fix: IOU Scan - In dark mode, the damaged PDF - file is barely visible.

Details

Fixed Issues
$ https://github.com/Expensify/App/issues/39356
PROPOSAL: https://github.com/Expensify/App/issues/39356#issuecomment-2029731489
Tests

Open workspace chat
Tap plus icon > request money > scan
Upload corrupt pdf attached below
Verify the "Failed" message is clearly visible in both dark and light mode.

[x] Verify that no errors appear in the JS console

Offline tests

Open workspace chat
Tap plus icon > request money > scan
Upload corrupt pdf attached below
Verify the "Failed" message is clearly visible in both dark and light mode.

QA Steps

Open workspace chat
Tap plus icon > request money > scan
Upload corrupt pdf attached below
Verify the "Failed" message is clearly visible in both dark and light mode.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/85894871/b898f14b-5e38-4653-82f8-36e1ae1a4984

Android: mWeb Chrome
https://github.com/Expensify/App/assets/85894871/847b8896-ecac-441e-8587-830241f6ea2e

iOS: Native
https://github.com/Expensify/App/assets/85894871/1f8e5078-2f82-4a0e-a1a3-1a884f88c6a2

iOS: mWeb Safari
https://github.com/Expensify/App/assets/85894871/48155d81-4d32-4b58-950f-c94716620ed5

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/85894871/3acb99d6-847b-4d1b-9446-02810c446096

MacOS: Desktop
https://github.com/Expensify/App/assets/85894871/1c2164dd-8060-4781-b24c-70d06b2ca8bb

@Krishna2323 When you push the changes for native devices, please ping me.

@Expensify/design Please confirm the styling of the Failed to load PDF file. text?

I would think instead of showing text like we are, we'd show our default fallback receipt view, something like this that @dannymcclain had suggested previously:

Though I think I would change out the icon to use some kind of broken image icon, something like this but in our brand style:

@shawnborton I thought in a different issue we had settled on not allowing someone to upload a corrupted image/PDF? We show the standard modal that says "Error uploading attachment..." This was the mock for that:

Am I misremembering something?

You are definitely right. So yeah, that begs the question, how did we trigger this particular "Failed to load PDF" scenario?

@shawnborton @dannymcclain To reproduce this issue, I am following this steps:

Go to workspace chat > + > Split Expense > Scan > Upload a corrupt pdf

https://github.com/Expensify/App/assets/35371050/1ca7f71b-8b01-4981-9923-150f29d21817

Corrupt pdf
corrupted.pdf

Yeah, I feel like a much better solution is to prevent a corrupted PDF from being uploaded in the first place.

@hayata-suenaga @shawnborton If we are going to prevent a corrupted PDF from being uploaded, then we can close this PR and close/hold the issue in favor of https://github.com/Expensify/App/issues/34032

That's where my head is at... it seems like we won't ever be able to get to this scenario once we prevent corrupted images/files from being uploaded.

it seems like we won't ever be able to get to this scenario once we prevent corrupted images/files from being uploaded.

I don't think we know the image/file is corrupt purely on the client, so isn't it possible you'll run into this when uploading a corrupt file offline?

Hmm good question, who can confirm that?
If that's the case, I like doing something like my suggestion here.

Yeah I think any of these would be great! I think I like doing something super generic in case this is a pattern we'd repeat not only for PDF-specific receipts but for all receipts in general that are corrupt. So that being said, I lean towards the first option?

Yeah first option makes sense to me—that way we could use it for any file type of receipt that borked.

I don't think we know the image/file is corrupt purely on the client, so isn't it possible you'll run into this when uploading a corrupt file offline?

I'm not familiar with how we're handling the PDF upload on the client side, but we try to display a preview of the PDF before the user attempts to send it. If the PDF is corrupted, we know that when we try to display the preview.
@Krishna2323, could you confirm that you seem to be working on the issue to prevent the corrupt PDF from being uploaded?

Regardless, I think it's a good idea to still implement the fallback view if the attachment is corrupted though.

Ok, so I guess, we agree on showing fallback for corrupted files, will start implementing the changes today and update.

Yes sorry if that wan't clear 🙇 thank you for pushing the PR forward 👍

I don't think we have Receipt Slash in the app, can you pls share it.

@Krishna2323 No prob!
Receipt Slash.svg.zip

Hmm I could have sworn we already have a pattern in place for receipt placeholders, like these:

Can we match that size? From Figma, it looks like they are at 80x80.

Two questions:
Why can't we detect the corrupted PDF immediately, the same way we do for images?
How did you get a camera view like this where the capture button is off to the right? Looks like we're missing a flash icon?

Why can't we detect the corrupted PDF immediately, the same way we do for images?

Also curious about this. It feels pretty weird to drop the user back at the contact selector when they acknowledge the warning. They don't need to pick a new contact, they need to upload a different file. So that feels pretty strange to me.

Hmm yeah, I'm a bit stumped so I would be curious for what our engineering team thinks.
I think the reality is, we haven't had a single customer ever report this corrupted PDF flow. This was a super edge case flow reported by our QA team. So I'm hesitant to spin the wheels too much here, given that I think this is not going to be any kind of mainline flow that we should optimize for.
If what you are proposing is the best we can do with our current limitations, I think it's probably our best bet.

@dannymcclain @shawnborton, we can push the user to file selector page instead of participants selector but we can't detect the pdf immediately. For images we use react-native-image-size package which works on all platforms but we don't have any package to detect corrupt pdf's. We first need to render it with different libraries for native and web and if the render fails we get the error. You can try uploading a protected pdf, you won't get any alert until you reach the confirmation page.

My knowledge of PDF flow is also very limited. But one question. We still display the corrupt PDF on the front end before uploading the PDF to the backend, right? If the file is corrupted, why the front end can display the file?

@Krishna2323 Remove HOLD from the PR title and resolve the conflicts In local I am getting conflicts with the latest main branch

resolve the conflicts In local I am getting conflicts with the latest main branch

@Krishna2323 bump for this comment

@Krishna2323 Please Update Tests Section something like this:

1. Open App > Press FAB Menu > Submit expense > Select the Scan tab
2. Upload corrupt pdf(attached below) > Select any participant
3. Verify That We can see the Attachment error Modal, and in the background, we can see the [Receipt Slash Image](https://github.com/Expensify/App/pull/40607#issuecomment-2082744344)

Below the Tests Section, attach the corrupt pdf file for testing. Also, can you help me understand how I can test these steps? I can test the following:
4. Verify transactions preview also shows receipt slash component
5. Click on transactions preview and verify receipt slash component is also displayed on transaction preview
6. Open transaction details page and verify receipt slash component in place of receipt place holder

The merge freeze is being gradually being lifted with the urgent PRs being merged first
I don't think this PR needs to be merged urgently, so let's wait until we have complete merge freeze lift 😄

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to production by https://github.com/luacmartins in version: 1.4.81-11 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Editor: Dropdown for exported enum variables shows only the first option

Godot version
v4.0.alpha.custom_build [947a1fa09]
System information
Linux (Fedora 35)
Issue description
When exporting a variable typed as an enum, the editor dropdown shows only the first enum value. Tested with both nodes and resources.
If an existing resource or scene is opened where the enum was already set, it shows as the first option but still has the reset button.
The enum in this screenshot has three values:

Steps to reproduce
Create a script extending Node or Resource. Add a variable statically typed as an enum and export it. Example:
extends Node

enum NamedEnum { VALUE_1, VALUE_2, VALUE_3 }

@export var enum_var: NamedEnum

Try to set this value in the editor, you will see only value 1.
Also happens when using buit-in enums:
extends Node

@export var builtin_enum: Viewport.DebugDraw

Only debug draw disabled is available.
Minimal reproduction project
EnumIssue.zip
Includes a node exporting both a custom enum and a built-in enum, and a resource exporting a custom enum.

Did some git bisecting, #60881 seems to be where this was introduced.

Thanks for reporting, i was facing the same issue. I found the cause and submited a PR.

Fixed by #60989


diesel setup doesn't work with some postgres URIs

For example, I need to specify a username and password, so I've created an  .env file containing:
DATABASE_URL=postgres://dev:password@localhost/my_db
When I run diesel setup it panics with:
Creating database: my_db
thread '<main>' panicked at 'FATAL:  database "dev" does not exist
', /home/user/.multirust/toolchains/nightly/cargo/registry/src/github.com-88ac128001ac3a9a/diesel_cli-0.5.3/src/main.rs:235
note: Run with `RUST_BACKTRACE=1` for a backtrace.

Sorry about that!
Turns out that when dropping/creating a database, we connect to the user's DATABASE_URL minus the database, so if a user is supplied, we would connect to postgres://user@localhost, which postgres interprets as us wanting to connect to the user database as well.
Anyways, I pushed up a fix here: https://github.com/sgrif/diesel/pull/248


Fix ReDoS in lodash

Update the lodash.trimend version to fix Regular Expression Denial of Service (ReDoS) in lodash (https://github.com/advisories/GHSA-29mw-wpgm-hmr9)

Hi,
Thanks for the PR:
We needed the update across multiple sub libraries and it was addressed as part of https://github.com/microsoft/Recognizers-Text/pull/3125.
Closing as duplicate in favor of mentioned PR.


User feedback

[ ] submitting during healthcheck is a pain
[ ] split languages automatically when delim ;, , found
[ ] grammar check

split languages automatically when delim ;, , found

It now occurs to me that most of my language codes are ISO 639-1 (two-letter). Since these cannot be confused with the three-letter codes used in Lindat, could the importer recognize them as well?


"Kernel not found" doesn't list all possible kernels

I am opening a new notebook and the kernel is not found.  I get this dialog box:

The problem is that list is not complete.  It is missing two kernel options, "R [default]" and "SAS", which I can see at a later point (if I select "Continue without kernel"):

This is inside AEN.  Relevant package versions:
$ conda list | grep -i -e nb_ -e notebook -e jupyter
_nb_ext_conf              0.2.0                    py35_0    defaults
jupyter                   1.0.0                    py35_3    defaults
jupyter-client            4.3.0                     <pip>
jupyter-console           4.1.1                     <pip>
jupyter-core              4.1.0                     <pip>
jupyter_client            4.3.0                    py35_0    defaults
jupyter_console           4.1.1                    py35_0    defaults
jupyter_core              4.1.0                    py35_0    defaults
nb_anacondacloud          1.1.0                    py35_0    defaults
nb_conda                  1.1.0                    py35_0    defaults
nb_conda_kernels          1.0.3                    py35_0    defaults
notebook                  4.2.1                    py35_0    defaults

If any of those are out of date, please advise the safe way to update them.
I know some of these are out of date, but I do not know how or where to safely update them.

The problem is that list is not complete.

I think it is complete... that notebook was saved with a python kernel and when it can not find a Python3 kernel name, it shows you all the "python" kernels available, not the R or the SAS... this is a behavior from upstream trying to match compatible kernels...

OK, good to know, and that behavior makes sense.  In my case, I was switching between language kernels inside the notebook.


mdx extension is not detected as content page

with the option strictNuxtContentPaths: true, files with mdx extension are not visible in generated sitemap.
Probably there is a strict checking just for md extension.

Oops, thanks. Have pushed it now


[python] Sync TileDB-Py and TileDB-SOMA Python query-condition code

There have been several indepedent updates to the query condition code in TileDB-Py and TileDB-SOMA Python APIs that should be synced. Divergence has been introduced by both repos. For example, although the TileDB-Py code has support for query conditions on  dimensions and Boolean types, these are still missing in TileDB-SOMA. On the other end, TileDB-SOMA has been updated to use attrs but TileDB-Py still uses dataclasses.

@nguyenv did #1756 achieve this goal, as a side effect?

No, and in fact both #1756 and #1793 have introduced more divergences from the TileDB-Py code. For example, init_query_condition take in completely different arguments, and the TileDB-SOMA code retrieves info from a passed in ArrowSchema instead of TileDB ArraySchema (good because this means we getting rid of TileDB-Py dependencies in the SOMA code). This issue should probably be closed since the query condition codes are too different at this point.


(Bug) email confirmation step missing

When using self custody on dev env after entering email there's no confirmation step to enter code.

[ ] make sure we have this step when email is missing from login method (ie signup using passwordless/self custody or facebook mobile)
[ ] verify we have this step on production, when user signs up using passwordless

maybe dup of #3617

DEV
https://images.zenhubusercontent.com/166818382/c4862376-db9e-4fcb-8daf-4c0b84fc9b41/2a55ed14_0bbc_4c21_bd60_8bd1c574b91e.mp4
https://images.zenhubusercontent.com/166818382/7ee0a7db-0490-4313-bd7a-fdf8038e80be/b0d1f523_5afa_4ea3_aa74_668d90438311.mp4


About immersion status bar for Android

I want to get support for Android immersion status bar.
When i use the wxc-minibar，i don't want I need to add support for immersion status bar in Android program
Hope weex-ui adds Android and IOS native component library to facilitate the use of some native component-specific features

I'm sorry
Now weex UI is more targeted at UI layer components while allowing users to be incompatible.
Here, About immersion status bar for Android needs to modify native code, which is not suitable for UI layer implementation. It can modify native side support by itself.


Summoned minecarts not linking

BkCommonLib version: 1.12.2-v4
TrainCarts version: 1.12.2-v2 Build 246
Spigot version: 1.12.2 git-Spigot-c3093ef-7e94e65

Problem or bug:
Minecarts spawned with the summon command are not capable of linking when pushed together.
Expected behaviour:
Summoned minecarts should link to provide a wider choice of customisation, such as blocks on top of them, display text, etc.
Steps to reproduce:

Spawn two carts with /summon minecart
Try to push them together

does the minecart you manually place down link up?

Yes, they do link

but what is the purpose of using summon instead of placing them down?

Customization. I was trying to show a custom name, with custom blocks rotated inside. Things that this plugin does not yet support. If I were to use simple minecarts, of course I'd place them down normally, not take the longest way

Try using the traincart built in spawner sign https://minecraft.gamepedia.com/Custom_servers/Bukkit/TrainCarts/Signs/Spawner , and then use entitydata to alter the nbt of the minecart, and see if it works

btw, if you have a discord account, you can join the official Traincarts / BKCommonLib discord server here for a more interactive and direct support https://discord.gg/P34bUPT

Will try when I got a bit of time. I already joined the discord group, asked, and nobody answered for hours, so I just quit and filled an issue here instead


Update http4s-blaze-client, ... to 0.21.31

Updates

org.http4s:http4s-blaze-client
org.http4s:http4s-blaze-server
org.http4s:http4s-circe
org.http4s:http4s-dsl

from 0.21.28 to 0.21.31.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s" } ]

labels: library-update, semver-patch

Superseded by #611.


Update README1.md

a discription is needed to fill this space
This is my first pull request.

this is where i get to rip on you!


Version 1.0.1

Landed in https://github.com/metarhia/metawatch/commit/9a2dba3ec38f11f4cbf9afb5725fbbbcf5a0ea92


[hotfix][doc] fix doc of externalized checkpoint

What is the purpose of the change
This PR intend to fix the incorrect doc of externalized checkpoint.
Brief change log

fix doc of externalized checkpoint

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
no
Documentation
no

Travis failed is unrelated, it cause by the concurrency problem in DefaultOperatorStateBackend when performing checkpoint, which this PR try to fix.

The configuration (config.md)should be generated from the config options by now, so not be manually edited.
(@zentol could you chime in here?)

@StephanEwen @alpinegizmo Thanks for your comments, I will revert the changes of config.md and address the comments concerning to the  `file://` only for local setups.

@zentol I have addressed your comments.

In Flink 1.5, all checkpoints are externalized. There is no notion of externalized checkpoints any more, just a setting to configure the retention (retain on cancel, retain on fail, never retain).
The state.checkpoints.dir is the recommended option for the directory where checkpoints are stored (and state.backend.fs.checkpointdir is a deprecated key for the same setting).

Hi guys, I think maybe the doc that related to the Externalized Checkpoint should be updated for 1.5. Are there any other suggestions? And if there is anyone who is willing to take over this plz feel free to do that, I would glad to close this PR if someone willing to take over this...


Tests don't run in node

I don't have a html file to set because I want to run pure JS tests in node.
I tried to set my JS file via the context menu but then running the tests does not work.

Hi there - unfortunately it can only work with the html at the moment because it parses the html output. What package are you using to run the tests?


Remove an element from array

🚀 Feature
(A clear and concise description of what the feature is.)
Have you read the Contributing Guidelines on Pull Requests?
YES
Motivation
This is a good level problem for building programming concepts.
Pitch
This is a good level problem for building programming concepts. Moreover, this is an important interview prep question which should be practiced for good coding.

I am a gssoc21 participant, and I would like to contribute to this issue in java. plz assign this to me.

/assign


Should #helpers be class_eval-ing the passed in mod?

 def helpers(new_mod = nil, &block)
          if block_given? || new_mod
            mod = new_mod || Module.new
            define_boolean_in_mod(mod)
            inject_api_helpers_to_mod(mod) if new_mod

            inject_api_helpers_to_mod(mod) do
              mod.class_eval(&block)
            end if block_given?

            namespace_stackable(:helpers, mod)
          else
            mod = Module.new
            namespace_stackable(:helpers).each do |mod_to_include|
              mod.send :include, mod_to_include
            end
            change!
            mod
          end
        end

mod could be new_mod which means it is having methods injected into it. If you assign something like helpers Some::Poor::ExternalModule it now will get methods included into it from the block due to the mod.class_eval(&block).
Is that correct? This doesn't seem like a side-effect we would want.

That definitely sounds reasonable. Try writing a test for this and hopefully a fix.


Update bitcoind dependency to 0.21.0 to fix M1 builds

same M1 build failure that (https://github.com/RCasatta/bitcoind/pull/38) solved is causing electrsd not to build on M1, which is causing bitcoindevkit build failures on my machine
Compiling electrsd v0.12.0 (/Users/futurepaul/dev/rust/other-peoples-code/electrsd)
error[E0425]: cannot find function `download_filename` in this scope
  --> /Users/futurepaul/.cargo/registry/src/github.com-1ecc6299db9ec823/bitcoind-0.20.0/build.rs:49:29
   |
49 |     let download_filename = download_filename();
   |                             ^^^^^^^^^^^^^^^^^ not found in this scope

error[E0277]: the size for values of type `str` cannot be known at compilation time
   --> /Users/futurepaul/.cargo/registry/src/github.com-1ecc6299db9ec823/bitcoind-0.20.0/build.rs:62:9
    |
62  | /         println!(
63  | |             "filename:{} version:{} hash:{}",
64  | |             download_filename, VERSION, expected_hash
65  | |         );
    | |__________^ doesn't have a size known at compile-time
    |
    = help: the trait `Sized` is not implemented for `str`
note: required by a bound in `ArgumentV1::<'a>::new`
   --> /Users/futurepaul/.rustup/toolchains/stable-aarch64-apple-darwin/lib/rustlib/src/rust/library/core/src/fmt/mod.rs:293:20
    |
293 |     pub fn new<'b, T>(x: &'b T, f: fn(&T, &mut Formatter<'_>) -> Result) -> ArgumentV1<'b> {
    |                    ^ required by this bound in `ArgumentV1::<'a>::new`
    = note: this error originates in the macro `$crate::format_args_nl` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0277]: the size for values of type `str` cannot be known at compilation time
   --> /Users/futurepaul/.cargo/registry/src/github.com-1ecc6299db9ec823/bitcoind-0.20.0/build.rs:67:19
    |
67  |           let url = format!(
    |  ___________________^
68  | |             "https://bitcoincore.org/bin/bitcoin-core-{}/{}",
69  | |             VERSION, download_filename
70  | |         );
    | |_________^ doesn't have a size known at compile-time
    |
    = help: the trait `Sized` is not implemented for `str`
note: required by a bound in `ArgumentV1::<'a>::new`
   --> /Users/futurepaul/.rustup/toolchains/stable-aarch64-apple-darwin/lib/rustlib/src/rust/library/core/src/fmt/mod.rs:293:20
    |
293 |     pub fn new<'b, T>(x: &'b T, f: fn(&T, &mut Formatter<'_>) -> Result) -> ArgumentV1<'b> {
    |                    ^ required by this bound in `ArgumentV1::<'a>::new`
    = note: this error originates in the macro `$crate::__export::format_args` (in Nightly builds, run with -Z macro-backtrace for more info)

Some errors have detailed explanations: E0277, E0425.
For more information about an error, try `rustc --explain E0277`.

Thanks for reporting, i just updated bitcoind to fixed version and released 0.13


feat: upgrade deps, new prefix option, cleaning and typos

Hello, this PR contains a few improvements:

An update of all dependencies
A few typographical corrections
use of any rather than interface{} or http.NoBody, rather than nil
A new prefix option

This new prefix option is useful when you have an API whose server spec contains a subfolder. In my case, it's /api. But as I register my api with server.RegisterHandlersWithBaseURL(s.EchoServer, server.NewStrictHandler(s.APIService.API, nil), "/api").
The prefix is not visible to the router, so the validator can't see it either. Setting this new option to /api virtually removes the prefix. And makes the validator work properly.
close: https://github.com/oapi-codegen/echo-middleware/issues/11

@jamietanna gentle ping, can I have your input here ?

@jamietanna hello ?


Set-CustomLayoutsPage example

Changed example to include a SPCustomPage parameter, not a string or url

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Set-SPCustomLayoutsPage.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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.


Update with info on cloning an existing projects in VS

Minor edit based on new support for cloning an existing repo with Dev Drive in VS.

Thanks @Mikejo5000! Merged.


[multistage] UNION queries produces wrong sorting results

SELECT val FROM (
SELECT val, ord FROM (
  SELECT CAST(1 AS INT) AS val, 1 AS ord
  UNION ALL
  SELECT CAST(3 AS INT) AS val, 3 AS ord
)
UNION ALL
SELECT val, ord FROM (
  SELECT CAST(2.1 AS DOUBLE) AS val, 2 AS ord
  UNION ALL
  SELECT CAST(4.1 AS DOUBLE) AS val, 4 AS ord
)
)
ORDER BY ord

throws exception in final order-by comparison b/c of different data type (using either both DOUBLE or both INT are fine.

Hi, I first reported the bug, and here is a pdf file that I made to report it. It has some test cases and results I explored.


model license?

Hello! Thank you so much for sharing the code and the model.
I would like to ask, what protocol is the pre-training model dpt_large? Can I use it in a business project?
Your reply is very much needed.


Sidekick Bot down

I have several servers and on one of them the Sidekick bot suddenly just appears as offline and I cannot fix it. Everywhere else it's working I can even PM the bot without problems. Can you help me out? (I already tried kicking and reinviting the bot, fiexed nothing, problem stayed the same)

Issued a full restart.
On Thu, Nov 29, 2018, 2:35 PM Sarafelen<EMAIL_ADDRESS>wrote:

I have several servers and on one of them the Sidekick bot suddenly just
appears as offline and I cannot fix it. Everywhere else it's working I can
even PM the bot without problems. Can you help me out? (I already tried
kicking and reinviting the bot, fiexed nothing, problem stayed the same)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ArtemGr/Sidekick/issues/154, or mute the thread
https://github.com/notifications/unsubscribe-auth/AACjd2XguTJNfdzqUSNDRueCvvfwKfQsks5uz8aHgaJpZM4Y5fCV
.

Thx now it is back online and working!


Fix #127, add "pushbutton" sound (questionable), make raccoon guy better

Thanks for making me a member, Hugo. What privileges does it give me? Not editing rights, apparently. (Not complaining.)

Sorry for the delay in responding. Would it be possible for you to remove the 2x2 tile property for the time being? I had a different idea for implementing those without taking over a tile property slot.
I wanted to implement tile grouping like I did in HEC.  It's somewhat similar to the existing "objects" feature, except it ties the group to the top-right tile:

It's in Release 1.4, so removing it might be a bit odd, but it'd take me about 5 minutes if you really want me to.

Alright, now that I finally had time to work out these features, it's time to merge. I've tested them as much as I could, but do make sure to test it further so we might find lurking bugs and errors 👍

It's a bit odd that you pushed commits to my fork, but ok. I don't think the "unbind" key is working, FYI.

I figured that, since I was going to merge your fork, I'd use the opportunity and push everything together :P
The "unbind group" key has to be used in the tiles screen, like so:

And what exactly is odd about renaming objects? From my testing, it seems to behave fine.


Weird pair of commas added to each cloze selection on Mac

Exactly what it says. If I try to cloze this:

Here is an example sentence.

and cloze "example", I'd expect it to do this:

Here is an {{c1::example}} sentence.

Instead, it does this (commas bolded for emphasis):

Here is an {{c1::example**,,**}} sentence

Any way I can remove those commas without individually removing each of them, which is a pain when I'm adding dozens or hundreds of cards at once, each with multiple clozes?

I don't have any Mac. I will see if I can get any Mac to test.


No such file or directory when using User Assigned Identity

symptoms
When runing rover with MSI authentication, landing zone operations fail with /tf/rover/functions.sh: line 470: ***: No such file or directory.
rover versions:
aztfmod/rover:0.13.6-2104.1611
aztfmod/rover:0.14.10-2104.1611
aztfmod/rover:0.15.0-2104.1611
error
@calling process_actions
@calling verify_parameters
landingzone                   : '/__w/2/s/landingzones/caf_solution/add-ons/azure_devops_agent'
@calling_deploy
@calling get_storage_id
@calling_get_logged_user_object_id
 - AZURE_ENVIRONMENT: AzureCloud
 - ARM_ENVIRONMENT: public
 - logged in Azure with User Assigned Identity: (MSIResource-***)
/tf/rover/functions.sh: line 470: ***: No such file or directory
Error on or near line 470; exiting with status 1
Error on or near line 563; exiting with status 1

If you repro the issue, you can use the following devrover: aztfmod/rover-preview:0.15.0-2104.260258 and let us know your feedbacks!


Optimize using time-memory tradeoff

Instead of searching for quadruplets (a,b,c,d) s.t. a^5+b^5+c^5+d^5 = N^5, pre-calculate all possible N^5-d^5 and search for triplets (a,b,c), reducing time complexity by an order of magnitude. This runs around 50-100x faster in my experience.
Also, refactor to only return the first result, and added a small test.

yyogo, Thanks! Now, I'm worried it will run too fast. I wonder if we can set it so the input is how many counterexamples to look for and then it goes that far and stops.

@yyogo Thanks! HundredLuck merged your speed up with theirs. The new version of the web page is now 100-times faster. (https://carlkcarlk.github.io/shortestpaper/) Thanks!


Monitoring vespa process appropriately to avoid the lost of document updating

Hi.
Would you please let me ask a couple of questions regarding vespa monitoring?
Current status.

We use vespa with several servers.
The Version is vespa-7.105.45-1.

https://github.com/vespa-engine/vespa/tree/vespa-7.105.45-1

We have several vespa clusters for redundancy
Each cluster has 30 vespa servers
When we do product release, we deploy each cluster one by one

Trouble.
A few days ago, we deployed vespa clusters.
After that, we realized the number of documents in a cluster was different from the other clusters as below.

Vespa cluster 1 documents: 1000
Vespa cluster 2 documents: 1000
Vespa cluster 3 documents: 1001    <-  vespa cluster 3 has surplus document!
Vespa cluster 4 documents: 1000

In addition, we found that this surplus document was the one removed a few weeks ago.
Which means possibility that update process did not have worked well at some vespa server in that cluster.
Our assumption is that one ( or more ) server had been detached from vespa cluster before deployment.
If one ( or more ) server had detached from vespa cluster with some reason, and it had not been updated appropriately for a few weeks,
then returned to cluster at deployment, it explains why document remains in one cluster.
When server detach from cluster or has any trouble in update process, we need some kind of alert at that time.
But no abnormal alert occurred in this case.
Although we monitor vespa-runserver process, this is probably not enough.
We monitor vespa-runserver by Prometheus Process-exporter as follows.
process-exporter --config.path=/etc/process_exporter/all.yaml --children=false

$ cat /etc/process_exporter/all.yaml
process_names:
- comm:
  - vespa-runserver

ref. https://github.com/ncabatoff/process-exporter
Scenario we assumed
We are assuming a scenario as follows.

One (or more) vespa server had detached from the whole vespa cluster accidentally before deployment.

We did not know that event, and did not even notice the failure of deleting document. ( process monitoring is not enough?)

All vespa clusters was deployed to production release. Probably, at this time, the detached vespa server was re-attached to vespa cluster.

We found the number of documents in one cluster was different from the other clusters.

Question
Is there any possibility of server detaching from vespa cluster with no alert, although we monitor vespa-runserver process?
If it may happen, could you let us know what kind of monitoring we should do besides vespa-runsuerver?

I think your analysis of what happened is correct. See https://docs.vespa.ai/documentation/elastic-vespa.html#data-retention-vs-size - after two weeks, knowledge that a document is deleted, is lost.
in https://cloud.vespa.ai/ nodes are wiped before adding back to avoid this problem. https://cloud.vespa.ai/  also automates failed node detection and handling. In your case, you will have to build something similar. Get started with https://docs.vespa.ai/documentation/content/content-nodes.html to understand processes, in particular the cluster controller (its status page is quite useful). https://docs.vespa.ai/documentation/slobrok.html is also relevant. The cluster state tracks changes to the cluster, when nodes go in and out
I am sorry I do not have a simple answer. It is complicated to build a service that handles this well, without creating other problems. Nodes can go in and out as servers boot for whatever reason.
We can discuss options in this ticket after you have reviewed the documentation / looked at the cluster state

Thank you for your answer to my question quickly.
Based on your comment, we read some vespa documents as follows.
vespa-get-cluster-state CLI

https://docs.vespa.ai/documentation/reference/vespa-cmdline-tools.html#vespa-get-cluster-state

State API

https://docs.vespa.ai/documentation/content/api-state-rest-api.html

However, if we could fetch these cluster-state information from prometheus metrics, it would be best.
Prometheus metrics like this

http://`vespa`:19092/prometheus/v1/values

Do you have any plans to add such information to prometheus metrics?

I am sorry, no not really, as we export a limited set to prometheus afaik
as this is open source, it is possible to build extensions, CC @gjoranv

I understand.
We would greatly appreciate if you kindly build some extensions.
For the time being, we might use CLI or API to get cluster-state-information.
Thank you.

Yes, I understand your issues. The problem of handling nodes going in and out of applications is difficult (failed node detection, node/data retention, if/how/when to put back in), managing nodes overall is a really big problem, so it is hard to solve. Vespa Team has spent many man-years integrating features and procedures into https://cloud.vespa.ai/ so other teams do not need to do it themselves.
I will close this issue, as the metrics export is not planned to change shortly (but this is open source, so feel free to contribute!)


The number of layers cannot exceed 5 in version 12.1.2 of the operating system in Iphone

    In version  12.1.2（16C101）  of the operating system in Iphone,if you place more than 5 layers(VectorImage) on the map,    chrome,Safari and apps on the phone will appear white screen.      But that won't happen on the iphone5 or the inpone XR.
    And which i use is version 6.0.0-beta.3 .

This has been fixed already. See #9584.

Thanks a lot!


Superset - auto cancel oracle query parameter

In superset if there is a long running query running more than 3 mins (for example), is it possible to kill the query automatically and take the next one in the queue. application should auto-cancel the query. Also Query time out option is set but still on backend query is running. It's not cancelling query execution on database server.  Let us know any solution for this.

@nirajtambat is this still an issue in newer versions of Superset?
@villebro do you think this might be related in shape to your recent query cancellation PR?

I think we'd need to implement query cancellation for Oracle. See here for details: https://stackoverflow.com/questions/9545560/how-to-kill-a-running-select-statement
I think it should be fairly straight forward if you have an Oracle instance handy.


Replace eslint with biome (JavaScript, TypeScript, JSX, and JSON)

Replaces eslint with biome as the default JavaScript, TypeScript, JSX, and JSON formatter/linter. As a bonus, biome is supposed to support CSS and HTML in a near future.
This PR removes the need to maintain a NPM package in the root of platform/web. It relies instead on npx and the ability to download/use packages.
All the current commit files formatting was mostly done automatically, but tests are needed to make sure nothing has been impacted.
About closure-compiler, exceptions to the biome config has been added to make sure that object literals aren't flagged as replaceable.
Impacts on DotNET, Network and Tests
It just relates to JSON formatting (using tabs instead of spaces).
JSDoc

Runs JSDoc with npx instead of using a locally installed version of jsdoc.

Supersedes
Supersedes #91134

In my personal opinion, we should wait with biome until it can check scripts inside html files. For the moment, all js code in misc/dist/html/editor.html and misc/dist/html/full-size.html is ignored by it.

I think there's been a misunderstanding regarding pre-commit; the hooks aren't required to use the repository's node environment. Instead, it constructs isolated node environments which are reused on subsequent calls. As such, the need for the separate npx logic can be discarded; it's already functionally identical to what pre-commit achieves.

diff
 .pre-commit-config.yaml | 18 +++++++++---------
 misc/scripts/biome.sh   | 39 ---------------------------------------
 2 files changed, 9 insertions(+), 48 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 73feca5d32..7a76f9f080 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -91,29 +91,29 @@ repos:

       - id: biome
         name: biome (JavaScript, TypeScript, JSX, and JSON formatting)
-        language: system
-        entry: misc/scripts/biome.sh
-        pass_filenames: false
+        language: node
+        entry: biome
+        args: [check, --write, --colors=force]
+        files: \.(js|jsx|ts|tsx|json|jsonc|css|html)$
+        additional_dependencies: ["<EMAIL_ADDRESS>
-      # Requires `npx` to be installed. (Install node.js)
       - id: jsdoc
         name: jsdoc
-        language: system
-        entry: npx
+        language: node
+        entry: jsdoc
         files: ^platform/web/js/engine/(engine|config|features)\.js$
         args:
-          - --yes
-          -<EMAIL_ADDRESS>           - --template
           - platform/web/js/jsdoc2rst/
           - platform/web/js/engine/engine.js
           - platform/web/js/engine/config.js
           - platform/web/js/engine/features.js
           - --destination
-          - ''
+          - ""
           - -d
           - dry-run
         pass_filenames: false
+        additional_dependencies<EMAIL_ADDRESS>
       - id: copyright-headers
         name: copyright-headers
diff --git a/misc/scripts/biome.sh b/misc/scripts/biome.sh
deleted file mode 100755
index 630e8e3d39..0000000000
--- a/misc/scripts/biome.sh
+++ /dev/null
@@ -1,39 +0,0 @@
-#!/usr/bin/env bash
-
-set -uo pipefail
-
-FORCE_NPX=${FORCE_NPX:-no}
-FORCE_NPX=$(echo "$FORCE_NPX" | tr "[:upper:]" "[:lower:]")
-
-if type "biome" > /dev/null && [ "$FORCE_NPX" = "no" ]; then
-	printf "\e[3mUsing local \`biome\` at \"$(which biome)\"\e[0m\n"
- 	biome_exec=biome
-else
-	biome_version="1.7.3"
-	printf "\e[3mUsing \`npx --yes @biomejs/biome@$biome_version\`\e[0m\n"
-	biome_exec="npx --yes @biomejs/biome@$biome_version"
-fi
-
-# Loops through all code files tracked by Git.
-git ls-files -- '*.js' '*.jsx' '*.ts' '*.tsx' '*.json' '*.jsonc' '*.css' '*.html' \
-                ':!:.git/*' ':!:thirdparty/*' ':!:platform/android/java/lib/src/com/google/*' ':!:*-so_wrap.*' |
-while read -r f; do
-    # Run `biome`.
-    "$biome_exec" check --write "$f" &> /dev/null
-done
-
-diff=$(git diff --color)
-
-# If no diff has been generated all is OK, clean up, and exit.
-if [ -z "$diff" ] ; then
-    printf "\e[1;32m*** Files in this commit comply with the biome style rules.\e[0m\n"
-    exit 0
-fi
-
-# A diff has been created, notify the user, clean up, and exit.
-printf "\n\e[1;33m*** The following changes must be made to comply with the formatting rules:\e[0m\n\n"
-# Perl commands replace trailing spaces with `·` and tabs with `<TAB>`.
-printf "%s\n" "$diff" | perl -pe 's/(.*[^ ])( +)(\e\[m)$/my $spaces="·" x length($2); sprintf("$1$spaces$3")/ge' | perl -pe 's/(.*[^\t])(\t+)(\e\[m)$/my $tabs="<TAB>" x length($2); sprintf("$1$tabs$3")/ge'
-
-printf "\n\e[1;91m*** Please fix your commit(s) with 'git commit --amend' or 'git rebase -i <hash>'\e[0m\n"
-exit 1


Test Case Pass

@jlperla I took a stab at implementing the test case you set up on Discourse. This is pretty rough, but gets the job done. Let me know what you think.
using Base.Test

# Type defs.
abstract type AbstractValue end
struct Value1 <: AbstractValue end
struct Value2 <: AbstractValue end

abstract type AbstractAlgorithm end
struct Algorithm1 <: AbstractAlgorithm end
struct Algorithm2 <: AbstractAlgorithm end

# Initialize objects.
v1 = Value1()
v2 = Value2()
a1 = Algorithm1()
a2 = Algorithm2()

# Write function methods
function f(v::Value1; alg = a2, kwargs...)
    return f(v, alg; kwargs...)
end

function f(v::Value1, alg::Algorithm2; N = 5, kwargs...)
end

function f(v::Value1, alg::Algorithm1; N = 10, kwargs...)
end

function f(v::Value2; alg = a1, kwargs...)
    return f(v, alg; kwargs...)
end

function f(v::Value2, alg::Algorithm1; N = 10, kwargs...)
end

function f(v::Value2, alg::Algorithm2; kwargs...)
    MethodError("This Method is ill-defined.")
end

# Run tests.
@testset "Dispatch Behavior" begin
    @test f(v1) == f(v1, a2, N=5) #i.e. defaults to Algorithm 2 with N=5 as the default
    @test f(v1, N = 7) == f(v1, a2, N=7) #Can change the default value
    @test f(v1, a1) == f(v1, a1, N=10) #can use use Algorithm1 (with the different default)
    @test f(v2) == f(v2, a1, N=10) #i.e. uses algorithm 1 by default
    @test f(v2, N = 8) == f(v2, a1, N=8) #Can change the default value
    @test_throws MethodError f(v2, a2) #Not defined! Should throw
end

No, that is not the point of it. Wait on this issue


Mobile scrolling not working

I tried the mobile scrolling on the demo and on my project and neither of them are triggering the next page.  I had a very similar issue earlier and it was because I wasn't listening to @HostListener('touchmove', ['$event']).  Could this be related?  I tried this with my my iphone 6 and blisk.

hi.
try http://orizens.github.com/echoes-ng2
it works - though i don't have @HostListener - so it might relate somehow.

That doesn't work for me with blisk-- still same issue.

What is blisk?
On Sun, Oct 16, 2016, 00:24 Jonathan Reyes<EMAIL_ADDRESS>wrote:
That doesn't work for me with blisk-- still same issue.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/orizens/angular2-infinite-scroll/issues/89#issuecomment-254012048,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1oRMXnC9mC2qr7x0vT1mMk2XTphdsvks5q0USCgaJpZM4KXbyj
.
--
Oren Farhi
Front End Engineer & Javascript Consultant
http://www.orizens.com

https://blisk.io/
But even when I load it on my iphone it doens't get past the bootstrap (still says Loading...).   Any thoughts?

no.
I just tried echoes-ng2 and plnkr demos on safari (mac) - and it works fine currently.

I know, I am talking about mobile.  Blisk is mobile testing, and on  my iphone for example your echoes doens't work either.  It is the scrolling on mobile (using touchevents) that is causing the issues...  It works perfectly on desktop browsers.

currently, i have no way of testing and debugging on blisk.
also, blisk is out of scope - unless you or anyone else are willing to assist with this.
thanks.

Yes I understand blisk is out of scope but I also mentioned that the echoes
doesn't work on my iPhone either.  I also can't get it to trigger events
which as mentioned I may attribute to a touch event issue...
On October 19, 2016 at 7:32:16 PM, Oren Farhi<EMAIL_ADDRESS>wrote:

currently, i have no way of testing and debugging on blisk.
also, blisk is out of scope - unless you or anyone else are willing to
assist with this.
thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/orizens/angular2-infinite-scroll/issues/89#issuecomment-254867559,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJNoFmgGyGp6onwZqG2VOMDMGqx59zsks5q1kYQgaJpZM4KXbyj
.


Fixed Button implementation in pricing/costs

- What I did

Fixed button implementation in pricing-costs section
- How I did it
Added a reference to get an @sign page.
Removed Center in HTML.
- How to verify it
Check the page to make sure that the changes were applied
- Description for the changelog

We can make a shortcode to center this button


display of flag, double hyphen, parser upgrade

This is a Bug Report
Double hyphen literal characters are not displaying correctly when embedded in <code> element.
Problem:
Previously, the docs site turned off the BlackFriday parser smartDashes setting (config.toml).
With the recent change to Hugo 0.70.0 (Goldmark parser), the dashes are no longer displaying
as dashes when the command string is embedded in the <code> element.
One affected page:
https://kubernetes.io/docs/reference/kubectl/overview/#operations
Proposed Solution:
Investigate whether the parser can be configured or determine how to display the dashes by removing the use of <code> elements.
For additional information, see pull requests #20848, #20909.

/kind bug
/priority important-soon
because people copy-and-paste from that page a lot.

Relates to #20335

i think we got close to a dozen reports about this already.

@neolit123 , Did you see/review PR #21272?

i think i saw it, yes. is there a reason it's not merged yet?

i think i saw it, yes. is there a reason it's not merged yet?

Not that I know of

Things all look good now. Closing this.
/close


[SPARK-40622][WIP]do not merge(try to fix build error)

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Can one of the admins verify this patch?


Add spell check pin changes

Changes from PR: https://github.com/Azure/azure-sdk-for-js/pull/17142/checks

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

@weshaggard -- Subbed in npx for npm exec


Documentation: Tiny typo fix

Uses cases -> use cases

Codecov Report

Merging #1911 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1911   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files           1       1
  Lines        4056    4056
  Branches      717     717
======================================
  Hits         4056    4056

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 a04bcec...4f85f3d. Read the comment docs.

Codecov Report

Merging #1911 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1911   +/-   ##
======================================
  Coverage     100%    100%
======================================
  Files           1       1
  Lines        4056    4056
  Branches      717     717
======================================
  Hits         4056    4056

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 a04bcec...4f85f3d. Read the comment docs.

Thank you @jordanstephens 👍


fix(.git-location) : Handles case where .git is a file instead of dir…

When using git submodules, the .git folder is actually a file containing a property that tells us where we should be looking for the files that are usually in the .git folder.
We can use the contents of that file to find where the COMMIT_EDITMSG is actually located.

Current coverage is 100.00%

Merging #28 into master will not affect coverage as of 804b71c

@@            master     #28   diff @@
======================================
  Files            1       1
  Stmts           71      82    +11
  Branches         0       0
  Methods          0       0
======================================
+ Hit             71      82    +11
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 804b71c

Powered by Codecov. Updated on successful CI builds.

I'll have to pull this down and test it out on a project. Hopefully will get it merged soon.

Looks good to me! Thanks!

Thanks for getting it merged in! One less forked npm module to depend on!
On Sun, Mar 20, 2016 at 11:51 PM, Kent C. Dodds<EMAIL_ADDRESS>wrote:

Merged #28 https://github.com/kentcdodds/validate-commit-msg/pull/28.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/kentcdodds/validate-commit-msg/pull/28#event-596621404

Thanks for contributing :-)


Revert "Use JSON to marshal errors from children"

Reverts Homebrew/brew#4752
This breaks brew formula-analytics and in turn https://formulae.brew.sh.

@woodruffw Can reapply this when it's tested and working nicely with brew formula-analytics. I'm a bit concerned about how this blew up, though; is it monkeypatching all uses of JSON or something?

is it monkeypatching all uses of JSON or something?

I think the issue is the extra context being added to #to_json by require "json/add/core": there are probably a few places where we're expecting #to_json to return just a JSON string object and now returning something more complicated instead.
In particular, I'm guessing that the 's' keys are coming from Symbol: https://github.com/flori/json/blob/master/lib/json/add/symbol.rb, and that the json_class keys are coming from JSON.create_id (which get thrown into every one of these extended serializations).
We can probably avoid this by just doing require 'json/add/exception' instead of json/add/core, since Exception serialization is the only thing we need. I'll open a new PR with the fixes (and hopefully a test).


TypeError: cannot convert dictionary update sequence element #0 to a sequence

Description
In openapi 3 mode if we create a method such as post or put that accepts json and returns json along with type: object in schema then the method fails with the below error.
Traceback (most recent call last):
  File "/c/w/connexion/venv/src/flask/flask/app.py", line 2310, in wsgi_app
    response = self.full_dispatch_request()
  File "/c/w/connexion/venv/src/flask/flask/app.py", line 1833, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/c/w/connexion/venv/src/flask/flask/app.py", line 1737, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/c/w/connexion/venv/src/flask/flask/_compat.py", line 40, in reraise
    raise value
  File "/c/w/connexion/venv/src/flask/flask/app.py", line 1831, in full_dispatch_request
    rv = self.dispatch_request()
  File "/c/w/connexion/venv/src/flask/flask/app.py", line 1817, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/c/w/connexion/venv/lib/python3.7/site-packages/connexion-2018.0.dev1-py3.7.egg/connexion/decorators/decorator.py", line 48, in wrapper
    response = function(request)
  File "/c/w/connexion/venv/lib/python3.7/site-packages/connexion-2018.0.dev1-py3.7.egg/connexion/decorators/uri_parsing.py", line 143, in wrapper
    response = function(request)
  File "/c/w/connexion/venv/lib/python3.7/site-packages/connexion-2018.0.dev1-py3.7.egg/connexion/decorators/validation.py", line 172, in wrapper
    response = function(request)
  File "/c/w/connexion/venv/lib/python3.7/site-packages/connexion-2018.0.dev1-py3.7.egg/connexion/decorators/parameter.py", line 108, in wrapper
    request.files, arguments, has_kwargs, sanitize)
  File "/c/w/connexion/venv/lib/python3.7/site-packages/connexion-2018.0.dev1-py3.7.egg/connexion/operations/abstract.py", line 275, in get_arguments
    has_kwargs, sanitize))
  File "/c/w/connexion/venv/lib/python3.7/site-packages/connexion-2018.0.dev1-py3.7.egg/connexion/operations/openapi.py", line 268, in _get_body_argument
    body_arg.update(body or {})
TypeError: cannot convert dictionary update sequence element #0 to a sequence
<IP_ADDRESS> - - [04/Mar/2019 21:10:14] "POST /v1.0/greeting HTTP/1.1" 500 -

Expected behaviour
connexion should work as expected returning json as the argument .
Actual behaviour
Error shown in the description
Steps to reproduce
To reproduce the error use the helloobj example shown in this branch.
https://github.com/zalando/connexion/compare/master...ngcloud:fix/dict-error-post
cd  examples/openapi3/helloobj
python hello.py

# Visit the swagger ui and make a post request
# http://localhost:9090/v1.0/ui/#/default/hello.post_greeting
# curl -X POST "http://localhost:9090/v1.0/greeting" -H  "accept: application/json" -H  "Content-Type: applicaton/json" -d "{\"name\":\"fluffy\"}"

Additional info:
Output of the commands:

python --version

Python 3.7.2+

pip show connexion | grep "^Version\:"

Version: 2018.0.dev1

I have attempted a fix here - https://github.com/zalando/connexion/commit/518d1dd3d711782aa56eb406b7fb346443d94271

Hey @prabhu - you have a typo applicaton/json in your spec and your curl request.
You need to use applicat i on/json (note the missing 'i').

Wow, I am feeling ashamed a bit here :) But thank you very much for this!

No worries!
On Thu, Mar 7, 2019 at 5:45 PM Prabhu Subramanian<EMAIL_ADDRESS>wrote:

Wow, I am feeling ashamed a bit here :) But thank you very much for this!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/zalando/connexion/issues/892#issuecomment-470413521,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAlPSUBEc8Ub3gMxGaEXJ8MngHkoX75mks5vULx0gaJpZM4bdO76
.

I have attempted a fix here - 518d1dd

I got same error. My headers were named correctly. Application is somewhat complex (OpenAPI spec is about 7000lines) and the server stub was generated with OpenAPI generator
This error appeared in every PUT and POST request so far, and changes in OpenAPI spec did not change anything in my knowledge. Spec is here for anyone interested: spec
However, adding your potential fix fixed my problem!

@Nicceboy Glad the patch helped. I think you need this patch because of the content-type used being
application/vnd.mason+json . If you change it to application/json it should work without the patch

I tried out with application/json as well, but no luck.  application/vnd.mason+json is on Accept and Response Content-Type headers only, not in any request Content-Type header.

Looks like there was typo in my OpenAPI spec after all. requestBody in component section was defined to be '*/*'. Change for application/json made it work like a charm, without additional fixes.

I ran into a lot of confusion here, as I was trying to allow for multiple content media types in the request body (which is supported in openapi3), like in the following example adapted from here:
[...]
    requestBody:
      required: true
      content:
        application/json:
          schema:
            x-body-name: user
            $ref: "#/components/schemas/User"
        application/:x-www-form-urlencoded
          schema:
            x-body-name: user
            $ref: "#/components/schemas/User"
[...]

When removing the second media type everything works as it should, but I have yet to understand how to handle multiple media types. (I am sending requests through the swagger-ui, as well as through flask's .post on an app.test_client())
P.S. This is partly an invitation to reply if you know how to make this work, partly meant as a google-able note for people making the same mistake.

I also ran into this issue. It took a lot of googling to figure out that this cryptic error means "default body doesn't have a schema for the body's mime type, maybe the specific mime time was omitted or misspelled" In my case, the request body was json but in my openAPI.yaml the request body had a mimetype of text


fix(maxAge): delete cache entries older than max age.

Fixes #14

Could you please propose how I would do that?

Managed to get a working test for this.

You can just check if the size of the cache is 0 after the expired time. Not necessary to override the delete method.

No - because the cache is repopulated by virtue of the second call - with every call the cache.size will always be 1. The only way to test this is to spy on the side effect of deleting from cache.

Any idea when this PR will be released?  Sounds like something we want to fix.

@SamVerschueren @keithamus Are you two in a deadlock or what's going on?
If there is really a memory leak fixing it would be important, no? I'm about to use this in production in several parts of my infrastructure, so I got a little worried stumbling on this issue by accident...

@0x80 I dont think we're in a deadlock. The PR has tests, and I've explained why those tests are what is needed and cannot be simplified.
Happy to take more feedback from @SamVerschueren or @sindresorhus about what is needed to get this merged.
If you're about to deploy this to production - you could always publish a version on your own namespace that includes this patch.

@keithamus I will do that then.
If it's hard or impossible to improve the tests, and it won't hurt to merge this, then I hope @SamVerschueren or someone can approve this PR.
The last thing a memoization lib needs is a memory leak... 😄

I don't see how this fixes the problem though? The delete call to the map that has been introduced here will only run just before we make a set call to the map, so it won't have any effect at all.
I think that a proper fix would cover the following test case:
test('maxAge option deletes old items', async t => {
	const f = () => 'foobar';
	const cache = new Map();
	const memoized = m(f, {maxAge: 100, cache});

	t.is(memoized(1), 'foobar');
	t.is(cache.has(1), true);

	await delay(50);
	t.is(cache.has(1), true);

	await delay(50);
	t.is(cache.has(1), false);
});

@LinusU if deletions were observable outside of calls to the memoized function as per your test case that would add a lot of complexity to this library which Im unsure is necessary. Adding for e.g. a setTimeout call to clear the cache means this library does a lot more (and has more side effects) than at current.

if deletions were observable outside of calls to the memoized function as per your test case that would add a lot of complexity to this library which Im unsure is necessary

This would be the only way to solve #14 though?
Since the problem that that issue raises is that any cached value will stay in the cache until a new value replaces it. This is an issue if you e.g. are caching web fetches, but only want to cache them for 5 minutes. If you at some point make 5000 different requests, for some domains that you will never access again, those 5000 responses will now be stored in memory until the process is terminated.

Did you understand what I meant with this?

The delete call to the map that has been introduced here will only run just before we make a set call to the map, so it won't have any effect at all.

As far as I can see, the patch that you have proposed doesn't change the behaviour of the module in any way. By extension, it also doesn't address the issue raised in #14.

those 5000 responses will now be stored in memory until the process is terminated

I understand this. I think in these instances it may be preferable to manually clear. I'm just concious of side effects. Introducing timers into libraries like this can cause unpredictability - like gc pauses which become hard to debug because a library (that you might not expect to) is firing off events at somewhat unpredictable times.

Did you understand what I meant with this?

I did. While calling Map with a new set operation does indeed clear this, for my own purposes I need specifically to call delete (I'm working with a subclassed version of Map that moves operations to disk, which is where I found this never actually calls delete in the way I want it to).
Perhaps I should rephrase the description as this might not actually fix #14 in the way folks may expect it to.

I agree with @LinusU that it does not solve #14 and it only helps when the new call fails like you mention @keithamus. Think the changes make sense for that part.

This also fixes a bug where if a subsequent call to fn may throw an error

This is good 👍

I've added an additional test to explicitly cover the case of a function throwing and the cache still being deleted.

@keithamus Thank you and sorry for the slow reply. I've released it as a patch version.


HTTPS client regression?

After upgrading from 0.7.26-rc.1 to 0.7.26, this code:
requestHTTP("https://steamcommunity.com",
	(scope HTTPClientRequest req) {},
	(scope HTTPClientResponse res) {}
);

causes error:
Task terminated with uncaught exception: Peer failed the certificate validation: 20

I found that reason is commit 7e8fcc48b527251f26fdb1d5a8fb88bd43a62fcf
After reversing appropriate diff, error has gone.

Yeah I noticed the same thing - posted some more details here:
http://forum.rejectedsoftware.com/groups/rejectedsoftware.vibed/thread/27364/#post-27369
Don't totally understand the surrounding code but it looks like before it was ignoring the fact that it couldn't verify the server certificate for client requests (due to lack of a certificate store) but that new code triggers and fails SSL requests pretty uniformly now.
Interesting it doesn't trigger if you connect to the example HTTPS server. I'm guessing that's because that uses a self-signed certificate or similar which probably hits a slightly different error return in the switch statement below where it ignores it.


How to provide authentication information for schema registry?

I have tried to use docker run -e SCHEMA_REGISTRY_GITOPS_SCHEMA_REGISTRY_URL=http://user:password@localhost:8085 ....  but it returns Unauthorized exception
ERROR io.confluent.kafka.schemaregistry.client.rest.exceptions.RestClientException:
Unauthorized; error code: 401

Should it be provided somehow differently or may be special options should be used?

After some investigation it seems to me there is an issue with schema-registry client which doesn't handle basic authentication properly. https://github.com/confluentinc/schema-registry/issues/1759

Hi @vitalvi and thanks for using schema-registry-gitops!
I got it working with these environment variables:
SCHEMA_REGISTRY_GITOPS_SCHEMA_REGISTRY_URL=https://localhost:8085
SCHEMA_REGISTRY_GITOPS_SCHEMA_REGISTRY_BASIC_AUTH_CREDENTIALS_SOURCE=USER_INFO
SCHEMA_REGISTRY_GITOPS_SCHEMA_REGISTRY_BASIC_AUTH_USER_INFO=user:password # in plain text, no base64 used

Can you check if that works for you, too? If so I can also put that in the README, I remember I struggled with that for quite some time, too. This is what the last part of https://docs.confluent.io/platform/current/schema-registry/security/index.html#configuring-the-rest-api-for-basic-http-authentication suggests for the client and all variables beginning with SCHEMA_REGISTRY_GITOPS_SCHEMA_REGISTRY_ will be forwarded to the client.

Hey @domnikl,
yeah, I confirm, these properties work for me too. Thanks a lot for your help.


[Question]: OPENBLAS blas_thread_init: pthread_create failed for thread x of 64: Operation not permitted

Describe your problem
Help!
On server B，I load docker images which are saved on server A.
And then "docker-compose up -d" to start the ragflow project.
When I try to check the ragflow status with "docker logs -f ragflow-server"
I got a lot of OpenBLAS errors as follows:

May this help.
https://stackoverflow.com/questions/52026652/openblas-blas-thread-init-pthread-create-resource-temporarily-unavailable


CharacterSet: minor improvements to match darwin

we don't actually impl NSCoding methods, so NSUnimplement them
punctuationCharacters was incorrectly migrated as punctuation

@swift-ci please test and merge


Feature: Squared Image

The feature from #241, moved into the Advanced dropdown as requested.

This is great! Thanks for the contribution, @molnarmark 👍


Add mezzanine-modal-announcements to third party modules

Thanks Josh
I added it to djangopackages too: https://www.djangopackages.com/packages/p/mezzanine-modal-announcements/

Sweet, thanks!
On Thu, Apr 16, 2015 at 8:23 PM, Stephen McDonald<EMAIL_ADDRESS>wrote:

Thanks Josh
I added it to djangopackages too:
https://www.djangopackages.com/packages/p/mezzanine-modal-announcements/
—
Reply to this email directly or view it on GitHub
https://github.com/stephenmcd/mezzanine/pull/1269#issuecomment-93867091.


Dark Mode for the portfolio

Adds Dark Mode theme toggle in the navbar.
Fixes #1
Add JS folder for scripts

This is great. Tested on my local. Thanks!


"We currently do not support a name of "test"."

OS?

Windows 7

Versions.
angular-cli: 1.0.0-beta.22-1
node: 6.9.2
os: win32 x64
Repro steps.
ng new test
The log given by the failure.
We currently do not support a name of "test".
Mention any other details that might be useful.
ERROR while installing angular-cli
npm WARN deprecated<EMAIL_ADDRESS>graceful-fs v3.0.0 and before will fail on node releases >= v7.0. Please update to<EMAIL_ADDRESS>as soon as possible. Use 'npm ls graceful-fs' to find it in the tree.
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>use uuid module instead
npm WARN deprecated<EMAIL_ADDRESS>ReDoS vulnerability parsing Set-Cookie https://nodesecurity.io/advisories/130
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
\AppData\Roaming\npm\ng -> \AppData\Roaming\npm\node_modules\angular-cli\bin\ng

<EMAIL_ADDRESS>install \AppData\Roaming\npm\node_modules\angular-cli\node_modules\execSync
node install.js

[execsync v1.0.2] Attempting to compile native extensions.
{ Error: spawn node-gyp ENOENT
at exports._errnoException (util.js:1022:11)
at Process.ChildProcess._handle.onexit (internal/child_process.js:193:32)
at onErrorNT (internal/child_process.js:359:16)
at _combinedTickCallback (internal/process/next_tick.js:74:11)
at process._tickCallback (internal/process/next_tick.js:98:9)
at Module.runMain (module.js:606:11)
at run (bootstrap_node.js:394:7)
at startup (bootstrap_node.js:149:9)
at bootstrap_node.js:509:3

code: 'ENOENT',
errno: 'ENOENT',
syscall: 'spawn node-gyp',
path: 'node-gyp',
spawnargs: [ 'rebuild' ] }
[execSync v1.0.2]
Native code compile failed!!
Will try to use win32 extension.

<EMAIL_ADDRESS>install \AppData\Roaming\npm\node_modules\angular-cli\node_modules\node-zopfli
node-pre-gyp install --fallback-to-build

[node-zopfli] Success: "\AppData\Roaming\npm\node_modules\angular-cli\node_modules\node-zopfli\lib\binding\node-v48-win32-x64\zopfli.node" is installed via remote

<EMAIL_ADDRESS>install \AppData\Roaming\npm\node_modules\angular-cli\node_modules\node-sass
node scripts/install.js

Downloading binary from https://github.com/sass/node-sass/releases/download/v3.13.1/win32-x64-48_binding.node
Download complete  ] - :
Binary saved to \AppData\Roaming\npm\node_modules\angular-cli\node_modules\node-sass\vendor\win32-x64-48\binding.node
Caching binary to \AppData\Roaming\npm-cache\node-sass\3.13.1\win32-x64-48_binding.node

<EMAIL_ADDRESS>postinstall \AppData\Roaming\npm\node_modules\angular-cli\node_modules\node-sass
node scripts/build.js

Binary found at \AppData\Roaming\npm\node_modules\angular-cli\node_modules\node-sass\vendor\win32-x64-48\binding.node
Testing binary
Binary is fine

As the error message indicates, we don't support that name for creating a new app. Please choose another name.

For testing this you can use somthing like "ng new angular-test"

Any name should be possible. What is the reason to not support name "test"?

just hit this myself, wondering what the rational behind requiring a hyphen in the name?  Looks like this is a re-occurring issue/question #912 #2645
I did a little more testing to see what was a valid naming scheme.
my-app, my.app, myApp, MyApp all work my_app did not, nor did my app,  my|app, my+app, my#app, my:app (note all were tested with appropriate shell quoting)  .
my#App got a bit more of a verbose error message (223 chars vs the 44 chars in response to ng new app)
My personal opinion here is the rules for the name convention should be included in the output of ng help new, it saves everyone's time in looking at github issues and head scratching on 'why does it not work?'  I'm big fan of 'rule of least surprise'  here not being able to create an app called 'test' is actually pretty surprising.
It would also be honest to doc the reason behind the rule (not required, just ya know nice) ie 'we parse the name and use it as part of a namespace and we do our best to figure out your naming schema', 'we think it looks pretty', or 'not really sure its just the way we have always done it'.


Remove atomic emulation

Since DPCPP now supports atomics, we don't need to emulate them by ourselves.

[x] Have you provided a meaningful PR description?
[x] Have you added a test, reproducer or referred to an issue with a reproducer?
[x] Have you tested your changes locally for CPU and GPU devices?
[x] Have you made sure that new changes do not introduce compiler warnings?
[ ] If this PR is a work in progress, are you filing the PR as a draft?

Fixes: #760, #650, #868

This PR should unblock these checks:
https://github.com/IntelPython/dpbench/pull/289

@ZzEeKkAa in a subsequent PR can you push changes to remove the dependency on spirv-tools?

#650 tested:
> set NUMBA_DPPY_DEBUG=1
> dpbench -b gpairs -i numba_dpex_k run --no-skip-expected-failures

================ Benchmark GPairs (gpairs) ========================

================ implementation numba_dpex_k ========================
implementation: numba_dpex_k
framework: numba_dpex
framework version: 0.21.1+61.g7d9bcca0.dirty
input size: 8512
setup time: 219.6217ms (219621700 ns)
warmup time: 962.5928ms (962592800 ns)
teardown time: 101.4μs (101400 ns)
max execution times: 4.226ms (4226000 ns)
min execution times: 2.7134ms (2713400 ns)
median execution times: 3.15995ms (3159950 ns)
repeats: 10
preset: S
validated: Success

This change is ready to go. I've tested all the mentioned issues.
These issues simply does not make sense now: #760, #868, #711 does not work, see my latest comment there.

@ZzEeKkAa in a subsequent PR can you push changes to remove the dependency on spirv-tools?

Did it here in a separate commit.


[improve] Print stack trace when got excpetion in ServerCnx

Motivation
I meet a issue that NPE happend in ServerCnx.handleSend() but the debug log does not print stack trace.

Modifications
print stack trace when got excpetion in ServerCnx
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

/pulsarbot rerun-failure-checks

Codecov Report

Merging #19593 (14206e6) into master (02b25a3) will decrease coverage by 0.45%.
The diff coverage is 30.00%.

@@             Coverage Diff              @@
##             master   #19593      +/-   ##
============================================
- Coverage     62.35%   61.91%   -0.45%
+ Complexity    25873    25647     -226
============================================
  Files          1844     1844
  Lines        135280   135343      +63
  Branches      14875    14886      +11
============================================
- Hits          84352    83793     -559
- Misses        43183    43789     +606
- Partials       7745     7761      +16

Flag
Coverage Δ

inttests
24.68% <13.75%> (+0.05%)
:arrow_up:

systests
25.27% <18.75%> (-0.12%)
:arrow_down:

unittests
59.17% <20.00%> (-0.38%)
:arrow_down:

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

Impacted Files
Coverage Δ

...va/org/apache/pulsar/broker/service/ServerCnx.java
55.61% <0.00%> (+0.93%)
:arrow_up:

...java/org/apache/pulsar/admin/cli/CmdFunctions.java
47.20% <0.00%> (-0.51%)
:arrow_down:

...rg/apache/pulsar/broker/service/BrokerService.java
59.22% <7.84%> (-0.85%)
:arrow_down:

...che/bookkeeper/mledger/impl/ManagedLedgerImpl.java
78.97% <92.85%> (+0.09%)
:arrow_up:

...okkeeper/mledger/impl/ShadowManagedLedgerImpl.java
51.23% <100.00%> (ø)

...apache/pulsar/common/functions/FunctionConfig.java
96.49% <100.00%> (+0.12%)
:arrow_up:

...he/pulsar/functions/utils/FunctionConfigUtils.java
63.23% <100.00%> (+0.23%)
:arrow_up:

...g/apache/pulsar/client/api/ServiceUrlProvider.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...r/client/impl/UnAckedMessageRedeliveryTracker.java
0.00% <0.00%> (-88.29%)
:arrow_down:

...ache/pulsar/client/impl/UnackMessageIdWrapper.java
0.00% <0.00%> (-65.52%)
:arrow_down:

... and 130 more


FeedbackChallenge : Blockchain Authentication Mechanism

FeedbackChallenge : Blockchain Authentication Mechanism
In the spirit of Project Catalyst's #FeedbackChallenge we are assessing to what extent the Catalyst Funding Process and it's Proposals are open source as part of the proposal Quality-Assurance-DAO. Your proposal has been selected because "open source" is mentioned in it's text or comments.
We would appreciate it if you could respond to the following questions (in the reply to my comments below) :

Open - Is your Organisation open-source ?
Accessible -  Can community contributors issue pull requests or raise issues on your source code ?
Transparent - Can community contributors access and propose changes to the source text of your documentation ?

To keep the assessment transparent each comment is recorded as a GitHub Issue on Quality-Assurance-DAO's proposal repository (Apache-2.0 License) where a more detailed explanation can be found.
Together we can build a community of open source contributors on the Cardano platform.
Thank You
Stephen Whitenstall, QA-DAO
Detailed explanation
Your Issue reference includes a more detailed explanation of the methodology of this FeedbackChallenge : Open Source. There is also a Open Source Assessment Project Board and a GitHub bot post Issues in my channel #quality-assurance-dao on Project Catalyst's Discord.
We believe this is important because many developers are concerned with the lack of a clear commitment to open source and fear their free contributions will be locked into a proprietary platform.
Open Source Assessment Method

[x] Draft a basic Open Source Assessment Method.
https://github.com/Quality-Assurance-DAO/DAO-Open-Source/tree/main/Documents/Assessments

Basic Open Source Assessment

Open - Is your Organisation open-source ?
Accessible -  Can your users issue pull requests or raise issues on your organisation's source code ?
Transparent -  Can your users access and propose changes to the source of your organisation's documentation ?

Method

[x] Do text search of "open source" on Catalyst Projects.
[x] Post assessment questions in comments of proposal.
[x] Raise Issue for each proposal's assessment.
[ ] Record response to assessment.
[ ] Close Issue once proposal submitted and record responses

Report

[ ] Collate and publish report of findings with a Creative Commons License

gargiulo.gianni (@gargiulo.gianni)
Idea Submitter

swhitenstall Hi,
The answer is yes to all the questions.

I will be implementing the solution under the gimbalabs github organization. code will be open sourced.
It is of course accessible and contributes will be more than welcome!
Yes. While our team will be leading and driving the development and adoption of the feature, any feedback/pr will reviewed and merged in accordance to the goals and objectives the project tries to achieve.

Thank you for your response, gargiulo.gianni (gargiulo.gianni). It has helped a lot.
I have captured them at [ https://github.com/Quality-Assurance-DAO/F5-Developer-ecosystem-Proposal/issues/69] and moved the status of the Issue to "Done".
Note : The Issue can still be altered and commented upon via GitHub until Fund 5 Proposals are finalized.
As I set out in the "Detailed explanation" section of each Issue : I will collect all data in a report of findings to be published on GitHub with a Creative Commons License after Fund 5 Proposals are finalized.
Good Luck with your proposal.
Stephen Whitenstall QA-DAO


You must press Close to save options

Description
After making any change to options you must press Close in the bottom right hand corner. The changes will not be saved if you click another addon, close by pressing "X" in the top right, close by pressing escape.
To Reproduce
Steps to reproduce the behavior:

Click any setting checkbox (e.g. Lock minimap button position)
Close config or click away to another config screen
Open config. See that Lock minimap button position is unchecked.

Expected behavior
Any changes to a config option is expected to remain no matter how you close the window.
Desktop (please complete the following information):

Version v3.11

Additional context
Season of Discovery

Re-opening this issue since the fix in #252 only affected the Dungeon/Raid filter settings panels.
The custom search patterns tab, for example, still requires a press of the close button
(unless you tab to an expansion's filter tab and then close the settings via any manner to force the implementation of this fix)

Fixed with #277


Missing - CVE IDs of affected Libraries

Hello Team,
Final generated report does not contain CVE ID information for affected libraries. It would be very useful to have the CVE IDs for the affected libraries in the final report. This will allow the Security Subject Matter Experts to analyze the CVE of the library and advice the teams on whether the alert really affects the used functionality of the open source code or not.
Thanks!
Rahul

Hello Rahul,
This feature is not in the scope of this open-source project but can be developed for you in the scope of the customized project.
Thank you,
WhiteSource PS team


Update text for Kaggle 1 Page to add code and publication

Update the text on the Kaggle 1 page (https://humanatlas.io/kaggle-one) based on the suggested edits in this document (https://docs.google.com/document/d/1FSRoQbaXa64xmc1VcH4lu_wqUCLxZsy6dXK3XdTFXbI/edit?usp=sharing)
TODO:

[x] Add "Publication" section containing link to published paper and citation.
[x] Add "Code" section containing link to GitHub repo.

Reviewed. Closing issue.


Provide URLRequest for JSS endpoints

New feature
Short description
Once the SessionManager is fully configured, it should be possible to retrieve URLRequests for all the models supported by JamfKit.
Missing functionnality
The SessionManager should be able to produce URLRequest that can be used by any user against the network communication framework of their choice.
Those requests should be available for the 4 methods currently supported by JSS:

PUT
GET
POST
DELETE

Implementation
TBD

Implemented by #72.


bulk operations with mongoskin and mongodb 3.x?

I've read the various bits of literature, and I'm seeing the same problem that the questioner in
http://stackoverflow.com/a/25636911
was seeing.
My code looks like something like this (I wrote it in coffeescript and am converting to js pseudocode):
coll = db.collection('foobar');
bulk = coll.initializeUnorderedBulkOp();
messages.forEach(function(entry) {
    bulk.insert(entry);
});
bulk.execute(function (err, result) {
   if (err) throw err
   inserted += result.nInserted
});

bulk is an object
bulk.insert works just fine
bulk.execute is undefined

The answer in the stackoverflow question said, "only the callback flavor of db.collection() works,
so I tried:
db.collection('foobar', function (err, coll) {
    logger.debug("got here");
    if (err) throw(err);
   bulk = coll.initializeUnorderedBulkOp();
   ... same code as before

We never get to "got here" implying that the "callback flavor" of db.collection() was dropped for 3.0?  Unfortunately, my python is way better than my JS prototyping skills, so looking at the skin source code doesn't make any sense to me.
What is the right way, with mongoskin 2.1.0 and the 2.2.0 mongodb JS driver, to do a bulk operation, or is this not implemented at all anymore?

bump?


failed, network error in downloading PASTIS24 data

Hi, thanks for your hard work on this project.
When I try to download the PASTIS24 from google drive, I kept facing the network error in the middle of downloading. This issue applies to other browsers, accounts, computers as well from what I can try. So I am wondering would you please try to upload it again or maybe share a google drive folder so that in worst case I could still download one by one? Any quick response is very much appreciated.

Btw, I found no pre-trained models in this link - the pre-trained models and tensorboard files. Should any pre-trained models been uploaded as well?

Hi @June01, thanks for pointing this out. I have uploaded the PASTIS24 in parts, I hope this helps with download issues.  After downloading you can combine all files together and unzip.
zip -F PASTIS24_25part.zip --out PASTIS24.zip
unzip PASTIS24.zip -d .

Hi @June01, thanks for pointing this out. I have uploaded the PASTIS24 in parts, I hope this helps with download issues. After downloading you can combine all files together and unzip.
zip -F PASTIS24_25part.zip --out PASTIS24.zip
unzip PASTIS24.zip -d .

Hi, I am very interested in this project.
According to this link, I successfully downloaded the dataset, but there is only pickle24x24 in the dataset. I can't find fold-paths. So when I run this project，will occur FileNotFoundError. Could you help me solve this problem？Thanks.

Hi @June01, thanks for pointing this out. I have uploaded the PASTIS24 in parts, I hope this helps with download issues. After downloading you can combine all files together and unzip.
zip -F PASTIS24_25part.zip --out PASTIS24.zip
unzip PASTIS24.zip -d .

Hi, I am very interested in this project. According to this link, I successfully downloaded the dataset, but there is only pickle24x24 in the dataset. I can't find fold-paths. So when I run this project，will occur FileNotFoundError. Could you help me solve this problem？Thanks.

@CChBen are you still having trouble getting the fold-paths? I just downloaded the data and everything seems to be there

Hi @michaeltrs, I believe that the files in the fold-paths directory are used to store the paths of files in Pickle24x24. Therefore, I wrote a code to generate these files myself.

@CChBen that is correct. You will need the same folds if you want to compare your results with our paper or Garnot 2021 (same paths). Let me investigate what is the issue with the data, in the meantime I have uploaded the folds as a separate file to download.


Add PlayerAnimator and BendyLib

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

PR is all messed up because of the Gradle features I still need to implement in ForgeKonfig, I'll take another look at it and implement it once I fix that.


Allow automatic discovery of OpenAPI spec files

Do you think it's going to be necessary to filter out files that have an explicit configuration stanza for them, for files that require custom arguments or other workarounds?

I think the scheme is very flexible. You get a DiscoveredFile and are supposed to return a List of GuardRailTasks. That means you can inspect the file, return an empty list to filter it or anything in between.

ping :smile_cat:

@sideeffffect Hey, I was trying to figure out what didn't work about the default stuff. I use that functionality all the time with the CLI, I was really surprised to see it not work here, but that shouldn't block your PR. Sorry about that

:heart:


Startup fail / NullPointerException

startup error: https://hastebin.com/ikoquxozux.css

Solved, I didn't see last update :)


Awesome work! Might want to fix typo

"./appleLoops.py --package-set garageband--mandatory-only"
Needs a space after "garageband".

Thanks, and uh, yeah, just testing to make sure people are reading it... Well, that's my excuse and I'm sticking with it! :)
Fixed :)

LOL...I'm stealing that one. :):):)


Source Map Issue

Bug Report:

What is the expected behavior?

Upon loading the site in the browser i expect the source map be available

What is the current behavior?

attempts to use '/performance-now.js.map' however it appears to just spew the sites content (html, not json) or 404 (new site 404 vs my lightly edited)

Steps to reproduce:

Follow setup directions (clone/npm install)

start project with 'npm start dev'

navigate to the entry point (http://localhost:3000) and load up devtools (Test on Chrome and Firefox)

Open devtools and look at console output. Firefox had the most detail but this is from a clean project (404) but my lightly edited project loads that file but its just the html content of the site, not json map

Useful Info:

Your environment:

Node version: 9.5.0
Browser: current Chrome/Firefox
OS: Windows 10

Other Info:
From poking around it looks like the vendor.js is created correctly and points to the performance-now.js.map however I havent made any progress past this

This because vendor.js is handled by AutoDllPlugin and the AutoDllPlugin does not inherit from the parent config by default, so those sourcemap things will leave untouched and that's the problem because some of vendor entries listed in webpack_config/config.js depend on some packages which reference sourcemapURL in thier code.
Those packages are react-headroom, react-helmet and styled-components (styled-components itself contains sourceMapURL as well) ... you can remove them from the list if 404 errors  look annoying to you.
AutoDllPlugin  has option to inherit config from parent but it marked as experimental and buggy. (https://github.com/asfktz/autodll-webpack-plugin#options)

@ball6847
Awesome, thanks for the explanation!
@jrich523 Could I close it?


recipes-kernel: Linux 5.13 bump to rev a14713f34448 (v5.13.5)

Changes are from stable v5.13.4..v5.13.5.
Signed-off-by: Aníbal Limón<EMAIL_ADDRESS>

@ndechesne It builds but sdx55-mtp fails.


The panel to add a dependency is missing

Describe the bug
The turbo package manager is missing from the user interface of an angular-material project after forking from the project's documentation. It is not possible to add additional packages to the derived project.
Link to the blitz that caused the error
https://stackblitz.com/edit/angular-xzfzqt
Steps to reproduce

Open project
Look at left panel
Click through side tabs

Expected behavior
One would expect to see the turbo package manager available.
Note that parity with local is based on launching the desktop interface for stackblitz from the browser.
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
Actual outcome:

Expected outcome from another project:

Platform
Browser name  = Chrome
Full version  = 98.0.4758.80
Major version = 98
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/98.0.4758.80 Safari/537.36
performance.memory = {
  "totalJSHeapSize": 88237791,
  "usedJSHeapSize": 77948007,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = d7261b7448b4a0bf35c1ba6d98461d9487544022

Additional context
The blitz itself is forked from an example in the angular material documentation.

@kvjrhall Thanks for the report. The project you have linked is not one of the classic Angular-based projects, but a WebContainer one, which means it's actually a full Node application running in your browser.
Instead of the dependency panel, you have access to a terminal where you can add dependencies as you would in a local setup:
npm install lodash

# These work as well
# yarn add lodash
# turbo install lodash

Thank you for the clarification! I suspected it was going to be something
like that. I'm new to the platform and it's not quite foolproof enough for
me yet.
On Fri, 11 Feb 2022, 15:32 Roberto Vidal, @.***> wrote:

@kvjrhall https://github.com/kvjrhall Thanks for the report. The
project you have linked is not one of the classic Angular-based projects,
but a WebContainer one, which means it's actually a full Node application
running in your browser
https://blog.stackblitz.com/posts/introducing-webcontainers/.
Instead of the dependency panel, you have access to a terminal where you
can add dependencies as you would in a local setup:
npm install lodash
These work as well
yarn add lodash
turbo install lodash
—
Reply to this email directly, view it on GitHub
https://github.com/stackblitz/webcontainer-core/issues/591#issuecomment-1036597703,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APDOKNN57XETZC7MMUMGZETU2VW5RANCNFSM5OFCTMJA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>