fix: spacing in picasso stories

FX-4261
Description
⚠ The https://picasso.toptal.net/?path=/story/tutorials-how-to-use-spacings--how-to-use-spacings story is updated in https://github.com/toptal/picasso/pull/3871
This pull request updates spacings in Picasso Storybook. Most of the changes are coming from running the codemod
npx @toptal/picasso-codemod v38.1.0 **/*.example.ts*
npx @toptal/picasso-codemod v38.1.0 .storybook/**/*.example.ts*

There are custom adjustments (for example, converting top={7} to style={{ paddingTop: '7rem' }}), they were kept to have the same view of stories.
How to test

Temploy
Check any story that uses Container with spacing properties, they should be updated

Development checks

[N/A] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[x] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[N/A] Annotate all props in component with documentation
[N/A] Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[N/A] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[N/A] Covered with tests (visual tests included)

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers

@toptal-anvil ping reviewers


Fixed naming convention for the UDF

Updated the naming for the Skyflow search function to skyflow_search to be consistent with Python and Snowflake conventions.

Thanks @thefalc!
@NavidKagalwalla Sean pointed out that actually Snowflake uses a snake case convention for these function names so he proposed changing them for the submission to Snowflake. We can chat when you're back.


Debug API "/health" and "/readiness" need more distinction in documentation

I don't get differences between these two apis in documentation, and they look to be the same.

.


Update README with Model Specifications

Added an overview of the model as discussed in response to #14.
Adding more info on the the model specs before they proceed to download the checkpoints should help folks ensure they have the necessary resources to effectively utilize Grok-1.

Can someone also add the minimum gpu required or hardware required to run the model ?

This is great, thank you!


Add spanish translation for stratum-v2

Hello @kouloumos I can't see what the error is  :(

Hello @kouloumos I can't see what the error is :(

all good @BlueeeMoon , this doesn't have to do with you. It's something we need to change in the linting process.

Hello @kouloumos I can't see what the error is :(

all good @BlueeeMoon , this doesn't have to do with you. It's something we need to change in the linting process.

Super, thank you very much, I'll keep an eye on it :)


Add seed_stack module

A new module that wraps up the concepts of "controller" and "worker".
A controller has the following:

Zookeeper
Mesos master
Marathon
Consul server
Dnsmasq for Consul DNS
Consular

A worker has:

Mesos slave
Consul client
Docker
Dnsmasq for Consul DNS
Nginx
Consul Template
Templates to configure nginx for service routing

The intention is for this to eventually be a standalone Puppet module that is published on Puppet Forge. For now, this is an easy place to get started. We need to have published Puppet modules for Marathon and Consular before this can be a standalone module.
Other things to do:

Nginx webserver/loadbalancer (a-la Unicore)
Docker registry stuff (not sure where this should be run)

Do we not perhaps need consul clients and dnsmasq on the controllers as well?

@jerith: A Consul server does all the same stuff as a client it just does leader election stuff so just having a server on the controllers is fine.
It probably makes sense to have Dnsmasq on the controllers so that they can easily use Consul DNS but it's not used for anything currently afaik...

@jerith: Added dnsmasq to controllers

:+1:


Strong name signature not valid for this assembly

Hi,
After I updated to version 2.0 I'm getting exception from ClickOnce:
Following failure messages were detected:
+ Strong name signature not valid for this assembly DataAnnotationsValidator.dll.
To solve the problem I compiled the code locally and included it with my application without a signature.
Version 1.x worked fine.

I see the same in my IIS application when pulling 2.0 from NuGet.  Also of note is that the version number in the DLL seems to have gone backwards from <IP_ADDRESS> to <IP_ADDRESS>: only the NuGet path shows 2.0.
Thanks,
Alex

I have the same issue

I just published version 2.1.0 (might still take a while to update the nuget index before you can see it to upgrade) after rebuilding with signature, please lemme know if you still have issues


More fixes, especially for more restrictive setups

This is mostly unreviewable unless you ignore whitespace or look at one commit at a time


Cells with HTML  tags do not appear in filter choices

(Original issue 28 created by altmany on 2012-09-30T10:31:53+00:00)
What steps will reproduce the problem?
1. create a JTable with a cell that includes "<html><img src='image.gif'>"
(i.e., just an icon image, no text)
2. filter the table; try to select the image rows from the filter

What is the expected output? What do you see instead?
I expect to see the cell image in the list of filter options. Instead, I see nothing
- I cannot filter for rows that contain images.

What version of the product are you using? On what operating system?
4.4

Please provide any additional information below.
This is a direct consequence of the fix for issue #23 - If the cell contents are entirely
within HTML tags, then the filter disregards the cell and the user cannot filter accordingly.
HTML <Img> tags are a special case that should get special treatment.

Original issue reported on code.google.com by altmany on 2012-09-30 10:31:53

(Original issue 28 last updated on 2012-09-30T10:31:53+00:00)
(Issue automaticaly closed due to status in Bitbucket: resolved)

(Original comment by coderazzi on 2012-10-03T01:51:02+00:00)
Hi,

There is basically to behaviors associated to each filter editor in the filter header:
text-oriented editors perform text searches, implementing logic such as wildcards and
comparison operators. The implementation done for issue 23 implied extracting the valid
HTML information from the content, which by default is to exclude those HTML tags -this
behavior can be modified by providing a different Parser implementation, by the way-.

The other type of filter editors is choice-oriented, where the user can only choose
one option from the associated combobox. In this case, what is already possible is
to define a net.coderazzi.filters.gui.ChoiceRenderer for that column. Note that a basic
implementation for this Renderer, using a JLabel would already show perfectly your
image -or any HTML content- (well in so far as the JLabel does it correctly).

Two caveats:
1- The height of the editor is fixed (larger ones can be created by modifying the TableFilter'
settings). Care must be taken to have proper renderers.
2- The default implementation will still be accessing the content as text: that is,
not because the library detects HTML in the column, it should be expected to use HTML
renderers. That is up to the user.

So, in summary, I believe your issue can be solved by setting a ChoiceRenderer that
delegates into a JLabel to render the HTML content -and such renderer could be, indeed,
part of the FilterHeader library-.

What do you think?

Cheers,

 Lu.

Original issue reported on code.google.com by coderazzi on 2012-10-03 01:51:02

(Original comment by coderazzi on 2012-10-03T01:51:36+00:00)
Hi,

There are basically two behaviors associated to each filter editor in the filter header:
text-oriented editors perform text searches, implementing logic such as wildcards and
comparison operators. The implementation done for  issue 23  implied extracting the
valid HTML information from the content, which by default is to exclude those HTML
tags -this behavior can be modified by providing a different Parser implementation,
by the way-.

The other type of filter editors is choice-oriented, where the user can only choose
one option from the associated combobox. In this case, what is already possible is
to define a net.coderazzi.filters.gui.ChoiceRenderer for that column. Note that a basic
implementation for this Renderer, using a JLabel would already show perfectly your
image -or any HTML content- (well in so far as the JLabel does it correctly).

Two caveats:
1- The height of the editor is fixed (larger ones can be created by modifying the TableFilter'
settings). Care must be taken to have proper renderers.
2- The default implementation will still be accessing the content as text: that is,
not because the library detects HTML in the column, it should be expected to use HTML
renderers. That is up to the user.

So, in summary, I believe your issue can be solved by setting a ChoiceRenderer that
delegates into a JLabel to render the HTML content -and such renderer could be, indeed,
part of the FilterHeader library-.

What do you think?

Cheers,

 Lu.

Original issue reported on code.google.com by coderazzi on 2012-10-03 01:51:36

(Original comment by altmany on 2012-10-03T23:04:52+00:00)
Sounds like a good plan. But why not automatically use the HTML renderer if <html> is
detected in 1+ cell(s) of the column? I think this would be more intuitive. Of course
the user can always switch between text/HTML/custom renderers, I'm just saying regarding
the default behavior.

Original issue reported on code.google.com by altmany on 2012-10-03 23:04:52

(Original comment by coderazzi on 2012-10-04T00:49:24+00:00)
I think that letting the developer choosing how to handle the column is better, also
from a performance point of view.
Furthermore, the header handles automatically changes in the table, so how to react
if one cell in the column has HTML info, and after an update, it doesn't?
It is not just a matter of handling the content differently, but the user interaction
would change as well (with a renderer, the user cannot enter free text on the editor).

Finally, although the solution of using a renderer would solve the problem of handling
images:
1- Perhaps it is still better searching for text.
2- I am not confident on how the rendering of the cell will look on the editor. That
is, the rendering will show the image, but obvious work like fitting the content to
the editor cell will be missing...

Cheers!

Original issue reported on code.google.com by coderazzi on 2012-10-04 00:49:24

(Original comment by coderazzi on 2012-11-18T00:54:22+00:00)
As proposed, a new renderer called HtmlChoiceRenderer can be setup on the columns with
html content.
Code included already on the repository, will be included on release 4.5.0

Original issue reported on code.google.com by coderazzi on 2012-11-18 00:54:22


Patch logging refactor

do less for logging.

@yunstanford does this PR improve baseline performance? Can you post a benchmark?

it has nothing to do with perf improvement. this pr just refactored the logging pattern, setup, disable, etc.
As for performance, it really depends on logging infra or how user wanna setup the logging. i'd like to let user make the choice.

Resolved conflicts.

@yunstanford sorry can you resolve the conflicts one more time please?

ok, resolved.


The .dll download link on the website points to an old version

The .dll download link on the website points to v3.1.229 while the current version is 4.13.0
Can you update it to point to the latest version?
(Normally I'd just use NuGet, but I'm working on a project where NuGet is not allowed.)

Fixed!

Oh, and thanks for reporting :+1:

Assuming it gets merged, I have removed these download links from the website.
When plain DLLs are needed they can still be acquired via NuGet.  A .nupkg is just a zip file containing certain files and it is easy to extract DLLs from it.  The nupkg can be downloaded NuGet.org without needing to use any NuGet client.


Coc api fix

the main change is page request,reason is here .so many changes was made to fix it,and some api broken.please review .
some pojo object need update too.

Will need to look into this further


Starfails parses aegea batch ls incorrectly to find failed jobs

??aafa57c3-91b1-4dea-a530-9a6af1a861
Traceback (most recent call last):
  File "/home/shayanhoss/miniconda3/envs/utilities-env/bin/aegea", line 23, in <module>
    aegea.main()
  File "/home/shayanhoss/miniconda3/envs/utilities-env/lib/python3.6/site-packages/aegea/__init__.py", line 82, in main
    result = parsed_args.entry_point(parsed_args)
  File "/home/shayanhoss/miniconda3/envs/utilities-env/lib/python3.6/site-packages/aegea/batch.py", line 452, in describe
    return clients.batch.describe_jobs(jobs=[args.job_id])["jobs"][0]
  File "/home/shayanhoss/miniconda3/envs/utilities-env/lib/python3.6/site-packages/botocore/client.py", line 357, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/home/shayanhoss/miniconda3/envs/utilities-env/lib/python3.6/site-packages/botocore/client.py", line 661, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.errorfactory.ClientException: An error occurred (ClientException) when calling the DescribeJobs operation: Provided name \udc94\udc82aafa57c3-91b1-4dea-a530-9a6af1a861 is not supported pattern[a-zA-Z_0-9-]{1,128}```

│a3ce933c-edf3-4d69-afe0-167d0ef63095│42 minutes ago│
│422bed55-99af-491d-a6da-b35cf624b896│29 minutes ago│```

Parsing at 2-37 and should be between 4-39 to grab full `job_id`

Parsing at 2-37 and should be between 4-39 to grab full job_id

I think this was fixed by #31 because the starfails script is now using the aegea API.


lazy val leads to illegal premature access error.

This could be related to the following closed issue (https://github.com/propensive/magnolia/issues/95)
I've came up with code example which partially based on magnolia's examples. Full showcase repo could be found here.
Based o such trait and derivation rules:
trait Decoder[T] {
  def decode(value: String): T
}

object Decoder {
  /** decodes strings */
  implicit val string: Decoder[String] = (s: String) => s

  /** decodes ints */
  implicit val int: Decoder[Int] = _.toInt

  def gen[T]: Decoder[T] = macro Magnolia.gen[T]

  type Typeclass[T] = Decoder[T]

  def combine[T](ctx: CaseClass[Decoder, T]): Decoder[T] = value => {
    val (_, values) = parse(value)
    ctx.construct {
      param =>
        param.typeclass.decode(values(param.label))
    }
  }

  def dispatch[T](ctx: SealedTrait[Decoder, T]): Decoder[T] = param => {
    val (name, _) = parse(param)
    val subtype   = ctx.subtypes.find(_.typeName.full == name).get
    subtype.typeclass.decode(param)
  }

  private def parse(value: String): (String, Map[String, String]) = ???
}

I am trying to provide a convenient type WithDecoder which contains implicit val dec: Decoder[T]. I've described WithDecoder as follows
abstract class WithDecoder[T]()(implicit decoder: MaterializedDecoder[T]) {
  implicit val dec: Decoder[T] = decoder.value
}

final class MaterializeDerivationMacros(val c: blackbox.Context) {
  import c.universe._

  def materializer[T: c.WeakTypeTag]: c.Expr[MaterializedDecoder[T]] = c.Expr[MaterializedDecoder[T]] {
    q"{ new ${weakTypeOf[MaterializedDecoder[T]]}(_root_.issue.Decoder.gen[${weakTypeOf[T]}]) }"
  }
}

final case class MaterializedDecoder[T](value: Decoder[T])
object MaterializedDecoder {
  implicit def materialize[T]: MaterializedDecoder[T] = macro MaterializeDerivationMacros.materializer[T]
}

When I've tried to compile the following code snippet, compile throws an error
object models {
  case class Test(s: String)
  object Test extends WithDecoder[Test]
}

Error:(5, 23) Implementation restriction: access of lazy value paramTypeclass$macro$3$1 in object Test from <$anon: Function0>, would require illegal premature access to object Test in object models
  object Test extends WithDecoder[Test]

So, the question is there any way to fix this?
Could somebody help with that?

Wow, that looks hairy... :( Thanks for the clear bug report, though.
I don't remember enough about #95 to give an authoritative answer, but maybe @joroKr21 does. Do you think you could experiment with reverting #96 in Magnolia, or changing that val to a def instead, and seeing if that makes any difference?

@propensive, yep, I could play around. I'll let you know if those changes make any difference.

@propensive, yep, I could play around. I'll let you know if those changes make any difference.

I'm afraid this won't work. You can't define an object that references itself in the extends clause.
And the companion object is referenced in any typeclass generated by Magnolia.
The error message is misleading but I believe it hides a fundamental restriction.
Shapeless has the same problem. Here is an example using vanilla Scala:
scala> :paste
// Entering paste mode (ctrl-D to finish)

trait Default[A] {
  def default: A
}

object Default {
  def instance[A](value: A): Default[A] = new Default[A] {
    def default = value
  }
}

abstract class WithDefault[A](instance: Default[A]) {
  implicit def default: Default[A] = instance
}

case class Test(s: String)
object Test extends WithDefault(Default.instance(Test("")))

// Exiting paste mode, now interpreting.

object Test extends WithDefault(Default.instance(Test("")))
                                                 ^
<pastie>:16: error: illegal cyclic reference involving object Test

object Test extends WithDefault(Default.instance(Test("")))
                    ^
<pastie>:16: error: illegal inheritance;
 self-type Test.type does not conform to WithDefault[Test]'s selftype WithDefault[Test]

object Test extends WithDefault(Default.instance(Test("")))
                                                ^
<pastie>:16: error: no arguments allowed for nullary constructor Object: ()Object

I was wrong - we don't actually need a reference to the companion object: #219
I suspect when you say object Test extends WithDecoder[Test] WithDecoder finds the implicit Test.dec (i.e. itself) and creating a loop.

hi, @joroKr21, thanks a lot  for your answers. What I've found is that if I return just val in this place https://github.com/propensive/magnolia/blob/master/core/shared/src/main/scala/magnolia.scala#L190
(I've just replaced this)
if (!fullauto || shouldBeLazy) q"lazy val $name: $tpe = $rhs"
      else q"val $name = $rhs"

with
q"val $name = $rhs"

the example project will compile.
But it looks like you've already found the cause.

Hi, no I haven't found the reason. Although now that you mention it the condition looks suspicious.
Why is it lazy if !fullauto? My guess is that it should be lazy only when... well shouldBeLazy.


ATSAME54P20A support

ATSAME54 and ATSAME54P20A support.
Datasheet:
http://ww1.microchip.com/downloads/en/DeviceDoc/60001507E.pdf

I tried to glean from existing code any order of boards and features (sometimes 51 is before 21, so I would have put 54 above 51, vice versa)


ReadMe cleanup, created additional guides in docs/en

Created docs/en/Features.MD
Created docs/en/Use.md
ReadMe updates and formatting.

Nic - please review but don't merge just yet, I may make some more tweaks. But let me know your thoughts.
These will probably be the 3 pages for the Github pages site.


added regex functionality for allow lists in the analyzer

Change Description
Although the allow_list functionality is useful, it is not very practical as regex functionality is not handled here. I added this while still allowing the regular allow list to exist.
Issue reference
This PR fixes issue NONE
Checklist

[ x] I have reviewed the contribution guidelines
[ x] I have signed the CLA (if required)
[ x] My code includes unit tests
[ x] All unit tests and lint checks pass locally
[ x] My PR contains documentation updates / additions if required

@microsoft-github-policy-service agree

Thanks! How would you suggest to differentiate this from the original allow list capabilities? When should users use which one?

/azp run

The regex functionality would make the original allow lists obsolete. This regex allow list includes the functionality of the regular allow lists.
This regex makes the allow list more useful. For example if I want to allow a certain company to pass then i would like all of its variants to pass: company, COMPANY, Company, Company group, Company LLC. This is not possible with the current setup.
If we would include this regex_allow_list, then we could in essence delete the regular allow list.
If one would like to keep the regular setup they would have to change everything from "company" to "\bcompany$" with no flags.

Van: Omri Mendels @.>
Verstuurd: zondag, april 14, 2024 2:36:02 p.m.
Aan: microsoft/presidio @.>
CC: NarekAra @.>; Mention @.>
Onderwerp: Re: [microsoft/presidio] added regex functionality for allow lists in the analyzer (PR #1357)
Thanks! How would you suggest to differentiate this from the original allow list capabilities? When should users use which one?
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/presidio/pull/1357#issuecomment-2054035236, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZFSVQFWIX5ZR5WTCBAG6IDY5JZ27AVCNFSM6AAAAABGFY3CVSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJUGAZTKMRTGY.
You are receiving this because you were mentioned.Message ID: @.***>

Is it not possible because of regex flags? Or something else?

If you could give an example where you call the new regrx_allow_list is called, that'd be great.

Let's take an example:
I love travelling to New York City.
I love new york.
I love London!
The NER would flag New York City, new york and London as a location. Let's say that we would allow everything new york to be whitelisted and London should be flagged. In the current setup we would have to list all possible scenarios of new york being mentioned:
"New York", "new york", "New york", "NEW YORK CITY"... This is because currently the allow list are based on exact matches. This is not practical, my allow lists currently have 10+ versions of a certain word. Would be much better to whitelist based on regex. Then simply "New York" would cover all these cases.

From: Omri Mendels @.>
Sent: Monday, April 15, 2024 4:58:55 PM
To: microsoft/presidio @.>
Cc: NarekAra @.>; Mention @.>
Subject: Re: [microsoft/presidio] added regex functionality for allow lists in the analyzer (PR #1357)
If you could give an example where you call the new regrx_allow_list that'd be great.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/presidio/pull/1357#issuecomment-2057068343, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZFSVQAFWXGQ7NXI7Y346WDY5PTK7AVCNFSM6AAAAABGFY3CVSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXGA3DQMZUGM.
You are receiving this because you were mentioned.Message ID: @.***>

If we made the deny list case insensitive, would that support your use case?

A deny list applies to the pattern recognizer which is used to detect entities. In my use case I want to remove entities.
The goal is to have:
I love travelling to New York City.
I love new york.
I love !
London and new york are detected as LOCATION by Spacy NER, but only London will be masked as I will pass regex_allow_list = ["New york"] and regex_flags: re.IGNORECASE
Without the regex_allow_list I would have to pass ["New York City", "new york"] and many more variations for future use cases.

From: Omri Mendels @.>
Sent: Tuesday, April 16, 2024 6:49:35 AM
To: microsoft/presidio @.>
Cc: NarekAra @.>; Mention @.>
Subject: Re: [microsoft/presidio] added regex functionality for allow lists in the analyzer (PR #1357)
If we made the deny list case insensitive, would that support your use case?
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/presidio/pull/1357#issuecomment-2058222137, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZFSVQCBNHOVHW6TRRPK3KDY5SUV7AVCNFSM6AAAAABGFY3CVSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJYGIZDEMJTG4.
You are receiving this because you were mentioned.Message ID: @.***>

Sorry, I meant allow list :) if we make the existing allow list case insensitive, would that be ok? I would rather not add more parameters or capabilities as it's confusing to users, and I'd rather not remove parameters as it's not backward compatible

This would improve the allow lists a lot, however it will not include variants of a word that are sometimes unpredictable. New york can be written in at least 5 different ways, each with different possibilities for capitalization.
I understand that we cannot remove the allow list as is, as it would break applications. That's why I let the regular allow_list as is.
What we could do is include all the functionality in allow_list. We could add a boolean as an additional parameter that signifies if the allow list should be interpreted as regex strings or as exact matches, and an additional parameter to modify the flags. These two can be turned off by default, which means the functionality is the same as it is now.
I do believe this is a beneficial update, as per my usecase I do not use the allow list from Presidio at all, I mody the results of the analyzer myself with regex.

From: Omri Mendels @.>
Sent: Tuesday, April 16, 2024 10:40:06 AM
To: microsoft/presidio @.>
Cc: NarekAra @.>; Mention @.>
Subject: Re: [microsoft/presidio] added regex functionality for allow lists in the analyzer (PR #1357)
Sorry, I meant allow list :) if we make the existing allow list case insensitive, would that be ok? I would rather not add more parameters or capabilities as it's confusing to users, and I'd rather not remove parameters as it's not backward compatible
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/presidio/pull/1357#issuecomment-2058552494, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AZFSVQGGKWJJEAHUHQGR3X3Y5TPWNAVCNFSM6AAAAABGFY3CVSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJYGU2TENBZGQ.
You are receiving this because you were mentioned.Message ID: @.***>

Enhancing the existing capability with a parameters on how to process it sounds like a great improvement!

/azp run

Great, thanks for reviewing! Indeed, a big improvement :)


[Feature Request] - KSegmented added dark mode

Clear and concise description of the problem
No response
Suggested solution
No response
Describe alternatives you've considered
No response
Screenshots or Videos
No response

remove the class  k-icon--base__dark from the icon
or
modify the bg color of the KSegmented component


Added anim under UI > Animations section.

Added the animation library I'm working on to the "awesome list" 😎
Project URL
https://github.com/onurersel/anim
Description
From the README file:
anim is an animation library written in Swift with a simple, declarative API in mind.
It supports a bunch of easing functions and chaining multiple animations. It's a wrapper on Apple's UIViewPropertyAnimator on its core.
Why it should be included to awesome-ios (optional)
The library has an easy-to-use syntax with full control over animation durations and delays in-between. I think it is a proper addition to the list.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Error

:no_entry_sign:
Found 2 link issues

          1 Warning

:warning:
Too many changes (when adding, please keep it to one project per Pull Request)

Link issues by awesome_bot

Line
Status
Link

1040
301
https://tapglue.github.io/ redirects tohttp://www.tapglue.com/

2214
301
https://github.com/s4cha/AssetChecker redirects tohttps://github.com/freshOS/AssetChecker

  Generated by :no_entry_sign: Danger

Hey @onurersel, awesome lib 👍

Thank you @lfarah ! Cheers for adding it to the list 🎉


 This app uses the new features of apps SDK. You can also check that current Rinkeby apps still work as expected

Travis automatic deployment:
https://pr1594--safereact.review.gnosisdev.com/volta/app

@francovenica you can use this deployed app to test the new functionalities of SDK
https://ipfs.io/ipfs/QmfMq8NgxdDfai1qKv7bGbT8DegYX4mNzmpi9AkY1VJZBS

I added the app using the URL Dani provided and it worked, but I had to refresh the page since adding it just show the loading icon forever (waited 20 mins till I refreshed manually to actually see the app)
It seems the app create just test tx that do nothing of real effect, so I don't think the app itself requires any testing or validations, still I played with the amount of gas for the tx, and even putting a low number like "1000" and not getting the "Gas estimation is OK" sign the tx went through and it was mined just fine.
A tx in volta: https://volta-explorer.energyweb.org/tx/0x5c00c154b84664c0687f6a94603517b7d4db2c7e21662e681d4590134b0d4325/internal-transactions
Volta safe used: 0x538622a690A651D852277c07334dC3377E22cdeF
Tx in rinkeby: https://rinkeby.etherscan.io/tx/0xe4f86cfaabacedc0096c492d6671b68341008c72e3e424dbafa4d7d7b8b03fa1
rinkeby safe use: 0x9913B9180C20C6b0F21B6480c84422F6ebc4B808
I tried in Rinkeby other apps: Compound, idle, sablier. With WC I tested just the connection with the "Wallet connect example" page
It looks good for me

Travis automatic deployment:
https://pr1594--safereact.review.gnosisdev.com/rinkeby/app

Travis automatic deployment:
https://pr1594--safereact.review.gnosisdev.com/volta/app


Widgets not rebuilt when using Consumer

I'm very new to Riverpod so please be gentle (I did read the docs). I have a StateNotifier<GlobalState> which contains 'global' app state, like ThemeMode. I can't make it work to change the application theme using riverpod.
Here is the full code (interactions with Riverpod marked with  // <================):
import 'package:flutter/material.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';

@immutable
class GlobalState {
  final ThemeMode themeMode;

  GlobalState({@required this.themeMode});

  GlobalState copyWith({
    ThemeMode newThemeMode,
  }) {
    return GlobalState(
      themeMode: newThemeMode ?? themeMode,
    );
  }
}

class GlobalStateNotifier extends StateNotifier<GlobalState> {
  GlobalStateNotifier(GlobalState state) : super(state);

  @override
  GlobalState get state => super.state;

  void changeThemeMode(ThemeMode themeMode) {
    state = state.copyWith(
      newThemeMode: themeMode,
    );
  }
}

final globalStateProvider = StateNotifierProvider(
  (ref) => GlobalStateNotifier(GlobalState(themeMode: ThemeMode.system)),
);

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return ProviderScope(
      child: Consumer(
        builder: (context, watch, child) {
          return MaterialApp(
            theme: ThemeData(
              primarySwatch: Colors.blue,
              visualDensity: VisualDensity.adaptivePlatformDensity,
            ),
            darkTheme: ThemeData.dark(),
            themeMode: watch(globalStateProvider).state.themeMode,
            home: MyHome(),
          );
        },
      ),
    );
  }
}

class MyHome extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Test'),
      ),
      drawer: MyDrawer(),
      body: Center(
        child: Text('Test'),
      ),
    );
  }
}

class MyDrawer extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Drawer(
      child: ListView(
        children: [
          ThemeMenuButton(),
        ],
      ),
    );
  }
}

class ThemeMenuButton extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Consumer(
      builder: (context, watch, child) {
        final themeToLabel = {
          ThemeMode.system: 'System',
          ThemeMode.light: 'Light',
          ThemeMode.dark: 'Dark',
        };

        final themeMode =
            watch(globalStateProvider).state.themeMode; // <================

        return PopupMenuButton<ThemeMode>(
          initialValue: themeMode,
          onSelected: (newThemeMode) {
            Navigator.pop(context);
            if (themeMode != newThemeMode) {
              context
                  .read(globalStateProvider)
                  .changeThemeMode(newThemeMode); // <================
            }
          },
          itemBuilder: (context) => themeToLabel.entries.map((entry) {
            return PopupMenuItem<ThemeMode>(
              value: entry.key,
              child: Text(entry.value),
            );
          }).toList(),
          child: ListTile(
            leading: Icon(Icons.bedtime),
            title: Text('Theme (${themeToLabel[themeMode]})'),
          ),
        );
      },
    );
  }
}

When I debug, I can actually see that when I choose a different ThemeMode, the state in StateNotifier is changed to the new instance with the new mode, and the value in the menu in the drawer also changes to the mode I selected; however, the Consumer.builder callback is never called again and the application doesn't react to the change. When I trigger a hot-reload, the theme does change, which means the state is correct, but the build is not
What am I missing?

The problem is that you are watching
final themeMode =
            watch(globalStateProvider).state.themeMode;

when you should be watching:
final themeMode =
            watch(globalStateProvider.state).themeMode;

There is a subtle difference when using a StateNotifier since the thing being provided to the tree is the StateNotifier subclass and not the state of the StateNotifier. To subscribe to changes to the state you have to specifically indicate that in the call to watch.

Right, thank you, this does work indeed.
This is very surprising indeed, I hope you guys can come up with a solution for it.


Right click on an instance / object in the Scene tree and "Copy path" to then paste in script editor

Operating system or device - Godot version:
Any OS
Issue description (what happened, and what was expected):
I'm regularly finding reference to instances using get_node("").  I would love to be able to right click on an instance in my scene tree and "copy path" so I can paste it into the script editor without having to work it out in my head.  It would speed up the workflow and reduce errors.

ah, I made it so it's now possible to drag it, but forgot that, will add it
the next days
On Fri, Aug 19, 2016 at 9:22 PM, therealrobster<EMAIL_ADDRESS>wrote:

Operating system or device - Godot version:
Any OS
Issue description (what happened, and what was expected):
I'm regularly finding reference to instances using get_node(""). I would
love to be able to right click on an instance in my scene tree and "copy
path" so I can paste it into the script editor without having to work it
out in my head. It would speed up the workflow and reduce errors.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/6211, or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z24LXFUp5b5nYVcLutDXGEHYOBEXuks5qhkivgaJpZM4Jo8z_
.

Dragging is even better.  Great idea.  Thank you
On 20 August 2016 at 10:48, Juan Linietsky<EMAIL_ADDRESS>wrote:

ah, I made it so it's now possible to drag it, but forgot that, will add it
the next days
On Fri, Aug 19, 2016 at 9:22 PM, therealrobster<EMAIL_ADDRESS>wrote:

Operating system or device - Godot version:
Any OS
Issue description (what happened, and what was expected):
I'm regularly finding reference to instances using get_node(""). I would
love to be able to right click on an instance in my scene tree and "copy
path" so I can paste it into the script editor without having to work it
out in my head. It would speed up the workflow and reduce errors.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/6211, or mute the thread
<https://github.com/notifications/unsubscribe-auth/AF-
Z24LXFUp5b5nYVcLutDXGEHYOBEXuks5qhkivgaJpZM4Jo8z_>
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/6211#issuecomment-241167389,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOxL4P73kMPLXZrEvuAtkFILJU8YSef0ks5qhk7fgaJpZM4Jo8z_
.

I think this was implemented in the current master branch, right?

This feature has been implemented in Godot 3.0, so I'm closing this issue. You can also drag-and-drop the node onto the script editor.


finatra-* readme-blog-link

Problem
The link for finatra's blog in the Readme file is broken
Solution
Update the blog link to point to finagle's blog (https://finagle.github.io/blog/)
Relevant Issues
#569 bad link in readme.md 404

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Your patch was merged in here, thank you!


Assertion during C-string hashing.

I get an assertion when using a C-string oplist in a dictionary. This is caused by m_core_hash, which is used by default. The check for alignment fails to be explicitly: assert ( (( (uintptr_t)p & (sizeof(uint64_t)-1) ) == 0) || (length <= sizeof(uint32_t)));
The C-string's address is 0x00007ff79367a50c which is not 64-bit aligned really. The string in question is not allocated dynamically, it's a literal value.

Seems like the C-string literals are not obliged to be aligned at all.

And it seems like you're aware of this:

This renders using a dictionary with string literals unusable, which is kinda wide usage actually

I have added a Dan Bernstein's hashing to my branch:

Indeed.
I have fixed this in master, pretty much like you propose but I am using the internal HASH infrastructure to let the user select the hash function.
Thank you for your report (and proposal).

That's cool you have those additional flexibility with those hashing macro terms. I've now accomodated your method and also've added XOR version of DJB as an addition to existing presets.

Also, I see you've removed the M_CSTR_HASH function which may be a good idea considering the new function is specific, but I would personally advocate for still naming it m_core_cstr_hash instead of m_core_str_hash since cstr is more explicit and clear to work with C-strings (just like the former macro is).

Done


undebuggable bug in windows.ui.xaml when using cas

Hi
I am using the connectedanimationservice to animate an ellipse from one page to the other. This works fine 9/10 times but sometimes I get a crash to which the debugger can’t seem to attach.
I am on build 14393 with the latest sdk. Here is the error event info:
Faulting application name: myappp.exe, version: <IP_ADDRESS>, time stamp: 0x56c69c5d
Faulting module name: Windows.UI.Xaml.dll, version: 10.0.14393.0, time stamp: 0x57899a84
Exception code: 0xc000027b
Fault offset: 0x00000000004d04ff
Faulting process id: 0x1aec
Faulting application start time: 0x01d1e2ef0aa8bdd8
Faulting application path: C:\Users\dev\Desktop\myyapp\bin\x64\Debug\AppX\myapp.exe
Faulting module path: C:\Windows\System32\Windows.UI.Xaml.dll
Report Id: 7af3379c-5d91-4740-bc76-21b3f40b2083
Faulting package full name: myapp.myapp_<IP_ADDRESS>_x64__4zg0btev1pvta
Faulting package-relative application ID: App
This is probably not a lot to go on. Here is the code I use:
            try
            {
                ConnectedAnimationService cas = ConnectedAnimationService.GetForCurrentView();

                cas.PrepareToAnimate("item", imageellipse);
            }
            catch { }

            ((Frame)ParentPage.Parent).Navigate(typeof(Detail), item);

And on  the receiving page:
protected override async void OnNavigatedTo(NavigationEventArgs e)
{
        try
        {
            ConnectedAnimationService cas = ConnectedAnimationService.GetForCurrentView();

        cas.DefaultDuration = TimeSpan.FromSeconds(1);

            cas.GetAnimation("item").TryStart(detailellipse);
        }
        catch { }

    }

Could you attach a project that demonstrates the issue?
Also, is the crash in PrepareToAnimate or GetAnimation?

Hi Jaster,
here is some info from hockeyapp:
Date/Time:       2016-07-21T03:43:22.963Z
OS Version:      Windows 10.0.14393.0
Report Version:  104
Exception Type:  System.Runtime.InteropServices.COMException
Crashed Thread:  3
Application Specific Information:
An outgoing call cannot be made since the application is dispatching an input-synchronous call.
An outgoing call cannot be made since the application is dispatching an input-synchronous call.
Exception Stack:
Windows.ApplicationModel.Core.UnhandledError.Propagate()
I currently unfortunately don't have time to create a project that I can post up. I hope this info helps.

I'm not able to repro this without more information, sorry. 😞
Closing this issue.


Try to create a new account, command doesn't work

cleos --url http://node2.liquideos.com:8888 system newaccount --stake-net '0.1 EOS' --stake-cpu '0.1 EOS' --buy-ram-kbytes 4 asdfghj12345 asdfghj55555 EOS7Divrf81apTrzXTxwocYCM8LvhLDqfpsF36pgTQtbpAKZV6hnZ EOS7Divrf81apTrzXTxwocYCM8LvhLDqfpsF36pgTQtbpAKZV6hnZ

ERROR: ExtrasError: [EOS7Divrf81apTrzXTxwocYCM8LvhLDqfpsF36pgTQtbpAKZV6hnZ EOS7Divrf81apTrzXTxwocYCM8LvhLDqfpsF36pgTQtbpAKZV6hnZ]

I have created and tried several keys, doesn't solve the problem. Is something wrong in the command I typed in?

Look at the issue #4565

The issue linked by damianoazzolini contains the solution to your problem.  To stake 0.1 EOS you must specify "0.1000 EOS".


cant click on cards

to go to a particular language section user have to click on "click here" button , want to change it so that user can click on that card to go to that language page

Go ahead


Support Kahu build system

Issue/Feature Description:
Feature handle kahu build system

[x] Basic static code checker (go fmt, go vet, goimports)
[x] Build binaries
[x] Build images

Why this issue to fixed / feature is needed(give scenarios or use cases):
Improve kahu build experience
How to reproduce, in case of a bug:
NA
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)
NA

The issue is handled in PR #26


Use co blocks for pause and resume operations

I'm likely going to try to centralize how we handle things like 409, but until I do that, it'll stay here for now.

:+1:


Refs #23359 -- Corrected showmigrations help text for the --database option.

Just happened to notice this: python manage.py showmigrations isn't really synchronizing anything. Or at least I'd understand "synchronize" to mean "apply migrations" (in the sense of the old python manage.py syncdb) so felt like it might be good to adjust this wording.
Had a look at some of the other descriptions and they all tend to specifically call out what the command does, so I stuck to that convention:

Thanks @felixxm ! 🎉


[#130728055] Document Route Services

WIP: I am adding a example with authentication

#130728055 Document Route Services
What?
Extend our documentation to tell users they can use the Cloud Foundry Route Services feature, explain how they can set it up and explain the risks/impact using it might have.
How to review?
Have a read
Who?
Anyone but @keymon

Anyone in the team can review this.
Should the WIP title be removed?

Removed. Maybe @benh-gds wants to review this?

I've tested the process and made the changes myself. Someone else can merge now.

@jollery I've made the changes requested

I've put the "[external link]" back


[Bug]: consumption/budget - add parameter for thresholdType

Description
The module consumption/budget have the properties thresholdType hardcoded to 'Actual' and can't be used for forecast notifications.

@dibcar , thanks for bringing this up. I just converted the issue into a bug


integrate codecov for UT check

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage

What this PR does / why we need it:
Which issue(s) this PR fixes:

resolve #8
Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

/lgtm
/approve


Added Traditional Chinese (zh_tw.json)

Added Traditional Chinese localization

Oh I'm sorry, I just realized, but I had the default branch set for 1.19 which this PR is targeting, but that branch is a few versions behind, so it has some old / changed translation keys

Oh I'm sorry, I just realized, but I had the default branch set for 1.19 which this PR is targeting, but that branch is a few versions behind, so it has some old / changed translation keys

Oh okay i will check the new one


Fix critical error on registration with confirmation mode

In case confirmation is activated:

When more than 1 user registered, the 'uid' column will be left empty until confirmation complete. That will lead to duplication of 'uid-provider' index as uid are both nulled but providers are the same 'email'

Can you add a test confirming the bug?

The problem appeared again in version 1.1.4 with rails 6.1


Fix #9 Traceback info fault

Hi, I've fixed typo. #9
Also I'll be happy, if that fix will be at pip asap.

Sorry, for the confusion but this got fixed in the CI branch a while ago. I merged it now into master.
https://github.com/Croydon/flask-xml-rpc-reloaded/pull/4
Hopefully I will have time at some point to set up the CI finally....


Solutions with spaces in filename fail to restore packages

🐛 Bug Report
If a solution has a space in the filename packages will fail to restore, due to the filename not being quoted when calling nuget
I assume the line in question is here, and just needs escaped quotes adding around {fileNameCommandLine}
https://github.com/NuKeeperDotNet/NuKeeper/blob/868a7073e91b0fd1b50ec4f4cadd0ec4944e7adc/NuKeeper.Update/Process/NuGetFileRestoreCommand.cs#L59
Expected behavior
Solution restores correctly
Reproduction steps
Make a solution that has a space in the .sln file name. Attempt to run nukeeper
Configuration
Version: 0.26.1
Platform if applicable:

[X] :hammer_and_wrench: NuKeeper CLI
[ ] :sparkles: GitHub
[ ] :robot: AzureDevops
[ ] :checkered_flag: Bitbucket
[ ] :earth_americas: Gitlab
[ ] :tv: Gitea
[ ] :whale: Docker

Not stale

Not stale


Update README.md

Updated link from https://console.aws.amazon.com/iam/home/ to https://console.aws.amazon.com/iam/home. Older one was pointing to 404.

Pull accepted. Thank you.


[resotocore] Fix preliminary hangup of workers

Codecov Report

Merging #577 (b08a9f5) into main (461491b) will increase coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #577      +/-   ##
==========================================
+ Coverage   88.44%   88.49%   +0.04%
==========================================
  Files          64       64
  Lines        8315     8315
==========================================
+ Hits         7354     7358       +4
+ Misses        961      957       -4

Flag
Coverage Δ

resotocore
88.49% <0.00%> (+0.04%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

resotocore/core/web/api.py
63.31% <0.00%> (ø)

resotocore/core/web/content_renderer.py
93.05% <0.00%> (+2.77%)
:arrow_up:

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 461491b...b08a9f5. Read the comment docs.


InitContainers dont work by default when moving from unversioned client to client-go

I was Migrating an app using Init Containers from unversioned client to client-go
All i needed to do in the unversioned client, was populate api.Container into InitContainers field of the PodSpec.
the migrated app stopped working in client-go.
i had to explictily enable serialization to annotations to make the init containers work in client-go.

I think that is a bad user experience. By default Init Containers or any other alpha feature should work, without requiring any special serialization.
I had to call the following function to make this work https://github.com/kubernetes/kubernetes/blob/master/pkg/api/v1/pod/util.go#L91
which is not part of the client-go code. So i had to duplicate that in my app.

@smarterclayton we had dealt with this when we migrated kubernetes to use versioned client. Is initcontainer going to be stable soon? If so, i don't want to import those conversion functions to client-go.

Looks like initContainers will be available as a field in 1.6 https://github.com/kubernetes/kubernetes/pull/38382
Which "unversioned" version where you using? I don't think those fields were added until 1.6.


Master -> Live

Docs Build status updates of commit 724d40a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/mixedrealityspatialdatapackager.md
:warning:Warning
View
Details

mixed-reality-docs/hologram-stability.md
:white_check_mark:Succeeded
View

mixed-reality-docs/mixedrealityspatialdatapackager.md

Line 4, Column 9: [Warning-author-not-found] Invalid value for author: 'alfred-msft' is not a valid GitHub ID.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit e19dde2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/hologram-stability.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 1e0805a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/hologram-stability.md
:white_check_mark:Succeeded
View

mixed-reality-docs/porting-guides.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit c3e39f1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/hologram-stability.md
:white_check_mark:Succeeded
View

mixed-reality-docs/porting-guides.md
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-deploying.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit ccfb2c8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/hologram-stability.md
:white_check_mark:Succeeded
View

mixed-reality-docs/porting-guides.md
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-deploying.md
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-winRT.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


⚠ Avatar API Endpoint has degraded performance

In c84735e, Avatar API Endpoint (https://avatar.roblox.com/v1/avatar-rules) experienced degraded performance:

HTTP code: 200
Response time: 1820 ms

Resolved: Avatar API Endpoint performance has improved in 2d50cbc.


Add mesh and operators for spherical domain

Summary

Add (dimensionless?) spherical domain to mesh.py, independently of the existing Cartesian domain
Add grad and div operators (Finite Volumes) for the spherical domain

Mesh shall be unchanged. Will use if domain is "negative particle" or "positive particle" to determine when spherical operators should be used.

Closed by #142


Jest run fails with TypeScript

Hi all, I've encountered the following error when running jest with Typescript. Running jest with --no-cache flag results in the same error. I'm running jest with global jest-cli.
 TypeError: Cannot read property 'getMockFunction' of undefined

      at Runtime._createRuntimeFor (node_modules/jest-cli/node_modules/jest-runtime/build/index.js:753:25)
      at process._tickCallback (node.js:369:9)

I've tried preprocessor from jest-examples but with the same result.
Jest settings in package.json file:
  "jest": {
    "moduleFileExtensions": [
       "js",
      "tsx",
      "ts",
      "json"
    ],
    "scriptPreprocessor": "<rootDir>/node_modules/ts-jest/preprocessor.js",
    "moduleDirectories": [
      "node_modules"
    ],
    "moduleNameMapper": {
      "^.+\\.(jpg|jpeg|png|gif|eot|otf|webp|svg|ttf|woff|woff2|mp4|webm|wav|mp3|m4a|aac|oga)$": "<rootDir>/config/jest/FileStub.js",
      "^.+\\.css$": "<rootDir>/config/jest/CSSStub.js"
    },
    "setupFiles": [
      "<rootDir>/config/polyfills.js"
    ],
    "testPathIgnorePatterns": [
      "<rootDir>/(build|.storybook|node_modules)/"
    ],
    "testEnvironment": "node",
    "testRegex": "(/__tests__/.*|\\.spec)\\.(js|jsx|ts|tsx)$",
    "globals": {
      "__TS_CONFIG__": {
        "module": "commonjs",
        "jsx": "react"
      }
    }
  }

Appreciate any advice. Thank you.

Can you provide a repo that we can npm install and npm test to take a look?

Thanks @cpojer. I appreciate your help with this and maybe some direction on how to debug such issues.
Heres is the link: https://github.com/pavloko/react-typescript

Update. I downgraded Jest to 15.1.1 and everything works. I hope this helps. Thanks again.

@pavloko Your repro on Jest v15 works fine. You can easily migrate to Jest v17 by replacing scriptPreprocessor with:
"transform": {
  "^.+\\.(ts|tsx)$": "<rootDir>/config/jest/preprocessor.js"
},


CIS (Custom item sounds)

You could create sound ids. For example, in the properties file entity.player.attack.sweep=citresewn.sweep. Like if you had a music disc and you wanted to change the sound, and then you use a sounds.json file to create the soundpath.

Sadly I cannot do music discs because the exact item in the jukeboxes is not synced to clients.
However maybe some item related sound events could be a neat addition.
Added to the idea board(in Projects)


[Question] 多标签预测问题

You must follow the issue template and provide as much information as possible. otherwise, this issue will be closed.
请按照 issue 模板要求填写信息。如果没有按照 issue 模板填写，将会忽略并关闭这个 issue
Check List
Thanks for considering to open an issue. Before you submit your issue, please confirm these boxes are checked.
You can post pictures, but if specific text or code is required to reproduce the issue, please provide the text in a plain text format for easy copy/paste.

[ ] I have searched in existing issues but did not find the same one.
[ ] I have read the documents

Environment

Linux16.04:
Python 3.8:
Kashgari 2.0:

Question
1、请问训练的时候，准确率达到90%，测试的时候为什么结果很差，即使预测训练集的数据，也不是很好，例如：
train：X = [[“你“, ”名字“, ”叫什么”], [“你“, ”名字“, ”叫什么”, “今年”, “几岁”]]
Y = [["name"], ["name", "age"]]
test:   X = [[“你“, ”名字“, ”叫什么”]]
Y = [("name", 1.0),  ("age", 0.98)……]
2、预测时，短文本小于3个词会报错
ValueError: Negative dimension size caused by subtracting 5 from 4 for '{{node model_1/conv1d/conv1d}} = Conv2D[T=DT_FLOAT, data_format="NHWC", dilations=[1, 1, 1, 1], explicit_paddings=[], padding="VALID", strides=[1, 1, 1, 1], use_cudnn_on_gpu=true](model_1/conv1d/conv1d/ExpandDims, model_1/conv1d/conv1d/ExpandDims_1)' with input shapes: [?,1,4,100], [1,5,100,128].
请问这是怎么回事？

问题1 貌似是多标签分类不能callbacks的原因。


Globalize-runtime cannot have two unit formatters with same options but different number formatters

Globalize version: 1.2.3
Globalize compiler version: v0.2.4
Example code (might not compile, but hopefully illustrates the point):
// compilation time - note that the only difference in formatters is the number formatter options
const unitFormatOptions1 = { numberFormatter: globalizeInstance.numberFormatter({minimumIntegerDigits: 1}) };
const unitFormatOptions2 = { numberFormatter: globalizeInstance.numberFormatter({minimumIntegerDigits: 2}) };
compileFormatters([
  globalizeInstance.unitFormatter('hour', unitFormatOptions1),
  globalizeInstance.unitFormatter('hour', unitFormatOptions2)
]);

// ... now to the runtime
const x = globalizeInstance.formatUnit(3, 'hour', unitFormatOptions1);
const y = globalizeInstance.formatUnit(3, 'hour', unitFormatOptions2);

Expected: x == "3" and y== "03"
Actual: x == "3" and y == "3"
The problem is that the unit formatter takes a number formatter (a function) as one of its options, but the runtime keys are generated by JSON.stringifying the options (see here), and JSON.stringify stringifies functions as empty string.
So, the runtime key is resolved to the same thing in both formatters, even though they have different number formatters.

#705 is related and may have the same solution


Add a license

Thanks for this project, it looks very useful!
It would be great if you could include a license so we can use/modify it (https://choosealicense.com/no-permission/)

Thanks for the heads up about the license - been meaning to sort one, but it's my first project and just never got round to it - went with MIT as it seemed the easiest.
If you do modify / improve it - do let me know so I can add them to the repo 👍


[Feature]: Umbrella bug for Device Request Prompt / Web Bluetooth API

Feature description
BiDi spec: https://github.com/w3c/webdriver-bidi/pull/523
Chromium mapper issue: https://github.com/GoogleChromeLabs/chromium-bidi/issues/1455
Web Bluetooth spec: https://github.com/WebBluetoothCG/web-bluetooth/issues/613

Merged so far:

https://github.com/puppeteer/puppeteer/pull/11157
https://github.com/puppeteer/puppeteer/pull/11152
https://github.com/puppeteer/puppeteer/pull/11153
https://github.com/puppeteer/puppeteer/pull/11166
https://github.com/puppeteer/puppeteer/pull/11159

cc @nondebug FYI


[Microsoft.Web/sites/config] AppSettings not working correctly

Bicep version
Bicep CLI version 0.3.1 (d0f5c9b164)
Describe the bug
Unable to set app settings with Microsoft.Web/sites/config
No errors on the deployment run through CI/CD pipeline but the app settings section remains empty on the portal
resource portalConfig 'Microsoft.Web/sites/config@2020-06-01' = {
  name: '${portal.name}/web'
  location: location
  properties: {
    appSettings: [
      {
        name: 'SettingA'
        value: 'SettingAValue'
      }
    ]
  }
}

To Reproduce
Normal web app deploy bicep

server plan Microsoft.Web/serverfarms
web app Microsoft.Web/sites
config Microsoft.Web/sites/config

Deployments all green, check portal, web app application settings is empty, no app settings created
Additional context

Able to set with  Microsoft.Web/sites properties: siteConfig: appSettings
Able to set with Microsoft.Web/sites/config/appsettings
Not able to set with Microsoft.Web/sites/config/web

Anybody able to replicate the same or am I just doing something wrong?
Or does Microsoft.Web/sites/config not actually have appSettings like the arm template docs suggest and the issue with the VS code extension not having the types just masking the issue?

I'm not sure why it's not working. Your template looks correct. @bmoore-msft may know, otherwise we can connect you with someone from the web team.
As an aside, this was previously a gap in our type system that we have since fixed in the main branch, but not yet released. You can try the experience now with the nightly build if you'd like. If you use the 'parent' property or nest the config resource inside the sites resource and we will provide better validation:
resource foosite 'Microsoft.Web/sites@2020-10-01' = {
  name: 'foo'
  location: 'bar'

  resource fooconfig 'config' = {
    name: 'web'
    properties: {
      appSettings: [
        {
          name: ''
          value: ''
        }
      ]
    }
  }
}

or
resource foosite 'Microsoft.Web/sites@2020-10-01' = {
  name: 'foo'
  location: 'bar'
}

resource fooconfig 'Microsoft.Web/sites/config@2020-10-01' = {
  parent: foosite

  name: 'web'
  properties: {
    appSettings: [
      {
        name: ''
        value: ''
      }
    ]
  }
}

Kind of confused now cause my solution of using Microsoft.Web/sites/config/appsettings doesn't work on a new deployment now
not really sure if its a template or more of a deploy issue, the deployment says the step was 'green' but don't really see any more details
Any tips on how to see more details on if its updating the app settings or not during deploy etc?

@alex-frankel tried the parenting as you suggested and that seems to work better, cheers

@chull434 - the config resources for web are kinda funky...  the resource "type" is always "config" and then the name property is the type of config.  All of them when authored as a resource, only have a "properties" body afaiki...  So in your original post, just remove the appSettings property:
resource portalConfig 'Microsoft.Web/sites/config@2020-06-01' = {
  name: '${portal.name}/web'
  location: location
  properties: {
    {
        name: 'SettingA'
        value: 'SettingAValue'
    }
  }
}

and some other options...  (appSettings example)
resource portalAppSettings 'Microsoft.Web/sites/config@2020-06-01' = {
  name: '${portal.name}/appSettings'
  location: location
  properties: {
    {
        name: 'AppSettingA'
        value: 'AppSettingAValue'
    }
  }
}

or on the parent (perhaps simplest):
resource portalConfig 'Microsoft.Web/sites@2020-06-01' = {
  name: '${portal.name}'
  location: location
  properties: {
    siteConfig: {
      appSettings: [ ... ]
      connectionStrings: [...]
      randomWebConfigSetting: 'randomWebConfigValue'
  }
}

@chull434 - going to close this since it looks like you got it resolved


persistent devices.js

Hi

zigbee2mqtt add-on version (if edge, please report commit hash):
edge: zigbee2mqtt version 1.6.0 (commit #14a9ec9f11e12ab43401b8c4945158965e0167e0)
Operating environment (HassOS, Virtual Machine, Device/platform running Hass.io):

HassOS 2.8 / RPi 3B+
Description of problem:
I'm trying to add a new device (IKEA bulb LED1836G9) and I'm using the edge version of the add-on, but I can't find a way to copy my custom "devices.js" from "/share/zigbee2mqtt" inside the docker container of zigbee2mqtt-edge. Is the config option "zigbee_shepherd_devices" implemented in the edge version, because it works under the 1.6 master branch, but under edge I get this following warning in the HA logs. (please see bellow)
Your entire configuration from the frontend (with sensitive fields redacted):
// replace this line with your pasted configuration

Your logs from Hass.io
19-09-23 12:03:36 WARNING (MainThread) [hassio.addons.validate] Unknown options zigbee_shepherd_devices

it's irrelevant now, the device got into zigbee2mqtt database


data zoom (in toolbox) stops working after changing axis type

General Questions

Issue Type

[ ] I have a question to ask about how to use ECharts to ...；我想提问如何使用 ECharts 实现某功能
[x] I have a bug to report；我想要报 bug
[ ] I have a feature to request, e.g.: I'd like a new feature that ...；我需要一个新功能
[ ] I have a feature to enhance, e.g.: The current feature should be improved in the way that ...；我希望改进某个功能
[ ] There's something wrong with the documents；文档有些问题
[ ] Others, or I'm not sure which issue types to choose here；其他，或我不知道应该选什么类型

Issue Details

Zooming stops working after changing axis type from linear to log scale (although it doesn't seem to matter from which type to which type)

Expected Behavior

The zooming feature still works
Current Behavior

The zooming feature doesn't work. Here is the error:

Cannot read property 'getMinMaxSpan' of undefined

Online Example

See a demo here: https://codesandbox.io/s/4x0xx62no9
The zoom feature works at first, but after changing it to log scale, it no longer works. If you change it back to linear scale, the zoom feature doesn't work there as well.

Topics

Anything Else We Need to Know

Environment

ECharts version；ECharts 版本: {DESCRIBE_HERE}
4.2.0-rc.2

[ ] It happens only on certain browsers or operating systems. 对于特定浏览器或操作系统才会出现的问题，请提供相应环境信息：{BROWSER_VERSION_OR_OS_INFORMATION_HERE}

@gnijuohz 请问这个问题有进展了吗


Failed to locate: "CL.exe".

Hi,
Can somebody help me with this error? What i must install?

Can you even try to search online for help? This is not the place to ask compilation help


Chat bot auto train english?

Hi, I'm newbie in this field also with chatterbot, I want to ask about the trainer, I'm implementing a chat bot in Vietnamese which is not supported currently, so I've to create the data by my own. It took me a while but generally I've done it. But when I choose trainers to train with this dataset, the return value still english and when I export the chat bot it's still english. Can someone help me?

Are you using chatterbot corpus for training your bot?
Because that doesn't support your language as far as I see: https://github.com/gunthercox/chatterbot-corpus/tree/master/chatterbot_corpus/data

Are you using chatterbot corpus for training your bot?
Because that doesn't support your language as far as I see: https://github.com/gunthercox/chatterbot-corpus/tree/master/chatterbot_corpus/data
yes I'm using chatterbot corpus, as my knowledge about chatterbot, it's using spacy for NLP, so I've installed a pipeline for my language and develop the dataset for my language, add it to the link you posted. Generally I've done it but even though I didn't assign my bot to train english, it's still automatically train it (because when I export the bot data, it's still have English data). i don't know why.

yes I'm using chatterbot corpus, as my knowledge about chatterbot, it's using spacy for NLP, so I've installed a pipeline for my language and develop the dataset for my language, add it to the link you posted. Generally I've done it but even though I didn't assign my bot to train english, it's still automatically train it (because when I export the bot data, it's still have English data). i don't know why.

What do you mean by "add it to the link you posted"? opened a PR?
and by the way, are you using the same database for both English trained bot and the bot trained in your language?
I mean training data are saved in database as far as I know.


Image Optimization

[X]  Add WP Smush Pro (wpmudev plugin)
[ ] Create appropriate thumbnail sizes for homepage

Visual regression test failed!

Lighthouse test passed! The score of 64 isn't less than the acceptable score of 28 (5 less than the score of 33 on the master branch)
View the full Lighthouse performance report for f/image-optimization and compare it to the Lighthouse performance report for the master branch


chore: enqueue merge job on comment

Signed-off-by: Aylei<EMAIL_ADDRESS>What type of PR is this?

[ ] API-change
[ ] BUG
[ ] Improvement
[ ] Documentation
[ ] Feature
[x] Test and CI
[ ] Code Refactoring

Which issue(s) this PR fixes:
This PR refine the mergify rule:

only 1 approve is required, same as current branch protection rule
to avoid unwanted merge, there should be an explicit /merge chatops comment to mark the PR as can-merge

What this PR does / why we need it:
Not Available
Special notes for your reviewer:
Not Available
Additional documentation (e.g. design docs, usage docs, etc.):
Not Available

/merge


Compatibility 233

Description
This pr implements compatibility for 2023.3 (rc and over) and fix deprecated method usage warning.

Fixes #163
Compatibility for ide 2023.3.

Thank you for the fix. When do you think you will have the new release?

@komapa this may help:
https://github.com/open-policy-agent/opa-idea-plugin/issues/165#issuecomment-1928199667


KOGITO-4941: [DMN Editor] Ctrl-B always converts field to structure a…

…nd nests
For more details see:

https://issues.redhat.com/browse/KOGITO-4941

This is a cherry-pick of https://github.com/kiegroup/kie-wb-common/pull/3658


Update fs2-kafka to 1.4.1

Updates com.github.fd4s:fs2-kafka from 1.0.0 to 1.4.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Migrations

https://raw.githubusercontent.com/fd4s/fs2-kafka/be2b888a1cf04de2d6416fda9d48bca37a8b056e/scalafix/rules/src/main/scala/fix/Fs2Kafka.scala

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.fd4s", artifactId = "fs2-kafka" } ]

labels: library-update, semver-minor, scalafix-migrations

Superseded by #107.


Is there any hooking system?

I want to bind some tasks before and after database connection and before/after query execution as well. Is it possible?

You can create a class extends from Medoo.
class MyMedoo extends Medoo
{
    function execute($statement, $map)
    {
        // beforeExec();
        parent::exec($statement, $map);
        // afterExec();
    }
}


Upgrade to xmldom 0.8.4

Hi,
apparently Dependabot-Users are getting alerted when using msw or msw/interceptors due to the latter using an outdated version of xmldom
Would it be possible for you guys to quickly upgrade?
see:
https://github.com/jindw/xmldom/issues/150
https://nvd.nist.gov/vuln/detail/CVE-2022-39299

I see somebody already created a PR for this:
https://github.com/mswjs/interceptors/pull/295

Fixed by #295. Thanks everyone for working on this.


intl dependency upgrade

charts_common and charts_flutter currently restrict their intl dependency to a version < 0.18.0.
This is incompatible with the latest flutter (master branch). Would it possible to lift/loosen the constraint?

same problem here

You can override the dependency.
`dependency_overrides:
forall packages using intl
intl: ^0.18.1`


Split worker from processing service

At the moment the processing service does all the work of executing jobs. This limits how much work and what kind of work can be done.
By moving the job processing into a separate service we can scale the job execution (which is potentially slow) independently from the processing of signals (which should be fast).

This is fixed in 0.2.0 by moving all the processing / executing in to the 'server' section and only processing one item per server invocation.


There is an error with your site.

Application error
An error occurred in the application and your page could not be served. If you are the application owner, check your logs for details. You can do this from the Heroku CLI with the command
heroku logs --tail

Heroku turned off the free dyno server. Thanks.


UvDriver cannot scan empty dir

For this code:
<?php
require __DIR__ . '/vendor/autoload.php';

use Amp\File;
use Amp\Loop;
use App\App;

Loop::run(function () {
    $flist = yield File\scandir("/tmp/tmp/");
    var_dump($flist);
});

/tmp/tmp is empty dir

Run php72 -dmemory_limit=1024m /path/to/php.php, it throw a excepiton:
PHP Fatal error:  Uncaught Amp\File\FilesystemException: Failed reading contents from /tmp/tmp/ in /opt/pkgist/vendor/amphp/file/lib/UvDriver.php:420
Stack trace:
#0 [internal function]: Amp\File\UvDriver->Amp\File\{closure}(NULL, 0)
#1 /opt/pkgist/vendor/amphp/amp/lib/Loop/UvDriver.php(174): uv_run(Object(UVLoop), 1)
#2 /opt/pkgist/vendor/amphp/amp/lib/Loop/Driver.php(130): Amp\Loop\UvDriver->dispatch(true)
#3 /opt/pkgist/vendor/amphp/amp/lib/Loop/Driver.php(70): Amp\Loop\Driver->tick()
#4 /opt/pkgist/vendor/amphp/amp/lib/Loop.php(76): Amp\Loop\Driver->run()
#5 /opt/pkgist/bin/gc2(11): Amp\Loop::run(Object(Closure))
#6 {main}
  thrown in /opt/pkgist/vendor/amphp/file/lib/UvDriver.php on line 420

But run env AMP_LOOP_DRIVER='\Amp\Loop\EvDriver' php72 -dmemory_limit=1024m /path/to/php.php, it works normally:
array(0) {
}

I think @bwoebi needs to update php-uv to support libuv 1.0. In the mean time I committed a work-around in 04f047aec618a3aefdedf0d41c46bc66c301f603.

Thanks @trowski, fixed it upstream: https://github.com/bwoebi/php-uv/commit/72aa710358672ed037e1b3dc60696f2928f95413


The plugin device_apps could not be built due to the issue above

Hello
Im use v1.14.6 flutter
but
I have an error The plugin device_apps could not be built due to the issue above
What's the solution?
What should I do?
Please help me a lot

Where is the "issue above"?


Add load following dispatch example

Add Load Following Dispatch Example

#262 Introduces a new feature: heuristic load following dispatch. Create an example notebook in the style of HOPP v2.0+ examples.
Proposed solution

There's an existing example that has become out of date. Rework this into a notebook.

@genevievestarke Do we have an example for this?


Very bad at language ability

Check them.
https://github.com/PKU-YuanGroup/MoE-LLaVA/issues/3#issuecomment-1916162040
https://github.com/PKU-YuanGroup/MoE-LLaVA/issues/5#issuecomment-1918518044


Wiki update for Sending-Mail section

Wiki Update
The example here needs to be removed or changed since Send-DbcMailMessage has been excluded from the module.
https://github.com/sqlcollaborative/dbachecks/tree/9330fe53b50c385e0d8ce8e463279fd778eeb82b#sending-mail
A suggestion on how to send emails still from @SQLDBAWithABeard:
https://github.com/sqlcollaborative/dbachecks/issues/539#issuecomment-418649934
$ouputDirectory = (Get-DbcConfigValue -Name app.maildirectory)
$filename = $outputDirectory + '\file.xml'
Invoke-Dbccheck -OutputFile $fileName -OutputFormat NunitXML

$outputpath = $ouputDirectory + "\index.html"
$reportunit = "ModulePAth\bin\ReportUnit.exe"
& $reportunit $ouputDirectory

$htmlbody = Get-Content -Path $outputpath -ErrorAction SilentlyContinue | Out-String

Send-MailMessage -To -From -SMTP -body $htmlbody

o/ @SQLDBAWithABeard
We can close this off, right?
#605 seems to handle this


A problem about the results.

Hi, Thanks for your wonderful works.
I have tried your codes, but It seems the test result doesn't match with the result in your paper. I just follow all the steps from your webside and the config file is the copy from template_fw0.5.yml. Then I run "bash scripts/eval.sh" and I test two objects (ape and cat). The result is shown as follows:
For ape:
2d projections metric: 0.9857142857142858
ADD metric: 0.780952380952381
ADD2 metric: 0.1380952380952381
ADD5 metric: 0.4219047619047619
5 cm 5 degree metric: 0.9876190476190476
mask ap70: nan
seq_len: 1050
For cat:
ADD metric: 0.9421157684630739
ADD2 metric: 0.2275449101796407
ADD5 metric: 0.6117764471057884
5 cm 5 degree metric: 0.9940119760479041
mask ap70: nan
seq_len: 1002
I cannot find the results in your paper
I don't know why?

Hi, do you install the environment with the docker file provided? We have tested locally and the results should match or are at least close to those reported in our paper.

OK, I install it using miniconda.
I will try docker later.

Hi, may I ask another question?
the runtime analysis in table 2 is for one iteration or five iteration? It seems not clear.
Thanks.

Table 2 provides the runtime of each individual module per forwarding.

OK, I test it in my PC with 0.7179 second per image using gpu 1660.

Hi，Did you get the same result with the ape model. For ape, I also get 78.1.

OK, I install it using miniconda. I will try docker later.

Hi，Did you get the same result with the ape model. For ape, I also get 78.1.


File issues magic/v4

#1276 fixed and updated to pass on 6

Should https://github.com/OISF/suricata-verify/pull/1253 be closed in favor of this PR ?

Merged in #1288