Enhance difference printing for equal waves

Output the name for free waves if present. Since IP9 free waves can have names via

Function Dostuff()
	Make/FREE=1 important_name
	print NameOfWave(important_name)
End

Drop unused trailing [0] dimensions in difference table
Comparing the output of text wave with differing strings can be hard. Maybe we can provide a colorized diff of the string as well?
Can we remove trailing zeros for integers in

Dimensions  |Labels|Value    |
------------|------|---------|
[3][0][0][0]|      |15.000000|
            |      |17.000000|

Can we also display enough digits so that the there is always a difference visible?

Last point: print number to string with increasing digit number until diff is present

The first bullet point is already done in the current release. If I use the following code:
static Function TestWave()
	Make/FREE=1 wv1 = { 15 }
	Make/FREE=1 wv2 = { 17 }
	CHECK_EQUAL_WAVES(wv1, wv2)
End

I get the following result:
  Assuming equality using mode WAVE_DATA for waves wv1 and wv2; detailed: Waves difference:
Wave1: 0x0851d19340
Wave2: 0x0851d194c0
Dimensions  |Labels|Value    |
------------|------|---------|
[0][0][0][0]|      |15.000000|
            |      |17.000000|
: is false. Assertion "CHECK_EQUAL_WAVES(wv1, wv2)" failed in line 10, procedure "Proc0"

The names wv1 and wv2 are already included in the output.

Good find that NameOfWave already helps us here.

@Garados007 On second thought. Can we have the free wave names in
Wave1: 0x082651f8b0
Wave2: 0x082651f2d0

instead of the memory addresses if these are available?

I think we can have both. Something like this:
Wave1: wv1 (0x082651f8b0)
I don't know if the memory address is somewhere needed but this solution will still keep it.

@Garados007 Fine with me.

I am trying to solve the trimming of trailing zeros of the numbers. There is a format specifier %g that will do this automatically. The current used %f will always keep six digits after the decimal point.
This will look like this:
static Function TestWave()
	Make/FREE=1 wv1 = { 15, 0.003401, 1e-10 }
	Make/FREE=1 wv_2 = { 17, 0.003,<PHONE_NUMBER> }
	CHECK_EQUAL_WAVES(wv1, wv_2)
End

Output:
Dimensions|Labels|Value      |
----------|------|-----------|
[0]       |      |15         |
          |      |17         |
[1]       |      |0.003401   |
          |      |0.003      |
[2]       |      |1e-10      |
          |      |1.23457e+09|

(this example contains the change in ddd0261)

Another option will be to use the output of %f and trim the characters by "hand" (code).

The solution with %g looks inconvenient for comparing the numbers for case 2 as the exponent is not the same.
I think for output with the same exponent %g has no option. So the way would be to go with %f and trim trailing zeroes.
The floating point values have a certain precision depending on type that translates for decimal representation in a certain number of digits.  Thus, if you enforce too many digits to be converted to a string the last digits will be non-zero for all not exactly representable numbers, e.g.:
printf "%.20f\r", 1.55
1.55000000000000004441

Double precision numbers use 53 bits for the mantisse, thus in decimal you get 53 * log(2) = 15.955 digits.
So with trunc(log(n)) you get the digits in front of the dot and 15.955 minus that value give you the digits after the dot.
For double precision I would expect something like that to work:
val = floor(15.955 - trunc(log(n)))
val = val <= 0 ? 0 : val
format = "%." + num2istr(val) + "f"
sprintf str, format, n

and then cut the zeroes at the end.
Does that make sense?

Yes. I could change the code so it will produce this:
Dimensions|Labels|Value                     |
----------|------|--------------------------|
[0]       |      |15                        |
          |      |17                        |
[1]       |      |0.00340100005269051       |
          |      |0.00300000002607703       |
[2]       |      |0.000000000100000001335143|
          |      |1234567936                |

There are some nasty stuff at the end but this has something to do with the internal representation of the numbers.
For a future issue it could be nice to align these numbers at their decimal point.

With 249d783 it will show all digits of the full number and only trim some insignificant zeros at the end of the number. So it will be possible to view the whole number including the difference. This will solve the bullet point 5.
Bullet point 6 is a little bit trickier because I have to check for the start of the difference and have to reduce the number of shown digits. With the current solution you only have something around 16 digits and I think this is enough to see.

Colorized diffs are impossible with the current version of Igor Pro. I cannot color the output to the history. I didn't found any references in the docs and I tried many stuff but nothing worked (html, markdown, rtf, label formatting). There is an old forum post with this issue but since then no update.

I think the issue here is the following:
In your test case you create waves of the type single precision float32.
You put these as argument in GetNiceStringForNumber where it is implicitly converted to double.
The bit count for float32 mantisse is 23, while you use currently 53 for double for all inputs.
If you replace the 15.955 depending on the input floating point type with either
53 * log(2) or 23 * log(2) then the latter would give you about 6.9 digits. With that you would not get the "nasty stuff" as you describe it.

Oh yes. I forgot the thing about float and double. I will try to look at it tomorrow. Maybe I can get more nice results.

Colorized diffs are impossible with the current version of Igor Pro. I cannot color the output to the history. I didn't found any references in the docs and I tried many stuff but nothing worked (html, markdown, rtf, label formatting). There is an old forum post with this issue but since then no update.

Good point. I forgot about that forum post. And I was the OP ...


Search within PT Settings

Description of the new feature
It would be nice to have a search box to search for options within PT Settings. This isn't immediately necessary, but will be more useful as new tools and options are added.
For example, I have trouble focusing on the long list of options in FancyZones settings, it usually takes 8-16 seconds for me to find a specific option. If I was able to search for a keyword, it would be much easier to locate the specific setting.
This should function similar to Windows Settings search, but it would be nice if you could toggle the settings directly from the opened search, unlike Windows Settings where you have to click the specific option before changing the setting (Not a big deal if there are technical limitations to doing this).
Alternatively, a Ctrl+F solution would be acceptable, but I think a dedicated search box would be better overall.

If you'd like to see this feature implemented, add a 👍 reaction to this post.

We could add the search in the title bar like in a lot of MSFT products like Teams
Azure Portal (I know it's a website but it still looks similar)

Microsoft Word

With the current title bar, It will not look good. We will have to wait for #1547

I think we should add the search as first element in the nav bar. Like in W10 settings app.

I think we should add the search as first element in the nav bar. Like in W10 settings app.

I think @htcfreek's idea is better because PowerToys settings uses NavigationView

Yes, I also agree that the NavBar would probably be the best option. It would probably be good since it's not tied to the Title bar, especially now, as it still uses the older style.

Hello admin, we need this feature in 2024.
The issue still persists, with the added features/modules in PT, this issue has become more prevalent!
Thanks admin

Examples of implementation of this function in other applications
Notepad++ (Search Menu & Preferences and Filter fields)

PDF-XChange Editor (Find field)


NuGet packages don't use the Apache 2 license

Copied from https://rx.codeplex.com/workitem/70
The Rx, Ix, and other related packages in NuGet (such as  http://www.nuget.org/packages/Rx-Main/)
has the link to the license page.
However it refers to  http://go.microsoft.com/fwlink/?LinkID=261272, not Apache License.
According to the license page in this project site (https://rx.codeplex.com/license), all resources by
this project are licensed under the Apache License 2.0, and I had found no exception clause.
I think that the link to the license page of NuGet packages might be a mistake and should be fixed, or
if this is correct, the notice of special licensing for NuGet packages should be mentioned in this project site.

+1 to fix this issue as soon as possible ... elsewhere it's useless for further usage.
(only as part of your non-public / personal projects)
Please be so kind and fix, it should not be that hard .. and it's know issue for several months.

Fixing this issue would make my life so much easier.
Pls fix, thx

Fixed in master now


add coverage action

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@363e1d8). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #47   +/-   ##
=======================================
  Coverage        ?   30.18%
=======================================
  Files           ?        5
  Lines           ?      212
  Branches        ?        0
=======================================
  Hits            ?       64
  Misses          ?      148
  Partials        ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Support in Unity for Cesium ion servers running in single-user authentication mode.

Requires https://github.com/CesiumGS/cesium-native/pull/841.
As described in the above ticket, we don't currently support Cesium ion Self-Hosted servers that are running in single-user authentication mode. This is inconvenient, especially since single-user mode is both the easiest to setup and the default authentication mode in Cesium ion Self-Hosted.
This PR changes the behavior of Cesium for Unity to request the current authentication mode from the /appData endpoint before connecting, ignoring the authentication flow and token options if in single-user mode.

@azrogers I get a crash running this branch when trying to resume an ion session. I believe it's because of CesiumIonSessionImpl::Resume at line 304: this blog of code:
https://github.com/CesiumGS/cesium-unity/blob/9be3acb7ef6cee0665555c2b4913d9beb5a0c770/native~/Editor/src/CesiumIonSessionImpl.cpp#L304-L310
this->_appData is nullopt, which is causing an accessor error.

@azrogers I'll merge this after CI passes. Thanks!


Remove structure slops to reduce cache pressure

Many shared structs are ordered in a way that, when built on a LP64 system, will have unused bytes due to alignment.  These structs may be reduced by carefully reordering them, and maybe changing certain types of members to something that packs better.
For instance, struct ntttcp_stream_server has a sizeof 144, spanning over 3 cache lines.  According to pahole, it could be at least 129, which is 5 bytes over using just 2 cache lines.  By changing all the bool elements to a single integer holding flags, this would fit 2 cache lines.

Yes, good catch. The alignment is really something we should fix.


\hat{\jmath} doesn't match latex output

(La)TeX code:
The code of (La)TeX you tried to render:
\hat{\jmath}
\hat{\imath}

This is reproducible in the demo. \hat{\imath} is also subtly wrong, but I only noticed later, it's barely different.
Screenshot

Expected behavior:
The hat should be further to the right, like in this screenshot.

Thanks for the report! Likely the same underlying issue as #1821, which is (sometimes?) incorrect italic correction on accents. This example is likely a better one to run with.


cancel timeout when waiting for driver extension

Use Future.timeout instead of a race between two futures. timeout knows to cancel the Timer when the Future resolves. Otherwise, when a test passes the process gets blocked until the Future.delayed resolves because timers create ports that prevent the Dart VM from exiting.

LGTM, but fails Travis. Looks like you need to rebase past Hixie's point changes.


Update README.md

Add a TODO that gives us a chance to flex on Discord

Make sure to add reviewer requirements, branch protections, and build testing(CI style?). I am able to merge this as soon as I opened it


Multi-key repeat problem

I map shift_r-> esc (meta for emacs). The other week I accidentally deleted some of karabiner settings and after my best attempt to reconstruct them my multi-key repeat behavior has changed.
Previously when I held <shift_r + n> the entire multikey was repeated to emacs and it scrolled my window nicely. Now it scrolls one step and then starts printing 'n'.
Apologies if this is a duplicate, but I couldn't find anything with a quick google and look through what I hoped to be relevant options in karabiner. Thanks in advance for your help!

The repeat behavior has been changed due to the software architecture changes between Karabiner and Karabiner-Elements.
Karabiner-Elements repeats only the last key event.
Please consider to use option+n (or another modifier which is recognized as meta in emacs) instead of esc+n.


Problem with setup.py install

Everything seems to be running fine with the install until cysift. Here is what I am getting out when I run 'setup.py install'. Everything up to that point seems to build correctly. Probably user (me) error.
C:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\VC\Tools\MSVC\14.16.27023\bin\HostX86\x64\cl.exe /c /nologo /Ox /W3 /GL /DNDEBUG /MD -Icyvlfeat -IC:\U
sers\10dallasj\PycharmProjects\PPSuite\venv\lib\site-packages\numpy\core\include -IC:\Users\10dallasj\PycharmProjects\PPSuite\venv\include -IC:\include -IC:\include
"-IC:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\VC\Tools\MSVC\14.16.27023\ATLMFC\include" "-IC:\Program Files (x86)\Microsoft Visual Studio\2017\Bu
ildTools\VC\Tools\MSVC\14.16.27023\include" "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.6.1\include\um" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.
17763.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\shared" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\um" "-IC:\Program
 Files (x86)\Windows Kits\10\include\10.0.17763.0\winrt" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\cppwinrt" /Tccyvlfeat\sift\cysift.c /Fobuild\
temp.win-amd64-3.7\Release\cyvlfeat\sift\cysift.obj
cysift.c
c:\users\10dallasj\pycharmprojects\ppsuite\venv\lib\site-packages\numpy\core\include\numpy\npy_1_7_deprecated_api.h(12) : Warning Msg: Using deprecated NumPy API, di
sable it by #defining NPY_NO_DEPRECATED_API NPY_1_7_API_VERSION
cyvlfeat\vl/host.h(280): warning C4005: 'isnan': macro redefinition
C:\Program Files (x86)\Windows Kits\10\include\10.0.17763.0\ucrt\corecrt_math.h(276): note: see previous definition of 'isnan'
cyvlfeat\sift\cysift.c(645): fatal error C1083: Cannot open include file: 'cy_util.h': No such file or directory
error: command 'C:\\Program Files (x86)\\Microsoft Visual Studio\\2017\\BuildTools\\VC\\Tools\\MSVC\\14.16.27023\\bin\\HostX86\\x64\\cl.exe' failed with exit status
2

Traceback (most recent call last):
File "setup.py", line 97, in
build_extension_from_pyx('cyvlfeat/fisher/cyfisher.pyx'),
File "setup.py", line 68, in build_extension_from_pyx
include_dirs.append(os.environ['LIBRARY_INC'])
File "D:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\os.py", line 681, in getitem
raise KeyError(key) from None
KeyError: 'LIBRARY_INC'

Traceback (most recent call last):
File "setup.py", line 97, in
build_extension_from_pyx('cyvlfeat/fisher/cyfisher.pyx'),
File "setup.py", line 68, in build_extension_from_pyx
include_dirs.append(os.environ['LIBRARY_INC'])
File "D:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\os.py", line 681, in getitem
raise KeyError(key) from None
KeyError: 'LIBRARY_INC'


Updated script params in workflow

Changes in this PR:
Params for script check-users-in-space-developer-role.ps1 in TV-check-users-in-space-developer-role have changed, updated.

 Refactor check-users-in-space-developer-role to PS coding standards


Add option to lint the buffer on change with flake8

Environment data

VS Code version: 1.33.0
Extension version (available under the Extensions sidebar): 2019.3.6558
OS and version: Windows 10 Pro Build 17134
Python version (& distribution if applicable, e.g. Anaconda): 3.7.3
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: flake8 3.7.7

Expected behaviour
There should be an option to lint the buffer when it changes/while typing. I'm flake8 user myself, so I'm talking specifically about flake8, but I think this isn't available for any linter (I haven't checked though), so this feature could be added to other linters as well. Currently, there's only Microsoft Python Language Server that can lint the buffer on change, but that doesn't allow you to use external linters like mypy or flake8.
Actual behaviour
Such option isn't available and the best you can do is enable autosave feature and enable lint on save, but this isn't really useful, if you don't want the file to be constantly saved and/or don't want the file to be saved in some random moment.
Explanation
This feature is considered pretty standard in other popular editors that can be considered competitors of VS Code like PyCharm, Atom or Sublime Text and I think would make great addition and would also make switching to VS Code a lot more fluent.
And since other editors already have this feature, this surely is possible, it might be good to also add configurable interval, which would prevent too frequent linter calls.

Duplicate of #408


chore: update Dockerfiles

https://github.com/fullboar/vc-showcase/issues/4
Updates frontend and backend dockerfiles, along with the Docker-compose file to run properly from base dir. Updates README.

@weslord

Handy for dev, but I think the goal here is to keep the Dockerfiles production-ready?


can we get some docs pls Dr. ?

also is python base pypi module packaging template on it?

> sentients beings come to life

Docs will come once I finish tearing apart the user interface!
For now, you should be able to piece together usage from:
pip install zetch
zetch init to auto create the zetch.config.toml file. The genned file contains a schema directive that should give intellisense to the config file, plus you can look at the schema json file in the link directly to see all the options.
zetch help then shows all the command line options. (zetch defaults to zetch render)
For the templating language, we use https://docs.rs/minijinja/latest/minijinja/
their docs should suffice for that part for now.


feature: Transparency colorscheme support

Did you check the docs?

[X] I have read all the drop.nvim docs

Is your feature request related to a problem? Please describe.
Will you be able to add transparant colorscheme support in the future plan?
Describe the solution you'd like
i use onedark colorscheme with transparency on
Describe alternatives you've considered
nothing special, maybe something that can related to colorscheme config that will make transparancy works
Additional context
My Alpha.nvim dashboard becoming black because it doesn't support transparency out of the box

Same goes for catppuccin/nvim with transparent_background=true. Seems to be an issue with winblend combined with an alpha value on the Normal highlight group. Probably related to the upstream neovim/neovim#18576.

Hi,

👁 I can see through 👁
That's Kitty & mini.starter
I'm not sure that related to Drop or Nvim, maybe it's your terminal (alacritty ?)

Hmm, how does your colorscheme define Normal?

I use ellisonleao/gruvbox.nvim

Alright, it sets Normal and NormalFloat as guibg=NONE, maybe we could (I'll do it tomorrow if nobody did, don't have time for it now) use a minimal vimrc which just installs ministarter, drop, and sets the minimal highlights for a transparent background (on the starter).

Hi,

👁 I can see through 👁
That's Kitty & mini.starter
I'm not sure that related to Drop or Nvim, maybe it's your terminal (alacritty ?)

yes i'm using alacritty as my default terminal, how does your kitty support transparency?

I'll do it tomorrow if nobody did

I'm so sorry, haven't gotten to it yet. I'll be back after my vacation and then I'll be on it ASAP

Just a friendly reminder @dtomvan, unless your vacation is still going on, if then, no worries!

Oh, yeah I am so sorry. Didn't take note of it properly. Thanks for the reminder!

the dark background and disabling transparency are happening for me too:

vs:

Any updates on this? Also getting a black background with transparent background.

Also having the same issue

Would love to see an update. I love the idea of this plugin but it just looks terrible in my terminal


Fails on raspbian stretch

raspbian stretch defaults to php7 so this fails. The best way I got it to work was to install, allow it to fail, then install nginx and php7, manually edit the default to enable php7 by commenting out the lines and adding index.php as a root.
The camera module I got apparently has some issues so I won't know if it's 100% before I get a new sensor, but the raspbian errors are real, so I thought I'd report them here.
It's not really PR ready as I've no clue which lines failed other than apt-get install lines, or what the setup does besides destroying nginx config (could rm /etc/nginx/sites-enabled/default instead of nuking it's contents).

Thanks. I am currently doing some tests with latest stretch raspbian to try to resolve the issues with this change.

I installed 6.3.15 on a fresh install of Stretch lite at the weekend and didn't experience any problems with PHP. dpkg -l php* shows:
php5            5.6.22+dfsg-2
php5-cli        5.6.22+dfsg-2+b3
php5-common     5.6.22+dfsg-2+b3
php5-json       1.3.9-1
php5-readline   5.6.22+dfsg-2+b3

Thanks for that report. Can you tell me which web server you chose (apache, nginx lighttpd)?
I am also experimenting changing the php to 7.0

@mfraser I was on stretch full, but I'm happy to retry with lite (shouldn't be any difference)

@roberttidey I am using apache.

I'm on stretch too with apache2 and it's working well.

@ebretteville & @mfraser are you both running the install script? What manual intervention are you performing?

Fresh install, update && upgrade, git pull and the script, thats it.

Seen it https://github.com/silvanmelchior/RPi_Cam_Web_Interface/commit/681bda993407c87ccf84f6ddc13e34da89c9ee2b. I'll try again and close if it fixes.

Also note that the latest install script has option to use php7 or php 5. I have only tested php7 with apache installs so far.

Will try to update next week from 6.3.14 to 6.14 and try to use PHP7 that require less ressources. We'll keep you posted

what I'm doing is baking an image before running install, so I can restore that, git pull, and retry. I'll run all the web servers under php7.
The initial idea was to get steps to setup without web-server etc, so it could be barebones dropped into an existing pi without playing around with web-server packages or config.

So I'm running this under stretch-lite as @mfraser mentioned that was working. I think it might be an idea to add --no-install-recommends to the apt-get install to keep it light (I'm happy to PR any suggestions btw)
The following additional packages will be installed:
  apache2-bin apache2-data apache2-utils ffmpeg fontconfig-config
  fonts-dejavu-core gpac-modules-base liba52-0.7.4 libaacs0 libapr1
  libaprutil1 libaprutil1-dbd-sqlite3 libaprutil1-ldap libass5 libasyncns0
  libavc1394-0 libavcodec57 libavdevice57 libavfilter6 libavformat57
  libavresample3 libavutil55 libbdplus0 libbluray1 libbs2b0 libcaca0 libcairo2
  libcdio-cdda1 libcdio-paranoia1 libcdio13 libchromaprint1 libdc1394-22
  libdrm-amdgpu1 libdrm-freedreno1 libdrm-nouveau2 libdrm-radeon1 libebur128-1
  libegl1-mesa libfaad2 libfftw3-double3 libflac8 libflite1 libfontconfig1
  libfreenect0.5 libfribidi0 libgbm1 libgl1-mesa-dri libgl1-mesa-glx
  libglapi-mesa libglu1-mesa libgme0 libgpac4 libgraphite2-3 libgsm1
  libgstreamer1.0-0 libharfbuzz0b libice6 libiec61883-0 libjack-jackd2-0
  libllvm3.9 liblua5.2-0 libmad0 libmariadbclient18 libmp3lame0 libmpg123-0
  libogg0 libopenal-data libopenal1 libopencv-core2.4v5 libopencv-imgproc2.4v5
  libopenjp2-7 libopenmpt0 libopus0 libpgm-5.2-0 libpixman-1-0 libpostproc54
  libpq5 libpulse0 libraw1394-11 librubberband2 libsdl1.2debian libsdl2-2.0-0
  libsensors4 libshine3 libsm6 libsnappy1v5 libsndfile1 libsndio6.1
  libsodium18 libsoxr0 libspeex1 libssh-gcrypt-4 libswresample2 libswscale4
  libtheora0 libtwolame0 libtxc-dxtn-s2tc libva-drm1 libva-x11-1 libva1
  libvdpau-va-gl1 libvdpau1 libvorbis0a libvorbisenc2 libvorbisfile3 libvpx4
  libwavpack1 libwayland-client0 libwayland-cursor0 libwayland-egl1-mesa
  libwayland-server0 libwebp6 libwebpmux2 libx11-xcb1 libx264-148 libx265-95
  libxcb-dri2-0 libxcb-dri3-0 libxcb-glx0 libxcb-present0 libxcb-render0
  libxcb-shape0 libxcb-shm0 libxcb-sync1 libxcb-xfixes0 libxcursor1
  libxdamage1 libxfixes3 libxi6 libxinerama1 libxkbcommon0 libxrandr2
  libxrender1 libxshmfence1 libxss1 libxtst6 libxv1 libxvidcore4 libxxf86vm1
  libzmq5 libzvbi-common libzvbi0 mesa-va-drivers mesa-vdpau-drivers
  mysql-common php-common php7.0-common php7.0-json php7.0-opcache
  php7.0-readline ssl-cert va-driver-all vdpau-driver-all x11-common
Suggested packages:
  www-browser apache2-doc apache2-suexec-pristine | apache2-suexec-custom
  ffmpeg-doc gstreamer1.0-plugins-base php-pear libbluray-bdj libfftw3-bin
  libfftw3-dev alsa-base jackd2 libportaudio2 opus-tools pulseaudio
  libraw1394-doc lm-sensors sndiod speex default-mysql-client
  postgresql-client openssl-blacklist
The following NEW packages will be installed:
  apache2 apache2-bin apache2-data apache2-utils ffmpeg fontconfig-config
  fonts-dejavu-core gpac gpac-modules-base gstreamer1.0-tools liba52-0.7.4
  libaacs0 libapache2-mod-php7.0 libapr1 libaprutil1 libaprutil1-dbd-sqlite3
  libaprutil1-ldap libass5 libasyncns0 libav-tools libavc1394-0 libavcodec57
  libavdevice57 libavfilter6 libavformat57 libavresample3 libavutil55
  libbdplus0 libbluray1 libbs2b0 libcaca0 libcairo2 libcdio-cdda1
  libcdio-paranoia1 libcdio13 libchromaprint1 libdc1394-22 libdrm-amdgpu1
  libdrm-freedreno1 libdrm-nouveau2 libdrm-radeon1 libebur128-1 libegl1-mesa
  libfaad2 libfftw3-double3 libflac8 libflite1 libfontconfig1 libfreenect0.5
  libfribidi0 libgbm1 libgl1-mesa-dri libgl1-mesa-glx libglapi-mesa
  libglu1-mesa libgme0 libgpac4 libgraphite2-3 libgsm1 libgstreamer1.0-0
  libharfbuzz0b libice6 libiec61883-0 libjack-jackd2-0 libllvm3.9 liblua5.2-0
  libmad0 libmariadbclient18 libmp3lame0 libmpg123-0 libogg0 libopenal-data
  libopenal1 libopencv-core2.4v5 libopencv-imgproc2.4v5 libopenjp2-7
  libopenmpt0 libopus0 libpgm-5.2-0 libpixman-1-0 libpostproc54 libpq5
  libpulse0 libraw1394-11 librubberband2 libsdl1.2debian libsdl2-2.0-0
  libsensors4 libshine3 libsm6 libsnappy1v5 libsndfile1 libsndio6.1
  libsodium18 libsoxr0 libspeex1 libssh-gcrypt-4 libswresample2 libswscale4
  libtheora0 libtwolame0 libtxc-dxtn-s2tc libva-drm1 libva-x11-1 libva1
  libvdpau-va-gl1 libvdpau1 libvorbis0a libvorbisenc2 libvorbisfile3 libvpx4
  libwavpack1 libwayland-client0 libwayland-cursor0 libwayland-egl1-mesa
  libwayland-server0 libwebp6 libwebpmux2 libx11-xcb1 libx264-148 libx265-95
  libxcb-dri2-0 libxcb-dri3-0 libxcb-glx0 libxcb-present0 libxcb-render0
  libxcb-shape0 libxcb-shm0 libxcb-sync1 libxcb-xfixes0 libxcursor1
  libxdamage1 libxfixes3 libxi6 libxinerama1 libxkbcommon0 libxrandr2
  libxrender1 libxshmfence1 libxss1 libxtst6 libxv1 libxvidcore4 libxxf86vm1
  libzmq5 libzvbi-common libzvbi0 mesa-va-drivers mesa-vdpau-drivers motion
  mysql-common php-common php7.0 php7.0-cli php7.0-common php7.0-json
  php7.0-opcache php7.0-readline ssl-cert va-driver-all vdpau-driver-all
  x11-common zip

Apache setup with defaults seems to create files in folder outside of web-root

As installed with script the folder is /var/www/myfolder where myfolder is defined in script set up (default html)

@roberttidey what I'm showing is that without an existing apache, with no change, other than to set php to 7 (as it was blank), the config for apache seems to put files into /var/www/html contents, but then relocate web root to /var/www, not /var/www/html

That is correct. The web root is deliberately left at /var/www. To access the camera one uses http://ip/html or whatever other folder one wishes to use.
The reasoning here is that some people also use the apache server for other accesses and want to keep those in different folders. One could use different port addresses to distinguish between these and at one point a couple of years ago it was like that but most people didn't want to put into different port addresses particularly if they were using port forwarding, SO it was changed to keep the web root constant.
For those, including myself, who use this as a single app device then one sets the folder to empty so the files go into /var/www and the access is http://ip

So it's meant to be that way for apache only? (I'm just trying to understand, raise issues related to stretch here as an epic, so nothing is missed, all is documented somewhere)

nginx default install with php7 seems to follow apache
default 403 for /

502 for /html (likely socket-file issue from virtualhost)

looks like https://github.com/silvanmelchior/RPi_Cam_Web_Interface/blob/master/etc/nginx/sites-available/rpicam.1#L16 is the culprit, adjusting fixes

Thanks. I have done that merge.

I think if the /html/ is intentional, I'm happy to close this :smile:

OK.


Confirmation Code Revocation

Moved from: https://wso2.org/jira/browse/IDENTITY-7064
In case of a forgery, we may need to expire the confirmation link, manually before the configured time (without resending the confirmation link).
Subtask of https://wso2.org/jira/browse/IDENTITY-7062

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


[QA] Share lists deleted user as 'user'

Setup via docker-compose-eos-test.yml from branch fix-yml-for-rc5 on localhost
From the WEB UI:

moss creates user testy2
marie shares folder testpic3 with testy2
moss deletes user testy2
marie inspects the sharing of her testpic3

An unnamed user is seen  in the share.
Expected behaviour: deleted user is leaves no traces in the shares he had received.

for QA-team: retest with owncloud & ocis storage drivers and add tests for this case (I think API tests should do because I guess the API does respond some wrong values)

marie inspects the sharing of her testpic3

Above action makes call to 2 different APIs,
API one (buggy 🐛)
curl --user moss:vista \
https://localhost:9200/ocs/v1.php/apps/files_sharing/api/v1/shares\?path\=/shareit\&reshares\=true

Expected behavior
<ocs>
   <meta>
      <status>ok</status>
      <statuscode>100</statuscode>
      <message>OK</message>
   </meta>
   <data/>
</ocs>

Actual behavior
<ocs>
    <meta>
        <status>ok</status>
        <statuscode>100</statuscode>
        <message>OK</message>
    </meta>
    <data>
        <element>
            <id>dba65967-73de-4f0a-95bf-904f9645f15f</id>
            <share_type>0</share_type>
            <uid_owner>058bff95-6708-4fe5-91e4-9ea3d377588b</uid_owner>
            <displayname_owner>Maurice Moss</displayname_owner>
            <permissions>15</permissions>
            <stime>1607673851</stime>
            <parent/>
            <expiration/>
            <token/>
            <uid_file_owner>058bff95-6708-4fe5-91e4-9ea3d377588b</uid_file_owner>
            <displayname_file_owner>Maurice Moss</displayname_file_owner>
            <additional_info_owner/>
            <additional_info_file_owner/>
            <state>0</state>
            <path>/shareit</path>
            <item_type>folder</item_type>
            <mimetype>httpd/unix-directory</mimetype>
            <storage_id>1284d238-aa92-42ce-bdc4-0b0000009157</storage_id>
            <storage>0</storage>
            <item_source>MTI4NGQyMzgtYWE5Mi00MmNlLWJkYzQtMGIwMDAwMDA5MTU3OmEwNjExMWMxLWJhYWUtNDRiZi1hZGRlLWQ2ZTMyYjY4YjRlZg==</item_source>
            <file_source>MTI4NGQyMzgtYWE5Mi00MmNlLWJkYzQtMGIwMDAwMDA5MTU3OmEwNjExMWMxLWJhYWUtNDRiZi1hZGRlLWQ2ZTMyYjY4YjRlZg==</file_source>
            <file_parent/>
            <file_target>/shareit</file_target>
            <share_with>0951297f-9ad5-4bab-b45e-339a0c764fad</share_with>
            <share_with_displayname>doe</share_with_displayname>
            <share_with_additional_info/>
            <mail_send>0</mail_send>
            <name/>
        </element>
    </data>
</ocs>

API two (works as aspected)
curl --user moss:vista \
https://localhost:9200/ocs/v1.php/apps/files_sharing/api/v1/shares\?path\=/shareit\&shared_with_me\=true

Expected and Actual Behavior
<ocs>
   <meta>
      <status>ok</status>
      <statuscode>100</statuscode>
      <message>OK</message>
   </meta>
   <data/>
</ocs>

the first request you are talking about if requesting all initiated shares the second one is getting all received shares. So the first request demonstrates the bug

Added Bug demonstration test in https://github.com/owncloud/core/pull/38205/files

Added Bug demonstration test in https://github.com/owncloud/core/pull/38205/files

re-tested with local server oCIS version 1.11.0.
It is reproduced

The issue is fixed and cannot be reproduced in the latest version so closing it.

The issue is fixed and cannot be reproduced in the latest version so closing it.

Test coverage was added in https://github.com/owncloud/core/pull/38205


Email formatting

I think, we need this formatting (or an ability of adding this by using blocks/delimiters).

@JWo1F could you provide an example of email formatting? Or you mean email validation?

hmm, don't think email would need any blocks/delimiters lol
Will close this for now, thanks.


pushing latest changes to live

Docs Build status updates of commit 61a661b:
:warning: Validation status: warnings

File
Status
Preview URL
Details

BingMaps/rest-services/routes/optimized-itinerary.md
:warning:Warning
View
Details

BingMaps/articles/bing-maps-on-azure-marketplace.md
:bulb:Suggestion
View
Details

BingMaps/rest-services/routes/calculate-a-truck-route.md
:bulb:Suggestion
View
Details

BingMaps/rest-services/routes/optimized-itinerary.md

Line 1, Column 1: [Warning-title-missing] Missing required attribute: 'title'. Add a title string to show in search engine results.
Line 4, Column 23: [Warning-yaml-header-syntax-error] While parsing a block mapping, did not find expected key.
Line 1, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'. NOTE: This Suggestion will become a Warning in late October, 2020.

BingMaps/articles/bing-maps-on-azure-marketplace.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'. NOTE: This Suggestion will become a Warning in late October, 2020.

BingMaps/rest-services/routes/calculate-a-truck-route.md

Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'. NOTE: This Suggestion will become a Warning in late October, 2020.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Add Generics RawServer and TypeProvider

Reference issue: #191.

More @fastify/typescript's area than mine. 😸


Lexical error at '\n'

According to @reactormonk, he's getting
GHC/Integer.hs:0:2: lexical error at character '\n' on compiling eta

This looks mostly like an encoding problem but we'll need to investigate.

LC_ALL is empty
╰─╼ env | grep LANG                                                                                ╾─[1]──┄
LANG=en_GB.utf8
LANGUAGE=

Archlinux:
Linux dynames 4.11.9-1-ARCH #1 SMP PREEMPT Wed Jul 5 18:23:08 CEST 2017 x86_64 GNU/Linux

btrfs filesystem with no special settings.

I took a look at this: https://wiki.archlinux.org/index.php/locale#LC_ALL:_troubleshooting
According to the examples shown, it seems that your LANG should actually be en_GB.UTF-8. I wonder if that missing - is causing problems!

To test that you can set LC_ALL=en_GB.UTF-8 in a terminal session and run ./cleaninstall.sh after that.

Same error. I should probably see if I package the problem into a VM.

Can you try setting LANG=C in a terminal session and running ./cleaninstall.sh?
And by the way, have you tried the binary installation?

Works meaning you're able to compile and run Eta programs now? I can add this to the docs if so.

... it somehow only ever works the first time, afterwards, I get the error. Not sure if it's connected to not specifying LANG=C at some point.

Not clear on what's going on - so it works the first time you run ./cleaninstall.sh and future times the build fails with the error above?

Exactly.

Are you doing LANG=C every time or just the first time? You'll need to export it to your environment whenever you work with eta programs for now. We should probably remove the locale-dependent bits in the compiler so it works regardless of locale.

It may be that not adding LANG=C somehow poisons the build, but I'm not 100% sure.

It may be that not adding LANG=C somehow poisons the build, but I'm not 100% sure.

Somehow I cannot reproduce this, do you still get the same error nowadays @reactormonk ?

Running ./cleaninstall.sh now works.

Sounds great! Feel free to reopen if it appears again 😄


[6.4.x] Workaround for dependency issues in OptaPlanner kie-server test

Importing the kie-platform-bom somehow confuses the KIE Maven builder
as it is not able to find the droolsjbpm-integration.pom during the
test execution. Since this branch (6.4.x) is no longer being supported,
it does not make sense to invest more time into proper solution.
@ksuta could you please take a look? I know this is not ideal solution, but its IMO good enough considering this branch is no longer actively maintained (+ the tests are working on 6.5.x and master).

+1


Reorder CMakeLists.txt

PR Type

[ ] Breaking change
[ ] Feature
[x] Patch

Brief Description

This PR reorders the CMakeLists.txt so the cmaize_add_package() call comes before our nwx_add_pybind11_module() call. This ensures that the package's install paths and rpaths are set up correctly before creating the Python bindings library.

@ryanmrichard I think this PR can be closed. Please reopen it if you think otherwise!


attempt to fix

Codecov Report

Merging #74 (337b09f) into master (3f8cfda) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #74   +/-   ##
=======================================
  Coverage   96.13%   96.13%
=======================================
  Files           5        5
  Lines         414      414
=======================================
  Hits          398      398
  Misses         16       16

Continue to review full report at Codecov.

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


Add a Gitter chat badge to README.rst

urda/nistbeacon now has a Chat Room on Gitter
@urda has just created a chat room. You can visit it here: https://gitter.im/urda/nistbeacon.
This pull-request adds this badge to your README.rst:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Moving a little fast aren't we gitter?


Add CSV data validation to bulk add endpoint

See https://github.com/multinet-app/multinet-girder/pull/5#issue-266334281

Pretty sure this isn't needed. CSVs uploaded without a _key field gain one during loading to Arango. CSVs without both a _from and _to field are simply assumed to be node data files. No further validation is really needed.
The only thing I can think is to display a warning if a file has only one of _from and _to, since this is most likely a broken edge definition table. But that is probably overkill.


Update skills section

Add personal skills

Java - SpringBoot | TypeScript - Angular | JavaScript - HTML - CSS | Node.js - Express.js | MySQL | MongoDB | Docker | RabbitMQ | Fast learning | Focus | Leadership | Teamwork


cleanup + add verify + add discard.email list

Removed old domains from banlist
Add discard.email public domain list file + parser
Add optional dns-verify method to check if hostname has MX record

Thanks for the PR @smeinecke


How to ensure track alignment on ABR

Hi! (You must think I am crazy at this point)
Well I am trying and succeding (sometimes) to create multiple qualities for a stream. It works with stream proccesses both with ffmpeg and AV. The problem is that consuming the streaming with HLS (hls.js) sometimes receives all the video tracks sometimes just the original one:
index.m3u8
#EXTM3U
#EXT-X-INDEPENDENT-SEGMENTS
#EXT-X-MEDIA:TYPE=AUDIO,GROUP-ID="aud",LANGUAGE="und-1",NAME="AAC-1",URI="1/index.m3u8?mTrack=2&iMsn=67&tkn=668057077"
#EXT-X-STREAM-INF:AUDIO="aud",CODECS="avc1.64002a,mp4a.40.2",RESOLUTION=1920x1080,FRAME-RATE=60,BANDWIDTH=10618598,AVERAGE-BANDWIDTH=8984967
2/index.m3u8?mTrack=2&iMsn=67&tkn=668057077
## NOTE: Track 3 is available, but ignored because it is not aligned with track 2.
## NOTE: Track 4 is available, but ignored because it is not aligned with track 2.

Sometimes the tracks are able to align and I get all tracks and so I am able to swap between them.
My Hls config in the frontend is this one:
const hlsConfig: Partial<HlsConfig> = {
  debug: false,
  enableWorker: true,
  lowLatencyMode: true,
  backBufferLength: 90,
  maxBufferLength: 300,
}

A stream proccess for 720p with ffmpeg looks like this:

Sometimes it just works, then you load the stream event and it doesn't anymore.
Is there any way to ensure track alignment even if it takes a bit for the player to load the stream??
Thank you and I know I am a pain in the ass!! Have a nice day

Hmm, are there multiple encodes on the same stream?
MistServer will by default take the "last" added live track and use that as a source. So you might need to mask every encode from processes. You can do that by setting the Source track mask to not be usable for processes. That way you can never re-encode from an already encoded stream.
Chances are that the times it worked sometimes are when all processes managed to start encoding from the original source track.
Normally you should be able to force keyframe alignment with the  -force_key_frames source which you set by  the keyframes Match input keyframes which does make it a tad weird that it is not aligning though...

It seems opting out on proccessing is not available
I opted out from the output track for proccessing but that didn't work, still getting misaligned tracks, maybe I have too many stream proccesses?

I think I am able to "fix" this for now by encoding the original stream in the same quality. Example OBS streams 1080p, I encode into another ouput with same quality. Those then are always (or at least seems to be) algined. The problem is you have one more proccess for each stream

The only problem facing know it's the bitrate it's absolutely ignored. For example I set 6000k minimun bitrate and output has 1200k, but I guess that's a problem for another day, and tomorow it's friday, so I guess another week haah

Heya,
I see I said it a bit confusing. So let me make it a bit more clear, as I definitely could do better there:
You need the Source track to only be available for processing
Then all the output tracks available for viewers (and optionally processing)

Even easier honestly is to just make the whole bitrate tree a single command through MKV-exec. It would be more reliable considering you can then at least force all those qualities to have the same keyframe interval. The multiple processes clearly isn't doing well for you:

ffmpeg -loglevel quiet -hide_banner -fflags nobuffer -i -  -map v:0 -map v:0 -map v:0 -map a:0  -c:v:0 h264 -b:v:0 5000k -s:v:0 1920x1080 -c:v:1 h264 -b:v:1 2000k -s:v:1  1280x720  -c:v:2 h264 -b:v:2 1000k -s:v:2 854x480 -c:a copy -force_key_frames source -cluster_time_limit 0 -f matroska -

Chances are your CPU will not be happy with this, you can try to alleviate that by setting -preset veryfast right behind -c:a copy or using a hardware encoder like h264_nvenc at every place it says h264 assuming you have an Nvidia card available.

That should generate a video with the following qualities:

1080p at ~5000kbps
720p at ~2000kbps
4380p at ~1000kbps

The reason you see the bitrate "ignored" is because FFmpeg will by see the bitrate as a target, if it's capable of doing the target quality with lesser bitrate it will. You can counter this with setting a constant bitrate.
That would be adding a -minrate X ,  -maxrate X and -bufsize x to every -b:v:0/1/2
-minrate X this is the target minimum bitrate,
-maxrate X this is the target maximum bitrate,
-bufsize X  This is the size of the buffer it can use to cut these up. Usually target 1 to 5 times the target bitrate in order to make sure the target bitrate can be met.
But honestly I would first try it without the minrate/maxrate/bufsize and see if you're happy with that quality.

Hi, sorry for the late reply. This absolutely fixes both the bitrate and track alignment. The only problem now It's I am not able to inhibit some track based on source width and height.
Before I would use your built-in solution like:
"source_track": ">1920x1080,~1920x1080",
But now with the single ffmpeg command I don't know how to approach this. Using ffmprobe seems to consume the stream and ffmpeg doesn't work

I went with that and it works great. One thing that would make this greate would be to be able to pass video input resolution to a custom script somehow. I tried using ffprobe and it feeds with the input video and ffmpeg does not work anymore. This works for certain fixed resolution but there is no way I know of to mantain both the input video resolution (transcoded for alignment) and new downscaled streamings.
Example: Input is 1536x864. Trancode 1536x864 and generate all downscaled streamings down to 854x480


integrate Hibernate Validator in example app

WDYT, should we do this?
So, on the one hand, it makes the build.gradle more complex.
On the other it shows that Hibernate Reactive still works with other stuff in the Hibernate ecosystem.
@emmanuelbernard?

P.S. @emmanuelbernard there is a pretty bad behavior in Hibernate Validator when a HV000183 error occurs because you don't have EL on the classpath.
Instead of logging a message it just throws a IntegrationException which is then immediately caught and swallowed by TypeSafeActivator.activate() with just the following debug level message in the log:
LOG.debug( "Unable to acquire Bean Validation ValidatorFactory, skipping activation" );

So not having EL in the classpath results in Hibernate Validator completely silently not working.

I can’t vouch for the build complexity but having and testing such example is really good. It avoids bad surprises late in the integration chain.

@gsmet you aware of this HV behavior of swallowing the EL missing exception ?

I can’t vouch for the build complexity

Well, it's two lines. Which isn't perfect, since it should be just one line.
But it's certainly manageable ;-)

I can’t vouch for the build complexity but having and testing such example is really good.

OK, I pushed it manually

IIRC, that's not an issue in HV, that's how they wanted it to work in JPA.
I haven't checked but my recollection is that setting javax.persistence.validation.mode to CALLBACK gives you a proper exception as it expects HV to be there.
If set to AUTO which is the default, if you have an error, it's just not enabled. I personally think it was a mistake but I was told it was supposed to work that way.
Also, I must admit I'm more and more leaning towards making EL a non-optional dependency (that you could exclude if you want and set up the ad hoc config) as it's needed with the default configuration. I never understood why it was optional considering you need it if you keep everything default.

If set to AUTO which is the default, if you have an error, it's just not enabled. I personally think it was a mistake but I was told it was supposed to work that way.

But in this case the problem wasn't thet bean validator was missing, it's that it was there, and one off its dependencies was missing.
I'm pretty sure that's got to be a visible error to the user.

Also, I must admit I'm more and more leaning towards making EL a non-optional dependency

Yes, I think it should be.

On Wed, May 27, 2020 at 6:00 PM Gavin King<EMAIL_ADDRESS>wrote:

If set to AUTO which is the default, if you have an error, it's just not
enabled. I personally think it was a mistake but I was told it was supposed
to work that way.
But in this case the problem wasn't thet bean validator was missing,
it's that it was there, and one off its dependencies was missing.

Yes exactly. As soon as you have an error bootstrapping HV, whatever it is,
it gets swallowed and the HV integration is disabled in ORM.
Not sure if it's the JPA spec mandating that or if someone thought it would
be a good idea. It's like that since far before I joined the project. I
remember Gunnar not being too happy about it either.
--
Guillaume

Well I have the spec in front of me. It says:

If the auto validation mode is specified by the validation-mode element or the javax.persis- tence.validation.mode property, or if neither the validation-mode element nor the javax.persistence.validation.mode property is specified, and a Bean Validation provider is present in the environment, the persistence provider must perform the automatic validation of entities as described in section <IP_ADDRESS>. If no Bean Validation provider is present in the environment, no lifecycle event validation takes place.

And almost exactly the same text is copy/pasted into the Javadoc of ValidationMode.
So it's definitely not the case that this behavior is mandated by the spec.

Well I have the spec in front of me. It says:

If the auto validation mode is specified by the validation-mode element or the javax.persis- tence.validation.mode property, or if neither the validation-mode element nor the javax.persistence.validation.mode property is specified, and a Bean Validation provider is present in the environment, the persistence provider must perform the automatic validation of entities as described in section <IP_ADDRESS>. If no Bean Validation provider is present in the environment, no lifecycle event validation takes place.

And almost exactly the same text is copy/pasted into the Javadoc of ValidationMode.
So it's definitely not the case that this behavior is mandated by the spec.

Right. It was not my intend to swallow "any issue" when I wrote that bit.

I have a vague recollection of a discussion where the issue was that we
didn't have a proper exception to differentiate between no implementation
and any other error.
I think we probably need to add a specific exception type in the spec so
that we can handle this case differently from other errors.
Again, vague recollection. And I'm on PTO so I will check that another day
:).
On Wed, May 27, 2020 at 9:18 PM Emmanuel Bernard<EMAIL_ADDRESS>wrote:

Well I have the spec in front of me. It says:
If the auto validation mode is specified by the validation-mode element
or the javax.persis- tence.validation.mode property, or if neither the
validation-mode element nor the javax.persistence.validation.mode property
is specified, and a Bean Validation provider is present in the environment,
the persistence provider must perform the automatic validation of entities
as described in section <IP_ADDRESS>. If no Bean Validation provider is present
in the environment, no lifecycle event validation takes place.
And almost exactly the same text is copy/pasted into the Javadoc of
ValidationMode.
So it's definitely not the case that this behavior is mandated by the spec.
Right. It was not my intend to swallow "any issue" when I wrote that bit.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hibernate/hibernate-reactive/pull/200#issuecomment-634883905,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJYOBJN64EQETHU24WHIQ3RTVRRPANCNFSM4NL77UFQ
.

At least we could make it work for HV, which is the only reasonable implementation anyways out there these days.

I have a vague recollection of a discussion where the issue was that we
didn't have a proper exception to differentiate between no implementation
and any other error.

Well note that all the code I'm seeing is completely internal to Hibernate Validator. I'm not sure if Hibernate ORM subsequently reswallows exceptions, I didn't get that far.

Sure, that could be an option if people are open to it.

IIRC, if you set the JPA validation mode to callback, you will have a proper error if you don't have an EL implementation. So it must be done in ORM.
I'll have a look when I'm back from PTO.


A7-1-2:  Query proposes to add constexpr to non-static data members (and other problems)

Affected rules

A7-1-2

Description
1. constexpr on non-static data members
This query should exclude variables that are non-static data members. This is because the compiler raises an error in case a non-static data member is marked as constexpr.
Example
class a_class
{
  int y;
  public:
  // CodeQL says to make 'x' as constexpr.
  // But when we do so, there is a compiler error: "error: non-static data member ‘x’ declared ‘constexpr’"
  int x = 10;
  a_class():y(x)
  {
  }
  int getY()
  {
    return y;
  }
};

int main()
{
  a_class o;
  return o.getY();
}

Fixing as per the alerts raised by CodeQL in the test case of this query (e.g. here or here) also raises the error.
test.cpp:55:3: error: non-static data member ‘m2’ declared ‘constexpr’
   55 |   constexpr int m2 = 0; // NON_COMPLIANT

test.cpp:130:3: error: non-static data member ‘m1’ declared ‘constexpr’
  130 |   constexpr int m1 = 0;

2. constexpr on member variables updated in un-instantiated templates
This query alerts to add constexpr to member variables which are only used in uninstantiated template functions. At first glance, it seems to be correct because since the template function is not instantiated, the member variable isn't getting accessed anywhere and hence becomes a candidate for adding constexpr. However, when the user attempts to fix it, the compiler raises an error and points to the modification in the uninstantiated template function. So should we avoid member variables of templates?
Example
#include <vector>
#include <memory>

template <typename T>
class a_class
{
  // CodeQL says to make "size_" as constexpr because the "add()" function below isn't instantiated and hence,
  // there is no access of "size_", but if we do this, the compiler complains that "size_" is updated in add.
  // "test.cpp:12:7: error: increment of read-only variable ‘a_class<T>::size_’"
  std::size_t size_ = 0;
  std::vector<T> values_{};
  public:
  void add(T arg) {
    values_[size_] = arg;
    ++size_;
  }
};

int main()
{
  a_class<int> o;
  return 0;
}

Note: The error about size_ being a non-static data member and constexpr also exists but even if we make it static, we cant fix the error because compiler complains about the update on size_ in the uninstantiated template function add().
3. constexpr for range-based for loop variables
I'm not able to provide a reproducible case but this query alerts on variables used in range-based for loop.
Example
void a_function()
{
  auto origin = std::vector<int> {1, 2, 3, 4, 5, 6, 7, 8, 9};
  auto values = std::vector<std::unique_ptr<int>>{};
  for (auto& value : origin) { // value can be constexpr
    values.emplace_back(std::make_unique<int>(value));
  }
}

The reason is that when the range based for loop is de-sugared there is an assignment like the following:
for (..)
auto &value = *(__begin) // operator call to * with __begin variable generated by the compiler front end.
...

We have an operator* as an initializer expression and it has a variable access to the __begin variable. The line;  “any(Call call | isCompileTimeEvaluatedCall(call))...” here is True for this case because the operator *  is marked as constexpr by CodeQL and the getAnArgument() here for this call doesn’t return any arguments. Please note that, the qualifier for this call expression is the __begin variable which is compiler generated. So should we avoid such calls which has a qualifier that has VariableAccess to variables that are compiler generated?

I've a fix for (1) and (3). Requesting @lcartey to provide valuable comments!

For (1), I think we should report non-static members, but only if they are genuinely never modified. This is because the rule states:

The constexpr specifier shall be used for values that can be determined at compile time.

And the value can be determined at compile time if it's not genuinely modified. You are correct in saying that the user will also need to make the variable static in this case, in order to get the program to compile. We could potentially update the message and/or help file to explain this.
For (2), CodeQL doesn't generate template members unless it needs to, so we don't get visibility into that code unless add gets called. The change I think we should make here is not reporting member variables as constexpr candidates if there exists a member function for which we don't have a definition - as we don't have the information in these cases to make the decision on whether it can be constexpr or not.
For (3), I think we discussed this separately, but the for-range-loop generated variable is wrongly flagged because we don't check that qualifiers of calls are isCompileTimeEvaluatedExpressions, which leads us to the wrong conclusion. There's a separate issue that these variables aren't marked as compiler generated, which I have been unable to replicate in unit tests to date.

Thanks @lcartey for the excellent feedback! I'll modify the PR as per your suggestions!

Kindly check the updated PR: #794


Typescript types dependencies in standalone mode

Hi!
I'm trying to use @escape.tech/graphql-armor-max-depth as a standalone module, so I added it together with @escape.tech/graphql-armor-types, and imported it like so import { maxDepthPlugin } from<EMAIL_ADDRESS>
So I'm wondering, is this module supposed to be used with skipLibCheck: true in the tsconfig.json, should it actually work out of the box, or am I doing it wrong?
Thanks!

First thing to do is to make sure you install that dependency:
npm install @escape.tech/graphql-armor-max-depth
Make sure you customize the limits when declaring/defining armor:
const armor = new ApolloArmor({
costLimit: {
enabled: Boolean(LimitENABLED),
maxCost: Number(maxCount),
objectCost: Number(objectCount),
scalarCost: Number(scalarCount),
depthCostFactor: Number(depthFactorCount),
ignoreIntrospection: Boolean(ignoreIntrospectionENABLED),
},
maxAliases: {
enabled: Boolean(maxAliasesENABLED),
n: Number(maxAliasesCount)
},
maxDirectives: {
n: Number(maxDirectivesCount)
},
maxDepth: {
n: Number(maxDepthCount)
},
maxTokens: {
n: Number(maxTokenCount)
}
});

Hey @Marsup,
I seemed to have overlooked one.
Could you please install the optional dependencies @escape.tech/graphql-armor-types?
Let me know if that resolves the issue!

@simpyparveen the module is present, the other aren't, rightfully so, since I'm not going to use them.
@c3b5aw it is already installed, it is the cause of the error, since it imports modules that aren't declared in its package.json and that I haven't installed since I have no use for them.

@simpyparveen the module is present, the other aren't, rightfully so, since I'm not going to use them.
@c3b5aw it is already installed, it is the cause of the error, since it imports modules that aren't declared in its package.json and that I haven't installed since I have no use for them.

Alright, any fix in mind ? Otherwise, I'll try to reproduce during the week.

I am using ApolloServer, plugins on Escape's docs are for Envelope. I missed that.

Setting skipLibCheck to true in your tsconfig.json file can potentially help resolve TypeScript type checking issues, but it's important to understand what this option does. This will make TypeScript skip type checking of declaration files (.d.ts files) that are included as part of all your dependencies.

If the module you are trying to use does not come with its own type declarations, TypeScript might not be able to infer the types. In this case, you might need to provide your own type declarations or use TypeScript's declare keyword to declare the module type. Create a file with a .d.ts extension in your project and declare the module.

Try clearing the TypeScript cache by deleting the node_modules folder, the package-lock.json (or yarn.lock) file, and then reinstalling your dependencies.

Inactive thread, closing.


Adding Sitemap to the Layout

It will help improve the SEO score of websites.
P.S. - It is also a good idea to add it to the robots.txt, but we can't use relative paths there.

As you said, specifying a sitemap in a robot.txt file would be better. But then, users would need to manually add the sitemap path. So, it's better for users to specify sitemap in robots.txt file on their own.
Thanks for your contribution.


💾 Optimize Memory Usage

Optimize memory usage, especially in the chunked processing, to reduce the program's overall footprint. CPU usage optimization is optional but is greatly appreciated if implemented.
Hint:

psutil might work really well

Not really needed, the app seems to manage memory without any issues for now.


Refactor exceptions, code cleanup, and move to delegate auth state changes

What kind of change does this PR introduce?
From a high level, this PR changes the following:

Dramatically simplifies the exception handling.
Streamlined error mapping between server errors and the new client exception
Simplified the persistence plugin functions
Fixes & hardening for auth state changes
Adds a bunch of additional tests, including tests for auth state changes and persistence
Lots of misc doc fixes
Replaces the console debug write with a function to listen for debug output
Added info on the changes to the README

Sorry for a lot of changes all at once, hopefully this isn't a giant PITA to review.
I think the most "controversial" thing is tweaking the persistence functions to no longer be async. I decided to remove that as the data involved is tiny, the main thread calling refresh is already running on another thread, and it makes it a lot easier to write the persistence plugin. Could go either way on this one.
I did run all the test suite against the local docker setup. :)

Awesome. :)
A few As & Qs...
I moved away from EventHandlers because they don't have any way to provide type information, and there are some other language/syntax quirks. I believe that EventHandlers were originally introduced to support some of the UI builder tools back in the day when MS was a lot more interested in that sort of thing. At least in the Unity community they are pretty much considered deprecated nowadays. Switching to straight delegates brings all the usual type safety gains.
RE switching to ISupabaseSessionHandler.cs absolutely, realized I should do that in the shower this morning lol. Keeping them simple delegates and adding this interface will make it lol easy to implement persistence.
Since we are breaking stuff anyways, I think I'm going to change the method names from Persistor to just SaveSession/LoadSession/DestroySession. I think it's easier/clearer plus apparently the spell-checker in my IDE complains about Persistor lol.
RE: SendMagicLink vs SignIn, I found it a bit confusing as the method looked like it was the same as a regular sign in with email. I can change the method name back, but as a user how do I know which is which? This might just be something to add to the doc for the method?
I think I'd like to add one extra method to the API to expose as well, the http://localhost:9999/settings endpoint. I'd like to write a Unity component to make it really easy to get set up with Supabase and this method will make it a lot easier for folks getting started (I can add a widget in the Editor to validate the config is correct).
I'll go ahead and make the changes and send 'em over. I will go ahead and change the method signatures back to SignIn and take a stab at adding my interpretation of what's going on in the flow.

I'm doing a code review one more time and I'm noticing that the SignIn method behavior is inconsistent RE: wiping out the session as a side-effect. Is the general idea that any sign in should wipe out session, or not...?

Ok, I think I got everything. Take a look, LMK if you have any Qs. FWIW I tried flipping the persistence stuff to use an interface, but delegates are not methods but instance variables and so the interface wouldn't work unless I flipped them to methods (adding more indirection complexity) and/or an abstract class (which wound up just being three methods and a constructor anyways).
The way it works now still works nicely with automatically generating types in the IDE and it's all typesafe, so I think it's fine ¯_(ツ)_/¯
If you think it looks good I think my next step is testing it out in Unity. I'm hoping to release a demo Unity project that will make it dead simple for someone to just copy and paste their Supbase config into Unity, hit a validate button in the Editor that will ping the new settings, and they'll be off to the races. Hopefully that will be a very short, easy video too. Then I'll add a second video showing the same thing but with Sign in with Apple.
 Oh well.

Ok, now that I'm using the full Supabase I see how the options interact a bit. Minor tweaks to make that easier to work with.
Few other minor things: I think that an abstract class for the persistence class works well. Also adding debug implementations of the various handlers that just write to console to make it easier for folks getting started.
Hoping to do these tonight/tomorrow.

Okay - I've thought about the delegate changes you proposed all day and I can get behind it!
I'll just have to transfer these paradigm changes to the other repos before doing a release upstream on supabase-csharp as these are going to warrant a semver major release (realtime-csharp has needed some love for a little while anyways and the delegates ought to clean up a lot of the code there!)
Good!

lol you were totally right RE: the persistence interface. Was totally overthinking/overcomplicating it w/delegates, etc. It's now just a simple interface.

Ok, last tweak for now. I moved setting the persistence from the client options to a method. This allows me to set the persistence without relying on the parent Supabase libs to tweak the options there for testing.

Cool, anything you need me to do at this point?
I think the main thing I'd like to do next is beat on it a bit more pointing at a live project to stress a bunch of edge cases. In particular, to make sure the flow works nicely for things that aren't covered by the docker stuff, like the flow for users if auto-confirm is turned off. I also think it would be good to add in some more errors to cover stuff like incorrectly configured nonce values coming back. Mostly I'm hoping those will just be adding more scenarios to the reasons file in the exception. IMHO those can be another PR if you want to close this one, they shouldn't break contracts just add some more hints for when things go sideways.

Nope - thanks so much. You've been a tremendous help!


Support different Hosting Environments

Is your feature request related to a problem? Please describe.
Actually only standard hosting environment is support. So Connect-MS365 can not be used for Germany Cloud or Chinese Cloud
Describe the solution you'd like
Provide additional parameter to define if other environment to be used.
Additional context

change ConnectionUri for New-PSSession

For Connect-SPO use -Region parameter
Gültige Werte sind: default | ITAR | Deutschland | China
https://docs.microsoft.com/de-de/powershell/module/sharepoint-online/connect-sposervice?view=sharepoint-ps#description


Errors on npm start. Dependencies mismatch?

Followed the guide but got issues on the last part.

On the web folder run npm start, and I get this:

D:\Portable Apps\broadcast-box\web>npm start

><EMAIL_ADDRESS>start
> dotenv -e ../.env.development react-scripts start

Browserslist: caniuse-lite is outdated. Please run:
  npx update-browserslist-db@latest
  Why you should do it regularly: https://github.com/browserslist/update-db#readme
(node:13972) [DEP_WEBPACK_DEV_SERVER_ON_AFTER_SETUP_MIDDLEWARE] DeprecationWarning: 'onAfterSetupMiddleware' option is deprecated. Please use the 'setupMiddlewares' option.
(Use `node --trace-deprecation ...` to show where the warning was created)
(node:13972) [DEP_WEBPACK_DEV_SERVER_ON_BEFORE_SETUP_MIDDLEWARE] DeprecationWarning: 'onBeforeSetupMiddleware' option is deprecated. Please use the 'setupMiddlewares' option.
Starting the development server...
Compiled with warnings.

[eslint]
src\App.js
  Line 1:17:  'useContext' is defined but never used         no-unused-vars
  Line 6:22:  'CinemaModeContext' is defined but never used  no-unused-vars

src\components\header\index.js
  Line 6:9:  'navigate' is assigned a value but never used  no-unused-vars

Search for the keywords to learn more about each warning.
To ignore, add // eslint-disable-next-line to the line before.

WARNING in [eslint]
src\App.js
  Line 1:17:  'useContext' is defined but never used         no-unused-vars
  Line 6:22:  'CinemaModeContext' is defined but never used  no-unused-vars

src\components\header\index.js
  Line 6:9:  'navigate' is assigned a value but never used  no-unused-vars

webpack compiled with 1 warning

My goal is to broadcast to a class using WLAN, and encode at the same time so they have a repository to rewatch the class.
Also no WebRTC option in latest OBS Studio (v30.0.0)? Should I use WHIP instead?

Hi @Dogway! Sorry I didn't get to this sooner. I fixed those eslint errors. I am also going to add something to Github Actions to make sure that doesn't happen again.
You did everything right. When you opened http://localhost:3000 did you see Broadcast Box? Were you able to point OBS at it?
I am going to close hoping that everything worked :) If not please re-open or join the discord to debug together https://discord.gg/An5jjhNUE3

Thanks, I will check this on Monday on the office and let you know then.

Yes, it worked now locally. Didn't test on a local network yet, maybe for a classroom I'd need to stream to internet though.

Great to hear @Dogway !
the second you hit issues reach out, I am here to help

Ok, I tested over LAN from Laptop to my Desktop, and also over WAN from Laptop to the smartphone. Both had the same error.

In OBS I set the server as http://localhost:8080/api/whip
Stream Key as StreamTest and start streaming
In a browser on the network's guest PC I access broadcast box through the IP given by "npm start", like: http://<IP_ADDRESS>:3000

It actually shows broadcast box panel but when I input the key and click Watch, this is the error I get in both cases (LAN and WAN)


Update dependencies automatically

Update dependencies in "Update package" workflow. See it working here:
https://github.com/Ana06/VM-Packages/pull/21
Allow to also trigger the workflow manually. The workflow still runs automatically on Wednesdays.
Note that the script only update fixed dependencies and many of our dependencies are not fixed. The script relies on metapackages having just one dependency with same name and version (only first 3 segments). I have renamed notepad++ to follow this. We should document this. Should we rename/adapt other packages?

The naming and version fixing should be documented and optionally checked via the linter.


Implement custom_metadata interop test in PHP

Turns out PHP doesn't expose the trailing metadata in the API, so this interop test is currently unimplementable in PHP. Implementing that interop test is a good check that we actually expose the right API.

Also see #6580.


Add better error message if parent path exists but isn't a directory

Fixes #433
Instead of a confusing error message like:
AttributeError: 'SQLiteDict' object has no attribute '_local_context'`

Show this error message:
FileExistsError: Parent path exists and is not a directory: <path>

This applies to SQLiteDict, SQLitePickleDict, and FileDict.
This also creates _local_context earlier in SQLiteDict.__init__() so any other potential initialization errors won't attempt to reference _local_context before it's created.

Codecov Report

Merging #434 (74d183a) into master (e9f425d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #434   +/-   ##
=======================================
  Coverage   98.72%   98.72%
=======================================
  Files          17       17
  Lines        1173     1176    +3
  Branches      144      144
=======================================
+ Hits         1158     1161    +3
  Misses          5        5
  Partials       10       10

Impacted Files
Coverage Δ

requests_cache/backends/sqlite.py
98.71% <100.00%> (+0.02%)
:arrow_up:

Continue to review full report at Codecov.

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


[SPARK-43934][SQL][PYTHON][CONNECT] Add regexp_* functions to Scala and Python

What changes were proposed in this pull request?
This PR want add regexp_* functions to Scala, Python and Connect API.
These functions show below.

rlike

regexp

regexp_count

regexp_extract_all

regexp_instr

regexp_like

regexp_substr

Why are the changes needed?
Add regexp_* functions to Scala, Python and Connect API.
Does this PR introduce any user-facing change?
'No'.
New feature.
How was this patch tested?
New test cases.

ping @HyukjinKwon @zhengruifeng cc @cloud-fan

looks pretty good, let's wait for the discussion https://github.com/apache/spark/pull/41516#discussion_r1225030166 , since regexp_like and regexp are alias to rlike

@HyukjinKwon @zhengruifeng The GA failure is unrelated to this PR.

would you mind rebasing to the lasted master?

Has been rebased. Please wait for GA passed.

merged to master

@zhengruifeng Thank you very much!


BottomNavigationBar Icon mechanism is too restrictive

Framework revision 9c1a24fa72 (4 days ago), 2016-11-18 21:22:32
Engine revision cea5ed2b9b
Tools Dart version 1.21.0-dev.6.0

Currently it is very difficult to set an image asset other than an Icon for the BottomNavigationBar. There doesn't seem to be a way to convert an AssetImage to an Icon and DestinationLabel requires an Icon value.
From Gitter (@abarth), re: DestinationLabel:

@required Icon icon
looks like an oversight
one sec
hum... it needs to determine the size of the icon as part of its animation

It would be great if we could either use a "generic" image asset of some kind or be able to use a widget that supports image rendering.

ImageIcon is supposed to address this need, we'll have to make BottomNavigationBar allow any widget and then find another way to read the size. Or alternatively, have Icon and ImageIcon implement some interface that exposes the size, and have it require that instead.

Regarding the Icon and ImageIcon: is the intention that they always be square? In the case of ImageIcon it appears that it uses ImageFit.scaleDown as the fit mode. Not sure if there is a benefit to allowing non-square images.
Regardless, would it just make sense for both classes to extend a common interface/class so that size could be returned for both?

Yeah, the intent is that icons be square.
Exactly how to get the size isn't something I've looked into. If it's possible to get the size during layout that's probably the right solution rather than limiting what widgets go here.


Updated Docker container build

Hi, I've made an updated docker container build to take advantage of the new ENV variables.
FROM alpine:latest

ENV MANGAL_VERSION=v2.1.1
ENV MANGAL_CONFIG_PATH=/config/mangal.toml
ENV MANGAL_DOWNLOAD_PATH=/downloads
ENV MANGAL_USER=abc
ENV MANGAL_UID=1000
ENV MANGAL_GID=1000

WORKDIR "/config"

RUN mkdir -p "${MANGAL_DOWNLOAD_PATH}" && addgroup -g "${MANGAL_GID}" "${MANGAL_USER}" && adduser \
    --disabled-password \
    --gecos "" \
    --home "$(pwd)" \
    --ingroup "${MANGAL_USER}" \
    --no-create-home \
    --uid "${MANGAL_UID}" \
    "${MANGAL_USER}" && \
    chown abc:abc /config "${MANGAL_DOWNLOAD_PATH}"

ADD https://github.com/metafates/mangal/releases/download/${MANGAL_VERSION}/mangal-linux-amd64 /usr/local/bin/mangal

RUN chmod +x /usr/local/bin/mangal

USER "${MANGAL_USER}"

ENTRYPOINT ["/usr/local/bin/mangal"]

I think MANGAL_CONFIG_PATH is kind of incorrectly named, as of v2.1.1 it points to a FILE directly not a PATH, this might cause misinterpretation as i did when i was updating my docker container. I think it should take PATH which gives you the possibility in the future to migrate config files when breaking changes happens.
For example, right now config is at mangal_v1.toml, in the future you make breaking change, you can create mangal_v2.toml and migrate settings from mangal_v1.toml.
Also, making more config options configurable via ENV variables is really helpful. prime example at least in my case is formats.default i always set it to cbz

Hi! You mean making MANGAL_CONFIG_PATH point to the config directory and then mangal will choose which config file to use?

Hi! You mean making MANGAL_CONFIG_PATH point to the config directory and then mangal will choose which config file to use?

In essence yes, it should point to directory instead of a file.

Okay, that makes more sense, agree. Will do!


Give all submit buttons a submit attribute

IE7 needs submit buttons to have an attribute of type="submit" for them to work and a lot of ours don't.
This fixes that by adding the necessary attribute where needed.

These should be converted to use the button pattern from the toolkit which fixes this bug already.

@quis switched to use those partials in 310835a

:+1:
You should give @andrewchong a heads up before you merge/deploy this because it's very likely to have an impact on the tests.

@quis I've had a look through the functional tests and, as far as I can see, the targeting of buttons is limited to the following methods:

has_button?
click_link_or_button
have_button

...all of which use the :button or :link_or_button selectors, the button bit of which matches either input or button tags.
(See https://github.com/jnicklas/xpath/blob/59badfa50d645ac64c70fc6a0c2f7fe826999a1f/lib/xpath/html.rb#L22).
I'll still talk to @andrewchong but thought it was worth checking first.

@tombye I think we are fine with this in terms of impact on the functional test. I believe I have already addressed the button issue from the last time @quis brought my attention to the changes he made. If there is any impact from your changes the resolution will not be a big problem.


(refactor): display name fix

Describe your changes :

I worked on fixing the display name issue and also showing the user profile picture.

Type of change :

[] Bug fix
[x] Improvement
[] New feature
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] Documentation

Frontend Preview (Screenshots) :

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

Closes : #5651

@harsh-vador I can't see profile pic related changes. Can you confirm that?

@harsh-vador I can't see profile pic related changes. Can you confirm that?

The profile picture is associated with the username, so changes made above automatically syncs with the profile picture of the user.


while testing teleoperation, the joints of puppet robot are twitching

Hello,
First and foremost, I want to express my gratitude for your valuable work on the tutorial.
I followed the quick guide to install the software and hardware, but when I tested teleoperation, the joints of puppet robotic arm kept twitching. What is the reason for this? Is there any solution available ?
I appreciate your prompt response and thank you once again for your assistance.
https://github.com/user-attachments/assets/bbdcfabf-7d18-4704-b154-67c5cf4335b2

OS: Ubuntu 20.04 + ROS 1 noetic


Uploader not sending data after transition to Atlas.

 I know a similar issue is closed, but now I have exactly the same problem after migrating to Atlas. My cell phone uploader cannot send any data the the DB. I can though upload svg values manually (and see them on the my NS site) using the script by:
https://github.com/cjo20/ns-api-uploader
I am not sure what really happened with the uploader. Any help is certainly appreciated. Thanks!
Uploader Error Code
Do not know if there is an error. Here are the last lines from the Heroku CLI:
2020-09-03T21:02:44.941092+00:00 app[web.1]: got data-received event, requesting reload
2020-09-03T21:02:45.082527+00:00 app[web.1]: GRANTED:  <IP_ADDRESS> api-secret api:treatments:read
2020-09-03T21:02:45.086667+00:00 heroku[router]: at=info method=GET path="/api/v1/treatments.json?find%5Bcreated_at%5D%5B%24gte%5D=2017&find%5Bcreated_at%5D%5B%24lte%5D=2020-09-02T20%3A57%3A10Z&find%5Bkey600%5D%5B%24regex%5D=ALARM&count=20" host=kiki-vigilante-last.herokuapp.com request_id=e9db5f84-85c8-40ea-8e8c-1506fcf35d0d fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=7ms status=200 bytes=1318 protocol=https
2020-09-03T21:02:45.235428+00:00 heroku[router]: at=info method=DELETE path="/api/v1/treatments/5f4e8bee5987e50805f2cbc8" host=website.herokuapp.com request_id=434d67cc-ba36-436b-a923-3151add424cc fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=3ms status=200 bytes=230 protocol=https
2020-09-03T21:02:45.235601+00:00 app[web.1]: GRANTED:  <IP_ADDRESS> api-secret api:treatments:read
2020-09-03T21:02:45.235674+00:00 app[web.1]: GRANTED:  <IP_ADDRESS> api-secret api:treatments:delete
2020-09-03T21:02:45.235900+00:00 app[web.1]: got data-received event, requesting reload
2020-09-03T21:02:45.392514+00:00 app[web.1]: GRANTED:  <IP_ADDRESS> api-secret api:treatments:read
2020-09-03T21:02:45.392666+00:00 app[web.1]: GRANTED:  <IP_ADDRESS> api-secret api:treatments:delete
2020-09-03T21:02:45.392936+00:00 app[web.1]: got data-received event, requesting reload
2020-09-03T21:02:45.393565+00:00 heroku[router]: at=info method=DELETE path="/api/v1/treatments/5f4e30845987e50805b05d95" host=website.herokuapp.com request_id=f97db33b-2608-469f-8d8e-dce68adf3c35 fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=5ms status=200 bytes=230 protocol=https
2020-09-03T21:02:45.543530+00:00 heroku[router]: at=info method=DELETE path="/api/v1/treatments/5f4e19135987e508059e4259" host=website.herokuapp.com request_id=b2279bed-0fd6-4048-a441-07b3b29842e2 fwd="<IP_ADDRESS>" dyno=web.1 connect=0ms service=3ms status=200 bytes=230 protocol=https
2020-09-03T21:02:45.543680+00:00 app[web.1]: GRANTED:  <IP_ADDRESS> api-secret api:treatments:read
2020-09-03T21:02:45.543756+00:00 app[web.1]: GRANTED:  <IP_ADDRESS> api-secret api:treatments:delete
2020-09-03T21:02:45.543956+00:00 app[web.1]: got data-received event, requesting reload
Steps to Reproduce (for bugs)

Severity Score
4

Are there any errors on the uploader log screen on the phone?

Thanks Pogman for the reply. I tried yesterday to upload a picture, but Safari did not let me. I am trying it now with Chrome.
I do not see any errors on the uploader, turning the whole thing more puzzling.
Thanks for your help!

Ok this looks like it could be a full mongo database. The limit is 500mb and can fill over time depending on pump type and features used. Nightscout will report a success when data is sent even in this situation.
Go into your NS website admin page and clean some of the old data out.

Thanks Pogman!
Sorry to have created a comment and probably closing it so rapidly. Yesterday, out of doubts, I had created an alternative NS site on my Heroku space, and after not being able to upload data, linked it to the same Atlas database. I could see on both sites the same data, but not upload any from the cell phone.
This morning, waiting for your suggestion (which most probably it was correct) I deleted the app I created yesterday, and suddenly it started to upload! I am not sure if my action liberated some space (it probably did), but the correlation with the repair is too strong.
Summary: I do not know what did I do to repair it, but it works now.
Again, thanks a lot for your support! And sorry for stealing some of your time.

Thanks Pogman!
Sorry to have created a comment and probably closing it so rapidly. Yesterday, out of doubts, I had created an alternative NS site on my Heroku space, and after not being able to upload data, linked it to the same Atlas database. I could see on both sites the same data, but not upload any from the cell phone.
This morning, waiting for your suggestion (which most probably it was correct) I deleted the app I created yesterday, and suddenly it started to upload! I am not sure if my action liberated some space (it probably did), but the correlation with the repair is too strong.
Summary: I do not know what did I do to repair it, but it works now.
Again, thanks a lot for your support! And sorry for stealing some of your time.

Thanks Pogman!
Sorry to have created a comment and probably closing it so rapidly. Yesterday, out of doubts, I had created an alternative NS site on my Heroku space, and after not being able to upload data, linked it to the same Atlas database. I could see on both sites the same data, but not upload any from the cell phone.
This morning, waiting for your suggestion (which most probably it was correct) I deleted the app I created yesterday, and suddenly it started to upload! I am not sure if my action liberated some space (it probably did), but the correlation with the repair is too strong.
Summary: I do not know what did I do to repair it, but it works now.
Again, thanks a lot for your support! And sorry for stealing some of your time.

Hello, I hav a similar issue with the uploader after migrate my mondodb to Atlas.
I update the nightscout app to 14.0.3 version, and then made the migration according to Mongo-Atlas web page instructions.
I made the changes adding the new VAR "MONGO_CONNECTION".
My nightscout site is available, the size of my DB is less than 100MB. But while I try to delete data as recommend above I receive an error message.
The error log of the 600 serie uploader shows this:

Any help?

Check out this guide direct from mLab and pay particular attention to the mongo string to set in heroku convig vars.
https://docs.mlab.com/how-to-migrate-nightscout-sandbox-heroku-addons-to-atlas/
It may help to join the "nightscout for medtronic" facebook group for help with transition problems.
The uploader log only knows what a nightscout server tells it and in your case all we get is the "internal server error" response,

Saludos. En mi caso, no se comunica con la bomba. Es la primera vez que conecto el Contour next link al móvil

@VanesaRL Can you please open a separate new issue with some more detailed information (messages from the pump, etc)? It looks like your problem has nothing to do with Atlas (this issue).


Monitor telemetry exporter: no way to report exception from consumer span.

Due to https://github.com/Azure/azure-sdk-for-js/commit/21ee8ad656a709bbfe86a0fcc54d3ac1361c6de5#diff-91578a36a9cd93bc063569087dd9e7421dd054203bbf445a117c780ad4192e4aR438 all exception reported from consumer span will be ignored.
In this commit you accept exceptions either from remote span or internal span. However consider scenario:

Kafka message arrives
you extract context from headers -> context with no recording span is created.  Span's isRemote  property is true;
you call startActiveSpan to create span for message handling using extracted context and kind = SpanKind.Consumer.
newly created span is no longer "remote" because it is child of remote span - so exceptions are ignored.

Yes, we can set kind to SpanKind.Internal, but it breaks visual span representation in UI and semantics overall...
Note, that in this context we have enough information to report "resolved" exception.

@vputsenko I believe based on the scenario you mentioned above, in point 4 you say the "newly created span is no longer "remote" because it is a child of a remote span". The logic from line 438 here specifies that if the parent of the span is remote, that the exception will be created, regardless of if the span the exception is recorded on is INTERNAL. Please let me know if that logic covers your case as expected. Thanks!


:robot: triggering CI on branch 'release-next' after synching from upstream/main

/lgtm
/approve

/lgtm

/retest


fixed: wrong props to SDP onClose(#1674)

Fixes #1674.

Coverage remained the same at 84.14% when pulling fdc848b7c97f69e8301aed33e7e7bfe2d6b5c1a6 on AlokTakshak:issue-1674 into 11de2016bf6e7111ddebd5a7f7fc1fe929a3600c on airbnb:master.

Merged in 3290bb4


Importing/Migrating data from GoAccess < 1.4

As the title states, is it possible to import or migrate B+Tree data to the On-Disk Persistance Storage? I couldn't find any information about that. Is this possible at all?

Yes, I still have plans on putting something together, not sure if I can push it out on the upcoming release as is almost out the door. If not in v1.4.1, more likely on v1.4.2. Thanks for the heads up!


Adding a rate us page

A rate us page helps user to know about the website better, it enhances user interaction. Please assign me this issue under jwoc 2024.

A rating page would require a backend, which i am not planning on implementing in the project to keep it's simplicity intact.
Thus, a rating page is currently not suited for our page.


Updated README file

Just went through the process of creating a new test project which imports angular-patternfly, I noticed these items where missing from the README file.
@jeff-phillips-18 , @dgutride

LGTM


[Icon Request] Support for .cfg file

[x] I'm sure this issue is not a duplicate.

[x] I'm submitting an icon request:

Type: filename

Icon Name: cfg

Sample original Icon:

Filenames: example.cfg

More info: It's the same as an .config file, but shortened. config

What language are you using?

Not really a programming language, but Valve's Source engine.

This file extension is natively supported by vscode. https://github.com/microsoft/vscode/blob/master/extensions/ini/package.json#L21
You should see the icon. If not check what is messing with it.

https://www.streamable.com/7iwrh
Video that shows the problem. I think nothing's messing with it.
I tried uninstalling and installing, but it's still broken.

Try F1 > Icons > Activate VSCode Icons

https://www.streamable.com/vi74n

I don't and I can't know what's going on with your installation but as I have proved, cfg is already supported natively by vscode.
Some more advice I can give is:

disable all extensions except vscode-icons
check your user and workspace settings for any file association highjacking


medianalysis plist can no longer be deleted in OS X Monterey

I have the issue described in the troubleshooting section where everything works great for about 5 minutes, but then it freezes up to a slow stutter.
The file system where the mediaanalysis plist is located is not read only and I cannot find any way to delete that.

Hey there
Can you tell me which mac you have, so I can narrow it down a bit ?

Sorry, yes. I have a mac mini 2012 i7 Quad Core running macOS 12.2 (Using OpenCore Lgacy Patcher) to allow it to install. I ran launchctl unload /System/Library/LaunchAgents/com.apple.mediaanalysisd.plist just now and so far it is running smooth. However, I know I would have to do this every time I reboot. Not sure if anything else would reload that launchagent either.

Sadly there's not much you can do, it's just macOS trying to run stuff in the background and I have 0 control over it :(
Your best bet is launching photos once in a while so the import can go through. I believe that fixes things.
Switching from 4K to 1080p H.264 would probably fix this too but it's not a great solution.

For more information, this is a longstanding bug. What you did MAY have fixed it for a while, but no guarantee.  I knew it was not your fault. It has been running for 10 minutes or so flawlessly, so we will see and I can report back.
I am running at 1080p h264.

Is there a way to run that command automatically on startup?

I'm not 100% sure you need to, but I think the easiest way to do it is via automator, see here : https://stackoverflow.com/questions/6442364/running-script-upon-login-mac/6445525#6445525

Just an update. It has run for several hours after unloading that agent with no issues. So it seems almost a certainty that the service was the issue. I guess we'll see if is starts back up.

Ok, after running all night it worked perfectly after unloading. I was even able to run them perfectly in 4K HEVC.
Of course, as suspected the issue did come back when I rebooted so I did the following.

File > New, type Application

Add a Run Shell Script action, which adds an embedded bash script, and either paste

launchctl unload /System/Library/LaunchAgents/com.apple.mediaanalysisd.plist

Save the *.app bundle and add it to the Login Items list in System Preferences > User & Groups > Login Items.
Select the "Hidden" check box if you don't want to see the app pop up on startup

Note:
This is a per user solution so if you have additional users, you would need to add the app to each user's login items.
The embedded script runs with the default "C" locale.
$PATH is fixed to /usr/bin:/bin:/usr/sbin:/sbin, which notably does NOT include /usr/local/bin
The working dir. is the current user's home directory.

Ok, after running all night it worked perfectly after unloading. I was even able to run them perfectly in 4K HEVC.
Of course, as suspected the issue did come back when I rebooted so I did the following.

File > New, type Application
Add a Run Shell Script action, which adds an embedded bash script, and either paste

launchctl unload /System/Library/LaunchAgents/com.apple.mediaanalysisd.plist

Save the *.app bundle and add it to the Login Items list in System Preferences > User & Groups > Login Items.
Select the "Hidden" check box if you don't want to see the app pop up on startup

Note:
This is a per user solution so if you have additional users, you would need to add the app to each user's login items. The embedded script runs with the default "C" locale. $PATH is fixed to /usr/bin:/bin:/usr/sbin:/sbin, which notably does NOT include /usr/local/bin The working dir. is the current user's home directory.

Thanks a whole lot for following up, though it's terrible it came back it's great to have a solution. I'll try to put a link tomorrow on the FAQ to this as this may help many people.
Take care!

No problem. Thanks for pointing me in the right direction. Have a great one!


Use '//' for JSON comments.

Comments in any form aren't technically real JSON, so does it really matter? JSON only semantically accepts key-value pairs.

@ejona86 tells me that it's a common extension to support comments in JSON using //.

Google JSON style guide doesn't accept comments in real JSON in any form, though it does use comments in the document to explain things, which was, I think the purpose here as well, correct?

Yes, the purpose of these comments is to explain things.

Right, which was already achieved. Switching from # to // seems to be like using Vulcan rather than Klingon - neither is really appropriate anyway, so why the need?

The common convention for comments in JSON is to use //.  That's what we use in our other docs, and we should do the same here for consistency.

Alright, consistency justifies Vulcan over Klingon. LGTM.

Since this is a doc-only change, I'm not going to bother noting the test failures, since they're clearly unrelated.


Doc changes - Azure WebApps

Purpose of this pull request
This pull request...
Select the type of change

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions, updating sections
[x] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)
SUMO-243953

@ankurch627, I made some minor layout changes (see commit "Updates from review").
The dashboard images don't load in my local build. When I go to the URLs, it says "AccessDenied". Do you need to upload the images to S3 and make them public (like you needed to do for this PR)?

@jpipkin1 I'm seeing the same issue on my local build

@kimsauce, the dashboard images still don't load for me (see comment above). Work with @ankurch627 to ensure the images work before publishing this PR.

@kimsauce @jpipkin1  this one we can wait till tomorrow @ankurch627 is doing some changes and will reupload the screenshots

Hi @ankurch627 , since the dashboard images have been delaying the release of another document (September apps recap), I've temporarily commented them out and merged the changes. I'll go ahead and open a new ticket to address these dashboard images separately.
Thank you for your understanding!