Python requirement not satisfied (IDFGH-3074)

I am running on Windows 10 with 20191231 version of the MSYS GNU Make environment.
I updated to latest version of master (v4.2-dev-1097-g2e14149bf) about 2 hours ago.
Without having changed ANYTHING else I suddenly I have the following errors:

The following Python requirements are not satisfied:
gdbgui>=<IP_ADDRESS>
pygdbmi<=<IP_ADDRESS>

I have tried al the following commands:
pacman -Syu to update everything
pacman -Ss gdbgui  and  pacman -S  mingw-w64-i686-python2-gdbgui
pacman -Ss pygdbmi  and  pacman -S  mingw-w64-i686-python2-pygdbmi
The first option above returns no response.
The second options just return
error: target not found: mingw-w64-i686-python2-?????
Any suggestions?
All help appreciated.

Hi @ammaree
Installing this package: mingw-w64-i686-gcc in msys2 should help to run
python -m pip install -r "$IDF_PATH/requirements.txt"

without an error.
Another option is to run the actual tools/windows/windows_install_prerequisites.sh, which installs that package and all required python packages afterwards.

Same issue on v4.2-dev-1115-ga7f87f091 on debian, pip install running correctly but these packages seem to be missing.
Patched with python -m pip install --user pygdbmi==<IP_ADDRESS> gdbgui for now (and seems related to #5133 ?)

@ryankurte if you have a minute, could you please try what is suggested in https://github.com/espressif/esp-idf/issues/5133#issuecomment-614536173 and post the result in that issue? Thanks in advance.

@david-cermak
Step 1: To start the process I reverted back to Master from v4.1 and tried to build the application. Output as below.
Step 1 output.txt
Step 2: As per your suggestion I then ran ' python -m pip install -r $IDF_PATH/requirements.txt" ' console output below.
Step 2 output.txt
Step 3: In spite of the error messages I then tried to build again, the console output result from that being.
Step 3 output.txt
Step 4: I then ran 'pacman -Syu'  once system upgrade completed (without any errors) I terminated (not exit) session, restarted session and ran 'pacman -Syu' again. Again no errors
Step 5: I then, based on #5133 ran install.sh from my IDP directory which completed with an error after downaloading the latest 2020r1 tols again. Console output is below.
Step 5 output.txt
Ultimately, at the end of all this, the error is still exactly the same complaint.

ammaree@giga MINGW32 /c/Dropbox/devs/ws/z-appl/irmacs

make -j9 all
/c/Dropbox/devs/ws/z-support/ESP32_TFT_library/components/mkspiffs/Makefile.projbuild:13: warning: overriding recipe for target 'clean'
/c/Dropbox/devs/ws/z-sdk/esp-idf/components/app_update/Makefile.projbuild:53: warning: ignoring old recipe for target 'clean'
Toolchain path: /opt/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc
Toolchain version: esp-2020r1
Compiler version: 8.2.0
The following Python requirements are not satisfied:
gdbgui>=<IP_ADDRESS>
pygdbmi<=<IP_ADDRESS>
The recommended way to install a packages is via "pacman". Please run "pacman -Ss <package_name>" for searching the package database and if found then "pacman -S mingw-w64-i686-python2-<package_name>" for installing it.
NOTE: You may need to run "pacman -Syu" if your package database is older and run twice if the previous run updated "pacman" itself.
Please read https://github.com/msys2/msys2/wiki/Using-packages for further information about using "pacman"
make: *** No rule to make target 'check_python_dependencies', needed by '/c/Dropbox/devs/ws/z-appl/irmacs/build/include/config/auto.conf'.  Stop.
ammaree@giga MINGW32 /c/Dropbox/devs/ws/z-appl/irmacs

@igrr python is /usr/bin/python, version 2.7.16, building with this dockerfile, I don't have time to patch the lines from 5133 in rn but will get back to it when I can.

@ryankurte Ah okay, this issue looks simpler than #5133.
Looking at the dockerfile, you are only adding the toolchain and $IDF_PATH/tools to the PATH:
ENV PATH="$PATH:/root/esp-idf/tools/"
ENV PATH="$PATH:/root/.espressif/tools/xtensa-esp32-elf/esp-2019r2-8.2.0/xtensa-esp32-elf/bin/"

You aren't adding the path to Python virtual environment, which is created by install.sh That path will look like /root/.espressif/python_env/idf4.2_py2.7_env/bin. Once you add it to the PATH, python will be taken from the virtual environment, and the packages installed by install.sh will be found.
FWIW, the complete list of paths added by export.sh should look like this:
  /path/to/esp-idf/components/esptool_py/esptool
  /path/to/esp-idf/components/espcoredump
  /path/to/esp-idf/components/partition_table
  /path/to/esp-idf/components/app_update
  $HOME/.espressif/tools/xtensa-esp32-elf/esp-2020r1-8.2.0/xtensa-esp32-elf/bin
  $HOME/.espressif/tools/xtensa-esp32s2-elf/esp-2020r1-8.2.0/xtensa-esp32s2-elf/bin
  $HOME/.espressif/tools/esp32ulp-elf/2.28.51-esp-20191205/esp32ulp-elf-binutils/bin
  $HOME/.espressif/tools/esp32s2ulp-elf/2.28.51-esp-20191205/esp32s2ulp-elf-binutils/bin
  $HOME/.espressif/tools/openocd-esp32/v0.10.0-esp32-20200406/openocd-esp32/bin
  $HOME/.espressif/python_env/idf4.2_py3.7_env/bin
  /path/to/esp-idf/tools

By the way, we do have an esp-idf docker image which may satisfy your use case: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-guides/tools/idf-docker-image.html.

@ammaree
Sorry, not sure if you have actually installed the mingw-w64-i686-gcc via pacman -Ss as indicated before.
Could you please run one of the below commands?

pacman --noconfirm -S --needed mingw-w64-i686-gcc
./tools/windows/windows_install_prerequisites.sh (this script)
Apologies if you've already done so, but that wasn't clear to me from your previous post

Hi @david-cermak
I have done that many many times, BOTH options with the same result.
See attached console output.
New Text Document.txt

Here is output from second option, the script file.
Fix 2 script.txt
This time the error message at the end was different so I moved all the python27 files into a subdir and run the script again.
The script completed successfully, no errors, but when I tried to run "make -j9 all" it terminated complaining about not being able to find python27.exe and hence not being able to run the prerequisites checking script.
When I move the python27 files (all except python.exe) back into the C:\msys32\mingw32\bin directory the normal error, pasted below, returned.

#make -j9 all
Toolchain path: /opt/xtensa-esp32-elf/bin/xtensa-esp32-elf-gcc
Toolchain version: esp-2020r1
Compiler version: 8.2.0
The following Python requirements are not satisfied:
gdbgui>=<IP_ADDRESS>
pygdbmi<=<IP_ADDRESS>
The recommended way to install a packages is via "pacman". Please run "pacman -Ss <package_name>" for searching the package database and if found then "pacman -S mingw-w64-i686-python2-<package_name>" for installing it.
NOTE: You may need to run "pacman -Syu" if your package database is older and run twice if the previous run updated "pacman" itself.
Please read https://github.com/msys2/msys2/wiki/Using-packages for further information about using "pacman"
make: *** No rule to make target 'check_python_dependencies', needed by '/c/Dropbox/devs/ws/z-appl/irmacs/build/include/config/auto.conf'.  Stop.

So we are back right at the start

@ammaree
Thanks for trying out both options and posting results!
It seems that your MSYS2 package is based on some older image, where the default python.exe is still python2.7 and the problematic packages are having trouble compiling for python2.7
One way to solve the issues is to use the latest package https://docs.espressif.com/projects/esp-idf/en/latest/esp32/get-started-legacy/windows-setup.html#toolchain-setup with python3 being a default python.
It might be also possible to somehow set the python3 to be executed when running python, but would recommend updating the MSYS package.

@david-cermak I dont think this is the case.
Attached a screenshot of the msys ZIP file downloaded (and installed) on April 5th and again today.
As you can see both are the exact same base name and size.

After installing the new download the C:/msys/mingw32/bin directory contains the same python2.7

I follow the EXACT instructions as per the link you provided, again cloned the master repo and exactly the same error with the newly downloaded msys32 (as per 1st attachment) and esp-idf repo just download.

@ammaree Ah, right, the actual version is has python2 as default python (sorry thought of the upcoming package which should default to python3)
but running the update script, specifically the line https://github.com/espressif/esp-idf/blob/master/tools/windows/windows_install_prerequisites.sh#L37 should make the default one to python3.
In your case, however, this it is not happening for some  reason, as you've indicated above that the script finished okay only when python2.7 and related been removed.
Also seeing from your listing python.exe is has the exact size as python2.7.exe (and no python3 present!)
Could you please try to simply run:
pacman --noconfirm -S --needed mingw-w64-i686-python

and post the result here? if finished with errors, might also need to do pacman --noconfirm -Syu and restart the msys terminal before that.
Would be also interesting to see the listing c:\msys32\mingw32\bin for python*.exe files as posted above. (In my case, after this command, the default python.exe points to python3.8 /or is the exact copy of python3.8.exe)

Step 1:

pacman --noconfirm -S --needed mingw-w64-i686-python

warning: mingw-w64-i686-python-3.8.2-2 is up to date -- skipping
there is nothing to do
Step 2:

pacman --noconfirm -Syu

:: Synchronising package databases...
mingw32 is up to date
mingw64 is up to date
msys is up to date
:: Starting core system upgrade...
there is nothing to do
:: Starting full system upgrade...
there is nothing to do
Step 3:

ls -l /mingw32/bin/python*.exe

-rwxr-xr-x 1 ammaree ammaree 97280 Apr  9 16:12 /mingw32/bin/python.exe
-rwxr-xr-x 1 ammaree ammaree 16384 Apr 20 19:20 /mingw32/bin/python2.7.exe
-rwxr-xr-x 1 ammaree ammaree 16384 Apr 20 19:20 /mingw32/bin/python2.exe
-rwxr-xr-x 1 ammaree ammaree 16384 Apr 20 19:20 /mingw32/bin/python2w.exe
-rwxr-xr-x 1 ammaree ammaree 97280 Apr  9 16:12 /mingw32/bin/python3.8.exe
-rwxr-xr-x 1 ammaree ammaree 97280 Apr  9 16:12 /mingw32/bin/python3.exe
-rwxr-xr-x 1 ammaree ammaree 95744 Apr  9 16:12 /mingw32/bin/python3w.exe

@david-cermak
OK, I have found the problem. In "SDK tool configuration" option "Python interpreter" python2.7 was specified. By removing this the error went away.
I guess, since has been a very long ongoing project (since IDF 2.x), the configuration has never been completely restarted.
Thanks for the help

Same issue on v4.2-dev-1115-ga7f87f091 on debian, pip install running correctly but these packages seem to be missing.
Patched with python -m pip install --user pygdbmi==<IP_ADDRESS> gdbgui for now (and seems related to #5133 ?)

Or even better (this fixed it in my case):
sudo -H python -m pip install pygdbmi==<IP_ADDRESS> gdbgui==<IP_ADDRESS>

i have done all of the above. a few times. still getting:
The following Python requirements are not satisfied:
gdbgui>=<IP_ADDRESS>
did not found "SDK tool configuration" option "Python interpreter" python2.7
also python -m pip install pygdbmi==<IP_ADDRESS> gdbgui==<IP_ADDRESS> did not worked.
got all versions of python installed.
would appreciate any help... (but please no re-install install or whatever because this is just unhelpful atall, also judging by the fact that the users themselves usually provide espressif with working workarounds.... see entire section above.)
thanks in advance

did you try an newer version of python, for example python3?
You can choose which python version to use by typing
update-alternatives --install /usr/bin/python python /usr/bin/python3.8 1

where should i write update-alternatives?
to my understanding the "python virt env" is trying to use another version of python (instead of a newer one.) also tried to replace the python.exe (shortcut) with another version (2.7), errors changed and i am not even going to get into that...
would like to state again; "SDK tool configuration" option "Python interpreter" python2.7 not located. no such entry in the sdkconfig file.

when i role back to version 4.1 "everything" works, but now i have to change method calls such as ''esp_event_handler_instance_register'' which do not exist in version 4.1
this is honestly the most frustrating environment i have ever came across, and i am truly sorry to be so blunt!

huh, i resolved this waay back but it has suddenly recurred in all our CI builds somehow :-/

hey good people of espia, i got the issue resolved but cant figure out why\how exactly...
TBH i am using an extension for visual studio (not espressif eclipse, sadly the windows build was problematic for me but ill try to do it on a linux machine) and i believe much of the faulty behavior came from some mix of dependencies between the extension i am using and other toolchain artifacts of espressif.
in the end i just uninstalled everything i could think of that relates to espressif installation\python\python environment\CMAKE(for sure problematic, atleast for me on windows) and microsoft awful env variables, re install the standalone sysgcc toolchain, went to sleep, by morning things worked out.
one thing i cant point to for sure is the mixing of python environment on my pc, when i started playing with which python executable is getting the calls, errors began to change.
also i was using c++ and i don't think it was that wise to do, especially in the lower levels of the esp32 firmware app entry point.
keep up the good work, and stay strong <3
just to share my little small and narrow point of view, the toolchain itself is pretty much great, from gcc and openocd etc
the problems i was having were mostly to do with higher level of interfacing between me and the IDE of choice, cmake and such
keep up the good work, and stay strong <3

Looks like (at least in our release/v4.2 derived branch) there are contrary dependencies for gdbgui and pygdbmi in requirements.txt, resolved with the following patch:
diff --git a/requirements.txt b/requirements.txt
index 6a19513ad..f711ebd26 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -11,7 +11,7 @@ future>=0.15.2
 cryptography>=2.1.4
 pyparsing>=2.0.3,<2.4.0
 pyelftools>=0.22
-gdbgui>=<IP_ADDRESS>
+gdbgui==<IP_ADDRESS>
 pygdbmi<=<IP_ADDRESS>
 # The pygdbmi required max version <IP_ADDRESS> since <IP_ADDRESS> is not copatible with latest gdbgui (>=<IP_ADDRESS>)
 reedsolo==1.5.3

right on the money brother ryan, well put
saw this exact same error, made me reinstall sysgcc, maybe that was the ticker.
also noticed some probs with the python env processing of the requirements.txt, python was 2.7, switched it to 3..

Same issue on v4.2-dev-1115-ga7f87f091 on debian, pip install running correctly but these packages seem to be missing.
Patched with python -m pip install --user pygdbmi==<IP_ADDRESS> gdbgui for now (and seems related to #5133 ?)

Or even better (this fixed it in my case):
python -m pip install pygdbmi==<IP_ADDRESS> gdbgui==<IP_ADDRESS>

This solved my problem

Struggled with that for days. Solved for me too!


Unnecessary peerDependencies cause installation errors

Rebass now has peerDependencies for both styled-components and Emotion.
That means you now have to install both to avoid getting constantly yelled at by npm/yarn.
If you don’t have Emotion, this error is output by yarn:
warning "rebass ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning "rebass ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>
And if you don’t have styled-components, this is the error message:
warning "rebass ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| >=3.0".
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| >= 3.0".

The easiest fix for this would be to simply remove the peerDependencies, since there’s no reason to need both installed. You could also make separate rebass-emotion and rebass-styled-components that could peerDepend on their respecitive style backends.

Should be fixed in v2.3.0


ScrollView Problem

When I add FlexibleCalendarView inside a ScrollView the calendar has problem showing correctly

You need to specify some height to your calendar inside scroll view and it should work.


Web proxy, switch authentication to PAM

pam template for squid already pushed in https://github.com/opnsense/core/commit/ddca8d435043ffa748c95945bc0bf6efcab2a1f2


Adding APIService to the sort order

This ensures that APIService resources are installed last and uninstalled first.
This is still a work in progress.
Fixes #2641

This change is

@technosophos @adamreese @michelleN can one of you guys check this out and tell me if I'm on the right track?

@arschles Yes, this looks correct.

@technosophos ok, I've removed the WIP from this

@seh thanks for pointing me in that direction - I've made the change in https://github.com/kubernetes/helm/pull/2650/commits/58a2d7b5c125871cd21b0593a792f3706516b3da


Re-add datetime cast to fix date subtraction

Date subtraction to get frequency is failing since the values are strings. Adding the datetime cast works to get the forecast to work. We can find a more permanent solution later.

Looks like folks are busy, so I'm going to go ahead and merge this so that the recipe is working for visitors.

Sorry -- Yes, I had missed the ping to approve earlier.  Thanks for the fix.


ExtraClasspath support

Hi,
In jetty-maven-plugin, we can use extraClasspath to add extra classpath. Is it possible to do this in gretty? If not, could this new feature be added to gretty.
Rice

Just find that classPath does not work in tomcat. Is this tomcat's problem?
On Wed, Oct 1, 2014 at 8:40 PM, Andrey Hihlovskiy<EMAIL_ADDRESS>wrote:

Good that you found the solution.
Sorry for lacking documentation on this function, I'll add it as soon as I
can.
—
Reply to this email directly or view it on GitHub
https://github.com/akhikhl/gretty/issues/86#issuecomment-57456965.


Error in AnomalyDetection.rpca

Hi,
I am using the R version of Anomaly Detection and am coming across an error. Here are the details:
Error:
AnomalyDetection.rpca(ts)
Error in data.frame(X.transform = X.transform, L.transform = L.transform,  :
arguments imply differing number of rows: 231, 227
In addition: Warning message:
In matrix((j - j.global.mean)/j.global.sd, nrow = frequency) :
data length [227] is not a sub-multiple or multiple of the number of rows [7]
Here's how to reproduce it:
ts <- c(54,28,56,98,105,3,96,70,56,56,30,48,42,42,70,2,48,63,70,66,99,1,54,112,21,56,3,9,4,8,14,15,2,20,8,8,1,3,8,6,6,10,1,8,9,7,6,9,6,18,8,16,28,30,6,4,32,20,16,16,88,2,6,16,12,12,20,4,2,16,18,14,12,18,2,8,24,30,8,10,81,196,6,32,21,27,8,7,9,9,40,8,40,24,10,6,18,52,120,196,180,27,32,100,112,96,484,2,18,88,78,78,120,18,16,108,91,72,108,10,654,2169,1252,2480,4704,4995,612,174,3328,2940,2424,2504,8008,104,666,2184,1806,1866,2930,380,87,1736,2817,2135,1896,2736,187,18,90,40,80,140,150,30,20,160,100,80,80,132,3,9,24,18,18,30,6,3,24,27,21,18,27,3,117,76,152,252,285,39,4,256,190,144,144,572,2,36,104,102,114,170,34,13,136,162,140,126,171,15,258,513,404,808,1344,1425,270,152,912,1000,736,720,4312,86,168,776,564,576,990,188,86,472,918,728,576,819,89)
AnomalyDetection.rpca(ts)
Kindly let me know how the issue can be resolved. I am stuck on a project due to the above issue.
Thanks.

The full signature for AnomalyDetection.rpca is given below: You can see that it assumes the data has a periodicity (called frequency) of 7 unless another value is passed. Your data must be divisible by 7 if you don't pass a value for frequency when calling the function. I'd recommend using frequency=1 if you don't have an expectation of periodicity in your data.
AnomalyDetection.rpca(X, frequency = 7, dates = NULL, autodiff = T,
forcediff = F, scale = T, L.penalty = 1,
s.penalty = 1.4/sqrt(max(frequency, ifelse(is.data.frame(X), nrow(X),
length(X))/frequency)), verbose = F)

According to the documentation, 'If X is a vector it will be cast to a matrix of dimension frequency by length(X)/frequency'. Based on my understanding, setting frequency =1 will convert the input into a 1-d array. Does it make sense to perform PCA on 1-d array?

Can you pad the values with the median of the previous weeks if the data is not divisible by 7 or the periodicity?


Fix broken log search syntax

Purpose of this pull request
This pull request...Fix broken log search syntax

Select the type of change:

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

This broke links on https://help.sumologic.com/docs/search/search-cheat-sheets/log-operators/; see #3525 for more info.


Can't drag and drop file to Razor HTML editor to create link, script, or img tags

This issue has been moved from a ticket on Developer Community.

[regression] [worked-in:16.9]
In VS Enterprise 2019 v. 16.10.2, it is no longer possible to drag a JavaScript, CSS, or image file from Solution Explorer onto a .cshtml Razor HTML editor surface to automatically create a corresponding script, link, or img HTML tag.

Original Comments
Feedback Bot on 6/24/2021, 11:12 PM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Original Solutions
(no solutions)

Another link: https://devdiv.visualstudio.com/DevDiv/_workitems/edit/1414391

Still not working in Vs 2022

https://developercommunity.visualstudio.com/t/Drag-and-drop-not-working-in-html-for-sc/1616706?

Yeah this no longer working is really a drag. 🤣
But seriously, please fix. Thanks

kicking_the_can_down_the_road_meme.gif


[1.11.2] JEI integration completely broken.

With the latest Mekanism version (Currently 20cf847) JustEnoughItems fails to load almost all all recipes that are done in a crafting bench. This causes JEI's recipe viewer to not work with Mekanism.
JEI version: jei_1.11.2-4.3.2
Issues for reference: #4319 (Confirmed fixed original issue, doesn't fix JEI integration though.)

What do you mean with fails to load? In my workspace they show up and it had a + to move items to the workbench.

Downgrading JEI fix the recipes not showing. I'm using jei_1.11.2-<IP_ADDRESS> and everything is fine. Not sure why tho.

I saw there where some changes to the api between 4.2 and 4.3. I'll look what's changed and why it breaks compatability

Indeed seems to be some API change shenanigans.

Could be that Mekanism is using an outdated API which has been removed.

Well... Looking at the start-up log explains a lot:
https://gist.github.com/thommy101/833a45670461a39bed96b0109257c99e
There's something wrong with the implementation and/or the new API of JEI.

Opened an issue (help request) over at the repository of JEI. Hopefully someone can point in the direction to look as I don't see anything wrong (aside from using in the latest version deprecated stuff, which should still work in a grace period)

I looked into it more and Mekanism is using a JEI internal class, BrokenCraftingRecipeException.
The location of this class has changed, so Mekanism's integration broke.
I recommend removing the try/catch from mekanism.client.jei.crafting.ShapedMekanismRecipeWrapper#getIngredients since any crash there should already be handled by JEI.
Since Mekanism does not update very frequently, it's incredibly important that it avoids importing any JEI internal classes. If there are any imports of mezz.jei. that aren't mezz.jei.api. they can break, so they should be removed.

Then we have no Mekanism-Receips when we use the fresh .266 Release?
And when the JEI-Api completly is removed then must the standard very basic receipes showable, or?

Sorry I only gave a technical explanation of the issue and solution.
At a high level, Mekanism just needs to fix their JEI plugin.

No Problem was an important technical explanation. Good point for search.
Only jei_1.11.2-<IP_ADDRESS>.jar are usably at the moment with Mekanims 1.11.2 Dev-Versions.

Strange Things will be happend:).
jei_1.11.2-<IP_ADDRESS>.jar
will be shown as JEI <IP_ADDRESS>4 in MOD-Options and set automaticly S:filterText=@mekanism when i try to get an Mekanism-Recipe.
???

I have it resolved, i think my test jars from mekanism has caused this strange error:).

I've got it fixed locally. Removed all non API calls to JEI and changed the dependencies to compile against the API instead of the full jar of JEI.
I'll open a PR later today.

Fixed on my end, thanks a bunch!

Nice! Maybe a good idea to use the API as deobfcompile instead full JEI source. That way your certain that you don't use unwanted code.
https://github.com/mezz/JustEnoughItems/wiki/Getting-Started#why-compile-against-the-api


Prd 895 navigation test

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable) [https://github.com/datahub-project/datahub/compare/master...acryldata:datahub:PRD-895-Navigation-Test?expand=1]
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

wrong branch


Add activity PMIP to source_id EC-Earth3

Our Paleo climate team would like to contribute to PMIP also using EC-Earth3. Could you please add PMIP to the activity_participation list for EC-Earth3? We will continue to contribute to PMIP using EC-Earth3-LR and EC-Earth3-Veg-LR and EC-Earth3-GrIS.

@shutingdk that change is now in place, take a peek at CMIP6_source_id.html and let me know of any other edits

Looks good. Thank you.


Paul's Big Sprint

Value: $0.00Closes: Friday, March 23, 2018Location: Victoria In-person work NOT requiredOpportunity DescriptionContrary to popular belief, Lorem Ipsum is not simply random text.  Richard McClintock, a Latin professor at Hampden-Sydney College in Virginia, looked up one of the more obscure Latin words, consectetur, from a Lorem Ipsum passage, and going through the cites of the word in classical literature, discovered the undoubtable source. Lorem Ipsum comes from sections 1.10.32 and 1.10.33 of "de Finibus Bonorum et Malorum" (The Extremes of Good and Evil) by Cicero, written in 45 BC. This book is a treatise on the theory of ethics, very popular during the Renaissance. The first line of Lorem Ipsum, "Lorem ipsum dolor sit amet..", comes from a line in section 1.10.32.
The standard chunk of Lorem Ipsum used since the 1500s is reproduced below for those interested. Sections 1.10.32 and 1.10.33 from "de Finibus Bonorum et Malorum" by Cicero are also reproduced in their exact original form, accompanied by English versions from the 1914 translation by H. Rackham.Acceptance CriteriaHow to ApplyGo to the Opportunity Page, click the Apply button above and submit your proposal by 16:00 PST on Friday, March 23, 2018.We plan to assign this opportunity by Friday, March 30, 2018 with work to start on Tuesday, March 20, 2018.Proposal Evaluation Criteria

This was a test.


Doesn't show content for feed

Hello,
When adding the feed https://thenib.com/feeds/rss, the content embedded in <![CDATA[ does not show.
I'm note sure if that's a feed issue or a news reader/parser issue.
Thank you,
Marius

It’s a bug in the feed — that content is essentially empty.
  <content:encoded><![CDATA[
  <div>

  </div>
  ]]></content:encoded>


Impact ?

Hi, Happy new year! I saw this commit which looks extensive. What's the impact of this change?
https://github.com/neatnik/steganographr/commit/7ab5bb2b21c5561eba281bb29d83cc17e3af0457

Ah, sorry, didn't mean to introduce any confusion. There are no functional changes at all—turns out by mistake I committed a modified version meant to integrate with my website's new content framework. Didn't mean to do that! I'll fix it now.

Ah, sorry, didn't mean to introduce any confusion. There are no functional changes at all—turns out by mistake I committed a modified version meant to integrate with my website's new content framework. Didn't mean to do that! I'll fix it now.

Hi, thanks. I was just wondering if I need to test a new release :-)

Hi, thanks. I was just wondering if I need to test a new release :-)


Multiple calls to Connection.Close() results in error

Basically this:
	Error:		Received unexpected error:
			write tcp <IP_ADDRESS>:51848-><IP_ADDRESS>:5672: use of closed network connection

When trying to close several times.

And what do you expect to happen? Closing a connection multiple times is an error: this is not something the protocol or TCP sockets allow you to do.

I'm closing this because there's no information on what you expected to have happened and the current error message is not that cryptic.
See #219 for what I believe all client libraries should do.

I believe multiple calls to Close should result in last error or no-op if connection is already closed. There is already sync.Once inside - just wrap the logic.


multiple parameters for init method

Is it possible to have multiple parameters for init method when I need to pass more than 1. Of course I can create custom object with those parameters as property but this is kind of cumbersome.
thanks :)

I don't think a change is needed here.
It is not difficult to pass in params with object[].
This is much faster than creating an object with custom properties.
Here's an example I just finished testing locally (obviously you want to run the null checks in real code).
In the ICommand:
public ICommand AddContactCommand
{
	get {
		return new Command(async () => {
			// Initialize the array with three different classes.
			object[] threeBodyProblem = new object[] { _repository, Contacts[0], "string" };
			await CoreMethods.PushPageModel<ContactPageModel>(threeBodyProblem);
		});
	}
}

In the ContactPageModel's Init method:
public override void Init(object initData)
{
	var result = initData as object[];
	var indexZero = result[0] as Repository;
	var indexOne = result[1] as Contact;
	var indexTwo = result[2] as string;
}

I recommend offering this as a solution and not modifying the methods.

Another alternative, that also might be nicer to use when C# 7 comes out (with the new Tuple deconstruction feature etc.).
CoreMethods.PushPageModel<SomeModel>(Tuple.Create(1, "a"));

public override void Init(object initData)
{
    var parameters = (Tuple<int, string>)initData;
}

Another way of getting multiple parameters, is to add a Init(params object[] parameters), and then you will have two Init methods. But I think the answer to @C0D3Name is better.

There is also another alternative that might work.
If the PushPageModel can take a Action<TViewModel>, then it should be possible to do something like:
CoreMethods.PushPageModel<SomeModel>((vm) => vm.Init("a", 1));

public override void Init(string a, int b) //Can however be called whatever you want.
{
    //
}

This should then be strongly typed and everything, as it don't depend on any baseclass.

@oddbear tuple is a good way of resolving this issue but requires c#7.
@C0D3Name I dont think this is a good solution. because in some cases  I might need to pass over different types. I handle it as below. In this case your using Object will fail. many people dont like to work with Object type I believe.
if (initData is Customer)
{
}

@EmilAlipiev
It is possible to cast to another type from an object. This satisfies the test within your if statement. I have written a simple, 1-file console application to prove this point, it utilizes similar logic to your example if statement. Attached is the output from running the application.
`
using System;
namespace ConsoleApplication1 {
public class Class1 {
	public string ClassName { get { return "Class1"; } }
}
public class Class2 {
	public string ClassName { get { return "Class2"; } }
}

class Program {
	static void Main(string[] args) {
		ClassTester(new Class1());
		ClassTester(new Class2());
		ClassTester(new object());

		Console.ReadKey();
	}

	private static void ClassTester(object theParameter) {
		if(theParameter as Class1 != null) {
			var classCast = theParameter as Class1;
			Console.WriteLine(classCast.ClassName);
		}
		else if(theParameter as Class2 != null) {
			var classCast = theParameter as Class2;
			Console.WriteLine(classCast.ClassName);
		}
		else {
			Console.WriteLine("Could not cast object to a known type.");
		}
	}
}

}
`

Sorry about replying an old issue, but I do have a similar issue.
I wonder, is it possible to fetch some values from the previous PageModel?
Or maybe I could just send the current instance of the PageModel to the next page, so I don't have to pack everything in an object and unpack in the new PageModel class. Any suggestion?

Sounds like this would not scale easily.
One navigation to the page would probarbly work fine, but when you can access a page from 3-4 others (or 100)... then I would guess this to be really hard to maintain.
I am thinking the code inside the Init from just 4 diffrent previous pages would be something like:
switch(model)
{
    case model is AViewModel: /* DoSomething */ break;
    case model is BViewModel: /* DoSomething */ break;
    case model is CViewModel: /* DoSomething */ break;
    case model is DViewModel: /* DoSomething */ break;
}

However, the previous viewModel can have a common interface (or similar), to reuse the "fetch logic".

Sounds like this would not scale easily.

Thanks for your reply.
Or maybe send an instance of NextPageModel could be an alternative way. But by doing this, it actually moves the "init" stuff to a constructor. Is this feasible or is there any violation against the design of fresh MVVM?
Cheers.


Invalid HTML for TOC

It seems that the TOC produces one </ul> too much.
See the third issue here:
https://validator.w3.org/nu/?doc=https%3A%2F%2Ftypemill.net%2Finfo%2Fmarkdown-test

fixed in 1.4.0

published with version 1.4.0 today


remove JSONCodable protocol

I've bumped into a problem while using xcode9.
Error I was getting was:

'JSONDecoder' is ambiguous for type lookup in this context

The only way to fix this is to put class in proper namespace by add prefix JSONCodable like JSONCodable.JSONDecoder, but there is a problem that compiler want to use protocol called the same, not lib name
the only idea I've figure out is to remove this protocol

I'd really like that patch if it works.

In the file that produces that error, you can also do
import class JSONCodable.JSONDecoder
It disambiguates what you are referencing for Xcode !


Capitalization of heading filter should match documentation

Documentation states that the heading search disregards capitalisation.
This isn't quite true in my experience. I was trying to search for a heading called "Events" and was using "heading includes Events" which resulted in no matches. However, "heading includes events" worked. So it's not a case that is disregards the capitalisation but rather that the search term NEEDS to be lower case, regardless of the capitalisation of the actual heading.
This may be a separate issue but I also cannot get it to search for descriptions including an emoji (I've tried things like "description includes :alarm_clock: " is the actual emoji inserted using the emoji toolbar and also searching for the emoji name with various syntax like :alarm: etc
Cheers

Thanks! I will update the readme.
The emoji topic should indeed be its own issue. Can you please open one? Thank you!

As a small note further on this I've done some testing.
path requires case to exactly match that of the target
heading requires lowercase but will match any target case

Fixed in the latest 1.1.2 release!


autofocus

as pointed out by @henrypinkard in https://github.com/micro-manager/pycro-manager/issues/266#issuecomment-809565527, the concept of autofocus should be somewhere

Following up on my experiences from: https://github.com/pymmcore-plus/pymmcore-plus/issues/51#issuecomment-1397585287
I think adding stage descriptors to the base ZPlan would essentially accomplish this:
    primary_focus_stage: str = None
    z_stack_stage: str = None

In the case of my example in the linked issue I would set primary_focus_stage to PFS-Offset and z_stack_stage to Focus.
The idea being:

If both are specified then primary is used to find the center, and z_stack to perform the z_stack.
If only one is given then use that for everything
If neither are given use whatever is returned by core.getDefaultFocusDevice

closed by #96


Add corporate Hooli-esque template!

Pull Request
Submission for ETHDenver UD website template bounty.
Contribution type

[x] New template contribution
[ ] Existing template update
[ ] Issue fix
[ ] README.md update

Contribution Description
Hooli-esque corporate financial overview template.

[x] I would like my template to be added to the Unstoppable Domains' website builder

Unstoppable Domain Name:
pauljhansel.crypto

Hey,
Matias from Unstoppable Domains here.
Thanks for submitting a template!
We will be reviewing all pull requests this week.
Best,

Hi, thank you for your submission. We've decided not to accept your submission as part of the bounty competition because it's lacking quality. If you would like to improve the design, fix the scroll, and mobile responsiveness feel free to leave this pull request open


ERC721 Staking contract methods not working in Google Chrome

getStakeInfo from ERC721 Staking contract in Google Chrome. In Brave working

Hello @alexey727, could you please share your contract address for the staking? We'll try to replicate it on our end. Looking forward to your prompt response.

Hello @warengonzaga 0xFA73ac9A2632228E82Cc4f6Eb459083bE19FB1c1 in BSC testnet


python310Packages.snapcast: 2.3.1 -> 2.3.2

Diff: https://github.com/happyleavesaoc/python-snapcast/compare/2.3.0...2.3.2
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 219977 run on x86_64-linux 1

2 packages built:

python310Packages.snapcast
python311Packages.snapcast


OAuth2.Client.get_token!/2 not deserializing token response

OAuth2.Client.new(
    token_url: "/auth/oauth/token",
    client_id: System.get_env("FRESH_BOOKS_CLIENT_ID"),
    client_secret: System.get_env("FRESH_BOOKS_CLIENT_SECRET"),
    site: "https://api.freshbooks.com",
    redirect_uri: "https://dev.website.ca/integrations/install/fresh-books",
)
|> OAuth2.Client.get_token!(code: code)

Produces
%OAuth2.Client{
  authorize_url: "/oauth/authorize",
  client_id: "asdf1234asdf",
  client_secret: "1234asdf1234",
  headers: [],
  params: %{},
  redirect_uri: "https://dev.website.ca/integrations/install/fresh-books",
  ref: nil,
  request_opts: [],
  serializers: %{},
  site: "https://api.freshbooks.com",
  strategy: OAuth2.Strategy.AuthCode,
  token: %OAuth2.AccessToken{
    access_token: "{\"access_token\":\"f7d067b72cab8af4a1db9865FAKETOKENbdff58b3060kshdlkjfghk\",\"token_type\":\"Bearer\",\"expires_in\":43200,\"refresh_token\":\"bdcd900efdfbd2ec8c5c01lksdFAKETOKENed419f1a62f995915698c4\",\"scope\":\"profile:write\",\"created_at\":1574976814}",
    expires_at: nil,
    other_params: %{},
    refresh_token: nil,
    token_type: "Bearer"
  },
  token_method: :post,
  token_url: "/auth/oauth/token"
}

Notice access_token is still a serialize JSON object.
I am happy to create a PR! In the mean time I will manually deserialize, but that is way less cool.

What is the value of the content-type header in the response? The token is generally decoded automatically based on that.
Also...see the README about configuring serializers.

16:08:25.332 request_id=Fdu-RqUZCumywD0AAHCE [debug]   OAuth2 Provider Request
  url: "https://api.freshbooks.com/auth/oauth/token"
  method: :post
  headers: [{"accept", "application/x-www-form-urlencoded"}, {"content-type", "application/x-www-form-urlencoded"}, {"authorization", "Basic AUTHORIZATIONTOKENREMOVED"}]
  body: "client_id=CLIENTIDREMOVED&code=CODEREMOVED&grant_type=authorization_code&redirect_uri=https%3A%2F%2Fdev.website.ca%2Fintegrations%2Finstall%2Ffresh-books"
  req_opts: []

16:08:25.637 request_id=Fdu-RqUZCumywD0AAHCE [debug] OAuth2 Provider Response %OAuth2.Response{body: "{\"access_token\":\"c453f20126311bb0acf659c2a2b53e6e38d0a2d269d2e62a087FAKETOKEN\",\"token_type\":\"Bearer\",\"expires_in\":43199,\"refresh_token\":\"d3180b81dc8e13ed124b304284185160481508bcd1d653dFAKETOKEN\",\"scope\":\"profile:write\",\"created_at\":1575061705}", headers: [{"server", "nginx"}, {"date", "Fri, 29 Nov 2019 21:08:26 GMT"}, {"content-type", "application/json; charset=utf-8"}, {"content-length", "255"}, {"vary", "Accept-Encoding"}, {"cache-control", "private, no-store"}, {"pragma", "no-cache"}, {"etag", "W/\"717a0351d29f57fc29d46432b63b7060\""}, {"x-request-id", "755e7f82-aef2-4a69-8658-ae7a66a48c0e"}, {"x-runtime", "0.098058"}, {"x-newrelic-app-data", "PxQBWV5TCBABVlVSBAUHUFQTGhEhCQ0WQg1UDl1KG0ZcDlwPQk5QEAFXQwRETwgdAEhRTAcID19RBgUDUldbVAFXD1ALFBkDH0dbBwgFAltcB1VSV1ANBgBQQz8="}, {"x-ratelimit-limit", "20"}, {"x-ratelimit-remaining", "19"}, {"x-ratelimit-reset", "1575061766"}, {"retry-after", "59"}, {"access-control-allow-origin", "*"}, {"x-frame-options", "SAMEORIGIN"}, {"x-xss-protection", "1; mode=block"}, {"x-content-type-options", "nosniff"}, {"content-security-policy", "default-src 'self'"}, {"x-content-security-policy", "default-src 'self'"}, {"strict-transport-security", "max-age=31556926; includeSubDomains; preload"}, {"referrer-policy", "strict-origin-when-cross-origin"}, {"via", "1.1 google"}, {"alt-svc", "quic=\":443\"; ma=2592000; v=\"46,43\",h3-Q050=\":443\"; ma=2592000,h3-Q049=\":443\"; ma=2592000,h3-Q048=\":443\"; ma=2592000,h3-Q046=\":443\"; ma=2592000,h3-Q043=\":443\"; ma=2592000"}], status_code: 200}

If you dig through the response headers: {"content-type", "application/json; charset=utf-8"}
Without having looked into the source, maybe the ; charset=utf-8 bit is throwing it off

@ericdude4 the issue is that you have not supplied a serializer in your client as shown here: https://github.com/scrogson/oauth2#configure-a-serializer
The following should fix your issue:
OAuth2.Client.new(
    token_url: "/auth/oauth/token",
    client_id: System.get_env("FRESH_BOOKS_CLIENT_ID"),
    client_secret: System.get_env("FRESH_BOOKS_CLIENT_SECRET"),
    site: "https://api.freshbooks.com",
    redirect_uri: "https://dev.website.ca/integrations/install/fresh-books",
)
|> OAuth2.put_serializer("application/json", Jason)
|> OAuth2.Client.get_token!(code: code)

I went through the same issue and I confirm that adding OAuth2.put_serializer("application/json", Jason) fix the problem.

Just got back to this and found that OAuth2.put_serializer("application/json", Jason) was not quite correct (wrong module, function undefined). The correct function is
OAuth2.Client.put_serializer("application/json", Jason)

Thank you for your help

Ah yep. My bad.


VTDecoderXPCService Memory Consumption

Hey there, I'm not sure if it's related to videogular or not but I noticed my site takes a memory / cpu hit when playing videos. The VTDecoderXPCService spikes in my Activity Monitor and really slows everything down. Any ideas?

Update: It appears the latest versions of Chrome and Firefox have this issue but NOT Safari. Thoughts?

I suggest to open an issue on Chrome and Firefox teams. If it's something blocking they'll fix it probably soon.


Chromium support on intel gpu

Describe the bug
Hello, when using chrome://gpu in the distro's native chromium (rpm-ostree), some things are disabled:
Graphics Feature Status

Direct Rendering Display Compositor: Disabled
Raw Draw: Disabled
Skia Graphite: Disabled
Vulkan: Disabled
WebGPU: Disabled
WebNN: Disabled

In the integrated Steam browser, in the normal interface, which you access with alt+tab, during some game, some things are disabled:
Graphics Feature Status
Canvas out-of-process rasterization: Disabled
Direct Rendering Display Compositor: Disabled
Raw Draw: Disabled
Video Encode: Software only. Hardware acceleration disabled
Vulkan: Disabled
WebGPU: Disabled
What did you expect to happen?
Compatibility to see all enabled
Output of rpm-ostree status
State: idle
Deployments:
● ostree-image-signed:docker://ghcr.io/ublue-os/bazzite:stable
                   Digest: sha256:e282521ee0e418a11ae423d8cfd9c83893545c9dd2e4e7226fda1dc8f1389e6c
                  Version: 40.20240901.1 (2024-09-02T21:12:18Z)
          LayeredPackages: chromium elisa-player libreoffice vlc wine

  ostree-image-signed:docker://ghcr.io/ublue-os/bazzite:stable
                   Digest: sha256:688da1a245efc6fdae27743964c8f5ebe380f057064844c7066d3f851e92651c
                  Version: 40.20240901.0 (2024-09-02T05:05:54Z)
          LayeredPackages: chromium elisa-player libreoffice vlc wine

Hardware
Laptop Lenovo Ideapad 3 15itl6 82md
intel i3 1115G4
Ram 8gb
Extra information or context
I believe it is a bug, or compatibility. I wanted to leave the report, to help the project. 👍

Please try the flatpak, we do not support layered browsers under any circumstance. It is a security risk to only update your browser with the OS.

Thanks.
I installed Chrome, via the bazzite portal.
In Chrome it looks like this:
Graphics Feature Status

Direct Rendering Display Compositor: Disabled
Raw Draw: Disabled
Skia Graphite: Disabled
Video Encode: Software only. Hardware acceleration disabled
Vulkan: Disabled
WebGPU: Disabled
WebNN: Disabled

You'll need this:
https://wiki.archlinux.org/title/Chromium#Hardware_video_acceleration

thanks,
I tried here, but it didn't find the .flags file here 😅

I couldn't find the flags in Chrome 😅
I uninstalled Chromium through the terminal, when I right-clicked on chromium, and in the uninstall option, when I opened discover it didn't find chromium. (sudo rpm-ostree uninstall chromium)
I did a little research here, and started Chrome with this command:
flatpak run com.google.Chrome --enable-features=VaapiVideoDecoder,VaapiIgnoreDriverChecks,Vulkan,DefaultANGLEVulkan,VulkanFromANGLE,VaapiVideoDecodeLinuxGL,ignore-gpu-blocklist,enable-zero-copy
It improved performance a lot, videos in 8k60fps were better too.
Of the disabled ones, it looked like this:
Graphics Feature Status

Direct Rendering Display Compositor: Disabled
Raw Draw: Disabled
Skia Graphite: Disabled
Video Encode: Software only. Hardware acceleration disabled
WebGPU: Disabled
WebNN: Disabled


dupRadar for large files

After rerunning several large RNAseq runs (15GB markDuped BAM files) with dupRadar, I found out that these are probably simply taking too long for the default of 2 hours which is specified in base.config. After automated rescheduling, its still running over 4 hours and thus ultimately crashes. Manually giving the pipeline > 4 hours time for each step solves the issue and the results look good.
I don't see this as a severe bug, but would like to pinpoint the issue and also bring up, that we might keep track of the enhancement in nextflow here: https://github.com/nextflow-io/nextflow/issues/731 that ultimately aims to have a possibility to check how big a file is and modify resource consumption based on file size for example in the future. That should provide the possibility to make such "extreme" cases easier to handle.

Problem can be fixed by using "Dynamic computing resources" for the process.
time = { check_max( 2.h * task.attempt, 'time' ) }

Can you add that in a PR?
I guess the dynamic stuff would be great to have once its there... but that should resolve it already...

Sure, I will make a PR later with other changes together.


Update DATA-DOG/go-sqlmock to semver

Because of https://github.com/DATA-DOG/go-sqlmock/issues/161 need changes in code. Need replace gopkg.in to github.com.

PR #24

PR #24

+1
I am getting the following error:
github.com/bar/foo imports
        github.com/heptiolabs/healthcheck tested by
        github.com/heptiolabs/healthcheck.test imports
        gopkg.in/DATA-DOG/go-sqlmock.v1: cannot find module providing package gopkg.in/DATA-DOG/go-sqlmock.v1

This PR should fix the dependency.
See a similar issue here: https://github.com/InVisionApp/go-health/pull/68
/ping @mattmoyer


Anerdw: Watch filesplice.com

Anerdw requests the watch of the watch_keyword filesplice\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

filesplice\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Possible bug with Masthead Component

Multiple errors on masthead due to the navigation links
-->
Detailed description
When navigation links are added to the masthead multiple error messages are produced. Error messages can be reproduced on the sandbox from the Masthead documentation: https://githubbox.com/carbon-design-system/carbon-for-ibm-dotcom/tree/master/packages/react/examples/codesandbox/components/MastheadL1
I think it is because of how the navigationL1 links are set up on the mastheadL1Data prop.
Some of the error messages are:
Warning: Failed prop type: Invalid prop className of type boolean supplied to ForwardRef(SideNavMenuItem), expected string
Warning: Each child in a list should have a unique "key" prop.
For a full list see the recording below or follow the steps to reproduce.
Is this a feature request (new component, new icon), a bug, or a general issue?
bug/ issue with documentation code
Is this issue related to a specific component?
The Masthead component
What browser are you working in?
Chrome 93.0.4577.82
What version of Carbon for IBM.com are you using?
1.24.0
Steps to reproduce the issue

Step one: Open https://githubbox.com/carbon-design-system/carbon-for-ibm-dotcom/tree/master/packages/react/examples/codesandbox/components/MastheadL1
Step two: Find and copy browser panel URL: eg: https://r0i89.sse.codesandbox.io/ on new tab
Step three: Go to developer tools to find error messages. CMD+Option+J on Chrome

Additional information

Screen Recording:

Thank you.

Issue moved to carbon-design-system/carbon-for-ibm-dotcom #7224 via ZenHub


Pin pandas version to fix unit tests

There has been a change in Pandas version 1.3.0 around the test utility function assert_frame_equal (see the release notes). Although I didn't look into exactly why this bug fix breaks some of our tests, regressing to Pandas 1.2.5, which is the latest version that is older than 1.3, fixes our tests.
I was able to reproduce the test failures locally via pip install pandas==1.3.0, and then fix via pip install pandas==1.2.5.
It may be worth someone with knowledge of the code and tests having a quick look to make sure the tests in question are not relying on broken behaviour in assert_frame_equal().
The build now completes.

@mfitz thanks for this. The tests seem to be breaking due to different column sorting behaviour between pandas 1.3.0 and 1.2.5. The failing tests were comparing identical dataframes, but with different column ordering.
We could either pin pandas to 1.2.5 or align the dataframe columns in the failing tests before checking for equality. I tested the latter option and seemed to pass the tests (it was a small change). Please let me know what you think.

I'd quite like to remove assert_frame_equal()

@mfitz thanks for this. The tests seem to be breaking due to different column sorting behaviour between pandas 1.3.0 and 1.2.5. The failing tests were comparing identical dataframes, but with different column ordering.
We could either pin pandas to 1.2.5 or align the dataframe columns in the failing tests before checking for equality. I tested the latter option and seemed to pass the tests (it was a small change). Please let me know what you think.

I prefer fixing the tests and then pinning to the latest pandas @Theodore-Chatziioannou


No symlinks for VirtualBox command line executables

Would it be appropriate for cask to manage the creation (and subsequent deletion upon uninstall) of symlinks to VirtualBox command line executables in /Applications/VirtualBox.app/Contents/MacOS/, including VBoxManage and VBoxHeadless? i.e.:
ln -s /Applications/VirtualBox.app/Contents/MacOS/VBoxManage /usr/local/bin/VBoxManage
ln -s /Applications/VirtualBox.app/Contents/MacOS/VBoxHeadless /usr/local/bin/VboxHeadless

Another question: does cask check whether the filesystem is case-sensitive? and if so, can it perform additional tasks? such as lowercase equivalents as well:
ln -s /Applications/VirtualBox.app/Contents/MacOS/VBoxManage /usr/local/bin/vboxmanage
ln -s /Applications/VirtualBox.app/Contents/MacOS/VBoxHeadless /usr/local/bin/vboxheadless

Of course, the uninstall process should include:
rm -f /usr/local/bin/VBoxManage
rm -f /usr/local/bin/VBoxHeadless
rm -f /usr/local/bin/vboxmanage
rm -f /usr/local/bin/vboxheadless

Sure! This kind of functionality is supported out of the box:
binary '/Applications/VirtualBox.app/Contents/MacOS/VBoxManage', :target => 'vboxmanage'
binary '/Applications/VirtualBox.app/Contents/MacOS/VBoxHeadless', :target => 'vboxheadless'

Add those lines, submit a pull request and you're on your way!

Never mind, it was a quick change so I went ahead and did it myself.

For simple app bundles, we symlink included binaries regularly (e.g. skim, vmware-fusion); symlinks are always converted to lowercase (e.g. diffmerge).
Installers do not offer the convenience of a dedicated stanza, but arbitrary pre- and post-install commands can be set with postflight and preflight. We use them sparingly, but I would be amenable to recover simple behavior available to other artifacts.

@ndr-qef Why would preflight and postflight be necessary here when binary will do?

binary represents an artifact, meaning that it must be available prior to installation. (The stanza expects a path relative to the downloaded archive, albeit absolute paths will also work.) There is no guarantee that binary artifacts will be symlinked after the pkg installer completes.

@qtfk, the binaries are now symlinked. For case-sensitive filesystems, would you find lowercase preferrable?

I would, but would still suggest creating both, as on other systems.


Amend Scopes and HTTP Response Code regexes

refs #42
Please forgive me, these are the first lines of go I've ever written and I haven't done any reading on the language. Happy to rev the PR.

I signed it!

Thanks @MikeRalphson.
Do you think we should have stricter regexes for paths (^/) and extensions (^x-)?

Do you think we should have stricter regexes for paths (^/) and extensions (^x-)?

I found just one regex for matching the actual contents of a URL path (rather than extracting one):
^(?:(?:\.\./)|/)?(?:\w(?:[\w`~!$=;\-\+\.\^\(\)\|\{\}\[\]]|(?:%\d\d))*\w?)?(?:/\w(?:[\w`~!$=;\-\+\.\^\(\)\|\{\}\[\]]|(?:%\d\d))*\w?)*(?:\?[^#]+)?(?:#[a-z0-9]\w*)?$

though this includes the query and fragment parts which would need to be removed. I can't vouch for its correctness, and I must confess I have reasons for not wanting this to be overly strict if the OAS adopts this schema as their official one. I wouldn't object to a PR against the OAS to add a regex requirement though.
The OAS seems to put no other restrictions on extension names, any Unicode string is a valid JSON/YAML property name if suitably escaped.


Confusing feedback when loading Burrito UI with lot of layers

When loading Burrito UI with a lot of layers (50+ in my case, and it will keep growing), the backend requests to get all layers takes more than 8 seconds. While the current CPU/Memory requests of my instance might be a bottleneck, the interface feedback is very confusing

it display that there 0 layers, leading to think that all my layers have disappeared
there is no loading indicator, which would hint that I should just wait

Furthermore, the current implementation of the backend requests, without any pagination might not be scalable. Event with a loading indicator waiting several seconds is not a pleasant experience

I think we should consider this issue resolved and open a new enhancement issue to add layers pagination later. @Alan-pad


EventStore AggregateStream never returns null

I'd expect that EventStore.AggregateStream(streamId) method would return null if there are no events (or not even a stream) for that stream identifier, but it actually creates a new stream instead (which all property values set to defaults) and saves both stream and aggregate in database (aggregate is defined as inline).
I've also seen null check on this thread, specifically:
public T Load<T>(string id, int? version = null) where T : AggregateBase
{
    using (var session = store.LightweightSession())
    {
        var aggregate = session.Events.AggregateStream<T>(id, version ?? 0);
        return aggregate ?? throw new InvalidOperationException($"No aggregate by id {id}.");
    }
}

but as far as I could test, this method will never throw InvalidOperationException.
I'm just interested whether this is expected behaviour and how should I use this API correctly. From what I've tested so far, I can load the document via documentSession.Load<MyDocument>(streamId); (this returns null if it doesn't exist), but I'm not sure if I can use this normally (meaning append new events internally on this object, as if it was fetched via AggregateStreamAsync, or AggregateStreamAsync is somehow different from Load - in regards to fetching latest aggregate that's stored in database and using it later on).
Thank you.

@huberzeljko I'm aware of that issue. In my opinion, it's a bug. We won't be able to fix that in v3, as it'd be kinda big breaking change, however, if others agree I can change that behaviour to be consistent in v4 (in my opinion it should return null). @jeremydmiller do you happen to remember why is it like that?
Regarding AggregateStream vs Load. Load is getting the snapshot directly from a document that represents the latest version of the stream. AggregateStream  works as follows:

get all events for the selected stream id (ordered by the occurrence - stream version),
create aggregate based on default constructor or creates an uninitialized object,
for each event, call Apply method for Aggregate.

Unfortunately, it does not check if there were events at all.
If you'd like to get null then you'd need to perform additional query - session.Events.FetchStreamState it should return null if stream was not found. See more in https://martendb.io/documentation/events/streams/

This has been fixed in the V4 "projections" branch.

This has been fixed in the V4 "projections" branch.


Char 05 DDPG: step index and episode index

for i in range(args.num_iteration):             state = env.reset()             for t in range(args.max_episode):
from the above code we can infer that i stands for the i-th step, and t stands for the t-th episode.
However,  it is shown in code:
print('Episode {},  The memory size is {} '.format(i, len(agent.replay_buffer.storage)))
that i is used for counting episode.
So do we need to change the positions of args.max_episode and args.num_iteration  ?

Thanks for your issue.
Note that i stands for the number of episodes. And t stands for the length of one trajectory, i.e, the agent would stop a rollout when t>args.max_episode.


Add initial support for Twitter V2 Timelines

In a second commit, it will be good to add the documentation for Timelines.


chore: v0.4.0

chore: v0.4.0

Codecov Report

Merging #36 (4e6f6fe) into main (77123bc) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #36   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           30        30
=========================================
  Hits            30        30

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 77123bc...4e6f6fe. Read the comment docs.


Fix adapter not adding event when result is 0 for single insertion

Codecov Report
Base: 90.50% // Head: 90.48% // Decreases project coverage by -0.01% :warning:

Coverage data is based on head (822acfb) compared to base (ecaf334).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #34      +/-   ##
===========================================
- Coverage    90.50%   90.48%   -0.02%
===========================================
  Files           79       69      -10
  Lines         1990     1797     -193
===========================================
- Hits          1801     1626     -175
+ Misses         189      171      -18

Flag
Coverage Δ

flat_generator
90.48% <ø> (ø)

flat_orm
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

flat_orm/lib/src/adapter/insertion_adapter.dart

flat_orm/lib/src/adapter/migration_adapter.dart

flat_orm/lib/src/adapter/query_adapter.dart

flat_orm/lib/src/util/primary_key_helper.dart

flat_orm/lib/src/sqflite_database_factory.dart

...src/extension/on_conflict_strategy_extensions.dart

flat_orm/lib/src/adapter/update_adapter.dart

flat_orm/lib/src/adapter/deletion_adapter.dart

flat_orm/lib/src/migration.dart

flat_orm/lib/src/callback.dart

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 for CONCAT operator

Search before asking

[X] I have searched the EvaDB issues and found no similar feature requests.

Description
I'm planning to add a new CONCAT operator to EvaDB that combines context from multiple textual rows, inspired by the WINDOW operator introduced here:
https://www.vldb.org/pvldb/vol8/p1058-leis.pdf
Motivation
The current workflow for LLM-based text apps is as follows:
story_table (id INTEGER, paragraph TEXT(1000));
CREATE TABLE feature_table AS
SELECT SentenceFeatureExtractor(paragraph), paragraph FROM story_table;

Each row in the story_table contains a single paragraph in the input document. The resulting feature_table contains the embedding for a single paragraph in each row.
Any subsequent processing, such as similarity search or indexing, only works at the paragraph level. This often results in poor LLM accuracy. For example, below are the results of a sample question using the story_qa app:
Enter your question (type 'exit' to stop): How is Natasha described?

Natasha is not described in this particular context. The context mentions a question asked to Natasha about someone else and then moves on to various conversations and descriptions of other people present in the scene.

If we could instead combine paragraphs while creating the embeddings and index, we have better control over the context given to the LLM model. The output for the same question when concatenating the input with 1 preceding and 1 succeeding paragraph:
Natasha is described as a "black-eyed, wide-mouthed girl not pretty but full of life," who is at the "charming age when a girl is no longer a child though the child is not yet a young woman." She is also prone to bouts of laughter and seems to have a close relationship with her mother.

The existing GROUP BY operator combines multiple paragraphs into a new group. However, it can only group multiple paragraphs into a single Pythonic list and cannot add context to each individual data point.
Proposed workflow
The proposed CONCAT operator concatenates multiple rows to add additional context to each data point. The operator can only be used with a SELECT query to augment the textual column. It cannot filter or alter the existing tables.
An example workflow with the CONCAT operator is as follows:
story_table (id INTEGER, paragraph TEXT(1000));
SELECT id, paragraph FROM story_table
CONCAT BY paragraph
ROW BETWEEN 2 PRECEDING AND 2 FOLLOWING;

Here are a few things to discuss:

This operation could be done on the app side as well, where users load text of different lengths into EvaDB. However, the optimal length is difficult to predict, and its impact on the dollar cost is non-trivial to calculate. Using an operator might allow us to optimize the dollar cost to the end user (see point 2).
The token usage depends on the window size. Depending on the use case, the user could choose the window size to save on dollar cost - e.g., small windows if the document contains simple information like FAQ.
The CONCAT operation (and subsequent processing) would be expensive for large documents due to the repeating of tokens. We must find ways to avoid or offset this cost.
The related WINDOW operator is useful for structured data (as proposed in the paper) - it might have use cases in forecasting and statistical analysis. If possible, we could find a consistent syntax for structured + unstructured data.
What does the CONCAT operator mean for audio and videos? GROUP BY might suffice for them.

@jarulraj  @gaurav274  @xzdandy  @jiashenC Any early feedback and discussion is highly appreciated, thanks!
Use case
No response
Are you willing to submit a PR?

[X] Yes I'd like to help by submitting a PR!

Another constraint to think about is with LLMs, we have a context limit -- so number of rows does not matter
SELECT id, paragraph FROM story_table
CONCAT BY paragraph
TOKEN BETWEEN 100 PRECEDING AND 100 FOLLOWING;

Can we just use the underlying database system -- Postgres?

Thanks for the suggestions!
Postgres supports the CONCAT/WINDOW functionality. I verified that we can get the desired outcome using the following query:
cursor.query("""
  USE pg_db {
    SELECT id, string_agg(paragraph, '.')
    OVER (ROWS BETWEEN 3 PRECEDING AND 3 FOLLOWING)
    FROM story_table; }
""")

To handle the context limit of LLM models, we could consider implementing a MAX_TOKEN constraint on the output of the above query. The models already (e.g., SentenceTransformer) truncate the input sentences based on the model limits (e.g., model.max_seq_length). The MAX_TOKEN constraint might still help us control the model cost more explicitly.

I like the Postgres syntax better. Providing a window functionality and string_agg, which can also be used for creating a moving windows on videos, if replacing string_agg with some function like image_agg.
Given the following query works now, can we rely on the underlying database instead of introducing the window function to EvaDB for now, which is non trivial and requires high efforts?
cursor.query("""
  USE pg_db {
    SELECT id, string_agg(paragraph, '.')
    OVER (ROWS BETWEEN 3 PRECEDING AND 3 FOLLOWING)
    FROM story_table; }
""")

If there is a usecase that we can not elegantly push down to the underlying database, please share. Thanks!

Sounds good. For now, let's go with Postgres queries for window function capabilities.


[single-machine-performance] regression_detector: post via infile, not stdin

This commit is an experiment to see if the PR comment formatting changes if the regression report is passed to pr-formatter via its --infile argument instead of via stdin.

What does this PR do?

Motivation
This PR is an experiment to test for interactions between the regression report and the pr-commenter tool because I was unable to reproduce in unit tests of the regression detector the strange formatting observed in regression detector PR comments from Agent CI.

Additional Notes
Work on improving PR comment formatting is also ongoing in the Single Machine Performance regression detector, with patches submitted to that tool as well.

Possible Drawbacks / Trade-offs
This experiment may yield the same results as the existing regression detector PR commenter setup. If so, that result may rule out interactions between the PR commenter and the regression detector report output as a source of the anomalous comment formatting output; in that case, this PR can be deleted safely.
Otherwise, this PR may improve regression detector PR comment formatting.

Describe how to test/QA your changes

A PR comment should post in the comment thread, which would indicate whether this experiment yields any change in output. Whether or not a change in output occurs, the resulting output will be useful for diagnosing causes of the odd formatting.
Reviewer's Checklist

[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[x] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Hm. If there is a bug due to interaction between the regression detector output and something in the PR commenting process, that flag is not at issue. Closing this PR because it is not an improvement, and the comment output has given me the additional information I needed to proceed further.


Export text messages from the phone

I am proposing this document without being fully sure if it is useful enough.
It contains the instructions for exporting the message texts (*) of SMS, MMS and IM from a Sailfish device to a computer, allowing further editing of the texts if needed. The normal backup/restore and SQL dump just make the messages readable.
(*) No MMS attachments
There are now three docs with "Messages" in their names. Perhaps the time to create a subfolder? That would also help with the index numbers of the README files - there are some problems here in the M area.

I agree with everything that has been written so far. @jovirkku do you think you could do that subfolder thing still in this PR?

Yes, I'll make the Messages subfolder.

I have added the Messages subfolder hierarchy now:
Support/Help_Articles/Messages

Support/Help_Articles/Messages/Messages_App/
Support/Help_Articles/Messages/Messages_Exporting_as_Text/
Support/Help_Articles/Messages/Messages_Troubleshooting/

Please rebase


css 404 error after build

Hi there,
I'm building my first middleman site and am experiencing some issues with the stylesheet link. The css was running before, but after running 'bundle exec middleman build', I am getting a 404 in the stylesheets. I've added a default to the config.rb file, tried changing the file names etc but nothing seems to be working.
Any advice? I'm struggling a lot.
Thanks!

Can you share your repository?

Hi @elizaplowden If you can record your process, it might be easy to quickly figure out the cause.

https://github.com/elizaplowden/attomo
Here is the link to the repo.
It is a Middleman template with some basic html/css while I was learning to use Dato CMS. The integration was working and the app was running fine locally until I ran the build command. I believe my files are named correctly and the stylesheet is linked properly in layout.erb. I commented out the minify_css activation in config.rb and also added 'set :css_dir, 'stylesheets'' in the hope that this might fix the issue, but it didn't.
Thank you so much for your help! I'm really stuck.

Can you try commenting out any line that says "sprockets" including Gemfile and config.rb

@tdreyno I'm not sure the issue I've found is related to this one, but a clean middleman install won't build the CSS file. It throws the following error messages:
DEPRECATION WARNING: autoprefixer-rails was deprected. TypeError: Cannot read property 'version' of undefined
Disabling autoprefixer in the config.rb file actually fixes the problem (well, the CSS compiles):
# activate :autoprefixer do |prefix|
#   prefix.browsers = "last 2 versions"
# end

@htmlboy Looks like I need to publish a new version of middleman-autoprefixer. Until then, you can point your Gemfile at git
gem "middleman-autoprefixer", github: "middleman/middleman-autoprefixer"

Fixed by:
-gem 'middleman-autoprefixer', '~> 2.10'
+gem 'middleman-autoprefixer', '~> 3.0'

By doing this update
  * middleman (newest 4.4.0, installed 4.3.11, requested ~> 4.3) in group "default"
  * middleman-autoprefixer (newest 3.0.0, installed 2.10.1, requested ~> 2.10) in group "default"
  * middleman-livereload (newest 3.4.7, installed 3.4.6) in group "development"

I had a lot of TypeError: Cannot read property 'version' of undefined errors and the CSS was not built anymore while reviewing the CHANGELOGs no breaking changes were announced about that.
The previous message I put fixed the issue but also this is a good idea if you plan to use ruby 3.0.x:
-gem 'middleman', '~> 4.3'
+gem 'middleman', '~> 4.4'

unstale

unstale

I am also having an issue with autoprefixer. I'm using ruby version 3.1.0p0, and just installed Middleman and initted a new project. My gemfile reads:
source 'https://rubygems.org'

gem 'middleman', '~> 4.2'
gem 'middleman-autoprefixer', '~> 2.7'
gem 'tzinfo-data', platforms: [:mswin, :mingw, :jruby, :x64_mingw]
gem 'wdm', '~> 0.1', platforms: [:mswin, :mingw, :x64_mingw]

And when I run the build or server commands, the CSS doesn't compile unless I comment out autoprefixer. The error output is:
error  build/stylesheets/nora-brown-design.css
TypeError: Cannot read properties of undefined (reading 'version')
...

How can I get autoprefixer working?

unstale


Suggest Edit: chrisgrieser / Supercharged Citation Picker

Gallery URL
https://alfred.app/workflows/chrisgrieser/supercharged-citation-picker/
Website
No response
Forum
No response
About
No response
Notes
As noticable in this forum thread, it is not apparent from the Gallery Page that the citation picker can do more than just inserting citation ;) (Though the name does not convey it, sure – feature creep is real I guess.)
Not sure how much should go on the Gallery page, but I think the section "Feature Overview" and "Table of Contents" here, maybe together with an easier to find link to the
Also, I think it might make sense to drop the retirement for pandoc. It's only needed for an auxiliary feature that maybe 20% of the users are using. Telling the remaining  80% to install a dependency they'll never use feels unnecessary. The workflow already notifies the user when they are trying to use the feature without having pandoc installed.

Page revamped. If you feel some clarification is still needed, might be easier to edit the README and send it back.


BulkInsert/BulkSaveChanges ignores schema and fails

Describe the bug
When using Bulk operations that use merge on tables that are not in the default schema the operation will fail.
To Reproduce
Steps to reproduce the behavior:

Create an entity that is using a table that is not on the default schema. (eg. [Jobs].[JobSite])
Attempt to perform a BulkInsert
Error will occur Invalid object name 'JobSite'

Expected behavior
The bulk insert/save should succeed
Additional context
I'm pretty sure that the issue is caused by the usage of entityType.GetTableName() instead of entityType.GetSchemaQualifiedTableName() as the sql generated looks like this:
MERGE JobSite AS t USING [Jobs].[#tmp_be_xx_JobSite] AS s ON 1=2 WHEN NOT MATCHED THEN INSERT ([ApproximateBaitSquareMeters],[ChemicalLitresUsed],[ClientVer],[Comments],[CreatedBy],[DateCreated],[DateUpdated],[DateVisited],[IsActive],[IsNeighbouringSite],[JobId],[JobSiteActivityAreaId],[JobSiteStatusId],[Reason],[Rescheduled],[SiteId],[SiteStatusComment],[SpatialSiteId],[TreatmentMethodFootRatio],[TreatmentMethodId],[UpdatedBy]) VALUES ([ApproximateBaitSquareMeters],[ChemicalLitresUsed],[ClientVer],[Comments],[CreatedBy],[DateCreated],[DateUpdated],[DateVisited],[IsActive],[IsNeighbouringSite],[JobId],[JobSiteActivityAreaId],[JobSiteStatusId],[Reason],[Rescheduled],[SiteId],[SiteStatusComment],[SpatialSiteId],[TreatmentMethodFootRatio],[TreatmentMethodId],[UpdatedBy]) OUTPUT $Action,[s].[_be_xx_id],[inserted].[Id],[inserted].[RowVer],[inserted].[TemporalEndTime],[inserted].[TemporalStartTime] ;
Note the begining where it is MERGE JobSite where it should be MERGE Jobs.Jobsite

This bug has been resolved. Please try again with the latest version.


Tables + MVC & Devise implementation

Summary 📒
For this milestone, we:

Created the tables + MVC for

User
Food
Recipe
Recipe food

Added device & login/registration page.
Allow the creation of a new user, given a name + email.

🟢 STATUS: APPROVED 🟢
Hello @Gopxfs 👋,
Your project is complete! There is nothing else to say other than... it's time to merge it. 😁
Excellent work and congratulations! 🎉

Highlights
✔ Linters are passing.
✔ Added proper migrations to create the schema file with default values.
✔ Implemented log-in and sign-up.
✔ Added validation and relationship between models.
Nice work!
Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Update setup-steps-for-extensible-key-management-using-the-azure-key-…

…vault.md
Updated SQL Server Connector Version and supported key types.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We have closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions

Updated version


Add basic EspoCRM support

Adds a minimal integration to EspoCRM which add minimal Toggl Button in bradcrumbs on each detailed view.

@Hofundur can you please add some pictures of where you positioned the buttons?

Hey, we're cleaning up our backlog and since this PR has been inactive for a while, we're going to close it for now. Anyone please feel free to revisit it some time, and we will gladly review it again 🙇