Product pages does not load - stuck on pending

I installed the starter and most pages work correctly, except for the product page.
When navigating to a product page (either by clicking a product in the collections page, or pasting the url into the address bar) the document is stuck on pending in the network tab, with the following url: http://localhost:8333/_next/data/development/products/sweatshirt.json?handle=sweatshirt

I tried to change ports, restarted the server, Chrome and Safari, cleared cache, and more. Nothing worked :(
Also might be worth mentioning that on Firefox all API requests are stuck in a weird state, seems like they are not even pending. I'm not sure it's related, as other browsers are working mostly fine. Attaching screenshot:

Environment details:
Mac Mojave 10.14.6
Running backend on Docker
Storefront on host machine with `yarn dev`
Project generated from this template yesterday
Node v16.17.0
Yarn 1.22.19

For my part, everything works on dev, but in prod, all the product pages "504 - Gateway Timeout".

Hi @itailulu have you found any fix for that?


Suggest moving manifest values to gradle

I would suggest moving the following information (generated by flutter create) out of the manifest and into the Gradle configuration:

versionCode
versionName
minSdkVersion
targetSdkVersion

Also, you might want to move the Application label (android:label) to strings for consistency.

@jakobr-google

@jason-simmons do you have any opinion on this?  I don't know much about gradle.

Compared our template against Android Studio (version 2.3.3) and we are indeed not consistent:
Flutter has the following in android/app/src/main/AndroidManifest.xml:
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    package="com.yourcompany.flutterapp"
    android:versionCode="1"
    android:versionName="0.0.1">

    <uses-sdk android:minSdkVersion="16" android:targetSdkVersion="21" />
    ...

Android Studio has it in android/studioapp/app/build.gradle:
android {
    compileSdkVersion 25
    buildToolsVersion "25.0.3"
    defaultConfig {
        applicationId "com.example.mit.studioapp"
        minSdkVersion 25
        targetSdkVersion 25
        versionCode 1
        versionName "1.0"
        testInstrumentationRunner "android.support.test.runner.AndroidJUnitRunner"
    }

FYI
the android-release-guide needs some update to reflect these changes.

Good catch @namiwang! Working on resolving in https://github.com/flutter/website/pull/653


Output file contains more delimiters than needed for COPY FROM input file

The lines in the output file from bde_copy always end with a delimiter, like:
bde_copy -o pri_id t/data/pab1.crs /tmp/pab1.unl > /dev/null && cat /tmp/pab1.uni
4457326|
4457326|
4457326|

Later on, there's software that pass that output file to a COPY <table> FROM PostgreSQL construct, which failes because the input contains more columns than expected (the presence of a final delimiter means there's another null field at the end).
I'm not sure this is necessarely a bug with bde_copy because it could be an expected format instead, and then the bug would be in the caller/user.

There are no tests currently for using -o switch, see #26

Found: the pipe character (|) is part of the default line terminator !
/usr/local/bin/bde_crs /tmp/pab1.unl | grep XXX
XXX line_terminator switch: '|\x0A'

\x0A is linefeed.
Does it make sense for lineterminator to contain the pipe character, by default ?
If this is really intended, then the callers should override the default when the output is intended to be used with "COPY FROM" ...
@imincik do you know why the pipe character is part of that default line ending ?
Could it be the default line ending is read from input ? Maybe my removing the CR character from the input the result was that the last 2 chars read from the BDE file ended up being |\n...

I finally found where lineterminator is set!
/usr/local/share/bde-copy/bde_copy.cfg

Basically bde_copy has a default configuration file path, and make install installs such configuration file, and that configuration file has a default line terminator of |\n:
# Output field and line terminators
#
# The characters can include \xHH where HH are define a character as a hex value
# \r, \n, \t, \s for carriage return (\x0D), new line (\x0A), tab (\x0I) and
# space (\x20).  Use \\ for a backslash character. Any other character after a
# backslash is unaltered.
#
# Multi-character terminators are valid

field_separator |
line_terminator |\x0A

This explains the reason why the pipe is there. Is that a sane default to be installed ? This is best answered by @palmerj I guess. Me I'll focus on whether linz_bde_uploader makes sure that a proper configuration is passed to it in order to get a working format to use with COPY FROM

linz_bde_uploader fix was submitted as a PR here: https://github.com/linz/linz_bde_uploader/pull/93
An enhancement to linz_bde_copy to possibly avoid parsing the global config is in #27
A fix to remove the pipe from the line_terminator in the default configuration is in #28

Can this be closed now?

Yes, I think subtasks are all in place. Closing.


Accessibility (WCAG level AA) Links in the site's link bar have insufficient contrast

Download Paciello Group's Contrast Analyser from http://www.paciellogroup.com/resources/contrastAnalyser.  This tool measures foreground-to-background contrast ratios using the same algorithm defined by the W3C's Web Content Accessibility Guidelines.
Navigate to http://nuget.org/.
Compare the contrast ratio of the links in the link bar in the site footer against their backgrounds.

Result:  It's 4.46:1.
Expected:  This is below the threshold of 4.5:1 given in WCAG.  We should be able to fix this easily by slightly adjusting our color palette.

We have a new gallery design.


Problems with Visual Studio Created C/C++ ?!

Describe the bug
Ghidra seems to not like the executables that are written in Visual C++ compiled with Visual Studio 2017 Enterprise. In particular it does not recognize the library functions (e.g. printf) (in contrast to that it works for the basically same Code compiled with gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.10)). also it seems to not recognize objects of type 'const char * []' or in other words a Array of strings. additionally it seems like its discriminating for loops.
To Reproduce
Steps to reproduce the behavior:

fetch the executable for my Baby-Shark-singing-Program as found here: https://github.com/Marc-Bender/Baby-Shark/tree/binaries
decompile it with everything default

Expected behavior
should be clear…?!
Environment (please complete the following information):

OS: [Windows 10 x64]
Java Version: [11.0.2]
Ghidra Version: [9.0 Build PUBLIC 2019-FEB-28 1236 EST]

Are you using a PDB for type information?  Is so, there is a known issue where no distinction is made between char* and char[n] or multi-dimensional arrays when comparing PDB data vs source code.  PDB seems a bit deficient in this regard.
Could you please provide some samples of expected data types vs results in Ghidra and where they occur (i.e., function signature or data).

well let me just take screenshots of just About everything that happens… (to be honest i have no clue what a pdb is ...)

at this Point i have to ask myself where the main function is because there is no such function recognized
so lets use the educated guess that the strings Baby Shark Mommy Shark and so on are used there
therefor search for strings

lets jump to the Memory Location of that string

and search for the use of that string

jump to that single occurence…

this now Shows something that Looks remotely similar to the original source Code but:

you should see that this is hardly anything useful to the average Joe when compared to the original Code sideby side.

actually let me load a grab a gcc compiled Version of that and demonstrate what happens then

well this now Looks quite different it still Shows some strange stuff but it atleast is able to make matters a Little more human intelligable...

Can you post what is located at 0x411960, and also the address that 0x411960 refers to?
The function name and arguments are not being pulled up from the thunk to the external library.  My suspicion is the binary was compiled statically and the call to printf or printf has not been recognized for some reason.

@emteere actually the suspicion Sounds plausible. but cant you figure out the Memory Contents yourself? as i already pointed out the Code for the Program i tested ghidra with and found this issue is here on GitHub (link somewhere on the way up) and i actually did post the executable there as well. so everything should be reproducable by anyone without any assistance of mine.

I have a similar issue with basically the hello world example 🤔

i have same problem,look like it's can't  identify C lib function

Having basically the same issue, decompiled code is really hard to be understood. Using the latest Ghidra 9.1.2 Public release.

I experienced the same issue with Microsoft Visual Studio Professional 2017.


Edit talks v2: fetch speakers

This PR allows to fetch already available speakers for a talk. It introduces 2 API-Endpoints:

/v2.1/talks/[talk-id]/speakers will give a list of speakers for the talk [talk-id] with references to the user-url if the talk has been claimed
/v2.1/speakers/[speaker-id] will give a list with one entry for the speaker [speaker-id].

Both results look like the following JSON where the list for the /v2.1/talks/[talk-id]/speakers can contain multiple entries and the list for the /v2.1/speakers/[speaker-id] will only contain 1 entry (the entry for the requested speaker-entry).
Depending on whether the user has claimed the talk the full_name and twitter_username entries will contain a value or not and if the talk is claimed the entry user_uri is available or not.

I'm closing this as the talk claims issue JOINDIN-637 overlaps.


Challenge doesn't accept hex shorthand

Challenge Use Hex Code to Mix Colors has an issue.
User Agent is: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36.
For red and green, the challenge doesn't accept hex shorthand (#F00,#0F0)
My code:

I am red!

I am green!

I am dodger blue!

I am orange!

I found the same issue. Do not know if it is relevant since the material has not touched the subject of hex color short hand.

For me, using shorthand for red passes the tests but not for green.


Uncaught SyntaxError: Unexpected token ','

The following error is thrown in debug console on FF 87.0, Chrome 89.0.4389.114, and Brave v1.22.70. Wordpress v5.5.1. wp-browserupdate v4.3.1
I removed all plugins for my site and the error still occurs.
The debug console shows this line with error 'Uncaught SyntaxError: Unexpected token ','
var $buoop = {required:{e:,f:,o:,s:,c:},test:false,newwindow:true,style:"",insecure:,unsupported:,mobile:,shift_page_down:,api:2021.03};
The source code links to this code block:

This seems to be an arrow with the Wordpress-Plugin.
Here the values are missing: {e:,f:,o:,s:,c:}.
Sould be somthing like {e:11,f:77,...}
Can you contact the author of the Wordpress plugin or check if there is an error in your wordpress installation?


Cant download anything

Hi I just startet using this but when i enter a URL and a folder i only get errors

tis is the log to the picture "Total files downloaded: 0
Total images downloaded: 0
Total videos downloaded: 0
Errors: 1
Warnings: 0
Total download time: 0:00:00.100677
Files skipped because they were already downloaded:
Archivos fallidos:
Iniciando descarga...
Service extracted: onlyfans from site: coomer.su
Descargando post único...
Fetching user posts from https://coomer.su/api/v1/onlyfans/user/kittyxkum?o=0
Error fetching user posts: 404 Client Error: NOT FOUND for url: https://coomer.su/api/v1/onlyfans/user/kittyxkum?o=0
No post found for this ID.
All downloads completed or cancelled.
Logs successfully exported to resources\config\logs\log_20241126_171707.txt
Logs successfully exported to resources\config\logs\log_20241126_171707.txt
Iniciando descarga...
No se pudo extraer el ID del usuario.
URL no válida
Iniciando descarga...
Service extracted: onlyfans from site: coomer.su
Downloading all user content...
Starting download process...
Fetching user posts from https://coomer.su/api/v1/onlyfans/user/kittyxkum?o=0
Error fetching user posts: 404 Client Error: NOT FOUND for url: https://coomer.su/api/v1/onlyfans/user/kittyxkum?o=0
No posts found for this user.
All downloads completed or cancelled.
Logs successfully exported to resources\config\logs\log_20241126_171751.txt
"
how can i fix it?

Looks like the v1 api is either down or no longer in service.  It just dropped out about 2 hours ago and Cyberdrop-dl stopped functioning a day or 2 ago.
https://coomer.su/api/v1/onlyfans/user/(any creator)?o=0 is coming back with {"error":"Not Found"}

Coomer is making changes in the api, I already uploaded a version that solves it but they changed it again, I will wait a little bit until they stop making changes and I will modify the newest version.

Thanks. I was wondering if they were screwin around. Kinda funny to see I guessed right. I assume there will be a fresh build afterwards?

I'll wait for them to stop playing with the API, and in a few hours I'll upload a new one lol.

I'm going to upload the solution on discord instead of github for now.


WIP: STYLE: Fixed issue with GDCMImageIO docs

Fixed issue with GDCMImageIO doxygen (s. screenshot), updated one comment.
No code changes.
Before:

After:

@issakomi thanks for doing this :100: . Maybe changing the commit message prefix and the PR prefix to DOC: would be preferred, as this is a documentation issue. Would you mind doing that and push forcing, please? Once that is done, and hence the WIP removed, and hopefully all dashboard checks passing, this will be ready to be merged. Thanks.

Sure, i shall do it. Does someone have an idea what to do that with the statement below?
  /** Macro to access Rescale Slope and Rescale Intercept. Which are
   * needed to rescale properly image when needed. User then need to
   * Always check those value when access value from the DICOM header */
  itkGetConstMacro(RescaleSlope, double);
  itkGetConstMacro(RescaleIntercept, double);

It doesn't provide any useful information, IMHO, but, in fact, some info could be not bad, because there are examples with re-using of reader's gdcmIO for writer and re-scale values can easily cause problem, there were already issues and posts about it. I can also let it be as it is or reduce to just "Macro to get Rescale Slope and Rescale Intercept."  Please let me know.

I don't know what the problem was, especially if this PR fixes it.

Hrmm... This got me thinking we have this "grouping" script: https://github.com/InsightSoftwareConsortium/ITK/blob/master/Utilities/Doxygen/itkgroup.pl
It may be wise to run this script to determine it this auto grouping is a cause of the issue.

As rescale slope and intercept are automatically applied, the docstring for them could be changed/shortened.

As rescale slope and intercept are automatically applied, the docstring for them could be changed/shortened.

I think so too, a user can only read the value, not really access (change) it in itkGDCMImageIO.

Thanks for doing this @issakomi ! This is ready to be merged I'd say.

It may be wise to run this script to determine it this auto grouping is a cause of the issue.

@blowekamp may be that can be investigated locally/separately, and @dzenanz can merge this?

It may be wise to run this script

@blowekamp I have never used it. Can you run it?

and @dzenanz can merge this?

With the approval, either the PR author or maintainers can merge.

It may be wise to run this script

@blowekamp I have never used it. Can you run it?

Identifying a potential cause of the odd/erroneous Doxygen behavior was my contribution to this PR.

Unfortunately, as already expected, the PR didn't fix main issue with broken documentation of the class. There are still many fixed typos and is was renamed appropriately (just "Updated" instead "Fixed"), so i think there is no need to revert. Locally build documentation for this class was OK, further investigation are required. :(

@dzenanz
The problem with documentation is not limited to GDCM IO, i have clicked random class just now and already the first one has similar issue

Maybe turn this into an issue so we don't forget about it?

@dzenanz
OK, thanks, i shall open the issue

Each night I merge some PRs, I get the email below. Maybe the warnings in there are related to broken docs?
A submission to CDash for the project Insight has warnings. You have been identified as one of the authors who have checked in changes that are part of this submission or you are listed in the default contact list.
Details on the submission can be found at https://open.cdash.org/build/6817866.
Project: Insight
Site: blaster.kitware
Build Name: Linux-Ubuntu-GCC-Doxygen
Build Time: 2020-10-14 05:30:02
Type: Nightly
Total Warnings: 20
Warnings (first 5 included)
home/kitware/DashboardITK-build/Utilities/Doxygen/Modules/ITKCommon.dox line 28 (https://open.cdash.org/viewBuildError.php?type=1&buildid=6817866
Preprocessing /home/kitware/Dashboard/.../ITK-build/Utilities/Doxygen/Modules/ITKCommon.dox:28: warning: end of file with unbalanced grouping commands
s/Tests/ITK-build/Utilities/Doxygen/Modules/ITKBiasCorrection.dox...
Parsing file /.../ITK-build/Utilities/Doxygen/Modules/ITKBiasCorrection.dox...
Preprocessing /.../ITK-build/Utilities/Doxygen/Modules/ITKBinaryMathematicalMorphology.dox...
Parsing file
Utilities/Doxygen/Modules/ITKIOMeta.dox line 20 (https://open.cdash.org/viewBuildError.php?type=1&buildid=6817866
Parsing file /.../ITK-buil/.../ITK-build/Utilities/Doxygen/Modules/ITKIOMeta.dox:20: warning: end of file with unbalanced grouping commands
d/Utilities/Doxygen/Modules/ITKIOHDF5.dox...
Preprocessing /.../ITK-build/Utilities/Doxygen/Modules/ITKIOImageBase.dox...
Parsing file /.../ITK-build/Utilities/Doxygen/Modules/ITKIOImageBase.dox...
Preprocessing /.../ITK-build/Utilities/Doxygen/Modules/IT
Utilities/Doxygen/Modules/SmoothingRecursiveYvvGaussianFITK-build/Utilities/Doxygen/Modules/Thickness3D.dox line 171 (https://open.cdash.org/viewBuildError.php?type=1&buildid=6817866
Parsing file /.../ITK-build/Utilities/Doxygen/Modules/SmoothingRecursiveYvvGaussianF/.../ITK-build/Utilities/Doxygen/Modules/Thickness3D.dox:171: warning: reached end of file while inside a '```' block!
The command that should end the block seems to be missing!
ilter.dox...
Preprocessing /.../ITK-build/Utilities/Doxygen/Modules/SphinxExamples.dox...
Parsing file /.../ITK-build/Utilities/Doxygen/Modules/SphinxExamples.dox...
Preprocessing /.../ITK-build/
Modules/Core/Common/include/itkTriangleCellTITK/Modules/Core/Common/include/itkVariableLengthVector.h line 1471 (https://open.cdash.org/viewBuildError.php?type=1&buildid=6817866
Preprocessing /.../ITK/Modules/Core/Common/include/itkTriangleCellT/.../ITK/Modules/Core/Common/include/itkVariableLengthVector.h:1471: warning: end of file with unbalanced grouping commands
opology.h...
Parsing file /.../ITK/Modules/Core/Common/include/itkTriangleCellTopology.h...
Preprocessing /.../ITK/Modules/Core/Common/include/itkTriangleHelper.h...
Parsing file /.../ITK/Modules/Core/Common/include/itkTriangleHelper.h...
Preprocessing /.
home/kITK/Modules/Core/QuadEdgeMesh/include/itkQuadEdge.h line 452 (https://open.cdash.org/viewBuildError.php?type=1&buildid=6817866
Parsing file /home/k/.../ITK/Modules/Core/QuadEdgeMesh/include/itkQuadEdge.h:452: warning: end of file with unbalanced grouping commands
itware/Dashboards/Tests/ITK/Modules/IO/GIPL/include/itkGiplImageIOFactory.h...
Preprocessing /.../ITK/Modules/IO/JPEG/include/itkJPEGImageIO.h...
Parsing file /.../ITK/Modules/IO/JPEG/include/itkJPEGImageIO.h...
Preprocessing /.../ITK/Modules/IO/JPEG/incl
-CDash on open.cdash.org

Yep, I had noticed the warnings being consistently reported in CDash. I tried to have a look at where it comes from and potential fixes but unfortunately did not succeed in finding the source of the warnings.


webapp:fix the bug that listing linux webapp returns nothing

Fix #4000

[x] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[x] Each command and parameter has a meaningful description.
[x] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #4006 into master will not change coverage.
The diff coverage is 75%.

@@           Coverage Diff           @@
##           master    #4006   +/-   ##
=======================================
  Coverage   70.59%   70.59%
=======================================
  Files         427      427
  Lines       27528    27528
  Branches     4209     4209
=======================================
  Hits        19434    19434
+ Misses       6839     6835    -4
- Partials     1255     1259    +4

Impacted Files
Coverage Δ

...ice/azure/cli/command_modules/appservice/custom.py
73.44% <75%> (ø)
:arrow_up:

...li-cloud/azure/cli/command_modules/cloud/custom.py
14.45% <0%> (ø)
:arrow_up:

...nent/azure/cli/command_modules/component/custom.py
16.23% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
31.25% <0%> (ø)
:arrow_up:

src/azure-cli-core/azure/cli/core/util.py
70.66% <0%> (ø)
: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 da3a797...43ff192. Read the comment docs.


Fatal error on indexing.

Given file causing fatal error:
<?php
class A
{
    protected $smth = null;
    public function __construct()
    {
        $name = 'test';
        $b = new B;
        $this->smth = $b->$name;
    }
}

class B {
    public $test = 'value';
}

Error:
PHP Catchable fatal error:  Object of class PhpParser\Node\Expr\Variable could not be converted to string in /home/smeagol/.composer/vendor/mkusher/padawan/src/Padawan/Framework/Complete/Resolver/NodeTypeResolver.php on line 95
Catchable fatal error: Object of class PhpParser\Node\Expr\Variable could not be converted to string in /home/smeagol/.composer/vendor/mkusher/padawan/src/Padawan/Framework/Complete/Resolver/NodeTypeResolver.php on line 95

Line $this->smth = $b->$name; is the problem, if I change it to $smth = $b->$name; it's fine and $this->smth = $b->test; is also all right.
Tested on master branch (works all right on 0.2.1).

can you try https://github.com/mkusher/padawan.php/issues/63

tested, the fix works as charm!

tested, the fix works as charm!


Add languages for Office 365 update downloads - specifically for proofing tools only

Hi,
the "Add languages for Office 365 update downloads" details the required steps to ensure SCCM will download the required content to service additional languages but is this for instances where the language is completely different, a language pack is installed and/or only proofing language has also been installed?
thanks.

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

ID: d1da48af-4783-bd17-67f3-f265df93503f
Version Independent ID: 17432b22-04cb-f47f-1634-e5b7fc99c37d
Content: Manage Office 365 ProPlus updates - Configuration Manager
Content Source: sccm/sum/deploy-use/manage-office-365-proplus-updates.md
Product: configuration-manager
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

This would help soooooo much. We have a monthly updates package that includes our company supported language packs but there are some countries that want their language pack which we cannot add as the package will become way to large to distribute globally. Adding some proofing tools would solve this to a point. Please add this function!

Danxavy - submit this through userVoice, you'll get more attention there than in these comments.

Thanks again for the feedback. We moved this content to a new GitHub repository, so this issue is no longer tracked with the original article. As part of that process, we reviewed our backlog of feedback, and made some tough decisions on items that we don't think we'll realistically ever address. If you're passionate about this specific feedback, please refile it on the current article, now under docs.microsoft.com/mem/configmgr.


appdata: add vcs-browser support

These URL is visible on Flathub and GNOME Control Center.
More information: https://www.freedesktop.org/software/appstream/docs/chap-Metadata.html#tag-url

LGTM, thanks!


TRT Spor 2 and TRT Spor Yildiz

Hi,
TRT Spor 2 changed its name to TRT Spor Yildiz so these channels are the same. Could you please merge them?
Source: https://tr.wikipedia.org/wiki/TRT_Spor_Yıldız

Fixed in https://github.com/iptv-org/epg/commit/d086f1859bca9a350901073f22f6952cf7bb8214


make the data browser optional

Add a UI Option for the Data Browser (similar to the one for publication).  Default should be to not show it.
https://github.com/twosigma/beakerx/pull/7232

I have just downloaded the latest docker beakerx/beakerx.  I seem not able to find the "data browser" option.  Where can I turn it on?

Hi thank you for your question.
The browser is on master and will be released soon (this week).
The option to turn it on will be in the regular UI Options panel.


debug report tries to read from possibly stack allocated memory

in cube.c, VkPipelineColorBlendAttachmentState att_state[1] is allocated on the stack.
The debug report will try to read from it later, from vkCmdDraw, reporting bogus values.

Yes, this is a bug due to the fact that safe_struct generator treats 'VkPipelineColorBlendStateCreateInfo' as an exception which then prevents a deep copy of the attachments. Correct fix is to update safe_struct generator to handle static arrays such as the "blendConstants[4]" in "VkPipelineColorBlendStateCreateInfo".
I'll get around to this eventually but low priority at the moment.

Submitted #580 to fix this issue

Fix was pushed to master


Better doc for jax.clear_caches()

Information that may be useful there:
(1) Is this function thread-safe?
(2) When should this function be called?  Is this
(2.a) something just for people who want to benchmark compilation (and want to start with a "clean slate")
or
(2.b) something that regular users should use in-between large sequential parts of a JAX program?  E.g., a JAX program that does some intensive computation and, after that, does some other intensive computation.  Should one call jax.clear_caches() in between?  If so, is this the only jax.clear_() function one should call?

Thanks for the questions!

Is this function thread-safe?

The Python caller to jax.clear_caches() would need to hold the GIL, and so by virtue of the GIL I think it should be Python-thread-safe. In general JAX's internal caches assume they are protected by the GIL. Did you have some other scenario in mind?

(2) When should this function be called?

It could in principle be useful in either scenario you mentioned, but if user code needed it I would treat that as a bug, either in JAX (e.g. in a cache eviction policy) or in the user code (e.g. accidentally recompiling too much). In other words, if you have user code that needs jax.clear_caches() to work, please open an issue about it!

If so, is this the only jax.clear_() function one should call?

Yes, this is meant to be the one-stop API for clearing all JAX's internal caches.
Did I miss anything?


multiple conflicting/hiding variables and class members -> compiler warnings/fails

In many places of the codebase (including the headers of the precompiled Windows depthai distribution), the code uses duplicate variable names which conflict and/or hide other variables (e.g. class members). Leading to compiler warnings and in some cases compiler failures.
Fix is to not conflict/hide variables with same names. Can be found easily with a compile. And fixed easily by changing conflicting names (e.g. following Google style guide).
Setup

all compilers and OS
throughout codebase, yet can also be seen in the headers of precompiled Windows distrib:  depthai-core-v2.11.1-win64

Repro

Enable warnings and errors in compiler. e.g. in MSVC it is with /W4
Compile an app that uses the depthai-core-v2.11.1-win64 and simple code that cascades to headers like common/Rect.hpp or pipeline/Node.hpp. Most any code, even just a dai device, will do this.

Result
...
[build] Microsoft (R) C/C++ Optimizing Compiler Version 19.29.30136 for x64
[build] Copyright (C) Microsoft Corporation.  All rights reserved.
[build]
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/common/Rect.hpp(100): error C2220: the following warning is treated as an error
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/common/Rect.hpp(100): warning C4458: declaration of 'width' hides class member
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/common/Rect.hpp(129): note: see declaration of 'dai::Rect::width'
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/common/Rect.hpp(100): warning C4458: declaration of 'height' hides class member
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/common/Rect.hpp(130): note: see declaration of 'dai::Rect::height'
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/common/Rect.hpp(116): warning C4458: declaration of 'width' hides class member
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/common/Rect.hpp(129): note: see declaration of 'dai::Rect::width'
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/common/Rect.hpp(116): warning C4458: declaration of 'height' hides class member
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/common/Rect.hpp(130): note: see declaration of 'dai::Rect::height'
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/datatype/RawFeatureTrackerConfig.hpp(99): warning C4305: 'initializing': truncation from 'double' to 'float'
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/datatype/RawFeatureTrackerConfig.hpp(105): warning C4305: 'initializing': truncation from 'double' to 'float'
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai-shared/datatype/RawFeatureTrackerConfig.hpp(174): warning C4305: 'initializing': truncation from 'double' to 'float'
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai\pipeline\node/SPIOut.hpp(62): warning C4458: declaration of 'id' hides class member
[build] C:\repos-nobackup\depthai-core-v2.11.1-win64\include\depthai\pipeline\Node.hpp(199): note: see declaration of 'dai::Node::id'
...

Expected
No warnings or errors. To not use conflicting variable names.
Workaround
No good workaround. Setting flags to ignore errors/warnings would need to be somehow only applied to these headers. But the headers are not isolated to depthai compile units and instead included in user compile units. Therefore disabling the compiler errors/warnings would then put the user code at risk by not reporting problems in that user's code.

For example, you can change
https://github.com/luxonis/depthai-core/blob/41de567ea52443368224145b9e54a4f858abaeb8/include/depthai/pipeline/node/SPIOut.hpp#L46-L48
to
    void setBusId(int newId) {
        properties.busId = newId;
    }

This was necessary for my code to work. I also did the same kind of thing in https://github.com/luxonis/depthai-shared/blob/main/include/depthai-shared/common/Rect.hpp

@DBraun, luxonis is often happy to receive PRs. Since there are many of these, a reasonable approach could be for someone like you to submit a PR that fixes a subset of them. And then over time, the issue is progressively resolved.
Could you submit a PR with some of your changes and then we can all look/discuss. :-)

@diablodale Sorry, when I said "you" I meant Luxonis ;). There's probably a lot of places to refactor, and I didn't catch them all, just the ones that caused errors for me. I would prefer Luxonis make the changes since they know the codebase better.


Update README.md

Without dependent patch-db users will get error: failed to compile + the beta is over ;)

Closing in favor of #18


Make images of reusable components smaller

Some of our reusable component images are quite large, especially anything cuda related, which leads to long download / startup times. We should try to minimize the images of our reusable components.

Is this more of an issue for the remote runners (vertex, kfp) rather than the local one?
I think for the local runner pulling and running images takes minimal amount of time (once the large image is downloaded once).
For the remote one:

KFP: depends on the pulling policy
Vertex: I don't think Vertex image pulls can be cached so this might be where the bulk of the time is spent.

We also saw that pulling it once locally can take a long time, especially if you need to pull each image for a pipeline. Some images are >3GB.

Can we build two versions of the cuda images? One image with cuda and one without cuda support? We could use the non-cuda images for local testing. When a gpu is available, we could pull the image with cuda support.

@RobbeSneyders is that the case of all the images share the same base image (cuda/pytorch)? In that case, the large image will only have to be pulled once and all the layers on top of it (additional dependencies) should be lightweight.
@mrchtr that could work but it has to be estimated at compile time, we could do it based on the GPU config in the component Op. however, I'm not sure if that's the problem we're trying to solve (lightweight testing). I think we also want fast pulling when running with GPU

Building base Fondant images might make sense as well since installing Fondant adds 377MB to the docker image due to our dependencies.
We should investigate if the Fondant install layer differs between images if they're built separately, even if their Dockerfile is the same (the current case). If they're pulled separately for each image, a Fondant base image would help here by only having this layer (and other shared layers) pulled once.


Fix tautological conditions

As discovered by https://github.com/go-gitea/gitea/pull/30729.

There's another one at modules/indexer/code/indexer.go:181 that err is always nil

There's another one at modules/indexer/code/indexer.go:181 that err is always nil

Fixed in https://github.com/go-gitea/gitea/pull/30735/commits/392a25b1e89684017ff6b9f7ea81d0a85b73c0b8 by removing the impossible to reach block.

Two more issues fixed in git_test.go, I think this is ready for review/merge and I will handle the remaining errors in https://github.com/go-gitea/gitea/pull/30729.


GeoJson Polygons disappear when zoom in

I added a service response to my scene as a GeoJsonDataSource. At glance, it shows but when i zoom in it disappear.
This is my code:

https://codepen.io/geojorge/pen/XWodMxX

Hi there,
I'm having trouble requesting the GeoJSON resource with the provided key.
I did notice that terrain is enabled. You can either set Cesium.GeoJsonDataSource.clampToGround = true;, or omit the terrain option if not needed.

I set clampToGround = true, but it doesn't work properly.


Work around link limit with manifest download (#427)

Codecov Report

Merging #455 into master will increase coverage by 0.38%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #455      +/-   ##
==========================================
+ Coverage   85.12%   85.51%   +0.38%
==========================================
  Files          39       39
  Lines        1829     1940     +111
==========================================
+ Hits         1557     1659     +102
- Misses        272      281       +9

Impacted Files
Coverage Δ

hca/dss/util/__init__.py
93.54% <100%> (+2.63%)
:arrow_up:

hca/util/__init__.py
87.94% <0%> (+0.88%)
: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 4d844f3...a924e8d. Read the comment docs.

@hannes-ucsc I've responded to your review comments, and gone a bit further...
I changed the behaviour so that any errors encountered with linking are logged and then copying is done instead. Let me know if you think this isn't a good idea.


ci(vault): Migrate codecov token to vault

What does this PR do?
Migrate the codecov token to vault
https://datadoghq.atlassian.net/browse/ACIX-448
Motivation
Vault migration
Describe how to test/QA your changes
Ci is :green:
Possible Drawbacks / Trade-offs
Additional Notes

/merge

/merge


Nightlies are not working

Try running, sbt -Dsbt.version=1.3.0-bin-20190617T192002 and it will fail to resolve util-interface. I think this is because this is a pure java library and deployment is skipped by the validator.

Perhaps it's one step more indirect: it's being skipped because of its crossScalaVersions.

Should be fixed now.


Test osx failed ports

The test results of this PR determine how to update the baseline.
DO NOT merge this PR.

Results:
REGRESSION: open62541:x64-osx
NEW ADDED PORT FAILURE: libb2:x64-osx (ignore if triplet is not supported by upstream port)
NEW ADDED PORT FAILURE: tfhe:x64-osx (ignore if triplet is not supported by upstream port)
REGRESSION: freeglut:x64-osx
REGRESSION: wpilib:x64-osx
REGRESSION: glib:x64-osx
REGRESSION: libguarded:x64-osx
NEW ADDED PORT FAILURE: akali:x64-osx (ignore if triplet is not supported by upstream port)
REGRESSION: qhull:x64-osx
REGRESSION: cppmicroservices:x64-osx
NEW ADDED PORT FAILURE: libaaplus:x64-osx (ignore if triplet is not supported by upstream port)
NEW ADDED PORT FAILURE: libmagic:x64-osx (ignore if triplet is not supported by upstream port)
REGRESSION: bullet3:x64-osx
REGRESSION: pbc:x64-osx
REGRESSION: sfgui:x64-osx


update userBlackList

Coverage remained the same at 100.0% when pulling af7da7e47ba5930691c643ec84b279ec0722d5b0 on Lakate:testBot into 18c8d7555ee1f7ba439049f7e44683c22f7ed22a on FrodoTheTrue:master.


change prepublishOnly to prepare so package builds from git install

This will build the package for people who install it bare from a Git repo, which could be handy if someone wants to fork it and install their fork to test something (like me).
I'm sure it's possible you had a reason to use prepublishOnly, just putting this here in case it's helpful.

Thanks for the suggestion!


fix inconsistent model updates on change

Without initiating a digest with each model update, there are inconsistent model updates... reference here:
example

@cgallarno any thoughts on this? Thanks.

Sorry, for the delayed response, This looks good. Merging it in.


SpriteBatch not work in Chrome

Description
doc/examples/sprite_batch not work in flutter web (Chrome). Crashed on render method of SpriteBatch
Development environment
Flame version:
branch develop
======== Exception caught by rendering library =====================================================
The following UnimplementedError was thrown during paint():
UnimplementedError

When the exception was thrown, this was the stack:
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 216:49  throw_
lib/_engine/engine/html/canvas.dart 521:5                                     drawAtlas
packages/flame/sprite_batch.dart 73:11                                        render
packages/flame/components/sprite_batch_component.dart 21:17                   render
packages/flame/game/base_game.dart 113:6                                      renderComponent
...
The following RenderObject was being processed when the exception was fired: GameRenderBox#4579d
...  parentData: <none> (can use size)
...  constraints: BoxConstraints(w=500.0, h=966.0)
...  size: Size(500.0, 966.0)
RenderObject: GameRenderBox#4579d
  parentData: <none> (can use size)
  constraints: BoxConstraints(w=500.0, h=966.0)
  size: Size(500.0, 966.0)
==========================================================================================

Flutter doctor:
[✓] Flutter (Channel beta, 1.24.0-10.2.pre, on Mac OS X 10.15.7 19H15 darwin-x64, locale ru)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Xcode - develop for iOS and macOS (Xcode 12.1.1)
[✓] Chrome - develop for the web
[✓] IntelliJ IDEA Ultimate Edition (version 2020.3)
[✓] VS Code (version 1.51.1)
[✓] Connected device (3 available)

Runtime
This issue is related to running on which platform? (Select all that apply)

[ ] Android :android:
[ ] iOS :apple:
[x] Web :browser:

Minimal reproducible code (Required for bugs)
https://github.com/flame-engine/flame/tree/develop/doc/examples/sprite_batch

SpriteBatch uses an API that is not available on web, I wonder if we should add an assert on it to check if this is running on web. Or if we could somehow implement the Spritebatch API without using drawAtlas. Any thoughts @wolfenrain ?

https://github.com/flutter/flutter/issues/32217 canvas.drawAtlas added to the ToDo list with priority 5

SpriteBatch uses an API that is not available on web, I wonder if we should add an assert on it to check if this is running on web. Or if we could somehow implement the Spritebatch API without using drawAtlas. Any thoughts @wolfenrain ?

This is still my plan indeed, haven't gotten around on implementing it. It has some problematic caveats with it because of the RSTransforms that are used internally by the SpriteBatch. We cant reuse the transforms for other draw calls. So we need to introduce a different way of caching the original values.
The RSTransform only keeps the values that it calculates based on the arguments that go into its constructor. But we need the original arguments. One idea is wrapping the RSTransform into our own class that saves both the transform and the original values, so we can use either one of them depending on the platform.


checkers don't run if files are open with FZF

I'm testing FZF, pretty much adding this module to my .vimrc: https://github.com/junegunn/fzf/blob/master/plugin/fzf.vim
When a file is open with this command, syntastic doesn't run automatically on save. It works fine otherwise: typing :e to re-open the file in place, or running :SyntasticCheck works fine.
Any hint about what signal gets lost and how it can be fixed?
My vim configuration is this: https://github.com/dvarrazzo/dotfiles/blob/f116a4b14cc5d4effff240d576451b52c921700a/vim/vimrc#L240

Uhm: I'm afraid the problem does not repeat consistently. In the meantime I have discovered g:syntastic_debug so I'll try to use it next time something like this happens.

Syntastic relies on BufWritePost to run checks on write, that's basically all there is to it.  On a quick look I see no reason why FZF would interfere with it, but then again syntastic UI was written for Vim 7.4, and there were corner cases that were never handled correctly.  So if you expect robust behavior that didn't happen even in its prime.  You should consider using ALE instead, as it's written for Vim 8 and it's actively maintained.


RN fix description

Fix incorrect description that was copy-pasted from skylab mod

merged, thank you!


bower.json: fixed version number

Prevents "mismatch Version declared in the json (1.0.5) is different than the resolved one (1.0.6)" warning from Bower.

Oh... I think Bower will still fetch the old variant.


Use correct path to install neon node

This PR

uses path testnet/stacksnode instead of testnet to install stacks-node

Thanks for catching this friedger! We've fixed this as part of the PR to add mining support.


Suggestion to combat how long the list of bank tags are getting

The bank tags on the left side is great and it should stay there, but my suggestion is to add a new left click option which opens a menu containing all the bank tags in a 6 x 4 grid (when the screen is the smallest size).
You would be able to scroll down in this menu if you had even more bank tags, which I know I sure do. Also you would be able to re-arrange the icons within this menu and it wouldn't effect the order of the icons on the left side of the bank.
An imgur of this idea can be seen here for additional clarity https://i.imgur.com/Mg8ugri.png

lmao you have described the existing feature of bank tags "tag tab tab" almost exactly. I have a PR here https://github.com/runelite/runelite/pull/13332 but I wouldn't mind adding a menu swap for the "view tag tabs" button to this plugin since that PR is fairly stale.

Also you would be able to re-arrange the icons within this menu and it wouldn't effect the order of the icons on the left side of the bank
This is the only thing you asked for that isn't already possible with tag tab tab as far as I can tell.

Thanks for the quick reply. I don't see "tag tab tab" in the bank tabs plugin.


Allow to configure remote origin for copying links

Reported by @rchiodo: If you have a repo fork and your workflow is to commit to the fork, then PR to upstream, today we will copy links for the fork. Often the user really wants to copy links relative to the upstream, e.g. when sending permalinks to collaborators on the same repo. We may want to introduce a workspace setting to allow users to configure which remote is used when copying links.

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


[Bug] : Service Card have extra image in the end of card

@khushi-joshi-05
check this ,

in the second card's bottom have some unnecessary image

This issue is already solved.
Try to contribute to different issues.


Update sidekiq workers

What this PR does:

Do not retry the UpdatesSynchronizerWorker
Add custom queue sync

Notes
Disable the backtrace logging now that we use a service for that ( Sentry ).
https://github.com/mperham/sidekiq/wiki/Error-Handling#backtrace-logging

Enabling backtrace logging for a job will cause the backtrace to be persisted throughout the lifetime of the job. This can cause your Redis memory usage to grow without new jobs being added if a large quantity of jobs are failing repeatedly and being requeued.

You should use caution when enabling backtrace by limiting it to a couple of lines, or use an error service to keep track of failures.

 Do not retry UpdatesSynchronizerWorker


U-SQL Built-in Extractors : Incorrect information on time zone normalisation

Link to article:
U-SQL Built-in Extractors
Problem:
Article states that:

U-SQL’s built-in extractors normalize all date time values to UTC -07:00 and then drop timezone information if present.

In reality it normalises the values to the time zone of the environment executing the job that would be in force at the time of the value, if that makes sense.  (it also doesn't "drop" the time zone i.e. there is time zone information present in the output)
So running on Azure this is what happens:

2018-20-10T12:00Z => 2018-20-10T05:00-07:00
2018-20-11T12:00Z => 2018-20-11T04:00-08:00

Running on a local machine e.g. mine in the UK:

2018-20-10T12:00Z => 2018-20-10T13:00+01:00
2018-20-11T12:00Z => 2018-20-11T12:00+00:00

(note the differences are due to daylight saving)

@MikeRys Can you take a look at this feedback?


Ping using specific interface (SO_BINDTODEVICE error)

Your Windows build number:  (Type ver at a Windows Command Prompt)
Microsoft Windows [Version 10.0.17134.165]

What you're doing and what's happening:  (Copy&paste specific commands and their output, or include screen shots)

install WSL feature
install Debian GNU/Linux from MS store
open the app (opens bash command line)
sudo su  (optional, result same with and without)
ping -I eth3 google.com or ping -I eth3 -4 google.com or ping -I eth3 -6 google.com

What's wrong / what should be happening instead:
It should execute a ping using a specific network device (here eth3 - hosts lan connection)
What actually happens for ipv4 destination (-4 argument or ipv4-only host)

ping -I eth3 -4 google.com
ping: SO_BINDTODEVICE: Invalid argument

For ipv6 enabled destination
ping -6 -I eth3 google.com
setsockopt(SO_BINDTODEVICE): Protocol not available

Strace of the failing command, if applicable:  (If some_command is failing, then run strace -o some_command.strace -f some_command some_args, and link the contents of some_command.strace in a gist here).
Gist with straces: https://gist.github.com/Skywalker-11/4e9fa7550918fdeb11a0de451e2ce904

Ping without -I argument for using a specific interface is working correctly

Fair enough given #69 (MESSAGE) but really this is spiritually dupe #1349. nmap does the same thing here.
socket(AF_INET, SOCK_DGRAM, IPPROTO_IP) = 4
capget({version=_LINUX_CAPABILITY_VERSION_3, pid=0}, NULL) = -1 EFAULT (Bad address)
capget({version=_LINUX_CAPABILITY_VERSION_3, pid=0}, {effective=0, permitted=1<<CAP_NET_ADMIN|1<<CAP_NET_RAW, inheritable=0}) = 0
capset({version=_LINUX_CAPABILITY_VERSION_3, pid=0}, {effective=1<<CAP_NET_RAW, permitted=1<<CAP_NET_ADMIN|1<<CAP_NET_RAW, inheritable=0}) = 0
setsockopt(4, SOL_SOCKET, SO_BINDTODEVICE, "eth3\0", 5) = -1 ENOPROTOOPT (Protocol not available)


Module exports should appear in completion list as functions, not properties

Testing https://github.com/Microsoft/vscode/issues/12598

@waderyan very like we receive property from the tsserver in this case.

Repro'd

Moving to TS

https://github.com/Microsoft/TypeScript/issues/11219


feat(log): allow usage of multiple same log channels

Right now, it isn't possible to correctly declare AppendLogChannel more than once in LogConfig. It will result in same log, being logged in same channel multiple times.
This changes this behaviour, by storing Monolog instance by spl_object_id, not class name.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.455%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
7256

Relevant Lines:
8800

💛  - Coveralls

Thanks!


Cannot switch to old mode now.

npm install passbook --save
<EMAIL_ADDRESS>node_modules/passbook
└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
Error: Cannot switch to old mode now.
    at emitDataEvents (_stream_readable.js:730:11)
    at IncomingMessage.Readable.resume (_stream_readable.js:715:3)
    at ClientRequest.<anonymous> (/Users/dev/repo/pass/node_modules/passbook/lib/pass.js:299:20)
    at ClientRequest.g (events.js:180:16)
    at ClientRequest.EventEmitter.emit (events.js:95:17)
    at HTTPParser.parserOnIncomingClient [as onIncoming] (http.js:1688:21)
    at HTTPParser.parserOnHeadersComplete [as onHeadersComplete] (http.js:121:23)
    at Socket.socketOnData [as ondata] (http.js:1583:20)
    at TCP.onread (net.js:527:27)

update node to v0.12 solved this issue.

I am receiving this error running node v.0.12.4 and<EMAIL_ADDRESS>

I was able to resolve the error by commenting out line 32 of /lib/pass.js
301  response.pipe(file)
302  //response.resume();


How to define a multi-readfile project in samplenames?

Hi. I am trying to use the Genome-Binning-Pipeline to process one of my pacbio metagenome project. After reading the Sample-Config-Example.yaml, the way I understand it, is that one read file and one contig file are the way to do it. But in my case, I have co-assembled three read files into one contig file, for example: three pacbio read files p1, p2 and p3 were assembled into assembly.contig.fasta.
So please, how should I proceed with the tool? Thank you.

Unless the reads all come from the same sample or population, we generally do not recommend performing co-assembly using HiFi reads.
If you choose to work with co-assembled data, your best strategy will be to merge all of your HiFi reads into a single fasta file. You will want coverage estimates that are based on all of the reads that were used for assembly, rather than just one set of reads. This should allow the binning step to work as intended.
Please let me know if you have any other issues - documentation is in progress and will be available soon.


Allow plain text in calls

Issue Ref: Fix #698
Description:
Only set the header 'application/json` if the payload is a JSON object.
TODOs:

[X] Ready to review
- [] Automated Tests
- [] Docs

LGTM


Fix chef-13 deprecation warnings and fix spec typo

Signed-off-by: Seth Thomas<EMAIL_ADDRESS>Description
Eliminate deprecation warnings and fix typo
Issues Resolved
Fixes #81
Check List

[X] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[X] New functionality includes testing.
[X] New functionality has been documented in the README if applicable
[X] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

This doesn't fix the build entirely but that will require a follow up PR to address.


⚠ Bofrost.fr has degraded performance

In 8817a35, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 810 ms

Resolved: Bofrost.fr performance has improved in 2966f49 after 306 days, 9 hours, 31 minutes.


document that packaging needs to be installed before flash-attn

Do we also need to mention that ninja is required?

Do we also need to mention that ninja is required?

I haven't seen that ninja has been problematic, but we can address in another pr


No way of training the codebook

Hi!
Could you please explain how the codebook vectors are updated if the codebook vectors are not required to be orthogonal?

embed tensors in both Euclidean and CosineSim codebooks are registered as buffers, so they can't be updated at all
There is no loss on the codebook vectors that moves them closer to the input

Am I missing something? It seems that right now there is no way of updating the codebook vectors without the orthogonal loss.

@RafailFridman they are updated through an exponential moving average of the cluster statistics during training shown here

It seems totally broken to me. Please see the following code and output
import torch
torch.manual_seed(3026076450)
x = torch.randn((1, 4, 2))

from vector_quantize_pytorch import VectorQuantize
vq = VectorQuantize(
    dim = 2,
    codebook_size = 4,     # codebook size
    decay = 0.1,             # the exponential moving average decay, lower means the dictionary will change faster
    commitment_weight = 0.   # the weight on the commitment loss
)

print("codebook")
print(vq.codebook)
print(vq._codebook.embed_avg)

for _ in range(1000):
    y, *_ = vq(x)

print("codebook")
print(vq.codebook)
print(vq._codebook.embed_avg)

print("x, quantized x")
print(x)
print(y)

codebook
tensor([[ 0.8001, -0.2969],
        [-0.6842, -0.4783],
        [-0.6681,  0.4355],
        [ 0.1892, -0.0102]])
tensor([[[ 0.8001, -0.2969],
         [-0.6842, -0.4783],
         [-0.6681,  0.4355],
         [ 0.1892, -0.0102]]])
codebook
tensor([[ 8.0007e+04, -2.9693e+04],
        [-6.8416e+04, -4.7833e+04],
        [-6.6815e+04,  4.3555e+04],
        [ 4.7304e-02, -2.5412e-03]])
tensor([[[ 0.8001, -0.2969],
         [-0.6842, -0.4783],
         [-0.6681,  0.4355],
         [ 0.1892, -0.0102]]])
x, quantized x
tensor([[[ 0.8369,  0.9166],
         [ 0.1462,  0.1903],
         [ 0.3105, -0.5587],
         [ 0.1310,  0.1395]]])
tensor([[[ 0.0473, -0.0025],
         [ 0.0473, -0.0025],
         [ 0.0473, -0.0025],
         [ 0.0473, -0.0025]]])

Looks like this commit removed the update of self.embed_avg:
ema_inplace(self.embed_avg, embed_sum.t(), self.decay)

https://github.com/lucidrains/vector-quantize-pytorch/commit/8716f68d5549f5252d81e45651271f466d639356#diff-8bcd9c958b614ce130dc4091c094d4cfcc2023716823c181e3a9edaddbcd433dL250

@tasptz Find the same issue that embed seems not be updated during training.

@lucidrains Could u help to take a look? Why the update of emb_avg is removed after involving the multihead VQ feature.
ema_inplace(self.embed_avg, embed_sum.t(), self.decay)


Increase chrome timeout

Fixes https://github.com/matrix-org/complement-crypto/issues/9

Given it only happens infrequently today, a 50% increase feels more than sufficient. The docs for this seem to suggest Chrome can just decide to not print out its listening port, which is why this timeout exists in the first place:
	// Chrome will sometimes fail to print the websocket, or run for a long
	// time, without properly exiting. To avoid blocking forever in those
	// cases, give up after a specified timeout.
	wsURLReadTimeout time.Duration

If this does happen, it feels bad to wait a whole minute when 30s would probably have been fine.


Self service css/images

AB#7927 Supporting Self
Changes

Added new site brand API.  Gets site image and CSS file URLs from the site brand endpoint in the user service.  Has separate feature toggles self_service_site_images and self_service_css.
Removed kibble/api/css.go as the new site brand API makes it redundant.
Added new site brand model.  Data from the site brand API is mapped onto this model and it is stored on the site model.  Has helper functions for easy usage in core-template.  Also added tests for this model.
Changed render file writing function location from kibble/render/translations_api_renderer.go to kibble/render/file_renderer.go... I guess that makes it feel a bit more generic?
Changed render function to GET and write an email logo to the build output if an email logo URL is present in the site brand model.  NOTE: Unfortunately, the email logo must live in the build output as all one million of our email templates have hardcoded locations for this image.

EDIT 2022-08-11 (jackh)
Site Model Change

Removed CSSFilename property from the site model.  This property is no longer required thanks to the new site brand model but it unfortunately is a breaking change.  However, this property is not used by any clients in prod currently so IMO we can get away with it.

Misc Changes

Updated the golangci-lint github action because it was failing and wouldn't tell me why.  It works now.
Added DS_Store to gitignore.
Bumped golang.org/x/sys in go mod/sum because I couldn't build Kibble on my machine without doing that apparently?

I'm happy to approve once those last few comments have been addressed.  Better wait on @stajs approval before merge though.

Does this need to be blocked?

The CSS GET endpoint also had a breaking change though so there is currently no released version of Kibble that can handle  CSS files.  However, the only client in production with the feature turned on is the test client so not a big deal I guess.

I've updated the PR description a bit.  Could I please get some other eyes on this @additiverich @matthewthomsonnz @gscragg !

Nice work.

I'll let Bryny know!


Update request: greetd.regreet 0.1.1 → 0.1.2

Package Information

Package name: greetd.regreet
Latest released version: 0.1.2
Current version on the unstable channel: 0.1.1
Current version on the stable/release channel: 0.1.1

Checklist

[x] Checked the nixpkgs pull requests

Notify maintainers

@fufexan
Note for maintainers: Please tag this issue in your PR.

Add a :+1: reaction to issues you find important.

You said you checked the PRs?
https://github.com/NixOS/nixpkgs/pull/367520

Oops


Generated files not added to sources

Hi,
We have a build file with a genrule for handling different configurations. The genrule looks like this:
filegroup(
    name = "selected_config",
    srcs = select({
                ":dev_mode": ["config/dev.py"],
                ":staging_mode": ["config/staging.py"],
                ":prod_mode": ["config/prod.py"],
                "//conditions:default": ["config/dev.py"]
           })
)

genrule(
    name = "make_default_link",
    srcs = [":selected_config"],
    outs = ["config/default.py"],
    cmd = "cp $(location :selected_config) $@"
)

So on build, we generate a file default.py based on the selected configuration. That file will be located in the bazel-genfiles folder but we are able to import it in python just like any other source file from a py_binary rule.
Our issue is that Intellij is unable to resolve the imports on the default file because the location of the generated file bazel-genfiles is not added to the project configuration. We can add it ourselves manually on the Project structure as a source directory but it will be erased every time we resync the project.
It would be great if the genfile location could be added as a source dir or else have the ability to add any arbitrary folder from the .bazelproject file.
This issue might be a duplicate of https://github.com/bazelbuild/intellij/issues/205 but it seemed to me to be a relatively different kind of issue.

Hi there! We're doing a clean up of old issues and will be closing this one. Please reopen if you’d like to discuss anything further. We’ll respond as soon as we have the bandwidth/resources to do so.


confusion about the experiment of coef distribution

Hi!
I noticed that you drew 8 coefficients in the diagram which arep0,p1,p2, ..., p7. Does it mean that the function of x is $$x(t) = p_0t^7 + p_1t^6 + p_2t^5+...p_6t+p_7$$

@Chnnnne  Hi, sorry for the late reply. The formulation of the monomial basis polynomial in this paper is
$$x(t)=\sum_{i=0}^{n}{p_i}t^i $$


Multiple Publications:

Is your feature request related to a problem? Please describe.
We LOVE your newspaper script - it's amaazing. But we ran into a bit of an RP issue (lol)
We have a member wanting to post a "tabloid" paper (rumours, comedy, etc) but some players feel like it discredits the 'newspaper' as an official source of neews (basically mixing the satire amongst real news)
Describe the solution you'd like
Offer the ability to have different Publications / Categories. So Weazel News could have one section and Daily Globe could have another. Even adding tabloids or a section for comics
Describe alternatives you've considered
Copy Pasting the script and changing the name of the script/events/etc
Additional context
To go even further, being able to assign different jobs to different categories would be amazing.
Or assign one category to one ITEM, and not have all categories available from one item (each newspaper item only opens its respective category.)

The solution you provide is actually already possible. You can freely add another tab to the menu, where you list another type of news within. After that, copy/paste the database query for requesting a specific type of news, and you're good to go. Then you'll just need to add another type of "reporter menu", where the type of news that gets posted differs from the original one.
Hope it helps.


Reduce JobQueue interface

I was working in JobQueue for other reasons and ran across a small cleanup.  The JobQueue::shutdown() method is unused and untested, so I removed it.  There were two JobQueue::getLoadEvent* methods, one of which returned a std::unique_ptr and the other returned a std::shared_ptr.  The lone caller of the shared_ptr interface only needed a unique_ptr.  So I removed the shared_ptr interface and renamed the unique_ptr method.

Codecov Report

Merging #2099 into develop will decrease coverage by <.01%.
The diff coverage is 87.5%.

@@             Coverage Diff             @@
##           develop    #2099      +/-   ##
===========================================
- Coverage    68.84%   68.83%   -0.01%
===========================================
  Files          684      684
  Lines        50367    50356      -11
===========================================
- Hits         34674    34665       -9
+ Misses       15693    15691       -2

Impacted Files
Coverage Δ

src/ripple/core/JobQueue.h
91.66% <ø> (ø)
:arrow_up:

src/ripple/overlay/impl/PeerImp.cpp
0% <0%> (ø)
:arrow_up:

src/ripple/app/paths/PathRequests.cpp
36.03% <100%> (ø)
:arrow_up:

src/ripple/app/paths/Pathfinder.cpp
86.13% <100%> (-0.06%)
:arrow_down:

src/ripple/app/paths/Pathfinder.h
100% <100%> (ø)
:arrow_up:

src/ripple/app/misc/NetworkOPs.cpp
58.7% <100%> (ø)
:arrow_up:

src/ripple/app/misc/Validations.cpp
59.53% <100%> (ø)
:arrow_up:

src/ripple/rpc/impl/RPCHandler.cpp
61.84% <100%> (ø)
:arrow_up:

src/ripple/core/impl/JobQueue.cpp
87.43% <100%> (+1.26%)
:arrow_up:

src/ripple/beast/asio/io_latency_probe.h
97.01% <0%> (-1.5%)
:arrow_down:

... and 2 more

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 1dbc5a5...a6a43e7. Read the comment docs.

@bachase: Yeah, when I was working on this code I thought briefly about following that suggestion.  But I did decide to limit the scope of this to just these few things bugging me in JobQueue.  I'm trying not to get distracted while I'm dealing with other JobQueue related work.

Merged as 1a238048d5428f674f6116c7dd53459d94780f85


Eviction and automatic fetching from page state store

This adds proper eviction to the page state store, which means pages won't be stored in the state forever. By default, 25 pages with state are allowed. Past that number, the oldest pages will have their states removed. Additionally, this will minimize the number of network requests by avoiding any network requests to pages that already have a state in the PSS (which would have been preferentially used anyway). That should speed up Perseus apps exceptionally, and it's not a breaking change due to the priorities introduced in v0.3.4.
The eviction system is a breaking change however, as it removes the Default implementations for both RenderCtx and PageStateStore, while also adding the eviction system, which may break some uses of freezing/thawing.

Alright, the system to prevent unnecessary network requests is done! There's one quirk: a page is only 'cached' properly after it's been loaded through the subsequent loads system. In other words, the very first page the user loads won't be properly cached unless they go back to from another page. The reason is that full caching requires having the <head> as a string, which we don't have for that first page, since the head comes pre-interpolated. Reading it from the HTML is risky, since it will probably already have been modified by JS, and I don't want to send it through with a JS variable, because that would potentially increase the size of the first page HTML files dramatically.
Also, the PSS does have a (configurable) maximum size. By default, if a user visits page A, then 25 others, none of which are page A again, the PSS will drop the information for page A from the cache.

I've also expanded this to begin building the preloading infrastructure in the client-side systems, which will eventually grow into a system for pages to automatically declare the preloading of other pages.
Note: tests should also be fixed after the merge from main.

I've now built the user-facing preload system, along with an example to accompany it! You can now preload any page in a Perseus app with the minimal number of network requests, fully ahead-of-time, with a single line of code! (I'll flesh this out fully as mentioned above in a future PR.)

Alright, I've enabled initial page caching under an enabled-by-default feature flag by using a <meta> tag to delimit the end of the head (this is created in such a way that it would be very hard for the user to add essential content after it). That can then be used to deduce what the head was and cache it. That should make this ready for merging!


moveo_moveit/ArmJointState.h: No such file or directory. Tried everything in my current power. No luck

Hi all,
I am trying to get moveo to work on Ubuntu 20.04 with ROS Noetic and am getting the "moveo_moveit/ArmJointState.h: No such file or directory" error. I tried all suggestions that I could find, but no luck. Manual copy of this file results in a cstring error, which, when resolved, causes a bunch of other issues, possibly related to the Eigen library.
I even tried to revert back to Ubuntu 16 with ROS Kinetic. Same issue. Deleting ros_lib does not solve the issue.
When I load the rosserial library (I tried 0.9.1 which causes an issue with cstring, I tried 0.7.8 and 0.7.9 which cause the ArmJointState error), I noticed that it actually installs a Rosserial_Arduino_Library instead of ros_lib.
Is there anyone who has gotten this to work on Ubuntu 20.04 with ROS Noetic? I am at a loss and have been working on this on/off for days. I am sure I can figure it out after spending another few days, but any help would be appreciated.
I have the simulation working, its going wrong at ros serial. For now my beautifully built robot arm sits aimlessly pointing at the ceiling....

What the heck! Working on this for days, and I just decided to write the above issue, downloaded Rosserial 0.9.1 one more time and now it compiles!!! No idea why it works now.

heya buddy , how did you manage to fix it? im struggling with this at the moment.

I resolve the issue: https://github.com/jesseweisberg/moveo_ros/issues/37#issuecomment-896127857

You need to source the moveo_ros workspace before building library for arduino.
Here is the link of  updated readme: Tenveis:arduino_compile
You can find the instructions in Troubleshoot section.
Hope this helps.


bump postgresql versions

https://www.postgresql.org/about/news/postgresql-133-127-1112-1017-and-9622-released-2210/

I believe the failing tests for 9.3 and 9.4 are expected, as per https://github.com/aptible/docker-postgresql/pull/114
For >= 9.5 the tests are passing - should this be merged, or will this just wait on the fix for the EOL versions/postgis?


Feature Request: Cloud autosave/autobackup/sync

As per title, having the tabs stored/retrieved/synched by one of the many cloud services (the easiest service of your choice to start) would be a great addition.

What would be your purpose ?
I don't find any improvement in relation to exporting it like a file.
Also, It will add a lot of dependencies to external API, that I am not really keen on.
For now, this is not my priority.

@NeatWolf: As Tabs/Groups are stored as ordinary bookmarks, you may use the bookmark sync add-on of your choice to accomplish this task. FF has it's own sync service already(?).

From now, I prefer stopping the use of bookmarking auto save due to the many problems it can create. However, once it will be fixed in will be a solution.
Another one (dirtier) would be:

To have the cloud application that synchronizes a folder on your computer
Create a link from the saved file of the extension in your synchronized folder (it is easily done on Linux, I don't know for Mac & Windows)

Saved file path example:
~/.mozilla/firefox/{letter&number).{profile name, often<EMAIL_ADDRESS>Be aware that this file is the one that makes possible to retrieve your groups on FF start. You might lose data if you modify it!


Modular resolver structure pt. 3

Move resolvers into structure
box_transfer/
    agreement/
        crud.py
        fields.py
        mutations.py
        queries.py
    shipment/
        crud.py
        fields.py
        mutations.py
        queries.py

 2.0 modular, domain-centered file structure in BE


Extra Blank Spaces Between Elements In Rendered Template

Version
2.5.13
Reproduction link
https://jsfiddle.net/aoxo2rod/8/
Steps to reproduce
Run the JSFiddle
What is expected?
The background should be green, styled by ul:empty, as the children of ul are all conditionally not rendered: empty v-for and v-if that evaluate to false.
What is actually happening?
The background is red, ul:empty is not used because there is a blank space inside the ul tags, which CSS doesn't consider to be :empty.

The multiple children elements are important, a single li element with v-if="false" will produce the correct result. It appears Vue is putting a blank space between the elements, even if they are conditionally not rendered (appearing as  instead). It's worth noting that if you manually remove the extra spaces using developer tools, everything looks as expected,  does not affect :empty.
Furthermore, this only reproduces if there is a newline between the elements. Modifying the provided JSFiddle to only have the two li with v-if="false" will produce the correct result if there's no newline between them. So Vue seems to be turning the newline between the elements into a single space, which is causing the issue.

The same thing happens with regular html and empty lines, the css does not apply:

If you want to have no spaces you can, however, remove any white spaces in the template:

  {{ item }}

Not closing because I'm not sure about the new line being replaced by an empty space but to me, this behaviour is more consistent than automatically removing whitespaces as using newlines in regular HTML would yield the same result

If you want to have no spaces you can, however, remove any white spaces in the template:

That's not really a practical solution, though. Putting everything on one line turns it into an unreadable mess for anything more than the most trivial of templates.

Not closing because I'm not sure about the new line being replaced by an empty space but to me, this behaviour is more consistent than automatically removing whitespaces as using newlines in regular HTML would yield the same result

But Vue does collapse everything inside the ul into a single line, it's already chomping the newlines, it's just turning them into spaces instead. If it wants to preserve newlines, then sure, that would be consistent with how vanilla HTML acts, but it doesn't currently do that, and as such the results are unintuitive.
@Justineo, using preserveWhitespace does seem to get the intended result, but as you said, it's only available via vue-loader at the moment. Is there any way to activate this for a single node? It's a bit icky to have to turn it on for an entire project if you only want that behavior for a single template, or even just a single part of a single template. It'd be great if there was a directive like v-preserve-whitespace="false" that could be used on an individual node. I briefly looked at writing my own directive to get that end result, but the only reasonable hook to use seemed to be componentUpdated and changing the element's innerHTML at that point led to inconsistent results, it's probably not a good place to touch the HTML.

Is there any way to activate this for a single node?

Like Evan said here (except that this might not be solved by CSS), I think it's better that we keep it consistent across the whole project. Custom directives may work if you just remove whitespace-only text nodes instead of setting innerHTML directly.

Like Evan said here (except that this might not be solved by CSS), I think it's better that we keep it consistent across the whole project.

That doesn't really work for reusable components, a library can't (or at least shouldn't) force users to change project-wide settings. It's also not applicable if they're not using vue-loader, at the moment.

Custom directives may work if you just remove whitespace-only text nodes instead of setting innerHTML directly.

You're right, that seemed to make it work nicely. Example directive below for anyone who comes across this looking for a solution:
function trimEmptyTextNodes (el) {
  for (let node of el.childNodes) {
    if (node.nodeType === Node.TEXT_NODE && node.data.trim() === '') {
      node.remove()
    }
  }
}

Vue.directive('trim-whitespace', {
  inserted: trimEmptyTextNodes,
  componentUpdated: trimEmptyTextNodes
})

So the open question remains regarding Vue chomping the newlines into a single space. Is there a rationale for this behavior? If it chomped the newlines into nothing, rather than a single space, this would work out of the box.

Collapsing new lines and white spaces is consistent with the behavior of HTML.

HTML renders any consecutive whitespaces (including newlines and everything) as a single space. So keeping the single spaces results in the exact same render output while reducing the payload. This is intended and is in fact consistent with what you'd expect with plain HTML.

@yyx990803 Unless you use
 tag (or, in general, white-space: pre-wrap;).

so have this code in my template:


Fix: Use unique key for S3Files for folder copy progress

When copying a folder from S3 to local, in copy(from s3Folder: S3Folder, to folder: String...) FolderUploadProgress's init was using S3File.name to calculate total size of all the files.  S3File.name will not be unique if there are two or more files with the same filename in different subfolders. For example if you tried to download this folder from S3 it would assert assert(folderProgress.finished == true):
file.txt
subfolder/file.txt

In release 0.4.1


docs: add documentation for BFS (#56)

Hello, I made the documentation for BFS.
Please Check the ## Syntax section. I understood not exactly what I did here.
Also I'm not sure with the content at ### BFS is particularly useful for: , because I not used this algorithms in praxis.
I have only experience with dijkstra in robot programming.

Thanks for your contribution 🚀
It looks very detailed, I will take a closer look tomorrow. Tagging #56

Thanks, LGTM 👍🏻


Add C# call flags API

Fixes #6766 and adds a general API to pass call flags like Cacheable or Idempotent.
Probably worth adding some tests before merging, but it should be ready for first pass of review.

After rethinking the CallOptions API a bit, I decided to make the newly introduced CallFlags internal and CallOptions now only exposes  .WithWaitForReady()   and IsWaitForReady (both marked as experimental for now). That exposes the necessary knobs while minimizing the API impact.
PTAL.


Add info about auto regeneration of TLS certs

MinIO Operator v4.3.2 or below

No auto renewal of certificates

MinIO Operator v4.3.3 to v4.5.3

MinIO Operator renews Tenant Certificate when the Tenant Certificate has 48 hours or less before expiration, introduced on https://github.com/minio/operator/pull/872

MinIO Operator v4.5.4 and above (including latest v5.0.14)

MinIO Operator renews Tenant Certificate when the Tenant Certificate duration has reached 80%, Introduced on https://github.com/minio/operator/pull/1332.

Example:
Certificate was issued 01/01/2023 and expires on 12/31/2023 (365 days).
Operator will evaluate the elapsed duration, once the day 292 have been reached (Oct 19th 2023) Operator will renew the tenant certificate

Note:

Depends on Operator being running at the moment the certificate reaches 38 hours or less
Removed TLS certificate for Operator Console in 5.x.


[SMALLFIX]  modified:   integration/fuse/src/test/java/alluxio/fuse/A…

…lluxioFuseFileSystemTest.java

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/17190/
Test PASSed.


Wayland display setting

Summary: Added clientDisplayName setting to get rid of rdk patch
Type: Feature
Test Plan: UT
Jira: N/A

https://jenkins.code.rdkcentral.com/job/github-triggered/34090/ started

Job started https://jenkins.code.rdkcentral.com/job/manual-verification/3296/

https://jenkins.code.rdkcentral.com/job/build-component/72079/ started.

Sanity tests for ['RPI3 Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/11665/

Sanity tests for ['RPI3 Hybrid/MC'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/11666/

Job started https://jenkinsrdkm.cmf.code.rdkcentral.com/job/manual-verification/90/

https://jenkinsrdkm.cmf.code.rdkcentral.com/job/build-component-platform/5101/ started.

Sanity tests for ['BCM Reference Refapp2'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/11698/


Rodemacher to Brame Energy Center Mapping Error

The FERC Plants Rodemacher 1, 2, and 3 should be mapped to the same plant as the Brame Energy Center in EIA -- they are the same plant, just different names, and so should have the same PUDL ID.

Did we update the plant mapping spreadsheet to fix this issue when we integrated last year's new data? @cmgosnell @swinter2011


[NMessengerViewController] Get visible cell nodes

Hi,
I want to get the messenger view visible cell nodes. Do you know how i can achieve that ?
Thank you for your help

@AndreMarvell MessengerView has a MessengerNode actual it is an ASTableNode. So you can easily access visible cell nodes:
self.messengerView.messengerNode.visibleNodes

Thank you for your answer @dangthaison91.
The property seems to be available on messengerNode but unfortunately, i have an error on the build :
"Value of type 'AsTableNode' has no member visibleNodes"


[WIP] Implement http proxy

Implement http proxy!
Should still need refine some code and add more unittest
You could test with
mpserver --mode http
curl -x localhost:5580 http://www.google.com
curl -x localhost:5580 http://www.google.com -p
curl -x localhost:5580 https://www.google.com -k

Still WIP, but if you could help review and advice that would great!
Thanks

Ref: #202

Coverage decreased (-0.1%) to 85.812% when pulling b914c73253a60f1d311e5b8972d1eb53ba52354f on http-proxy into 689ac52c7502e6f46f4f28b25c4dbc25cfe5ec81 on master.

Coverage decreased (-0.1%) to 85.799% when pulling ab6eadf35b0d7260887a067bde8486706130aa74 on http-proxy into 689ac52c7502e6f46f4f28b25c4dbc25cfe5ec81 on master.

Coverage increased (+0.5%) to 86.443% when pulling 161cb8358b3461ca25cd28ce8c316bccb6fad136 on http-proxy into 689ac52c7502e6f46f4f28b25c4dbc25cfe5ec81 on master.

I'll take a look at it. Thanks for the great work.

Coverage increased (+0.5%) to 86.462% when pulling 9ba646be74430c50c329187c9ffe81390c1e5a4e on http-proxy into 689ac52c7502e6f46f4f28b25c4dbc25cfe5ec81 on master.

merged thanks.


Feature: extract rails snowpack config into an npm package

On second thought, I prefer users being able to see the config.


Fix odo log to pass container name

What type of PR is this?
/kind bug
What does does this PR do / why we need it:
passes container name for odo log, we are already passing container name for odo log -f, It seems it got missed from odo log
Tests was not able to catch this as tests are written with nodejs devfile which uses a single container.
Which issue(s) this PR fixes:
Fixes #3711
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:

can we have an regression test for this?

can we have an regression test for this?

Adding, apart from this there are some issues with springboot devfile to stream logs, looking on to them, Marking this as WIP

Any update on this @adisky ?

Any update on this @adisky ?
@girishramnani It would get fixed only with update in springboot devfile. The scripts inside not piping stdout to odo log, once we update the image, it would get resolved. This change is also required, but only with this change it is not working.

@girishramnani PTAl, updated integration tests with multi containers devfile.

works
/approve

/lgtm

[odo]  ✗  image stream nodejs with tag latest not found in openshift and mbupgvlpas namespaces

/retest


(fix) specify exception when photo filter creation fails

Coverage increased (+0.02%) to 90.652% when pulling 4fea737c03b10ab844a1b7125c8f41a9dbccd385 on feature-socialmedia into 891f6f973daa48e041522fdb754843c8f202059c on develop.


a showMonthPicker will be welcomed

A picker just pick year and month

sorry. it's beyond the spec of this package currently, but you can fork it and just remove the right column


A small suggestion to group tasks

https://docs.github.com/en/issues/tracking-your-work-with-issues/using-issues/adding-sub-issues

Not sure - it doesn't show up for us... 😅


Add profiler instrumentation to data provider and performance documentation

This change adds information re: adding profiler instrumentation to custom components and adds a discussion of the Unity Profiler to the performance guidance page.

/azp run mrtk_docs

/azp run


SetupTime part of TransportTime

As suggested by @lafave here : https://github.com/graphhopper/jsprit/pull/224
SetupTime is now part of the TransportTime/Cost. It just induced to add a parameter.

Got a few update on it, following the actual state of the master branch

Is there any plans for this PR?


[Sprites] guitar back and some resprites

About the PR

welding masks icon repsrites
acoustic guitar backpack sprite
health analyzer icon resprite
Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: lzk

tweak: Changed sprites

It is missing screenshots

shit... branches somehow merged together

health analyzer meta was not updated in #16125, so i dont think that should be updated here


FEATURE: CIP30 data sign doesn't support generating hashed content

Hardware wallets, e.g. ledger typically will only return message as a blake224 hash. If this is the case they typically add: hashed: TRUE in the unprotected header section.
Please add support for ability to sign hashed content to the java class.
https://github.com/bloxbean/cardano-client-lib/blob/24959cbe7928788d2426a7dd32bbf45ed08c190b/cip/cip30/src/main/java/com/bloxbean/cardano/client/cip/cip30/CIP30DataSigner.java#L22
Typically I would expect a new overloaded method where I can sing the payload and the signature will be using blake2b224.

Example:
payload (text):
{"action":"LOGIN","actionText":"Login","data":{"event":"CARDANO_SUMMIT_AWARDS_2024","network":"MAIN","role":"VOTER","walletId":"stake1uypayp2nyzy66tmcz6yjuth59pym0df83rjpk0758fhqrncq8vcdz","walletType":"CARDANO"},"slot":"136068943"}

sig: 84582aa201276761646472657373581de103d205532089ad2f7816892e2ef42849b7b52788e41b3fd43a6e01cfa166686173686564f5581c1c1afc33a1ed48205eadcbbda2fc8e61442af2e04673616f21b7d0385840954858f672e9ca51975655452d79a8f106011e9535a2ebfb909f7bbcce5d10d246ae62df2da3a7790edd8f93723cbdfdffc5341d08135b1a40e7a998e8b2ed06

key:
a4010103272006215820c13745be35c2dfc3fa9523140030dda5b5346634e405662b1aae5c61389c55b3

payload (hex):
7b22616374696f6e223a224c4f47494e222c22616374696f6e54657874223a224c6f67696e222c2264617461223a7b226576656e74223a2243415244414e4f5f53554d4d49545f4157415244535f32303234222c226e6574776f726b223a224d41494e222c22726f6c65223a22564f544552222c2277616c6c65744964223a227374616b6531757970617970326e797a793636746d637a36796a757468353970796d3064663833726a706b30373538666871726e6371387663647a222c2277616c6c657454797065223a2243415244414e4f227d2c22736c6f74223a22313336303638393432227d

Thanks @matiwinnetou . We will look into it.

https://github.com/bloxbean/cardano-client-lib/pull/453/files


Wrong path?

In the section
https://builderbook.org/books/builder-book/book-and-chapter-models-internal-api-render-chapter#internal-apis
at some point we read

In this case, three smaller objects are:
{ credential: 'include' },
opts (which is emptry by default)
{ headers }.
Put it together, you finally get:
server/lib/sendRequest.js :

I think the path
server/lib/sendRequest.js

should be instead
lib/api/sendRequest.js

@realtebo good catch! You are correct. I fixed this in the book content.


Thank you

Thanks for sharing this project. I hope it develops further.

Stay tuned!!

Thats true ?

Hey, @productdevbook thanks for raising the issue.
Could you please elaborate what's the issue exactly?
Thanks

@hiashutoshsingh iam add button error app null check operator used on a null value library mds

Hey, @productdevbook we have published a new version (0.0.2), please try again with the new version.
Thanks.

@akshatkaneria dont fixed

@akshatkaneria thank you fixed


Returning from message view scrolls the message list

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
6.301
Where did you get the app from?
F-Droid
Android version
12
Device model
Pixel 4a
Steps to reproduce

have unread messages that cover more than one screen
Have unread messages being displayed always at the top of the list
Read the top message
Return to message list
message list scrolls down to the now read email

Expected behavior
Previously it stayed at the top of the list which seems more sensible to me. The view scrolling potentially somewhere way down after reading an email at the top of the unread pile is certainly unexpected.
Actual behavior
Message list scrolls to the bottom of the unread pile/top of the read pile.
Logs
No response

It's actually not necessary to have more than one unread email for this to trigger. it just needs to be sorted on top as unread but further down in the list in it's read state to trigger the scrolling,

I found the commit that introduced this behavior: https://github.com/thundernest/k-9/commit/8b546cadd6e23d64d88814f3fdf37b4c1fd6331a
I too am frustrated by this new behavior. Can it please be gated by a setting? I really don't want my message list to auto-scroll at all, as it messes with my muscle memory when rapid-firing though lots of messages.

Is this still a problem in K-9 Mail 6.400?

I'm not able to test 6.400 right now since it hasn't hit F-droid yet, but I tested 6.312 and the issue is not present in that version.
Thanks!

but I tested 6.312 and the issue is not present in that version.

Same here!


Fix crashes from null model

editor.getModel(resource) can return null at times - e.g. if the user navigated away from the editor in a React app. This checks for the model existing first before attempting to call getModeId.
Prior art: monaco-json does this as well
Fixes: https://github.com/remcohaszing/monaco-yaml/issues/106
FWIW the typing for getModel returns ITextModel | null - not sure why this is not impacting the type checking of the code in current state.

FWIW the typing for getModel returns ITextModel | null - not sure why this is not impacting the type checking of the code in current state.

The types  | null and  | undefined are ignored in TypeScript without strict null checks. I would like to enable this eventually, but it wasn’t enabled when I was made a maintainer.

Released as v3.2.1 :tada:


Set policyName in DescribeScalingPolicies input request

When calling ReadOne for ScalingPolicies the controller does not set
policyName into the input causing it to list all the policies related to a
resource and (possibly) pick the wrong one. This PR adds to a controller
the ability to set a policyName to the request, making it easier to read the
latest state.
Description of changes:

Set policyName in DescribeScalingPolicies input request

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Sync offline, Controller errors are gone with the fix in https://github.com/aws-controllers-k8s/applicationautoscaling-controller/pull/105/commits/c3e71e98b8d9e970475f978eeb9aadc4340e89ea
Next step is to fix/add a test. SageMaker endpoint can scale on 2 different metrics on same dimension, so use that instead of dynamoDB test


malformed JSON when saving sframe array of loaded images using export_json()

Getting malformed JSON:
[
{"id":0,"path":"./data/104/26979/train/blank.jpg","image":Height: 432 Width: 768}
,{"id":1,"path":"./data/104/26979/train/intro.jpg","image":Height: 432 Width: 768}

Height and Width are missing quotes and I think they're should be parentheses around them like this:
[
    {
        "id": 0,
        "path": "./data/104/26979/train/blank.jpg",
        "image": {
            "Height": 432,
            "Width": 768
        }
    },
    {
        "id": 1,
        "path": "./data/104/26979/train/intro.jpg",
        "image": {
            "Height": 432,
            "Width": 768
        }
    }
]

Code looks like this:
reference_data = tc.image_analysis.load_images('{}/train'.format(directory))
reference_data = reference_data.add_row_number()
reference_data.export_json('reference.json')

@pusateri - thanks for detailed bug report. I can reproduced the issue.
I recommend we fix it by just making Height and Width top level keys. They should probably also be lowercase too:
[
    {
        "id": 0,
        "path": "./data/104/26979/train/blank.jpg",
         "height": 432,
         "width": 768
    },
    {
        "id": 1,
        "path": "./data/104/26979/train/intro.jpg",
        "height": 432,
        "width": 768
    }
]

That sounds fine. I took a look at the code and it tries to re-use the CSV code to generate JSON too. It seems like there would be less chance of this happening again if it used the standard python JSON module. That's a bigger change and maybe could become a TODO.

@TobyRoseman Each top level key should represent one column of the SFrame; ideally when ingested again, the same JSON should result in the same SFrame. I think the right fix (for JSON export/import of Image type) is to encode the actual contents of the image into the column named "image", perhaps as a base64-encoded data:// URL.

I could see taking two fixes here; one to make the JSON valid (but not necessarily useful) with an image column, and one to make it actually serialize properly and persist across export/import.

The export of JSON should definitely be fixed.


Front Camera

Is there a way to set the default camera to the front-facing one for a phone that has two?

I would need that too :) Did you figure it out?

I ended up writing a custom solution from scratch.
What are you working on?

When I try the demo, it seems to use the front-facing camera - ie the one that I use for selfies - and all I see is my ugly mug. Is there a way to switch it to the back-facing camera, which is more normal for scanning qr-codes?

I see this is explained here:
https://github.com/PolymerElements/app-media/blob/master/explainer.md
To get the back-facing camera, instead of the front-facing one, it's supposed to be:
<app-media-stream
    video-constraints='{"facingMode":"environment"}'
...

I have yet to get this to work (though I've not tried too hard, yet).

It didn't work well when I tried, going to try again...
On Mon, Sep 9, 2019 at 11:07 AM Max Waterman<EMAIL_ADDRESS>wrote:

I see this is explained here:
https://github.com/PolymerElements/app-media/blob/master/explainer.md
To get the back-facing camera, instead of the front-facing one, it's
supposed to be:
<app-media-stream
video-constraints='{"facingMode":"environment"}'
...
I have yet to get this to work (though I've not tried too hard, yet).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/LostInBrittany/granite-qrcode-scanner/issues/12?email_source=notifications&email_token=AAFRLTAPQ326R5S6WA27X63QIYG3RA5CNFSM4GGZKBT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6GZLRQ#issuecomment-529372614,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFRLTFYAUTFBMIL4OSQ2M3QIYG3RANCNFSM4GGZKBTQ
.

Well, at least in my computer, it happily ignores the facingMode because front and rear cameras are seen as different devices.
In my computes, app-devices component detects a list of devices:
0: InputDeviceInfo
deviceId: "8a6154290bc7f4e9b98161546d765b4c5ae005d3dbbdfc17f6ae1c3a95f58ab2"
groupId: "e66ecc99b39ba620ebec27a29b0c6bff7d9e98d3fef7ddb6085eeaae85cd530b"
kind: "videoinput"
label: "Microsoft Camera Front"
__proto__: InputDeviceInfo
1: InputDeviceInfo
deviceId: "dbb66427a250c2f94038617ca80463f9c08fff81219c85aeceb16357d374cd3a"
groupId: "eb6b7491b4bca62b71b59fd84534b8c36aa671bd8a7b8182c46fb9b2b607b73a"
kind: "videoinput"
label: "Microsoft Camera Rear"
__proto__: InputDeviceInfo
2: InputDeviceInfo
deviceId: "17358a27853d4a14a08de118fded502902e64930b27212b68c546406f39c98ed"
groupId: "d46b6ec87897e1e30ba783d80a8f5c75e170d8cee1ab36573297f88cd143f7aa"
kind: "videoinput"
label: "Microsoft IR Camera Front"
__proto__: InputDeviceInfo

By default I choose the first one, and as it is the front camera, the environment  value of facingMode is ignored...
And I can't simply choose the device in function of the intended facing  mode, as the only info I get it's the device name, and it isn't standard across devices...
Any ideas or suggestions are welcome...

I've found a way to make it work on Firefox but oddly enough not on Chrome.
Now there is a userMode attribute. If you set it, you get the frontal (user) camera,
if not the read (environment) one... if you're in Firefox, in Chrome I only get the frontal
one on my Surface.
Can you test it, @davidmaxwaterman ?


Readme for Trie

What does this PR do?
Adds a small readme for the Trie Data Structure, giving a short and crisp explanation as to what a Trie actually does.
However it also resulted in another small change in the README.md of BMI Calculator App because there was a file named README.md there too. The contents of that README.md weren't altered but I suppose some internal hash in the git structure changed as a result of having 2 files with the same name in 2 different directories.
Test Plan
Test Plan : Proof read for typos, add more info if maintainers ask.
Related PRs and Issues
Yeah, here's the related issue : https://github.com/dubesar/Java-A-Z/issues/425
Have you read the Contributing Guidelines?
Yes, I've gone through it !

@mujtaba1747 please add in a detailed README so that the concept of trie can be made easy by reading that,

@dubesar, the markdown isn't proper, it isn't showing properly on the page
I was editing the markdown when you merged my branch to master :(
When, I was done making changes, I made another pull request : https://github.com/dubesar/Java-A-Z/pull/477
Not trying to spam, I just wish this repo to be perfect :)


pull request for e2e test at 02/01/2017 05:45:10

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fix typo in README.md

"Numer" -> "Number"

Thank you for fix it!
This PR is duplicated with #27 so I'll close this PR.


protobuf: add the frame proto and defination

This PR is to fix #24 , we can use this protobuf defination to easily serialize and transfer the frame data.

Closes #24


Default to 0 supply

solved in https://github.com/thirdweb-dev/js/pull/2356/commits/22455503f55bbd6d141f3ca798c8a8051fab4104


tcnative Windows build

The tcnative page says:
"If you are on Windows, you have to:
Build APR by yourself (or get the 64-bit DLL from our CI server),"
But the link to the CI server fails every time, been trying every day for the last 4 days.  It would be very valuable to have the Windows 64 bit binary for those of us without much Windows functionality, but who have customers using Windows.  thanks!

@Gadreel I think we not provide this anymore. You should just build APR yourself like explained here:
http://netty.io/wiki/forked-tomcat-native.html#wiki-h3-6