XREF issue. Using IONIC platform

Hey,
I'm still new to this so sorry if this is repetetive. Anyways, I am trying to genereate a PDF file by using jsPDF.
My goal is to convert a html document (things inside a div tag) to a PDF. I receive a datauri string which I store inside a pdfValue then convert that to Binary and call upon your PDFJS.getDOcument(withBinaryStringValue) function.
I only included two files in my project, pdf.js and pdf.worker.js. I obtained these files by following the instructions.
When i check the logs in logcat, I get the following issues:
D/CordovaLog(12117): : Line 0 : The provided value 'moz-chunked-arraybuffer' is not a valid enum value of type XMLHttpRequestResponseType.
I/chromium(12117): [INFO:CONSOLE(0)] "The provided value 'moz-chunked-arraybuffer' is not a valid enum value of type XMLHttpRequestResponseType.", source:  (0)
D/CordovaLog(12117): file:///android_asset/www/app/toPDF/pdf.worker.js: Line 232 : Error: Invalid XRef stream
I/chromium(12117): [INFO:CONSOLE(232)] "Error: Invalid XRef stream", source: file:///android_asset/www/app/toPDF/pdf.worker.js (232)
D/CordovaLog(12117): file:///android_asset/www/app/toPDF/pdf.worker.js: Line 233 :     at error (file:///android_asset/www/app/toPDF/pdf.worker.js:233:17)
D/CordovaLog(12117):     at Object.XRef_readXRef [as readXRef] (file:///android_asset/www/app/toPDF/pdf.worker.js:4307:15)
D/CordovaLog(12117):     at Object.XRef_parse [as parse] (file:///android_asset/www/app/toPDF/pdf.worker.js:3905:28)
D/CordovaLog(12117):     at Object.PDFDocument_setup [as setup] (file:///android_asset/www/app/toPDF/pdf.worker.js:3062:17)
D/CordovaLog(12117):     at Object.PDFDocument_parse [as parse] (file:///android_asset/www/app/toPDF/pdf.worker.js:2943:12)
D/CordovaLog(12117):     at file:///android_asset/www/app/toPDF/pdf.worker.js:2498:26
D/CordovaLog(12117):     at LocalPdfManager_ensure [as ensure] (file:///android_asset/www/app/toPDF/pdf.worker.js:2493:12)
D/CordovaLog(12117):     at LocalPdfManager.BasePdfManager_ensureDoc [as ensureDoc] (file:///android_asset/www/app/toPDF/pdf.worker.js:2425:19)
D/CordovaLog(12117):     at file:///android_asset/www/app/toPDF/pdf.worker.js:34209:22
I/chromium(12117): [INFO:CONSOLE(233)] "    at error (file:///android_asset/www/app/toPDF/pdf.worker.js:233:17)
I/chromium(12117):     at Object.XRef_readXRef [as readXRef] (file:///android_asset/www/app/toPDF/pdf.worker.js:4307:15)
I/chromium(12117):     at Object.XRef_parse [as parse] (file:///android_asset/www/app/toPDF/pdf.worker.js:3905:28)
I/chromium(12117):     at Object.PDFDocument_setup [as setup] (file:///android_asset/www/app/toPDF/pdf.worker.js:3062:17)
I/chromium(12117):     at Object.PDFDocument_parse [as parse] (file:///android_asset/www/app/toPDF/pdf.worker.js:2943:12)
I/chromium(12117):     at file:///android_asset/www/app/toPDF/pdf.worker.js:2498:26
I/chromium(12117):     at LocalPdfManager_ensure [as ensure] (file:///android_asset/www/app/toPDF/pdf.worker.js:2493:12)
I/chromium(12117):     at LocalPdfManager.BasePdfManager_ensureDoc [as ensureDoc] (file:///android_asset/www/app/toPDF/pdf.worker.js:2425:19)
I/chromium(12117):     at file:///android_asset/www/app/toPDF/pdf.worker.js:34209:22", source: file:///android_asset/www/app/toPDF/pdf.worker.js (233)
D/CordovaLog(12117): file:///android_asset/www/app/toPDF/pdf.worker.js: Line 224 : Warning: Unsupported feature "unknown"
I/chromium(12117): [INFO:CONSOLE(224)] "Warning: Unsupported feature "unknown"", source: file:///android_asset/www/app/toPDF/pdf.worker.js (224)
D/CordovaLog(12117): file:///android_asset/www/app/toPDF/pdf.js: Line 224 : Warning: Unsupported feature "unknown"
I/chromium(12117): [INFO:CONSOLE(224)] "Warning: Unsupported feature "unknown"", source: file:///android_asset/www/app/toPDF/pdf.js (224)
D/CordovaLog(12117): file:///android_asset/www/app/toPDF/pdf.worker.js: Line 224 : Warning: Indexing all PDF objects
I/chromium(12117): [INFO:CONSOLE(224)] "Warning: Indexing all PDF objects", source: file:///android_asset/www/app/toPDF/pdf.worker.js (224)

When reporting an issue, please make sure that you read https://github.com/mozilla/pdf.js/blob/master/CONTRIBUTING.md. At the very least, you need to provide the PDF file for us to debug; either as a link, or uploaded to GitHub.

I'm also a bit confused here. You say that you are using jsPDF (which is a different project than PDF.js), so how exactly does PDF.js come into play here?

PDFJS.getDOcument(withBinaryStringValue)

See also https://github.com/mozilla/pdf.js/blob/master/src/display/api.js#L223, and make sure you are not passing binary string instead of URL. Are you sure the above is not  PDFJS.getDocument({data: withBinaryStringValue})?

@Snuffleupagus do you mean the PDf.js file itself? Or the actual PDF that is generated (because it doesn't generate anything).
@timvandermeij Well i needed jsPDF to get the datauri output by using jspdfObject.output('datauri')
then i used PDFJS.get(jspdfObject.output('datauri')) to display the document.
@yurydelendik Correct me if i am wrong, but isn't datauri a base64 encoded string? Unless that is the same as a binary string.
Thanks for your patience and help guys,

Correct me if i am wrong, but isn't datauri a base64 encoded string? Unless that is the same as a binary string.

If withBinaryStringValue is a datauri with base64 encoded string (the variable name is misleading), then XHR on legacy WebKit and IE browsers don't support reading from the data URI scheme. Feed decoded binary data (typed array, js array or string) to the getDocument using the syntax I proposed in the comment above.

Alright, so I found a method that converts base64 to a binary string. After converting, I pass that array into getDocument. The file generates on the PC but not on android. Also, some methods for PDFJS are showing as "unresolved'. Is that because i am lacking libraries?  I only included pdf.js and pdf.worker.js after building these files. To give you a better idea of whats going on, i'll post the cose here
$scope.toPDF = function(){
                var doc = new jsPDF();
                var source = $('#test').html();
                doc.fromHTML(source, 15, 15, {
                    'width': 170
                });
                var PAGE_TO_VIEW = 1;
                var SCALE = 1.0;
                var pdfValue = doc.output('datauri');
                var pdfAsArray = convertDataURIToBinary(pdfValue);
                window.open(pdfValue);
               // window.open("app/recent_transactions/recent_transactions.html");
        //PDFJS.getDocument('recent_transactions.html');
                PDFJS.getDocument(pdfAsArray).then(function(pdfDocument){
                    return pdfDocument.getPage(PAGE_TO_VIEW).then(function (pdfPage) {
                        // Creating the page view with default parameters.

PDFPageView is unresolved type*****
var pdfPageView = new PDFJS.PDFPageView({
id: PAGE_TO_VIEW,
scale: SCALE,
defaultViewport: pdfPage.getViewport(SCALE)
// We can enable text/annotations layers, if needed
//textLayerFactory: new PDFJS.DefaultTextLayerFactory(),
//annotationsLayerFactory: new PDFJS.DefaultAnnotationsLayerFactory()
});
// Associates the actual page with the view, and drawing it
**setPdfPage is also an unresolved type.
pdfPageView.setPdfPage(pdfPage);
return pdfPageView.draw();
});
});
};
var BASE64_MARKER = ';base64,';
    function convertDataURIToBinary(dataURI) {
        var base64Index = dataURI.indexOf(BASE64_MARKER) + BASE64_MARKER.length;
        var base64 = dataURI.substring(base64Index);
        var raw = window.atob(base64);
        var rawLength = raw.length;
       var array = new Uint8Array(new ArrayBuffer(rawLength));

        for(var i = 0; i < rawLength; i++) {
           array[i] = raw.charCodeAt(i);
        }
        return array;
    };

Just a note
i found the code following getDocument online. It pretty much filled the contents of a blank pdf page that was generated before I added that section of code.

To deal with legacy browsers you need to include web/compatibility.js

I tried adding the compatibility but nothing changes. I ended up displaying the PDF in a canvas on the HTML page but it would be optimal if I can get a separate window to open.

Actually, since it's custom solution, please provide link to the complete code and OS and browser versions. I'll close as incomplete for now. Please provide all information requested at https://github.com/mozilla/pdf.js/blob/master/CONTRIBUTING.md to reopen.


Remove proxy to Brigade API from dashboard command

This responds to Kashti creating a proxy to the API, removing the need for brig to create it - https://github.com/Azure/kashti/pull/187
To test, you need to deploy the Kashti chart according to the instructions in https://github.com/Azure/kashti/pull/187 and brig dashboard - this only starts the proxy to the Kashti pod.
closes #549

Rebased off master

Should we merge this now or wait until the angular5 branch gets merged onto master in Kashti, @flynnduism ?

Following this change, should this be removed from the Kashti docs?

The example above can be used along with a few kubectl port-forward commands to
locally access your Kashti UI. See the Install Guide for more.


TypeDoc does not escape html entities from tag content

Typedoc version: 0.12.0
Node versions: 10.10.0, 8.12.0 LTS
Issue: voodooattack/when-ts#2
Problem: typedoc does not escape html entities from generic types inside a @type clause.
Example from my project:
  /**
   *
   * @type {HistoryManager<S extends MachineState>}
   * @private
   */
  private _history: HistoryManager<S>;

Output:

type

{HistoryManager<S extends MachineState>}

This causes browsers to think there's a malformed strikethrough <S> tag, and it gets applied to the rest of the page. See here for an example.

I spent some time looking into this and determined that it is (mostly) intended behavior, though certainly unintuitive. The issue isn't the _history: HistoryManager<S> line, it is the comment above it.
TypeDoc renders the content of tags as markdown, and markdown supports arbitrary HTML content, so we render {HistoryManager<S extends MachineState>} exactly as we see it, which leads to this issue. Now, arguably we should be stripping types from the comment before rendering it, but this is kind of tricky since for some comments {} has no special meaning, while for others it describes a type.

With #1198's resolution, @type tags are now stripped, which fixes this issue.


First commits

🗣 Description
This pull request provides a Terraform configuration to provide a singular AWS S3 bucket to store AWS Lambda deployment artifacts in CyHy environments.

💭 Motivation and context
As we have added AWS Lambdas to the CyHy configuration they each have gotten their own S3 bucket to store deployment artifacts. This will, in my opinion, only get messier as we add more Lambdas to the environment configuration. I believe that a single bucket to store deployment artifacts for a given CyHy workspace is both more manageable and makes more sense in the long run. This will allow us to remove the existing artifact buckets for the assessment-data-import and findings-data-import Lambadas.

[!NOTE]
Once this PR is approved and merged I will deploy it to production and begin work on dismantling the aforementioned bucket configurations . This deploy should not be problematic because this configuration should have no conflicts with existing infrastructure.

🧪 Testing
Automated tests pass.

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] All new and existing tests pass.

@dav3r and @jsf9k I force-pushed after updating the lambda_artifact_s3_bucket to lambda_artifacts_s3_bucket to better represent that it will store multiple artifacts (which aligns with the bucket name etc.). I managed to forget to pull down from origin before rebasing and had to manually recreate the commit with @dav3r's suggestions. Just wanted a double-check to make sure I didn't botch something else.

Should we also update the name of the lambda_artifact_bucket.tf file?

Please see 2426069.


Missing word?

Beroski: Japanese derived word for a bundle/ wrap. A bundled item in cloth most likely, a square piece of clothing, to secure something like food, precious belongings and easy to carry either on head or carried.
Aikel kall el bliull er a beroski el bekerkard a kelir a rubak. The food wrapped in red are designated for the old men.

Charl confirms. Mamed el obang el omail a kall er obong el kellir a rubak el ngara mechesang.

From: Gnalu @.>
Sent: Sunday, July 24, 2022 2:53:28 AM
To: tekinged/missing @.>
Cc: Subscribed @.***>
Subject: [tekinged/missing] Missing word? (Issue #31)
This message has originated from an External Source. Please use proper judgment and caution when opening attachments, clicking links, or responding to this email.
Beroski: Japanese derived word for a bundle/ wrap. A bundled item in cloth most likely, a square piece of clothing, to secure something like food, precious belongings and easy to carry either on head or carried.
Aikel kall el bliull er a beroski el bekerkard a kelir a rubak. The food wrapped in red are designated for the old men.
—
Reply to this email directly, view it on GitHubhttps://github.com/tekinged/missing/issues/31, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAPT2PX6FEX3ITMKKT2JEHDVVQWRRANCNFSM54OKM3KQ.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
Seagate Internal


Code refactor

Current coverage is 41.60% (diff: 100%)

No coverage report found for master at c41c142.

Powered by Codecov. Last update c41c142...9046a2d


constrain node version

package.json has
  "engines": {
    "node": ">=0.10.0"
  }

This should perhaps have an upper limit since with node 8.6.0
$ gulp build
/Volumes/Sites/vector/node_modules/require-dir/index.js:93
            if (!require.extensions.hasOwnProperty(ext)) {
                                    ^

TypeError: require.extensions.hasOwnProperty is not a function
    at requireDir (/Volumes/Sites/vector/node_modules/require-dir/index.js:93:37)
    at Object.<anonymous> (/Volumes/Sites/vector/node_modules/gulp-git/index.js:4:18)
    at Module._compile (module.js:624:30)
    at Object.Module._extensions..js (module.js:635:10)
    at Module.load (module.js:545:32)
    at tryModuleLoad (module.js:508:12)
    at Function.Module._load (module.js:500:3)
    at Module.require (module.js:568:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/Volumes/Sites/vector/gulp/scripts.js:7:11)

and with node 4.4.2 gulp completes without error

I was following http://vectoross.io/docs/building-vector-source
Another option is to mention the constraint there

Hello everyone,
This is to confirm that indeed there is a problem depending on your node installed version. I am on Ubuntu 16.04 with node v8.8 and I have the following error when I run gulp build:
 if (!require.extensions.hasOwnProperty(ext)) {
                                    ^

TypeError: require.extensions.hasOwnProperty is not a function
    at requireDir (//vector/node_modules/require-dir/index.js:93:37)
    at Object.<anonymous> (/vector/node_modules/gulp-git/index.js:4:18)
    at Module._compile (module.js:612:30)
    at Object.Module._extensions..js (module.js:623:10)
    at Module.load (module.js:531:32)
    at tryModuleLoad (module.js:494:12)
    at Function.Module._load (module.js:486:3)
    at Module.require (module.js:556:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/vector/gulp/scripts.js:7:11)

I tried the installation on dockerized nodes. So I grabbed the node:6.11 docker, and tried the installation inside with node v6.11.4 inside; everything went well as expected. I did the same with node:7.10 docker image with node v7.10.0, it works with a warning when you gulp build
[15:18:08] git describe (log : false)
(node:373) DeprecationWarning: Calling an asynchronous function without callback is deprecated.

I grabbed the next version node:8 on the docker hub, with node v8.7.0 inside and got the same error as before on my machine.
/vector/node_modules/require-dir/index.js:93
            if (!require.extensions.hasOwnProperty(ext)) {
                                    ^

TypeError: require.extensions.hasOwnProperty is not a function
    at requireDir (/vector/node_modules/require-dir/index.js:93:37)
    at Object.<anonymous> (/vector/node_modules/gulp-git/index.js:4:18)
    at Module._compile (module.js:624:30)
    at Object.Module._extensions..js (module.js:635:10)
    at Module.load (module.js:545:32)
    at tryModuleLoad (module.js:508:12)
    at Function.Module._load (module.js:500:3)
    at Module.require (module.js:568:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/vector/gulp/scripts.js:7:11)

The build works again when you comment from 93 to 95 of file node_modules/require-dir/index.js before doing gulp build again. Package require-dir is installed with har-validator when you bower install apparently (I just did a quick grep, not sure which other tool nodejs people use)
I hope this helps.

@sofianinho workaround is to run gulp with an older version of node. 4.4 worked for me

@tom-- thanks for the workaround
I also referenced that in the post with other working versions. Basically it works fine with a simple warning up to version 7.10  of node. Version 8 of node ends up with the error. You can avoid it by commenting the lines I showed after you do a "bower install". The module that seems to cause the error is require-dir.

Issue was solved in master (https://github.com/Netflix/vector/pull/173).


Restic 'list' broken on Windows/amd64

... but 'ls' 'restore' 'check' 'backup' still works.
Output of restic version
restic 0.8.3
compiled with go1.10 on windows/amd64
How did you run restic exactly?
C:\Users\user\Desktop\restic>restic init -r c:\restictest
enter password for new repository:
enter password again:
created restic repository f0d1dfdeca at c:\restictest

Please note that knowledge of your password is required to access
the repository. Losing your password means that your data is
irrecoverably lost.

C:\Users\user\Desktop\restic>restic list -r c:\restictest
Fatal: type not specified

C:\Users\user\Desktop\restic>restic list -r c:\restictest\
Fatal: type not specified

C:\Users\user\Desktop\restic>restic backup -r c:\restictest c:\bin
enter password for repository:
password is correct
scan [c:\bin]
scanned 596 directories, 14100 files in 0:01
[0:28] 100.00%  1.783 GiB / 1.783 GiB  14696 / 14696 items  0 errors  ETA 0:00
duration: 0:28
snapshot 262da89f saved

C:\Users\user\Desktop\restic>restic list -r c:\restictest\
Fatal: type not specified

C:\Users\user\Desktop\restic>restic list -r c:\restictest
Fatal: type not specified

C:\Users\user\Desktop\restic>restic check -r c:\restictest
enter password for repository:
password is correct
create exclusive lock for repository
load indexes
check all packs
check snapshots, trees and blobs
no errors were found

C:\Users\username\Desktop\restic>restic ls -r c:\restictest 262da89f
enter password for repository:
password is correct
snapshot 262da89f of [c:\bin] at 2018-05-17 14:49:35.4865423 +0200 CEST):
\bin
[listing of files as intended]

C:\Users\user\Desktop\restic>restic version
restic 0.8.3
compiled with go1.10 on windows/amd64

C:\Users\user\Desktop\restic>

C:\Users\username\Desktop\restic>restic restore -t extracttest -r c:\restictest 262da89f
enter password for repository:
password is correct
restoring <Snapshot 262da89f of [c:\bin] at 2018-05-17 14:49:35.4865423 +0200 CEST by FLEX\username@hostname> to extracttest

[files restored correctly]

What backend/server/service did you use to store the repository?
Local NTFS file system
Expected behavior
Ask for passphrase + List the snapshots in the repository
Actual behavior
Exits with the message: '''
Fatal: type not specified
'''
Does not ask for passphrase.
Steps to reproduce the behavior
See above in 'how do did you run restic exactly'?
Do you have any idea what may have caused this?
Do you have an idea how to solve the issue?
Did restic help you or made you happy in any way?
Restic still makes me smile because of the possibilities that lies in it; because of the diverse backends and because it has a native win64 port. You guys are awesome - keep up the good work! Thanks!

Oh, this is not a bug, but a user interface issue: the list command is a low-level one to list objects in the repo:
$ restic list --help

The "list" command allows listing objects in the repository based on type.

Usage:
  restic list [blobs|packs|index|snapshots|keys|locks] [flags]

So in order to list snapshots in the repo you could use restic list snapshots - but that's not very user friendly.
The command you're looking for is the restic snapshots command.

My bad; it comes from using borg ; 'borg list' is equivalent of 'restic snapshots'.
Sorry. Soooorrryyyyy.
:-)

Don't worry about it, I like to see the positive thing here: A confusing error message got improved :)
Thanks for taking the time to report this!


less unsafe in ref_ring

via https://github.com/rust-random/rand/pull/960/commits/de9dfdd86851032d942eb583d8d438e06085867b#diff-fd713d05cafb058540808aeb61e4fb7ac7817c7e1fde91e422153ec5fc0b61dfR227-R232


Workspace storage compatibility for HDFS and LFS

Signed-off-by: Al-assad<EMAIL_ADDRESS>see issue #251

This pr was largely complete, note that most of the changes are unverified and rely on other tasks in issue #246 for overall verification. Next we'll focus on the other subtasks in #246

@wolfboys PTAL


should render ReactNode in connectedCallback not constructor?

in constructor create and render ReactNode, let we can't setProps before render ReactNode.
because  HTML attr is not support Camel-Case。
so create and render  ReactNode in connectedCallback better

Hi @WeiRu1016. Thanks for using this lib and your contribution to this project. Would you be able to create an example with described issue in https://codesandbox.io/ or similar?

sorry, This is demo: https://codesandbox.io/s/react-web-component-ugmyj9?file=/src/index.tsx
because textValue is camelCase props, html5 attributes is not support camelCase attributes.
so use setProps.
but before setProps, react component has created and mounted. props textValue is required props, but now is undefined.
if ExampleComplete used textValue to acc, will cause render error!!!


Sema: Suppress unavoidable deprecation diagnostics in _Concurrency

The _Concurrency module must use some deprecated declarations in its implementation in order to maintain backward compatibility. Since these references are unavoidable, the deprecation diagnostics emitted about them are an unhelpful nuisance. This change introduces targetted logic for suppressing these specific deprecation diagnostics as a workaround until there is a language feature for suppressing deprecation diagnostics.
Also, remove a couple of superfluous trys.

@swift-ci please test


Some useful (English language) macros for testing

These days I just do:
\begin[papersize=a3,class=fontproof]{document}
\setTestFont[filename=Coolangatta-SemiBold.otf,size=25pt]

\proof:arrowroot

\proof:bringhurst

\proof:allkernpairs\proof:jafkerns

\end{document}

getting this error with current sile master and both @jvgaultney's examples and the one above:
/usr/local/share/sile/packages/bidi.lua:155: Bidi run too big? U_BUFFER_OVERFLOW_ERROR

Right you are, I need to also turn \bidi-off. This is because the jafkerns paragraph is extremely long. Ideally we should make that buffer dynamically assigned.

OK, SILE now dynamically assigns that buffer...

works for me, thanks


Shift+Drag selection is broken

Shift+Drag selection does only select nodes, but not lines and areas.

I'm pretty sure it is that way by design, not broken.
I've always assumed the usecase for this feature is realigning a badly drawn segment:

Yes, it's this way by design.

I have changed the description in the shortcut list in the OSM wiki, which was wrong then.
Another usecase for block selection would be copying a complex building, maybe including the relations for 3d-modeling. This doesn't work with the current design.
What if it were selecting all fully enclosed objects? I believe it would still work for your usecase. Either a way wouldn't be selected, because any node is outside of the box, or the way would be moved in total anyway, because all nodes of the way are selected and moved.


Crash when loading from launch 4/19

Duplicates https://github.com/Tyler-IN/MnB2-Bannerlord-CommunityPatch/issues/169

Same issue here.


feat: add gocryptfs

Add rfjakob/gocryptfs

any reason not to use aqua:rfjakob/gocryptfs?

Oh nevermind. I thought I had to add it to the registry, but it's already there


What tool is Imgbot using to achieve its results?

Is it optipng, imagemagick or sth else?

It is Magick.NET (https://github.com/dlemstra/Magick.NET) that has it's own code and some ImageMagick code to optimize the files.

I have no experience with ImageMagick or your .NET library but I have some with Oxipng and it seems to always be better than Imgbot. I can tell you more about this tool and link some PRs - Imgbot ones and mine with Oxipng for the same files so you can compare.
If you are not interested / don't see a way to make use of this tool in Imgbot then just close this issue.

For PNG compression it might be a good idea to search for an alternative. Feel free to research this.

I did some tests and then forgot about this. A few days ago I discovered another tool - https://github.com/JayXon/Leanify and it seems to outperform anything else.
Running more tests and will post results later.
The problem is - leanify optimizes many formats but there is no option to exclude some of them so if it sees JSON or xml files it makes them hard to read - minified.

Unless someone PRs a "fix" to Leanify the only way to use it for image-only optimization is to make a copy of the repo first, run leanify on the copy, and then move all image files to the original directory. This can be easily done with a bash script.

The ImageOptim app has a wide range of tools to investigate possibly using: https://imageoptim.com/mac
Specifically, it uses all of:

oxipng
pngcrush
mozjpeg
zopfli
advpng
jpegoptim
gifsicle
svgo
svgcleaner
pngout


Option to define parameters in projects

Is your feature request related to a problem? Please describe.
I have a mani.yaml with a larger number of repos..as intended. Given a simple brief example:
projects:
    p1:
      path: x
      url: https://.../repos/x.git
    p2:
      path: y
      url: https://.../repos/y.git
tasks:
  upd:
    desc: Update all given projects.
    output: text
    commands:
      - name: switch to master
        cmd: git switch master
      - name: pull update
        cmd: git pull --all -p

So in general the usage is fine but I have one issue. I have a lot of git repos (200+) but not all using the same naming for the default branch. Some use main some use master and others use dev... so the definition of a task like the above is
not usable for all cases.
Describe the solution you'd like
It would be great having a way to define a kind of a parameter for the task something like (just an idea):
projects:
    p1:
      path: x
      url: https://.../repos/x.git
      parameters:
        - branch: main
    p2:
      path: y
      url: https://.../repos/y.git
      parameters:
        - branch: master
tasks:
  upd:
    desc: Update all given projects.
    output: text
    commands:
      - name: switch to ${branch}
        cmd: git switch ${branch}
      - name: pull update
        cmd: git pull --all -p

Maybe there should be used a different syntax than using ${..} to separate them from the environment variables given for all tasks via env:.....

That makes sense, you'd want some env variables that are specific for each project. I think an easy fix is just to introduce the same env property for projects as well. env supports both simple strings as well as shell expansion.

To be honest using the term env reminds me a lot about environment variables which here is not the case ...but of course that would work as you described that would be great...

yeah, the env naming is perhaps not the best, but I chose it because it gets loaded as an environment variable when you run commands. Shell expansion is also there for flexibility purposes. Anyway, please try the latest release 0.11.1, and see if the current solution works for you.

Close as the new release supports this.


Make Hnsw99Codecs Extensible

Description
Lucene99HnswVectorsFormat and Lucene99HnswScalarQuantizedVectorsFormat are not extensible. With the new changes to how max vector dimension is set in the codec, users should be able to extend these classes to override the getMaxDimensions method. The current alternative is to make a delegating class that extends from KnnVectorsFormat which isn't the nicest UX.

Lucene prefers the delegation pattern. To prevent uncontrolled subclassing most of the components in Lucene are final.


Remove acronym in SUDs dataset description

Darn, one last comment-- the dataset subtitle says "Substance Abuse Disorders". We should be consistent and use the term "substance use disorders" even though the facilities themselves are listed in SAMHSA as specializing in treatment of Substance Abuse.

Sounds good. Does Substance Abuse Treatment Services still work as the dataset title?

Yes according to SAMHSA I think Substance Abuse Treatment Services works still: https://www.samhsa.gov/data/substance-abuse-facilities-data-nssats


[SPARK-45273][CORE][UI] Support for set the access host in http header

What changes were proposed in this pull request?
The PR supports to set the access host in http header.
Why are the changes needed?
It can avoid http head attack.
Does this PR introduce any user-facing change?
No
How was this patch tested?
add new test
Was this patch authored or co-authored using generative AI tooling?
No

Please give me a review when you have time.@srowen
Thank you so much.

I don't see any explanation of what this does or why it is useful

@chenyu-opensource Please add more detail in description.

I don't see any explanation of what this does or why it is useful

I had submit a new commit for description.
This pr is  to solve the problem of request header attacks. We should add filtering logic to set access whitelist for request headers.

There is still no explanation of what problem this solves

There is still no explanation of what problem this solves

HTTP Host header attack is a network security vulnerability that utilizes the "Host" header in HTTP requests. The Host header in the HTTP protocol is used to identify the target server of the HTTP request, indicating which host name or IP address the client wants to access the web application.
If the server directly trusts the Host header without verifying its legitimacy, an attacker may use this controllable variable to inject the Host and manipulate the server's behavior.
This pr used a whitelist method to solve this problem.

What 'trusts' a host header in Spark? what happens if it's spoofed?
This is still not explaining the issue.

What 'trusts' a host header in Spark? what happens if it's spoofed? This is still not explaining the issue.

What I mean is that there are no restrictions on request headers in Spark itself, which means that the client can freely pass through the server, so attackers can use this variable parameter to attack.
This pr helps us that when setting the parameter 'host' in HTTP header request, we can verify this parameter instead of directly letting it pass.

Attack what?

Attack what?

It may cause the transmission of malicious code and It may cause cache pollution and password reset.
I have already used this method to solve this problem in the production environment, and after vulnerability scanning, the problem did not occur.

How does this arise? Use<EMAIL_ADDRESS>if needed. I am not clear what attack you have in mind or whether it can affect spark, so, no this would not be useful unless there's an argument it solves something


Scripts no longer pull in transitive dependencies

The following script should work without the explicit commons-lang dependency:
@Grapes([ @Grab('io.github.http-builder-ng:http-builder-ng-core:0.13.2'),
          @Grab('org.apache.commons:commons-lang3:3.5'),
          @Grab('org.jsoup:jsoup:1.9.2') ])
import groovyx.net.http.HttpBuilder
import java.util.concurrent.Executors
import java.util.concurrent.ExecutorService

final int MAX_THREADS = 10
final ExecutorService myPool = Executors.newFixedThreadPool(MAX_THREADS)

def client = HttpBuilder.configure {
    execution.maxThreads = MAX_THREADS;
    execution.executor = myPool;

    request.uri = 'http://www.cnn.com/'
}

def result = client.get()

println(result.select('title').text())

This is most likely due to either the removal of the println statements in build.gradle (not likely) or the explicit additions to the generated pom file during maven central publishing.
Also, this may simply be due to the addition of a method use from that dependency - which would cause it to fail if missing. (most likely)

This issue was really about explicitly using the library in our code. This has been fixed in the development branch, pending 0.14.0 release.


Chore: Improve script to create and update docker images

I'm moving this code commit from the branch chrome-headless-rebase, because next update of the scanner will be deployed probably on Kubernetes, and these improvements help us to deploy very easily.

@molant done!

👍 merge whenever you are ready.

From: Jesus David García Gomez<EMAIL_ADDRESS>Sent: Monday, October 29, 2018 10:02
To: webhintio/online-service
Cc: Antón Molleda; Mention
Subject: Re: [webhintio/online-service] Chore: Improve script to create and update docker images (#170)
@molanthttps://github.com/molant done!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/webhintio/online-service/pull/170#issuecomment-433991028, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAlBgoTJZeFijO8UEtkc6cTIce1NRH4Oks5upzR_gaJpZM4X7Mm0.


ENH: Add ImageRegionRange to itk::Experimental namespace

Added ImageRegionRange<TImage> to the itk::Experimental namespace,
supporting modern C++ iteration (bidirectional) over a range of pixels
of an image region.
Replaced the classic ImageRegionConstIterator<ImageType> by
ImageRegionRange<const ImageType>, when doing bounding box computation
in ImageMaskSpatialObject. A performance improvement of 5 % or more
was observed.

While the existing ImageBufferRange iterates over the entire image buffer, this ImageRegionRange<TImage> would iterate over the image region, specified by the user. (By default, it would do the requested region)
Note: Still under construction!

When the region of interest has only a few pixels per row, the proposed ImageRegionRange appears much faster than the classic itk::ImageRegionConstIterator. For example, with region size [2, 65535], I observed that the proposed range appears more than 10 times (!) as fast as the classic ITK iterator.
When  the region of interest has a large number pixels per row,  the proposed range and the classic ITK iterator appear to have a similar performance.

@N-Dekker I am really liking these new C++11 interfaces.
Just an FYI, I wrote the ImageScanlineIterator. This iteration patter had two loops, the inner loop which had a trivial check in "IsAtEndOfLine" which was just a comparison. While the "NextLine" operations did the work of moving the iterator through the N dimensions. There was quite a bit of speed up because the interloop could be easily vectorized by the compiler. This is generally my go to iterator for performance.
A filter that may significantly benefit from better iterator is the BilateralImageFilter, I believe 10x practical speed up in the execution is possible.

@blowekamp

Just an FYI, I wrote the ImageScanlineIterator. This iteration patter had two loops, the inner loop which had a trivial check in "IsAtEndOfLine" which was just a comparison. While the "NextLine" operations did the work of moving the iterator through the N dimensions. There was quite a bit of speed up because the interloop could be easily vectorized by the compiler. This is generally my go to iterator for performance.

Thanks Bradley! Very interesting! Actually I already noticed that the old itk::ImageRegionConstIterator is optimized for iteration within one and the same row (or 'span'): https://github.com/InsightSoftwareConsortium/ITK/blob/v5.0.0/Modules/Core/Common/include/itkImageRegionConstIterator.h#L233
Such an optimization appears very beneficial in general, for many use cases. However, when the number of pixels per row is small, it can yield a performance penalty instead! I think that's why the Experimental::ImageRegionRange that I'm proposing here is much faster than old itk::ImageRegionConstIterator, for a region that has only a few pixels per row.

@N-Dekker Your link is to an "if" statement. Traditionally compiler auto-vectorization has difficulties with if statements when trying to vectorize the code because it's trying to do multiple loop passes is a vectorized fashion. That is why the ImageScanlineIterator removes the if statement in the operator++, and  IsAtEndOfLine, is just a comparison.
One suggestion I believe Hans made, was to extend the "optimized" inner loop to be the maximum continuous region, which could be multiple slices. Food for your thoughts.

@blowekamp Just did a preliminary performance comparison between ImageScanlineConstIterator, ImageRegionConstIterator, and ImageRegionRange, on VS2017 Release. For an image region with a small number of pixels per row,  ImageRegionRange still appears fastest. But for an image region with a large number of rows, your  ImageScanlineConstIterator is the winner  :-)
Some (preliminary) results:
Region size [2, 1048575]:
ImageRegionConstIterator.....  0.0603093 seconds
ImageRegionRange.............  0.00438488 seconds
ImageScanlineConstIterator...  0.0342728 seconds
 -- Slowest: ImageRegionConstIterator.....
 -- Fastest: ImageRegionRange.............

Region size [65535, 65535]:
ImageRegionConstIterator.....  3.51961 seconds
ImageRegionRange.............  3.39576 seconds
ImageScanlineConstIterator...  2.19651 seconds
 -- Slowest: ImageRegionConstIterator.....
 -- Fastest: ImageScanlineConstIterator...

@N-Dekker awesome!!

For new use cases (user code), I think the choice should be either Experimental::ImageRegionRange or itk::ImageScanlineIterator. Either of these two always seems a better choice than the old itk::ImageRegionIterator. At least, that's what I think...!


IDVA6-427 fix all page titles in frontend

Link to Jira:
https://companieshouse.atlassian.net/browse/IDVA6-427
Changes:

Title set for pages specified in the ticket

CI: Security warnings found!

CI: Security warnings found!


Update all projects to use the new Maven plugin (original repo updates)

The updates on this CL are from the commits to the original repository (https://github.com/googleinterns/step/commits/master) after May 18, which is the date that I cloned the original repository. I have to include these changes in order to deploy my portfolio to cell-step-2020.appspot.com.

Note: There are 82 files changed in this CL, which is obviously a very large amount of changes. I simply copied the most recent version of files or versions from the current repo (https://github.com/googleinterns/step) and verified that my project still works as expected.

Sorry, accidentally deleted branch above (you can ignore that).


Fix download feature

Short description of what this resolves:
Fix for download feature bug
Includes changes from #1257, but I'm not ready to separate it right now.
Changes proposed in this pull request:
Fixes: #1258
How Has This Been Tested?

Screenshots (if appropriate):
Checklist:

[x] I have read the contribution guidelines.
[ ] My change requires a change to the documentation and GitHub Wiki.
[ ] I have updated the documentation and Wiki accordingly.

Thanks
I will review it once  #1257 is merged.
Also can you look into the webview settings. Current UI and UX isnt working as expected. We need to fix that too in this version.

Rebased on top of last changes so it ready for review.

Merged, Thanks


"Syntax Error"  when using ES6 npm library in angular 7 app

I get the above console error when using the library from your npm package. I only get this error in IE11

@cbuchert can you not provide support for IE11 in your library?

@sir-captainmorgan21 - Well, we certainly could, but we've chosen not to. As @cbuchert mentioned above, using babel is the recommended workaround.


Gallery: Color contrast too low

An a11y review found the following contract issues:

[ ] Contrast of text in drawer should possibly be increased from 2.99 to 4.50 for smaller text or 3.00 for larger text
[ ] Contrast of Hamburger menu icon in main Flutter Gallery home page should be increased from 1.11 to 3.00 or higher
[ ] Upper left hand arrow back button contrast  in Contacts Profile should be increased from 1.94 to 3.0 or higher (issue also found in other sections of app as well)

I've written https://github.com/flutter/flutter/pull/48752 to address the first and third issues. The second point is no longer an issue I believe, as the hamburger menu icon is currently passing contrast ratio guidelines.


Command scheduling, event handling improvements

This is the second part of my PR (#631), which was previously reverted to remove changes unrelated to PC port.
This PR adds the following features:

Command scheduling - schedule command, which allows to run another command at a set interval. Any supported command can be executed. It has the following syntax:

schedule set <id> <period(ms) <cmd...> - add/update a scheduled task

id is a user-chosen integer identifier of the task
period is the interval in milliseconds
cmd is the command to run; it can consist of multiple space-separated words

schedule del <id> - remove a previously scheduled task
schedule start <id> - start (enable) a previously scheduled task (enabled by default)
schedule stop <id> - stop (disable) a previously scheduled task, without removing it

Event variable substitution (I couldn't find a better name) - replace %var% tags in commands of objects' "action" field with a value from the event JSON. For example, this jsonl:

{"page":2,"id":105,"obj":"label","text":"Brightness"}
{"page":2,"id":106,"obj":"slider","action":{"changed":"p2b105.text %val%"}}

will update the label's text whenever the slider emits a changed event. Any supported command can be executed, and any value from the JSON event can be used in substitution.
NOTE: That feature is disabled by default on non-PC builds, not to impact performance/memory usage on ESP32/Arduino builds. In particular, it requires deserialization of the JSON event prior to publishing, as well as running std::string::replace() a few times. It can be enabled using HASP_USE_EVENT_DATA_SUBST=1.

Setting "action" on all types of controls - previously, only a few controls allowed to run commands on events (script_event_handler()). Now, all controls call event_send_object_data() only, which then calls script_event_handler().

If there is an "action" tag on the object, the command is executed and MQTT message is not published.
An example can be seen above (with the slider).

Publishing MQTT messages along with event commands - if the "action" tag of an object has an additional property "pub": true, the action command is ran AND a message is published to MQTT.

This property defaults to false, which matches the behavior from previous versions.
Example:
{"page":2,"id":106,"obj":"slider","action":{"changed":"p2b105.text %val%","pub":true}}

This slider's events will be published to MQTT, additionally the changed event will run a command.

I have one more "feature" that was left out of #631, that is setting LVGL max framerate. If you want, I can add it to this PR, otherwise I will probably create another one.

@kuba2k2 it would be best if each of these point (1..4) could be separate PR's :)

These features are directly related to each other. I think it doesn't really make sense to split it, after all this PR is only 3 commits. You can review these commits separately and it will (more or less) match these 4 points.

Hi, any news about this? :slightly_smiling_face: Anything that needs to be fixed before merging?

Yes, the substitution may not apply to all commands. It should apply to everything that is executed as a text command. Since it is only useful in event actions (because otherwise data will be empty anyway, e.g. on MQTT command) it should apply to all commands executed this way.
I do not have an ESP32 platform with a compatible LCD, that's why I can't test it. I can only say that I was very careful to keep the existing behavior of the code, not to impact other users.


Nomad-exporter transiently fails due to allocation metric collission

We realised that nomad-exporter failed with some level of frequency, specially after starting to use nodes that allowed higher levels of "container density"; it started returning 500 upon scrape. The response contained several lines, flooded with the same thing:
* collected metric nomad_allocation_cpu_system_mode label:<name:"alloc" value:"oosomjob.oosomejob[0]" > label:<name:"datacenter" value:"oosomdc" > label:<name:"group" value:"oosomjob" > label:<name:"job" value:"oosomjob" > label:<name:"node" value:"oosomnode" > label:<name:"region" value:"global" > gauge:<value:380.915828904299 >  was collected before with the same name and label values

This happened because a canary allocation, labeled as above, was scheduled in the same node as the canary of the previous deployment of the job.
Note that canaries will always be labelled to have alloc = XXXX.YYYY[0], so a collision happens sometimes.
The same thing will happen if the 2nd,3rd,4th.. allocations of the job are scheduled in the same node as the previous allocation of the deployment.
After removing the allocation(s) introducing this collision on nomad-exporter, the exporter stops responding 500.
I believe we should add an extra label with alloc_id in order to introduce a unique identifier to the metrics. Given that we are already filtering out metrics of stale allocations, this shouldn't introduce a cardinality bomb.
WDYT @pcarranza ?

Nah, we need a way of identifying the canary instead. That way we will not be introducing an ID which can derive into a bomb.

Note that this not only affects canaries; the alloc label has XXXX.YYYY[N] possible values, where N:[0, $job_redundancy). If any of these ordered elements happens to be scheduled in the same  node as in the case of the previous deployment (i.e. not a canary, but another alloc in the middle of an ongoing deployment).

Gotcha, then it's not about canaries, but about the allocation being in the same place with the same name and index in the list.
Then the difference will be the version, or the deployment id. Would that make sense?

@dgonzalezruiz I've something for you to test.
I moved the development to gitlab to have some actual CI, you can now find it here: https://gitlab.com/yakshaving.art/nomad-exporter
This adds a job_version to allocations and deployments, removes the deployment_id to reduce cardinality (not a lot, but meh, it does make sense) and failed from the task count because it was basically a useless metric.
Though, if you were running this as a binary I will recommend you to run it as a container within the nomad cluster, this way you can control memory usage (to prevent it from burning all the available memory) - you can find a nomad-exporter.nomad hcl file which you can use as a baseline.
I've been testing it by installing multiple versions of the same thing (the nomad exporter itself) in a single node, and I could not see any collision.


Generating images takes 15 minutes

On AWS machine (g2.2xlarge), I am trying to generate an image based on the model.
The command I execute is:
Testing - Takes around 15 minutes
python generate.py sample_images/tubingen.jpg -m models/composition.model -o sample_images/output.jpg

I haven't used MS-COCO anywhere as I am using the existing model.
From what I read in README, this should take ~1 second, however it takes 15 minutes.
Here are the steps that I followed (nothing new, just followed the readme):

Git Clone
Setup model "sh setup_model.sh"
Generate image.

On a Titan X it takes 1 second. This is a far superior card to what Amazon has available. Their cards are 2 generations old and have 1/3rd of the graphics memory. The GPU instances on AWS are their most limited, but I've been talking to them and they're working on it.

try add -g 0?

Adding -g to the generate.py worked. Thanks @markz-nyc
However, it gave 'cudaErrorMemoryAllocation: out of memory'. With a smaller image it worked fine as mentioned in #3


DuckDB file lock when Execute dbt SQL Feature

Expected behavior
To have the option to close the connection after executing the query and to remove the lock on the DuckDB file.
Actual behavior
I'm currently working on a DBT project that connects to a DuckDB database file. As you may know, the .duckdb file extension can only be opened by a single process at a time, unless in ReadOnly mode.
When I execute an SQL query in a model using the "Execute dbt SQL (Ctrl+Enter)" button, it opens the "QUERY RESULT" tab and displays the results, which is perfect! However, the connection to the database remains active and the .duckdb file is locked.
For now, I've been using the lsof command to identify the process that is using the file, and then killing it. After that, the "Execute dbt SQL" function no longer works, and I have to reload the workspace to use it again.
Steps To Reproduce

Set Dbt Profile with a Duckdb connection :

dbt_profile:
  target: dev
  outputs:
    dev:
      type: duckdb
      path: "/path-to-duckdb-file/my-duckdb-database-file.duckdb') }}"

Then Open an sql file to execute with dbt

Then "Execute dbt SQL (Ctrl+Enter)"

See Result Query :

After that there is not option to close the connection, so the duckdb file is locked.

Log output/Screenshots
No response
Operating System
Debian GNU/Linux 11 (bullseye)
dbt version
core: 1.5.0
dbt Adapter
duckdb: 1.5.0
dbt Power User version
v0.13.0
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

@xrafhue I believe this issue was resolved in 1.5.1: https://github.com/jwills/dbt-duckdb/issues/162


Experimental Component - Loading - Design

Design
Create final design specifications that are ready for production. Functionality and variations should match as is Carbon components.

[ ] confirm Security design matches duo intent
[ ] confirm Sketch file sizing/spacing specs
[ ] screen grab sizing guidance and add to GitHub issue

Loading design
Sketch file: https://ibm.box.com/s/57ncx15w6t8o290jf04i33gtw3vx9sll


POST Request magically turns to GET request giving me "Method Not Allowed"

I have a PHP file example.php and inside this file l have entered the code below
$client->request('POST', $postURL, ['headers' => ['content-type' => 'application/json', 'Authorization'=>'Token t6wdjjd7787sjjhd87954frr']],  ['json' => $post_data] );
when l visit my web browser to test the request  localhost:8000/example.php
I get the error 405 GET not allowed, meanwhile the request am making is a POST request
What am l doing wrongly here?

 You can use Postman or Paw.

@Konafets Thanks for your quick response
With PostMan it works perfectly. I am writing a cronjob to call this example.php file every 1 hour. So l want the GuzzleHttp request to work in this case for me.
Do you mean, l cannot call the example.php with the browser, as it will turn the request to GET? How do l test example.php file?
Thank you

By looking at the Access log at the API's end, it receives GET instead of POST. I think $client->post must call the API using the POST, why is the browser taking control of GuzzleHttp POST Request?
at=info method=GET path="/app/study/create/" host=www.example.com request_id=731be894-76ce-4561-b060-228b54cc20ca

Unless you have <form method="POST">, every request of the browser is done via GET. But a form is in your case not applicable anyway, since a cron job is never executed by the browser but via CLI. So, you need to write a Command Line Script which gets called by the cron job. You can execute the CLI script also manually.
If you are using a Framework like Laravel or Symfony the best way to this is to write a command.

I think there is a misunderstanding here: in your case there are actually two requests: one Madrid by Guzzle, which is a POST request, the other is the request to your webserver. I assume the 405 response was not a Guzzle error and was returned by your webserver. The Guzzle method is not being overwritten by your webserver.
As @Konafets pointed, you'd probably better run this script from CLI. Even for testing purposes you can use cURL from CLI (or Postman), but your browser will always send GET to your webserver.

Thank you guys for the explanations. It does make sense to me now.

It could be that a redirect is happening in your code, this is where you can lose the POST request method, that turns into a GET methods in the end.


adding crwdasarja as developer to crowdstrike-security plugin

Link to GitHub repository

https://github.com/jenkinsci/crowdstrike-security-plugin
Adding @asarja-crwd as a developer/contributor to crowdstrike security plugin.
When modifying release permission
List the GitHub usernames of the users who should have commit permissions below:

@crwd-etatarevic
@crwdharshitac

This is needed in order to cut releases of the plugin or component.
If you are modifying the release permission of your plugin or component, fill out the following checklist:

### Release permission checklist (for submitters)
- [x] The usernames of the users added to the "developers" section in the .yml file are the same the users use to log in to [accounts.jenkins.io](https://accounts.jenkins.io/).
- [x] All users added have logged in to [Artifactory](https://repo.jenkins-ci.org/) and [Jira](https://issues.jenkins.io/) once.
- [x] I have mentioned an [existing team member](https://github.com/orgs/jenkinsci/teams) of the plugin or component team to approve this request.

Removal and addition is approved @jenkins-infra-bot


fix readme. information about toggle function

Just a quick fix about the readme.
The toggle function was not working.
I fixed it. I let you take a look at.
Thanks

Coverage remained the same at 98.551% when pulling 42453dd311f18829b79837c3fc6f25fc8fc44d94 on nruiz64:feat/FixReadmeToggleInformation into 9069701e28c486b0b8ff880904e50cfdca7655f2 on alexcurtis:master.

Actually toggled is working. I close this PR.


Extract "_dropAreaMargin" and "elevation" from ReorderableListView

Use case
When you use a ReorderableListView whenever you make a longPress on a cell it is elevated by a certain value elevation and a bottom padding of a certain value _dropAreaMargin is added.
Proposal
It would be useful to be able to extract these parameters to make them customizable by the user by adding them to the ReorderableListView class constructor.

CC @darrenaustin

With the recent refactoring of the ReorderableListView (PRs: #74299 and #74697), we have added the ability to override the the default decoration for the item being dragged with a proxyDecorator property. It can be used to add your own custom look to the dragged item:
  Widget proxyDecorator(Widget child, int index, Animation<double> animation) {
    return AnimatedBuilder(
      animation: animation,
      builder: (BuildContext context, Widget? child) {
        final double animValue = Curves.easeInOut.transform(animation.value);
        final double elevation = lerpDouble(0, 6, animValue)!;
        return Material(
          child: child,
          elevation: elevation,
        );
      },
      child: child,
    );
  }

...

  ReorderableListView(
     ...
     proxyDecorator: proxyDecorator,
     ...
  )

Which mimics the default look from the ReorderableListView. You can wrap the item in whatever look you desire.


Running import in my project I got error ...

After installing AndreasElia/laravel-api-to-postman I tried to import my project, but got error :

master@master-laptop:/mnt/_work_sdb8/wwwroot/lar/PublishBuyDocs$ php artisan export:postman

   ReflectionException

  Class "O:47:"Laravel\SerializableClosure\SerializableClosure":1:{s:12:"serializable";O:46:"Laravel\SerializableClosure\Serializers\Signed":2:{s:12:"serializable";s:303:"O:46:"Laravel\SerializableClosure\Serializers\Native":5:{s:3:"use";a:0:{}s:8:"function";s:85:"function (\Illuminate\Http\Request $request) {
        return $request->user();
    }";s:5:"scope";s:37:"Illuminate\Routing\RouteFileRegistrar";s:4:"this";N;s:4:"self";s:32:"00000000000004370000000000000000";}";s:4:"hash";s:44:"Np6TXB8CPOs/P0pRtmBIG8RKahtC8oq1L+OoYoVPPmE=";}}" does not exist

  at vendor/andreaselia/laravel-api-to-postman/src/Commands/ExportPostmanCommand.php:195
    191▕             return new ReflectionFunction($routeAction['uses']);
    192▕         }
    193▕
    194▕         $routeData = explode('@', $routeAction['uses']);
  ➜ 195▕         $reflection = new ReflectionClass($routeData[0]);
    196▕
    197▕         if (! $reflection->hasMethod($routeData[1])) {
    198▕             return null;
    199▕         }

      +15 vendor frames
  16  artisan:37
      Illuminate\Foundation\Console\Kernel::handle()
master@master-laptop:/mnt/_work_sdb8/wwwroot/lar/PublishBuyDocs$ php -v
PHP 8.1.2 (cli) (built: Jan 24 2022 10:42:33) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.2, Copyright (c) Zend Technologies
    with Zend OPcache v8.1.2, Copyright (c), by Zend Technologies
master@master-laptop:/mnt/_work_sdb8/wwwroot/lar/PublishBuyDocs$ php artisan --version
Laravel Framework 9.4.1

In composer.json :
{
"name": "laravel/laravel",
"type": "project",
"description": "The Laravel Framework.",
"keywords": ["framework", "laravel"],
"license": "MIT",
"require": {
"php": "^8.0.2",
"andreaselia/laravel-api-to-postman": "^1.11",
"bensampo/laravel-enum": "^5.2",
"cviebrock/eloquent-sluggable": "^9.0",
"guzzlehttp/guzzle": "^7.2",
"laravel/framework": "^9.2",
"laravel/passport": "^10.3",
"laravel/sanctum": "^2.14.1",
"laravel/telescope": "^4.8",
"laravel/tinker": "^2.7",
"orangehill/iseed": "^3.0",
"spatie/laravel-permission": "^5.5"
},
"require-dev": {
"fakerphp/faker": "^1.9.1",
"laravel/sail": "^1.0.1",
"mockery/mockery": "^1.4.4",
"nunomaduro/collision": "^6.1",
"phpunit/phpunit": "^9.5.10",
"spatie/laravel-ignition": "^1.0"
},
"autoload": {
"files": [
"app/Library/helper.php"
],
"psr-4": {
"App\": "app/",
"Database\Factories\": "database/factories/",
"Database\Seeders\": "database/seeders/"
}
},
"autoload-dev": {
"psr-4": {
"Tests\": "tests/"
}
},
"scripts": {
"post-autoload-dump": [
"Illuminate\Foundation\ComposerScripts::postAutoloadDump",
"@php artisan package:discover --ansi"
],
"post-update-cmd": [
"@php artisan vendor:publish --tag=laravel-assets --ansi --force"
],
"post-root-package-install": [
"@php -r "file_exists('.env') || copy('.env.example', '.env');""
],
"post-create-project-cmd": [
"@php artisan key:generate --ansi"
]
},
"extra": {
"laravel": {
"dont-discover": []
}
},
"config": {
"optimize-autoloader": true,
"preferred-install": "dist",
"sort-packages": true
},
"minimum-stability": "dev",
"prefer-stable": true
}
In my routes/api.php :

<?php

use Illuminate\Http\Request;
use Illuminate\Support\Facades\Route;

use App\Http\Controllers\AuthController;
use App\Http\Controllers\PageController;

use App\Http\Controllers\AuthorsController;

/*
|--------------------------------------------------------------------------
| API Routes
|--------------------------------------------------------------------------
|
| Here is where you can register API routes for your application. These
| routes are loaded by the RouteServiceProvider within a group which
| is assigned the "api" middleware group. Enjoy building your API!
|
*/

Route::middleware('auth:api')->prefix('v1')->group(function () {
    Route::get('/user', function (Request $request) {
        return $request->user();
    });

    Route::post('/register', [AuthController::class,'register']);
    Route::post('/login', [AuthController::class,'login']);

    Route::group(['middleware' => 'auth'], function () {

//        Route::get('/profile', 'UserProfileController@index');
//        Route::put('/profile/{user_id}', 'UserProfileController@update');

        Route::post('/logout', [AuthController::class, 'logout'] );

        Route::resource('/authors', AuthorsController::class);

        // PAGES BLOCK BEGIN
        Route::resource('/pages', PageController::class);
        Route::group(['prefix' => 'pages'], function () {
            Route::put('{id}/publish', [PageController::class, 'publish']);
            Route::put('{id}/unpublish', [PageController::class, 'unpublish']);
//        Route::put('{id}/unpublish', 'PageController@unpublish');
        });
        // PAGES BLOCK END

    });

});

and in routes/web.php :

<?php

use Illuminate\Support\Facades\Route;

/*
|--------------------------------------------------------------------------
| Web Routes
|--------------------------------------------------------------------------
|
| Here is where you can register web routes for your application. These
| routes are loaded by the RouteServiceProvider within a group which
| contains the "web" middleware group. Now create something great!
|
*/

Route::get('/', function () {
    return view('welcome');
});

I have Postman v9.0.8.
What could raise this error and how can it be fixed?
Thanks!

Thanks for submitting this issue @sergeynilov, I'll take a look into it and get back to you after.

Good day @sergeynilov 👋
The issue is this route:
Route::get('/user', function (Request $request) {
    return $request->user();
});

If you comment it out, it should work as expected. I recommend using controllers for all routes, so if you do use that /user route, you could move it to a UserController or something.

Hello @sergeynilov, are you able to confirm whether or not disabling the /user route fixes the issue for yourself? I'm unable to reproduce this locally with the package, so any steps to reproduce this on a clean Laravel install would be greatly appreciated.
I'm going to close this (at least for now) until it's a reproducible issue, just incase it's user error.


SDCSRM-521 Add venom config file

Motivation and Context

To make the venom security tests more re-usable, the test config file has been removed from the baked docker image and now needs to live and be fetched from somewhere else. It makes most sense to me for it to live in the RH UI repo, with the code it is testing.
What has changed

Add venom test yml

How to test?

See the linked PRs, this has been tested in a dev pipeline already.
Links

https://jira.ons.gov.uk/browse/SDCSRM-521

It's unlikely for us to test it locally unless something is wrong, but would it be helpful to add a venom command to the makefile and possibly even to the readme?

It's unlikely for us to test it locally unless something is wrong, but would it be helpful to add a venom command to the makefile and possibly even to the readme?

I think the tests are currently bound to fail locally, since some headers are only present when it's serving https. It would definitely be convenient to be able to run them locally, though, even if they won't all pass. It would be useful for when we add or update the tests in future.


useRevoluteJoint causing expected instance of O

useRevoluteJoint(anchor, box, [
    [0, 0, 0], // Position of the joint in bodyA's local space
    [0, 0, 0], // Position of the joint in bodyB's local space
    [0, 0, 0] // Axis of the joint, expressed in the local-space of the rigid-bodies it is attached to.
  ]);

codesandbox

The last argument of the useRevoluteJoint needs to be a direction of spin:
useRevoluteJoint(anchor, box, [
    [0, 0, 0],
    [0, 0, 0],
    [0, 1, 0] // <----
  ]);

Will update the readme ✅


Add twitter color, font and tinder font

I picked Twitter and Tinder font from their websites
Then make CSS variable using Twitter brand color guidelines in App.css
I change from, old

to new

Looks like, old one more colorful, but this one more like Twitter's


Increase timeout for CircleCI

Increase timeout for one of the commands executed by CircleCI as it is failing occasionally

Sounds good to me.


[Feature Request]: Richer options for filter_config

Is your feature request related to a problem? Please describe.
At present, the filter_config handling in openai_utils.py considers only model properties such as name, base_url, api_type etc., which are parameters used for creating clients and making calls. Further, all filters operate as an AND (all conditions must be met), so we are missing OR semantics.
Describe the solution you'd like
Ideally we should be able to filter on other model properties such as model family, context window size, or token cost. This is especially important when using Azure, where different deployments could have different deployment names (e.g., "gpt4", "gpt_4", "autogen-gpt-4", etc.), and I want to basically select all models that are of the "gpt-4" family. Alternatively, I might want all models where the cost per 1k tokens is less than $0.03, etc.
It would also be useful if I could pass a filter_list ( of filter_dicts ) and this would rule-in models that match any of the provided filter_dicts. Provide such "OR" capabilities would alleviate some of the concerns noted above.
Additional context
No response

In config_list_from_json, the value of a filter key can be a list and we do use "OR" to match any element in the list.
Could you clarify which function you are referring to?

Closing this issue due to inactivity. If you have further questions, please open a new issue or join the discussion in AutoGen Discord server: https://discord.com/invite/Yb5gwGVkE5


Is there any notification when the front view controller size changing?

I want to add an animation on front view controller which corresponding to the width is changing. (When using gesture pan)
Is that possible, is this library provide such delegate to return the actual size during swiping?
Thank you

Yes!
// The following methods provide a means to track the evolution of the gesture recognizer.
// The 'location' parameter is the X origin coordinate of the front view as the user drags it
// The 'progress' parameter is a number ranging from 0 to 1 indicating the front view location relative to the
//   rearRevealWidth or rightRevealWidth. 1 is fully revealed, dragging ocurring in the overDraw region will result in values above 1.
// The 'overProgress' parameter is a number ranging from 0 to 1 indicating the front view location relative to the
//   overdraw region. 0 is fully revealed, 1 is fully overdrawn. Negative values occur inside the normal reveal region

(void)revealController:(SWRevealViewController *)revealController panGestureBeganFromLocation:(CGFloat)location progress:(CGFloat)progress overProgress:(CGFloat)overProgress;
(void)revealController:(SWRevealViewController *)revealController panGestureMovedToLocation:(CGFloat)location progress:(CGFloat)progress overProgress:(CGFloat)overProgress;
(void)revealController:(SWRevealViewController *)revealController panGestureEndedToLocation:(CGFloat)location progress:(CGFloat)progress overProgress:(CGFloat)overProgress;

The width is always the same, the position change


Make resent-sender header optional

In reference to #1880 a clean branch.

What are advantages of "resent-sender" header?
Or contrary what are disadvantages of removing this header (why it was being added at the first place)?
Some clients are showing this header, so making it optional sounds great. But it would be great to know what (bad) can I expect when turning this header off.
Thank you!

https://github.com/postalserver/postal/issues/767#issuecomment-473677331

@joostkalwij Thank you much for your detailed response! Much appreciated.
I want it removed as well, but at the same time I don't want to risk any delivery problems (obviously). That's my only concern here. I've checked the RFC 822 and my understanding is basically same as yours, meaning omitting this header should not have any delivery consequences nor issues.
It would be great if more "experts" in this field could chime in and verify our thoughts so we don't experience delivery issues when disabling this header.


A new version of the app is available each session

Overview
Currently the message that 'A new version of the app is available' and the prompt to reload is displayed to me during each session. If I click reload, the app reloads and the message goes away temporarily, but returns in about a minute when the app finishes loading.

References and additional details
Steps to repro:

Load app.shapeshift.com on web or mobile
When the prompt to reload the app is displayed, tap reload
Notice the prompt return in about 1 minute when app finishes loading

Acceptance Criteria

The prompt to reload the app is only displayed if a newer version of the app is available

Need By Date
Let's fix this asap, it's v annoying for our existing users
Screenshots/Mockups
No response
Estimated effort
No response

tshifty, ptt, and wesley all were unable to reproduce this issue. closing for now until we get more reports


Update xeus version

Similarly to xeus-cling, if you update to 0.18.1 and use blocking_input_request, it should also remove some code!


app is broken (singing application for MacOS)

I  packageDmg a dmg file,  I can install and run in current macos ,but when I copy this file to another macos , it can install but
can't run , the dialog show me it is broken。how can I fix it ?
jdk:15
kotlin("jvm") version "1.4.30"
id("org.jetbrains.compose") version "0.3.1"
current macos version:11.2
another macos version:11.4

One must sign and notarize app with Apple to run it on other machines, see https://github.com/JetBrains/compose-jb/tree/master/tutorials/Signing_and_notarization_on_macOS

@olonho thank you! I found in the test,The app build from Xcode can run  on other machines,and it not need Developer ID ,so why the compose need,do you know?

Do not think that one can run unsigned apps of other machines on recent macOS versions.

@qq7866196 probably that app is being signed by Xcode somehow. We cannot sign applications automatically, since we don't have certificate to sign it (it should be provided by developer).

I want use it development some tools, and share in my team. But I am  not an apple developer, it  not free.  I not need to publish to app store. hope it can support on the future version.

@qq7866196 it is not about publishing to appStore. MacOs verifies signature for any application that is run on a modern MacOS system. And an application should be signed by developer certificate, that, by default we don't have - developer shall provided it to build system, as described in the tutorial.

Closing as inactive

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


The create account link is hidden with the link to go to the Github project of the app.

Hello, It is not possible to go the create account page because the link to go there is hidden with another link togo to the Github project of the app.
Maybe add a spacing.

Hy @r-e-d-ant , thanks for the warning. Is it something you would be willing to contribute, maybe open up a PR with fix?

@r-e-d-ant wait for me to fix the pipelines problems before I can release it to production. See #189 for more info.

ok ok !
On Wed, Sep 21, 2022 at 5:13 AM Alvaro Israel Nunes Leite <
@.***> wrote:

@r-e-d-ant https://github.com/r-e-d-ant wait for me to fix the
pipelines problems before I can release it to production. See #189
https://github.com/AlvaroIsrael/gobarber-app/pull/189 for more info.
—
Reply to this email directly, view it on GitHub
https://github.com/AlvaroIsrael/gobarber-app/issues/187#issuecomment-1253152174,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APYZDORE74D7G2FK3EFTOBTV7J4NHANCNFSM6AAAAAAQOJ3PTI
.
You are receiving this because you were mentioned.Message ID:
@.***>


Installation Guide

Hey, is it released yet? Can you provide the installation guide?

No, I'm sorry it's not released yet, it's a work in progress and it currently only supports 2-3 basic operations (including websockets).


python312Packages.aranet4: 2.2.3 -> 2.3.0

Diff: https://github.com/Anrijs/Aranet4-Python/compare/refs/tags/v2.2.3...v2.3.0
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

1 package failed to build:

home-assistant-component-tests.aranet

4 packages built:

python311Packages.aranet4
python311Packages.aranet4.dist
python312Packages.aranet4
python312Packages.aranet4.dist


[mpg123] Add patch for wasm triplet, fix checktypesize

Seems that something about the wasm env ended up triggering an error when the CMakelists.txt calls check_type_size due to CheckTypeSize not being included, Working on submitting a report to the maintainers.
I attempted to update to 1.32.0 that just released a few hours ago, but it seems there's just issues with that release, I think they accidentally dropped a couple folders when they archived the release, as the install fails even on x64-windows for the same reason (can't install the manual due to the file/folder not existing) I'll report that as well after reporting the check_type_size fix.

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Please do these steps for proper versioning:

revert the changes in versions/
increment port-version in ports/mpg123/vcpkg.json, and commit this change.
run vcpkg x-add-version mpg123, and commit the new changes.

If your modification is completed, please revert to "ready for review". I will initiate the code merge.

Working on submitting a report to the maintainers.

Please make sure it's submitted to them. Thanks for the fix!


[Question] Any thoughts on supporting azure data studio?

Hi guys -
Not a big deal if you can't, do you guys have any thoughts on supporting azure data studio?
I use it quite a bit for my work with MSSQL, and apparently it's built on top of vscode - and having the power of vim would be really nice :).
Looks like adamskt was able to get vscodevim to install and run on azure data studio before.  https://github.com/VSCodeVim/Vim/issues/2677
Unfortunately if I try to install from VSIX the way he describes in that issue, I get this:

:(

If the API is compatible then why not? I'll investigate

@asvetliakov What did your investigation find? :)

I’d be very interested in this too.

It's likely it works now because https://github.com/vscode-neovim/vscode-neovim/pull/668


Allow disabling New Version info banner

Describe the bug
0.12 release of ui-server brought in feature to display a banner at top by parsing the github releases page. However, contacting public github.com is restricted from our company production environment.
To Reproduce
Deploy 0.12 version after 0.12.1 or higher version is available and observe this behavior.
Expected behavior
Provide a config switch to turn off this feature.
Screenshots
N/A
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version - 0.12

Additional context
Add any other context about the problem here.

resolving with #524


Allow the Controller and Webhook Containers to run with the securityContext: readOnlyRootfilesystem: true

🛑 Important:
Please open an issue to discuss significant work before you start. We appreciate your contributions and don't want your efforts to go to waste!
For guidelines on how to contribute, please review the CONTRIBUTING.md document.
Purpose of this PR
Provide a clear and concise description of the changes. Explain the motivation behind these changes and link to relevant issues or discussions.
Close #2218
Proposed changes:

Mount an emptyDir onto the controller container to write Spark artifacts to
Mount an emptyDir onto the webhook container to write certficates to
Set the default securityContexts for the controller and webhook to readOnlyRootfilesystem: true

Change Category
Indicate the type of change by marking the applicable boxes:

[ ] Bugfix (non-breaking change which fixes an issue)
[X] Feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that could affect existing functionality)
[ ] Documentation update

Rationale

Please see this issue for discussion around this feature.
This feature improves the default security posture of the Spark Operator Helm Chart and allows organizations that require readOnlyRootfilesystem: true to deploy the helm chart.
Checklist
Before submitting your PR, please review the following:

[X] I have conducted a self-review of my own code.
[X] I have updated documentation accordingly.
[X] I have added tests that prove my changes are effective or that my feature works.
[X] Existing unit tests pass locally with my changes.

Additional Notes

@npgretz Could you do a rebase as there is a merge conflict?

@npgretz Could you do a rebase as there is a merge conflict?

Done. Thank you for making me aware!

Sorry for the small mistakes holding up the merge; this is my first open-source contribution. I am just figuring out the process.

@npgretz Thanks for the contribution to enhance the security of the Spark operator. Just a few small nits, could you address them and regenerate the helm README by running make helm-docs? Then everything will be fine for me.

It would be better to use the K8s yaml style for values.yaml file, i.e.， the array need not be indented.

Thank you for your suggestions! Could ask for clarification on this suggestion. Is this referring to the array of objects for the volumes and volumeMounts I added for the controller and webhook pods? I am not sure how I could create these arrays of objects without the indentation and have valid YAML.
I also did not find a reference for K8's YAML Style. Could you direct me to a resource?
Thank you, again!

@npgretz If you run kubectl get pod -o yaml to fetch the manifest of any pod, you will find the array objects are not indented:
$ kubectl get pod -n spark-operator spark-operator-controller-69844fbfb5-nqwts -o yaml
...
spec:
  containers:
  - args:
    - controller
    - start
    - --zap-log-level=info
    - --namespaces=default
    - --controller-threads=10
    - --enable-ui-service=true
 ...

Is this referring to the array of objects for the volumes and volumeMounts I added for the controller and webhook pods?

Yes, you can simply remove the indentation so that we can maintain a consistent YAML style.

@ChenYi015, thank for you for explaining that. I hadn't realized the format of the templated manifests was off. I have adjusted the deploy templates for the controller and webhook to correctly indent volumes and volumeMounts.

/lgtm


'SLKTextInput.h' file not found for 1.9.2

An import error of SLKTextView.h occurs when I use the framework in version 1.9.2.

Oh, Sorry it was revised in 4c6e3ef

I'm having the same problem in 1.9.2. I can successfully build 1.9.0 and 1.9.1.
Built with:
carthage version
#=> 0.16

carthage update --platform iOS SlackTextViewController

This fixes it for me–thanks @dzenbot


How much time an alert takes to fire

Hi ,
I have an alert rule configured  in Prometheus is like this :
- alert: K8SSchedulerDown
expr: absent(up{job="kube-scheduler"} == 1)
for: 5m
labels:
severity: critical
And the AlertManager is configured with default 'repeat_interval' to 3 minutes.
I have 2 questions :

How much time it will take for the above alert  to move to PENDING state , and How much time it will take to move to FIRING state. Is 5(Inactive to Pending )+5 (Pending to Firing)  minutes , it takes to FIRING.
If the alert will be moved to FIRING state after 5 or 10 minutes , then why I am getting alerts in 3 minutes(is it due to the repeat_interval settings)

This post might also be interesting: https://pracucci.com/prometheus-understanding-the-delays-on-alerting.html

@debbiswal With a 1m eval duration and 1m for duration, that is the maximum time it should take, yes. Please take further usage questions to the mailing list and/or IRC channel though...

Thanks @juliusv .


Container Editor assigns wrong ID to container

Likely due to the way the editor is called

Issue resolved in a patch


[AST] Missing space when printing generic params

Using -dump-parse on func foo<T>(bar: T) {} results in:
(source_file
  (func_decl "foo(bar:)"<T>
    (parameter_list
      (parameter "bar" apiName=bar))
    (brace_stmt)))

Notice there is no space between "foo(bar:)" and <T>.
Add a space to correct the formatting error.

@swift-ci please smoke test and merge.

Looks like the test failure is legit.

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge


Etag + md5 question

I'm playing with python3 conversion, and came across the following :
I see that here the Etag is set to a md5.new(url) object. Shouldn't it be the md5.new(url).hexdigest() ? Or does django do some magic ?
Thanks.

Perhaps -- It would be great to get a test for this showing it failing, and a patch that fixes it.

Actually, it should use the hashlib module. md5 has been deprecated since Python 2.5. However, it seems like the else branch in _badge_return is unused. _badge_return used by the project_badge view which passes redirect=True by default. I will open a pull request to simply it.


The rect calculation for the Semantics tree is incorrect in landscape mode

It appears there is always an assumption that the system nav bar is on the right side of the screen, but a clockwise rotation to landscape places the nav bar on the left side of the screen.

This information is not supplied by the WindowInsets - these continuously report insets/padding/stable insets of zero.  Instead we need to detect a left system nav bar somehow.


Questions for nested/multi layer schemas missing specific tags

When having schema that does use nested objects questions for the filed using field name but are missing whole tags/path
For example instead of "please provide kafka credentials name" user is being asked "name". This is confusing to user etc.

We could build a question from the path of the field in the json schema.
{
   "kafka": {
       "name": {"type": "string"}
   },
   "connector": {
      "name": {"type": "string"},
      "details": {
         "size": {"type": "number"}
      }
   }
}

Prompts the user with:
Enter "name" in "kafka": ...
Enter "size" in "details" in "kafka": ...
Enter "name" in "connector": ...

Not the best but it could be a start

Actually that is exactly what we need

size from connector.details as format sound much better


Docker image auto build and deploy

Please, add automated docker build and push at docker hub in CI Pipeline.
I can write pipeline for CircleCI or Travis.

I was looking on Kitematic yesterday and there're quite a few unofficial dockers for hugo, most of which are not well-maintained or poorly constructed (i.e. not using scratch).
The current Dockerfile in the project repo makes some assumptions, such as to which ports to expose and volumes to use, which, while nice, would still require some adjustments for use in some Compose projects.
Given the base image, env vars and apt packages will change between builds having an official image will help prevent the need to use something like Docker-in-Docker, which adds a funky abstraction to running hugo in containerized environments.
TL;DR: Please update the CI/CD for deploys to DockerHub as OP suggests.

For those looking for a quick way to build the 0.44 with Docker I've created a Dockerfile to build tagged versions of Hugo here (path subject to change):
https://git.habd.as/comfusion/after-dark/src/branch/fathom/docker/hugo/Dockerfile

+1 for Composeability

I think this is still relevant because Hugo users usually perform some kinda autobuild for their websites, usually done in a CI/CD pipeline, usually based on Docker for easiness and reusability.
This would make the project more usable, and still more maintainable because CI would do the dirty job.

I would also like to see an officially maintained Docker image be added. :+1:

There is an official docker image on dockerhub. But it is stale like this issue.

There is an official docker image on dockerhub. But it is stale like this issue.

That gohugoio account isn't official.

So it's a well crafted fake that drains reputation then.

Gitlab provides one.
I also think that the project should provide one, it would be cleaner for our CICD pipelines :)

LGTM on dockerhub?

It is a good practice to wrap a command line tool into a docker image, I believe.
Could you please release an official image for us?
Or you have other concerns about this?

We (I) plan to use GitHub to publish docker images.
https://github.com/gohugoio/hugo/pull/6405#issuecomment-548280568

FYI.

How can one help with this? Sure, if people want to use hugo in their automated, containerized environments, they can just clone the repo and do the docker build itself. But it is pretty much standard to provide a dockerized version of an application. It's the direction the industry has been moving in for the past years.
When I tried to use the gohugoio/hugo image last week and noticed that it is based upon a version of hugo that is two years old, I was very confused. Then I noticed that it is not THIS repo that the image is built upon, but a fork. So I thought: "Alright, some 2 years ago, someone broke the docker build. And apparently it was very hard to fix or something. So they switched to a working fork, and then forgot it or deprioritized it or whatever." So I thought, I'm going to fork that repo and fix the damn build, do something for the OS community.
I was even more confused when I cloned my fork, ran docker build  on it and everything worked. I don't understand. Docker Hub as an autobuild feature. Providing an up-to-date docker image should be as easy as re-connecting THIS repo to the gohugoio/hugo repo  on Docker Hub and telling it to auto-build everytime something gets merged into master. But then  again, if it was  that easy, it would have been done, right? So what am I missing?

@Christoph-Harms I recommend this one: https://hub.docker.com/r/klakegg/hugo/tags

That gohugoio account isn't official.

I overlooked that comment of yours, @bep. That explains it. Shame that it "looks" official, because the account names on Docker Hub and Github are the same. :/ It gives a taste of, I don't know, unreliability? It's not exactly what I mean, but I don't have a better word. It puts hugo in a bad light. Kind of.  :)

@Christoph-Harms I recommend this one: https://hub.docker.com/r/klakegg/hugo/

Looks great, thank you! I didn't even find that one when browsing Docker Hub. Only old forks or specialized builds.

@Christoph-Harms @klakegg is reliable. I understand that the "official Hugo docker situation" isn't ... ideal, but it's a priority/time thing. I have now added the klakegg Docker to the Hugo install page so it should be easier to find.


#210 [Accommodation] Make fields non-obligatory if the status is "withdrawn:

https://safeshelter.youtrack.cloud/issues?preview=FEEDBACK-17


A small correction in spelling

A small correction in spelling

@vikranthsuravarapu : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Typo fix verified.


error when creating a new project by CLI command

I'ma newbie in angular. When I try run new project like tutorial, i always get a error as below:
** Angular Live Development Server is listening on localhost:8082, open your browser on http://localhost:8082/ **
Date: 2018-07-26T06:46:54.486Z
Hash: 4d3e973e83c839f26163
Time: 4843ms
chunk {main} main.js, main.js.map (main) 1.91 kB [initial] [rendered]
chunk {polyfills} polyfills.js, polyfills.js.map (polyfills) 671 bytes [initial] [rendered]
chunk {runtime} runtime.js, runtime.js.map (runtime) 5.22 kB [entry] [rendered]
chunk {styles} styles.js, styles.js.map (styles) 15.6 kB [initial] [rendered]
chunk {vendor} vendor.js, vendor.js.map (vendor) 325 kB [initial] [rendered]
ERROR in node_modules/@angular/platform-browser-dynamic/src/compiler_factory.d.ts(18,22): error TS2720: Class 'CompilerImpl' incorrectly implements class 'Compiler'. Did you mean to extend 'Compiler' and inherit its members as a subclass?
Property 'getModuleId' is missing in type 'CompilerImpl'.
node_modules/@angular/platform-browser-dynamic/testing/src/compiler_factory.d.ts(12,22): error TS2720: Class 'TestingCompilerImpl' incorrectly implements class 'TestingCompiler'. Did you mean to extend 'TestingCompiler' and inherit its members as a subclass?
Property 'getModuleId' is missing in type 'TestingCompilerImpl'.
i ｢wdm｣: Failed to compile.

What versions of NPM, Angular-CLI, and Node are you using?

I had the same problem and solved it by updating @angular/platform-browser and @angular/platform-browser-dynamic to "^6.1.0" in my package.json. Before the versions had been stuck at "6.0.9" which was different to the versions of angular (6.1.0) and angular-cli (6.1.1).

Thanks for reporting this issue. However, you didn't provide sufficient information for us to understand and reproduce the problem. Please check out our submission guidelines to understand why we can't act on issues that are lacking important information.
If the problem persists, please file a new issue and ensure you provide all of the required information when filling out the issue template.

@sclode workarround Seems to have fixed my issue.