$ref values must be RFC3986-compliant percent-encoded URIs

Hi team,
First, thanks for this amazing tool...
Recently, we started the validation process of our APIs (version 2.0) using Swagger-Hub (https://app.swaggerhub.com) & Swagger Editor (https://editor.swagger.io). While with Swagger-Hub everything went well - some minor warnings -, with Swagger Editor we surprisingly started reciving errors (with structures that haven't changed from version 1.0 and successfully passed the validation process) of type: Semantic error at paths... $ref values must be RFC3986-compliant percent-encoded URIs...all the errors are related to simple generic structures used in our responses and the references to them seem to be using not allowed characters...like this '[' and ']:
schema:
$ref: '#/definitions/Page[Voyage]'
-- here is the error (Semantic error at paths./api/v201804/Voyages/Pages.get.responses.200.schema.$ref
$ref values must be RFC3986-compliant percent-encoded URIs)
and, basically, this is the definition
Voyage:
description: Represents a voyage
required:
- id
- vesselId
- startDate
- endDate
type: object
properties:
-- description of the properties
'Page[Voyage]':
description: Represents a page of resources
required:
- data
- paginationContext
type: object
properties:
data:
description: 'Defines the data. Type: array'
type: array
items:
$ref: '#/definitions/Voyage'
paginationContext:
$ref: '#/definitions/PaginationContext'
description: 'Defines the pagination context. Type: PaginationContext'
PaginationContext:
description: Represents the pagination context
required:
- page
- size
- pages
- totalRows
type: object
properties:
-- description of the properties
Could you, please, tell us what we are going wrong?
Thanks

same trouble here

I wasn’t aware of this requirement before and therefore thr value that’s generated by default doesn’t account for it. So, let’s keep tracking here as a bug.
In the meantime, you can override the default behavior via the “SchemaId” confit setting (see readme)

Correction: the setting is CustomSchemaIds not SchemaId

Same issue. And if you fix the compliance by escaping [ and ] with %5B and %5D respectively, you end up with "5B" and "5D" in the properties because the generator only removes the % symbols. It's a bit of a mess.

Having a similar issue
For some reason our webapp generates the following
$ref: '#/definitions/KeyValuePair[String,IEnumerable[String]]'
and has KeyValuePair ... as part of our model section
In the definitions section it has
KeyValuePair... and a warning next to it saying the definition isn't being used but there are at least 3 refs to it.
I tried the suggestion around the schemaIds but that didn't fix it.
Is there a way to set model names i.e replace KeyValuePair with something else

@protegesolutions  sorry I should have mentioned that I tried to manually edit on SwaggerEditor and match the references and that fixed the problem. Because this definition is created automatically I was hoping their would be a way to edit/swap the keyvaluepair ... via code or some kind of annotation or something.

@gtd4 Ah, gotcha. If you're generating code from the definition, you should be able to just go ahead with the $ref error. We moved from using editor to generator and the $ref/definition stuff seems to be fine. Someone's just gotten a tad overzealous with the rules, I reckon, because to conform to the $ref validation, you break the resultant generated code.

My property was indeed a KeyValuePair<string, string> as others mentioned before. The outputed result in the swagger.json was:
  phone_numbers:
    uniqueItems: false
    type: array
    items:
      $ref: '#/definitions/KeyValuePair[String,String]'

It took me a while to figure out from this thread what was meant with using the CustomSchemaIds. Experimenting with it, it's just a matter or playing around with the C# type name, incl. generic arguments, and transforming the name into something that's url-compliant. Like, for example:
swagger.CustomSchemaIds((type) => type.ToString()
    .Replace("[", "_")
    .Replace("]", "_")
    .Replace(",", "-")
    .Replace("`", "_")
);

Some notes on why the type.ToString():

Can't use .FullName because this will not solve the problem but only adds additional rubbish to the $ref
Can't use .Name, because this will break if you have the same type with different generic arguments (also will include the backtick '`' symbol, by the way)

Not proud of this. And it makes a huge mess of the definition. But it does seem to fix the "error". Maybe it help others, until a better solution can be conjured up. Would love for that to happen! So +1.

Was there any good solution for this?

Ideally want to end up with
Paged[Voyage]
PagedVoyage

My property was indeed a KeyValuePair<string, string> as others mentioned before. The outputed result in the swagger.json was:
  phone_numbers:
    uniqueItems: false
    type: array
    items:
      $ref: '#/definitions/KeyValuePair[String,String]'

It took me a while to figure out from this thread what was meant with using the CustomSchemaIds. Experimenting with it, it's just a matter or playing around with the C# type name, incl. generic arguments, and transforming the name into something that's url-compliant. Like, for example:
swagger.CustomSchemaIds((type) => type.ToString()
    .Replace("[", "_")
    .Replace("]", "_")
    .Replace(",", "-")
    .Replace("`", "_")
);

Some notes on why the type.ToString():

Can't use .FullName because this will not solve the problem but only adds additional rubbish to the $ref
Can't use .Name, because this will break if you have the same type with different generic arguments (also will include the backtick '`' symbol, by the way)

Not proud of this. And it makes a huge mess of the definition. But it does seem to fix the "error". Maybe it help others, until a better solution can be conjured up. Would love for that to happen! So +1.

With your example i ended up with this which is not desired by any means:
GroHealth.Web.ViewModels.Utils.PagedList_1_GroHealth.Entities.HealthMetrics.BloodGlucose_

I did face similar issues as already mentioned. Even if the URL was encoded, the validator still was complaining for not finding a reference to the definition
Semantic error at paths./api/Attribute/Filter.post.responses.200.schema.$ref $refs must reference a valid location in the document
Which meant that I had to also update the definitions to match the new encoded URL.
What I did to get around this, was to convert the "[" for any operation response that had a schema reference to a generic type, into a dot ".", using an IOperationFilter. I also removed the "]"
Using an IDocumentFilter, I updated all Definitions of generic type, using the same pattern as above.
So the generic type List[MyObject] becomes List.MyObject which is still unique, but also compliant.
In a more complex generic type a KeyValuePair<String,String> should probably become KeyValuePair.String.String
As long as the Definition key is unique and compliant, the validator seems happy.
Sharing in case someone finds it helpful.

Hi gtrifidis, thanks for the solution.
Please, do you mind sharing the code of the 2 class implementing IOperationFilter and IDocumentFilter ?
Thanks a lot

As mentioned above, the CustomSchemaIds config method provides the flexibility to play around with different strategies. Additionally, in the next version (previews available on myget.org), the default implementation has been simplified to hopefully provide preferable names for generic types out-of-the-box. For example, List<Product> will be described as a ProductList, KeyValuePair<string, int> will be described as a StringStringKeyValuePair etc.
For reference, here's the code:
private string DefaultSchemaIdSelector(Type modelType)
{
    if (!modelType.IsConstructedGenericType) return modelType.Name;

    var prefix = modelType.GetGenericArguments()
        .Select(genericArg => DefaultSchemaIdSelector(genericArg))
        .Aggregate((previous, current) => previous + current);

    return prefix + modelType.Name.Split('`').First();
}

As mentioned above, the CustomSchemaIds config method provides the flexibility to play around with different strategies. Additionally, in the next version (previews available on myget.org), the default implementation has been simplified to hopefully provide preferable names for generic types out-of-the-box. For example, List<Product> will be described as a ProductList, KeyValuePair<string, int> will be described as a StringStringKeyValuePair etc.
For reference, here's the code:
private string DefaultSchemaIdSelector(Type modelType)
{
    if (!modelType.IsConstructedGenericType) return modelType.Name;

    var prefix = modelType.GetGenericArguments()
        .Select(genericArg => DefaultSchemaIdSelector(genericArg))
        .Aggregate((previous, current) => previous + current);

    return prefix + modelType.Name.Split('`').First();
}

Thanks, this worked a treat

And for those who need the integration with NSwag Studio and get the old naming (TypeOfType etc.),
based en the previous, here is the code :
`                c.CustomSchemaIds(DefaultSchemaIdSelector);
            string DefaultSchemaIdSelector(Type modelType)
            {
                if (!modelType.IsConstructedGenericType) return modelType.Name;

                var prefix = modelType.GetGenericArguments()
                    .Select(DefaultSchemaIdSelector)
                    .Aggregate((previous, current) => previous+ current);

                return modelType.Name.Split('`').First() + "Of"  + prefix;
            }`

for anyone who still has a problem with this, can try to resolve the way i did. It is basically not accepting any special characters in your definitions which is referred under your schema in the API. So all I did was to change to a meaningful definition name and use it in the schema. The error is gone. Here is the example of the problem and its solution.
Problematic definition vs ref
"schema":
{
"$ref": "#/definitions/Tuple[List[Int32]. List[Int32]]"
}

under defintions: (model sections)
"Tuple[List[Int32]. List[Int32]]" : { ... }
Here is the solution to the above example problem - Rename definition and reference it as defined
"schema":
{
"$ref": "#/definitions/TupleListIntegers"
}

under defintions: (model sections)
"TupleListIntegers" : { ... }
The error goes off.

As mentioned above, the CustomSchemaIds config method provides the flexibility to play around with different strategies. Additionally, in the next version (previews available on myget.org), the default implementation has been simplified to hopefully provide preferable names for generic types out-of-the-box. For example, List<Product> will be described as a ProductList, KeyValuePair<string, int> will be described as a StringStringKeyValuePair etc.
For reference, here's the code:
private string DefaultSchemaIdSelector(Type modelType)
{
    if (!modelType.IsConstructedGenericType) return modelType.Name;

    var prefix = modelType.GetGenericArguments()
        .Select(genericArg => DefaultSchemaIdSelector(genericArg))
        .Aggregate((previous, current) => previous + current);

    return prefix + modelType.Name.Split('`').First();
}

That's what helped me to solve the issue!


Compilation problem

Hi all,
I am new on the LoRa/rpi world and I am trying to run this example.
I have all the files cloned to my rpi 3 and the wiringPi library installed.
I´m trying to run the following code:
g++ -o LoraWan main.cpp -lwiringPi -lstdc++
but, intead of a clean compilation, i get the following error:
main.cpp:(.text+0x9b8): undefined reference to `bin_to_b64(unsigned char const*, int, char*, int)'
main.cpp:(.text+0xf80): undefined reference to `bin_to_b64(unsigned char const*, int, char*, int)'

Do you guys know how to fix this?
Thanks

Solved it.... I forgot to put the path to the base64.c file.
g++ -o LoraWan main.cpp -lwiringPi -lstdc++ <path>


Routable tabs on 2nd page has a swipe back issue on IOS

Hi so yeah I noticed that  while routable tabs on index main page are ok. 2nd pages have this issue swipeback issue on IOS..
https://user-images.githubusercontent.com/128444952/226510129-2f00a555-3a9d-4fef-84d6-8e41ed11a3d9.mp4

Your issue is closed because it is not filled correctly. Before you open an issue please review the contributing guideline.
Please fill the issue template correctly and:

Clearly describe the issue including steps to reproduce when it is a bug.
Make sure you fill in the earliest version that you know has the issue.
Provide live link or JSFiddle/Codepen or website with isssue

It will save a lot of time for other contributors to check your issue and fix it.


[UI] [Peer Registration] [Register Peer] Can not observe the bottom of drop down form.

For replay this bug you need:
Test case

Go to SS and login under any account.
Click on 'Register peer' menu icon and fill out the form.
Go to 'Peer Registration' page and click on 'Register Peer' menu icon.

Result: you can't observe  the bottom of drop down form.

Commit ID 	81e10d4983973e23cf9e3a8ce818fffb460ca97a

resolved


Correctly parse the Quorum config section

Instead of incorrectly having an array of Quorums, there should only be one root Quorum with a set of nodes and sub-quorums.

How do you create sub-entries in YAML? I couldn't figure that out. That's why it was initially an array at the top level, with the first entry implicitly meaning "root".

quorum:
  # threshold as a percentage
  threshold: 66%
  nodes:
    - GBFDLGQQDDE2CAYVELVPXUXR572ZT5EOTMGJQBPTIHSLPEOEZYQQCEWN
    - GBYK4I37MZKLL4A2QS7VJCTDIIJK7UXWQWKXKTQ5WZGT2FPCGIVIQCY5
  subQuorums:
    - name: quorum-1.1
      threshold: 66%
      nodes:
        - GBFDLGQQDDE2CAYVELVPXUXR572ZT5EOTMGJQBPTIHSLPEOEZYQQCEWN
        - GBYK4I37MZKLL4A2QS7VJCTDIIJK7UXWQWKXKTQ5WZGT2FPCGIVIQCY5
      subQuorums:
        - name: quorum-1.1.1
           threshold: 66%
           nodes:
             - GBFDLGQQDDE2CAYVELVPXUXR572ZT5EOTMGJQBPTIHSLPEOEZYQQCEWN
             - GBYK4I37MZKLL4A2QS7VJCTDIIJK7UXWQWKXKTQ5WZGT2FPCGIVIQCY5
             - GBYK4I37MZKLL4A2QS7VJCTDIIJK7UXWQWKXKTQ5WZGT2FPCGIVIQCY5

Something like this ?

#209

The quorum config has been greatly simplified now, and I've added a unittest. This is the diff: https://github.com/bpfkorea/agora/compare/dde37dda787fecfb0fba8befaf95e394ad85e7f4..57326a31983a4c712b5b39384733751e164c9fba

Ah I have to fix the integration configs too.

Needs a rebase, but ready for merge

Rebased.


Add UART serial example, restructure examples

This PR will refactor the examples structure, adding a non-working UART serial example, with its own custom transport. The purpose of this example is to be able to run the Pico in a debug session, from VS Code.
Currently, the example connects to the Micro-ROS agent successfully, then stalls after only 1 msg.data published. Build this example with make uart_example in Debug mode.
The original USB example is in its own folder, and can be built by make usb_example in Release mode, then copy .uf2 to Pico. This example is working normally.
The reason for creating this draft PR is to discuss further why the UART serial custom transport makes the Pico stall. Previous discussions in https://github.com/micro-ROS/micro_ros_raspberrypi_pico_sdk/issues/942 and https://github.com/micro-ROS/micro_ros_raspberrypi_pico_sdk/issues/937.

Does this PR have the UART issue?


Basic i18n support

Hey @liyasthomas,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: b384afe0-10f9-11ea-b8ff-d5ab8a371226

Hey @liyasthomas,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 655fdba0-10fe-11ea-b8ff-d5ab8a371226

Retry DeepCode" on this PR.

Retry DeepCode

Hey @liyasthomas,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 87f69310-10ff-11ea-b8ff-d5ab8a371226

@NBTX please review this.

Hey @liyasthomas,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: db1f0900-110e-11ea-b8ff-d5ab8a371226


feat(templates): ts-node by default (#123)

Backport
This will backport the following commits from 2.x to 1.x:

feat(templates): ts-node by default (#123)

Questions ?
Please refer to the Backport tool documentation

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 879

Questions ?
Please refer to the Backport tool documentation


Navigation

Individual RECORD functions are documented, but one has to search them out separately. It would be very convenient to have a list of other RECORD functions on one side of the page documenting each individual function.

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

ID: b74f4650-4bde-bd87-0881-ef6fac238b48
Version Independent ID: 35fcd85e-4d49-e0f7-98ba-90ddb3e2bb27
Content: FIND Function (Record) - Dynamics NAV
Content Source: dynamics-nav/FIND-Function--Record-.md
Product: dynamics-nav-2018
GitHub Login: @jswymer
Microsoft Alias: edupont

Thanks for the feedback. We're working on improving the documentation for the AL language and will take your feedback into consideration. Thanks again.


Colour Blindness accessibility

Is your feature request related to a problem? Please describe.
One of my players is colourblind and he can't see 75% of the character sheet.
Describe the solution you'd like
There are international standards for colours so user accessibility is high for them as well.
What the person told me is that it is mostly due to not having enough contrast between background and text.
I hope you can make an easy fix soon by fixing the colour palette or a "for now" fix by not having dark grey text on light grey backgrounds etc.
Many Thanks
Great tool otherwise

Do you mean the pathmuncher import screen, pathbuilder itself or the pf2e character sheet?

Hi,  thanks for the reply. It is thepf2e character sheet. I have attached a screenshot.
Many thanks

Thanks, this is the repo for the Pathmuncher tool, you'll want to report this at the PF2e system project: https://github.com/foundryvtt/pf2e


  This is the expected behavior of the build system.

OPS Build status updates of commit fd883e1:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamics-gp/TOC.md
:warning:Warning
View
Details

dynamics-gp/TOC.md

Line 0: [Warning] Invalid file link:(~/payroll/US19JanTaxR1.md).

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


User entering invalid prompt option at the root level twice causes an error in Node SDK 2.0.2

A prompt at the root level dialog will produce errors if the users enters and invalid choice twice in a row.
Repro:
Create a bot with the following dialog:
bot.dialog('/', [function (session) {
    var msg = new builder.Message(session)
        .attachments([
            new builder.HeroCard(session)
                .title("Test Card")
                .text("Select an option")
                .buttons([
                    builder.CardAction.imBack(session, "yes", "Yes"),
                    builder.CardAction.imBack(session, "no", "No")
                ])
        ]);
    builder.Prompts.choice(session, msg, "yes|no");
}, function (session, results) {
	session.send(`You chose ${results.response.entity}`);
}]);

When prompted, enter junk text twice in a row.
Result:
Bot will respond with "Sorry. Something went wrong." and "ERROR: Too many calls to session.endDialog()." will be logged to the Node.JS output.

Hi @elpollouk I am not able to repro - the bot will show the Yes/No dialog again if I use junk input. Please open a new issue if you see this behavior again, thanks.


readme: Add links to CHANGELOG.md and LICENSE

For convenience, added links to the CHANGELOG.md and LICENSE that are referenced in README.md.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #378   +/-   ##
=======================================
  Coverage   85.45%   85.45%
=======================================
  Files          40       40
  Lines        8629     8629
=======================================
  Hits         7374     7374
  Misses       1255     1255

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 50f3a60...d01cce7. Read the comment docs.


GitHub CI: Add 'check-update' and 'check-docs'

'check-update' runs a 'make update' to check that it wasn't forgotten.
'check-docs' runs 'make doc-nits'.  We have that as a separate job to
make it more prominent.

I added 'check-update' specifically because 'make update' was forgotten lately, which went unnoticed.

... and well, it didn't take long to fail.  Good :wink:

make update commit added

There's also cmd-nits

Any reason not to do this in 1.1.1-stable?

Any reason not to do this in 1.1.1-stable?

Other than "didn't think of it"? No...

Labeled for 1.1.1 as well

There's also cmd-nits

And md-nits. But neither of these are on 1.1.1 so they should be probably in a separate PR as this one can be easily cherry-picked.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged
master:
8175476b81c6b54bfa1c8555b35561099b202c4d GitHub CI: Add 'check-update' and 'check-docs'
f3f2dd9dfb5fe89fff44c413e9d6c788e0fa6300 make update
1.1.1:
5a5d87a936ceeca1648288e1efe4296687193b16 GitHub CI: Add 'check-update' and 'check-docs'


Spyder Screenshot Contest: please participate!

From pierre.raybaut on 2011-04-20T15:53:11Z
To participate to this first Spyder Screenshot Contest, you just have to:

Make sure you realize that there is absolutely no price to win except the satisfaction to participate to a communication effort around your favorite IDE

Attached here your screenshot(s)

The best screenshots will be uploaded permanently and shown in a new Screenshot Gallery Wiki page.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=635

From<EMAIL_ADDRESS>on 2011-04-29T07:44:13Z
and one more taken from ubuntu
Attachment: ubuntu_console_and_friends.png

From<EMAIL_ADDRESS>on 2011-06-13T07:39:20Z
Pierre, what font are you using in your code editor?

From pierre.raybaut on 2011-04-22T06:24:25Z
Spyder on Ubuntu 10.04
Attachment: spyder_ubuntu_editor.png spyder_ubuntu_console.png spyder_ubuntu_profiler.png

From pierre.raybaut on 2011-06-14T13:09:50Z
Liberation Mono

From pierre.raybaut on 2011-09-05T13:48:18Z
Status: Done

From<EMAIL_ADDRESS>on 2011-10-10T08:48:13Z
Deep blue-themed spyder, minimal setup (k)ubuntu 10.04, heavily customized bespin qt style.
Attachment: Spyder_editor.png Spyder_console.png


Create an azure VM via python with init script

Related command
No response
Resource Provider
virtual machine
Description of Feature or Work Requested
I am trying to create a virtual machine on azure via python and having it execute a script on launching time (init script).
For this purpose, I am using the custom_data as follows:

the custom_data is created this way:

For the variables inside the init_script:
WORKER_SCRIPT_PATH : is the string path to my shell script (an sh file)
and the other variables are simple strings.
Now after I execute the script, everything seems to work fine and the VM is created but when I check on azure, I don't find he script!

AM i missing something?
Minimum API Version Required
N/A
Swagger PR link / SDK link
N/A
Request Example
No response
Target Date
22/09/2022
Additional context
No response

@zhoxing-ms for awareness


Typo in doc: tf.keras.losses.CategoricalCrossentropy

Please make sure that this is a documentation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:doc_template
System information

TensorFlow version: 1.13
Doc Link: tf.keras.losses.CategoricalCrossentropy

Describe the documentation issue
In the example code
loss = cce(
    [[1., 0., 0.], [0., 1., 0.], [0., 0., 1.]],
    [[.9, .05, .05], [.5, .89, .6], [.05, .01, .94]])

I think the second array in the second line should be [.05, .89, .06] instead of [.5, .89, .6], as they're supposed to be summed to one.
We welcome contributions by users. Will you be able to update submit a PR (use the doc style guide) to fix the doc Issue?

Thank you for noticing this and submitting the PR! Approving now. 🙂


Track 'Obviyo'

Vendor site - https://www.obviyo.com/
Category - Personalisation / Shopify Apps
Example Site - https://www.pothosbeauty.com/products/charcoal-edge-styler
Detection - Look for script from domain '//h30-deploy.hiconversion.com/'
FYI - https://blendcommerce.com/blogs/shopify/hiconversion-is-now-obviyo
@nurbek91  - After raising this, I realised we already have an entry for 'HiConversion' under A/B Testing category. I think more appropriate categories are what I listed above and let's get this changed to it's new name 'Obviyo'

Hey @rockeynebhwani , thanks for info!!!
Removed HiConversion , add Obviyo
Category : Personalisation, Shopify app , requires: Shopify, update pricing, icon, desc.


Adds dnf plugin

Useful aliases for dnf
fixes #4184

Hi, please read my comment https://github.com/robbyrussell/oh-my-zsh/pull/2452#issuecomment-127736656


IllegalArgumentException: reason.size() > 123

There's this relatively new code in: okhttp/src/main/java/okhttp3/internal/ws/RealWebSocket.kt:
    if (reason != null) {
      reasonBytes = reason.encodeUtf8()
      require(reasonBytes.size <= CLOSE_MESSAGE_MAX) {
        "reason.size() > $CLOSE_MESSAGE_MAX: $reason"
      }
    }

The problem is that this causes "IllegalArgumentException: reason.size() > 123" and there's no "nice" way to fix it.
Of course I can do:
   int code = 1000;
   String reason = ("disconnected: " + url).substring(0,123);
   webSocket.close(code, reason);

but this is not ideal, because that 123 can change. So I would rather do:
   String reason = ("disconnected: " + url).substring(0,(int) WebSocketProtocol.CLOSE_MESSAGE_MAX);

but this doesn't work because it's internal.
So at least change it to public, but even better solution maybe would be to do this (the substring) in okhttp, and instead of an exception just log a warning.

123 won't change.

It should be acknowledged that it would be erroneous to truncate arbitrary reason strings by 123, as that max length refers to bytes of UTF-8 text, not characters, nor String chars (UTF-16 code units). The error message OkHttp is providing is a bit misleading in that regard.

Yep, agreed.  We shouldn't arbitrarily truncate, it's awkward with partial unicode strings etc.  So clients should do this, reporting the error is the best we can do as an API.  If you feel strongly, submit a PR to improve the message to include encoded bytes or similar. But unlikely we fix this internally if it hasn't gotten attention so far.
Won't fix without a PR.


[Feature] Out of box support for RSocket in sleuth similar to WebFlux

With RSocket being added into spring boot core similar to WebFlux and RSocket is being projected for inter micro services communication tracing becomes more important in RSocket environment.  Thus case for out of box support for RSocket becomes more important.

Any update on this task? Is it planned or have already some code/branch that I could have a look into? thanks!

Nope, not yet. We have it on our radar though.

Done via https://github.com/spring-cloud/spring-cloud-sleuth/commit/216d681b04a2fc228ca5628e4f5c4593d9b33171

It's available in Release Train 2021.0.0-SNAPSHOT or Sleuth version 3.1.0-SNAPSHOT


Not working in IE10+

Unfortunately, it seems like it is not working in IE10+ like most blur-plugins.

Not working in IE11, forEach() method appears as an error


presence of while loop breaks literal_unroll compilation

I reproduce this issue by running on numba 0.56.0 on google colab.
I cannot understand why the presence of a while loop later in the block creates a getitem problem for literal_unroll on heterogenous tuples.
from numba import njit,literal_unroll
@njit
def f1( t ):
    for t_i in literal_unroll(t):
        print('.')

    return 0

@njit
def f2( t ):
    for t_i in literal_unroll(t):
        print('.')

    x=0
    while x>0:
        x= x-1

    return 0

print('f1')
f1( (0,0.0) ) # works
print('f2')
f2( (0,0.0) ) # fails

The unexpected compile fail for f2 is the following:
TypingError: Failed in nopython mode pipeline (step: nopython frontend)
No implementation of function Function(<built-in function getitem>) found for signature:

 >>> getitem(Tuple(int64, float64), int64)

There are 22 candidate implementations:
      - Of which 22 did not match due to:
      Overload of function 'getitem': File: <numerous>: Line N/A.
        With argument(s): '(Tuple(int64, float64), int64)':
       No match.

During: typing of intrinsic-call at C:\Users\pconlon\AppData\Local\Temp/ipykernel_18736/1219060107.py (11)

File "..\..\..\..\..\AppData\Local\Temp\ipykernel_18736\1219060107.py", line 11:
<source missing, REPL/exec in use?>

[X] I have tried using the latest released version of Numba (most recent is
visible in the change log (https://github.com/numba/numba/blob/main/CHANGE_LOG).
[X] I have included a self contained code sample to reproduce the problem.
i.e. it's possible to run as 'python bug.py'.

What I expect to happen:

the f2 should compiles just like f1 and does not have a compilation failure

Thanks for the report - I can reproduce the compilation failure.

A smaller reproducer:
from numba import njit, literal_unroll

@njit('(Tuple((int64, float64)),)')
def f2(t):
    for t_i in literal_unroll(t):
        pass

    while True:
        x = 0

#8321 fixes.


How can I measure LCP and FID in single page application?

This was discussed in a previous issue. See: https://github.com/GoogleChrome/web-vitals/issues/119


snowflake_grant_account_role mixed role_name and parent_role_name

Terraform CLI and Provider Versions
terraform {
required_providers {
snowflake = {
source  = "Snowflake-Labs/snowflake"
version = "~> 0.76"
}
}
Terraform Configuration
terraform version
Terraform v1.7.5
on darwin_arm64
+ provider registry.terraform.io/hashicorp/aws v5.41.0
+ provider registry.terraform.io/snowflake-labs/snowflake v0.87.2

Expected Behavior
Based on documentation terraform should grant parent_role_name to role_name
https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/grant_account_role
##################################
grant account role to account role
##################################
resource "snowflake_role" "role" {
name = var.role_name
}
resource "snowflake_role" "parent_role" {
name = var.parent_role_name
}
resource "snowflake_grant_account_role" "g" {
role_name        = snowflake_role.role.name
parent_role_name = snowflake_role.parent_role.name
}
Actual Behavior
Parent and child roles are mixed. Instead of granting parent to child, terraform grants child to parent.
Steps to Reproduce
In Terraform
resource "snowflake_role" "A" {
  name                                = "A"
  provider                            = snowflake.user_admin
}

resource "snowflake_role" "B" {
  name                                = "B"
  provider                            = snowflake.user_admin
}

resource "snowflake_grant_account_role" "A_TO_B" {
  provider  = snowflake.security_admin
  role_name = "B"
  parent_role_name = "A"
}

In snowflake
SHOW GRANTS TO ROLE A;

How much impact is this issue causing?
High
Logs
No response
Additional Information
No response

Hey @lvrbanec 👋
Not sure what this is about. The snowflake_grant_account_role resource is mimicking what's available on Snowflake (here's the documentation). I would say this is a normal behavior that a child role is granting to the parent role because it's giving the parent role the same grants as the child one.  Here's a more detailed article on how Snowflake's access management works: https://docs.snowflake.com/en/user-guide/security-access-control-overview. Please, let me know If you're concerned that there's something wrong with the provider. Ideally, please link Snowflake documentation saying the connection should be reversed to the one we have right now.

Hey @sfc-gh-jcieslak
Thanks for the answer. You are right, I expected that Snowflake would show parent to child. I totally misread the documentation cause of my expectations. Many many thanks for clearing it out for me and spending your time looking into this issue.
Wish you all the best

Resolved


I18n/tr

Changes

Add missing turkish :tr: translations
Remove translated lines from _missing_translations_tr.json file

Currently, localsend seems to fully support Turkish language.
Question
Can I add my username to tr section in translators.dart ?

Of couse, you can add yourself to translators.dart 👍


mkvtoolnix: 32.0.0 -> 33.0.0

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/mkvtoolnix/versions.
meta.description for mkvtoolnix is: '"Cross-platform tools for Matroska"'.

Checks done (click to expand)

built on NixOS
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 33.0.0 with grep in /nix/store/12jsjsybcq2zdfgl555y98rq0mpkfy10-mkvtoolnix-33.0.0
directory tree listing: https://gist.github.com/79f8199ac7cb96f49cc8ccdcad955aeb
du listing: https://gist.github.com/cd97619d14a5e10af64025dec91d2cda

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

7 total rebuild path(s)
2 package rebuild(s)
2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
mkvtoolnix
mkvtoolnix-cli

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/12jsjsybcq2zdfgl555y98rq0mpkfy10-mkvtoolnix-33.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A mkvtoolnix https://github.com/r-ryantm/nixpkgs/archive/94edef49ced1e0b4de799082c4aebe8480ec7779.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/12jsjsybcq2zdfgl555y98rq0mpkfy10-mkvtoolnix-33.0.0
ls -la /nix/store/12jsjsybcq2zdfgl555y98rq0mpkfy10-mkvtoolnix-33.0.0/bin

cc @codyopel @fuuzetsu @rnhmjoj for testing.

@GrahamcOfBorg eval


Please get in contact with me

Hi Benjamin Wan,
Some other developer showed me an app claming its done by him. I finally found out you are the producer of the code.
I would like to integrate your project into out appliction. So we might have a job for you.
Please reach out to me by email<EMAIL_ADDRESS>Thank you

I added a Apache License.
It's just a demo, not being able to provide technical support.

Hi Benjamin
Sorry I was on leave
We are not asking for technical support. We are interested in hire you for a project.
Can you please get in contact with me by email?
Thank you very much

Hi Benjamin
Can you please geet in touch with us? I dropped you another email


OrderBy with .Contains(Guid) expression generates invalid SQL

Using SqlServer database provider, and Blog with a Guid primary key Id.
var guids = new List<Guid>();
await db.Blogs.OrderBy(x => guids.Contains(x.Id)).ToListAsync();

This gets translated into
SELECT *
      FROM [Blogs] AS [x]
      ORDER BY CASE
          WHEN 0 = 1
          THEN CAST(1 AS BIT) ELSE CAST(0 AS BIT)
      END DESC

When executed, sql server throws the error
A constant expression was encountered in the ORDER BY list, position 1.
When included in more complicated queries (with Skip().Take(), the query fails to compile. (Edit - I think this exception/issue is different since this occurs with string contains as well)
System.NullReferenceException: Object reference not set to an instance of an object.
   at Microsoft.EntityFrameworkCore.Query.Expressions.InExpression.Equals(InExpression other)
   at System.Collections.Generic.List`1.FindIndex(Int32 startIndex, Int32 count, Predicate`1 match)
   at Microsoft.EntityFrameworkCore.Query.Expressions.SelectExpression.AddToProjection(Expression expression, Boolean resetProjectStar)
   at Microsoft.EntityFrameworkCore.Query.ExpressionVisitors.RelationalProjectionExpressionVisitor.Visit(Expression expression)
   at Microsoft.EntityFrameworkCore.Query.ExpressionVisitors.RelationalProjectionExpressionVisitor.VisitNew(NewExpression newExpression)
   at System.Linq.Expressions.NewExpression.Accept(ExpressionVisitor visitor)

I would expect this query to compile similarly as other Contains() expressions. If I change the the list and Id to ints, the following query is correctly generated. Strings work as well. DateTimeOffset does not work.
SELECT *
      FROM [Blogs] AS [x]
      ORDER BY CASE
          WHEN [x].[Id] IN (1, 2, 3, 4)
          THEN CAST(1 AS BIT) ELSE CAST(0 AS BIT)
      END DESC

Further technical details
EF Core version: Latest version, 2.1.0-preview1-27020
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system:  win 10 64bit
IDE: VS Studio 2017 15.4.0 Preview 4.0

await db.Blogs.Where(x => guids.Contains(x.Id)).OrderBy(x => x.Id).ToListAsync();

@MarcoLoetscher That is a filter query, I'm trying to order the data.

In first query we do generate InExpression but the list is empty so we optimize that bit out to replace it with 0 = 1. I verified that above SQL does throw exception. We need to refine rules to apply optimization because we should not be generating incorrect SQL. Marking this as bug for that. The second expected SQL would be generated for any case of parameter values being non-empty list.
I am not sure why it would fail only for DateTimeOffset but implementation of Equals in InExpression does not seem to be null-safe.

@smitpatel I've narrowed down the cause of the NRE on the InExpression.Equals().
I doubt this is related.
This only throws under specific conditions. Posts must be a collection navigation property.
The guids variable could also be a list of int, the same exception occurs.
var guids = new List<Guid> { Guid.Empty };
await db.Blogs
    .Include(x => x.Posts)
    .OrderByDescending(x => guids.Contains(x.Id))
    .Skip(1)
    .ToListAsync();


Shared Library not found

I have followed all the step to import the liboqs library in python. Unfortunately when i try to run the library it shows the error (Shared library is not found). Kindly help me to resolve this issue.

Stale issue, cannot reproduce


"ibc-setup balances" becomes unusable if one registry chain is down

About
If at least one chain is unreachable, the ibc-setup balances command outputs an error (fast-fail).
Current result
error: connect ECONNREFUSED <IP_ADDRESS>:443 {"stack":"Error: connect ECONNREFUSED <IP_ADDRESS>:443\n    at TCPConnectWrap.
afterConnect [as oncomplete] (node:net:1137:16)","config":{"url":"https://rpc.cosmos.network:443","method":"post","data":"
{\"jsonrpc\":\"2.0\",\"id\":979132555359,\"method\":\"status\",\"params\":{}}","headers":{"Accept":"application/json, text/
plain, */*","Content-Type":"application/json;charset=utf-8","User-Agent":"axios/0.21.1","Content-Length":65},"transformRequest":
[null],"transformResponse":[null],"timeout":0,"xsrfCookieName":"XSRF-TOKEN","xsrfHeaderName":"X-XSRF-TOKEN",
"maxContentLength":-1,"maxBodyLength":-1},"code":"ECONNREFUSED"}

Expected result

should inform about unreachable chain
should still print balances for other chains

Yes, I found this before and forgot to write an issue.
I would just catch that and logger.error() a message for all chains that it couldn't connect to.
And then print out normally only the ones it could connect to with a non-zero balance


add toFloat() to prototype

Added the toFloat() prototype function for conveniently parsing a Big in a chained form.
Before:
var result = parseFloat(Big(35).div(4));

After:
var result = Big(35).div(4).toFloat();

The documentstion (here)[http://mikemcl.github.io/big.js/#faq] describes how a Big can be converted to a JavaScript number.
Do I want to remove that section and document the new method, and add tests and create a new release just for the sake of a toFloat method, which is not really necessary any way, and with this being a minimalist library?
Hmmm...
Is toFloat better than toNumber?
Your contribution is clean and modest which is appealing...
Thank you, I'll consider it.

No, maybe you're right. Since I posted the PR yesterday I figured a better way (at least for my usage).
Using .toFloat() would have its pitfalls:
Big(354).div(7).toFloat()             // returns number 50.57142857142857
Big(354).div(7).toFixed(2).toFloat()  // that obviously can't be done

I would serve no purpose for simple uses like the mentioned above, where you would have to go back to parseFloat.
But...
Using the unary plus operator:
+Big(354).div(7)             // returns number 50.57142857142857
+Big(354).div(7).toFixed(2)  // returns number 50.57

It's short, it's native and works for exponentials as well.

@javorosas And if you want the coercion operator after, you can do * 1, so
Big(354).div(7).toFixed(2) * 1

I think the + is more intuitive and readable, but nice to have options.


hello.js example fails

Hi, I'm trying to get spooky up and running and am getting errors on the example scripts.
Any thoughts on what's not working?
spookyjs 0.2.5
osx: 10.9.5
phantomjs: 1.9.2
casperjs: 1.0.4
I get this error
[ { file: '/Users/john/dev/<project_dir>/node_modules/spooky/lib/../lib/bootstrap/casper.js',
    line: 112,
    function: '_require' },
  { file: '', line: 9, function: 'anonymous' },
  { file: '/usr/local/Cellar/casperjs/1.0.4/libexec/bin/bootstrap.js',
    line: 160,
    function: '_require' },
  { file: '/Users/john/dev/<project_dir>/node_modules/spooky/lib/bootstrap.js',
    line: 54,
    function: '' } ]
{ [Error: Child terminated with non-zero exit code 1] details: { code: 1, signal: null } }```

✖ 1 of 79 tests failed:

bootstrap/emit "before each" hook:
Error: Child terminated with non-zero exit code 1
at Spooky. (/Users/john/Desktop/sandbox/SpookyJS/tests/util/hooks.js:28:19)
at Spooky.EventEmitter.emit (events.js:95:17)
at Spooky. (/Users/john/Desktop/sandbox/SpookyJS/lib/spooky.js:216:18)
at ChildProcess.EventEmitter.emit (events.js:98:17)
at Process.ChildProcess._handle.onexit (child_process.js:789:12)

Can you provide more code? from the error only it will be difficult to help. By the way, i would suggest to use the latest supported versions: Phantomjs 1.9.7 and Casperjs 1.1.0

Thank you! I upgraded phantom and casper to the versions you suggested and Spooky no longer crashes.
Though the script then had trouble with Wikipedia's use of SSL. I added the appropriate parameters (as far as I can tell), and submitted a PR: https://github.com/SpookyJS/SpookyJS/pull/150


Error when using upload-dir flag with 'ddev composer create'

Is there an existing issue for this?

[X] I have searched the existing issues

Expected Behavior
Successful creation of the project with the specified upload directory.
Actual Behavior
Error occurs during the project creation process when the upload-dir flag is set to 'public'. The following error message is displayed
rsync: [receiver] mkstemp "/var/www/html/web/public/.gitignore.IYVNFK" failed: No such file or directory (2)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1333) [sender=3.2.3]
Failed to create project: ComposeCmd failed to run 'COMPOSE_PROJECT_NAME=ddev-<project_name> docker-compose -f /<generic_user>/Sites/<project_name>/www/.ddev/.ddev-docker-compose-full.yaml exec -w /var/www/html -T web bash -c set -eu && ( rsync -rltgopD "/tmp/TQYtMb/" "/var/www/html/")', action='[exec -w /var/www/html -T web bash -c set -eu && ( rsync -rltgopD "/tmp/TQYtMb/" "/var/www/html/")]', err='exit status 23', stdout='', stderr='rsync: [receiver] mkstemp "/var/www/html/web/public/.gitignore.IYVNFK" failed: No such file or directory (2)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1333) [sender=3.2.3]'

Steps To Reproduce

Execute the following command to configure DDEV:
ddev config --project-name=<project_name> --project-type=craftcms --docroot=web --create-docroot --upload-dir=public
Run the 'ddev composer create -y --no-install --no-scripts ' command.
Observe the error message mentioned above during the project creation process.

Anything else?
It is important to note that the issue described above only occurs when using the upload-dir flag. Without the upload-dir flag, the project creation process works correctly.

Hi @bavodeclercq ! Sorry you're having trouble and thanks for stopping by.

Please do the requested ddev debug test. Since you omitted it on creation of this issue, just put the output into a gist on gist.github.com and edit your submission with a link to the gist. Right now I have no way to know what OS or Docker provider you're using
Please show to full reproduction steps that you used, including the actual ddev composer create command.
If you were trying this with mutagen, you'll probably want to try it without mutagen, as when you're using mutagen the upload_dir will be bind-mounted, and since it's a subdirectory of the docroot, maybe that messes up Craft's composer build?

Hi @rfay,
The original post has been updated with the requested information. The output of the ddev debug test is provided directly as a code block within the post.
Unfortunately, the actual ddev composer create command cannot be shared as the project in question is a private repository.
Yes, you are correct. When attempting the process without using mutagen, the project creation process works correctly.

Please see if you can demonstrate the problem using the craft quickstart from the docs.

Unfortunately, I won't be able to demonstrate the problem using the craft quickstart from the docs as my private repository is a scaffolding project of Craft 4, where the web/public directory is used as a volume. The default craft package does not have any volume directories by default. However, I have conducted a similar test using the web/cpresources folder, and I encountered the same error message.

My apologies for not providing clear instructions earlier. To demonstrate the problem, you can use the following commands:

ddev config --project-type=craftcms --docroot=web --create-docroot --upload-dir=cpresources
ddev composer create -y --no-scripts craftcms/craft

Please note that this is just a one-to-one example resembling my private repository, but the cpresources directory would not typically be used as the upload_dir. It is being utilized here to replicate the error scenario, as this directory is also within the docroot and contains only a .gitignore file.
Regarding the error message, it is still present in the OP, but it is hidden behind a toggle under the "Actual Behavior" section. You can expand that section to view the error message.

Thanks so much for taking the time to recreate and explain this @bavodeclercq - I was able to recreate it.
Please test the artifacts in https://github.com/ddev/ddev/pull/5042#issuecomment-1612001153 and see if they solve the problem for you.
More detail about how to test a PR in https://ddev.readthedocs.io/en/latest/developers/building-contributing/#testing-a-pr

I have tested the artifacts provided in #5042 as instructed, but unless I made a mistake during the PR testing process, I am still experiencing the same problem:
> ddev -v
ddev version v1.22.0-alpha3-15-gf4b0c5ef
Instrumentation is opted in, but AmplitudeAPIKey is not available. This usually means you have a locally-built ddev binary or one from a PR build. It's not an error. Please report it if you're using an official release build.

> ddev config --project-type=craftcms --docroot=web --create-docroot --upload-dir=cpresources
Creating a new ddev project config in the current directory (/Users/<user>/Sites/<project_name>/www)
Once completed, your configuration will be written to /Users/<user>/Sites/<project_name>/www/.ddev/config.yaml

Created docroot at /Users/<user>/Sites/<project_name>/www/web
You have specified a project type of craftcms but no project of that type is found in /Users/<user>/Sites/<project_name>/www/web
Configuration complete. You may now run 'ddev start'.
Instrumentation is opted in, but SegmentKey is not available. This usually means you have a locally-built ddev binary or one from a PR build. It's not an error. Please report it if you're using an official release build.
Instrumentation is opted in, but AmplitudeAPIKey is not available. This usually means you have a locally-built ddev binary or one from a PR build. It's not an error. Please report it if you're using an official release build.

> ddev composer create -y --no-scripts craftcms/craft
Network ddev_default created
Downloading https://github.com/docker/compose/releases/download/v2.18.1/docker-compose-darwin-aarch64 ...
docker-compose 50.50 MiB / 50.50 MiB [==============================] 100.00% 3s
Download complete.
 Container ddev-ssh-agent  Created
 Container ddev-ssh-agent  Started
ssh-agent container is running: If you want to add authentication to the ssh-agent container, run 'ddev auth ssh' to enable your keys.
 Network ddev-<project_name>_default  Created
 Container ddev-<project_name>-db  Created
 Container ddev-<project_name>-web  Created
 Container ddev-<project_name>-web  Started
 Container ddev-<project_name>-db  Started
Starting mutagen sync process... This can take some time.
Mutagen sync flush completed in 1s.
For details on sync status 'ddev mutagen st <project_name> -l'
 Container ddev-router  Created
 Container ddev-router  Started
Warning: MOST EXISTING CONTENT in the composer root (/Users/<user>/Sites/<project_name>/www) will be deleted by the composer create-project operation. Only .ddev, .git and .tarballs will be preserved.
Removing any existing files in composer root
Executing composer command: [composer create-project --no-scripts craftcms/craft --no-install /tmp/JDKbnW]

Creating a "craftcms/craft" project at "/tmp/JDKbnW"
Info from https://repo.packagist.org: #StandWithUkraine
Installing craftcms/craft (2.0.10)
  - Installing craftcms/craft (2.0.10): Extracting archive
Created project in /tmp/JDKbnW
Moving install to composer root
rsync: [receiver] mkstemp "/var/www/html/web/cpresources/.gitignore.lewgiQ" failed: No such file or directory (2)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1333) [sender=3.2.3]
Failed to create project: ComposeCmd failed to run 'COMPOSE_PROJECT_NAME=ddev-<project_name> docker-compose -f /Users/<user>/Sites/<project_name>/www/.ddev/.ddev-docker-compose-full.yaml exec -w /var/www/html -T web bash -c set -eu && ( rsync -rltgopD "/tmp/JDKbnW/" "/var/www/html/")', action='[exec -w /var/www/html -T web bash -c set -eu && ( rsync -rltgopD "/tmp/JDKbnW/" "/var/www/html/")]', err='exit status 23', stdout='', stderr='rsync: [receiver] mkstemp "/var/www/html/web/cpresources/.gitignore.lewgiQ" failed: No such file or directory (2)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1333) [sender=3.2.3]'

I am not able to recreate your problem with this PR on colima.
I note that you're using the unsupported docker provider Orbstack. Would you mind trying one of the supported Docker providers?
I've updated the PR, would love to have you test it with colima or Docker Desktop, https://github.com/ddev/ddev/pull/5042#issuecomment-1612001153 - please make sure you do a ddev poweroff

I've asked a colleague who has Colima installed to test the PR, and he encountered the same problem.
$ ddev -v
ddev version v1.22.0-alpha3-20-g4f6c436d

Projects/test
$ ddev config --project-type=craftcms --docroot=web --create-docroot --upload-dir=cpresources                                                                                <user>, 8:50:12
Creating a new ddev project config in the current directory (/Projects/test)
Once completed, your configuration will be written to /Projects/test/.ddev/config.yaml

Created docroot at /Projects/test/web
You have specified a project type of craftcms but no project of that type is found in /Projects/test/web
Configuration complete. You may now run 'ddev start'.

 This can take some time.
Mutagen sync flush completed in 4s.
For details on sync status 'ddev mutagen st whatever -l'
 ddev-router Pulling
 Container ddev-router  Created
 Container ddev-router  Started
Warning: MOST EXISTING CONTENT in the composer root (/Projects/test) will be deleted by the composer create-project operation. Only .ddev, .git and .tarballs will be preserved.
Removing any existing files in composer root
Executing composer command: [composer create-project --no-scripts craftcms/craft --no-install /tmp/GVfUUH]

Creating a "craftcms/craft" project at "/tmp/GVfUUH"
Info from https://repo.packagist.org: #StandWithUkraine
Installing craftcms/craft (2.0.10)
  - Downloading craftcms/craft (2.0.10)
  - Installing craftcms/craft (2.0.10): Extracting archive
Created project in /tmp/GVfUUH
Moving install to composer root
rsync: [generator] failed to set times on "/var/www/html/web/cpresources": Read-only file system (30)
rsync: [receiver] mkstemp "/var/www/html/web/cpresources/.gitignore.Q2KqXx" failed: Read-only file system (30)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1333) [sender=3.2.3]
Failed to create project: ComposeCmd failed to run 'COMPOSE_PROJECT_NAME=ddev-whatever docker-compose -f /Projects/test/.ddev/.ddev-docker-compose-full.yaml exec -w /var/www/html -T web bash -c set -eu && ( rsync -rltgopD "/tmp/GVfUUH/" "/var/www/html/")', action='[exec -w /var/www/html -T web bash -c set -eu && ( rsync -rltgopD "/tmp/GVfUUH/" "/var/www/html/")]', err='exit status 23', stdout='', stderr='rsync: [generator] failed to set times on "/var/www/html/web/cpresources": Read-only file system (30)
rsync: [receiver] mkstemp "/var/www/html/web/cpresources/.gitignore.Q2KqXx" failed: Read-only file system (30)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1333) [sender=3.2.3]'

Thanks for the feedback @bavodeclercq . I've pushed another commit which solves the issue for me. Please give it another try, thanks!

I have retested the PR, and apart from the fact that I now need to use upload_dirs (presumably due to another update), I can confirm that the issue is resolved. Thanks!

@rfay The example commands I provided were solely for the purpose of reproducing the problem. In my private repository, which is a scaffolding project of Craft, I have a folder (with a gitignore file) specifically designated for user-generated files, where the issue was occurring.

I am following the exact same steps as in my example with cpresources, which are:

ddev config --project-name=<project_name> --project-type=craftcms --docroot=web --create-docroot --upload-dir=public
ddev composer create -y --no-install --no-scripts <package>

So, similar to cpresources, my upload_dir (public) is also created during the composer create step and includes a gitignore file.
I'm not sure if these steps are what you mean by "specific setup." If there are any other specific details or configurations you would like me to provide, please let me know.

Thanks, https://github.com/ddev/ddev/pull/5132 will fix it, thanks for your patience on all of this, and for testing and raising the issue in the first place.
I'd love it if you'd test the artifacts at https://github.com/ddev/ddev/pull/5132#issuecomment-1634563891


[zephyr_mirror] add support for zcrypt'ed classes with colons

The regex used for parsing .crypt-table didn't allow colons in class
names. This commit changes the [^:] token with \S, meaning that class
names can now contain colons but can no longer contain whitespace.
I think this should be fine, since zcrypt is only used for MIT zephyr,
where (by convention) class names do not contain whitespace.
Additionally, it should not be possible for us to accidentally consume a
field-separating colon as part of the class capture group because the
regex enforces that all field-separating colons are followed by one or
more whitespace characters, whereas the class name cannot contain
whitespace.

Automated message from Dropbox CLA bot
@jhurwitz, it looks like you've already signed the Dropbox CLA. Thanks!

I looked at the Barnowl implementation of zcrypt to see if there was anything I could learn from it, but their approach (https://github.com/barnowl/barnowl/blob/master/zcrypt.c#L433) is different enough (searching for crypt-<class>: and then parsing that line, rather than parsing every line and trying to extract the class name from that) that the small incremental change I made in this PR seems safer.

Merged, thanks @jhurwitz!  I guess I should deploy this to the locker as well?


[BI-1372] Add ontology sort option for entity + attribute

Description
Story: https://breedinginsight.atlassian.net/jira/software/c/projects/BI/boards/1?modal=detail&selectedIssue=BI-1372
Fix previous error with sorting (mistook trait description for entity + attribute in previous bug fix).
Added Trait column sorting to import preview.
Dependencies
biapi: BI-1372
Testing
See bi-api PR.
Checklist:

[ ] I have performed a self-review of my own code
[ ] I have tested my code and ensured it meets the acceptance criteria of the story
[ ] I have create/modified unit tests to cover this change
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to documentation
[ ] I have run TAF: <link to TAF run>

passed Test.


Can blob reservation discounts apply to/across multiple blob containers/storageAccounts/subscriptions simultaneously???

Page doesn't seem to explicitly answer whether a customer could use a single reservation to obtain reserved pricing for multiple storage containers...  (e.g. - customer purchases 1PB of reserved capacity and desires to create/use 2 separate 500TB blobContainers to exhause/exercise that reservation)

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

ID: b4da9483-7120-9e2c-9861-9dba80034b56
Version Independent ID: 61588662-b963-e5a6-f39d-235244fc46cc
Content: Optimize costs for Blob storage with reserved capacity - Azure Storage
Content Source: articles/storage/blobs/storage-blob-reserved-capacity.md
Service: storage
GitHub Login: @tamram
Microsoft Alias: tamram

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

@tamram
Could you please review add comments on this, update as appropriate.

@Naveenommi-MSFT - Tamra looks to be on extended OOF...  Someone else you can pass to?

@micrileyatx
Thank you for bringing this to our attention.
I've delegated this to content author @stevenmatthew, who will review it and offer their insightful opinions.

@micrileyatx
Thanks for reaching out, and I hope we can stay in touch. I think we're okay to close this, but let me know if you think we should add/update the overview paragraph for "Scope". Happy to apply your feedback.

#please-close


Plugin API feedback / improvements

While working on https://github.com/docker/docker/pull/25548, I noticed some things in the plugin API that may need improvement; putting them in an issue so that I don't forget, and we can discuss.

Plugins can only be referenced by name on the API (not by ID). Referencing by ID may be more consistent with other API endpoints, and also keeps control over possibly "ambiguous" URLs (see other points below)

Due to using names in the URL we may run into issues with plugins installed from third-party registries (my-registry:5000/my/plugin:latest). I'm not sure routing those URL's will always work; they look a bit "scary".

Potentially ambiguous URLs (for example, inspecting a plugin called helloworld/enable would produce URL /plugins/helloworld/enable). This is not a direct issue, because inspect uses GET, and enable POST, but just posting in case.

Some endpoints don't return a message in the response. I didn't check if other endpoints consistently do this, but a "success" message may be nice (e.g. /plugins/(plugin)/enable only responds with HTTP/1.1 200 OK, but no message (e.g. {message: "successfully enabled plugin foo"}

/cc @anusha-ragunathan @tiborvass @bfirsh

Sounds like due to (2) and (3), URL encoding plugin names could get very hairy. Would it be practical to only allow calls with ID, then have a separate endpoint which allowed clients to inspect with a URL encoded name to resolve to ID? e.g. GET /plugins?name=foobar
Agree about (4), but that's a common issue throughout the entire API.

Due to using names in the URL we may run into issues with plugins installed from third-party registries (my-registry:5000/my/plugin:latest). I'm not sure routing those URL's will always work; they look a bit "scary".

Is this even working? The doc page for docker plugin install only mentions the Docker Hub as the host where plugins can come from. Are there already patches in docker/distribution to enable this with the open source registry? If not, is there a plan to enable this in the open source registry?
@tiborvass @anusha-ragunathan @bfirsh @stevvooe ?

@runcom : The doc for docker plugin install is incorrect. It needs to be updated to mention that any registry with version 2.3 and above with work.

I'll try to capture some of my concerns here. Let me know if you need me to expand.
To tell you the truth, I am not a huge fan of the plugin API design, as a whole. Things like PluginSetting are very confusing. Things like PluginEnv are very different from the rest of the API.
For the plugin setting use case, I would just pull this all into a top-level ACL object that uses field paths to declare "exports":
type Exports struct {
  // Path is a slash represented path to a sibling referenced by this export.
  Path string

  Description string
}

type MyObject struct {
   Exports []Export
   // .. interesting hierarchy of fields
}

Potentially ambiguous URLs (for example, inspecting a plugin called helloworld/enable would produce URL /plugins/helloworld/enable). This is not a direct issue, because inspect uses GET, and enable POST, but just posting in case.

Don't do this. It limits future modification and introduces the possibility of ambiguous urls. We did this in the v2 registry API but it took a lot of work to lock it down appropriately and takes extra analysis to ensure holes are introduced.
Also, I would recommend to make provisions to use the plugin manifest digest as the plugin id out of the box.
Sorry for the brevity. I hope this helps.

Small feedback item:
While toying around with the Authz plugin API I noticed that the calls from the Docker daemon  to the plugin don't include a  Content-Type: application/json header.
Worked around it in my Node.js plugin by forcing the Authz server to treat the body as JSON.
Not a huge problem, but non-standard none the less ;-)


Finalize dataset copy UI fixes #451

Move census table and chart classes

@akegan screenshots plz :)

added comment


CPU collision from shape textures

Wave data cameras asynchronously readback the shape render targets to the CPU, where they can then be sample to give water displacement/height information.
There are a number of things that worked out super well:

The code/implementation is simple and fairly short (but relies on experimental APIs, as detailed below)
I've not implemented an interpolation function and I'm not sure its required, point sampling gives good results for me
Collision data is naturally lod'd, as are the shape textures. This gives lower res shape in the distance.
Collision data will match almost 1:1 with rendered surface, so no disturbing artifacts where objects appear to be floating in air or submerged
Collision data will naturally include any shape rendered - this includes wave particles, dynamic sims, and any other custom shape. This also includes height attenuation in shallow water.

Two demo scripts are provided which demonstrate sampling displacement at a position and height at a position.
This is using an experimental unity api which only supports DX11/12 currently, but Unity plan to role it out to other platforms. This api added a dependency on unity version 2018.1 so I upgraded the project as well, and no issues were encountered while upgrading.
I've not noticed any bugs so far, but I've only tested on DX11. However, there are currently a couple of limitations:

There is a small latency between requesting the shape textures and getting them through, typically 2 or 3 frames in my experiments.  If you click Freeze waves on the debug GUI with one of the collision probe scripts active, you can observe a slight shift as the requests catch up. I hope this will not be an issue.
Shape data is point sampled - interpolation could be added if needed
There is no blending across shape textures, which can lead to slight pops and slight error in collision vs visual surface. Blending could be added if needed.

I implemented the new system, seems the new texture look up performance wise is on par with the GetHeightFast I implemented, so thats good. Will update by branch shortly, I made GetHeightFast check the cache then if that fails use the texture, but if the lod is -1 it falls back to the approximate function

Thanks Dizzy! Good to hear feedback about the public API. I initially wanted to force users to call the suggest lod function and give the sample area. This means that for the case that many samples are taken over an area, the lod index only has to be computed once, and a large sampling area will filter out small wavelengths that would be susceptible to aliasing and would not make a big difference to a calculation such as buoyancy.
However this is clearly confusing and I've added functions that simply take a point and return the height/displacement. These are less likely to be efficient this way, but probably much easier to understand. Does this make sense? If you have feedback let me know in the next day or two, otherwise ill get it merged in!


Fix the parsing issue of preprocessing macros within the module's port declarations.

Fix In SystemVerilog, macro definition parsing error #13

I'm not sure if it's because my system is macOS, but the differences between the src/parser.c generated by tree-sitter and the original one are quite significant. The tree-sitter tests have all passed. Could you please evaluate the impact of these changes?
Thank you.

Could you please evaluate the impact of these changes?

If tests are passing it should be perfectly fine.
Thanks @donniek for your contribution :)


Docker "Provide token : " input

Hello!
I am trying to run an authenticated scan using the docker image v1.0.0.
It discovers ok based on the kubeconfig but when I get to the prompt that reads
Provide token :
I can't enter anything. I am trying to paste in a token. Can't type anything either.
It does say choose one of the below options (use arrow keys) but it falls right through to the prompt.
Am I doing it wrong?
Thank you

I was able to paste it into a docker container just now, when you paste it, it doesn't show what you pasted (presumably to stop shoulder surfers from seeing your secret token, believe this behavior of hiding it is intentional)

Oh, ok. Thank you for clarifying. I will try that.


Copy Implementation

SEO copy implemented and set to be visible by default.
Button at bottom of copy which scrolls users back up to Astrostation and hides copy.
Button added to settings which makes copy visible again.
Tooltip added to tutorial for hiding SEO copy.

ONLY ISSUE:
On mobile, height of Astrostation is not adjusting to flex around inner components, resulting in overlap with InfoSection.
I won't be able to look into this until the weekend, if anyone can pinpoint a fix.

Based on this comment in the description:ONLY ISSUE:
On mobile, height of Astrostation is not adjusting to flex around inner components, resulting in overlap with InfoSection.
I won't be able to look into this until the weekend, if anyone can pinpoint a fix.
We should mark this as WIP because I do not want to merge anything that introduces new known issues

Agreed


Proofreading the README

I started to follow the README step by step to try to make it more beginner friendly. The ultimate goal is to have a "copy-pasteable " example so user can try kedroboot out easily. thi is just the beginning, and I'll resume it later.
Here is my journey:
Step 1 : Start from scratch from an empty starter
kedro new --starter=spaceflights-pandas
pip install kedro-boot==0.2.0
conda create -n temp python=3.10 -y
kedro registry list # warning

PB
Failed to load kedro_boot.app.fastapi.cli commands from EntryPoint(name='fastapi', value='kedro_boot.app.fastapi.cli:fastapi_command', utils.py:11                              group='kedro_boot'). Full exception: No module named 'uvicorn'
Solution
pip install -e . # to get pandas and all. This is more du to the starter but confusing
Step 2: try to start a Kedro Server
kedro boot fastapi
PB
no command name fastapi
Solution
This is due to pyctuator not installed => indeed the readme shows "pip install kedro-boot[fastapi]", my bad but the error message is very confusing.
PB 2
"No config patterns were found for 'fastapi' in your config loader" # weird because there is a try/except but it is a MissingConfigError instead of a KeyError https://github.com/takikadiri/kedro-boot/blob/5cacf4556996b122e6a01ce401ef54d85e1badcf/kedro_boot/app/fastapi/app.py#L41-L42
Solution 2

add empty fastapi.yml

💡 Maybe we should add a "kedro boot fastapi init" command ?

update settings.py

CONFIG_LOADER_ARGS = {
    "base_env": "base", # needed
    "default_run_env": "local", # needed
    "config_patterns": {
        "fastapi": ["fastapi*/"],
    },
}

Step 3
The result of the previous step serves on localhost:5000 {"detail":"Not Found"}. this is not niformative. i guess the next step is to create a pydantic model for the dataset to model but this is not very clear.

💡 Maybe we should add a resolver and read it form a metadata in the catalog?

# catalog.yml

my_data:
    type: ...
    filepath: ...
    metadata:
        fastapi:
            schema: ${pydantic:path/to/file.py}

Step 1 and 2 are normally solved. The code is in the main branch. Can you please repass those tests on it before the patch release ?
I'll keep this issue open as it aim at more higher level goal (have a "copy-pasteable" example)

Overall, I get all the information needed in the README, but it is quite hard to follow because it introduces a lot of concepts and we have to go back and forth to understand what to do and how theses concepts are related.
I think it is important to be able to read the README from top to bottom linearly, evntually not explaining all concepts  (we can refer to other sections), because it helps to have something "just working" to try out instead of having to understand concepts first.
My preference order to introduce concetps would go like this :

Standalone mode

Introduce KedroBootSession ("standalone mode) and give examples of how one can use it
Introduce some lightweight explanation about the CompilationSpecs with examples

KedroBoot Apps

Introduce the app concept and give examples and taht it inherits automatically from a KedroBootSession
Introduce the  mapping between CompilationSpecs and namespaced pipeline, and finally an example of an app running  from end to end

kedro boot --app monte_carlo_pi.app.MonteCarloApp --pipeline monte_carlo

Usage: kedro boot [OPTIONS] COMMAND [ARGS]...
Try 'kedro boot -h' for help.

Error: No such option: --app

The run command is missing. It should therefore be:
kedro boot run --app monte_carlo_pi.app.MonteCarloApp --pipeline monte_carlo

In Use Case 1 code example
from kedro_boot.framework.compilation.specs import CompilationSpec

ModuleNotFoundError: No module named 'kedro_boot.framework.compilation

The import could be updated to
from kedro_boot.framework.compiler.specs import CompilationSpec


Update React 16 → 18

Resolves #7
Updated React from v16 to v18.
I did see the warnings about peer dependencies you mentioned, @jenniferabowd, however, I saw those both when upgrading to v17 and then to v18.
Although I saw these warnings, the vulnerability counts remained the same, and the project appears to function the same as it did before. Is this okay for this PR, or is there more work required for this issue?

@jenniferabowd Never mind! I managed to take care of those peer dependency warnings as well by updating…

react-test-renderer
react-redux
redux

So this would be the solution to update everything this PR accomplishes in a single command:
npm install react@latest react-dom@latest react-test-renderer@latest react-redux@latest redux@latest

I just split it into 2 different commits 🙂

@brandonmcconnell if you are doing this for Hacktoberfest, feel free to split up the PR since it is addressing several issues.

@jenniferabowd Sure, I'll do that. Thanks!

Oops! I already merged it! However, I see you assigned yourself a few things.

@jenniferabowd No worries!


Fixed exceptions in some JDBC drivers when working with chars

Not all JDBC drivers support java.lang.Character (for whatever reason). Notably MariaDB, Oracle and SQL Server. I've added a FieldConverter to convert the SQL arg into a String which is accepted by all 3 (+ 1 for JTDS) JDBC drivers. Note that the underlying SQL type remains unchanged so this should have no backwards compatibility concerns.
The new CharacterCompatFieldConverter currently resides in this repository but it should probably be moved into ormlite-core. Perhaps into a new package com.j256.ormlite.field.converters where we can also move BooleanNumberFieldConverter too (currently resides in BaseDatabaseType).
This fixes some exceptions first reported in issue #2.

Hey @Bo98 tried to ping you via your email.  Any interest in becoming an ormlite maintainer?  Not a ton of extra work but I'd love to get another pair of eyes on it.  Please ping me at my contact form.  https://256stuff.com/gray/email/


Currying types for deep fusions

A proper explanation is lacking, but...
 (.))

-- Three BiFunctions
fused a b c d = h (g (f a b) c) d
fused = (((h .) . g) .) . f

fuse f g h = (((h .) . g) .) . f
fuse = flip (flip . (((.) . (.)) .) . flip ((.) . (.)))

-- Four BiFunctions
fused a b c d e = i (h (g (f a b) c) d) e
fused = (((((i .) . h) .) . g) .) . f

fused f g h i = (((((i .) . h) .) . g) .) . f
fuse = flip (flip . ((flip . (((.) . (.)) .)) .) . flip (flip . (((.) . (.)) .) . flip ((.) . (.))))

So, carve some time and see if there's a way to emulate a pattern like this in Java

See also:
https://github.com/hiljusti/z/tree/explore/currying-types


it seems that google-authenticator doesn't work with locked screen?

my google-authenticator enabled  ubuntu 17.10 fail to unlock when system was auto-locked.
i have to restart...

Reopen if you add way more details.

Hi! This is Mohammad Mahin Uddin and the beginer user of Bitcoin.I am a poor Bangladeshi student. I bought 0.22722849 BTC from my tution fees and receive this 0.22722849 BTC in your bitcoin.com wallet app. I have deleted my bitcoin.com wallet mistakenly from my iPhone 6 and I did not write its back phases for recover fund.
How can I recover my bitcoin address which had generated in your bitcoin.com's wallet app. I don;t have recovery passes.
Can you please send me my wallet address of this bitcoin.com wallet app. I have fews information which will be useful for you.

icloud email ID is<EMAIL_ADDRESS>Notification email ID<EMAIL_ADDRESS>Receiving bitcoin wallet address are following:

thanks you for your response, i  am sending you all transection associated with my wallet ID.
1.Receiving Wallet address :16mnR2U7SVi5B7TSxuKBCV5gMwBZtfiDLY  and it's transection Id; 7b732acf6722b7dafa846876240446a1c11a1df4c0724ec91133fc6a0966f321
2.Receiving Wallet address:1GZufdYXQTZcw6kkAxhP8RXW5817b8HfbX and its transection ID: 68208d6b3a4d9352ee265b2a2340ffbff9c90dd994db745797bd8ed60ce4b544
3. Receiving Wallet address:  1FPbQwrMZnYP3NTKvdbAWzVW2KBKjYygVR  and its transection ID: fb1fb83072e7a5b286bc6f67786599596d96d343a78afb4863c0b4b41318d2e3
Please send me my wallet information from this following information.
waiting for you.
Your lovely,
Mohammad Mahin Uddin


Change in path api does not follow semantic versioning guidelines.

Firstly, thanks so much for creating this package.
As described, the change of path from a getter to a function does not follow semantic versioning guidelines (should bump a major version). This will cause confusion in the users, as dart will load the latest version that's compatible with the version specified, but that will cause error in the code when you try to compile and run it.
Because this took me a while to find out and the code linter would essentially not give any help with that regard (but the code does not run), I think it may be best if this could be made into a breaking change next time :-).
Thanks again for the package!

@wxxedu sure, my bad, i will make sure to bump major version with breaking change next time


sasjs build on SAS 9 is not recognising the jobs folder

When sasjs build creates the deployment.sas program, both jobs and services are being created in the services folder:

Expected:  Jobs should go in a jobs folder.  This means adjusting the part which creates the SAS (deployment) program.

This is also a problem for TESTs


Is this lib dead?

Hi,
I see that this lib has a lot of really great PRs pending which include serious bug fixes and a lot of performance improvements. Especially the PRs from @timurrrr.
Many issues are unanswered as well as many topic in the discussions.
The last commits cover minor changes (link fix... WHO CARES?!?), no real improvements.
So: is this lib dead?!? Maybe it should be removed from the Arduino list or replaced by a better maintained fork?

Hi @Jojo-A. Did you found a maintained alternative?

Thanks, I will try it. Probably the bug I've noticed is not present there.

It runs rock-stable in my application.

Are you using VScode + Platformio?
What platform version?
After hours of investigation, I found that using a version of platform-espressif32 newer than 3.5.0 (and 4.2.0 in another case) makes the ESP32 reboot in a loop when connected to a CANBUS network (95kbps). Using ESP32 devkitC v1 and v4 along with SN65HVD230 transceiver combo.
A simple sniffing sketch like this makes the device reboot in a loop:
https://github.com/adamtheone/canDrive/tree/main/01_canSniffer_Arduino
Recent versions of the Arduino IDE (2.x) have the same bad outcome. Arduino IDE 1.8.19 (and platform-espressif32 @ 3.5.0) work fine.

Are you using VScode + Platformio?
What platform version?

Oh, this is some time ago already.
I just used Arduino IDE v2.0.3 and the BSP for esp8266 3.1.1 and the BSP for ESP32 2.0.6 (iirc).
Please open another issue to discuss your bootloop problem. I started this issue to find out how good (or bad) this lib is maintained.

This issue helped to solve my problem. I was pulling out my hair for 6 hours with 2 esp32 wroom module connected  with MCP2551 transceiver chip. And I finally came across this issue.
The fork mentioned worked out of the box.
https://github.com/timurrrr/arduino-CAN
Anyway thanks for this wonderful Library.


Enhacement | security-keys: Update versions constraint + Update module version

What?

Update and test Ref Architecture layers  with tf latest versions, Leverage CLI latest version and SSO feature enabled.

How?

Update tf version constrains ~> v1.2.7
Update tf aws provider constrains  ~> v4.10
Update module version to the latest available

Environment Versions

Leverage CLI : v1.8
Terraform:  v1.2.7
provider registry.terraform.io/hashicorp/aws v4.46.0

Layers
Why?
Keeping Leverage Reference Architecture up to date.
References
GitHub issue https://github.com/binbashar/le-tf-infra-aws/issues/370

$ leverage --version
leverage, version 1.8.0

$ leverage terraform version
Terraform v1.2.7
on linux_amd64
+ provider registry.terraform.io/hashicorp/aws v4.46.0


Ability to import parts with a chosen part id that is not currently in use

Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find a similar feature request

Problem statement
I want to import parts from a spreadsheet, and I have already assigned unique IDs to them. I want to use these unique IDs as their part id in InvenTree so that if I ever need to update them, I can reimport and the new things will populate to the correct place. Currently, the documentation says that to add a new part, import it with part id blank and the next part id will be assigned to it.
Suggested solution
So the solution would be to make it search for existing part ids (as it already does). If one is not found (currently it throws an error) then insert a new part with the not found part id and the other information in the row being imported.
Describe alternatives you've considered
An alternative would be to give back a list of generated Part Ids in the order that they are assigned so that they can be inserted into the file type that was used for the import, but this seems less ideal.
Examples of other systems
No response
Do you want to develop this?

[ ] I want to develop this.

I could also help to make this happen, but I would need a bit of guidance/help with starting.

That is not compatible with how the database works - the database backend assigns unique IDs and we cannot pass our own ID values to it. The "ID" value must be blank for new items being imported.
Note that there is currently a large project to completely overhaul the data import framework: https://github.com/inventree/InvenTree/pull/6911

IMO this is not planned - can we close this?

Closing, this will not be supported.


Memory/Ram Icon

Icon Request

Icon name: Ram/Memory
Use case: Show ram amount on a server, etc
Screenshots of similar icons:
https://fontawesome.com/icons/memory?s=solid

that's a good idea, but recreating the one you sent while respecting the guidelines seems impossible. we would need to make some exploration

@raphael141714 created on in #527 , but it does need some refactoring to match the guidelines.

I mean it looks pretty much exactly like a stick of ram just not as wide when you look at like thin for example

Closed as duplicate of #527


Add indicator of what model responded to message and menu to try response with a different model

For example..at bottom of message add greyed out "response by chatgpt-3.5" with a little menu, that has buttons to "retry with gpt-4/claude/etc". This would also in the future support the full set of params eg temperature, prompt, etc. Maybe this would tie in with threads(https://github.com/tarasglek/chatcraft.org/issues/40), receipts(https://github.com/tarasglek/chatcraft.org/issues/61)
This would make it easy to compare models plus their params via "retry with gpt4", make for some good blog posts.
One potential UI like..is clicking on the "response by chatgpt-3.5" would open sidebar where we can tweak prompt, model, etc and click retry

I think we have the pieces in place to explore this more easily now.  Consider:

We could put a "gear" icon or something near the date or name of model, and it could open an accordion showing a form with model params used to generate it.  It could also include a "Retry" button.
Or we could integrate this into the "dots" menu on the right somehow.
Let's say I have a chat like this:

message 1
message 2
message 3
message 4
message 5

I am interested in having message 3 get retried with a different model.  What is the UX?  I tweak the params and click "Retry" and then it sends [message 1, message 2, message 3] to the new model (i.e., ignores 4 and 5) and then replaces the text in message 3 when it comes back?  Or does this delete message 3 and create a new message in its place?

oooh..if we integrate it into the menu, then it will be really slick..just add a "retry-with" menu that opens up to submenu of all available models

How about my other question though: what does retrying an arbitrary message in a chat mean in terms of the context you send?

tabbed messages is an interesting concept

Threads might be interesting too. I was thinking tabs because I like how OpenAI does it (although these aren't really functional after the first message - but maybe it would work?). I haven't fully considered all the implications tho.

i'd be ok with this too

In addition, see/test https://github.com/tarasglek/chatcraft.org/pull/98, which adds in support for initially sending with a given model without needing to dig into settings.
I'll think on the ideas in https://github.com/tarasglek/chatcraft.org/issues/59#issuecomment-1581067673, thanks for adding them.

This has landed.


Added no man's sky

Guess my whole attempt at renaming the binary in MO was flawed removed that attempt and it should work now.


BatchNormalization layer has different output when loading a checkpoint saved from TF 2.0

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Official Tensorflow Docker Container (2.0.0, 2.0.0-beta1, latest) with py3 and gpu support
TensorFlow installed from (source or binary): Docker
TensorFlow version (use command below): 2.1
Python version: 3
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 10.0 / 10.2
GPU model and memory: Tested on RTX 2080Ti, Titan V, GTX 1080

Describe the current behavior
In my project I trained several (adversarial) segmentation models using both Tensorflow 2.0.0 and 2.0.0-b1.
Upon upgrading Tensorflow to version 2.1 I noticed a decrease in the accuracy of my models between 5~15% (Dice Score) with respect to the version 2.0.
I ran several tests loading the same checkpoints in TF 2.0 and in TF2.1 by comparing the layer outputs and I noticed that starting from the first BatchNormalization layer the outputs between the two versions are different (even though the loaded weights are the same). BatchNormalization behavior is consistent between all 2.0 releases, but changes for 2.1.
Describe the expected behavior
The output of BatchNormalization layer (and thus of the network) should be the same when using TF 2.0 and TF 2.1 if loading the same checkpoint.
Standalone code to reproduce the issue
I tried reproducing the issue using the examples or pretrained networks, but the differences are not noticeable as they are in my models. I wrote a notebook that can be run on both 2.0.0 and 2.1 to highlight the differences.
https://drive.google.com/open?id=1vw3vaPU32lqI2jAOvp0toXIgD84R_Tx4
Besides the notebook there are also the input data I used for testing and the output of the last layer for both TF2.0-b1 and TF2.1 (TF2.0 alpha and stable are not included as they are the same of TF2.0-b1). I'm not including individual layers outputs due to size constraints.
In the pics fodler can find also some visual examples of the network outputs.
Other info / logs
I'm attaching just an example of just one mismatching output for the same input. In the drive folder you can find the test for an entire batch.

I couldn't understand what this could be related to, I'd like to have some insights of what could be the cause.
Thank you!

@edoardogiacomello,
I tried to reproduce the issue but did not observe much difference in the output with TF v2.0, TF v2.1 and TF v2.2.0-rc3. Please find the attached gist. Thanks!

@edoardogiacomello,
I tried to reproduce the issue but did not observe much difference in the output with TF v2.0, TF v2.1 and TF v2.2.0-rc3. Please find the attached gist. Thanks!

I'm sorry, I may have made a mistake while managing the different containers. I checked again and I've been able to reproduce the error in version 2.0.0-beta1 (both on Docker and Colab), so the change may have happened between 2.0.0b1 and 2.0. Starting from 2.0 (stable) the results are consistent with future versions, as you showed.
I'd like to find out what's causing this difference in BatchNormalization outputs and if there's a way to adapt my checkpoints to the stable version without a complete re-training.

@edoardogiacomello Please feel free to close the issue as this was resolved already. You could use stable version of TF2.1 or TF2.0. Thanks!

@jvishnuvardhan BatchNormalization layer may have been fixed (it changed for sure), but this issue is referred to loading a model that is trained with a previous version and it doesn't behave the way it should. I don't see how using a stable version could fix the issue, which is indeed related to loading a beta checkpoint in a stable version.
Could you give a reference on what the issue was or how it has been resolved? I'd like to avoid re-launching about 8 months of training (or getting stuck into beta for the whole project) if there could be a workaround. Thank you :)

There were a number of changes between the beta and 2.1, and we would recommend retraining or adjusting weights as necessary. For long-lived models, it is best to use stable versions of TF, as nightlies and RCs sometimes contain issues that require non-backwards compatible changes versus the subsequent release.


Link to the license translation proposal form seems broken

Hi Rob,
Can you please have look at the link below? It seems the link to the translation proposal form on the wiki is not working.
https://wiki.creativecommons.org/index.php?title=Form:4.0_Translation&action=edit&redlink=1
Thanks!

Thank you for reporting this.
The CC0 Translation form is also missing:
https://wiki.creativecommons.org/index.php?title=Form:CC0_Translation
I cannot find anything in the logs (in particular, I don't think I accidentally deleted these pages during the cleanup last year).
@mattl is this an SMW removal issue?

@robmyers Yes, this is SMW.

@mattl is there a substitute in MediaWiki we can use, would Civi or GForms be a good replacement, or should we just ask people to "Please email  with the following details..."?

I'm not clear what the form was -- if it was just a "Hey, you should do Spanish next" kinda thing, then it could just be an "email us" piece of text. For anything more structured, it could just be a Google Form, or a Civi Profile. Civi Profile if any personal information.

It seems to be a multi-page form, so I cannot recover it from the Internet Archive:
https://web.archive.org/web/20150502224343/https://wiki.creativecommons.org/Form:4.0_Translation
@peterspdx What would your preferred replacement for these forms be? What information is needed to start the translation process?

@soohyunpae I'd contact Diane with any translation requests while we're working on sorting this out.

@robmyers Great, Thanks!

There's certainly a preferred form or method for starting a translation and notifying CC of someone's intention to do so. It consists of basic information about the team, timeline, language, the plan for accomplishing the translation, etc. Basic info is always good, and to have it populate a page that is publicly viewable is the idea.
I'm not aware of new translations being proposed, but agree that for the time being I am happy to act as recipient of such basic information until we figure this out.
@mattl, @robmyers, let's put this on an upcoming tech - legal call. I'll add it to an agenda for a week or so out.
@soohyunpae let me know if there's any impending new translation in the works.


UnicodeDecodeError during installation

Hi Uwe/Martin,
I am trying to install stardist on a CentOS system with python-3.5, keras-2.2.4, tensorflow-1.6.0. I am getting a UnicodeDecode error after trying to install stardist via pip. The error message is the following:
(tensorflowNEW) -bash-4.2$ pip install stardist
Collecting stardist
Using cached https://files.pythonhosted.org/packages/88/d5/51972b9c6e210a34c44c263a1ac60d525271c5d90a802df5ecc9d66287fa/stardist-0.3.3.tar.gz
ERROR: Command errored out with exit status 1:
command: /data/u934/service_imagerie/v_kapoor/anaconda2/envs/tensorflowNEW/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-d3zbl816/stardist/setup.py'"'"'; file='"'"'/tmp/pip-install-d3zbl816/stardist/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info --egg-base pip-egg-info
cwd: /tmp/pip-install-d3zbl816/stardist/
Complete output (7 lines):
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-install-d3zbl816/stardist/setup.py", line 46, in
long_description = f.read()
File "/data/u934/service_imagerie/v_kapoor/anaconda2/envs/tensorflowNEW/lib/python3.5/encodings/ascii.py", line 26, in decode
return codecs.ascii_decode(input, self.errors)[0]
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 769: ordinal not in range(128)
----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

Hi,
This seems to be similar like https://github.com/mpicbg-csbd/stardist/issues/16
Could you try if the link mentioned there helps?

Ok, there was indeed an offending non-ascii character.
If you install the latest version (stardist 0.3.4) it should be fixed.

Yep that fixed it. Thanks a lot Martin.


Review levelized cost

@OFR-IIASA thanks for this. I added the part related to relation_cost. As this cost can be related to relation_total_capacity, relation_new_capacity and relation_activity, I added all of them. I also improved the documentation and corrected some inconsistencies in indexes.
I just notice that your PR is opened under IIASA's remote. I assume you checked this before.

@behnam-zakeri would you please assign the PR to yourself or other people working so it doesn't come in the list for non-assigned PRs.

@behnam-zakeri it seems this PR was automatically closed as the branch to which this PR was originally made has been deleted. Can you reopen this PR and change the destination branch to main?


Divided by zero

Divided by zero should raise an exception or return some flag to identify the divided by zero case. currently execute function returns 0 in case of divided by zero i.e 2/0 or 0/0

Better support for divide by zero is now available.


New sample script: Get all the Teams a specific user is part of

Docs

Author
Veronique Lengelle (@veronicageek)

Original Post
https://veronicageek.com/powershell/powershell-for-m365/get-all-teams-a-user-is-part-of-using-powershell-pnp/2020/10/

Description
Get all the Teams a specific user is part of

Keywords
Microsoft Teams

Happy to be assigned to it. Can submit a PR tomorrow.

Awesome! Thank you @veronicageek ❤


Add DaskBakery

Thanks to @alxmrs's heroic efforts, Beam has a DaskRunner. This PR adds that runner as a configurable option here.
This should be a broadly useful deployment target, and was specifically motivated by our struggles in https://github.com/pangeo-forge/pangeo-forge-recipes/issues/618 to establish the usefulness of the LocalDirectRunner's usefulness for single-machine parallelization. Pangeo Forge users with this use case can leverage Dask's LocalCluster for this purpose going forward.
I've so far been able to confirm the scaling functionality of the DaskRunner + LocalCluster with the following example (h/t @TomNicholas for help with this):
# dask_runner_test.py

import time

import apache_beam as beam
from apache_beam.pipeline import PipelineOptions
from apache_beam.runners.dask.dask_runner import DaskRunner
from dask.distributed import LocalCluster
from dask.distributed import Client

def sleep(x):
    time.sleep(2)
    return

input_data = list(range(1, 30))
worker_counts = [1, 2, 4, 8, 16]

if __name__ == "__main__":
    for n_workers in worker_counts:
        print(f"Mapping function 'sleep' across {n_workers = }")

        cluster = LocalCluster(n_workers=n_workers, threads_per_worker=1)
        client = Client(cluster)
        client.wait_for_workers(n_workers)
        options = [f"--dask_client_address={client.scheduler.address}"]

        start_time = time.time()
        with beam.Pipeline(runner=DaskRunner(), options=PipelineOptions(options)) as p:
            result = (
                p
                | "CreateInput" >> beam.Create(input_data)
                | "Map" >> beam.Map(sleep)
            )

        end_time = time.time()
        elapsed_time = end_time - start_time
        print(f"Elapsed time with {n_workers = }: {elapsed_time} seconds")

➜ python dask_runner_test.py
Mapping function 'sleep' across n_workers = 1
Elapsed time with n_workers = 1: 58.47139596939087 seconds
Mapping function 'sleep' across n_workers = 2
Elapsed time with n_workers = 2: 30.290652751922607 seconds
Mapping function 'sleep' across n_workers = 4
Elapsed time with n_workers = 4: 16.250579833984375 seconds
Mapping function 'sleep' across n_workers = 8
Elapsed time with n_workers = 8: 8.254508972167969 seconds
Mapping function 'sleep' across n_workers = 16
Elapsed time with n_workers = 16: 4.343317985534668 seconds

This is pretty fucking cool!
How do we test it?

How do we test it?

We can use a Dask LocalCluster in Github Actions. I will add this test once I have it working locally.

Update:
Features missing from the current Beam DaskRunner, which we need to make this work for Pangeo Forge:

Side inputs. This is addressed by https://github.com/apache/beam/pull/27618 which I am now working on getting merged. (See that thread for current status.)
Combiners. I will need to make a follow-up PR to beam to implement this after Side inputs go in.

Combiners. I will need to make a follow-up PR to beam to implement this after side inputs go in.

Note to self: looks like this can be implemented with dask.bag.Bag.foldby, which AFAICT is conceptually identical to Beam's CombineFns.


PLAT-79165: Updated the Touchable sample to better demonstrate its intent

Checklist

[x] I have read and understand the contribution guide
[ ] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[x] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
The Touchable sample is confusing regarding what it's attempting to demonstrate.
Resolution
Added a visual element to show where the tap-start radius is checking for when pulses should be fired.

Updated to hopefully avoid the linting issues and provide slightly better separation. It's a little wonky but works as expected.