Possible to go from NSAttributedString, to "simple" tags?

In this great library, it's unclear:
Say I have an attributed string, which has (let's say) ONLY bold and italic sections.
I want to PRODUCE THIS OUTPUT:
This string has only some bold and some italic.
As you know, you can certainly generate html using Apple's built-in generator, but it produces wildly complex CSS output (see below for example).
So, I know that with SwiftRichString you can go FROM  something like
This string has only some bold and some italic.
to an NSAttributedString - but can you go in the other direction???
Sorry it's a bit unclear from the doco - or maybe I'm just lame  :)
Example of Apple's crappy full-css output...,
if you use .attributedText.data
><!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">

<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<meta http-equiv="Content-Style-Type" content="text/css">
<title></title>
<meta name="Generator" content="Cocoa HTML Writer">

So, Apple made this stuff OK

<

Hi,
No, currently the transformation from attributed string to html-tag-like version is not supported and it's not in the scope of the library.
I'll evaluate it but frankly it's not a priority for the library.
If you want, I'll be happy to accept a PR for it.


Add DockerFile for ChromeDevTools

Note that I was not able to verify it fully works, as it throws an
authentication error on typescript/typescript on the Docker Hub.
This is part of #39568
CC @weswigham

typescript/typescript is built locally from the Dockerfile in the repo root before running the docker tests. gulp run tests --ru=docker --t=devtools should work.

I was able to make it run with:
npm run gulp -- runtests --ru=docker --t=chrome-devtools-frontend-next

I am not sure how best to make the error log as useful for you. The CMD now makes sure that into only shows any failures, if there are any. This should remove all noise from Ninja and produce a stable output.

@typescript-bot user test this

@weswigham It's been a while since we worked on this PR. Could you let me know what the latest status update for it is?


Generic source parameters for options

This allows automatic inference of source types if provided in options.
const layer = new ImageLayer({
  source: new ImageWMS()
});
// layer is now of type `ImageLayer<ImageWMS>`

Also in TypeSript this disallows construction with incorrect source types.
const layer = new ImageLayer<ImageWMS>({
  source: new ImageSource()
});
// was allowed before, should result in a type error now

Here is a picture of the resulting jsdoc. It is problematic that the type of the source is now seen as the generic parameter (ImageSourceType). Also the generic parameters are not documented.
It would be possible to replace ImageSourceType with the type that it has to be at least (module:ol/source/Image).

I added a simple parsing of the comments for template value name / type pairs that will result in docs that look like before. Still missing any template parameter, though.


Material UI Icons cause to "invalid hook call" issue

Hi. I use a simple template example sign-in in my own project
After starting the project Chrome is showing me error "Invalid hook call. Hooks can only be called inside of the body of a function component."

Using trial and error I've found that the line "LockOutlinedIcon " causes the problem.
<Avatar className={classes.avatar}>
          <LockOutlinedIcon />
</Avatar>

Any icon I use through import LockOutlinedIcon from '@material-ui/icons/LockOutlined'; causes the error.
Environment
Material-UI core - 4.3.1
Material-UI icons- 4.2.1
React - 16.8.6
React-dom - 16.8.6

I also have a similar problem with using withMobileDialog(), it brings up the following error when compiled in a production build:
uncaught at F Invariant Violation: Minified React error #321; visit https://reactjs.org/docs/error-decoder.html?invariant=321 for the full message or use the non-minified dev environment for full errors and additional helpful warnings.

Error occurs for me when using any icon. Seems this issue happens with next.js. I tried messing with the pages under examples/nextjs-with-typescript by adding an icon and the same error occurred.

same with @ramonjesusburgos
When added the Customized snackbars example code in examples/nextjs-with-typescript, https://material-ui.com/components/snackbars/ I got this error.
Warning: React.createElement: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.

After removing this line, there is no error any more.
<Icon className={clsx(classes.icon, classes.iconVariant)} />

@oliviertassinari could you please reopen this issue?

@ithinco I would encourage you to report your issue on StackOverflow, you should likely find better luck.

I get this error every time i use material ui icons. And i can't find an answer on stackoverflow. Its really hurting my progress.

I had the exact same problem if I used npx create-next-app but if I used the MUI example code as base, it worked as expected

I had the exact same problem if I used npx create-next-app but if I used the MUI example code as base, it worked as expected

Solved ! I had the exact same problem i am getting the error
"Invalid hook call. Hooks can only be called inside of the body of a function component."
while importing any icon from @mui/icons-material iin my case i've installed the @mui/icons-material out of my react folder, i've deleted the package.json file and entered into react folder "cd my-app" and again i've run npm i @mui/icons-material and that's all Worked Perfectly for me !

Solved ! I had the exact same problem i am getting the error "Invalid hook call. Hooks can only be called inside of the body of a function component." while importing any icon from @mui/icons-material iin my case i've installed the @mui/icons-material out of my react folder, i've deleted the package.json file and entered into react folder "cd my-app" and again i've run npm i @mui/icons-material and that's all Worked Perfectly for me !

did you just delete package.json? did not delete dependencies in node_module?

Solved ! I had the exact same problem i am getting the error "Invalid hook call. Hooks can only be called inside of the body of a function component." while importing any icon from @mui/icons-material iin my case i've installed the @mui/icons-material out of my react folder, i've deleted the package.json file and entered into react folder "cd my-app" and again i've run npm i @mui/icons-material and that's all Worked Perfectly for me !

did you just delete package.json? did not delete dependencies in node_module?

I never saw 'invalid hook call', but I did see 'Warning: React.createElement: type is invalid', and it was bringing the whole page down.
Weirdly, in Vite dev mode, I could sometimes get the icon to display when I started with a string and swapped in an icon with hot module reloading (HMR). But not all the time, and never at first render.
What fixed it for me was switching from:
import InfoIcon from '@mui/icons-material/Info'

...to:
import { Info } from '@mui/icons-material'


Disconnect-Rubrik not working after PowerShell 7.3.9

Current Behavior
After succesfull export of database by use of the PowerShell-API, we call the disconnect function Disconnect-Rubrik. On Powershell versions greater than 7.3.9 this error is returned:

Invoke-WebRequest: C:\Program Files\WindowsPowerShell\Modules\Rubrik\Private\Invoke-RubrikWebRequest.ps1:28
Line |
28 | … $result = Invoke-WebRequest -UseBasicParsing -SkipCertificateCheck …
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| The given key 'Content-Type' was not present in the dictionary.

Expected Behavior
Close connection and invalidate token used to communicate with Rubrik server
Steps to Reproduce

Install latest PowerShell release 7.4.2
Install Rubrik-SDK 6.0.1 or 9.0.0
Connect-Rubrik -Server $cluster -Id $serviceAccountId -Secret $secret
Disconnect-Rubrik -Server $cluster

Context
No response
Failure Logs
No response

I get the same error with Remove-RubrikMount

Hi, could you please fix this issue soon, as it prevents us from being on the last version of PowerShell. It was reported 42 days ago, and still no person is assigned to fix it.

try using this as a workaround.
Invoke-RestMethod -Uri https:///api/v1/session/me -Method DELETE -Header $RubrikConnections[0].header -SkipCertificateCheck

Hi, i have tried the workaround sugested by jairulez but it is not working for us.
I get the Error : Request needs authentication! and i dont want to run with the parameter  -SkipCertificateCheck in production

I am looking forward to the release of the changes done by mwpreston

Hi Mike Preston! I can see you have fixed this issue, do you have a link where i can download the new rubrik.sdk module? Version 9.0.1 ?


bm_25 + semantic

https://github.com/langchain-ai/langchain/discussions/10619
https://python.langchain.com/docs/integrations/retrievers/bm25
https://python.langchain.com/docs/integrations/retrievers/merger_retriever

@pseudotensor could you please give me more context on this?

Lexical + semantic search is beneficial, esp when combined with reciprocal metric to merge them.
However, as-is, h2ogpt doesn't use retrievers specifically, but manages the retrieval itself, so bm_25 would have to be done same way, and we merge ourselves, or maybe switch to retrievers (at least custom retriever).


[bug]: occasionally nodes server takes ~5 minutes to validate requests with dataURLs

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Linux
GPU
cuda
VRAM
No response
What version did you experience this issue on?
nodes
What happened?
Working on the canvas migration to nodes. We need to send images to the server. Currently doing this with base 64 dataURLs as strings. Sending the whole graph at once.
Usually this works, but occasionally, session creation requests do not get a response for about 5 minutes, then I get a 400.
Attached is an example fetch() you can paste into the browser console. Every so often it just sits at pending for 5 min then fails.

fetch.txt
Screenshots
No response
Additional context
No response
Contact Details
No response

Turns out this was VSCode's built-in port forwarding being flaky.


Parity with Roboto’s weights

Since Roboto and Roboto Slab are under one superfamily, I think it would make sense for them to be available in the same weights(Added weights noted below). Medium falls between masters and would be an instance. To get Roboto Slab Black an additinal master is needed. In this case extrapolation actually works quite well and yields a result that needs minimal correction to be used as a master.
Added weights would mean 6 matching weights to Roboto:

Sample of the extrapolated Black with minor corrections on a few glyphs:

File size impacts:
1 static ttf ≈ 170kB
3-master VF ≈ 220/230kB
4-master VF ≈  275kB
(vf file sizes without VTT hinting)

Since Roboto and Roboto Slab are under one superfamily, I think it would make sense for them to be available in the same weights

I agree.
Perhaps you can still do 3 masters, Thin Regular Black?

Yep, changes are very subtle and some improvements on /uni0423 with longer inner serifs and /Thorn with a bit more space between the bowl and serifs.
Spacing shifts by a couple of font units and there are no significant changes to color, So I believe we are good on this front as well
I will move forward with the 3 masters as Thin, Regular, and Black

Sounds good, thanks!


TitanGraph Example Logging Configuration

Running the titan example test cases prints some unimportant logging information like:
1901 [pool-8-thread-6-ScalaTest-running-GremlinScalaSpec] INFO  com.thinkaurelius.titan.graphdb.configuration.GraphDatabaseConfiguration  - Set cluster.partition=false from store features
1901 [pool-8-thread-6-ScalaTest-running-GremlinScalaSpec] INFO  com.thinkaurelius.titan.graphdb.configuration.GraphDatabaseConfiguration  - Set default timestamp provider MICRO
1901 [pool-8-thread-6-ScalaTest-running-GremlinScalaSpec] INFO  com.thinkaurelius.titan.graphdb.configuration.GraphDatabaseConfiguration  - Generated unique-instance-id=8d4cb72d8621-max-mac2
1909 [pool-8-thread-6-ScalaTest-running-GremlinScalaSpec] INFO  com.thinkaurelius.titan.diskstorage.Backend  - Initiated backend operations thread pool of size 8
1917 [pool-8-thread-6-ScalaTest-running-GremlinScalaSpec] INFO  com.thinkaurelius.titan.diskstorage.log.kcvs.KCVSLog  - Loaded unidentified ReadMarker start time Timepoint[1422346756411000 μs] into com.thinkaurelius.titan.diskstorage.log.kcvs.KCVSLog$MessagePuller@2aa0b4b
2928 [pool-8-thread-6-ScalaTest-running-GremlinScalaSpec] WARN  com.thinkaurelius.titan.graphdb.transaction.StandardTitanTx  - Query requires iterating over all vertices [()]. For better performance, use indexes
2929 [pool-8-thread-6-ScalaTest-running-GremlinScalaSpec] WARN  com.thinkaurelius.titan.graphdb.transaction.StandardTitanTx  - Query requires iterating over all vertices [()]. For better performance, use indexes
2931 [pool-8-thread-6-ScalaTest-running-GremlinScalaSpec] WARN  com.thinkaurelius.titan.graphdb.transaction.StandardTitanTx  - Query requires iterating over all vertices [(^label = saturn)]. For better performance, use indexes
2933 [pool-8-thread-6-ScalaTest-running-GremlinScalaSpec] WARN  com.thinkaurelius.titan.graphdb.transaction.StandardTitanTx  - Query requires iterating over all vertices [(^label = saturn)]. For better performance, use indexes

How to turn this off properly?

Good question. Wow, looking at the dependency tree it looks like we get
quite a number of logging frameworks as transitive dependencies.
That, plus your comment on Neo4j made me think declaring the databases
as dependencies is not a good idea...
Can you try your luck with a log4j.xml?
http://logging.apache.org/log4j/2.x/manual/configuration.html

I refactored the examples a little bit for my needs and it works as expected. Of course its some kind
of a hackish solution, one has to consider its usefulness.
E.g.
class GremlinScalaSpec extends FlatSpec
  with Matchers
  with InMemoryConnect
  with DefaultLogging
  with DefaultExecutionContext {

  "Gremlin-Scala" should "connect to Titan database and pull out Saturn's keys and shutdown cleanly" in {
    val cFuture = connect()
    cFuture onComplete {
      case Success(c) =>
        c.isOpen() shouldBe true
      case Failure(t) => throw new ConnectException(t.getMessage)
    }

    val gsTry = Await.ready(cFuture, Duration.Inf).value.get
    val gs = gsTry match {
      case Success(g) => GremlinScala(g)
      case Failure(t) => throw new Exception(t.getMessage)
    }

    (1 to 5) foreach { i ⇒
      gs.addVertex().setProperty("name", s"vertex $i")
    }
    gs.addVertex("saturn", Map("name" -> "saturn"))

    gs.V.count().head shouldBe 6

    val traversal = gs.V.value[String]("name")
    traversal.toList.size shouldBe 6

    gs.V.has(T.label, "saturn").count().head shouldBe 1

    val saturnQ = gs.V.has(T.label, "saturn").head
    saturnQ.property[String]("name").value shouldBe "saturn"

    gs.close
  }
}

With:
import org.apache.log4j.Logger
import org.apache.log4j.Level

object InMemoryConnect {

  trait SilentLogging {
    implicit val out = new java.io.ByteArrayOutputStream()
    Logger.getRootLogger.setLevel(Level.OFF)
  }

  trait DefaultLogging {
    implicit val out = Console.out
    Logger.getRootLogger.setLevel(Level.ALL)
  }

  trait DefaultExecutionContext {
    implicit val ec = ExecutionContext.global
  }
}

trait InMemoryConnect {
  private val conf = new BaseConfiguration()

  def connect()(implicit out: OutputStream,
                ec: ExecutionContext = ExecutionContext.global): Future[TitanGraph] = {

    Console.withOut(out) {
      conf.setProperty("storage.backend", "inmemory")
      import com.thinkaurelius.titan.core.TitanFactory
      Future { TitanFactory.open(conf) }
    }
  }
}

Simply switch from:
class GremlinScalaSpec extends FlatSpec
  with Matchers
  with InMemoryConnect
  with DefaultLogging
  with DefaultExecutionContext

to:
class GremlinScalaSpec extends FlatSpec
  with Matchers
  with InMemoryConnect
  with SilentLogging
  with DefaultExecutionContext {

to suppress all the DB logging.
This works on the ShortestPath adapted to inmemory Titan as well. :)

Thanks for sharing! Disabling the root logger should not normally happen in code, and these example projects should log more rather than less, so I'll leave it as is.


JS lambda template

placeholder, we need one like https://github.com/confluentinc/dtx-template-registry/commit/dd3be442555b3c01687a9336dafa5e3fe8c77f8d

double-check support for JS by AWS

UPDATE: AWS supports all major langs and even some custom runtimes like Dart.


Remove the role designation from a Service Account

What would you like to be added:
I am wondering why a particular role is being associated with the service account.  It looks like we can remove that part and let the containers in the pods set their own role they want to assume using the shared SA JWT token in the pod.
You can allow the same service account to assume several roles.  I ran into this issue when trying to add service accounts to the sidecar injector.  If a developer sets a SA on a pod for something like access to a particular CRD, I have to annotate the SA or provide a new one.  Either way I have to elevate the side car injector privileges within the cluster.  If the user has their own SA for IAM setup, I have to overwrite that annotation or somehow force a new App Mesh policy into the existing role.
It does not appear that the SA annotation needs to be directly linked to a role.  It just needs to signify this SA token should load a JWT for AWS.
Why is this needed:
Several roles in AWS can be associated with a Service Account just like you can bind several K8s roles to a service account.

Thanks for filing this issue. At this point we cannot change the existing behavior of looking up the annotation on a ServiceAccount because that would result in a breaking change for our customers.
The primary reason we put the annotation on the ServiceAccount rather than on the pod is because the identity within the Kubernetes cluster is what the IAM role's trust policy has specified, not the pod itself. The other reason we lookup the role annotation on the service account is to encourage a least privilege model where one Kubernetes identity assumes a single role. You are correct that a Service Account is capable of assuming multiple roles, however it is more complex to keep track of which roles are bound to a particular SA.
If there is a need for specifying a different role than the annotation, I do want to support that use case. Currently the webhook short-circuits if either AWS_ROLE_ARN or AWS_WEB_IDENTITY_TOKEN_FILE are present and doesn't modify the container. What do you think about keeping the existing behavior, but if AWS_ROLE_ARN is present in a container, not overwriting it with whatever the ServiceAccount specifies? This would have the effect of letting you use different identities for applications using the existing flow, but specifying a different role for a sidecar for something like AppMesh. Would that suit your use case?

Thanks for responding.
I just want to follow with a few points.
I agree the trust policy should trust the service account but it is a one to many relationship.  Basically this current implementation ties authz to authn, in theory if not in actual practice.
I say in theory because there is no security boundary by explicitly naming a role on a service account, it is essentially a flag that says "load a token."  For those less experienced this could lead to a misinterpretation of trusted entities for a service account.  It does not actually enforce the concept of least privilege.  It actually confuses authorization as it provides the image of least privilege where there is none.  The role annotation actually has no link to privilege.  So service accounts could be given permissions on a Role through IAM and there would be a disparity with what is in the code describing services on the cluster and what is actually implemented in AWS.
I think the OIDC approach was a fantastic take on IAM permissions for pods, but I think explicitly linking one role to a service account is misleading and ultimately not how it should be used.
The service has only been out a for a few months I think this would be a great opportunity to change course before adoption increases.

I am following with a quick note for those that are new to this.  The token that this tool loads right now is in no way bound to a IAM role.  It is just a token that establishes a Service Account's identity.  The annotation on the Service Account in no way binds that identity to one particular role.  The extent at which it is bound is only in the env variable.  If ANY other role sets the same Assume role policy, that pod can use the same token to assume a different role.

This will not be changed so I am closing the issue.


pinentry.exe always shows prompts in the background

Several times, I've done a check-in, then gone on with some other task not noticing that there's a pop-up hidden, waiting for my PIN. Signed commits are new to me, so I'm not used to having to do that.
I know almost nothing about golang. This project is the first time I've even looked at golang source code, so take what I say with a grain of salt.
I see in credui.go that you call win.SetForegroundWindow. I assume that your win.FindWindow works. I can't spare the time to figure out how to debug golang right now (sorry).
I did find a couple of articles that might help. The problem might be that the process that launches pinentry.exe is not in the foreground. If it's not a GUI app, then this may not be practical to solve.
https://github.com/skratchdot/open-golang/issues/16#issuecomment-693687428
https://stackoverflow.com/a/19136480/645431

I too have this problem.
I can't find the documentation for where this timeout is supposed to go?

From the README:


feat: upgrade the Files API

For https://github.com/ipfs/interface-ipfs-core/pull/378

Just missing moving the addFromFs, addFromUrl and addFromStream tests to https://github.com/ipfs/interface-ipfs-core/pull/378

Merging as the only failures are ping related


10.10.2 pre-release version Xcode issue.

Same here

This should be fixed by #57, please verify

I downloaded the zip file, build it and everything went OK. Just a few minutes before, Alcatraz version didn't worked. Thanks

#57 works in 10.10.2(14C94b).

cant open my xcode anymore... I will remove this plugin

@CavalcanteLeo This should have already been resolved at the time you commented. Have you tried upgrading/reinstalling? If the problem persists can you open a bug with the details. Thanks.

I updated the binary .zip in 'releases' to make it easier to update.


bazel: Allow to distdir all dependencies

To use --distdir option of Bazel (which allows to use previously
fetched tarballs instead of downloading dependencies during
build), all dependencies should use http instead of git and need
to have sha256 sums specified.
Risk Level: low
Testing: n/a
Docs Changes: updated bazel/EXTERNAL_DEPS.md
Release Notes: n/a
Signed-off-by: Michal Rostecki<EMAIL_ADDRESS>

@lizan PTAL

@lizan Comments addressed


[Event Request] codeunit 56 "Sales - Calc Discount By Type" - OnBeforeApplyInvDiscBasedOnAmt

New Event at the beginning of the function ApplyInvDiscBasedOnAmt:
procedure ApplyInvDiscBasedOnAmt(InvoiceDiscountAmount: Decimal; var SalesHeader: Record "Sales Header")
var
    TempVATAmountLine: Record "VAT Amount Line" temporary;
    SalesLine: Record "Sales Line";
    SalesSetup: Record "Sales & Receivables Setup";
    DiscountNotificationMgt: Codeunit "Discount Notification Mgt.";
    InvDiscBaseAmount: Decimal;
begin
    OnBeforeApplyInvDiscBasedOnAmt(InvoiceDiscountAmount, SalesHeader);
    with SalesHeader do begin

[IntegrationEvent(false, false)]
procedure OnBeforeApplyInvDiscBasedOnAmt(InvoiceDiscountAmount: Decimal; var SalesHeader: Record "Sales Header")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 41884.


Add slide parameter to changeInterval listener

Description
The changeInterval event is emitted when slideOnRemove happens. When customers subscribe to this event to create revertibles for undo-redo, this event should be ignored as it is not a user edit. But there is nothing in the information passed to the listener to make this discernible from user-edited interval moves.
This change adds an additional parameter to the event listener: "slide" is a boolean that indicates whether the event is a slide caused by position removal.
I also considered populating the op to the listener (it's not set in these cases) so it could be checked to see if it was a position remove or a change interval. But this had quite a large fan out and API breaking changes (although in APIs that I think we don't really want to surface externally).
I also considered making a new slide event and splitting that off from changeInterval, but that felt a bit redundant and is a breaking change in behaviour (although I don't think there are many users yet).
Breaking Changes
This adds a new parameter to the listener, but code that doesn't expect it will continue to work. And code that is expecting it but uses an old version will get undefined.

Does this change warrant a changeset? Since there is a public API change?


checkpoint

Hi, thank you for your excellent work~ It seems that the provided checkpoint is not right.
I run datasetGAN/test_deeplab_cross_validation.py, and the wrong is as follows:
"File "datasetGAN/test_deeplab_cross_validation.py", line 357, in test
classifier.load_state_dict(checkpoint['model_state_dict'])
Missing key(s) in state_dict: "backbone.conv1.weight", "backbone.bn1.weight", "backbone.bn1.bias", "backbone.bn1.running_mean", "backbone.bn1.running_var"....
Unexpected key(s) in state_dict: "module.layers.0.weight",.."
The parameters in the classifier and the provided .pth model are not consistent.

Hey we didn't provide deep lab checkpoints.

Thanks for your reply！So I have to train one？ Is there a shortcut if I only want to have a try？ Do you provide a demo or colab？


Added unit tests for Globus endpoint transfers and transfer tests for service.

This PR adds tests for file transfers between a pair of Globus endpoints that is publicly available for using their Python SDK. It should be merged after #23.

Okay, I rebased to prod the CI system to running tests and it obviously found something. Time to clean out some more cobwebs.

I think Globus now does better error checking via their API, which is great! Just have to fix some issues that it has identified.


 Pressing CTRL+C shows the graceful shutdown message but gets stuck forever (or at least for too long). There's probably some goroutine or something not listening for the context cancelation properly.
Not super important, but quite annoying.

Seems to be a problem with Libp2p. Opened an issue with them: https://github.com/libp2p/go-libp2p/issues/2968.


Fix Scoring Protocol

It seems like env expects a None value

    env          -- dict; environment in which to run tests. If None, an

                 empty dictionary is used instead.

Making None be the default value for env in DoctestSuite fixes the scoring protocol
Old error when running the scoring protocol:
Traceback (most recent call last):
  File "ok/__main__.py", line 9, in <module>
  File "ok/client/cli/ok.py", line 132, in main
  File "ok/client/protocols/scoring.py", line 51, in run
  File "ok/client/sources/ok_test/models.py", line 99, in score
TypeError: run() missing 1 required positional argument: 'env'

LGTM, merging.


Fix missing geometry type exceptions

Fixes #7728
It had two origins:

Race-condition when the query-schema-model was updated while the
modal was closing, the model listeners weren’t removed until the modal
animation finished. Changed to be removed straight away.
Option view required geometry type, even though it might not be
available (yet, or due to failing node results), so relaxed the check

@alonsogarciapablo could you try this branch locally and confirm that it solves your problem, please?

\o/

Checking!

👍  👍  👍  Code looks 👍  and it works 👍  locally.


how to use params in sendCypherQuery ?

I noticed the readme doesn't really talk about params in sendCypherQuery. Is this like preparted statements in SQL?

This helped
https://github.com/neoxygen/neo4j-neoclient/blob/5526f847928584455f7c4bd20ecd50b0cd657146/tests/Neoxygen/NeoClient/Tests/Functional/UseCaseTest.php#L33-L34
I wonder thought ... is the {   } notation something from this library or neo4j server? Because it might be confusing to use this as placeholder.

It is from the Neo4j server, it is used in all documentations you will find about parameters.
Also you might read : http://www.sitepoint.com/discover-graph-databases-neo4j-php/

Can we add a small text to readme.md to explains how prepared statements work?


chore: 06-solomachine rename update to UpdateState

Description

Rename update to UpdateState

ref: #878

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

Codecov Report

Merging #1110 (51465ee) into damian/solomachine-verify-clientmsg (d6d9920) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@                           Coverage Diff                           @@
##           damian/solomachine-verify-clientmsg    #1110      +/-   ##
=======================================================================
- Coverage                                79.70%   79.70%   -0.01%
=======================================================================
  Files                                      151      151
  Lines                                    10910    10908       -2
=======================================================================
- Hits                                      8696     8694       -2
  Misses                                    1788     1788
  Partials                                   426      426

Impacted Files
Coverage Δ

...dules/light-clients/06-solomachine/types/update.go
95.00% <100.00%> (-0.17%)
:arrow_down:


tbdex test vectors

adding tbdex test vectors to tbdex-rs

Leaving 2 todos and merging
// TODO: Fix offering test vector / web5-js bug - https://github.com/TBD54566975/tbdex/issues/346
// TODO: The quote test vector needs to be updated - https://github.com/TBD54566975/tbdex/blob/main/hosted/test-vectors/protocol/vectors/parse-quote.json


[ML][Pipelines] Support serverless compute in pipeline & add end-to-end tests

Description

bypass getting compute from service when it's serverless
add end-to-end tests, and skip in playback mode for now

All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


How to prepare a custom training data ?

Is there any tool to annotate words and gives this format of train.txt ?

The following repository contains the code I used to convert medical text data to the CoNLL train.txt format: https://github.com/mxhofer/i2b2_2009-to-CoNLL
This article provides additional details: https://towardsdatascience.com/deep-learning-for-named-entity-recognition-3-reusing-a-bidirectional-lstm-cnn-on-clinical-text-e84bd28052df
Hope this helps!


Producers websockets should be non-SSL

No need to incur encryption cost for this.
Actually should the whole site be http? 🤔

One issue here would be on the authentication. If the page is served in session over https://, the websocket on ws:// (i.e. non-encrypted might not carry the session cookie 🤔

Closing this as won't do. SSL is required to use many of the functionality now:

media stream APIs
iframe embedding from https source like vdo.ninja


Breaking the circular dependency between adapters and the gush package

Currently all the adapters have a dependency on the gushphp/gush, but gushphp/gush requires all the adapters, which causes a circular dependency between the packages. Making changes in the interface a real "challenge".
The original idea was to fix this is using a composer plugin, but this would only solve one problem.
Because the gushphp/gush package still requires all the adapters :angry:
Also we don't actually use plugins or external adapters like bldr does, using the phar is the only supported method.
Since the https://github.com/gushphp/gush/blob/master/bin/gush will only enable adapters that are installed, we don't actually need the adapters to make gush work, we only need them for the phar building process.
So here are some options I think of:

use a dedicated composer.json for the phar building (if at all possible)
move adapters to the require-dev section; this will however also install the dev-tools which is not what we want.
Create a new repository named gush-application which contains the phar-builder, a composer.json for installing gushphp/gush and the adapters. The original package does no longer require the adapters.

Better ideas and suggestions are welcome :+1:

repos removed :+1:
I think we should ask @matthiasnoback about this.
I personally like 1, since bldr (@aquaesi tool) can do this without problems ?

great team work @sstok :open_hands: :ok_hand:

I discovered something :flushed: the reason why adding "gushphp/gush" as dev-requirement for some adapters doesn't work is because the branch-alias is missing in those adapter packages!
If we are working on an adapter simply add the branch-name as branch-alias and gush should have no problem being installed as an dependency :smile: Why did I not discover this sooner :hammer:
"extra": {
        "branch-alias": {
            "dev-master": "1.11-dev"
            "dev-my-new-feature": "1.12-dev"
        }
    }

When add gushphp/gush as (dev-)requirement to say the gushphp/gush-github-adapter and install the deps for that package the gushphp/gush requires the gushphp/gush-github-adapter package which is provided by the current package.
BUT! The gushphp/gush-github-adapter package has no branch-alias so Composer will use the branch-name (master) as the provided version dev-master which does not fulfill the version-requirement for the gushphp/gush-github-adapter which is ~1.10. And so Composer gives-up.
But if you add a branch alias to gushphp/gush-github-adapter composer will known that master is the same as 1.11.0 fulfills the requirement.
Alternatively you set the COMPOSER_ROOT_VERSION env variable to 1.11.0 as its works the same as the branch-alias for the root-package (the one you are running the composer command with now).

I tried it and its working :+1:


Add instrumentation for tasty

This adds spans for all of:

Individual test cases
Test groups
Resource setup and teardown

It also correctly handles test parallelism.

@ocharles this is just the library that I wrote for us to use


TI Program - Add Extensions and Improve Statistics for TI Assembly Programs

Describe the enhancement
There's missing extensions in linguist for TI Assembly which are part of TI Program.
It was brought to my attention by @TIny-Hacker that statistics for TI Programmers are widely skewed towards the most basic programs (with #5446 missing which exemplifies some basic programs in newer formats) such as while majority of the community uses TI Assembly but not all the extensions are included as there's no z80 assembly syntax highlighting for ".8xk" / ".8xk.txt" while ".83p" and ".83p.txt" are among the extensions for TI Assembly that are written in assembly are missing.
References:
https://en.wikibooks.org/wiki/TI_83_Plus_Assembly
https://github.com/github/linguist/blob/1345d0122a82119ae01abc366346fc9b022cc444/lib/linguist/languages.yml#L6675-L6685
Please see conversation reference at https://twitter.com/darkmastermindz/status/1626954142756356096
Requirements
I can help with contributing some initial PRs and testing as well if needed.
- [ ] Identify all TI Assembly extensions
- [ ] Discuss syntax highlighting and statistics for TI Programs (TI Assembly vs TI Basic)
- [ ] Contribute to enhancements for each in PRs

After some thought I feel like TI Program is mostly just assuming TI-BASIC, or at least TI z80 BASIC, since (e)Z80 ASM/C aren't necessarily platform-specific. Technically, .8xp is a binary file anyway, so most source code (even for basic programs) isn't uploaded with a .8xp extension, but just a text file like here: https://github.com/NinjaWeedle/CEPORT. Binary files aren't recognized as source, so unless you specifically create a text file with your code and then change the extension, it won't detect it.
Given that assembly programs for TI calculators are often made with the extension .asm or .z80, they are counted as simply "Assembly" and not specifically TI, which is fine given that other devices (for example, gameboy) use z80 assembly as well.
Extensions for other TI calculator platforms which use some form of TI programming language could be nice to have, like .82p, .85p, .86p, .89p, .v2p, .9xp, and .tns. I'm not exactly sure why .8xk is supported, since it's specifically a compiled app binary and the source for that would be .asm or .z80 (not .8xk).

I've created a grammar for z80 TI-BASIC here which could possibly help with syntax highlighting.
What makes the most sense to me is to remove .8xk and .8xk.txt, add .83p and .83p.txt, and possibly rename from "TI Program" to "TI-BASIC" so there's less ambiguity.

Hi @Alhadis - previously you discussed about TI Programs and ASM in #3330 - I would like to hear some input on this as well on this before starting a PR. I definitely would like TI Programs to be accurately represented. :D

Oh man, this is a blast from the past, both this repo and calculator programming in general. It's nice to see there's still an interest and community around it :)

Technically, .8xp is a binary file anyway, so most source code (even for basic programs) isn't uploaded with a .8xp extension, but just a text file like here. Binary files aren't recognized as source, so unless you specifically create a text file with your code and then change the extension, it won't detect it.

I agree with this... .8xp is the extension for the source files on the calculator itself, but it's stored as binary and not normal plaintext source code. Linguist can't recognize binary files (or at least that was the case 6 years ago) so people need to translate them with a tool like this then upload the resulting text under the .8xp.txt extension.
IIRC I didn't realize Linguist couldn't recognize binary files when I first added the .8xk extension, and I thought it would help GitHub recognize compiled apps. I have no problem with removing .8xk now if that is still the case. (and .8xk.txt probably shouldn't have existed to begin with).
I know very little about assembly, so I can't speak to that, except that I agree marking all .asm as calculator programs would be bad lol.

possibly rename from "TI Program" to "TI-BASIC" so there's less ambiguity

I hesitate to rename "TI Program" to "TI-BASIC". Before the color TI-84s came out a lot of programs were made with axe parser, which is more powerful than TI-BASIC, compiles into assembly, and has its own syntax that's similar to but distinct from TI-BASIC. But it's been a long time since then, and from my googling it seems that Axe hasn't been updated in years, so perhaps it's no longer relevant.

Searching here I see some repos using .8xp.txt like I intended, and some that just uploaded the plaintext as .8xp. I don't see any instances of .8xk or .8xk.txt so I doubt they'll be missed if we do remove them.

I hesitate to rename "TI Program" to "TI-BASIC". Before the color TI-84s came out a lot of programs were made with axe parser, which is more powerful than TI-BASIC, compiles into assembly, and has its own syntax that's similar to but distinct from TI-BASIC.

I agree with this as well. Initially when I suggested renaming it I wasn't thinking of third-party languages like AXE on monochromes and ICE on the colored calculators. While it isn't used much anymore (given that most students use color calculators now), it (along with ICE) still is widespread enough that I don't think renaming makes sense anymore.

@scottmangiapane @TIny-Hacker I think a good approach to solution to also have is to maybe make TI Program a parent group and then split these into separate categories based on highlighting requirements much like how Typescript is implemented in languages.yml so we can solve the problem of statistics first without changing the current implementation.
https://github.com/github/linguist/blob/1345d0122a82119ae01abc366346fc9b022cc444/lib/linguist/languages.yml#L30-L31


add gzip

addressing #278

[x] added compression
[x] fixed site optimization
[ ] fixed image optimization
[ ]  fixed text optimization

I don't see any image warnings, but I didn't see any previously either, so I can't confirm that this is optimizing the images on the site.
This was solved
Compressing https://react-webpack-node.herokuapp.com/ could save 8.1KiB (34% reduction).

These were not solved.
Compressing https://react-webpack-node.herokuapp.com/assets/app.js could save 196.6KiB (71% reduction).
Compressing https://react-webpack-node.herokuapp.com/assets/styles/main.css could save 1.9KiB (53% reduction).

@jrodl3r @psimyn @IljaDaderko  any ideas on how to get the text compression to work properly?

I think app.use(express.compress()) before the express.static line should handle js/css

@psimyn perfect.

👍 🎉


fix jest-puppeteer globals import

Summary
This PR contains a fix to import the jest-puppeteer globals in test/e2e/common.ts and reenable types resolution.

The fix is in branch my_branch.
How did you test this change?
The end to end test suite passes 100%.

⚠ Warning ⚠
I had to do a npm i and update package-lock.json since npm ci errored. I don't know if the new package-lock.json should be merged or discarded.

Oh wow what awesome service! I didn't think to check there would be a breaking change (given the minor rev) but this makes sense. So cool!!!

You're welcome ! The follow-up is part of contributing as well 👍


Errors in documentation?

Hi, when running cargo doc, the following errors are produced for this package:
 Documenting cortex-m v0.5.2
error: `[::task::Poll]` cannot be resolved, ignoring it...
   |
note: lint level defined here
  --> /home/emifre/.cargo/registry/src/github.com-1ecc6299db9ec823/cortex-m-0.5.2/src/lib.rs:42:9
   |
42 | #![deny(warnings)]
   |         ^^^^^^^^
   = note: #[deny(intra_doc_link_resolution_failure)] implied by #[deny(warnings)]
   = note: the link appears in this line:

            [`Poll::Pending`](::task::Poll). The future will *also* register the
                              ^^^^^^^^^^^^
   = help: to escape `[` and `]` characters, just add '\' before them like `\[` or `\]`

error: `[::task::Context::waker]` cannot be resolved, ignoring it...
   |
   = note: the link appears in this line:

            [`cx.waker()`](::task::Context::waker)). Once a task has been woken up,
                           ^^^^^^^^^^^^^^^^^^^^^^
   = help: to escape `[` and `]` characters, just add '\' before them like `\[` or `\]`

error: Could not document `cortex-m`.

I hope this helps to fix it :)

The bad reference is part of the core documentation comments (and only hits cortex-m because it denies warnings), and is still there in 2018-06-27 nightly (but was not there in stable 1.27, which works fine).
I think that the breakage is related to task::Poll being behind the futures_api feature gate (but the Future trait that returns a Poll and mentions it in its docs is behind the same gate).
Until the root cause is found, I suggest that the #![deny(warnings)] be overridden by an #[allow(intra_doc_link_resolution_failure)].

This appears to be fixed as of nightly-2018-08-01.


segmentation label shows all '0'

Hello, I got image segmentation label issue in Carla, but haven't found any one mentioned it. Not sure if it is a bug or not.
I am using Ubuntu 16, I directly download the version 0.7.0 binaries from website. I add the image segmentation in the sensor data as:
camera2 = Camera('CameraSeg', PostProcessing='SemanticSegmentation')
and store the segmentation ground truth into my local directory. But the segmentation image is all black, and I saw the server terminal confirmed that the segmentation is enabled.
Anyone encountered the similar issues? Thank you!

Hi @sufengniu,
The server encodes the segmentation label in the red channel as a number from 0 to 12, so it looks black at first sight.
In Python you can convert this labels using carla.image_converter to get the CityScapes palette. But if you have to convert a bulk of images the best way would be to compile and use the app at "Util/ImageConverter" as it is much faster converting lots of images. Just store all the images in a folder as you do now and run the image converter
$ cd Util/ImageConverter
$ make
$ ./bin/image_converter -c semseg -i path/to/image/folder -o output/folder

I also want to do this conversion for my picture, but I tried that CityScapes palette but picture is still black
Can you give us a brief description of the specific methods or steps? i really want it.


Errors (IndexError) on silent waves (degenerate pathological cases)

  File "/home/nyanpasu64/miniconda3/envs/wavetable/lib/python3.6/site-packages/waveform_analysis/freq_estimation.py", line 92, in freq_from_autocorr
    start = find(d > 0)[0]
IndexError: index 0 is out of bounds for axis 0 with size 0

If freq_from_autocorr is given a silent signal d contains a series of 0.
I feel the best solution is to just return some arbitrary value without crashing.

I don't remember if it happens in practice, but a while back I found that parabolic(f, 0) compares to f[-1] (which is wrong but doesn't crash) and parabolic(f, len(f)-1) compares to f[len(x)] which raises IndexError.

Idea: parabolic(f, x)
# if x < 0 or x >= len(f): raise IndexError(add meaningful message?)
if x <= 0 or x >= len(f) - 1:
    return (x, f[x]) # generates IndexError for x >= len(f), but not x < 0

Also, since freq_from_autocorr performs DC removal, I feel it's unacceptable (for my use cases) for a utility function to crash given any DC input, due to an unhandled internal IndexError. I think to make the function easier to integrate into surrounding code, it should provide a result (f=0?) with the same type as normal execution, or maybe raising ValueError (though I still don't like the idea of a frequency estimator crashing and taking down the entire app, given hard-to-estimate input where I probably don't care about the frequency).

@nyanpasu64
In what context are you using it that it's unacceptable to raise an exception?  A silent WAV file has no frequency, so I'm not sure what you would expect it to do here.  It certainly shouldn't return an arbitrary value in Hz.  It would make sense to return 0, since it's just a DC signal at 0 Hz.  It could also return None or NaN.

It's been years since I've worked with this library (and recently I tried getting some of my code to run again, but waveform-analysis fails from importing scipy.signal.kaiser which has been moved to scipy.signal.windows.kaiser). I'm no longer familiar with the code involved here. Looking back at my old code it appears I catch IndexError and give it a value of 0 instead. That might be a good idea?

In what context are you using it that it's unacceptable to raise an exception?

I think leaking an exception from an implementation detail is a bit messy compared to an intentional exception or return value, and having the caller handle an exception might be slower than branching on the return value.

I think leaking an exception from an implementation detail is a bit messy compared to an intentional exception or return value,

Yeah I agree.  Just not sure which to do.
https://github.com/endolith/waveform-analysis/discussions/35


Weird math

Why are you doing this?
			sky_data[abs(elevation-el_end),(azimuth-az_end)]=signal_strength

https://github.com/saveitforparts/Carryout-Radio-Telescope/blob/4f95d72945620df74f8c8e25442b5345cd2abd0d/carryout_scan.py#L159C1-L159C68
Elevation will start at el_start and end at el_end, so it will always be equal or greater to el_start.
Just to elevation-el_start and you'll go from 0 to el_end-el_start. Then you can drop the abs and do the same for the azimuth.
I'd also move the np.savetxt(f"raw-data-" + timestr +".txt", sky_data) line 161 outside the inner loop, then you'll save only once per azimuth level, filesystem write could be slow.

I was trying to make sure the array wrote in the same direction as the
scan, and every other line I swap the azimuth to go backwards. Probably a
better way to do that.
Originally I had another output to a preview image that updated every pixel
live, so you could watch it scan. I took that out for this version. I'm not
sure I really need to save time in the inner loop, I already have a couple
time.sleep delays so I don't run faster than the motors can handle.
But yes there are probably lots of improvements, I'm bad at coding :-)
On Fri, Jan 12, 2024 at 6:24 PM aza @.***> wrote:

Why are you doing this?
  	sky_data[abs(elevation-el_end),(azimuth-az_end)]=signal_strength

https://github.com/saveitforparts/Carryout-Radio-Telescope/blob/4f95d72945620df74f8c8e25442b5345cd2abd0d/carryout_scan.py#L159C1-L159C68
Elevation will start at el_start and end at el_end, so it will always be
equal or greater to el_start.
Just to elevation-el_start and you'll go from 0 to el_end-el_start. Then
you can drop the abs and do the same for the azimuth.
I'd also move the np.savetxt(f"raw-data-" + timestr +".txt", sky_data)
line 161 outside the inner loop, then you'll save only once per azimuth
level, filesystem write could be slow.
—
Reply to this email directly, view it on GitHub
https://github.com/saveitforparts/Carryout-Radio-Telescope/issues/1, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AUBUBKLRSQUVRWZ5IHS4SSDYOHH4JAVCNFSM6AAAAABBY4FXA2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGA3TSOJSGMYDIMY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

All good, I was trying to spot where the misalignment might come from and that is the only thing That I saw that could be it.


Everything works great, except transparency

I've restarted LibreWolf several times, the rest of the userChrome seems to function correctly. On KDE Plasma with Kvantum using a definitely very transparent theme (Gently + Kvantum Gently). Worth noting that the higher I set the transparency setting, the more broken it seems to get

Nevermind, answer is in closed issues. Perhaps you should add a note to the readme?

What issue are you referring to?

If you mean about transparency; https://github.com/QNetITQ/WaveFox/issues/149

Issue #149 has nothing to do with Linux. It looks like you mean issue #102. I'll add a link to this issue in the description.

Added a link.

No, I wasn't referring to that. There isn't any transparency happening. Look at the screenshot...

Then I don't know. Transparency works for me. Maybe it's your Linux distribution, maybe it's the OS theme, or maybe the style version doesn't match the browser version.

Nah, I actually decided to not use this For both of the reasons I stated before, which both remain unresolved

I don’t understand, are you trying to demand something from me now? I don't owe anything to anyone.

Nevermind, answer is in closed issues.

After that, I concluded that you solved your problem yourself, but it turns out not. Next time, if there is one at all, express your thoughts more precisely.

I literally sent you a screenshot of the problem

Well, yes. And then you literally wrote

Nevermind, answer is in closed issues. Perhaps you should add a note to the readme?

I thought that you solved your problem yourself and wanted to know how. You sent me a link to a bug report that has nothing to do with Linux. And then it turns out that you haven’t solved your problem. Why then write that you solved it? You just misled me.
But this doesn’t matter anymore, because then you started insulting me.  You're just an uncultured boor who disrespects other people. I will not tolerate such an attitude. Until you change your attitude, don’t write here.
P.S.
Sorry if I offended you in any way. I did not want. But here you are completely wrong.

"Starting with Firefox 117, transparency is not supported by the browser."
 Those are not the same thing

Starting with Firefox 117, transparency is not supported by the browser.

This only applies to Windows (Mica / Acrylic). Each system has its own methods for using transparency. On Linux, transparency is supported and works if the conditions in the description are met.

 Those are not the same thing

Didn't try to solve your problem? However, my actions say otherwise.

Have I responded to your problem? Yes.
Was I trying to figure it out? Had tried. And I figured it out.
I rechecked the transparency on Linux and everything works for me. So I can't confirm your problem. Other users don’t write bug reports to me about this, as you can see.

Are you tired of complaining to me yet? Well, I don’t owe anything to anyone! How much more do I need to explain this? Read the license terms. Only politeness and mutual respect can help. There is no other way!  There are only horizontal connections, not vertical ones.


Compact Multi-Class Boosted Trees

The issue related to the performance of the BoostedTree classifier in terms of accuracy.
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution: Linux Ubuntu 16.04, Windows 10
TensorFlow installed from (source or binary): pip install TensorFlow
TensorFlow version (use command below): 2.4.1
Python version: 3.6

Describe the model
BoostedTree classifier is a model introduced by N Ponomareva, T Colthurst, G Hendry.2017.
It is built over the Xgboost idea and learning is done through building one layer of decision tree regressor over N boosting iteration.
Describe the expected behavior
As the related paper discovered, for ten trees, the accuracy of the xgboost is about 60%, and for the TF model is 73%.
This experiment had done over the Cover type dataset.
Describe the current behavior
I did the same experiments over the same dataset and identical configuration, but the results have a huge difference.
In this experiment, the accuracy of xgboost is about 70%, and the TF model score is about 58 percent.
It is worthy to note the fact the logistic regression at least achieves 70 percent.
About the training time, if you plot the training time over boosting iteration you will have a Neutral trend which should be linear as the gradient boosting is a linear ensemble mode.
Standalone code to reproduce the issue
I reproduced the model here.
Also, you may find the related experiments here.

@samanemami  I tried to reproduce the issue  but facing error while importing data. Please check the gist here.

@saikumarchalla Please find the modified gist here

Was able to reproduce the issue using TF version 2.4 and Nightly version. Thanks!

After running different experiments, I found that you reduced the tree number in XGBoost, hence, it reduced the XGBoost accuracy.


Get-RubrikEvent - Enhancement request

Describe the solution you'd like
I would like to be able to exclude and include events based on the event type. I would like the following changes to be made:

Allow for Include and Exclude based on Event Type
Provide Default Output for all Event Types, by specifying an object type to the output
Include a proper date object for all Event Types
Investigate which parameters that are supported by the API could be added as parameters to this function

Added exclusion based on both Event & Object type, time objects are present & default view has been added.


Refactor migrations commands tests

Ensure that running the tests doesn't write in the code tree.

@andrewgodwin Can you do a quick sanity check? Most importantly look at the docstring of temporary_migration_module in the second commit. Thanks!

Looks good to me - a lot cleaner than the previous tests' behaviour.


start_isabelle_server doesn't work on Windows

The reason is like this https://stackoverflow.com/questions/44633458/why-am-i-getting-notimplementederror-with-async-and-await-on-windows
There is no reason why start_isabelle_server should use asyncio

The reason is like this https://stackoverflow.com/questions/44633458/why-am-i-getting-notimplementederror-with-async-and-await-on-windows

There is another reason: Isabelle server doesn't for Windows doesn't exist. The recommended way to run it under Windows is Cygwin. Even if asyncio runs without error, we will do something else to make it work.

There is no reason why start_isabelle_server should use asyncio

There is a very good reason, in reality,  since isabelle server not only runs the server, but it doesn't exit and waits for an interruption.

If one redefines a server starting functions like that:
def start_isabelle_server(
    log_file: Optional[str] = None,
    name: Optional[str] = None,
    port: Optional[int] = None,
) -> Tuple[str, asyncio.subprocess.Process]:
    """ start Isabelle server """
    # this line enables asyncio.create_subprocess_exec on Windows: https://docs.python.org/3/library/asyncio-platforms.html#asyncio-windows-subprocess
    asyncio.set_event_loop_policy(asyncio.WindowsProactorEventLoopPolicy())

    async def async_call():
        isabelle_server = await asyncio.create_subprocess_exec(
            os.path.join(
                os.environ["HOMEDRIVE"],
                os.environ["HOMEPATH"],
                "Isabelle2021-1",
                # here one should use a shiny new bat-file
                "Cygwin-Isabelle-Server.bat"
            ),
            stdout=asyncio.subprocess.PIPE,
            stderr=asyncio.subprocess.PIPE
        )
        return (await isabelle_server.stdout.readline()).decode(
            "utf-8"
        ), isabelle_server

    return async_run(async_call())

And Cygwin-Isabelle-Server.bat contains the following:
@echo off

set TEMP_WINDOWS=%TEMP%
set HOME=%HOMEDRIVE%%HOMEPATH%
set PATH=%HOME%\Isabelle2021-1\bin;%PATH%
set LANG=en_US.UTF-8
set CHERE_INVOKING=true

"%HOME%\Isabelle2021-1\contrib\cygwin\bin\bash" --login -i -c "isabelle server"

Then everything work on Windows. So, it's probably a good idea to include a new bat-file to Isabelle distribution. It should take arguments needed to start the server, and then start it using Cygwin, hiding this complexity from the user.
Without such a bat-file it's very hard to work with Isabelle inside a Cygwin bash inside a Windows subprocess from Python.


Request: Atom package dependencies

Some atom packages may be plugins of some other atom packages, so I suggest atom to install all packages listed at field package-dependencies of file package.json, it makes ability to integrate/extend atom packages easier.
At this moment, I can use atom-package-dependencies to depend on other atom packages, the thing is:

Dependencies (which were installed implicitly & automatically) wouldn't be uninstall automatically when their dependents is uninstalled, so users must uninstall them manually & explicitly.
Users can disable/uninstall dependencies even when their dependents haven't been disabled/uninstalled, without any warnings or errors, the dependent packages still run without their dependencies.

Atom Version: 1.6.2
OS and Version: Windows 10 Pro 64-bit (10.0, Build 10586)

This is a duplicate of #2412. You may want to subscribe there for updates.


An ask mode fix was introduced into release/m89 so bumping task versi…

…on to ensure it doesn't mask the fixes in master

LGTM


Replace a cut/rev/cut/rev train by a single sed command.

Sed is not available on syno.
Fixes #19

Pull request is for Master branch, Closing pull request without merge, however. I will take the changes and push them onto Dev branch.
Thank you so much for this fix as I hope it helps those using other systems that do not support rev


Added United States Cities

Added Cities of United States.

Congrats 🎉, your PR got merged. To be eligible for the swags from iNeuron X ClueLess. Follow these steps.

SignIn / SignUp in ClueLess Website

Go to the profile section from the top right-hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard!
And don’t forget to share your achievement with your friends. @sdeepanshu17


ais-dotnet-bing-maps-realtime.dib - FindKernel "Kernel" does not contain a definition for "FindKernel"

Hi - I tried running "ais-dotnet-bing-maps-realtime.dib", and it threw the following error:
var jsKernel = Kernel.Root.FindKernel("javascript");
jsKernel.RegisterCommandType<VesselPositionCommand>();

Error: (1,28): error CS1061: 'Kernel' does not contain a definition for 'FindKernel' and no accessible extension method 'FindKernel' accepting a first argument of type 'Kernel' could be found (are you missing a using directive or an assembly reference?)
I updated the "FindKernel" method to "FindKernelByName" and that seems to have worked.

Thanks for reporting that. I did have updated notebooks locally, but I hadn't pushed them up. I've just done that, so they should all be back in a working state.


use theme assets from RC for now

Pre-Flight checklist

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

What are the relevant tickets?
Partially resolves https://github.com/mitodl/ocw-studio/issues/685
What's this PR do?
When we set up the Concourse integration, the release branch of ocw-hugo-themes was hard coded into the pipeline definition.  This PR changes that to release-candidate for now so we can test ocw-hugo-themes changes using the RC instance of ocw-studio.
How should this be manually tested?

Set your env variables to the same as RC
Create a test course and add some basic content and metadata to it, enough to save and publish it
Run docker-compose run web ./manage.py backpopulate_pipelines --filter <site slug>, replacing <site slug> with the test course you just created
Click the preview button on your test site
Go to https://cicd-qa.odl.mit.edu/?search=team%3A"ocw" group%3A"draft" and find your pipeline
Verify that a build has been triggered and that the first step is pulling the release-candidate version of ocw-hugo-themes as in the screenshot

Screenshots (if appropriate)

There is a new build but it's not running: https://cicd-qa.odl.mit.edu/teams/ocw/pipelines/draft/jobs/build-ocw-site?vars.site="test-pr-686"


[sokol_gfx] Assign labels as debug names to D3D11 resources

This change assigns the "label" field form resource desc structs to D3D11 resource objects, using
SetPrivateData with WKPDID_D3DDebugObjectName. This code is enabled only when SOKOL_DEBUG is defined.
Note: this requires sokol_gfx to also link with dxguid.lib.
Note: some resources have multiple "sub-resources", e.g. a shader has a vertex and a fragment shader resource. In those cases the label is the same for all sub-resources.
This is useful especially in RenderDoc (docs) and in D3D11 debugging messages.

Good change, but I want to give it a whirl before merging.
We should try a change though which I think/hope allows us to get rid of the dxguid dependency:
In sokol_app.h I'm defining any required COM IIDs inline like this:
https://github.com/floooh/sokol/blob/91ec4b5fa95b1e3e3b2e21cfeac85d03cdc9a8ef/sokol_app.h#L6317-L6319
Maybe we can do the same inline definition for the WKPDID_D3DDebugObjectName ID.
I seem to remember that linking with dxguid caused problems in some scenarios (like compiling via Zig which uses the mingw2 headers instead of the Windows SDK).
Also another change request regarding this block:
#if defined(__cplusplus)
#define _sg_d3d11_SetPrivateData(self, guid, DataSize, pData) (self)->SetPrivateData(guid, DataSize, pData)
#else
#define _sg_d3d11_SetPrivateData(self, guid, DataSize, pData) (self)->lpVtbl->SetPrivateData(self, &guid, DataSize, pData)
#endif

Please create a wrapper function like this instead (all D3D API functions called by sokol-gfx have such a wrapper:
https://github.com/floooh/sokol/blob/91ec4b5fa95b1e3e3b2e21cfeac85d03cdc9a8ef/sokol_gfx.h#L9354-L9360
That's all for now :)

PS: did you also check that calling SetPrivateData with a nullptr works?

Oh yeah, defining the GUIIDs inline is a very good idea, I didn't notice the ones that were already there, that's why I linked dxguid.

Please create a wrapper function like this instead

The reason why I didn't create a function like this is because SetPrivateData is called on many different types, all child of ID3D11Device. And while the polymorphism works in C++, the C version needs to explicitly use the VTable. So I made a macro similar to _sg_d3d11_Release or _sg_d3d11_AddRef. I'm not aware of any way to make this work in C, apart from writing the function for each resource type. Should I do that instead?

did you also check that calling SetPrivateData with a nullptr works in case no label is defined?

Oops, good catch! Somehow I completely forgot about that, since I always set the labels in my own code. But looking at the docs it should be fine.

The reason why I didn't create a function like this is because SetPrivateData is called on many different types

Ah right, in that case the macro totally makes sense. But can you add a little explanation comment above the macro? Because such things tend to come up after a while again ;)

But looking at the docs it should be fine.

Agreed, looks like nullptr is valid.

Alright, I added the GUID constant and comments in the last commit

Last changes look good.
I just noticed a problem though I overlooked at first, those strlen calls on nullptrs:
(UINT)strlen(desc->label)

..AFAIK strlen on a nullptr is UB.
My suggestion would be to change the  _sg_d3d11_SetPrivateData into something simpler and remove any redundancies (for the C++ version):
_sg_d3d11_setlabel(self, label) (self)->SetPrivateData(_sg_win32_refguid(_sg_d3d11_WKPDID_D3DDebugObjectName), label ? (UINT)strlen(label) : 0, label)

...calling this would then look a bit simpler:
_sg_d3d11_setlabel(buf->d3d11.buf, desc->label);

Yeah the setlabel macro is a very good idea, it makes it quite a lot simpler. I didn't know about the strlen UB, thanks for letting me know.

Giving the PR a whirl now...

Nice, it also works in the Visual Studio graphics debugger:

And merged. I also added a blurb to the changelog.


[Bug]: Log 413 error in plugin execution

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When the payload size is more than 100MB, 413 response is thrown.
This error is not logged, Add an entry to error log
Steps To Reproduce
Upload any file > 100MB to any plugin
Public Sample App
No response
Issue video log
No response
Version
Cloud

use s3 plugin
try to upload a huge file >100MB
there is a timeout
Expected: an error should be shown in debugger
Actual: due to timeout, 413 error is thrown, but we are not logging it.

File: https://drive.google.com/file/d/1Hb3VjKuzyrAdRvQCn51iK-8li6DliN_l/view?usp=sharing
S3_ACCESS_KEY: AKIAVWHAAGIQN3JBRMYM
S3_SECRET_KEY: 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
BucketName: assets-test.appsmith.com
cc: @Irongade

Hi @Irongade
Tried to reproduce the issue with the above mentioned steps but was unable to view the error.
As per expectation I observe we have to display the following error :
Appsmith Error : Payload too large. File size cannot exceed ${maxFileSize}MB.
Appsmith Error code : AE-APP-4013
Appsmith Error message : API execution error
However currently we are not displaying the above said error. Please let me know if there is any way to repro this
cc: @ChandanBalajiBP

This PR is blocked due to added requirements to handle this issue using 100 Status code errors.


✨ Add timeline shortcode

@mirceanton please have a look before I merge to dev

@nunocoracao I implemented it on a dev branch on my website and it looks very nice! I think this is perfect. Thank you very much!
Preview: https://deploy-preview-69--mirceanton.netlify.app/about/#experience

Looks good yeah, thanks for sharing @mirceanton


is there a kafka sample or documentation ?

@iamiddy
:+1: More samples would be nice.
I found this today but I haven't had a chance to test the approach. It only has a producer, no consumer.
https://github.com/jbrisbin/tfl-ingest

Hi,
Unfortunately we haven't put together a set of dedicated examples yet (we welcome contributions!). Generally speaking though, taking any of the current examples and switching to the Kafka binder should work.
Hope this helps,
Marius

How do you switch to the Kafka binder? Does this mean changing all pom.xml files of the modules?

a straight forward (but compile time only) way to control which binder should be used could be maven profiles for the various binders. Although i'm not a big fan of this approach it would also make sure that the size of the fat JAR stays reasonable (in contrast to including all binders in the packaging phase and using properties to control which binder should be applied). It really depends on whether you need this runtime configurable or nor.

From the simple point of view of the uberjar - compiling the 'correct' binder is the way to go.
Using the module launcher, you can modify the classpath of the launched module at runtime (regardless what binder they have been packaged with) via boot properties, i.e.  --includes=org.springframework.cloud:spring-cloud-stream-binder-kafka:1.0.0.BUILD-SNAPSHOT --excludes=::*-binder-redis to swap the binder without recompiling.
If you are using Spring Cloud Data Flow, you can currently pass include/exclude properties per module (e.g. module.as well as using the module.*.includes idiom).

Hi @iamiddy, @Blackbaud-KevinHutson, @olmoser, @frosenberg: Hopefully, @mbogoevici's suggestions were helpful. We will be addressing documentation in general with examples via: #212 as soon as we wrap-up #138, #172, and #173.
Please let us know if you have any other questions or feedback. Closing this for now.


How to fix Dynamic Linking Error?

Hi,
I read here about similar issues but it couldn't be helpful for my case.
my problem:
I wrote RunCmd.dll C++ on windows 7 which run synchronic command line(process), the dll was tested successfully using C#, but when using FFI module I got error:
C:\Program Files\nodejs\node_modules\ffi\lib\dynamic_library.js:74
    throw new Error('Dynamic Linking Error: ' + err)
          ^
Error: Dynamic Linking Error: "RunCmd":

my code: (java script)
var nativeC = new ffi.Library("RunCmd", {
  "run_sync_process": ["void", ["LPTSTR"]]
});

console.log('running notepad');
var run = nativeC.run_sync_process;

run('notepad.exe');

I also try:
var libA = ffi.DynamicLibrary('RunCmd');

and got the same error
Any suggestion?

@TooTallNate We're having this problem over at cjlarose/argon2-ffi#20. We're trying to debug it there.

@PlasmaPower Can you open a new issue with specifics please?

This problem was solved today. The reason for the problem is that my x64 Node.js called the DLL file compiled by x86, which caused the problem. So, the solution is:

If you have the source code of the DLL, choose the same encoding method as your Node.js (for example, my Node.js is x64, and my DLL also chooses the x64 version to compile)
If you don't have the source code of the DLL, try reinstalling the Node.js installation package for x86 or x64.


Naming of three italian provinces

"Valle d'Aosta" should be "Aosta"
"Reggio nell'Emilia" should be "Reggio Emilia"
"Monza e della Brianza" should be "Monza e Brianza"
Sources:
http://www.upu.int/fileadmin/documentsFiles/activities/addressingUnit/itaEn.pdf
https://en.wikipedia.org/wiki/ISO_3166-2:IT
Reasoning:
"Valle d'Aosta" is the name of the region, but the name of the province is just "Aosta".
"Regio Emilia" is consistently used by UPU, ISO, and the italian site used as a source for Wikipedia: http://www.comuni-italiani.it/035/index.html
The official site of Monza e Brianza seems to use that form, just like ISO does.
These 3 are also the forms reported to us by our Italian users.

It seems this has now been done in the dataset, and the naming matches UPU/ISO.


tmt reportportal plugin not reporting multi-host test results properly

When importing test results from a multi-host test to RP, just a single (the first) test case instance is reported, despite the fact that the same test was running also on other guests.

This is being worked on in https://github.com/teemtee/tmt/pull/3135


Mismatching pip installation and website documentation?

Hi Laplace Developers!
I am not entirely sure if I am doing some incorrect installation wise, but as of yesterday when I install your package via pip pip install laplace-torch I get a different version than the one that is currently available both here on GitHub and on your documentation website. I know this because the function RunningNLLMetric() is not available in the old version.
I can fix this for myself, but thought that it was perhaps of interest for you to know.
Many thanks

Yes, currently the main branch is still in a pre-release stage and thus not been uploaded to the pip repo yet. This should be done by the end of this month. We're waiting for all the open pull requests to be merged.
https://github.com/aleximmer/Laplace/milestone/2

For now, please install it from the main branch:
pip install<EMAIL_ADDRESS>

Cool - thanks very much for the swift reply!

Hello again,
I have now tried using installing from the dev branch above, which works fine upon installation (locally), but I run into dependency conflicts when later trying to install from a requirements file generated by running pip freeze. The error I get is the following:
INFO: pip is looking at multiple versions of laplace-torch to determine which version is compatible with other requirements. This could take a while.

The conflict is caused by:
    The user requested asdl 0.1 (from git+https://github.com/kazukiosawa/asdl@011a942b2698b9ec33b0c8c47c96bd49335e5d80)
    laplace-torch 0.1a2 depends on asdl 0.1 (from git+https://github.com/kazukiosawa/asdl)

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

My requirements file contains the following lines after installing from Git:
asdl @ git+https://github.com/kazukiosawa/asdl@011a942b2698b9ec33b0c8c47c96bd49335e5d80
curvlinops-for-pytorch @ git+https://github.com/f-dangel/curvlinops@d509cd6c332cda0e3bbb7b6bd9c5f37a6aa023b1
laplace-torch @ git+https://git@github.com/aleximmer/laplace@d969f659d3cfad250874d29dc9c92484130a6846

My expertise in the environment management area is limited, so it is possible that I am doing something wrong here. I tried alternatively using pip list --format freeze, but this just points directly to the 0.1a2 version of Laplace rather than the Git version.
I am able to find a solution to this myself although hacky, but nevertheless thought that I would make you aware of this issue (although, once again, it is not unlikely it is me causing the error).

Yeah I think it's because of pip... What I usually do is to put the manual installation procedure in the README of my project. E.g. https://github.com/wiseodd/laplace-bayesopt?tab=readme-ov-file#installation


Fix: Allow custom output file names instead of defaulting to test.svg

I fixed the issue https://github.com/BTheDragonMaster/RAIChU/issues/29

Thank you!


Identify and correct problems with augmented_dataset_csv/issues

The ExampleGen component of the TFX pipeline officially supports few data formats for ingestion. As we have decided to go with CSV, this requires a conversion of the contents of augmented_dataset/ from .txt to .csv.
An initial conversion has been made in augmented_dataset_csv/, but beam_dag_runner.py rejects the files which appear to be of the correct format.
It would also be conducive to merge the rows in ground_truth.csv with their respective issues, but this crashes my machine for an unknown reason.

https://www.tensorflow.org/tfx/data_validation/api_docs/python/tfdv/DecodeCSV
https://www.tensorflow.org/api_docs/python/tf/io/decode_csv
https://tools.ietf.org/html/rfc4180#page-2

A decision was made to convert to tfrecord instead of csv. Will reopen if reversed.


[Wearable] [VSCode] Checkbox/radio widget when selected are not filled in

Technical
[Wearable] [VSCode] Checkbox/radio widget when selected are not filled in
Description
Steps to reproduce

Open sampleproject in Design Editor
D&D checkbox widget
Go to Live Preview
Click on the checbox widget

Expected behaviour
Checkbox is marked
Actual behaviour
Checkbox is not marked

W Sample TAU 1.1.4.zip

From I can see the problem occurs due to the fact that the image needed for the checkbox component is not copied to the project directory.
I'll investigate it and try to fix it tomorrow.

I've found the root of this issue.
It's because we have wrong link in tau.circle (which is because we have both tau catalogue that contains tau.circle and tau.circle itself copied alone - without files it's referring to).
It should work without this lone double tau.circle.
We probably should restrict conditions under which we copy libs like tau.circle to projects directory.

BTW I've also fixed error caused by _setIcon in component-element and added several tests while working on this case : )

thanks to #358 paths now works fine :

works fine


exp/orderbook: Improve performance of path finding implementation

I have implemented 3 changes which improve the performance of the path finding implementation:

remove visited map[string]bool and instead scan through the visited []xdr.Asset slice to determine if we have visited a node at some point in the current path of the DFS
abort the dfs if we have already visited all the terminal nodes
abort the dfs if we're at the max path limit before processing all outgoing edges

The first optimization takes advantage of the fact that doing a linear scan over a small list is faster than a map lookup. Since we're limiting the depth of our DFS to at most 5, the visited list will be small enough that iterating over the list is faster than a map lookup.
The second optimization is effective when we are processing path finding requests with just 1 or 2 destination assets. Most of the path finding traffic fits this case. When we have a few destination assets there is no point in continuing the DFS after we have visited the destination assets.
The third optimization is the most impactful one. Assume we have a max path length of 3, this means our path should have no more than three edges, src -> asset1 -> asset2 -> dest.  The dfs function was enforcing the max path length by checking that the length of the visited list  was not greater than maxPathLength. This check was done at the very beginning of the function.
I realized that we should move this check to a later point in the code. Specifically, we should first check if we are at a terminal node and then we can check if the maxPathLength condition. By doing that we eliminate the case where we are at the end of a maxPathLength path. We do not need to traverse the node's outgoing edges because the path is going to exceed maxPathLength.
Here are the benchmarks for these changes:
goos: darwin
goarch: amd64
pkg: github.com/stellar/go/exp/orderbook
cpu: Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
BenchmarkMultipleDestinationAssets
BenchmarkMultipleDestinationAssets-12    	     260	   4426879 ns/op
BenchmarkTestData
BenchmarkTestData-12                     	       9	 119356122 ns/op
PASS

As a basis of comparison, here are the benchmarks for the code without the changes:
goos: darwin
goarch: amd64
pkg: github.com/stellar/go/exp/orderbook
cpu: Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
BenchmarkMultipleDestinationAssets
BenchmarkMultipleDestinationAssets-12    	      21	  55507880 ns/op
BenchmarkTestData
BenchmarkTestData-12                     	       1<PHONE_NUMBER> ns/op
PASS

There is room for improvement but the other ideas I had would affect the accuracy of the results. The nice thing about these improvements is that the output of the path finding implementation remains identical. Once the orderbook becomes sufficiently large that the path finding implementation becomes too slow even with these improvements, we can start looking at changes which sacrifice accuracy for efficiency.

Wow - that's a factor 12 speedup! :tada:

Awesome!!


How add Medium account?

I can't add my medium account, I tried several combinations without success. When I add    /@username   my result is   https://medium.com//@username with double slash before @
When I add @username I had a error
Some ideas? thank you and sorry for my english

Wrap your handle with quotation marks, like '@username' or "@username"


Add local registry to existing test framework

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: yutongz
Assign the PR to them by writing /assign @yutongz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: kimikowang
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: yutongz
Assign the PR to them by writing /assign @yutongz in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kimikowang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
cff041cb95a48ddf5bd718c738a3c08974a22718
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kimikowang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
dd74305298ff5c5928d61ed793e55680df63e0dd
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kimikowang: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
3c697243a7db38fee62859b8548337e13f123ee4
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram This is part of the swat effort. Let's not delete it for now.
@nmittler @ozevren @kimikowang @gargnupur @JimmyCYJ Now we have multiple branches on different things. Shall we create a new swat branch and move all works there? Since different people are working on different things, we should not step on others anyway.

The new PR I am going to submit will make use of local test setup in another branch. I think merging them together would be a good idea.

@ozevren Since most of the changes are not related to the product itself, we can generally merge to master (maybe periodically) as long as they are in reasonable state, and is not enabled until the part is ready. They don't necessarily all land at the same time.


Multiple fields cursor, sorting and repo options

Add support for multi columns cursor (#3 #10) - allow to pass cursor_field: [:column1, :column2].

for multi cursor :after_cursor accepts a map where keys are cursor fields, e.g after_cursor: %{column1: x, column2: y.

Add custom ordering of results (#14) - e.g. order: :desc
Allow to pass Ecto Repo options that are forwarded to Repo.all/2 - e.g. timeout: 100_000 is useful if starting with after_cursor on large tables.
Add validations when invalid options are passed

cursor_field that isn't atom or list of atoms
after_cursor that contains map keys that aren't present in cursor_field (typos)
result doesn't contain cursor_field (invalid select)

Thanks for your PR. I'm looking for a simple library to use in our company project


(wip) remove Traversal: adapt to changes (now it compiles)

Tests don't compile so far. Will either continue next week or give to @mpollmeier to improve upon.

Tests now compile as well, but some still fail (not done yet). The diff is somewhat giant, though.
I'd like to also get rid of the whole joern scripts stuff -- that currently errors out with ammonite telling me that something is wrong, but not what is wrong.
We already removed all scripting nonsense from codescience, any objections to removing it here as well? (the existing scripts would be turned into ordinary scala files).
Now that we can see how it feels to get rid of Traversal, what do you think @fabsx00 @ml86 @mpollmeier ?
The most annoying things are:

.head and .headOption become .next() and .nextOption() on Iterator.
There is a tendency to be clueless about what types one operates on. Hence sprinkled .l on things that are already lists.
There is the annoying issue of conclicting imports (we talked about that at length with @mpollmeier @ml86 ).


Updated FieldCheckboxes component

Description
Refactoring and adding features to the FieldCheckboxes component.

Same usage as before.
New usage with composable components FieldCheckboxesItem and FieldCheckboxesCheckAll :<FieldCheckboxes
  name="colors"
  label="Colors"
  required="You need to check at least one color"
>
  <Stack>
    <FieldCheckboxesCheckAll>All colors</FieldCheckboxesCheckAll>
    <Divider />
    <FieldCheckboxesItem value="red">Red</FieldCheckboxesItem>
    <FieldCheckboxesItem value="green">Green</FieldCheckboxesItem>
    <FieldCheckboxesItem value="blue">Blue</FieldCheckboxesItem>
  </Stack>
</FieldCheckboxes>

You can checkout the associated story file for all usages.
Performances
Value was managed using zustand library due to poor performances caused by re-rendering in a lot a FieldCheckboxesItem.
It also resolves issue #85.

Closed in favor of #93


Add cas_roles to permission scheme

@rsskmr can you add us to the codeship perms, if possible?

@cdaringe FYI: the codeship tests will always fail until we figure out a way to mock our database. It's doable, but we aren't there yet.

@dylancwood roger that!

@rsskmr Why is this PR into release rather than develop?

Looks good. Let me know what you think about using application as the context for application-wide roles, instead of meta.
Also, please create a new PR that will merge into develop, rather than release.

Closing, was in release review mode, me bad


Added TicTacToe game using VanillaJs

Hey there,
I have added a link to a project tutorial for building a TicTacToe game using VanillaJs

Looks fine, but this article only available for paid members.

How about opening that article in an incognito window?


Use of ayon_common

Is there an existing issue for this?

[x] I have searched the existing issues and added correct labels.

Description
Current Behavior
This addon is still using some functionality from ayon-launcher - it is importing ayon_common that is available only there. I am not sure why it is working elsewhere, but when calling certain functions of ayon-core outside of "normal" flow, it crashes because of that. For example in unit tests testing some ayon-core functions.
Expected Behavior
It shouldn't import module that isn't available.
Steps To Reproduce:
Create some unit tests for ayon-core functions. I don't really know when this is getting triggered, but it will fail on import of ayon_common.
Additional context:
No response
Relevant log output:
=============================
No module named 'ayon_common'
=============================
Traceback (most recent call last):
  File "E:\projects\ynput\repos\ayon-core\client\ayon_core\addon\base.py", line 338, in _load_ayon_addons
    mod = __import__(basename, fromlist=("",))
  File "C:\Users\annat\AppData\Local\Ynput\AYON\addons\ayon_third_party_1.1.3-dev.1\ayon_third_party\__init__.py", line 3, in <module>
    from .addon import ThirdPartyDistAddon
  File "C:\Users\annat\AppData\Local\Ynput\AYON\addons\ayon_third_party_1.1.3-dev.1\ayon_third_party\addon.py", line 5, in <module>
    from .utils import (
  File "C:\Users\annat\AppData\Local\Ynput\AYON\addons\ayon_third_party_1.1.3-dev.1\ayon_third_party\utils.py", line 10, in <module>
    from ayon_common import (
ModuleNotFoundError: No module named 'ayon_common'

This should be solved by #14


Would like your email

I am interested in discussing some changes to your code to allow this to work with pingtest.net. This would make the data very useful to VoIP implementations. I would willing to pay you for the changes. I can be reached at chris -a-t- intutivetechnology -d-o-t- com

I think that would be a complete rewrite. This code is based on other code I found at the webz. If pingtest.net has not already been dissected then doing so may be a major undertaking involving packet analysis and such and so on and this is very different from some changes to this code. And I don't even have the time to go and figure out if anyone has already done work on it elsewhere in some other language.


fix: ci lint for mod registry and image

do golangci-lint on registry&image module,  and fixed issues occurred

 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.

should not ignore errors! @justadogistaken


macOS: don't include pugl.mm for external UIs

On macOS: don't include pugl.mm for external UIs

yeah like that, but it is better to keep the OBJS_UI stuff together, you moved them to a place where it doesnt belong IMO.
the old place seemed better, we can just add another ifneq in there

I first had the if in line 92 but moved it below for two reasons:
1.) UI_TYPE is evaluated and modified before line 175,
2.) OBJS_UI is also modified below in line 199
But if you still want I can move the if back to the top at line 92. IMHO both places have their pros and cons.

Thanks, and sorry being so picky.


PasswordlessViewController.swift

Auth.auth().sendSignInLink
show me Error : Value of type 'Auth' has no member 'sendSignInLink'
how to solve this ???

What version of Firebase are you using?


Dont allow service fns without self

This PR is based on #130, so I could check the tests locally as well. Feel free to not pull those in, or tell me and I'll remove them.

This PR fixes #129, as it simply compile-errors when trying to use a first argument that is not either a:

An actual receiver (&self, &mut self, self: Box<val>, etc...)
A reference to itself impl Service { fn foo(instance: &Service) { ..} }

Anything else will fail the equality check and stop compilation.
The error message is not the best, but should give some indication on what went wrong.
Sadly at macro-level there is no way to verify type equality.
There's also IMO quite some unsoundness around lifetimes, but this PR should preserve any previous behaviour (as it only emits a check).

I moved the function into the reference project because that's where all the samples belong.
While I like that there is a compile error now, I'm not a big fan on the ergonomics. There is nothing here telling me the issue is a missing self parameter. Even worse it says expected this to be u32, which makes it sound as if the compiler just didn't see that u32.
8  | impl ServiceIgnoringMethods {
   |      ^^^^^^^^^^^^^^^^^^^^^^ type mismatch resolving `<ServiceIgnoringMethods as TypeEq>::This == u32`
   |
note: expected this to be `u32`
  --> crates\reference_project\src\patterns\services\ignored.rs:27:24
   |
27 |     pub fn test(_test: u32) -> Result<(), Error> {
   |                        ^^^
note: required by a bound in `ignored::_::{closure#0}::assert_type_eq_all`
  --> crates\reference_project\src\patterns\services\ignored.rs:27:24
   |
27 |     pub fn test(_test: u32) -> Result<(), Error> {
   |                        ^^^ required by this bound in `assert_type_eq_all`

Yeah, sadly I don't see a good way to improve this, since you can't really control this error message.
What one could do is make the assert_type_eq_all function's name dependent on the type it checks?
So one could instead read assert_first_argument_compatible_with_ServiceIgnoringMethods or something like that?


chore: limit browser to getting page content

Temporarily remove all tools other than Get Page Contents from from the Browser bundle. This is the core functionality we want to demo.

We decided to leave these tools in since they're working. Closing out.


How i can create theme without bootscore namespace

Before this, I was working with Underscore .
It was a good feature that allowed to create a template and the whole theme was based on the name of the template, this is also in another starter https://them.es/starter-bootstrap/ There is, and it is very important that my client does not realize that I am using a ready-made template called bootscore, because I have problems in selling and convincing the client.

You can just do a complete search using your editor, and replace "bootscore" with whatever.
But adding to that, it's not nice to deceive a client with trying to hide that you may use a starting base for your theme.

Also keep in mind that bootScore is MIT licensed, so copyrights are to be preserved.

https://them.es/starter-bootstrap/ is MIT , i want to create theme like this

Then read my first message. I'm just reminding you that you can't remove all references and copyrights of bootScore, because of the MIT license. The renaming everything is up to you.


HDDS-7227. Implement Changed placement policy interface to allow existing nodes to be specified for Pipeline Placement Policy.

What changes were proposed in this pull request?
Support pipeline placement policy algorithm to adapt usedNodes. When usedNodes are passed in pipeline placement policy consider usedNodes to determine anchor node and based on that return new requested nodes based on rack aware policy.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7227
How was this patch tested?
Added Unit tests.

@sodonnel, Can you please help to review.


Add KV Certficates Async Client Unit Test

Adds missing tests of KV Certificates Async Client.
This will also unblock Sync stack integration.

API change check
API changes are not detected in this pull request.


UIMA-6367: JCas cover annotation created in PEAR context replaced during index operations

fixed trampoline map
added test that checks annotation instance in mocked pear context

It is great that we have the test and the fix, but the test name is not self-explanatory and the test has no JavaDoc. We should add a properly explanatory JavaDoc to this test, otherwise if it at some point breaks, it takes ages until somebody understands the scenario that is tested here.


Restructure in preparation for merging in NSLS-II content

This PR rearranges the documents into the structure planned in #137.
No substantive changes are made to content.

Self-merging with an in-person nod from @henryiii


Eslint Prep Work

tslint is being deprecated.  There is some tooling to support the conversion to eslint and some typescript specific linting.  This is an initial pass to update the code for the low hanging fruit.  Best  reviewed at the commit level.  There are very few logic changes.
typescript-eslint has more info about converting.  Initial playing with the conversion required turning indent rules off - the errors and warnings were not consistent.  The typescript-eslint team acknowledged in an issue they do not use the indent rules and use prettier instead.

Thanks for the review.  Please go ahead with the merge.  I will work on my git skills for rolling back, updating and replaying history on my own time.  I had initially done all the changes at once and then broke them up into commits.  Maybe not the best strategy but there were a number of linting messages to get through.  The use of '+' caused numerous indent issues until I turned it off.

I had initially done all the changes at once and then broke them up into commits.

That's exactly how I perform similar tasks...
I'll just merge it, the shape of the commits is not all that important ;-)