Use JLine to allow Ublu a full-featured command line

The setup could use a little more love (to enable things like proper completion based on commands, functions, and existing tuples, as well as command history that writes to a file), but for now, this enables Ublu to work with a full-featured command line, with emacs keybindings and line history.

This doesn't require native code to be built, does it?
On Thu, Dec 15, 2016 at 4:24 PM, Taylor C. Richberger <
<EMAIL_ADDRESS>wrote:

The setup could use a little more love (to enable things like proper
completion based on commands, functions, and existing tuples, as well as
command history that writes to a file), but for now, this enables Ublu to
work with a full-featured command line, with emacs keybindings and line
history.
You can view, comment on, or merge this pull request online at:
https://github.com/jwoehr/ublu/pull/5
Commit Summary

dpoint needs to allow a crit and warn alert comparator to work
without value
add jline jar
trial emplacement of jline
set up jline in interpreter
clean up Interpreter file, and cull warnings

File Changes

A JLINE_LICENSE
https://github.com/jwoehr/ublu/pull/5/files#diff-0 (34)
A lib/jline-3.1.2.jar
https://github.com/jwoehr/ublu/pull/5/files#diff-1 (0)
M nbproject/project.properties
https://github.com/jwoehr/ublu/pull/5/files#diff-2 (2)
M src/ublu/Ublu.java
https://github.com/jwoehr/ublu/pull/5/files#diff-3 (3)
M src/ublu/util/Interpreter.java
https://github.com/jwoehr/ublu/pull/5/files#diff-4 (30)
M src/ublu/util/SysShepHelper.java
https://github.com/jwoehr/ublu/pull/5/files#diff-5 (10)

Patch Links:

https://github.com/jwoehr/ublu/pull/5.patch
https://github.com/jwoehr/ublu/pull/5.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jwoehr/ublu/pull/5, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEZAhFUu4bFbG4SJZ4AYi4_etujWHPGdks5rIcxCgaJpZM4LOr7A
.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

Should not, but now that I think of it, I don't think it will work for anything other than Windows and Unix (or at least something that allows POSIX tty conventions).  It works through a DLL for Windows (which is packaged in and extracted on use automatically) and through tty for Linux/Unix/POSIX.
If the Ublu interpreter needs to run anywhere else, this pull request should probably be denied.

Yeah it has to work in QSH on IBM i and in USS on z/OS.
Can we make it a props or command line option?
On Thu, Dec 15, 2016 at 11:39 PM, Taylor C. Richberger <
<EMAIL_ADDRESS>wrote:

Should not, but now that I think of it, I don't think it will work for
anything other than Windows and Unix (or at least something that allows
POSIX tty conventions). It works through a DLL for Windows (which is
packaged in and extracted on use automatically) and through tty for
Linux/Unix/POSIX.
If the Ublu interpreter needs to run anywhere else, this pull request
should probably be denied.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jwoehr/ublu/pull/5#issuecomment-267528153, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEZAhKEA7THolRONF8S0D88wxcwE8fnBks5rIjI3gaJpZM4LOr7A
.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

Y'know, really, so far, all the external libs Ublu uses are there to
provide functionality. I'd really like to avoid adding libs just to enhance
the language.
For instance: You can look at ArgArray.assimilateQuotation() or whatever I
called it and ArgArray.assimilateBlock() and make an
ArgArray.assimilateCString() rather than drag Antlr into things. May be
more work but it will get you into the internals of the Ublu interpreter.
On Thu, Dec 15, 2016 at 11:41 PM, Jack Woehr <
<EMAIL_ADDRESS>wrote:

Yeah it has to work in QSH on IBM i and in USS on z/OS.
Can we make it a props or command line option?
On Thu, Dec 15, 2016 at 11:39 PM, Taylor C. Richberger <
<EMAIL_ADDRESS>wrote:

Should not, but now that I think of it, I don't think it will work for
anything other than Windows and Unix (or at least something that allows
POSIX tty conventions). It works through a DLL for Windows (which is
packaged in and extracted on use automatically) and through tty for
Linux/Unix/POSIX.
If the Ublu interpreter needs to run anywhere else, this pull request
should probably be denied.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jwoehr/ublu/pull/5#issuecomment-267528153, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEZAhKEA7THolRONF8S0D88wxcwE8fnBks5rIjI3gaJpZM4LOr7A
.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

Oh and the point you raised earlier about inline comments. I do have an
inline comment mechanism \ ${ this is a comment }$
On Thu, Dec 15, 2016 at 11:49 PM, Jack Woehr <
<EMAIL_ADDRESS>wrote:

Y'know, really, so far, all the external libs Ublu uses are there to
provide functionality. I'd really like to avoid adding libs just to enhance
the language.
For instance: You can look at ArgArray.assimilateQuotation() or whatever I
called it and ArgArray.assimilateBlock() and make an
ArgArray.assimilateCString() rather than drag Antlr into things. May be
more work but it will get you into the internals of the Ublu interpreter.
On Thu, Dec 15, 2016 at 11:41 PM, Jack Woehr <jwoehr@absolute-performance.
com> wrote:

Yeah it has to work in QSH on IBM i and in USS on z/OS.
Can we make it a props or command line option?
On Thu, Dec 15, 2016 at 11:39 PM, Taylor C. Richberger <
<EMAIL_ADDRESS>wrote:

Should not, but now that I think of it, I don't think it will work for
anything other than Windows and Unix (or at least something that allows
POSIX tty conventions). It works through a DLL for Windows (which is
packaged in and extracted on use automatically) and through tty for
Linux/Unix/POSIX.
If the Ublu interpreter needs to run anywhere else, this pull request
should probably be denied.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jwoehr/ublu/pull/5#issuecomment-267528153, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEZAhKEA7THolRONF8S0D88wxcwE8fnBks5rIjI3gaJpZM4LOr7A
.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

That should work; just leaving the members in but never initializing them otherwise.  We could also have the property default be to use it for Linux, Windows, AIX, or BSD, and not otherwise (eg. ublu.jline defaults to 1 for systems we know will work and 0 otherwise, changed on demand at start-up or an environment variable, so it can be set in the user profile).
As far as avoiding unnecessary extra libs, I can sympathize with that.  Given that this one is just quality-of-life, it could be easily left out.

Well, I don't want to neglect quality of life.  If you can make the command line work better
just under Unix systems that's cool. The truth is Windows command shell has
its own line repeat and history mechanism, don't need a library for it.
ublu works better under COMMAND.EXE than in an XTerm right now!!
On Thu, Dec 15, 2016 at 11:51 PM, Taylor C. Richberger <
<EMAIL_ADDRESS>wrote:

That should work; just leaving the members in but never initializing them
otherwise. We could also have the property default be to use it for Linux,
Windows, AIX, or BSD, and not otherwise (eg. ublu.jline defaults to 1 for
systems we know will work and 0 otherwise, changed on demand at start-up or
an environment variable, so it can be set in the user profile).
As far as avoiding unnecessary extra libs, I can sympathize with that.
Given that this one is just quality-of-life, it could be easily left out.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jwoehr/ublu/pull/5#issuecomment-267529490, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEZAhC9qeL9nFwPy2npt-z7Rsa2P-lnlks5rIjT2gaJpZM4LOr7A
.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

The other thing is, I'm retro. I don't want 100 open source libraries as
part of Ublu. I can't decide whether to trust them. Basically, I trust
JTOpen, Postgresql and TN5250J. I may drag in a couple of other IBM jars,
one for the z/VSE operating system and one for IBM MQ. But I'd like to
leave it at that. The rest I'd really like it if possible that we
hand-craft it, even if we're always a little cruder than the norm. I want
control more than elegance.
On Thu, Dec 15, 2016 at 11:53 PM, Jack Woehr <
<EMAIL_ADDRESS>wrote:

Well, I don't want to neglect quality of life.  If you can make the command line work better
just under Unix systems that's cool. The truth is Windows command shell has
its own line repeat and history mechanism, don't need a library for it.
ublu works better under COMMAND.EXE than in an XTerm right now!!
On Thu, Dec 15, 2016 at 11:51 PM, Taylor C. Richberger <
<EMAIL_ADDRESS>wrote:

That should work; just leaving the members in but never initializing them
otherwise. We could also have the property default be to use it for Linux,
Windows, AIX, or BSD, and not otherwise (eg. ublu.jline defaults to 1 for
systems we know will work and 0 otherwise, changed on demand at start-up or
an environment variable, so it can be set in the user profile).
As far as avoiding unnecessary extra libs, I can sympathize with that.
Given that this one is just quality-of-life, it could be easily left out.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jwoehr/ublu/pull/5#issuecomment-267529490, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEZAhC9qeL9nFwPy2npt-z7Rsa2P-lnlks5rIjT2gaJpZM4LOr7A
.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

On the other hand, I want you and other users to be happy. So if I have
to compromise, so be it.
On Thu, Dec 15, 2016 at 11:56 PM, Jack Woehr <
<EMAIL_ADDRESS>wrote:

The other thing is, I'm retro. I don't want 100 open source libraries as
part of Ublu. I can't decide whether to trust them. Basically, I trust
JTOpen, Postgresql and TN5250J. I may drag in a couple of other IBM jars,
one for the z/VSE operating system and one for IBM MQ. But I'd like to
leave it at that. The rest I'd really like it if possible that we
hand-craft it, even if we're always a little cruder than the norm. I want
control more than elegance.
On Thu, Dec 15, 2016 at 11:53 PM, Jack Woehr <jwoehr@absolute-performance.
com> wrote:

Well, I don't want to neglect quality of life.  If you can make the command line work better
just under Unix systems that's cool. The truth is Windows command shell has
its own line repeat and history mechanism, don't need a library for it.
ublu works better under COMMAND.EXE than in an XTerm right now!!
On Thu, Dec 15, 2016 at 11:51 PM, Taylor C. Richberger <
<EMAIL_ADDRESS>wrote:

That should work; just leaving the members in but never initializing
them otherwise. We could also have the property default be to use it for
Linux, Windows, AIX, or BSD, and not otherwise (eg. ublu.jline defaults to
1 for systems we know will work and 0 otherwise, changed on demand at
start-up or an environment variable, so it can be set in the user profile).
As far as avoiding unnecessary extra libs, I can sympathize with that.
Given that this one is just quality-of-life, it could be easily left out.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jwoehr/ublu/pull/5#issuecomment-267529490, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEZAhC9qeL9nFwPy2npt-z7Rsa2P-lnlks5rIjT2gaJpZM4LOr7A
.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

Taking this in to people's data centers, we really have to be sure we
aren't trojaned.
On Thu, Dec 15, 2016 at 11:57 PM, Jack Woehr <
<EMAIL_ADDRESS>wrote:

On the other hand, I want you and other users to be happy. So if I have
to compromise, so be it.
On Thu, Dec 15, 2016 at 11:56 PM, Jack Woehr <jwoehr@absolute-performance.
com> wrote:

The other thing is, I'm retro. I don't want 100 open source libraries as
part of Ublu. I can't decide whether to trust them. Basically, I trust
JTOpen, Postgresql and TN5250J. I may drag in a couple of other IBM jars,
one for the z/VSE operating system and one for IBM MQ. But I'd like to
leave it at that. The rest I'd really like it if possible that we
hand-craft it, even if we're always a little cruder than the norm. I want
control more than elegance.
On Thu, Dec 15, 2016 at 11:53 PM, Jack Woehr <
<EMAIL_ADDRESS>wrote:

Well, I don't want to neglect quality of life.  If you can make the command line work better
just under Unix systems that's cool. The truth is Windows command shell has
its own line repeat and history mechanism, don't need a library for it.
ublu works better under COMMAND.EXE than in an XTerm right now!!
On Thu, Dec 15, 2016 at 11:51 PM, Taylor C. Richberger <
<EMAIL_ADDRESS>wrote:

That should work; just leaving the members in but never initializing
them otherwise. We could also have the property default be to use it for
Linux, Windows, AIX, or BSD, and not otherwise (eg. ublu.jline defaults to
1 for systems we know will work and 0 otherwise, changed on demand at
start-up or an environment variable, so it can be set in the user profile).
As far as avoiding unnecessary extra libs, I can sympathize with that.
Given that this one is just quality-of-life, it could be easily left out.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jwoehr/ublu/pull/5#issuecomment-267529490, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEZAhC9qeL9nFwPy2npt-z7Rsa2P-lnlks5rIjT2gaJpZM4LOr7A
.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

Anyway, please fix calljava if you are inspired to do so and can do so
without breaking existing code. Try to avoid a performance hit. I want to
use calljava more and more to create extensions rather than expand the base
system.
On Thu, Dec 15, 2016 at 11:58 PM, Jack Woehr <
<EMAIL_ADDRESS>wrote:

Taking this in to people's data centers, we really have to be sure we
aren't trojaned.
On Thu, Dec 15, 2016 at 11:57 PM, Jack Woehr <jwoehr@absolute-performance.
com> wrote:

On the other hand, I want you and other users to be happy. So if I
have to compromise, so be it.
On Thu, Dec 15, 2016 at 11:56 PM, Jack Woehr <
<EMAIL_ADDRESS>wrote:

The other thing is, I'm retro. I don't want 100 open source libraries as
part of Ublu. I can't decide whether to trust them. Basically, I trust
JTOpen, Postgresql and TN5250J. I may drag in a couple of other IBM jars,
one for the z/VSE operating system and one for IBM MQ. But I'd like to
leave it at that. The rest I'd really like it if possible that we
hand-craft it, even if we're always a little cruder than the norm. I want
control more than elegance.
On Thu, Dec 15, 2016 at 11:53 PM, Jack Woehr <
<EMAIL_ADDRESS>wrote:

Well, I don't want to neglect quality of life.  If you can make the command line work
better just under Unix systems that's cool. The truth is Windows command
shell has its own line repeat and history mechanism, don't need a library
for it. ublu works better under COMMAND.EXE than in an XTerm right now!!
On Thu, Dec 15, 2016 at 11:51 PM, Taylor C. Richberger <
<EMAIL_ADDRESS>wrote:

That should work; just leaving the members in but never initializing
them otherwise. We could also have the property default be to use it for
Linux, Windows, AIX, or BSD, and not otherwise (eg. ublu.jline defaults to
1 for systems we know will work and 0 otherwise, changed on demand at
start-up or an environment variable, so it can be set in the user profile).
As far as avoiding unnecessary extra libs, I can sympathize with that.
Given that this one is just quality-of-life, it could be easily left out.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jwoehr/ublu/pull/5#issuecomment-267529490, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEZAhC9qeL9nFwPy2npt-z7Rsa2P-lnlks5rIjT2gaJpZM4LOr7A
.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

--
Absolute Performance, Inc.
12303 Airport Way, Suite 100
Broomfield, CO 80021
NON-DISCLOSURE NOTICE:  This communication including any and all
attachments is for the intended recipient(s) only and may contain
confidential and privileged information.  If you are not the intended
recipient of this communication, any disclosure, copying further
distribution or use of this communication is prohibited.  If you received
this communication in error, please contact the sender and delete/destroy
all copies of this communication immediately.

I'm going to try to use Go to build a terminal-like interface to Ublu to make it easier to use interpretively.


LIIKUNTA-210 | Fix broken filter keywords

The issues were caused by behaviour that differs from the default behaviour of NextJS and were brought about with the change where we begun to use our custom stringification routine by default. Unfortunately it didn't have support for query as a way of providing search parameters, nor did it retain the current pathname if a new one was not provided. The stringification routine Next uses is relatively complex and this regression highlights the difficulty of maintaining logic that attempts to mimic that. We should look to replace the current escape with a better approach, but I didn't spend time on that now.

Codecov Report

Merging #87 (3be584f) into main (6a52c81) will increase coverage by 0.25%.
The diff coverage is 92.59%.

@@            Coverage Diff             @@
##             main      #87      +/-   ##
==========================================
+ Coverage   71.64%   71.90%   +0.25%
==========================================
  Files          76       76
  Lines        1192     1210      +18
  Branches      308      313       +5
==========================================
+ Hits          854      870      +16
- Misses        337      339       +2
  Partials        1        1

Impacted Files
Coverage Δ

src/domain/i18n/router/utils.ts
82.22% <90.47%> (+5.29%)
:arrow_up:

src/domain/i18n/router/Link.tsx
93.33% <100.00%> (-0.42%)
:arrow_down:

Continue to review full report at Codecov.

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


Add tree L systems

[ ] Add stack of saved states
[ ] Add way for bracket notation to be interpreted

Added stack, now can generate rudimentary tree-like L-systems. However, the structure is not open - I think some lines are being drawn that shouldn't be. Need to look into this

Fixed branch joining bug, branching tree structures now generate correctly


Bump tested AGP versions to incldue 7.0.1 and 7.1.0-alpha10.

Build failed https://gradle.com/s/42f73msi2kgoo


Add LibreOffice

Add libreoffice
Project URL
https://www.libreoffice.org/
Category
Productivity
Description
LibreOffice is a powerful office suite – its clean interface and feature-rich tools help you unleash your creativity and enhance your productivity.
LibreOffice includes several applications that make it the most powerful Free and Open Source office suite on the market.
Checklist

[ ] Edit applications.json instead of README.md.
[ ] Only one project/change is in this pull request
[ ] Screenshots(s) added if any
[ ] Has a commit from less than 2 years ago
[ ] Has a clear README in English

@shlomoklahr Could you review my comments?

Sorry! Will fix


Insert-Update Entity with Computed column in sql

When inserting or updating  an entity with computed column in sql, I get the error:

Column 'inserted.Status' cannot be referenced in the OUTPUT clause because the column definition contains a subquery or references a function that performs user or system data access. A function is assumed by default to perform data access if it is not schemabound. Consider removing the subquery or function from the column definition or removing the column from the OUTPUT clause

Computed columns should not be included in insert or update statements.

Are you using RC1? Probably already fixed in https://github.com/aspnet/EntityFramework/issues/1441

I am using RC1 and I have set MaxBatchSize to 1.

This is fixed in RC2, which will ship shortly.

I am having the same issue in rc2.
in my model i have registered the property like this
[DatabaseGenerated(DatabaseGeneratedOption.Computed)]
public SubjectStatus? SubjectStatus { get; set; }

Insert statement is running fine.
But on update i have this issue because my computed column uses a Scalar-valued Function.
the sql statement that is created on update is the following.
UPDATE [Subjects] SET [CouncilId] = @p1, ... [SubjectGroupId] = @p14
OUTPUT INSERTED.[SubjectStatus]
INTO @inserted0
WHERE [Id] = @p0;
SELECT [SubjectStatus] FROM @inserted0;

I tested it a few minutes ago and the problem isn't solved for update statements

@xrkolovos @ampalla can you open a new issue, the specific scenario covered by this issue is fixed.


Error when trying to make inference with a local model

System Info
text-generation-inference
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
I have a local llama-2-7b-chat model trained with autotrain. My new model is stored in /data/my-llama folder
autotrain llm --train --project_name my-llama --model meta-llama/Llama-2-7b-hf --data_path . --use_peft --use_int4 --learning_rate 2e-4 --train_batch_size 2 --num_train_epochs 3 --trainer sft
Auto-train finished ok
I deploy that model with text_generation-inference
docker run --shm-size 1g -p 8080:80 -v $PWD/data:/data ghcr.io/huggingface/text-generation-inference:1.0.1 --model-id /data/ --disable-custom-kernels
but I have this error
ERROR download: text_generation_launcher: Download encountered an error: /opt/conda/lib/python3.9/site-packages/bitsandbytes/cextension.py:34: UserWarning: The installed version of bitsandbytes was compiled without GPU support. 8-bit optimizers, 8-bit multiplication, and GPU quantization are unavailable.
warn("The installed version of bitsandbytes was compiled without GPU support. "
Traceback (most recent call last):
File "/opt/conda/bin/text-generation-server", line 8, in
sys.exit(app())
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/cli.py", line 147, in download_weights
local_pt_files = utils.weight_files(model_id, revision, ".bin")
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/utils/hub.py", line 90, in weight_files
raise FileNotFoundError(
FileNotFoundError: No local weights found in /data/ with extension .bin
Expected behavior
No error

It does work.
I can't debug your environement, but something is wrong between where the local model lives, and the -v $PWD/data:/data argument.

Yes, all right. But when I launch
docker run --shm-size 1g -p 8080:80 -v $PWD/data:/data ghcr.io/huggingface/text-generation-inference:1.0.1 --model-id /data/myllama/ --disable-custom-kernels
I have this error
/opt/conda/lib/python3.9/site-packages/bitsandbytes/cextension.py:34: UserWarning: The installed version of bitsandbytes was compiled without GPU support. 8-bit optimizers, 8-bit multiplication, and GPU quantization are unavailable.
warn("The installed version of bitsandbytes was compiled without GPU support. "
Traceback (most recent call last):
File "/opt/conda/bin/text-generation-server", line 8, in
sys.exit(app())
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/cli.py", line 81, in serve
server.serve(
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/server.py", line 184, in serve
asyncio.run(
File "/opt/conda/lib/python3.9/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/opt/conda/lib/python3.9/asyncio/base_events.py", line 647, in run_until_complete
return future.result()
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/server.py", line 136, in serve_inner
model = get_model(
File "/opt/conda/lib/python3.9/site-packages/text_generation_server/models/init.py", line 117, in get_model
config_dict, _ = PretrainedConfig.get_config_dict(
File "/opt/conda/lib/python3.9/site-packages/transformers/configuration_utils.py", line 617, in get_config_dict
config_dict, kwargs = cls._get_config_dict(pretrained_model_name_or_path, **kwargs)
File "/opt/conda/lib/python3.9/site-packages/transformers/configuration_utils.py", line 672, in _get_config_dict
resolved_config_file = cached_file(
File "/opt/conda/lib/python3.9/site-packages/transformers/utils/hub.py", line 388, in cached_file
raise EnvironmentError(
OSError: /data/myllama/ does not appear to have a file named config.json. Checkout 'https://huggingface.co//data/myllama//None' for available files.
rank=0
2023-08-22T19:14:35.330308Z ERROR text_generation_launcher: Shard 0 failed to start
2023-08-22T19:14:35.330333Z  INFO text_generation_launcher: Shutting down shards
Error: ShardCannotStart
The output model trained with autotrain contains these files:
user_dev@myllm:~/data/myllama$ ls
README.md            adapter_model.bin          checkpoint-6  special_tokens_map.json  tokenizer.model        training_args.bin
adapter_config.json  adapter_model.safetensors  runs          tokenizer.json           tokenizer_config.json  training_params.json
There isn't any config.json file

As you can see, there is no config.json.

Yes, tgi will not work cross repo, you need both the load and the original model is the same location, or have a pre-fused model.

@Narsil but if adapter_config.json contains base_model_name_or_path a model name (e.g. meta-ai/llama-2-hf), according to https://github.com/huggingface/text-generation-inference/pull/762 it should be downloading this model as the next step?

on-demand/ cloud burst

Not really imo, and it certainly wasn't designed with that in mind.
It's designed for running large LLM, on long running servers, and be the most efficient possible on those.
All the rest is just bonus.
Supporting LORA as first class citizen is something we have in mind : https://github.com/huggingface/text-generation-inference/issues/907#issuecomment-1689454243
However, supporting non merged models atm is a lot of engineering work not worth the effort in our opinions. (Supporting MANY loras on the same deployment is much more attractive, for maybe a little bit more engineering).
Cost wise it would be much more beneficial that the extra storage cost we're talking about here.
@chris-aeviator I'm sorry if you feel angry or let down, but you're more than welcome to fork and add support if we cannot provide. This is what open source is for !

add 30-100 $ for every copy per month,

Really ? Something seems to be wrong there. EFS or S3 shouldn't cost nearly as much.

Re: s3
very conservative 2 TB egress, loads my model 12 times per month to a server location of my choice.

Don't want to mess around here or get emotional though.
What's holding adapters back? For single GPU deployment with B&B 4 bit I can (POC) successfully load a model https://github.com/ohmytofu-ai/tgi-angryface/blob/8028ebb93c58fe58673c80a16c482b3e1cad6807/server/text_generation_server/models/causal_lm.py#L489 and the implementation took very little time (might be something wrong with it still, I didn't test thoroughly yet).

But... you shouldn't be moving around paying egress costs (unless you're running on a different cloud). So there's no bandwidth cost if you're using S3 from the same region in AWS.
EFS should work (not cheap either) so you can keep models there all the time.
For a little more engineering work (not that much, but more maintenance) you can get NFS to work like EFS and that would be quite cheap.
Currently what happens is just that the model is loading the base, and adding the adapter on top, saving that back onto the docker, and loading from it.
The issue with the code you linked is that is works just for CausalLM models, not FlashCausalLM ones nor Seq2SeqLM.
And this is where the engineering debt starts, because you also don't have sharing nor GPTQ/BITSANDBYTES support.
The code you're calling is the "graceful degradation" path, not the main one.
With the current method, everything comes for free (you just have to have a bit of disk on the host node to save the fused model).
I just realized there was indeed an issue when deploying from docker with the current code though: https://github.com/huggingface/text-generation-inference/pull/935. It didn't trigger locally because when creating the code it created the fused version, and things worked afterwards, whereas in docker everything is created at every boot (still cached everything from the hub in the /data folder).
So there's a little bit of boot latency, but zero engineering maintenance cost basically (and you get all new features pretty much for free without us having to think about peft).
As I said, lora ARE interesting, but we're going to try and exploit the lora to the fullest when we do, which is actually running multiple lora with the same backbone on a single server. Engineering/maintenance cost will be high, but so will the benefit. Ideally you could keep a single server running, and serve pretty much all loras on it.
It's still an idea for now, but we have brainstormed it a bit, it's just a matter of making it prio #1.
I'm not sure everyone realizes but we're still a pretty small team (TGI is like 1 person basically). Things to do are plenty, we're permanently thinking about what's the biggest impact feature, we just cannot deliver everything.
Happy, to continue the discussion about your use case though. We both agree  30-100 $ is not a real good cost for maintaining loras.

Appreciate your time taken drafting a technical solution. However the TLDR is 99% of people cannot do what you propose. There is no one cloud provider as of August 2023 that will provide you with more than a handful of GPUs.

Lambda - no GPUs avail
Runpod - some GPUs
AWS - a single A10G with the default quota. Each request for quota increase, even if declined, costs around 13€ in "business support fees"

That's why more dynamic deployment schemes are essential for "the GPU poor" right now.


Update the page to use the Microsoft.Graph powershell cmdlets?

Since the AzureAD Powershell modules have been on the deprecation path for a long time, shouldn't this page be updated to use the Microsoft.Graph powershell cmdlets?

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

ID: a9b38e3c-c054-794a-70af-ce9d2ec70507
Version Independent ID: 04653c31-670f-ebd4-da07-072ea5503483
Content: Revoke user access in an emergency in Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/enterprise-users/users-revoke-access.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @barclayn
Microsoft Alias: barclayn

@gabriel-robert-fx
Thanks for your feedback! We will investigate and update as appropriate.

Hi @gabriel-robert-fx
Related to Microsoft graph cmdlets, the articles are in the process of being updating. The content owners (@barclayn) are working on it. It will go live soon in future. Other than that, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Models - Additions/Edits to Models

User/Post/Comment/Message - index incoming.

problem with merge


Changed 'js' to '.js' in resolve.extensions (ui.frontend -> webpack.commons.js)

Description
Node modules were not being properly resolved by Webpack because the file extension was 'js' instead of '.js' on line 15 of webpack.commons
Related Issue
https://github.com/adobe/aem-project-archetype/issues/226
Motivation and Context
Allows importing of entire modules vs. only importing their inner /dist js files
How Has This Been Tested?
Built webpack with '.js' instead of 'js'
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@austen-wade Thank you for providing this fix!


MRO computation in RustPython doesn't seem to implement C3 correctly

Feature
I was experimenting with RustPython's implementation of multiple inheritance and it doesn't seem to implement the C3 method resolution order algorithm correctly.
Example 1:
class A:
    pass

class B(A):
    pass

class C(A, B):
    pass

This should give an error, but doesn't in RustPython.
Example 2:
class A:
    def f(self):
        print("A")

class B(A):
    def f(self):
        super().f()
        print("B")

class C(A):
    def f(self):
        super().f()
        print("C")

class D(B, C):
    def f(self):
        super().f()
        print("D")

D().f()

This should print "A, C, B, D" but prints instead "A, B, D", missing C completely in the mro.
Python Documentation
There seems to be no great description of the C3 algorithm as part of the docs, but it's described as part of the 2.3 release notes here: https://www.python.org/download/releases/2.3/mro/

This is now fixed and can be closed.


[WIP] Multi-Lingual Tokenization

@vsabolcec @beme248 please let me know when you are ready for me to review

@guipenedo the PR is ready for review

LGTM! Is there any need for the cache ?

@guipenedo just explained. Approved


Cannot upgrade from v1.12.10393.0 to v1.13.11431.0 with chocolately due to Microsoft.VCLibs.140.00.UWPDesktop

Windows Terminal version
1.12.10393.0
Windows build number
10.0.19044.0
Other Software
N/A
Steps to reproduce
choco upgrade -y microsoft-windows-terminal
Expected Behavior
Upgrade suceeds.
Actual Behavior
>  choco upgrade -y vcredist140
Chocolatey v1.1.0
Upgrading the following packages:
vcredist140
By upgrading, you accept licenses for the packages.
vcredist140 v14.32.31326 is the latest version available based on your source(s).

Chocolatey upgraded 0/1 packages.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
> choco upgrade -y microsoft-windows-terminal
Chocolatey v1.1.0
Upgrading the following packages:
microsoft-windows-terminal
By upgrading, you accept licenses for the packages.

You have microsoft-windows-terminal v1.12.10393.0 installed. Version 1.13.11431.0 is available based on your source(s).
Progress: Downloading microsoft-windows-terminal 1.13.11431.0... 100%

microsoft-windows-terminal v1.13.11431.0 [Approved]
microsoft-windows-terminal package files upgrade completed. Performing other installation steps.
Progress: 0% - Processing                                                                                ERROR: The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: Deployment failed with HRESULT: 0x80073CF3, Fehler bei Updates des Pakets (Abhängigkeits- oder Konfliktüberprüfung).

Das Paket Microsoft.WindowsTerminal_1.13.11431.0_x64__8wekyb3d8bbwe kann von Windows nicht installiert werden, da es von einem Framework abhängig ist, das nicht gefunden wurde. Stellen Sie das von CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US veröffentlichte Framework Microsoft.VCLibs.140.00.UWPDesktop mit einer neutralen oder x64-Prozessorarchitektur und Mindestversion 14.0.30035.0 zusammen mit dem zu installierenden Paket bereit. Die derzeit installierten Frameworks mit
Das Paket Microsoft.WindowsTerminal_1.13.11431.0_x64__8wekyb3d8bbwe kann von Windows nicht installiert werden, da es von einem Framework abhängig ist, das nicht gefunden wurde. Stellen Sie das von CN=Microsoft Corporation, O=Microsoft Corporation, L=Redmond, S=Washington, C=US veröffentlichte Framework Microsoft.VCLibs.140.00.UWPDesktop mit einer neutralen oder x64-Prozessorarchitektur und Mindestversion 14.0.30035.0 zusammen mit dem zu installierenden Paket bereit. Die derzeit installierten Frameworks mit dem Namen Microsoft.VCLibs.140.00.UWPDesktop sind: {Microsoft.VCLibs.140.00.UWPDesktop_14.0.27629.0_x64__8wekyb3d8bbwe}

NOTE: For additional information, look for [ActivityId] 59f0e28b-853c-0008-5a62-f4593c85d801 in the Event Log or use the command line Get-AppPackageLog -ActivityID 59f0e28b-853c-0008-5a62-f4593c85d801

The upgrade of microsoft-windows-terminal was NOT successful.
Error while running 'C:\ProgramData\chocolatey\lib\microsoft-windows-terminal\tools\chocolateyInstall.ps1'.
 See log for details.

Chocolatey upgraded 0/1 packages. 1 packages failed.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).

Failures
 - microsoft-windows-terminal (exited -1) - Error while running 'C:\ProgramData\chocolatey\lib\microsoft-windows-terminal\tools\chocolateyInstall.ps1'.
 See log for details.

This looks very similar to https://github.com/microsoft/terminal/issues/3097, however this seems solved long ago and all dependencies should be shipped with the app.

Thanks for the report! We don't actually maintain the choco package for the Terminal ourselves (as discussed in #2044). As noted in the README:

If you have any issues when installing/upgrading the package please go to the Windows Terminal package page and follow the Chocolatey triage process

I suspect whoever does maintain the choco package needs to use the Windows 10 versions of the Terminal package, instead of the Windows 11 versions. Sorry about that!

Okay, reported downstream to the maintainer(s) at https://github.com/mkevenaar/chocolatey-packages/issues/136

Thanks for the report! We don't actually maintain the choco package for the Terminal ourselves (as discussed in #2044). As noted in the README:

If you have any issues when installing/upgrading the package please go to the Windows Terminal package page and follow the Chocolatey triage process

I suspect whoever does maintain the choco package needs to use the Windows 10 versions of the Terminal package, instead of the Windows 11 versions. Sorry about that!

I checked and its using this file Microsoft.WindowsTerminalPreview_Win10_1.14.1451.0_8wekyb3d8bbwe.msixbundl

This script here finally made it work: https://github.com/microsoft/terminal/issues/3097#issuecomment-1345242698


Joystick trigger buttons

We need to add LT/RT to both joystick implementations, since these are used for both Live Mana and changing Time of Day. iirc at least with XINPUT these are handled differently than the other buttons, since they are analogue.

[x] Add LT/RT to common interface
[x] Add LT/RT to linux implementation
[x] Add LT/RT to windows implementation

Technically we already have LT/RT in the common interface. If we're happy with just doing ON/OFF for them, we can tick off the first box. Not sure SoS even supports anything for the analogue input anyways.

Implemented for windows in #43

Also implemented for linux in #43
This should be fully implemented


enh: standard space querying

while progressing in https://github.com/poldracklab/fmriprep/pull/1955, we find ourselves extending the queries that i believe the get_ helper methods were trying to solve. This PR improves the capabilities of Spaces querying for standard spaces.
I've only updated one get method for feedback, and if this looks alright, we can extend this to the others.

Is this necessary before an fMRIPrep release?

this aims to simplify filtering of spaces in the above PR, so ideally before we release 20.0.0

let's get this as is - we can touch up a few things in a new PR


Configurations/10-main.conf: correct AIX targets.

I'm adding WIP, because it's just one way to solve the problem, and it's open question if it's more suitable. See #5659 for background info. On additional not, manual says that ranlib needs -X option too. I don't seem to have the problem, but it might be problematic on older systems...

I think this solution has merits.  The other would be to change that definition of ARFLAGS in the Makefile template to also include $target{arflags}, but that seems a bit over the top for the moment.  So I would say that you can take this out of WIP.

WIP is removed, I've verified this with aix-gcc and aix64-gcc...

On related side note, I feel that I'd prefer to harmonize usage of some of the parameters. As discussed in referred issue, there are arflags and ARFLAGS. Then there are ar and AR, ranlib and RANLIB, maybe something else. One case should be sufficient in all these cases.
BTW, on side note to side note. My understanding is that one can omit CC => sub { env('CC') } and ranlib => sub { env('RANLIB') } from uClinux targets. Because that's how it works behind curtains now anyway...

On related side note, I feel that I'd prefer to harmonize usage of some of the parameters. As discussed in referred issue, there are arflags and ARFLAGS. Then there are ar and AR, ranlib and RANLIB, maybe something else. One case should be sufficient in all these cases.

I agree, with an exception...  for flags, we keep the user overridable ones in upper case attributes and the non-overridable ones in lower case attributes.  How that matters for flags to ar is up to us to decide.

BTW, on side note to side note. My understanding is that one can omit CC => sub { env('CC') } and ranlib => sub { env('RANLIB') } from uClinux targets. Because that's how it works behind curtains now anyway...

100% correct.

we keep the user overridable ones in upper case attributes and the non-overridable ones in lower case attributes.

It's totally appropriate for cflags and alike, but I suggest that it's an overkill for arflags and alike. In other words I'm talking about harmonization specifically for flags/attributes like arflags, not for all flags.
On second thought one might find having for example ranlib vs. RANLIB usable. (Once again, for example). But then it should be possible to actually "wire" ranlib directly into rule. In mean if RANLIB is defined, then rule would invoke $(RANLIB), but if ranlib is defined, then rule should invoke specifically $target{ranlib}, i.e. without referencing the make variable. So I suppose one can also put it as "harmonization to upper case" vs. "hardwiring lower-case into rules".


Update Quansight Labs link

👋 there was a typo, and after fixing the typo, a redirect to https://labs.quansight.org/.

Thanks a lot!!!!


Autofix is not working in multiline cases

Hi,
Exactly the rule I was looking for  👍  and was about to write myself (but struggled quite a bit with it)
I found an issue tho with the auto-fixing when having multiline statements (using the implicit option)
      <button
        [disabled]="
          !(myCondition.fooBarBaz.zapZapZap && myCondition.fooBarBaz.zipZapZup) &&
          this.implicitVar === 'something'
        "
      ></button>

This won't get fixed correctly and also the wrong property is underlined in my code editor: implicitVar instead of this

Correct. I didn't complete the plugin yet. It's kind of blocking on angular-eslint/angular-eslint#384

It looks like I found an solution in fb7c29c. Will release a new version soon.

Lot's of issues where fixed in version 0.1.0, but still some left, based on my own findings:

					<tr
						*ngIf="
							currentMutation.address.opmerking !== null &&
							currentMutation.address.opmerking.length > 0
						">

Potential issue: multi-line *ngIf.

@Component({
	template: `
		<form clrStepper [formGroup]="form">
			<clr-stepper-panel formGroupName="group">
				<button clrStepButton="next" navigateToNextStep="true">Next</button>
			</clr-stepper-panel>
		</form>
	`
})
class TestComponent {
	@ViewChild(ClrStepButton, { static: false }) public button: ClrStepButton;

	public form = new FormGroup({ group: new FormGroup({}) });
}

Which resolves to incorrect HTML with this. before the double quote:

More use-cases have been patched in version 0.2.0.
Aldo, all tests are green, after running ng lint --fix in my project, not all cases are fixed correctly (specifically see comment above).
Fixing those edge-cases manually, they don't report.
Side-note: I also noticed differences in lint fixing when running ng lint --fix and saving file with settings.json:
	"editor.formatOnPaste": true,
	"editor.formatOnSave": true,
	"editor.formatOnType": true,
	"editor.codeActionsOnSave": {
		"source.fixAll.eslint": true // Fix all fixable ESLint rules.
	},

I'll do some more tests, but for now, I'm marking this issue as fixed.


Icon lost/bill error (community modpack)

Describe the bug
This bug came up when i was copy and paste make X per Colonist and with only alpaca wool as the ingredent i was pasting the bill on all of my things until i got to make leather armor and pasted in the bill, all icons and the bug log went nuts icons missing in the schedule for copy and past as well as the bottom right corner icons are missing
To Reproduce in description
Steps to reproduce the behavior:

Go to 'tailor bench'
Click on 'some bills and add leather armor last'
Scroll down 'threw them making them a non leather and paste the bill on the leather armor from vanilla exp'
See error

Error Log
https://pastebin.com/Lsm1s4Di
https://pastebin.com/9r6J5zJ3
https://pastebin.com/KbcibP0X
Screenshots
If applicable, add screenshots to help explain your problem.

TY Niilo

TY Niilo

your error logs are not accessible, you may have them set to private or something.

I was unable to reproduce the issue with 2.2.19 preview


Mono_logs folder gobbles up disk space

Godot version:
3.1 Alpha Mono
OS/device including version:
KDE Neon
Issue description:
After some months of using Godot, I noticed I suddenly ran out of disk space. Using k4dirstat, I found the culprit: the folder ~/.local/share/godot/mono/mono_logs grew to a whopping 6.18 GB, filled with nothing but text files containing logs:

Steps to reproduce:
Not sure, it seems to be triggered by using Godot Mono for a while.

CC @neikeq I can't confirm or deny this since deleting ~/.local/share/godot/mono is part of my rebuild workflow, but Godot should definitely delete old logs.

Godot should delete files that are 5 or more days old. Are there log files older than 5 days?

@neikeq Not sure, I deleted the folder before I could check the date. I'll wait for the next 5 days and see if the logs are still there.

@neikeq Whoops, forgot about this, but yes, after waiting 5 days, the logs are indeed removed automatically. That reduces the urgency of the issue, but still, it would be nice if Godot actually ensures the logs don't go beyond a certain size, e.g. 100MB.

I wonder how did you manage to get it to gobble 6.18 GB of disk space. I run Godot continuously and mine does not reach a single MB. It would be good to have more details on this.

I used to have the same problem but not any more. So, possibly that 5-day auto cleaning feature might be a recent addition to the engine and the log files mentioned by the reporter could have been made with an older version of Godot?

That code was there way before the module was merged with the official Godot repo. Also I find it hard to believe it can get to 6 GB so quick just because older logs weren't deleted. It has to be something else.
If anyone ever manages to reproduce this again, it would be great to be able to have a look at the files.

I'm not sure if this helps, but I have a few logs that are several MB, the largest being 4.3 MB. It's almost entirely (99%) the following over and over again:

[0x7f9dabdfe700] worker unparking, timeout? no interrupted? no (in domain Mono, debug)
[0x7f9dabdfe700] worker parking (in domain Mono, debug)
[0x7f9dabbfd700] try unpark worker, success? yes (in domain Mono, debug)
[0x7f9dabbfd700] request worker, unparked (in domain Mono, debug)
pipe_read: reading up to 1024 bytes from pipe 25 (in domain Mono, debug)
pipe_read: read 1024 bytes from pipe 25 (in domain Mono, debug)
[0x7f9dabbfd700] try unpark worker (in domain Mono, debug)
[0x7f9dabbfd700] try unpark worker, success? yes (in domain Mono, debug)
[0x7f9dabbfd700] request worker, unparked (in domain Mono, debug)
[0x7f9db03fe700] worker unparking, timeout? no interrupted? no (in domain Mono, debug)
[0x7f9dabbfd700] try unpark worker (in domain Mono, debug)

Most of my sessions with Godot are fairly short though (10 minutes?), so if I was using Godot constantly throughout the day I suppose it could possibly become an issue.

Is this still reproducible in the current master branch or latest beta build?

I haven't been able to use Godot much over the last month, so my logs are all pretty small.
But, if the following was the reason why they were getting so large, it still has the same logs over and over as before. These statements seem to regularly accumulate as Godot is being used. I'm not sure how important they are or I might dig for and remove them myself.

worker unparking, timeout? no interrupted? no (in domain Mono, debug)
worker unparking, timeout? yes interrupted? no (in domain Mono, debug)
worker finishing (in domain Mono, debug)

worker unparking, timeout? no interrupted? no (in domain Mono, debug)
worker unparking, timeout? yes interrupted? no (in domain Mono, debug)
worker finishing (in domain Mono, debug)

Those logs are not from Godot, so they must be emitted by Mono itself.

I've been using C# Godot (3.1 stable) for past month and my mono_logs is just 10MB, with logs only from last 5 days or so. I also often see a message on startup that some logs are deleted, so I guess this might be fixed.

Sounds like if some logs grow to extreme sizes within minutes/hours, this issue would still be relevant, as the existing logic only purges logs older than 5 days without capping the max size of the logs kept.
It can happen that an error message gets spammed to logs dozens of times per seconds, thus growing log sizes to several GBs as seen by OP initially. And given that this can be due to Mono itself and not our code, we have to handle this IMO.

I'm changing the default log level for debug (and release debug) builds in #28139 from "debug" to "info". Let me know if that improves things.

It's probably better after #28139.
We still need to eventually add some logic to prevent logs from growing to unreasonable sizes if some errors are being spammed e.g. each frame. That applies to non-Mono logs too.

Sometimes a Mono log file is generated that takes up the entire disk space. I'm not sure when and why this happens.
I provide a part of the log after which endless spam errors begin.
Godot v3.2.3.stable.mono.official

File properties screenshot

Log chunk

...
Assembly Ref addref GodotTools[0000000062170010] -> System[0000000018A04BD0]: 4 (in domain Mono, info)
Loading reference 0 of D:\Program Files (x86)\Steam\steamapps\common\Godot Engine/GodotSharp/Tools/GodotTools.IdeMessaging.dll asmctx DEFAULT, looking for netstandard, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 (in domain Mono, info)
Assembly Ref addref GodotTools.IdeMessaging[0000000062170BE0] -> netstandard[0000000062170880]: 4 (in domain Mono, info)
Loading reference 1 of D:\Program Files (x86)\Steam\steamapps\common\Godot Engine/GodotSharp/Mono/lib/mono/4.5/Facades/netstandard.dll asmctx DEFAULT, looking for System.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089 (in domain Mono, info)
Image addref System.Core[00000000621784D0] (asmctx DEFAULT) -> D:\Program Files (x86)\Steam\steamapps\common\Godot Engine/GodotSharp/Mono/lib/mono/4.5/System.Core.dll[000000006228D0D0]: 2 (in domain Mono, info)
Prepared to set up assembly 'System.Core' (D:\Program Files (x86)\Steam\steamapps\common\Godot Engine/GodotSharp/Mono/lib/mono/4.5/System.Core.dll) (in domain Mono, info)
Assembly System.Core[00000000621784D0] added to domain GodotEngine.Domain.Scripts, ref_count=1 (in domain Mono, info)
Assembly Ref addref netstandard[0000000062170880] -> System.Core[00000000621784D0]: 2 (in domain Mono, info)
Loading reference 0 of D:\Program Files (x86)\Steam\steamapps\common\Godot Engine/GodotSharp/Mono/lib/mono/4.5/System.Core.dll asmctx DEFAULT, looking for mscorlib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089 (in domain Mono, info)
Assembly Ref addref System.Core[00000000621784D0] -> mscorlib[0000000018A05E60]: 8 (in domain Mono, info)
GC_MINOR: (Nursery full) time 5.74ms, stw 5.83ms promoted 204K major size: 1088K in use: 343K los size: 1024K in use: 80K (in domain Mono, info)
GC_MINOR: (Nursery full) time 0.27ms, stw 0.28ms promoted 0K major size: 1088K in use: 343K los size: 1024K in use: 80K (in domain Mono, info)
selector_thread_wakeup_drain_pipes: recv () failed, error (10054)
 (in domain , warning)
selector_thread_wakeup_drain_pipes: recv () failed, error (10054)
 (in domain , warning)
selector_thread_wakeup_drain_pipes: recv () failed, error (10054)
 (in domain , warning)
selector_thread_wakeup_drain_pipes: recv () failed, error (10054)
 (in domain , warning)
selector_thread_wakeup_drain_pipes: recv () failed, error (10054)
 (in domain , warning)
...

Should we disable Mono logs by default to work around this issue?

Should we disable Mono logs by default to work around this issue?

As @akien-mga suggested we should add logic to avoid the log files from growing to unreasonable sizes. Like discarding old data once we reach a certain threshold.

The same problem warning spam (selector_thread_wakeup_drain_pipes: recv () failed, error) seems to be a thing with Unity log files as well: https://issuetracker.unity3d.com/issues/editor-freezes-slash-crashes-when-connected-to-nordvpn?page=3#comments
@Morb0 are you using a VPN?

Unity seems to have fixed it with a patch on their Mono fork: https://github.com/Unity-Technologies/mono/commit/37dc7aae643fa1d86f183044379352eac074d96e

@neikeq, I have vpn installed but it is not active

I downloaded version 3.2.3 (mono, 64bit) a week ago. I did not start making games yet, but after keeping Godot open on my computer for like 5 days, it suddenly created a huge 40GB log file in AppData\Roaming\Godot\mono\mono_logs. My SSD was completely full and it even broke my desktop with graphical glitches everywhere 0_0. Unfortunatly I could not open the log to see what's inside, I immediately removed it to prevent further harm to my computer... I think I'm going to use standard version as I have no plan to use C# !

We no longer use the Mono_logs folder in 4.0, so I'll move this issue to the 3.x milestone.


Added Unity 2021.1.6f1 version 2021.1.7f1.

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Welp, seems like i broke your automation by adding the real PackageName D: I'll fix it when this gets submitted

@OfficialEsco sometimes I wish the code would do what we want instead of "EXACTLY" what we told it to do.

@OfficialEsco sometimes I wish the code would do what we want instead of "EXACTLY" what we told it to do.

Weeeeell, i think this is more about what @jedieaston told his script to do instead of the new things i want his script to do 🙄
Since a few programs have their PackageName like "AppName Version (Architecture)" you guys over at the WinGet team probably have a workaround for this anyways 😄

We are working through challenges with matching PackageName and what is registered in ARP. We support multiple architectures in the same manifest now. I'm hoping to see consolidation for PackageIdentifiers not to include architecture. We recently went through this exercise with Visual Studio Code. They are back down to one for stable and one for insider. Both architectures, and User vs. Machine are availabe via a single manifest.

We are working through challenges with matching PackageName and what is registered in ARP. We support multiple architectures in the same manifest now. I'm hoping to see consolidation for PackageIdentifiers not to include architecture. We recently went through this exercise with Visual Studio Code. They are back down to one for stable and one for insider. Both architectures, and User vs. Machine are availabe via a single manifest.

Its the ARP which includes Version and Architecture i mean btw, which is why i tested this manifest as PackageName: Unity 2021.1.6f1

VC++ require Architecture tho, but you have Firefox which has Firefox 88.0 (x64) en-US as ARP

With unity, we probably should stick with the package name excluding the version. We do some normalization work to try to ignore versions and architectures. With the VC++ libraries, a user might want multiple architectures installed on their machine, so those are likely instances where they are different packages that should include the architecture in the name.

Shoot, I didn't even see that. Sorry @OfficialEsco! I'm going to add an additional argument to take care of this, and I need to remember to look at the PR as well.


Feature req: Collapse to frame title

I'm writing a docs site in which having some code blocks that show an entire code file is a useful thing, but I don't want these to be shown in full by default. I can surround these with details/summary tags to make these collapsed, but I think it might be neat to have this built in to the frame plugin so you can just see the title of the code block when collapsed.
Is this something that would be easy to achieve?

Yes - that's perfect.
I think as a plugin it probably needs to render some text / an arrow inside the frame title, but that code will work fine for my our needs as-is. Thanks so much!

Agreed, the collapse UI can be improved for sure. The plugin was just meant as an example to show how you can implement a feature like this using the plugin API. Feel free to adapt it to your own needs.
I love to hear that it works well for you. Happy coding!


Backport "HBASE-28784 Exclude samples and release-documentation zip of jaxws-ri from output tarball" to branch-2.5

Backports https://github.com/apache/hbase/pull/6157

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 26s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

_ branch-2.5 Compile Tests _

+1 :green_heart:
mvninstall
4m 19s

branch-2.5 passed

+1 :green_heart:
compile
6m 28s

branch-2.5 passed

+1 :green_heart:
spotless
1m  7s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 48s

the patch passed

+1 :green_heart:
compile
5m 23s

the patch passed

+1 :green_heart:
javac
5m 23s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
xmllint
0m  1s

No new issues.

+1 :green_heart:
hadoopcheck
29m 17s

Patch does not cause any errors with Hadoop 2.10.2 or 3.2.4 3.3.6 3.4.0.

+1 :green_heart:
spotless
0m 52s

patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m 15s

The patch does not generate ASF License warnings.

54m 27s

Subsystem
Report/Notes

Docker
ClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6161/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6161

Optional Tests
dupname asflicense javac codespell detsecrets xmllint hadoopcheck spotless compile

uname
Linux ac908501f473 5.4.0-192-generic #212-Ubuntu SMP Fri Jul 5 09:47:39 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / 1a531e2c024f6a21e5ce6d169b887735b4c9ef57

Default Java
Eclipse Adoptium-11.0.23+9

Max. process+thread count
79 (vs. ulimit of 30000)

modules
C: . U: .

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 55s

the patch passed

+1 :green_heart:
compile
2m  8s

the patch passed

+1 :green_heart:
javac
2m  8s

the patch passed

+1 :green_heart:
javadoc
2m  8s

the patch passed

+1 :green_heart:
shadedjars
5m  0s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

+1 :green_heart:
unit
272m 57s

root in the patch passed.

304m 22s

Subsystem
Report/Notes

Docker
ClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6161/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6161

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 08c03c5461b7 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / 1a531e2c024f6a21e5ce6d169b887735b4c9ef57

Default Java
Eclipse Adoptium-17.0.11+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6161/1/testReport/

Max. process+thread count
7829 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6161/1/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 45s

Docker mode activated.

-0 :warning:
yetus
0m  5s

Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck

_ Prechecks _

_ branch-2.5 Compile Tests _

+1 :green_heart:
mvninstall
2m 26s

branch-2.5 passed

+1 :green_heart:
compile
1m 46s

branch-2.5 passed

+1 :green_heart:
javadoc
1m 38s

branch-2.5 passed

+1 :green_heart:
shadedjars
4m 45s

branch has no errors when building our shaded downstream artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 20s

the patch passed

+1 :green_heart:
compile
1m 45s

the patch passed

+1 :green_heart:
javac
1m 45s

the patch passed

+1 :green_heart:
javadoc
1m 37s

the patch passed

+1 :green_heart:
shadedjars
4m 42s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

+1 :green_heart:
unit
345m 44s

root in the patch passed.

373m 33s

Subsystem
Report/Notes

Docker
ClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6161/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6161

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux e184d8f569f7 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / 1a531e2c024f6a21e5ce6d169b887735b4c9ef57

Default Java
Temurin-1.8.0_412-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6161/1/testReport/

Max. process+thread count
6756 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6161/1/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 49s

Docker mode activated.

-0 :warning:
yetus
0m  5s

Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck

_ Prechecks _

_ branch-2.5 Compile Tests _

+1 :green_heart:
mvninstall
4m  4s

branch-2.5 passed

+1 :green_heart:
compile
3m 23s

branch-2.5 passed

+1 :green_heart:
javadoc
2m 53s

branch-2.5 passed

+1 :green_heart:
shadedjars
6m 42s

branch has no errors when building our shaded downstream artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 55s

the patch passed

+1 :green_heart:
compile
3m 50s

the patch passed

+1 :green_heart:
javac
3m 50s

the patch passed

+1 :green_heart:
javadoc
3m 28s

the patch passed

+1 :green_heart:
shadedjars
7m  9s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

-1 :x:
unit
353m  4s
/patch-unit-root.txt
root in the patch failed.

396m 15s

Subsystem
Report/Notes

Docker
ClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6161/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6161

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 53d73dd8e122 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / 1a531e2c024f6a21e5ce6d169b887735b4c9ef57

Default Java
Eclipse Adoptium-11.0.23+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6161/1/testReport/

Max. process+thread count
4815 (vs. ulimit of 30000)

modules
C: . U: .

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6161/1/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

UT failures are unrelated. This changes should not cause any UT failure, so merging!


Delete any Near Duplicate Images

Script Title -
The script will be  written in python and will look in a directory full of images to see any similarities or any possible duplicates between them and delete them to save storage.
Programming Language

[x] Python

Happy Coding

Can I work on this please?

Can I work on this please?

Sorry, I will be working on this myself


Is it possible to have the menu items become active automatically on scroll?

While I'm scrolling the page, I'd like to highlight my menu items automatically, so that user understand where he/she is in the navigation. I haven't seen something in the documentation like this. Is this possible or can I add it myself?

Hi,
no it's not: https://github.com/Nolanus/ng2-page-scroll/wiki/FAQ#is-it-possible-to-detect-the-current-scroll-section

Thanks @Nolanus


No default jsDelivr CDN file set

This package doesn't have a default file set. You can set it via jsdelivr, browser, or main field in package.json

Fixed for the community fork Fomantic-UI by https://github.com/fomantic/Fomantic-UI/pull/1613


document the differences between the three cephcsi images

What new functionality do you want?
I would like a paragraph of documentation that describes the differences in the three images we publish:

cephcsi/cephfsplugin
cephcsi/rbdplugin
cephcsi/cephcsi

The differences between these three are not clear to me. I suspect cephfsplugin and rbdplugin are going to go away at some point, but this is just a wild guess. If they are going away, why are they still around now? When should users expect them to disappear?
How would the end user gain value from having this feature?
Users have a clear understanding of what the three images are, and why consume one or the other.
This is also important to know how we want to package and ship these downstream in Red Hat's OpenShift Container Storage 4 product.

What new functionality do you want?
I would like a paragraph of documentation that describes the differences in the three images we publish:
1. `cephcsi/cephfsplugin`

2. `cephcsi/rbdplugin`

3. `cephcsi/cephcsi`

The support matrix will be updated soon after the v1.1.0 release with the docker images or version and that should avoid this confusion.

The differences between these three are not clear to me. I suspect cephfsplugin and rbdplugin are going to go away at some point, but this is just a wild guess. If they are going away, why are they still around now? When should users expect them to disappear?

We do have to maintain those for earlier versions of CSI spec. Say for example , v1.0 and 0.3. But eventually it will go out or will vanish. From csi v1.1.0 it will be just one iimage which is "cephcsi"

How would the end user gain value from having this feature?
Users have a clear understanding of what the three images are, and why consume one or the other.
This is also important to know how we want to package and ship these downstream in Red Hat's OpenShift Container Storage 4 product.

Downstream should be just one image which is "cephcsi" and it will be consumed by Rook.

I was discussing the ceph-csi project with someone, and the question of "what is the canary image" came up. It would be great to document the technical explanation of the "canary" tag for the container, because I don't think that's common terminology like "master" or "latest". In fact, maybe we should rename "canary" to "master".

@humblec any update on this one, moved to release-1.1.1


Service APIs support

We have plan to support Service APIs but without concrete steps to do it, and we need more and more feedbacks from the community, more communications with the Service APIs sig.
This issue is used to track our new trend about Service APIs.

use #644  instead


interation of set_eng_float_format and pivot_tables

There appears to be an interaction between the option for float display length and pivot_table methods on integers. This issue arose when working on a dataset with both floats and integers - but this arose on operations on integers part of the dataset.
The following example throws an error from the format.py module.
Swap the comments on the formatting options  (undoing the setting for floats) and it runs fine.
In my dataset - there were no missing values following the pivot table operation and the error still occurs. In the sample code below, NaNs may be causing the formatting issue, but that didn't appear to be the case for me.
import pandas as pd
import numpy as np

#pd.set_option('display.float_format', None)
pd.set_eng_float_format(accuracy=1)

df = pd.DataFrame(data = np.random.randint(25, size=(10, 3)), columns = list('abc'))
df.pivot_table(values='a', index='b', columns='c')

cheers
Chris

not so friendly, needs a nan-check before handing off to Decimal.
want to do a pull-request?

OK - I'll have a go since you've classified it as a novice level issue. Last time I tried (with a csv issue in 2014), I got lost very quickly.  I'll get back to you in a week or so.
cheers

@ChristopherShort ok thanks. lmk.

Hi,
I have fixed the bug. Is this fix going to released 0.18.2, or in the next major release. I am not sure about the file that I have to update in doc/source/whatsnew/vx.y.z.txt, would I need to update v0.18.2
Regards,
Yadu

0.18.2

yadu - thanks - this has been on my conscience for a couple of months (the 1 week blew right past).
I'll find something else I can contribute too.

No worries. cheers.


make ubuntu dockerfile to have user ubuntu instead of root

What's the issue?
make the ubuntu image has the ubuntu user instead of root.
Definition of Done

New dockerfile.ubuntu

The PR was merged.


Nginx with passenger?

Issue description
I can't find any passenger module or config option to enable passenger with nginx
Are there any plans to add passenger to nixpkgs?

Hi there! Recently I just got Nginx + Passenger working with my Rails application on NixOS, the derivations are located here (nixos-configs/pkgs/servers) and a sample NixOS configuration with Rails is at here.
It's currently not clean and may do some redundant work in the derivation, but I hope it can help people who want to use Passenger for now.

I found that puma is better then passenger, so I dont need this anymore

Well, what I need is a server that runs multiple apps, keeps them alive, and automatically scales. I think I'll stick to Nginx + Passenger for this kind of purpose 😉.

<digging this thread up from the dead>
I want to do some experiments with provisioning nginx+passenger app servers with nixops, so this would be nice to have in nixpkgs.
@zetavg are you still using your config? I see the last commit was years ago...


voice connection of  destroyed with 1006. wait for events.

While playing music, the following logs appear and the music does not play.
[2021-01-12 22:05:04,235] [connector.py] [discodo.VoiceConnector:connector] [INFO]: voice connection of<PHONE_NUMBER>47612488 destroyed with 1006. wait for events.

We need how it happened.

We need how it happened.

To be able to reproduce
Can you write the environment you are using and detailed information?

To be able to reproduce
Can you write the environment you are using and detailed information?

https://websockets.readthedocs.io/en/stable/faq.html#what-does-connectionclosederror-code-1006-mean
If it can be reproduce, tell us with websockets DEBUG logs.

https://websockets.readthedocs.io/en/stable/faq.html#what-does-connectionclosederror-code-1006-mean
If it can be reproduce, tell us with websockets DEBUG logs.

I tried to reproduce the situation, and as a reproduce, I got the following results.
It happened while playing music for more than an hour.
And the discodo server output the following logs:[2021-01-15 22:05:49,029] [connector.py] [discodo.VoiceConnector:connector] [INFO]: voice connection of<PHONE_NUMBER>42827796 destroyed with 1006. wait for events. websocket connection is closing.
I'm using Raspberry Pi 4B and OS is Raspberry Pi OS
The Python version is 3.8.6 and discodo module version is 2.3.9

I tried to reproduce the situation, and as a reproduce, I got the following results.
It happened while playing music for more than an hour.
And the discodo server output the following logs:[2021-01-15 22:05:49,029] [connector.py] [discodo.VoiceConnector:connector] [INFO]: voice connection of<PHONE_NUMBER>42827796 destroyed with 1006. wait for events. websocket connection is closing.
I'm using Raspberry Pi 4B and OS is Raspberry Pi OS
The Python version is 3.8.6 and discodo module version is 2.3.9

I tried to reproduce the situation, and as a reproduce, I got the following results.
It happened while playing music for more than an hour.
And the discodo server output the following logs:`[2021-01-15 22:05:49,029] [connector.py] [discodo.VoiceConnector:connector] [INFO]: voice connection of<PHONE_NUMBER>42827796 destroyed with 1006. wait for events.
websocket connection is closing.
`
I'm using Raspberry Pi 4B and OS is Raspberry Pi OS
The Python version is 3.8.6 and discodo module version is 2.3.9

Please give us aiohttp DEBUG logs.

I tried to reproduce the situation, and as a reproduce, I got the following results.
It happened while playing music for more than an hour.
And the discodo server output the following logs:`[2021-01-15 22:05:49,029] [connector.py] [discodo.VoiceConnector:connector] [INFO]: voice connection of<PHONE_NUMBER>42827796 destroyed with 1006. wait for events.
websocket connection is closing.
`
I'm using Raspberry Pi 4B and OS is Raspberry Pi OS
The Python version is 3.8.6 and discodo module version is 2.3.9

Please give us aiohttp DEBUG logs.


list_usbserial on Windows

Got this output on Windows - COM5 is a USB serial port, that's true, but it's for my 1532:0b00 aka HDK - not sure what the other foolishness is about (well, I know there's a WMI path there, but don't know what that thing masquerading as a VID/PID is)
C:\Users\Ryan\Desktop\build\OSVR-Core-vc12-ninja\bin>osvr_list_usbserial.exe
013e:013e \\WINDOWS-HS7N0FC\ROOT\CIMV2:Win32_SerialPort.DeviceID="COM5"

Inclined to not ship this on Windows until it works.
Concerned that it might indicate some underlying bug in the USB Serial Enum code on Windows, which IIRC @mars979 originally worked on?

If appears that if you hit line 188 in USBSerialDevInfo_Windows.h, the VID and PID are not detected or set. I'm guessing this is what's happening in your situation, but will have to investigate more thoroughly to know for sure.
(The one platform I didn't test... <grumble>)


Fix for Builder Issue #470

This PR adds support for canonical name + path resolution when a dependency + path is excluded (build: false) in an SFX build.

Thanks, it would be amazing to get a couple of tests on this as well in https://github.com/systemjs/builder/blob/master/test/canonicals.js.

I've been testing various changes.
bestAlias = alias + canonicalName.substr(alias.length);  does not work as canonicalName in the normal case let's say for parse is<EMAIL_ADDRESS>so it turns into<EMAIL_ADDRESS>and<EMAIL_ADDRESS>of course the other mappings are equally messed up.
Another potentially nasty corner case is what if two left hand values share the same right hand value. The last left hand value will always be matched as one is sequentially traversing the System.map array.

Ok I think a aliasedName = bestAlias + canonicalName.substr(map[bestAlias].length) then at the end?
In terms of preference, the rules for map config specificity are based on only accepting a new match with the specificity of alias.split('/').length. When two matches have the same specificity, the first one should win. See the algorithm at https://github.com/systemjs/systemjs/blob/master/lib/core.js#L26.
The getBestMatch function seems badly named too at this point. The algorithm can definitely be cleaned up. It's all well-defined stuff though.
There is one other thing here and that is that this matching is supposed to happen in canonical space for completeness. I'm not too worried about it for now, but just to mention the idea is that canonicalName should be decanonicalized via loader.decanonicalize(canonicalName) and the map targets should also be decanonicalized this way, and this is the space we should do the matching in. That will handle cases where people manually map into jspm_packages paths like map: { jquery<EMAIL_ADDRESS>}.

Sorry I mean URL space not canonical space in the last paragraph.

This seems to work for the bestAlias return path:
  if (bestAlias)
    return bestAlias + canonicalName.substr(loader.map[bestAlias].length);

It's pretty simple too. I'll certainly have to defer to you understanding regarding the rest of your comment about the inner workings.

Sure, happy to go ahead with just that fix for now.

I got lazy and created a new PR: https://github.com/systemjs/builder/pull/485
Please ignore this one.


Trying BOND on new datasets and languages

Thank you for the great work. Would you be able to provide the Gazetteers data and distant label generation code? I would like to try BOND on new datasets. @cliang1453 @HMJiangGatech
Thank you in advance

Sure, could you please share your email address?

Thank you Chen. I sent you an email two days ago. I can resend again.

Thanks for resending! It looks like the previous one was filtered out by the system. I've replied back with the code and data.


Reworked generateStartDateText to avoid short circuiting

Pre-Flight checklist

[X] Testing

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

What are the relevant tickets?
#523
What's this PR do?
Hopefully the final fix for dashboard dates.
How should this be manually tested?
As a user with enrollments, log in and navigate to the dashboard. The enrollments should display with the proper start date.
As a user with items in the cart, navigate to the cart. The cart items should display the course start date properly. Additional items (added manually through the Django admin) should also display properly.
Any background context you want to provide?
A production build of the app may be useful for testing - the code works (and has worked) successfully on dev builds but failed once it was minimized.

Codecov Report

Merging #555 (08dc688) into main (b37ff0e) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #555      +/-   ##
==========================================
+ Coverage   86.87%   86.89%   +0.02%
==========================================
  Files         114      114
  Lines        4816     4816
  Branches      766      766
==========================================
+ Hits         4184     4185       +1
+ Misses        522      520       -2
- Partials      110      111       +1

Impacted Files
Coverage Δ

ecommerce/discounts.py
80.70% <0.00%> (+1.75%)
:arrow_up:

Continue to review full report at Codecov.

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


Bluesky: change fediverse handle format?

I'm considering changing our Bluesky handle format.
Right now, we use subdomain labels liberally to convert fediverse addresses like<EMAIL_ADDRESS>to Bluesky handles like user.example.com.ap.brid.gy. This works, and it's simple, but it has drawbacks. We've been struggling to plan for how to scale handle verification, #744.
If I switch to the - character for the fediverse address part, eg user-example-com.ap.brid.gy, the resulting handles are more complicated, but I can switch to Bluesky's HTTPS handle verification method, and I'd only need a single wildcard DNS record and a single wildcard SSL cert.
I'm inclined to do it.

Just in case it's not on your radar, DNS names do have a 63-byte label length limit. (Label being the bit between dots.)
I'd guess it's unlikely that this would be a problem in practice but I don't have anything really substantive to support that feeling.

Thanks, yes! 63 chars per label, 253 chars total.
I also need to look at allowed characters. For DNS it's just alphanumeric and -, right? WebFinger usernames may allow a few more, eg .s, which users would also need to translate to -s.

Based on https://datatracker.ietf.org/doc/html/rfc7565#section-7 and https://datatracker.ietf.org/doc/html/rfc3986#section-2.3 , WebFinger usernames also allow _ and ~.

I also need to look at allowed characters. For DNS it's just alphanumeric and -, right?

Kinda. DNS labels are arbitrary bytes but case insensitive to a-z. Hostname labels are alphanumeric and dash (for the most part). To enable internationalisation IDNA (RFC 5890 and RFC 5891) then represents a normalised subset of unicode in a hostname label compatible format. If that's something you want to handle dnspython has some helpers for marshalling names to and from IDNA.

Some platforms support - and _ in their handles (even though WebFinger may not allow a - in the username). Even domains can have a - in it.
You would need to account for this when you create the Bluesky version of the handle.

Yup, absolutely true. Ugh.
I'm sure I (we) can come up with an encoding that can turn every fediverse handle into a single DNS compliant label, and it might still be worth it to avoid the DNS zone scaling issue...but UX-wise, it's getting uglier and uglier. Sigh.

Would making it two levels help? (i.e. separating username and server)
username.social-example-com.ap.brid.gy for<EMAIL_ADDRESS>That would reduce the number of zones, although that may not matter depending on how you implemented creating the zones.

Yeah I've considered various multi-level formats like that. The DNS scaling concern (more in #744) is mainly the number of records, regardless of how they're organized into zones.
The win of going fully flat under ap.brid.gy, as is proposed here, is that we can drop DNS entirely and use ATProto's HTTPS-based handle resolution method instead. That scales better in number of accounts, but SSL certs only support a single level of wildcards, ie *.ap.brid.gy but not *.*.ap.brid.gy, so the namespace would need to be flat. 🤷

Hi Ryan, nice to hear from you! :-)
On Thu, Feb 22, 2024 at 11:06 AM Ryan Barrett @.***>
wrote:

Yeah I've considered various multi-level formats like that. The DNS
scaling concern (more in #744
https://github.com/snarfed/bridgy-fed/issues/744) is mainly the number
of records, regardless of how they're organized into zones.
The win of going fully flat under ap.brid.gy, as is proposed here, is
that we can drop DNS entirely and use ATProto's HTTPS-based handle
resolution method
https://atproto.com/specs/handle#https-well-known-method instead. That
scales better in number of accounts, but SSL certs only support a single
level of wildcards, ie *.ap.brid.gy but not ..ap.brid.gy, so the
namespace would need to be flat. 🤷
—
Reply to this email directly, view it on GitHub
https://github.com/snarfed/bridgy-fed/issues/830#issuecomment-1959990887,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADSHPCHZIFCLJD6H6PPLTTYU6CJ5AVCNFSM6AAAAABDEDH4OOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNJZHE4TAOBYG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

And with the number of subdomains, AutoSSL for each one will become a pain.
The tricky part will be handling domains and usernames with - in it.

Given that this is only a win if it's entirely flat (https://github.com/snarfed/bridgy-fed/issues/830#issuecomment-1959990887), and that means we'd need to come up with a more complicated encoding to handle -, _, ~, and similar characters, I'm now less inclined to do this. Sigh.

Decided against this, sadly.


Having multiple images in viewpager is not zooming

In Viewpager which is horizontal swiping, the first image is zoomable but the rest of the images are not able to zoom.

I'll need to look more into this, but I'm wondering if ViewPager is consuming the touch events.

So, what if it's true?
I have a ZoomageView in my recyclerview (Horizontal) and I cannot zoom anyway.

Please try latest snapshot

@jsibbold The update you did for 1.3.0 seems to have improved functionality when ZoomageView is used in conjunction with a view pager. Do you have an ETA on when 1.3.0 will be released?

@shirtleton I'll try to get that release out by end of week

@shirtleton sorry it took me a bit longer, but 1.3.0 should be good and up on jcenter. Let me know if any issues arise getting it.


Extend the AST with spans instead of just locations

Currently only a location is stored for each AST node indicating the first character of the expression. If instead a span where stored for each expression the error messages could be significantly be improved.
The current way of storing a location for each node is probably not needed as well as for many expressions the location/span could be calculated from the data it contains.

Implemented imprecise spans in df133667d37f7547a1e17a049d826356ed1d82f7

Closed by #123


Training stuck at 0%

I'm currently using a docker image which include all required pytorch and CUDA 8.0 dependencies.
I just cloned the repo and executed the demo_voc2007 with the default argument values.
The training stuck at 0%, nothing happens.
Is there a known reason why is this happening?

Hi Tauranis, I encountered the same problem as you. Do you have any solution now?