No odom messages being published

Im following the turtlebo3-UE tutorial but I get an error in rviz:
[controller_server-6] [INFO] [1660832507.617404925] [local_costmap.local_costmap]: Timed out waiting for transform from base_link to odom to become available, tf error: Invalid frame ID "odom" passed to canTransform argument target_frame - frame does not exist
When I check ros2 topic echo /odom, no messages are being published. I checked the turtlebot actor in UE and see there is no moveComponent. Do I need to manually add in a move component? this step is not included in the readme. Any help is appreciated

Same issue here.

Thanks for testing and sorry for not responding.
Please try devel branch

I added examples https://rclue.readthedocs.io/en/latest/examples.html please try it first.
there is a jammy branch which works with Ubuntu 22.04

close issue since no activity


When providing fetcher: null , middleware is defaulting it to undefined

Bug report
We're building a custom middleware were we interact with fetcher.
We check if fetcher === null, but we receive fetcher as undefined inside middleware callback.
 const myCustomMiddleware = (useSwrNext) =>
  (key, fetcher, config) => {
    const extendedFetcher =  fetcher === null ? null : myFetcher
    ....

The issue is that fetcher comes undefined even if we set it as null in our useSWRImmutable("myKey", null, options)
Types doesn't allow us to define fetcher as undefined in useSWRInmutable.
Expected Behavior
Middleware returns fetcher as null when we set it null in useSWRInmutable
SWR version.
"swr": "^1.3.0"

@jquintozamora Thank you! I've created a PR to fix this.


⚠ Secret Website 3 has degraded performance

In 855b50f, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2472 ms

Resolved: Secret Website 3 performance has improved in 683cdaa.


Lab 9 RLS

Lab 9 RLS
Task: Configure row-level security (optional)
Step: 5
Description of issue
Step: In the Members box, commence entering Salespeople_Australia.
Issue:  Salespeople_Australia is not exist in the ADD of cloudslise.

Hi Alex - the reason the task is optional is because it depends on the creation of a Salespeople_Australia security group. If you refer to the text directly above those instructions, you'll see:
"This task relies on the existence of a Salespeople_Australia security group in the tenant you are working in. This security group does NOT automatically exist in the tenant. If you have permissions on your tenant, you can follow the steps below. If you are using a tenant provided to you in training, you will not have the appropriate permissions to create security groups. Please read through the tasks, but note that you will not be able to complete them in the absence of the existence of the security group. After reading through, proceed to the Clean Up task."


Fix resource groups tests

With the new boto3 version the handling for resource groups changed.

Coverage decreased (-0.05%) to 94.292% when pulling 731951fe96f3bcd87fb8dc88ffa2060b6af65175 on gruebel:fix-resource-groups-tests into 50d71eccbee12ce5cf2c1ecf61bfcbd60317976d on spulec:master.

Hi @bblommers,
could you rebuild this PR, it is a fix which is needed for the previous PRs. I don't understand why the batch test is failing, I had the same problem in the past. Very mysterious!


Adds a draft proposal for returning contents of dict without meta_data

this is just a draft, and probably will need to have it changed slightly for python3. Tell me if this is what you are looking for @caphrim007

All I was doing in the meantime was running to_dict() and then doing a pop('_meta_data', None). I think the function needs a name like attrs or something like that. no_meta, while true, is also not necessarily the purpose of the method.
The use case is that I was to get all of the things that would be considered attributes of the resource if I were to do a normal curl of the endpoint.

I think properties is fine

this will fail for 2.6 python as we use dict comprehension. Since we are about to drop python 2.6 support from Travis, I am going to merge this regardless


The argument type 'DioCacheManager' can't be assigned to the parameter type 'Interceptor'.

I'm using  dio_http_cache: ^0.2.8 with dio: 3.0.9

Found that adding DioCacheManager(...).interceptor works. The docs need to be updated

Sorry, I didn't scroll over to see it

Repurposing this issue

Hi, thanks for your issue;
I'm sorry for that, And I have updated README.
baseUrl: it’s optional; If you don't have set baseUrl in CacheConfig, when you call deleteCache, you need provide full path like "https://www.google.com/search?q=hello", but not just "search?q=hello".


What's the status of kotlin class generation?

What's the status of Kotlin class generation? The README file says it's subject to change. When is it planned to release the final/stable version of that feature?
This issue should be labeled with the Question label.

It's should be ok, we keep fixing some small issues but the overall the structure of generated models should be stable now.

I guess we can close this.


Update react-scripts to 4.0.1

This updates react-scripts from 4.0.0 to 4.0.1
Includes and closes #212

Thanks for approving @SheldonCheng ;) Branch is rebased and up-to-date. Unfortunately CI is failing for PR head branches not contained in the source repo since the DOCKER_USER and DOCKER_PASSWORD secrets are unavailable.


Bump Speech to 0.25.1 [ci skip]

Release notes coming momentarily.

LGTM

Do not force long-running requests to be LINEAR16 only in Speech. (#3291)
Add new encodings. (#3292)
Do not validate sample rate. (#3290)


Update README.md

Running the example is not clear enough. Also you have to look through code in order to enter the credentials

Thanks!


Fix raster join docs

didn't correctly update this from PR #495

s22s branch circle ci: https://2998-174014757-gh.circle-artifacts.com/0/rf-site/raster-join.html


[Docs]: Update documentation for navigateTo to aling with latest changes

Is there an existing issue for this?

[X] I have searched the existing issues

Documentation Link
https://docs.appsmith.com/reference/appsmith-framework/widget-actions/navigate-to
Describe the problem
Following PR changes UI and functionality of navigateTo and this needs to be updated on documentation
https://github.com/appsmithorg/appsmith/pull/14856
Describe the improvement
UI an functional changes improvements for navigateTo()

@ramsaptami could you please summarize the changes needed in the navigateTo docs? cc @rimildeyjsr

@Pranay105 Attaching a screenshot of how the navigateTo looks right now

But having said that, we have a project where the action selector (the entire bit under Events on widgets) is changing so we can overhaul all of them together


Emulate text entry in FlutterDriver

This should be a good-enough fix for #6737
/cc @goderbauer

Will land on green.


Add build doc to readme.

Fixes https://github.com/dotnet/MQTTnet/issues/1362.

Please move this to the Wiki instead. In my opinion this is nothing very important which should be present on the first page which is shown to the users.

It's documented under https://github.com/dotnet/MQTTnet/wiki/Build.


UNET model input precision conversion fix

This PR fixes an issue reported both here https://stackoverflow.com/questions/75213370/stable-diffusion-with-openvino-failed-to-set-input-blob-with-precision-i64-if and here https://github.com/bes-dev/stable_diffusion.openvino/issues/118
The UNET model expects f64 data type for the input named "t" while the values generated by tqdm are 64-bit ints.

:+1:


Kernel Version Not Supported

Following error produced on a Samsung Chromebook 3.  Trying to install kernel headers on chroot (trusty) to support installation of virtualbox.
sudo sh setup-headers.sh

Your kernel version 3.18.0-13436-g8b496ef with architecture amd64 is not supported

How can I obtain support for kernel version 3.18.0-13436-g8b496ef?

First, vitualbox modules are currently broken on chromebooks see https://github.com/divx118/crouton-packages/issues/45 Also if that is fixed it will be a pain to keep support for the newer 3.18.0 kernels since they use a subversion now which with each chrome update is changed. I would suggest enabling USB boot and install a legacy linux version on a fast USB stick or SD-card to dual boot into. Then you can have your own kernel.


Gradle optimization failure on gradle 7.2 and Kover 0.4.0

Getting tons of
Execution optimizations have been disabled for task ':kmath-ejml:test' to ensure correctness due to the following reasons:
  - Property 'jvmArgumentProviders.$0' was implemented by the Java lambda 'kotlinx.kover.KoverPlugin$$Lambda$2209/0x00000008426af840'. Reason: Using Java lambdas is not supported as task inputs. Please refer to https://docs.gradle.org/7.2/userguide/validation_problems.html#implementation_unknown for more details about this problem.

Actually, it is raised in each jvm test source set.

Can confirm that the warning is gone in 0.4.1.

Seems to be working without warning or slowdowns on 0.4.1


[HOLD] Add druid logs

Add druid logs

Hold, wait for parser to be merged.

Superseded by https://github.com/DataDog/integrations-core/pull/4561


zh:update page of about_notes

update page of about_notes

[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: fleeto
If they are not already assigned, you can assign the PR to them by writing /assign @fleeto 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:

content_zh/OWNERS

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

Hi @liangyuanpeng. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

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.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: gyliu513
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:

content_zh/OWNERS [gyliu513]

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


Custom API llama-cpp-python complains about missing OpenAI API Key

I followed the instructions here, https://github.com/abetlen/llama-cpp-python/tree/main/docker#open-llama-in-a-box, and I can see it running on port 8000 but when I tried following these steps, https://github.com/madox2/vim-ai/wiki/Custom-APIs#llama-cpp-python, I get the following error:

Missing OpenAI API key

Hi, recently has been added new config option enable_auth. I believe that setting it to 0 would resolve your issue

That config option is already added.

Well, that option is supposed to disable api key check, if not then it's a bug. Could you share your configuration?

It's literally just this:
cat ~/.vimrc
let g:vim_ai_chat = {
\  "options": {
\    "endpoint_url": "http://localhost:8000/v1/chat/completions",
\    "enable_auth": 0,
\  },
\}

I have checked once again and it is working as expected in my environment. Note that your configuration is related just to AIChat command. AI and AIEdit commands need to be configured separately. Please also verify that you are using latest version of the plugin

Do vim_ai_complete and vim_ai_edit work with those options for a custom endpoint work too ? I got AIChat to work with ollama but not the others two ... (maybe ollama is missing some endpoints but i can't say which you requiere from the docs).

Do vim_ai_complete and vim_ai_edit work with those options for a custom endpoint work too ? I got AIChat to work with ollama but not the others two ... (maybe ollama is missing some endpoints but i can't say which you requiere from the docs).

If ollama does not support completions, you can configure AI, AIEdit with chat engine

Closing as this works as intended


The implementation of TextMapPropagator#inject and TextMapSetter#unsafeSet

Currently, the TextMapPropagator defines the inject method as:
trait TextMapPropagator[F[_]] {
  def inject[A: TextMapSetter](ctx: Vault, carrier: A): F[Unit]
}

How are we going to make it work in an actual application?
For example, we would like to propagate span details downstream via http4s Response.
def propagateSpanMiddleware[F[_]: TextMapPropagator: Local[*[_], Vault]](service: HttpRoutes[F]): HttpRoutes[F] = Kleisli { (req: Request[F]) =>
  service(req).flatMap { response: Response =>
    Local[F, Vault].ask[Vault].flatMap { tracingContext =>
      TextMapPropagator[F].inject(tracingContext, resp) // well, it returns F[Unit], what now?
    }
  }
}

implicit val textMapSetter: TextMapSetter[Response] =
  new TextMapSetter[HttpRequest] {
    def unsafeSet(carrier: Response, key: String, value: String): Unit =
      ???
  }

The current behavior is forced by the API contracts of Java's OpenTelemetry implementation.
The example above can be hacked as:
def propagateSpanMiddleware[F[_]: TextMapPropagator: Local[*[_], Vault]](service: HttpRoutes[F]): HttpRoutes[F] =
  Kleisli { (req: Request[F]) =>
    service(req).flatMap { response: Response =>
      Local[F, Vault].ask[Vault].flatMap { tracingContext =>
        val headers: mutable.Map[String, String] = Map.empty
        TextMapPropagator[F].inject(tracingContext, headers).map(_ => httpRequest.addHeaders(headers))
      }
    }
  }

implicit val mutableMapSetter: TextMapSetter[mutable.Map[String, String]] =
  new TextMapSetter[mutable.Map[String, String]] {
    def unsafeSet(
        carrier: mutable.Map[String, String],
        key: String,
        value: String
    ): Unit =
      carrier.update(key, value)
  }

It works, but we can offer a graceful alternative by incapsulating operations with mutable states.

Can we ease the pain with the utility method?
trait Tracer[F[_]] {
  def propagationContext: F[Map[String, String]] = {
    val mutableMap = scala.collection.mutable.Map.empty[String, String]
    for {
      current <- Local[F, Vault].ask
      _ <- propagators.textMapPropagator.inject(current, mutableMap)
    } yield mutableMap.toMap
  }
}

Hm, we can simplify it even more.
trait CarrierTextMapSetter[A] {
  def set(carrier: A, context: Map[String, String]): A
}

implicit val forHttpResponse: CarrierTextMapSetter[Response] =
  new CarrierTextMapSetter[Response] {
    def set(carrier: Response, context: Map[String, String]): Response =
       carrier.addHeaders(context)
  }

trait Tracer[F[_]] {
  def propagate[C: CarrierTextMapSetter](carrier: C): F[C] = {
    val mutableMap = scala.collection.mutable.Map.empty[String, String]
    for {
      current <- Local[F, Vault].ask
      _ <- propagators.textMapPropagator.inject(current, mutableMap)
    } yield CarrierTextMapSetter[C].set(carrier, mutableMap.toMap)
  }
}

And example can be:
def propagateSpanMiddleware[F[_]: Tracer](service: HttpRoutes[F]): HttpRoutes[F] = Kleisli { (req: Request[F]) =>
  service(req).flatMap { response: Response =>
    Tracer[F].propagate(response)
  }
}

The question is the following: do we really need to introduce a high-level abstraction, or consumers (http4s, etc) can do the magic on their side?

This is one of the few conflicts between the specification and functional programming, and it hurts:

Carriers used at Inject are expected to be mutable.

I like @iRevive's helper, except the F in the return type is a leaky implementation detail of being forced to go through the effectful inject.  I wonder whether we could justify an unsafe method on TextMapPropagator, because Tracer.propagate is pure from the outside.

@lacarvalho91 where do you get the Local[F, Vault] from when you make an instance of OtelPropagatorBackend?

I had hoped that when I do ask it would get the current context at that point but I'm probably not understanding what's actually happening very well. I had a look at the tracer and it does seem that it does a very similar thing, ends up doing an Ask

@lacarvalho91
I took your code and it seems to be working:
  def run: IO[Unit] =
    IOLocal(Vault.empty)
      .flatMap { implicit ioLocal: IOLocal[Vault] =>
        val otel4s = OtelJava.local[IO](GlobalOpenTelemetry.get())

        otel4s.tracerProvider.tracer("Http4sExample").get.flatMap { implicit tracer: Tracer[IO] =>
          val armeria = ArmeriaCatsBackend.usingDefaultClient[IO]()
          val backend = new OtelPropagatorBackend[IO, Any](
            armeria,
            otel4s.propagators.textMapPropagator
          )

          Tracer[IO].rootSpan("test").use { _ =>
            backend
              .send(
                basicRequest
                  .get(uri"https://httpbin.org/get")
                  .response(asStringAlways)
              )
              .flatMap(response => IO.println(response.body))
          }
        }
      }

And I get:
[info] headers from Vault ctx: HashMap(X-B3-SpanId -> 603771c3f54dbb8e, X-B3-Sampled -> 1, X-B3-TraceId -> c77b12ac34f5c6441b1a954c9734fc0d)

Your application may lose tracing details somewhere in the middle, e.g. by using Tracer[IO].noopScope().

Hmm OK interesting, thanks for doing that. What could cause span details to be lost? I'm not explicitly calling noopScope() anywhere

I've pretty much copied what you did:
object Test extends IOApp.Simple {
  implicit def localForIoLocal[F[_] : MonadCancelThrow : LiftIO, E](implicit
                                                                    ioLocal: IOLocal[E]
                                                                   ): Local[F, E] =
    new Local[F, E] {
      def applicative =
        Applicative[F]

      def ask[E2 >: E] =
        Functor[F].widen[E, E2](ioLocal.get.to[F])

      def local[A](fa: F[A])(f: E => E): F[A] =
        MonadCancelThrow[F].bracket(ioLocal.modify(e => (f(e), e)).to[F])(_ =>
          fa
        )(ioLocal.set(_).to[F])
    }

  override def run: cats.effect.IO[Unit] = {
    IOLocal(Vault.empty).flatMap {
      implicit ioLocal: IOLocal[Vault] =>
        val otel4s = OtelJava.local[cats.effect.IO](GlobalOpenTelemetry.get())
        for {
          tracer <- otel4s.tracerProvider.tracer("my-app").get
          armeria = ArmeriaCatsBackend.usingDefaultClient[cats.effect.IO]()
          backend = new OtelPropagatorBackend(armeria, otel4s.propagators.textMapPropagator)
          _ <- tracer.rootSpan("test").use { _ =>
            backend.send(
              basicRequest
                .get(uri"https://httpbin.org/get")
                .response(asStringAlways)
            )
          }
        } yield ()
    }
  }
}

class OtelPropagatorBackend[F[_], P](delegate: SttpBackend[F, P], textMapPropagator: TextMapPropagator[F])(implicit
    L: Local[F, Vault]
) extends DelegateSttpBackend[F, P](delegate) {

  override def send[T, R >: P & capabilities.Effect[F]](request: Request[T, R]): F[Response[T]] = {
    responseMonad.flatMap(L.ask[Vault]) { vault =>
      val headers: mutable.Map[String, String] = mutable.Map.empty
      responseMonad.flatMap(textMapPropagator.inject(vault, headers)) { _ =>
        System.out.println(s"headers from Vault ctx: $headers")
        delegate.send(request.headers(headers.toMap))
      }
    }
  }
}

object OtelPropagatorBackend {
  implicit val mutableMapSetter: TextMapSetter[mutable.Map[String, String]] =
    new TextMapSetter[mutable.Map[String, String]] {
      def unsafeSet(carrier: mutable.Map[String, String], key: String, value: String): Unit = carrier.update(key, value)
    }
}

I'm still getting: headers from Vault: HashMap(). No idea what I'm doing wrong :cry:

got it working...it was a problem with how the auto configure and the java agent were configured, that was tough to debug. thanks for looking into it!

It's frustrating the whole thing depends on the environment variables, and it's really hard to get the current configuration state.
I wonder whether it's possible to make a simple report upon instantiation of Otel4s. For example, printing some basic info: which propagators are configured (e.g. b3 and w3c), whether we use noop tracer, etc.

I did not realize about the implicit IOLocal and constructed it like the example, and didn't get the vault filling. This post was super useful, but also we should see if we can make that more clear in the docs.

What was misconfigured?  There is a reference that we should highlight.  The easy one to forget is otel.java.global-autoconfigure.enabled, but that logs a warning.

@rossabaker I thought it was because I didn't have the Java agent enabled, but turns out that it is definitely correct to not have it enabled (it was messing with gRPC at least). So I'm not really sure what the culprit was, I got into a bit of a hole trying things. I've got a feeling that I was inheriting some otel related SBT config from a parent module that was messing things up but :shrug: .
The main problem though is that I think it is easy to assume that the problem is with the code, so I spent a lot of time hacking the code thinking that was the problem when it wasn't that at all.
What I'm working on is a demo showing otel4s trace context propagation In the following journey:
http4s server -> STTP client
                      -> fs2 gRPC client -> fs2 gRPC server

Which I've got working now...but I had to do some craziness on the gRPC server side when doing tracer.joinOrRoot because gRPC header keys are lowercase but it seems like the B3 propagator is case sensitive :sob:, which of course isn't an otel4s problem:
    val rename = Map(
      "x-b3-traceid" -> "X-B3-TraceId",
      "x-b3-spanid"  -> "X-B3-SpanId",
      "x-b3-sampled" -> "X-B3-Sampled"
    )

What are your thoughts on getting all this added to the examples? (excluding the workaround for the case sensitivity problem...I'll work out how to resolve that first)

Examples would be great, though in the build, we should (with a comment) partition dependencies that exist only in examples from those we need to maintain compatibility for.  We wouldn't want to couple our repo with fs2-grpc dependencies, but we can certainly show off how it works.

We wouldn't want to couple our repo with fs2-grpc dependencies, but we can certainly show off how it works.

Note: anything we add as dependency here, even for an example, could create a nuisance cycle if that dependency decides to develop a first-class integration with otel4s.

With teeter-tottering Scala Steward updates?  It shouldn't be a cycle on any classpath, and if anything did a first-class integration, I'd say we could remove it from here and have the examples over there.  Might not work if there was a comprehensive example, though...

should this be considered fixed by #182 ?


image pull failed

Problem:
Warning  Failed     16m (x2 over 16m)   kubelet            Failed to pull image "ack-agility-registry.cn-shanghai.cr.aliyuncs.com/ecp_builder/open-local:v0.7.1": rpc error: code = NotFound desc = failed to pull and unpack image "ack-agility-registry.cn-shanghai.cr.aliyuncs.com/ecp_builder/open-local:v0.7.1": failed to resolve reference "ack-agility-registry.cn-shanghai.cr.aliyuncs.com/ecp_builder/open-local:v0.7.1": ack-agility-registry.cn-shanghai.cr.aliyuncs.com/ecp_builder/open-local:v0.7.1: not found

me too

The image service should be recovered now, sorry for the inconvience

hello， @peter-wangxu
Unable to access ack-agility-registry.cn-shanghai.cr.aliyuncs.com.
is the  image service down again?

Please use latest version which use docker hub image

@peter-wangxu  thank you!! the main branch is okay 🎉🎉
Have you considered adding a tag for the latest version? I noticed that the latest release is still 0.7.1, so I encountered the image issue while using 0.7.1


Current layer is wrong in HASS while correct in Fluidd

Version of the moonraker integration
1.3.7
Describe your setup
Elegoo Neptune 4 Pro with Klipper v0.10.0-530 (cannot update) and Fluidd v1.30.5
Describe the bug
The values of the sensor "current layer" is always wrong. Usually by 5-6 layers but not always.
The correct value is visible in Fluidd:

but the sensor value is really off:

my plan is to create an HASS automation to be notified when X layer is reached on specific prints but I can't do that if the most important value is wrong
Moonraker Logs
There are no errors in the HASS log related to this

I would need a copy of the debug log during a print to troubleshoot.
thanks
https://moonraker-home-assistant.readthedocs.io/en/latest/support/logs.html#get-a-copy-of-your-ha-logs

its happening on EVERY print, its always 5-6 layers wrong. why don't the addon just read from the same place as fluidd?

I don't know where Fluidd reads it
Both this integration and FLuidd reads what is available out of the Moonraker API, and make some calculation to interpret the current layer.

if it helps, from what i see fluidd current layer matches what i see in cura

In your case you don't have current_layer this cause the integration to guess the layer by manually calculating it. Now there is a chance that Fluidd calculate it differently.
Based on the Moonraker Documentation, you can add the current_layer

info: This is a dict containing information about the print provided by the slicer. Currently this is limited to the total_layer and current_layer values. Note that these values are set by the SET_PRINT_STATS_INFO gcode command. It is necessary to configure the slicer to include this command in the print. SET_PRINT_STATS_INFO TOTAL_LAYER=total_layer_count should be called in the slicer's "start gcode" to initalize the total layer count. SET_PRINT_STATS_INFO CURRENT_LAYER=current_layer should be called in the slicer's "on layer change" gcode. The user must substitute the total_layer_count and current_layer with the appropriate "placeholder syntax" for the slicer.

i'm not reslicing all my files for this. not to mention there is no event for "on layer change" in cura
if fluidd calculates it correctly, this addon must be able to as well. that's why i opened this issue

Looks,
Others don't have this problem, it calculate the right value for me and others,
I won't be spending time on this

"works for me, will close"


Load translations from Laravel

This allows translations to be overridden by the developer if necessary.
This is a companion to #657 but is necessary for overloading translations to work correctly.

Hi @austintoddj,
So, since we are just using the config('app.locale') to pass the locale (I didn't catch this earlier), we don't need to supply the trans function with that locale. It is automatically provided the current locale when it is instantiated in the translations service provider.
That's the only place I differ from your requested change. 😃

Good to go. 😁


bug: Documentation

It's not very clear from the documentation what to do for Android, for instance if we don't want to have the user click on "install Google barcode", but do it automatically on the first scan:

shoudl we call isGoogleBarcodeScannerModuleAvailable each time or only the first time?
in this case, how do we know if the plugin was already successfully installed?

It's not very clear from the documentation what to do for Android, for instance if we don't want to have the user click on "install Google barcode", but do it automatically on the first scan:

This demo app is only for developers to test the individual plugin APIs and is not meant for the end user.
See also https://github.com/capawesome-team/capacitor-mlkit/discussions/84#discussioncomment-7249913.

shoudl we call isGoogleBarcodeScannerModuleAvailable each time or only the first time?

You should call it each time.

in this case, how do we know if the plugin was already successfully installed?

If isGoogleBarcodeScannerModuleAvailable(...) returns { available: true }, then the module was successfully installed.

Okay! That wasn’t immediately clear to me 😅I thought is available meant « is compatible with my phone and I can go on and download it », and not « is already installed »Perhaps renaming it to is installed would clarify?On Nov 2, 2023, at 11:36, Robin Genz @.***> wrote:
It's not very clear from the documentation what to do for Android, for instance if we don't want to have the user click on "install Google barcode", but do it automatically on the first scan:
This demo app is only for developers to test the individual plugin APIs and is not meant for the end user.
See also capawesome-team/capacitor-mlkit#84 (comment).
shoudl we call isGoogleBarcodeScannerModuleAvailable each time or only the first time?
You should call it each time.
in this case, how do we know if the plugin was already successfully installed?
If isGoogleBarcodeScannerModuleAvailable(...) returns { available: true }, then the module was successfully installed.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Perhaps renaming it to is installed would clarify

Yes, I will probably do that in the next major release.
In this case i am closing this issue.


spelling error

"the" was "hte". corrected.

@hobpoblin : Thanks for your contribution! The author, @rkarlin, has been notified to review your proposed change.

Merge typo fix


update NSUUID.UUIDString to uuidString to match Darwin version

as of Xcode 8.

Thanks!

@swift-ci please test


Module 'OHHTTPStubs' has no member named 'removeAllStubs' in 9.0.0

I had updated the OHHTTPStubs to 9.0.0,
Following the official document as below,
"remove any stubs you installed after each test — to avoid those stubs to still be installed when executing the next Test Case — by calling [HTTPStubs removeAllStubs] in your tearDown method."
but when i calling OHHTTPStubs.removeAllStubs()
it appear Module 'OHHTTPStubs' has no member named 'removeAllStubs',
which function should I write or instead,
thanks for any comment in advance

i found it rename to HTTPStubs
so call HTTPStubs.removeAllStubs() would be solved it.

@AliSoftware Could you please rename it in the documentation?


Fix Swagger UI issues for JsonTypeDefinitionBeaconRestApi

To allow the removal of the old api logic, we need to be able to instantiate swagger-ui and the swagger-docs endpoint using the new typedef based api.
Validator-api already has swagger-docs being published, but it's not loading a swagger-ui. This is probably the simpler of the 2 scenarios, as it's already a completely typedef based solution using the RestApiBuilder.
RestApiBuilder.openApiDocsEnabled() currently adds swagger-docs endpoint to the validator-api, so it's just the swagger-ui that's missing.

@zilm13 can you spend a bit of time (no more than a day) investigating this and writing up an idea of how we might solve this problem? It's possible that webjars may be the solution, but I haven't really dug into it.
If we can get an idea of the amount of work involved, we can go from there... I'll have a look over whatever you are thinking tomorrow if that works...

one thing that may not have been clear initially - javalin has a plugin, but we want to use our own webjar and manage it ourselves so that it's not coupled to the jenkins version we use...


[RaisedButton] Fix alignment between text and icons

Align text inside the button and also with the icon, if any.

Do you have an example where this is useful (It will help us understand the use case)?
Thanks

Yes is a visual fix:

I'm wondering, would removing the other.hasOwnProperty('href') here https://github.com/callemall/material-ui/blob/master/src/enhanced-button.jsx#L297 fix it?

Not sure if I understood the code but I think that line is not working because that button had an href but the vertical align is not in the css

I had checked better the code and that line apply vertical align to the   surrounding a div with a div and then the span of text and the one for  icon , so is not going to fix the issue we are talking but need to be there to maintain the sibling elements aligned. If removed will break the line

That sounds like a good idea :+1:

@octavioamu Thanks.

Happy to help, thanks you @oliviertassinari

Could you fix https://github.com/callemall/material-ui/pull/3364 so we can merge this one too?


Sails-mysql migrate: 'drop' doesn`t work

models: { migrate: 'drop' } leads to this error
[Error (E_UNKNOWN) Encountered an unexpected error] Details:  Error: ER_ROW_IS_REFERENCED: Cannot delete or update a parent row: a foreign key constraint fails

Error (E_UNKNOWN) :: Encountered an unexpected error
Error (E_UNKNOWN) :: Encountered an unexpected error
: ER_ROW_IS_REFERENCED: Cannot delete or update a parent row: a foreign key constraint fails
    at Query.Sequence._packetToError (node_modules/sails-mysql/node_modules/mysql/lib/protocol/sequences/Sequence.js:48:14)
    at Query.ErrorPacket (node_modules/sails-mysql/node_modules/mysql/lib/protocol/sequences/Query.js:83:18)
    at Protocol._parsePacket (node_modules/sails-mysql/node_modules/mysql/lib/protocol/Protocol.js:280:23)
    at Parser.write (node_modules/sails-mysql/node_modules/mysql/lib/protocol/Parser.js:73:12)
    at Protocol.write (node_modules/sails-mysql/node_modules/mysql/lib/protocol/Protocol.js:39:16)
    at Socket.<anonymous> (node_modules/sails-mysql/node_modules/mysql/lib/Connection.js:96:28)
    at readableAddChunk (_stream_readable.js:177:18)
    at Socket.Readable.push (_stream_readable.js:135:10)
    at TCP.onread (net.js:542:20)
    --------------------
    at Protocol._enqueue (node_modules/sails-mysql/node_modules/mysql/lib/protocol/Protocol.js:141:48)
    at PoolConnection.query (node_modules/sails-mysql/node_modules/mysql/lib/Connection.js:201:25)
    at dropTable (node_modules/sails-mysql/lib/adapter.js:275:22)
    at node_modules/sails-mysql/lib/adapter.js:287:11
    at node_modules/sails-mysql/node_modules/async/lib/async.js:52:16
    at iterate (node_modules/sails-mysql/node_modules/async/lib/async.js:260:24)
    at Object.async.forEachOfSeries.async.eachOfSeries (node_modules/sails-mysql/node_modules/async/lib/async.js:281:9)
    at Object.async.forEachSeries.async.eachSeries (node_modules/sails-mysql/node_modules/async/lib/async.js:214:22)
    at __DROP__ (node_modules/sails-mysql/lib/adapter.js:285:15)
    at afterwards (node_modules/sails-mysql/lib/connections/spawn.js:84:5)
    at node_modules/sails-mysql/lib/connections/spawn.js:40:7
    at Handshake.onConnect (node_modules/sails-mysql/node_modules/mysql/lib/Pool.js:59:7)
    at Handshake.Sequence.end (node_modules/sails-mysql/node_modules/mysql/lib/protocol/sequences/Sequence.js:96:24)
    at Handshake.Sequence.OkPacket (node_modules/sails-mysql/node_modules/mysql/lib/protocol/sequences/Sequence.js:105:8)
    at Protocol._parsePacket (node_modules/sails-mysql/node_modules/mysql/lib/protocol/Protocol.js:280:23)
    at Parser.write (node_modules/sails-mysql/node_modules/mysql/lib/protocol/Parser.js:73:12)
    at Protocol.write (node_modules/sails-mysql/node_modules/mysql/lib/protocol/Protocol.js:39:16)
    at Socket.<anonymous> (node_modules/sails-mysql/node_modules/mysql/lib/Connection.js:96:28)
    at readableAddChunk (_stream_readable.js:177:18)
    at Socket.Readable.push (_stream_readable.js:135:10)
    at TCP.onread (net.js:542:20)

BTW how can I get normal error message and trace?

Hello,
You're probably applying this to an existing database. The error message is really clear.

ER_ROW_IS_REFERENCED: Cannot delete or update a parent row: a foreign key constraint fails

Sails doesn't use foreign key constraints. So sails probably didn't create the schema.

Good answer, @RWOverdijk 👍
@Crusader4Christ it'd be possible to improve the experience here by making some enhancements to the drop method in the MySQL adapter  (e.g. given ER_ROW_IS_REFERENCED, it wait a few hundred ms and attempt to retry once before failing).  But realistically, that gets into the territory of magic that I'm not quite comfortable with.  To really make this awesome, I think we'd need to augment describe so that it returns information about physical-layer FK constraints.  My intuition there is that doing that would be a bit too much to live in core, but let me know if any other way jumps out.


add new editing API, suitable for modifying several nodes at once

r? @viorina

bors delegate=viorina

bors r+


Media Repo did not follow SRV records

jki.re has a SRV record:
(env) ➜ dig -t SRV _matrix._tcp.jki.re

; <<>> DiG 9.11.5-P4-5.1-Debian <<>> -t SRV _matrix._tcp.jki.re
;; global options: +cmd
;; Got answer:
;; ->>HEADER<<- opcode: QUERY, status: NOERROR, id: 39016
;; flags: qr rd ra; QUERY: 1, ANSWER: 1, AUTHORITY: 0, ADDITIONAL: 1

;; OPT PSEUDOSECTION:
; EDNS: version: 0, flags:; udp: 512
;; QUESTION SECTION:
;_matrix._tcp.jki.re.           IN      SRV

;; ANSWER SECTION:
_matrix._tcp.jki.re.    299     IN      SRV     10 5 8080 jki.re.

;; Query time: 115 msec
;; SERVER: <IP_ADDRESS>#53(<IP_ADDRESS>)
;; WHEN: Wed Jan 08 17:30:17 GMT 2020
;; MSG SIZE  rcvd: 74

But matrix-media-repo doesn't seem to have followed it:
Jan 08 18:13:27 synapse media_repo[1397]: INFO[2020-01-08 17:13:27.648 Z] Host is valid - processing request            contentLength=0 contentType="" host=matrix.amorgan.xyz method=GET queryString="" remoteAddr=<IP_ADDRESS> requestId=REQ-2151 resource=/_matrix/med
Jan 08 18:13:27 synapse media_repo[1397]: INFO[2020-01-08 17:13:27.649 Z] Getting media record from database            allowRemote=true contentLength=0 contentType="" filename="" host=matrix.amorgan.xyz mediaId=tvsXlXaChyGvbqUZnSXxNXgs method=GET queryString="" remo
Jan 08 18:13:27 synapse media_repo[1397]: INFO[2020-01-08 17:13:27.654 Z] Downloading remote media                      nocontext=true worker_blockForMedia=true worker_requestId="remote_download:jki.re_tvsXlXaChyGvbqUZnSXxNXgs" worker_requestMediaId=tvsXlXaChyGvbqUZn
Jan 08 18:13:27 synapse media_repo[1397]: INFO[2020-01-08 17:13:27.654 Z] Getting server API URL for jki.re
Jan 08 18:13:27 synapse media_repo[1397]: INFO[2020-01-08 17:13:27.656 Z] Server API URL for jki.re is https://jki.re:8448 (cache)
Jan 08 18:13:27 synapse media_repo[1397]: INFO[2020-01-08 17:13:27.656 Z] Doing federated GET to https://jki.re:8448/_matrix/media/v1/download/jki.re/tvsXlXaChyGvbqUZnSXxNXgs?allow_remote=false with host jki.re
Jan 08 18:13:27 synapse media_repo[1397]: ERRO[2020-01-08 17:13:27.718 Z] Unexpected error locating media: Get https://jki.re:8448/_matrix/media/v1/download/jki.re/tvsXlXaChyGvbqUZnSXxNXgs?allow_remote=false: dial tcp [2a01:7e00::f03c:91ff:fe6e:411b]:8448: connect: c
Jan 08 18:13:27 synapse media_repo[1397]: INFO[2020-01-08 17:13:27.718 Z] Replying with result: *api.ErrorResponse &{Code:M_UNKNOWN Message:Unexpected Error InternalCode:M_UNKNOWN}  contentLength=0 contentType="" host=matrix.amorgan.xyz method=GET queryString="" remo

Or maybe the cache ate it.
Thus leading to clients not being able to see the media.

Works for me:
INFO[2020-01-08 17:39:09.674 Z] Getting server API URL for jki.re
INFO[2020-01-08 17:39:09.755 Z] Server API URL for jki.re is https://jki.re:8080 (SRV)
INFO[2020-01-08 17:39:09.755 Z] Doing federated GET to https://jki.re:8080/_matrix/federation/v1/version with host jki.re

This looks like some weird DNS bug in go and/or Ubuntu 18.04


Just a kind suggestion: update the example of launching preview using chrome in OS X

I've tried the new version OmniMarkupPreviewer. Glad to see Markdown's strikeout and subscript can work well at the same time(Before that if the two are loaded, only strikeout can work ), and jquery.min.map problem also been solved.
At the same time, previewing markdown in Safari is a little disturbing. Because Safari continuously scrolls up and down as you type in ST3, which makes you almost cannot preview. Finally I decided to give chrome a try, and the result is exciting. It works perfect with any problems! But when I followed the example of launching preview using chrome in default OmniMarkupPreviewer.sublime-settings file, which told me to set browser_command to ["open", "-a", "chrome", "{url}"], it didn't launch chrome(actually nothing happened). And finally I found ["open", "-a", "Google Chrome", "{url}"] worked well for me ...
If you can fix the Safari issue, that will be best.  :) Otherwise, chrome browser seems to be the better choice. And please update the example of launching preview using chrome in OS X. That may help a lot of people(In last few days I've been used Safari to preview and endure the terrible scrolling).
Thanks for your awesome app! :)

I found why strikeout and subscript can't work well at the same time before.
Actually I made a mistake. When using default github theme they can work together.
But when using your own theme,  you need to place customized css&template files into ${packages}/User/OmniMarkupPreviewer/  folder, which is recommended.
If you unfortunately place your files into OmniMarkupPreviewer installed folder(~//Library/Application\ Support/Sublime\ Text\ 3/Packages/OmniMarkupPreviewer/), it will result Markdown's strikeout and subscript can work well together.
Sorry for bothering.

@ashfinal
Thanks for you comprehensive information, but I can't reproduce it on OSX Safari, does your markdown contains any special elements, large images, for example?

Thank you for reply. No special elements, Not too much images(4-5 pieces), and the resolution is all below 600x600. Safari works well only when have one page to render, if your markdown is over than one page. You continue typing in ST, Safari will scroll up then quickly down. You type a few character, it scroll up and then down … so on and so on.
Chrome don't have this problem. It scrolls smoothly.
I think maybe it's because Safari's render strategy is different?
BTW, I'm on Yosemite 10.10.2 Safari 8.0.3
Anyway, Chrome works well. So far, that's enough for me. : )
If you still want to figure what's happening, someday I can give you a screencast.

When using this browser_command setting in the Preferences -- where do we find the specific name of the browser that will get the setting to work?
ie. "browser_command": ["open", "-a", "WHERE DO WE FIND BROWSER NAMES THAT WILL WORK", "{url}"],
Thanks


Fix typo in docs for iterable's take

cc @floitschG

Can't you just update your patch?

Can't you just update your patch?

Sure, no problem. I will change to count.

Moved to https://codereview.chromium.org/1287453002/


Are Personal-ish Sites Allowed?

Your issue
Hello. I am one person currently running my web forum https://basementcommunity.com.
It's an internet discussion board for general and focused topics with no real intent on making money or being a business, but really just a hobby project for myself with a goal of creating a community of people fatigued from modern social media.
That being said, I'm a huge a11y advocate and pride myself on making my site as accessible as possible. But I'm not planning on revealing my identity willingly for the foreseeable future.
So the question is, are sites like mine allowed to join the webring?
Code of conduct

[X] I agree to follow this project's code of conduct

Hey there, I want to communicate a couple of things:

Thank you for your interest, and for considering this webring for the community you cultivated. Seeing things like that warms my heart.
Also thank you for your patience with me getting back to this. I've been going through a lot of job, life, and health-related stress lately, and things like this had to be deprioritized.

I've been thinking about the request via the lenses of the site's mission and your expressed desire for anonymity. I take community building and privacy seriously, and have a ton of respect for others who do as well.
I think where I land on this issue and its companion PR is that communities are less in line with the overall goal of the webring. Quoting the tagline here:

A webring for digital accessibility practitioners.

A discussion board seems a little less inline with that goal, and consequently I don't think I should add the Basement community forum. It also raises the question of the accessibility of user generated content, which increases the surface area of content to monitor for inclusive practices to a degree that I don't have the ability to stay on top of. Basically, I want the webring list to have more of a promise of accessible content than not and that complicates things.
As for anonymity, I do think that is something the webring should (and does) support. Aliases are acceptable for a web presence on the webring. The security of the chain of technology used to create that anonymity is beyond the scope of this website, but the underlying considerations about it is why I draw the line at requiring HTTPS.
All this said, I'd like to keep this issue open until the following questions are addressed, if you're willing:

Are there opportunities for improvement in the documentation to better communicate these concerns? Both on the site itself and GitHub?
Is there any other information you feel is not present or could use improvement?

And again, thank you for your interest and patience here.

Closed for inactivity.


Bluebird Promise referred instead of default promise

Promise.delay is used in scheduled OOB however bluebird promise was not referred. This resulted in runtime error TypeError: Promise.delay is not a function as seen in schedule worker logs. This PR should fix it.

Pull Request Test Coverage Report for Build 866

1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 86.838%

Totals

Change from base Build 864:
0.002%

Covered Lines:
5894

Relevant Lines:
6557

💛  - Coveralls

@sauravmndl : can you please review/approve this PR as it is needed today for test cycle fixes ?


Support databaseId at ProviderContext

I've supportted the databaseId at ProviderContext.
In this change, developers can support an SQL for multiple database using provider method.
This is equivalent to _databaseId of special variable for XML based mapper.
WDYT?

Looks good. Thank you, @kazuki43zoo !

thx!


Digital Twin KIT Release 23.12 Operations View Acceptance Criteria

Release 23.12 KIT Operations View

[ ] Quick Setup Guide and Deployment Scripts in Repository

Documentation for the operations view must be available in Eclipse Tractus-X as MD files.
A reference implementation must be deployable via versioned and released HelmCharts with ArgoCD and as local deployment. In addition, a quick setup guide must be provided.

Quick Setup Guides
Deployment Scripts (e.g. local, helm charts, terraform)

Best Practice:

eclipse tractusx-docs-artefacts
How-to-create-a-KIT-website

Artifact Repository:

eclipse-tractusx

Contacts:

Responsible: PO/BO (from relevant Domain)
Responsible for Process: PO / BO (from relevant Domain)
Consult: KIT-CONTACT1, KIT-CONTACT2 + System Team
Review by: PCA/BDA

Quick Setup Guide and Deployment Scripts in Repository --> This is provided in the reference implementations' repositories linked here.

@danielmiehle @maximilianong kindly provide your approval.


Merging Feature/ux refactor into mrtk_development

Overview
this merges all work done on ObjectManipulator and BoundsControl into mrtk_development experimental section.
Reviews for individual PRs into ux_refactor were already done

/azp run


Adding bmi calculator to dailyUseTools

resolves #51
Title : B.M.I calculator
Number of files/folder added : Folder: 1 (BMI calculator Files : 2 (bmi-calculator.html and screenshot)
changes made : Added bmi calculator using index.html theme, updated README.md
I have -
Read contibuting.md
Followed all the guidelines
updated readme.md
added your name in the footer of my bmi-calculator.html file
provide below mentioned details if you want your name to be added in contributors.html
twitter: 2apreety18
GitHub: https://github.com/2apreety18

@2apreety18 Awesome work on  this one. 💚🎉. keep contributing and happy coding.✌


ros2 launch xarm_gazebo _robot_beside_table_gazebo.launch.py

Error: [ruby $(which ign) gazebo-1] QStandardPaths: XDG_RUNTIME_DIR not set, defaulting to '/tmp/runtime-root'
[ruby $(which ign) gazebo-1] MESA: error: Failed to query drm device.
[ruby $(which ign) gazebo-1] libGL error: glx: failed to create dri3 screen
[ruby $(which ign) gazebo-1] libGL error: failed to load driver: iris
[ruby $(which ign) gazebo-1] libGL error: failed to open /dev/dri/card1: No such file or directory
[ruby $(which ign) gazebo-1] libGL error: failed to load driver: iris
[ruby $(which ign) gazebo-1] Library [/keti_ws/install/ignition-sensors6/lib/libignition-sensors6.so] does not export any plugins. The symbol [IgnitionPluginHook] is missing, or it is not externally visible.
[ruby $(which ign) gazebo-1] [Err] [SystemLoader.cc:103] Failed to load system plugin [libignition-sensors6.so] : couldn't load library on path [/keti_ws/install/ignition-sensors6/lib/libignition-sensors6.so].

Expected Behavior: To load the camera and robot in the same environment and have camera topics published correctly such that once bridged with ros2 one can visualize correct tf tree and have correct frame_ids for all camera topics

Actual Behavior: Fails to load the new sensor plugin built from source that is suppose to fix the frame_id issue for points

I am seeing that the PluginHook definition was changed in https://github.com/gazebosim/gz-plugin/blob/gz-plugin3/loader/src/Loader.cc#L537. Could be a mismatched version of gz-plugin.  Need to dissect the .so file to figure out.

Okay, figured it out. What we were missing is that /usr/lib/aarch64-linux-gnu/libignition-sensors6.so is actually NOT a plugin, but merely a shared library that the plugin /usr/lib/aarch64-linux-gnu/ign-gazebo-6/plugins/libignition-gazebo-sensors-system.so links to.
What this means is that we simply run make install from gz-sensors6, WITHOUT any modifications to the SDF file.

Nikola Raicevic
1:37 PM
I ran the image, and the camera plugin ran successfully but still have this:  ros2 topic echo /camera_01/camera_depth/points | grep "frame"
frame_id: sensor_d455/camera_01_link/cameradepth
Nikola Raicevic
1:44 PM
Seems like gz_frame ID did not fix it <gz_frame_id>${prefix}_depth_optical_frame</gz_frame_id>

By the way, this should work for depth image => pointcloud: https://github.com/ros-perception/image_pipeline/blob/rolling/depth_image_proc/launch/point_cloud_xyzrgb.launch.py
4:09
hopefully something we can just apt install


chore: add available to sett fragment

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Dev or production Dependency?

I noticed this shows up under require rather than require-dev in composer.json.
Is there any reason?

It's a dev dependency.
Thanks for notice. I will update it on next release.


feat(MarshalJSON): marshals slice of SubDiff.diffs to JSON

Need a way to marshal the diffs to json so we can send the diffs over the API /diff endpoint in qri.
func (d *SubDiff) MarshalJSON() ([]byte, error)
Marshals the SubDiff.diffs (using the d.Diff.Deltas() function) to json.

hey! So, when yall get the chance, I’ve made a small change to datasetDiffer (added a MarshalJSON function), but its failing the linting step on circleCI (not when I run golint locally):
datasetDiffer_test.go:1:1: don't use MixedCaps in package name; datasetDiffer should be datasetdiffer
Found 2 lint suggestions; failing.
Exited with code 1

Not sure if I should rename the package and try again or if this is a bs lint suggestion


Added UMD for CommonJS, AMD and globals

For people using browserify to bundle their js files (like me) or those using AMD. The extra code does not hurt the ones using globals either.

Hi @marcandre , it would be possible to accept this?
Thank you so much!

Sorry to have forgotten about this. Merged.

Thanks @marcandre !


feat: Add Arbitrum, Optimism and Polygon support

What I did
Added support for Arbitrum and Optimism to Infura Provider so that it doesn't say NetworkError: 'infura' is not a valid provider for network 'mainnet' anymore
fixes: #44 and #45 and #24
How I did it
Similar to how it is done in ape-alchemy https://github.com/ApeWorX/ape-alchemy/pull/17
How to verify it
I wrote test for it
Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

flake failed because the comment was too long 😅 pushed a fix, sorry

@fubuloubu do you know how to fix CI in that PR? would love it to get merged, but seems like I didnt touch anythin CI related, and the tests are failing.
Tests are requiring WEB3_INFURA_PROJECT_ID set. What is your usual approach to this? Seems like it always was the case for the CI in thar repo. I can add mocks, I guess, but it seems that you had a workaround before

Tests are requiring WEB3_INFURA_PROJECT_ID set. What is your usual approach to this?

No, basically because this is your first time contributing, you don't have access to the secret necessary to run the CI

@eliseygusev fix that one issue, and I'll run the CI locally and merge this (since the CI isn't going to pass right now for you)

Tested locally with my api key


fix(typography): enable lyon font path setting

Description
The current settings in typography.scss assume a directory structure that some projects do not have (i.e. /assets/fonts). This causes webpack builds to fail as it is difficult/impossible to store the Lyon fonts in a location that can be found.
Fixes #409

@Hopkinsa I think something is wrong with the PR - I don't think you changes 1071 files 😅

@Hopkinsa I think something is wrong with the PR - I don't think you changes 1071 files 😅

@Hopkinsa I think something is wrong with the PR - I don't think you changes 1071 files 😅
Something definitely wrong, thats not what I pushed!  Will have a look to see what's happened

Incorrect files included in PR


search Feature add  #578

Search Feature Added issue number #578

hii @jayk-gupta can you pls added level 2 or level 3 tag as it take many  changes and effort

Thanks for your contribution!

The search is not working and there is a bug in website index.js file


What is obj_mac.h used for

The notes said "Do not edit ".what is obj_mac.h used for? What would happened if I add a #define?

If I need to add a new algorithm , is it necessary to add a new nid in obj_mac.h?

No you shouldn't edit this directly.  Instead add your object to: crypto/objects/objects.txt then run make update.

@paulidale Is obj_mac.h generated from the crypto/objects/objects.txt ?

Yes.

Thanks!And  I have another question~~When I add my object and make,build happens:
./libssl.so: undefined reference to `EVP_sm4_128_gcm'
collect2: error: ld returned 1 exit status
Makefile:6459: recipe for target 'apps/openssl' failed
make[1]: *** [apps/openssl] Error 1
make[1]: Leaving directory '/home/bowen/openssl-1.1.1d'
Makefile:174: recipe for target 'all' failed
make: *** [all] Error 2
Here is what I did:
1.added my object to crypto/objects/objects.txt  and make update
2. added EVP_add_cipher(EVP_CIPHER *cipher)
3.implemented corresponding evp interface.
Here is the question:
How to sovle the build error？ It's there any document or instruction for adding an algorithm ? Thank u again!!

EVP_sm4_128_gcm isn't being exported from libcrypto.  make update should cover this too.
Note that new EVP_xxx algorithm functions are not being accepted.  Instead algorithms should be accessed by name, preferably using, in this case, EVP_CIPHER_fetch().

Thanks for advice! It didn't work.But I find out that obj_mac.h doesn't generate code after adding objects to crypto\object\obects.txt but other related files did such as obj_dat.h and obj_mac.num. It seems to be the core of the problem.How to solve it?

Building of these files is available only on unix-like systems via make update

Thanks for help.Yes my system is ubuntu 14.And I run make clean make depend and make update . But As I said obj_dat.h and obj_mac.num they were updated , except for obj_mac.h.All the files related should be updated once I add object to objects.txt,right?

I deleted some code from obj_mac.h and then run make update  and then  no code was recover as imagined......

If you look at make update definition (e.g. via make -n -B update to see the real commands), you'll see that the regeneration also involves as intermediate obj_mac.num that you did not edit.
BTW, I don't recommend making such experiments - it's possible to break the build completely.

Thanks a lot! It's useful!


UNDER REVIEW: Mix-in for user reactions to prior propositions - decision attribution

mix-in that is used for attribution of an Experience event to one or more Decision events.

@cluby-adobe - Please review as this involves ExperienceEvent

The use cases that would need this mix-in are still being detailed.

@jbeckert - Please let me know when this is ready for formal xdm review

@prabhum2 This can be reviewed for the mechanical aspects, the structure, naming, style guidelines etc.
We still have an outstanding discussion for how one of the clients of the mixing will use it but if you consider it to be used by non-Adobe solution clients it is general enough to fit the needs of sending back reactions and responses to prior propositions made in the experience decisioning sub domain. At that abstract level, everything is there.

@clubycoder The decisioning domain has always been an adobe extension. Can we address this in a different PR and timeframe. The current mix-in works in and with the datatype schemas of that extension domain. For instance, it is not common practice that a core component makes references to an extension component. The decisioning domain defines the proposition event and decision options class in the first place and the new mix-in makes reference to those schemas. It would be questionable if this is even allowed, that a core component references those extensions. It is the other way around. Making the mixin completely generic and not refer to any decision proposition or decision options, robs it of its value because anything could be stored in the mixin properties.
There was resistance in the past by the stewards of the XDM system to pull in decisioning as a core sub-domain. This should be done holistically and not by moving one mixin of the domain to the core  system
If the term "attribution" is reserved for other concerns, then this is easy to change.
What about "response", "reaction", or "interaction", "lifecycle"?
The issue I see with the first two is that there are a number of Experience Events that will use the mix-in which aren't strictly user responses or reactions. Like the event that presents (shows, displays, renders) the decision proposition.
I would favor lifecycle but could also see interaction as a good term.


fix: property pane table binding issue fixed

Description
When REST API with which returns response in non array format is bound to table widget, We are not able to see table's data property populated with that api name. This PR fixes the issue

PR fixes following issue(s)
Fixes #25432

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Bug fix (non-breaking change which fixes an issue)

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[ ] Jest
[x] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview

/ok-to-test

/ok-to-test

/ok-to-test

/ok-to-test


build: check-fbp-bin should know about modules

With alldefconfig or allmodconfig where tests depend on node types built
as modules the tests will not run due to lack of symbol definition -
since they were not linked to modules.
Since we don't configure the fbp tests but discover'em with directory
listing and skip codes we can't specify dependencies, this patch changes
the fbp-bin compilation to always depend on all modules - again, we
don't know exactly the ones we really depend on but for test suite
purpose it's ok to link against all modules.
Signed-off-by: Leandro Dorileo<EMAIL_ADDRESS>

actually we're already linking against the modules... we just have special requirements for the ubuntu's gcc version. I'm sending a new patch for that.


Fix sequences upsert

A nice and large PR bringing two, main changes:

Refactor "upsertItems" for less duplication of code (although not perfect)
Upsert sequences headers including column definitions/schema

Finally finished the big PR. Now support for handling upsert of sequence rows with modified column schema is also supported.


items can react to forceful equip

Give items the ability to trigger effects when force-equipped. Pass through the attacker to check for perks and such.

Yeah, there are - but I think I covered all the dynamic ones, and my thinking is that in scenes it can/should? be handled by the scene.

Depends on what you need it for

Could go over them and pass the relevant pc/npc through if you think that's preferable though, it wouldnt take that long - but yeah, that's my thinking, in scenes you're usually equipping a specific item, so you know if it overrides afterForceEquip and can pass the npc through if necessary
in case you're wondering what my use case is, its adding a base level of hypnosis when forcing visors onto people with certain perks: func on_forceEquip(_attacker):

Maybe it's better to have a onEquippedBy(otherCharacter, forced=false) function on the item then. Just for the future. Don't worry about modifying all of the calls
It almost feels like it would be nice to have a BaseCharacter function too. Something like
getCharacter("rahi").forceEquipBy("pc", GlobalRegistry.createItem("ballgag"))
just a cleaner api. Dunno

I did notice you're a lot more liberal with introducing new functions than I'd be. Probably I'm just used to languages that have method overloading :) I can rejig it, no problem.

Something like this, maybe? I've left them in Inventory.gd for now, though, I dunno about having one set of item equipping functions in Inventory and another in BaseCharacter, so if you wanna move it there you'd have a better idea of what you want it to look like.

Yeah thank you ^^ I will take a better look and merge when not sleepy

Thankies


Install correct PyTorch nightly in GH action

For the GH action about running torch.compile, when using the nightly options, install from the correct index (used to be test, now is nightly).

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Silence usage on internal errors

Codecov Report

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

@@           Coverage Diff           @@
##           master     #783   +/-   ##
=======================================
  Coverage   48.14%   48.14%
=======================================
  Files          28       28
  Lines        2476     2476
=======================================
  Hits         1192     1192
  Misses       1098     1098
  Partials      186      186

Continue to review full report at Codecov.

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


Check Groovy UnitTests

Some Groovy based unit tests depend on async calls. They often fail on Travis since the wait time is too small
In addition some Groovy tests are commented out. We should check if we can remove the tests or try to add them (maybe in a fixed version)

duplicate of https://github.com/canoo/dolphin-platform/issues/452


New command: project rename

Usage
spfx project rename [options]
Description
Fully rename a SPFx solution/component by updating all references to the original name used when creating a solution.
This would be very handy as sometimes solutions evolve to be something that was not fully expected when they were created and names may no longer fully describe the solution purpose.
Having a command to "replace" all instances of the name on the right places would be a great experience
Options

Option
Description

-n, --name <name>
The new name for the solution/component

Additional Information
Sorry...no idea...

Sorry for the delay in getting back to this thread, and sorry if this will be non sense :) It came out as I got frustrated updating a solution that was created with a name that was no longer appropriate as the solution evolved dramatically into a different direction :) Updating all the references manually to tidy up things is a very boring task :)
I'm thinking on two different commands:
Update all the instances of the solution name provided during the creation process, and perhaps also the Id:

yo-rc.json: update libraryName and libraryId
deploy-azure-storage.json: update container property
package-solution.json: update name, id and zippedPackage
package.json: update name
readme.md: update title

Update the name and id of web parts or extensions

*.manifest.json: update id and all name references
config.json: components reference
update React component created by default - rename files and replace names in code
mystrings.d.ts: update references to previous name

Thanks for the clarification @joelfmrodrigues, it makes indeed sense to create two commands to keep things simple. I'll create specs for both of them and we can take it from there. Would you like to help building one of them?

Closing this issue in favor of two new issues with specs for the new commands.

For sure as I would like to check how the project is setup and give it a go (it has a great reputation 😁), but I anticipate I may only have time after the end of the month. I still have to add two reusable controls to the PnP repository and that will likely take all my spare time.
If no one else does it before the end of the month, then I will give it a try 👍

Awesome! Thanks again for the idea and the transparency about your availability 👏


fzn-ortools: seems to give up on div and mod if divisor can be negative

What version of OR-Tools and what language are you using?
Version: 9.6
Language: MiniZinc
Linux Ubuntu 22.04
What did you do?
$ cat /tmp/latest.mzn
var -10..10: A;
var -10..10: B;
var -10..10: C;
constraint
  A div C=B;
$ minizinc --solver ortools /tmp/latest.mzn
=====UNKNOWN=====

The same thing happens for mod.

yes. We reject models where divisor or mod span across 0.


typing: Fix ioredis typing

I tried to use this with tsc compiler then it throws an error
'Redis' only refers to a type, but is being used as a namespace here.

this PR should fix the problem

Should be fixed in latest release #422


Delete disk modal

📝 Description
Adding a delete disk modal
🎥 Demo
https://user-images.githubusercontent.com/67270715/161136779-d14e8310-82a9-4b8a-9b78-697eb04c74cd.mp4
Signed-off-by: Aviv Turgeman<EMAIL_ADDRESS>

/retest

/lgtm

/hold

/lgtm

/unhold


Calculate scheduled headway per event

In order to display colored dots in the data dashboard, scheduled_headway needs to be filled in.
Unfortunately, we don't immediately know what the scheduled headway is from a particular vehicle, so we need to calculate it ourselves from GTFS. It also might be possible to request them on-demand, once per day, from the MBTA v3 API.

What's the preferred way to define the scheduled_headway for the first stop in a route for a given day? There's about 4 hours of dead time. We could grab yesterday's stop_times.txt if we have the previous trip_id for that route, but I'd also buy if this is somewhat of a nonsensical notion and we are happy to slap a NaN on that bad boy

This was fixed in https://github.com/transitmatters/gobble/pull/7!


[request] ESP32 esp-idf branch update

Hi,
is there plan to switch esp-idf to more recent release (release/3.1 at least).
I would like to use esp_http_client component and it is incompatible with current tcpip_adapter component used in AWSFreeRTOS (bunch of redefinition, etc).
When i replace tcpip_adapter then smartconfig_ack is complaining.

Hello,
We currently use a tailored version of ESP-IDF, and while we have been accepting updates from Espressif on those core components, we have not yet been able to bring in some of the framework's newer components and features. Unfortunately, it looks like the HTTP client/server components are among those which we don't yet support.
I did bring this issue up internally; thank you for helping us prioritize our ongoing work. We are investigating ways to keep our dependencies more up-to-date across all of our supported platforms, but I'm sorry for the inconvenience in the meantime. Feel free to let us know if you have any further questions.

Hi,
basically what i need is option to call http(s) calls from IoT thing. My first thought is esp_http_client because i know it, but maybe there is some option in AWSFreeRTOS that i didnt find yet. Any help is appreciated, because its ongoing project.

Unfortunately, I don't think we have a built-in solution for HTTP functionality yet. HTTP[S] client connections are on our roadmap, but we don't have a timeline for those features yet.
When you say that the esp_http_client component is incompatible with the tcpip_adapter component that we use in our demos, do you mean that you get build errors when you copy the HTTP client component from a recent version of ESP-IDF into lib/third_party/mcu_vendors/espressif/esp-idf/components/? If so, could you share a log of some of those errors?


Reduce CI, move to node 14

Codecov Report

Merging #165 (f75c785) into main (762fc12) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #165   +/-   ##
=======================================
  Coverage   41.91%   41.91%
=======================================
  Files         119      119
  Lines        5750     5750
  Branches      876      876
=======================================
  Hits         2410     2410
  Misses       3190     3190
  Partials      150      150

Continue to review full report at Codecov.

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


Missing _compliant init

Should it be initialized to True or False?
Should it come with a version bump to 1.2.4?

Yes, it should. But in stepper this value is a duplicata of self._config[Stepper._MODE_COMPLIANT] witch is already initialized to true. We should juste only use this one and remove the duplicata.
We found other issues on this example. We will test it and fix it as soon as possible.
We are currently working on a 1.3.0 rev for Pyluos, with big release of all libs.

We are currently working on a 1.3.0 rev for Pyluos, with big release of all libs.

I've seen that. I'm still using 1.2.3 though because the dev revision has breaking changes, in particular modules are no longer named modules, and that requires re-flashing the gate, Luos 0.7.0... which I couldn't compile so far. I would go for a few fixes on 1.2.3 in parallel of the new revision, which would allow to publish a (first) complete and working release for ROS 2.

Ok I will create a new 1.2.4 revision with the Steppers updates.

Revision 1.2.4 is created.


flutter: Error: NoSuchMethodError: The method 'signUp' was called on null.

Describe the bug
flutter: Error: NoSuchMethodError: The method 'signUp' was called on null.

Receiver: Instance of 'NoSuchMethodError'
Tried calling: message
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:53:5)
#1      _LoginSignupPageState.validateAndSubmit. (package:aspire/ftu/signup.dart:66:29)
#2      State.setState (package:flutter/src/widgets/framework.dart:1233:30)
#3      _LoginSignupPageState.validateAndSubmit (package:aspire/ftu/signup.dart:64:9)
#4      _InkResponseState._handleTap (package:flutter/src/material/ink_well.dart:779:19)
#5      _InkResponseState.build. (package:flutter/src/material/ink_well.dart:862:36)
#6      GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:182:24)
#7      TapGestureRecognizer.handleTapUp (package:flutter/src/gestures/tap.dart:504:11)
#8      BaseTapGestureRecognizer._checkUp (package:flutter/src/gestures/tap.dart:282:5)
#9      BaseTa<…>
Expected behavior
The email and password should send to firebase for signup.
Screenshots

Smartphone (please complete the following information):

Device: Pad Pro (12.9-inch) (4th generation)

Add a root page for init the auth can solve this issue


Added correct graceful shutdown.

Fixes https://github.com/taskiq-python/taskiq/issues/380
Fixes https://github.com/taskiq-python/taskiq/issues/325
This PR rewrites graceful shutdowns completely and now it should be able to gracefully wait for all tasks to complete and implement proper reloading.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 48.27586% with 15 lines in your changes missing coverage. Please review.

Project coverage is 72.85%. Comparing base (317c1d9) to head (3dd6cdd).

Files with missing lines
Patch %
Lines

taskiq/cli/worker/run.py
0.00%
11 Missing :warning:

taskiq/receiver/receiver.py
80.00%
3 Missing :warning:

taskiq/api/receiver.py
66.66%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #381      +/-   ##
==========================================
- Coverage   73.16%   72.85%   -0.32%
==========================================
  Files          62       62
  Lines        1871     1890      +19
==========================================
+ Hits         1369     1377       +8
- Misses        502      513      +11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


Implement time Duration

thanks

I will revert this change.
The goal is that all types supported out-of-the-box are compatible with how serde_json serializes them. std::time::Duration is, by default, serialized as {secs: .., nanos: ..}.


🐛 Bug Report: [Techdocs] some mkdocs-material features are not working

📜 Description
Some mkdocs-material features are not working as expected. For example:
---
status: deprecated
icon: material/emoticon-happy
---

like described here: https://squidfunk.github.io/mkdocs-material/reference/
👍 Expected behavior
It's expected to work as described here and in the same way as when running mkdocs serve locally

👎 Actual Behavior with Screenshots

The reference config is not displayed
👟 Reproduction steps
Add the following lines at the top of a Markdown file to display a status for a page:
---
status: new
---

run
techdocs-cli serve -v

it works when using the mkdocs-material docker-image
docker run -it --rm -p 8000:8000 -v ${PWD}:/docs --name mkdocs-material squidfunk/mkdocs-material

📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, but I'm happy to collaborate on a PR with someone else

I just encountered the same issue. I'm pretty sure this is because Backstage does not support the CSS classes that mkdocs uses for the icons.

Not stale.

This was raised in #5279 which was closed in favor of #3998 which doesn't appear to have resulted in any changes that would support the new mkdocs-material CSS (or any extra styles added inside the docs).
From what I can see there is a set of styles manually copied from mkdocs-material and hardcoded into the techdocs plugin. Obviously this hasn't been updated with all of the optional styles (eg: md-status--new). Long term, I feel like the best thing to do would be for a new RFC to be opened around improving this experience.
Short term, I worked around it by adding both my extra css as well as the default status variables into my app\public\index.html:
<head>
...
    <style>
      :root {
        --md-status--updated: url('data:image/svg+xml;charset=utf-8,<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24"><title>cellphone-arrow-down</title><path d="M17,1H7A2,2 0 0,0 5,3V21A2,2 0 0,0 7,23H17A2,2 0 0,0 19,21V3A2,2 0 0,0 17,1M17,19H7V5H17V19M16,13H13V8H11V13H8L12,17L16,13Z" /></svg>');
        --md-status: url('data:image/svg+xml;charset=utf-8,<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24"><path d="M11 9h2V7h-2m1 13c-4.41 0-8-3.59-8-8s3.59-8 8-8 8 3.59 8 8-3.59 8-8 8m0-18A10 10 0 0 0 2 12a10 10 0 0 0 10 10 10 10 0 0 0 10-10A10 10 0 0 0 12 2m-1 15h2v-6h-2v6Z"/></svg>');
        --md-status--new: url('data:image/svg+xml;charset=utf-8,<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24"><path d="m23 12-2.44-2.78.34-3.68-3.61-.82-1.89-3.18L12 3 8.6 1.54 6.71 4.72l-<IP_ADDRESS> 3.68L1 12l2.44 2.78-.34 3.69 3.61.82 1.89 3.18L12 21l3.4 1.46 1.89-3.18 3.61-.82-.34-3.68L23 12m-10 5h-2v-2h2v2m0-4h-2V7h2v6Z"/></svg>');
        --md-status--deprecated: url('data:image/svg+xml;charset=utf-8,<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24"><path d="M9 3v1H4v2h1v13a2 2 0 0 0 2 2h10a2 2 0 0 0 2-2V6h1V4h-5V3H9m0 5h2v9H9V8m4 0h2v9h-2V8Z"/></svg>');
        --md-status--encrypted: url('data:image/svg+xml;charset=utf-8,<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24"><path d="M12 1 3 5v6c0 5.55 3.84 10.74 9 12 5.16-1.26 9-6.45 9-12V5l-9-4m0 6c1.4 0 2.8 1.1 2.8 2.5V11c.6 0 1.2.6 1.2 1.3v3.5c0 .6-.6 1.2-1.3 1.2H9.2c-.6 0-1.2-.6-1.2-1.3v-3.5c0-.6.6-1.2 1.2-1.2V9.5C9.2 8.1 10.6 7 12 7m0 1.2c-.8 0-1.5.5-1.5 1.3V11h3V9.5c0-.8-.7-1.3-1.5-1.3Z"/></svg>');
      }

      .md-status--updated::after {
        -webkit-mask-image: var(--md-status--updated);
        mask-image: var(--md-status--updated);
      }
    </style>
...
</head>

Not stale.

Yes, @darylgraham is correct that in the case of the status icons the CSS hasn't been copied across to support this. In the short term, we can add those so they're at least supported.
A larger change that moves away from this necessity will take longer and require input from more parties, but is something we can start looking at.

@cftad - Hey, I have a PR open to fix this. LMK if this does or doesn't address or issues.
Thanks


License is missing

There is no license in the repository.
Under which license is the documentation licensed?

It's MIT - I just added the file.
Thanks!

On a second thought - MIT is for software, so perhaps we need to pick a different license? Some CC license? Any suggestions?

I mean technically there still is code here; in both snippet form and some js stuff to generate the website. So the MIT one should still suffice you'd think?

From https://choosealicense.com/non-software/

Any open source software license or open license for media is applicable to software documentation.

So I think MIT is fine.


Stream json parsing

Is there a way to parse the JSON string from a network stream (JSON as bytes stream or data chunks)?

There isn't, no.  sajson relies on having the entire input document available.  (Fortunately, sajson parses at a rate of hundreds of MB per second, so streaming should not be necessary except in the most extreme cases.)

Ah, yeah.  If keeping the entire JSON document in memory is not desirable, sajson is not a good fit.  :/  I would look for a streaming parser.
Cheers!


Add gemv masked to JIT plus some fixes

Added masked gemvs to the JIT, reduces the library size from 5.9 to 1.7M
Fixed some issues with JITing the new large vectorized kernels
Some random code reorgs with a small attempt to reduce CPU binary footprint

Currently the MLX light build command (e.g .without the CPU back-end) gives you a C++ binary of 2.8MB and a MTL Lib of 1.7 MB (all in 4.5MB).
If you include the CPU back-end the C++ binary goes up to 8. MB, so all in 9.7 MB.


added token date checks & signature is now forced to lowercase hex before validation.

For some reason, i couldn't get token verification to work as-is..
signature = basexx.from_url64(str:sub(dotSecond+1)) was returning binary, but
['HS256'] = function(data, signature, key) return signature == tohex(hmac.new (key, 'sha256'):final (data)) end, was returning lowercase hex.
So, forced extracted message signature to lowercase hex.
Also, added token date (nbf, and exp) checks if they are defined within the token.
Hope this helps..

Would you mind adding some tests that describe the behavior and fail with the old code and then pass with the new?

Hi Robert,
Apologies on delayed response - crazy week.
I'm happy to help, but not quite understanding what you're asking. Are you
asking to enhance the code for robustness? or something else?
fyi.. i committed a minor change that permits errors in json decode to flow
upward so they can be handled by the calling lua code. otherwise, nginx
fails with 500 error. I don't know if i should do another pull request? or
if you will see the second commit? Please advise..
Thnx!
:-)
-Mark

From: Robert Andrew Ditthardt<EMAIL_ADDRESS>Sent: Monday, June 13, 2016 8:06 PM
To: Olivine-Labs/lua-jwt
Cc: mhassman; Author
Subject: Re: [Olivine-Labs/lua-jwt] added token date checks & signature is
now forced to lowercase hex before validation. (#9)
Would you mind adding some tests that describe the behavior and fail with
the old code and then pass with the new?

You are receiving this because you authored the thread.
Reply to this email directly, view
https://github.com/Olivine-Labs/lua-jwt/pull/9#issuecomment-225744640 it
on GitHub, or mute
<https://github.com/notifications/unsubscribe/AA37PCEf8UXue0FbDzUxKGyKj2q0Za
qyks5qLfCAgaJpZM4I0FPn> the thread.
<https://github.com/notifications/beacon/AA37PKfPPdqjvjmpTOkdxZOttpkvLK8Iks5
qLfCAgaJpZM4I0FPn.gif>

Specifically I'm asking for tests to be written in the spec folder so that running busted confirms that this code works. There are existing tests over all of the functionality in this library.

This PR add nginx as dependency.
I think better use os module for that.

I have idea to add options.checks table which will allows provide some arbitrary values (and may be functions). E.g. it can provide current timestamp to checks nbf and exp. Also e.g. server name to check iss.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_ccboc05ncb/clone/recipe/meta.yaml:16: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_ccboc05ncb/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_ccboc05ncb/clone/recipe/meta.yaml:41: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_ccboc05ncb/clone/recipe/meta.yaml:41: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_ccboc05ncb/clone/recipe/meta.yaml:41: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_ccboc05ncb/clone/recipe/meta.yaml:41: missing_description: The recipe is missing a description
Errors were found


Add Tutorial/Setup page for VS Code + Cortex Debug

Below is copy/pasted from my response on Slack
There isn't a specific official example yet. I've been trying to figure out a way to make VS Code projects for all of the Daisy Examples that can work from the root folder as a single workspace, but I don't know that it can work with how VS Code, and the extension are set up.
For now, if you have the DaisyExamples folder, you can pick any example and add the .vscode folder from this template. (I haven't used vs code for debugging in a few weeks. So I may have forgotten a step or two). I added a README to show what edits you have to make to the files (for the final binary, and the compiler path), but then you should be good to go.
I haven't tested this yet, but it came from a working project. I may have some time later today to test this, and I'll plan on making a wiki page specifically for this ASAP!
CortexDebugFiles.zip

 VS Code Tutorial Wiki Page

We're going to go back to the original separate wiki page for installing Vs code and replacing the existing Getting Started page with that content as a starting point.
Ideal table of contents:

Get the Source

Install the Toolchain
2a. Install OpenOCD (Optional, only required for flashing over JTAG instead of USB)

Install the VS Code IDE
3a. Install Cortex Debug

Run the Blink Example
4a. Flash Over USB
4b. Flash Over JTAG

Debugging

Troubleshooting

added


Reorganize the docs README ToC to have related concepts listed under proper hierarchy

Description of the Issue:

Reorganize the Component Schematic section so that the main Application Model constructs are grouped together (Component, Trait, Application Configuration) and then related concepts should be nested under their parent (ie workloads under Component)

I can open a PR for this one. I think there were a few knit issues with this

Traits is a H3 header whereas all the other OAM constructs are H2s
It would be helpful to have a 2-3 sentence intro under the **Concepts ** to give the user context as to what the sub-headers are

Thanks for picking it up! Can you update the issue labels when you pick it up and start working on it please.

I don't have permission to issue labels otherwise I would :(

Can we close this, @suhuruli?

Yes!

#451 closes this.