Update link to Awesome Mastodon

url to list: https://github.com/hyperupcall/awesome-mastodon
explanation: Mastodon is a very popular decentralized platform for hosting a social networking platform. It's built on top of ActivityPub and has cemented it's legacy as an alternative social media platform since the hostile takeover of Twitter last year.
This does not add a new list; it updates the link to Awesome Mastodon. Unicorn.
The hueyy fork is reasonably popular and seems to be actively maintained.
I was somewhat confused since the PR template implies all PRs are requests for new awesome lists. Which means this pull request title is apparently invalid. I wasn't sure if reviews  were necessary in this case (looking at #2459 for instructions), so I did it anyways:
Requirements for your pull request

[x] Don't open a Draft / WIP pull request while you work on the guidelines. A pull request should be 100% ready and should adhere to all the guidelines when you open it. Instead use #2242 for incubation visibility.
[x] Don't waste my time. Do a good job, adhere to all the guidelines, and be responsive.
[x] You have to review at least 2 other open pull requests.
Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration. Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions. Comments pointing out lint violation are allowed, but does not count as a review.
[x] You have read and understood the instructions for creating a list.
(N/A, changing/update link, not adding) [] This pull request has a title in the format Add Name of List. It should not contain the word Awesome.

✅ Add Swift
✅ Add Software Architecture
❌ Update readme.md
❌ Add Awesome Swift
❌ Add swift
❌ add Swift
❌ Adding Swift
❌ Added Swift

[x] Your entry here should include a short description about the project/theme of the list. It should not describe the list itself. The first character should be uppercase and the description should end in a dot. It should be an objective description and not a tagline or marketing blurb.

✅ - [iOS](…) - Mobile operating system for Apple phones and tablets.
✅ - [Framer](…) - Prototyping interactive UI designs.
❌ - [iOS](…) - Resources and tools for iOS development.
❌ - [Framer](…)
❌ - [Framer](…) - prototyping interactive UI designs

[x] Your entry should be added at the bottom of the appropriate category.
[x] The title of your entry should be title-cased and the URL to your list should end in #readme.

Example: - [Software Architecture](https://github.com/simskij/awesome-software-architecture#readme) - The discipline of designing and building software.

[x] No blockchain-related lists.
[x] The suggested Awesome list complies with the below requirements.

Requirements for your Awesome list

[x] Has been around for at least 30 days.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
[x] Run awesome-lint on your list and fix the reported issues. If there are false-positives or things that cannot/shouldn't be fixed, please report it.
[x] The default branch should be named main, not master.
[x] Includes a succinct description of the project/theme at the top of the readme. (Example)

✅ Mobile operating system for Apple phones and tablets.
✅ Prototyping interactive UI designs.
❌ Resources and tools for iOS development.
❌ Awesome Framer packages and tools.

[x] It's the result of hard work and the best I could possibly produce.
If you have not put in considerable effort into your list, your pull request will be immediately closed.
[x] The repo name of your list should be in lowercase slug format: awesome-name-of-list.

✅ awesome-swift
✅ awesome-web-typography
❌ awesome-Swift
❌ AwesomeWebTypography

[x] The heading title of your list should be in title case format: # Awesome Name of List.

✅ # Awesome Swift
✅ # Awesome Web Typography
❌ # awesome-swift
❌ # AwesomeSwift

[x] Non-generated Markdown file in a GitHub repo.
[x] The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.
[x] Not a duplicate. Please search for existing submissions.
[x] Only has awesome items. Awesome lists are curations of the best, not everything.
[x] Does not contain items that are unmaintained, has archived repo, deprecated, or missing docs. If you really need to include such items, they should be in a separate Markdown file.
[x] Includes a project logo/illustration whenever possible.

Either centered, fullwidth, or placed at the top-right of the readme. (Example)
The image should link to the project website or any relevant website.
The image should be high-DPI. Set it to maximum half the width of the original image.

[x] Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
[x] Includes the Awesome badge.

[x] Should be placed on the right side of the readme heading.

Can be placed centered if the list has a centered graphics header.

[x] Should link back to this list.

[x] Has a Table of Contents section.

[x] Should be named Contents, not Table of Contents.
[x] Should be the first section in the list.
[x] Should only have one level of nested lists, preferably none.
[x] Must not feature Contributing or Footnotes sections.

[x] Has an appropriate license.

We strongly recommend the CC0 license, but any Creative Commons license will work.

[x] Tip: You can quickly add it to your repo by going to this URL: https://github.com/<user>/<repo>/community/license/new?branch=main&template=cc0-1.0 (replace <user> and <repo> accordingly).

[x] A code license like MIT, BSD, Apache, GPL, etc, is not acceptable. Neither are WTFPL and Unlicense.
[x] Place a file named license or LICENSE in the repo root with the license text.
[x] Do not add the license name, text, or a Licence section to the readme. GitHub already shows the license name and link to the full text at the top of the repo.
[x] To verify that you've read all the guidelines, please comment on your pull request with just the word unicorn.

[x] Has contribution guidelines.

[x] The file should be named contributing.md. Casing is up to you.
[x] It can optionally be linked from the readme in a dedicated section titled Contributing, positioned at the top or bottom of the main content.
[x] The section should not appear in the Table of Contents.

[x] All non-important but necessary content (like extra copyright notices, hyperlinks to sources, pointers to expansive content, etc) should be grouped in a Footnotes section at the bottom of the readme. The section should not be present in the Table of Contents.
[x] Has consistent formatting and proper spelling/grammar.

[x] The link and description are separated by a dash. Example: - [AVA](…) - JavaScript test runner.
[x] The description starts with an uppercase character and ends with a period.
[x] Consistent and correct naming. For example, Node.js, not NodeJS or node.js.

[x] Does not use hard-wrapping.
[x] Does not include a CI (e.g. GitHub Actions) badge.You can still use a CI for linting, but the badge has no value in the readme.
[x] Does not include an Inspired by awesome-foo or Inspired by the Awesome project kinda link at the top of the readme. The Awesome badge is enough.

Reviews

Review 1
Review 2

Closes #2724

It still needs to follow the guidelines: https://github.com/sindresorhus/awesome/blob/main/pull_request_template.md#requirements-for-your-awesome-list
For example:

The description starts with an uppercase character and ends with a period.

This is all so new I could use a tutorial on all this? I'm total NOB

unicorn

I suppose replying with the keyword sensibly doesn't ping... @sindresorhus this PR is ready.
A related question - is awesome-lists still a thing? If so, I would like to transfer my awesome-mastodon to the organization if you concur. I would also like to ask you for the permission to help maintain awesome-lists/awesome-bash; there are are various issues with the list (dead links, renamed links, improper formatting, non-compliance with awesome-lint, large issue/pr queue) that I would like to fix. I have a lot of experience with Bash and I'm quite active on GitHub, so I think it would be a good fit.

Is this to help me outside the PR I'm not sure what I'm trying to compute

A related question - is awesome-lists still a thing?

I prefer for people to have their list on their account, because people feel more ownership then and is more likely to continue maintaining. The thinking of the org was to take over popular lists that are no longer maintained.

I would also like to ask you for the permission to help maintain awesome-lists/awesome-bash;

Invited you.

That makes sense - in that case I will keep the awesome-mastodon list on my account. Thank you for the invite - I have accepted it.


Implement "Rooms filter" on tracks page

Implement "Rooms filter" on tracks page. All items should be clickable.

@mariobehling Claiming 🚀

has this issue been resolved yet? if not can i claim it?

@ItachiUchiha1998 I'm working on this

can i work on this issue?

@shubham3211 Go ahead!

ok :+1:


Refactors + add pin board module

There's a lot of refactors. A lot.
Feel free to learn more from the cone school of Rust™
(Also added a pin board module at request from @ryanccn —  simply set PIN_BOARD_TARGET to the ID of the pin board channel,
and PIN_BOARD_SOURCES to a comma-separated list of IDs of the channels the bot should listen for pins.)

i'll merge this after i setup 8ball's sever for it

🥺


Add LICENSE.md

Port the existing MIT LICENSE in the README.md to LICENSE.md file.

It already exists, do you another license?

Sorry, this has already been done and I forgot to close the issue - my bad! 🙈


Error on clearLayers - this._icon is undefined

I'm having a problem when I call the Leaflet's clearLayers function. In 99% of the use of the application this function is executed correctly, but in some moments an exception is launched from the clearLayers function
Call that triggers the error

getMapLayersLeaflet is just a function that searches for layers or creates if it doesn't exist (singleton idea)

Error

Original Code: leaflet-src.js in function _removeIcon. The error is coming from the call  remove(this._icon), because it's coming in this function this._icon with undefined

Debug Code: Showing that it is coming undefined in the _removeIcon function

Result 1

Result 2

I believe that a race condition is occurring, at some point that it has 2 expenses at the same time, this is an assumption
Initial Solution:  The solution was to check if this._icon exist on leaflet-src.js in function _removeIcon.

I don't know if this solution was correct
I'm running the latest version of Leaflet v1.7.1, the error also occurs in previous versions
The browser that is running the code is the Google Chrome Version 81.0.4044.92 (Official Build) (64-bit)

We need to know how to reproduce the issue, otherwise it's hard to choose where it should be fixed.

We need to know how to reproduce the issue, otherwise it's hard to choose where it should be fixed.

I have the same issue. layer.clearLayers(); works most of the time, but will sometimes throw an exception because this._icon is undefined. Some further info: I have a WebSocket that receives GPS coordinates from an external source (at ~10Hz), and in the onmessage callback of the WebSocket, I call layer.clearLayers(); L.marker(message.position, {icon}).addTo(layer); (icon is  defined earlier).
Call stack:
ri<EMAIL_ADDRESS>_removeIcon<EMAIL_ADDRESS>onRemove<EMAIL_ADDRESS>removeLayer<EMAIL_ADDRESS>removeLayer<EMAIL_ADDRESS>removeLayer<EMAIL_ADDRESS>eachLayer<EMAIL_ADDRESS>clearLayers<EMAIL_ADDRESS>make_listener (http://localhost:8080/static/js/app/map.js#70)
dispatch_message (http://localhost:8080/static/js/app/api.js#84)
onmessage (http://localhost:8080/static/js/app/api.js#25)
...

I close this issue until we have a reproduceable example


SSR Fails if module uses top-level await or imports module using top-level await

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[ ] I have tried the npm run clear or yarn clear command.
[ ] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[ ] I have read the console error message carefully (if applicable).

Description
Usage of top-level await in a component rendered as part of a docusaurus page leads to server-side rendering errors (but works on client side).
This was first encountered when importing an ES module using top-level await, but the same behavior can be replicated by adding a top-level await statement to the default pages/index.tsx. We get a separate error if we try to mark the example app as an ES module, but bypassing that, we get this same issue from importing the component. It looks like webpack may apply transformations automatically when encountering top level await. https://webpack.js.org/configuration/experiments/#experimentstoplevelawait
Reproducible demo
https://github.com/NickGerleman/docusaurus-await-repro
Steps to reproduce

Ad the following snippet to pages/index.tsx in a new Docusaurus V3 app

const a = await new Promise((resolve) => {
  setTimeout(() => resolve(42), 0);
})

Run yarn build --dev

Expected behavior
Build succeeds
Actual behavior
✖ Server
  Compiled with some errors in 5.88s

Warning: React.jsx: type is invalid -- expected a string (for built-in components) or a class/function (for composite components) but got: object. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
    at LoadableComponent (webpack-internal:///./node_modules/react-loadable/lib/index.js:138:32)
    at Route (webpack-internal:///./node_modules/react-router/esm/react-router.js:648:29)
    at Switch (webpack-internal:///./node_modules/react-router/esm/react-router.js:850:29)
    at Route (webpack-internal:///./node_modules/react-router/esm/react-router.js:648:29)
    at ClientLifecyclesDispatcher (webpack-internal:///./node_modules/@docusaurus/core/lib/client/ClientLifecyclesDispatcher.js:15:261)
    at PendingNavigation (webpack-internal:///./node_modules/@docusaurus/core/lib/client/PendingNavigation.js:17:150)
    at Root (webpack-internal:///./node_modules/@docusaurus/core/lib/client/theme-fallback/Root/index.js:20:16)
    at BrowserContextProvider (webpack-internal:///./node_modules/@docusaurus/core/lib/client/browserContext.js:21:127)
    at DocusaurusContextProvider (webpack-internal:///./node_modules/@docusaurus/core/lib/client/docusaurusContext.js:21:496)
    at ErrorBoundary (webpack-internal:///./node_modules/@docusaurus/core/lib/client/exports/ErrorBoundary.js:16:269)
    at App (webpack-internal:///./node_modules/@docusaurus/core/lib/client/App.js:29:241)
    at LinksCollectorProvider (webpack-internal:///./node_modules/@docusaurus/core/lib/client/LinksCollector.js:17:125)
    at Router (webpack-internal:///./node_modules/react-router/esm/react-router.js:267:30)
    at StaticRouter (webpack-internal:///./node_modules/react-router/esm/react-router.js:763:35)
    at r (webpack-internal:///./node_modules/react-helmet-async/lib/index.module.js:17:7350)
    at Capture (webpack-internal:///./node_modules/react-loadable/lib/index.js:294:30)
Error: Docusaurus server-side rendering could not render static page with path / because of error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
    at render (webpack-internal:///./node_modules/@docusaurus/core/lib/client/serverEntry.js:35:346)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async /Users/ngerlem/github/my-website/node_modules/p-map/index.js:57:22 {
  [cause]: Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
      at renderElement (webpack-internal:///./node_modules/react-dom/cjs/react-dom-server.node.development.js:6109:9)
      at renderNodeDestructiveImpl (webpack-internal:///./node_modules/react-dom/cjs/react-dom-server.node.development.js:6170:11)
      at renderNodeDestructive (webpack-internal:///./node_modules/react-dom/cjs/react-dom-server.node.development.js:6142:14)
      at renderContextProvider (webpack-internal:///./node_modules/react-dom/cjs/react-dom-server.node.development.js:5986:3)
      at renderElement (webpack-internal:///./node_modules/react-dom/cjs/react-dom-server.node.development.js:6083:11)
      at renderNodeDestructiveImpl (webpack-internal:///./node_modules/react-dom/cjs/react-dom-server.node.development.js:6170:11)
      at renderNodeDestructive (webpack-internal:///./node_modules/react-dom/cjs/react-dom-server.node.development.js:6142:14)
      at renderIndeterminateComponent (webpack-internal:///./node_modules/react-dom/cjs/react-dom-server.node.development.js:5851:7)
      at renderElement (webpack-internal:///./node_modules/react-dom/cjs/react-dom-server.node.development.js:6012:7)
      at renderNodeDestructiveImpl (webpack-internal:///./node_modules/react-dom/cjs/react-dom-server.node.development.js:6170:11)
}

Error: Unable to build website for locale en.
    at tryToBuildLocale (/Users/ngerlem/github/my-website/node_modules/@docusaurus/core/lib/commands/build.js:55:19)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async mapAsyncSequential (/Users/ngerlem/github/my-website/node_modules/@docusaurus/utils/lib/jsUtils.js:44:24)
    at async Command.build (/Users/ngerlem/github/my-website/node_modules/@docusaurus/core/lib/commands/build.js:82:21) {
  [cause]: Error: Failed to compile due to Webpack errors.
  Error: Docusaurus server-side rendering could not render static page with path / because of error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: object. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
      at /Users/ngerlem/github/my-website/node_modules/@docusaurus/core/lib/webpack/utils.js:207:24
      at /Users/ngerlem/github/my-website/node_modules/webpack/lib/MultiCompiler.js:554:14
      at processQueueWorker (/Users/ngerlem/github/my-website/node_modules/webpack/lib/MultiCompiler.js:491:6)
      at process.processTicksAndRejections (node:internal/process/task_queues:77:11)
}

[INFO] Docusaurus version: 3.0.0-rc.1
Node version: v18.16.1

Your environment
Docusaurus version: 3.0.0-rc.1
Node version: v18.16.1
Self-service

[ ] I'd be willing to fix this bug myself.

It looks like webpack async modules work out of the box with import, but will return a promise if sync required. https://webpack.js.org/blog/2020-10-10-webpack-5-release/
I tried to dig into how react-loadable as used by Dodusaurus works but didn’t get very far. I suspect there is some code somewhere doing sync style require and getting a promise back.

Granted it has been a while, but I think it's a duplicate of https://github.com/facebook/docusaurus/issues/7238. TLA still doesn't work in Docusaurus because of all the legacy setup we have.

Closing as a duplicate


Bluetooth: Mesh: Miscellaneous corrections to minor problems

Fixed the problem in the Provisioner provisioning
formatted printing and added key update function for CDB.
Signed-off-by: Lingao Meng<EMAIL_ADDRESS>

cc @haoyanbiao

I have submitted all the units separately. I suggest merging the current units first. After all, they will solve some problems in the current code, and then I will complete them in another pr.

Close this, please move to https://github.com/zephyrproject-rtos/zephyr/pull/32567


Chrome's tokens don't work, you need to use Firefox. Made that clear in the README.

You mean the cookie? I just checked and they are indeed different, but logging in still works.

Yup the cookie. Here's my cookie on Chrome:
csrftoken=[VALUE];
messages=[VALUE];
__stripe_mid=[VALUE];
LEETCODE_SESSION=[VALUE];
_ga=[VALUE];
_gid=[VALUE];
_dd_s=[VALUE];
_ga_CDRWKZTDEX=[VALUE]

Compared to Firefox:
csrftoken=[VALUE];
_ga_[VALUE?]=[VALUE];
_ga=[VALUE];
_dd_s=rum=[VALUE];
_gid=[VALUE];
gr_user_id=[VALUE];
[VALUE?]_gr_session_id=[VALUE];
[VALUE?]_gr_session_id_sent_vst=[VALUE];
LEETCODE_SESSION=[VALUE];
messages=[VALUE];
[VALUE?]_gr_last_sent_sid_with_cs1=[VALUE];
[VALUE?]_gr_last_sent_cs1=[VALUE};
[VALUE?]_gr_cs1=[VALUE]

Cookie parser looks only for csrftoken and LEETCODE_SESSION tokens, which both are present in both cookies.
What error do you get when pasting the cookie from chrome?
    local csrf = str:match("csrftoken=([^;]+)")
    if not csrf or csrf == "" then return nil, "Bad csrf token format" end

    local ls = str:match("LEETCODE_SESSION=([^;]+)")
    if not ls or ls == "" then return nil, "Bad leetcode session token format" end

Cookie parser looks only for csrftoken and LEETCODE_SESSION tokens, which both are present in both cookies. What error do you get when pasting the cookie from chrome?
    local csrf = str:match("csrftoken=([^;]+)")
    if not csrf or csrf == "" then return nil, "Bad csrf token format" end

    local ls = str:match("LEETCODE_SESSION=([^;]+)")
    if not ls or ls == "" then return nil, "Bad leetcode session token format" end

Whoops didn't get this notification. I got the Bad csrf token format error.


[BEAM-7557] - Migrate DynamoDBIO to AWS SDK for Java 2

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

@cmachgodaddy I was trying to rebase this locally but I get some classes removed, it is weird, can you please fully rebase it and get it green. I will fix any minor pending fix and merge when done, thanks.

Closing this in favor of #9086


clarify CQs have no state

fixes https://github.com/influxdata/docs.influxdata.com/issues/262

@rkuchan @rossmcdonald

+1, looks good to me

👍


New splitter component

Implementation of the new Splitter component. Theme settings are missing. Variables already implemented in _splitter.scss

Hi @GioeEng,
Thank you for the pull request and the enormous amount of work. We appreciate the effort!
We will have to test the behavior of the component before we merge it though.
Also here are a few comments that I have after a quick glance over the implementation (they are mostly about code style and naming conventions).

The events of Radzen components currently do not begin with On - we prefer Resize vs. OnResize.
All CSS classes that we use have a common prefix ".rz-" e.g.  class="rz-collapse" vs. class="collapse"
We use camel case notation for C# and JS variable names e.g. pane vs i_pane and paneLength vs pane_length.

Could you please update the code to adhere to those conventions?

Hi @akorchev,
Within the time available, reported problems were fixed and some bug fixes were performed

Thank you @GioeEng! I just played around with this pull request and things look very promising! Great stuff.
We just need to style it a bit and implement design-time support in Radzen. By the way we want to give you a free Radzen Professional subscription (if you want one of course). Just shoot an email to<EMAIL_ADDRESS>

Thanks @akorchev ,
actually I didn't focus on styles as I don't have all the tools needed for theme setup assistant. I'm thinking of using the Razden library to migrate my AspNet.Core (MVC) projects and its library of javascript controls. There are still several controls and features I can implement (from the old library) but I'm waiting for Microsoft to improve Blazor integration with VS (2022? https://devblogs.microsoft.com/dotnet/introducing-net-hot-reload/) because working without the edit & continue and without Hot Reload properly functional is really time consuming.
It's still a pleasure to collaborate, you are doing a good job.

We have a design for the splitter in mind and will implement it some time next week (hopefully).


fix for issue 1169 that works without explicit separators

This contains a fix for path-element fixing (Delegated PR for @lucian-r)
fix for v3 is in #1718

Related to #1169


install expat:x64-windows

Package: expat:x64-windows
Vcpkg version: 2018.11.23-nohash
tar is having issues with the README symlink in the expat package on windows:
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:56 (message):
    Command failed: C:/Program Files/CMake/bin/cmake.exe;-E;tar;xjf;C:/Users/USER/git/vcpkg/downloads/libexpat-libexpat-R_2_2_6.tar.gz
    Working Directory: C:/Users/USER/git/vcpkg/buildtrees/expat/src/TEMP
    Error code: 1
    See logs for more information:
      C:\Users\USER\git\vcpkg\buildtrees\expat\extract-err.log

CMake Error: Problem with archive_write_header(): Can't create ''
CMake Error: Current file: libexpat-R_2_2_6/README.md
CMake Error: Problem extracting tar: C:/Users/ignaz/git/vcpkg/downloads/libexpat-libexpat-R_2_2_6.tar.gz

Running tar xjf libexpat-libexpat-R_2_2_6.tar.gz give the following error:
libexpat-R_2_2_6/README.md: Can't create '\\\\?\\C:\\Users\\USER\\git\\vcpkg\\buildtrees\\expat\\src\\TEMP\\libexpat-R_2_2_6\\README.md'
tar: Error exit delayed from previous errors.

You can reproduce it by running vcpkg install expat:x64-windows
UPDATE:
It works in a Windows VM I created so the issue is somewhere else :man_shrugging:

in case anyone has the same issue:
Alexej Harm found the setting that causes this issue.
The UTF-8 beta in intl.cpl > Administrative > Change system locale... needs to be unticked for cmake (3.14.2) to function properly.


Add a helper to generate/export public keys and auth header from clients more easily

closing in anticipation of switching to server-sent events


Adding missing structs from linux/user.h

Adding structs from linux/user.h (or sys/user.h depending on OS version). This adds the linux user_regs_struct and user struct from sys/user.h into libc for x86 and x86_64. Also, bumping the version number.
As an aside, I was wondering if this was also the right place to add the bit offsets for the EFLAGS register ? These aren't defined anywhere in the linux system libraries instead the definition seems to just be in the x86 and x86_64 documentation but they are useful for anyone looking to interpret eflags. If libc is the right place for these constants I'd like the opportunity to add them before it's merged in. :)

Thanks for the PR! Looks like there's some style errors though?
Could you also back out the version bump? That typically happens through dedicated PRs.

Done and done.
Any thoughts on constants for the EFLAGS register?

so there appears to still be some errors, it seems some of the generated tests fail? And also "error: requires freeze lang_item". cargo build and test both pass on my own pc and I'm not too familiar with the testing involved. Any guidance would be appreciated although I'll still be looking into it so hopefully will figure it out...

Any thoughts on constants for the EFLAGS register?

Ah sorry I missed that! If these are constants in glibc or in standard C headers, sure! Otherwise though I'd recommend a separate crate for now.
For the test failures the nightly failures are ok (I'll work around them elsewhere) but for there's a few other non-nightly failures I think?

@xd009642 To test on your local PC you can run cd libc-test && cargo run.
You may need to change the test to include <sys/user.h>.
Note that _libc_fpstate is from <sys/ucontext.h>. It's not the same as  user_fpregs_struct from <sys/user.h>. It looks like you renamed user_regs_struct to _libc_state to match that, but you should keep the original name instead. You'll have to fix this to get the test to pass.

Ah okay, thanks @philipc, I probably won't have a chance to do this until Tuesday as I'm away from my home pc.

Right I've addressed the issues and everything passes on my home machine. Hopefully it should all be good with travis now. Sorry about the delay!

@bors: r+
Looks great, thanks!

:pushpin: Commit 6acbf87 has been approved by alexcrichton

:hourglass: Testing commit 6acbf87 with merge 80e1422...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 80e1422851d45015ebd6d351ebc16db55ee24be4 to master...


Add Mix music mother be like by OrionGXT

Song Name
Mix music mother be like
Artist Name
OrionGXT
Source
Youtube
Youtube Link
Feiw5IjZbrs
Direct File Link
No response
Song ID
1000
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 111618119
Downloaded: Yes

accept


Build errors with any Google API for iOS

My project builds perfectly fine when I do not add a reference to Xamarin.Firebase packages, but when I do, I get the error "Could not load file or assembly 'Microsoft.Win32.Registry" while building the project. I have tried different Xamarin.Build.Download versions 0.4.11, 0.7.1, 0.8, 0.9, and finally 0.10. None of them solved my problem! I have tried clearing the .nuget cache, as well as bin and obj folders. Still no hope!
I am using Visual Studio on Windows, and connecting to my Mac via the LAN. I have also cleared the .nuget cache from my Mac. Also, I am installing the latest version of Xamarin.Firebase.iOS.Core v6.6.0.
As I read through the related issues for other Google API packages ( #365 or #372 ) others have reported the same build errors as I am getting.

Seconded. I'm experiencing the exact same issue. Found myself going around in circles, it seems the new 0.10 gives:
The "XamarinDownloadArchives" task could not be initialized with its input parameters.
And the previous versions give either the Win32 error or Sharing Violation errors.

@AmirMahdiNassiri @Minneth Can you share a test case, please? We have tested again XBD and seems to be working ok from our side.

@SotoiGhost Yes, of course. I even created a new Xamarin Forms solution using Visual Studio 2019, and only added the Xamarin.Firebase.iOS.Core v6.6.0.  I have saved them in the attached build log file.
Please let me know if any additional data is required.
Build Log.txt

@AmirMahdiNassiri Can you add the following line to your test case please? Let me know your results.
Firebase.Core.App.Configure();

@SotoiGhost Oh, I can confirm that putting this line in AppDelegate.FinishedLaunching ensures a successful build! Thank you so much for your amazingly quick replies! My apologies that I didn't know this is a requirement for a successful build.
Have a great day, and keep up the great work.

@SotoiGhost Sadly, after putting my next line of code, which is Crashlytics.Configure(); the error reappears! I tried clearing obj and bin directories and installed Xamarin.Build.Download v0.10 on the iOS project.

I've already got this in my code and no benefit, unfortunately.

From: Israel Soto<EMAIL_ADDRESS>Sent: Friday, March 6, 2020 8:24:21 PM
To: xamarin/GoogleApisForiOSComponents<EMAIL_ADDRESS>Cc: Minneth<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [xamarin/GoogleApisForiOSComponents] Build errors with any Google API for iOS (#381)
@AmirMahdiNassirihttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAmirMahdiNassiri&data=02|01||a7f44a903ce6478f95b108d7c20c5b31|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637191230625620831&sdata=x4IbGRMSEkWZ6yopJicNofc66PoYQwwXSen%2Be73boIk%3D&reserved=0 Can you add the following line to your test case please? Let me know your results.
Firebase.Core.App.Configure();
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fxamarin%2FGoogleApisForiOSComponents%2Fissues%2F381%3Femail_source%3Dnotifications%26email_token%3DABQ4QRGXPSS2I4ZEPMQ75ALRGFLXLA5CNFSM4LDFWEVKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOCXXBI%23issuecomment-595950469&data=02|01||a7f44a903ce6478f95b108d7c20c5b31|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637191230625620831&sdata=Ct%2Bjo%2F7S3yAwy%2BZ%2BgJ98TV%2BbocVpuwCNe3CFP%2BpGnuY%3D&reserved=0, or unsubscribehttps://eur05.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABQ4QRDQGCZYHG52L4XDZU3RGFLXLANCNFSM4LDFWEVA&data=02|01||a7f44a903ce6478f95b108d7c20c5b31|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637191230625630822&sdata=scQ8VmqejxP0goOZjgTqWS9020dqKILmtXPdFWWWylE%3D&reserved=0.

@AmirMahdiNassiri I added Crashlytics.Configure(); and worked for me. The only thing that might differ is the environment data. Are you up to date on stable channel?

@SotoiGhost Yes, you're right. It worked after clearing obj and bin and restarting Visual Studio! What's the problem with those native linkings?!

Glad to know that this is working!
The problem is that Firebase and Google have a quite complex dependency chain (by dependency I mean GoogleUtilities, Protobuf, GTMSessionFetcher, etc.), so, the linker tries to do it best to remove unused code to reduce app size. But, the linker is not perfect and makes mistakes sometimes (even more when native frameworks use swizzling as Firebase does), so it strips dependency symbols thinking that no-one use them, but Firebase frameworks use them somehow and viceversa. That's the reason of Firebase linking errors even when you haven't coded anything.
I'm closing this issue as it seems that the error is fixed. If you still facing something related to this, please, feel free to reopen it! Happy coding!

@SotoiGhost I see. It's not the first time for me that the linking process does annoying things. Entity Framework core had some issues with the linking process as well!
Thank you for all the information, and I really appreciate all your quick replies. Happy coding!

Hello, Compliment of the day, how are you today ? i am Happy to get together with you,My name is Serach , I'm a military medical nurse from israel. Please let not my profession, race or nationality inch your mind, I have gone across your profile and became interested in you, we can be good friends, contact me through my private email address/<EMAIL_ADDRESS> ////


Update production AmiGO after upstream data fix

The UM-BBD URLs at UMN are not working, so each GO term that has UM-BBD as Source leads to a page that no longer exists. For an example, please visit http://amigo2.berkeleybop.org/amigo/term/GO:0018920
or
http://amigo.geneontology.org/amigo/term/GO:0018920
Edith Wong followed up with Emanuel from eawag and below is his response:

The UM-BBD/PPS has been transferred to Eawag-BBD/PPS under the general address of http://eawag-bbd.ethz.ch. Thus, you can now find the glyphosate data, for instance, under http://eawag-bbd.ethz.ch/gly/gly_map.html.
Please note, however, that as of around the middle of next year, Eawag-BBD/PPS will go offline and be completely replaced with the successor system enviPath (https://envipath.org). With regards to contents we intend to completely migrate all data from Eawag-BBD/PPS, but the form will be different.
Right now we are working on a feature of the new system that allows to easily map the old addresses to the new ones.
If you have joined the email list<EMAIL_ADDRESS>you will receive detailed information about this mapping as soon as it is finalized.

These URLs should be updated.

Possibly related update needed for this file:
https://github.com/geneontology/go-site/blob/master/metadata/db-xrefs.yaml

database: UM-BBD
name: University of Minnesota Biocatalysis/Biodegradation Database
generic_urls:

http://umbbd.msi.umn.edu/
entity_types:
type_name: entity
type_id: BET:0000000

I believe that this is not an AmiGO issue, but rather an upstream data issue. The proper location would be geneontology/go-site. I'll leave this open for "hotfix" resolution with @stuartmiyasato after upstream data is fixed.

Sorry @kltm, you are right. I should have added that to go-site. -- had the wrong repo on screen.

Links to geneontology/go-site#165

Does anybody know of the upstream data has been fixed for this case? I can't find any actionable error either...

I've attempted to fix upstream, and it looks fine on http://tomodachi.berkeleybop.org/amigo/term/GO:0018920 .
@stuartmiyasato, at your leisure, could you do a refresh of the amigo data ("make install" or "make refresh" IIRC) and close out this item?

Sorry, or @kkarra .

This would have finally been solved by the new AmiGO release.


Documentations to 3d cockipt

Hi guys there is document
For understand where to start to improove 3d cockipt of deltaglider?

Hi!
There is documentation explaining the workings of (generic) virtual cockpits (and that is currently being improved), but there is nothing specific about any vessel, as each vessel will have its own buttons, lights, etc.

How to open current vessel graphics?

I think you should post in the forum, as these tickets are not the correct place to request help.

Thx


OpenAPI validation fails on circular references

Hi, maintainers.
There is a valid usecase using circular references in arrays in openapi schema but I'm getting this error.
Error: failed to add runners (hello.yaml): runner req error: infinite circular reference detected: Row: Response -> Row -> Row [33:11]
libopenapi has a parameter to ignore them, which is IgnoreArrayCircularReferences.
https://pb33f.io/libopenapi/circular-references/
But runn is not able to being accessed to this option.
https://github.com/k1LoW/runn/blob/main/runner_option.go#L13-L16
I would like to make this behavior configurable so love to hear what you think of this.
Sample files
runners:
  req:
    endpoint: https://example.com
    openapi3: ./openapi.yaml
steps:
  TestHello:
    req:
      /api/hello:
        get:
    test: |
      current.res.status == 200

openapi: 3.1.0
info:
  title: For debugging
  version: 1.0.0
paths:
  /api/hello:
    get:
      responses:
        "200":
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/Response"
          description: Debugging
components:
  schemas:
    Response:
      type: object
      properties:
        rows:
          type: array
          items:
            $ref: "#/components/schemas/Row"
      required:
        - rows
    Row:
      type: object
      properties:
        name:
          description: Name of the row
          type: string
        rows:
          description: A collection of row
          type: array
          items:
            $ref: "#/components/schemas/Row"
      required:
        - name
        - rows

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx

@takanohi Thank you for your request.
Even with the circular reference check disabled, libopenapi (libopenapi-validator) could not validate this response.
I don't know if this will have the results you are hoping for, but I would like to take action because it is effective in some use cases.

@k1LoW Thank you for quick updates! I confirmed it was fixed too.


Error on connect Firebird 3

Driver jaybird-jdk17-3.0.2.jar  no compatible with firebird 3
Error:  Incompatible wire encryption levels requested on client and server  [SQLState:28000, ISC erros code:335545064]
help

You need to update driver version to 3.x:
Open driver edit dialog->Download/Update->Click on Version column. Select 3.x version in dropdown.

I have installed 3.0.1 driver. It still does not work.Get the same error description. I use Windows10. Please help. Tried the other drivers also.

The default setting for WireCrypt is incompatible with jaybird: https://github.com/FirebirdSQL/jaybird/wiki/Jaybird-and-Firebird-3#jaybird-3

@mrotteveel where should I put WireCrypt = Enabled? Adding it in driver properties does not work


Connection resets under highly concurrent traffic

Hi All,
I'm using Apache Bench [1] to push some load through a vanilla dropwizard application. I'm hitting the admin ping resource with around 250 concurrent connections for a duration of about 30 seconds [2]. During these tests I see "connection resets" [3]. This test is NOT using keep-alive and so every request is creating a new TCP socket.
With keep-alives turned on the tests pass with flying colours.
Is there any advice / guidance as to the max number new of connections  / second? And / or is there some configuration settings that I can I tweak to set / discover this limit. I've tried maxThreads  with no success.
Thanks for any advice.
[1] https://httpd.apache.org/docs/2.4/programs/ab.html
[2] ab -c 250 -t 30 -n 10000000 http://localhost:8081/admin/ping
[3] `This is ApacheBench, Version 2.3 <$Revision: 1826891 $>
Copyright 1996 Adam Twiss, Zeus Technology Ltd, http://www.zeustech.net/
Licensed to The Apache Software Foundation, http://www.apache.org/
Benchmarking localhost (be patient)
apr_socket_recv: Connection reset by peer (54)
Total of 14514 requests completed`

@DrMattFaulkner Thanks for writing this up!
Which version of Dropwizard have you been using? What was the exact configuration you've been using in your Dropwizard application?
And finally, could you please try repeating the benchmarks with Dropwizard 2.0.0-rc1?

Was this tested with OSX? There are some issues with ab.
Can it be reproduced with vegeta or wrk or siege?

Only just seen these replies.
@joschi  I've been using version 1.3.9 of dropwizard. The configuration is standard:
`server:
type: simple
applicationContextPath: /
adminContextPath: /admin
connector:
type: http
logging:
level: INFO
loggers:
com.genesys: DEBUG
`
Let me repeat the tests with DW 2. I have tried something other than ab, but let me repeat those tests to get up to date information.

@DrMattFaulkner Did you find time to re-run the tests with Dropwizard 2.0.0-rc1?


Passwords Encrypted

id : X
phishlet : instagram
username : X
password:#PWD_INSTAGRAM_BROWSER:6:157415674334:AfQAL7s8TTSOrmDsvbvD8g36+pMIyUAqH+vlKuJfv9qYQDNhMT7xe+to/G18rTfA/Mr8xgoUQgHuD34UiImBvY237ykx8lhHIn6OuniKNQ7/ThxfFZZTJZ3y
tokens : captured
Why in password instead of the password I get this?

Instagram encrypts the password use AES GCM before sending it to the server. Necessary inject own script to intercept password

Instagram encrypts the password use AES GCM before sending it to the server. Necessary inject own script to intercept password

this not happening to me

Hey, it is also happening to me. It shows me the right password but then a crypted one and save the crypted one.
Same for Facebook but it replaces it with "false".
Do someone have a solution ? Or know how to manually replace a password in a saved session. Thanks :)

Does anyone have any solution yet?
Facebook is doing it too for their mobile site.
Basically they add an input element named "encpass" (with encrypted pass) and they remove the "name" attribute on "pass" field so it doesn't get posted.
If i understood correctly, we will try to get the password from fields that have "pass" (value defined in phishlet) in their name. This is why sometimes we get "false" in the password value, it comes from "had_password_prefilled" field.
I will see if I can inject something to keep the original value  of the password.

Facebook are doing client side encryption for the mobile version, so the password will be encrypted then sent with POST request(with email), Evilginx is man in the middle so it’s capturing the Network traffic and in this case it’s the encrypted password  and the plain text email.
You need to inject JS to the phishing page to capture the password manually.
Simple example can be: if the user clicks/touch on the login button, grab the password filed and send it to your server.
It’s pretty simple injection, I can post it if needed.
I think the reason for client side encryption ONLY on mobile is due to the fact that mobiles phones are using public/free Wifi more often, and they more vulnerable to man in the middle attack?

Facebook are doing client side encryption for the mobile version, so the password will be encrypted then sent with POST request(with the email), Evilginx is man in the middle so it’s capturing the Network traffic and in this case it’s the encrypted password and the plain text email.
You need to inject JS to the phishing page to capture the password manually.
Simple example can be: if the user clicks/touch on the login button, grab the password filed and send it to your server.
It’s pretty simple injection, I can post it if needed.
I think the reason for client side encryption ONLY on mobile is due to the fact that mobiles phones are using public/free Wifi more often, and they more vulnerable to man in the middle attack?

<EMAIL_ADDRESS>can u help me also?

id : X
phishlet : instagram
username : X
password:#PWD_INSTAGRAM_BROWSER:6:157415674334:AfQAL7s8TTSOrmDsvbvD8g36+pMIyUAqH+vlKuJfv9qYQDNhMT7xe+to/G18rTfA/Mr8xgoUQgHuD34UiImBvY237ykx8lhHIn6OuniKNQ7/ThxfFZZTJZ3y
tokens : captured
Why in password instead of the password I get this?

i can help you solve the issues

i not working telegram
…
On Sat, Jan 25, 2020 at 8:53 PM spam441 @.***> wrote: contact me on telegram if u guys need any help @IAmBecomeDeath https://github.com/IAmBecomeDeath — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#422?email_source=notifications&email_token=AN6QN2E5XXIMM6Q7HLOMCN3Q7QY6NA5CNFSM4JPWYCTKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJ4344Q#issuecomment-578403954>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN6QN2AFFOU4UPIAZT63DXTQ7QY6NANCNFSM4JPWYCTA .

i can help you with the<EMAIL_ADDRESS>

Can you help me too with the Facebook encryption password?
<EMAIL_ADDRESS>Thank you !


Fix the NPE when no segment exists in RealtimeSegmentSelector

When no HLC or LLC segment exists, return empty list
Fix the external view version in RoutingManager for non-existing external view

Codecov Report

Merging #5138 into master will increase coverage by 0.11%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master    #5138      +/-   ##
============================================
+ Coverage     65.49%   65.61%   +0.11%
  Complexity       12       12
============================================
  Files          1051     1051
  Lines         54120    54125       +5
  Branches       8068     8070       +2
============================================
+ Hits          35447    35514      +67
+ Misses        16052    15985      -67
- Partials       2621     2626       +5

Impacted Files
Coverage Δ
Complexity Δ

...uting/segmentselector/RealtimeSegmentSelector.java
90.32% <0%> (-3.02%)
0 <0> (ø)

...rg/apache/pinot/broker/routing/RoutingManager.java
81.49% <50%> (+3.01%)
0 <0> (ø)
:arrow_down:

...elix/core/relocation/RealtimeSegmentRelocator.java
34.21% <0%> (-10.53%)
0% <0%> (ø)

.../impl/dictionary/FloatOnHeapMutableDictionary.java
68.29% <0%> (-8.54%)
0% <0%> (ø)

...org/apache/pinot/broker/queryquota/HitCounter.java
92.3% <0%> (-7.7%)
0% <0%> (ø)

...lix/core/realtime/PinotRealtimeSegmentManager.java
76.41% <0%> (-5.65%)
0% <0%> (ø)

.../impl/dictionary/LongOffHeapMutableDictionary.java
67.74% <0%> (-4.31%)
0% <0%> (ø)

...e/operator/dociditerators/BitmapDocIdIterator.java
60.71% <0%> (-3.58%)
0% <0%> (ø)

...e/operator/dociditerators/SortedDocIdIterator.java
55.55% <0%> (-2.78%)
0% <0%> (ø)

...rg/apache/pinot/core/operator/CombineOperator.java
66.29% <0%> (-2.25%)
0% <0%> (ø)

... and 20 more

Continue to review full report at Codecov.

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

Can we add a unit test case or integration test case?

Added a unit test to cover the NPE: #5139
The existing integration tests already cover the RoutingManager. The version issue is a little bit tricky to cover though..


Add functionality to bulk assign content items to taxons

This is required as part of the functionality to bulk assign content
items to taxons, based on the alpha taxonomy data that is currently
being produced by Information Architects in the Finding Things team.
This change adds a method which inspects the locally-downloaded taxonomy
data and groups list of taxons to each base path. Part of this change
includes moving the actual file creation to the #populate method rather
than doing this on initialize. Any client of the ImportFile class will
have to ensure the file is created via #populate before calling anything
else.
Trello: https://trello.com/c/bsZAwZ35

:+1: I'll Merge tomorrow once someone else has had a look


Switch box-reflection example to refresh envmap every 10 frames

To improve out of the box performance on mobile devices

Just thinking... would it make more sense for the update frequency to be controlled 'once per second' instead of per number of frames?


Editing title and meta description for teaching internships

Trello card
https://trello.com/c/KM5iYuWx/2682-launch-of-marketing-internship-programme
Context
Inputting content and backlinks, ready to look at formatting with Pete on Monday
Changes proposed in this pull request
Guidance to review

looks good to me :)


Service Idler should not scale pods on Kubernetes Clusters

Describe the bug
The Lagoon Service Idler is scaling pods in force mode on Kubernetes Clusters even though we don't have support for auto unidling on Kubernetes Clusters yet.
Also we actually use oc idle and it seems running this against a Kubernetes cluster will just scale the Deployment to 0 even though Kubernetes should not support oc idle (it's an OpenShift command)
To Reproduce
run https://github.com/amazeeio/lagoon/blob/master/services/auto-idler/idle-services.sh with force, see deployments on Kubernetes cluster scaled to 0.
Expected behavior
Kubernetes clusters are skipped.

this is now handled by aergia


Archive this package in favor of TranscodingStreams?

I'm reasonably certain this package is deprecated in favor of TranscodingStreams and has been for a few years. Should we just archive it and put a big, fat sign up on the README that redirects people to TranscodingStreams?

Possible duplication of #61. In that issue, @StefanKarpinski mentions TranscodingStreams.jl, so maybe they're orthogonal? I don't think I've ever used either directly, so I'm not sure.
I do think it should be either archived or transferred to a different Org, its presence in BioJulia is mostly historical accident.

IIRC, Kenta Sato wrote this package to replace BufferedStreams - the NoopStream works like BufferedStreams, but better.

I think the anchor functionality is missing in the other packages.

TranscodingStreams is much slower for me:
julia> using BufferedStreams, TranscodingStreams

julia> function readchars(io)
           s = 0
           while !eof(io)
               s += read(io, UInt8)
           end
           return s
       end;

julia> open(tmp) do io
           @time readchars(BufferedInputStream(io, 128*2^10))
       end;
  0.136337 seconds (3 allocations: 128.172 KiB)

julia> open(tmp) do io
           @time readchars(NoopStream(io; bufsize=128*2^10))
       end;
  1.012437 seconds (5 allocations: 128.234 KiB)

So I think there is till value in this package.

@KristofferC good point. Since all of BioJulia's IO goes through TranscodingStreams, I think what your comment implies is that we should do our best to make TranscodingStreams much faster than it is currently. And I agree we shouldn't retire BufferedStreams until we can reasonably claim that users can switch to NoopStream without any loss of performance.

Unfortunately, I dont have merge permissions for this repo (I'm not a maintainer). In fact, I'm pretty sure no-one is, and this package is unmaintained despite having 1,012 dependents.

Since it is in the BioJulia org someone should have some power to be able to add new collaborators?

Whoops, I conflated this package with TranscodingStreams. I can look at #64 tomorrow

I think this can be closed?

At least until the performance in the replacement package has been improved to at least match this package.


Viewer CSS and JavaScript aren't served gzip-compressed

The DocumentCloud Viewer CSS and JavaScript aren't served gzip-compressed from the Amazon S3 servers.
Example URL:
http://s3.amazonaws.com/s3.documentcloud.org/viewer/viewer.js

Response Headers:
Date: Mon, 22 Dec 2014 17:42:35 GMT
ETag: "c461f0969585f2aaff54e405967d12be"
Last-Modified: Tue, 16 Dec 2014 18:18:14 GMT
Server: AmazonS3
x-amz-id-2: ukt2TSsLb5DM0RUlxdYsr/1pozp/gsREOVRJ9xlWIlPlniZjqA3vf8ZACmCu7mP95yWV78oKMXs=
x-amz-request-id: 16053EED6EADE536

While Amazon S3 doesn't support on-the-fly compression, you can compress the files locally and then upload them. You'll need to add the Content-Encoding: gzip header to the files.
A better solution might be to use CloudFront.

Totally legit, but since that's a platform/hosting issue (not a Document Viewer issue itself), closing this in favor of your issue here: https://github.com/documentcloud/documentcloud/issues/207


revision 2

Got to line 304 but it's getting pretty technically dense so slow going

I’m not used to github, I don’t think i am making changes correctly.
can you confirm i am using it right?
On 3 May 2017, at 11:58, brunoreis <notifications<EMAIL_ADDRESS>wrote:
@brunoreis approved this pull request.
Thanks a lot JP.

In PHPGraphQLArticle.mdhttps://github.com/brunoreis/palestraGraphQL/pull/2#discussion_r114594519:

-END OF JP REVISION 1
+GraphQL usually has only one endpoint and we make all requests there with queries like this. To find out what queries are available, we define a schema on the server. The server defines a schema that list the queries, it's possible arguments and return types.
Suggestion: To expose available queries, we define a schema on the server. The server schema list the queries, it's possible arguments and return types

In PHPGraphQLArticle.mdhttps://github.com/brunoreis/palestraGraphQL/pull/2#discussion_r114594754:

-In GraphQL, all you need is to change the query to say what fields you need.
+How would we do that in REST? Well, we probably would need to pass a parameter to specify this return type and code it inside our controller. In GraphQL, all we need to update is the query to say what fields are needed.
... all we need is to update the query to say... or ... all we need is to update the query to specify...

In PHPGraphQLArticle.mdhttps://github.com/brunoreis/palestraGraphQL/pull/2#discussion_r114595281:

-In GraphQL the client can specify in the query what fields are needed and only those will be returned on the response.
-Now, let's move into another direction and think that we need more info. We are looking at a list of articles and we want to see those articles and the last 5 comments on them. In rest, we could do something like:
+Let's consider another scenario. We are looking at a list of articles and want to the last 5 comments for each of them. In REST, we could do something like:
This is not another scenario, but we are deepening on the scenario above. Getting more info for each post. What about: "Let's move further"... or something like that?

In PHPGraphQLArticle.mdhttps://github.com/brunoreis/palestraGraphQL/pull/2#discussion_r114596883:

-I started using Youshido's lib and later migrated to Webonyx. To integrate with symfony, I used the OverblogGraphQLBundle. Having that GraphQL server layer in place, meanwhile I was evolving with the app, I tried a lot of layer compositions and code organizations to find a balanced and clean one.
+I started using Youshido's lib but later migrated to Webonyx. To integrate with Symfony, I used the OverblogGraphQLBundle. Having that GraphQL server layer in place, meanwhile I was evolving with the app, I tried a lot of layer compositions and code organizations to find a balanced and clean one. ISSUES WITH THIS PARA
What issues? Let me try to improve it...
"
I started using Youshido's lib but later migrated to Webonyx. To integrate it with Symfony, I used the OverblogGraphQLBundlehttps://github.com/overblog/GraphQLBundle to make the server available. Having that GraphQL server layer in place, meanwhile I was evolving with the app, I tried a lot of layer compositions and code organizations to finally find a balanced and clean one.
"
If it's worse than before, just throw it away. :-)

In PHPGraphQLArticle.mdhttps://github.com/brunoreis/palestraGraphQL/pull/2#discussion_r114597430:

-We are going to work on a very simple example because our goal here is to explain about our architecture, but this App can give you interesting data structures to serve as an example to deeper study:
+We are going to cover a very simple example because our goal here is simply to explain the architecture.If you are interest in diving deeperm this App can give you some other data structures to study:
I think there is a typing error here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/brunoreis/palestraGraphQL/pull/2#pullrequestreview-36074599, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AauZE68iN89RmNt9JEYd2UBWkxWJP1Zaks5r2LIygaJpZM4NOg4h.

@jpstones, I just merged those changes. This PR was already merged. I accepted it, so it was merged. So this is what I did: I made a git pull from the server, changed data, added, commited and made a push straight to the server.
This can be done in places we are ok with. To keep reviewing I think it's better to keep with the pull requests as we were doing.
Thanks.


ci: 1411 implement version

Fixes #1411
Test Plan

How do we know the code works?

.
Checklist

[x] Unit tested

Codecov Report

Merging #1412 (a1064d2) into master (2b84392) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1412      +/-   ##
============================================
- Coverage     77.34%   77.32%   -0.03%
  Complexity      719      719
============================================
  Files           250      250
  Lines          4834     4834
  Branches        922      922
============================================
- Hits           3739     3738       -1
  Misses          596      596
- Partials        499      500       +1


Rule: duplicate-annotation

I didn't know this worked until now, but it does:
# METADATA
# title: foo

# METADATA
# title: bar
rule := true

rego.metadata.rule() will simply return the last metadata block annotations, while chain() will give you all of them. I don't think it makes any sense to have duplicate, potentially "conflicting" annotations on the same scope, like in the example above. We should of course still allow the same attributes to descripe different scopes though (e.g. one description for document and one for rule).
Not sure about the category, but I would think this is likely a mistake, so leaniing towards bugs.

This does make some sense for differently scoped annotations, or multiple rule scoped annotations describing the specific incremental definition. So I'll close this as won't fix-


Change cookie policy link in cookie banner

The current cookie policy which covers wellcome.org and wellcomecollection.org is being replaced with a dedicated cookie policy page for wellcome.org as they are releasing their new cookie consent management today.
Temporarily (until our new cookie policy page and preference centre are live) we need to link to this temporary cookie policy page from our existing cookie banner: https://wellcome.org/wellcome-collection-cookie-policy

Change is in prod (cache clearing might be needed to get it)


Regression in grunt-webfont

grunt-webfont started to screw some icons during font generation, though, strangely, not all of them.
See https://github.com/sapegin/grunt-webfont/issues/321 for details.

Well, turned out that reverting to 0.4.2 doesn't help much :(


Update MD file on new branch 2017-11-27_11-41-44

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.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.


cube-ctl: avoid 'ln' errors running prep

Not all devices conform to "/dev/tty" so if you happen to run
'cube-ctl prep' on say dom0 you will get 'ln' errors as we attempt to
symlink files like:
<EMAIL_ADDRESS>We check to ensure we only attempt to create symlinks when the
matching is successful and we extract only a number.
Signed-off-by: Mark Asselstine<EMAIL_ADDRESS>

@zeddii any chance this change can be applied?

oops. I had missed this. Will have a look at it shortly.

merged. (and again sorry, I was distracted working on overc on qemuarm64 and somehow missed this).


Allows the plugin to work when it is minified.

This change allows the plugin to work even if it is minified (from the non minified dist version).

Thanks  :+1:
Normally i use ng-annotate for this purpose. but forgot to add here..


Using with ionic 2

Hello,
I'm relatively new to hybrid development - I was wondering if it's possible to use this with Ionic 2, and if so how would I go about using it?

@davidtweaver we are using It with ionic 2 without issue


can't open big file

VSCode Version: Code 1.13.1 (379d2efb5539b09112c793d3d9a413017d736f89, 2017-06-14T18:21:47.485Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author
Version

html-snippets
abusaidm
0.1.0

debugger-for-chrome
msjsdiag
3.1.5

vetur
octref
0.8.3

Steps to Reproduce:

can't open big file

can't open big file


flake: add checks.benchmark and apps.benchmark

Adds a check that runs the benchmark from the result of the
pluton:bench:perf package that our Flake outputs. This can be built
like nix build .#checks.x86_64-linux.benchmark for example. It will
also be ran via nix flake check.
Alternatively, if you want to get the effects that are generated by the
benchmark such as bench.csv, bench.json and bench.html you can use
nix run, like nix run .#benchmark.
In the implementation, checks.benchmark follows apps.benchmark.program.
Ideally, anyone modifying this code should make all changes to the
apps.benchmark.program before even attempting to modify the check. The
check is only intended to run in CI.

@MatthewCroughan Thanks! How can I tell if nix flake check is run by Hercules CI? It is not super clear just by looking at here.

How can I tell if nix flake check is run by Hercules CI?

Right now, this repo is not configured to properly use Hercules with your Flake. So only one single output derivation is being built. When it is properly configured by using https://github.com/hercules-ci/flake-compat-ci, it will build every output of the flake, which is equivalent to running nix flake check and more, as every output is built.

I had copied over that flake-compat-ci snippet from DanaSwap: https://github.com/Plutonomicon/pluton/blob/1ef06333b8148a5f62cee79b96ac644f2733e146/flake.nix#L136-L139
Is there anything more that needs to be done in flake.nix?
--
I think Hercules should recurse and build them all, but I cannot tell by looking at its dashboard. It shows logs on a per-derivation basis; rather than showing the entire build log beginning with the pluton derivation.

Right now, it's just using the derivation that is output by default.nix at the root of the repo, rather than recursing into the attributes of the Flake. I've made a PR to resolve this. https://github.com/Plutonomicon/pluton/pull/22


Why multiple request asked at the same time?

At starting of the app, I have READ_SMS permission and WRITE_EXTERNAL_STORAGE permission after signup.
Now the issue is as soon as I allow READ_SMS permission, it asks me for WRITE_EXTERNAL_STORAGE permission after that only(signup not done). How to stop this behavior and ask user permission only when required.

@vipulasri I don't quite understand your issue, can you post some code that reproduces the problem?

@samtstern I mean I am asking two different permissions on two different activities or fragments but as soon as first activity asks for a permission only related to that activity (Ex. Send OTP screen asks for READ_SMS) then another permission (i.e WRITE_EXTERNAL_STORAGE) on another activity and that screen is not yet called or displayed. Still, after allowing first READ_SMS permission, it asks for second permission WRITE_EXTERNAL_STORAGE.
@Override
    public void onRequestPermissionsResult(int requestCode, @NonNull String[] permissions, @NonNull int[] grantResults) {
        super.onRequestPermissionsResult(requestCode, permissions, grantResults);
        EasyPermissions.onRequestPermissionsResult(requestCode, permissions, grantResults, this);
    }

Do I have to change onRequestPermissionResult() method calling super

@vipulasri so you have two fragments in the same Activity requesting different permissions?  Where do you actually call reuestPermissions?

@samtstern No, I have one request permission in a fragment associated with another activity and second request permission in different activity.

@vipulasri please show me how you are requesting the SMS permission.

@AfterPermissionGranted(RC_READ_SMS_PERM)
    public void initiateOtp() {
        //do nothing, not needed as sms has already been sent
        if (EasyPermissions.hasPermissions(getActivity(), Manifest.permission.READ_SMS)) {
            initiateVerification();
        } else {
            EasyPermissions.requestPermissions(this, getString(R.string.readsms_permission), RC_READ_SMS_PERM, Manifest.permission.READ_SMS);
        }
    }

And where do you request the external storage permission?  I need some context with these code snippets, the behavior you're describing is guaranteed to be some strange issue in how your code is set up so only showing me basic code will not get to the root of the problem.

I request the external storage permission(if not permitted), whenever the user has to select a picture from the device and upload it. And request external storage permission is implemented in BaseActivity so that I can reuse the feature when needed.

Are you re-using the same request code for each (RC_READ_SMS_PERM)?  Try using different request codes in the requestPermissions and @AfterPermissionGranted annotations.

I am already using different request codes for both permission requests.

@vipulasri I am going to need to see more code to debug, there has not been enough information provided in this thread to lead to the source of the problem.  If you'd prefer this can be done on StackOverflow.

@samtstern I figured out the problem. Problem was I was having external storage permission in my base activity and the activity which was requiring permission for read sms was extending BaseActivity which was creating issue as onRequestPermissionsResult was calling its super method thus calling base activity onRequestPermissionsResult.


Change RegExpExtract to use isNull checks instead of contains_re

fixes https://github.com/NVIDIA/spark-rapids/issues/7196
Currently Regexp Extract uses contains_re to handle the case where cudf returns a NULL when there is a valid value but it doesn't match the regex but with java/Spark it needs to return an empty string. The contains_re is inefficient and about doubles the time of the extract.  instead of using that we can just use isNull and And.
The cases:
    // | Input  | Java  | cuDF  |
    // |--------|-------|-------|
    // | ''     | ''    | ''    |
    // | NULL   | NULL  | NULL  |
    // | 'a1a'  | '1'   | '1'   |
    // | '1a1'  | ''    | NULL  |

So here we can just check if the result from CUDF is NULL and the input is not null then we put in an empty string.  This PR has changed to use that logic.
Tested via integration tests and tested on customer query where this had good speed up. Query went form 3.3 minutes to 2.5 minutes.  Looked and nsys trace and the contains_re is gone. This was taking about 36ms per call and now this code takes about 3ms per call.

build

build


Add Guest and Hugepages memory to kubevirt_vm_resource_requests

What this PR does
Before this PR:
Guest and Hugepages memory requests were not included in the kubevirt_vm_resource_requests metric
After this PR:
The metric also exposes these values
jira-ticket: https://issues.redhat.com/browse/CNV-51408

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

Add Guest and Hugepages memory to kubevirt_vm_resource_requests

/retest

@machadovilaca will this allow us to get the allocated memory for every VM , both running and non-running?
We need to have this even if a request was not set specifically.
It also needs to be the exact number that the User requested.
So for example, If the user asked for 2GB memory and we are creating the VM with 2.15GB, the metric should report 2GB.

@machadovilaca will this allow us to get the allocated memory for every VM , both running and non-running? We need to have this even if a request was not set specifically. It also needs to be the exact number that the User requested. So for example, If the user asked for 2GB memory and we are creating the VM with 2.15GB, the metric should report 2GB.

It is the exact memory requested in the VM spec. And the user must always set one of these values, Resources.Requests, Memory.Guest or Memory.Hugepages

/approve

/lgtm

/cherry-pick release-1.4


can not compile v1.4.0 on Linux

here is the terminal log :
$ cargo build --release --verbose
       Fresh cfg-if v1.0.0
       Fresh lazy_static v1.4.0
       Fresh unicode-width v0.1.9
       Fresh bitflags v1.3.2
       Fresh vec_map v0.8.2
       Fresh ansi_term v0.11.0
       Fresh strsim v0.8.0
       Fresh textwrap v0.11.0
       Fresh libc v0.2.103
       Fresh log v0.4.14
       Fresh atty v0.2.14
       Fresh thread-id v3.3.0
       Fresh colored v2.0.0
       Fresh clap v2.33.3
       Fresh simple-logging v2.0.2
   Compiling walleye v1.4.0 (/home/tissatussa/Compiled/Walleye_v1.4.0)
     Running `rustc --crate-name walleye --edition=2018 src/main.rs --error-format=json --json=diagnostic-rendered-ansi --crate-type bin --emit=dep-info,link -C opt-level=3 -C lto -C codegen-units=1 -C metadata=c0314d45e350ab32 -C extra-filename=-c0314d45e350ab32 --out-dir /home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps -L dependency=/home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps --extern clap=/home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps/libclap-92b2ce1d83fefcb6.rlib --extern colored=/home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps/libcolored-af5d305c32b4ddbc.rlib --extern log=/home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps/liblog-f28a7c1eb8fddef7.rlib --extern simple_logging=/home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps/libsimple_logging-d9170c4c5c5a3ed5.rlib`
error[E0277]: `[Option<(board::Point, board::Point)>; 100]` is not an iterator
   --> src/engine.rs:214:16
    |
214 |     for mov in search_info.pv_moves {
    |                ^^^^^^^^^^^^^^^^^^^^ borrow the array with `&` or call `.iter()` on it to iterate over it
    |
    = help: the trait `Iterator` is not implemented for `[Option<(board::Point, board::Point)>; 100]`
    = note: arrays are not iterators, but slices like the following are: `&[1, 2, 3]`
    = note: required because of the requirements on the impl of `IntoIterator` for `[Option<(board::Point, board::Point)>; 100]`
    = note: required by `into_iter`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0277`.
error: could not compile `walleye`

Caused by:
  process didn't exit successfully: `rustc --crate-name walleye --edition=2018 src/main.rs --error-format=json --json=diagnostic-rendered-ansi --crate-type bin --emit=dep-info,link -C opt-level=3 -C lto -C codegen-units=1 -C metadata=c0314d45e350ab32 -C extra-filename=-c0314d45e350ab32 --out-dir /home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps -L dependency=/home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps --extern clap=/home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps/libclap-92b2ce1d83fefcb6.rlib --extern colored=/home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps/libcolored-af5d305c32b4ddbc.rlib --extern log=/home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps/liblog-f28a7c1eb8fddef7.rlib --extern simple_logging=/home/tissatussa/Compiled/Walleye_v1.4.0/target/release/deps/libsimple_logging-d9170c4c5c5a3ed5.rlib` (exit code: 1)

@tissatussa
Probably the author pushed a commit that broke the build.  I've just pulled the main branch (tagged 1.4.0) and it cleanly compiled and built on Debian 11 Bullseye, on Rust 1.55 (installed directly from the Rust-Lang website).

I got the same problem compiling on my laptop running Ubuntu. Strange that it compiled fine on windows but with this release it should compile with both
https://github.com/MitchelPaulin/Walleye/releases/tag/1.4.1

I forced pushed a fix to the 1.4 release. Strange that it would compile on windows but not on linux, might be an issue with the cargo version because the CI also runs ubuntu-latest and it seemed to pass.

now it compiles and runs OK !
only thing is the version number : it still has 1.4.0 ..

I decided to just update the 1.4.0 release directly rather than make a 1.4.1 since the change was so small. Happy it's working!


SYSTEM Requirements for covsim

Hello,
I need to ask a very basic question. What are the basic requirements (RAM, Processor, etc) for running this innovative software for my analysis?. I have COVID-19 data comprised of daily deaths, daily cases etc.
Thanks.

From my experience as a user of that software there are no special requirements for simple analyses.
RAM: Small size of simulations (like 1e3-1e4 agents) runs far below 250MB of RAM.
Realistic simulations of all individuals for a country of size 6mln - 40GB of RAM, 40mln - 250GB of RAM (includes covasim + additional data structures to read realistic data + several realistic connection layers).
Any CPU is capable to run covasim. We tested it on Intel i5/i7 “U” CPUs of 7th-9th generations and Power9 platforms. Using numba gives only around x2.5 speed improvement over disabling numba. We also haven’t observed any speed increase by enabling multithreading.
So I would guess you can perhaps run it even on raspberry pi.

Thanks for your question @furqan915 and for your comprehensive response @rlatkows ! That's correct, there are no specific requirements for running Covasim -- if you can run Python with Matplotlib you can run Covasim. If you do not have a very fast processor though, you might want to use a smaller population size (number of agents) so the simulations take less time to run however. You can scale up to the full population size using the pop_scale parameter. E.g., rather than running a simulation of 2 million agents, you could run with 200,000 agents with pop_scale=10 and get nearly the same result, but 10x faster.

Hi @furqan915  -- just letting you know that this question has been added to the FAQ in version 3.0.2.


Restore sanity.

I battled this for-loop to the brink of insanity, and emerged on the
other side victorious.  This monster behaves like a regular for-loop
except that it has the side "benefit" of crashing when the result set
has length 0 (if you take out the crash, it runs first with i=j=0 and
then i=j=-1 and then stops).

Hi there. This monster loop is really the product of CoffeeScript to Javascript conversion because the original implementation used CoffeScript as the impl source.
You are absolutely right - i inspected the script and it does produce bad JS that results in attempts to access result and erroneous index.
However, I do not want to change the Javascript implementation of this fragment too much (even though the generated JS is indeed a piece of monster conditional) as will loose the alignment with original code thus the ability to apply fixes.
I think the best way would be to just return if result.length == 0?
mycb = (result) ->
    #console.log "mycb result #{JSON.stringify result}"

    /// I WOULD ADD THIS CONDITION AS A FIX
    return if result.length <= 0

    last = result.length-1
    for i in [0..last]
      r = result[i]
      type = r.type
      # NOTE: r.qid can be ignored
      res = r.result

      q = mycbmap[i]

      if q
        if q[type]
          q[type] res

    return

@brodybits - Chris I created an issue on you repo for this.

Hi @andpor.  I was truly hoping this was machine-generated :).  Returning early if result.length === 0 was also my first idea, but I wanted to figure out that for-loop.  Not changing it too much makes sense though!

Aha!  It even makes sense what's going on when I look at the coffeescript.  Thanks for clarifying everything!


added support for running node 8.10 async/await handler

Hi!
I added support for node 8.10 async handlers that are supported by AWS Lambda now.
https://aws.amazon.com/blogs/compute/node-js-8-10-runtime-now-available-in-aws-lambda/
I also added a /test directory that has the documentation from the readme as well as an async handler function.

Hey Daniel!
Thank you for this pull request. I have added async/Promises support naturally without any input parameters.
Check the new version here: https://github.com/Max-Kolodezniy/aws-lambda-local/commit/0465c7039eb0552d2884328eb60695edd194e777

Thanks for the update! It's much better to not have to have a special flag so thanks for making that change. I wasn't sure how to tell if the handler used callback or async/await so your solution was very enlightening!


Docs: add Docker version/API version matrix

Add a table to the documentation to find which API-version relates to which Docker version

ping @moxiegirl @SvenDowideit :-)

A little unrelated, but since it seems the docker minor version, and the API version are a one-to-one mapping, why are they not the same? Why does docker 1.9.x not use API 1.9?

Why does docker 1.9.x not use API 1.9?

Beats me :D I always have to look up which API version relates to which Docker version (hence this table :))

@phemmer So that they don't have to be 1-to-1 if they don't need to be.

@phemmer So that they don't have to be 1-to-1 if they don't need to be.

I would still think the 2 can be related.

API version would never be ahead of docker version, as changing the API requires changing docker itself.
If the API doesn't change between docker versions, the API can keep the older version number. E.G. Docker 1.9.0 could use API 1.8.0.
And would this really ever happen? Would there ever be a minor release that doesn't change the API?

ping @moxiegirl ptal, also let me know if you like the suggestion @phemmer made https://github.com/docker/docker/pull/16764#discussion_r41149870, or if the current layout is okay to you

@thaJeztah I think that is a case of six of one and half dozen of the other. We now have footnotes that all say the same thing.  UX wise --- the columns are easier to hit for more people. So, I say go with the column.

LGTM as is.

Author LGTM, merging as-is, but thanks for the suggestion @phemmer, we can always change in the future


Update config.json

Thanks very much. There is an issues too with custom configs I'm trying to solve. Have a nice day


"files" searches the whole disk

Environment (please complete the following information):

OS: MacOS
Vim version: 8.2.2350
vim-clap version: master
Have you reproduced with a minimal vimrc: no
Have you updated to the latest plugin version: yes
Have you upgraded to/compiled the latest Rust binary: yes

Describe the bug
After today's pull, :files now shows results from all home directory instead of the root of the project's git
Screenshots

Works for me. I don't see how https://github.com/liuchengxu/vim-clap/pull/614/files affects this.

Works for me. I don't see how https://github.com/liuchengxu/vim-clap/pull/614/files affects this.

It's not today's changes. Maybe it was there for a while and I didn't notice. I tried to checkout vim-clap to some commit before, and still have the issue. Will try to figure out.

It's not today's changes. Maybe it was there for a while and I didn't notice. I tried to checkout vim-clap to some commit before, and still have the issue. Will try to figure out.

Added let g:clap_disable_run_rooter = v:true to my vimrc and it fixed the issue. Probably clap's rooter messes with something I have in my config.

Added let g:clap_disable_run_rooter = v:true to my vimrc and it fixed the issue. Probably clap's rooter messes with something I have in my config.

Seems like I have the same issue with grep as well. Pretty sure I didn't have this issue yesterday.

Seems like I have the same issue with grep as well. Pretty sure I didn't have this issue yesterday.

You should try to reproduce with a minimal vimrc then.

You should try to reproduce with a minimal vimrc then.

I figured something now. If I invoke files/grep on buffer/file in vim which is not yet committed into the git repo, it start the search from ~. At the same time, if I invoke it from the buffer/file that is already committed in the repo, it searches the project itself. Here is a screencast:
https://user-images.githubusercontent.com/849640/105350046-79da0580-5be2-11eb-854a-fbfddbdcc374.mov

I figured something now. If I invoke files/grep on buffer/file in vim which is not yet committed into the git repo, it start the search from ~. At the same time, if I invoke it from the buffer/file that is already committed in the repo, it searches the project itself. Here is a screencast:
https://user-images.githubusercontent.com/849640/105350046-79da0580-5be2-11eb-854a-fbfddbdcc374.mov

Can not reproduce, works for me even for the uncommitted files.

Can not reproduce, works for me even for the uncommitted files.


Allow select cases to be numbers

Currently strings which are purely numbers are not allowed to be used as select cases. This PR should fix this issue. However, due to the nature of select statements, we will still need to pass in a String as a placeholder i.e.
"conditionalString": "{level, 0{Novice} 3{Advanced} other{Amazing}}"

must be referenced with AppLocalizations.of(context).conditionalString('$level').
Fixes #116459.

@thkim1011 are you still wanting to land this one?


Import Identity from external source

As a user I'd like to import identity from either a website calling the provider, or by pasting in the identity.
Two formats that should be supported:
{
  "_trapdoor":"1235452352523",
  "_nullifier": "1234567",
  "_secret": "123567",
  "_identityCommitment":"123567"
}

['0x00000','0x00000'] //  (nullifier, trapdoor)

Acceptance Criteria:

[ ] Add import method
[x] Add confirmation screen and input name
[x] Add manual identity import with pasting the values
[ ] Save identity to the storage
[ ] Save host and update metadata type (account is optional)
[ ] Indicate that identity is imported in identity list
[ ] Support history and notification

https://github.com/Discreetly/IdentityCommitmentNullifierCircuit/tree/main

Blocked by ffjavascript fork. It's not possible to integrate it with idc-nullifier package because it depends on snarkjs.


settleAsync() and allAsync() only handle GET requests

I tried making a request object with options: { method: 'POST' } but it looks like these methods only support GET if I'm reading wrapToArray() and wrapToObject() correctly.
It would be useful to be able to pass the method in the options object.

Hey @blowback,
First of all thank you to have some time to open a issue.
Yeah, you are right, first I implemented it just to use as GET for my necessity.
Since you brought this subject to the table I'll implement it and this feature should be in the next release.

Spec:
To change that behaviour we need refactor these  two functions below to verify the if the user passed the object with the key method and after that validation call the correspondent function to each one.
function wrapToArray(myRequests) {
  return _.map(myRequests, function(myRequest){
    return exports.get(myRequest.url, myRequest.options);
  });
}

function wrapToObject(myRequests){
  return _.mapValues(myRequests, function(myRequest){
    return exports.get(myRequest.url, myRequest.options);
  })
}

Hey @blowback, just released version 0.8.0 which have this feature implemented. ( 12a1f705587728cbbe1b4c3d073693754e9d9e15 )
Give me feedback if everything it is ok for you.


Add option to show a banner with an input field

It would be great if such one, similar to the one used by the Messages app, could be added.
Thanks for this really great library!

Thanks for your suggestion, will look into it for future updates when I have the time.

That would be helpful

+1
I don't think any other in-app notification libraries have this - though I have asked about it https://github.com/hyperoslo/Whisper/issues/67

WhatsApp recently improved their in-app notifications to include a swipe to respond feature. And it's quite a pain to use because if you swipe above a certain threshold, the iOS Notification Center picks up the gesture. So it sometimes works and sometimes doesn't.

I am closing this one. I just don't think this works well after playing with WhatsApp's implementation. It is too close to the notification center, and pulling down in most cases opens the notification center, instead of the pan gesture of the notification.

Makes sense, thanks for the update!

I still think that this feature should be implemented. Wouldn't it be possible to restrict the gesture area to the lower part of the notification banner? For an additional effect the banner's height can be slightly increased. Unfortunately I am not good at ObjC (I prefer Swift), otherwise I would have tried to implement it.

There is no point anymore. With iOS 10, you can choose to display remote and user (new term for local) notifications natively. So really, I will be deprecating this framework soon. It is good for what it does on iOS 8 & 9, but with 10 you just get the real, native system implementation.

@LeoNatan - do you mean notifications will be shown even if the app is active?

Yes indeed, when handling notifications using the new API (UNUserNotificationCenterDelegate), you have an option to specify whether the notification should be displayed inside the application.


Adds datafile text field for Data File object query

For querying Data File objects- I added the field that properly specifies the text of that datafile.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.01%. Comparing base (e64620b) to head (89c72cf).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   96.01%   96.01%
=======================================
  Files           6        6
  Lines         602      602
=======================================
  Hits          578      578
  Misses         24       24

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add route to count comments, add count to GET request result

Addresses #78.
Get a count for a url with host:port/count?url=someurl.with/comments.
Also adds count to comment GET request. Successful response for /get is now:
{"success":true,"message":"","comments":[...],"count":5}
Response for /count:
{"success":true,"message":"","count":1}

unack https://github.com/adtac/commento/pull/85/commits/befd5b3928e296e86744825e07854df0a844c09c

@smbkr ping here too :)

@adtac so sorry! been massively busy lately! I'd still like to fix this & my other PR with the suggested changes, will take a look at my lunch today & will also attempt merging master into this branch so that conflicts can be avoided.

@smbkr no problem, it's open source; everyone is just a volunteer.
And thanks for getting this going again!

@adtac I've just pushed some commits which I believe address all the issues raised - I've removed unnecessary casts and pointers, changed the switch in the countComments method to a standard if/else, and also added the updated count when creating a comment.
I've also removed omitempty from the count, as the value 0 is treated as empty, and we want this to be returned in the response when the actual count is zero. Previously I had used this as I was not returning a count when creating a comment, so I wanted that key to be omitted, however I think it makes sense to return the updated count after a new comment is created. This was a count is returned with /get and /create, and can be retrieved on its own with /count. What do you think?

@smbkr Neat stuff, thanks!
Just one tiny thing: please use rebases instead of merging from master. This helps us maintain a linear history and they are good for a number of reasons.
Alternatively, you can hand over the PR to me and I'll cherry-pick your commits (author information will be intact) in a larger PR. I'll probably be doing this anyway since there are multiple huge PRs close to merging, so it's really not a problem for me.

Cheers @adtac - I'm happy for you to cherry pick. Same goes for the other PR. I'm used to just merging master into my working branch since that's the way we do it at my work, however I can see the benefit of rebasing instead. I'll try that next time!

Thanks for letting me know, @smbkr :)
I'll comment here when they get merged through a cherry-pick (just as a heads-up).


CLI is laggy due to slow imports

In particular, jax and matplotlib imports really slow things down. It would be nice if at least dinf --help and dinf-plot --help were more responsive. I like having imports at the top of files, but maybe make an exception in the cli code?
(venv) t490:src $ time python -c "import jax"

real    0m0,689s
user    0m0,879s
sys     0m1,236s
(venv) t490:src $ time python -c "import matplotlib"

real    0m0,246s
user    0m0,379s
sys     0m0,735s
(venv) t490:src $ time python -c "import matplotlib.pyplot as plt"

real    0m0,620s
user    0m0,745s
sys     0m0,758s
(venv) t490:src $ time python -c "import dinf"

real    0m0,982s
user    0m1,110s
sys     0m1,342s
(venv) t490:src $ time python -c "import dinf.plot"

real    0m1,552s
user    0m1,656s
sys     0m1,337s
(venv) t490:src $ time dinf -h > /dev/null

real    0m0,985s
user    0m1,093s
sys     0m1,303s
(venv) t490:src $ time dinf-plot -h > /dev/null

real    0m1,542s
user    0m1,650s
sys     0m1,342s

Get an awesome profile of import times using importtime builtin and tuna package:
python -X importtime -c 'import dinf' 2> /tmp/import.log
tuna /tmp/import.log  # opens graphic in browser

We now lazily import scipy, which helps a lot. Also, jax has done a lot of work to reduce import times. The lag is still not great, but it's much better than it was and it's probably as good as it's going to get.


SetLocal's don't work

In common.nim, in the definition of []= for self: CallFrame self.stack[self.getAbsIndex(idx)] = val is used, but the []= syntax is invalid for sequences. This crashes the REPL on every SetLocal call.

Might be #5 in disguise.
since []= need to refer to indexes that exist, but so does SetLocal actually

Update, see src/frame.nim instead


No acceleration on 1.0.0 example with USE_NCCL=1

I'm performing the image classification fine-tuning example on 4 V100 on a p3.8x:
python fine-tune.py --pretrained-model imagenet11k-resnet-152 --gpus 0,1,2,3 --data-train caltech256-train.rec --data-val caltech256-val.rec --batch-size 64 --num-classes 256 --num-examples 15240 --num-epochs 1 --lr 0.005 --kv-store nccl

Surprinsingly, the throughput I've got was even 3% lower than with --kv-store device
I compiled mxnet 1.0.0 with:
 make -j $(nproc) USE_OPENCV=1 USE_BLAS=openblas USE_CUDA=1 USE_CUDA_PATH=/usr/local/cuda USE_CUDNN=1 USE_NCCL=1 USE_NCCL_PATH=/usr/local/nccl/

Using NVIDIA Driver 384.81 and CUDA 9.0, on  Deep Learning Base AMI (Ubuntu) Version 2.0 (ami-10ef8d6a)

nccl is only faster in some cases, usually when batch size is small

I've tested with batch size 16 with only 1 GPU. Quite small batch size.  And 64=16x4 with 4 GPUs. Without acceleration in both cases.


ControlFlowIndentation apparently produces wrong indentation

Certain sources seem to cause the ControlFlowIndentation to produce wrong indentation. One example is https://github.com/Netflix/hollow/blob/d5309ac0a2cde4af37e9f7177db067ad0de144f5/hollow/src/main/java/com/netflix/hollow/core/read/filter/TypeFilter.java where the recipe does this (observed on public.moderne.io):
                  .flatMap(i -> {
                String field = os.getFieldName(i);
                Action fa = rule.apply(type, field);
-                            if (fa == next) return Stream.empty();
-                            TypeActions child = newTypeActions(type, field, fa);
-                            Action descendantAction = fa.recursive ? fa : action;
-                            if (descendantAction.recursive && os.getFieldType(i) == REFERENCE) {
-                                String refType = os.getReferencedType(i);
-                                HollowSchema refSchema = schemas.get(refType);
-                                assert refSchema != null;
-                                Stream<TypeActions> descendants = descendants((t,f) -> descendantAction, refSchema);
-                                return Stream.concat(Stream.of(parent, child), descendants);
-                            } else {
-                                return Stream.of(parent, child);
-                            }
+                    if (fa == next) return Stream.empty();
+                    TypeActions child = newTypeActions(type, field, fa);
+                    Action descendantAction = fa.recursive ? fa : action;
+                    if (descendantAction.recursive && os.getFieldType(i) == REFERENCE) {
+                        String refType = os.getReferencedType(i);
+                        HollowSchema refSchema = schemas.get(refType);
+                        assert refSchema != null;
+                        Stream<TypeActions> descendants = descendants((t,f) -> descendantAction, refSchema);
+                        return Stream.concat(Stream.of(parent, child), descendants);
+                    } else {
+                        return Stream.of(parent, child);
+                    }
            });

case SET:

The diff shown here is a bit misleading. The diff shown in the SaaS makes it clearer that the indentation looks wrong. Also note that AFAICT it is not an issue of tabs vs. spaces.


Extension sadly stopped working.

"Loading gists..." in loop.
"Error running command gistpad.refreshShowcase: Remote end closed socket abruptly.. This is likely caused by the extension that contributes gistpad.refreshShowcase."
Tested with new created GH token.
Already reinstalled extension.
Tested on fresh VSCode.
"GitHub Gist Explorer" Extension works again.

Hmm I'm not able to repro this 🤔
Did you say you were able to work around it by signing out and back into GH?

++ just came here after trying gistpad for the first time. In my case I am unable to sign in -- I get the browser popup (macos - can be safari or chrome), but the 'signing in to github.com' pop up pane never disappears

Seems to be working great for me as well. So I'm going to close this as no-repo. Let me know if you run into any issues. Thanks!


coverage: Report mean coverage (feature request)

I'd like to calculate mean coverage as per this post.
bedtools coverage reports the number of a features that overlap b. Perhaps it could also report mean coverage or alternatively the sum of the lengths of the a features that overlap b, from which mean coverage could easily be calculated.
This works
bedtools coverage -d -abam a.bam -b b.bed |datamash -g 1,2,3 mean 5

brew install datamash

Added -mean option in #247.

Thanks, Neil!


Update CHANGELOG

@mpkorstanje, I've updated the changelog. Would you mind going through the polyglot steps? I don't have perms to do it...

I've added you to build team. You should have permissions pending your acceptance of the invitation.

Yup. Thanks!

Created a new pull request to fix the installation of Berp. Included the CHANGELOG in that.