Properly handle non-UTF8 filenames in Zip

	at java.base/java.lang.StringCoding.throwMalformed(StringCoding.java:685)
	at java.base/java.lang.StringCoding.decodeUTF8_0(StringCoding.java:768)
	at java.base/java.lang.StringCoding.newStringUTF8NoRepl(StringCoding.java:965)
	at java.base/java.lang.System$2.newStringUTF8NoRepl(System.java:2197)
	at java.base/java.util.zip.ZipCoder$UTF8.toString(ZipCoder.java:60)
	at java.base/java.util.zip.ZipCoder.toString(ZipCoder.java:87)
	at java.base/java.util.zip.ZipInputStream.readLOC(ZipInputStream.java:301)
	at java.base/java.util.zip.ZipInputStream.getNextEntry(ZipInputStream.java:123)
	at xyz.docbleach.module.zip.ArchiveBleach.sanitize(ArchiveBleach.java:44)
	at xyz.docbleach.api.bleach.CompositeBleach.sanitize(CompositeBleach.java:74)
	at xyz.docbleach.api.BleachSession.sanitize(BleachSession.java:71)
	at xyz.docbleach.cli.Main.sanitize(Main.java:81)
	at xyz.docbleach.cli.Main.main(Main.java:54)
Caused by: java.nio.charset.MalformedInputException: Input length = 1
	... 13 more

Process finished with exit code 1

Sample file: e35d68feda25f401da03883da1e9c437
Archive:  VirusShare_e35d68feda25f401da03883da1e9c437
Zip file size: 1978422 bytes, number of entries: 4
drwx---     3.1 fat        0 bx stor 13-Jun-24 09:44 bulletstorm-trainer18/
-rwxa--     3.1 fat  2030080 bx defN 11-Mar-12 21:16 bulletstorm-trainer18/BS+28Tr-LinGon.exe
-rw-a--     3.1 fat      893 tx defN 13-Jun-24 09:48 bulletstorm-trainer18/+�+���+��+��.txt
-rw-a--     3.1 fat      151 tx defN 13-Mar-29 17:14 bulletstorm-trainer18/+�+���+��+��.url
4 files, 2031124 bytes uncompressed, 1977223 bytes compressed:  2.7%

Full zipinfo:
Archive:  VirusShare_e35d68feda25f401da03883da1e9c437
The zipfile comment is 241 bytes long and contains the following text:
======================== zipfile comment begins ==========================
�������԰ http://www.cr173.com

�������������������û���⡣

�ٶ�һ�¡��������԰������ϲ������Ŷ��

--------------------------------------------------
--------------------------------------------------

��ѹ���룺www.cr173.com
========================= zipfile comment ends ===========================

End-of-central-directory record:
-------------------------------

  Zip archive file size:                   1978422<PHONE_NUMBER>1E3036h)
  Actual end-cent-dir record offset:       1978159<PHONE_NUMBER>1E2F2Fh)
  Expected end-cent-dir record offset:     1978159<PHONE_NUMBER>1E2F2Fh)
  (based on the length of the central directory and its expected offset)

  This zipfile constitutes the sole disk of a single-part archive; its
  central directory contains 4 entries.
  The central directory is 572<PHONE_NUMBER>00023Ch) bytes long,
  and its (expected) offset in bytes from the beginning of the zipfile
  is 1977587<PHONE_NUMBER>1E2CF3h).

Central directory entry #1:
---------------------------

  bulletstorm-trainer18/

  offset of local header from start of archive:   0
<PHONE_NUMBER>000000h) bytes
  file system or operating system of origin:      MS-DOS, OS/2 or NT FAT
  version of encoding software:                   3.1
  minimum file system compatibility required:     MS-DOS, OS/2 or NT FAT
  minimum software version required to extract:   1.0
  compression method:                             none (stored)
  file security status:                           not encrypted
  extended local header:                          no
  file last modified on (DOS date/time):          2013 Jun 24 09:44:28
  32-bit CRC value (hex):                         00000000
  compressed size:                                0 bytes
  uncompressed size:                              0 bytes
  length of filename:                             22 characters
  length of extra field:                          36 bytes
  length of file comment:                         0 characters
  disk number on which file begins:               disk 1
  apparent file type:                             binary
  non-MSDOS external file attributes:             000000 hex
  MS-DOS file attributes (10 hex):                dir

  The central-directory extra field contains:
  - A subfield with ID 0x000a (PKWARE Win32) and 32 data bytes.  The first
    20 are:   00 00 00 00 01 00 18 00 d4 e6 d4 5d 7c 70 ce 01 d4 e6 d4 5d.

  There is no file comment.

Central directory entry #2:
---------------------------

  There are an extra -36 bytes preceding this file.

  bulletstorm-trainer18/BS+28Tr-LinGon.exe

  offset of local header from start of archive:   52
<PHONE_NUMBER>000034h) bytes
  file system or operating system of origin:      MS-DOS, OS/2 or NT FAT
  version of encoding software:                   3.1
  minimum file system compatibility required:     MS-DOS, OS/2 or NT FAT
  minimum software version required to extract:   2.0
  compression method:                             deflated
  compression sub-type (deflation):               normal
  file security status:                           not encrypted
  extended local header:                          no
  file last modified on (DOS date/time):          2011 Mar 12 21:16:50
  32-bit CRC value (hex):                         3083c7e1
  compressed size:                                1976519 bytes
  uncompressed size:                              2030080 bytes
  length of filename:                             40 characters
  length of extra field:                          36 bytes
  length of file comment:                         0 characters
  disk number on which file begins:               disk 1
  apparent file type:                             binary
  non-MSDOS external file attributes:             000000 hex
  MS-DOS file attributes (20 hex):                arc

  The central-directory extra field contains:
  - A subfield with ID 0x000a (PKWARE Win32) and 32 data bytes.  The first
    20 are:   00 00 00 00 01 00 18 00 20 37 fb bf b7 e0 cb 01 66 31 2f 59.

  There is no file comment.

Central directory entry #3:
---------------------------

  There are an extra -36 bytes preceding this file.

  bulletstorm-trainer18/+�+���+��+��.txt

  offset of local header from start of archive:   1976641
<PHONE_NUMBER>1E2941h) bytes
  file system or operating system of origin:      MS-DOS, OS/2 or NT FAT
  version of encoding software:                   3.1
  minimum file system compatibility required:     MS-DOS, OS/2 or NT FAT
  minimum software version required to extract:   2.0
  compression method:                             deflated
  compression sub-type (deflation):               normal
  file security status:                           not encrypted
  extended local header:                          no
  file last modified on (DOS date/time):          2013 Jun 24 09:48:12
  32-bit CRC value (hex):                         558ee91a
  compressed size:                                570 bytes
  uncompressed size:                              893 bytes
  length of filename:                             38 characters
  length of extra field:                          89 bytes
  length of file comment:                         0 characters
  disk number on which file begins:               disk 1
  apparent file type:                             text
  non-MSDOS external file attributes:             000000 hex
  MS-DOS file attributes (20 hex):                arc

  The central-directory extra field contains:
  - A subfield with ID 0x000a (PKWARE Win32) and 32 data bytes.  The first
    20 are:   00 00 00 00 01 00 18 00 aa 21 43 e3 7c 70 ce 01 1c 87 ee 5b.
  - A subfield with ID 0x7075 (UTF8 path name) and 49 data bytes. The first
    24 UTF8 bytes in the extra field (V1, ASCII name CRC `f37f78d7') are:
    62 75 6c 6c 65 74 73 74 6f 72 6d 2d 74 72 61 69 6e 65 72 31 38 2f e8 a5.

  There is no file comment.

Central directory entry #4:
---------------------------

  There are an extra -36 bytes preceding this file.

  bulletstorm-trainer18/+�+���+��+��.url

  offset of local header from start of archive:   1977332
<PHONE_NUMBER>1E2BF4h) bytes
  file system or operating system of origin:      MS-DOS, OS/2 or NT FAT
  version of encoding software:                   3.1
  minimum file system compatibility required:     MS-DOS, OS/2 or NT FAT
  minimum software version required to extract:   2.0
  compression method:                             deflated
  compression sub-type (deflation):               normal
  file security status:                           not encrypted
  extended local header:                          no
  file last modified on (DOS date/time):          2013 Mar 29 17:14:02
  32-bit CRC value (hex):                         fcf30365
  compressed size:                                134 bytes
  uncompressed size:                              151 bytes
  length of filename:                             38 characters
  length of extra field:                          89 bytes
  length of file comment:                         0 characters
  disk number on which file begins:               disk 1
  apparent file type:                             text
  non-MSDOS external file attributes:             000000 hex
  MS-DOS file attributes (20 hex):                arc

  The central-directory extra field contains:
  - A subfield with ID 0x000a (PKWARE Win32) and 32 data bytes.  The first
    20 are:   00 00 00 00 01 00 18 00 b8 1b 98 c1 5d 2c ce 01 1c 87 ee 5b.
  - A subfield with ID 0x7075 (UTF8 path name) and 49 data bytes. The first
    24 UTF8 bytes in the extra field (V1, ASCII name CRC `1b3e623c') are:
    62 75 6c 6c 65 74 73 74 6f 72 6d 2d 74 72 61 69 6e 65 72 31 38 2f e8 a5.

  There is no file comment.
4 files, 2031124 bytes uncompressed, 1977223 bytes compressed:  2.7%


Add custom twig loader to workaround cache issue

Proof of concept solution for liip/LiipThemeBundle#79.
Apologies if I have missed something, but this seems as though it can be solved relatively simply by using a custom Twig filesystem loader, as stof suggested. This solution does not solve the 'referencing a non-themed template from a themed template' issue.
Suggestions are welcome.

one of the unit tests needs to be fixed. also there are some minor CS issues (specifically missing space after if and before (
@stof any feedback here?

also it would be good to have a functional test to illustrate that this works.

This won't work as is, because the class name for the compiled template still does not take the theme into account, so the Twig_Environment will not load the template a second time

Ah! Good point, I hadn't considered that. I'll have a look and see what changes would be necessary to solve the compiled class name issue.

@stof, I believe you are incorrect in your diagnosis. When determining the class cache name, Twig_Environment::getTemplateClass() uses the return value from Twig_LoaderInterface::getCacheKey(). The behaviour of the default Twig_Loader_Filesystem::getCacheKey() is to simply use the value returned by findTemplate(), which returns the full path to the file.
Twig_FilesystemLoader is an ancestor of our new loader so we have no problem, because findTemplate() returns the full path to the template, including the theme name (e.g. 'themes/themename').
I will fix the CS, the broken test, and create a new test for this feature.

@stof can you check once more?

alternatively @dantleech do you have time to look at this PR?

Would be great to add some tests for the compiler pass and the Loader (using prophecy would make this task easier).

Thanks for taking a look @dantleech! Lots of feedback there, I'll try to address it today and update this PR.

ping @xanido

anyone have time to wrap this up?

it wasn't fixed with #123 ?

@dantleech / @xanido can you get this wrapped up?

ping


stats: fix req/resp body/hdrs sizes tracking

Using the upstream host to fetch the cluster doesn't
work from ActiveStream, because it may not be available anymore
by the time we look it up. So use ClusterInfo instead.
Fixes #15879.
Signed-off-by: Raul Gutierrez Segales<EMAIL_ADDRESS>

I'll follow-up with updated unit tests and an integration tests.

/wait

/wait

/wait

Need to an integration test.
/wait

Btw, I think we could merge this as is and I'll follow-up with an integration test to avoid regressions. Thanks!

@jmarantz added an integration test, in case you want to do another pass -- thanks.

@jmarantz added an integration test, in case you want to do another pass -- thanks.

... and I verified that the integration test does not pass/finish without the new approach (using cluster info to get that stats instead of the upstream host).

Ok moved things into the router, will follow-up with unit tests in a bit (integration test is still good). I'll take a look at supporting retries too, if it'll need significant refactoring we can probably do that in a follow-up?

Tests coming up in a bit.
/wait

cc: @mattklein123 for visibility (given there will be more work in this area, e.g.: to properly compute these stats for retries).

@snowp Ok, I think is ready for another pass -- thanks!

FYI this patch is generating stats nicely for us in prod 👍

/wait

/wait

/wait

/wait


Reuse variable in pattern matches for equality

This discussion comes up once in a while with newcomers, i.e. "why can't I write this?"
(defun equal?
  ([x x] 'true)
  ([_ _] 'false))

For now, we have to do something like this:
(defun equal?
  ([x y] (when (=:= x y))
   'true)
  ([_ _]
   'false))

It might be nice to reuse variables in pattern matches for equality as this related issue on Alpaca suggests.

This felt more "functional" but I have any strong objections to this. It would not create any compatibility issues though I would not allow it in a lambda only in a match-lambda.

:+1: for allowing variable reuse (for matching) in match-lambda!

We can do this. In the next push to develop.

I'd like this behaviour in case as well, but I (think I) want the compiler to complain if I try to, e.g.
``lfe
(defun foo (y)
(let ((x 42))
(case y
(x 'forty-two)
(_ 'not-forty-two))))

I am looking at adding implicit equality testing of variables in patterns I intend to leave the variable scoping as it is. This means that variables occurring in a pattern are new fresh variables and if you want to test them against variables outside the scope then you will need to do that explicitly. So @yurrriq in your case you are stuck with:
(defun foo (y)
  (let ((x 42))
     (case y
       (x1 (when (=:= x1 x)) 'forty-two)
       (_ 'not-forty-two))))

Perfect!

I have just pushed a first version of this into 'develop'. It seems to work but needs testing.
Variable scoping is still observed so explicit tests need to be added for variables already bound before the match. This differs from Erlang but we have kept importing variables into bitseg lengths.


rough-sketch-html

search bar needs header as well as adjustments to background and css styling

Corrected line 33 and 32 left in 26 and we can adjust later through git hub
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows
From: @.>
Sent: Monday, January 23, 2023 8:03 PM
To: @.>
Cc: Richard @.>; @.>
Subject: Re: [NewguyenGitHub/Recovery-Radar] rough-sketch-html (PR #5)
Merged #5https://github.com/NewguyenGitHub/Recovery-Radar/pull/5 into main.
—
Reply to this email directly, view it on GitHubhttps://github.com/NewguyenGitHub/Recovery-Radar/pull/5#event-8340209461, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A4FO7WKSBPQLYAUUKBJ2L6DWT5BBTANCNFSM6AAAAAAUESKJ4M.
You are receiving this because you authored the thread.Message ID: @.***>


Debug LODs in editor

Is it possible to view LOD changes in editor view or is it supposed to work in render only? I set up a scene with LODSpatial and a camera, but when I move the camera, LODs do not change in the camera preview viewport.

@Calinou Maybe there is a way to modify a scene in-memory, without saving the changes to the scene file? Mark these changes as "ephemeral" or something. Probably this would be helpful to visualize Remote changes as well.

Maybe there is a way to modify a scene in-memory, without saving the changes to the scene file? Mark these changes as "ephemeral" or something.

Godot can only do this if you add new nodes using a tool script (and don't set an owner for them), not if you modify existing nodes.


Add custom header for passthrough Google IAM

Currently the library won't work if the gitlab is under google IAM, need to add custom additional header to insert IAM token in the request.

I'm wondering if this is as simple as adding a new RequestOptionFunc and then when creating a new client just passing it into WithRequestOptions.

Yup, just simple custom header could work. But getting the IAM header is another thing, since it's not visible in regular network console.


missing support for custom namespace attributes and values

E.g. <html xmlns="http://www.w3.org/1999/xhtml" xmlns:havunen="http://www.example.com/trueadm"> will give issues. And so will  abc:def if set as attribute value.
Inferno only deals with xlink and xml. No custom.
Does the JSX plugin accepting this?
This is also after the specs. But various website to link to here, and to avoid trigger something. I suggest use Google to search for more info if needed.

I made the decision a while back not to support custom namespaces, it's too messy to handle edge cases properly and it's not something I want Inferno to support at this time. If people need custom namespace support, I'd suggest they explain a good use-case for today's modern web.

I can't find any documentation about this missing? Stated in readme you say
you are close to react api. You are not
On Apr 27, 2016 2:18 AM, "Dominic Gannaway"<EMAIL_ADDRESS>wrote:

Closed #197 https://github.com/trueadm/inferno/issues/197.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/trueadm/inferno/issues/197#event-641885245

@Kflash We have no documentation yet. Our readme says nothing of the kind. In principle, Inferno is compatible with the standard React API, allowing painless transition from React to Inferno.
Like I said before, I'm happy to build something in when there is a demand for such a thing. I personally see no demand and from my experience, it's not something people do.
Inferno's goal is not to be a React clone, it's to be a React-like library with insanely fast performance.


请作者开放预训练模型

@zhuxyme TATT has no pre-trained model, you can directly train from scratch. For the TP Generator, you can download weights following ReadMe.md and perform the initialization.

I think what he calls pre-training parameters refers to the completely trained model parameters that can get the results in the paper.

TATT has no pre-trained model, you can directly train from scratch. For the TP Generator, you can download weights following ReadMe.md and perform the initialization.

YES

是的

@zhuxyme  Please send your request to our email<EMAIL_ADDRESS>and provide your institute name and research purpose, we will provide you the trained parameter.


Update Q to promises in javascript

A tool for creating and composing asynchronous promises in JavaScript

No thanks. Bluebird is better.


[GitHub Migration] Pagination limited on repos with many issues

Description
When trying to migrate repositories with a large number of issues and PRs, I get following error:

Migrate repository from https://github.com/TeamNewPipe/NewPipe failed:
error while listing repos: GET https://api.github.com/repos/TeamNewPipe/NewPipe/issues/comments?direction=asc&page=1112&per_page=27&sort=created:
422 In order to keep the API fast for everyone, pagination is limited for this resource. Check the rel=last link relation in the Link response header to see how far back you can traverse. []

You might need to use a different method to retrieve data when this error occurs.
Gitea Version
1.21.11
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
self-hosted via docker
Database
PostgreSQL

Hm, the rate limit should be handled.


Two editors on same page, content from sidebar ends up in the second editor

Expected Behavior
Content added from sidebar should end up in the same editor as sidebar are in.
Current Behavior
If I add content from sidebar in editor number on content ends upp in sidebar number two.
Steps to Reproduce

Have two editors at same page
Add content from sidebar in editor number one.

Browser, device, and OS

Mac
Chrome 79

Hi, @davidsoderberg ! Are you using the same editorState for both editors ? If this is the case, each publisher must have their own editorState:
import React from "react";
import ReactDOM from "react-dom";
import {MegadraftEditor, editorStateFromRaw} from "megadraft";

import 'megadraft/dist/css/megadraft.css'

class Editor extends React.Component {
  constructor(props) {
    super(props);
    this.state = {editorState: editorStateFromRaw(null)};
  }

  onChange = (editorState) => {
    this.setState({editorState});
  }

  render() {
    return (
      <MegadraftEditor
        editorState={this.state.editorState}
        onChange={this.onChange}/>
    )
  }
}

class App extends React.Component {
  render() {
    return (

First editor

        <Editor/>

Second editor

        <Editor/>

    )
  }
}

ReactDOM.render(
  <App />,
  document.getElementById('container')
);

No they use thier own state exactly like you are doing in the example.

After tested this with another editor it seems to be in my code the issue are.


18 support multiple nrf cloud accounts

Delete migration section from README.md
Will delete migration cli after PR is approved
FingerPrint of health check device will be suffix with account name i.e. 29a.ch3ckr.exeger

@pudkrong you merged this PR without following the commit style, which resulted in the merge not releasing a new major release. In the future please squash PRs into on (or few commits), following the commit message guides before merging them so we get a nice release history.
This is also a BREAKING CHANGE, please provide the migration instructions in the description of the release at https://github.com/hello-nrfcloud/backend/releases/tag/v4.0.0

I've now corrected this mistake and force-pushed a squash commit in https://github.com/hello-nrfcloud/backend/commit/adcc67ae03f6566758cc89786956eb2ab6682278


update debian/ubuntu repository

spec test seems to be broken on upstream with strict variable on ::remi class.

Ahh that remi module.. I'm going to fix that now and will merge this in a bit 👍

@shoekstra centos6 is still failing after upstream merge

I've got some time off at the moment so will take a proper look. Going to refactor a bit since ownCloud changed how they package the app.

Have stopped using the remi module so the pain it brought is now gone for good.  I had already started on supporting ownCloud, so applied my local changes to update the repos which means this PR is no longer needed.
I'll push a new version to the forge shortly.


Title bar and menu unclickable (ALSO ZERO CONTACT OR SUPPORT FOR PAYING CUSTOMERS 😒)

After latest upgrade cannot use Franz File/Edit/View/Services/etc nor the windows min/max/close buttons in the title bar. They are there, but do not respond to clicks, nor standard Alt-F etc
To Reproduce
Upgrade to latest version of Franz
Expected behavior
Expect all menus and features to respond to user input....
Screenshots

Desktop (please complete the following information):

OS: Windows 10

Removing app and re-installing 5.8.0 - menus work again.

Fresh install, same problem!
The website forces the download of 5.9.0Beta1, but it cannot click anything.
And indeed, how is it possible that I am paying for this and you provide 0 contact information on your website?
This is very rude. Your service is not cheap!

Exactly the same here, please assist!

Based on all the other issues here, I cancelled my subscription.
Seems nobody of the team cares to respond to anything here.

Looks like this is a good alternative: https://getferdi.com/


load checkpoints

Hi!
Great code, thanks for sharing.
How can I use checkpoint files in order to continue training sessions after pausing?
Eyal

You can use model.load_state_dict(torch.load('CHECKPOINT_PATH', map_location=lambda storage, loc: storage)) at this line https://github.com/rosinality/glow-pytorch/blob/master/train.py#L164.

I guess that the next line should be at :
https://github.com/rosinality/glow-pytorch/blob/master/train.py#L166.
optimizer.state_dict(torch.load('checkpoint/optim_012001.pt', map_location=lambda storage, loc: storage))
Am I right?

Yes, if you also want to load optimizer states.

Codes have been changed, I can not find a proper way to load the checkpoint and I tried codes mentioned about which does not work,  any solution for loading the checkpoints?


Return model on update

Is your feature request related to a problem? Please describe.
When performing an update, I am missing the option to retrieve the updated model. I explicitly have to get the updated model from the DB.
Describe the solution you'd like
I would like to leverage the sqlalchemy Update returning clause which allows returning the data after the update has been performed.

That's a good point and should be pretty simple to implement. Would you like to work on it maybe?
Let's add optional returning parameter as a list of fields and chain the stmt with .returning if it's present.

Yes, I can give it a try. :)

Unfortunately, that would probably mean we need to remove the commit arg and ideally we do the same as we do for get and return the data as a model. I would even suggest we split the current function into update and update_multi.


createElement does not exist in default preact.js library. This breaks typescript

I see createElement being exposed in rollup aliases. I'm not sure why there are so many rollup targets. Its confusing.
why isn't createElement just bundled by default as an alias of h?

@nojvek Use the provided TS definition, the React one doesn't make sense for Preact.  If you want 100% compat with React, you should alias preact-compat, which does export all of those things by default.
The extra alias is provided for those who cannot change their JSX pragma.  In the latest Typescript, you can set it to anything you want (including h): https://github.com/Microsoft/TypeScript/issues/9582#issuecomment-260202346

Ahhh I see. It was jsxFactory compiler option.
I think I'll submit another PR for a little preact-typescript example.
Works with you?
On Monday, November 14, 2016, Jason Miller<EMAIL_ADDRESS>wrote:

@nojvek https://github.com/nojvek Use the provided TS definition, the
React one doesn't make sense for Preact. If you want 100% compat with
React, you should alias preact-compat, which does export all of those
things by default.
The extra alias is provided for those who cannot change their JSX pragma.
In the latest Typescript, you can set it to anything you want (including h):
Microsoft/TypeScript#9582 (comment)
https://github.com/Microsoft/TypeScript/issues/9582#issuecomment-260202346
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/developit/preact/issues/403#issuecomment-260398103,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-JVF6-ecIagMK74n4UwQDoJMny69eRks5q-JbxgaJpZM4KxkSH
.

A typescript example would be amazing 😍
Repo or gist, either is fine. If you link it here, we can also add it to the website.

@nojvek Should we keep this issue open, or close it since you're considering a more documentation-related PR? Cheers

There are two ways to look at it.
JsxFactory is very new. It's not in the major Typescript build. I had
to install tsc@next which is in beta. I also had to change vscode
Typescript.sdk settings. All in all it seemed a bit of pain.
On the other hand
createElement is just an alias of h and only a minor change for compat
reason.
Export {h, h as createElement,...
I prefer having createElement as it will work with most tsc builds and sdks
in atom, vscode, sublime out of the box. It's not that disruptive and only
adds couple of bytes to the dist code.
I also think we should document the new way of using preact but show
the fallback incase someone's having an older tsc compiler.
I'll create a nice medium article and get back to you.
Thoughts?
On Wednesday, November 16, 2016, Jason Miller<EMAIL_ADDRESS>wrote:

@nojvek https://github.com/nojvek Should we keep this issue open, or
close it since you're considering a more documentation-related PR? Cheers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/developit/preact/issues/403#issuecomment-260949627,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-JVGyjF8ADR8HQQ2tC5FkTLmArkRKtks5q-wi3gaJpZM4KxkSH
.

I'm pretty resistant to adding that alias to the default build - it's come up before and it just doesn't seem worth the cost given most people don't use it. There are currently no aliases at all in preact's core, and createElement() is entirely a compatibility thing - it is not a name anyone would choose if it weren't already a name React picked out of a hat.  Generally that means it's something that gets left to preact-compat, where all the crazy old aliases and legacy support stuff ends up.
One option is to alias preact to preact/aliases, which would get you h() and createElement() when importing 'preact' - seems like the best of both as long as it is respected by IDEs and whatnot (I don't really use editor integrations so I'm not super up-to-date on what they support).  Another would be to create a standalone package like preactsx or something that does the re-export, maybe easing installation and linking.
Agreed regarding documentation though - it'd be good to show the old and the new way, and I think that'd actually be a great reference for people wondering why TypeScript has added the new jsxFactory option.  It was difficult to make the pitch for why it was preferable over the old limited option, so maybe your medium post can serve as an in-depth explanation of the advantages.

Closing since TypeScript 2 is out of beta now and supports the custom pragma nicely, and we also now ship preact/aliases to cover this case. Let me know if I'm being presumptuous though :octocat:


Parser option

I want to use babel-eslint, but seems like there is no parser option.
/cc @shinnn @adametry

You can set "parser" option in .eslintrc.

Temporary I put it there, but missed parser option in Gulp plugin seems like inconsistent behavior.

This is because ESLint API doesn't accept parser option. We can set it only with .eslintrc  for the moment.

Currently, eslint doesn't move the 'parser' option from the CLIEngine to the file-specific config; however, there does seem to be one route that works currently: baseConfig.
Try this:
gulp.task('lint', function() {
	return gulp.src(['**/*.js'])
		.pipe(eslint({
			baseConfig:{
				parser: "babel-eslint"
			}
		}))
		.pipe(eslint.format());
});

The drawback on this one is that it effectively sets the "reset" option. If you already configure the eslint rules, this may not be a problem.

For what it's worth, eslint is considering clearing the baseConfig by default in v1.0.0.

FWIW: There is an accepted issue at ESLint (https://github.com/eslint/eslint/issues/3127) to be able to specify parser from the CLI (and CLIEngine).  It is currently waiting for someone to step up and implement it.

Setting the parser in .eslintrc gives the following error
{
"parser": "babel-eslint",
"rules": {
"strict": 0
}
}
0:0  error  Cannot find module 'babel-eslint'
It appears this package isn't included as a dependency for gulp-eslint.
Are there any plans to incorporate it?

As @IanVS mentioned, Eslint added support for the "parser" option in CLIEngine. Gulp-eslint does very little aside from migrating legacy options before passing the config to CLIEngine. Effectively, the option can be used with gulp-eslint now as well.

@jefferyasharp, while we use babel-eslint to test parser support, there is no direct dependency on babel-eslint in "production" use. If your package uses babel-eslint, it should be installed as a dependency there. Also, there may be some installation method that cause Eslint to have trouble finding a parser, such as globally installing or (sym)linking to Eslint. If either of these are possible causes of the error you're experiencing, consider installing babel-eslint relative to the path eslint is installed.
If the issue remains, please create a separate issue to discuss and track it.


CASTEP interface for sumo-bandstats

Hello, does the sumo-bandstats work with CASTEP?

Not yet. This is indicated in the "Feature Support" table of the README (in the "band analysis" row.)


Using StyleSheet.master.toHTML in production breaks the build

Environment
Binaries:

Node: 8.9.3
Yarn: 1.5.1
npm: 5.8.0
Watchman: 4.9.0

npmPackages:
styled-components:

wanted: 3.2.5
installed: 3.2.5

Steps to reproduce
Using gatsby.js with the gatsby-plugin-styled-components plugin, which uses babel-plugin-styled-components with the ssr: true flag to build the styles.
In development, i.e. running gatsby develop, the style tag in the head of the page with data-styled-components contains all the page style properties and the stylesheet is extracted correctly.
On production, the style tag is empty and the stylesheet is not extracted, but breaks with an error.
Expected Behavior
In production
By using __DO_NOT_USE_OR_YOU_WILL_BE_HAUNTED_BY_SPOOKY_GHOSTS from styled-components and getting the StyleSheet from there, I'm expecting that by running styleSheet.instance.toHTML() to get the style tag in a string.
Actual Behavior
In production
By running the toHTML method, it breaks with the following error:
Uncaught TypeError: Cannot read property 'cssText' of undefined

I could pinpoint that the error might be caused by trying to build the stylesheet using the data-styled-components style tag which is empty.

Ok, I finally managed to reproduce the issue in a new environment, since I cannot share the original repo. Check this out: https://github.com/pbogdan03/gatsby-styled-components-test. npm i and gatsby develop should work fine and display the generated styles, but gatsby build && gatsby serve should spit out an error.
I'm not sure what's causing it exactly. In either case, when trying to reproduce the issue and not seeing the error, the generated <style data-styled-components> tag was empty as opposed to the development build which had the used styles in there.
Hopefully running the build will make matters more clear.

https://github.com/pbogdan03/gatsby-styled-components-test/blob/master/src/layouts/Wrapper.js
What’s the intent of this? The Boolean value is explicitly telling the StyleSheet that this is a non-server environment. Also it’s probably not safe to reset the StyleSheet at a semi-random place of your code, since you’ll lose the registered components and hence their order and specificity

Resetting the stylesheet is something I've seen in a discussion here, anyway, removing it doesn't fix the issue I'm afraid. And the check is for the first pass of gatsby build which renders the static files and needs to be told which parts of the code need to be run only in a browser environment.

Ok, thanks for the explanation, I think I understand what you mean now.
I can't use the SSR API as I need to get the state of the rendered page at that moment, including viewport size etc. and save the information on the client side in real time. Gatsby isn't the issue here (I think). It works as intended, rendering the static files to be served to the client. I'm just using Gatsby for the blogging functionality, the rest of the app still needs to get the style properties in use at that moment and recreate the view for saving snapshots of it.
The children prop is used for testing purposes, to see of what's in the component at that moment, I'm not using it like that in the actual app.


Add Cookiebot scripts to codebase

Questions for Fountain

how big is the script in the head? Can its performance impact on the users be minimized?

Copy this script and insert it as the very first script within the HEAD-tag of your website to enable the consent banner.

Copy this script and insert it in the HTML on the specific page and at the exact position where you would like the Cookie Declaration to appear.

https://www.cookiebot.com/en/help/

https://support.cookiebot.com/hc/en-us/articles/360009192739-Google-Tag-Manager-and-Automatic-cookie-blocking

    Please

    accept marketing-cookies

    to watch this video.

YouTube requires you to [accept marketing cookies] to watch this video. Alternatively, you can watch the video on [here].


add confirmation dialog

Description

Added a feature related to an open issue of adding a confirmation dialog that allows user to confirm the deletion process.
Related Issue
Delete a task should have a confirmation dialog

Screenshots (if appropriate):

@all-contributors add @uKiJo for code

@uKiJo , dialog should be responsive and centered on small devices

@uKiJo , dialog should be responsive and centered on small devices

I think it is completely responsive, by centering, do you mean vertically?

@uKiJo , dialog should be responsive and centered on small devices

I think it is completely responsive, by centering, do you mean vertically?

Yup


Fix Routing

As a user, I want to be able to navigate between web pages.
Currently have to go to /topics/forum{id} to view forum. However, topics tiles show up on homepage so user should be able to access from there. I.e /forum{id}

Merged with Implement Routing #4
#4


Add some examples

We should have a folder for examples so people can see how this works.

As I posted this, I saw https://github.com/kubernetes-sigs/lws/tree/main/docs/example.
Maybe in the top level README, we should link to this so its easier to find.

Opened up #93.


contribution.md

i just create the contribution file add all the steps how to contribute and also clean the readme file.thats are looking quite messy

@Mukesh-ghildiyal You can create a PR mentioning the above changes.


Installation Error

Problem encountered on https://dotnet.microsoft.com/en-us/learn/dotnet/hello-world-tutorial/install
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
I have installed .NET 7.0 from https://dotnet.microsoft.com/download
When I use 'dotnet --info' I get this info:
Host:
Version:      7.0.4
Architecture: x86
Commit:       0a396acafe
.NET SDKs installed:
No SDKs were found.
.NET runtimes installed:
Microsoft.AspNetCore.App 6.0.15 [C:\Program Files (x86)\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 7.0.4 [C:\Program Files (x86)\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 5.0.12 [C:\Program Files (x86)\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.15 [C:\Program Files (x86)\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 7.0.4 [C:\Program Files (x86)\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 5.0.12 [C:\Program Files (x86)\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.15 [C:\Program Files (x86)\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 7.0.4 [C:\Program Files (x86)\dotnet\shared\Microsoft.WindowsDesktop.App]
I am unsure how this could be. Do I need to install the SDK to a certain location?

Hello you need to install .NET SDK from https://dotnet.microsoft.com/en-us/download . Currently you have installed .NET runtime instead of SDK.

Dup - https://github.com/dotnet/core/issues/8327.


fix type of keywords entry in frontmatter (in /registry/)

Describe the proposed changes
in /registry/ :
change keywords's type from array to string.
Project version
current stable
Related issue
contributes to #435
Signed-off-by: Gaetan de Villele<EMAIL_ADDRESS>

Deploy preview processing.
Built with commit 9f328ad482cfc3acdb54a1e91f9935f30fd76628
https://app.netlify.com/sites/docker-docs/deploys/582402f971e20a1b25e0db24

Thanks @gdevillele :)


[8.x] Allow api resource to not wrap when merging per resource

Currently, when merging your data is wrapped in data because of MergeValue#data, more info here:
https://github.com/laravel/framework/issues/21052
^, I tried doing the work around but then you end up with the key and null value, i dont want the key present at all when the merge constraint is false.
This PR allows a API resource to not wrap when merging.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.

@taylorotwell  -- can you provide a work around since you closed the PR? I'm not sure how i can avoid wrapping the return of mergeWhen with a data key

a consistent workaround i found was to create my own MergeValue and overwrite mergeWhen on my resource
use Illuminate\Http\Resources\MergeValue as LaravelMergeValue;

/**
 * This allows us to avoid wrapping the data returned from here by utilizing JsonSerializable.
 *
 * Class MergeValue
 * @package App\Http\Resources\Json
 */
class MergeValue extends LaravelMergeValue implements \JsonSerializable
{
    /**
     * @var mixed
     */
    public $data;

    /**
     * @return mixed
     */
    public function jsonSerialize()
    {
        return $this->data;
    }
}

@taylorotwell  -- is the thinking that this flexibility is not something the framework would like to support or is the approach in this PR not sufficient?


Simplify the handling of OOP problems

This PR also significantly speed up the tests.

Codecov Report

Merging #108 (f05e2e4) into main (9657b09) will increase coverage by 0.27%.
The diff coverage is 96.87%.

:exclamation: Current head f05e2e4 differs from pull request most recent head 9d495c8. Consider uploading reports for the commit 9d495c8 to get more accurate results

@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
+ Coverage   95.08%   95.36%   +0.27%
==========================================
  Files          24       25       +1
  Lines         977      949      -28
==========================================
- Hits          929      905      -24
+ Misses         48       44       -4

Impacted Files
Coverage Δ

src/ProbNumDiffEq.jl
75.00% <ø> (-1.48%)
:arrow_down:

src/initialization/classicsolverinit.jl
96.96% <85.71%> (+6.69%)
:arrow_up:

src/modelingtoolkitize.jl
100.00% <100.00%> (ø)

src/perform_step.jl
98.11% <100.00%> (-0.21%)
:arrow_down:

src/solve.jl
100.00% <100.00%> (ø)

src/caches.jl
97.10% <0.00%> (-2.90%)
:arrow_down:

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 9657b09...9d495c8. Read the comment docs.


Remove target option

Ref #2221. This simplifies svelte.config.js and src/app.html, and removes the tight coupling between them, by getting rid of the target option. Instead, the init script hydrates the init script's parentNode. As a bonus, it makes it easier to embed multiple SvelteKit apps on a single page, for the rare cases where that's necessary.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[x] Ideally, include a test that fails without this PR but passes with it.

Tests

[ ] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpx changeset and following the prompts. All changesets should be patch until SvelteKit 1.0

I presume that any other elements that were previously in the app.html body now need to be moved to the primary layout.svelte? Some of my sites are using an external service (Snipcart) that needs some configuration passed via divs and an external script tag. I usually put them in the app.html body, but this update seems to be clearing the body tag indiscriminately, even if I wrap the %svelte% placeholder in a nested div. Although perhaps there's something I've implemented incorrectly from my end.


X00T_DotOS_masumsaief

Please come with real account

Please come with real account

sir this is my real account
http://t.me/masumsaief00

Please come with real account

sir this is my real account,why i use fake account
http://t.me/masumsaief00

OK proceed


Question about the lemma 1.6 and code implementation

Hi. In your paper the lemma 1.6 is talking about the distribution of X and Y=sin(pi/2*X). We have known that the output of linear layer is in normal distribution if taking specific initialization method uniform(-c,c). However, in your code, the activation is just torch.sin(x) not torch.sin(pi/2 * x). Is there something I missed?

Hi. In your paper the lemma 1.6 is talking about the distribution of X and Y=sin(pi/2*X). We have known that the output of linear layer is in normal distribution if taking specific initialization method uniform(-c,c). However, in your code, the activation is just torch.sin(x) not torch.sin(pi/2 * x). Is there something I missed?

Same question, do you have any idea now?


Error loading layout files

Version
v2.3.2
Platform
MacOS Sequoia 15.0
What steps will reproduce the bug?

Install Deno v1.46.3 and Lume v2.3.2, Lume CMS v0.6.3
In the /src folder create a index.md file. In the front matter add layout: layouts/base.vto
Create the folder _includes/layouts and add the file base.vto with boilerplate html
Run lume serve

How often does it reproduce? Is there a required condition?
Every time.
What is the expected behavior?
Lume should create the file index.html in the _src directory using the base.vto template.
What do you see instead?
Lume throws an error: The layout file "/_includes/layouts/base.vto" doesn't exist
Here is the full output:
error: Uncaught (in worker "") (in promise) Error: Error rendering the layout of the page /index.md
            throw new Error(
                  ^
    at https://deno.land/x/lume@v2.3.2/core/renderer.ts:176:19
    at eventLoopTick (ext:core/01_core.js:175:7)
Caused by: Error: The layout file "/_includes/layouts/base.vto" doesn't exist
    at Renderer.#renderLayout (https://deno.land/x/lume@v2.3.2/core/renderer.ts:279:15)
    at https://deno.land/x/lume@v2.3.2/core/renderer.ts:159:52
    at concurrent (https://deno.land/x/lume@v2.3.2/core/utils/concurrent.ts:10:33)
    at eventLoopTick (ext:core/01_core.js:175:7)
    at async Renderer.renderPages (https://deno.land/x/lume@v2.3.2/core/renderer.ts:155:7)
    at async Site.#buildPages (https://deno.land/x/lume@v2.3.2/core/site.ts:641:5)
    at async Site.build (https://deno.land/x/lume@v2.3.2/core/site.ts:541:9)
    at async buildSite (https://deno.land/x/lume@v2.3.2/cli/utils.ts:48:3)
    at async onmessage (https://deno.land/x/lume@v2.3.2/cli/build_worker.ts:23:16)
error: Uncaught (in promise) Error: Unhandled error in child worker.
    at Worker.#pollControl (ext:runtime/11_workers.js:204:19)
    at eventLoopTick (ext:core/01_core.js:175:7)

Additional information
No response

Is the _includes folder inside the src folder?

Sorry, that was the issue.
It would be helpful to update this line in Getting Started > Create a Layout to be more clear:

Create a new directory _includes and the file layout.vto inside it.

To:

Create a new directory _includes inside the src directory. Create the file layout.vto inside it.


Fixed spelling errors

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Noticed some spelling errors and wanted to fix them to improve clarity.
Modification
Changed two spelling mistakes, should be obvious why.
BC-breaking (Optional)
N/A
Use cases (Optional)
N/A
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
The documentation has been modified accordingly, like docstring or example tutorials.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Dear mattcasey02,
First of all, we want to express our gratitude for your significant PR in the MMDetection project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
If you are Chinese or have WeChat，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. We look forward to seeing you there! Join us ：https://discord.gg/raweFPmdzG
Thank you again for your contribution❤
Best regards！ @mattcasey02


Feature/readme overview

Added:

animated overview gif images (needs update for verify)
descriptions for global Verify and Given methods
Changed:

updated readme with overview, examples, description of whole process

 Prepare basic readme documentation


Build Daily Intake Form Component

@dagaither @kingdonniemoe looks like this was taken care of with another issue. Can we close this?

Already added. No branch


Config sync no longer shows up in admin panel for Strapi 4.1.5, strapi-plugin-config-sync 1.0.0

Bug report
Describe the bug
After upgrading to strapi 4.1.5 and strapi-plugin-config-sync 1.0.0, Config sync no longer shows up in the admin panel.
Steps to reproduce the behavior

update from strapi 4.1.3 and strapi-plugin-config-sync 1.0.0-beta.8 to the versions mentioned above
build the admin panel: strapi build
start strapi
config sync no longer shows up in the admin panel 😞

The command line interface still works, though.
Expected behavior
Config sync still shows up in the Strapi admin panel.
System

Node.js version: 16.13.1
NPM version: 8.3.0
Strapi version: 4.1.5
Plugin version: 1.0.0
Database: sqlite
Operating system: Windows 11

Hey @alexdevmotion,
To have plugin be less invasive, I've moved the admin tool to the Settings in your Strapi app.
This plugin is more of a background tooling, so no need to have a dedicated menu item IMO 🙂

Good thinking and thank you for this amazing tool, I find it's really a must have for my Strapi apps.


Geolocation with feature display

Consider offering a convenient way to access and display geolocation on the map - https://openlayers.org/en/latest/examples/geolocation.html

User requirement was stated as follows:

Geoposition / Location anzeigen und anwenden
anwenden = automatically zoom to position on startup in ParcelMap, e.g. during field visits on tablet

@StefanBrand could this be solved with a map control? So the user clicks on a button within the map and is zoomed to the position?

Since we must support both mobile and in-house usage, we should start off with a button that users can push on demand.


Is --version flag no longer supported?

I was using code to check the exact version of an installed changeset CLI with changeset --version.
I'm not sure if there was an announcement, but I just realized that its behavior changed to be same as changeset version and broke my workflow.
Is that an intended change or a bug?

my guess is that it was accidentally broken by https://github.com/changesets/changesets/pull/1392 , cc @bluwy


La edit master bots how-to authentication

Docs Build status updates of commit 1250d4a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/bots/how-to/authentication/add-authentication.md
:warning:Warning

Details

msteams-platform/bots/how-to/authentication/auth-flow-bot.md
:white_check_mark:Succeeded

msteams-platform/bots/how-to/authentication/add-authentication.md

Line 77: [Warning] Invalid file link:(~/bots/how-to/concepts/build-and-test/app-studio-overview).

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.

Docs Build status updates of commit 2853696:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/bots/how-to/authentication/add-authentication.md
:warning:Warning

Details

msteams-platform/bots/how-to/authentication/auth-flow-bot.md
:white_check_mark:Succeeded

msteams-platform/bots/how-to/authentication/add-authentication.md

Line 77: [Warning] Invalid file link:(~/bots/how-to/concepts/build-and-test/app-studio-overview).

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.

Docs Build status updates of commit 56c638a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/bots/how-to/authentication/add-authentication.md
:warning:Warning

Details

msteams-platform/bots/how-to/authentication/auth-flow-bot.md
:white_check_mark:Succeeded

msteams-platform/bots/how-to/authentication/add-authentication.md

Line 77: [Warning] Invalid file link:(~/bots/how-to/concepts/build-and-test/app-studio-overview).

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.

Docs Build status updates of commit 79ac1cc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/authentication/add-authentication.md
:white_check_mark:Succeeded

msteams-platform/bots/how-to/authentication/auth-flow-bot.md
:white_check_mark:Succeeded

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.


[API] Inconsistency of layer/imagerySet config options

ol.source.MapQuest uses the layer option to configure the style, whereas ol.source.BingMaps uses imagerySet. ol.source.Stamen also uses layer.
I think all should use the same. What should be used? My preferred options would be style or layer. Other ideas?

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

With this ticket untouched for such a long time, I guess I'm happy enough with the way things are. Closing.


Quickview Modal - Clicking on Owner's name in Details tab is wrong

When you click on a Owner's name in the Details tab of the Quickview modal it should show you a list of all Apps that person has submitted.  Instead it currently (incorrectly) shows you a list of all of the Apps that you have submitted.

Issue moved to ozone-development/ozp-react-commons #51 via ZenHub.io


This should be Inbound Customer Journey but text and video is Outbound. Somethings wrong or not updated or wrong link to site???

BR /matts

Dokumentinformation
⚠ Redigera inte det här avsnittet. Det krävs för docs.microsoft.com ➟ länkning till GitHub-problem.

ID: e53d8830-0af8-cc74-3197-e48f2cbfccda
Version Independent ID: a7ad13f9-e0c9-21c9-b313-fd65ade4903d
Content: Inbound marketing with Dynamics 365 for Marketing
Content Source: ce/marketing/create-inbound-customer-journey.md
Service: dynamics-365-marketing
Product: unspecified
GitHub Login: @kamaybac
Microsoft Alias: kamaybac

The text actually does describe inbound journeys, though it starts by contrasting them with outbound journeys. The video was placed here for general reference about customer journeys, but I see now that it provides the wrong focus so I am going to remove it and reorder the opening paragraph.

Thank you, Matts, for reporting this issue.

NP, my pleasure…
/matts
Matts Eklund
Avega Group
mobil +46 (0) 70 632 05 44
matts.eklund<EMAIL_ADDRESS>Från: Karl Maybach<EMAIL_ADDRESS>Skickat: den 17 maj 2018 10:46
Till: MicrosoftDocs/dynamics-365-customer-engagement<EMAIL_ADDRESS>Kopia: Matts Eklund<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Ämne: Re: [MicrosoftDocs/dynamics-365-customer-engagement] This should be Inbound Customer Journey but text and video is Outbound. Somethings wrong or not updated or wrong link to site??? (#19)
Thank you, Matts, for reporting this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/dynamics-365-customer-engagement/issues/19#issuecomment-389792329, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AliwQH0Ssd2I1UO7i2gqZVn9_whD7oXQks5tzTirgaJpZM4UCkQK.

Hi Bishal,
I was assigned this, and closed it, 20 days ago. Is there something else I need to do here?
Thanks
. Karl
From: Bishal Goswami<EMAIL_ADDRESS>Sent: Wednesday, June 6, 2018 10:21 AM
To: MicrosoftDocs/dynamics-365-customer-engagement<EMAIL_ADDRESS>Cc: Karl Maybach<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/dynamics-365-customer-engagement] This should be Inbound Customer Journey but text and video is Outbound. Somethings wrong or not updated or wrong link to site??? (#19)
Assigned #19https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-customer-engagement%2Fissues%2F19&data=02|01|Karl.Maybach%40microsoft.com|8a2d9e9bb1ae42422d1f08d5cb86712a|72f988bf86f141af91ab2d7cd011db47|1|0|636638700650031744&sdata=gwCcF%2FZlcGNaSU2UMDfNxjtx%2F2V4SUnmfqlzxQkfNaE%3D&reserved=0 to @kamaybachttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkamaybac&data=02|01|Karl.Maybach%40microsoft.com|8a2d9e9bb1ae42422d1f08d5cb86712a|72f988bf86f141af91ab2d7cd011db47|1|0|636638700650041748&sdata=GRmZ9O5eSi3pm9bZLW4tS3nuQ%2FihIZ2JvLOoDBfrfdc%3D&reserved=0.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fdynamics-365-customer-engagement%2Fissues%2F19%23event-1665517517&data=02|01|Karl.Maybach%40microsoft.com|8a2d9e9bb1ae42422d1f08d5cb86712a|72f988bf86f141af91ab2d7cd011db47|1|0|636638700650041748&sdata=XZ7VfHS6pHoggB4bNxNApLeeoHJ7ETfgh9CNefp%2BMpE%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAZNw8RM0x5kyopPzXL-tVPaCgoasu4BCks5t55DugaJpZM4UCkQK&data=02|01|Karl.Maybach%40microsoft.com|8a2d9e9bb1ae42422d1f08d5cb86712a|72f988bf86f141af91ab2d7cd011db47|1|0|636638700650051753&sdata=8s2zbw%2Bmx8c6QHiNyyW8x69ntneCEoXTuA8X8jedWME%3D&reserved=0.


Broker does not terminate when test is over

Run a single test that starts the broker, such as VariableExplorerTest.SimpleDataTest. The broker process starts but does not terminate when test is over. Attempt to run another test fails b/c build in VS fails.

I suspect that this is happening for some tests but not the others. For the broker to be properly shut down, the test class must invoke Dispose on its RHostBrokerConnector instance. If it doesn't do so, the broker will hang around. So whenever you hit this, list the tests for which it's happening.
(The usual watchdog protection for shutting the host down automatically doesn't trigger, because it checks whether the parent process is still running - and the parent process in that case is VS test runner execution process, which is reused between test runs).

For this particular test (and any other test that uses VsRHostScript), the broker isn't created directly, but is rather requested via IRInteractiveWorkflowProvider:
                VsAppShell.Current.ExportProvider.GetExportedValue<IRSessionProvider>(),
                VsAppShell.Current.ExportProvider.GetExportedValue<IRInteractiveWorkflowProvider>().GetOrCreate().BrokerConnector,

So it's the provider that's responsible for disposing it. Which it will do if it's disposed itself, but this never happens in practice. I'm not sure what exactly is going wrong there - the provider is obtained via MEF, and my understanding is that MEF is responsible for disposing components that are disposable, when container is shutting down. But in this case, Dispose is simply never called. @AlexanderSher, are any of these assumptions wrong?

I'm not sure what exactly is going wrong there - the provider is obtained via MEF, and my understanding is that MEF is responsible for disposing components that are disposable, when container is shutting down.
The problem here is that there is no test fixture - MEF is initialized in TestVsAppShell singleton, so with these tests broker instance is shared between tests.

https://github.com/Microsoft/RTVS/commit/16bbfb33f3c03917ba44003ed507f46f3920cfac


[feature] Add structure overview

Hi, Just in case you want to spend some more time on this plugin: There is a feature from the GoYacc plugin that I'm missing in the Bison plugin, and that is to have a structural overview over the grammar.
The implementation in the GoYacc plugin looks reasonably short, and there is documentation for writing a StructureViewFactory, so it looks doable.
This would allow to see the grammar rules by pressing Ctrl+F12, and also to quickly jump to a particular grammar rule.

Done!


Algorithms, Code Snippets & Data Structures Implemented in Julia.

I did not find any code snippets for Julia Language.
It is a relatively new language and it is being used by many and it should be added.
I will make a PR to add this.

Reopening the issue for tracking other additions to Julia.


Feature Request: Stacked Notifications with Auto and Manual Dismissal in Bootstrap

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I have read the contributing guidelines

Proposal
Provide a new feature in Bootstrap called "Stacked Notifications" that allows users to easily create, manage, and dismiss multiple notifications that are stacked vertically on the screen.

Notification Stacking: Notifications will appear stacked on top of each other in the top-right corner of the screen, with the newest notification appearing at the top.
Automatic Dismissal: Notifications can be configured to automatically dismiss after a set amount of time.
Manual Dismissal: Each notification will include a close button that allows users to manually dismiss notifications.
Clear All Notifications: Provide a method to clear all active notifications at once.

Motivation and Context
Benefit:
This feature enhances user experience by providing a streamlined way to manage multiple notifications on a webpage. Users can quickly see and respond to notifications without them overlapping or being missed.
This feature is especially useful for:

Web Applications: Where multiple notifications may need to be displayed simultaneously, such as error messages, success alerts, or user prompts.
Improving Accessibility: Helps users who might be overwhelmed by multiple pop-ups or dialogs by providing a clear, organized notification system.
User Control: Users can easily manage notifications without being distracted by multiple elements on the screen.

Targeted Use Cases:

Developers: Can easily integrate a notification system without building one from scratch.
Users: Who need to manage alerts and messages efficiently in a clutter-free way.
Websites: That require real-time updates, such as dashboards, admin panels, or e-commerce sites.

By implementing this feature, Bootstrap can offer a robust notification management system that improves user interaction and accessibility.

Hey there, thanks for your contribution.
I feel like the feature request your asking for is partially existing already with Toasts (with autohide option set to false or a long delay) using the right placement on the Toast container.
So if I understand well, you are asking for a "dismiss all" button that would remove all the Toasts inside a given container. To my understanding, something like this plugged on a button would do the trick. I don't know if we really need this:
document.getElementById('toast-container').children.forEach(child => {
  Toast.getOrCreateInstance(child).hide();
});

Might also be a duplicate of #38424.


fix(files): rejecting file and other invalid url types

Goal
Reject items that are files, ftp, tel links as a bad user input error
https://pocket.sentry.io/issues/5612403011/?project=5761820&query=is%3Aunresolved issue.priority%3A[high%2C medium]&referrer=issue-stream&sort=date&statsPeriod=24h&stream_index=3

The only problem is that this won't let urls without http work -- e.g. getpocket.com alone or www.getpocket.com

@kschelonka its run after we auto append https to urls like that


Intercepting the response of the request

Issue/New Feature
Is there any way to intercept the response of all the request? Currently, the SDK provides a way to intercept the request and modify it before sending but can we also intercept after we got the response.
If this is not possible currently, then is there any chance to include this in the future roadmap?
Use-case
Suppose you are crawling some page with the virtual scrolling, now there is utils function to virtual scrolling but we don't have any way to fetch the data from the API calls. The data needs to be extracted from the HTML which is not as clean as the JSON API response.

I assume you're talking about PuppeteerCrawler since you mentioned scrolling. You can use Puppeteer itself to listen to the responses:
async function handlePageFunction({ page }) {
    page.on('response', async (httpResponse) => {
        // filter for responses you're interested in, e.g.:
        if (!httpResponse.url().includes('data')) return;
        // do something with the response, for example:
        const data = await httpResponse.json();
        await processData(data);
    }
}

If you want to capture all responses, you might want to set this listener up in the gotoFunction already. Hope this helps.

Thank you @mnmkng

Hi guys,
Sorry to post in a closed issue. I had similar problem, I needed to fetch the response from a graphQL query (ajax POST request) in a page visited via crawler. In my case it's Playwright crawler, but it's very similar to Puppeteer.
Also, since 2020 SDK was updated and gotoFunction was replaced with navigation hooks. Registering listeners inside handlePageFunction doesn't seem to work, or at least I was not able to make it run properly, Instead I used with gotoFunction approach. Sharing the snippet, maybe it will help others:
const crawler = new Apify.PlaywrightCrawler({
  preNavigationHooks: [
    async ({page}) =>
      page.on('response', async response => {
        if (response.request().url().includes('data')) {
          const data = await response.json(); // ajax response
        }
      })],
  handlePageFunction: async ({request, page}) => {
    // your code
    await page.waitForLoadState('networkidle') // wait until all ajax calls are made in page
  }
});


UTF8 - IO Encoding

This is a bit of a weird one since it's a general python problem, but it also affects this tool.
$ echo "øl"
øl
$ echo "øl" > test
$ jinja2 test
øl
$ jinja2 test > test2
Traceback (most recent call last):
  File "/bin/jinja2", line 11, in <module>
    load_entry_point('jinja2-cli==0.7.0', 'console_scripts', 'jinja2')()
  File "/usr/lib/python2.7/site-packages/jinja2cli/cli.py", line 364, in main
    sys.exit(cli(opts, args))
  File "/usr/lib/python2.7/site-packages/jinja2cli/cli.py", line 290, in cli
    sys.stdout.write(output)
UnicodeEncodeError: 'ascii' codec can't encode character u'\xf8' in position 0: ordinal not in range(128)

and workaround this is
$ export PYTHONIOENCODING=utf-8
$ jinja2 test > test2
$ cat test2
øl

This is explored here https://stackoverflow.com/q/1473577/558194
So, to be a general Unix tool this should ideally support UTF-8, thus I would suggest implementing something to address potential issues.

Thanks for the report, I had some logic backwards. :)


pg_hba error with 9.4

Hi, I'm trying to install Postgres 9.4. However, the server recipe is trying to access pg_hba in a version 9.3 directory structure.
 Any errors should be visible in the
output above. Please fix your recipes so that they properly complete.

This is working for me now. Thanks!


[TME-959] Fix data-kjb-disable-search placement

Description
JIRA: TME-959
In SageSearch, we added the attribute of data-kjb-disable-search to  <form> to "Prevent the default Kajabi-Products Search.js from binding to this input"; however, in products' search, we are checking for data-kjb-disable-search on an input instead.
Screenshots
n/a
Test notes
This should be tested against products
1.) Go to https://www.kajabi.test/admin/sites/1/landing_pages?website_page=all, and type Home
2.) Click outside of the text input and it should no longer auto submit
3.) Go back to the search input and press enter, and it should now submit.
Estimated impact
Fixes bug where in products, any page that was using the SageSearch component was autosubmitting when it should not have been (users will now have to press enter to submit the form)

[ ] Go to https://www.kajabi.test/admin/sites/1/products and search for a product.  It should not execute query until you press enter

@philschanely yes.  it seems like this was always on (but just on the wrong element).  i also just reverted the work i did here because I thought that was the initial fix but the search.js file here already takes care of not executing search correctly when canceling

@darrellrivera Ah ok!


🪸 Topic overview - subscriptions (hashtag1150)

hashtag What is currently missing?
![Screenshot 2023-05-15 at 11 54 07](
hashtag Questions
hashtaghashtaghashtag UI

Is "Grouped by teams" necessary when the data is displayed in a table which has a "Filter by teams" feature?

Not necessary, render all without grouping in table

Should the User Subs / Prefixed Subs and TxnID subs options be tabs, rather than a filter dropdown like other ways to organize the data?

should be tabs because the data rendered in the table is different, designs to come

what is the checkmark icon representing in the table? The only action available there is to delete the ACL.

it was an extra feature to allow users to approve pending ACLs from this screen, but will not be part of this epic

in the Angular app, there are options to view service account password and offset info, but not in the new design. Are we not implementing those features?

we will, but designs are to come
![Screenshot 2023-05-15 at 14 33 56](
![Screenshot 2023-05-15 at 14 33 59](

hashtaghashtaghashtag API

There is no param exposed by the getTopicOverview endpoint to filter by ACL Type, team, or any search. These could be implemented only in the front end but only if the aclInfoList, prefixedAclInfoList and transactionalAclInfoList are not paginated. Their data type is AclInfo[], and I do see pagination properties on AclInfo. However, I do not see pagination info in the payload returned from that endpoint in staging (even for an array with > 10 elements):
![Screenshot 2023-05-15 at 14 31 44](

that data is never paginated

There is no "Time of request" in the ACL info data, but it is shown in the designs
All properties are optional in the payload we get from getTopicOverview
AclInfo has ccl_ssl and acl_ssls / acl_ip and acl_ips. What is the difference? Should there be only one property for it?

hashtag TODO

[x] hashtag1228
[x] hashtag1229
[x] hashtag1230
[x] hashtag1231
[x] hashtag1232

That's done right?? 🎉  Will close the epic!


Error on Android with Angular and APP_INITIALIZER

Which platform(s) does your issue occur on?

Android
@nativesrcript/android 7.2.0 w api-level 30
emulator

Please, provide the following version numbers that your issue occurs with:

CLI: 7.2.1
Cross-platform modules: 7.1.4
Runtime(s): 7.0.1 (android)

Please, tell us how to recreate the issue in as much detail as possible.
When using the plugin in a nativescript angular project with an async APP_INITIALIZER the app dies due to the early call to installGestures() here: https://github.com/nativescript-community/ui-drawer/blob/master/src/index.ts#L14
Proposed solution is to remove this call as it is called in the install() function exported by this plugin.
To reproduce:

Create a blank ns-angular app.
Add the plugin
Add an async APP_INITIALIZER & launch screen as described here: https://blog.nativescript.org/angular-launch-animations/
Run on Android emulator/device.

Error is that a view/frame doesn't exist when installGestures() is called, causes app death.
Workaround for me for now has been to back off to the RadSideDrawer plugin.
Is there any code involved?
Will create demo and get it in here.

Exact error is: JS: ERROR Error: a page is needed to attach a gesture
Demo is here: https://github.com/jamescodesthings/demo-ui-drawer-issue-5

@jamescodesthings your error seems to come from somewhere else. I mean i think you need to call installGestures again.
The error you extracted means the install method has NOT been called.
Why is the drawer plugin required from the launch screen? makes no sense and i think should not happen.

Closing. Cheers @farfromrefug
The issue for me was a bit of a strange one but I think one of the problems here was I'd oversimplified my demo repo.
I've updated the demo repo with it working as expected now. Cheers.


License

Hi, I've just used this, in a new project. It's just for learning, but I let my friends used too.
I'd like to know if I have to put something inside my project, like a link to your repository or if I have to say that it's not belong to me, or something like that.

Hello. This project follows the MIT License.
In my opinion, other than that, you can put a reference to NES.css if you want, but you don't have to.
I will keep this issue open for a while if anyone else would like to say or add anything else to this 👍

@guastallaigor , I agree with you 👍
@Flip32 Enjoy using NES.css 🎮
And if you think NES.css is a good product, I would apprecate it if you could recommend it to everyone around you. 😆


🚧 [WIP] Community Events

Let's discuss what Outreach team can do for the Node.js Community Events.

Community events goal includes Node.js related meetups, conferences, workshops, and more. We are currently figuring out ways to promote Node speakers, events, and build consistent meetups workflow to increase participation in such events.

Let's kick start the discussion around the following topics:

Should we do an Org account on Meetups.com
Meetup in a Box
Make a workflow that's inclusive of current Node.js related meetups

For "meetups in a box" we don't need to start from scratch.
We can create prototypes of successful Node.js meetups by analysing the formats the folks use here and make it easier for new meetups to reproduce the format they choose.

If you would like I can connect this team to the organizers of the Node.js Israel monthly meetup :]

We can even improve the currently existing meetup formats and then integrate with the meetups in a box plan. ✌

@benjamingr that would be rad. Are they on the Node.js Slack group?

UPDATE: @amiller-gh discussed with folks from Meetup.com, and they refused to offer non-profit pricing which is a bummer.
We are now thinking about having a country or statewide meetups instead. So, that could only be less than 300 meetups at the most. Where we could ask different meetup groups to show up as well people from any city would be able to do meetups without having to worry about paying $30/month.
This will also help us keep these meetups more consistent. But even these meetups would not be head-to-head with already running meetup programs.


want to add fresh quotes extension

this extension give you a fresh quotes
i make pull request pls review it mam

Hey please assign this issue to me as I have a great knowledge of styling so I will be obliged to represent the best extension for this.

I am assigning the issue to @adi2002vicky since he has posted the issues first. @Atharv-110 sure you can take up the modification after @adi2002vicky submit a pr or you can add another issue

@ridsuteri i got that thanks

@adi2002vicky are you working on this? or should I assign it to someone else?

I can work on it if you allow me @ridsuteri

pls check pr sir @ridsuteri

pls check pr sir @ridsuteri

since your last PR was plagiarised, I am assigning this issue to @Atharv-110 now

any updates @Atharv-110 ?

Yes @ridsuteri I need few days like 3 to 4 as my mid semester examinations are going on.

@Atharv-110 working on it or should i close this issue?

@ridsuteri till when do you expect me to do this as i am stuck in some urgent work for this week?

@ridsuteri till when do you expect me to do this as i am stuck in some urgent work for this week?

Umm the earlier the better, maybe submit it after this week!

@ridsuteri till when do you expect me to do this as i am stuck in some urgent work for this week?

Umm the earlier the better, maybe submit it after this week!

@ridsuteri  If this task is being delayed, then I'd be glad to work on it. I am a GSSoC'22 contributor.

go ahead @Sukriti-m !


Allow POST to /delay

The /delay endpoint currently returns a 405 (method not allowed) on POST. Any chance you could open it up to allow it? Thanks.

Can you enlighten us as to why this is necessary?

We've done this for our version running at yourapihere.com. I'm personally
a fan of opening up any of the non-method endpoints to allow any method. I
just haven't gotten around to it yet.
The use case is that many POST requests take awhile to process data and
this would be able to simulate that.
+1 for the request if you want to do a pull req
On Thu, Apr 23, 2015 at 2:29 PM, Ian Cordasco<EMAIL_ADDRESS>wrote:

Can you enlighten us as to why this is necessary?
—
Reply to this email directly or view it on GitHub
https://github.com/Runscope/httpbin/issues/224#issuecomment-95724318.

@johnsheehan sure, I guess I'm just trying to understand what is trying to be tested. Every HTTP library I've used across a couple different languages doesn't care if there's a delay in using a specific method. So, since HTTPBin is usually used to test things, I'm trying to understand the behaviour being tested.

Thanks guys. @sigmavirus24 I agree that the method shouldn't matter in a delay/timeout situation, I'm just trying to prove it to settle a bug report.
Thanks for this service, it's been extremely valuable in testing my project. Glad to contribute a pr, thought I'd need to learn me a little python but it looks pretty clear.

@tmenier yeah it makes sense to me. FWIW, if you're going to send the PR you'll be changing this line to look a bit more like this other line. You will need to specifically mention 'GET' in your paramter to methods though, otherwise you'll break the existing API.
Cheers,

Thanks for the tip. For completeness there's probably several other endpoints where this change might make sense for the same reasons, but I only did /delay for this PR.


Fix unguarded availability warnings in Messaging

Split from #385
Xcode 9 introduces warnings for unguarded API calls introduced in later versions.
Occurrences show up in Auth and Messaging:

$ pod spec lint FirebaseCommunity.podspec
-> FirebaseCommunity (0.1.4)
- WARN  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthAppDelegateProxy.m:122:10: warning: 'UIApplicationOpenURLOptionsAnnotationKey' is only available on iOS 9.0 or newer [-Wunguarded-availability]
- NOTE  | [FirebaseCommunity/Auth] xcodebuild:  /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator11.0.sdk/System/Library/Frameworks/UIKit.framework/Headers/UIApplication.h:535:51: note: 'UIApplicationOpenURLOptionsAnnotationKey' has been explicitly marked partial here
- NOTE  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthAppDelegateProxy.m:122:10: note: enclose 'UIApplicationOpenURLOptionsAnnotationKey' in an @available check to silence this warning
- WARN  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthURLPresenter.m:47:3: warning: 'SFSafariViewController' is partial: introduced in iOS 9.0 [-Wunguarded-availability]
- NOTE  | [FirebaseCommunity/Auth] xcodebuild:  /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator11.0.sdk/System/Library/Frameworks/SafariServices.framework/Headers/SFSafariViewController.h:27:12: note: 'SFSafariViewController' has been explicitly marked partial here
- NOTE  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthURLPresenter.h:41:12: note: annotate 'FIRAuthURLPresenter' with an availability attribute to silence
- WARN  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthURLPresenter.m:104:40: warning: 'SFSafariViewController' is partial: introduced in iOS 9.0 [-Wunguarded-availability]
- NOTE  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthURLPresenter.m:104:1: note: annotate 'safariViewControllerDidFinish:' with an availability attribute to silence
- WARN  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthURLPresenter.m:80:10: warning: 'SFSafariViewController' is only available on iOS 9.0 or newer [-Wunguarded-availability]
- NOTE  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthURLPresenter.m:80:10: note: enclose 'SFSafariViewController' in an @available check to silence this warning
- WARN  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthURLPresenter.m:81:33: warning: 'SFSafariViewController' is only available on iOS 9.0 or newer [-Wunguarded-availability]
- NOTE  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthURLPresenter.m:81:33: note: enclose 'SFSafariViewController' in an @available check to silence this warning
- WARN  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthURLPresenter.m:164:3: warning: 'SFSafariViewController' is only available on iOS 9.0 or newer [-Wunguarded-availability]
- NOTE  | [FirebaseCommunity/Auth] xcodebuild:  FirebaseCommunity/Firebase/Auth/Source/FIRAuthURLPresenter.m:164:3: note: enclose 'SFSafariViewController' in an @available check to silence this warning
- WARN  | [FirebaseCommunity/Database] xcodebuild:  leveldb-library/db/c.cc:274:56: warning: possible misuse of comma operator here [-Wcomma]
- NOTE  | [FirebaseCommunity/Database] xcodebuild:  leveldb-library/db/c.cc:274:21: note: cast expression to void to silence warning
- WARN  | [FirebaseCommunity/Database] xcodebuild:  leveldb-library/db/c.cc:275:56: warning: possible misuse of comma operator here [-Wcomma]
- NOTE  | [FirebaseCommunity/Database] xcodebuild:  leveldb-library/db/c.cc:275:21: note: cast expression to void to silence warning
- WARN  | [FirebaseCommunity/Database] xcodebuild:  leveldb-library/db/version_set.cc:1350:5: warning: code will never be executed [-Wunreachable-code]
- NOTE  | [FirebaseCommunity/Database] xcodebuild:  leveldb-library/db/version_set.cc:1349:7: note: silence by adding parentheses to mark code as explicitly dead
- WARN  | [FirebaseCommunity/Messaging] xcodebuild:  FirebaseCommunity/Firebase/Messaging/FIRMessaging.m:407:18: warning: 'application:openURL:options:' is only available on iOS 9.0 or newer [-Wunguarded-availability]
- NOTE  | [FirebaseCommunity/Messaging] xcodebuild:  /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator11.0.sdk/System/Library/Frameworks/UIKit.framework/Headers/UIApplication.h:326:1: note: 'application:openURL:options:' has been explicitly marked partial here
- NOTE  | [FirebaseCommunity/Messaging] xcodebuild:  FirebaseCommunity/Firebase/Messaging/FIRMessaging.m:407:18: note: enclose 'application:openURL:options:' in an @available check to silence this warning
- WARN  | [FirebaseCommunity/Messaging] xcodebuild:  FirebaseCommunity/Firebase/Messaging/FIRMessaging.m:699:20: warning: 'messaging:didReceiveMessage:' is only available on iOS 10.0 or newer [-Wunguarded-availability]
- NOTE  | [FirebaseCommunity/Messaging] xcodebuild:  FirebaseCommunity/Firebase/Messaging/Public/FIRMessaging.h:260:1: note: 'messaging:didReceiveMessage:' has been explicitly marked partial here
- NOTE  | [FirebaseCommunity/Messaging] xcodebuild:  FirebaseCommunity/Firebase/Messaging/FIRMessaging.m:699:20: note: enclose 'messaging:didReceiveMessage:' in an @available check to silence this warning
- WARN  | [FirebaseCommunity/Messaging] xcodebuild:  FirebaseCommunity/Firebase/Messaging/FIRMessagingContextManagerService.m:147:20: warning: 'setAlertTitle:' is only available on iOS 8.2 or newer [-Wunguarded-availability]
- NOTE  | [FirebaseCommunity/Messaging] xcodebuild:  /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator11.0.sdk/System/Library/Frameworks/UIKit.framework/Headers/UILocalNotification.h:44:47: note: 'setAlertTitle:' has been explicitly marked partial here
- NOTE  | [FirebaseCommunity/Messaging] xcodebuild:  FirebaseCommunity/Firebase/Messaging/FIRMessagingContextManagerService.m:147:20: note: enclose 'setAlertTitle:' in an @available check to silence this warning
- WARN  | [FirebaseCommunity/Messaging] xcodebuild:  FirebaseCommunity/Firebase/Messaging/FIRMessagingRmq2PersistentStore.m:107:26: warning: 'sqlite3_errstr' is only available on iOS 8.2 or newer [-Wunguarded-availability]
- NOTE  | [FirebaseCommunity/Messaging] xcodebuild:  /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator11.0.sdk/usr/include/sqlite3.h:3385:24: note: 'sqlite3_errstr' has been explicitly marked partial here
- NOTE  | [FirebaseCommunity/Messaging] xcodebuild:  FirebaseCommunity/Firebase/Messaging/FIRMessagingRmq2PersistentStore.m:107:26: note: enclose 'sqlite3_errstr' in an @available check to silence this warning
Analyzed 1 podspec.

Had to roll the Messaging change back for the 4.8.0 release due to @available only being available in Xcode 9.
TODO:  use +#if defined(__IPHONE_11_0) && (__IPHONE_OS_VERSION_MAX_ALLOWED >= __IPHONE_11_0) or something like that to check if the @available function is available. Otherwise, break out into helper functions.


A joint statement from partners from 18 EU member states

PQC-joint-statement.pdf
Thank you for assembling this.  You may want to add this document and the links on the last page.

Thank you for suggestion. The link to pdf you mentioned is already included in the list. BTW I have added some links from the last page.


[EASY] Standardize set_seed()

Method set_seed() was added to mtl.utils and was redefined as a class method in a test and Classifier(). Now all use the version in snorkel.model.utils.

Good call, formal type check now.


Support Carthage

Publish the component in Cathage as well. That could solve dependencies issues.

Scheme marked as shared so Carthage should be able to use it.


Fix travis flake8 version

This change prevent travis from installing flake8-3.x which is incompatible
with python-2.6

Thanks!


Set up project to use docker

Use docker to containerize the application

Docker and docker-compose setup already up and running.


Require contiguous qubits on circuits submitted to ionq devices

Fixes #39 .
The ionq backend doesn't allow us to insert no-ops anymore so we can't work around this restriction.
It means we can't attempt noise-aware placement, but this wasn't allowed anyway on ionq devices.

Looks like something we should put out in a release soon?

Do you know if any examples are effected by this?

Looks like something we should put out in a release soon?

Yes, I will do that next.

Do you know if any examples are effected by this?

No, none of the examples currently use braket.


attrs does not remove props

The current implementation of styled components in next-yak may is not correctly handling prop removal when using the attrs method. styled-components removes props when they are explicitly set to undefined in the attrs method, and we should ensure next-yak implements this behavior correctly.
Expected Behavior
Based on styled-components' implementation:

Props returned from attrs should be merged with the props passed to the component.
Props not returned from attrs should retain their original values.
Props explicitly set to undefined in attrs should be removed from the final props passed to the DOM element.

Example of Desired Behavior
Here's how the prop handling should work:
const StyledSvg = styled.svg.attrs<IStyledSvgProps>((
  { noColor, className, ...props }
) => ({
  className: className ? `StyledSvg ${className}` : "StyledSvg",
  // Explicitly set noColor to undefined to remove it
  noColor: undefined,
  $noColor: noColor,
  ...props
}))`
  display: inline-block;
  vertical-align: middle;
  pointer-events: none;
  fill: currentcolor;

  ${({ $noColor }) =>
    !$noColor &&
    css`
      fill: currentcolor;
    `}
`;

When used like this:
<StyledSvg noColor={true} />

The noColor prop should not be passed to the DOM, avoiding React warnings like:
Warning: React does not recognize the `noColor` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `nocolor` instead. If you accidentally passed it from a parent component, remove it from the DOM element.

After further research we found out that styled component wants to allow children to overwrite there parent props.
In the following example <Child /> should render tab-index=0 and type="submit":
const Parent = styled.button.attrs(() => ({
  type: "button",
  tabIndex: 0,
}))``;
const Child = styled(Parent).attrs(() => ({
  type: "submit",
}))``;

styled-components manages this by calling the attrs function of <Parent /> before the attrs function of <Child />


Yarn Issue

Couldn't start resource seatbelt
[         script:yarn] [yarn]	yarn install v1.22.5
[         script:yarn] [yarn]	[1/4] Resolving packages...
[         script:yarn] [yarn]	[2/4] Fetching packages...
[         script:yarn] Error: [yarn]	error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "4.18.0 || 6.14.0 || >=18.0.0". Got "16.9.1"
[         script:yarn] Error: [yarn]	error Found incompatible module.
Tried these commands:
yarn upgrade
yarn install
no luck

Duplicate of #14


Call to undefined method mergeConfigFrom

I installed "pragmarx/tracker": "2.0.10", on Laravel 4.2 after adding 'PragmaRX\Tracker\Vendor\Laravel\ServiceProvider', to the providers, i got
Call to undefined method PragmaRX\Tracker\Vendor\Laravel\ServiceProvider::mergeConfigFrom()

Am i doing something wrong?

i have same error :((


⚠ ETF has degraded performance

In 9298039, ETF (https://markets.hankyung.com/equity-product/etf) experienced degraded performance:

HTTP code: 200
Response time: 18118 ms

Resolved: 신규상장 performance has improved in f545279.


How to use ubi-boringssl builds?

Describe the solution you'd like
Either fix the ubi-boringssl builds, or document how to run these builds.
What is the added value?
Functionality with FIPS 140-2 compliant builds.
Give us examples of the outcome
Basic functionality.
Observations (Constraints, Context, etc):
Currently using v0.7.1 works great. But if you try to use v0.7.1-ubi-boringssl, all 3 pods (ES, cert controller, webhook) will error out with
cert-controller /bin/external-secrets: /lib64/libc.so.6: version `GLIBC_2.34' not found (required by /bin/external-secrets)

Is there additional helm values we need to be passing in to utilize these UBI images? Is anyone actually using these currently?

The ubi-boringssl images are quite fresh. It could be that there is a libc version mismatch, as the binary is not built inside a container. :thinking:


Improve documentation for Dropout and rngs argument in linen.Module.apply()

Here is an example of Dropout in a model definition:
https://github.com/google/flax/blob/d068512a932da3e05b822790a591bac391aeab36/examples/nlp_seq/models.py#L211
Here is the apply(), where rngs is passed in
https://github.com/google/flax/blob/d068512a932da3e05b822790a591bac391aeab36/examples/nlp_seq/train.py#L206-L207
However the rng is not very clearly explained in apply()
https://github.com/google/flax/blob/615f40be774e7ed66fd344e8291ac0d48ebcef7d/flax/linen/module.py#L749
The rngs seems to be passed to flax/core/scope.py
Here is the code for Dropout (linen)
https://github.com/google/flax/blob/9b4807840c5cb26ef5e29028e3558d404aee00a0/flax/linen/stochastic.py#L56-L57
Here is the code for  make_rng()
https://github.com/google/flax/blob/615f40be774e7ed66fd344e8291ac0d48ebcef7d/flax/core/scope.py#L441-L447
The documentation for rngs in apply() should have a (pointer to) list of names of possible rngs
And documentation for Dropout should mention how to pass in rng using apply(), without directly passing in like Dropout()(x,rng=rng).
Also probably need to mention the make_rng() fold_in the rng so each dropout layer will use different rng if there are multiple dropout layers.

We could mention that Dropout() requires an rng with the name dropout in its module documentation. The code is currently very short and it's easily visible, but I agree it would be better discoverable if it was mentioned in the class pydoc as well.
I also think that extending the Module.apply() could be extended with something like
    rngs: The rngs for the variable collections. For example :class:`flax.linen.stochastic.Dropout`
      requires an additional rng named `dropout`.

would be reasonable, since this is quite common and illustrative for newcomers.
WDYT @marcvanzee who was been thinking about the right level of verbosity in our docs
@cccntu why don't you give it a try updating the documentation and sending a PR?

@andsteing  Thanks for the suggestion. I will give it a try in a few days. :)


Add Pagination to Triple and Reification tables (and once they exist, also on Vocabulary tables)

This had been implemented in apis vanilla before, so it can be either taken from there or implemented from scratch.

already implemented (at least for triples)


Add API to access consumer metrics.

Here is my initial proposal to expose consumer metrics.
WDYT?

I have updated my PR based on the comments from the previous version.
If you decide to merge this are there any chances for backporting to 1.2 or 1.3. I know its an api change, but 1.3 is not RC, yet?

We can backport it to 1.3 without any problems; 1.3 will released on the same schedule as 2.0.
We can also backport it to 1.2, if necessary, but only on the concrete container classes (leaving the interface unchanged).

The Travis report says:
[ant:checkstyle] [ERROR] /home/travis/build/spring-projects/spring-kafka/spring-kafka/src/main/java/org/springframework/kafka/listener/MessageListenerContainer.java:49: Line has leading space characters; indentation should be performed with tabs only. [RegexpSinglelineJava]

The indents must always be only tabs.

Merged as https://github.com/spring-projects/spring-kafka/commit/1945f29d5518e3c4a9950ba82135420dfb61e808 and cherry-picked to 1.3.x.
I have polished JavaDocs a bit, add your name to the @author list and mentioned feature in the Docs.
Thank you for the contribution.
Looking forward for more!


details() helper is not reactive

I noticed that the details()isn't updated on mount, nor does it react when details are changed.
Repro:
https://stackblitz.com/edit/solidjs-templates-tbsg3q?file=src/App.tsx

@ivancuric I assume with you. details() retuns onMount an empty object. That's impratical.

@ivancuric that's a valid concern. My original intention was simply to return the track details from the slider instance. I can see how this would cause confusion b/c it's an accessor so the assumption is that there's a signal available.
I don't really see the need to track all the slider details for the 90% use case of a slider. I kind of leave it up to the user to build their reactivity needs around it.
I'd love feedback and some opinions on this though. I could be wrong!
I'll close the issue for now but definitely open to feedback.