allow KEYMAP with @a

I've debugged into vim and found the location, in which normal keymap application diverges when playing a register macro (@a).
It is in getchar.c: 2189
		    if (mp->m_keys[0] == c1
			    && (mp->m_mode & local_State)
			    && ((mp->m_mode & LANGMAP) == 0
				|| typebuf.tb_maplen == 0))

Both (no@a) and (@a) have
mp->m_mode == 32 == LANGMAP
local_State == 48 == 32+16

but only (@a) has: typebuf.tb_maplen > 0
Cited from getchar.c regarding the tb_maplen:

The head of the buffer may contain the result of mappings, abbreviations
and @a commands.  The length of this part is typebuf.tb_maplen.

THE LAST TWO LINES COMMENTED AWAY MADE KEYMAP WORK WITH @a COMMANDS.
So before: no LANGMAP for @a. That's what ((mp->m_mode & LANGMAP) == 0 || typebuf.tb_maplen == 0) says, anyway.
With patch: LANGMAP for @a, i.e. without the condition.
tb_maplen is only set in  ins_typebuf(), if the nottyped parameter is TRUE.
There are 12 such calls in check_abbr, do_execreg, exec_normal_cmd, server_to_input_buf, put_reedit_in_typebuf, put_in_typebuf.
As for abbreviation I found no difference in behavior before and after the patch.
Personally I would like to use KEYMAP with @a.
I also regard it as a general deficiency, when e.g. a russian guy cannot use @a to insert russian text.
But maybe there are unwanted side effects, I'm not aware of.
Bram, maybe you can think of side effects or know why this condition was put there in the first place.

Roland Puntaier wrote:

I've debugged into vim and found the location, in which normal keymap application diverges when playing a register macro (@a).
It is in getchar.c: 2189
  		    if (mp->m_keys[0] == c1
  			    && (mp->m_mode & local_State)
  			    && ((mp->m_mode & LANGMAP) == 0
  				|| typebuf.tb_maplen == 0))

Both (no@a) and (@a) have
mp->m_mode == 32 == LANGMAP
local_State == 48 == 32+16

but only (@a) has: typebuf.tb_maplen > 0
Cited from getchar.c regarding the tb_maplen:

The head of the buffer may contain the result of mappings, abbreviations
and @a commands.  The length of this part is typebuf.tb_maplen.

THE LAST TWO LINES COMMENTED AWAY MADE KEYMAP WORK WITH @a COMMANDS.
So before: no LANGMAP for @a. That's what ((mp->m_mode & LANGMAP) == 0 || typebuf.tb_maplen == 0) says, anyway.
With patch: LANGMAP for @a, i.e. without the condition.

The 'keymap' option is meant for when typing text in Insert mode.  It
does not apply to when typing commands, in Normal mode.  The 'langmap'
option is the other way around.
A register contains commands, which may start Insert mode and then
insert some text.  Thus it's a mix.  Translating all characters with
'keymap' (or 'langmap') will cause problems.
I wonder how you managed to get characters into a register that you want
to apply 'keymap' or 'langmap' to.  This translation should have already
happened when typing the characters.

tb_maplen is only set in  ins_typebuf(), if the nottyped parameter is TRUE.
There are 12 such calls in check_abbr, do_execreg, exec_normal_cmd,
server_to_input_buf, put_reedit_in_typebuf, put_in_typebuf.
As for abbreviation I found no difference in behavior before and after
the patch.
Personally I would like to use KEYMAP with @a.
I also regard it as a general deficiency, when e.g. a russian guy
cannot use @a to insert russian text.
But maybe there are unwanted side effects, I'm not aware of.
Bram, maybe you can think of side effects or know why this condition
was put there in the first place.

Yes, there are undesired side effects.

DENNIS:  Ah, now we see the violence inherent in the system.

DENNIS:  Oh!  Come and see the violence inherent in the system!
HELP! HELP!  I'm being repressed!
The Quest for the Holy Grail (Monty Python)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

On Wed, Dec 30, 2015 at 9:10 PM, Bram Moolenaar
<EMAIL_ADDRESS>wrote:

Roland Puntaier wrote:

I've debugged into vim and found the location, in which normal keymap application diverges when playing a register macro (@a).
It is in getchar.c: 2189
if (mp->m_keys[0] == c1
&& (mp->m_mode & local_State)
&& ((mp->m_mode & LANGMAP) == 0
|| typebuf.tb_maplen == 0))
Both (no@a) and (@a) have
mp->m_mode == 32 == LANGMAP
local_State == 48 == 32+16
but only (@a) has: typebuf.tb_maplen > 0
Cited from getchar.c regarding the tb_maplen:

The head of the buffer may contain the result of mappings, abbreviations
and @a commands. The length of this part is typebuf.tb_maplen.

THE LAST TWO LINES COMMENTED AWAY MADE KEYMAP WORK WITH @a COMMANDS.
So before: no LANGMAP for @a. That's what ((mp->m_mode & LANGMAP) == 0 || typebuf.tb_maplen == 0) says, anyway.
With patch: LANGMAP for @a, i.e. without the condition.

The 'keymap' option is meant for when typing text in Insert mode. It
does not apply to when typing commands, in Normal mode. The 'langmap'
option is the other way around.
A register contains commands, which may start Insert mode and then
insert some text. Thus it's a mix. Translating all characters with
'keymap' (or 'langmap') will cause problems.
I wonder how you managed to get characters into a register that you want
to apply 'keymap' or 'langmap' to. This translation should have already
happened when typing the characters.

tb_maplen is only set in ins_typebuf(), if the nottyped parameter is TRUE.
There are 12 such calls in check_abbr, do_execreg, exec_normal_cmd,
server_to_input_buf, put_reedit_in_typebuf, put_in_typebuf.
As for abbreviation I found no difference in behavior before and after
the patch.
Personally I would like to use KEYMAP with @a.
I also regard it as a general deficiency, when e.g. a russian guy
cannot use @a to insert russian text.

To insert text, I would use "a (or, in Insert mode, ^Ra), not @a. But
then, I very seldom use recording (about the only case is when I wan
something to go on repeating until it errors out). I have set apart
register q for those rare cases; but most of my other registers
contain text strings that I want to be ready to paste anytime. Some
are rather long text which I don't use very often but wouldn't
remember by heart, others are short but frequent. And of course if I
want to be able to repeatedly insert some Russian text I'll put it in
the register in Cyrillic; for English (or French or German or …) I
would put it there in Latin. The help for i_CTRL-R says "The text is
inserted as if you typed it, but mappings and abbreviations are not
used."; language-mappings (keymaps) are not used either, so that e.g.
in an HTML file in Russian, when I hit ^Ri what gets inserted is
(the contents of the register) and not <и> which wouldn't be a proper
HTML tag, even if at that point my "cyrillic-phonetic" keymap is
enabled ('iminsert' = 1).
I'm not very familiar with the 'langmap' option but I would expect
registers containing Cyrillic text to be pasted in Cyrillic and
registers containing Latin text to be pasted in Latin in that case
too. Does 'iminsert' apply to this option too? I couldn't find the
answer in the help for any of 'langmap', 'iminsert' and i_CTRL-^
(maybe translate even text typed in Insert mode when 'iminsert' is 0
[to insert Latin text] but not when it is 1 [to insert Cyrillic
text])?

But maybe there are unwanted side effects, I'm not aware of.
Bram, maybe you can think of side effects or know why this condition
was put there in the first place.

Yes, there are undesired side effects.

DENNIS: Ah, now we see the violence inherent in the system.

DENNIS: Oh! Come and see the violence inherent in the system!
HELP! HELP! I'm being repressed!
The Quest for the Holy Grail (Monty Python)
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net \
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\ an exciting new programming language -- http://www.Zimbu.org ///
\\ help me help AIDS victims -- http://ICCF-Holland.org ///

Dear Bram,

A register contains commands, which may start Insert mode and then insert some
text. Thus it's a mix. Translating all characters with 'keymap' (or 'langmap')
will cause problems.

Not ALL characters are translated, because the map hash includes the vim State:
local_State = get_real_state();
...
mp = curbuf->b_maphash[MAP_HASH(local_State, c1)]; //getchar.c:2158

The hash is generated with LANG_MAP:
(void)do_map(2, buf, LANGMAP, FALSE);

For vim.h:
#define LANGMAP		0x20	/* Language mapping, can be combined with
                   INSERT and CMDLINE */

This means keymap only applies to INSERT and CMDLINE, as it should be.
Note: With LANGMAP I meant and mean the define in the C code not :set langmap.
The latter is handled via LANGMAP_ADJUST using the global static garray_T langmap_mapga;
defined in options.c.
keymap on the other hand is done via the global or local maphash.
langmap is handled before keymap.
The C LANGMAP define only applies to keymap.
Note: Probably russians will have a russian keyboard layout and use :set
langmap instead of :set keymap. Russian is only an example, though.
Since keymap applies to INSERT and CMDLINE for direct typing,
it should do so also for @a macros. I think everybody will expect
@a to behave the same way as direct typing.

I wonder how you managed to get characters into a register that you want to
apply 'keymap' or 'langmap' to. This translation should have already happened
when typing the characters.

do ":help russian"
use the keymap as described there
insert some russian text
record a macro @a that inserts some russian text
play the macor @a

Yes, there are undesired side effects.

Please name the undesired side effects, else I can't decide, whether you think, there are,
or you know them.

On 12/30/2015 9:59 PM, Tony Mechelynck wrote:

The help for i_CTRL-R says "The text is inserted as if you typed it,
but mappings and abbreviations are not used."; language-mappings
(keymaps) are not used either, so that e.g. in an HTML file in
Russian, when I hit ^Ri what gets inserted is  (the contents of the
register) and not <и> which wouldn't be a proper HTML tag, even if at
that point my "cyrillic-phonetic" keymap is enabled ('iminsert' = 1).

Does 'iminsert' apply to this option too? I couldn't find the answer
in the help for any of 'langmap', 'iminsert' and i_CTRL-^ (maybe
translate even text typed in Insert mode when 'iminsert' is 0 [to
insert Latin text] but not when it is 1 [to insert Cyrillic text])?

:set keymap=russian-jcukenwin
Will set iminsert=1. So this means keymap set.
^Ri is completely independent of keymap and it remains so with my patch.

I'm not very familiar with the 'langmap' option but I would expect
registers containing Cyrillic text to be pasted in Cyrillic and
registers containing Latin text to be pasted in Latin in that case too.

If you had e.g. a russian keyboard layout, you wouldn't be able to use
vim, e.g. to move around in normal mode, like with the english
keyboard's hjkl. langmap will translate ролд to hjkl.
While langmap applies for normal mode, keymap works in INSERT and
CMDLINE mode.
Langmap will not apply for CMDLINE, so you will not be able to enter vim
commands like :g, because the g is п on the russian keyboard. So langmap
is probably not very useful.
Keymap on the other hand will keep the english keyboard in normal mode
and for : commands. So you can still enter :g. It applies for /, so you
can search for russian text. (Behavior not changed with my patch.)

On 12/31/2015 6:22 PM, Bram Moolenaar (Vim Github Repository) wrote:

Note: Probably russians will have a russian keyboard layout and use :set
langmap instead of :set keymap. Russian is only an example, though.
Since keymap applies to INSERT and CMDLINE for direct typing,
it should do so also for @a macros. I think everybody will expect
@a to behave the same way as direct typing.

No, the langmap and keymap options relate to the keyboard. Replaying a
register does not involve the keyboard. When replaying a register one
doesn't know where the characters came from, they might be typed or the
result of something else, e.g. a yank operation.

I wonder how you managed to get characters into a register that
you want to
apply 'keymap' or 'langmap' to. This translation should have
already happened
when typing the characters.

do ":help russian"
use the keymap as described there
insert some russian text
record a macro @a that inserts some russian text
play the macor @a

This is a bit vague... Perhaps the problem is that when recording into
a register the keymap/langmap is not applied, resulting in the wrong
characters in the register?
I look at keymap/langmap as a low level operation, which is applied when
typing on the keyboard, before it goes to the rest of Vim, which is
unaware of this filtering.
It's possible that this interferes with other things, such as terminal
codes and other mappings. It's quite complicated. Many changes in this
area are not backwards compatible and break someone's carefully tuned
setup. I have thought of re-implementing this, but haven't found the
time to dive into it.

As of now
//getchar.c:2515
if (keylen > typebuf.tb_maplen)
gotchars(typebuf.tb_buf + typebuf.tb_off
+ typebuf.tb_maplen,
keylen - typebuf.tb_maplen);
is before
//getchar.c:2610
i = ins_typebuf(s, noremap,
i.e. the current code does not put the result of the keymap mapping into
the register.
So my patch fits to the current implementation by applying the keymap to
chars in the register.
Since my last commit I have

left the original version as comment for reference in case of problems.
added langmap application to ':'-CMDLINE, as proposed by Tony Mechelynck,
in order to make langmap a better opposite to keymap.
I did LANGMAP_ADJUST after gotchars(),
because currently it is applied on chars from registers (@a).

I have also updated the documentation to mention:

langmap applies to ':'-CMDLINE mode (new)
keymap applies to non-':'-CMDLINE mode (current)

Best Regards, Roland

On 12/31/2015 6:22 PM, Bram Moolenaar (Vim Github Repository) wrote:

Note: Probably russians will have a russian keyboard layout and use :set
langmap instead of :set keymap. Russian is only an example, though.
Since keymap applies to INSERT and CMDLINE for direct typing,
it should do so also for @a macros. I think everybody will expect
@a to behave the same way as direct typing.

No, the langmap and keymap options relate to the keyboard. Replaying a
register does not involve the keyboard. When replaying a register one
doesn't know where the characters came from, they might be typed or the
result of something else, e.g. a yank operation.

I wonder how you managed to get characters into a register that
you want to
apply 'keymap' or 'langmap' to. This translation should have
already happened
when typing the characters.

do ":help russian"
use the keymap as described there
insert some russian text
record a macro @a that inserts some russian text
play the macor @a

This is a bit vague... Perhaps the problem is that when recording into
a register the keymap/langmap is not applied, resulting in the wrong
characters in the register?
I look at keymap/langmap as a low level operation, which is applied when
typing on the keyboard, before it goes to the rest of Vim, which is
unaware of this filtering.
It's possible that this interferes with other things, such as terminal
codes and other mappings. It's quite complicated. Many changes in this
area are not backwards compatible and break someone's carefully tuned
setup. I have thought of re-implementing this, but haven't found the
time to dive into it.

As of now
//getchar.c:2515
if (keylen > typebuf.tb_maplen)
gotchars(typebuf.tb_buf + typebuf.tb_off
+ typebuf.tb_maplen,
keylen - typebuf.tb_maplen);
is before
//getchar.c:2610
i = ins_typebuf(s, noremap,
i.e. the current code does not put the result of the keymap mapping into
the register.
So my patch fits to the current implementation by applying the keymap to
chars in the register.
Since my last commit I have

left the original version as comment for reference in case of problems.
added langmap application to ':'-CMDLINE, as proposed by Tony Mechelynck,
in order to make langmap a better opposite to keymap.
I did LANGMAP_ADJUST after gotchars(),
because currently it is applied on chars from registers (@a).

I have also updated the documentation to mention:

langmap applies to ':'-CMDLINE mode (new)
keymap applies to non-':'-CMDLINE mode (current)

Best Regards, Roland

Hello Bram,
I've seen the recently added (issues/572) test_langmap.vim's use of feedkeys().
That langmap should apply after feedkeys() does not fit
to my changes to the langmap implementation,
which apply langmap at a very early place (inchar())
and also apply langmap in ':'-command line mode.
The latter because langmap is intended to be complementary to
keymap and because langmap's intention likely was
to make vim usable from cyrillic or greek keyboards.
Personally I'm favoring my solution, but not passionately (As I've mentioned I'm more interested
in the keymap correction).
Anyway it's up to you to decide. If you reject this pull request,
allow me to open another pull request with only the originally intended change regarding keymap.
Thanks.
Best, Roland

Roland Puntaier wrote:

Hello Bram,
I've seen the recently added (issues/572) test_langmap.vim's use of
feedkeys().
That langmap should apply after feedkeys() does not fit
to my changes to the langmap implementation,

Note that depending on flags feedkeys() inserts a bit earlier or not.
Esp. whether the keys are mapped and/or typed.  Well, it's the same
buffer but with different flags (we may split it into more than one
buffer some day to make this easier).

which apply langmap at a very early place (inchar())
and also apply langmap in ':'-command line mode.
The latter because langmap is intended to be complementary to
keymap and because langmap's intention likely was
to make vim usable from cyrillic or greek keyboards.
Personally I'm favoring my solution, but not passionately (As I've
mentioned I'm more interested in the keymap correction).
Anyway it's up to you to decide. If you reject this pull request,
allow me to open another pull request with only the originally
intended change regarding keymap.

I haven't looked at your latest patch yet, but if langmap/keymap is
applied before where feedkeys() inserts characters, how can you test it?
Perhaps we need another flag in feedkeys() to insert keys even earlier?
--
Seen it all, done it all, can't remember most of it.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

":call feedfields()" is in :-Command-mode, where langmap according this pull request already applies.
Testing can still be done with old style tests. The current tests are in test_mapping.in.
They include (issues/572).

Any progress?

seems stalled, so let me close this


delete useless bloat

About the PR
removes useless bloat from the game files (RobustToolbox)
Why / Balance
people will lag less and be happy again
Technical details
delete the folder and its files, world peace
Media

Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

unoriginal

wait does that go into the actual thing oh damn didnt know was following the tutorial


Fix the tests and user nomad's user stanza instead

I see and hopefully even understand what it does. Thank you. Note, however, that Docker driver in Nomad relies upon Docker Image setting the correct user, if I understand the documentation correctly and correct me if I'm wrong. I've also missed this part of documentation before. I cannot say whether this presents a drawback for some people or not.
Also, this explains why my Docker slaves were running as root user, because that's default user and user under which sshd is running under and Jenkins is SSH-ing in as jenkins with a matching key. No biggie, just providing explanation/personal observation.

@jovandeginste, please, let me know and I'll merge it in.

I've merged it not to block the progress. It's possible to revert it, if needed. Once again, thank you @jovandeginste.

@zstyblik Not sure I understand your note.
As far as I understand, these things are to be considered:

When building a Docker image, you can specify (more than once) a user stanza
All subsequent stanza's will be executed as that user (if relevant). The CMD and ENTRYPOINT commands will be run with the final USER, unless overridden at runtime. The default user is root:root.
When launching a Docker container, you can override the user as which the CMD or ENTRYPOINT are executed, using the user flag (--user).
As Nomad launches "jobs" (in the general sense), it is useful to specify as what user the job should run.
This is relevant for raw exec, Docker containers, and in general any type of job. It provides the user stanza at task-level. In my tests yesterday it behaves as such (wrt. Docker containers):
** No User stanza given: use the default Docker user as specified in the image
** Any User stanza given will be equivalent to passing the --user parameter to the Docker run command
** If the value of user is the empty string (ie. --user=""), Docker still uses the default user as specified in the image
This led me to the implementation to always pass the switchUser as user in Nomad, even if it's empty (blank). The stack behaves as expected and does not change the user if it's blank. If you specify a user, it is passed to the Docker runtime engine. I did not test what happens if the user specified does not exist, but I don't think we should worry about that...

I hope I answered your question (and that my answer is not more confusing than clarifying :-))

@jovandeginste

Docker and rkt images specify their own default users.

facepalm as I've missed keyword default. Damn! And yes, I was like "oh, this doesn't make any sense to me. Why wouldn't you pass --user and rely on Docker Image?" Again, thank you for exhausting explanation and clarification. I'm off into the corner of shame since I've made a fool out of myself ... a bit :|

@zstyblik Don't fret yourself over this - we all learn new stuff (almost) every day. If you were considering yourself an absolute expert in the matter, you should probably revisit that stance. Otherwise it's just a regular thing being active on the Internet :-)


Remote Desktop using xRDP for RHEL 8

Hi Team.
I have installed RHEL 8 in GCP machine and when I am trying to login remote system through xrdp then it asking first
This account will be use to connect to localhost : Here I am giving my company mail id to login and password then it goes inside the xrdp system here if i create new user then i can login but unable to login using existing id and password
Connecting to sesman on <IP_ADDRESS>:3350
sesman connect Ok
sending login info to session manager
login failed for user xyz
the id with which i login first ideally it should using the same id , but as i said if i create new user and password and then login at second step is sucesfully login to remote desktop of RHEL
Please help me if any idea

Hi @mahendra03
I do not fully understand what you are asking.
xrdp is simply calling down to the normal user authentication facilities provided by the operating system.
If you are trying to use a user which the operating system doesn't know about, xrdp won't be able to log you in.
To find out if the system knows about a particular user, you can use this command:-
getent passwd <username>

Hi @matt335672  - I will try what you are suggesting.
Let me explain you what I am trying to do :
I am mean to say first i open mstsc for my mac and there I provide the password using which I login to macbook then it open new screen where I provide session xvnc and here I am confuse which user name and password to provide if i provide same passowrd and user name with which I login to VM then getting error
we are using gcp instance.

To do this, I think you will need to add your macbook login credentials to the GCP instance.
Alternatively, you may be able to get mstsc for the Mac to store your login credentials for xrdp so that you don't need to enter them on the xrdp login screen.
I can't comment on the overall security of this approach, as I don't know the rest of your system.You should get someone from your IT department to review the security as well.

@matt335672 Sure I ll check on this .... update you and thanks for the help


[FEATURE] typing final value from lookup or default

Is your feature request related to a problem? Please describe.
in trying to use an integer value the env call makes it into a string 'n'
not sure if this is RTFM on my part, i tried prefixing with !!INT !ENV ${WW_SENSOR_READ_TIMEOUT:90}
but gives me parse errors
Describe the solution you'd like
i'd like to get the value into the data type i would be expecting
Describe alternatives you've considered
i'm able to wrap getting the value and add conversion.
Additional context
sorry again if this is just due to my ignorance with python and the yaml format

Hi @tim-oe 👋
So from README, you can do:
...
!TAG tag:yaml.org,2002:float ${ENV_TAG2:27017}
...

which, in your case would be:
!ENV tag:yaml.org,2002:int ${WW_SENSOR_READ_TIMEOUT:90}

Can you try and let me know if this works for you?

that worked.
thank you!


README: minor correction and improvement

Starting block with "This" is bit fuzzy as there are multiple
preceding statements where "This" could relate to.

Few things stinging the eye while I made a read-through after some time passed from previous read.
Small but I get them documented in this PR while I still remember.
We can add more if we find.
One possible location to check fix is the explicit Docker versions that we list. This lists moves along and doc gets out of date, so we should either ad more "Docker tested" versions there, or simplify is to mention only major versions, or something else (what about removing the whole version list, does it help at all?)

Few things stinging the eye while I made a read-through after some time passed from previous read.
Small but I get them documented in this PR while I still remember.

We should simply merge them each time there is something. No need to wait...

One possible location to check and fix is the list of explicit Docker versions. This set moves along and README gets out of date, so we should either add more "Docker tested" versions there, or simplify mentioning only major versions, or something else (what about removing the whole version list, does it help at all?)

Let's remove it. The README.md is for users and they don't need Docker.

The README.md is for users and they don't need Docker.

but we have chapter in README "Build PMEM-CSI" which states "Use make build-images to produce Docker container images.", means we document building which requires Docker.
The statement we are removing, implicitly documents that dependency now.
What about we replace current stmt stating Docker versions, with something like "Docker is required to build PMEM-CSI" under "Software required" ?

"Build PMEM-CSI"

Let's move that to DEVELOPMENT.md. I don't think we need to document a Docker version there, because in practice it doesn't matter and we don't want to force developers to install some specific version just because that's the one we tested with.

ready for merging now?


chore: bump actions/checkout

https://github.com/actions/checkout\#checkout-v3

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


generated-sources/apt not being detected as a source directory when importing into eclipse

The hibernate JPA DDL  generated sources are not added to an eclipse project when importing. This is caused because there is no configuration in the POM to add them. It's easy to reproduce, just import the project using the POM in eclipse and notice all of the Entity_ classes cannot be found.
A fix would be to use the build-helper-maven-plugin so eclipse is aware of the sources. I'll submit a pull request later in case you're interested in just doing that.

I think this issue can also be closed, right @tkent?

@dacrome
Sorry for the slow response - yes, absolutely.


RCTSinglelineTextInputView.h file not found (RNTextInputMask.m)

Hello there.
I've done the steps in the docs, but no luck, my project on iOS won't build
#import "RCTSinglelineTextInputView.h"

in file RNTextInputMask.m (line 13)
Versions:
"react-native-text-input-mask": "^0.7.0",
"react-native": "0.52.x",
"react": "16.2.0",
Also using react-native-navigation if this matters

I'm having this also

Roll back to version 0.5.2 worked for me

@ppozniak I tried to build the library for 0.56 RN, seems it works good so I close the issue, if it still happens please let me know to reopen the issue

I'm having same issue on 0.58 RN with version 1.0.1. Any updates? I cannot roll back to before versioning 1 because of the Swift conversation of XCode 10.2. Thanks apple.

I found the solution. Make sure import of libRNTextInputMask.a after the libRCTText.a.
😄
Thanks.

Anyone found a solution?

I'm having same issue.
"react": "16.0.0-beta.5",
"react-native": "0.51.0",
"react-native-text-input-mask": "^1.0.1"

facing the same issue.

Facing the same issue here as well, seems to come up out of no where. Have had no prior build issues but am getting it today randomly with no code changes.

same issue

@ivanzotov this seems like a pretty persistent issue. Is there a reason why it's still closed?

I'm also facing same issue any idea for this.

Facing same issue #import <RCTText/RCTSinglelineTextInputView.h> 'RCTText/RCTSinglelineTextInputView.h not found'
Any one found and working solution, please share, i am using version 1.0.4
"react-native-text-input-mask": "^1.0.4"

I have resolved my issue by doing the following step.

As @dogukankotan mention I found the solution. Make sure import of libRNTextInputMask.a after the libRCTText.a.
add InputMask.framework - Go Library in Xcode -> RNTextInputMask.xcodeproj -> Under TARGETS(RNTextInputMask) -> Build Phases -> Link Binary With Libraries.

Same here. I am having the same problem. Is working on development but not in production

Same here. Any solution?


Rename MaybeApply to RandomApply

There is an issue in the keras_cv repository to rename the MaybeApply layer to RandomApply in order to match standard naming conventions both within and external to keras_cv.
This PR updates the guides in the keras-io repo that uses the MaybeApply layer to use the new name for that layer.
Here are some of the relevant links:

keras_cv issue
keras_cv PR

Note:
The updated guides haven't been tested since I believe we need to wait for a new release of keras_cv with the change before I can test this.
Who can review?
@ianstenbit @LukeWood @jbischof

I don't know if there are labels to let people know this PR shouldn't be merged until the new keras_cv release is cut and this PR is tested with the new keras_cv release. If there are, could someone please add it to this PR?

@ianstenbit this guide doesn't seem to be published on keras.io anyway. You can't even find it through Google search. If this is the only use case there's no harm updating before a release.

Ah, in that case we need to wait @fchollet. We'll push out v0.5.1 in the new few weeks

Thanks for the PR.
The RandomApply does not seem to exist in the latest stable KerasCV release -- @ianstenbit do you know what we should do here?

My plan was that we'd review this PR and only merge it when we cut 0.5.1 (see my comment here

We're past the keras-cv 0.5.1 release, let's merge this.


Fix SR-2868

Added a guard statement to significandWidth property of floating point types to fix SR-2868.

Resolves SR-2868.

This looks fine, but please also add one or more tests that exercise this in test/stdlib/Float.swift.

First, My initial fix broke the API contract for the return value of significandWidth of infinity values. It is corrected now.
I looked at test/stdlib/Float.swift to add a test, but I am not sure what is the correct place to put it: Should I create a new test specifically for testing significandWidth, or should I just add significandWidth check to each case (normal, subnormal, infinity, etc)?

Looking at things a bit more, I think the most natural place is as a new test in test/stdlib/FloatingPoint.swift.gyb:
FloatingPoint.test("${Self}.significandWidth") {
  expectTrue(-1, ${Self}.infinity.significandWidth)
  expectTrue(-1, ${Self}.nan.significandWidth)
  expectTrue(0, ${Self}(1).significandWidth)
  // ... some additional cases ...
}

Added *.significandWidth tests to test/stdlib/FloatingPoint.swift.gyb. I think this should be enough.

@swift-ci please test and merge.

@hooman Thanks for following up with the added tests.

@swift-ci please test.

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - d18514b77e03df7d40c179f350a02f96820b0eec
Test requested by - @stephentyrone

@hooman reading the logs, the Linux failure appears to be unrelated to your change (the same failure is present in some earlier unrelated test runs). I'll re-run the tests once the bots are green and we'll go from there.

OK, thank you.

@swift-ci please smoke test and merge.

@swift-ci Please test.

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - f03ab59d8f55a7614cb655cccf830b7f6d6ee845
Test requested by - @stephentyrone

@swift-ci Please smoke test.

@swift-ct Please smoke test Linux.

@swift-ci Please smoke test Linux.

Great! Thank you.


init command does not apply init migration on prod hasura

Maybe we should reset all migrations on prod when we do nhost init so the SOUCE STATUS and DATABASE STATUS are in sync.
https://hasura.io/blog/resetting-hasura-migrations/

All works now.


reset-password.js?

The mean 0.4.2 branch has a file under the scripts folder called reset-password.js.
https://github.com/meanjs/mean/blob/v0.4.2/scripts/reset-password.js
Is file absent by design? It does have mongoose dependency in mean.

I removed it as I was short on time and did not want to go to the effort of rewriting the script. I was hoping someone might make a PR at some point. It's somewhat "non-essential" however so not a huge priority. Would be nice to have though :)


Allow non-authenticated users to access cached requests / responses without authenticating

See server_service.py:
        """Actually make a request to an API."""
        # TODO: try to invoke the API even if we're not authenticated, and if
        #       it turns out the results are cached, then we can just hand back the results.

Closing because we currently have no plans to let external users access the crfm-models playground server.


should faild if the permission is not created when linkauth,but now it success.

void apply_eosio_linkauth(apply_context& context) {
//   context.require_write_lock( config::eosio_auth_scope );
auto requirement = context.act.data_as();
try {
EOS_ASSERT(!requirement.requirement.empty(), action_validate_exception, "Required permission cannot be empty");
  context.require_authorization(requirement.account); // only here to mark the single authority on this action as used

  auto& db = context.db;
  const auto *account = db.find<account_object, by_name>(requirement.account);
  EOS_ASSERT(account != nullptr, account_query_exception,
             "Failed to retrieve account: ${account}", ("account", requirement.account)); // Redundant?
  const auto *code = db.find<account_object, by_name>(requirement.code);
  EOS_ASSERT(code != nullptr, account_query_exception,
             "Failed to retrieve code for account: ${account}", ("account", requirement.code));
  if( requirement.requirement != config::eosio_any_name ) {
     **const auto *permission = db.find<permission_object, by_name>(requirement.requirement);**
     EOS_ASSERT(permission != nullptr, permission_query_exception,
                "Failed to retrieve permission: ${permission}", ("permission", requirement.requirement));
  }

this cause linkauth permission success when the requirement permission exists but the permission is not created by the account.

The objective fix to this bug is tracked in #6333. This issue is left to track a subjective mitigation that rejects eosio::linkauth if it links to a non-existing permission.

There is no real need for a subjective mitigation since this isn't a security issue and the proper fix has been implemented as part of the protocol feature introduced in #6831.


precise

Add option to use preciso for super precise mathematical calculations

Maybe have a precise "mode", so that when you turn on precise, everything has to be a string as input and output.

done


RubySpec tests not compatible with IEEE 754 round-nearest-to-even mode

Right now, MRI's sprintf uses IEEE 754 round-nearest-to-even mode (https://en.wikipedia.org/wiki/IEEE_floating_point#Roundings_to_nearest). We're trying to get Float#round to match that behavior (https://bugs.ruby-lang.org/issues/12548). It breaks RubySpec in a few places.
Looking at tests, some are clearly incompatible with that mode. For instance, from core/float/round_spec.rb around line 70:
it "returns rounded values for big values" do
    +2.5e20.round(-20).should   eql( +3 * 10 ** 20  )

This should round to (+2 * 10 ** 20) in that mode, not (+3 * 10 ** 20).
All four test failures below are similar -- they seem to depending on the rounding behavior of various types not following IEEE 754 round-nearest-to-even.
Here's Nobuyoshi Nakada's test output when using a local implementation of that mode:
1)
Float#round returns rounded values for big values FAILED
Expected<PHONE_NUMBER>00000000000
to have same value and type as<PHONE_NUMBER>00000000000

spec/rubyspec/core/float/round_spec.rb:70:in `block (2 levels) in <top (required)>'
spec/rubyspec/core/float/round_spec.rb:3:in `<top (required)>'

2)
Integer#round returns itself rounded if passed a negative value FAILED
Expected 200
 to have same value and type as 300

spec/rubyspec/core/integer/round_spec.rb:22:in `block (2 levels) in <top (required)>'
spec/rubyspec/core/integer/round_spec.rb:4:in `<top (required)>'

3)
Rational#round with no arguments (precision = 0) returns the truncated value toward the nearest integer FAILED
Expected 0
 to equal 1

spec/rubyspec/shared/rational/round.rb:17:in `block (3 levels) in <top (required)>'
spec/rubyspec/core/rational/round_spec.rb:3:in `<top (required)>'

4)
Time#strftime rounds an offset to the nearest second when formatting with %z FAILED
Expected "+01:01:04"
 to equal "+01:01:05"

spec/rubyspec/core/time/strftime_spec.rb:50:in `block (2 levels) in <top (required)>'
spec/rubyspec/core/time/strftime_spec.rb:7:in `<top (required)>'

I think those are mostly fine to be changed in ruby/spec for the next version, but the real decision has to happen on MRI's bug tracker whether this is bearable for compatibility.
I could imagine some Ruby software calling round and get surprising results by updating the Ruby version.
OTOH Float#to_s changed many times, but the compatibility did not look like a big concern (maybe because we still need printf-like formatting like "%.3f" % f to display floats correctly).

Matz says on the linked MRI bug that he believes we should change Float#round to match the behavior of sprintf. He hasn't expressed any versioning preference yet that I know of, though.

Right, that was my thinking this could get in 2.4 or later, but unlikely to be backported.
I would be happy to see a patch adapting the specs with ruby_version_is guards and testing the new behavior :smiley:

Opened PR #322 (https://github.com/ruby/spec/pull/322). Ruby 2.4.0dev hasn't integrated Nobu's patch yet, but I'll comment on the corresponding MRI bug that we have a PR for this.

Benoit says RubySpec PR #322 looks good. Just waiting for us to commit round-to-nearest-even mode to MRI so he can merge it. I've used Nobu's branch locally, including to create PR #322, and it works well for me. +1

Resolved with PR #322 merge.


webshop task : JVM exception occured

Faced the below error when I ran the webshop task. It seems the code is running in the docker (as  mentioned in issue24 ), can anyone pls help?
jnius.JavaException: JVM exception occurred: /root/workspace/src/tasks/webshop/web_agent_site/../search_engine/indexes does not exist or is not a directory. java.lang.IllegalArgumentException

I checked in the webshop docker:
(webshop) root@62bdd530bd59:/# cd /root/workspace/src/tasks/webshop
bash: cd: /root/workspace/src/tasks/webshop: No such file or directory

In another folder (root/webshop/search_engine), there are some relevant files:
(webshop) root@62bdd530bd59:~/webshop/search_engine# ls
convert_product_file_format.py  indexes_100   indexes_1k          resources      resources_100k  run_indexing.sh
indexes                         indexes_100k  lucene_searcher.py  resources_100  resources_1k

Here is the error information:
(agentbench) GP-TRT-2:~/AgentBench$ bash .assignments/2023-09-14-10-16-52.sh
Evaluating in docker localhost/task:webshop, Parameters: --task outputs/2023-09-14-10-16-52/llama2-7b/WebShop-dev/task.yaml --agent outputs/2023-09-14-10-16-52/llama2-7b/WebShop-dev/agent.yaml --output outputs/2023-09-14-10-16-52/llama2-7b/WebShop-dev
> [Warning] FastChat agent not available
{'module': 'src.tasks.WebShop', 'parameters': {'end': 280, 'name': 'WebShop-dev', 'num_envs': 3, 'start': 200, 'worker_limit': 3, 'workers': 6}}
{'module': 'src.agents.HTTPAgent', 'parameters': {'body': {'Key1': 'Value1', 'Key2': 'Value2'}, 'headers': {'Content-Type': 'application/json'}, 'name': 'llama2-7b', 'prompter': {'args': {'agent_role': 'assistant'}, 'name': 'role_content_dict'}, 'url': 'http://localhost:8000/v1/chat/completions'}}
[Evaluation] Loading Agent ...
[Evaluation] Successfully loaded Agent.
[Evaluation] Loading Task ...
> [Warning] OSInteraction task not available
> [Warning] ALFWorld task not available
> [Warning] DBBench task not available
Warning: Gym version v0.24.0 has a number of critical issues with `gym.make` such that the `reset` and `step` functions are called before returning the environment. It is recommend to downgrading to v0.23.1 or upgrading to v0.25.1
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/pkg_resources/__init__.py:121: DeprecationWarning: pkg_resources is deprecated as an API
  warnings.warn("pkg_resources is deprecated as an API", DeprecationWarning)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/pkg_resources/__init__.py:2870: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('mpl_toolkits')`.
Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
  declare_namespace(pkg)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/faiss/loader.py:28: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  if LooseVersion(numpy.__version__) >= "1.19":
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/setuptools/_distutils/version.py:345: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  other = LooseVersion(other)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/thefuzz/fuzz.py:11: UserWarning: Using slow pure-python SequenceMatcher. Install python-Levenshtein to remove this warning
  warnings.warn('Using slow pure-python SequenceMatcher. Install python-Levenshtein to remove this warning')
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/gym/envs/registration.py:516: UserWarning: WARN: Overriding environment WebAgentSiteEnv-v0
  logger.warn(f"Overriding environment {spec.id}")
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/gym/envs/registration.py:516: UserWarning: WARN: Overriding environment WebAgentTextEnv-v0
  logger.warn(f"Overriding environment {spec.id}")
> [Warning] Mind2Web task not available
> [Warning] KnowledgeGraph task not available
[Evaluation] Successfully loaded Task.
Evaluating task 'WebShop-dev' ...
Start Predicting All ...
  0%|                                                                                | 0/80 [00:00<?, ?it/s]> [Warning] FastChat agent not available
> [Warning] OSInteraction task not available
> [Warning] FastChat agent not available
> [Warning] ALFWorld task not available
> [Warning] DBBench task not available
> [Warning] FastChat agent not available
> [Warning] OSInteraction task not available
> [Warning] ALFWorld task not available
> [Warning] DBBench task not available
Warning: Gym version v0.24.0 has a number of critical issues with `gym.make` such that the `reset` and `step` functions are called before returning the environment. It is recommend to downgrading to v0.23.1 or upgrading to v0.25.1
> [Warning] OSInteraction task not available
> [Warning] ALFWorld task not available
> [Warning] DBBench task not available
Warning: Gym version v0.24.0 has a number of critical issues with `gym.make` such that the `reset` and `step` functions are called before returning the environment. It is recommend to downgrading to v0.23.1 or upgrading to v0.25.1
Warning: Gym version v0.24.0 has a number of critical issues with `gym.make` such that the `reset` and `step` functions are called before returning the environment. It is recommend to downgrading to v0.23.1 or upgrading to v0.25.1
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/pkg_resources/__init__.py:121: DeprecationWarning: pkg_resources is deprecated as an API
  warnings.warn("pkg_resources is deprecated as an API", DeprecationWarning)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/pkg_resources/__init__.py:2870: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('mpl_toolkits')`.
Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
  declare_namespace(pkg)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/pkg_resources/__init__.py:121: DeprecationWarning: pkg_resources is deprecated as an API
  warnings.warn("pkg_resources is deprecated as an API", DeprecationWarning)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/pkg_resources/__init__.py:2870: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('mpl_toolkits')`.
Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
  declare_namespace(pkg)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/pkg_resources/__init__.py:121: DeprecationWarning: pkg_resources is deprecated as an API
  warnings.warn("pkg_resources is deprecated as an API", DeprecationWarning)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/pkg_resources/__init__.py:2870: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('mpl_toolkits')`.
Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
  declare_namespace(pkg)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/faiss/loader.py:28: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  if LooseVersion(numpy.__version__) >= "1.19":
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/setuptools/_distutils/version.py:345: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  other = LooseVersion(other)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/faiss/loader.py:28: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  if LooseVersion(numpy.__version__) >= "1.19":
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/setuptools/_distutils/version.py:345: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  other = LooseVersion(other)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/faiss/loader.py:28: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  if LooseVersion(numpy.__version__) >= "1.19":
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/setuptools/_distutils/version.py:345: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  other = LooseVersion(other)
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/thefuzz/fuzz.py:11: UserWarning: Using slow pure-python SequenceMatcher. Install python-Levenshtein to remove this warning
  warnings.warn('Using slow pure-python SequenceMatcher. Install python-Levenshtein to remove this warning')
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/thefuzz/fuzz.py:11: UserWarning: Using slow pure-python SequenceMatcher. Install python-Levenshtein to remove this warning
  warnings.warn('Using slow pure-python SequenceMatcher. Install python-Levenshtein to remove this warning')
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/thefuzz/fuzz.py:11: UserWarning: Using slow pure-python SequenceMatcher. Install python-Levenshtein to remove this warning
  warnings.warn('Using slow pure-python SequenceMatcher. Install python-Levenshtein to remove this warning')
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/gym/envs/registration.py:516: UserWarning: WARN: Overriding environment WebAgentSiteEnv-v0
  logger.warn(f"Overriding environment {spec.id}")
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/gym/envs/registration.py:516: UserWarning: WARN: Overriding environment WebAgentTextEnv-v0
  logger.warn(f"Overriding environment {spec.id}")
> [Warning] Mind2Web task not available
> [Warning] KnowledgeGraph task not available
Products loaded.
Keys cleaned.
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/gym/envs/registration.py:516: UserWarning: WARN: Overriding environment WebAgentSiteEnv-v0
  logger.warn(f"Overriding environment {spec.id}")
/root/miniconda3/envs/webshop/lib/python3.8/site-packages/gym/envs/registration.py:516: UserWarning: WARN: Overriding environment WebAgentTextEnv-v0
  logger.warn(f"Overriding environment {spec.id}")
> [Warning] Mind2Web task not available
> [Warning] KnowledgeGraph task not available
Products loaded.
Keys cleaned.
Attributes loaded.
100%|███████████████████████████████████████████████████████████████████████████████████████████████████| 1000/1000 [00:00<00:00, 70730.25it/s]
Process SpawnProcess-1:
Traceback (most recent call last):
  File "/root/miniconda3/envs/webshop/lib/python3.8/site-packages/multiprocess/process.py", line 315, in _bootstrap
    self.run()
  File "/root/miniconda3/envs/webshop/lib/python3.8/site-packages/multiprocess/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/root/workspace/src/tasks/webshop_docker/__init__.py", line 38, in predict
    env = WebAgentTextEnv(observation_mode="text", human_goals=True)
  File "/root/workspace/src/tasks/webshop_docker/web_agent_site/envs/web_agent_text_env.py", line 61, in __init__
    self.server = SimServer(
  File "/root/workspace/src/tasks/webshop_docker/web_agent_site/envs/web_agent_text_env.py", line 299, in __init__
    self.search_engine = init_search_engine(num_products=num_products)
  File "/root/workspace/src/tasks/webshop/web_agent_site/engine/engine.py", line 206, in init_search_engine
    search_engine = LuceneSearcher(os.path.join(BASE_DIR, f'../search_engine/{indexes}'))
  File "/root/miniconda3/envs/webshop/lib/python3.8/site-packages/pyserini/search/lucene/_searcher.py", line 51, in __init__
    self.object = JLuceneSearcher(index_dir)
  File "jnius/jnius_export_class.pxi", line 270, in jnius.JavaClass.__init__
  File "jnius/jnius_export_class.pxi", line 384, in jnius.JavaClass.call_constructor
  File "jnius/jnius_utils.pxi", line 79, in jnius.check_exception
jnius.JavaException: JVM exception occurred: /root/workspace/src/tasks/webshop/web_agent_site/../search_engine/indexes does not exist or is not a directory. java.lang.IllegalArgumentException

The problem is discussed in (https://github.com/THUDM/AgentBench/issues/24#issue-1863168883) , close the issue


I have pandas-profiling installed, but running the task has no results

Describe the bug
A clear and concise description of what the bug is.
If the description consists of multiple non-related bugs, you are encouraged to create separate issues.
To Reproduce
We would need to reproduce your scenario before being able to resolve it.
Data:
Please share your dataframe.
If the data is confidential, for example when it contains company-sensitive information, provide us with a synthetic or open dataset that produces the same error.
You should provide the DataFrame structure, for example by reporting the output of df.info().
You can anonymize the column names if necessary.
Code: Preferably, use this code format:
"""
Test for issue XXX:
https://github.com/pandas-profiling/pandas-profiling/issues/XXX
"""
import pandas as pd
import pandas_profiling

def test_issueXXX():
    df = pd.read_csv(r'<file>')

    # Minimal reproducible code

Version information:
Version information is essential in reproducing and resolving bugs. Please report:

Python version: Your exact Python version.
Environment: Where do you run the code? Command line, IDE (PyCharm, Spyder, IDLE etc.), Jupyter Notebook (Colab or local)
pip: If you are using pip, run pip freeze in your environment and report the results. The list of packages can be rather long, you can use the snippet below to collapse the output.

Click to expand Version information

<<< Put your version information here >>>

Additional context
Add any other context about the problem here.

I have pandas-profiling installed, but running the task has no results

Could you provide the minimal information to reproduce this error?

the minimal code you are using to generate the report
which environment you are using (jupyter notebook, console or IDE) and packages (pip freeze > packages.txt)
a sample or description of the dataset (df.head(), df.info())

I encountered the same issue. It turned out to be caused by the lower version of jupyter_core that misses secure_write module.
I fixed it by updating the jupyter version:
pip install --upgrade jupyter_client

I encountered the same issue. It turned out to be caused by the lower version of jupyter_core that misses secure_write module.
I fixed it by updating the jupyter version:
pip install --upgrade jupyter_client

thank you very much.


Stick file extension on version numbers

xarray uses the file path to determine the read engine. It needs paths to end in '.nc'

This does not seem to be an issue. I'm marking as resolved for now.


feat: remove sismo-connect-solidity library dependencies

[x] rewrite SismoConnectVerifier tests
[x] rewrite HydraS3Verifier tests
[x] remove builder libraries dependencies in tests and deployment scripts (libraries are now deployed in sismo-core/sismo-connect-solidity)

In src/utils there is some AuthBuilder, ClaimBuilder libraries, fmt, etc. that contains a lot of "build" function. I don't think this is necessary as in the code we are always using the same build function.
I would also put them as an internal library instead of external one


C and C Headers have some unsafe code

Since the unsafe counter is naive and counts all C and C++ as unsafe it's strange that some code in .c and .h files is not unsafe. See the example in the README.md...

May be some problem with blank lines? Can you add examples ro repo?

I noticed it in the Rust repo (See the example output), but I haven't yet done any real investigating such as creating a quick .c file and seeing what happens.

It seems to be the same bug as #2.
int  test() {
	return /* test */ 0;
}

Result is
         Language  Files  Lines  Blanks  Comments  Code  Unsafe (%)
         --------  -----  -----  ------  --------  ----  ----------
         C Header  1      3      0       1         2     1 (50.00%)
         TOML      1      4      0       0         4
         --------  -----  -----  ------  --------  ----  ----------
Totals:            2      7      0       1         6     1 (16.67%)

Makes sense, and now I see why I didn't catch it before! Thanks! :smile:

That found some of the erroneous lines...but not all.

The only place, where count.code increments before unsafe check is
else if line.contains(ms) {
    debugln!("line contains a multi start");
    count.code += 1;
    is_in_comments = line.contains(count.multi_end().unwrap());
    debugln!("line also contained a multi end: {:?}", is_in_comments);
    continue;
    }

and there is continue

Let me fix it?

That was it! Sorry, I just pushed the patch!


How to set Rate Limit?

Details
Is there a way to set a rate limit like max 20 requests/minutes to avoid 429 Too Many requests on a prod site?

try: https://unlighthouse.dev/guide/recipes/improving-accuracy#reduce-parallel-scans

try: https://unlighthouse.dev/guide/recipes/improving-accuracy#reduce-parallel-scans

works. thanks


Have support in multiple channels

Each message published to the bus should have the channel name specified beforehand. If no channel was specified, this message will end up in the default channel. Maybe _default_channel is a good default.
Each subscriber subscribed to the bus should have the channel name specified. If no channel was specified, this subscriber will subscribe to the default channel.

In my opinion default_channel may no need, It will cause of confuse. Each channel should handle one type of data, each channel handle one business, actually one function to process data , eg: single responsibility principle .
Instead of default channel , could be throw exception.

Agree. A typical setup would look like this:
// Subscription
bus.Subscribe(
    subscriber: new Subscriber("a subscriber", message => { /* Consume the message */ })
    channel: "channel-name");

// Publishing
bus.Publish("a message", "channel-name");

For a small application, which uses a single channel, the name could be discarded for convenience. In this case, the channel defaults to _default_channel, which might also be configurable.
// Subscription
bus.Subscribe(new Subscriber("a subscriber", message => { /* Consume the message */ }));

// Publishing
bus.Publish("a message");

@badpaybad
Would you mind reviewing the above PR?


Can not generate nyc report after test

After updating to<EMAIL_ADDRESS>I can't generate report after testing.
<EMAIL_ADDRESS>works correctly.

node: v6.10.2
OS: windows 10
ava: v0.19.1

...
  "nyc": {
    "include": [
      "dist/src/**/*.js"
    ],
    "exclude": [
      "dist/src/public/**/*.js",
      "dist/**/*.spec.js"
    ],
    "extensions": [
      ".ts"
    ],
    "cache": true,
    "all": true,
    "check-coverage": false
  }
...

gulp.task("test", ["pretest"], () => {
    return gulp.src("dist/test/**/*.spec.js")
        .pipe(ava({
            verbose: true,
            failFast: false,
            powerAssert: true,
            nyc: true,
        }));
});

Maybe this problem relates to execa.


Telegraf Plugin - Ping?

Any thought on enabling Telegraf Ping plugin?  I know there are Linux permission particulars, but would be nice to have ping stats from the erlite-4 point of view. Would I be able to add another exec in the config, I see there is one in use already.
Thanks for the this!
https://github.com/influxdata/telegraf/blob/master/plugins/inputs/ping/README.md

The ping input should be already there. You can run $ telegraf --input-list to check this.


Include all Flixel classes without DCE

This PR makes it so game includes all unused Flixel classes without DCE
Includes 252 classes

Macro: just exists

Couldn't you just use a macro?
Also why do we need this?

I don't like how that's being done. I would rather just use macro than pushing lots of classes into an unused array at runtime.
See https://api.haxe.org/haxe/macro/Compiler.html#include

#12953 does this same thing but in a much better manner


Allow adding device name to track check-ins and sales

Is it a ?

[x] Feature Request

Feature Request
Feature Description
The EventBrite app has an option to add device name in Device Settings. The device name is used to track check-ins and sales. The Organizer App also needs to support the same.

Would you like to work on the issue?
Yes

First this needs to be supported by server

@iamareebjamal Right now we may only create the device name preference. This shared preference can then be used when attendee check-in and sales code could to be changed, i.e when it's implemented on the server.
I have created an issue on the server, and have requested suggestions to move forth.

OK


Vercel deploy broken

Environment
Vercel
Reproduction
use "@nuxtjs/i18n": "8.0.0-beta.11"
and deploy on vercel
Describe the bug
On i18n call /i18n/precompile => Error 500
vercel log => [nuxt] [request error] [unhandled] [500] ENOENT: no such file or directory, open '/vercel/path0/webapp/.nuxt/i18n/config-a7d47455.js'
I guess vercel can't create file after the build step, and i18n /precompile create the ./nuxt/i18n/config-....js after the build (when app is first loaded)
is there a way to precompile during the build or to allow vercel to create new build file ?
Additional context
https://github.com/nuxt-modules/i18n/commit/f7925d113a5c4ab3cadf5467c899d604f0d33214#r109373991
Logs

also link to this vercel issue : https://github.com/orgs/vercel/discussions/1396

Thank you for your reporting!
This issue is same as #2000
I'm working on this issue resolving.
please wait. 🙏
Thanks!


Patch 1.3.3

Description
Patch 1.3.3. Includes:
SCIP return solution if [total]nodelimit hit (#2279)
Update swig autogenerated files to swig 4.1.1 (#2273)
Update contributing docs to reflect usage of ruff over flake8/isort (#2302)
Change some O(n) lookups in scip_conif.py to O(1) (#2313)
Avoid sparse matrix multiply for identity matrix (#2315)
Fix gurobi interface issues (#2300)
Previously merged:
Fixing links to cvx_short_course (#2264)
These MRs were included in patch 1.4.2 but caused merge conflicts in this patch, so I did not include them:
Restore Mosek 9 compatibility (#2325)
Reformatting the installation page (#2268)
Updating NAG website links in doc (#2299)
Use new NumPy API (#2319)

Restore Mosek 9 compatibility (https://github.com/cvxpy/cvxpy/pull/2325)

This bug was never on 1.3.x

Reformatting the installation page (https://github.com/cvxpy/cvxpy/pull/2268)
Updating NAG website links in doc (https://github.com/cvxpy/cvxpy/pull/2299)

We don't deploy the 1.3.x docs so seems fine

Use new NumPy API (https://github.com/cvxpy/cvxpy/pull/2319)

It seems reasonable to assume if you're not updating CVXPY to the 1.4.x series, you're probably not updating NumPy to 2.x so this seems fine to me.


[BUG] Set-PnPSite with managed identity gives error: The remote server returned an error: (401) Unauthorized

Reporting an Issue or Missing Feature
Reporting an Issue
I try to run the "Set-PnPSite -LockState ReadOnly" commando with an managed identity. I got a 401 unauthorised error. See hyperlink below how I implement the managed identity with an principal identity. I give the managed identity "Sites.FullControl.All" permissions.
https://pnp.github.io/powershell/articles/azurefunctions.html#assigning-permissions-to-the-managed-identity
Expected behavior
Please describe what output you expect to see from the PnP PowerShell Cmdlets
I expect no errors. I expect the SharePoint site will have the lockstate "ReadOnly"
Actual behavior
Please describe what you see instead. Please provide samples of output or screenshots.
I see the error "The remote server returned an error: (401) Unauthorized" in the Azure function test console.
Steps to reproduce behavior
Please include complete script or code samples in-line or linked from gists
Follow this tutorial: https://pnp.github.io/powershell/articles/azurefunctions.html

Create an Azure function app
Create a function
Add PnP PowerShell module
Enable managed identity togle
Give managed identity the permissions "Sites.FullControl.All" to SharePointOnline
Add this script and do a test run:

Connect-PnPOnline "https://myCompany.sharepoint.com/sites/someTestSite" -ManagedIdentity
Set-PnPSite -LockState ReadOnly

What is the version of the Cmdlet module you are running?
(you can retrieve this by executing Get-Module -Name "PnP.PowerShell" -ListAvailable)
PnP PowerShell 2.2.0 and PowerShell 7
Which operating system/environment are you running PnP PowerShell on?

[X ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

I found the fix. You first need to connect to the SPO admin center site. After this you need to use the identity argument on the Set-PnPSite command to the target site collection:
Set-PnPSite -Identity $siteUrl -LockState ReadOnly


Issues when trying to login directly without a login button flow

Hi! I've tested dev branch and made a few changes related to #144 and my issue is that the app redirects back to B2C after log in on IE.
My flow tries to authenticate directly on my app without using any login button like some examples I've seen, but maybe is having some issues maybe because of a setTimeout on the constructor of msal.
I'm calling acquireTokenSilent at startup, but it seems is not working without wrapping it in a setTimeout.
I have an example that shows this behavior.
Thank you!

@joanjane The constructor of msal is written in an asynchronous manner. The code inside setTimeout (handleAuthenticationResponse) calls the callback defined by the user and hence the code flow switches control from the library to the app developer. If we do not put this code inside the setTimeout, then you will not have the instance of msal available inside the callback method. In your example, you call the   constructor function of msal on line 32 and since it is asynchronous, the call to acquireToken also goes out in parallel on line 48 which causes issues since login has not completed yet. If you remove the call to acquireToken on line 48 and run your sample. it should work as expected. You should only have a single call to acquireToken in your sample which is on line no 42 inside the callback.

@rohitnarula7176 The library don't execute the callback method when you're authenticated and you refresh the page, so removing line 48 on my initial sample implies that flow is not triggered in this case. Also, a login button is not what I need, I just simply want a redirection to login if the user is not authenticated, and stay on the website otherwise (which is a very common practice).
Finally, I've read a sample that uses msal isCallback method that fixes this case.
I've updated my gist and this seems to solve the issue.


FR: Multi tenancy support

Is there any work in progress or ETA for multi tenancy support like the Node.js SDK?
We really need this to take advantage of Cloud Identity Platform.
Thanks for your time.

This is close to a release. See the tenant-mgt branch.


Add Authentication Components and Logic

As a user
I need a secure way to register and log in
So that I can access personalized features
Details and Assumptions
* Authentication includes signup, login, and user sessions.

Acceptance Criteria
gherkin
Given a new user
When they register and log in
Then they can access personalized features securely

implementation phase


[Question]: How do I save my data when I plan to update the docker image and move the ragflow project?

Describe your problem
Hi, last week I tried to put the ragflow project on my PC and tried using it. I felt good and I want to move it to my server. I notice that the dokcer image version is not the latest. So, How can I save my existing data, and change the docker image, and move the ragflow project to the server computer? Thanks a lot.

The docker container didn't hold  any data.
If you want to migrate data, you could search about migrating docker volumes which held all the data.

The docker container didn't hold any data. If you want to migrate data, you could search about migrating docker volumes which held all the data.

Thanks very much. I found the volumes.


Get Latest Price Endpoint

Fetches the latest price based on current context Block Time for a market

Added in get latest price testing + get price tests

NOTE: One concern is whether there isn't a price written for a market based on the timestamp in ctx.BlockTime().Unix(). In the current code we just return found as false.


cop_codegen: replace 'parse + unwrap' with 'quote!'

Signed-off-by: Lonng<EMAIL_ADDRESS>
What have you changed? (mandatory)
cop_codegen: replace 'parse + unwrap' with 'quote!'.

Highlight the code
Remove the unwrap
Fix broken unit test

What are the type of the changes? (mandatory)
The currently defined types are listed below, please pick one of the types for this PR by removing the others:

Improvement (change which is an improvement to an existing feature)
Bug fix (change which fixes an issue)

How has this PR been tested? (mandatory)
Exists unit test.

/run-all-tests


Better handling of "New unread messages" tag/label

[ ]  In the topic narrow, I reached at bottom (verified from web https://chat.zulip.org/#narrow/stream/announce/topic/Streams) but still label was showing (I thinks it was showing for the homeView) but due to this composing new message becomes difficult.

[ ]  In the private chat narrow, after reaching at bottom it was showing label but text was not present in that.

@zulipbot claim

@vishwesh3 You can change the position prop for UnreadNotice component in Chat.js to top for narrows. The second one looks to me as an android specific issue.
The PR was quick one so there might be some issues still, thanks for reporting. A number of things are planned for the new messages indicator which I'll be working on after my exams :)

@vishwesh3 Have you pulled the latest master?
The initial release did not account for 'muted' streams and topics. I fixed that very recently.

I have downloaded from play store. ok will test on latest master

@vishwesh3 do you think this issue is still relevant?

@borisyankov we can close this. :)


Docs: use 1.3.0 version instead of 1.3.0-SNAPSHOT

Error occurred of not found when adding Morphia 1.3.0-SNAPSHOT

66666666

not sure what this is supposed to mean.  please post to the mailing list if you have questions.


Makes ESCDialog and ESCDialogOption instantiable without an ESC string

… string and other dialog optimizations

Thanks a lot!


TV Show poster is is blank

The poster on the discord RPC shows properly when I am watching a movie, but it doesn't show when I start watching a TV Show. It just shows the default poster. Also the buttons don't show up when watching a TV show too

where can i find the log to post?
I also i should let you know, i manually changed the base URL, in the globals.py file because kodi is running on a separate system within my lan

That should be fine.
I don't think it currently writes it to a file, this was a project when I was starting programming. Just set your logging level to DEBUG, run it in your terminal, start a TV show and post the output here.

Closing this as it's an issue with an addon that I sadly can't fix. This script assumes you have local Kodi files.


Switches the back and beta positions on the floor based ARVIR

Brings it in line with how having it work on a push instead of the current design for modals
#trivial

👍


Fix login bug

Fix #317 Switching back to CLI login after going to GUI once will fail

Coverage decreased (-0.05%) to 90.094% when pulling b2ac81caf1c40b085c1b4d5dca2472ec3b7bbcd0 on jelneo:general_bug_fixes into e3888bab4d63954de1a3cf62b297c8ec13534f09 on CS2103AUG2017-T17-B1:master.


Should BlobId type be aligned with attachments type in Observation schema?

Description
Currently running into a slight mismatch in types and not entirely sure where a change needs to be done:
BlobId, which is returned by our BlobApi when creating what are eventually observation attachments:
type BlobId = {
  type: 'photo' | 'audio' | 'video'
  variant: ... // omitted for brevity
  name: string
  driveId: string
}

Attachment, defined by @mapeo/schema in the Observation schema:
type Attachment = {
  type: 'photo' | 'audio' | 'video' | 'UNRECOGNIZED'
  hash: string
  name: string
  driveDiscoveryId: string
}

The primary mismatch is the type field for each. My guess is that @mapeo/core should be updated to align with schema

Thoughts:

type is correct as-is: an observation that is read from a hypercore could have an attachment type that is UNRECOGNIZED - it could have been created by a future version of CoMapeo and this version does not know about it yet. However the blob API should not allow us to create a blob of type UNRECOGNIZED, and should not allow reading it - it is the responsibility of the API consumer to ignore attachments that are UNRECOGNIZED (because we don't know how to display them - they could be any kind of file, e.g. sensor readings, 3-d models etc.
driveId for the BlobId should be changed to driveDiscoveryId (need to change the BlobAPI code to ensure the discovery key is used)
Having both hash and name feels redundant, since they are both unique identifiers. However thinking about it more, if in the future we allow users to attach files from outside Mapeo, it is possible that a user could attach the same file twice (e.g. same hash), but maybe we should just check for that and return the ID without writing a copy. That can probably be done in a later release if we support external attachments. So, TL;DR, I think we only need the hash. We can call it name and say it is the sha256 hash, or we can remove the name property and call it sha256 or hash. Any preferences? ... However, thinking about this more, keeping the name short (e.g. random 8-bytes) vs. the 32-byte sha does keep the size of the blobIndex core down (3 variants * 24 bytes less per index record, or 3 * 48 bytes since will be hex-encoded string = 144 bytes) at the expense of adding 8 extra bytes to the data core (in the attachment record) for the name, so maybe... keep them both. However reading a blob should not require the hash, but the returned value from creating a blob should include the hash. Clear as mud?
variant shouldn't be on the Attachment type, because the attachment refers to all variants.

driveId for the BlobId should be changed to driveDiscoveryId (need to change the BlobAPI code to ensure the discovery key is used)

Pretty sure this is already doing what you're suggesting, and it's just a matter of confusing naming
https://github.com/digidem/mapeo-core-next/blob/1f4abac31b84fcc5a73a9db2ef7f8fafbffeb3ba/src/blob-store/index.js#L74
https://github.com/digidem/mapeo-core-next/pull/294#pullrequestreview-1655015372 (lol)

Clear as mud?

i'm...not sure what i just read 😂

To summarize my jumbled comments above:

type - no change necessary. Front-end should handle UNRECOGNIZED on read, but it is not valid for writes.
BlobId.driveId is fine to leave for now, it is the discovery ID, just needs some renaming along with many other things for consistency at a later stage.
Keep both hash and name in the attachment record. Use name for lookup. hash is just used for verification. Other props for verification may be added to attachment records in the future.
variant is not on the attachment record because each attachment could have more than one variant. In the future we might want to populate this field at read-time to append the attachment variants that are actually available.
TL;DR no action required, closing.


Occasional pauses/frame drops during puzzles

During puzzle mode, I experience occasional bursts of 5-10 dropped frames. These are very sporadic and maybe only occur every 20 minutes, if that.
I think they may coincide with a texture being loaded, or a sound being played. Maybe we could track this down with the debugger, if we stress test the game by loading customers constantly.

Upon further investigation, these frame drops were actually pretty common but usually only resulted in 2-3 dropped frames. I tracked them down to loading the textures and shader properties, which took about 16 milliseconds. These properties are now loaded in a background process.

I've reverted c321eb48ed67cefb21415f8d82f97230532b0c81 to close this issue for now. The performance improvement which caused all of these bugs is branched at https://github.com/Poobslag/turbofat/tree/avoid-jitter-when-loading-creatures, and some attempted fixes are branched at https://github.com/Poobslag/turbofat/tree/eyes-partially-loaded-bug.

I've attempted a new fix which seems better. It focuses on shaders, and still loads the textures up front but updates the shaders gradually over time. The old fix is still available on 59e1f6e.


model text-davinci-003 has been deprecated

openai:debug {
error: {
message: 'The model text-davinci-003 has been deprecated, learn more here: https://platform.openai.com/docs/deprecations',
type: 'invalid_request_error',
param: null,
code: 'model_not_found'
}

same error...


Logo

Hi. I am a graphic designer. I would like to volunteer to design a logo for this repository that you can use in the readme file. What do you say?

That sounds great! Just keep in mind though that it may change in the future, and I might not be able to put it in the actual app itself. I'd like to see the logo though, if you're up for it.

I designed a logo like this for you. Is there anything you want to add ? Which one do you prefer ?

Falcons official font is Raleway, font weight 600. Can we see that?
Also, monochrome would be better, and not shaded. Bottom left looks great.

Lowercase everything, except for F. Try to make the logo the same height as the F. Then make the font bold too.
Looking great!

Of course

What do you think?

Great, but the font doesn't look quite like Raleway, and the font is too thin.

I forgot to thicken the font.

Are you sure the font is Raleway? This doesn't look like Raleway.

Can you really link to Raleway?

https://fonts.google.com/specimen/Raleway

I think it was.

Ah, that's better. If you could make it bolder, I'll place that in the readme.md.
Thanks for the help!

Merged. Thanks for the logo!


If you are hovering an highlighted object and it is deactivated, the highlight keeps there

Hiya,
if you are hovering an object

and it is deactivated, the highlight keeps there

I believe this has been fixed in the beta version of the plugin up here: https://github.com/ValveSoftware/steamvr_unity_plugin/releases/tag/2.1b2


Can't mass-assign protected attributes: key

Can't mass-assign protected attributes: key when creating a new feature with flipper ui and Flipper::Adapters::ActiveRecord. Adding attr_accessor :key to the adapter workaround the problem.

@TLmaK0 thanks, which version of rails are you on?

@AlexWheeler Rails 3.2.14

great, did you add attr_accessor :key or attr_accessible :key? attr_accessor :key attr_accessible is a rails method for allowing specific attributes to be mass assigned and would fix your issue.
The problem with adding it to the adapter is that attr_accessible has been taken out of rails versions > 4.0.  It looks like you'll also have issues with enabling gates via the UI since the adapter code uses mass assignment there too.
@jnunemaker curious on your opinion on this.
Changing the code to not use mass assignment and instead do something along the lines of
feature_class.create! do |f|   f.key = 'key'   etc. end
over feature_class.create!({ mass assigned attrs }) would make this work with all versions of active record or we could check the version at runtime
if Gem.loaded_specs['activerecord'].versiom > Gem::Version.new("4.0.0")
  attr_accessible(:key)
end

other thoughts?

@AlexWheeler
You are true. I used attr_accessible, sorry :(
And as you say, I have also problems with Gates (attributes :key, :value, :feature_key)
Thanks 👍

@AlexWheeler I should have written up my thoughts. Sorry. Yeah, I was thinking we just don't use mass assignment and then we can avoid the issue. Something like this:
instance = @gate_class.new
instance.feature_key = feature.key
instance.key = gate.key
instance.value = thing.value.to_s
instance.save!

Same thing for other gate and feature usages. I just hadn't whipped a PR together yet. If someone wanted to that would be great.

cool will get a PR up tonight, thanks


Large memory usage for LPSubscriptionMap

Hey there, on our live server (Paper) LuckPerms is using 400MB for a map and it's entries, something related to subscriptions. Is this a normal amount of usage?

Uhh no, that doesn't seem normal.
LPSubscriptionMap is just a replacement for a map already in the Bukkit PluginManager instance, with a few changes to make it faster, and work properly with the way LP handles subscriptions.
Soo, most likely another plugin doing something silly.
I've pushed this c0cd197bd0d3f336405f63ae7ceed6d3b6e84c29 change to ensure players can't get added (since LP handles those separately)


Add option to resolve SQS queue by ARN

Fixes #561
:loudspeaker: Type of change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[ ] Refactoring

:scroll: Description

This PR introduces the option to use the queue ARN to resolve an SQS queue when specifying an SQSListener. This enables the user to listen to queues from other accounts than the account used by the application/sqs client.
Limitation: As the used AmazonSQS client is region-specific, queues from other regions cannot be resolved. However, this can still be achieved using the queue url.
:bulb: Motivation and Context

See issue #561
Since it is common to access AWS resources via the ARN, this should also be possible for the SqsListener.
:green_heart: How did you test it?
I built the project locally and ran the unit tests. In addition, I used a small sample application to check locally that the ARN is resolved correctly when specified as value in the SqsListener annotation.
:pencil: Checklist

[x] I reviewed submitted code
[x] I added tests to verify changes
[x] I updated reference documentation to reflect the change
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

@tomazfernandes do you think its possible to include this feature also in 3.0 implementation?

Merged to 2.4.x. @tomazfernandes @techadri would be nice to port it to main branch too

Sure, @techadri feel free to open a PR if you’d like and thanks for the 2.x PR!
Otherwise I can look into it, but since I’m already a little bit behind schedule perhaps it’d be better to leave it to 3.1?

Also fine to leave it for 3.1

@tomazfernandes @maciejwalkowiak I will take a look and probably create a PR for version 3.x as well.


ENH: ImageRandomCoordinateSampler UseMultiThread use ITK's ThreadPool

Internally calling multiThreader.SingleMethodExecute(), instead of overriding ThreadedGenerateData.
When 10'000 random samples are retrieved, a significant performance improvement is observed by enabling UseMultiThread: from ~48 millisecond (single-threaded) to ~40 millisecond (multi-threaded). When 100'000 random samples are retrieved, even from ~123 to ~56 millisecond: Tested on a Windows 10 PC (6 cores, 12 logical processors), VS2019 Release build.
Follow-up to pull request https://github.com/SuperElastix/elastix/pull/973 commit ca9e9ecdf5a1bf7fca2df887feaf5c87365c5b81 "ENH: ImageRandomSampler UseMultiThread now using ITK's ThreadPool"

It should be noted that the original code (before this pull request) also used ITK's ThreadPool, by default. Only in a different way. Call stack original version:
CommonGTest.exe!itk::ImageRandomCoordinateSampler<itk::Image<int,2>>::ThreadedGenerateData(const itk::ImageRegion<2> & __formal, unsigned int threadId) Line 241	C++
CommonGTest.exe!itk::ImageToVectorContainerFilter<itk::Image<int,2>,itk::VectorDataContainer<unsigned __int64,itk::ImageSample<itk::Image<int,2>>>>::ThreaderCallback(void * arg) Line 284	C++
[External Code]
CommonGTest.exe!itk::ThreadPool::AddWork::__l3::<lambda>() Line 97	C++
[External Code]
CommonGTest.exe!itk::ThreadPool::ThreadExecute() Line 211	C++
[External Code]

Call stack with this pull request:
CommonGTest.exe!itk::ImageRandomCoordinateSampler<itk::Image<int,2>>::ThreaderCallback(void * arg) Line 174	C++
[External Code]
CommonGTest.exe!itk::ThreadPool::AddWork::__l3::<lambda>() Line 97	C++
[External Code]
CommonGTest.exe!itk::ThreadPool::ThreadExecute() Line 211	C++
[External Code]


'Microsoft.Subscription/aliases' Resource is not documented

Hi there,
The resource 'Microsoft.Subscription/aliases' can not be found in the ARM template reference under https://docs.microsoft.com/en-us/azure/templates/.
It would be very helpful when the resource is also described in the ARM template reference. Please add it there.
Thanks!

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

ID: bd3632da-89e0-cbd4-2d92-311eaacd33b3
Version Independent ID: 9a59177a-4183-cdd8-dee3-9dbeca53544b
Content: Programmatically create Azure subscriptions with the latest APIs
Content Source: articles/cost-management-billing/manage/programmatically-create-subscription.md
Service: cost-management-billing
Sub-service: billing
GitHub Login: @bandersmsft
Microsoft Alias: banders

@J0F3 Thanks for the feedback. We have this issue to the content owner to review further and update as appropriate.

Aliases is now available at https://docs.microsoft.com/en-us/azure/templates/microsoft.subscription/aliases

Aliases is now available at https://docs.microsoft.com/en-us/azure/templates/microsoft.subscription/aliases

#please-close

#please-close


Width argument not respected when set to large value

PrettyPrinter version: 0.18.0
Python version: 3.7.4
Operating System: MacOS 10.14.6

Description
I want to print a simple dict with long str values without line wrapping.
What I Did

Expected:
{
    'foo': 'a very long bar a very long bar a very long bar a very long bar a very long bar a very long bar ',
    # 'baz': 'if there were other keys, they would be here...',
}

Noticed this as well, looking at the pprint() definition, this might be working a intended:
   :param indent: number of spaces to add for each level of nesting.
   :param stream: the output stream, defaults to ``sys.stdout``
   :param width: a soft maximum allowed number of columns in the output,
                 which the layout algorithm attempts to stay under.
   :param depth: maximum depth to print nested structures
   :param ribbon_width: a soft maximum allowed number of columns in the output,
                        after indenting the line
   :param max_seq_len: a maximum sequence length that applies to subclasses of
                       lists, sets, frozensets, tuples and dicts. A trailing
                       comment that indicates the number of truncated elements.
                       Setting max_seq_len to ``None`` disables truncation.
   :param sort_dict_keys: a ``bool`` value indicating if dict keys should be
                          sorted in the output. Defaults to ``False``, in
                          which case the default order is used, which is the
                          insertion order in CPython 3.6+.


Add tighter bounds on AnyNamedTuple

AnyNamedTuple should have a Tuple lower bound and a Product upper-bound

How important is the change? AnyNamedTuple was intended as a marker "only named tuples can go here". It was not supposed to have any functionality beyond that.

Let's discuss this on the next Scala Core

How important is the change? AnyNamedTuple was intended as a marker "only named tuples can go here". It was not supposed to have any functionality beyond that.

I don't know how important it is, but it's a surprising element that NamedTuple is not a Product. Why shouldn't a method that accepts a Product argument not be able to use NamedTuple? Why should we be able to use Product internal methods like productIterator on NamedTuple? Regarding the Tuple lower bound, it could go with/without I guess.

It has been decided that named tuples will remain experimental in 3.6.x, so it should still be valid to make significant changes to the implementation (like the changes suggested in this PR).
Before we announce it as stable, we will likely make it a preview feature, as tracked in:

https://github.com/scala/scala3/issues/22042

@Gedochao why not do the same (preview) for the new given syntax?

@Gedochao why not do the same (preview) for the new given syntax?

What does preview even mean ?

What does preview even mean ?

@hamzaremmal Think of it as a stage 2, promoted experimental feature.
A middle step before announcing something as stable, promoting it to be tried out more, but still subject to changes.
The process will likely be announced along with Scala 3.7.x series.

@Gedochao why not do the same (preview) for the new given syntax?

@soronpo Feature previews were freshly decided on just yesterday, with named tuples being the only ones we know already will be shipped with it.
We haven't considered other features for it yet (and we don't have an implementation in place yet, either).

@Gedochao why not do the same (preview) for the new given syntax?

FYI after some discussions in the background, it seems we wouldn't do that. The new given syntax will likely ship as stable in 3.6.2.

See https://contributors.scala-lang.org/t/pre-sip-named-tuples/6403/234?u=odersky for a counter-argument why we would not want to do this.

See https://contributors.scala-lang.org/t/pre-sip-named-tuples/6403/234?u=odersky for a counter-argument why we would not want to do this.

That is a good argument, yet it is fixable, no? Create a private IArray with the product names that is injected at runtime for named tuples. It is just weird that named tuples are not Products.

That is a good argument, yet it is fixable, no? Create a private IArray with the product names that is injected at runtime for named tuples. It is just weird that named tuples are not Products.

That would be a big ask with potentially lots of code size overhead IMO. The next question how do to avoid multiple IArrays and then it goes down the drain. By comparison, I think its acceptable to force a _.toTuple if you want a product.


Java EWS - ResolveName not retrieving all email ids from Outlook

I am trying to retrieve list of users from Outlook using the Outlook API Exchange Webservices in a java stand alone application. To retrieve the list, I am using "resolveName". I find that sometimes when outlook returns multiple email ids for a name, the list that resolveName returns does not contain the last address. This does not happen with all the names that have multiple email ids returned by outlook.
Example - if the name "John Test" returns 5 emails ids when manually searched for in Outlook, the method resolveName returns only 4 addresses.
Does anybody have an idea of what might be happening here, or if I should use some method other than resolveName?

I'm a bit confused about Outlook API. Could you show piece of code and raw EWS request and response for ResolveName request?

The code that I use is this -
ExchangeService exchangeService = new ExchangeService(ExchangeVersion.Exchange2010_SP2);
ExchangeCredentials credentials = new WebCredentials(username, password, domain);
exchangeService.setCredentials(credentials);
exchangeService.setUrl(URI.create("url to exchange"));
NameResolutionCollection nameResults = exchangeService.resolveName("FirstName,LastName");
for (NameResolution name : nameResults) {
emailAddr = name.getMailbox().getAddress();
}
When I manually lookup "FirstName,LastName" in outlook, I get 5 matching email addresses. But the above code snippet returns only 4 email addresses.

It's quite interesting. Do you see any differences between contacts (contact that isn't found and contacts that are found)?


Vertical Alignment (Ionicons)

The vertical alignment of Ionicons seems to be off in the latest release.

Seems to happen on most icons. Not just one shown.

@alexcurtis Did you find any solution to this?

@alexcurtis @DVassilev I have the same issue, did you find any solution? The issue only appears for Ionicons.

Anyone find a solution for this? I'm having the same issue with Ionicons. As a workaround I've been manually applying paddingTop rules to the icons to get them to be vertically aligned, but the amount of paddingTop required varies for different icons and sizes so this is very tedious.

Have you guys experimented with lineHeight?

lineHeight doesnt help. we should reopen this.

+1

+1 on this.

@oblador can we reopen this? @alexcurtis looks gone.

I think the issue still exists in iOS and here is my workaround. I modified IonicIcon.ttf file; instructions are here: https://medium.com/@martin_adamko/consistent-font-line-height-rendering-42068cc2957d
I just changed lineGap value to 0 in the font file, and now alignment seems to be correct on both platforms in my app.
Before:

After:

@sampsasaarela just to clarify, you modified the Ionicons.ttf file located in /node_modules/react-native-vector-icons/Fonts/Ionicons.ttf? Meaning that any other developers working on the project would have to make the same modification to the file on their local machine, and also that you'd have to make this modification again every time you remove your node_modules folder and reinstall the project's dependencies?
Also, when following those instructions, do you simply end up with a modified Ionicons.ttf file, meaning that you can just copy this modified file somewhere and place it in the /node_modules/react-native-vector-icons/Fonts/ directory whenever you need to (e.g. the scenarios above)? Or do those instructions also result in other changes, e.g. to the Xcode project's configuration? If it's the former, could you upload the modified Ionicons.ttf file somewhere for people to be able to use?

@jordanmkoncz yes that is the case.
Currently, I have postinstall script in my project which will fix the alignment everytime developer run npm install. Of course, this is not the optimal solution but works for us for now.
Here is the script:
FONT_PATH=./node_modules/react-native-vector-icons/Fonts
FONT_FILE=$(FONT_PATH)/Ionicons.ttf
XML_FILE=$(FONT_PATH)/Ionicons.hhea.xml
ftxdumperfuser -t hhea -A d $(FONT_FILE) && sed -i '' 's/lineGap="46"/lineGap="0"/' $(XML_FILE) && sed -i '' 's/ascender="448"/ascender="471"/' $(XML_FILE) && ftxdumperfuser -t hhea -A f $(FONT_FILE)

Any reason why this is closed? I'm still experiencing this - on Android too.

Thanks @sampsasaarela, that's very helpful!
@gregblass For some reason @alexcurtis closed the issue and seems to be no longer looking at it, so I created a new issue at https://github.com/oblador/react-native-vector-icons/issues/566 which will hopefully be noticed by one of the maintainers.

Could it be reopened by some repository contributor? @oblador


Fix issues preventing dev version from conflicting with production build

When running a production build, intended to be a fulltime browser, there are issues when also developing Tofino. Some off the top of my head:

Running tests opens new tabs in production build
Running dev version when production build is running sometimes(?) doesn't start the UA service

#1385

Running dev version when production build is running sometimes(?) doesn't start the UA service

I very much expect this is just about port overlapping.  What might happen (should happen?) is that we observe an existing UAS and connect to it, thereby blurring the lines yet further.  I've filed https://github.com/mozilla/tofino/issues/1419 to talk about a possible solution.


Storie Documentation without component

When a Storie does not have a declared component, the auto-generated documentation does not work and displays this error message.
This case is an just an example. Some Stories don't need component.
icon.stories.js
export default {
  title: "Styles/icon",
  argTypes: {
    size: {
      control: {
        type: "select"
      },
      options: [
        "xs",
        "sm",
        "lg",
      ]
    },
    rotate: {
      control: {
        type: "select"
      },
      options: ["0", "90", "180", "270"]
    }
  }
};

TypeError: Cannot read properties of undefined (reading 'name')

Specs
"storybook": "^7.4.6",
"storybook-addon-vue-slots": "^0.9.21",

Thank you for all your hard work :pray:
I'll try to get some time to fix this.

The same error occurs when trying to write custom mdx docs to component


Update README.md's repository structure and status for upcoming 1.0 release.

Update https://github.com/grpc/grpc#repository-structure--status for the upcoming 1.0 release..
Include a column listing supported versions for each language.

@zhongql Please pay close attention to this release.


Is this normal? (See pic)

Aside from the project not being maintained, I've seen this:

That doesn't seem right... It is a partner, or someone had that slip in though a PR?

I let the domain alm.tools expire as https://alm-tools.github.io/ is free and good enough as a snapshot.
Sadly someone decided to buy it and use it for spam 😢. Just one more reason that I don't buy domains for OSS projects anymore :rose:

Then you should at least change the website link in the README to https://alm-tools.github.io/ so people do not get scammed.

Done :rose:


Added translit for Russian characters

Hi @maxatwork , sorry completely missed this PR! Thanks for the fix