popen handbrake_commend in text mode

This fixes an issue with printing multi-byte UTF-8 characters to the console on Windows.
Basically, printing a bare UTF-8 continuation byte ("\xC3"), without any following bytes, to the Windows console raises an error in ruby:
 ruby -e 's="\xc3"; print s'
Traceback (most recent call last):
        2: from -e:1:in `<main>'
        1: from -e:1:in `print'
-e:1:in `write': Input/output error @ io_write - <STDOUT> (Errno::EIO)

While this works:
ruby -e 's="\xc3\xb1"; print s'
ñ
(Note this doesn't seem to be a problem on Unix platforms — at least not Linux/Mac).
In the transcoding script, we're currently reading output from HandbrakeCLI in binary mode one byte at a time and printing it to the console one byte at a time.  Therefore if we encounter a multi-byte UTF-8 character, we have this problem.
By popen-ing the HandbrakeCLI in text mode, each_char iterates over multi-byte characters atomically, rather than iterating over each byte in the multi-byte sequence, thus avoiding the problem.
I'm not sure if you had a good reason for getting the HandbrakeCLI output in binary mode to begin with, so feel free to reject / go with an alternative fix as appropriate :)

@joshstaiger Thanks for submitting this patch! Nice detective work, sir. I'm glad it fixes the problem you were seeing and I don't dispute your logic. But... :)
...Two years ago @samhutchins changed that same code to use binary file mode when reading and writing console and log output from HandBrakeCLI. This eliminated redundant information and "console spew" on Windows by suppressing the EOL <-> CRLF conversion.
You can see that change here: https://github.com/donmelton/video_transcoding/commit/2859a278ed5fab2c2abd2b48a8210a14d9effef8
So, I want @samhutchins to review your patch to make sure that taking this won't cause a regression.
Also, we're using binary file mode now with all the other tools and all over the code so we need to think about whether we need to make changes elsewhere.
Sam?

Hmm, interesting.  I'm running the transcode script under wsl in Windows.  In that case I'd assume that no EOL conversion is necessary - which might explain why I'm not seeing any "console spew".  I don't have a regular Windows install outside wsl handy to test otherwise.
I had thought that maybe setting STDOUT to binary mode would fix the problem, but I'm still seeing the issue while writing to the console even if I do that (at least in ruby 2.5.1).
Alternatively, you could keep binary mode and specifically check for UTF-8 multi-byte prefix codes, buffering print output accordingly.  eg: throw bytes into a buffer while their high-order bit is set before printing.

I'll test on a non WSL installation soon

Yeah, this re-introduces the console spew
I think the failure on multi-byte characters is worse than the spew, so I'd be fine with reverting my original change. In an ideal world, we'd buffer the output to get the best of all worlds

Also worth noting, there were several commits to change the read mode that would need reverting

@samhutchins Thanks for testing so quickly, Sam!
Yeah, spew is preferable to a crash. I wish we could avoid both but buffering, as @joshstaiger suggests, is lot harder than it looks because of the way HandBrakeCLI overwrites lines to update progress.
Of course, if anyone wants to submit a working patch for that, I will gladly review it! :)

@samhutchins ...

Also worth noting, there were several commits to change the read mode that would need reverting

Yeah, the changes to binary file mode are all over the code with several tools. Should we change all of them? And do you want me to work up a patch for that after I merge @joshstaiger's pull request?

We should be consistent, so I think we should undo the binary mode everywhere. I’ll have a go at a buffering solution sometime soon

Or @joshstaiger could do it, lol

I just threw together https://github.com/donmelton/video_transcoding/pull/264, which seems to fix the crash without having to revert the binary mode.
There are probably some edge cases where it might drop a multi byte character if it occurs at the very end of the stream, but I'm not sure that's worth worrying about.

@joshstaiger Actually, it's the end-of-stream characters that were the problem for me dealing with the progress overwrite issue. But I'll try your patch out later this evening when I'm back at my development machine.

@joshstaiger OK, since I just took your patch from #264, I'm obviously going to reject this patch. But, boy, am I glad you submitted this in the first place! :)
Thanks, again!


#167316007 An endpoint to flag a suspecting property - [persisting data to a db]

What does this PR do?
An API endpoint for users to flag a suspecting property is created.
Description of tasks
The following endpoint should be working;
POST /api/v1/property/flag
after

creating a controller[API endpoint] to flag a suspecting property
creating validations for the controller
creating a route to access the endpoint
writing tests for the controller

How should this be manually tested/checked?
After cloning this repo, cd into this project, run it by typing the command _npm star_t, and test for the particular endpoint on postman, using localhost:5000/api/v1/property/flag
For the test, run npm test
Any background context you want to provide?
While testing, if you encounter any error and error messages, properly follow the error guidelines to get it working, this is so because the endpoint is well validated.
What are the relevant pivotal tracker stories?
167316007

Codecov Report

Merging #61 into development will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           development    #61   +/-   ##
==========================================
  Coverage          100%   100%
==========================================
  Files               11     14    +3
  Lines              309    350   +41
==========================================
+ Hits               309    350   +41

Impacted Files
Coverage Δ

SERVER/Middleware/validateUsers.js
100% <ø> (ø)
:arrow_up:

SERVER/Routes/index.js
100% <100%> (ø)
:arrow_up:

SERVER/Middleware/validateFlags.js
100% <100%> (ø)

SERVER/Tests/flagsTest.js
100% <100%> (ø)

SERVER/Controllers/flags.js
100% <100%> (ø)

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 7980aff...aa8c568. Read the comment docs.


mergeDeepWith providing keyPath instead of key.

The current mergeDeepWith passes the two values and the key that they share - would it make more sense to provide the entire keyPath? With just the key, it's pretty difficult to understand where the object exists within the structure.

Good idea!

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.

:+1:

:+1:


Tested it!

I tested render node & api server on MacOS Sierra 10.12.6 and Windows Server 2016 with AE CC 2018 Version 15.0.0 (Build 180). Works fine 👍

oh, great news, so it is actually working fine
and was it mp4 rendering, or avi/mov, or maybe any other ?

@Inlife on both systems it was mov

oh i see
and i almost sure new AE doesn't allow to setup the h264 output


Support for IP_PKTINFO control message

We are looking into a possibility of adding IP_PKTINFO /IPV6_PKTINFO  control message support into libuv. And I would like to consult this change before writing any code.
Problem
We have an application that responds on UDP. The listening UDP sockets are bound to INADDR/in6addr_any because we occasionally need to change IPs without restarting the application. In this setup, the source IP of the outgoing message can be chosen incorrectly as the operating system will take first address valid for the destination route.
This is what IP_PKTINFO solves for us. When this socket option is enabled and a datagram is received via recvmsg(), the destination address/interface is included in the control message. This control message can be then passed to sendmsg() and correct source IP is chosen as a result.
It seems that all supported platforms have similar socket option. I know (and have tested) on Linux, FreeBSD, and macOS.
Questions

Will you consider including this features into libuv upstream?
How do you think the API should look like? Can we do it without changing the API? Or should we add a new variant of receive and send?

Other past efforts
I have found some efforts to add this interface, none successful:

https://groups.google.com/forum/#!topic/libuv/YQBkiHG2bMk (adds new API)
https://github.com/nodejs/node-v0.x-archive/issues/6589#issuecomment-29289512 (modifies existing API)

It's come up a few times before (as you've seen) but the stumbling block has been cross-platform support.  With that in mind:

Will you consider including this features into libuv upstream?

Only if if it can be reasonably supported on a majority of supported platforms.  See https://github.com/nodejs/node-v0.x-archive/issues/6589#issuecomment-29287181 - I think that's still the state of things.
cc @libuv/collaborators

Only if if it can be reasonably supported on a majority of supported platforms. See nodejs/node-v0.x-archive#6589 (comment) - I think that's still the state of things.

I have seen this working reliably on Linux, FreeBSD, and OS X. We enabled this feature in Knot DNS some time ago, and yes - there was small workaround needed for OS X but it works fine to my knowledge. I cannot speak for Windows and Solaris.
So assuming we can make it work on these platforms, how do you think the API should look like?

how do you think the API should look like?

I don't have a proposal in mind ATM, ideally you can come up with something and we can comment on it to reach a point where we are all happy with it.  Some thoughts:

Consider making this for v2, that lifts the backwards compatible API / stable ABI requirement
I will request a LEP
Don't forget about Windows when making your design, it's a fundamentally different beast sometimes

Cheers!

Awesome. Thank you.

We need support for this in NodeJS. Anything that gives us the ability to get this information would be good. If the information is not available due to platform issues have those platforms return NULL or throw an error when specifying the option.

@krisives You've seen https://github.com/libuv/libuv/issues/1213#issuecomment-276896697, yes?  If you want to see this happen, that's where you should start.

Hey. I've workaround the missing IP_PKTINFO by implementing custom read/write operations and uv_pool_t as suggested by Saúl in offline conversation. There are some differences in how the control messages are interpreted on Linux and macOS. But once this is understood, it works very well. I don't have a code for Windows though.
I still would like to turn this into abstract libuv interface. I will do my best to find some time ASAP to write a proposal how the API could look like.

@fcelda Where is the code for IP_PKTINFO I wanted to check it? Thank you!

@saghul, was a LEP ever drafted?
Is there an active branch/fork somewhere where this is being developed?
We are in need of this to support a transparent proxy but from a time perspective, we may end up implementing our own solution using poll on a descriptor as an interim solution while we work through the libuv/uvw changes to support in the long term.

@bmagistro For what it's worth I ended up writing a C++ solution because of this not being ultimately available on nodejs as well as doing something like what you are talking about with a thin layer to do the ipv6 stuff.

Thanks for the info.
I took a closer look at IP_PKTINFO the other day and it doesn't quite meet our needs.  For the proxy we will need IP_RECVORIGDSTADDR since we also need to get the original destination port so going to put things on hold and just roll our own for now using poll_t.

@bmagistro Just to be clear (mostly for other readers) IP_RECVORIGDSTADDR is for BSD whereas you can get this info using IP_PKTINFO in Linux. On Linux you want to set the socket option IPV6_RECVPKTINFO to 1 then use the CMSG macros to pull out the cmsg_type == IP_PKTINFO
I only mention this because the flags and constants are frankly confusing and not amazingly documented, and they vary between platform.

Hi all, this seems like a much needed functionality. @fcelda do you know why is this conversation stalled? Is it 'windows' part? I can test this on FreeBSD/Linux/OSX if that can help moving this forward. @fcelda also, Is it possible for you to share the patch? Thanks!

Thanks for the patience. I'm not bringing any good news. I cannot share the code for uv_pool_t with IP_PKTINFO (IPV6_RECVPKTINFO) as it is, sorry. Also, I didn't get far in turning it into a LEP or patch. The Windows support is huge unknown for me, I didn't find time to research if there is an equivalent interface in Windows and I even don't have environment to test in.

fcelda wrote:
I have seen this working reliably on Linux, FreeBSD, and OS X. We enabled this feature in Knot DNS some time ago, and yes - there was small workaround needed for OS X but it works fine to my knowledge.
How did you get it to work on MacOSX?
When I do "rc = setsockopt(s, IPPROTO_IP, IP_PKTINFO, &opt, sizeof opt);
with opt being aqn unsigned int containing 1, rc = -1 and errno gets INVALIDARG.

@nbolyard It works for me. It's difficult to tell what's different in your setup with the amount of context you gave. Do you really have IPv4 UDP socket? For IPv6, you need to pass IPPROTO_IPV6 and IPV6_RECVPKTINFO into setsockopt.
#include <netinet/in.h>
#include <stdio.h>
#include <sys/socket.h>
#include <sys/types.h>
#include <unistd.h>

int main(int argc, char *argv[])
{
	int sock = socket(AF_INET, SOCK_DGRAM, 0);
	if (sock < 0) {
		perror("socket");
		return 1;
	}

	int on = 1;
	int ret = setsockopt(sock, IPPROTO_IP, IP_PKTINFO, &on, sizeof(on));
	if (ret != 0) {
		perror("setsockopt");
		close(sock);
		return 1;
	}

	close(sock);

	return 0;
}

$ gcc -Wall ./pktinfo.c && ./a.out; echo $?
0
$ uname -v
Darwin Kernel Version 17.7.0: Thu Jun 21 22:53:14 PDT 2018; root:xnu-4570.71.2~1/RELEASE_X86_64


Are paths updated for VS2019?

I'm viewing the version 2019 of this page but the paths still contains "2017" inside. Can you replace "2017" by "2019"?
I checked that these new paths work as expected.

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

ID: e8177d9b-f8d7-419e-d81d-ce25a44b084f
Version Independent ID: 43a69770-1eab-06e1-868c-3d8f1c32716d
Content: Generating Files with the TextTransform Utility - Visual Studio
Content Source: docs/modeling/generating-files-with-the-texttransform-utility.md
Product: visual-studio-windows
Technology: vs-ide-modeling
GitHub Login: @gewarren
Microsoft Alias: gewarren

I updated the paths accordingly, and the updated documentation should be live by tomorrow.


cannot build rev 0.5.4 without updating cargo.lock

the cargo.lock update to 0.5.4 was not included in the 0.5.4 release

Ah that was sloppy :) Can you clarify in what circumstance this is an issue? I just tried both depending on 0.5.4 from a library and installing 0.5.4 via cargo install and neither seem to have a problem. I just want to understand the severity.

See

I'm trying to package this for nixpkgs and normally a ref is specified for the package version but cargo will try to update the cargo.lock when building for ref=0.5.4 and the build will fail due to a hash mismatch

Gotcha, and thanks for contributing a NixOS package! Your workaround makes sense to me, or if you prefer you could start with 0.5.2 since both 0.5.3 and 0.5.4 were just config changes for the Debian package. I can also cut a 0.5.5 if you're unable to proceed, but I'd lean towards one of these workarounds if they make sense to you.

0.6.0 is now out, so I'm going to close this. Please let me know if you're still having difficulty!


Update for query branch

Blocked on https://github.com/cosmos/cosmos-sdk/pull/5901

This PR is blocked on:

[ ] Resolving the opaque packet verification issue

It also contains some work to prepare the ground for the query work to navigate between chains.

On the path negotiation side, I have a rly paths find command that queries all of the configured chains and returns all the paths between them. Needed functionaility:

[ ] rly pth find-or-gen [src-chain-id] [dst-chain-id] [src-port-id] [dst-port-id] [name] - Uses any existing primitives found on src or dst to create a path between two ports.
[ ] pathA.Equal(pathB) needed to deduplicate existing paths in the config

What is the opaque packet verification issue? Are certain verifyMembership calls failing which didn't previously, or something like that? Theoretically the SDK-side test-cases should cover this.


Quick and dirty prototype of compression/decompression

Hi Augustin,
In order to resurrect my interest on compression support for websockets, I have done a (very) quick and (VERY) dirty implementation of compression/decompression optional support using what you started implementing.
I hope that some things are of interest. I did a simple test communicating with wscat, and it just works 👍
There are many more things to be done. I will try to push a bit further if I find time.

OK after a pass on RFC7692, here are the things which need to change:

Written in the extension name: permessage-xxx; the compression/decompression needs to happen at the message level, not frame level. It does not mean that the per frame evaluation of extensions is wrong though, as some extensions may have a per frame impact.
The Frame check procedure needs to evolve; with extensions some reserved bits are going to be in use
Interesting check to put in place:

An endpoint MUST NOT set the "Per-Message Compressed" bit of control
frames and non-first fragments of a data message.  An endpoint
receiving such a frame MUST Fail the WebSocket Connection.

the permessage decompression must reset the rsv1 bit to zero before giving the message to next extension

Argh. I was working on the same feature in my branch, but it wasn't ready for submitting as a PR.
I pushed my work-in-progress state here: https://github.com/aaugustin/websockets/pull/178/commits/aab9747cb2c426d9d023bb9d5de4dcc2efa5dff8

It looks like we've taken a similar approach.
We both noticed that my initial framework for handling extensions was largely inadequate, due to its lack of handling of parameters. Negotiation must be possible and the server must be able to return a different (but compatible) set of parameters. (I didn't know that before diving into RFC 7692.)
My version deals with both the client and the server side. It doesn't pass the Autobahn test suite, for reasons I haven't understood yet. Many tests pass, some don't.

Well if you're on it, that's a great thing!
Instead of trying to push forward my draft, I will have a look at your last code and see if I can spot reasons why autobahn does not pass 👍

The good news is that we're much more likely to bring this feature to completion if we're two people working on it :-) I'm happy to see that you've make a checklist against the RFC, that's my approach for developing this library.
FYI I like this library but don't have any use case for it anymore, so I'm never sure if the code I write works in real life, beyond test cases and the Autobahn test suite.
Here's a plan -- please let me know if you have other ideas:

We should compare the two branches and take the best features from both -- likely by copying them from your prototype to my branch, which attempts to cover the entire feature.

I'd like to rework the extensions negotiation framework in my branch. It doesn't need to be very flexible yet. I'd rather make a release that supports compression and improve negotiation later, as most people won't customize it anyway but would benefit from compression.

We need to make the Autobahn test suite pass. Help is this area is highly appreciated.  You must modify the configuration (compliance/*.json) like this:

   "cases": ["12.*", "13.*"],
   "exclude-cases": [],

to test compression. Adjust cases to run a subset of tests because they can be quite slow. Once we believe everything works we can do a full run.
compliance/README gives some pointers, if you're having trouble let me know and I'll add details.

Leaving this open for now, but let's follow up in #178!

I actively use this library, for home project but especially at work! Using it for one year now at least, and still working great ;-) It only lacks compression, which would make a huge difference when talking with mobile devices with low bandwidth...
I just pushed the latest code I was working on (I got carried on after starting!), so that you have my latest things. I just started looking at autobahn test suite but didn't get any pass yet, which is not a surprise as I was more focussing on the "permessage" rather than "deflate". The DEFLATE implementation can be easily inspired by other libraries as well.
Perfectly OK with your plan.


Provide auth layer

An auth layer is required to restrict access to the XR Debug stream. When enabled, only auth peers can use the debugger.

Use HTTP Digest auth.

[ ] 128-bits or more entropy password (generated)
[ ] Nonce with IP, timestamp, private key

Looks like SHA-256 isn't supported but by Firefox.
https://bugs.chromium.org/p/chromium/issues/detail?id=1160478
https://caniuse.com/mdn-http_headers_www-authenticate_digest_sha-256


Add privacy notice

This is a first draft for adding a privacy notice. This still needs legal review and may me incomplete. Also, wording improvements are welcome.

Resolves #61

Sorry, kind of missed the review comments on this one. One major issue I have is that while the entire page is in English, the privacy notice is now in German, which feels both inconsistent and unprofessional to me. Do we have an English translation or should we just translate it ourselves?


Use wildcard domains and A12+ auto-verify

I recommend buiding the apk with the keystore used for signing release keys and seeing if it shows Verified Links correctly. It shud work (as I verified it with the alternate domain and tachi mangadex extension shows up correctly)
Summary of the PR:

Use android:autoVerify to trigger verifying deeplink in A12+
Use wildcard domains for deeplink intents. This essentially lets us deeplink https://www.mangadex.org by only verifying for https://mangadex.org (tested and works for tachi mangadex ext)
Removes the mangadex.cc link intent. It shouldnt be in wildspread use anyways. Also its just a 301 redirect now so cant be verified. Ref: https://developer.android.com/training/app-links/verify-site-associations#publish-json

I think you removed the chapter deeplink by accident?

I think you removed the chapter deeplink by accident? Or do all links now try to open in neko. Even if it doesn't support them?

No no
Only links supported are /manga /title and /chapter
Eg: https://mangadex.org/author/f85a5b93-3c87-4c61-9032-07ceacbb9e64/fujimoto-tatsuki ->  doesnt work
I tidied up the xml and it should be clearer now


chore(master): release 0.2.1

:robot: I have created a release beep boop
0.2.1 (2022-12-19)
Bug Fixes

repair package (030785a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/indykite/indykite-sdk-node/releases/tag/v0.2.1 :sunflower:


GCP: Architecture and Date positioning is not consistent

The positioning of architecture and date is closer to the vertical centre of the page than it is the case with Azure and AWS.

Closed as it is solved in our v2 redesign.


Fix Jellyfin not working

Motivation for changes:
fix the issue #3078
Detailed changes:

use 3.1.120 emby component code and create jellyfin components based on that

Addressed issues:

Fixes #3078

Config usage if relevant (new plugin or updated schema):
now i added components for jellyfin. uses will be similar to emby but replace 'emby_' with 'jellyfin_'
for example : we can get jellyfin list of movies based on the config below
  get_jellyfin_list:
    jellyfin_list:
      server:
        host: http://<IP_ADDRESS>:8096
        username: username
        password: password
        return_host: lan
      list: movies
    accept_all: yes
    list_add:
      - entry_list: jellyfin_movie_list

To Do:

Maybe someone with better programming skills and api knowledge fix the current emby code so that jellyfin can work again from just current emby components.

Maybe someone with better programming skills and api knowledge fix the current emby code so that jellyfin can work again from just current emby components.

Not a terribly good idea.  We make no promises to ensure compatibility with the Emby API.

Maybe someone with better programming skills and api knowledge fix the current emby code so that jellyfin can work again from just current emby components.

Not a terribly good idea. We make no promises to ensure compatibility with the Emby API.

Ohh, i see then it would be good idea to separate emby and jellyfin code with only caveats that it will use older emby plugin code ( last version where jellyfin was working).
On a side note as it is still showing 1 workflow awaiting approval and might take a while to merge it, if someone wants a docker image working with jellyfin they can use below command in the meantime
  docker pull ghcr.io/arbuilder/flexget

commenting so that it doesn’t get buried behind bump pull requests.

I'm the developer of the emby plugin. Was this working with jellyfin? Are the API's of emby the same of Jellyfin?

From what i see your fork is from a much older version of the actual emby component, things like remote server login and some other changes are missing. Emby component keeps evolving and those changes would be missing from jellyfin integration. Using the same code would be perfect, but with both projects going their separate ways, it's a matter of time until something is different in one of them. I don't have Jellyfin installed to test the integrations, I could install it to see what I can do, but no promises here.

From what i see your fork is from a much older version of the actual emby component, things like remote server login and some other changes are missing.

@soloam using emby endpoint api didn't worked anymore so i replaced it with jellyfin endpoint api which worked as a fix.

I could install it to see what I can do, but no promises here.

Nice 👍

Awesome work, but we can't use host like that. Everything else looks good to me :)

Awesome work, but we can't use host like that. Everything else looks good to me :)

Thanks, I am actually not a coder so I made the changes in Emby plugin so that it can work with Jellyfin. This Jellyfin plug-in was working nicely so I did a pull request.


Add Addrinfo#inspect_sockaddr spec

Oh my, I'm seeing this failures again.

@etehtsea Thanks!
I restarted the build, probably one of the spec restored is not closing the server thread/port properly.


Scaffold SessionManagerField

Overview
Perquisite work to build out a new CMS component to display session data in the Member admin UI.
See https://github.com/silverstripe/silverstripe-session-manager/issues/12 for the designs and more details.
Acceptance Criteria

[x]  Set up build tooling

Base configuration on Module Template
Use current release of silverstripe-webpack

[x]  Create empty SessionManagerField component and Entwine wrapper
[x]  Create basic SessionManagerField PHP class
[x]  Create extension to attach SessionManagerField to Member UI
[x]  Validate that new field is rendering correctly
[x]  By default this component is viewed by the current user only

Notes

The module template has some out of date dependencies that will need updating as part of this issue. Let’s look to commit the updates back into the module template repository.

Shifted final AC regarding position of field to #12.


Please add bin to listary.json.

Please add bin to listary.json.
    "bin": [
      "Listary.exe"
    ],

why? on the portable version the Listary.exe is in the root location.

I see.


⚠ Samsung has degraded performance

In f647baa, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3130 ms

Resolved: Samsung performance has improved in c535308.


Add token classifiction for named entity recognition.

Description

Add TokenClassification class and class TokenClassificationProcessor class for named entity recognition. So far, only pretrained BERT model is supported.
Related Issues

Checklist:

[ ] My code follows the code style of this project, as detailed in our contribution guidelines.
[ ] I have added tests.
[ ] I have updated the documentation accordingly.

Should the named_entity_recognition.py script be renamed to token_classification.py? I don't know other use case of token classification though. :) @saidbleik thoughts?

Should the named_entity_recognition.py script be renamed to token_classification.py? I don't know other use case of token classification though. :) @saidbleik thoughts?
As a task name, NER is more common. I guess we're using token classification because of BertForTokenClassification.


Update ngrx-store-freeze to 0.2.4

I am getting this with the latest code
$ npm install
...
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^5.0.0 but none is installed. You must install peer dependencies yourself.

The latest version of ngrx-store-freeze is 0.2.4 and eliminates the warning.

Fixed ;) Thank you for reporting!


fixed a bug in delete action

Can you please comment on what case this is covering? It is not so clear to me.

In fact I saved a publish action then I wanted to delete it, after pressing the delete button I got a 404 error then I checked the logs and saw the request was default to id = 0 when I replaced it with that code, the delete button worked

What version of the plugin are you using? I cannot replicate this issue.

I wasn't using the latest version I updated the version and it works now


Add 4 Projects

Description 📜

I have added 4 projects of mine including 3 HTML projects and one is my portfolio website
Fixes # <issue_number_here>

Domain of Contribution 📊

[ x] Web Development
[ ] App Development
[ ] Flutter Projects
[ ] Python Projects
[ ] C++ Projects
[ ] Clone Projects
[ ] HTML Projects
[ ] Javascript Projects
[ ] Machine Learning Projects
[ ] Others

Checklist ✅

[x ] I follow Contributing Guidelines & Code of conduct of this project.
[x ] I have performed a self-review of my own code or work.
[x ] I have commented my code, particularly in hard-to-understand areas.
[ x] My changes generates no new warnings.

Screenshots / Gif (Optional) 📸

fill the description template correctly and delete all the irrelevant checkboxes and other things

fill the description template correctly and delete all the irrelevant checkboxes and other things

I have done all the required changes. Please review it @ron2111


Histogram legend is missing the first interval

Example:
seq 1000 1000000 | jp -input csv  -x '[*]' -type hist -bins 15

When my terminal is wide enough then the 15 intervals are correctly included below each bar.
But when the terminal to small for inline display, the then displayed legend just includes 14 intervals, i.e. the interval for the zeroth bar is missing.
That means the legend looks like this:
1  [67600,134200)
2  [134200,200800)
3  [200800,267400)
4  [267400,334000)
5  [334000,400600)
6  [400600,467200)
7  [467200,533800)
8  [533800,600400)
9  [600400,667000)
10 [667000,733600)
11 [733600,800200)
12 [800200,866800)
13 [866800,933400)
14 [933400,1e+06]

The inline display:
[1000,67600)   [67600,134200)   [134200,200800) [200800,267400)  [267400,334000) [334000,400600)  [400600,467200) [467200,533800)  [533800,600400) [600400,667000)  [667000,733600) [733600,800200)  [800200,866800) [866800,933400)  [933400,1e+06]

I had the same issue and got it working by subtracting the height of the runes.
Unfortunately it seems this project is no longer maintained so here's what I changed locally:
        if labelsRight {
                for i, group := range table.Columns {
-                       c.GetBuffer().WriteRight(c.GetBuffer().Height-i, paddingX+1+chartWidth/c.RuneSize().Width, []rune(Fi(i)))
-                       c.GetBuffer().WriteRight(c.GetBuffer().Height-i, paddingX+4+chartWidth/c.RuneSize().Width, []rune(group))
+                       c.GetBuffer().WriteRight(c.GetBuffer().Height-i-c.RuneSize().Height, paddingX+1+chartWidth/c.RuneSize().Width, []rune(Fi(i)))
+                       c.GetBuffer().WriteRight(c.GetBuffer().Height-i-c.RuneSize().Height, paddingX+4+chartWidth/c.RuneSize().Width, []rune(group))
                }
        }

Before

After


Add tests for PyMC functionality

I think the PyMC code can be quite simplified, but without a test suite I wouldn't venture refactoring it :)

@rohanbabbar04 can you work on this? You can use the notebook I shared with you as a starting point. To add context @ricardoV94 is a PyMC expert, so his help will be very valuable.

Sure, I will look into this and add tests...


Faster is acyclic test

This branch implements the fast acyclic test based on the depth first search. If there are cycles then the old algorithm is used, in particular the output stays the same.
In my project (5000+ webpack modules) this speeds up the circular dependency check from about 4300ms to 30ms (of course only if there are no cycles, if there are then nothing changes)
FIxes #65

This should work great if a particular project enforces no cycles (e.g. via failOnError plugin setting, or via other measures).
However, if it's not generally enforced, a project would tend to have one or two "well-known" circular dependencies. In this case the extra isAcyclic check before the actual cycles enumerating would probably add some overhead? I would assume it's somewhat negligible (given its O(N) complexity) - but any thoughts on that? @vbraun

If the overhead is too great (not saying it necessarily is), looks like making the "passQuickly" optional would be reasonably easy?
  if (passQuickly && isAcyclic(graph))
    return;

Would be great to get this in - my team is concerned about adding this plugin because of performance (we'd have failOnError on, so would expect failure to be the unusual case).

Well in my experience the overhead (in case of cycles present) is <1%, so I'd call than negligible.
If you have cycles that you are fine with then you really need a setting that somehow excludes that cycle from error reporting. Though I'd argue that your time would be much better spent breaking that cycle than figuring out how to configure webpack to allow a particular circular dependency.

On webpack 5 this branch shows the following warning:
 [DEP_WEBPACK_DEPRECATION_ARRAY_TO_SET] DeprecationWarning: Compilation.modules was changed from Array to Set (using Array method 'filter' is deprecated)


robyn_refresh error : Error in UseMethod("mutate")

Project Robyn
Describe issue
🟢 I created a set of models using robyn_run and then selected a model from OutputCollect (select_model <- "5_268_6"). I then created RobynModel-5_268_6.json using robyn_write, defined a calibration_input dataframe before calling robyn_refresh.
🟢 The JSON file is successfully imported, the input data is read successfully, the feature engineering runs, and the initial model is successfully recreated according to messages.
🔴 However, on step Building refresh model #1 in manual mode I get the following error : Error in UseMethod("mutate") :    no applicable method for 'mutate' applied to an object of class "NULL".
I noticed the same issue being reported here
Provide reproducible example
ℹ dt_input is the same dataframe that was used to build the initial model (with window_end used in robyn_inputs)
select_model <- "5_268_6"
ExportedModel <- robyn_write(InputCollect, OutputCollect, select_model, export = create_files)
json_file <- "Robyn_202310021749_init/RobynModel-5_268_6.json"
RobynRefresh <- robyn_refresh(
json_file = json_file,
dt_input = refresh_data,
dt_holidays = dt_prophet_holidays,
refresh_steps = 20,
refresh_mode="manual",
refresh_iters = 1000,
refresh_trials = 5,
calibration_input = calibration_input,
plot_folder = "robyn_working_directory/Robyn_202310021749_init/refresh")
Environment & Robyn version

Robyn version: 3.10.3
R version : 4.2.1

Just tested it on the latest dev version and I can't reproduce your error. Would you pls install the latest and retry? remotes::install_github("facebookexperimental/Robyn/R")

Thank you very much for the reply. Closing this issue now


MLOps - understand the limit of Jupyter Notebook and the solution to have ML product iteration

捋 MLOps 背後解決的問題跟作法，並且簡單描述出來

took 2 hrs
done


Remove createJSModules from packages of Android templates

This was a breaking change in RN v0.47.2. Not sure if overriding this method is necessary for native modules to work in earlier versions -- if so, we'll need to check the version in package.json and update the templates accordingly.

And the compiler fails if that method isn't overridden in earlier versions :( looks like we have to do the version check

Had the same issue with react-native-orientation. It's a breaking change for Android. Anything above v0.47 needs to have it removed, otherwise it breaks compilation. Any project before v0.47 definitely needs the @Override call.


Asset path fix

All files created by the plugin on Android are now placed in a proper subfolder inside the app cache directory, folder called wizAssets.
Database is not removed anymore when requesting the deletion of all assets in cache via /.
Fixed paths problems when passing a file or directory with an uri starting with a slash.

Ping @jrouault for review.

Updated. I think everything we discussed is included 😃 .

Updated! Ready for approval :)


Fix: Use regionTier in the Tasks panel to show next region unlock reuirement

This fixes the tasks panel to correctly show the next region unlock requirement

What is currently displayed:


Adding support for oracle

Automating swap configuration (prereq for Oracle 19c)
Automating Oracle 19c installation and configuration
Supporting Oracle with Connections Wizards
Connections and Docs updates to support Oracle
Updating documentation and examples to support multiple database backends
Connections MT updates and fixes
SSO update - avoid doing it twice
J2EE role refactoring and improvements
Updated default Helm version
WAS cleanup - fixing the order of execution to avoid running it multiple times or partial cleanups
Make it optional to (not) create IIM cache

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Dejan Menges seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Rizkee/feature/cell name and mod below of

Added additional param for below_of locator to account for empty cells and added new table end condition on cell name

@s-ricks please merge from main and fix the conflict


MudTable checkboxes works incorrect with records

Bug type
Component
Component name
MudTable
What happened?
Checkboxes work incorrectly.
Expected behavior
Checkboxes should work correctly using record, as they work with classes.
Reproduction link
https://try.mudblazor.com/snippet/wuQGardueypllRRK
Reproduction steps

Just change Items element at OnSelectedItemsChanged callback

Relevant log output
No response
Version (bug)
6.0.14
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[x] I agree to follow this project's Code of Conduct

@henon Here is an issue demonstrating the problem with records.

@sharpgraf please let me understand why you use Record type in a table with multi-selection when the record values can change. The multi-selection needs to compare the row objects against each other in order to determine which objects are selected.
Note: Using records is OK if the record's values won't change.
@Mr-Technician I think the only way to solve this problem if people are going to insist on using records is to add a Comparer to Table similar to what was added in Select and initialize the inner hashset with that comparer.

For the record, when to use Record: https://stackoverflow.com/questions/64816714/when-to-use-record-vs-class-vs-struct
I still think the usage in a multi-selectable table is just wrong if the objects are not immutable.

@sharpgraf please let me understand why you use Record type in a table with multi-selection when the record values can change. The multi-selection needs to compare the row objects against each other in order to determine which objects are selected.
Note: Using records is OK if the record's values won't change.

Here I briefly describe what I want to achieve, i need to do some extra step to achieve it and now i am using immutable records.
https://try.mudblazor.com/snippet/QuwGEWkvSGbsxkXl

Until the update is out yet, you can use some simple workaround.

Implement IMudTableRecord like this public Guid MudTableRecordId { get; init; } = Guid.NewGuid();
Call UpdateHashSet at OnSelectedItemsChanged

Example: https://try.mudblazor.com/snippet/mawwOMadHHoLXbvC
It works for me and hope it helps !

@henon  @Mr-Technician  i do not think it's okay too
Try to select first element, we can not just because it has new reference, i think we need comparer for classes too in this case .
https://try.mudblazor.com/snippet/mamwuWOnzdlRDMcm

@sharpgraf Comparers will work for classes as well if your use case requires it.

@Mr-Technician  Yeah , that makes sense, just wanted to be sure it would be included :)

@sharpgraf Of course you have to ensure equality!!! Like this: https://try.mudblazor.com/snippet/cuwwuikdzJIgepKs

but yeah, a comparer would also do the job

but yeah, a comparer would also do the job

I mean, it would be "semantically odd" to have comparer only for records, but not for classes :)


Tweak to the Laravel 8 Install Doc

In the docs for Laravel 8 you have to swap out the composer dependency

However, Laravel 8 out of the box has the old Ignition as around 2.5
"facade/ignition": "^2.5",

Whereas this version is around ^1.0.  This will cause a composer error if you just change the name.
So just to make it more clear you might want to add something that shows swapping that
"spatie/laravel-ignition": "^1.0"

Thanks for the suggestion! We've added a bit more information (should appear later today).


⚠ Auth has degraded performance

In 9fc3d6d, Auth (https://auth.k2lin.com) experienced degraded performance:

HTTP code: 200
Response time: 15079 ms

Resolved: Auth performance has improved in a3126f0.


in parameters obfuscate mutability of referenced object

auto do_something(in std::string arg) -> void {
    // arg is "foo"
    some_arbitrary_function_call();
    // arg is "bar"
}

If the argument was instead spelled std::string const& arg I contend that it's "clearer" that this is merely a readonly reference, and transferring control to any other function could have arbitrary effects on the referenced object. I think the above example is admittedly somewhat contrived, but far more likely to happen with coroutines, e.g.
auto do_something(in std::string arg) -> std::generator<int> {
    // arg is "foo"
    co_yield(1);
    // arg is "bar"
}

Worse is that, for an unfamiliar type, the presence of this behaviour is unpredictable to an author/reviewer because 'cheap to copy' is unpredictable (and may even be changed over time via a trait).
I wrote more about this general issue here in a post a few years ago https://gist.github.com/alex-weej/74d943d7c2d9b881f809cd8183c02de3

in T t indeed implies only that you can't write to t. If T is not immutable, it does not guarantee that t won't change elsewhere.
Note: you don't even need a lambda to cause this to happen; just call two functions, each on its own thread, passing the same object, once as inout, once as in.
There will probably be a need to have a flag like copy to enforce a copy.
That would then be the way to avoid cases where concurrency might otherwise change the value of the argument during your function. Just like you would perhaps have written std::string const arg now.
I'm not sure if it would be useful to have the opposite, an in that explicitly never copies (producing the equivalent of const int&) for a function that does not modify the value, but does want to observe changes to it. But I suppose in int& would already cover that case.

If the argument was instead spelled std::string const& arg I contend that it's "clearer" that this is merely a readonly reference, and transferring control to any other function could have arbitrary effects on the referenced object.

Agreed, saying in  could suggest a copy.
Indeed the current proposal doesn't provide a syntax to force a copy. Of course, you could just omit all modifiers and have a by-value std::string argument, but this is indeed makes you consider 6 options for argument passing instead of 5 as claimed in the proposal. So I see it as a teachability problem.

@Zastai

That would then be the way to avoid cases where concurrency might otherwise change the value of the argument during your function.

Don't get fooled that you need concurrency for this to be a problem. You don't

That would then be the way to avoid cases where concurrency might otherwise change the value of the argument during your function.

Don't get fooled that you need concurrency for this to be a problem. You don't

True, you could also call code from the function that mutates arg (because it has write access to it).
But then at least you have a chance of being aware of this, and behaving accordingly. With concurrency, all bets are off.

That would then be the way to avoid cases where concurrency might otherwise change the value of the argument during your function.

Don't get fooled that you need concurrency for this to be a problem. You don't

True, you could also call code from the function that mutates arg (because it has write access to it).
But then at least you have a chance of being aware of this, and behaving accordingly. With concurrency, all bets are off.

Yes although the non-concurrent case would be much more common than the concurrent one. And in the concurrent one you have to plan what to synchronize anyway, which requires a bit more of a high level approach that would also help to prevent such issues. But I agree that in the concurrent case it can be harder to spot the bug in the code.

I'd like to point out that while both multithreading concurrency and calls to functions (hidden behind co_await/co_yield, or just a function call with no visible definition) can lead to a hypothetical observable problem, it far more commonly leads to a failure to optimize. This is somewhat explicitly acknowledged by writing either:

T const& - save the copy, but assume the worst re aliasing and have to re-read the value several times in the function's code gen
T const - make a copy (or move construct!), and then be able to optimize under the assumption that its value cannot change.

Finding it interesting that with coroutines, an in argument being a T const& is actually quite dangerous because it's far more likely that the object will be destroyed (not just mutated!) after the first suspension.

I've updated the first post with an explanation of this thought.

Thanks! Note this is mentioned in the paper (section 1.3.1, in parameters) in this Note:

There is one design point that we should validate with field experience: If an “in” parameter is aliased by another parameter or global, then although the in parameter is treated as const in that it cannot be modified directly, its value may change through the alias. This could be a pitfall, which we should determine by getting field usability experience. However, the [P1179R1] Lifetime rules default to banning passing non-owning Pointers that alias, so this may not be a problem in practice.

I've now added a note about coroutines right after that, as follows -- it will be in the next version of 708.pdf I push sometime in the next few days:

For coroutines, an in parameter passed by pointer (much like today a const& parameter) can be invalidated/dangling at the first suspension point, particularly if it binds to an rvalue argument. This case is intended to be covered by [P1179R1] Lifetime rules extended for coroutines, where after the first suspension point we would warn about any use of a const& or in parameter (and potentially other by-pointer passing).

There's a separate issue with coroutines about whether that suspension can happen on function entry so that it would never be safe to refer to even a const& parameter today.
Thanks @alex-weej and everyone!


linkage build errors on Linux since upgrading to 6.0.2

when ForFoundationOnly.h and ForSwiftFoundationOnly.h end up being imported, compile fails due to different linkage of __CFAllocatorRespectsHintZeroWhenAllocating().
suggest removing it from ForFoundationOnly.h or adding CF_PRIVATE.
<module-includes>:1:10: note: in file included from <module-includes>:1:
1 | #include "CoreFoundation.h"
  |          `- note: in file included from <module-includes>:1:
2 |

/opt/swift-6.0.2-RELEASE-ubuntu24.04-aarch64/usr/lib/swift/CoreFoundation/CoreFoundation.h:77:10: note: in file included from /opt/swift-6.0.2-RELEASE-ubuntu24.04-aarch64/usr/lib/swift/CoreFoundation/CoreFoundation.h:77:
 75 | #include "CFConstantKeys.h"
 76 |
 77 | #include "ForSwiftFoundationOnly.h"
    |          `- note: in file included from /opt/swift-6.0.2-RELEASE-ubuntu24.04-aarch64/usr/lib/swift/CoreFoundation/CoreFoundation.h:77:
 78 |
 79 | #if TARGET_OS_OSX || TARGET_OS_IPHONE || TARGET_OS_WIN32 || TARGET_OS_LINUX

<unknown>:0: error: could not build C module 'CoreFoundation'
/opt/swift-6.0.2-RELEASE-ubuntu24.04-aarch64/usr/lib/swift/CoreFoundation/ForSwiftFoundationOnly.h:108:20: error: declaration of '__CFAllocatorRespectsHintZeroWhenAllocating' has a different language linkage
106 | _CF_EXPORT_SCOPE_BEGIN
107 |
108 | CF_PRIVATE Boolean __CFAllocatorRespectsHintZeroWhenAllocating(CFAllocatorRef _Nullable allocator);
    |                    `- error: declaration of '__CFAllocatorRespectsHintZeroWhenAllocating' has a different language linkage
109 |
110 | struct __CFSwiftObject {

/opt/swift-6.0.2-RELEASE-ubuntu24.04-aarch64/usr/lib/swift/CoreFoundation/ForFoundationOnly.h:71:9: note: previous declaration is here
 69 | #endif
 70 |
 71 | Boolean __CFAllocatorRespectsHintZeroWhenAllocating(CFAllocatorRef _Nullable allocator);
    |         `- note: previous declaration is here
 72 | typedef CF_ENUM(CFOptionFlags, _CFAllocatorHint) {
 73 |     _CFAllocatorHintZeroWhenAllocating = 1

Hmm, still seeing this when adding CF_PRIVATE to ForFoundationOnly.h for reasons I haven't yet determined (although if CF_PRIVATE is already defined, ForSwiftFoundationOnly.h won't define it, so that could be it).
Is there a reason it needs to be in ForFoundationOnly.h?

It is there for historical reasons. We can move it to CFInternal.h and make it CF_PRIVATE.

@lhoward are you saying this is new to 6.0.2 (vs 6.0)?

@lhoward are you saying this is new to 6.0.2 (vs 6.0)?

I am, although it doesn't really make sense (given the prototype in ForFoundationOnly.h was committed in 2021). I'll try to find out the real reason.

I can't easily explain why it's just started happening with one particular package I'm trying to build.
But from first principles it seems that this always should have raised an error:

CoreFoundation.h (presumably the umbrella header for the CoreFoundation module) includes ForSwiftFoundationOnly.h
ForSwiftFoundationOnly.h defines CF_PRIVATE, if undefined, to extern __attribute__((__visibility__("hidden")))
ForSwiftFoundationOnly.h includes ForFoundationOnly.h
ForFoundationOnly.h declares __CFAllocatorRespectsHintZeroWhenAllocating() without CF_PRIVATE
ForSwiftFoundationOnly.h declares __CFAllocatorRespectsHintZeroWhenAllocating() without CF_PRIVATE

The only reasons I can think of as to why this didn't raise an error by default:

somehow CF_PRIVATE was being defined to the empty string (seems unlikely)
something in the compiler flags (perhaps related to C++ interoperability?) raised the language linkage error in this particular case

Actually, it's nothing to do with the visibility attribute. The issue is that __CFAllocatorRespectsHintZeroWhenAllocating in ForFoundationOnly.h is not within extern "C" (via _CF_EXPORT_SCOPE_BEGIN).
Moving __CFAllocatorRespectsHintZeroWhenAllocating after _CF_EXPORT_SCOPE_BEGIN in ForFoundationOnly.h also fixes it. Of course #5126 looks like a better fix.
Why did I only see this now? Perhaps it was there all along since Swift 6 and something has changed WRT C++ interoperability (which I am using)?

It makes sense that this could have started in 6.0 vs 6.0.2. Thanks for the investigation.

Tangential: __CFSafelyReallocate() and __CFSafelyReallocateWithAllocator() are also declare with incorrect (but not conflicting) linkage when imported as C++. Suggest wrapping them in _CF_EXPORT_SCOPE_BEGIN.

Let's file a new bug/get a new PR for reallocate ones.


Add option to not cache

Loading cache JSONs is slow and doing it every session creates a lot of very unnecessary overhead, especially when a single session just does something simple like load 1 Pokemon.

Implemented in this commit. Created an EmptyCache class that always returns False when testing if the request exists in the cache.


Revisit pattern of caching IOptions.Value / IOptionsSnapshot

per @sebastienros today the pattern where we cache the value of the options in middleware everywhere prevents dynamically configuring options per tenants.  We should revisit this to see if this is something we should change.  As part of this we can also see if it makes sense to switch to IOptionsSnapshot anywhere.

Is there any news/ideas about caching pattern for Options? (https://github.com/aspnet/Options/issues/167)

Might as well just use https://github.com/aspnet/Options/issues/161 to track revisiting any options patterns in 2.0


Bonjour reserves *.local domains

Causes lag with ipv6

Don't use dev either -- see https://iyware.com/dont-use-dev-for-development/


mynewt: allow overriding the startup routine

Add a new Mynewt option that allows for overriding the default hal_system_start routine; this allows for startup customizations required by some BSPs.
There's also a commit with some example custom code that will be dropped before merge!

/cc @agross-korg

lgtm

go ahead and drop the sample.  it's not needed now.


Add User@PC Name to main menu ListView and use it in keylogger

Hi @MaxXor
Please, I want to know if there is any problem if we add User Name and PC name to main menu listview like [User@PC-Name] to make it easier to know which client we deal with
And If we can make the folder created in [Clients Folder] will be like User@PC-Name instead of IP as ISP in many countries don't offer static IP so there will be new folder every time the router restarts
I wait for you opinion about that :smile:

this is a very useful feature. i support this.

Will be added before the release of the next version. :) I want this in the next release.

Good. Didn't even think about it.

Awesome idea this is very helpful, and will also help to segregate clusters of clients that share the same IP address.


max search results for full-text search

Redis Doc:
https://redis.io/commands/ft.aggregate/
REDUCE {func} {nargs} {arg} … [AS {name}]
reduces the matching results in each group into a single record, using a reduction function. For example, COUNT counts the number of records in the group. The reducers can have their own property names using the AS {name} optional argument. If a name is not given, the resulting name will be the name of the reduce function and the group properties. For example, if a name is not given to COUNT_DISTINCT by property @foo, the resulting name will be count_distinct(@foo).
Hiermit sollte möglich sein, eine Query zu schreiben, die eine Zahl zurückgibt, wie viele Einträge in der DB gefunden wurden.
    req = aggregations.AggregateRequest("redis").group_by(
        "@parent", reducers.count_distinct("@title")
    )

https://github.com/redis/redis-py/blob/master/tests/test_search.py

Number of query results is already returned to the result object

written new method that returns just the number of found objects for a specific full-text search query


Actions require an explicit HttpMethod binding for Swagger

I customize a ActionModelConvention Used to set the default HttpMethod, it works fine, but the swagger can not generate the document Because it can't get to the HttpMethod from ApiDescription

please tell us how to do it ?

I rewrote the SwaggerGenerator to achieve my goal.


Rebase should allow additional options

Currently if you use rebase it doesn't allow any other options to be specified
svn2gitnet --verbose --rebase

Too many arguments.
Type 'svn2gitnet --help' for more information

It would also be useful to allow specifying username/password with rebase since sometimes those aren't cached during the original clone
This causes svn2gitnet to use the currently logged in user for logging in to the SVN repo which is incorrect
svn2gitnet --username <username> --password <password> --rebase

Thanks. This is a good improvement. I'm considering adding it to the upcoming release.


Save/Resume Feature

Right now I’m imagining implementing this with two new argument flags:

Something called like --store, which would make a .json file that contains all the paths contained within a folder (In other words it would store the work listdupes does during its “Gathering files…” phase) and then quit once that work is done. The json file should also store the date of its creation.

Something like --read, which would read files listed in that .json file, checksum files as usual, and cache the resulting checksums in another .json file (Probably a unix style invisible dotfile since it’s a temporary cache).  The --read flag should notify the user if a notably old file is being read. ("This file is over a week old", "This file is over a month old" etc.

So if the application is quit while being run with the --read argument it would stop reading stored paths, write any remaining checksums to the cache file along with the index it's on, and then quit.
It should also be possible for listdupes to make an intelligent guess about when a server or other external volume it's reading from has been disconnected, and quit after a relatively long period of disconnection (Say 1 second). In which case, the next time it's started with --load, it should resume its work from before the disconnection.

Implemented in v6.0.0 Beta 1


collapse the old header

Hi,
How should I collapse the previous cells when I expand a new one ?

IMHO no

Sorry, I haven't saw the How. Right now you cannot collapse/expand programatically

I've added the ability to expand/collapse sections programatically. Please see https://github.com/LaurentiuUngur/LUExpandableTableView/releases/tag/2.1.0


fixfix array_with_const with larger than max_array_size

Proposed changes
when we use func array_with_const with large size there has not limit so, we will make be core like this

Issue Number: close #xxx

run buildall


Strip comments from SAML Response XML during scan

Let's say you get this kind of email claim in SAML Response
<saml:Attribute
  Name="http://schemas.xmlsoap.org/ws/2005/05/identity/claims/email" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri">
  <saml:AttributeValue xsi:type="xs:string">
    jon<EMAIL_ADDRESS>  </saml:AttributeValue>
</saml:Attribute>

using Samly you will get this assertion:
%Samly.Assertion{
  attributes: %{
    "email" => "jon<EMAIL_ADDRESS>    ...
  },
  ...
}

but with modifying the response and adding an empty comment
<saml:Attribute
  Name="http://schemas.xmlsoap.org/ws/2005/05/identity/claims/email" NameFormat="urn:oasis:names:tc:SAML:2.0:attrname-format:uri">
  <saml:AttributeValue xsi:type="xs:string">
    jon<EMAIL_ADDRESS>  </saml:AttributeValue>
</saml:Attribute>

the envelope signature verification will pass, and you will get
%Samly.Assertion{
  attributes: %{
    "email" => ["jon@snow", <EMAIL_ADDRESS>    ...
  },
  ...
}

If your app expects multiple emails and takes the first one, it might assume the identity of a different user within the same organization.
We've been running this kind of fix ourselves for a while now https://github.com/Recruitee/samly/commit/056298df41891b2f65400c0195469843bf55b49a but now thought it might be better to update it in the core if you agree.

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


Pass the new state into the reducer done callback

💰💰💰

Coverage remained the same at 95.238% when pulling 6c23319e4f4bca4cb98c2df7f9b452720b801370 on nicknikolov:master into 82ece0fd4dbcacb020bc6b0dbd1e0460e85b1bdb on yoshuawuyts:master.

Coverage remained the same at 95.238% when pulling 6c23319e4f4bca4cb98c2df7f9b452720b801370 on nicknikolov:master into 82ece0fd4dbcacb020bc6b0dbd1e0460e85b1bdb on yoshuawuyts:master.

Might be cool to write a test for this, validate this behavior for future refactors C:

8.2.0 :tada:


DeprecationWarning when running py.test

Hello,
When running tests using quantiphy, I got this warning :
python3.6/site-packages/quantiphy.py:1268: DeprecationWarning: Flags not at the start of the expression '\\A\\s*(?P<sign>[-+]?)' (truncated) 15:21:54     for pattern, get_mant, get_sf, get_units in [
Anyone aware of this ?
Regards

Up ?


[toggl-track] Fix timer not stopping from main window

Description
Fixes #13800
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Thanks for the quick review @mil3na 🙏


Publish new Pod version?

Would it be possible to update the podspec and push out a v0.3.3 with the latest merged code?
Thanks!

Darren

I'll try and get this done ASAP, thanks for the reminder.


Add functions to create address

Refactoring address types to Algebraic Data Types. Making companion objects for P2PKHAddress and P2SHAddress and adding helper functions inside of each to make it easier to create their corresponding type.

Coverage increased (+0.4%) to 91.374% when pulling d4452a1d3556945bb1f6e97c1ce9eb82fe021be5 on Christewart:add_functions_to_create_address into 71630141a9a9bcf46a090a7c6ae452716ccdbdf4 on bitcoin-s:master.

tACK


[BEAM-1566] Flink: upgrade apache commons

R: @aljoscha
Any concerns about an upgrade here? My understanding is that Apache Commons is good about versioning so this should not be breaking anything.
Prefer an exact version or a range?

FWIW, Flink currently depends on 1.4.1: http://mvnrepository.com/artifact/org.apache.flink/flink-core/1.2.0

I think this shouldn't be a problem, the dependency was added in Flink to support some additional decompression handlers. A range should be fine.
LGTM.

Yes and frankly, sdk-java-core should be shading this.

Coverage decreased (-0.02%) to 69.144% when pulling 187222b9f515b224eb69ec7c861ce80526a85f1a on dhalperi:flink-fixup into 063fbd4ed4ed34dece6490e791f0e5c91ec52895 on apache:master.


same issue

remote: -----> Building on the Heroku-20 stack
remote:  !     No default language could be detected for this app.
remote:                         HINT: This occurs when Heroku cannot detect the buildpack to use for this application automatically.
remote:                         See https://devcenter.heroku.com/articles/buildpacks
remote:
remote:  !     Push failed

@RENJITHVS : Were you able to upload the project to heroku?
I still don't find a solution.

yes ,I can able to upload my project to Heroku from Windows , I  have done these steps

Go to 'Getting Started on Heroku with Python'
Create a Heroku account
install pipenv
Install git ( check git --version)
Install Heroku CLI
Login heroku
Create a virtual environment
Run manage.py not gonna run - pip freeze nothing installing
Check which version django, requests you have and install it
Run manage.py and then stop it
Go to Django Heroku
Create a Procfile and
Install django-heroku
Add stuff to settings.py file
Install guincorn (u can't run  it on windows! No problem move on..!)
pip freeze - requirements.txt (use the angle bracket after pip freeze. Youtube doesn't allow angle bracket in description :/ )
heroku create  (to create an app on heroku)
git status git commands - (git push heroku master)
Open up the website)
Admin panel not working. Heroku run bash. Migrations

Host your website completely free here. Have Fun!


Custom React component in toolbarContainer

Hi @JiHong88 ,
I am using toolbarContainer to define the tool And I am able to add a custom React component to the end of the toolbar container.
My question is, is it possible to add between the suneditor icon?

Thanks

@gopal1996 To add a custom button, you need to create a custom plugin.
http://suneditor.com/sample/html/customPlugins.html


Vertical distance between sections is too small compared to spacing between h3 h4, etc.

Describe the issue
See title, a good example can be found here:
https://4mrnhq.csb.app/#typography
When viewed in isolation, the vertical spacing in this section looks ok to me. However, when you now scroll up just a little such that the section "Preview" becomes visible, it should be apparent that the vertical distance to that section is too small.
Current Behavior
Vertical distance between sections is too small.
Expected Behavior
Vertical distance between sections should be enlarged so that sections become clearly recognizable as such.
Reproduction URL
https://4mrnhq.csb.app/#typography
Environment
Latest Brave on Pop!OS

idk if this spacing is enough

basically i doubled the spacing in the section style:

idk if this spacing is enough

Thanks! I'm not sure whether this is the right solution. The weird thing is that the spacing looks perfect in the docs, but apparently these don't include pico.css directly. Assuming that they do via a bundler, I'm wondering whether this is just an oversight that is easily corrected?

we can assume this are the simplest solution, but it depends highly of the concrete value of the block-spacing-vertical variable
Regarding the spacing, it may be due to not bundled that specific change at the time of inserting it but I wouldn't know for sure.


plan to support babel-cli debugging?

Hello.
VS Code supports only 'node' type. But it's not enough to test my project. I need to exec command directly such as babel and eslint. So I'm asking to you where options for aforementioned?
Thank you!

Hello,
Did you try using the preLaunchTask in launch.json.
More details about debugging and preLaunchTask can be found here

I saw about preLaunchTask. It looks need gulp tasks.
I just want to build purely as babel-node ./tools/run.js (https://babeljs.io/docs/usage/cli/)
babel actually can be executed in script side but I prefer command than gulp.

This should be possible by introducing a task that does just that.
Please read our tasks doc on how to do that.
Then you can reference this task from the preLaunchTask


Expression 2 editor - highlight copies of selected word fails when occurrences are separated by one character

Yet another miniscule one, I'm running revision 4031 (latest as of time of writing)
The words are highlighted only when more than one character, doesn't matter which, separates the last occurrence.

This is intended behaviour, the editor intentionally has a check to make sure only whole words are found.
'-- [^a-zA-Z0-9_] ensures we only find whole words, and the gsub escapes any regex command characters that happen to be in str
local pattern = "^a-zA-Z0-9_" .. str:gsub("[%-%^%$%(%)%%%.%[%]%*%+%?]", "%%%1") .. "()[^a-zA-Z0-9_]"`

could possibly be fixed by stepping back 1 char after finding

It's also possible to do this with a frontier pattern:

%f[set], a frontier pattern; such item matches an empty string at any position such that the next character belongs to set and the previous character does not belong to set. The set set is interpreted as previously described. The beginning and the end of the subject are handled as if they were the character '\0'.

> test = "same.same|same |same|  same//same/same same same same same"
> test:gsub("%f[%a]same%f[%A]", print)
same
same
same
same
same
same
same
same
same
same
same

@AbigailBuccaneer fixit


Feat: Footer not taking up full width, emply space on right side

Describe the feature
Footer not taking up full width, emply space on right side
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@sanmarg Assign me this issue, I will also improve the UI of footer which will make it a better UI

Hey, can you please assign me this issue if it is not solved? This is my first time making an open-source contribution, and I really want to showcase my skills. I am very familiar with languages like Java, HTML, CSS, and JS. Apart from coding, I'm an ux/ui freelancer as well.

Its because of the slider present.

Nope. Anyways leave it


Feature request: Add traversability information

Hi @victorreijgwart!
Context
Wavemap is a great tool to efficiently store occupation probabilities in 3D. However, some applications (like path planners for ground robots) have the additional need for traversability information. It would be desirable that the traversability information could also be queried at different resolutions for a more efficient path search.
High-level changes
Inspired by your plan to integrate color (https://github.com/ethz-asl/wavemap/issues/57), I assume the plan would be adding:

a map data structure that stores traversability alongside the occupancy
a measurement integrator that also fuses traversability information when updating the map
extensions of the ROS interface (wavemap_server) to forward the traversability point cloud to the integrator

I would be happy to contribute but I kinda got stuck in the first point (creating a new map data structure) because I can't really see where or how the traversability information would be stored. My (limited) understanding of wavemap is that the occupancy probability of a certain node is given by a linear combination of Haar coefficients from the root node up to the desired termination height.
If we take a volumetric data structure such as HashedWaveletOctree, we see that it stores a std::unordered_map of HashedWaveletOctreeBlock. Each HashedWaveletOctreeBlock has a float scale and a Ndtree of height - 1 where each NdtreeNode is a std::array<float> storing Haar coefficients at each height.
My questions are:

Would we like (is it even possible?) the traversability information to also be expressed via Haar coefficients? That is, using e.g. HaarCoefficients<CellInfo, 3>, where CellInfo contains two floats (occupancy and traversability).

If so, I guess that the Harr functions such as ForwardLifted would change?
Otherwise, would HashedWaveletOctreeBlock just have an additional float traversability_; variable? But then I don't see how this info could be expressed in different resolutions.
Maybe we would just have another Ndtree that encodes traversability, but then I guess we would need to somehow synchronize both Ndtrees.

A bit more generic: I understand that each HashedWaveletOctreeBlock has its own Ndtree, is that right? If so, are the Ndtrees  somehow connected to each other?

Thank you very much!
Miguel

Hi Miguel,
Thank you for this question! This feature would definitely be great to have.
As you suggested, the steps would, in general, be very similar to integrating color information.
I have a few questions to make sure I understand your setup correctly.

Do you perform the traversability classification in image space (e.g. pixel-wise predictions based on RGB-D images)?
How do you usually filter the traversability estimates? Assuming you store the posterior per-voxel, do you apply the measurement updates using log odds? Or using a weighted average?

Regarding your questions:

My (limited) understanding of wavemap is that the occupancy probability of a certain node is given by a linear combination of Haar coefficients from the root node up to the desired termination height.

This is correct. The structure and information stored in the map are identical to Octomap. The key difference is that while octomap stores the occupancy probability of each node as an absolute value, we encode the occupancy probabilities using differences. The main advantage of doing this is that when you update the map at a lower resolution, all the affected high-resolution cells are updated implicitly - so when you update the map at a resolution of 20cm (e.g. for LiDAR), in an area with a maximum resolution of 1cm (e.g. from RGB-D), all probabilities are updated correctly even though you only pay for a 20cm update.
Although you could store the difference between a node and its 8 children, some of this information is redundant. Specifically, the average value of the 8 children is identical to the value of the parent. Haar wavelets, therefore, only need 7 difference coefficients. In code, the InverseHaar transform takes 1 scale coefficient (value of parent) + 7 detail coefficients (differences) and returns the values of the 8 children. Note that for Haar transforms, a node's scale coefficient directly corresponds to the average value over the volume covered by the node.

If we take a volumetric data structure such as HashedWaveletOctree, we see that it stores a std::unordered_map of HashedWaveletOctreeBlock. Each HashedWaveletOctreeBlock has a float scale and a Ndtree of height - 1 where each NdtreeNode is a std::array<float> storing Haar coefficients at each height.

Yes, that's right :)

Would we like (is it even possible?) the traversability information to also be expressed via Haar coefficients?

Yes, this would be possible. If you query traversability information at a lower resolution, would you like the map to return the average traversability over the subvolume covered by the node? Or would you rather have the minimum/maximum traversability over the given subvolume? The Haar transform is only suitable if you want the lower resolutions to capture averages. In case you'd rather have the lower resolutions perform min/max reductions or another non-linear operation, it'd be better to store the values of interest directly as absolute values (at the loss of implicit multi-res updates).

A bit more generic: I understand that each HashedWaveletOctreeBlock has its own Ndtree, is that right? If so, are the Ndtrees  somehow connected to each other?

They're completely independent. Each octree has a few private variables (e.g. the max tree height). These get passed in through the constructor whenever a new block is created. Let me know if this answers your question.

I can't really see where or how the traversability information would be stored.

In the code, the type of the data stored in each octree node is specified here.
Best wishes,
Victor

Hi Victor,
Answering your questions:

Do you perform the traversability classification in image space (e.g. pixel-wise predictions based on RGB-D images)?

Our current classificator works with the 3D point cloud, but that may change in the near future.

How do you usually filter the traversability estimates? Assuming you store the posterior per-voxel, do you apply the measurement updates using log odds? Or using a weighted average?

This may also change, but we currently treat traversability as occupancy (log odds and threshold for classification).

If you query traversability information at a lower resolution, would you like the map to return the average traversability over the subvolume covered by the node?

Yes, I think this is quite reasonable, and --if I have understood it correctly-- that way we would be using full potential of the Haar transforms.
Thank you very much for the clarifications, I appreciate them. Unfortunately, I think I overestimated my current availability to work on this in the coming weeks, but it is definitely a feature of great interest and I'll try to come back to it when I have some more bandwidth.
Best regards,
Miguel


⚠ PrivateHD has degraded performance

In 16d56ca, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 3731 ms

Resolved: PrivateHD performance has improved in dbaa082.


how to make daemonset upgrade sequentially

I have a ceph helm chart that has daemonsets in it. The daemonsets start up ceph osd. There could be multiple daemonsets, e.g. ceph-osd-dev-sdc, ceph-osd-dev-sdd, ceph-osd-dev-sde.
I want to upgrade these daemonsets sequentially during helm upgrade, i.e. daemonset ceph-osd-dev-sdd waits till ceph-osd-dev-sdc is complete before it can upgrade.
Ideas and pointers are appreciated.

@technosophos I saw your comments on some related issues.

Quick thoughts ...
The daemonsets could share a local volume and set a state file in that volume.  The init script for the containers waits on that state to flip to -ready before proceeding with the upgrade logic.  Readiness checks can then also return based on the same state.
Alternatively could provide a prereq env to daemonsets, they check for the prereq and wait till it is up to speed.  This can also be used as a readiness check.


Cannot deploy Hubot on Heroku - npm file breaks build

I'm trying to deploy the Hubot app on Heroku and I'm getting this build error which seems to be linked to the usage of the backtick "`" symbol in the npm's lib unsupported.js file.
(Line 28 of this file: https://github.com/npm/npm/blob/latest/lib/utils/unsupported.js)
Is there any chance of getting this fixed, or a suggestion of a workaround?
Natalias-Tardis:myhubot natalia$ git push heroku master
Counting objects: 15, done.
Delta compression using up to 4 threads.
Compressing objects: 100% (11/11), done.
Writing objects: 100% (15/15), 17.50 KiB | 0 bytes/s, done.
Total 15 (delta 0), reused 0 (delta 0)
remote: Compressing source files... done.
remote: Building source:
remote:
remote: -----> Node.js app detected
remote:
remote: -----> Creating runtime environment
remote:
remote:        NPM_CONFIG_LOGLEVEL=error
remote:        NPM_CONFIG_PRODUCTION=true
remote:        NODE_VERBOSE=false
remote:        NODE_ENV=production
remote:        NODE_MODULES_CACHE=true
remote:
remote: -----> Installing binaries
remote:        engines.node (package.json):  0.10.x
remote:        engines.npm (package.json):   unspecified (use default)
remote:
remote:        Resolving node version 0.10.x...
remote:        Downloading and installing node 0.10.48...
remote:        Detected package-lock.json: defaulting npm to version 5.x.x
remote:        Bootstrapping npm 5.x.x (replacing 2.15.1)...
remote:
remote: /tmp/build_29ae4dc962b314270f344f772059ec1b/.heroku/node/lib/node_modules/npm/lib/utils/unsupported.js:27
remote:         console.error(`a bug known to break npm. Please update to at least ${r
remote:                       ^
remote: SyntaxError: Unexpected token ILLEGAL
remote:     at Module._compile (module.js:439:25)
remote:     at Object.Module._extensions..js (module.js:474:10)
remote:     at Module.load (module.js:356:32)
remote:     at Function.Module._load (module.js:312:12)
remote:     at Module.require (module.js:364:17)
remote:     at require (module.js:380:17)
remote:     at /tmp/build_29ae4dc962b314270f344f772059ec1b/.heroku/node/lib/node_modules/npm/bin/npm-cli.js:19:21
remote:     at Object.<anonymous> (/tmp/build_29ae4dc962b314270f344f772059ec1b/.heroku/node/lib/node_modules/npm/bin/npm-cli.js:92:3)
remote:     at Module._compile (module.js:456:26)
remote:     at Object.Module._extensions..js (module.js:474:10)
remote:        npm  installed

@uCreateStudio since the change needed is to npm, recommend opening an issue in that repo. Looks like the change was made here: https://github.com/npm/npm/commit/2a11f021561acb1eb1ad4ad45ad955793b1eb4af#diff-1240a890730acc445832e67a85023f96


Optional plain JSON result

Summary
I've had a hard time to read the output of jwt in my programming tool in spite of the -j option, because in error-case it doesn't return plain json as expected:
The JWT provided is invalid because Error(Base64(InvalidByte(85, 61)))
{
"header": {
"typ": "JWT",
"alg": "ES256",
[...]
Especially the weird color-information in the error should be deactivatable by a new option (like --plaintext oslt).
Steps to reproduce
Run jwt -j with an expired token for instance
Expected behavior
Return plain JSON wrapping the current one with something like that without color codes:
{
"result: {
"error": "The JWT provided is invalid because Error(Base64(InvalidByte(85, 61)))"
}
"header": {
"typ": "JWT",
"alg": "ES256",
[...]

Sorry for the late reply @hingehe. Can you tell me a bit more about your programming tool? Is it a TTY or not? Anything else I could look for to strip color and formatting?

I am working with a EAI Tool called "inubit".

I execute your script with a "CLI Execution module" on a linux system (so I think it is a TTY-Tool in your sense). The result (refer my previous post) has weird characters. My tool tries to wrap everything into an XML structure. This fails, whit these color coding characters, because of charset foo.
I fixed the problem with replace (regular expressions) in my CLI script, so I remove those characters. But this is of course not the best option. An optional parameter to suppress any color codings would be perfect for my needs.

Hi again @hingehe, sorry for the delayed response. The output crate I use is bunt, which uses termcolor itself. bunt will automatically print to stdout with the auto color mode. In that mode, output on windows won't be colored, but also if the environment variable NO_COLOR is defined, then it won't output colors.
That being said, can you try defining NO_COLOR in your environment and see if that works for you? If it does, I'll find a place to document it 😄


Question: Which update type can i use?

Describe what you want
Hello @ all,
I have function like this:
	async update(id: bigint, data: any): Promise<void> {
		await db.update(memberships).set(data).where(eq(memberships.id, id));
	}

Currently i have data as any. How i use  it with typesafe?
The set method has:
export declare class PgUpdateBuilder<TTable extends PgTable, TQueryResult extends QueryResultHKT> {
    ...
    set(values: PgUpdateSetSource<TTable>): PgUpdate<TTable, TQueryResult>;
}

... and PgUpdateSetSource:

export type PgUpdateSetSource<TTable extends PgTable> = {
    [Key in keyof TTable['_']['columns']]?: GetColumnData<TTable['_']['columns'][Key]> | SQL;
} & {};

I use drizzle-typebox (https://orm.drizzle.team/docs/typebox) but there is no update type.
Is there any trick?

Update type is basically the insert type, but with all properties optional.

Unfortunately, this is not so easy because, for example, I do not send the ID.
I call the function as follows:
cRepo.update(membership.id, {
    availableUnits: 5,
});

If I exclude the ID as follows:
const membershipInsertSchema = createInsertSchema(memberships);
export type NewMembership = Static<typeof membershipInsertSchema>;
export type UpdateMembership = Omit<NewMembership, "id">;

Then the above error disappears. But then another error comes:
  Type '{ availableUnits: number; }' is missing the following properties from type 'UpdateMembership': companyId

The companyId is a mandatory field! That's why it doesn't work. I think the update scheme is missing in drizzle-typebox!

Oh i found a solution:
export type UpdateMembership = Partial<NewMembership>;


Live

Test PR

Docs Build status updates of commit 3d0cfa7:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Test PR


couldn't find "libffmpeg.so"

I always get this when trying on the emulator, how do you get it to compile and run in emulators?

You're building it within AS? I just threw the following line:
CGE_USE_VIDEO_MODULE = 1
Over here


What is meaning "Loras are baked in into the converted model. "?

Does it mean if want LORA works we need to merge the LORA into checkpoint model before convert to onnx ?

You need to go into settings>extra networks and find the textbox for loras to add all the loras you want to be added into the TRT model when you convert.

You need to go into settings>extra networks and find the textbox for loras to add all the loras you want to be added into the TRT model when you convert.

I did what you said to convert to a onnx file and then convert to a .trt file, but the trigger word of lora still not work when create via text to image.

hmm... okay. I'm not sure then. I've got very limited knowledge of TensorRT. I've been trying to get answers about how they calculated the size of the shape on the NVIDIA repo but have yet to get a response. I would say that at this point in time you might just go with merging the LORA into the checkpoint then converting it over since it isn't working with the Extra Networks.


Fixes array analysis for inplace binary operators.

As title. The expr.fn is now the inplace operator in the case
of an "inplace_binop" expr node. Adds test in array analysis to
ensure that empty slices hold equivalent (it works correctly without
this change but it may catch future problems). Adds a test to parfors
slice tests that checks the issue, that binary operators were
failing array analysis.
Fixes #3461 , Fixes #3554

This is ready once the comments are resolved.

Codecov Report

Merging #3558 into master will decrease coverage by 0.01%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master    #3558      +/-   ##
==========================================
- Coverage   80.69%   80.67%   -0.02%
==========================================
  Files         393      393
  Lines       80425    80287     -138
  Branches     9160     9146      -14
==========================================
- Hits        64896    64775     -121
+ Misses      14109    14095      -14
+ Partials     1420     1417       -3

There's a conflict.

RE https://github.com/numba/numba/pull/3558#issuecomment-446685473, thanks, merged and patched up.

This looks ready to merge.


[Feature Request]: Creating seller dashboard.

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
I want to add seller dashboard. Where seller can update the data of orders status. It will manage all the order shipped and deliver etc. and it can only access by certain email. By allowing only certain emails it will help seller to give access and make the order tracking more real time.
Use Case
While user tracking the order. Seller can give the update in real time.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@MAVRICK-1  can you please assign this to me.

@Hemu21 can you make a admin dashboard first , which will update and track all the orders

I think seller will update the orders tracking. Seller him self is the admin right for their own products.

I got your point , that's the next step. first have to create admin who have access to every order

@Hemu21 next step will be to create a seller dashboard


used a cronjob to perform the dbclean

fixes #51
Implementing the external DB Clean Task recommended as per the docs https://docs.pact.io/pact_broker/docker_images/pactfoundation#running-the-clean-task-from-an-external-source

Got rid of the DB Clean inside the Deployment
Utilized a CronJob to perform the cleanup
Generalized logging & db env vars (included them in the helm helpers file) as they are now re-usable across the CronJob & the Deployment

@ChrisJBurns would be great if you can have a look at this one 😉

Thanks for this @sherifkayad  I'll give it a review when I've got a bit more time, I'll have to make sure that it all templates as expected as well

@ChrisJBurns TYT and let me know if anything needs to be changed

@sherifkayad  Just had a look at this, we should probably make the CronJob optional for the database clean just to keep things simple in the vanilla install. So by default, if someone installs a single Pact Broker with 1 replica, then that single instance can be the container that runs the clean. But for folks who have more of a replicated setup of the Pact Brokers, we can allow them to enable the DBClean CronJob, a bit of documentation in the README.md about this would be a good thing too, just so users are aware of the ways the DB Clean can be done :+1:
So a few tweaks, but all round great addition! :+1:

@sherifkayad thanks for the feedback. Let me take care of that and submit something you can have a look at. Will ping you once I add something there.

@ChrisJBurns can you please have a look now? I also added some docs in the README about that new feature and overall about the DB Cleanup Task.

@ChrisJBurns any updates from your side to the latest changes done?

@sherifkayad  Yep, I plan on taking a look on Friday as it's bank holiday here in the UK :+1:

Awesome! Same here in Germany .. Happy Easter holidays and let me know if there is anything I can do


Fix sub deprecation

use view instead

Coverage remained the same at 95.918% when pulling ba75ab0bfd5cd09a0dccb7c5135cd058c5188e95 on compat into 680976c8b9fa394a2b603b1b9b6f295d6add0fbf on master.

Coverage remained the same at 95.918% when pulling 6c20eb3179950c7c0b58c35144dccc0bed14ff05 on compat into 680976c8b9fa394a2b603b1b9b6f295d6add0fbf on master.

Coverage remained the same at 95.918% when pulling c2781d2e33dd8154753f0274b6deb8307c5dbaf2 on compat into 680976c8b9fa394a2b603b1b9b6f295d6add0fbf on master.


Sort company contacts based on published date.

By default this sorts by ID descending (Highest ID # to lowest) which outside of recreating contacts to alter the order there isn’t any other means to change the order, rather than force recreation etc. of existing contacts, sorting on published date gives some sort of control point. (And effectively works as a drop in replacement, as presumably any higher ID numbers have a more recent publish date thus putting it at the top anyways)

 Is there a way to re-order contacts on a Leaders page?


Claiming IoT rewards doesn't work with the latest iOS app (v2.0.8)

Hi,
Claiming IoT rewards doesn't work with the latest iOS app (v2.0.8). I'm running latest iOS 16.5.
Already tried to re-install the app and remove & import the wallet.
Steps to reproduce:

Open the app
Click on the second tab on the bottom menu
Click on "Claim all Rewards"
Click on "Add all to account" on the next screen
Click "Approve" (I have enough SOL)
Wait for "Claiming your Rewards..." to complete...
When 6. is completed (with success message) no rewards added + no activity in the app

My last wallet activity is from 12.05.2023 which means that the app worked on that date.

hi I have the same problem have u solve it ??

Hello is the same with me, could someone solve this??

Where can I find this pubkey?

You should be able to click the settings gear, top left, then copy address

Me too.

Can either of you send a video of this happening? I checked out that wallet address and there's no signs of failure on it. Last activity was 2 months ago, and bugs have been fixed since then.

Sure, how can I send it privately to you?

Actually, @ilieff85 I can confirm your wallet doesn't have enough Sol. That's why it is failing to claim. The next update will be more clear about this.

@ChewingGlass I have balance of 0.00089588 SOL and as far as I see the network fee is 0.0000075 SOL.

Yeah so on Solana, every wallet needs a minimum of 0.00089 sol to be considered rent-exempt. You cannot go below that value unless you go all the way to 0.
The next release will be more clear about this


how build a custom Flutter web engine and use it for run my web app？

hi@VladyslavBondarenko @yjbanov
this is my trouble:
how build a custom Flutter web engine ? i have buit it by using :felt build. Succeeded.
i have build my flutter web engine and get the files such as: flutter_web_sdk，lib, kernel.
but i don't know how to use it to build my web app, is there a complete command for me ?

You can find the full documentation here: https://github.com/flutter/flutter/wiki/Compiling-the-engine#compiling-for-the-web
flutter run --local-engine=host_debug_unopt -d
Is the command to run with the local engine.

Its documented in several places, closing.

Another link: Running a Flutter app with a local engine: https://github.com/flutter/flutter/wiki/Debugging-the-engine#running-a-flutter-app-with-a-local-engine

hi@ferhatb
this is my cmd：
flutter run --debug --dart-define=FLUTTER_WEB_USE_SKIA=true --local-engine= host_debug_unopt --local-engine-src-path= D:\flutter\Gitcode\flutter\src -d chrome
but An exception was thrown: Target file "host_debug_unopt" not found.

@yzg-github each "local-engine" value is a different build of the engine. It seems you built the engine for android (android_debug_unopt_arm64), but not for web.
You can check what local engines are built, checking within your clone of the engine repo, inside the src/out directory:
dit@dit:~/github/engine$ ls src/out
compile_commands.json  host_debug  host_debug_unopt  host_release

For web, I've been using this command: ninja -C out/host_debug_unopt, but felt build does the same. Remember to gclient sync and do all the steps prior to running felt (following the instructions linked on top by @MichealReed)

@ditman
i have build my flutter web engine and get the files such as: flutter_web_sdk，lib, kernel in  host_debug_unopt.  i think i have built web engine successed.
i know android_debug_unopt_arm64 is for android also.
but i don't know how to use my web engin to my web app. i used : flutter run --debug --dart-define=FLUTTER_WEB_USE_SKIA=true --local-engine= host_debug_unopt --local-engine-src-path= D:\flutter\Gitcode\flutter\src -d chrome.
but it not work,and an error: Target file "host_debug_unopt" not found.
is there any parameters i need to set?

@yzg-github I normally don't pass the --local-engine-src-path option. If you make your clone of the flutter repo and the engine be in the same directory, something like this:
D:\flutter\Gitcode\flutter (your clone of flutter)
D:\flutter\Gitcode\engine (your clone of the engine)
You don't need the --local-engine-src-path, and your command can be just:
flutter run -d chrome --local-engine=host_debug_unopt --dart-define FLUTTER_WEB_USE_SKIA=true
(If I were you, I'd try with less options, and then add more, so start with:
flutter run -d chrome --local-engine host_debug_unopt and start adding other options after you get that running)

@ditman
Thank you for your suggestion, I will try again according to your suggestion.

@yzg-github I don't know, I've never tried to reuse an engine asset across platforms. You're probably on your own there.

@ditman
i tried to make the clone of the flutter repo and the engine be in the same directory.and run :
flutter run -d chrome --local-engine=host_debug_unopt
or:
flutter run -d chrome --dart-define FLUTTER_WEB_USE_SKIA=true
my test web app work well.
but when i run:
flutter run -d chrome --local-engine=host_debug_unopt --dart-define FLUTTER_WEB_USE_SKIA=true
my test web app just shou me An empty web. but there is no error occured in my shell console. and i have changed the canvasKitBaseUrl point to myself canvaskit.wasm, like this:
const String canvasKitBaseUrl = '/flutter/Gitcode/engine/src/third_party/skia/out/canvaskit_wasm_debug/'
the file does exist.
i don't know why this hanppend. could you give me some suggestions? Thank you very much!

@yzg-github check the Javascript Error console and the Network panel in Chrome Devtools to see if there are any errors there. If your page is blank, there must be some error showing there (something is broken, or something didn't load).

@ditman
Thank you very much for your help!

@ditman
when i use File(path) to open a file, there is an error : no such file or directory. but the file is exist. what's the reason? could you help me?

@yzg-github you should ask that in Stack Overflow, so more people sees your question and can help you. Your questions are not related to the original ticket. (I'll hide your question as off-topic!)

ok i will ask this in Stack Overflow.thanks!


Ignored files fail ember test

I have some files that I want to ignore, and have ignored them using an .eslintrc file.  However, the fact that they are ignored raises an error, which then fails the ember test run.  Would it be possible to prevent these warning from failing the build?  I feel like, if the user has explicitly listed the files they want to ignore, then even these warnings should be ignored.

Addressed with https://github.com/jonathanKingston/broccoli-lint-eslint/issues/30


Incorrect Characteristic equality for DiscoveredCharacteristic

On Android at least (have not checked for the other platforms) the equality check of DiscoveredCharacteristic may be incorrect if the list of descriptors is non-empty:
https://github.com/JuulLabs/kable/blob/5d6e00e01ac53ea5c76b69fc4b8b897dbb578c75/core/src/commonMain/kotlin/Characteristic.kt#L24-L28
This is because the equality of a DiscoveredCharacteristic will be checked against it's list of descriptors, and on Android the BluetoothGattDescriptor object does not provide an equals function. So, if we have two descriptor objects in memory that represent the same descriptor on a peripheral, then they will erroneously be seen as not equal.
In most cases this will not be an issue, but the Observers class relies on the correctness of Characteristic equality.

More specifically, the following code should take into account instanceId on Android:
https://github.com/JuulLabs/kable/blob/260f0cdc7ed409ef2777057f5446c70fee7e9e91/core/src/commonMain/kotlin/ObservationEvent.kt#L18-L20