Changing only part of date (input) does not change form's pristine state (Chrome)

Have a look at this plnkr.
When (in Chrome) I click the year and type something - say 2015 - and blur the field, model value does not change.

However, when I select the date from calendar and then I delete the year value (select it and hit backspace), then field & form error are set as expected:

Not sure if Angular's or Chrome's issue as in Firefox it is working fine (although there is no date picker).

I did a little looking into here, it looks like if you type all of the date values (mm/dd/yyyy, say 03/14/2015), validation does occur.
This may have to do with how partial date entry is validated - I suspect it has to do with either the fact that it is an input of type date.

I've noted this issue also. Looking into angular.js I notices that validators are not trigger when a part of the date is changed or cleared.
I've als noticed that clearing the complete date (by using backspace in each part) does not trigger any validator.

As explained in https://github.com/angular/angular.js/issues/11622#issuecomment-109571015, I think this is due to the fact that an invalid or partial date sets the input's value to an empty string.
(Further edits on the already invalid date, will not have an effect on the input's value (it will remain empty), until a valid date is picked.)
It is related to how the browser does/doesn't emit input events upon partial input.
Responding to keydown on date inputs (as suggested by @Narretz), sounds like a viable solution.

A quick POV showed that the problem is not only the lack of firing input events.
Off the top of my head, another part of the problem must be the fact that once the input is an empty string, further (invalid) edits do not impose a change in the value ($viewValue), thus no validation is triggered (this.just a speculation though).

@gkalpak  There's a condition inside $commitViewValue that forces validation etc. when the input has a native validator, even if the viewValue is empty: https://github.com/angular/angular.js/blob/master/src/ng/directive/ngModel.js#L652
The problem is that there's no change event in the following cases, because the browser does not register a change in the input's value (as gkalpak has pointed out)

when adding data into the input, the change event is only fired after all field (day, month, year) have been filled.
when removing data from the input, the change event is fired as soon as the date becomes invalid, but not when all data is cleared.

I think

@Narretz, I thought there was another issue, because my original POC (see https://github.com/angular/angular.js/issues/12207#issuecomment-137675456) didn't work. It turned out to be a timing issue.
So, triggering input on keyup, seems to solve the issue.
The real problem is more related to how browsers fire input events on date inputs though (as described previously), so Angular might have to work around that.
@Narretz, wdyt about imulating input on keyup (or something) for date-family inputs ?

@gkalpak Yeah, let's give it a shot. Do you want to open a PR? And I wonder if we should restrict it to Chrome, because FF and IE don't have datepickers (at least on desktop)

@Narretz, sure, PR is on the way.
Good idea restricting it to browsers that support input[type="date/..."] (currently Chrome and Edge - hopefully more in the future). Date inputs report type === 'text' in non-supporting browsers.

@gkalpak to add to your comment of 11 Sep 2015 you said:
when adding data into the input, the change event is only fired after all field (day, month, year) have been filled.
This is partially true, it depends which keys you use to edit the date. If I type the following (wrong) date in one flow then no validator is triggered: 31112015. Even if I leave the field it is still not triggered. When I return to the field and change the year it is not triggered. When I change the day or month the it gets striggered.

@TimoPot, indeed, I was referring to filling the fields with valid data. If not, the input remains empty (aka unchanged).

Ah missed that. Well actually the input is not empty after all I've just entered a value ;-)
am I correct to notice that this issue is not solved yet?

Also manually invoke $validate() or $setTouched() or $setDirty() on the date field does not help to get the date validated. Is there a known workaround?

@TimoPot The problem is that for Angular the input is actually empty, because the browser does not set the value property of the input when invalid data is entered (for date and number).
I think we need to listen to key events in addition to the input / change event. I believe @gkalpak worked on a PR for this, we should try to get it in after 1.5.

The workaround I've used (and is more or less what is included in the PR) is to trigger an input event, whenever a keyup event is fired on date fields. [Demo pen]
I'll resume work on the PR, so we can get this fixed soon.

Oke. I assume the Demo pen will follow.
Thanks.

@TimoPot, btw, it's not a proper solution, because one needs to take into account keys that do not change the input (e.g. modifiers etc) and possibly other stuff. But it's good enough for a quick workaround.


i3en instances are in the website but not in the json file

https://www.ec2instances.info/instances.json doesn't contain i3en instances. could you please update it?

Thanks!


Add workaround for Script Console background color

The Script Console has the same low contrast as the build console (#62). With syntax colouring it's impossible to read.
@Jorg3Lucas this should be integrated into jenkins-core-themes eventually.

@Jorg3Lucas conflicts resolved.


Corvus installs successfully, but run-corvus fails with "configparser.NoOptionError: No option 'phsf' in section: 'Executables'"

I've been trying unsuccessfully to install Feff10 & corvus in a singularity apptainer under Ubuntu 20.04 (as well as on my base system under Ubuntu 22.04) for several days now. In the apptainer, I'm able to build Feff10 no problem, but I can't get Corvus-1.0.9 to work. I am following the installation instructions [here](file:///home/joseph/Downloads/Corvus_Installation_Instructions-2.pdf). Using miniconda3, I do conda activate, conda install --channel conda-forge pymatgen, and python -m pip install . according to the instructions in 'install.sh' and I get
Successfully built corvus
Installing collected packages: corvus
  Attempting uninstall: corvus
    Found existing installation: corvus 1.0.9
    Uninstalling corvus-1.0.9:
      Successfully uninstalled corvus-1.0.9
Successfully installed corvus-1.0.9

but then run-corvus leads to ModuleNotFoundError: No module named 'imp' (https://github.com/times-software/Corvus/issues/1). This is because miniconda3 comes with python 3.12 by default, so using the miniconda3 base environment according to the instructions in 'install.sh' doesn't work, since python >3.12 no longer has the imp module that's imported in controls.py. However, even if I create a conda env with python 3.9, in which I can also successfully install corvus-1.0.9, I get further python errors on running run-corvus:
Traceback (most recent call last):
  File "configparser.py", line 767, in get
    value = d[option]
            ~^^^^^^^^
  File "__init__.py", line 1014, in __getitem__
    return self.__missing__(key)            # support subclasses that define __missing__
           ^^^^^^^^^^^^^^^^^^^^^
  File "__init__.py", line 1006, in __missing__
    raise KeyError(key)
KeyError: 'phsf'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "run-corvus", line 8, in <module>
    sys.exit(oneshot())
             ^^^^^^^^^
  File "controls.py", line 669, in oneshot
    configure(config)
  File "controls.py", line 275, in configure
    path2code = rcp.get('Executables', code)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "configparser.py", line 770, in get
    raise NoOptionError(option, section)
configparser.NoOptionError: No option 'phsf' in section: 'Executables'

These are the same errors I get if I, e.g., comment out import imp in controls.py and I'm not sure where to go from here. I'm pretty sure I'm doing everything as right as possible during the installation process. I'm using a clean system inside a singularity apptainer. I'm installing all the dependencies. Not sure what I'm doing wrong. I should note that I'm installing the other dependencies for corvus-1.0.9 via pip as follows:
pip install ipython-genutils ipykernel ipython traitlets lmfit pymatgen
as it seems like only pymatgen fails without conda. Pymatgen can be installed with pip, but it's an older version that corvus-1.0.9 doesn't accept. You can upgrade the pip-installed pymatgen like pip install --upgrade pymatgen and it prints that it's been upgraded to the needed version, but the corvus installer still claims that it's the wrong version. Following instead conda install --channel conda-forge pymatgen. the corvus installer is happy, but it clearly hasn't worked as run-corvus still fails.

Hi Joseph,
How are you running corvus? Typing run-corvus alone will not produce
anything but an error (corvus specific). Are you running one of the example
input files? If so, only some of them will work, because many of them
require other scientific software beyond FEFF10.
Cheers,
Josh
On Fri, Oct 4, 2024 at 2:21 PM josephzsombor @.***>
wrote:

I've been trying unsuccessfully to install Feff10 & corvus in a
singularity apptainer under Ubuntu 20.04 (as well as on my base system
under Ubuntu 22.04) for several days now. In the apptainer, I'm able to
build Feff10 no problem, but I can't get Corvus-1.0.9 to work. I am
following the installation instructions here. Using miniconda3, I do 'conda
activate', 'conda install --channel conda-forge pymatgen', and 'python -m
pip install .' according to the instructions in 'install.sh' and I get
Successfully built corvus Installing collected packages: corvus Attempting
uninstall: corvus Found existing installation: corvus 1.0.9 Uninstalling
corvus-1.0.9: Successfully uninstalled corvus-1.0.9 Successfully installed
corvus-1.0.9
but then run-corvus leads to "ModuleNotFoundError: No module named 'imp'" (
#1
https://urldefense.com/v3/__https://github.com/times-software/Corvus/issues/1__;!!K-Hz7m0Vt54!hVE3uM9FzWq--Y6NHZDOYC4CcuUiduVAtN-bXfbrdFKt_nM_3mhou6NkGucTzj7Kjk72g_dnnq7ONKL9iUYd3Q$).
This is because miniconda3 comes with python 3.12 by default, so using the
miniconda3 base environment according to the instructions in 'install.sh'
doesn't work, since python >3.12 no longer has the imp module that's
imported in controls.py. However, even if I create a conda env with
python 3.9, in which I can also successfully install corvus-1.0.9, I get
further python errors on running run-corvus:
`Traceback (most recent call last):
File
"/home/joseph/Documents/programs/chemistry/Feff/ubuntu-focal-feff/home/miniconda3/lib/python3.12/configparser.py",
line 767, in get
value = d[option]
~^^^^^^^^
File
"/home/joseph/Documents/programs/chemistry/Feff/ubuntu-focal-feff/home/miniconda3/lib/python3.12/collections/
init.py", line 1014, in getitem
return self.missing(key) # support subclasses that define missing
^^^^^^^^^^^^^^^^^^^^^
File
"/home/joseph/Documents/programs/chemistry/Feff/ubuntu-focal-feff/home/miniconda3/lib/python3.12/collections/
init.py", line 1006, in missing
raise KeyError(key)
KeyError: 'phsf'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File
"/home/joseph/Documents/programs/chemistry/Feff/ubuntu-focal-feff/home/miniconda3/bin/run-corvus",
line 8, in
sys.exit(oneshot())
^^^^^^^^^
File
"/home/joseph/Documents/programs/chemistry/Feff/ubuntu-focal-feff/home/miniconda3/lib/python3.12/site-packages/corvus/controls.py",
line 669, in oneshot
configure(config)
File
"/home/joseph/Documents/programs/chemistry/Feff/ubuntu-focal-feff/home/miniconda3/lib/python3.12/site-packages/corvus/controls.py",
line 275, in configure
path2code = rcp.get('Executables', code)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File
"/home/joseph/Documents/programs/chemistry/Feff/ubuntu-focal-feff/home/miniconda3/lib/python3.12/configparser.py",
line 770, in get
raise NoOptionError(option, section)
configparser.NoOptionError: No option 'phsf' in section: 'Executables'
These are the same errors I get if I, e.g., comment outimport impin
controls.py and I'm not sure where to go from here. I'm pretty sure I'm
doing everything as right as possible during the installation process. I'm
using a clean system inside a singularity apptainer. I'm installing all the
dependencies. Not sure what I'm doing wrong. I should note that I'm
installing the other dependencies for corvus-1.0.9 via pip as follows:pip
install ipython-genutils ipykernel ipython traitlets lmfit pymatgenas it
seems like only pymatgen fails without conda. Pymatgen can be installed
with pip, but it's an older version that corvus-1.0.9 doesn't accept. You
can upgrade the pip-installed pymatgen likepip install --upgrade pymatgen
and it prints that it's been upgraded to the needed version, but the corvus
installer still claims that it's the wrong version. Following instead
'conda install --channel conda-forge pymatgen. the corvus installer is
happy, but it clearly hasn't worked as run-corvus still fails.
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/times-software/Corvus/issues/2__;!!K-Hz7m0Vt54!hVE3uM9FzWq--Y6NHZDOYC4CcuUiduVAtN-bXfbrdFKt_nM_3mhou6NkGucTzj7Kjk72g_dnnq7ONKIPTSyLNA$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AA2RF2ZSEYENQMNTHOTQJK3ZZ4BG5AVCNFSM6AAAAABPMUAJ4WVHI2DSMVQWIX3LMV43ASLTON2WKOZSGU3DONBSGY2TMNY__;!!K-Hz7m0Vt54!hVE3uM9FzWq--Y6NHZDOYC4CcuUiduVAtN-bXfbrdFKt_nM_3mhou6NkGucTzj7Kjk72g_dnnq7ONKKGDRGhqg$
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Joshua Kas (He/Him/His)
Research Assistant Professor
University of Washington

Hi @jjkas
I am just following the guide here: file:///home/joseph/Downloads/Corvus_Installation_Instructions-2.pdf
It says running run-corvus with no argument should just print Provide target properties or Workflow.. I also tried navigating to examples/Feff_XANES/GeCl4 and running run-corvus -i GeCl4.in to no avail. It also printed the same KeyError: 'phsf'

OK. It looks like the configuration file is messed up. Can you try the
following?

Remove the corvus.conf file.

rm ~/.Corvus/corvus.conf

Run corvus again

run-corvus
The output should look as follows:
Configuration file not found. Will search for feff10.
System: Darwin
Checking path: /Users/jjkas/
jfeff10.app/Contents/Resources/JFEFF/feff10/mac
Writing corvus.conf
Provide target properties or Workflow

Now you should be able to run corvus again and get the normal output.

run-corvus
Provide target properties or Workflow
On Fri, Oct 4, 2024 at 4:17 PM josephzsombor @.***>
wrote:

Hi @jjkas
https://urldefense.com/v3/__https://github.com/jjkas__;!!K-Hz7m0Vt54!i5RnfoPAUnrHkPnS_ok39u-lJawPZfV_PV3MiZxcX9SZ9-ahEm9_2YSQZOp1Ivb7jZCJYNW3KovjKJiOa7BAxw$
I am just following the guide here:
file:///home/joseph/Downloads/Corvus_Installation_Instructions-2.pdf
It says running run-corvus with no argument should just print Provide
target properties or Workflow.. I also tried navigating to
examples/Feff_XANES/GeCl4 and running run-corvus -i GeCl4.in to no avail.
It also printed the same KeyError: 'phsf'
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/times-software/Corvus/issues/2*issuecomment-2394779892__;Iw!!K-Hz7m0Vt54!i5RnfoPAUnrHkPnS_ok39u-lJawPZfV_PV3MiZxcX9SZ9-ahEm9_2YSQZOp1Ivb7jZCJYNW3KovjKJhgnc70JA$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AA2RF25RVAD6EA3PHHG4ZADZZ4OZBAVCNFSM6AAAAABPMUAJ4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJUG43TSOBZGI__;!!K-Hz7m0Vt54!i5RnfoPAUnrHkPnS_ok39u-lJawPZfV_PV3MiZxcX9SZ9-ahEm9_2YSQZOp1Ivb7jZCJYNW3KovjKJgxDwYWlA$
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Joshua Kas (He/Him/His)
Research Assistant Professor
University of Washington

Hi @jjkas,
I tried deleting corvus.conf, but run-corvus just prints the same error: configparser.NoOptionError: No option 'phsf' in section: 'Executables'. Maybe I didn't install it properly somehow?

Not sure why it's doing that. I get that error if the configuration file is
messed up. What is in your configuration file? Can you cat it and send the
output to me?
cat ~/.Corvus/corvus.conf
On Fri, Oct 4, 2024 at 7:51 PM josephzsombor @.***>
wrote:

Hi @jjkas
https://urldefense.com/v3/__https://github.com/jjkas__;!!K-Hz7m0Vt54!hHvDBJgrDkOxY3gqcXmWVlSzaffYltjBgDzQrUTeDr6KJm8xMctRqcQnO2iH_G0k5AtAfWm4Lvv4D-hzu1hALg$
,
I tried deleting corvus.conf, but run-corvus just prints the same error: configparser.NoOptionError:
No option 'phsf' in section: 'Executables'. Maybe I didn't install it
properly somehow?
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/times-software/Corvus/issues/2*issuecomment-2394873641__;Iw!!K-Hz7m0Vt54!hHvDBJgrDkOxY3gqcXmWVlSzaffYltjBgDzQrUTeDr6KJm8xMctRqcQnO2iH_G0k5AtAfWm4Lvv4D-gf6loYNw$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AA2RF26Y2E5DJAC7NK3ZXOTZZ5HYHAVCNFSM6AAAAABPMUAJ4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJUHA3TGNRUGE__;!!K-Hz7m0Vt54!hHvDBJgrDkOxY3gqcXmWVlSzaffYltjBgDzQrUTeDr6KJm8xMctRqcQnO2iH_G0k5AtAfWm4Lvv4D-hznEsobg$
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Joshua Kas (He/Him/His)
Research Assistant Professor
University of Washington

I just copied corvus.conf.template and edited in the directory where feff installed its executables:
[Executables]
dmdw     : <path to dmdw dir>
feff     : /home/joseph/Documents/programs/chemistry/Feff/ubuntu-focal-feff/home/feff10/bin/MPI
abinit   : <path to abinit executables dir>
nwchem   : <path to nwchem executables dir>
orca     : <path to orca executables dir>
gaussian : <path to gaussian executables dir>
vasp     : <path to vasp executables dir>
siesta   : <path to siesta executables dir>
ocean    : <path to ocean executables dir>
cif2cell : <path to cif2cell executables dir>

[Defaults]
prefix      : Corvus
inputsuffix : .inp
savesuffix  : .nest
checkpoints : off
parallelrun :

Yes. This is the problem. If corvus does not find corvus.conf either in the
installation directory or ~/.Corvus/
then it will produce one. Can you try two things?

Make sure that there is no corvus.conf file anywhere. Remove the
corvus.conf file in ~/.Corvus/ and also in the installation folder if it
exists.

Now run-corvus again.

If this doesn't work, edit the corvus.conf so that it looks as follows:

[Executables]
dmdw     :
feff     : /home/joseph/Documents/programs/chemistry/Feff/ubuntu-focal-feff/home/feff10/bin/MPI
abinit   :
nwchem   :
orca     :
gaussian :
vasp     :
siesta   :
ocean    :
cif2cell :
phsf     :
[Defaults]
prefix      : Corvus
inputsuffix : .inp
savesuffix  : .nest
checkpoints : off
parallelrun :
The addition of the empty phsf : should fix everything, but let me
know if it doesn't.
Cheers,
Josh
On Fri, Oct 4, 2024 at 7:58 PM josephzsombor @.***>
wrote:

I just copied corvus.conf.template and edited in the directory where feff
installed its executables:
[Executables]
dmdw     :
feff     : /home/joseph/Documents/programs/chemistry/Feff/ubuntu-focal-feff/home/feff10/bin/MPI
abinit   :
nwchem   :
orca     :
gaussian :
vasp     :
siesta   :
ocean    :
cif2cell :
[Defaults]
prefix      : Corvus
inputsuffix : .inp
savesuffix  : .nest
checkpoints : off
parallelrun :
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/times-software/Corvus/issues/2*issuecomment-2394875424__;Iw!!K-Hz7m0Vt54!jllLmZcY5vCKehkqphMfK-SnVHjVqStssXPtEIgI5v3YsPWdRqFIE6O6S54fdbB9vwdGvl1f3NCaZ5d0EDheuw$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AA2RF2ZFBDP5SEDEBWGUCX3ZZ5IU3AVCNFSM6AAAAABPMUAJ4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJUHA3TKNBSGQ__;!!K-Hz7m0Vt54!jllLmZcY5vCKehkqphMfK-SnVHjVqStssXPtEIgI5v3YsPWdRqFIE6O6S54fdbB9vwdGvl1f3NCaZ5fWBPcx8g$
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Joshua Kas (He/Him/His)
Research Assistant Professor
University of Washington

Hi Josh,
thanks for all your help! I'm afraid mayble my install is all squirrelly somehow. Where is corvus supposed to put its executables after it's installed? I'm just installing it by navigating to the installation directory and running python -m pip install . I don't see a hidden directory anywhere, and when I do run-corvus, it doesn't produce a new corvus.config file. All that said, I did edit that line into the file and try run-corvus again. Now, I get this new error:
    collections.MutableMapping.register(ParseResults)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'collections' has no attribute 'MutableMapping'

OK. Edit the file corvutils/pyparsing.py
and change the line
import collections
to
import collections.abc as collections
That should fix the problem.
By the way, what version of python are you using now? This version of
corvus should work with python 3.9x, but just to check ...
We're getting closer;)
Cheers,
Josh
Joshua Kas (He/Him/His)
Research Assistant Professor
University of Washington
On Fri, Oct 4, 2024 at 8:21 PM josephzsombor @.***>
wrote:

Hi Josh,
thanks for all your help! I'm afraid mayble my install is all squirrelly
somehow. Where is corvus supposed to put its executables after it's
installed? I'm just installing it by navigating to the installation
directory and running python -m pip install . I don't see a hidden
directory anywhere, and when I do run-corvus, it doesn't produce a new
corvus.config file. All that said, I did edit that line into the file and
try run-corvus again. Now, I get this new error:
collections.MutableMapping.register(ParseResults)
^^^^^^^^^^^^^^^^^^^^^^^^^^

AttributeError: module 'collections' has no attribute 'MutableMapping'
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/times-software/Corvus/issues/2*issuecomment-2394881959__;Iw!!K-Hz7m0Vt54!gXPzV9SQllf9dewAPuA6UmVXdDxSNEHtgKIiVl7PjIzRvqMaAxTN9U7uCUfNWlrgpIHa4r4GsimngR5Ca2lqoQ$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AA2RF22WT6N3CCXAOWRN5YDZZ5LMXAVCNFSM6AAAAABPMUAJ4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJUHA4DCOJVHE__;!!K-Hz7m0Vt54!gXPzV9SQllf9dewAPuA6UmVXdDxSNEHtgKIiVl7PjIzRvqMaAxTN9U7uCUfNWlrgpIHa4r4GsimngR4fiUa0MQ$
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi Josh,
I edited that line. Here's the new error:
corvutils/pyparsing.py:1950: SyntaxWarning: invalid escape sequence '\g'
  ret = re.sub(self.escCharReplacePattern,"\g<1>",ret)
Provide target properties or Workflow

I think I must be doing something fundamentally wrong at this point. I'm using python 3.12.2, but that's just becuase I followed the instructions in install.sh and did conda activate, so I'm in conda's base environment, which has python 3.12.2. I also tried installing in a conda env with python=3.9. If I deactivate, then I have python 3.10.14 and I get a different error:
corvutils/pyparsing.py:1950: SyntaxWarning: invalid escape sequence '\g'
  ret = re.sub(self.escCharReplacePattern,"\g<1>",ret)
Provide target properties or Workflow

Would it be better if I just stick with the conda env so that I make sure it's in python 3.9?

Feff10 won’t compile on your base system? What’s your base system, and what
compiler?
Joshua Kas (He/Him/His)
Research Assistant Professor
University of Washington
On Fri, Oct 4, 2024 at 9:30 PM josephzsombor @.***>
wrote:

So, if I make sure I'm using python 3.9, it now works!
conda create --name corvus-env python=3.9
conda activate corvus-env
run corvus
and I get
Provide target properties or Workflow
!!!!
Unfortunately, I'm not sure in exactly what order I did things to get this
working now. I'm going to try to make a new apptainer and start from
scratch. Also not sure how to activate a conda env inside an apptainer,
but should be possible.
It would also be fine to run it from a conda env on my base system, but
unfortunately feff10 won't compile on my base system, so it seemed like
containerizing it would be the easiest/most robust solution.
—
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/times-software/Corvus/issues/2*issuecomment-2394920832__;Iw!!K-Hz7m0Vt54!gi1gtu2gVg4I5TAF_CgnH59ym3YluRokCk0MTSW0gC7XjhcOhiHhLp7h_eBs8ULNzFyQX9YHt_v5c9AhK6jJ5Q$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AA2RF2ZNFV744QXM3QXAI6TZZ5TP5AVCNFSM6AAAAABPMUAJ4WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJUHEZDAOBTGI__;!!K-Hz7m0Vt54!gi1gtu2gVg4I5TAF_CgnH59ym3YluRokCk0MTSW0gC7XjhcOhiHhLp7h_eBs8ULNzFyQX9YHt_v5c9DLvKSPkg$
.
You are receiving this because you were mentioned.Message ID:
@.***>

okay, so I can now run the germanium tetrachloride example. Just had to install cif2cell in corvus-env:
conda install conda-forge::cif2cell

My computer is on Ubuntu 22.04 and I'm trying to install feff10 by running make mpi with F90 = gfortran. In my fresh singularity container, I could get it to work right away as follows (using fakeroot):
apt install gfortran
apt install build-essential
apt install openmpi-bin openmpi-doc libopenmpi-dev

then as a user:
git clone https://github.com/times-software/feff10
cd feff10/src
cp Compiler.mk.default Compiler.mk #and uncommont F90 = gfortran, comment F90 = ifort
make mpi #works inside the countainer. idk why it doesn't work on my base system

but in my base system, I get errors like this:
rror: Type mismatch between actual argument at (1) and actual argument at (2) (COMPLEX(8)/INTEGER(4)).
parallel.f90:286:21:

  238 |       call MPI_BCAST(buf,count,MPI_INTEGER,source,                      &
      |                     2
......
  286 |       call MPI_BCAST(buf,count,MPI_DOUBLE_PRECISION,source,                         &
      |                     1
Error: Type mismatch between actual argument at (1) and actual argument at (2) (REAL(8)/INTEGER(4)).
make[1]: *** [Makefile:457: PAR/parallel.o] Error 1
make[1]: Leaving directory '/home/joseph/Documents/programs/chemistry/Feff/feff10/src'
make: *** [Makefile:179: mpi] Error 2

It's probably because of something weird I've done to the system. I have huge amounts of packages and repos on here. So I usually prefer to just package any important program I need as a singularity container and use it from there so that it remains stable.


403 Forbidden Errors When Accessing Via Port Forwarding

As a temporary workaround until #13 is complete, I have setup a Vagrant box with Windows and Node.js installed on it.  From within the box, the hosted function works perfectly fine.
In order to ease the pain of debugging, I have enabled port-forwarding on 7071 and 5858.
When attempting to access the function externally via the forwarded ports I am receiving a 403 error (see below).
 ~  curl -X POST -d "" http://localhost:7071/api/elements/order
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN""http://www.w3.org/TR/html4/strict.dtd">
<TITLE>Forbidden</TITLE>
<META HTTP-EQUIV="Content-Type" Content="text/html; charset=us-ascii">

Forbidden URL

HTTP Error 403. The request URL is forbidden.

Any thoughts?

You don't say what the destination host and port are for the forwarding? The CLI local server is hardcoded to use localhost (<IP_ADDRESS>). May be it checks the source of the request too?
I dont supposed the Functions' Proxy option is available locally but might be worth trying.

Vagrant port forwarding binds to all ports unless otherwise specified (I don't specify).  This means that it is bound to localhost as well.  I've tested this by shutting down the host and trying the same call again; it returns a connection refused when func host start isn't running.
I was thinking it checks the source of the request, but if I specify --cors *  shouldn't that mitigate that?

Related to #174.

@lindydonna I was actually trying to work around #174 as well.  Vagrant allows binding to all interfaces including <IP_ADDRESS>.  By all accounts, this should work.  I'm just not sure what element within the code is throwing the 403.

@jordanyaker I believe it's because the port is not exposed at all. @ahmelsayed can explain in more detail.

@lindydonna @ahmelsayed if the port wasn't exposed, I would think that I would be getting an empty error like the example below (when the CLI isn't running) instead of a 403:
curl -X POST -d "" http://localhost:7071/api/elements/order
curl: (52) Empty reply from server


HDFS-16776 Erasure Coding: The length of targets should be checked when DN gets a reconstruction task

Description of PR
JIRA: HDFS-16776
The length of targets should be checked when DN gets a EC reconstruction task.For some reason (HDFS-14768, HDFS-16739) , the length of targets will be larger than additionalReplRequired which causes some elements in targets get the default value 0. It may trigger the bug which leads to the data corrupttion just like HDFS-14768.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 42s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
38m 32s

trunk passed

+1 :green_heart:
compile
1m 35s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
1m 35s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
1m 16s

trunk passed

+1 :green_heart:
mvnsite
1m 39s

trunk passed

+1 :green_heart:
javadoc
1m 18s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 38s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 36s

trunk passed

+1 :green_heart:
shadedclient
23m  9s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 18s

the patch passed

+1 :green_heart:
compile
1m 22s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
1m 22s

the patch passed

+1 :green_heart:
compile
1m 19s

the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
1m 19s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 57s

the patch passed

+1 :green_heart:
mvnsite
1m 26s

the patch passed

+1 :green_heart:
javadoc
0m 54s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 28s

the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 27s

the patch passed

+1 :green_heart:
shadedclient
22m 50s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
237m 55s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
1m  3s

The patch does not generate ASF License warnings.

347m  2s

Reason
Tests

Failed junit tests
hadoop.hdfs.TestReconstructStripedFile

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4901

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 3c9fd80694fb 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 824b84ca6dd20791f7c766e1dfa0299a76dd6949

Default Java
Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/1/testReport/

Max. process+thread count
3044 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 39s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
38m 17s

trunk passed

+1 :green_heart:
compile
1m 42s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
1m 29s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
1m 16s

trunk passed

+1 :green_heart:
mvnsite
1m 43s

trunk passed

+1 :green_heart:
javadoc
1m 23s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 42s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 35s

trunk passed

+1 :green_heart:
shadedclient
23m  1s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 26s

the patch passed

+1 :green_heart:
compile
1m 23s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
1m 23s

the patch passed

+1 :green_heart:
compile
1m 24s

the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
1m 24s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
1m  1s
/results-checkstyle-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs-project/hadoop-hdfs: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

+1 :green_heart:
mvnsite
1m 26s

the patch passed

+1 :green_heart:
javadoc
0m 55s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 27s

the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 28s

the patch passed

+1 :green_heart:
shadedclient
22m 54s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
238m 55s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
1m  7s

The patch does not generate ASF License warnings.

348m  1s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4901

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux f257db2e44d1 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 5b4abf4b7a804fdca0db78107d73ebe65cea719b

Default Java
Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/2/testReport/

Max. process+thread count
3649 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 44s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
38m 40s

trunk passed

+1 :green_heart:
compile
1m 42s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
1m 36s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
1m 19s

trunk passed

+1 :green_heart:
mvnsite
1m 40s

trunk passed

+1 :green_heart:
javadoc
1m 24s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 37s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 33s

trunk passed

+1 :green_heart:
shadedclient
23m  3s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 22s

the patch passed

+1 :green_heart:
compile
1m 24s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
1m 24s

the patch passed

+1 :green_heart:
compile
1m 18s

the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
1m 18s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 57s

the patch passed

+1 :green_heart:
mvnsite
1m 28s

the patch passed

+1 :green_heart:
javadoc
0m 57s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 29s

the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 25s

the patch passed

+1 :green_heart:
shadedclient
22m 49s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
239m 23s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
1m  8s

The patch does not generate ASF License warnings.

349m  4s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4901

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux e39135260415 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 77586f5dde4506bc4382b6b78690b37b0d731f42

Default Java
Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/3/testReport/

Max. process+thread count
3072 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 45s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
40m 44s

trunk passed

+1 :green_heart:
compile
1m 43s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
1m 33s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
1m 15s

trunk passed

+1 :green_heart:
mvnsite
1m 47s

trunk passed

+1 :green_heart:
javadoc
1m 26s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 37s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 31s

trunk passed

+1 :green_heart:
shadedclient
22m 49s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 22s

the patch passed

+1 :green_heart:
compile
1m 22s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
1m 22s

the patch passed

+1 :green_heart:
compile
1m 16s

the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
1m 16s

the patch passed

+1 :green_heart:
blanks
0m  1s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 58s

the patch passed

+1 :green_heart:
mvnsite
1m 26s

the patch passed

+1 :green_heart:
javadoc
0m 58s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 32s

the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 23s

the patch passed

+1 :green_heart:
shadedclient
22m 41s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
237m 29s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
1m 10s

The patch does not generate ASF License warnings.

348m 41s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4901

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets

uname
Linux 82192a094eb7 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / c679261f247a39cbd1dc1dcca143a23008b90cc8

Default Java
Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/4/testReport/

Max. process+thread count
3662 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4901/4/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

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

This message was automatically generated.

Merged. Thanks for your contribution, @Kidd53685368.

As mentioned in HDFS-16739, in order to satisfy the storage policy, the length of targetIndices will be more than the actual number of blocks that need to be reconstructed.
This change may cause the reconstruction task to never succeed to satisfy the storage policy (unless the mover is manually triggered to make the block satisfy the storage policy).


Bug: Issue with Total Time for Cycle Time Summary Data

Issue:
The Total Time field for Cycle Time Summary Data throws a series of validations when a user enters time that is 3 or more digits. We need to add an error message (DTO check) to inform the user that the fields only accepts values from 0-99
Testing
Tester: @JanellC

bug resolved


[Event Request] table 5900 "Service Header" procedure Validate() field Status

Please can you add the following code/event to the above procedure...
case Status of
    Status::"In Process":
        begin
            if not LinesExist then begin
                "Starting Date" := WorkDate();
                Validate("Starting Time", Time);
            end else
                UpdateStartingDateTime();
        end;
    Status::Finished:
        begin
            TestMandatoryFields(ServLine);
            if Status <> xRec.Status then
                if "Notify Customer" = "Notify Customer"::"By Email" then begin
                    TestField("Customer No.");
                    Clear(NotifyCust);
                    NotifyCust.Run(Rec);
                end;
            if not LinesExist then begin
                if ("Finishing Date" = 0D) and ("Finishing Time" = 0T) then begin
                    "Finishing Date" := WorkDate();
                    "Finishing Time" := Time;
                end;
            end else
                UpdateFinishingDateTime();

            OnValidateStatusFinishedOnAferUpdateFinishingDateTime(Rec, xRec);       //NEW CODE
        end;
end;

The event publisher should be as follows...
[IntegrationEvent(false, false)]
local procedure OnValidateStatusFinishedOnAferUpdateFinishingDateTime(var ServiceHeader: Record "Service Header"; xServiceHeader: Record "Service Header")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 10893.


Getting lombok dependency missing error on non-java project

The recent version of the lombok plugin tries to add itself to every project, regardless of the project type. I'm working on a golang project, but every time, when I start intellij or open the project, I'm getting the following error:
11:51 AM	Lombok Dependency is missing
	  Project "go-restful-middleware" does not seem to have lombok dependency added.
      This plugin does not provide it and will not function correctly without.
	  Please make sure to add it manually and refresh your project.
	  Click here to see how to add it to your project.

You can disable plugin in the configuration for your project:

But I will look for a chance to disable it automatically for non java projects by default.

Cool. Thanks.

It also shows warnings in all projects not using lombok.
Did you consider adding a configuration option to turn warnings off completely?
I just like how it used to be before the update.

For projects where you are not using Lombok you can untick "Enable Lombok plugin for this project".
As a background - main reason for this check being added is due to the fact that many users of this plugin do not understand the fact that they need lombok and keep reporting issues, even though we are clearly stating that the Lombok dependency is needed.
I can understand that there is some frustration about some warnings and we can let users opt-out of this, just like with version check - but it's a per-project configuration.

per-project configuration
I use lombok in maybe 1 project out of 10. Did you consider adding an option to disable all warnings altogether, not on a per-project basis?

Well if its a non-java project we can improve detection and not give warnings (ideally)
However configuration of the plugin is saved per project today, and we will have to investigate if something different could be done and when.
You can also change project defaults and set Lombok to disabled completely (even better, to avoid triggering augmentations for a "fast bailout" etc) or disable some features as default and only enable where you need on a per-project basis.
We do take developer experience seriously and will think if there are good way to change/improve this. For now, as a work-around at least, this is the best I can give you.

To mitigate this issue to some extend, I've made #266 that should allow you disable "missing lombok dependency" warning for the project that is non-java or where your project is properly configured but has non-java modules.

In addition, there has been discovered a bug with the new warnings - default setting was supposed to be disabled for some of the warnings (like out-of-date warning) but it was not. This is also fixed in #266.  See #267 bug for more details.
Default "state" for both "missing lombok" and "out-of-date lombok" should be disabled. So this should solve even bigger chunk of the issue :)

I too find it annoying that the plugin gives me a warning in all projects where lombok is not used. Maybe you could add a "Don't show this again" option?

To sum it all up.

There is an issue with the release build where warnings got enabled automatically, even though default was ment to be "off".
A new configuration to disable lombok dependency (not only version) has been added
It might be a good idea to lead you into configuration for version check and what was mentioned in point 2 to disable warnings.

As the initial effort to reduce annoyance caused by the bug (point 1) and also make it a bit more flexible (point 2), the fix #266  has been merged.
Point 3 is a good way forward when warnings are going to be enabled by default, but this is not a deal breaker given 1. and 2. are fixed.

New version with mentioned fix #266  and some other improvements was released few minutes ago!

For projects where you are not using Lombok you can untick "Enable Lombok plugin for this project".

This doesn't help when only some of the modules in a project use Lombok.


fix random ordering of mimetypes in the doc strings on generate

fixes #1646

This is great. Could you please add a test case to assert the ordering?

Missed @fredbi 's comment, agree that there should be a test case.

added the unit test and passed ci. :smile:

Thank you so much.


issue: thread 'main' panicked at 'called Result::unwrap() on an Err value: Error { pos: 2:6, kind:

I'm trying to convert my xlm I used in the old karabiner and I got the following error:
./karaconv -i ~/OneDrive/Documents/emacs/karabiner/private.xml -o ~/.config/karabiner/karabiner.json
thread 'main' panicked at 'called Result::unwrap() on an Err value: Error { pos: 2:6, kind: Syntax("Invalid processing instruction: <?xml") }', libcore/result.rs:945:5
note: Run with RUST_BACKTRACE=1 for a backtrace.
It is pretty simple and small. just want to prevent karabiner to use it when I'm on vnc or emacs.
Here is the file:
https://gist.github.com/fkgruber/9c39fa2e8b46bb7648a2efe74d1ff398

Same error here with the 0.3 binary:
$ karaconv -i ....xml -o ....json thread 'main' panicked at 'called Result::unwrap()on anErrvalue: missing fieldappendix', libcore/result.rs:945:5 note: Run with RUST_BACKTRACE=1 for a backtrace.
File: https://gist.github.com/akidee/ca0d815ac0ec2c1b852acf47c46959d2


System.StackOverflowException

Hi
I've just updated to the latest version of CompareNETObjects from version v4.59.0 and now I'm getting a System.StackOverflowException whenever I try to do any kind of list operations like IndexOf!
I've manage to create a very simple xUnitTest file that demonstrates the problem. If you compile the test with v4.59.0 it is working but with v4.63.0 you will get an exception.
UnitTest1.cs.txt
What am I missing?
Best regards,
Jens Christiansen

Jens,
Beginning with 4.62 and higher the .NET Compare Objects library no longer uses the HashCode to determine if it is the same object but the reference equals.  Because you have overridden Equals to something other than Reference Equals that is why you are getting the stack overflow exception.  You will need to rewrite your code to account for this or use version 4.59.


Fixing Bearer authentication header references

Found a few references to Authentication header with the wrong Bearer value

I had to move .docc folders because they were breaking xcodebuild under Xcode 14.0.

Rebased :)

Thanks! You are right, there is no colon.


WIP Reconcile Tasks

This depends on #59
I have a couple things left to do but this is getting there.

Thanks for your contribution! The HA features here are awesome.
I left some comments that you can look at, but I would like to revisit this after your first PR is merged and I take a second pass at the review.

@rubbish, If you have time, could you take a look? I would like to get this merged soon.

Done with this review pass (ignoring tests for now); found a few potential bugs, some inefficiencies and unnecessary code, and some minor nits.

@adam-mesos, responded to all concerns. @adam-mesos or @rubbish, please take a final look before I merge.

I have reviewed this and also pair programmed with @nicgrayson. I messaged @adam-mesos and the plan is to merge and if he has any additional feedback, I will review it and definitely submit any changes in an follow up PR if he deems they are needed. Thanks for the great team work, @adam-mesos, @rubbish, @nicgrayson. :+1:


Feature/add tx

te

Coverage decreased (-18.9%) to 72.414% when pulling bda3c8e9e7ac8e7b5628779b79eb0b696d9351d3 on feature/add-tx into 2e5ccb7b26b07140e53d7a682fbb7394e8d6aaee on master.

Coverage decreased (-18.9%) to 72.414% when pulling bda3c8e9e7ac8e7b5628779b79eb0b696d9351d3 on feature/add-tx into 2e5ccb7b26b07140e53d7a682fbb7394e8d6aaee on master.

Coverage decreased (-12.0%) to 79.31% when pulling<PHONE_NUMBER>070a9428038504024896331704aa73 on feature/add-tx into 2e5ccb7b26b07140e53d7a682fbb7394e8d6aaee on master.


Update mingw-w64

Who is running that mirror?
And what's up with the official one, if there's commits that are missing?

What missing commits are you talking about?

The ones since end of march, which the official(?) mirror repo does not have.

This new mirror is up to date, but if the previous one was the official one then it's not updated anymore.

Yes, that's exactly what and why I'm asking.
What is the current mirror situation? And who is running it? What happened to the official one? (And where are they actually mirroring from?)


Liquid: add simple empty keyword

Hey @RunDevelopment here's an easy one :)
I have one quick question though: I want to add an object token that is any one of the following keywords, sourced from the liquid reference
address , all_country_option_tags , article , block , blog , cart , checkout , collection , color , comment , country , country_option_tags , currency , current_page , current_tags , customer , customer_address , date , discount_allocation , discount_application , external_video , filter , filter_value , font , forloop , form , fulfillment , generic_file , gift_card , group , handle , image , line_item , link , linklist , localization , location , measurement , media , metafield , model , model_source , order , page , page_description , page_image , page_title , paginate , part , policy , product , product_option , recommendations , request , robots , routes , rule , script , search , section , selling_plan , selling_plan_allocation , selling_plan_group , shipping_method , shop , shop_locale , sitemap , store_availability , tablerow , tax_line , template , theme , transaction , unit_price_measurement , user_agent , variant , video , video_source

I was going to go with the naive new RegExp([words].join('|')) but I was wondering if you have any better suggestions?

Thanks for the PR @JaKXz!
Use a regex literal, so /\b(?:address|all_country_option_tags|...)\b/. It's the easiest option IMO. (Using regex literals also means that our static analysis tools can deal with them, which is nice.)

Thank you for contributing @JaKXz!


fix gofmt error.

refer to go report card: https://goreportcard.com/report/github.com/go-gitea/gitea#gofmt
Signed-off-by: Bo-Yi Wu<EMAIL_ADDRESS>

Current coverage is 3.03% (diff: 100%)

Merging #168 into master will not change coverage

@@            master      #168   diff @@
========================================
  Files           33        33
  Lines         8096      8096
  Methods          0         0
  Messages         0         0
  Branches         0         0
========================================
  Hits           246       246
  Misses        7830      7830
  Partials        20        20

Powered by Codecov. Last update 904deb7...221284f

Agreed, bindata should not be included

@strk @lunny My apologies. I removed bindata.

LGTM

LGTM


Unwanted retry when returning 408

i am returning 408 from my webapi controller if a call to the backend times out. i want that code to be returned to the client but what happens is that the action that generated the timeout is called again and again.
Is there any way to disable this behavior?

Wait...hmmm
Sent from my Samsung Galaxy , an AT&T LTE smartphone
-------- Original message --------From: JanEggers<EMAIL_ADDRESS>Date: 11/27/17  12:46 AM  (GMT-08:00) To: aspnet/Home<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [aspnet/Home] Unwanted retry when returning 408 (#2289)
i am returning 408 from my webapi controller if a call to the backend times out. i want that code to be returned to the client but what happens is that the action that generated the timeout is called again and again.
Is there any way to disable this behavior?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/aspnet/Home","title":"aspnet/Home","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/aspnet/Home"}},"updates":{"snippets":[{"icon":"DESCRIPTION","message":"Unwanted retry when returning 408 (#2289)"}],"action":{"name":"View Issue","url":"https://github.com/aspnet/Home/issues/2289"}}}

There's no retry on the server side, that must be coming from the client.

that is strange i dont have retry code in place and after i changed returncode to 500 the method was no longer called repeatedly.

What does a Fiddler trace show?

hi there @Tratcher thx for pointing me to fiddler, seems chrome does it internally devtools show one request but there are several.

can you advice on a meaningful http errorcode if a backend service has a timeout, im basicly having some async code in a try catch and dont want to return 500 in case of a TimeoutException or TaskCancellationException..
would 504 be the right thing to return?

Yes 504 is a good status code for that.

thx again


refactor!: reduce name conflicts

What I did
This PR makes a breaking change to how the message struct signing info is obtained, which reduces naming conflicts (should eliminate all common ones) when using this package (e.g. #9)
fixes: #9
fixes: #32 (00b4e8c)
fixes: #2 (f704f81)
fixes: #1 (1b901d0)
How I did it
Liberal use of sunder properties, some magic
How to verify it
Check with some common signing flows (I added EIP2612 Permit and Gnosis SafeTx messages)
Checklist

[x] Passes all linting checks (pre-commit and CI jobs)
[x] New test cases have been added and are passing
[x] Documentation has been updated
[x] PR title follows Conventional Commit standard (will be automatically included in the changelog)

LGTM Some EIP-7127 stuff that worked before continue to work for me after this PR. Is there any other testing you would like done?

Yeah, I think this refactor is looking decently good, although it definitely is a breaking change so I wanted to understand all of the downstream impact before releasing it
Also was trying to refactor into a better dataclass library, but none had the support for the featureset we're using from dataclassy

Once this is mergeable, I'd like to update the pin in ape core: https://github.com/ApeWorX/ape/blob/main/setup.py#L122

if statements / logic branching in tests should be avoided. Test cases should be narrower and one shouldn't have to question about whether the branches are still relevant.

In this scenario, it's just switching out to a different value, which will never change because the transaction def aligns with an on-chain deployment, and must produce a specific hash

I would have expected the tests for create_safe_tx_def to be in test_common.py. It would be easier to audit this way.

Updated this


Unit test

Prelim PR (still have to add coverage to tenders and tickets)

Prelim PR (still have to add coverage to tenders and tickets)

cool 👍


[BUG] Rke2/k3s clusters have the error failed calling webhook even after webhook is installed for a few minutes

Information about the Cluster
Rancher Server Setup

Rancher version: v2.7.7-rc4 upgraded from v2.7.6 also seen on v2.7.6
Installation option (Docker install/Helm Chart): Helm Chart

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE1, v1.26.7, 1.4.8

Proxy/Cert Details: Self-signed

Information about the Cluster

Kubernetes version: v1.26.8+k3s1 and v1.26.8+rke2r1
Cluster Type (Local/Downstream): Custom  3 worker, 1 etcd, 1 cp RKE1

Describe the bug
On a restore to a migrated rancher server, rke2 and k3s clusters are stuck for a while in error state waiting for webhook for a few minuts even though webhook is updated/installed.
To Reproduce

Create a rancher server on v2.7.7 - HA1 server
Create 2 downstream clusters - rke2 and k3s node drivera
Take a backup - b1
Create an RKE1 cluster by following the guide https://rke.docs.rancher.com/installation
Bring down the nodes on HA1 server.
Restore the backed up file b1 on to the new cluster from step 4 by following the guide https://ranchermanager.docs.rancher.com/how-to-guides/new-user-guides/backup-restore-and-disaster-recovery/migrate-rancher-to-new-cluster
Install rancher on the new cluster - HA2
Check the nodes status on the HA2 server

Result
RKE2 and K3s clusters are stuck in errored state for a few minutes even after webhook is installed and active:
                 Failed to update fleet-default/anuk3sbig cluster.x-k8s.io/v1beta1, Kind=Cluster for rke-cluster fleet-default/anuk3sbig: Internal error occurred: failed calling webhook "default.cluster.cluster.x-k8s.io": failed to call webhook: Post "https://webhook-
                     service.cattle-system.svc:443/mutate-cluster-x-k8s-io-v1beta1-cluster?timeout=10s":dial tcp <IP_ADDRESS>:443: connect: connection refused

Expected Result
Expected the clusters to come back up active as the webhook is installed and active.
Additional context
This was noticed in previous rancher server versions as well  v2.7.6

One note from the team 1/webhook side: This error message doesn't (on it's own) indicate that the webhook is down for the full duration of cluster downtime. Rancher's controllers (and others with a sane back-off policy) can enter an exponential back-off after several failures, that may result in the controllers not retrying an operation until well after the webhook is available. To resolve this issue, the relevant controllers can implement a less aggressive exponential back-off, that may reduce the "extra downtime" between when the webhook is ready, and when the controllers are re-running for the clusters.
It's possible that the webhook is down for this entire time, but this should result in issues for other parts of the system (such as issues editing global/cluster roles). Absent any evidence of a broader impact (or specific evidence that the webhook is down for this entire time) I would recommend that team 2 investigate further why this specific component (v2 prov) seems to think the webhook is down for such a long time.

@anupama2501 I tried a few times but was unable to reproduce this. Were you seeing this behaviour consistently?

@bfbachmann I noticed this yesterday while performing a backup/rollback it was noticed on an upgrade as well as on a roll back. While the webhook was active, the rke2 and k3s clusters were in error state waiting for the webhook to be active. It took a few minutes for the clusters to recover. This is sometimes noticed for a longer time ~10min and sometimes it is ~3 minutes.
Upgrade perfomed from v2.7.6 to v2.8.0-rc1 rancher HA server with local RKE1 cluster
Downstream clusters - rke2 [1 etcd, 1 worker and 1 cp] and k3s[3 etcd, 3 worker and 2 cp] node drivers

I've also had upgrade scenarios (2.7.6 to v2.8-head) where the webhook error persisted for at least 15 minutes after the upgrade.

@bfbachmann I can't think of a time when I've personally seen it never come up, but I've seen it take as long as three hours for the webhook to come up and clusters to then become active

I'm going to close this as it is not reliably reproducible. I have seen the webhook take a while to come up before (it's often the last thing to boot), but much of this depends on the resources (CPU, RAM) available on the host system, and also the resource constraints placed on Docker (if you're using RKE1).
I think that if we see persistent issues where the webhook reliably takes a long time to book in certain cases, we can reopen this.


Update to wgpu-0.12

wgpu-0.12 isn't out yet, but will be out shortly, so putting this PR up so we can be on top of it, and to show other people that the work has already done so they don't duplicate work.

Good idea.

Are we able to merge this?

GTG


Enable BAM for e2e again after it was disabled in 1392

Description
Enable BAM for e2e again after it was disabled in #1392
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)
[x] e2e tests configuration

/hold

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.69%. Comparing base (3c71e79) to head (39e811b).
Report is 33 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1396   +/-   ##
=======================================
  Coverage   94.69%   94.69%
=======================================
  Files          58       58
  Lines        2505     2505
=======================================
  Hits         2372     2372
  Misses        133      133

/retest

/override "Red Hat Konflux / ols-e2e-tests-rosa / lightspeed-service"

/retest
/unhold

/hold

We won't be enabling BAM, it is now in unsupported state.


Check if all country listings are broken down by category and sub-category

Some mock data just has a list of classes.  That means we don't know category or sub-category.
That is probably mistaken.  Check with the real data.

Yes, lists of classes are always broken down by Category. For Britain, they are also broken down by sub-category.

Ok, so we always have Category.
What about Sub-Category for non-British units?

Aah, I've realised we do have this data.
In this image, "Composite" is Category and "PB" is Sub-Category.  So, we just have to move our existing sub-category up one level, and grab sub-category from designator.

@IanMayo, please mark this issue as "Ready to implement". This looks to be important.

Defer this, it's related to the previous strategy.


Remove pin of outdated R version

Today, I was reminded of #479, which recorded some problem of possibly interfering versions 4.2.3 and 4.3.0 of R in our CI test suite. The current release is 4.4.1, so this workaround/pin to 4.2.3 might be outdated. This PR tries removing the pin to see if our test suite is still working fine with the latest version of R.
How to review

Read the diff and note that the CI checks all pass.

PR checklist

[x] Continuous integration checks all ✅
[x] Update tests setup
~[ ] Add, expand, or update documentation.~ Just CI.
~[ ] Update release notes.~ Just CI.

This is working for now, it seems; let's see if it re-introduces some flakiness.


Market Table Suggestion - Volume (Last to Total)

I would like to make a suggestion to change the volume on the market page to display the total volume for the coin rather than showing the volume of the last trade.
Thoughts everyone?

What do you think @3s3s?

Duplicate of #121

@knkrth you referenced a issue which doesn't have a solution and additionally it has been closed.
Anyone else worked on this?

@daygle At the moment no one.


mat-tab-group that starts in hidden container does not show selection in header

What is the expected behavior?
When a tab group starts inside a container element that is hidden using [hidden]. Once the container element becomes visible the tab should render correctly and show the default selection in the header.
What is the current behavior?
If the mat-tab-group is inside a container ( for example <div>) that is hidden using [hidden] when a component renders the first time. When you set hidden to false afterwards to show the tab group the selection is not visible in the tab header. After you click to a different tab the header refreshes and then the selection is shown as expected.
What are the steps to reproduce?
Here is a StackBlitz sample that reproduces the issue. Once you click the button to make the tab group visible you can see that the header has no selection.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?

Angular: 7.0.0 and 7.1.2
Angular Material: 7.0.1 and 7.1.1
OS: Windows: 10
TypeScript: 3.1.3
Browser: Version 70.0.3538.110 (Official Build) (64-bit)

Is there anything else we should know?

I have a related issue: When a tab group with many elements is nested inside another element, such as another tab group, the arrow buttons to scroll through the tabs are only visible if the tab group is visible right after loading the page. If it is hidden at first you have to select an item for the arrows to show up.
I have also created a StackBlitz to illustrate the problem.
I assume that our issues have the same cause and would be happy to hear if there is any workaround.

Same problem here, demo: https://stackblitz.com/edit/angular-material-tabs-problem

Hi @andrewseguin, I see you closed this issue while it still persists and the link you added to something I suppose to be a workaround doesn't actually work.

@andrewseguin couldn't get your workaround to properly work!

@andrewseguin Your example does not work. Is there a plan for a resolution on this?

@andrewseguin Your workaround only draws the underline, but does not focus the text on the tab correctly and still appears unselected. Is there a plan for an update on this?

For anyone coming across this issue. You can: add a @ViewChild reference to the mat-tab-group element in the .ts file and then call childRef.realignInkBar() when you change the parameter to un hide the element.
Stackblitz: https://stackblitz.com/edit/realighinkbar-example

For anyone coming across this issue. You can: add a @ViewChild reference to the mat-tab-group element in the .ts file and then call childRef.realignInkBar() when you change the parameter to un hide the element.
Stackblitz: https://stackblitz.com/edit/realighinkbar-example

Note that when you select the second tab, for example, hide the tabs and then show them again, the selected tab does not return to the first one. It remains the last selection.

For anyone coming across this issue. You can: add a @ViewChild reference to the mat-tab-group element in the .ts file and then call childRef.realignInkBar() when you change the parameter to un hide the element.
Stackblitz: https://stackblitz.com/edit/realighinkbar-example

Showing me compile error on viewchild:
Argument of type '{ static: boolean; }' is not assignable to parameter of type '{ read?: any; }'.

For anyone coming across this issue. You can: add a @ViewChild reference to the mat-tab-group element in the .ts file and then call childRef.realignInkBar() when you change the parameter to un hide the element.
Stackblitz: https://stackblitz.com/edit/realighinkbar-example

Note that when you select the second tab, for example, hide the tabs and then show them again, the selected tab does not return to the first one. It remains the last selection.

You can use selectedIndex on the  element to store the current tab index.
See updated Stackblitz: https://stackblitz.com/edit/realighinkbar-example

For anyone coming across this issue. You can: add a @ViewChild reference to the mat-tab-group element in the .ts file and then call childRef.realignInkBar() when you change the parameter to un hide the element.
Stackblitz: https://stackblitz.com/edit/realighinkbar-example

Showing me compile error on viewchild:
Argument of type '{ static: boolean; }' is not assignable to parameter of type '{ read?: any; }'.

Is this in the Stackblitz example or in your own project? If in your own project you may be on an older Angular varsion which does not require the {static: boolean} option -you can just remove it.

For anyone coming across this issue. You can: add a @ViewChild reference to the mat-tab-group element in the .ts file and then call childRef.realignInkBar() when you change the parameter to un hide the element.
Stackblitz: https://stackblitz.com/edit/realighinkbar-example

Note that when you select the second tab, for example, hide the tabs and then show them again, the selected tab does not return to the first one. It remains the last selection.

You can use selectedIndex on the element to store the current tab index.
See updated Stackblitz: https://stackblitz.com/edit/realighinkbar-example

I've tried it before without success, but now it works in your example inserting this.selectedIndex = 0;
toggleExpandCard() {
this.selectedIndex = 0;
this.tabs.realignInkBar(); // Remove this line to 'break' the example
this.isHidden = !this.isHidden;
}
I'm gonna try on my project, thank you very much!


Add variation text when include_text is enabled

Add variation text (e.g., :skin-tone-2:) alongside emoji text when include_text is enabled. This allows the full colon name to be selected when copying HTML that includes an emoji span.
Other changes:

Moving emoji_span helper function into the test preamble and removing it from individual spec files.

Happy to make any changes to accommodate style or process preferences.

Coverage increased (+0.1%) to 90.393% when pulling 65c20b77ba2ca6460f700ddc8a739814922edabf on montlebalm:add_variation_text into ed21f14a5cf719a03bd8a65fd5ef470b7917d2df on iamcal:master.

can you remove the change to the emoji-data sha? i'd like to keep the data version stable here.
if you think this warrants a minor version bump, please roll up the notes from 3.0.3 into 3.1.0 insce i never cut that release

I could go either way on the version. It does change existing behavior someone might rely on, but it's admittedly minor and could be considered a bug fix.

Coverage increased (+0.1%) to 90.393% when pulling b1e4c8dd87775592a9931bc832712dbcdb0b9115 on montlebalm:add_variation_text into ed21f14a5cf719a03bd8a65fd5ef470b7917d2df on iamcal:master.

@iamcal I reverted the change to emoji-data and rolled up the notes from 3.0.3 into 3.1.0.

merged, thanks!


Can XVG do anything besides resetting it to zero?

As far as I can tell.. XVG Verge is working as developed. I recorded the entire process when I signed up and was never issued nor asked to create a password. I just followed the instructions on the Verge site to use Orbot and the Verge app, but those instructions have changed over the years.  So I don't have a password and never have. When I go to send coins to someone, I am asked for a password and then the Verge app suggests that "if the problem persists, try restoring your wallet." I could restore my wallet, but so far every single forum post that I see from everyone having the exact same problem who don't have a password becaue they never got one restored their wallet only to lose their coins. I have not seen anyone ever able to send any XVG coins or restore their wallet or do anything other than give money to XVG Verge "developers" in the form of "lost" password that they never had. It's a huge scam as far as I can tell. A good reason not to let your friends buy XVG Verge.
If I can't find a way to get a password issued, then I will lose my coins by restoring my wallet, record the whole thing, then combine it with the video I made while signing up and not being given a password, and expose the fraud that is XVG Verge. I'll restore my wallet to zero (so I can give them more money?) and record the whole thing. First I'll backup my phone though. In case there's any way to actually do anything with XVG Verge besides look t it and reset it to zero.

Ok, I will await an "anwser" then, whatever that is.


Add Show/edit configurations

Show/edit configurations.
- Should you track changes in a handout?
- Should you only track changes in a handout?
- name of the handout to use?
- ???

Added on Commit f6d9f3d0.


[FEATURE] Add AntiChatLag

Is your feature request related to a problem? Please describe.
I get mad when kids send thousands of unicode characters in chat or /whisper them to me, because it causes a crash, and I was wondering if it was possible to stop these messages or filter them out.
Describe the solution you'd like
I would like if there was a module that would filter these messages.
I've tried muting them, but the chat logs don't save, so I cant /ignore them.
Additional context
I dont have much else to say / give.

make it urself retard its likw 2.5 lines

@CascadeDemon bro ive never coded in java before
idk how it works

https://github.com/shxrkyylol/Calypso
use youtube Guides den lel

@CascadeDemon btw if its so easy make it urself

@CascadeDemon btw if its so easy make it urself

???? i alr have that module degen lelz

@CascadeDemon btw if its so easy make it urself

???? i alr have that module degen lelz

bro why u so mean? can u send it over please?

if possible, put the error log and lag message on gist.github.com and i could make a pr adding it here https://github.com/3arthqu4ke/3arthh4ck/blob/main/src/main/java/me/earth/earthhack/impl/modules/misc/chat/Chat.java#L28

@mdo992 the message looks like this

There is a solution to this, of course, load the pop sound lag from the chat and you will crash once, then you will not live, but you have to do it when you leave the server and enter it all the time.

@shasdfasdqw could you try explaining it better? i am not understanding what the "pop sound lag" and when I "leave the server and enter it all the time"

@shasdfasdqw could you try explaining it better? i am not understanding what the "pop sound lag" and when I "leave the server and enter it all the time"

my english is bad when you enter or leave any server

poplag crashes the game when a guy pops it sorry i said pop sound lag there will be pop lag
in the chat section, pop lag will load, turn it on, it will crash for a moment.


Fix the issue https://github.com/TransitApp/SVProgressHUD/issues/481.

A potential approach for fixing the issue.

Did this remove the background circle drawing/graphic?
Here's a screenshot that shows what I mean:

I'm not sure. In what context are you seeing this ? What methods are you calling. Can you reproduce it with the Demo app ?
By the look of it, is seems that you're calling some variant of showProgress: ?
If I run it on the Demo app, in the commit preceding my change, the appearance looks similar:

What would your expected behaviour be ?

As I recall, there was a very faint grey circle behind the actual progress circle, which is what is missing here. Or am I imagining things? :P

... I can not remember a "faint grey circle".

Found a screenshot by googling. Here's more or less how it looked like for me before:

Oh I see now. Will investigate this.

The "issue" is not related to this bugfix. Will go deeper into history.

Should be fixed.


Fix a bug in trajectory_sampler

Argument env was being passed to a wrong parameter.

This looks good. Thanks!


feat: Add support for sign function in pgsql

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Add support for sign function in pgsql.
associated issue: #8900

Checklist For Contributors

[x] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[ ] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

Click here for Documentation
Types of user-facing changes

SQL commands, functions, and operators

Release note

It's not ready to be merged now and I will submit the test in subsequent commits.  I opened this so shat we can discuss  incorrect code changes here.
This is my first PR, so let me know if anything looks wrong.  Thanks :)

I think it's ready to be merged.  What should I do next ?

Hi @Guo-Shiyu , thanks for your interest in RisingWave and your contribution!
Could you also add the sign function for decimal/numeric as PostgreSQL supports that and RisingWave tries to be compatible with PostgreSQL?
https://www.postgresql.org/docs/9.1/functions-math.html

Could you also add some e2e tests for the function? Thanks you

Ping @Guo-Shiyu. The PR has been inactive for a week. Hope it can be merged soon.

superseded by #10819


Targetsify 03_historical_jread

@jread-usgs - thought it would be fun to see how quickly I could convert your scipiper pipeline into a targets one. Pretty quick! Plus, I couldn't help myself and added a short SB download step to get the data locally. Total build time is 8 minutes. PNG diffs are probably just because of my bad graphics rendering on Windows compared to your Mac - I am not seeing a visual difference otherwise.
Tagging @ceenell here too since we talked about it earlier this week

Cool! This does look look like a pretty slim refactor.
I pulled this down, reviewed it, ran it, and it worked well for me.
I don't want you to make this change (please don't!) but one reason I avoided sbtools is that these files were taking a ton of time to download with that API and were failing too. So I just manually downloaded them from the API. Perhaps they are much faster and more reliable now, but I think that step fails if any one of the three downloads fails, which could bee pretty error-prone when SB is flakey. But please don't change to a branch - I think we're good here.


Exclude WebSocket Requests from "slow response time" rules of SmartDetection

Hi there,
my WebApp handles WebSocket requests as well as normal HTTP requests.
These days, I received some "slow response time" notification emails from AI for this WebApp.
The response time of WebSocket Requests is naturally very high (time between open and close handshake).
Is it possible to exclude WebSocket requests from SmartDetection, so that I only get notifications if the response times of the normal HTTP requests are slow?
Best regards
Alex

This does seem like something that should be configurable out of the box.

Same issue. This should be accounted for. App Insights treats this as an issue but this behavior is expected.

Same issue. This should be accounted for. App Insights treats this as an issue but this behavior is expected.

Bumping because this should be fixed

Boing

4 years old problem and no solution yet?
Basically, "metricNamespace":"Microsoft.Web/sites","metricName":"HttpResponseTime" should not include WebSocket connections because they can last for hours and it is not what you expect to monitor or trigger alerts on.

"metricNamespace":"Microsoft.Web/sites","metricName":"HttpResponseTime"
^ This is not something produced by the Application Insights SDK.

Any update on this issue?

https://github.com/microsoft/ApplicationInsights-dotnet/pull/2372/files Fixed SDK to exclude long running SignalR connections.
SmartDetection is not part of this SDK.
"metricNamespace":"Microsoft.Web/sites","metricName":"HttpResponseTime" - this metric is also not part of this sdk, so unfortunately, can't help.

You could help by forwarding the issue to the right team

Okay, can you point me in the right direction with a link?

I do not know if teams outside of SDK have Github repo for reporting issues.
If you see issue with application insights sdk produced metrics after https://github.com/microsoft/ApplicationInsights-dotnet/pull/2372/files, this is the right forum.
If it is about smartdetection - my only suggestion is to open Azure support ticket.
"metricNamespace":"Microsoft.Web/sites","metricName":"HttpResponseTime" - If this is about who is producing this metric, my best guess is, its automatically done by Azure WebApps, so you need to create a support ticket for that too.

It's pretty weird when someone inside the company asks someone outside of the company to influence a product decision, especially going through support, which I suspect we've all had, at best, mediocre interactions with.

The issue is still happening and took me a while to understand that it wasn't a real problem with our application and I can't believe it hasn't been looked at yet after all these years...

The issue is still happening and took me a while to understand that it wasn't a real problem with our application and I can't believe it hasn't been looked at yet after all these years...

https://github.com/microsoft/ApplicationInsights-dotnet/pull/2372/files Has fixed this.

The issue is still happening and took me a while to understand that it wasn't a real problem with our application and I can't believe it hasn't been looked at yet after all these years...

https://github.com/microsoft/ApplicationInsights-dotnet/pull/2372/files Has fixed this.

Was this fix released? I have the latest SDK for .Net (2.4.3) and I'm still getting very high response times on application insights while the response time of the rest of the application seems to be ok. Any suggestions?

is this issue still open? @rafaelgrilli92 can you tell us if the issue is fixed in the updated AppInsights package?
we are at the latest SDK for .Net (8.0.6) and we too are seeing these response times, our overall average response times are not usable anymore.
we use Azure.Monitor.OpenTelemetry.AspNetCore (1.2.0)

can we assume that OpenTelemetry doesn't support this fix even though adopting OpenTelemetry is "recomended"?
the 'http.long_running' property is included in the request:

@akeijzer11 not really. I ended up setting up the Azure SignalR instead of using our backend server to handle the connections, so now the weird response times are gone.

we use Azure.Monitor.OpenTelemetry.AspNetCore (1.2.0)

I don't think OTel has fixed this, would be good to report this in the AzureMonitor OpenTelemetry repo https://github.com/Azure/azure-sdk-for-net/issues/new/choose
@rajkumar-rangaraj FYI, who owns the AzureMonitor Distro share plans for addressing this in OTel distros.
This is fixed in Application Insights.


Add ability to view document's metadata

Is your feature request related to a problem? Please describe.
Currently, it's impossible to view document's metadata which is useful, for CAS, expiration or xattrs examination
Describe the solution you'd like
Add a context menu on the document to view the document's metadata (xattrs keys/metadata)
Describe alternatives you've considered
Admin UI or N1QL

Kush:
The flow should be the following:

Right-click on the document
Click on "show metadata". (which opens the editor with the document metadata)
when the document is saved, the metadata should be updated

(we don't need to show the metadata in the tree, as the scenarios using it are a little bit less common)


FEAT env

Dear Adyen development team,
I created this PR based on the discussion that we had in this comment.

Hi @emidev98,
Thanks for submitting your first PR! 🎉
We'll review your contribution soon.
Cheers!

Hi @emidev98,
We really appreciate your contribution, but in this case we don't see a clear benefit in implementing the proposal.
I'm closing this PR, but feel free to contribute again in the future.
Cheers!


Custom ROS modes not visible in QGroundcontrol for Pixhawk 6X

Issue Summary
We are experiencing an issue using the PX4 ROS2 Interface Library with our custom ROS2 package (px4_ros2_trajectory_control, a slightly modified version of your experimental trajectory controller) to control our custom drone (Sarax). While everything works as expected in simulation and with Pixhawk 5X hardware, the ROS Trajectory Mode does not appear in QGroundControl when using Pixhawk 6X, despite identical software versions, workflow and setup.

Context

Goal: Use the PX4 ROS2 Interface Library to send velocity (trajectory) commands via ROS2 for safe motion control of Sarax (Pixhawk 6X).
Setup:

Ground station PC: Running QGroundControl.
Sarax: Equipped with a Jetson Pixhawk Baseboard (Pixhawk 6X).
Communication: Telemetry radio between the ground station and Pixhawk 6X.
ROS2 node: Runs directly on the Jetson board, translating Twist commands to PX4-compatible setpoints.

Observations

Simulation (X500 in Gazebo):
ROS Trajectory Mode appears in QGroundControl and functions correctly.

PX4 version: v1.15.0
Communication: Direct UDP
Hardware: Simulation only

Jetson with a separate Pixhawk 5X:
ROS Trajectory Mode does appear in QGroundControl.

PX4 version: v1.15.0
Communication: Telemetry radio or UDP
Hardware: Pixhawk 5X

Jetson with a separate Pixhawk 6X:
ROS Trajectory Mode does not appear in QGroundControl.

PX4 version: v1.15.0
Communication: Telemetry radio or UDP
Hardware: Pixhawk 6X

Jetson with build-in Pixhawk 6X:
ROS Trajectory Mode does not appear in QGroundControl.

PX4 version: v1.15.0
Communication: Telemetry radio or UDP
Hardware: Pixhawk 6X

Findings

The issue seems to be hardware-dependent:

Works: Pixhawk 5X (via telemetry radio or direct connection).
Fails: Pixhawk 6X (even with identical firmware and setup, also tested for firmware versions v1.15.1 and v1.15.2).

ROS Trajectory Mode is initialized correctly but never activated because it does not become visible in QGroundControl.
MAVLink console outputs for both hardware versions show no significant differences.
Flashing the firmware via QGroundControl or building from source made no difference.
Using telemtry or UDP for communication does also not seem to make a difference.
We also tried using separate PCs (Ubuntu 22.04) and a separate Pixhawk 6X to see whether this is an issue from the Jetson  Pixhawk board, but also this yielded the same results.

Request
Given that the issue seems tied to Pixhawk 6X hardware, could there be a firmware or PX4 ROS2 Interface Library compatibility issue affecting custom modes over telemetry? Is there a different way of activating the ROS mode other than from QGroundControl? We would appreciate any insights or suggestions for debugging this further.

Can you please clarify how are the Pixhawk's connected to the Jetson? (eg: ethernet, serial)

Good question!
I made some mistakes in the list above, let me clarify:

In the list above, communication (telemetry radio (serial) or Ethernet (UDP)) referred to the connection between PX4 and QGroundControl (QGC).
Observation 2. and 3. were done with a separate PC (Ubuntu 22.04) serving as onboard computer, not the Jetson Pixhawk board. Here, the PC was connected via Ethernet (UDP) to the Pixhawk's.

Here it was checked if the custom ROS mode would also show up on another PC on QGC over the telemetry radio (serial), which it did.

The Jetson and the build-in Pixhawk are connected via an Ethernet switch as shown here.

Here we only tested with telemetry radio as communication method for QGC as it wouldn't make sense to run QGC on the Jetson itself. However, from Observation 2. and 3. we can infer that the ROS mode not showing up is dependent on the Pixhawk that is being used and is independent from the Jetson.

Here's the corrected list:
Observations

Simulation (X500 in Gazebo):
ROS Trajectory Mode appears in QGroundControl and functions correctly.

PX4 version: v1.15.0
QGC Communication: Ethernet (UDP)
Hardware: Simulation only

PC with a separate Pixhawk 5X:
ROS Trajectory Mode does appear in QGroundControl.

PX4 version: v1.15.0
QGC Communication: Telemetry radio (serial) or Ethernet (UDP)
Hardware: Pixhawk 5X

PC with a separate Pixhawk 6X:
ROS Trajectory Mode does not appear in QGroundControl.

PX4 version: v1.15.0
QGC Communication: Telemetry radio (serial) or Ethernet (UDP)
Hardware: Pixhawk 6X

Jetson with build-in Pixhawk 6X:
ROS Trajectory Mode does not appear in QGroundControl.

PX4 version: v1.15.0
QGC Communication: Telemetry radio (serial)
Hardware: Pixhawk 6X

Thanks for the detailed report @NilsRublein. The mavlink messages for the modes are still under development and were thus not enabled on all boards. You need to add this line to the v6x board: https://github.com/PX4/PX4-Autopilot/blob/v1.15.1/boards/px4/fmu-v6x/default.px4board.
On main this is already the case now.

Thank you @bkueng, this worked!

Closing this issue as solved.


[New Port Request]

Library name: Open CASCADE Technology
Library description: A 3D CAD, CAM, CAE, ecc. C++ LGPL Library
Source repository URL: https://www.opencascade.com/content/latest-release
Project homepage (if different from the source repository):
Anything else that is useful to know when adding (such as optional features the library may have that should be included):

I'm sorry,
I've duplicated #4968


Remove ExpandDNSConfig from local kind cluster

How to categorize this PR?

/area dev-productivity
/kind cleanup
What this PR does / why we need it:
https://github.com/gardener/gardener/pull/8533 has upgraded the kind clusters to 1.28, so ExpandDNSConfig is set to true by default and does not need to be explicitly done.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Thanks 😄
/lgtm
/approve


Network Policies for Port

I want to use the network policy to block particular ports (22, 23, 80). Do we have any policy to block particular rules, as we did not see any rules regarding that?
Documentation is also minimal for network policy.
Any help will be appreciated

Looks like a request for help in its current form.
/kind support
(Feel free to remove the label if this is incorrect.)

@anuprd Thank you for reaching out and seeking assistance. We appreciate your query, but the GitHub issue tracker may not be the most suitable platform for obtaining advice on Kubernetes. We recommend exploring alternative resources such as the Kubernetes Slack community (http://slack.k8s.io/) or the Discuss Kubernetes forum (https://discuss.kubernetes.io/) for the support you need.

@anuprd
There is no network policy to block particular port but  you can allow range of ports by using egress.
you can try with third party network policy provider

As this is support issue you can raise it in Slack or in community forum . It will be useful to discuss the respective channel.
For now i am closing this issue Try to raise the issue, the bug report, feature request Which will enhance the development the look and feel of the website.
You may reopen or raise new issue with proper description.

/close


Elasticsearch::Transport::Transport::Errors::BadRequest with number_format_exception

Hi
I need to index a model related with some dynamic data structure.
So my as_indexed_json method looks like:
def as_indexed_json(options={})
  as_json(include: { user_data: { only: :doc }})
end

Where the doc is a jsonb column.
Now I got an error with one of the fields when indexing data. It looks like:
[400] {"error":{"root_cause":[{"type":"mapper_parsing_exception","reason":"failed to parse [user_data.doc.test_field]"}],"type":"mapper_parsing_exception","reason":"failed to parse [user_data.doc.test_field]","caused_by":{"type":"number_format_exception","reason":"For input string: \"25a\""}},"status":400}

It looks like it is mapped to number and it's a problem like https://www.elastic.co/guide/en/elasticsearch/hadoop/current/mapping.html#_incorrect_mapping
But how can I solve it? Can I somehow force it to be mapped as string with the jsonb column?

+1 same issue

Closing as this question is out of date. Please reopen if it's still relevant.