registerd images resolution

I can't find any parameter in the launch files to change image resolution, is there any option flag to adjust resolution? 1280*720 or 640*480

See this tutorial for a guide on how to set your camera parameters.
Let me know if you have further questions.


Parameters support

Description
Adds support of parameters adjustment
Resolves #37
PR Checklist

[x] Tests added
[x] Changelog updated

@em-pe Please have a look. Code climate reported too large client file, I'm just splitting it into client+generator, but it's better to to id in the separate PR, as it doesn't hide the params logic.

@em-pe Please have a look. Code climate reported too large client file, I'm just splitting it into client+generator, but it's better to to id in the separate PR, as it doesn't hide the params logic.


Android Studio badge doesnt copy properly

Hi,
When copying the android studio badge on https://home.aveek.io/GitHub-Profile-Badges/, it copies as https://img.shields.io/badge/Android Studio-3DDC84.svg?style=for-the-badge&logo=Android-Studio&logoColor=white, which has a space in the URL meaning that github wont take it as an image url, so it wont display. The correct formatting should have a _ between Android and Studio (https://img.shields.io/badge/Android_Studio-3DDC84.svg?style=for-the-badge&logo=Android-Studio&logoColor=white)
Thanks :)

Actually this happens for every icon with a space in its name.

Issue has been resolved with correct URI encoding

Thanks :)


Support pointer-based reverse lookups

Looks for use of dot notation in 'target' param on GET_MANY_REFERENCE actions.

Hi Christian,
thanks for contributing!!
Can I ask you the purpose of the val constant?
const val = params.targetPointerClass ? pointer : params.id;

Hm it appears there's no purpose. Dead code.


Using wav2vec2 with audio

Simple implementation of wav2vec2 leads to relatively poor transcription.  Try different audio data files as well as look deeper into the built-in models to see if there are better options.
More thorough comparison of noisy vs clean audio.

Finished


[Template] Install dependencies failing in nightly runs

Template nightly test runs are failing with:

Error message:
node common/scripts/install-run-rush.js install
========================== Starting Command Output ===========================
/usr/bin/bash --noprofile --norc /mnt/vss/_work/_temp/b90c7bd1-a602-4005-89a2-8e5cfa10b64f.sh
The rush.json configuration requests Rush version 5.63.0
Transforming /mnt/vss/_work/1/s/common/config/rush/.npmrc
--> "/mnt/vss/_work/1/s/common/temp/install-run/<EMAIL_ADDRESS>Installing @microsoft/rush...
npm ERR! code E500
npm ERR! 500 Internal Server Error - GET https://registry.npmjs.org/@pnpm%2Flink-bins - Network connection lost.
npm ERR! A complete log of this run can be found in:
npm ERR!     /home/cloudtest/.npm/_logs/2022-09-15T10_34_27_554Z-debug-0.log
Error: Unable to install package: Error: "npm install" encountered an error

For more details check here:

https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1852603&view=results

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.3713667,Service Bus:0.0630009,Network - Virtual Network:0.032647185'

Looks like some transient network problem. Recent js - template - tests runs are all green. Closing


Differentiating Between LCWIPs/schemes on browse page

At a glance, and even possibly with close inspection it's not possible to differentiate between which features belong to which scheme/LCWIP.
We have a 'bounding polygon' tag you can apply to polygons drawn in the sketcher. Maybe we want to encourage the use of this and make it used by the browse page?
I'm sure there are plenty of other options for this too.

Maybe we want to revisit the idea of a 3 tier system: superschemes/LCWIPs, schemes, interventions. Thiscould be reintroduced by putting all schemes from a file into a single superscheme @dabreegster

The boundary polygons have caused UX problems in the past; I'm not sure about encouraging more. https://github.com/acteng/atip/blob/a965c5baf86e6d7a66d570f77a1e428fbbceccce/src/lib/browse/data.ts#L44
If we had confidence the boundary polygons wouldn't overlap, we could render them as non-interactive objects and style clearly as boundaries using dashed lines. But also, manually drawing these sounds tedious and ripe for inconsistencies. We could calculate convex or concave hulls and see if that helps? Or do some kind of KNN matching.
As a brute force thing to try, we could color every scheme a different color. Since there's too many, we could cycle through just some colors, but try to use different colors for adjacent schemes.
Maybe in the initial tooltips, we could emphasize scheme too, not just feature name.
I don't understand the superscheme idea. We have many schemes (both from LCWIPs and other sources) next to each other on the map, and visually distinguishing them is hard -- that's the root problem we've got?

Hi, stupid question but the black lines are obviously routes included as part of different LCWIPs, however, is there are demarcation for the actual LCWIPs themselves? i.e. something to track which routes belong to which LCWIPs/any geographic demarcation of which LCWIPs are where?

I ask as it would be useful to denote which schemes belong to which LCWIPs

Just putting the specific ask here for more context


Latest version recommends Node JS 17 and above but it doesn't officially support by Homebridge.

Describe the bug
It's not really a bug but I just want to check to make sure that everything is alright as the latest version of the plugin suggests Node JS17 to be installed but Homebridge will not officially support the odd number version of Node JS so Node JS 17 is not going to be officially supported by Homebridge. In this case, will I still be able to use all the function of this plugin if I update it to the latest version but still staying with Node JS 16.X?
Does anyone have any experience of running the latest version of the plugin with Node JS 16.X or upgrading to Node JS 17 and how does that work with homebridge? It would be nice to get to hear from experienced users before jumping in either ways.
Thank you in advance.

Yes. It was a typo when updating package information. But shouldn’t be a problem.

Noted.
I'm updating now. Thanks for the confirmation.


Need Powershell cmdlets for Billing.

At present, the Billing information of the GCP Compute Engine is only possible via REST API's. Could not find related Powershell cmdlets anywhere on the https://googlecloudplatform.github.io/google-cloud-powershell/#/ page.

@kharadzv Currently the project is in maintenance mode and we do not plan to support adding any new cmdlets. Pull requests are welcomed though if you want to add new cmdlets for this.


Add option to auto prefix states so each is Fully Qualified

See discussion: https://github.com/StateSmith/StateSmith/discussions/125
I don't think this will be the default, but we can make it an option people can turn on. Could be very helpful for very large designs.

Here's a quick potential implementation: https://github.com/StateSmith/StateSmith-examples/tree/main/proposal-fully-qualified-state-ids
I didn't spend much time on it. If anyone is interested in this idea, can they please help test it?

I would still prefer this (from the discussion):
As a trade-off, the solution could be just to generate the state name in for "parentState_childState" only for the duplicates, and only up to the different parent name.
So, unless there are duplicate states (which I guess is rare), everything would look like it is now.
Is there anything you find could be a problem? I can imagine that logic is much more difficult to code.

I would still prefer this (from the discussion): As a trade-off, the solution could be just to generate the state name in for "parentState_childState" only for the duplicates, and only up to the different parent name.

Me too.

So, unless there are duplicate states (which I guess is rare), everything would look like it is now. Is there anything you find could be a problem? I can imagine that logic is much more difficult to code.

Here's a separate ticket for shortest fully qualified names: https://github.com/StateSmith/StateSmith/issues/138
Once we determine the strategy for naming, the coding will be easy. I just want to make sure that our naming strategy considers every possibility.

It could even be the default behavior.

Agreed. Once we are sure that the implementation is thoroughly tested.

Closing in favor of https://github.com/StateSmith/StateSmith/issues/138


Problem getting tracks, may be me

when trying to gettracks i receieve the following error
SyntaxError: Unexpected token e in JSON at position 0
at JSON.parse ()
at C:\node_modules\spotify-url-info\src\index.js:36:18
Spotify URL(s)
(https://open.spotify.com/playlist/37i9dQZF1E37tA0YCiV36c?si=d13a383ec4294e07)
Your Code
try{
let songs = await getTracks(args[1]);
}
catch(err)
{
console.log(err);
}
i was using a older version i believe on discord.js v12 and im having to update everything i used to call
let songs = await spAPI.getTracks(args[1]);

Can you try https://github.com/microlinkhq/spotify-url-info/releases/tag/v3.1.6 ?

Yeah I download that version as well as some older ones and I am getting the same error

shipped https://github.com/microlinkhq/spotify-url-info/releases/tag/v3.1.7 that should address your issue.
Can you confirm?

Looks like it, thanks a lot


Something doesn't make sense in listener, maybe a bug

In the code bellow, observedBlock is always latestBlock in any case, so contract.queryFilter(listener.eventName, observedBlock, latestBlock) will filter from latestBlock to latestBlock, this doesn't make sense
https://github.com/Bisonai/orakl/blob/bbe32cd19e3d840acb8a0a5b40c65e7db2840f81/core/src/listener/listener.ts#L38-L52
If the intent is to query events from the last observed block up to the latest block, the code should create const fromBlock = observedBlock before update observedBlock = latestBlock and edit query to contract.queryFilter(listener.eventName, fromBlock, latestBlock)

Hello @kodyfanz,
We are aware of this issues and currently investigating this as well. Due to the above described bug, some blocks can be missed.
Cheers,
Martin

This issue is duplicate of https://github.com/Bisonai/orakl/issues/465. Closing this issue. All further communication will be done through the #465


Bug fix: fix missing Migrations.migWriteError lang data

The "migWriteError" language data is not registered at system/Language/en/Migrations.php file, so, when write migration failure, it was only get text "Migrations.migWriteError".
I've added the "migWriteError" language fixed it with use existing "Migrations.writeError" data to system/Language/en/Migrations.php file. I provided unit test to it at tests/system/Commands/SessionsCommandsTest.php.
Checklist:

[x] Securely signed commits
[x] Unit testing, with >80% coverage

I checked that there is another file that uses "Migrations.writeError" language data at system/Command/Database/CreateMigration.php.
For consistency, I updated to use existing "Migrations.writeError", added path to parameter so the error is more verbose.

@MGatner travis build at 7.2:sqlite seems unrelated as due to redis

This has been failing on a few tests. Unfortunately I cannot override failed CI results so @lonnieezell will need to get involved.

I will close and re-open the PR to trigger build

@MGatner travis build green 🎉


ISPN-5369 SuspectException can be thrown without the target node leaving

https://issues.jboss.org/browse/ISPN-5369
Ignore SUSPECT events in the request correlator

just small comments. it looks good :)

Just so I understand.  So basically if you have a command being ran in the dispatcher and a node is suspected (not yet removed) we still let any pending request to wait still.  The request won't then throw the SuspectedException until the view change occurs with that member gone, right?

yes @wburns.

@pruivo K, great that is what I needed :)  I am surprised we never ran into this before.

@pruivo Updated.
@wburns I'm not that surprised, I think it's really unlikely to get a heartbeat from a node and unsuspect it right between the timeout expiring and GMS installing the new view.

@danberindei I meant more the fact that you would get a SuspectException and retry a command but find you send it to the same node over and over.  I agree regarding unsuspecting a node though.

testing and pushing....

integrated! thanks @danberindei !


On Android, after changing the navigation bar type, the keyboard height is wrong?

Description

Under the gesture navigation, the keyboard information obtained is:

{
   width:<PHONE_NUMBER>496094,
   screenX: 0,
   screenY:<PHONE_NUMBER>328125,
   height:<PHONE_NUMBER>058594
}

In this mode, the height of the navigation bar is calculated to be 15.238095783052017.

After changing to button navigation, the information obtained is:

{
   width:<PHONE_NUMBER>496094,
   screenX: 0,
   screenY:<PHONE_NUMBER>328125,
   height:<PHONE_NUMBER>2265625
}

In this mode, the height of the navigation bar is calculated to be 48.38095292590913.
The height:<PHONE_NUMBER>058594 does it include the height of the navigation bar? If included, the real keyboard height is<PHONE_NUMBER>058594 - 15.238095783052017.
For the second part, the real keyboard height is<PHONE_NUMBER>2265625 - 48.38095292590913.
The two calculation results are not equal. But in these two cases, their screenY values are the same.
So, the conclusion is that the keyboard height is wrong, am I right?
Device: Samsung Galaxy A51 (Android 11)
Version
0.66.3
Output of react-native info
System:
    OS: macOS 11.5.2
    CPU: (6) x64 Intel(R) Core(TM) i5-8500B CPU @ 3.00GHz
    Memory: 401.35 MB / 8.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 15.4.0 - /usr/local/bin/node
    Yarn: 1.22.17 - /usr/local/bin/yarn
    npm: 7.0.15 - /usr/local/bin/npm
    Watchman: Not Found
  Managers:
    CocoaPods: 1.11.2 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.5, DriverKit 20.4, macOS 11.3, tvOS 14.5, watchOS 7.4
    Android SDK: Not Found
  IDEs:
    Android Studio: 4.2 AI-202.76<IP_ADDRESS>86908
    Xcode: 12.5/12E262 - /usr/bin/xcodebuild
  Languages:
    Java: 11.0.8 - /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/javac
    Python: 2.7.16 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: Not Found
    react-native: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Steps to reproduce

In the gesture navigation mode, let the Textinput get the focus. (You can see that Textinput is just at the top of the keyboard)
Go to the phone settings and switch the navigation bar type to button navigation.
Restart the app and let Textinput get the focus. (You can see that the keyboard will cover the Textinput)

Snack, code example, screenshot, or link to a repository
Demo: https://github.com/1280103995/react-native-keyboard-debug/tree/main

This is a full screen, so the StatusBar and NavigationBar still have a height.
Then, keyboardHeight should be: frames.endCoordinates.height + getNavHeight().
Therefore, the moving distance of Textinput should be: keyboardHeight + StatusBar.currentHeight, so that it will not be covered by the keyboard.


[STM32] Support LTDC peripheral

It is already (partially) supported in stm32f4xx hal (https://docs.rs/stm32f4xx-hal/latest/stm32f4xx_hal/pac/ltdc/index.html) but embassy does not seem to have any support for this, yet.
I would like to use it with the STM32F4 Discovery kit for STM32F469.
An example for how this could work with, e.g., slint is found here https://github.com/slint-ui/slint/blob/master/examples/mcu-board-support/stm32h735g.rs though this is with an STM32H735G, but I guess it would work very similar.
As you can see in the example, support in the STM32yyxx-hals is not great and I think being able to interface with displays from embassy would be a great addition and make for some nice demos.
Currently I have no good ideas what an implementation would look like. So any input on this is welcome. Eventually I would like to also work on some abstractions over the exact display used, but that will supposedly need some more discussion once we have LTDC confirmed working.
In the meantime, supporting the peripheral on a PAC level from stm32-data would be great and higher level abstractions could be implemented once the PAC part is confirmed working.

Hi, I'd like to take a look at this as well!

Can be closed since https://github.com/embassy-rs/stm32-data/pull/468 and https://github.com/embassy-rs/embassy/pull/2903 are merged now.


Point to new domain

Since this project has moved to the wmcloud.org domain, change wmflabs.org to wmcloud.org

Thanks!


feat: Add disconnect button on auth config pages and form login callout banner

Description

Earlier, for disconnecting any authentication method viz. Google/Github, the super admin had to empty each of the fields and click Save & Restart button. This has been fixed in this PR by the addition of a disconnect button on each of the authentication method config pages.
Added a form login callout banner on the form login auth config page, which warns the super admin that the email IDs used in form login/signup will not be verified.

Fixes #9936
Type of change

New feature (non-breaking change which adds functionality)
This change requires a documentation update

How Has This Been Tested?

Disconnect button

Added config needed for enabling google authentication.
Once it is enabled, checked if disconnect button is seen on google authentication config page.
Clicked on disconnect button and checked if Google authentication gets disabled on login screen.

Form login callout banner

Went to form login authentication config page.
Checked if a red warning banner exists with a link to docs.appsmith.com

Screen recording of changes done on local machine:

Checklist:

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

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: feat/form-login-calout

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
55.67 (-0.08)
37 (-0.07)
35.13 (-0.67)
56 (-0.11)

:red_circle:
app/client/src/ce/configs/index.ts
97.3 (0)
86.14 (-1.49)
100 (0)
100 (0)

:red_circle:
app/client/src/ce/constants/messages.ts
76.95 (-0.03)
100 (0)
31.65 (-0.1)
81.8 (-0.14)

:sparkles: :new:
app/client/src/components/ads/ColorSelector.tsx
21.43
22.22
0
23.08

:sparkles: :new:
app/client/src/components/ads/DisplayImageUpload.tsx
28.57
12.5
0
32

:sparkles: :new:
app/client/src/components/ads/IconSelector.tsx
22.73
8.89
0
25.64

:sparkles: :new:
app/client/src/components/ads/NumberedStep.tsx
54.55
100
0
54.55

:sparkles: :new:
app/client/src/components/ads/RectangularSwitcher.tsx
26.67
30
0
30.77

:sparkles: :new:
app/client/src/components/ads/SearchInput.tsx
42.86
21.43
14.29
45.45

:sparkles: :new:
app/client/src/components/ads/ShowcaseCarousel.tsx
26.53
37.5
0
28.26

:sparkles: :new:
app/client/src/components/ads/Statusbar.tsx
45
44.44
0
45

:sparkles: :new:
app/client/src/components/ads/TabItemBackgroundFill.tsx
46.15
50
0
50

:sparkles: :new:
app/client/src/components/ads/Table.tsx
19.15
50
0
19.15

:sparkles: :new:
app/client/src/components/ads/TableDropdown.tsx
27.66
28.57
0
31.71

:sparkles: :new:
app/client/src/components/ads/UserProfileImagePicker.tsx
23.33
0
0
25

:sparkles: :new:
app/client/src/components/ads/index.ts
100
100
0
100

:sparkles: :new:
app/client/src/components/ads/formFields/UneditableField.tsx
75
50
0
75

:green_circle:
app/client/src/constants/ThirdPartyConstants.tsx
91.67 (0.76)
100 (0)
0 (0)
90.91 (0.91)

:sparkles: :new:
app/client/src/pages/Settings/FormGroup/Accordion.tsx
50
57.14
0
55

:sparkles: :new:
app/client/src/pages/Settings/FormGroup/RedirectUrlForm.tsx
68.18
100
0
68.18

:red_circle:
app/client/src/pages/Settings/FormGroup/TextInput.tsx
100 (0)
60 (-15)
100 (0)
100 (0)

:red_circle:
app/client/src/pages/Settings/FormGroup/group.tsx
92.68 (-4.62)
56.67 (-15.55)
100 (0)
92.5 (-4.72)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

/ok-to-test sha=20db1fe

/ok-to-test sha=20db1fe

/ok-to-test sha=769eeb8

/ok-to-test sha=28b1e9a

/ok-to-test sha=03c66a5


Opensnitch runtime error due to Python package not currently in nixpkg repo

Describe the bug
On unstable, when I run “opensnitch-ui” I get the following error:
/nix/store/dyjz8kf7id22i9js6lkknq71pvm3z1i6-python3.11-protobuf-5.28.2/lib/python3.11/site-packages/google/protobuf/runtime_version.py:112: UserWarning: Protobuf gencode version 5.27.2 is older than the runtime version 5.28.2 at ui.proto. Please avoid checked-in Protobuf gencode that can be obsolete.
  warnings.warn(
Traceback (most recent call last):
  File "/nix/store/0k12vibb7qw546nw3wpwvys1zaz9yz3v-opensnitch-ui-1.6.6/bin/.opensnitch-ui-wrapped", line 42, in <module>
    from opensnitch.service import UIService
  File "/nix/store/0k12vibb7qw546nw3wpwvys1zaz9yz3v-opensnitch-ui-1.6.6/lib/python3.11/site-packages/opensnitch/service.py", line 16, in <module>
    from opensnitch import ui_pb2_grpc
  File "/nix/store/0k12vibb7qw546nw3wpwvys1zaz9yz3v-opensnitch-ui-1.6.6/lib/python3.11/site-packages/opensnitch/ui_pb2_grpc.py", line 19, in <module>
    raise RuntimeError(
RuntimeError: The grpc package installed is at version 1.64.1, but the generated code in ui_pb2_grpc.py depends on grpcio>=1.66.1. Please upgrade your grpc module to grpcio>=1.66.1 or downgrade your generated code using grpcio-tools<=1.64.1.

The latest version of grpcio 2 available is v1.64.1. How would I go about getting grpcio v1.66.1 or later in NixOS?
I also tried installing grpc (v1.66.1) and python313 to no avail.
Steps To Reproduce
Steps to reproduce the behavior:

Switch from 24.05 to 'unstable'
Run opensnitch-ui

Notify maintainers
@onny
Metadata

system: "x86_64-linux"
host os: Linux 6.6.57, NixOS, 24.11 (Vicuna), 24.11.20241020.1997e4a
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.24.9
channels(root): "nixos-24.05"
nixpkgs: /nix/store/dyzl40h25l04565n90psbhzgnc5vp2xr-source

Add a :+1: reaction to issues you find important.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/opensnitch-runtime-error-due-to-python-package-not-currently-in-nixpkg-repo/54748/4

Once this pull request is merged into unstable, it should work. For now, you'll need to write an overlay for opensnitch-ui. I'll just wait.

#350432 merged successfully and the changes are now in the unstable branch. I was able to run opensnitch-ui without errors, therefore closing this issue.


New Lambda snippet to convert S3 Storage class - lambda-function-convert-s3-storageclass

Issue #, if available:
Description of changes:
New Lambda snippet to convert S3 Storage class
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for the contribution @arunt4ever


chore: fix codesandbox default privacy

Motivation
CodeSandbox repros we receive lately are private by default, which is annoying:

Apparently, new query string options have been implemented to prevent this, so let's figure what is the best for our use case.
https://twitter.com/CompuIves/status/1773028692752539825

Note to self: from this twitter conversation I understand we can use the new ?fork=true&privacy=public query string.
Using fork=true is not an ideal general UX to me because users get presented with a modal immediately even before seeing Docusaurus running in a sandbox, so let's only use privacy=public alone


Wrong disallowSpacesInCallExpression

<EMAIL_ADDRESS>const newObj = dataIsMap || dataIsSet ? new data.constructor : {};

if (dataIsMap) { // Warning

}

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/25209022-wrong-disallowspacesincallexpression?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

Thanks @kobezzza. Nice catch.
Reproduced with a failing test:
    it('should not report on an if statement followed by a NewExpression #(1594)', function() {
        checker.configure({ disallowSpacesInCallExpression: true, esnext: true });
        assert(checker.checkString('const newObj = dataIsMap || dataIsSet ? new data.constructor : {};\n\nif (dataIsMap) {\n\n}').isEmpty());
    });


Unable to present revocable credential

Steps to reproduce:

Run the ACA-py faber demo with revocation enabled
Issue a credential
Send Proof Request

Issue arises if a tails server is not setup. Please consult documentation at: https://github.com/hyperledger/aries-cloudagent-python/blob/main/demo/AliceGetsAPhone.md#run-an-instance-of-indy-tails-server and https://github.com/bcgov/bc-wallet-mobile/discussions/206


Display incorrectly on Kindle

The navigation of Keyboard Shortcuts always display in the front and can't close.

Miniflux is not supported on Kindle browser. As mentioned in the documentation, this software is tested only on Firefox, Chrome and Safari. I do not own any Kindle.

Kindle browser is the same as Internet Explorer.

Internet Explorer is not supported either. See https://docs.miniflux.net/en/latest/requirements.html#web-browsers

This won't be fixed because the mentioned browsers are not supported.


fix(build): use seperate target to install atc-router lualib

Sister PR https://github.com/Kong/atc-router/pull/61

Summary
Make the step slightly more robust.
Checklist

[ ] The Pull Request has tests
[ ] There's an entry in the CHANGELOG
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Full changelog

[Implement ...]

Issue reference

Fix #[issue number]

We should use the branch  in atc-router to verify it.


No cf_clearance cookie in Chrome or Edge?

I've tried using Google Chrome and Edge (yikes) to find the cf_clearance cookie but I don't see it. I do see __cfduid though. Did they rename it?

Yeah, it looks like they removed that cookie. I'm not sure if it was necessary before, I didn't test without it. You should be able to leave it blank and still download some older files. While I was testing this I found a change they made to newer comics so I'm going to post a new release without that cf_clearance cookie and a fix for the other issue.


UX regression: easily open containing folders of transfers

This is a UX regression though a small issue: in the previous (Qt) version, after a torrent was completed (and probably before, but I had not tested that) a user could double-click a transfer in the transfer list and it would open the containing folder in Explorer, but this no longer works.
An improved version would repeat this old functionality, but also

support hitting the Enter key to open the containing folder, and
allow the user to select multiple transfers from the list then hit the Enter key to open the containing folders for each transfer

Hey @vinajuliette! Thanks for reporting - we'll get it sorted for the next release.


Draft : Do not review:  Add MySqlOffline migration to Azure CLI

Description

Add capability to azure cli to migrate with mysql offline.
Right now in Draft mode.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

dms


uv_fs_statfs fails for paths to files on Windows

uv_fs_statfs does not work on Windows if the path passed to it points to a file. However, it works for directory or root paths. This is not consistent with Unix statfs.
This is happening because on Windows uv_fs_statfs is using GetDiskFreeSpaceW, which only accepts a root directory path as input (according to the docs). It seems to be working with regular directory paths as well on my Windows 10 Pro machine, but fails for paths to files.
uv_fs_statfs(loop, req, "C:\\", cb); // works for root directories
uv_fs_statfs(loop, req, "C:\\path\\to\\directory", cb); // works for regular directories
uv_fs_statfs(loop, req, "C:\\path\\to\\file.ext", cb); // fails for files
Refs: https://github.com/nodejs/node/pull/31351#pullrequestreview-344860172

Version: 1.34.2
Platform: Windows 10 x86-64

I'm not sure if there is a better way to do it on Windows, but libuv could detect the error and retry with the directory portion (GetFullPathNameW() could help there). Alternatively, we could always try to detect the directory portion first.

I've tried, but uv_fs_statfs() succeeds with:
uv_fs_statfs(loop, req, "\\\\?\\C:\\Windows\\", cb);
uv_fs_statfs(loop, req, "\\\\192.168.1.xxx\\C$\\Windows\\", cb);
uv_fs_statfs(loop, req, "..\\", cb);

So what if we try again in the parent directory if it fails?

Extracting the parent directory from the path could work but Microsoft's doc says that the GetDiskFreeSpaceW function only accepts the root directory of a disk. Not sure if passing the parent instead of the root will work on all versions of windows.

So what if we try again in the parent directory if it fails?

I think we're suggesting the same thing.

Not sure if passing the parent instead of the root will work on all versions of windows.

I'm not sure either.


onReady is not reinitilaized on restart

this._onReady is assigned once in the constructor, so calling this._onReadyCallbacks.resolve or this._onReadyCallbacks.reject does not cause any effect on the restart
I wonder how clients can register request and notificaiton handler on restart. A new resolved connection does not have them. For example this code seems to be executed only on first start, never on restart: https://github.com/redhat-developer/vscode-java/blob/1f6783957c699e261a33d05702f2da356017458d/src/extension.ts#L90

Good catch and good question. Since people can cache the promise there is no way how I can reinitialize it since promise states can't be altered from the outside.
I think the best we can do is to send a event and then people can re-fetch onReady

I made the necessary changes which will ship with the next version of the library.
You can then do
	client.onDidChangeState((event) => {
		if (event.newState === State.Starting) {
			client.onReady().then(() => {
				console.log('Server is ready');
			});
		}
	});

```


Enable cuda builds of charm to run non-cuda programs

Original issue: https://charm.cs.illinois.edu/redmine/issues/792

No body.

Original date: 2017-02-01 18:33:40

This seems like a significant bug to me if I understand it correctly and if it is still pertinent: you can't build Charm with CUDA support and run a simple Charm program that doesn't use CUDA?

Original date: 2017-02-01 20:18:24

This bug is fixed/obviated by patch ~~https://charm.cs.illinois.edu/gerrit/#/c/2063/~~ https://github.com/UIUC-PPL/charm/commit/a873294f7fc69b0ea9404863fd94dcd1991bf32d


Step14 - Recover a file

The temp drive letter is has two files.. "data loss readme......" and "IMC.hiv" where do I locate the file to paste to wwwroot?

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

ID: 959d21ed-cd7b-dece-d206-8a207f780dad
Version Independent ID: e440a3e5-f953-ac2f-334c-4d771ed2d009
Content: Tutorial - Back up Windows virtual machines in the Azure portal
Content Source: articles/virtual-machines/windows/tutorial-backup-vms.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

Thanks for the feedback! We are currently investigating and will update you shortly.

@cynthn Can you please share your insights on this one? Thanks.

@tapialuism I think you might be looking at the wrong drive letter. It sounds like you are looking at the temporary drive for the VM (possibly D:) and not the mount point for the backup (which would be a different drive letter, further down the alphabet).

@tapialuism Did you get a chance to take a look shared above? Please let us know if that helped. Thanks.

I will dive in today to take a look. I do recall the that the only drive
letter was D. I’ll check again if the results are the same I’ll repeat the
procedure beginning from installing IIS.
I will keep you informed.
On Fri, Oct 5, 2018 at 9:31 AM Karishma Tiwari - MSFT <
<EMAIL_ADDRESS>wrote:

@tapialuism https://github.com/tapialuism Did you get a chance to take
a look shared above? Please let us know if that helped. Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/16033#issuecomment-427424499,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ApBWaFx0lwsWW2ukMYgkgMKDGM5ECKYzks5uh4lWgaJpZM4XEj6I
.

I tried again I ran the script and received the error below and the script never finishes.
"Exception calling "GetDirectoryName" with "1" argument(s): "Illegal characters in path."
At line:52 char:1

$CurrentDir = [System.IO.Path]::GetDirectoryName($myInvocation.MyComm ...

  + CategoryInfo          : NotSpecified: (:) [], MethodInvocationException
  + FullyQualifiedErrorId : ArgumentException"

The error is referring to this line;
$CurrentDir = [System.IO.Path]::GetDirectoryName($myInvocation.MyCommand.Definition)

@tapialuism Did you try with the old recovery point which contains the iisstart.png file and not the recovery point from which you deleted the iisstart.png file.

@tapialuism I followed the tutorial and I was also not able to find a drive letter (Step 14). I am working with the product team to understand whether this is a bug or by design. In any case, we will make sure that the document is updated to reflect the expected outcome.
The workaround solution that you can use is :

Click on Search and Type 'Create and Format hard disk partitions' or 'Disk Management'.
From the disk management screen, identify the additional disks related to the recovery point. Right-click on it and click on 'Change driver letter and paths'. Select a drive letter, for example G: drive.
Got to G: drive and you will find iisstart.png file there.
You can now copy this file from G: and paste it to \inetpub\wwwroot

Please retry with the above steps and share the results with me. Thanks.

@Karishma-Tiwari-MSFT it sounds like what you are saying is the .exe that is downloaded isn't creating a mount point for the backup anymore? I was going to test it myself, since I ran through it multiple times when I wrote the doc. It may be that they changed the behavior of the .exe or it might be broken (which is what I think you suspect).

@cynthn That's correct. I am about to share the details/observations about it, with you, in an email.

@Karishma-Tiwari-MSFT FYI - when I ran the .exe, I got the following (password removed) which has "0 recovery volumes attached":
Microsoft Azure VM Backup - File Recovery

Please enter the 15 character password, shown on the portal, to securely connect to the recovery point: xxxxxxxxxx
Connecting to recovery point using ISCSI service....
WARNING: Waiting for service 'Microsoft iSCSI Initiator Service (MSiSCSI)' to start...
Connection succeeded!
Please wait while we attach volumes of the recovery point.
0 recovery volumes attached
*************  Open Explorer to browse for files  *************
After recovery, to remove the disks and close the connection to the recovery point, please click 'Unmount Disks' in step 3 of the portal.
Press 'Q/q' key to exit ...

@cynthn Yes, this is exactly what I am seeing. I shared the details in the email. Thanks for testing it out. :)

@tapialuism We will close this issue now. We are working internally on the issue with the Product team and will share any updates here. Until then, please follow the workaround as shared above.
Feel free to reopen, if you have additional questions. Thanks.

Thanks.

No one is bothering to reply...!

#reopen @SAIMMINHAS, the Issue is currently closed, so folks might not be getting notified. I thought this was fixed. Have you tried this recently?

Hi guys,
I've double checked again and it's sorted. 1 question though, when i run the File recovery script on the VM, it creates 2 mapped drives. One is 'C:' drive while the other is a temporary storage drive. Due to some reason, it does not creates another mapped drive which is also available in a vm by default. Any idea why this happens?
It's just that if i had to recover a file from that 2nd drive, then it should also be mapped through the script.

It's the above visible 'D:' drive that does not appears after the Script. Not sure why.

?????

@SAIMMINHAS Sorry for the delay on this. We would need a support engineer to do live troubleshoot with you on this. Can you please send an email to<EMAIL_ADDRESS>with your subscription Id and link to this thread. I will enable support request for you and provide further instructions on it.

Hi Karishma,
I've now done as asked.
Many thanks.

Thanks @SAIMMINHAS
I replied over your email.
I will close this issue, once you have it resolved, we can share the resolution here.


Header line prefix in XML can break the XML parsing

The "headerLinePrefix": "-- ", setting in the script causes the XML parser to fail (can be a mistake on the XML parser side, I don't know). After removing the prefix manually, everything works fine.

Very good Point! had the same issue, and this led to the complete pycharm project being corrupted.
changing the entry under XML to "headerLinePrefix": "", worked fine


handy error messages with fancy arrow

https://github.com/violence/warnie/blob/master/index.js

Looks like we don't need it here for node v6
Do we really need it for v4?


[]

Affected app
Name: [e.g. WhatsApp]
Package id: [e.g. com.whatsapp]

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
System
Android Version: [e.g. 10]
Custom ROM: [e.g. LineageOS 17.1]
microG
microG Core version: [e.g. <IP_ADDRESS>915]
microG Self-Check results: [e.g. All ticked]

Additional context
Add any other context about the problem here.

Affected app Name: [e.g. WhatsApp] Package id: [e.g. com.whatsapp]
Describe the bug A clear and concise description of what the bug is.
To Reproduce Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior A clear and concise description of what you expected to happen.
Screenshots If applicable, add screenshots to help explain your problem.
System Android Version: [e.g. 10] Custom ROM: [e.g. LineageOS 17.1]
microG microG Core version: [e.g. <IP_ADDRESS>915] microG Self-Check results: [e.g. All ticked]
Additional context Add any other context about the problem here.


Footer Links should have underline on hover

Footer Links should have underline on hover
It will be great to understand we are hovering on links if we add underline while hovering the links in footer
Now there is no response while we are on the footer , this change might give some life to the footer
if this feature can be approved then I can work on it

That would be superb. Open a PR and let me know :)
Thank you for your contribution.

I have done the changes and PR is sent #190


Action plugin 'set_actuator_async' switch statement index changed to zero-based

The pull request is related to this issue

Actually I thought about returning an error at first, but then I saw that neither the original method did any check on the parameters range (index is a const int). So I kept it that way.
I don't have full knowledge of how errors are handled in the async commands but, looking at the source code of other methods of the same class (e.g. goto_location_async), I presume that, in the else branch, I should use the method:
command_result_callback(Action::Result::CommandDenied, callback);
and then return
Is it right?

Yes, but CommandDenied is only for the case where the vehicle denies it, and it looks like we don't have an InvalidArgument for action. Let's merge it like this and add that later.
Thanks again.

I’m happy to help. Thank you!


Strange rails4 + establish_connection issue

Hey
I'm sorry if this is described vaguely, I'm afraid we haven't been able to pinpoint the problem yet.
Situation is as follows:

We have a legacy database for a number of our models. We connect to this using establish_connection in the models and also use self.table_name_prefix to change the table name.
We set up database_cleaner to use different strategies for this database (I honestly forgot why, apparently it was required at one point but I forgot the details):

DatabaseCleaner[:active_record, {connection: :mail_database_test}].strategy = :truncation

Upon upgrading to Rails4, I changed all observers to concerns. These just add an after_create, destroy and update callback to a bunch of models, notifying our logging system of changes made.

We noticed a number of tests started to break. Removing the above code (altered strategy) seemed to improve, but that's obviously not ideal. We also noticed that the tests didn't break immediatly, they only broke after we actually went and removed the observers themselves. If the observer class is still there (but empty) the tests work. If we remove the file, they break.
19) Api::AccountsController#update should update an account
     Failure/Error: @client = create :client
     ActiveRecord::StatementInvalid:
       Mysql2::Error: Table 'app_secondary_db_test.clients' doesn't exist: SHOW FULL FIELDS FROM `clients`

The clients table doesn't exist because this model actually uses the primary database.
We get the feeling this may be a problem with databasecleaner, possibly with the caching of the tables.
Completely out of ideas. Thanks!

@wendy0402 You could try with this alternative: https://gist.github.com/mgreenly/1109325


[Minor] improvement(docs): Optimize Trino related docs and fix some minor mistakes in docs

What changes were proposed in this pull request?
Refine some descriptions and correct mistakes in Trino-related docs.
Why are the changes needed?
Make users more aware of the documents.
Does this PR introduce any user-facing change?
N/A.
How was this patch tested?
N/A.

Everything has been resolved.

@yuqi1129 please rebase the code.

@yuqi1129 please rebase the code.

Done


Add LiveUpdateContribution.remove

Feature Summary
This feature provides intaface to POST /api/live/thread/delete_udpate which removes a live update.
References

https://www.reddit.com/dev/api#POST_api_live_{thread}_delete_update
#676 - feature request for reddit live
nmtake@e147a37 - API design discussion

Updated.

Thanks!


Error retrieving coinbase currency rates

Url works just fine on browser.
Tested with 1.8.2 but is not a regression error, I think coinbase rates never worked (at least since 1.7.x)
Connected to my own node via tor

debug log
2024-01-20 11:58:33,770 DEBUG [Thread-57] o.e.j.u.s.SslContextFactory [null:-1] Selected Protocols [TLSv1.3, TLSv1.2] of [TLSv1.3, TLSv1.2, TLSv1.1, TLSv1, SSLv3, SSLv2Hello]
2024-01-20 11:58:33,770 DEBUG [Thread-57] o.e.j.u.s.SslContextFactory [null:-1] Selected Ciphers   [TLS_AES_256_GCM_SHA384, TLS_AES_128_GCM_SHA256, TLS_CHACHA20_POLY1305_SHA256, TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384, TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256, TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305_SHA256, TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384, TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305_SHA256, TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256, TLS_DHE_RSA_WITH_AES_256_GCM_SHA384, TLS_DHE_RSA_WITH_CHACHA20_POLY1305_SHA256, TLS_DHE_DSS_WITH_AES_256_GCM_SHA384, TLS_DHE_RSA_WITH_AES_128_GCM_SHA256, TLS_DHE_DSS_WITH_AES_128_GCM_SHA256, TLS_ECDHE_ECDSA_WITH_AES_256_CBC_SHA384, TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384, TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA256, TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256, TLS_DHE_RSA_WITH_AES_256_CBC_SHA256, TLS_DHE_DSS_WITH_AES_256_CBC_SHA256, TLS_DHE_RSA_WITH_AES_128_CBC_SHA256, TLS_DHE_DSS_WITH_AES_128_CBC_SHA256, TLS_ECDH_ECDSA_WITH_AES_256_GCM_SHA384, TLS_ECDH_RSA_WITH_AES_256_GCM_SHA384, TLS_ECDH_ECDSA_WITH_AES_128_GCM_SHA256, TLS_ECDH_RSA_WITH_AES_128_GCM_SHA256, TLS_ECDH_ECDSA_WITH_AES_256_CBC_SHA384, TLS_ECDH_RSA_WITH_AES_256_CBC_SHA384, TLS_ECDH_ECDSA_WITH_AES_128_CBC_SHA256, TLS_ECDH_RSA_WITH_AES_128_CBC_SHA256, TLS_EMPTY_RENEGOTIATION_INFO_SCSV] of [TLS_AES_256_GCM_SHA384, TLS_AES_128_GCM_SHA256, TLS_CHACHA20_POLY1305_SHA256, TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384, TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256, TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305_SHA256, TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384, TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305_SHA256, TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256, TLS_DHE_RSA_WITH_AES_256_GCM_SHA384, TLS_DHE_RSA_WITH_CHACHA20_POLY1305_SHA256, TLS_DHE_DSS_WITH_AES_256_GCM_SHA384, TLS_DHE_RSA_WITH_AES_128_GCM_SHA256, TLS_DHE_DSS_WITH_AES_128_GCM_SHA256, TLS_ECDHE_ECDSA_WITH_AES_256_CBC_SHA384, TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384, TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA256, TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256, TLS_DHE_RSA_WITH_AES_256_CBC_SHA256, TLS_DHE_DSS_WITH_AES_256_CBC_SHA256, TLS_DHE_RSA_WITH_AES_128_CBC_SHA256, TLS_DHE_DSS_WITH_AES_128_CBC_SHA256, TLS_ECDH_ECDSA_WITH_AES_256_GCM_SHA384, TLS_ECDH_RSA_WITH_AES_256_GCM_SHA384, TLS_ECDH_ECDSA_WITH_AES_128_GCM_SHA256, TLS_ECDH_RSA_WITH_AES_128_GCM_SHA256, TLS_ECDH_ECDSA_WITH_AES_256_CBC_SHA384, TLS_ECDH_RSA_WITH_AES_256_CBC_SHA384, TLS_ECDH_ECDSA_WITH_AES_128_CBC_SHA256, TLS_ECDH_RSA_WITH_AES_128_CBC_SHA256, TLS_ECDHE_ECDSA_WITH_AES_256_CBC_SHA, TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA, TLS_ECDHE_ECDSA_WITH_AES_128_CBC_SHA, TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA, TLS_DHE_RSA_WITH_AES_256_CBC_SHA, TLS_DHE_DSS_WITH_AES_256_CBC_SHA, TLS_DHE_RSA_WITH_AES_128_CBC_SHA, TLS_DHE_DSS_WITH_AES_128_CBC_SHA, TLS_ECDH_ECDSA_WITH_AES_256_CBC_SHA, TLS_ECDH_RSA_WITH_AES_256_CBC_SHA, TLS_ECDH_ECDSA_WITH_AES_128_CBC_SHA, TLS_ECDH_RSA_WITH_AES_128_CBC_SHA, TLS_RSA_WITH_AES_256_GCM_SHA384, TLS_RSA_WITH_AES_128_GCM_SHA256, TLS_RSA_WITH_AES_256_CBC_SHA256, TLS_RSA_WITH_AES_128_CBC_SHA256, TLS_RSA_WITH_AES_256_CBC_SHA, TLS_RSA_WITH_AES_128_CBC_SHA, TLS_EMPTY_RENEGOTIATION_INFO_SCSV]
2024-01-20 11:58:33,771 DEBUG [Thread-57] o.e.j.u.s.SslContextFactory [null:-1] Customize SSLEngine[hostname=null, port=-1, Session(1705748313771|SSL_NULL_WITH_NULL_NULL)]
2024-01-20 11:58:33,771 DEBUG [Thread-57] o.e.j.u.c.ContainerLifeCycle [null:-1] HttpDestination[https://api.coingecko.com]@4be422c2(via <IP_ADDRESS>:9050),queue=0,pool=DuplexConnectionPool@1e98f758[c=0/64,a=0,i=0] added {DuplexConnectionPool@1e98f758[c=0/64,a=0,i=0],POJO}
2024-01-20 11:58:33,771 DEBUG [Thread-57] o.e.j.u.c.ContainerLifeCycle [null:-1] HttpDestination[https://www.sparrowwallet.com]@70d62f95(via <IP_ADDRESS>:9050),queue=0,pool=DuplexConnectionPool@7760dfbd[c=0/64,a=0,i=0] added {DuplexConnectionPool@7760dfbd[c=0/64,a=0,i=0],POJO}
2024-01-20 11:58:33,771 DEBUG [Thread-57] o.e.j.u.c.ContainerLifeCycle [null:-1] HttpDestination[http://mempoolhqx4isw62xs7abwphsq7ldayuidyx2v2oethdhhj6mlo2r6ad.onion]@10d565dd(via <IP_ADDRESS>:9050),queue=0,pool=DuplexConnectionPool@2e2dd41e[c=0/64,a=0,i=0] added {DuplexConnectionPool@2e2dd41e[c=0/64,a=0,i=0],POJO}
2024-01-20 11:58:33,771 DEBUG [Thread-57] o.e.j.u.c.ContainerLifeCycle [null:-1] HttpDestination[https://api.coinbase.com]@4e072efd(via <IP_ADDRESS>:9050),queue=0,pool=DuplexConnectionPool@3a59de31[c=0/64,a=0,i=0] added {DuplexConnectionPool@3a59de31[c=0/64,a=0,i=0],POJO}
2024-01-20 11:58:33,771 DEBUG [Thread-57] o.e.j.u.c.ContainerLifeCycle [null:-1] HttpDestination[https://api.coinbase.com]@7df6a7ed(via <IP_ADDRESS>:9050),queue=0,pool=DuplexConnectionPool@6322dc15[c=0/64,a=0,i=0] added {DuplexConnectionPool@6322dc15[c=0/64,a=0,i=0],POJO}
2024-01-20 11:58:33,771 ERROR [Thread-57] c.s.h.c.JacksonHttpClient [null:-1] getJson failed: https://api.coinbase.com/v2/exchange-rates?currency=BTC:java.lang.Exception: Http query failed: status=403
2024-01-20 11:58:33,771 WARN [Thread-57] c.s.s.n.ExchangeSource [null:-1] Error retrieving currency rates
com.sparrowwallet.nightjar.http.JavaHttpException: java.lang.Exception: Http query failed: status=403
	at com.sparrowwallet.nightjar@0.2.39/com.sparrowwallet.nightjar.http.JavaHttpClient.makeRequest(Unknown Source)
	at com.sparrowwallet.nightjar@0.2.39/com.sparrowwallet.nightjar.http.JavaHttpClient.requestJsonGet(Unknown Source)
	at com.sparrowwallet.nightjar@0.2.39/com.samourai.http.client.JacksonHttpClient.getJson(Unknown Source)
	at com.sparrowwallet.nightjar@0.2.39/com.samourai.http.client.JacksonHttpClient.getJson(Unknown Source)
	at com.sparrowwallet.sparrow@1.8.2/com.sparrowwallet.sparrow.net.HttpClientService.requestJson(Unknown Source)
	at com.sparrowwallet.sparrow@1.8.2/com.sparrowwallet.sparrow.net.ExchangeSource$2.getRates(Unknown Source)
	at com.sparrowwallet.sparrow@1.8.2/com.sparrowwallet.sparrow.net.ExchangeSource$2.getExchangeRate(Unknown Source)
	at com.sparrowwallet.sparrow@1.8.2/com.sparrowwallet.sparrow.net.ExchangeSource$RatesService$1.call(Unknown Source)
	at com.sparrowwallet.sparrow@1.8.2/com.sparrowwallet.sparrow.net.ExchangeSource$RatesService$1.call(Unknown Source)
	at javafx.graphics@18/javafx.concurrent.Task$TaskCallable.call(Unknown Source)
	at java.base/java.util.concurrent.FutureTask.run(Unknown Source)
	at javafx.graphics@18/javafx.concurrent.Service.lambda$executeTask$6(Unknown Source)
	at java.base/java.security.AccessController.doPrivileged(Unknown Source)
	at javafx.graphics@18/javafx.concurrent.Service.lambda$executeTask$7(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(Unknown Source)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
	at java.base/java.lang.Thread.run(Unknown Source)
Caused by: java.lang.Exception: Http query failed: status=403
	... 17 common frames omitted

Unfortunately Coinbase tends to block Tor exit nodes with a 403 (Forbidden) response. Since they do not provide an onion address, there's not much Sparrow can do about this. If you don't use Tor, you'll find it works fine.

Thanks @craigraw  that makes sense. What do you think about a switch or something to optionally obtain rates via clearnet? Just as suggestion

You can switch to Coingecko, which works over Tor.


Support hiding the option to Reset a single tasks

It seems that one cannot hide the option to reset a single task from the right-click menu like what one can do for other options.

I'd like to stick with the groups, for now. I have something else in my head that might replace the context menu options (usefully) =)

Looking forward to that! 😁


Dropping NumPy 1.10

Increasingly we discover issues with PRs after merging them on NumPy 1.10, which we then either need to fix ourselves or hope the submitter of the PR will kindly fix the issue. While these issues can normally be solved by skipping, adding compat functions, etc., it starts to raise the question: how long do we want to support NumPy 1.10. As of January NumPy 1.14 was relased and has since had 2 patch releases. The list of NumPy 1.15 issue is not too long either (roughly two thirds have already been resolved). To me this is a pretty good case for dropping, but am interested to hear other thoughts.
If we do feel strongly about keeping NumPy 1.10, could we discuss adding it as part of the PR builds to cutdown on strain caused by the issues mentioned above?

We need to keep numpy 1.10 for downstream dependencies. I suggest moving the 3.5 build to 1.10, and bumping the 3.4 to the latest numpy that has a 3.4 build.

Works for me. Thanks @jcrist. :)

We need to keep numpy 1.10 for downstream dependencies.

Actually, that's no longer the case. Looks like xarray and scikit-image have both dropped 1.10. In that case, I'd also be fine dropping support. Ping @mrocklin in case I missed any other dependents.

Fine by me
On Fri, Apr 20, 2018 at 11:59 AM, Jim Crist<EMAIL_ADDRESS>wrote:

We need to keep numpy 1.10 for downstream dependencies.
Actually, that's no longer the case. Looks like xarray and scikit-image
have both dropped 1.10. In that case, I'd also be fine dropping support.
Ping @mrocklin https://github.com/mrocklin in case I missed any other
dependents.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/3427#issuecomment-383142250, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszB2Z7i-BvBaBDSIDxXI1wNJYuXFBks5tqgX0gaJpZM4Tdrjg
.

See #3429. Always nice to delete code.

Indeed, very nice!


7.1.0 expected:'Number Token', actual:'"0"'

NEST/Elasticsearch.Net version: 7.1.0-ci20190429T050458
Elasticsearch version: 7.0.0
Description of the problem including expected versus actual behavior:
Request returns with an exception JsonParsingException: expected:'Number Token', actual:'"0"', at offset:518 whilst the property has been marked with attribute Text.
Steps to reproduce:

Logging via log4net elastic search module
Retrieving the logs as:

var results = _client.Search<MyLog>(s =>
                s.Sort(ss => ss.Descending(p => p.TimeStamp))
                    .Size(150)
                    .From(from)
                    .Index("log-something")
                    .Query(
                        q => q.QueryString(qs => qs.Query($"*{query}*")))
            );

"Enjoy" the exception

Provide DebugInformation (if relevant):
The entry it's failing on is "lineNumber": "0",
Returned JSON for an item is
{
    "timeStamp": "2019-04-29T05:50:49.0457231Z",
    "message": "Finished",
    "messageObject": {},
    "exception": {},
    "loggerName": "myproject",
    "domain": "myproject.exe",
    "identity": "",
    "level": "INFO",
    "className": "myproject.Lib.Regger",
    "fileName": null,
    "lineNumber": "0",
    "fullInfo": "myproject.Lib.Regger.Log(:0)",
    "methodName": "Log",
    "fix": "LocationInfo, UserName, Identity, Partial",
    "properties": {
        "log4net:Identity": "",
        "log4net:UserName": "username",
        "log4net:HostName": "somemachine",
        "@timestamp": "2019-04-29T05:50:49.0457231Z"
    },
    "userName": "username",
    "threadName": "3",
    "hostName": "somemachine"
}

It has been marked in the model as followed:
        [Text(Name = "lineNumber")]
        public long LineNumber { get; set; }

Yet the library tries to resolve it as an integer.

Seems the new serializer takes the property datatype for deserialization, as this was defined as long instead of string the deserializer failed.
Can be closed :)

@bluewalk The serializer is much stricter in 7.x, but the benefit is that it is much faster 😃

Hi RussCam I'm unable to solve the above mentioned issue.Actually I kept mapping to strict

How I need to check which field causing the issue?

@kartheek91 Have a look at the documentation on capturing the response. The error message provides the offset, so you should be able to ascertain which document is the culprit.

I kept mapping to strict.

strict mapping only controls If new fields are detected in a document, throw an exception and  the reject the document, so that new fields must be explicitly added to the mapping. What you probably want is to set coerce to false, which is true by default.


Article - Element spacing

Some images in articles are bumping really close to images. There should always be 30px space in between any element e.g. paragraph, image

@Norvard - I think this was an issue with the way WP was sending copy to us. Has this been fixed?
https://next.wellcomecollection.org/articles/queer-territory-claude-cahun-and-a-land-without-labels

@jamesgorrie No, it looks like that specific issue still stands.

This specific issue is the result of the paragraph of text above the image not being contained in a <p> tag (which would give it some bottom-margin). As @jamesgorrie says, this is the result of the way the data's being delivered to us from WP. Don't know if there's anything we could do at our end to auto-wrap instances of text like this in <p> tags – not sure how we'd determine whether or not they should be wrapped (and possibly overkill given it won't be an issue when we move away from WP). Perhaps @RussellDornan can fix this instance in WP (give a shout if it needs clarifying).

You're right @davidpmccormick it's probably not enough of an issue to worry about it just now. Just one more reason for us to move away from WP as soon as we can.

@RussellDornan don't worry about editing this instance in WP! I'm just going to close this issue, but it's just something to be aware of in this transitional time.


Can I draw multiple curves in a single figure ?

I use foo.add_scalar_dict({'train_acc':trainacc,'test_acc':testacc}) to send training and testing accuracy to the server, but they ended up in separate figures. How can I draw them in a single figure ?

Doesn't using the split on underscores option in the tensorboard UI solves this problem?

@albanD Thanks for your reply. The split on underscores shows two accuracies side by side like this.

But I would like to show these two accuracies in a single figure so that I can see how overfit the model is. Just like torch's logger .

The only way I know to trick tensorboard into doing that is to create two experiments with different names (train and test in your case) and then log a scalar with the exact same name in both (acc in your case).

@albanD Thanks, I will try that. BTW, Does pytorch has plan to implement logger or display module like Torch?

The torch tnt framework is planned to be moved to pytorch here: https://github.com/pytorch/tnt


Fixed text reflow when words are longer than the required width

Previously, if a word was longer than the required width, it would overflow. With this change, the word is wrapped so that it stays within the required width (same as CSS overflow-wrap: break-word). Escape codes are preserved when a word is cut.
So before it would look like this:

And now like this:

Cool, cool. Could you squash all the commits and add some tests for it, and I'll merge?

Sure, that's done.

Thanks! Good work. 🎉


feat: Add Realtime Call-Up Demo

Summary by CodeRabbit
Release Notes

New Features

Introduced a new React application for real-time voice calls using the Coze API.
Added comprehensive documentation in the README for setup and usage.
Implemented a custom hook for API interactions, managing authentication and data retrieval.

Styling Enhancements

Added new CSS files for application and call interface styling, ensuring a responsive and user-friendly design.

Configuration Updates

Included new ESLint and Webpack configurations to streamline development and ensure code quality.

New Files

Created essential files including package.json, tsconfig.json, and various entry points for the application.

@jsongo @Tecvan-fe  @chyroc  开源代码有没强制要全部使用英文文案


Explain scaffold generated files.

After the section on the [Update-Database] step, a sentence is included and makes no sense because haven't defined a "DbContext" and there is no created "Models/MovieContext.cs" file. IIRC, this was done in the 2.0 tut, but if you're following step-by-step along, this is absolutely confounding because we haven't done anything involving "DbContext" or a "MovieContext.cs" file in the 2.1 tutorial.
Quoted text:
The schema is based on the model specified in the [DbContext] (In the Models/MovieContext.cs file).

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

ID: 6719f08e-3bd7-dc1a-71df-f2ef9fbca9d8
Version Independent ID: 7096fdb3-612e-9e00-bd0b-8ea4886a09ce
Content: Add a model to a Razor Pages app in ASP.NET Core
Content Source: aspnetcore/tutorials/razor-pages/model.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

In 2.1 the scaffolder create the DbContext.

Yes, I noticed that - we set the DbContext to "RazorPagesMovieContext". I guess my point was there was no previous reference to what a DbContext is, that we've already created one, and that there's a referenced file path that doesn't exist ("Models/MovieContext.cs"). Unless this tutorial isn't meant to be used by n00bs, it was just a note of confusing text. I mean, I've been  crash-coursed into RazorPages, have been up to my ears in learning about it and it still threw me for a loop.

Would it be more clear if I wrote
The schema is based on the model specified in the RazorPagesMovieContext (In the Models/MovieContext.cs file).

What it needs is before Perform initial migration it needs a list of files generated. It should explain the DbContext. That would make it easier to follow
You're right, that's the wrong place.

That would explain a lot. Thanks!

@BrittanyDilts  It's in the data folder. I'll get this updated. Great feedback. It will be a few weeks before I can add the new section, but I can fix this part now.

Yep! listing the correct folder and filename would alleviate all confusion with that bit.  :D


Druid0.13.0 - 405 method not allow when I send post to coordinator url

Affected Version
Druid 0.13.0
Description
I post a lookup to coordinator url ,and it return a 405 error.
curl  -H "Content-type:application/json" -vvv -XPOST http://localhost:8081/druid/coordinator/v1/lookups/config/__default<EMAIL_ADDRESS>it return result is
 HTTP/1.1 405 Method Not Allowed
and My common config is
druid.extensions.loadList=["druid-parser-route","druid-kafka-indexing-service","mysql-metadata-storage","druid-histogram", "druid-datasketches", "druid-lookups-cached-global"]
I think i have added the lookup extension and the json content is ok

Hello，did you solve the issue?

I think you need to post to /druid/coordinator/v1/lookups/config not /druid/coordinator/v1/lookups/config/__default


Optimize Local Storage: Use local storage logic into a custom hook for reusability.

We can encapsulate local storage logic into a custom hook for reusability.

@mehul-m-prajapati yeah man.. work on it


Onboarder's Create Project Broken

Expected
A new project gets created and I'm redirected to that project
Actual
Clicking the create button causes the button to become disabled and nothing happens (no user-facing error message).

@sloria ?

@nanchen47 can you look at this?  We'll want this fixed asap with the science articles coming out today.

Regression was introduced by this hotfix: https://github.com/CenterForOpenScience/osf.io/pull/3208. @chennan47 Any ideas?

Just talked with @sloria. This should now be fixed.


iOS error: Running model failed:Invalid argument: NodeDef mentions attr 'dilations' not in Op<name=Conv2D; signature=input:T, filter:T ->

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): MacOS
TensorFlow installed from (source or binary): pip
TensorFlow version (use command below): 1.6
Python version: 3.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: none
GPU model and memory:none
Exact command to reproduce:

Describe the problem
I was just trying to make a classifier iOS app by simply replacing the inception pb file to my pb file for my classifier.
The iOS app example I was using is camera. It worked well with the original inception.
But when I changed the pb file I got an error as listed below. The pb file was generated with tensorflow-for-poest-2.
Then the model doesn't work. The camera is working but no classification message.
I assume there is some mismatch between versions of TF?
Source code / logs
2018-03-11 20:33:41.906421: E /Users/apple/OneDrive/P/tensorflow/tensorflow/examples/ios/camera/CameraExampleViewController.mm:327] Running model failed:Invalid argument: NodeDef mentions attr 'dilations' not in Op<name=Conv2D; signature=input:T, filter:T -> output:T; attr=T:type,allowed=[DT_HALF, DT_FLOAT, DT_DOUBLE]; attr=strides:list(int); attr=use_cudnn_on_gpu:bool,default=true; attr=padding:string,allowed=["SAME", "VALID"]; attr=data_format:string,default="NHWC",allowed=["NHWC", "NCHW"]>; NodeDef: conv/Conv2D = Conv2D[T=DT_FLOAT, data_format="NHWC", dilations=[1, 1, 1, 1], padding="VALID", strides=[1, 2, 2, 1], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_Mul_0, conv/conv2d_params). (Check whether your GraphDef-interpreting binary is up to date with your GraphDef-generating binary.).
[[Node: conv/Conv2D = Conv2D[T=DT_FLOAT, data_format="NHWC", dilations=[1, 1, 1, 1], padding="VALID", strides=[1, 2, 2, 1], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/cpu:0"](_recv_Mul_0, conv/conv2d_params)]]

Hi, I do have the same problem actually. This seems to be a version mismatch between the generating Tensorflow version and the interpreting one. See here
I recently built the tensorflow_core.a by myself with as stated here.

Hi, I've got the same issue. I retrained the model following the "Tensorflow for Poets" article in docker on macOS. Once launch the camera app it gives the same error.

@andrehentz could you take a look?

Ist seems like the version of tensorflow in the CocoaPods „tensorflow-experimental“ package is 1.1.0. I retrained a model with tf 1.1.0 and optimized it and it worked. Seems like using the CocoaPods version isn‘t a good idea.

@sascha-greiner-adam I tried to specify the tensorflow dependency as pod 'TensorFlow-experimental', '~> 1.1.0' and it still doesn't work. Maybe you could share a repo with a working project that has the right tensorflow pod?

You have to build the pb file with Version 1.1.0. Or if you check out tf version 1.1.0 from github, there is an iOS example without pod. You could copy that example, edit search paths and then check out latest. Build for iOS and then use your copied example app project. This worked for me. Monday I could give you more informations.

Could TensorFlow-experimental please be updated to have the latest binary?

@sascha-greiner-adam what the example gitHub URL?

Nagging Assignees @shivaniag, @andrehentz: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @shivaniag, @andrehentz: It has been 58 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

I have the same issue as well... appreciate if someone can advice how it can be fixed please?

same here!It will be great if someone can help fix it

Add this code where you freeze the checkpoint filed. Mine is called freeze_graph.py
for node in output_graph_def.node:
      if node.op == 'RefSwitch':
        node.op = 'Switch'
        for index in xrange(len(node.input)):
          if 'moving_' in node.input[index]:
            node.input[index] = node.input[index] + '/read'
      elif node.op == 'AssignSub':
        node.op = 'Sub'
        if 'use_locking' in node.attr: del node.attr['use_locking']
      if "dilations" in node.attr: del node.attr["dilations"]
      if "index_type" in node.attr: del node.attr["index_type"]

freeze_graph.py.txt

I think it's a version issue.I use the the libs generated by 'build_all_ios.sh',setting the header search path and lib search path,and the error disappeared when I load the retrained pb file

@cartmanguo Can you give me some more details on what you did? I am kind of new to this environment.

@shubchat  I download latest version of tensorflow,find the .sh file in /contrib/makefile.After compiling,this will give you four static libs,libtensorflow-core.a,libprotobuf-lite.a,libprotobuf.a,nsync.a
include them all in your project,in the XCode build setting tab,set the header search path and library search path
This guide may save you some time:
https://github.com/tensorflow/tensorflow/tree/master/tensorflow/examples/ios#building-the-tensorflow-ios-libraries-from-source

I faced the same issue too but managed to fix it by generating the .pb file using TensorFlow 1.1.* (according to answers in StackOverflow).
As I understand it, the protobuf framework version in our desktop generating the .pb file must be of the same version as the one residing in iOS after Pod installation.
https://www.tensorflow.org/mobile/linking_libs#protobuf_problems

... [T]his generated code needs to be linked against shared libraries for the exact same version of the framework that was used for the generator.

Nagging Assignees @shivaniag, @andrehentz: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @shivaniag, @andrehentz: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @shivaniag, @andrehentz: It has been 45 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

YC, do you maintain the cocoapod for tf mobile?

Nagging Assignees @miaout17, @shivaniag: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @miaout17, @shivaniag: It has been 30 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @miaout17, @shivaniag: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @miaout17, @shivaniag: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @miaout17, @shivaniag: It has been 44 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @miaout17, @shivaniag: It has been 59 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignees @miaout17, @shivaniag: It has been 74 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@hanspond We are checking to see if you still need help on this issue, as you are using an older version of tensorflow(1.x) which is officially considered as end of life. We recommend that you upgrade to 2.6 which is latest stable version of TF and let us know if the issue still persists in newer versions. Thanks!


Can support Distil-Whisper model?

Can support Distil-Whisper model?
I heard that the distillation model has a small scale, very high performance, very fast, and high recognition quality

Hello @lmxin123 ,
Yes, distil models should be supported as described here: https://github.com/sandrohanea/whisper.net/issues/126

thank you，i try it later
hope you can update to 1.6，thanks


Switch requirements N-S vs E-W

Hi,
It's a little ambiguous whether all these are required on North-South switches, or only on East-West. It appears East-West has stricter requirements, but those might be avoided if E-W is done switchless.
For branch-office / edge scenarios, it would be helpful to work with N-S switches that don't do .1Qbb and .1Qaz
Can this be clarified?
thanks.

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

ID: 37289c9d-b579-912c-1bf0-95cd71034a4b
Version Independent ID: 9b6a5ee4-4b17-657b-e1b4-5a8f4ccbfe49
Content: Physical network requirements for Azure Stack HCI - Azure Stack HCI
Content Source: azure-stack/hci/concepts/physical-network-requirements.md
Service: azure-stack
Sub-service: azure-stack-hci
GitHub Login: @jacobpedd
Microsoft Alias: jpeddicord

@gregnashnamoi
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@jacobpedd
Can you please check and add your comments on this doc update request as applicable.


Update composite monitor during detector updation

When detector is updated, if new monitors are created and/or old monitors are deleted we also need to update the composite monitor sequence.
Update detector flow would involve the following interactions with alerting:

create new monitors if any
update sequence of detector's composite monitor by removing old monitor ids(if any monitors need to be deleted) and add new monitor ids
if step one or two fail delete any newly created monitors, else delete monitors which are not required anymore

Followed the steps from the above. The PR that is created for parent issue containing all the changes:
https://github.com/opensearch-project/security-analytics/pull/316

This functionality is implemented, closing


Add DataTable.WriteCsv to return header

I do next
let csv = new CsvWriter(target)
    dataTable.WriteCsv(csv,true) |> ignore

Could I return headers in some form? Or have public method to calculate these?

A CsvWriter has no way to distinguish between a header record and a data record.


[BUG] system restore cannot roll out when volume name is different to the PV

Describe the bug (🐛 if you encounter this issue)
system restore cannot roll out when volume name is different to the PV
To Reproduce
Steps to reproduce the behavior:

Create volume(test-volume)
Create PV(test-volume-pv) and PVC(test-volume-pvc) from test-volume
Create a system backup
Delete the volume
Restore a system backup
System restore blocked

Expected behavior
Should complete restore.
Log or Support bundle
N/A
Environment
N/A
Additional context
None

cc @longhorn/qa

Verified passed on v1.4.x-head (longhorn-manager 2a65750). Create a volume and Create pv/pvc for this volume with different name from the volume name, the volume can be restored in a new cluster.


[AndroidTV, fireTV] Clicking down from menu in @rnv/template-starter -> Focus is lost

Describe the bug
Clicking down from menu in @rnv/template-starter -> Focus is lost, Reproduced only on AndroidTV and fireTV
To Reproduce

Open app
Focus on Home/My Page/My Modal
Click down -> focus is lost (more like it is not lost but focused somewhere on icon but that focus is not visible because clicking second time down results in focus on "Try Me!"

Expected behavior
Focus is on "Try Me!"
Desktop (please complete the following information):

OS: macOS Monterey 12.4
Node Version: 16.14.0
RNV Version: 0.36.0-canary.14
@rnv/template-starter: 0.36.0-canary.14

Additional context
Also when clicking from "Try Me!" to header, user needs to do one more extra click

App scope was changed so not an issue


actions: saveas: Fix crash at access without permission

In the moment os.Stat() fails due to not existing permission fileinfo.Name() will be access, which we need to prevent. In that case we can't be sure, if the file exists or not anyway. Missing privileges should lead to a call of saveBufToFile since this will take care of extending privileges (e.g. by invoking sudo).
Currently the behavior ignores existing files which can't be accessed due to missing permissions. They will be overwritten in the moment sudo is invoked without any further check if the user wants to overwrite possible existing ones.
On micro start with such a files as parameter we will receive the complaint:
stat [FILENAME]: permission denied

Press enter to continue

So maybe we need a more plausible but more complex solution to 1. stat with more privileges and 2. ask to load/read the file with more privileges.
Fixes #3080

Hello

Go docs say that new code shouldn't use os.IsPermission and use errors.Is(err, fs.ErrPermission) instead. At the same time the error is 100% returned by os package but just wanted to let you know
Adding || os.IsPermission(err) was the first "solution" I tried but it didn't work so I tried to add loging which didn't log anything:

if os.IsNotExist(err) || os.IsPermission(err) {
	if os.IsPermission(err) {
		WriteLog(err.Error())
	}

Does os.IsPermission(err) change anything for you? I tried all four situations without any results:

perm: yes, exist: no
perm: yes, exist: yes
perm: no, exist: no
perm: no, exist: yes

The else really works and solves the issue

stat with more privileges

What do you mean? How would it look?

ask to load/read the file with more privileges.

Do you mean separate function for file loading that will silently load file if can or prompt user for permissions?

I will update. Would be nice to force such updates with deprecated warnings or such.
Yes, because it doesn't change the problem caused by unconditional access to the fileinfo.Name(), but you will need it too.
2.1. In the moment you use SaveAs without the permission your action will be silently ignored, because there is no further prompt. By adding this condition we will invoke sudo or equiv. to do the save.
2.2. That's the reason why we should add a functionality to check if the non accessible file exists, because silently overwriting an existing file can cause other problems...even in the moment the user explicitly used super user rights. Sometimes the user doesn't see the consequences coming, so I recommend to do a small check including prompt before overwriting.

Do you mean separate function for file loading that will silently load file if can or prompt user for permissions?

The load/check needs to be done with more privileges, otherwise we can't test. Afterwards the possible YN prompt for overwrite can be displayed.

Hmmm.... We have two different "permission errors". I messed up them and thought that we have only one

Folder is readable by user but user don't have rights for file within
Folder isn't readable so user even can't know if file exist

I experimented with 1 kind so I didn't notice any changes with adding || os.IsPermission(err) but the 2 kind behaves as you described: silent ignore.
I understand now, thank you!


take in include in findMany (and other Prisma Client queries) unnecessarily slow

Problem
We've noticed that queries in the form of:
prisma.modelA.findMany({
  include: {
    modelB: {
      take: 3,
      orderBy: {
        createdAt: 'desc'
      }
    }
  }
})

are slow.
It turns out that Prisma fetches all of modelB and then picks for each of modelA the first 3 entries.
This is a big overhead especially if modelB is big.
Suggested solution
At least in PostgreSQL this can be optimized by using JOIN LATERAL.
I've provided an example repo that uses it and still achieves the same objective: https://github.com/Osman-Sodefa/prisma-join-lateral
Output
$ yarn start
yarn run v1.22.19
$ ts-node run.ts
find many by lateral join: 9.442ms
find many by include: 6.299s
same result? true
Done in 29.30s.

It would be nice if Prisma would use it on its own.

Thanks for the example repo ❤
There are two levels to the solution here:

We can make sure we run a better query that does not read all the data for the nested items
We can further optimize by using a lateral JOIN to avoid the second query overall

We will probably start with 1) before we then move on to 2). The benefit of 1) is that it works for most databases.

Hey @makeeno,
We’re excited to let you know that we’ve just added support for JOINs to resolve relations in Prisma 5.7.0 for PostgreSQL and CockroachDB, under the relationJoins preview feature.
Once you enable that feature, JOINs will be used to resolve one-to-one, one-to-many and many-to-many relations. Most of the queries listed on this documentation page will essentially now use joins. This means that, when possible, we now resolve your query in a single round-trip as opposed to the N round-trips we used to make for each relation.
I say when possible, because we fallback to the old query-based mechanism when you:

Use distinct
Use cursor
Use _count in your selection set

This is the very first step to bring JOINs to Prisma for querying relations. We plan on removing the current limitations and adding support for other databases in future releases.
Changing the way Prisma resolves relations is not a trivial change and is likely not going to work 100% on day one, so please bear with us as we continuously improve the feature, both in terms of bugs and performance improvements on the SQL queries we generate.
If you are curious, we have provided a bunch of Prisma query examples together with their associated SQL query output in our latest release notes
Finally, we have a dedicated preview feature feedback issue here if you have any thoughts about it.
Thank you for your patience!


Glitches out when players are fired

The pages ui disappears, still useable but the suits are out of their previous order and missing some of them.

This mod doesn't have pages by itself, that problem is from another mod.


Rate limits WRT subdomains and multiple hosts

First of all this project is fantastic, thank you!
#274 touches on the questions but not quite clear.
In my use case each of my users lives on their own subdomain
userA.example.com
userB.example.com etc
And I'm running a separate stack for each of them using nginx-proxy to route to them.
No one is routed to the base domain example.com
Does that mean that a different certificate will be issued for each subdomain ? Since LetsEncrypt rate limits on the base domain (example.com) will I be hitting rate limits?
If so is their a way to avoid this?
Second,
If we run this stack on multiple hosts each host will get its own certificate, I assume this will also hit a rate limit. I'm thinking of mounting the volumes from a distributed file system (EFS), is their another solution / best practice ?

The official LE rate limits doc might be clearer than #274
More specifically:

The main limit is Certificates per Registered Domain, (20 per week). A registered domain is, generally speaking, the part of the domain you purchased from your domain name registrar. For instance, in the name www.example.com, the registered domain is example.com. In new.blog.example.co.uk, the registered domain is example.co.uk. We use the Public Suffix List to calculate the registered domain.

If you have a lot of subdomains, you may want to combine them into a single certificate, up to a limit of 100 Names per Certificate. Combined with the above limit, that means you can issue certificates containing up to 2,000 unique subdomains per week. A certificate with multiple names is often called a SAN certificate, or sometimes a UCC certificate.

So 20 or 2000 new users a week depending on wether you use discrete certificates for each user's subdomain or group them into SAN certificates ...
BUT currently the project handles ACME account registration in a simplistic way: it creates a new account for each new base domain, no matter what. If you request 20 discrete certificates, you'll end up with 20 accounts key. And guess what, there is rate limiting on this too.

You can create a maximum of 10 Accounts per IP Address per 3 hours. You can create a maximum of 500 Accounts per IP Range within an IPv6 /48 per 3 hours. Hitting either account rate limit is very rare, and we recommend that large integrators prefer a design using one account for many customers.

So you might hit the 10 accounts per IP Address per 3 hours limit if you generate your 20 allowed certs per Registered Domain per week at once on the same host, or more probably if you use test certificates (because these too register new accounts, whose keys are wiped when you switch to production certificate). I use my own modified version of this project to circumvent this (it attempt to re-use a single account key for all domains and certificates).

I'm thinking of mounting the volumes from a distributed file system (EFS), is their another solution / best practice ?

Take a look at Docker Volume plugins.


Add IP and MAC Address Validator

This adds a very basic IP address validator.  It uses ipaddress from the Python3 stdlib and the backported version for Python2.  I'm unsure if ipaddress is in a default install of Python2.7.x.  It was in mine, but it's also listed on PyPI as a backport, so I added it to dependencies anyway.
Anyway, this will let you validate IPv4 and IPv6 addresses, optionally enforcing the version.  You could extend this a lot, but I don't want to boil the ocean.
I know you can do custom validators, but this seems like something that might be common enough that it is worth having upstream.
I tried to make sure I did all the right things with testing.  Please let me know if I missed something.

This would resolve part of #45, at least at a very basic level, although that issue specifically mentions netaddr whereas this PR uses ipaddress since it's in stdlib for Py3 and backported to Py2.7.

I added functionality for validating MAC addreses as well, mostly piggybacking off of the Regex validator that already exists.  A library like netaddr would definitely do this better, but I was trying to keep from bringing in external dependencies given that there's probably only a subset of people who want these features, and if those people feel strong enough they could always write custom validators that used netaddr.

Cool. Is there a reason you didn't want to use regex for the IP validation too? I looked and it seems available.

@blopker I had a few!  :)
First and foremost, ipaddress is in the Python3 standard library and does this (and much more) already.  Although it isn't in the Python2 standard library, it is backported, and Python2 support ends 1 January 2020 anyway.
I would have also needed to write more code to do the same thing with regex.  IP addresses are also more difficult than MAC addresses when it comes to regex -- for example, each octet can be a number between 1 and 255, which isn't easy to express or read as a regex (to me).  Another option might be more code to check those things.
Finally, although I only supplied a version constraint in this PR, the ipaddress module provides a wealth of other interesting possibilities, such as confirming that an IP is link local or not, enforcing specific prefix lengths, determining if the address is private or public, multicast or unicast, is a teredo address, a 6to4 address, etc.  Lots of possible constraints, all of which I can see use cases for, and some of which I have use cases for myself (just haven't written the code to implement the constraint yet, though I hope to in the future).

Ok.  I'm done writing more commits now.  :)  I'll update if you have feedback, but from my perspective, I'm "done."  I'll likely add more constraints to ip() in the future as I require them (just starting to build out some models that require this) and submit more PRs for that.

Fair enough :)
I think it's fine to use ipaddress, but meet me half way: most people will not be using this validator. Making everyone install the ipaddress backport seems like just wasted resources (and maybe a security risk, not sure who maintains it), especially for us internally. We'd be installing an extra package hundreds of times a day for something we wont be using for a while if ever.
The alternative is to remove ipaddress from setup.py and do a conditional import in the validator itself catching ImportError. The catch would reraise the exception with a message about needing to install ipaddress for Python 2. A similar message would also be in the docs.
Whacha think?

I can remove the dependency from setup.py and add it as a try/except.  However, I think I'd have to suppress the exception and not re-raise.  I haven't tested this specific case, but I would expect that you'll need that package no matter what since the re-raised exception would cause the application to exit.  Either I misunderstood what you meant or there's something else I don't understand (which is pretty common).

The idea would be to only import the package when the user actually needs it. If it's not there there the program can't continue anyway, so it might as well crash with a helpful message.

@blopker I've updated to implement the requested changes.

@blopker thanks!  I've updated to address the most recent issue.  Tests still passed the way it was, but I updated it anyway because you are correct.

Great! Thanks for sticking with this to the end.

Aaaand it's live on PyPi!


Test on Ruby 2.6

Ruby 2.6 has been released! Let's add it to the test matrix.

@patrobinson: Support for 2.1 and 2.2 has ended. Shall I remove builds for both versions?

@orien 👍


Move window close button into shell submenu

IMO actions that affect a session don't belong into the window menu.
That's also how iTerm handles it.

Agreed. Needs rebase

Got merged in the other pr ;)
Good to know that you're squashing.


Inlay hints can cause scrolling to jitter

Have word wrap and inlay hints on
Have a file where the addition of an inlay hint causes a line to wrap
Scroll around slowly using the mousewheel and notice you get jitter.
Scroll just right and you get infinite jitter.

@connor4312 We cannot reproduce this. Can you share more details? What's the exact view port when this happens? Can you do breakpoints here

ping @connor4312

I'm not seeing this today, where previously I saw it happen trivially in any large file. Closing unless it comes up again...


Failed to resolve module specifier "vue-cal"

After install npm i vue-cal
and in my js file:
import VueCal from 'vue-cal'
import 'vue-cal/dist/vuecal.css'
failed to import with this error message:
Failed to resolve module specifier "vue-cal"

If you are using Vue-cal in Vue + Typescript project, then you can't using module. Because author not add type defenitions to source code (*.d.ts files). You have 2 methods:

Easy method:
Add 2 lines to tsconfig.json file in your project folder. This switch off type checking in imported JS modules.
{
"compilerOptions": {
...
"noImplicitAny": false,
"allowJs": true,
...
}
}
Advanced method. Write .d.ts file yourself to complete type checking. Info https://www.typescriptlang.org/docs/handbook/declaration-files/introduction.html

Today I also met with the same problem. Solved using first method. I hope the author will add the necessary files to the project to support Typescript. I'm busy at the moment, but as soon as I'm free I can help the author with Typescript

@snurbol #168
there is .d.ts file for vue2, it might help you


Adding items at run time: introduce threshold on number of items left

The current behavior is:
Not enough items left? Add new items (default value: 2*num_items_per_iter).
A better option is to add new items when the number of items left falls below a threshold:
Number of items left < threshold? Add new items (default value: multiple of the threshold?).

Closing this issue because this is no longer the default behavior - @elucherini please feel free to reopen if I misunderstood!


Construct Stylesheets

What's changed, or what was fixed?

Added Constructable Stylesheets article

Target Live Date: 2019-02-11

[x] This has been reviewed and approved by (@jpmedley)
[x] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[x] I've staged the site and manually verified that my content displays correctly.

CC: @petele

Whoops!
There were 2 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/updates/2019/02/constructable-stylesheets.md#L111 - Common typo found: 'node' Should it be 'Node'?
src/content/en/updates/2019/02/constructable-stylesheets.md#L113 - Common typo found: 'node' Should it be 'Node'?

Warnings are from a code sample.

:tada: This has been pushed live to https://developers.google.com/web/


React Native TextInput the input text is not displayed

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
React Native When I input TextInput, the input text is not displayed. This is an accidental situation. How can I solve this problem?
React Native version:
"react-native": "0.59.5"
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.
1.This is an accidental situation
2.
<TextInput
style={styles.text_input}
placeholder={'请填写注册时的手机号'}
underlineColorAndroid='transparent'
returnKeyType ='done'
onChangeText={(text)=> {
this.setState({
phoneNumber: text
});
}}
/>
3.
text_input: {
fontSize: 14,
height: 40,
color:Colors.FontBlackColor
}
Expected Results
the input text is display
Snack, code example, screenshot, or link to a repository:

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
React Native When I input TextInput, the input text is not displayed. This is an accidental situation. How can I solve this problem?
React Native version:
"react-native": "0.59.5"
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.
1.This is an accidental situation
2.
<TextInput
style={styles.text_input}
placeholder={'请填写注册时的手机号'}
underlineColorAndroid='transparent'
returnKeyType ='done'
onChangeText={(text)=> {
this.setState({
phoneNumber: text
});
}}
/>
3.
text_input: {
fontSize: 14,
height: 40,
color:Colors.FontBlackColor
}
Expected Results
the input text is display
Snack, code example, screenshot, or link to a repository:

value={phoneNumber} as a props for TextInput
E.g
<TextInput
style={styles.text_input}
value={phoneNumber}
placeholder={'请填写注册时的手机号'}
underlineColorAndroid='transparent'
returnKeyType ='done'
onChangeText={(text)=> {
this.setState({
phoneNumber: text
});
}}
/>

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
React Native When I input TextInput, the input text is not displayed. This is an accidental situation. How can I solve this problem?
React Native version:
"react-native": "0.59.5"
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.
1.This is an accidental situation
2.
<TextInput
style={styles.text_input}
placeholder={'请填写注册时的手机号'}
underlineColorAndroid='transparent'
returnKeyType ='done'
onChangeText={(text)=> {
this.setState({
phoneNumber: text
});
}}
/>
3.
text_input: {
fontSize: 14,
height: 40,
color:Colors.FontBlackColor
}
Expected Results
the input text is display
Snack, code example, screenshot, or link to a repository:

value={phoneNumber} as a props for TextInput
E.g
<TextInput
style={styles.text_input}
value={phoneNumber}
placeholder={'请填写注册时的手机号'}
underlineColorAndroid='transparent'
returnKeyType ='done'
onChangeText={(text)=> {
this.setState({
phoneNumber: text
});
}}
/>


Fix vendoring on master

It looks like something snuck in and now make vendor fails on master. This should make things happy.

/hold

/lgtm

LGTM

/hold cancel


Request/response not reported in case where response contains bad JSON

The REST request and response cannot be found in the stdout report or in the log file for the case shown below; the error log contains no error information whatsoever.
It's very hard to solve the issue schemathesis reports without this information.
================================== Schemathesis test session starts =================================
platform Linux -- Python 3.6.9, schemathesis-2.7.1, hypothesis-5.37.4, hypothesis_jsonschema-0.18.0, jsonschema-3.2.0
rootdir: /home/mr_sandwich/PBJ/peanut-butter-jelly/tls/src
hypothesis profile 'default' -> database=DirectoryBasedExampleDatabase('/home/mr_sandwich/PBJ/peanut-butter-jelly/tls/src/.hypothesis/examples')
Schema location: file:///home/mr_sandwich/PBJ/peanut-butter-jelly/tls/src/../../RESTapi/openapi_pbj.yaml
Base URL: http://knife.sandwich.com:42188/pbj_FE
Specification version: Open API 3.0.0
Workers: 1
collected endpoints: 9

GET /pbj_FE/sandwich-server .                                                                 [ 11%]
DELETE /pbj_FE/sandwich-server .                                                              [ 22%]
POST /pbj_FE/sandwich-server .                                                                [ 33%]
POST /pbj_FE/flavor E                                                                         [ 44%]
GET /pbj_FE/flavor/{flavorID} .                                                               [ 55%]
DELETE /pbj_FE/flavor/{flavorID} .                                                            [ 66%]
PATCH /pbj_FE/flavor/{flavorID} .                                                             [ 77%]
PATCH /pbj_FE/flavor/{flavorID}/stats .                                                       [ 88%]
GET /pbj_FE/flavor/{flavorID}/stats .                                                         [100%]

=============================================== ERRORS ==============================================
______________________________________ POST: /pbj_FE/flavor ______________________________________
simplejson.errors.JSONDecodeError: Expecting ',' delimiter or '}': line 1 column 40 (char 39)

Expected behavior:
Both request and response should be reported in the log file for every error.
OS: ubuntu 18.04

I missed this issue. Thank you for reporting it :+1:
We definitely should add it if they are available (there are some cases when they aren't)

The fix is released in 2.7.3. Let me know if it solves the problem for you :)