TypeScript 2.4 weak types produces surprising Angular1 error

With TypeScript 2.4 a bunch of our Angular1 code produces a new error.
I reduced the code to something like the following.  (The interfaces/types here are from the Angular1 d.ts on DefinitelyTyped, but I simplified them a bit to make the error more obvious.)
interface IController {
  $onInit?(): void;
}

type IControllerConstructor = (new (...args: any[]) => IController);

type Injectable<T> = T|Array<string|T>;

interface IDirective {
  controller?: string|Injectable<IControllerConstructor>;
}

class Demo {
  data: string;
};

function demo(): IDirective {
  return {
    controller: Demo,
  }
}

TypeScript 2.3 accepts this code, while 2.4 produces this error:
Type '{ controller: typeof Demo; }' is not assignable to type 'IDirective'.
  Types of property 'controller' are incompatible.
    Type 'typeof Demo' is not assignable to type 'string | (new (...args: any[]) => IController) | (string | (new (...args: any[]) => IController))...'.
      Type 'typeof Demo' is not assignable to type '(string | (new (...args: any[]) => IController))[]'.
        Property 'push' is missing in type 'typeof Demo'.

The error message is quite a surprise!  I believe the underlying issue is that the "Demo" class is not seen to implement the IController interface anymore due to weak types, so TS picks the other union element in Injectable (the array).
I'm not sure how this can be improved, but perhaps we can think about the error?

Demo is not IController under the week type check. it does not implement any of the IController optional members, and it has other members of itself. you would have seen the same error if you had class Demo implements IContoller.
export class Demo implements IController {
    data: string;
};

there is some relevant discussion about this in https://github.com/Microsoft/TypeScript/pull/16047#issuecomment-304063779
Adding an index signature on IController should be sufficient allow the assignment.
the error message happens to be on the last constituent of the union type, here Array<T | string>, it is not possible for the compiler to know which branch was more useful in these cases..

Do you recommend we add an index signature to the d.ts files on DefinitelyTyped?  It's a bit weird because it's a lie, but I'm not sure how else to work around this.  Perhaps we could require users include the implements ... by implementing an (empty) $onInit method?

implements does not change the type. it just checks that the class is assignable to the interface.

Right, but that would a user-comprehensible place for us to recommend a fix.  When they try adding implements they'd get an error about how they don't implement the interface, and fixing that (by satisfying the interface) it would fix this error.
Otherwise, I'm not sure what the workaround should be -- maybe some Demo as IController at the use site?  Or would that hide other errors?

Right, but that would a user-comprehensible place for us to recommend a fix.

makes sense

and fixing that (by satisfying the interface) it would fix this error.

in this case, assuming i am understating the API correctly, the interface is meant to be weak. so not all users are expected to implement $onInit. so the fix would be adding an index signature on the type in DT.

Otherwise, I'm not sure what the workaround should be -- maybe some Demo as IController at the use site? Or would that hide other errors?

that would make the error go away too, but you do not want to do that every component.

I see.  So is "add an index signature" the general pattern for forcing an interface back to being weak?  We might need to sprinkle these around in various other places that use a similar pattern.

Just chiming in, the change was surprising for me too, but I managed to fix my project failing to build on TS 2.4.0 by sprinkling empty $onInit() for all my Controllers without $onInit().

The Definitely Typed d.ts for angular should be fixed now. @evmar "add an index signature" is the easiest solution for library authors to use. If you are using a library and normally want the weak type check, you can cast to the weak type to get an exception.

ryanelian - your suggestion worked. I was able to do the following to get it working:
export function MyDirective():ng.IDirective {
    return {
        templateUrl :  "my-directive.html",
        replace: true,
        restrict: 'E',
        controller: MyController
    }
}
export class MyController implements IController {
    //...additional controller code

    $onInit(){
    };
}

Why the heck would it be an OK solution to require to write an empty $onInit function when a controller is not used as a component or is a component and doesn't need that hook implemented!? So dumb...


Update Github Actions to use Poetry as the installer and discard the use of requirements.txt and package caching

Is your feature request related to a problem? Please describe.
Right now any new packages added to Poetry require the developer to run a one line command so that Poetry updates requirements.txt. This introduces issues for package management and potential issues for missing packages not synced.
Describe the solution you'd like
We need to replace the Poetry Github Action package to Snok/Poetry and also use actions/cache to speed up the CI/CD pipeline.
Describe alternatives you've considered
None
Additional context
None

@esfoobar-usds I closed https://github.com/usds/justice40-tool/issues/1100 as a duplicate because I think this is the real solution to that problem.

I notice that the current dockerfile references requirements.txt: https://github.com/usds/justice40-tool/blob/main/data/data-pipeline/Dockerfile#L33
That might also explain why we had that pandas warning turn into an error recently, and only in the docker compose setup, since I recently updated requirements.txt.
@esfoobar-usds Are you okay with me using poetry in the docker setup instead of requirements.txt as part of making this change?

Yes, we can install and run poetry on Dockerfile.
I have this example from one of my bootstrap projects.


Old context get lost when too many WebGL contexts exist

Hi,
When I use visdom, there are two things really annoying. One is the order of the plot is messed up(as #134 mentioned). Another one is old context get lost when too many WebGL contexts exist, even after sorting the plot. I save few 3d scatter plots every epoch, and I found the plots from past epoch disappeared even if I filter the plot and just allow few plot to show.

I think I can fix this part next few weeks if you think it's necessary.
Thanks.
Qi

The WebGL issue presumably happens because each Plotly plot has its own
renderer? There is supposedly an ongoing effort to address this issue
here. I'm not sure there
is an easy way to fix this in visdom?
On Sun, Aug 20, 2017 at 11:39 AM, QiChang<EMAIL_ADDRESS>wrote:

Hi,
When I use visdom, there are two things really annoying. One is the order
of the plot is messed up(as #134
https://github.com/facebookresearch/visdom/issues/134 mentioned).
Another one is old context get lost when too many WebGL contexts exist,
even after sorting the plot.
[image: disappeared plot]
https://user-images.githubusercontent.com/19438484/29496191-d0dd60e6-859b-11e7-87c4-9cf5264a2dbd.png
I think I can fix this part next few weeks if you think it's necessary.
Thanks.
Qi
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/facebookresearch/visdom/issues/137, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABHSrtEwKsyvOi4eUPfwtX3unwaIP-yCks5saFMmgaJpZM4O8pf_
.

Thanks for your timely reply, Ivdmaaten,
I think there are several ways to accelerate the process. I just quickly checked the implementation in Visdom. When filtering the plots, Visdom simply hides the other grid. I think if there are too many Plotly plots, it's reasonable to disable the older plots. However, if we filter and narrow down the number of results, Visdom should refresh the whole layout and just render few nodes.

Side notes, it's very strange plot.ly use SVG instead of WebGL to render the plot. I use Chrome which should support WebGL.

Closing this issue as more on the plotly side than visdom - even if we did disable plots on filter, how would we select which ones to re-enable moving forward? Using multiple envs seems to be the correct approach to prevent loss of context in visdom.
If somebody comes up with a good solution/behavior for this issue I'd be happy to accept a PR that implements their solution, otherwise we'll await plot.ly


Please make the conversion operator explicit

While your request is something to consider I don't have any current plans for updating this library.


Comparison operators for all comparables

This adds the default comparison operators, (<, <=, >= and >) for all types derived from Comparable.
While most comparables already implement those operators, this allows to save duplicate work when implementing own comparables. It also makes dealing with comparables that don't override the operators much easier.

Great extensions, thank you!


Index setup bourbon

This uses broccoli-ruby-sass found at
https://github.com/joefiorini/broccoli-ruby-sass and installed with npm.
This also installs bourbon and neat through bower.

I also have reason to believe the original index branch had problems ( #4 ) with sass and bourbon because ember-cli-sass uses libsass instead of ruby sass.


Panic related to ChaCha20-based header protection

We're running v0.14.0 (fork with minor changes) in production and we encounter the following panic from time to time.
panic: chacha20: counter overflow

goroutine 53078166 [running]:
.../vendor/github.com/marten-seemann/chacha20.(*Cipher).XORKeyStream(0xc0031697d8, 0xc00d4d2dc0, 0x5, 0x5, 0xc00d4d2dc0, 0x5, 0x5)
	/go/src/.../vendor/github.com/marten-seemann/chacha20/chacha_generic.go:167 +0xc3b
.../vendor/github.com/marten-seemann/chacha20.XORKeyStream(0xc00d4d2dc0, 0x5, 0x5, 0xc00d4d2dc0, 0x5, 0x5, 0xc00d4d2de5, 0xc00d4d2dc5)
	/go/src/.../vendor/github.com/marten-seemann/chacha20/chacha_generic.go:237 +0x12d
.../vendor/github.com/Psiphon-Labs/quic-go/internal/handshake.(*chachaHeaderProtector).apply(0xc00d4d2dc0, 0xc00e208009, 0x10, 0x5a3, 0xc00e208000, 0xc00e208005, 0x2, 0x5a7)
	/go/src/.../vendor/github.com/Psiphon-Labs/quic-go/internal/handshake/header_protector.go:109 +0xa9
.../vendor/github.com/Psiphon-Labs/quic-go/internal/handshake.(*chachaHeaderProtector).EncryptHeader(0xc00d4d2dc0, 0xc00e208009, 0x10, 0x5a3, 0xc00e208000, 0xc00e208005, 0x2, 0x5a7)
	/go/src/.../vendor/github.com/Psiphon-Labs/quic-go/internal/handshake/header_protector.go:98 +0x77
.../vendor/github.com/Psiphon-Labs/quic-go/internal/handshake.(*updatableAEAD).EncryptHeader(0xc0054fab40, 0xc00e208009, 0x10, 0x5a3, 0xc00e208000, 0xc00e208005, 0x2, 0x5a7)
	/go/src/.../vendor/github.com/Psiphon-Labs/quic-go/internal/handshake/updatable_aead.go:246 +0x8a
.../vendor/github.com/Psiphon-Labs/quic-go.(*packetPacker).writeAndSealPacketWithPadding(0xc00cd6aa90, 0xc0044ec5a0, 0xc00aa7f5e0, 0x1, 0x1, 0x0, 0x4cd, 0x0, 0x3, 0x7f9a46a5acd8, ...)
	/go/src/.../vendor/github.com/Psiphon-Labs/quic-go/packet_packer.go:579 +0x456
.../vendor/github.com/Psiphon-Labs/quic-go.(*packetPacker).writeAndSealPacket(0xc00cd6aa90, 0xc0044ec5a0, 0xc00aa7f5e0, 0x1, 0x1, 0x0, 0x4cd, 0x3, 0x7f9a46a5acd8, 0xc0054fab40, ...)
	/go/src/.../vendor/github.com/Psiphon-Labs/quic-go/packet_packer.go:534 +0x114
.../vendor/github.com/Psiphon-Labs/quic-go.(*packetPacker).maybePackAppDataPacket(0xc00cd6aa90, 0x0, 0x0, 0x0)
	/go/src/.../vendor/github.com/Psiphon-Labs/quic-go/packet_packer.go:401 +0x26a
.../vendor/github.com/Psiphon-Labs/quic-go.(*packetPacker).PackPacket(0xc00cd6aa90, 0xc003169c00, 0x0, 0xc0058630a0)
	/go/src/.../vendor/github.com/Psiphon-Labs/quic-go/packet_packer.go:274 +0x41
.../vendor/github.com/Psiphon-Labs/quic-go.(*session).sendPacket(0xc0027faa00, 0x1, 0xc003169f18, 0xc011d9eae0)
	/go/src/.../vendor/github.com/Psiphon-Labs/quic-go/session.go:1272 +0x80
.../vendor/github.com/Psiphon-Labs/quic-go.(*session).sendPackets(0xc0027faa00, 0x10a164e8d5836, 0x12cbc40)
	/go/src/.../vendor/github.com/Psiphon-Labs/quic-go/session.go:1187 +0x270
.../vendor/github.com/Psiphon-Labs/quic-go.(*session).run(0xc0027faa00, 0x0, 0x0)
	/go/src/.../vendor/github.com/Psiphon-Labs/quic-go/session.go:541 +0x5af
created by .../vendor/github.com/Psiphon-Labs/quic-go.(*baseServer).createNewSession
	/go/src/.../vendor/github.com/Psiphon-Labs/quic-go/server.go:461 +0x3dd

While ChaCha20 allows 2^32 blocks (https://tools.ietf.org/html/rfc7539#section-2.3.2), we observe that the chacha20 code will panic when the counter is at 2^32-1 and processing just 1 block, even though that should be a valid case: https://github.com/marten-seemann/chacha20/blob/36564989294fee5f3957d3e3fbfc655e10786ec0/chacha_generic.go#L165-L16.
Since QUIC ChaCha20-based header protection (https://tools.ietf.org/html/draft-ietf-quic-tls-24#section-5.4.4) initializes the counter with "4 bytes of [...] sampled ciphertext", it appears that it's possible that a counter value of 2^32-1 (0xffffffff) is selected and then chacha20 will panic.
We note that the upstream x/crypto/chacha20 has the same panic bahavior, and a number of other Go ChaCha20 implementations also panic when the counter is 2^32-1 and there's only 1 block to process. But it appears that OpenSSL allows a full 2^32 blocks to be processed before entering its overflow logic: https://github.com/openssl/openssl/blob/706457b7bda7fdbab426b8dce83b318908339da4/crypto/evp/e_chacha20_poly1305.c#L94-L104.
So, we believe that the correct fix is to allow the counter value 2^31-1 and only panic if more than 1 block is processed beyond that point.
We have implemented a proposed fix: https://github.com/Psiphon-Labs/chacha20/commit/899a4be528633ecf678f45e4f6b177d0f89b9e7c.

@rod-hynes Can you give me an update on the status of the fix? It seems like I can't find the corresponding CL that you prepared.
The SetCounter() method was merged a few days ago, but it looks like the overflow bug is still there.

As requested (https://github.com/golang/crypto/pull/108#issuecomment-583527753) I opened an issue: https://github.com/golang/go/issues/37157.
I don't think @lukechampine has created a CL yet?

Oops, slipped my mind, sorry -- will open a CL today!

Thank you @rod-hynes and @lukechampine! Already looking forward to dropping the awkward workaround I had to implement in quic-go to prevent this crash (#2331).

For future reference, this is the CL: https://go-review.googlesource.com/c/crypto/+/220591/.

@lukechampine Any progresss on this issue?

And another CL mentioning this issue: https://go-review.googlesource.com/c/crypto/+/224279/


Accept leading 0x on Elf image base addresses

This finally bugged me enough to make a PR :-)
When setting the Image Base in the options when importing an ELF file, the validator complains if the input is a valid hex number with a 0x prefix. This is mostly an issue when copy pasting an address.

There is a NumericUtilities class containing methods more appropriate to parse the string than Long.parseUnsignedLong. It would be better to just use one of them as they will handle the 0x accordingly.

There is a NumericUtilities or NumericUtils class containing methods more appropriate to parse the string than Long.parseUnsignedLong. It would be better to just use one of them as they will handle the 0x accordingly.
I've felt your pain on this one.

Which API did you mean? I can’t seem to see one which treats numbers without a 0x prefix as hex, and disallows a ‘-‘ to negate the value?
Happy to change it though, if there’s something more suitable.

There is a NumericUtilities or NumericUtils class containing methods more appropriate to parse the string than Long.parseUnsignedLong. It would be better to just use one of them as they will handle the 0x accordingly.
I've felt your pain on this one.

Which API did you mean? I can’t seem to see one which treats numbers without a 0x prefix as hex, and disallows a ‘-‘ to negate the value?
Happy to change it though, if there’s something more suitable.

NumericUtilities.parseHexLong(String)

https://github.com/NationalSecurityAgency/ghidra/blob/ecef7844fd5b224e55595a49acc570c99ddf56f9/Ghidra/Framework/Generic/src/main/java/ghidra/util/NumericUtilities.java#L164

https://github.com/NationalSecurityAgency/ghidra/blob/ecef7844fd5b224e55595a49acc570c99ddf56f9/Ghidra/Framework/Generic/src/main/java/ghidra/util/NumericUtilities.java#L164
I think that API supports negative values though?
https://github.com/NationalSecurityAgency/ghidra/blob/ecef7844fd5b224e55595a49acc570c99ddf56f9/Ghidra/Framework/Generic/src/main/java/ghidra/util/NumericUtilities.java#L177

If a user inputs a negative number the sign extended number up to 64 bits would be the result. I don't think it should matter but it is probably best to leave it up to the devs to decide.

Ok. Thanks for the feedback. I’ll leave it as is for now and see what they come back with. Happy to go with whatever best fits though.
Just so long as I don’t have to go back and delete a leading 0x prefix any more!


TypeError: Cannot join tz-naive with tz-aware DatetimeIndex

Ran this:
from pandas_datareader import data as pdr
import datetime
import fix_yahoo_finance

df = pdr.get_data_yahoo(
    tickers = '^GSPC',
    start   = datetime.date(2017, 5, 7),
    end     = datetime.date(2017, 6, 21)
)

Got this:

TypeError: Cannot join tz-naive with tz-aware DatetimeIndex

Sorry fixed with new build.


Make function supports to use package command

Master Issue: #8676
Motivation
We have a new package management service which can manage all
the packages. We can use that in the Pulsar Function will make the
function packages are easier to manage.
I reuse the 'packageUrl' for downloading the package from the packages
management service. So user can use the package name as the 'packageUrl'
to download it.
Modifications

Support new package type URL for creating and updating functions
Support to use package download command for Kubernetes runtime
Add tests for the changes

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


Add a README explaining the backpressure

It's not complete but perhaps a good starting point :)

I thought that the site would contain user focused documentation, whereas the README would contain instructions for contributors. I like when you don't have to click around to find the information :)
Feel free to move this to the documentation if you disagree though :)

Thanks a lot for this, @backuitist! I believe it's nice to have all high-level documenation in one place. I'll take a stab at moving this to the site and adding some notes from myself as well, when I get some time.

Neat! Thanks!


Non-shaka error caught in streaming_engine when loading HLS manifest in the presence of mux.js

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
2.5.5
Can you reproduce the issue with our latest release version?
Yes
Can you reproduce the issue with the latest code from master?
Unknown - I am using the google hosted very
Are you using the demo app or your own custom app?
Custom
If custom app, can you reproduce the issue using our demo app?
N/A - depends on my manifest
What browser and OS are you using?
Safari and Firefox on Mac OS X 10.13.6
For embedded devices (smart TVs, etc.), what model and firmware version are you using?
N/A
What are the manifest and license server URIs?
Manifest:
https://cod.ckcufm.com/cgi-bin/stream-server?broadcast_id=45185&program_id=31&streamtype=hlsmaster
Player as embedded:
https://cod.ckcufm.com/programs/ondemand-player.html?id=31356&token=O}6]sg\2(wUsgXPIcT7%2600%2C6UztHNm\HE`1msAt~b%2Bm'0sJp~u
License server: N/A
What did you do?
Playback of an HLS playlist in the presence of mux.js 5.5.1
What did you expect to happen?
Audio!
What actually happened?
Audio player loaded media but failed to start playing, with errors in the console that I was not able to figure out even in debug mode (see image). The error appears to be a non-Shaka error that Shaka is not happy to catch at this point in the code.

Note that if I remove the mux.js library from the head element, then as expected Shaka generates error code 4032 for this HLS stream. I am not an expert in deploying js projects, so it is possible that I have got something wrong with my mux.js build. However, I can't tell what this is because Shaka is eating the error.

Thank you for the report!  We will try to reproduce the issue with the manifest URL you included.

It looks like the check for captions isn't valid and we try to append embedded captions for the stream.  Since there isn't any video stream, we can't find its timestampOffset.  This is the culprit line, caused by the if statement checking for truthy values instead of checking for a non-empty list.


mysql 5.6 not working

docker run -p 3306:3306 tutum/mysql:5.6

.....
.....
Status: Downloaded newer image for tutum/mysql:5.6
=> An empty or uninitialized MySQL volume is detected in /var/lib/mysql
=> Installing MySQL ...
=> Done!
=> Creating admin user ...
=> Waiting for confirmation of MySQL service startup, trying 0/13 ...
=> Waiting for confirmation of MySQL service startup, trying 1/13 ...
=> Waiting for confirmation of MySQL service startup, trying 2/13 ...
=> Waiting for confirmation of MySQL service startup, trying 3/13 ...
=> Waiting for confirmation of MySQL service startup, trying 4/13 ...
=> Waiting for confirmation of MySQL service startup, trying 5/13 ...
=> Waiting for confirmation of MySQL service startup, trying 6/13 ...
=> Waiting for confirmation of MySQL service startup, trying 7/13 ...
=> Waiting for confirmation of MySQL service startup, trying 8/13 ...
=> Waiting for confirmation of MySQL service startup, trying 9/13 ...
=> Waiting for confirmation of MySQL service startup, trying 10/13 ...
=> Waiting for confirmation of MySQL service startup, trying 11/13 ...

What does the error log print after the time out of the startup of mysql?

Sorry for the short log. My host machine is Ubuntu 14.04 (hosted at DigitalOcean 1GB) and here is the full log:
root@dev:~# docker run tutum/mysql:5.6
=> An empty or uninitialized MySQL volume is detected in /var/lib/mysql
=> Installing MySQL ...
=> Done!
=> Creating admin user ...
=> Waiting for confirmation of MySQL service startup, trying 0/13 ...
=> Waiting for confirmation of MySQL service startup, trying 1/13 ...
=> Waiting for confirmation of MySQL service startup, trying 2/13 ...
=> Waiting for confirmation of MySQL service startup, trying 3/13 ...
=> Waiting for confirmation of MySQL service startup, trying 4/13 ...
=> Waiting for confirmation of MySQL service startup, trying 5/13 ...
=> Waiting for confirmation of MySQL service startup, trying 6/13 ...
=> Waiting for confirmation of MySQL service startup, trying 7/13 ...
=> Waiting for confirmation of MySQL service startup, trying 8/13 ...
=> Waiting for confirmation of MySQL service startup, trying 9/13 ...
=> Waiting for confirmation of MySQL service startup, trying 10/13 ...
=> Waiting for confirmation of MySQL service startup, trying 11/13 ...
=> Waiting for confirmation of MySQL service startup, trying 12/13 ...
Time out. Error log is shown as below:
2015-01-03 06:07:37 17 [Note] InnoDB: Setting file ./ibdata1 size to 12 MB
2015-01-03 06:07:37 17 [Note] InnoDB: Database physically writes the file full: wait...
2015-01-03 06:07:38 17 [Note] InnoDB: Setting log file ./ib_logfile101 size to 48 MB
2015-01-03 06:07:38 17 [Note] InnoDB: Setting log file ./ib_logfile1 size to 48 MB
2015-01-03 06:07:38 17 [Note] InnoDB: Renaming log file ./ib_logfile101 to ./ib_logfile0
2015-01-03 06:07:38 17 [Warning] InnoDB: New log files created, LSN=45781
2015-01-03 06:07:38 17 [Note] InnoDB: Doublewrite buffer not found: creating new
2015-01-03 06:07:38 17 [Note] InnoDB: Doublewrite buffer created
2015-01-03 06:07:38 17 [Note] InnoDB: 128 rollback segment(s) are active.
2015-01-03 06:07:38 17 [Warning] InnoDB: Creating foreign key constraint system tables.
2015-01-03 06:07:38 17 [Note] InnoDB: Foreign key constraint system tables created
2015-01-03 06:07:38 17 [Note] InnoDB: Creating tablespace and datafile system tables.
2015-01-03 06:07:38 17 [Note] InnoDB: Tablespace and datafile system tables created.
2015-01-03 06:07:38 17 [Note] InnoDB: Waiting for purge to start
2015-01-03 06:07:38 17 [Note] InnoDB: 5.6.19 started; log sequence number 0
2015-01-03 06:07:39 17 [Note] Binlog end
2015-01-03 06:07:39 17 [Note] InnoDB: FTS optimize thread exiting.
2015-01-03 06:07:39 17 [Note] InnoDB: Starting shutdown...
2015-01-03 06:07:40 17 [Note] InnoDB: Shutdown completed; log sequence number 1625977
2015-01-03 06:07:40 41 [Note] InnoDB: Using atomics to ref count buffer pool pages
2015-01-03 06:07:40 41 [Note] InnoDB: The InnoDB memory heap is disabled
2015-01-03 06:07:40 41 [Note] InnoDB: Mutexes and rw_locks use GCC atomic builtins
2015-01-03 06:07:40 41 [Note] InnoDB: Compressed tables use zlib 1.2.8
2015-01-03 06:07:40 41 [Note] InnoDB: Using Linux native AIO
2015-01-03 06:07:40 41 [Note] InnoDB: Using CPU crc32 instructions
2015-01-03 06:07:40 41 [Note] InnoDB: Initializing buffer pool, size = 128.0M
2015-01-03 06:07:40 41 [Note] InnoDB: Completed initialization of buffer pool
2015-01-03 06:07:40 41 [Note] InnoDB: Highest supported file format is Barracuda.
2015-01-03 06:07:40 41 [Note] InnoDB: 128 rollback segment(s) are active.
2015-01-03 06:07:40 41 [Note] InnoDB: Waiting for purge to start
2015-01-03 06:07:40 41 [Note] InnoDB: 5.6.19 started; log sequence number 1625977
2015-01-03 06:07:40 41 [Note] Binlog end
2015-01-03 06:07:40 41 [Note] InnoDB: FTS optimize thread exiting.
2015-01-03 06:07:40 41 [Note] InnoDB: Starting shutdown...
2015-01-03 06:07:42 41 [Note] InnoDB: Shutdown completed; log sequence number 1625987
150103 06:07:43 mysqld_safe Starting mysqld daemon with databases from /var/lib/mysql
2015-01-03 06:07:43 0 [Warning] Using unique option prefix key_buffer instead of key_buffer_size is deprecated and will be removed in a future release. Please use the full name instead.
2015-01-03 06:07:43 0 [Warning] Using unique option prefix key_buffer instead of key_buffer_size is deprecated and will be removed in a future release. Please use the full name instead.
2015-01-03 06:07:43 0 [Warning] TIMESTAMP with implicit DEFAULT value is deprecated. Please use --explicit_defaults_for_timestamp server option (see documentation for more details).
2015-01-03 06:07:43 999 [Warning] Using unique option prefix myisam-recover instead of myisam-recover-options is deprecated and will be removed in a future release. Please use the full name instead.
2015-01-03 06:07:43 999 [Warning] Using unique option prefix myisam-recover instead of myisam-recover-options is deprecated and will be removed in a future release.  The following is an example of working mysql container(5.5):
root@dev:~# docker run tutum/mysql:5.5
=> An empty or uninitialized MySQL volume is detected in /var/lib/mysql
=> Installing MySQL ...
=> Done!
=> Creating admin user ...
=> Waiting for confirmation of MySQL service startup, trying 0/13 ...
=> Creating MySQL user admin with random password
=> Done!
========================================================================
You can now connect to this MySQL Server using:

    mysql -uadmin -pNIEMoZa4nLsC -h<host> -P<port>

Please remember to change the above password as soon as possible!
MySQL user 'root' has no password but only allows local connections
========================================================================
150103 06:14:09 mysqld_safe Can't log to error log and syslog at the same time.  Remove all --log-error configuration options for --syslog to take effect.
150103 06:14:09 mysqld_safe Logging to '/var/log/mysql/error.log'.
150103 06:14:09 mysqld_safe Starting mysqld daemon with databases from /var/lib/mysql

2015-01-03 06:07:43 999 [Note] InnoDB: Initializing buffer pool, size = 128.0M
InnoDB: mmap(137363456 bytes) failed; errno 12
2015-01-03 06:07:43 999 [ERROR] InnoDB: Cannot allocate memory for the buffer pool

It seems that there is not enough memory to run mysql 5.6

I wonder what are the minimum requirements to run mysql 5.6 . Currently I tried on Ubuntu 14.04 with 1GB memory and it failed.

I use a 1G Ram VM and mysql 5.6 works fine.
The default InnoDB buffer pool size is 128MB, as long as the free ram is greater than it, mysql should work.
You may find this post useful: http://stackoverflow.com/questions/12114746/mysqld-service-stops-once-a-day-on-ec2-server

me too,Mysql 5.6 works bad, I have 512 Ram VM and mysql 5.5 works fine.

Maybe do:
docker run --name sql -p 3306:3306 -v /tmp:/tmp -e MYSQL_user="root" -e MYSQL_PASS="[PASS]" -e STARTUP_SQL="/tmp/db.sql" tutum/mysql:5.5
If it's inside the /tmp folder, it seems to work.


Add disable switch

Add tag disable on switch component

Hi!
Such switches are already introduced.
Thank you for your PR!


Experimenting with words supplied by API Server via Params

This is a development PR and for the time being should not be merged into main. There will be plenty of issues with the code. However, it can become a common branch for developing a workable solution.
development branch: Jr/dev play api words

After a bit more testing, I think this code is good to go. I'll mark for review and await change request or approval.


PhysicalBone3D and PhysicalBoneSimulator3D is not following animations

Tested versions
Reproducible in 4.3
System information
Godot v4.3.stable - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 4080 Laptop GPU (NVIDIA; <IP_ADDRESS>94) - Intel(R) Core(TM) i9-14900HX (32 Threads)
Issue description
In 4.3 PhysicalBone3D nodes are not following animations anymore. They are standing still in the rest position and not moving unless it is simulating physics. Breaking the ragdoll system and hit traces made with 4.2
Rest poses seem to be broken as well

Steps to reproduce
Click Skeleton -> Create Physical Skeleton on an animated Skeleton3D
Minimal reproduction project (MRP)
N/A

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

to fix your trouble check this fix, i see it in another issue,
https://app.mediafire.com/9mrkd33xulszl
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Could you provide a minimal reproduction project that shows the issue? It's much easier for contributors to debug and fix issues if they can make sure they can see what you see.

sure. wait a min.

PhysicalBone3DBug.zip

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."

PhysicalBone3DBug42.zip
And this is the 4.2 version for comparison

Made this gif to further demonstrate the issue.

I can confirm the regression with the MRPs. Starting with the 4.2 project, it seems to start breaking with 4.3.dev6.
This error gets spammed:
ERROR: Condition "bones[p_bone].physical_bone" is true.
   at: bind_physical_bone_to_bone (scene/3d/physical_bone_simulator_3d.cpp:129)

CC @lyuma @TokageItLab @fire

@akien-mga it is closed as completed but there are still issues.
As of 4.4 dev2 physical bones while following animations and set_bone_global_pose_override() method they are ignoring SkeletonModify3D and SkeletonIK3D. Since we are using physical bones for the bullet traces too this causes issues.
And set_bone_global_pose_override() is marked as depreciated. If we will have to use SkeletonModify3D instead this will cause further issues in the future.
You can look at the legs to see the difference between while SkeletonIK3D is applied and simulating physics.

@eddieataberk I sent https://github.com/godotengine/godot/pull/97007 to fix it. Note that SkeletonModifier updates depending on the order of the tree, so make sure to put PhysicalBoneSimulator after IK and other modifiers.

@TokageItLab Thank you. Tested it now physical bones follows the animation and modifiers correctly.
But everything else related to IK and animations seems broken now.

layered animations also have some issues some animations looks one way some another maybe there are other changes in the source build affecting it i am not sure.

@eddieataberk What I can think of where this happens is that the CallbackModeProcess does not match between Skeleton and AnimationMixer. They need to match, make sure that.

But if it's caused by something else, I don't have idea, I remember there was a bug in the import around AnimationLibrary, which may have something to do with it.

@TokageItLab I believe it's something to do with importer. I am losing my mind right now lol. Since I opened the project with the custom build. i tried deleting .godot file reverting all the files from github and reimporting it with 4.3 and 4.4dev2 and issue remains even on 4.3 now. It seems bone rotations are off for some reason.

i also noticed a bunch of this error. maybe related.

I am 100% certain the issue is from Animation Library workflow. I have no idea why it didn't happened before but the easiest solution I found is to combine all animations into a single file. I was enjoying the animation library workflow and in editor retargeting but it's not reliable as of 4.3 and 4.4. If anyone else is having the same issue spending a little bit of time in Blender is more robust solution.

Basically, I think it's a compatibility issue due to AnimationLibrary regression caused by the recently introduced StringName change. I believe the latest version has some fixes.

https://github.com/godotengine/godot/pull/96586
https://github.com/godotengine/godot/pull/95505
https://github.com/godotengine/godot/pull/96577
I forgot to note that such dangers lurk in the dev branch, sorry.


Empty message check

I noticed that if a message is an empty string, it goes through every if, so I added this little check to avoid it

I think this should stay as it is. Because there maybe commands which need empty lines. this chnage is to risky.

Oh, ok.
I just saw it was useless during the debug.
Should I close the PR?


bisecting broken: Script returned error

I have a script like
#!/bin/sh
! rustc /tmp/crash.rs  |& grep -Ii "query stack"

which returns 1 or 0 depending on  whether rustc says "query stack"  while processing a file.
When I try to run this after the update
cargo-bisect-rustc --preserve --script=/home/matthias/vcs/github/rust_bisect/script.sh --test-dir=/home/matthias/vcs/github/rust_bisect/foo --start 2024-06-15
CBR just quits:
fetching https://static.rust-lang.org/dist/channel-rust-nightly-date.txt
nightly date: 10 B / 10 B [===========================================================================================================] 100.00 % 755.04 KB/s determined the latest nightly is 2024-06-23
checking the start range to find a passing nightly
installing nightly-2024-06-15
testing...
RESULT: nightly-2024-06-15, ===> Script returned error

ERROR: the start of the range (nightly-2024-06-15) must not reproduce the regression

cc @ehuss
cargo-bisect-rustc 0.6.9

Can you say more about what the issue is? It looks like the start of your range is either failing to compile, or emitting the "query stack" message. What are the contents of crash.rs? What is the output with -vv?

Ooh, did only the CBR output change?
This looked like cbr could not proceed because of some error related to the script.

Yea, instead of saying "Yes" or "No", it tells you what happened at each step.

would be clearer if it would also say the exit code imo, something like
Script returned error (exit code: 1)

Yea, it would be nice to include the exit code. I'm not sure how difficult that will be since the code that renders the message may not have access to the exit code.


Allow DocumentReference.context.encounter

We have the situation that in an internal document exchange scenario between systems within a hospital (at least in Germany), the encounter context is pretty important both for querying as well as during the document provide interaction. So if we were do define profiles for this scenario we'd be incompatible with MHD due to the 0..0 restriction on DocumentReference.context.encounter.
I understand that encounter context is not relevant/needed in MHD/XDS context, but I would assume that in most internal scenarios, e.g. where FHIR replaces HL7 V2 MDM communication, the provision of encounter information will be mandatory.
The question this raises is: should these internally used DocumentReferences be reuseable - as is - for MHD or do vendors in fact need to actively remove the encounter reference?
Is the fact that the information would be dropped when mapping to XDS enough reason to completely rule out the element? In my interpretation of cardinalities, I would assume that the encounter reference should simply lack the mustSupport flag to indicate that it won't be processed within the MDH scope.
We are currently specifying such an internal document exchange and aim to harmonize with MHD as much as possible. However with the encounter reference we'd be forced to break compatibility.
see Zulip discussion here: https://chat.fhir.org/#narrow/stream/179223-ihe/topic/MHD.20DocumentReference.2Econtext.2Eencounter.20forbidden

I think that the .encounter could easily map to DocumentEntry.referenceIdList. The complexity that there are two FHIR elements that map to the one XDS element can be overcome. The use of referenceIdList does include carrying identifiers like encounter. So this is well within the intended use of the element.
MHD Document Consumer should always expect that any of the FHIR elements might be populated, so the current specification does not forbid returning it to a MHD Document Consumer.

committee discussion:
likely good element for MHDS backends.
With XDS a conversion from MHD to XDS can put the id into referenceIdList;
Data coming from XDS referenceIdList would not be expected (but not forbidden) from being translated into .encounter.

added a section to ITI-67 explaining populating Resource References -- <IP_ADDRESS>.2.1.5
removed the 0..0 on encounter
added a map for encounter to referenceIdList

@JohnMoehrke with ReferenceIdList, use the CX type for encounter to enable putting ReferenceIdList that are encounter types into the DocumentReference.encounter.identifier.
Table <IP_ADDRESS>.7-2 - urn:ihe:iti:xds:2015:encounterId

pull request #135 closed


⚠ Bofrost.lu has degraded performance

In 0c5e491, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1140 ms

Resolved: Bofrost.lu performance has improved in c6d399a after 6 minutes.


simplify usage of map method

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [JAV-XXX] Fixes bug in ApproximateQuantiles, where you replace JAV-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

fix #370

Coverage increased (+0.009%) to 86.935% when pulling cf9fabfd4d1418e5e1f5bd1a4f216e4c1c029b8d on linshihang:issue-370 into 50dfedb8fa26e3453837601acab3ec2a6193b59e on ServiceComb:master.


What would a response payload look like for an array of orders?

Hi,
I'm just looking into Siren and writing a C# library that can be used with NancyFX to return Siren responses.
As part of my demo I was going to use your Order example. On my / route in the API I return an array of orders but I'm unsure what the payload should look like.
Any chance you could help me out?
Thanks

It would look like the example on the main page except multiple orders inside an array
[
{
// order
},
{
// order
},
.....
]

Sorry I deleted my previous comment, I wasn't thinking.
You would probably want to return an orders collection where the entities are all the orders:
Here's a simple example:
{
  "class": [ "collection" ],
  "entities": [
    { // order 1},
    { // order 2},
    { // order N}
  ]
"links": [
    { "rel": [ "self" ], "href": "http://api.x.io/orders" }
]

So this is what I have. How does it look? The one issue I have is on the action of creating an order. I've put array as the type where you would POST an array of items to create the order but I'm pretty sure this is wrong.

{
    "class": [
        "collection"
    ],
    "properties": null,
    "entities": [
        {
            "class": [
                "order"
            ],
            "rel": [
                "http://localhost:8080/rels/order"
            ],
            "href": "http://localhost:8080/orders/45323",
            "properties": {
                "orderNumber": 45323,
                "itemCount": 2,
                "status": "Pending"
            },
            "links": null
        },
        {
            "class": [
                "order"
            ],
            "rel": [
                "http://localhost:8080/rels/order"
            ],
            "href": "http://localhost:8080/orders/6534",
            "properties": {
                "orderNumber": 6534,
                "itemCount": 5,
                "status": "Completed"
            },
            "links": null
        }
    ],
    "actions": [
        {
            "name": "add-order",
            "title": "Add Order",
            "method": "POST",
            "href": "http://localhost:8080//orders/",
            "type": "application/json",
            "fields": [
                {
                    "name": "orderitems",
                    "type": "array",
                    "value": null
                }
            ]
        }
    ],
    "links": [
        {
            "rel": [
                "self"
            ],
            "href": "http://localhost:8080/orders/"
        }
    ]
}

I've pushed my code although still WIP here if you want to see https://github.com/jchannon/Nancy.Siren

WRT to the general response it looks pretty good.  Personally I'd avoid putting null for properties, just put an empty hash or skip having properties at all.
I don't personally use posting of json, but generally the action looks right to me.
BTW, I just realized this is not the appropriate place for general how to questions.  The discussion group is more appropriate:  https://groups.google.com/forum/#!forum/siren-hypermedia
You should close this issue and if you have more questions ask at the discussion group.

Have continued it here - https://groups.google.com/forum/#!topic/siren-hypermedia/hU1UTiqSW80


Added option to delay completion request

Closes #19
Implemented delay between firing inline completion event from VSCode and inference logic to avoid unnecessary completion requests (i.e. when user is typing).
Added corresponding option to extension's settings (250ms seems perfect to me).

Very useful! Thanks!


Autocomplete/intellisense/types for "actions" in defineStore() typed as "any" (missing)

You can actually see this in the demo projects on stackblitz, when writing a func in actions, this is typed as any and doesn't provide any autocomplete/intellisense/types

export const useCounter = defineStore({
  id: 'counter',
  state: () => ({
    n: 2,
  }),
  getters: {
    double: (state) => state.n * 2,
  },
  actions: {
    increment(amount = 1) {
      this.n += amount // this is "any", not typesafe
    },
  },
})

It was missing the strict: true in the TS config

even with strict: true the error is still there.

I was hitting this problem after refactoring a mono repo. The original package had the strict: true setting while the new one had not. Why is this necessary? I think this should be in the docs. It is hard to find the origin of this configuration error.

I had a similar issue in my TypeScript code and was able to get around it by typecasting the actions to ThisType<MyAppState> something like:
type MyAppState = {
  counter: number;
}

export const useCounter = defineStore("xyz", {
  state: (): MyAppState => ({
    counter: 0
  }),
  action: {
    increment() {
    }
  } as ThisType<MyAppState>
});


Allow getting out of "modal mode" by pressing ESC

Currently, the only. way to get out of "modal mode" is the press M-m again. It would be nice to also allow ESC to achieve the same thing

So we could maybe add an extra configurable "exit" keybinding, i.e. M-m behaves as usual enter/exit but one extra key can be set to also exit (e.g. ESC, i or whatever).
I'll work on this soon. Please stand-by!

Nice! Could it be somehow "appending" the key? Basically, allow exiting out by pressing either ESC or M-m?

I believe it should be consistent, i.e. one keybinding does one thing. If we have several ways of doing the same thing, it would get confusing (e.g. "do I need to override two variables for the same thing"). It's easier to learn one thing and stick to it :)

Sure, of course whatever I say is only suggestion, you're the one making the final decision 🙂
And I guess we can close this now?


boards/esp32s3: Link stack checking function and data to SRAM when enable flash or PSRAM driver

Summary
This PR intends to fix the PSRAM support on ESP32S3 when Stack Smash Protection is enabled (CONFIG_STACK_CANARIES=1).
During PSRAM initialization and flash operations, the Cache needs to be disabled, so all data and code for the aforementioned scope is required to be placed in Internal RAM (DRAM and IRAM regions).
Impact
Only for PSRAM-enabled configurations on ESP32S3.
Testing

Successful execution of ramtest in esp32s3-devkit:psram_octal with CONFIG_STACK_CANARIES=1

@cwespressif
Please resolve the conflicts.

@cwespressif Please resolve the conflicts.

Done


Missing icon: solid > login

There is one icon missing in the 102 new icons, outline set: login.

That’s been noted here https://github.com/tailwindlabs/heroicons/issues/90 FWIW.

Added 👍


Add Task and TaskExecutor @open sesame 11/10 08:57

Task has several properties for its state and working information.
There are two types for Task, Synchronous and Asynchronous. and, their
behaviors will be differ when they are executed by TaskExecutor.
TaskExecutor class manages task's execution by their type.
Synchoronous tasks are exeucuted immediately, while asynchoronous tasks
are executed in thread manner.
For testing above two classes, unittest codes are also included.
The mock class for TaskExecutor is introduced to check work thread is
correctly working, and completion check is also working correctly.
Many tests are added to check asynchronous tasks' correctness.
Issue (#2034)
Signed-off-by: Jiho Chu<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2033. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2033-202211072209450.36585903167725-4d3190bd9a9cdad82a6b3bfecdf8dd395a27ef7d/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2033-202211091358570.69702196121216-5595a66632ab419f464054d31ee3994f392d2529/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2033-202211091709520.447026014328-5595a66632ab419f464054d31ee3994f392d2529/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2033-202211091739190.61138892173767-5595a66632ab419f464054d31ee3994f392d2529/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2033-202211100857320.028469085693359-5595a66632ab419f464054d31ee3994f392d2529/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2033-202211101036030.23885893821716-d526405b1d8924d80d3ae40a1dc019ec0bf16fd5/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2033-202211101140420.3255569934845-9d0f0837c3226eb26650ef347b41b3b45b3b7540/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2033-202211101653010.26751899719238-fe74e09c18a34f2bcc487154399c954ff061636e/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2033-202211111050430.44065093994141-fe74e09c18a34f2bcc487154399c954ff061636e/.


enable-rule, enable-decorator args

Is your feature request related to a problem? Please describe.
There is a way to skip rules and decorators but no way to enable them by args.
I think this causes a user experience problem. This means to enable any decorator or rule requires adding it explicitly to the .redocly.yaml file. This is problematic when someone doesn't even have a .redocly.yaml file -- they need to:

Create a .redocly.yaml file (which has its own challenges because commonly used operating systems hide dot files by default)
Learn the structure of the file and fill it out to enable a rule or decorator. Save the file.
Commit to source control, upload, or CLI push again.

Describe the solution you'd like
I would like to be able to specify enabling some rule or decorator at runtime. For example, removing internal APIs....
--enable-decorator=remove-x-internal,remove-unused-components
Describe alternatives you've considered
Auto-generating .redocly.yaml file on the fly from a wizard.
Additional context
I am envisioning what the usage may be like from within the API registry in workflows, where someone may be able to select from specific rules and decorators.

This one needs some thought. I'm not coming into this with certainty in this solution.
See here for some other context: https://github.com/Redocly/openapi-cli/pull/443#pullrequestreview-842344756

This one needs some thought. I'm not coming into this with certainty in this solution.
See here for some other context: https://github.com/Redocly/openapi-cli/pull/443#pullrequestreview-842344756

I'm not sure about --enable-rule as rules have to have a severity specified while a decorator may be just turned on.

Suggestions about parameters:
--enable-decorator=remove-x-internal{key:value},remove-unused-components
--enable-decorator=remove-x-internal@key:value,key2:value2,remove-unused-components

Thank you @andriyl  -- I like the {key:value,key2:value2}. But, don't proceed yet -- let's get more feedback.

As it turns out this issue exists for decorators and rules.

Do you mean ability to enable rule from CLI? or ability to specify options via CLI? Could you elaborate?

I suggest using space instead of , when passing the list of decorators or rules.
--enable-decorator=remove-x-internal remove-unused-components  as a result, we get here:
[ 'remove-x-internal', 'remove-unused-components' ].
Otherwise using coma: [ 'remove-x-internal,remove-unused-components' ]

This might also be solved with further improvements to the config format (started here: #87 ).

I think it should be closed after we updated redocly.yaml file format. @adamaltman what do you think?

This was resolved by https://github.com/Redocly/openapi-cli/pull/588.
Someone can create multiple entrypoints with different decorators setup.


Custom file name template

考虑支持自定义的文件名吗？
比如 中文版生成的文件名含有中文，import一个中文文件，感觉不是很靠谱，所以每次都要手动修改。
我觉得，文件名最好支持模版插值，可以取{tag}{difficulty}等几个meta信息填充，这样也可以满足不同人的需求。

@poppinlp Guessing this might be a regression in #149
If the user is using Chinese version LeetCode, the file name we generated will contain Chinese

@poppinlp Guessing this might be a regression in #149
If the user is using Chinese version LeetCode, the file name we generated will contain Chinese

I think these two issues are not the same function, maybe you didn't understand what I meant. I am suggesting to add a configuration item, just like outputFolder, does not affect the reading of local files.

Setting Name
Description
Default Value

fileName
The name of the generated file;{id}-{difficulty}-{tag}-{name}
{id}.{name}.{ext}

like this? @jdneo

@jdneo
I have not tested on the Chinese version LeetCode. That's my fault. I'll fix it and think about the test cases.
@Dec-F
Yep. You are talking about a different feature. I think I get your point.
What's your option about this? @jdneo I think this feature is reasonable but not high priority.

If this feature is necessary, I will submit a PR tomorrow or weekend. What do you think? @jdneo @poppinlp

@poppinlp Not your fault, we all missed that part, It's great if this can be fixed.
@Dec-F PR is welcome, you can send out it whenever you want.

@jdneo  I've sent a PR to fix the Chinese file name issue.
@jdneo @Dec-F Here maybe a plan to support the file name pattern:
We could pass the whole config into the genFileName function instead of the only endpoint right now to avoid too many params. And then generate the file name by the pattern in the setting. And don't forget to update the documents after that.
The variable in pattern could be:

id
name
difficulty
ext
companies(join the array in kebabcase?)
tags(join the array in kebabcase?)

for a certain problem, it may belong to multiple companies and tags, how to handle this senario?

On the one side, if we wanna support company and tag, of course, the tags and companies could be an array. So we may just merge the array items with a separate symbol such as underscore, dash, e.t.c.
On the other side, I think the first question is what should we support company and tag? For me, there's no such need. But maybe there could be a situation for others that they wanna grep the files by company name or tag. I'm not really sure about this.

I think there are two ways to deal with it.

Let the user choose. Treat like this
fileName option Support function, handed to user programming

What do you think?

@Dec-F Will your request get satisfied if we implement the comment here: https://github.com/jdneo/vscode-leetcode/issues/119#issuecomment-466778120

@magic-akari This is the issue.
I think the outputFolder and the fileName could be combined into one setting. Like leetcode.relativeFilePath.

I think the outputFolder and the fileName could be combined into one setting. Like leetcode.relativeFilePath.

Does this mean that outputFolder will be deprecated?

Yes. Make it deprecated and stop honor this setting in the future.
For now, we will honor both outputFolder and relativeFilePath. If the user set both two settings, relativeFilePath wins.

I prefer the following config format
{
    "leetcode.relativeFilePath": {
        "default": "{language}/{id}.{kebab-case-name}.{ext}",
        "java": "java/src/{id}-{BigCamelCaseName}/main.{ext}",
        "cpp": "cpp/src/{id}-{smallCamelCaseName}.cc",
        "rust": "src/p{id}_{snake_case_name}.{ext}"
    }
}

@magic-akari 可以简单一下，第一阶段先不考虑不同语言的命名习惯，今后可以根据不同语言生成不同样式的文件名。需要支持的变量可以有：

${id}
${difficulty}
${tag}
${name}

另一个问题，怎么比较好地区分folder和filename. 举例来说，有一个用户他只想自定义文件夹路径，但是使用了该配置项之后，他必须把文件名格式也补全。感觉似乎不是一个很好的用户体验

另一个问题，怎么比较好地区分folder和filename. 举例来说，有一个用户他只想自定义文件夹路径，但是使用了该配置项之后，他必须把文件名格式也补全。感觉似乎不是一个很好的用户体验

什么都不设置，会用 default 设置吧，最开始的时候生成一个。
倒是可以考虑不废弃 outputFolder，然后把 relativeFilePath 接在 outputFolder 后边，但是这样做同样是令人困惑的。

不废弃 outputFolder 的话，那么setting名称就是fileName了。让我再想一想这个 experience 应该是怎么样的

第一阶段先不考虑不同语言的命名习惯，今后可以根据不同语言生成不同样式的文件名。需要支持的变量可以有：

其实我就是因为遇到了命名的问题，才来这里寻找解决办法的。
比如现在默认的命名，有的语言没办法导入这种数字开头命名的文件。想导入函数做测试的时候就遇到麻烦了。

I think that merging into a configuration may not be a good idea. Atomic configuration understanding costs may be lower.
A good example is the path and filename in the webpack output configuration.

或者平铺这些设置，
{
    "leetcode.relativeFilePath": "{language}/{id}.{kebab-case-name}.{ext}",
    "leetcode.relativeFilePath.java": "java/src/{id}-{camelCaseName}/main.{ext}",
    "leetcode.relativeFilePath.cpp": "cpp/src/{id}-{PascalCaseName}.cc",
    "leetcode.relativeFilePath.rust": "src/p{id}_{snake_case_name}.{ext}"
}

I agree with @Dec-F's opinion. It makes me confused to merge outputFolder and fileName. IMO, the splitted setting leetcode.fileName makes more sense.

如果分开设置 filename 和 outputFolder，为每个语言设置独立的输出就会遇到困难。
我觉得每个语言可独立设置还是挺重要的，毕竟有的语言对名称有要求。

I don't get your point. Why splitting filename and outputFolder will make it different to generate different name style for different language?

Let's have a try.
We split the output config into filename and path. Now we have to write the config like following.
{
    "leetcode.output.default.path": "...",
    "leetcode.output.default.filename": "...",
    "leetcode.output.java.path": "...",
    "leetcode.output.java.filename": "...",
    "leetcode.output.cpp.path": "...",
    "leetcode.output.cpp.filename": "...",
    "leetcode.output.rust.path": "...",
    "leetcode.output.rust.filename": "..."
}

or
{
    "leetcode.output.default": {
        "path": "",
        "filename": ""
    },
    "leetcode.output.java": {
        "path": "",
        "filename": ""
    },
    "leetcode.output.cpp": {
        "path": "",
        "filename": ""
    },
    "leetcode.output.rust": {
        "path": "",
        "filename": ""
    }
}

There is too much noise in the first one.
The second, we cannot write config in vscode settings UI.

@magic-akari Yes, I agree with your point. But one thing we need to figure out is how to make this setting easy to understand. Because actually we combined two settings into one. What if:

The user only want to set the folder
Or the user only want to set the file name

Things will become tricky during the implementation, because we have to guess which part is for folder and which part is for the file name.

The user only want to set the folder
Or the user only want to set the file name

We can set the default value of leetcode.relativeFilePath.default.
It's easy to change it in vscode settings UI.

It will be logically equivalent to the current project.

So what's the expected behavior if the user set leetcode.relativeFilePath.java to: com.leetcode.problems. Or even com.leetcode.java(for folder name)

Do you mean com.leetcode.problems/${id}.${camelCaseName}.${ext} ?
Copy the deafult value and paste it into Java input.
These efforts are required when the user needs to set it up.

I mean, the user only want to set the folder path. So he set it to com.leetcode.problems or even com.leetcode.java. How will the extension know that this is for the file name or its just a folder name.

我们应该让用户输入完整的路径。而不是允许输入部分，然后猜测用户要做什么。
使用默认值的时候，用户什么都不需要做，一切逻辑和现在的项目逻辑等价。
当用户真的有需求，要改变某个语言的输出文件夹的时候，从默认值哪里复制粘贴出${id}.${camelCaseName}.${ext}，然后在前边补上路径。
需要个性化设置，不至于这一点努力都不愿意付出吧。

或许，有什么其他更方便的方法？
但是我觉得这样是最明确，不含糊的。

要么，放弃直接在 vscode settings 更改设置的便利。
{
    "leetcode.output": {
        "default": {
            "path": "...",
            "filename": "..."
        },
        "java": {
            "path": "...",
            "filename": "..."
        },
        "cpp": {
            "path": "...",
            "filename": "..."
        },
        "rust": {
            "path": "...",
            "filename": "..."
        }
    }
}

用户需要点击 Edit in settings.json

要么，放弃直接在 vscode settings UI 更改设置的便利。
{
    "leetcode.output": {
        "default": {
            "path": "...",
            "filename": "..."
        },
        "java": {
            "path": "...",
            "filename": "..."
        },
        "cpp": {
            "path": "...",
            "filename": "..."
        },
        "rust": {
            "path": "...",
            "filename": "..."
        }
    }
}

用户需要点击 Edit in settings.json

This might be something worth considering. But by calling leetcode.output. I'm worrying that some users might get confused with leetcode.workspaceFolder.

This might be something worth considering. But by calling leetcode.output. I'm worrying that some users might get confused with leetcode.workspaceFolder.

名字不是重要的，最后由你来拍版决定。现在需要确定这种设置的结构能否接受。

Hmm, please give me some time to think about it. @Dec-F Do you have any feedback?

Hmm, please give me some time to think about it. @Dec-F Do you have any feedback?

guaranteed atomic configuration is a very important factor. The template variables contain language information. You can customize the complete path and cover all languages. I don't think there is any problem.

@Dec-F Thanks.
@magic-akari Could you list all the variables that you want to support first?

请看这里
https://github.com/jdneo/vscode-leetcode/pull/380/files#diff-4f863ca9b6092167a0e40548789c2f3aR209-R230
我有个疑问。关于 tag 和 company.
现在的逻辑是，在多个值中选一个，然后生成文件。这里存在问题。
如果用户点击 code now 后，选择不同的值，就会新生成一个文件，而不是打开之前创建的。
我对这种行为感到困惑。

这确实是一个问题。关于多tag的情况你有什么好的建议吗？

我很需要这个功能。 我需要把文件名的改为 1-two-sum.cpp 。 因为如果是1.two-sum.cpp的话，编译输出的可执行文件是1.two-sum，tow-sum 会被当成后缀名，执行失败。

关于 tag 和 company 这种多 tag 的情况，我并没有什么好的想法或者解决方案。
因为我的需求里并不包含这两个。
如果能够了解需要这两个 tag 的人，他们的需求是什么，问题也许就能明晰一些了。

Has anyone started to implement this function?

Hi @Dec-F,
I was busy with other stuff these days. Sorry for that I did not respond to this issue in time.
Actually @magic-akari has provided a PR #380 for this, which I haven't been able to take a look so far. Maybe you can take a look and provide your thoughts?

Resolved in #380 by @magic-akari .
Nice job!


Unable to join a multiplayer server - Internal Server Error

Heya.  Awesome mod.  Unfortunately I think something broke.  Short version, I'm currently testing a 1.16.4 fabric modpack on a dedicated server without about 5 players currently, containing dankstorage-1.9a-1.16.3.jar, and things were working fine.  Then suddenly out of nowhere, I am no longer able to join the server.  My client tells me Internal Server Error, and my client console shows absolutely nothing.  So I intarweb searched the issue, and the most common reported thing is messed up player data.  So I asked the server host to send me the server's playerdata folder, replicated the server locally, then added the playerdata.  Sure enough, even in a fresh world, I get the same Internal Server Error.  Only, now that I have access to the server, I'm getting this in the server console -
[00:25:15] [Server thread/WARN]: Failed to handle packet for /<IP_ADDRESS>:3248
net.minecraft.class_148: Loading entity NBT
	at Not Enough Crashes deobfuscated stack trace.(1.16.4+build.7) ~[?:?]
	at net.minecraft.entity.Entity.fromTag(Entity:1663) ~[?:?]
	at net.minecraft.world.WorldSaveHandler.loadPlayerData(WorldSaveHandler:57) ~[?:?]
	at net.minecraft.server.PlayerManager.loadPlayerData(PlayerManager:344) ~[?:?]
	at net.minecraft.server.PlayerManager.onPlayerConnect(PlayerManager:138) ~[?:?]
	at net.minecraft.server.network.ServerLoginNetworkHandler.acceptPlayer(ServerLoginNetworkHandler:113) ~[?:?]
	at net.minecraft.server.network.ServerLoginNetworkHandler.redirect$bpk000$handlePlayerJoin(ServerLoginNetworkHandler:562) ~[?:?]
	at net.minecraft.server.network.ServerLoginNetworkHandler.tick(ServerLoginNetworkHandler:65) ~[?:?]
	at net.minecraft.network.ClientConnection.tick(ClientConnection:228) ~[?:?]
	at net.minecraft.server.ServerNetworkIo.tick(ServerNetworkIo:173) ~[?:?]
	at net.minecraft.server.MinecraftServer.tickWorlds(MinecraftServer:882) ~[intermediary-server.jar:?]
	at net.minecraft.server.dedicated.MinecraftDedicatedServer.tickWorlds(MinecraftDedicatedServer:312) ~[?:?]
	at net.minecraft.server.MinecraftServer.tick(MinecraftServer:811) ~[intermediary-server.jar:?]
	at net.minecraft.server.MinecraftServer.runServer(MinecraftServer:670) ~[intermediary-server.jar:?]
	at net.minecraft.server.MinecraftServer.method_29739(MinecraftServer:257) ~[intermediary-server.jar:?]
	at java.lang.Thread.run(Unknown Source) [?:1.8.0_271]
Caused by: java.lang.NullPointerException
	at Not Enough Crashes deobfuscated stack trace.(1.16.4+build.7) ~[?:?]
	at net.minecraft.item.ItemStack.handler$zmo000$areTagsEqualReturn1(ItemStack:6655) ~[?:?]
	at net.minecraft.item.ItemStack.areTagsEqual(ItemStack:383) ~[?:?]
	at tfar.dankstorage.event.MixinHooks.areItemStacksCompatible(MixinHooks.java:227) ~[dankstorage-1.9a-1.16.3.jar:?]
	at tfar.dankstorage.event.MixinHooks.doesItemStackExist(MixinHooks.java:232) ~[dankstorage-1.9a-1.16.3.jar:?]
	at tfar.dankstorage.event.MixinHooks.voidPickup(MixinHooks.java:150) ~[dankstorage-1.9a-1.16.3.jar:?]
	at tfar.dankstorage.event.MixinHooks.onItemPickup(MixinHooks.java:132) ~[dankstorage-1.9a-1.16.3.jar:?]
	at tfar.dankstorage.event.MixinHooks.interceptItem(MixinHooks.java:47) ~[dankstorage-1.9a-1.16.3.jar:?]
	at net.minecraft.entity.player.PlayerInventory.handler$bdi000$interceptItems(PlayerInventory:2690) ~[?:?]
	at net.minecraft.entity.player.PlayerInventory.insertStack(PlayerInventory) ~[?:?]
	at net.minecraft.entity.player.PlayerEntity.giveItemStack(PlayerEntity:1907) ~[?:?]
	at dev.emi.trinkets.api.PlayerTrinketComponent.fromTag(PlayerTrinketComponent.java:42) ~[trinkets-2.6.7.jar:?]
	at nerdhub.cardinal.components.api.util.container.AbstractComponentContainer.fromTag(AbstractComponentContainer.java:180) ~[f37fc192-c514-48a1-9d3c-a4d8ea68d0c7.jar:?]
	at net.minecraft.entity.Entity.handler$bdp000$fromTag(Entity:8692) ~[?:?]
	at net.minecraft.entity.Entity.fromTag(Entity:1654) ~[?:?]
	... 14 more

I'm not entirely sure, but it appears to be dank storage related.  There's no crash, so no crash log.  Deleting the playerdata with my UUID allows me to join.  I haven't tried to edit the NBT to try and narrow down if it's a specific item in one of my danks, although I'm thinking about it.  If you need any extra information, please let me know.  I'd give you a mod list, but the pack is up to about 140 mods and I'm not sure how to easily export a list of what mods I'm using without a crashlog.

Alright, so after a bunch of manual testing.. including manually removing dank storage items from the character one at a time..  I fixed the problem.  I don't think this is actually dank storage related.  One of the most recent updates to the modpack, removed the mod Artifacts.  Apparently when Artifacts is installed, it enables some Trinket slots and defaults to putting air blocks in them.  Then when you remove Artifacts and nothing is enabling those slots, having an air block inside the non-activated slot, blows up everything.  Using an nbt editor, I removed the entire trinkets tag, and everything seems to be working again (so far).  Sorry for the confusion.  I assume this is probably an issue I should report to the Trinkets people.

Alright, so after a bunch of manual testing.. including manually removing dank storage items from the character one at a time..  I fixed the problem.  I don't think this is actually dank storage related.  One of the most recent updates to the modpack, removed the mod Artifacts.  Apparently when Artifacts is installed, it enables some Trinket slots and defaults to putting air blocks in them.  Then when you remove Artifacts and nothing is enabling those slots, having an air block inside the non-activated slot, blows up everything.  Using an nbt editor, I removed the entire trinkets tag, and everything seems to be working again (so far).  Sorry for the confusion.  I assume this is probably an issue I should report to the Trinkets people.


refactor: simplify mapContributors configuration

Close #209

I've made a few changes, which are described in the commit message body and in the example in test. Please see https://github.com/nolebase/integrations/pull/211 .


Fees error message to improve for self payment

If a user is paying themselves, then the fees calculation output should not show a generic error message but rather show that payment can't be made

This issue has been moved to here https://github.com/GaloyMoney/galoy/issues/2432


"Show Inline Error Tooltips" toggle off behavior

A really minor bug: toggling the "Show Inline Error Tooltips" option off while an inline error tooltip is currently being displayed causes the tooltip to remain there permanently (or, at least until the option is toggled back on and the cursor is moved).
Repro steps:

Have "Show Inline Error Tooltips" option turned on.
Have two panes open, one open to a file of code and the other with settings open.
Create code that will be caught by a linter.
Put the cursor at the beginning of the offending line, causing a tooltip to appear.
Switch panes and go to the settings for the Linter package.
Toggle the "Show Inline Error Tooltips" option to off.
Go back to panel with code, tooltip remains open, moving cursor around does not remove it.

In the code panel, click to a line to remove the tooltip.

Code is now free of tooltips, as expected.

See the attached picture. Notice (a) the tooltip is displayed, (b) the "Show Inline Error Tooltips" is unchecked, and (c) the cursor location is several lines below the line with the tooltip.

Windows 10
Atom 1.4
Linter 1.11.3

I think this has been fixed in the UI rewrite

Confirmed, this has been fixed in the new UI


Horizontal Plotting of Traces

@DaloroAT when we do VSP time picking we prefer to have the traces plotted horizontally than vertically for accuracy in time picking purpose. Is it possible you can deploy it Visual Settings.

Just rotate on 90 degrees? Or is the reversion of trace order needed?
As I remember, time increases from the left side of the display to the right. Trace number increases from the top of the display to the bottom?

@DaloroAT Just rotate 90 deg.
You remember the other details correctly.

@DaloroAT Any chance you can priories this? I also could not find your whatsapp number. Can you please share it with me.
Regards

@HHTC0 I can, but I'm able to do it only on the weekends.

I also could not find your whatsapp number. Can you please share it with me.

I don't want to share my own phone number in public 😁 please, write to my e-mail placed on the main page of my profile. I will share it with you privately.

@DaloroAT I have sent you connection request on LinkedIn.

Hi @DaloroAT, are you planning to work on this sometime soon?

Hey @HHTC0
I'm on vacation until Friday, but I almost finished your request. I should complete it on the weekend.
This PR will contain bandfiltering as well and spectrum checking by windows.
I wanted to suggest you to try it with python, but I forgot to push on of the necessary file 😁 so I will ping you on the weekend.

Hi @DaloroAT thanks. Please enjoy your vacation and have good rest.
Having BPF and spectrum would be great addition. :)


add shabari to Contributors list

Hi this is my first commit on github, Thanks for the walk through tutorial. hopefully this is starting of many commits to come

Hi  @shabari004, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


How to obtain the listing name including external lib/function name as in the Listing windows

Assuming I have the such code in my binary:
      005c9cfc ff  15  54  f2  b7  00                   CALL      dword ptr [-> MSVCP140.DLL::std::basic_ios<char,struct_std::

but when I print this instruction from python script using such code
currentProgram.getListing().getInstructionAt(toAddr(0x5c9cfc))

what I'm getting is "only" CALL dword ptr [0x00b7f254].
Is there a way to obtain the same listing in script so that it include external's lib/function name?

Check out #1994 from yesterday.

OK, I guess it based on the same question from SO: https://reverseengineering.stackexchange.com/questions/25322/extracting-info-from-ghidra-listing-window


Feat: Enhancement in the Contact page .

Describe the feature
I think if the Contact section had border-bottom , placeholder in every text item so it would have better UI and also it is not aligned properly. I would also like to make it responsive.
Please Assign me this issue with the level and label of GSSOC'24
Add ScreenShots
Before:

After:

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

@agrwal697 Issue assigned to you


Adding playbooks to provision on Azure

These playbooks are an adaptation of the original AWS provisioner to run against Azure.  Note that these provision your network, subnet, resource group, etc which is different from the AWS playbooks that expect an existing VPC.

I thought I had mentioned this here. The refactor of the provisioner #141 needs to happen before we can add an other cloud environments. I'm all for adding more, but not under these circumstances.
Closing this PR. We can revisit this topic once #141 gets done.


Promote library stacktraces to ERROR channel

Use case: When developing a library, it is a bit unhelpful to receive errors like e.g. KeyError: xyz and nothing more.
While there are diagnostics that live on the DEBUG level, I believe that it's not their rightful place to be there.
Considering also better UX, I would add the stacktrace directly on the error itself like so:
KeyError: 'xyz'

Traceback (most recent call last):
[...]

I am not sure if there is some other reason that this happens like that; I'd like to at least debate the possibility to, at least conditionally, raise their level.
I have already read #1916, https://robotframework.org/robotframework/2.1.2/RobotFrameworkUserGuide.html#reporting-keyword-status ; I am thinking that, for CI/CD pipelines (where people attempt to be always-green), it will not change much; on failing executions, the error can be passed directly to the developer.

Tracebacks aren't shown by default because they are confusing for non-programmers and provide no valuable information when a keyword fails unless there's a bug in the library itself. It's already easy to use --loglevel debug if you want to see tracebacks and that's generally a good idea during library development. Is this not enough?

[1] I already use --loglevel trace:info just in case there is something I need.

It just feels bad that something so important (an uncaught exception), would be dug deep in DEBUG level (which, by definition, can be too chatty).
It can be also disappointing that you might need to rerun a test (potentially more than once), to capture your original error; apart from the time spent in doing that.

The above (and maybe a couple of more arguments) lead to behavior [1]; which I am not sure if it's the best course of action for everyday testing.

I used this way to decorate my function, but not convenient as debug (old version of RIDE needs --loglevel debug pattern to work)
import traceback
def trier(func):
    def infunc(*args, **kwargs)
        try:
            return func(*args, **kwargs)
        except:
            print(traceback.format_exc())
            raise

and then
@trier
def yourkeyword(...):
    ...

As I already wrote, tracebacks aren't shown by default because they are confusing for non-programmers and also because the provide no valuable information when the library works correctly. I know they are valuable when the library has a bug, but the problem is there's no way to reliably differentiate exceptions that are caused by bugs from exceptions that are explicitly raised by a keyword to signal a failure. And even if there was, showing bug tracebacks for non-technical users wouldn't be too helpful.
We are not going to change the behavior and tracebacks will be logged on debug level also in the future. If you want to make them more visible, you can easily create a listener that logs an error whenever there's a failure.


licensing

Also, as per licensing, botml v1 is open source (MIT) so usable as-is in most situations, whereas botml v2 is copyrighted, with the software free to use in non-commercial projets; any commercial use of this version 2 needs a license, as per https://github.com/BotML/botml-js/blob/2.0-dev/LICENSE.txt

can you explain more your thoughts on this?

Sorry for the verrryy late reply; the licensing information is all available to you in the License file.


Updating $header-font-family global variable has no effect

RE: src/assets/scss/_settings.scss section 10. Typography
Updating $header-font-family has no effect. The $global-font-family variable is used instead.
I forced changes using the app.scss file. Just thought the feedback might be helpful
My system details
OS: Windows 7 Pro
using Windows Shell and Foundation CLI

You guys are doing a stellar job of streaming the creation of email templates. Thank you!
Cheers!
Jules

Please ignore this ticket. I thought I was on foundation-emails


add multiturn-conversation dataset process

add multiturn-conversation dataset process
Fixes # not add multiturn-conversation dataset process

Gracias….. EXCELENTE TRABAJO 👏🏻
Karliz garcia
El El dom, 26 de mayo de 2024 a la(s) 21:04, Facebook Community Bot <
@.***> escribió:

Thank you for signing our Contributor License Agreement. We can now accept
your code for this (and any) Meta Open Source project. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/meta-llama/llama-recipes/pull/542#issuecomment-2132559959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFDKPYMN5SFUJN2J4JUEZSDZEKPENAVCNFSM6AAAAABIKKSNGCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZSGU2TSOJVHE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Karliz garcia
El El dom, 26 de mayo de 2024 a la(s) 23:49, Karla García <
@.***> escribió:

Gracias….. EXCELENTE TRABAJO 👏🏻
Karliz garcia
El El dom, 26 de mayo de 2024 a la(s) 21:04, Facebook Community Bot <
@.***> escribió:

Thank you for signing our Contributor License Agreement. We can now
accept your code for this (and any) Meta Open Source project. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/meta-llama/llama-recipes/pull/542#issuecomment-2132559959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BFDKPYMN5SFUJN2J4JUEZSDZEKPENAVCNFSM6AAAAABIKKSNGCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZSGU2TSOJVHE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


H3-java newInstance() couldn't recover when the temporary dynamic library file is deleted

When newing an H3Core instance, the function copies the dynamic library from resources to a temporary file in order to load the libraries and reuses the original temporary file for newing instances later.
In the event that the temporary file is deleted, creating new instances always crashes with no way to recover.
I suggest we add a file existence check to the existing file path and recreate if needed.

I think adding a file existence check makes sense. There will still be a race condition where the file is determined to exist and then is deleted (unless catching the file not found exception)

This is fixed in the v3.6.3 release.


Maven plugin is not using maven repositories

Describe the bug

contractsRepositoryUrl: URL to a repository with the artifacts that have contracts. If it is not provided, use the current Maven ones.

I've tried adding repositories to the maven project and to settings but none of those work.
Instead I see
[DEBUG] Using the following remote repos [remote0 (jar:file:/home/jakub/.m2/repository/com/fasterxml/jackson/core/jackson-databind/2.15.4/jackson-databind-2.15.4.jar!/META-INF/versions/9/, default, releases+snapshots)]

which makes no sense.
The downloading stubs fails with:
[DEBUG] Could not obtain transporter factory for remote0 (jar:file:/home/jakub/.m2/repository/com/fasterxml/jackson/core/jackson-databind/2.15.4/jackson-databind-2.15.4.jar!/META-INF/versions/9/, default, releases+snapshots)
org.eclipse.aether.transfer.NoTransporterException: Unsupported transport protocol jar

It works if I set the contractsRepositoryUrl (and credentials) manually in the plugin.
But I'd rather not repeat the repository configuration there and use the built in maven mechanism for resolving dependencies.
It's all rather confusing. I'm building with Maven, why do I have to setup this artifact resolution manually?

I'm not using contractMode=LOCAL and just add the contracts jar to module dependencies.
Pros:

don't have to duplicate the credentials setup
Cons:
the support for + version is broken, I have to specify the version explicitly in contractDependency

Even better I add the contracts jar to plugin dependencies.
It's not polluting the project dependencies, and it makes sense - it's the plugin that uses the artifact.
Interestingly you could use LOCAL or CLASSPATH mode with this approach to the same effect.
Using + still fails though
java.lang.IllegalArgumentException: For groupId [com.example.contracts] artifactId [example] and classifier [stubs] the version was not resolved! The following exceptions took place [org.eclipse.aether.transfer.MetadataNotFoundException: Could not find metadata com.example.contracts:example/maven-metadata.xml in local (/home/jakub/.m2/repository), org.eclipse.aether.transfer.MetadataNotFoundException: Cannot access remote0 (jar:file:/home/jakub/.m2/repository/com/fasterxml/jackson/core/jackson-databind/2.15.4/jackson-databind-2.15.4.jar!/META-INF/versions/9/) in offline mode and the metadata com.example.contracts:example/maven-metadata.xml has not been downloaded from it before]


Add support for IQ quantizarions

This change allows importing IQ type gguf quantization with ollama create.
This change carries the commit from #3657 while moving its changes around to the refactored project structure.
❯ ./ollama create nous-hermes-2-mistral:IQ_4XS -f /Users/bruce/models/nous-hermes-2-mistral/Modelfile
transferring model data
using existing layer sha256:737258efad6ba5cf7232de66715a26cadba67b0e4bdace5cf03cf49d1e4864a0
creating new layer sha256:d7285065edcb87b4852f1144dd090812df1b00ade49f74e234066ea9407a14bc
creating new layer sha256:d8ba2f9a17b3bbdeb5690efaa409b3fcb0b56296a777c7a69c78aa33bbddf182
creating new layer sha256:b2c4ee0a7317771fcbe7413c369d72ea911c63e6f52b2b0d6298a5a14c8e4983
writing manifest
success

❯ ./ollama run nous-hermes-2-mistral:IQ_4XS
>>> write some python

def print_fruits(fruits):
    for fruit in fruits:
        print(fruit)

Tested with:
IQ1_S
IQ1_M
IQ2_M
IQ3_XXS
IQ3_XS
IQ3_S
IQ4_NL
IQ4_XS
resolves #3622

What's happening with this one?

@BruceMacD would you please be able to update (rebase) your branch from main?

@sammcj sorry for the delay!

Nice work, this is awesome!

Awesome, great job! Just wanted to comment that IQ3_M seems to be absent from the list (referring to "filetype.go"), which means IQ3_M would not work?

I'm curious about which version of ollama will supported this feature.

@wwjCMP there hasn't been a final release in two weeks, but v0.1.39 which is currently marked as pre-release has it - https://github.com/ollama/ollama/releases

@wwjCMP there hasn't been a final release in two weeks, but v0.1.39 which is currently marked as pre-release has it - https://github.com/ollama/ollama/releases

thanks


Enable a tracer defined in the servlet context to override a tracer a…

…ssociated with the TraceFilter

It introduces yet another way how the tracer can be provided which can cause some confusion.
Why is not GlobalTracer sufficient?

If the OT dependencies are installed in the servlet container, then the GlobalTracer is global to the container - so only one tracer - which also means all webapps reporting spans for the same service name.
This PR enables the tracer to be managed with the servlet context lifecycle.

Could you please add this to readme and javadoc? Explaining when it should be used.

@pavolloffay done

@pavolloffay @jpkrohling thanks.


Question: Self-hosted Analytics Integration

All the analytics services available are either hosted by 3rd party organization.
Are there any options for using self-hosted analytics platform to be used with the organization.

There is a self-hosted version of Botmetrics you can install:
https://github.com/howdyai/botmetrics

Thanks. I was about to install that. Thanks for quick response.

I installed botmetrics in my own server and found that it's not showing web as a platform. But getbotmetrics.com web shows an option for web chat bots.
Is the repo not updated? Please let me know how to use analytics for web bots.

Check this out:
https://bot-metrics.com/docs#other
If you want a more recently updated stats solution, I would checkout dashbot
https://www.dashbot.io/docs/generic/

The self hosted version is largely the same, with the change of pointing the API host at your local instance:
https://github.com/botmetrics/botmetrics.js#setting-your-api-host-for-self-hosting

There is a major difference in self hosted version. General web bot option
is not available on GitHub repo.
On Thu 21 Jun, 2018, 7:07 PM Peter Swimm<EMAIL_ADDRESS>wrote:

The self hosted version is largely the same, with the change of pointing
the API host at your local instance:
https://github.com/botmetrics/botmetrics.js#setting-your-api-host-for-self-hosting
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/howdyai/botkit/issues/1368#issuecomment-399106138,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBx53BxaGT7MSp5_M_jhAMqtdvMZjM0ks5t-6GLgaJpZM4UW_6N
.

You will have to do some work on your own granted. Right now the hosted solutions are more robust and less work for you to do to support.

Agreed on this point. But due to corporate restrictions I can't use hosted
service.
Please consider this as a closed.
On Fri 22 Jun, 2018, 10:22 PM Peter Swimm<EMAIL_ADDRESS>wrote:

You will have to do some work on your own granted. Right now the hosted
solutions are more robust and less work for you to do to support.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/howdyai/botkit/issues/1368#issuecomment-399509139,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABBx5_b4yiguj_NhK5O8Epd3jr2lX1zdks5t_SDNgaJpZM4UW_6N
.


Make other screen tabs scrollable

Issue

close #36

Overview (Required)

Change from TabRow to ScrollableTabRow in OtherScreen

Links

https://github.com/DroidKaigi/conference-app-2021/pull/148

For FeedScreen

Screenshot

Before
After

I was thinking about raising an issue. Thank you for doing it first! 😄


Cannot install powershell modules on CentOS 7

I'm new to powershell. I've been told I need to install MSOnline to allow powershell to authenticate to Azure/AD.
I have powershell-6.0.0_beta.9-1.rhel.7.x86_64 on CentOS 7.4.1708
Running as root:
PS /root> Import-Module MSOnline
Import-Module : The specified module 'MSOnline' was not loaded because no valid module file was found
PS /root> Install-Module MSOnline -Force                                                                                Installing package 'MSOnline'
Get-Module lists nothing new, no files have appeared in /opt/microsoft/powershell/6.0.0-beta.9/Modules/ or in /root/.local/share/powershell/Modules, and Import-Module still fails
PS /root> Save-Module -Name MSOnline -Path /root/tmp
WARNING: Administrator rights are required to install or update.
PackageManagement\Save-Package : Unable to save the module 'MSOnline'.
A directory /root/tmp/MSOnline/<IP_ADDRESS>/ is created, but it's empty.
a) is there a way to make install work ?
b) why is it saying I need administrator rights if I'm running as root ?
c) is there a way to download modules manually and copy them into the .local directory ?

Hi @adaviel!
The MSOnline is the older module for managing Azure AD. The newer module is AzureAD. Neither module is supported on Linux platforms. They are currently only supported by Windows PowerShell 5.1 on Windows Platforms.
In the future, those modules may be made available in PowerShell Core and on Linux, but for the time being they are not supported.

So you are saying that Azure would not work even if I could install it.
Does anything actually work ?
E.g. if I try to install ACMESharp or Pscx it behaves in the same way - appears to download, but cannot be imported.

Correct. if you could download and install them they would still not function on Linux. I believe the AzureAD Module doesn't work on PowerShell Core on windows either due to it's reliance on ADAL which requires WinForms.
There are quite  a few modules that will work with Core and on Linux. You could try some from the PowerShellGallery just to eliminate if it is something related to installations.

As I say, I can't even seem to install them. I just tried psCheckPoint from your link, with the same results. Earlier, somehow in my playing around I did get MSOnline downloaded into the local modules directory - not sure how - but it did not show in Get-Module until I renamed it from MSOnline.<IP_ADDRESS>/ to just MSOnline/. Same with a utility "PsGet" I found online - the instructions said that PsGet.psm1 would work if placed in the Modules directory, but it had to be in a PsGet subdirectory.

if you do
Install-Module -Name psCheckPoint -Scope CurrentUser
Import-Module psCheckPoint
Get-Module psCheckPoint

What do you get?

PS /root> Install-Module -Name psCheckPoint -Scope CurrentUser
Untrusted repository
You are installing the modules from an untrusted repository
... .. Installing package 'psCheckPoint'  Downloaded 0.00MB out of 0.63MB   Installing package 'psCheckPoint'   Downloaded 0.31MB out of 0.63MB
[ooooooooo  ]  Installing package 'psCheckPoint' .. . Unzipping  [ooooo  ]
(the last is quickly overwritten by spaces; I ran it inside "screen" to try to capture it)
S /root> Import-Module psCheckPoint                                                                                Import-Module : The specified module 'psCheckPoint' was not loaded because no valid module file was found in any module
PS /root> Get-Module psCheckPoint
PS /root>
there is nothing created in .local/share/powershell/Modules/
If I run strace pwsh, when I run Install-Module there's a line
open("/opt/microsoft/powershell/6.0.0-beta.9/System.Management.Automation.resources.dll", O_RDONLY) = -1 ENOENT
but I see no writes to any file

Please open an issue here: https://github.com/powershell/powershellget


Bad state: no tests to run

Screenshot

Preconditions

Only one test in integration_test folder

//import 'package:flutter/cupertino.dart';
import 'package:flutter_test/flutter_test.dart';
import 'package:haven_bridge/main_common.dart';
import 'package:easy_localization/easy_localization.dart';
//import 'package:integration_test/integration_test.dart';
import 'package:patrol/patrol.dart';
import 'config.dart';

//import 'package:haven_bridge/main_qa.dart' as app;

void main() {
  patrolTest('sign in', config: patrolConfig, nativeAutomation: true,
      ($) async {
    //app.main();
    await EasyLocalization.ensureInitialized();
    await $.pumpWidgetAndSettle(const MyApp());

    await $.native.openApp();

    //await $.pumpWidgetAndSettle(const MyApp());

    //expect($('Saznajte više'), findsOneWidget);
    await $('Prijava').tap();

    await $(#emailTextField).enterText('luka.mlikota.zapovjednik@biss.hr');
    await $(#passwordTextField).enterText('password');
    await $(#loginButton).tap();

    expect($('Pregled intervencija'), findsOneWidget);
    await $.tap($('Sljedeće'));

    expect($('Odabir adrese'), findsOneWidget);
    await $.tap($('Sljedeće'));

    expect($('Voditelj intervencije'), findsOneWidget);
    await $.tap($('Sljedeće'));

    expect($('Uklanjanje vozila'), findsOneWidget);
    await $.tap($('Sljedeće'));

    expect($('Osvrt'), findsOneWidget);
    await $.tap($('Gotovo'));
    await $.pump(const Duration(seconds: 4));
    await $.native.grantPermissionWhenInUse();
  });
}

Latest version of patrol_cli (0.6.14)
Connected emulator or real device, in my case I'm using VOG L29 (mobile)  • DUM0220811001124 • android-arm64  • Android 10 (API 29)

PS C:\Users\lmlikota\haven-bridge-mobile> flutter devices
4 connected devices:

VOG L29 (mobile)  • DUM0220811001124 • android-arm64  • Android 10 (API 29)
Windows (desktop) • windows          • windows-x64    • Microsoft Windows [Version 10.0.22000.978]
Chrome (web)      • chrome           • web-javascript • Google Chrome 106.0.5249.103
Edge (web)        • edge             • web-javascript • Microsoft Edge 106.0.1370.37

Steps to reproduce

Try to run all tests using patrol drive --flavor development

Expected results

Test have been executed

Actual result

Test didn't even run. Patrol CLI can't find test?

PS C:\Users\lmlikota\haven-bridge-mobile> patrol drive --flavor development
No device specified, using the first one (DUM0220811001124)
✓ Forwarded ports (88ms)
✓ Installed server (1.7s)
✓ Installed instrumentation (1.6s)
Bad state: no tests to run
#0      TestRunner.run (package:patrol_cli/src/features/drive/test_runner.dart:43:7)
#1      DriveCommand.execute (package:patrol_cli/src/features/drive/drive_command.dart:250:23)
<asynchronous suspension>
#2      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#3      PatrolCommandRunner.runCommand (package:patrol_cli/src/command_runner.dart:210:18)
<asynchronous suspension>
#4      PatrolCommandRunner.run (package:patrol_cli/src/command_runner.dart:161:18)
<asynchronous suspension>
#5      patrolCommandRunner (package:patrol_cli/src/command_runner.dart:42:14)
<asynchronous suspension>
#6      main (file:///C:/Users/lmlikota/AppData/Local/Pub/Cache/hosted/pub.dartlang.org/patrol_cli-0.6.14/bin/main.dart:6:20)
<asynchronous suspension>

I've also tried:
PS C:\Users\lmlikota\haven-bridge-mobile>
PS C:\Users\lmlikota\haven-bridge-mobile> patrol drive --flavor development -t patrollogin.dart
Error: target file patrollogin.dart has invalid suffix
PS C:\Users\lmlikota\haven-bridge-mobile> patrol drive --flavor development -t login
Error: target file login has invalid suffix
PS C:\Users\lmlikota\haven-bridge-mobile> patrol drive --flavor development --target login
Error: target file login has invalid suffix
PS C:\Users\lmlikota\haven-bridge-mobile> patrol drive --flavor development --target ./integration_test/login.dart
Error: target file ./integration_test/login.dart has invalid suffix
PS C:\Users\lmlikota\haven-bridge-mobile> patrol drive --flavor development --target /integration_test/login.dart
Error: target file /integration_test/login.dart has invalid suffix
PS C:\Users\lmlikota\haven-bridge-mobile> patrol drive --flavor development --target ./login.dart
Error: target file ./login.dart has invalid suffix
PS C:\Users\lmlikota\haven-bridge-mobile> patrol drive --flavor development --target /integration_test/login.dart
Error: target file /integration_test/login.dart has invalid suffix

Hi @lmlikota, thanks for this (very detailed!) bug report.
Test files must end with _test.dart for Patrol to find them. The same applies to Flutter's own tools, e.g flutter test.

You can always expected detailed bug reports from me even though sometimes 🐞 are just in my head.
I'm going to severely facapalm myself now 😄 I wasn't aware of that fact, it would be nice to state it in docs. Can I help you with docs somehow?

Happy to help :) We should explain this better.
Thanks for offering help with the docs, but actually I'm adding it to docs right now (and fixing the docs site, which is down).

PR is up: https://github.com/leancodepl/patrol/pull/435


Call to action for Prep course

Currently the prep course isn't properly communicated on our surfaces, and we don't have a clear call to action.
I'm contacting a number of people, and need:

Updating the website to reflect the Python prep course
Clarify the call to action that we should be pressing people to do

Link added above the fold.


Cleanup: Unused files removed

I discovered some old files, which doesn't seemed to be used anywhere and are safe* to delete.
Goal: Just cleaning up the repo...

The solution does build and I looked through the different Powershell/CMD-Scripts and didn't found any hint that these files are used.

We have a few other services that make use of the HashCodeCombner - can you revert that one from the PR?

Is this file included as a link in other projects? Didn't found any reference to it.

We have a NuGet on an internal build server that includes it

Sneaky ;) I will take a look at it tonight.

done :)

Was committed to dev.


fix: check for sdk updates to update autoload

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#261  👈
#260
#259
#258
#257
#256
#255
#254
#253
#252
#251
#250
#249
#248
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @NathanFlurry and the rest of your teammates on  Graphite