Add null rendering test

Here's a failing test case.

okaayyy.. i could've sworn that failed on my machine...

okay, even worse: it seems CI will always pass, even if tests fail (!). Have a look at the CI logs and you should see the failure.

Relevant: https://github.com/juliangruber/tape-run/issues/27

I'm not entirely sure if it's worth adding this feature when a user could just as easily add a <noscript /> or <div /> instead. What do you think?

If undefined is handled, then I'd expect null to be too. This is a very common pattern after all:
return <div>
  { someCondition && <Navigation /> }
  { someCondition && <Content /> }
</div>
/* if someCondition is `null`, then we're screwed */

These tests were added in another branch last week :)


Select a range that does not consist only of siblings

Could you please show how i can select a range that does not consist only of siblings by using SHIFT + LMK?

You can not select non siblings with shift.

You can try to add this using a custom plugin. I have not tried this however (too many cases to consider - hidden nodes, closed nodes, not loaded nodes, etc)


makePrimitiveMap: use TupleSections (hlint warning).

:-1: I left it like that deliberately, I don't think this is any better.

I can revert and add the hlint ignore pragma instead in a following PR?

It's okay, I'm working pretty heavily on this code atm anyway. I'm not sure why you got it showing up as a warning when usually leftover hlints that aren't ignored fail the build.

That's because I am working on the bazel branch based on nixpkgs 18.09 which provide a more recent version of hlint with this new lint.


fix: don't show terminal suggestions when keybindings are sent through to shell

When terminal.integrated.sendKeybindingsToShell is true, we probably shouldn't show the suggestion widget since the user will only be able to interact with it via mouse. I thought the widget was broken until I checked my settings.
Part of #154662

@Tyriar for review


Syntax highlighting

On the Hugo website, this theme is tagged as having syntax highlighting, but even on the demo site, the code is all one color.  Is this mis-tagged, or is syntax highlighting under development?

Dream Plus does support syntax highlighting. It is for reasons unknown(to me), why the syntax highlighting fails for exampleSite(which is hosted as the demo site). I do know that it is caused by setting the parent directory as the themesDir in its config.toml.
So, in short, it works!

To use it follow the wiki instructions over here: Syntax Highlighting

I have found some discrepancy in my code which was causing the failure of syntax highlighting on the demo site. Will fix it tonight.

@cricketbird Done. Thanks for pointing it out. Shall we consider this issue solved?

@cricketbird You sure you used the proper lexer? Refer to this: https://gohugo.io/content-management/syntax-highlighting/

I just did (another) clean install and there is no highlighting in the exampleSite out of the box.  I then downloaded a syntax.css file (per the hugo Syntax Highlight link above) and renamed it highlight.css (per your wiki), and put it in static/css and still no highlighting.

It works in my case, and with the netlify builds as well. It should in your case as well. You must be doing something wrong. Which version of Hugo are you using by the way?

Assuming you're having issues implementing syntax highlighting. Read this:
Wiki/Syntax Highlighting

Closing due to inactivity. OP may reopen if the case arises.


Bug fix on UI to CoffeeEdit form value power accessible

The power accessible attribute is being treated as a boolean when it is a string. The UI will then display "Yes" when powerAccessible is set to a value and "No" when it is empty

Good eye @katielevy1! I've fixed the blog post in https://github.com/oktadeveloper/okta-blog/pull/401.


udpate user show

Coverage remained the same at 0.0% when pulling 09ecacd4da7b7ed48f2cdec7ca37fecf8ce488dd on jcottobboni:master into 5c49a4a7a8ca58cf3a241f7bf00da69af423dfa0 on gorails:master.


Enhancement: Revise 100 clear criteria for only ultimate clears

Revise "Please Do Other Content" role to only apply if a character has cleared an ultimate encounter 100 times.
Should not apply for any other content.

No longer needed. Yipee


this.state.lastIdealStyles[i] fails when children are removed from StaggeredMotion

Hi,
It might be my particular use case (cant share the code unfortunately), but i can say that the children are propagated via React.Children.map.
When  child is removed and react-motion attempts to lookup a style value via this.state.lastIdealStyles[i], it fails.
I've fixed this in my fork of react-motion and will gladly put in a pr, but my fix is just a bandaid as other cases could have the children being reordered.
It would seem like a better approach would be to store the previous styles and velocities in a key store by the components key.
edit
found https://github.com/chenglou/react-motion/issues/26

Looks like this need is covered by TansitionMotion including staggered animation. Just curious as to why StaggeredMotion is needed when a more predictable option is available?


Implicit use of tokio::net

Seems that the features is enabled by hyper_util but it's not enabled on this crate even though it's being used.
I was exploring the option to make wiremock wasm-unknown-unknown happy but sadly hyper_util doesn't allow it because it relies on tokio net which relies on mio. And mio only supports the wasm-wasi (there are OS socket after all)
While socket2 is not actually mandatory for the tokio feature itself but clien-legacy in hyper_util.
PS: Would be nice to enable resolver = "2" as well.

I'm open to the idea of making changes to enable easier usage in a WASM context. What would that entail?

PS: Would be nice to enable resolver = "2" as well.

That only affects the project you're currently building. Having it set on wiremock makes no sense difference for downstream users AFAIK.

I believe for WASM support we should target wasm32-wasi as it provides access to some OS resources.
The main issue I suppose is the Tcp socket where currently tokio::net is used and, in terms, mio.
as for the resolver, yes, you are right. features only show on the first occurrence of a given dependency in cargo tree and I had to hunt down which dep. enables a tokio features that requires mio

And btw mio supports wasm32-wasi and TcpListener/TcpStream:
https://docs.rs/mio/latest/wasm32-wasi/mio/net/index.html
Not sure if tokio will play nice though..


The URI returns http://:7086 and no data is generated.

When I run dotnet run on the event-sourcing sample, the URI returns http://:7086 and no data is generated.
Are there any omissions or errors in the procedures?
[Execution result]
C:\Temp\cosmos-db-design-patterns\event-sourcing\source> dotnet run
Unhandled exception. System.InvalidOperationException: The gRPC channel URI 'http://:7086' could not be parsed.

The results of both command runs are attached.
dotnet run.txt
func start.txt

Hey @paladique can you take a look at this when you get a chance? @enishimu1 is using this repo for a demo in a session.
Thanks!

Thanks for the output @enishimu1 - I'll take a look into this

The function needed to be changed to an isolated worker model, some things see PR here: https://github.com/Azure-Samples/cosmos-db-design-patterns/pull/34
Looking into other app issue as well.

[like]  Eiji Nishimura reacted to your message:

From: Jasmine Greenaway @.>
Sent: Tuesday, October 1, 2024 2:58:11 PM
To: Azure-Samples/cosmos-db-design-patterns @.>
Cc: Eiji Nishimura @.>; Mention @.>
Subject: Re: [Azure-Samples/cosmos-db-design-patterns] The URI returns http://:7086 and no data is generated. (Issue #33)
Thanks for the output @enishimu1https://github.com/enishimu1 - I'll take a look into this
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure-Samples/cosmos-db-design-patterns/issues/33#issuecomment-2386239261, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A33RTUCQPLRHGYTV5IFRJBLZZK2AHAVCNFSM6AAAAABO2JC666VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBWGIZTSMRWGE.
You are receiving this because you were mentioned.Message ID: @.***>

[like]  Eiji Nishimura reacted to your message:

From: Jasmine Greenaway @.>
Sent: Tuesday, October 1, 2024 4:15:43 PM
To: Azure-Samples/cosmos-db-design-patterns @.>
Cc: Eiji Nishimura @.>; Mention @.>
Subject: Re: [Azure-Samples/cosmos-db-design-patterns] The URI returns http://:7086 and no data is generated. (Issue #33)
The function needed to be changed to an isolated worker model, some things see PR here: #34https://github.com/Azure-Samples/cosmos-db-design-patterns/pull/34
Looking into other app issue as well.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure-Samples/cosmos-db-design-patterns/issues/33#issuecomment-2386437246, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A33RTUBGHIWFIGDJX3P62RTZZLDC7AVCNFSM6AAAAABO2JC666VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBWGQZTOMRUGY.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks for checking. I'll try this right away.

@paladique

"FUNCTIONS_WORKER_RUNTIME": "dotnet-isolated"
Now, [func start] started without any problems.
However, [dotnet run -- console.txt] still does not start. This time, [http://:] was returned.
Attached is the exception.
(https://github.com/user-attachments/files/17221128/dotnet.run.--.console.txt)

I merged @paladique PR for isolated mode. We are reviewing all of these now to make updates and fixes. We can take another look and get resolved. Thanks.


Show part of previous view controller image

Nice library. Thanks.
Is it possible to show a small part of the previously scrolled through image?

This is something that isn't built-in and as such would require internal modification.


A few thoughts...

Hey, thanks for posting this code on github! I am hoping to use it to update an older wiki that we use for a game. When I was going through your implementation, I came across a few things that I might recommend.

Include in the README how to override the default docker-composer file by using docker-compose.override.yml
Adding to the previous thought, add the ability to add or override the existing Dockerfile
Add ${DB_HOST} as an environment variable that can be changed easily
Separate/Remove all variables that are in LocalSettings.php that are not relevant to the build and place in ExtraLocalSettings.php

For example, $wgGroupPermissions['*']['createaccount'] = false; and etc...

Thank you for your time,
Rev

Hello Rev,
thanks for trying to use my code.

Please note that the docker-compose.yml file in the root directory is only intended for development purposes. The variables in that file are defined in a .env file in root directory that you can create by copying .env.default.

I added two new docker-compose example files that i recommend to use. However all these files should be treated as examples and rather be copied then overridden.
The preferred way to use this image is to pull its build from Docker Hub.
Command Line:
docker pull kristophjunge/mediawiki:latest

Docker Compose:
version: '2'
services:
  wiki:
    image: kristophjunge/mediawiki:latest

If you have to extend the image the proper way is to create a Dockerfile that extends the build on Docker Hub.

Dockerfile:
FROM kristophjunge/mediawiki:latest

I added a section "Extending this image" to the readme file.

The only place where variables in the form of ${} occur is in the docker-compose.yml in root directories which is again not intended to be used except for development.

To specify the database host for production use please use the environment variable.
Command-Line:
-e MEDIAWIKI_DB_HOST=dbhost

Docker Compose:
environment:
  MEDIAWIKI_DB_HOST: dbhost

I agree and have moved all configuration values that don't have a fixed default for a technical reason and that are not configurable by a docker environment variable into the default ExtraLocalSettings.php file. The moved configuration variables are $wgEnotifUserTalk, $wgEnotifWatchlist, $wgEmailAuthentication, $wgUseInstantCommons, $wgAuthenticationTokenVersion, $wgGroupPermissions.

If you still want to use this image please note that there are new versions with major changes. Have a look at the changelog.
Thank you,
Kristoph


Non-LEO orbits do not properly display

Orbital ring graphics do not properly display for orbits other than LEO.  MEO orbits display as partial arcs and GEO orbits only show dots (see attached image).
The orbital periods are correct coming out of SGP4 and into satrecs. The issue appears to be a time restriction (~90 mins) somewhere within the graphics potion of the application.  Thus, the LEO orbits display correctly (90 mins), the MEO orbits only show partial arcs (90 mins of a longer period), and GEO orbits (24-hour Period) only show a dot.

Take a look at https://github.com/Flowm/satvis/blob/master/src/modules/SatelliteProperties.js#L153
It sounds like you aren't passing it forward and backwards propagation parameters so it is defaulting to LEO.

Thanks for reaching out!
To be honest I've mostly focused on LEO satellites in the past as theses are the ones I'm actively involved with. The reported behavior is a combination of two things:

GEO satellites not having an orbit line at all:
Some entities (like the Sensor Cone) are intended/optimized for LEO satellites, so they were disabled for all satellites with an orbital period >12h.
-> As at least the Orbit visualizations still makes sense for GEO satellites I adjusted the filter to enable the orbit for GEO satellites.

MEO/GEO satellites not showing a full orbit:
As identified by @thkruz this stems from the fact, that the orbit line is based on a propagated position interpolated by the cesium framework. In this the sampling steps for propagation were hard-coded.
-> The new version is now calculating the sampling steps (along with the sampling interval for performance reasons) based on the orbital period as suggested.

A fixed version is now available in the next branch where I have been working  on some refactoring of the surrounding framework (vue3 upgrade, proper state management in the url) on and off for the past half year with the plan to merge it to the main branch in the near future.
Please reach out if there are any further questions or if I can assist the process of providing a cesium based visualization again on celestrak.org in any way 🙂

Florian,
Thank you!  We have forward the info you posted to our volunteer working this section.
We have a list of items we are working on, sent you a direct email for another issue we are addressing.

tagging @justinweaver1


[Feature request] Specify target to build in Dockerfile

Hi!
In for example a nginx+php-fpm deployment, I'm using a multistage docker build

Compile javascript assets
create a nginx image with all files
install php depenencies
create a php-fpm image with all files
Update deployment with the new nxinx and php-fpm images

@Richard87 can you assign me  this issue

@852hamza Sorry, I dont have access (I'm not a contributor)

ok


Finish Event not being emitted

I'm calling this from within a locomotive app, I would expect some sort of finish event to be emitted, but it seems like my callback is never called.
/*jslint node:true */
"use strict";
Var Busboy = require("busboy"),
      fs = require('fs');

module.exports = function (next) {
  if (this.req.method && this.req.method === "POST") {
      var busboy = new Busboy({headers: this.req.headers});
      busboy.on('file', function (fieldname, file, filename, encoding, mimetype) {
          if (typeof this.files === "undefined") {
              this.files = [];
          }

          this.files.push({field: fieldname, file: file});
      }.bind(this));

      busboy.on("field", function (key, value, keyTruncated, valueTruncated) {
          this.req.query[key] = value;
      }.bind(this));

      busboy.on("finish", function () {
         // THIS NEVER GETS CALLED
          console.log("\n\n\n INSIDE FINISH FUNCTION");]
          next();
      });

      this.req.pipe(busboy);
  } else {
      next();
  }
};

Are you actually reading from your file streams? If you don't, you won't see the finish event because the stream backpressure mechanism is doing its thing.

Okay, that does make sense...  I'd planned on adding a subsequent filter and passing the streams into that filter downstream (as it were).  This is due to not wanting to start another Asynch in the middle of the pipe and having the operation of that Asynch complete after the results were required.
I felt that it would be cleaner to do something along the lines of
 GetFiles(next);
 ProcessFiles(next);
 ProcessOutput();

  Run( GetFiles(ProcessFiles(ProcessOutput);

But if I need to do something with the file streams  I will look at:
 GetFiles(next) {
      foreach(file).processFiles();
  }
  ProcessOuput();

   Run();

I'm a little scared about the Asynch return timing as I outlined above but I'll see what I can do with it.

So I've been working towards resolving this and while I believe this is my error, I just can't seem to figure out what it is.
Here's what I've got for code:
/*jslint node:true */
"use strict";
var cloudinary = require('cloudinary'),
    Busboy = require("busboy"),
    fs = require('fs');

module.exports = function (next) {

    if (this.req.method && this.req.method === "POST") {
        var busboy = new Busboy({headers: this.req.headers});
           busboy.on('file', function (fieldname, file, filename, encoding, mimetype) {
                var stream = cloudinary.uploader.upload_stream(function (result) {
                this.req.query[fieldname] = result.url;
            }.bind(this));

            file.on('data', stream.write);
            file.on('end', stream.end);
        }.bind(this));
        busboy.on("field", function (key, value, keyTruncated, valueTruncated) {
            this.req.query[key] = value;
         }.bind(this));

        busboy.on("finish", function () {
           next();
        });
    this.req.pipe(busboy);
    } else {
        next();
    }
};

Which gives me:
_stream_writable.js:180
  if (state.ended)
           ^
TypeError: Cannot read property 'ended' of undefined
    at FileStream.Writable.write (_stream_writable.js:180:12)
    at FileStream.emit (events.js:95:17)
    at FileStream.<anonymous> (_stream_readable.js:765:14)
    at FileStream.emit (events.js:92:17)
    at emitReadable_ (_stream_readable.js:427:10)
    at emitReadable (_stream_readable.js:423:5)
    at readableAddChunk (_stream_readable.js:166:9)
    at FileStream.Readable.push (_stream_readable.js:128:10)
    at PartStream.onData     (/Users/crispensmith/Documents/ctstore/Site/node_modules/busboy/lib/types/multipart.js:215:28)
at PartStream.emit (events.js:95:17)
20 Mar 00:13:00 - [nodemon] app crashed - waiting for file changes before starting...

Any help you could provide would be greatly appreciated.  I'm quite new to streams so it's probably something very simple that I'm not catching.

What version of node are you using?

0.10.36 on a Mac

Ok I think I see the problem. You're passing stream.write and stream.end as event handlers. This won't work like you'd expect because when you pass those functions, it's passing the bare function with no context attached to it. So when a data event occurs for example, the function is called but now the this inside stream.write is not pointing to stream anymore.
Typically the way to solve this is to bind the function with stream.write.bind(stream) or using a closure like file.on('data', function(d) { stream.write(d); });. However since you already have a real Writable stream, you can just pipe the two together:
var stream = cloudinary.uploader.upload_stream(function (result) {
  this.req.query[fieldname] = result.url;
}.bind(this));

file.pipe(stream);

This way is much simpler and backpressure is handled automatically.

Thanks for a very thorough explanation, that makes excellent sense now.

Unfortunately my original fears about timing seem to have played out and the Cloudinary callback  may be called after the initial stream completes.  The way I see it there are two options: 1) I can wire in some sort of debouncing tool or 2) I can write to disk and then read from there.
In either case, I believe this can be closed as the original concern was resolved and stemmed from my lack of familiarity with streams & busboy.


Update notification receiver

This PR updates notification receiver.

Coverage remained the same at 100.0% when pulling 19db519c1bd5da4a19d7240019ab9e2aea205d1d on fisuda:feature/receiver into 05916d0684a705762cc708bd62ed31a2896d8801 on lets-fiware:main.


Performing an URLSessionUploadTask in background?

Hello,
Is it possible using Moya to perform an URLSessionUploadTask in background to send for example image even if the user return to home?
Thanks in advance!

Hey @XavierDK. You can create your own Manager and pass it to the MoyaProvider:
let configuration = URLSessionConfiguration.background(withIdentifier: "com.example.app.background")
let manager = Manager(configuration: configuration)

Let us know if it helps!

Thank you @sunshinejr! It's exactly what I was searching about.

Glad it worked! 🎉

It seems this is not possible any more with the latest version since it depends on Alamofire 5 which disabled background sessions: https://github.com/Alamofire/Alamofire/commit/1d5411cdde887cb9c8d282239fa4a44d42bd5e9f
Any other solutions?

How can I do that? As @teameh said it is no longer available this usage?

How can I do that? As @teameh said it is no longer available this usage?

@yunustek At the moment it looks like the way you can solve it by switching from Moya/Alamofire to native URLSession.

Okay thanks @polurezov8

For other people looking for a solution : a plugin can surround the requests with the start/end background tasks, like the following:
`class MoyaBackgroundPlugin : PluginType {
var bgTask : UIBackgroundTaskIdentifier = UIBackgroundTaskIdentifier.invalid
var numInFlight = 0
func prepare(_ request: URLRequest, target: TargetType) -> URLRequest {
    if numInFlight == 0 {
        bgTask = UIApplication.shared.beginBackgroundTask(withName: "Moya bg plugin") {
            if self.bgTask != UIBackgroundTaskIdentifier.invalid {
                UIApplication.shared.endBackgroundTask(self.bgTask)
                self.bgTask = .invalid
                self.numInFlight = 0
            }
        }
    }
    numInFlight+=1
    return request
}

func didReceive(_ result: Result<Response, MoyaError>, target: TargetType) {
    numInFlight -= 1
    if (numInFlight <= 0) {
        UIApplication.shared.endBackgroundTask(self.bgTask)
        self.bgTask = .invalid
    }
}

}`


Sudo-less environment

Codecov Report

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

@@          Coverage Diff          @@
##           master    #79   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           9      9
  Lines         270    270
=====================================
  Hits          270    270

Changes Unknown when pulling cbd955ca6f2bc40b532b2cd1388350b7f7025119 on no-sudo into ** on master**.

Changes Unknown when pulling cbd955ca6f2bc40b532b2cd1388350b7f7025119 on no-sudo into ** on master**.

Changes Unknown when pulling cbd955ca6f2bc40b532b2cd1388350b7f7025119 on no-sudo into ** on master**.

Changes Unknown when pulling cbd955ca6f2bc40b532b2cd1388350b7f7025119 on no-sudo into ** on master**.

Changes Unknown when pulling cbd955ca6f2bc40b532b2cd1388350b7f7025119 on no-sudo into ** on master**.


remove support in swagger

remove the specific case for swagger-express-middleware

Coverage increased (+0.9%) to 100.0% when pulling 70b2ebb8bf06c32ce5fb6859f99ea87c2f226db3 on remove-support-in-swagger into b1bc1ea353aeca3da76d2909d165955a3910d24f on master.


ss238 - Added new value 'Tissue' for DNA source

This will need to be done for the other manifests, (tube/library).
You'll also need to add the new type to the DNA_SOURCES array in sample.rb


Fixes 404 error when authenticating with v2 reg

As discussed here https://github.com/containerd/nerdctl/issues/715 nerdctl login was failing due to the missing trailing / - https://github.com/containerd/nerdctl/pull/724 is the original PR and as requested by @AkihiroSuda I have raised the same PR here instead

Tests failed have pushed a fix

This should be fixed in the login code not in the configuration. The paths are all prefixed with / when working with these values. The "login" endpoint in that case would be considered / appended to the path. The fix would end up changing the output of the configurations and is not consistent with the rest of the code.

Thanks for the input @dmcgowan I'm now a bit confused as to where to put this. I originally raised this fix in https://github.com/containerd/nerdctl/pull/724#discussion_r790359547 but needed to raise it here as it looks like nerdctl pulls these values from this function call https://github.com/containerd/nerdctl/blob/cce68fbf3b92239de64d6b9f7cf92b0d309dc8e2/cmd/nerdctl/login.go#L236 - I've checked this fix works so should the original PR be reopened and the adjustment made to nerdctl?

@dylanrhysscott probably https://github.com/containerd/nerdctl/blob/master/cmd/nerdctl/login.go#L262. The rh.Path represents the path to the API and the / would be added to get the HTTP endpoint. It is common for / to be stripped from paths and they should be explicitly added (or checked/cleaned then added) when forming the URL path.

@dylanrhysscott probably https://github.com/containerd/nerdctl/blob/master/cmd/nerdctl/login.go#L262. The rh.Path represents the path to the API and the / would be added to get the HTTP endpoint. It is common for / to be stripped from paths and they should be explicitly added (or checked/cleaned then added) when forming the URL path.

Thanks @dmcgowan  looks like I'll need to reopen my previous PR - please could you reopen this? https://github.com/containerd/nerdctl/pull/724 and close this one?

@dylanrhysscott probably https://github.com/containerd/nerdctl/blob/master/cmd/nerdctl/login.go#L262. The rh.Path represents the path to the API and the / would be added to get the HTTP endpoint. It is common for / to be stripped from paths and they should be explicitly added (or checked/cleaned then added) when forming the URL path.

Thanks @dmcgowan looks like I'll need to reopen my previous PR - please could you reopen this? containerd/nerdctl#724 and close this one? @AkihiroSuda

Sure


[BUG] Hyperf\Database\Connection::setPdo(): Send of 5 bytes failed with errno=32 Broken pipe！

Execute the command and paste the result below.
Command: uname -a && php -v && composer info | grep hyperf && php --ri swoole
Darwin lait-635.local 21.6.0 Darwin Kernel Version 21.6.0: Sun Nov  6 23:31:16 PST 2022; root:xnu-8020.240.14~1/RELEASE_X86_64 x86_64
PHP 8.1.18 (cli) (built: Apr 14 2023 05:13:59) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.18, Copyright (c) Zend Technologies
    with Zend OPcache v8.1.18, Copyright (c), by Zend Technologies
hyperf/amqp                        v3.0.18   A amqplib for hyperf.
hyperf/async-queue                 v3.0.18   A async queue component for hyperf.
hyperf/cache                       v3.0.18   A cache component for hyperf.
hyperf/code-parser                 v3.0.18   A code parser component for Hyperf.
hyperf/codec                       v3.0.1    A codec component for Hyperf.
hyperf/collection                  v3.0.18   Hyperf Collection package which come from illuminate/collections
hyperf/command                     v3.0.18   Command for hyperf
hyperf/conditionable               v3.0.0    Hyperf Macroable package which come from illuminate/conditionable
hyperf/config                      v3.0.18   An independent component that provides configuration container.
hyperf/context                     v3.0.18   A coroutine/application context library.
hyperf/contract                    v3.0.10   The contracts of Hyperf.
hyperf/coordinator                 v3.0.18   Hyperf Coordinator
hyperf/coroutine                   v3.0.16   Hyperf Coroutine
hyperf/crontab                     v3.0.18   A crontab component for Hyperf.
hyperf/database                    v3.0.18   A flexible database library.
hyperf/db-connection               v3.0.18   A hyperf db connection handler for hyperf/database.
hyperf/devtool                     v3.0.18   A Devtool for Hyperf.
hyperf/di                          v3.0.18   A DI for Hyperf.
hyperf/dispatcher                  v3.0.16   A HTTP Server for Hyperf.
hyperf/engine                      v2.8.1    Coroutine engine provided by swoole.
hyperf/engine-contract             v1.7.0    Contract for Coroutine Engine
hyperf/event                       v3.0.0    an event manager that implements PSR-14.
hyperf/exception-handler           v3.0.18   Exception handler for hyperf
hyperf/filesystem                  v3.0.18   flysystem integration for hyperf
hyperf/framework                   v3.0.18   A coroutine framework that focuses on hyperspeed and flexible, specifically use for build microservices and middlewares.
hyperf/guzzle                      v3.0.18   Swoole coroutine handler for guzzle
hyperf/http-message                v3.0.18   microservice framework base on swoole
hyperf/http-server                 v3.0.18   A HTTP Server for Hyperf.
hyperf/ide-helper                  v3.0.18   IDE help files for Hyperf.
hyperf/logger                      v3.0.18   A logger component for hyperf.
hyperf/macroable                   v3.0.0    Hyperf Macroable package which come from illuminate/macroable
hyperf/memory                      v3.0.0    An independent component that use to operate and manage memory.
hyperf/model-cache                 v3.0.18   A model cache component for hyperf.
hyperf/model-listener              v3.0.18   A model listener for Hyperf.
hyperf/paginator                   v3.0.18   A paginator component for hyperf.
hyperf/pipeline                    v3.0.16   Hyperf Macroable package which come from illuminate/pipeline
hyperf/pool                        v3.0.18   An independent universal connection pool component.
hyperf/process                     v3.0.18   A process component for hyperf.
hyperf/redis                       v3.0.18   A redis component for hyperf.
hyperf/serializer                  v3.0.0    A serializer component for Hyperf.
hyperf/server                      v3.0.18   A base server library for Hyperf.
hyperf/snowflake                   v3.0.18   A snowflake library
hyperf/stringable                  v3.0.18   Hyperf Stringable package which come from illuminate/support
hyperf/support                     v3.0.1    A support component for Hyperf.
hyperf/tappable                    v3.0.0    Hyperf Macroable package which come from illuminate/tappable
hyperf/testing                     v3.0.18   Testing for hyperf
hyperf/translation                 v3.0.18   An independent translation component, forked by illuminate/translation.
hyperf/utils                       v3.0.18   A tools package that could help developer solved the problem quickly.
hyperf/validation                  v3.0.18   hyperf validation
hyperf/watcher                     v3.0.18   Hot reload watcher for Hyperf
hyperf/websocket-server            v3.0.18   A websocket server library for Hyperf.
tangwei/dto                        v2.1.3    php hyperf dto

swoole

Swoole => enabled
Author => Swoole Team<EMAIL_ADDRESS>Version => 5.0.2
Built => Apr 27 2023 10:52:28
coroutine => enabled with boost asm context
kqueue => enabled
rwlock => enabled
sockets => enabled
openssl => OpenSSL 1.1.1n  15 Mar 2022
dtls => enabled
http2 => enabled
json => enabled
curl-native => enabled
pcre => enabled
c-ares => 1.18.1
zlib => 1.2.11
brotli => E16777225/D16777225
mysqlnd => enabled
async_redis => enabled

Directive => Local Value => Master Value
swoole.enable_coroutine => On => On
swoole.enable_library => On => On
swoole.enable_preemptive_scheduler => Off => Off
swoole.display_errors => On => On
swoole.use_shortname => Off => Off
swoole.unixsock_buffer_size => 262144 => 262144

Description:
hyperf所有组件都更新到最新版 后还是会出现这个错误。很乱。就感觉是mysql随时会断
[INFO] SQL[17.92 ms] select * from `system_menu` where `system_menu`.`deleted_at` is null order by `parent_id` asc, `sort` desc
[INFO] SQL[2.37 ms] select * from `system_menu` where `status` = '' and `name` like '%%' and `system_menu`.`deleted_at` is null order by `parent_id` asc, `sort` desc
[INFO] SQL[0.77 ms] select * from `system_menu` where `status` = '' and `name` like '%%' and `system_menu`.`deleted_at` is null order by `parent_id` asc, `sort` desc
[INFO] SQL[0.85 ms] select * from `system_menu` where `status` = '' and `name` like '%%' and `system_menu`.`deleted_at` is null order by `parent_id` asc, `sort` desc
[INFO] SQL[0.69 ms] select * from `system_menu` where `status` = '' and `name` like '%%' and `system_menu`.`deleted_at` is null order by `parent_id` asc, `sort` desc
[INFO] WebSocket [ user close connect for message server: id > 1, fd > 69, time > 2023-04-27 13:22:30 ]
[INFO] WebSocket [ user connection to message server: id > 1, fd > 85, time > 2023-04-27 13:22:32 ]
[INFO] SQL[4.01 ms] select `id`, `parent_id`, `id` as `value`, `name` as `label` from `system_menu` where `status` = '1' and `type` = 'M' and `system_menu`.`deleted_at` is null order by `sort` desc
PHP Notice:  Hyperf\Database\Connection::setPdo(): Send of 5 bytes failed with errno=32 Broken pipe in /Users/lait/server/github/MineAdmin/vendor/hyperf/database/src/Connection.php on line 641

Notice: Hyperf\Database\Connection::setPdo(): Send of 5 bytes failed with errno=32 Broken pipe in /Users/lait/server/github/MineAdmin/vendor/hyperf/database/src/Connection.php on line 641
[2023-04-27 21:22:32 $17770.0]  WARNING Server::check_worker_exit_status(): worker(pid=17776, id=5) abnormal exit, status=0, signal=11
A bug occurred in Swoole-v5.0.2, please report it.
The Swoole developers probably don't know about it,
and unless you report it, chances are it won't be fixed.
You can read How to report a bug doc before submitting any bug reports:
>> https://github.com/swoole/swoole-src/blob/master/.github/ISSUE.md
Please do not send bug reports in the mailing list or personal letters.
The issue page is also suitable to submit feature requests.

OS: Darwin 21.6.0 Darwin Kernel Version 21.6.0: Sun Nov  6 23:31:16 PST 2022; root:xnu-8020.240.14~1/RELEASE_X86_64 x86_64
GCC_VERSION: Apple LLVM 12.0.0 (clang-1<IP_ADDRESS>)
OPENSSL_VERSION: OpenSSL 1.1.1n  15 Mar 2022
PHP_VERSION : 8.1.18

[WARNING] Database connection refreshing.
[WARNING] Database connection refreshed.
[INFO] SQL[2.96 ms] select count(*) as aggregate from `system_queue_message` where     id IN (
        SELECT `message_id` FROM `system_queue_message_receive` WHERE `user_id` = '1'
        AND if ('1' <> 'all', `read_status` = '1', ' 1 = 1 ')
    )
[INFO] SQL[0.75 ms] select count(*) as aggregate from `system_queue_message` where     id IN (
        SELECT `message_id` FROM `system_queue_message_receive` WHERE `user_id` = '1'
        AND if ('1' <> 'all', `read_status` = '1', ' 1 = 1 ')
    )
PHP Notice:  Hyperf\Database\Connection::setReadPdo(): Send of 5 bytes failed with errno=32 Broken pipe in /Users/lait/server/github/MineAdmin/vendor/hyperf/database/src/Connection.php on line 657

Notice: Hyperf\Database\Connection::setReadPdo(): Send of 5 bytes failed with errno=32 Broken pipe in /Users/lait/server/github/MineAdmin/vendor/hyperf/database/src/Connection.php on line 657
[2023-04-27 21:19:09 $17770.0]  WARNING Server::check_worker_exit_status(): worker(pid=17775, id=4) abnormal exit, status=0, signal=11
A bug occurred in Swoole-v5.0.2, please report it.
The Swoole developers probably don't know about it,
and unless you report it, chances are it won't be fixed.
You can read How to report a bug doc before submitting any bug reports:
>> https://github.com/swoole/swoole-src/blob/master/.github/ISSUE.md
Please do not send bug reports in the mailing list or personal letters.
The issue page is also suitable to submit feature requests.

OS: Darwin 21.6.0 Darwin Kernel Version 21.6.0: Sun Nov  6 23:31:16 PST 2022; root:xnu-8020.240.14~1/RELEASE_X86_64 x86_64
GCC_VERSION: Apple LLVM 12.0.0 (clang-1<IP_ADDRESS>)
OPENSSL_VERSION: OpenSSL 1.1.1n  15 Mar 2022
PHP_VERSION : 8.1.18

Steps To Reproduce:
使用的代码是 https://gitee.com/mineadmin/mineadmin 拉下来后更新所有composer包到最新后正常运行
没有做任何改动.

把这个 Hyperf\ExceptionHandler\Listener\ErrorExceptionHandler::class 扔到 config/autoload/listeners.php 里


Consider if we should stringify non-string tokens

Following up on #695 and #712, which converted string attributes from strong-typed tokens to stringified tokens, we should consider what to do with other attribute types.
Namely, there are a few attributes in the CloudFormation spec that return string arrays and there one that returns JSON.
Also, we might want to consider what to do for tokens that represent numbers. There are non at the moment in CloudFormation, but an example would be port number.

I would say, same as for string values: we identify a range of special values in the domain that we use to represent tokens.
For numbers, we have two options:

Hide information in the unused bits of a NaN. That works in-memory (and is in fact sort-of what this unused space was intended for), but we'll have to take care at the JSII border when serializing them, to make it explicit that it's a NaN-w-extra information and JSII front-end implementors will have to carry these bits around as well. Might not be super portable.

Shrink the range of the double. We can reserve some bits at the upper or lower ranges of the range to encode token numbers. This is probably more portable than the previous one, but again we have to pay attention in jsii land that (a) all languages use doubles, not floats and (b) the stringification of the double contains enough detail that all bits are preserved and properly deserialized.

For lists, we'll have a special singleton list (["${THISISATOKENLIST}"]). However, the operations on lists are no longer trivial. The built-in operations MUST NOT be used anymore, because they cannot do the right thing in all cases.
Instead, we'll have to have functions which people use to operate on arrays, which either do the actual operation on a literal string list, or return a CloudFormation intrinsic in case of a tokenized list.

array.push(), array.concat(): prohibited, CloudFormation has no equivalent.
array.length: prohibited, CloudFormation has no equivalent.
array[5]: translates to Fn.select().
array.join(): maps to Fn.join()

We could potentially use ES6 Proxies to make this work transparently in JavaScript land (user cannot tell the difference and don't have to resort to special functions), but we'd have to stop serializing arrays as "just data" in that case and treat them like interfaces into jsii-land (think List<string> instead of string[]).

That reminds me, string.split() must also not be used anymore today, since it cannot work in the face of tokenized strings. It should produce a Fn.split() when encountering a Tokenized string (that you're NOT allowed to take the length of :stuck_out_tongue_closed_eyes: )


[2/N] executor pass the complete config to worker/modelrunner

continue of https://github.com/vllm-project/vllm/pull/9933

I personally find it confusing that we call the members/arguments vllm_config when it is of type EngineConfig. I would personally choose the name engine_config, but I dont have religion around it.
Other than the ModelConfig --> EngineConfig comment, approving to unblock

I personally find it confusing that we call the members/arguments vllm_config when it is of type EngineConfig. I would personally choose the name engine_config, but I dont have religion around it.

Agree .. alternatively EngineConfig could be renamed to VLLMConfig :)

@robertgshaw2-neuralmagic @njhill renamed to VllmConfig  👍


put nf-test pipeline level guidelines into guidelines

guidelines should be in guidelines 🙂

note that the deploy preview doesn't show update docs at the moment

yes, please


multi-backends solution

There are multi-backends in my platform,include arm cpu+mali gpu+different npus.
Can I use tvm to dynamic scheduling different backend?

Please post such question to the discuss forum.

please me give me a link for discuss forum.


Unknown attribute batterySupport with value enabled

Describe the bug
When running the example allVehicles.py I get an error message.
To Reproduce
Run allVehicles.py
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
The example should run without error.
Logs
/vehicles/XXXXX/domains/batterySupport/batterySupportStatus: Unknown attribute batterySupport with value enabled

Additional context
The error message causes issues with downstream use of the library, such as https://github.com/mitch-dc/volkswagen_we_connect_id

Hi @ColinRobbins: Sorry, I cannot reproduce this with my car. Do you have an ID with SW Version 2.4 or 3.0? Are these all messages you are getting?

Yes, they are the only messages I am seeing - everything else is OK.
How do I tell which S/W version?  I am guessing the latest as I’ve only taken delivery in the last few weeks.
Happy to try and produce debugging info, if guided as to what you need.

I added the missing attribute. Could you give the example a new try using the main branch?

Yes, this works in my test environment.
Thank you.
I doing so, I found a few dependancies missing, I'll make a PR shortly.

Perfect, I will make that part of the next release then.
Regarding the dependencies I answered in the PR.

Hi.
Did this make it into 0.45.1?
I am still seeing the error in 0.45.1, but not when I use your main branch.

No, will be part of 0.45.2. My car is currently in the workshop for the 2.4 update. When I have it back I can prepare an update

This is released in 0.46.0


Dependency on @hapi/joi is causing issues now that Joi has broken off of Hapi

The joi project is no longer maintained within the Hapi project. It is now in its own namespace. https://joi.dev/
As a result, when using hapi v20+ you must include both @hapi/joi (which is stale) and joi in your project dependencies to get @now-ims/hapi-auth-now to work. The project doesn't appear to list @hapi/joi as a dependency and this has led to build failures and workarounds.

We apologize for this issue and will update the package to use the appropriate dependencies

Thank you!


delete '/' to sync tf2

To support tf2, I deleted slash in front of topic name where it had set in global and local costmap param.yaml.
https://github.com/ros-planning/navigation/issues/794

Hey do you know when this will land on the release build servers?  At the moment in our workshops we have to build from source or patch the installed system files.  Neither are that desirable since this fix works great.

@ROBOTIS-Kayman @RyanJaehyunShim Could you guys please advise on the estimated release date for this fix^^?

@robotpilot I'd appreciate it if you guys could advise on the timeline for bloom-releasing this package to get this fix out.

@AAlon Sorry for late. We are going to work on bloom-releasing today.

@AAlon  FYI, the current state is as follows:
For ROS 1 Kinetic Kame:
-> https://github.com/ros/rosdistro/pull/22058
-> https://github.com/ROBOTIS-GIT/turtlebot3/releases/tag/1.2.1
For ROS 1 Melodic Morenia:
-> https://github.com/ros/rosdistro/pull/22059
-> https://github.com/ROBOTIS-GIT/turtlebot3/releases/tag/1.2.2


Automatic error sending with debugged middleware

The middleware was not working with Django 1.8.7, and was not documented in the Readme.

Hi @NicolasMarlier, thanks for the PR!
It looks like this is because from django.utils.deprecation import MiddlewareMixin is used, which is not available in django 1.8.7. To also maintain support for django 1.10, I think we should do something like this instead:
try:
    from django.utils.deprecation import MiddlewareMixin
except ImportError:
    MiddlewareMixin = object

Looking at https://docs.djangoproject.com/en/1.10/topics/http/middleware/#upgrading-pre-django-1-10-style-middleware, I see the advantage of using from django.utils.deprecation import MiddlewareMixin is to provide support for both new-style MIDDLEWARE and old-style MIDDLEWARE_CLASSES. I'd love to see us keep this compatibility if we can.
Adding some docs on the usage like you've done is great. It would be even better if we mentioned the new-style MIDDLEWARE usage as well.
Thanks again!

Hey Zach,
Thanks for your comments, I updated the code accordingly.
Best,

done

@nickciesinski LGTM, go ahead and merge.

@zachgoldstein Probably the wrong nick haha

ah blarg sorry. @NicolasMarlier ^^ LGTM


DateTime visualizer, leveraging NodaTime

Visualizer targets DateTime.

Debuggee-side
Will return different Instants (some sort of serializable representation of Instant, because NodeTime 3.0 doesn't support binary serialization) based on the Kind property:

Utc - returns a single Instant representing that UTC DateTime, using the g pattern
Local - returns a LocalDateTime (using the r round-trip pattern, and the local time zone id of the debuggee (as a string identifier)
Unspecified - returns both of the above

Because a LocalDateTime with a time zone applied to it could return 1, 2, or 0 dates, we'll actually return the following for Local or Unspecified:
public class AppliedZoneResult {
    public Instant? AsLocal1 { get; }
    public Instant? AsLocal2 { get; }
    public AppliedZoneResult(LocalDateTime local, DateTimeZone dateTimeZone) {
        try {
            AsLocal1 = local.InZoneStrictly(dateTimeZone).ToInstant();
        } catch (AmbiguousTimeException ambiguousTimeException) {
            AsLocal1 = ambiguousTimeException.EarlierMapping.ToInstant();
            AsLocal2 = ambiguousTimeException.LaterMapping.ToInstant();
        } catch (SkippedTimeException) {
            // intentionally blank
        }
    }
}

The Config should also contain two lists, for additional BCL and TZDB identifiers. They won't be used on the debuggee side, only on the debugger side.

Debugger-side

Converter for Instant that returns a two-line string: the UTC date, and the date in the local system time

Main window:

UTC date (using converter)
Local (for the debuggee) date time
Local (for debuggee) time zone
Local instant (combination of the above two, using converter)

Two datagrids, one for BCL ids, and one for tzdb ids. Each grid has two columns:

id
instant -- local date time applied to timezone with selected id

(Alternatively, a single grid with both types of ids.)

Config should also have a Period offset, which offsets all instants / local date times by the specified periods.
Every instant should have a context menu:

Open new period -> opens a dialog to select a period, and opens a new window with that period offset

Consider also integrating TimeZoneInfo Explorer.

Copy Watch Expression: integrates period offset, time zone and local date time (when appropriate)

We're making an assumption here. A LocalDateTime with a time zone applied to it will return a result (1, 0, or 2 dates). Can we assume the result will be the same for a given LocalDateTime (with the same calendar) and a given time zone, whether the time zone application is done in the debugger or debuggee?

Some implementation details:

Target type -- DateTime
Will return a different number of ZonedDateTimes based on DateTimeKind: UTC (1), Local (1, using timezone from debuggee), Unspecified (2)
Config allows returning additional ZonedDateTime: Windows identifiers, and tzdb identifiers
Converter to show ZonedDateTime with time zone and offset
Response class:

class Response : ISerializable {
    public List<KeyValuePair<string, ZonedDateTime>> Basic {get;set;}
    public List<KeyValuePair<string, ZonedDateTime>> Optional {get;set;}
}

UI:

grid of basic; grid of optional
quantity textbox
period element combobox
open in new window;

Now live at https://github.com/zspitz/DateTimeVisualizer.


bug:  codeium can not use

Did you check docs and existing issues?

[x] I have read all the LazyVim docs
[x] I have updated the plugin to the latest version before submitting this issue
[x] I have searched the existing issues of LazyVim
[x] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.10.2
Operating system/version
win11
Describe the bug

Steps To Reproduce
i only update the lazyvim, and the codeium can not to use
Expected Behavior
the codeium can not to use
Repro
vim.env.LAZY_STDPATH = ".repro"
load(vim.fn.system("curl -s https://raw.githubusercontent.com/folke/lazy.nvim/main/bootstrap.lua"))()

require("lazy.minit").repro({
  spec = {
    { "LazyVim/LazyVim", import = "lazyvim.plugins" },
    -- add any other plugins here
  },
})

Have you tried with fresh LazyVim installation from scratch? Because I can't reproduce it. Granted I'm on Linux and not Windows and maybe it has to do something with the OS environment.

I have tried reinstalling, but the problem still persists.It may be a problem with Windows

That can only happen if you override the config function for the blink spec, or you're not using the blink extra.

@folke According to your statement, if I annotate it, I should report the same error. but ......
https://github.com/user-attachments/assets/439b924d-9e01-47d8-9d7d-3a2574863f9c

And, I only use the Codeium plugin, which has the same issue

If you don't use LazyExtras, then you have to manually make sure you import the correct extras in the correct order.
In this case, the blink extra should be above the codeium one.
I really recommend using LazyExtras instead.

Thank you, I have been using your template for a long time, and it can still work like this. I am now using Extras


Makefile fixes for ubuntu

Thanks for adding this. I guess spray doesn't compile on Ubuntu without linking libz, otherwise why should you add this. Could you figure out which component requires libz? I'm on Fedora where it compiles without libz, too.

Libdwarf. It really should pick the pkg-config LIBS


Add scrollbar

Please makes sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's Contributing Guide.
[ ] Make sure you are merging your commits to dev branch.
[ ] Rebase before creating a PR to keep commit history clear.
[ ] Add some descriptions and refer relative issues for you PR.

conflict了

。。。。。。。。。。。。。。。。。。。。。。。。。。


update certificate

I've created a new certificate to use in our tools since our old certificate was deprecated  in the latest version of chrome.

@tosta1989,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Non Linear Ads

Non Linear Ads are not working in this player , can any one check this issue i am mentioning the links:
VAST 1.0 Non Linear
http://demo.tremorvideo.com/proddev/vast/vast1Nonlinear.xml
VAST 2.0 Non Linear
http://demo.tremorvideo.com/proddev/vast/vast2Nonlinear.xml

Hi there - we currently don't support non-linear ads in our iOS or Android SDKs. You can see a list of supported features here. Thanks!


Fermentrack dead after latest update

Latest update killed my instance.
ImportError: No module named backports.configparser
Is that the bottom of all the stderr logs.

Are you running under Python 3? How long ago was the previous upgrade (did you have the message to upgrade for a long time before clicking it, or did you just see it for the update I pushed two days ago?)
Nothing in the latest update should have triggered that aside from the upgrade of the environment that happens on every upgrade. It’s possible one of the dependencies introduced a bug - but that’s a strange bug to introduce.

I haven't heard anything more on this, so I'm going to go ahead and close the issue for now. If you are still having this problem please feel free to reopen!


Update compressors used for release assets

Our release assets are pretty large.
We're using gzip on them and there are better alternatives out there.
XZ (2-3x better than gzip) is I think ubiquitous in all tar implementations now. I think any user will be able to untar (even on windows without forcing need for 7z (e.g. either python tools or cmake tar command should suffice but we need to check)
Apple's lzfse might be best for macOS. We should switch hdiutil command in masonry to use either lzma or lzfse to reduce size of apple assets.

Did a test with hdiutil on macOS adding -format UXXX args to -create option

ULFO (lzfse-compressed image and is available in OS X 10.11+ only) gave only 5% better
ULMO (lzma or xz available on 10.15+ only) gave 18% better
UDZO (default zlib (level 1)) is our default
UDZO with -imagekey zlib-level=9 only 5% better

@griffin28 recommended having a look at set(CPACK_SOURCE_GENERATOR "TXZ")

I do get about 30% improvement in size of .dmg file using macOS Archive Utility. It creates a .cpgz file...
-rw-r--r--@ 1 miller86  3268  731145338 Oct 14 15:18 VisIt-3.3.1.dmg
-rw-r--r--  1 miller86  3640  516743650 Oct 14 23:27 VisIt-3.3.1.dmg.cpgz


Improve AIP accessibility

Long term goal...
De-couple from CPG as much as possible - the ideal state is "AIP in a box", with a set of defined inputs and documentation on how to generate/format them

Take PED/Ext IDs/Phenotypes from a single file - don't expect to be generating this at runtime using Metamist. That would then become the input for panel-HPO matching, and PanelApp queries
Fully define input/output paths & folders - don't expect to be generating paths on the fly using output_path or similar

Longer term:

Annotated VCF as source data for the process (SVs are... less simple). Requires a way of ingesting annotated VCFs into a MT, specifically a way to split the VEP CSQ field contents.
Define an annotation contract, e.g. a dependency on VEP nnn, with a way of providing ClinVar/SpliceAI annotations.
...?

Simpler/less pressing:

Swap out Peddy for peds - we only need a PED file object, the extra Peddy functionality just makes the install weightier, for no benefit in this project

Ahhhh I remember that tutorial! Very cool sneks
I think as a first pass I'd like to try abstracting all the elements that are tightly coupled to specific non-generic services, i.e. we get pedigrees , ext. IDs and phenotypes from metamist, so create a stage which uses metamist to create an intermediate file with pedigree data & phenotypes in prod-pipes, and AIP would just maintain a contract against a PED file with 2 additional cols. Bonus points if that develops on top of existing standards like ingesting that information through phenopackets.
The ambition being that we create a very tightly defined set of generic(ish) files needed to run AIP end to end, and the application itself can sit in a docker container with as few additional dependencies as possible and run as a black-box

Partially/mostly done -


Segmentation fault at end of Falcon7b T3K demo (perf-mode)

The Falcon7b T3K demo (perf-mode) is running into a segfault at the end of the test:

Pipeline failure (121506d): https://github.com/tenstorrent/tt-metal/actions/runs/9505905480/job/26202055404
Last passing run: https://github.com/tenstorrent/tt-metal/actions/runs/9440536599
fyi @uaydonat

Not able to reproduce locally on main (5196c48052) on sjc-nmrk-t3001

The issue starts occuring with cc77001, @abhullar-tt could you please take a look?

The issue starts occuring with cc77001, @abhullar-tt could you please take a look?

yeah, my changes should have been noops for WH but Ill take a look in case I missed something

Seeing this new segfault as well on Mixtral attn test:
pytest -svv models/demos/t3000/mixtral8x7b/tests/test_mixtral_attention.py
Fatal Python error: Segmentation fault

Current thread 0x00007f802bf61740 (most recent call first):
  File "/home/mtairum/tt-metal-new/ttnn/ttnn/multi_device.py", line 63 in close_device_mesh
  File "/home/mtairum/tt-metal-new/conftest.py", line 398 in t3k_device_mesh
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/fixtures.py", line 896 in _teardown_yield_fixture
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/fixtures.py", line 1012 in finish
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/runner.py", line 546 in teardown_exact
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/runner.py", line 188 in pytest_runtest_teardown
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_callers.py", line 103 in _multicall
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_manager.py", line 120 in _hookexec
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_hooks.py", line 513 in __call__
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/runner.py", line 241 in <lambda>
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/runner.py", line 341 in from_call
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/runner.py", line 240 in call_and_report
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/runner.py", line 136 in runtestprotocol
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/runner.py", line 116 in pytest_runtest_protocol
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_callers.py", line 103 in _multicall
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_manager.py", line 120 in _hookexec
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_hooks.py", line 513 in __call__
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/main.py", line 364 in pytest_runtestloop
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_callers.py", line 103 in _multicall
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_manager.py", line 120 in _hookexec
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_hooks.py", line 513 in __call__
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/main.py", line 339 in _main
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/main.py", line 285 in wrap_session
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/main.py", line 332 in pytest_cmdline_main
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_callers.py", line 103 in _multicall
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_manager.py", line 120 in _hookexec
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/pluggy/_hooks.py", line 513 in __call__
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/config/__init__.py", line 178 in main
  File "/home/mtairum/tt-metal-new/python_env/lib/python3.8/site-packages/_pytest/config/__init__.py", line 206 in console_main
  File "/home/mtairum/tt-metal-new/python_env/bin/pytest", line 8 in <module>
Segmentation fault (core dumped)

Still trying to repro the segfaul, Im unable to reproduce on t3005

@skhorasganiTT I don't think my commit necessarily caused this and Im not able to reproduce locally, you also mentioned that falcon is passing on CI?
There could be another source of the segfault

Yes I was able to reproduce it twice yesterday on sjc-nmrk-t3001 with that commit, but on latest main (21761c6862) it is not appearing anymore for Falcon7b. I agree, it could be something that was exposed but not caused by your commit. @mtairum would you be able to verify that the Mixtral error is due to cc77001? If it appears to be unrelated, maybe we can close this.

On latest main 0d24864536a482740ee73086ab80eac2f09e010d I'm not seeing the segfault on the mixtral attn test.
However I do see it with the mixtral demo:
pytest models/demos/t3000/mixtral8x7b/demo/demo.py::test_mixtral8x7b_demo[wormhole_b0-True-general_weights]

Were you able to verify whether it starts occurring with the commit mentioned above? If it is not related to that, we need to investigate further / find an appropriate owner for this.

I just tested Mixtral attn  on commit cc77001d07b56fb6a059751bdf6e1f850479f0d7, and don't see the segfault. On T3002.
I think you can close this issue. I'll open one for Mixtral with its segfaults.

Sounds good, thanks @mtairum


Add helper name to helper not found exception (#10)

Plus unit tests

I am no longer maintaining this project any more. I will be marking it as abandoned soon...shall I link to your fork? Will you be maintaining it?


Crypto Performance on ARMv8

I'm using rclone on an ARMv8, and the CPU usage is about 20x higher than it is on comparable x86 CPU, presumably due to crypto performance. I am seeing about 47% CPU usage on a 2.0GHz ARMv8 core for uploading at 20Mbit/s (speed of the uplink). A Nehalem CPU is showing about 2.3% CPU usage to do the same thing.
Is there a way to link against a different crypto library at least for the SSL part, to leverage more optimized crypto code? Linking against something like OpenSSL would also allow for leveraging asynchronous crypto offload engine using /dev/crypto, particularly on low end ARM CPUs that really need all the help they can get.

I suspect this is because on ARMv8 go doesn't use the built in AES instructions.
There is an issue about it here: https://github.com/golang/go/issues/18498 - fixing that will bring the speed back to openssl speeds.  It looks like some progres has been made on it - it might be in go 1.9.
Which hardware are you using?
If you try doing curl or wget to an https address on the ARMv8 board, how much CPU does that use for a given amount of throughput?
There is a project to use openssl from go here: https://github.com/spacemonkeygo/openssl - it requires a patched net/http which makes it a bit tricky to install.

I am using an APM X-Gene 1 (Gigabyte MP30-AR0 board). OpenSSL speed tests at 95MB/s on AES128 and 75MB/s on AES256.
wget shows about 25-30% CPU usage downloading at about 80Mbit on a https download, which is still far off what openssl seems capable of.
A better implementation in Go still wouldn't immediately address the hardware accelerator/coprocessor offload, though, via the /dev/crypto interface, which OpenSSL can do.

A better implementation in Go still wouldn't immediately address the hardware accelerator/coprocessor offload

Are there other crypto accelerators built in to your board other than the ARMv8 AES instructions which I believe are a standard part of ARMv8?

via the /dev/crypto interface, which OpenSSL can do.

Are you using linux? According to wikipedia /dev/crypto was never merged in mainline linux.
It looks like a great idea and go could directly target it if available...

The ARMv8 I use doesn't have a separate crypto coprocessor, but I have a LOT of other ARMs, including Nvidia Tegra and Marvell Kirkwood based ones that do have asynchronous crypto offload available via /dev/crypto, and they really do need it for something like this.
It is true that cryptodev was never merged into mainline, but it is pretty trivial to build it, and it exposes all of the crypto drivers built into the kernel, including the ones for hardware accelerators. Given that other crypto libraries (other than OpenSSL) have effectively non-existant support for /dev/crypto, for a long time now OpenSSL has been the only game in town for crypto offload.

If you wanted to have a go with https://github.com/spacemonkeygo/openssl
Then it would be relatively easy, you'd just need to add this in fs/http.go in the Transport method
t.DialTLS = func(network, address string) (net.Conn, error) {
        return openssl.Dial(network, address, context.TODO(), 0)
}

There might be a bit more stuff, but that plus the docs should give you a start.
I'm unlikely to do this by default with rclone as adding dependencies to C libraries breaks the cross compilation, but it could be set up so you could supply a build tag to make it work quite easily.

minio (another storage system) saw 100x speedup in crypto operations in Go on ARMv8 by using hardware support.
https://blog.minio.io/accelerating-sha256-by-100x-in-golang-on-arm-1517225f5ff4#.foika9d4j
The code base is at https://github.com/minio/sha256-simd

That's looks like a worthwhile option for sure, but it doesn't seem to come with the USP of OpenSSL which is /dev/crypto support for leveraging kernel crypto drivers, including the ones for hardware crypto offload.

If someone would do the same thing for the AES instructions and upstream that to the go project that would solve all the problems for ARMv8!

The upstream issue, "crypto/aes: add dedicated asm version of AES, AES-GCM for arm64", is now scheduled for Go 1.10 (slipped from Go 1.9).


IGListDiffable protocol does not inherit from NSObject

I cannot check if object isKindOfClass:
- (BOOL)isEqualToDiffableObject:(id<IGListDiffable>)object
{
   if(![object isKindOfClass:[SectionItemWide class]]) // <- error
     ;
   }
}

I have to ugly-cast first:
- (BOOL)isEqualToDiffableObject:(id<IGListDiffable>)object
{
    NSObject<IGListDiffable> *o = (NSObject<IGListDiffable> *)object;
    if ([o isKindOfClass:[SectionItemWide class]]) {
     ;
    }
}

Is this intentional?
New issue checklist

[TRUE] I have reviewed the README and documentation
[TRUE] I have searched existing issues and this is not a duplicate

General information

IGListKit version: HEAD
iOS version(s):10+
CocoaPods/Carthage version: latest
Xcode version: 8.2.1
Devices/Simulators affected: all
Reproducible in the demo project? (Yes/No): YES
Related issues: -

We removed <NSObject> b/c of Swift interop issues. If IGListDiffable inherits from the NSObject then all swift classes must conform to NSObjectProtocol, which is unnecessary.
It does make this a little annoying though, I agree. In Instagram we just make the parameter id (not id<IGListDiffable>) to make the class checks easy. Not sure if there's a better solution out there.

That doesn't make sense. id is just syntactic sugar for NSObject *. You shouldn't need any casting.
Sounds like SectionItemWide is forward-declared instead of imported... ?

@rnystrom Yes, thanks, it is the simplest. I'll do that... (remove conformance to the IGListDiffable). Language purists may drop dead now, but, oh, well... :-)
@jessesquires It all happens in SectionItemWide:
@implementation SectionItemWide

- (id<NSObject>)diffIdentifier
{
    return self;
}

- (instancetype)initWithIndex:(NSUInteger)index
{
    self = [super init];
    return self;
}

// change to:
//- (BOOL)isEqualToDiffableObject:(id)object
- (BOOL)isEqualToDiffableObject:(id<IGListDiffable>)object
{
    if ([self isEqual:object])
        return YES;

    NSObject<IGListDiffable> *o = (NSObject<IGListDiffable> *)object;
    if ([o isKindOfClass:[SectionItemWide class]]) {
        /// ...
    }
    // something something in the month of May
    return NO;
}
@end

oh my bad. just checked this.
bizarre.

@jessesquires ya it's a PITA, not really sure why id<Something> wouldn't conform to NSObject since its still an id...


gatk error after last night's (successful) upgrade

Hello!
After successful upgrade last night with
bcbio_nextgen.py upgrade -u development --tools --data
I have an error:
gatk -R ~/work/tools/bcbio/genomes/Hsapiens/GRCh37/seq/GRCh37.fa -T VariantsToTable -V 394-gatk-haplotype.decomposed.vcf.gz -F CHROM -F POS -F REF -F ALT -F DP -GF DP -GF AD -o test.table --allowMissingData
'import site' failed; use -v for traceback
Traceback (most recent call last):
  File "/home/naumenko/work/tools/bin/gatk", line 9, in <module>
    import os
  File "/home/naumenko/work/tools/bcbio/anaconda/lib/python2.7/os.py", line 400, in <module>
    import UserDict
  File "/home/naumenko/work/tools/bcbio/anaconda/lib/python2.7/UserDict.py", line 116, in <module>
    import _abcoll
  File "/home/naumenko/work/tools/bcbio/anaconda/lib/python2.7/_abcoll.py", line 456, in <module>
    KeysView.register(type({}.viewkeys()))
AttributeError: 'dict' object has no attribute 'viewkeys'

Without wrapper it works:
java -Xmx1g -jar ~/work/tools/bcbio/toolplus/gatk/3.6-0-g89b7209/GenomeAnalysisTK.jar -R ~/work/tools/bcbio/genomes/Hsapiens/GRCh37/seq/GRCh37.fa -T VariantsToTable -V 394-gatk-haplotype.decomposed.vcf.gz -F CHROM -F POS -F REF -F ALT -F DP -GF DP -GF AD -o test.table --allowMissingData
INFO  11:41:15,227 HelpFormatter - ---------------------------------------------------------------------------------
INFO  11:41:15,231 HelpFormatter - The Genome Analysis Toolkit (GATK) v3.6-0-g89b7209, Compiled 2016/06/01 22:27:29
INFO  11:41:15,234 HelpFormatter - Copyright (c) 2010-2016 The Broad Institute
INFO  11:41:15,234 HelpFormatter - For support and documentation go to https://www.broadinstitute.org/gatk
INFO  11:41:15,234 HelpFormatter - [Tue Dec 13 11:41:15 EST 2016] Executing on Linux 2.6.32-642.6.2.el6.x86_64 amd64
INFO  11:41:15,235 HelpFormatter - Java HotSpot(TM) 64-Bit Server VM 1.8.0_91-b14 JdkDeflater
INFO  11:41:15,238 HelpFormatter - Program Args: -R /home/naumenko/work/tools/bcbio/genomes/Hsapiens/GRCh37/seq/GRCh37.fa -T VariantsToTable -V 394-gatk-haplotype.decomposed.vcf.gz -F CHROM -F POS -F REF -F ALT -F DP -GF DP -GF AD -o test.table --allowMissingData
INFO  11:41:15,264 HelpFormatter - Executing as naumenko@qlogin1 on Linux 2.6.32-642.6.2.el6.x86_64 amd64; Java HotSpot(TM) 64-Bit Server VM 1.8.0_91-b14.
INFO  11:41:15,265 HelpFormatter - Date/Time: 2016/12/13 11:41:15
INFO  11:41:15,265 HelpFormatter - ---------------------------------------------------------------------------------
INFO  11:41:15,266 HelpFormatter - ---------------------------------------------------------------------------------
INFO  11:41:15,298 GenomeAnalysisEngine - Strictness is SILENT
INFO  11:41:15,734 GenomeAnalysisEngine - Downsampling Settings: Method: BY_SAMPLE, Target Coverage: 1000
WARN  11:41:15,977 IndexDictionaryUtils - Track variant doesn't have a sequence dictionary built in, skipping dictionary validation
INFO  11:41:16,178 GenomeAnalysisEngine - Preparing for traversal
INFO  11:41:16,192 GenomeAnalysisEngine - Done preparing for traversal
INFO  11:41:16,194 ProgressMeter - [INITIALIZATION COMPLETE; STARTING PROCESSING]
INFO  11:41:16,194 ProgressMeter -                 | processed |    time |    per 1M |           |   total | remaining
INFO  11:41:16,194 ProgressMeter -        Location |     sites | elapsed |     sites | completed | runtime |   runtime
INFO  11:41:36,677 ProgressMeter -            done    369157.0    20.0 s      55.0 s      100.0%    20.0 s       0.0 s
INFO  11:41:36,678 ProgressMeter - Total runtime 20.48 secs, 0.34 min, 0.01 hours
------------------------------------------------------------------------------------------
Done. There were 1 WARN messages, the first 1 are repeated below.
WARN  11:41:15,977 IndexDictionaryUtils - Track variant doesn't have a sequence dictionary built in, skipping dictionary validation

Before the update I used the wrapper for many times.
Could you please point out what is happening?
May this cause problems with variant calling in the variant2 pipeline?
Thanks!
Sergey

Sergey;
Sorry about the problem, I believe the issue was that the new wrapper around GATK uses Python and did not explicitly specify the path to the anaconda installed python. It likely picked up a different version and caused incompatibility issues. I pushed a fix that resolves this (and also bumps GATK to the recently released 3.7). You can update the wrapper with:
bcbio_conda install -c bioconda gatk

and will need to re-run gatk-register (you can use --noversioncheck if you want to use something older than 3.7). Hope this fixes things for you.

Thanks Brad, now it works.


Test Issue created at Sat, 03 Jul 2021 13:10:45 GMT

Test issue body  Sat, 03 Jul 2021 13:10:45 GMT

Comment on issue at Sat, 03 Jul 2021 13:10:47 GMT


Consolidate retry config for OpenAI

Extend the retry configuration strategy introduced in https://github.com/spring-projects/spring-ai/pull/1040 to the rest of the OpenAI model integration.
Main aspects:

A RetryTemplate can be autoconfigured for each model type integration, or else a default one is used.
The RetryTemplate is used to wrap the call to the model service for targeted HTTP retry logic instead of wrapping the entire implementation logic.
The model request object is assembled in a dedicated method for improved readability and for sharing it between standard and streaming implementations.

Thanks @ThomasVitale


Verified address - nested address

Description
When searching for an address in some cases multiple  addresses can fall under the same address.
In this case the result ends up being undefined undefined
We should look to handle this use case.

@ShadeWyrm I suspect this might be one of those situations where the retrievable property comes back false?

Yup! Perfect, will use that to update. :)


Bug/schema changes NSW public schools master dataset 20240530

fixing
#5
#4

ok so tested again and the source data looks to be fixed up. To re un do fixes to data types and column changes.


bug - changes are not propagated to java language server

Extension Name: vscode-gradle
Extension Version: 3.16.4
OS Version: Arch Linux
VSCode version: 1.95.3
Describe the bug
With multimodule gradle project, changing dependency version is not synchronized to the Java Lnaguage Server.
To Reproduce
Open this project, multimodule.zip

Go to services/module-a/src/main/java/com/one/demo/DemoApplication.java, navigate to SpringApplication and open the class file and check the version of the library where the file comes from, it will be shown it is 3.4.0
Go to buildSrc/build.gradle -> line 10 and change implementation 'org.springframework.boot:spring-boot-gradle-plugin:3.3.0
Repeat step one to check the version, it will still show the previous version

Another way of reproducing

Go to services/module-a/build.gradle, add some dependency, you can add, implementation 'io.micrometer:micrometer-registry-prometheus:1.11.3'
Go to services/module-a/src/main/java/com/one/demo/DemoApplication.java, and try to import PrometheusMeterRegistry, which comes from that package, https://github.com/micrometer-metrics/micrometer/blob/main/implementations/micrometer-registry-prometheus/src/main/java/io/micrometer/prometheusmetrics/PrometheusMeterRegistry.java

Expected behavior
When dependencies are changed, or any gradle configuration the JDT should be updated as well to reflect the changes applied

Removing a dependency also doesn't remove it from VS Code Gradle plugin tree list. Likely same core issue - not updating properly on changes to build.gradle file. Restarting seems to be only thing that works. Been sketchy for a while, now.


Publish redacted comments on the BoPS API

Description of change
Publish redacted comments with tag and date on the BoPS API
This is so bops-applicants can pick it up and show it
Story Link
part 1 of https://trello.com/c/Xsc1xWiH/1653-publish-comments-so-everyone-can-see-the-comments

Want to add an example of this in the swagger docs?

Will do in a separate PR


Dependency on commons-io:commons-io:2.17.1-SNAPSHOT

In our projects we are updating the com.commercetools.sdk:commercetools-http-client dependency to 17.15.1 and are running into the issue that the transitive dependency com.commercetools.sdk:commercetools-async-http-client:17.15.1 depends on commons-io:commons-io:2.17.1-SNAPSHOT that can't be found in the public Maven repositories.
Can this maybe be changed to a non-SNAPSHOT version?

The SDK is not pulling a snapshot version, it refers to 2.11.0 of commons-io.
https://github.com/commercetools/commercetools-sdk-java-v2/blob/2e4c00a8799c5104b0c237c05a0ad1035a6dbde4/commercetools/commercetools-async-http-client/build.gradle#L5
https://github.com/commercetools/commercetools-sdk-java-v2/blob/2e4c00a8799c5104b0c237c05a0ad1035a6dbde4/gradle-scripts/extensions.gradle#L19-L22
https://github.com/commercetools/commercetools-sdk-java-v2/blob/2e4c00a8799c5104b0c237c05a0ad1035a6dbde4/licenses/index.json#L539-L551
The AHC dependency depends on commons-io 2.6.
So it may be that the dependency tree on your side resolves it incorrectly.

Thanks for the quick response, it seems this was caused by <snapshots><enabled>true</enabled></snapshots> in my Maven settings.xml for the proxy in combination with the [2.8.0,) version.


Publisher custom video renderer shows video with inverted colour only

Describe the bug
In my application, we want to get publisher frame buffer to be used to draw skeleton using MLKit. For this as per my understanding we have to use Custom Video Renderer and from this class "onFrame(frame: Frame)" function we will get frame buffer and use the buffer for getting image. The issue is when we use the CustomVideoRenderer from examples of OpenTok, the video we gets has inverted colour.
Expected behavior
Expected behaviour is in case if we can get a custom renderer with normal video for Publisher class.
Additional context
Additionally, are we doing anything wrong for getting Publisher Frame? Also, I am not proficient in OpenGL implementation at all, if you can give example for "GLSurfaceView.Renderer" implementation.
Thanks a lot in advance.

Hi, I guess the issue is coming because of the YUV format of Frame Buffer. What I have tried at my end is using "io.github.crow_misia.libyuv" library to convert YUV format to ArgbBuffer format. But the issue is still same with green background in foreground. Can You please help on this?
Thanks

@WCRohit , just checked up with the team, the inverting was intentional.  Reason being, we wanted the "custom" part of the rendering to be evident.

@v-kpheng Thanks for your reply, Understood the reason. Closing the ticket.


[RELEASE] Data Prepper 2.0.1

Is your feature request related to a problem? Please describe
The Data Prepper maintainers are releasing 2.0.1.
Planned release date: Oct 20, 2022
Describe the solution you'd like
After the Data Prepper maintainers completes the staging release build, please run the promotion job. A Data Prepper maintainer will supply the DATA_PREPPER_BUILD_NUMBER parameter.
Parameters:

VERSION: 2.0.1
RELEASE_MAJOR_TAG: true
RELEASE_LATEST_TAG: true
DATA_PREPPER_BUILD_NUMBER - To be supplied later.

Describe alternatives you've considered
No response
Additional context
No response

@dlvenable Please use the new template to create the issue for releasing standalone components.

Thanks for pointing this out! We will use this going forward.

Please set:
DATA_PREPPER_BUILD_NUMBER - 31
https://github.com/opensearch-project/data-prepper/actions/runs/3313631040

Release on maven:

Docker:
https://hub.docker.com/r/opensearchproject/data-prepper/tags?page=1&name=2.0.1
ECR:
https://gallery.ecr.aws/opensearchproject/data-prepper

Maven central will be available soon:

https://repo1.maven.org/maven2/org/opensearch/dataprepper/data-prepper-api/2.0.1

Awaiting project website:

https://github.com/opensearch-project/project-website/pull/1092

Awaiting @dlvenable to complete the verification before closing the issue.
Thanks.

Thanks @peterzhuamazon for helping with the release. I have verified the binaries and the changes to project-website. Those are deploying now. I'll close this issue as there is nothing else I expect is needed.


Delete task shown in dropdown

When I want to change a task in a phase, I see previously used (deleted) task in the list

Is it a cache problem? Or the tasks are never removed?

Is it a cache problem? Or the tasks are never removed?

Looks like cache problem. Just like the queue problem

Makes sense.


args[property].trim() is not a function

2019-04-17 15:49:29 : DTeam                          !query affects=casting level by 3&int by 1
 47 /home/ntang/lorebot/lorebot.js:1154
 48                 if (args[property].trim().indexOf(' by ') > 0) {       // !query affects=damroll by 2,hitroll by 2
 49                                    ^
 50
 51 TypeError: args[property].trim is not a function
 52     at ProcessQuery (/home/ntang/lorebot/lorebot.js:1154:36)
 53     at Client.client.on (/home/ntang/lorebot/lorebot.js:1552:9)
 54     at Client.emit (events.js:160:13)
 55     at MessageCreateHandler.handle (/home/ntang/lorebot/node_modules/discord.js/src/client/websocket/packets/handler    s/MessageCreate.js:9:34)
 56     at WebSocketPacketManager.handle (/home/ntang/lorebot/node_modules/discord.js/src/client/websocket/packets/WebSo    cketPacketManager.js:103:65)
 57     at WebSocketConnection.onPacket (/home/ntang/lorebot/node_modules/discord.js/src/client/websocket/WebSocketConne    ction.js:330:35)
 58     at WebSocketConnection.onMessage (/home/ntang/lorebot/node_modules/discord.js/src/client/websocket/WebSocketConn    ection.js:293:17)
 59     at WebSocket.onMessage (/home/ntang/lorebot/node_modules/ws/lib/event-target.js:120:16)
 60     at WebSocket.emit (events.js:160:13)
 61     at Receiver._receiver.onmessage (/home/ntang/lorebot/node_modules/ws/lib/websocket.js:145:47)
 62 error: Forever detected script exited with code: 1
 63 error: Script restart attempt #1
 64 Lorebot ready!
 65 2019-04-17 15:50:57 : DTeam                          !query affects=casting level by 1,int by 1

Have you tested perhaps - error suggest to me it might be a casting issue. Like first cast the object to a string, then you could use the trim().
args[property].toString().trim()....

Have you tested perhaps converting to string - error suggests to me it might be a casting issue. Like first cast the object to a string, then maybe you could use the trim().
args[property].toString().trim()....

I did think of that and when I ran the test I received the same error, only instead of .trim it said .toString is the invalid function suggesting (in my opinion) that args[property] is where the issue lies.
If you look at args[property] when you do affects=affect1,affect2 versus affects=affect1&affects=affect2, it is different (output from querystring) and that difference is what I think is causing the problem but I'm unsure why or how to resolve it.  easier to remove the unnecessary .functions.

ideally fixed with 66ff52e6f4a7cbcc1e8a6051dbc6c61212d6134a


Swift 3 support

Swift 3 introduced breaking changes:
Compiling Swift Module 'URITemplate' (1 sources)
/home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/Sources/URITemplate.swift:149:40: error: 'joinWithSeparator' has been renamed to 'joined'
        return op!.prefix + expansions.joinWithSeparator(op!.joiner)
                                       ^~~~~~~~~~~~~~~~~
                                       joined
Swift.Sequence:3:17: note: 'joinWithSeparator' has been explicitly marked unavailable here
    public func joinWithSeparator(separator: String) -> String
                ^
/home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/Sources/URITemplate.swift:149:40: error: 'joinWithSeparator' has been renamed to 'joined'
        return op!.prefix + expansions.joinWithSeparator(op!.joiner)
                                       ^~~~~~~~~~~~~~~~~
                                       joined
Swift.Sequence:3:17: note: 'joinWithSeparator' has been explicitly marked unavailable here
    public func joinWithSeparator(separator: String) -> String
                ^
/home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/Sources/URITemplate.swift:179:20: error: 'joinWithSeparator' has been renamed to 'joined'
    return regexes.joinWithSeparator((op ?? StringExpansion()).joiner)
                   ^~~~~~~~~~~~~~~~~
                   joined
Swift.Sequence:3:17: note: 'joinWithSeparator' has been explicitly marked unavailable here
    public func joinWithSeparator(separator: String) -> String
                ^
/home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/Sources/URITemplate.swift:215:44: error: 'enumerate()' has been renamed to 'enumerated'
        for (index, variable) in variables.enumerate() {
                                           ^~~~~~~~~
                                           enumerated
Swift.Sequence:3:17: note: 'enumerate()' has been explicitly marked unavailable here
    public func enumerate() -> EnumeratedSequence<Self>
                ^
/home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/Sources/URITemplate.swift:242:32: error: 'reverse()' has been renamed to 'reversed'
    for match in Array(matches.reverse()) {
                               ^~~~~~~
                               reversed
Swift.Collection:3:17: note: 'reverse()' has been explicitly marked unavailable here
    public func reverse() -> ReverseRandomAccessCollection<Self>
                ^
/home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/Sources/URITemplate.swift:311:27: error: value of type 'Index' (aka 'String.CharacterView.Index') has no member 'advancedBy'
        let index = value.startIndex.advancedBy(prefix, limit: value.endIndex)
                    ~~~~~~^~~~~~~~~~ ~~~~~~~~~~
/home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/Sources/URITemplate.swift:322:54: error: 'joinWithSeparator' has been renamed to 'joined'
    return value.map { self.expand(value: "\($0)") }.joinWithSeparator(joiner)
                                                     ^~~~~~~~~~~~~~~~~
                                                     joined
Swift.Sequence:3:17: note: 'joinWithSeparator' has been explicitly marked unavailable here
    public func joinWithSeparator(separator: String) -> String
                ^
/home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/Sources/URITemplate.swift:335:21: error: 'joinWithSeparator' has been renamed to 'joined'
    return elements.joinWithSeparator(joiner)
                    ^~~~~~~~~~~~~~~~~
                    joined
Swift.Sequence:3:17: note: 'joinWithSeparator' has been explicitly marked unavailable here
    public func joinWithSeparator(separator: String) -> String
                ^
/home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/Sources/URITemplate.swift:444:7: error: 'joinWithSeparator' has been renamed to 'joined'
    }.joinWithSeparator(joiner)
      ^~~~~~~~~~~~~~~~~
      joined
Swift.Sequence:3:17: note: 'joinWithSeparator' has been explicitly marked unavailable here
    public func joinWithSeparator(separator: String) -> String
                ^
/home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/Sources/URITemplate.swift:492:9: error: 'joinWithSeparator' has been renamed to 'joined'
      }.joinWithSeparator(joiner)
        ^~~~~~~~~~~~~~~~~
        joined
Swift.Sequence:3:17: note: 'joinWithSeparator' has been explicitly marked unavailable here
    public func joinWithSeparator(separator: String) -> String
                ^
/home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/Sources/URITemplate.swift:547:7: error: 'joinWithSeparator' has been renamed to 'joined'
    }.joinWithSeparator(joiner)
      ^~~~~~~~~~~~~~~~~
      joined
Swift.Sequence:3:17: note: 'joinWithSeparator' has been explicitly marked unavailable here
    public func joinWithSeparator(separator: String) -> String
                ^
<unknown>:0: error: build had 1 command failures
error: exit(1): /home/saulius/.swiftenv/versions/DEVELOPMENT-SNAPSHOT-2016-03-24-a/usr/bin/swift-build-tool -f /home/saulius/workspace/Swifton/Packages/URITemplate-1.3.1/.build/debug.yaml default

I fixed it.
https://github.com/damuellen/URITemplate.swift/commit/8f1130eaaa73b57fe2bd7f97de903fb401eca2f1

@damuellen thanks! I think it's worth to keep both Swift 2 and Swift 3 compatibility.

@kylef any chance you could merge this?

As of URITemplate 2.0, Swift 3 is now supported.

Awesome! Thanks for the great work @kylef 🍻


Python tests not working with pytest 5.0.0

The constant _pytest.main.EXIT_TESTSFAILED is no longer available in pytest 5.0.

I did a quick checkout of 5.0 to see if the constant was moved but it appears to've been completely factored out (or the name was changed). I haven't yet looked at the history to see where pytest went instead. That would be my next line of investigation


Popup with Third party extensions

Closing in favor of #134, which is the same thing but cleaned up.


changed texts on datasources page

Description

I changed the texts on the datasources page ( from "Create new" to "Create new API" and from "CURL to CURL import").
Here is how the updated version looks.

Type of change

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

How Has This Been Tested?
Since, only texts were changed, I did not perform any new test. I however ensured that all tests passed.
Checklist:

[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
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

/ok-to-test sha=5db8cd2


Update in the grayscale style

The grayscale style and the green style have some slight differences in the fonts sizes and weights, some margins, etc.
The grayscale style needs to be updated to it's the actual green style but in gray.
Figma design has been shared.

PR #161


Costco availability not being picked up

Costco in zip 98382 has availability which you're not getting, and 4 other locations in the Seattle metro area had availability within the last 2 hours.

It's because they introduced a new serviceId (119).

I believe this should be working for Washington again. I think the issue today may have been slightly different from the original issue you reported a couple days ago, but it looks like Costco has finally unified all of their booking for all states under a single link. So that's actually good news as it simplifies a lot of things, but it tripped up my store updating/scanning process for extracting some of the other IDs necessary for appointment booking. I think this should be fixed by https://github.com/GUI/covid-vaccine-spotter/commit/aa3242de5ea48a09016d67e038597276b6abb0b9, and I'm now seeing Washington locations being picked up that weren't previously. So while I think this is working, I've seen some other small changes on their end that I'd like to dig more into just to double check. I'll try to investigate more tomorrow, but give a shout in the meantime if you notice anything else amiss. Thanks again for the heads up!

Yeah today's change is different from the one I originally filed... so we have the fun of redoing the scraper twice in one week. 🤦🏻♀

I think they haven't necessarily changed the offsite location link though it's hard to say for sure due to the J&J situation. https://book-costcopharmacy.appointment-plus.com/d0m1slrg/#/


Fix incorrect package name

Acceptance test was broken because of incorrect package name.

Codecov Report

Merging #53 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop      #53   +/-   ##
==========================================
  Coverage      50.27%   50.27%
  Complexity       343      343
==========================================
  Files             94       94
  Lines           1432     1432
  Branches          91       91
==========================================
  Hits             720      720
  Misses           670      670
  Partials          42       42

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 e18ca8a...55b1aa6. Read the comment docs.


Webpack failed to compile

Do you want to request a feature or report a bug?

Running on Windows 10
Not rendering .svg files

There's not enough info to tell exactly why. Please check the import carefully.
As per instructions provided at the issue template, support should go to Stack Overflow. Please re-open there.


Hyperparameter optimization

How are you optimizing hyperparameter space in this implementation?
Grid search or random search or bayesian?

I would recommend random search. See:
http://www.jmlr.org/papers/volume13/bergstra12a/bergstra12a.pdf

Is it already implemented in this model? and what is the loss function of this model with CRF as a classifier?

No, you would  need to implement it by yourself.
For the loss function see line 51:
https://github.com/UKPLab/elmo-bilstm-cnn-crf/blob/master/neuralnets/keraslayers/ChainCRF.py

okay.


fix for issue#10

What this PR does / why we need it:
vpn-seed-server pod never starts up and goes into CrashLoopBackoff
Which issue(s) this PR fixes:
Fixes #10
Special notes for your reviewer:
Release note:

Can you please also update the alpine image version in the shoot-client Dockerfile.


lighthouse improvements

i noticed that we were not completely accurate with our lighthouse benchmarking due to a the fact that h1 in our temporary design was too big on mobile and turned into the lcp-element which will not be the case once we are done.
the true lcp-element is a tricky one as it is in the featured-article block, which in turn has to read the query-index.json, so this is possibly kind of a worst case scenario.
/blog/ -> scripts.js -> featured-article.js -> query-index.json -> media_1298398237498273.webp -> wait for load.
as especially the mobile score is still highly constrained by bandwidth contention, i needed to make sure that nothing else gets loaded before we load the chain above. the good news is that we could improve from the current state even more by h2-push-ing 3 of the 4 files, however the impact of making sure that nothing else gets loaded before the actual LCP is quite dramatic... see below.
this is quite a significant change to block loading and sequencing, and i think it works well in this particular case. it may be harder in other cases where the images are actually already on the page.
current state: with ?lcp=simple
https://lighthouse--business-website--adobe.hlx.live/blog/?lcp=simple
https://developers.google.com/speed/pagespeed/insights/?url=https%3A%2F%2Flighthouse--business-website--adobe.hlx.live%2Fblog%2F%3Flcp%3Dsimple

improved state
https://lighthouse--business-website--adobe.hlx.live/blog/
https://developers.google.com/speed/pagespeed/insights/?url=https%3A%2F%2Flighthouse--business-website--adobe.hlx.live%2Fblog%2F&tab=mobile

@auniverseaway love those "Page Speed Insights Audit", they are more stable than i thought they would be...


Adding longer expression and opening it in the FEEL editor moves to the top of the screen

Describe the bug
When we add longer expressions in the respective field in the properties panel, try to open the editor it moves to the top of the screen
Steps to reproduce

Create a new BPMN diagram
In the properties panel choose a field supporting FEEL
Paste the test below or in general a longer text
open the editor
The editor jumps to the top of the screen
Dragging the editor is no longer possible

https://github.com/camunda/camunda-modeler/assets/85225325/3daa70f7-a4ac-41a0-918a-fedd439cac17
example expression:
replace(replace("### Roles this week

| Role | DRI |
|---|---|
| Moderator | @---moderator--- |
| Summary Writer | @---summaryWriter--- |

### Agenda

<!-- If information is documented that shall not be made available for all Camundi, please use and reference an [ELT confidental note](https://drive.google.com/drive/u/1/folders/1EVfpSkZIP3yY90brsNotytYh2SPuqsbJ)-->

#### Status Check-in

- [ ] Conduct status check-in: _ensure the ELT knows status of key-items on our roadmap. Consider next target release in [product hub](https://github.com/orgs/camunda/projects/9/views/31)_.

#### Discussion Topics

_Every ELT member can add discussion points here._

- [ ] ...

#### Management Bugs and Tasks

- [ ] Review [tasks](https://github.com/orgs/camunda/projects/52) that are currently `In progress`.
- [ ] Introduce new [tasks](https://github.com/orgs/camunda/projects/52) that were put into the `Inbox`, potentially assigning them (`/autoassign`) and moving to `Backlog` or `In progress`.

----------------------

Find recordings of all meetings [here](https://docs.google.com/document/d/1PfpCZ1eo8IdVvHdaehAH68VTQLJCRYaoxbih_mQsK9c/edit).

Find ELT confidental notes or link a new note [here](https://drive.google.com/drive/u/1/folders/1EVfpSkZIP3yY90brsNotytYh2SPuqsbJ).

Find all Weeklies (incl. past ones) [here](https://github.com/camunda/engineering-leadership-team/issues?q=label%3Aweekly+).

Cf. [weekly agenda](https://github.com/camunda/engineering-leadership-team/blob/main/.github/ISSUE_TEMPLATE/WEEKLY_NOTE.md), [manually create new agenda](https://github.com/camunda/engineering-leadership-team/issues/new?template=WEEKLY_NOTE.md&labels=weekly)", "---moderator---", nextModerator), "---summaryWriter---", nextSummaryWriter)

Expected behavior
The editor should not be misplaced
Environment

OS: [Windows 10]
Camunda Modeler Version: [5.17.0]
Execution Platform: [ Camunda 8]
Installed plug-ins: [...]

Additional context
No response

This probably has the same root-cause as https://github.com/camunda/camunda-modeler/issues/3969

Closing as a duplicate of https://github.com/camunda/camunda-modeler/issues/3969