[qtdeclarative] Build error

Host Environment

OS: Windows
Compiler: VS2022

To Reproduce
Steps to reproduce the behavior:
./vcpkg install qt:x64-windows
Failure logs
Installing 38/41 qtwebengine:x64-windows...
Building qtwebengine[core,default-features,geolocation,spellchecker,webchannel]:x64-windows...
-- Setting up python virtual environmnent...
-- Installing python packages: html5lib
-- Setting up python virtual environmnent...finished.
CMake Warning at ports/qtwebengine/portfile.cmake:85 (message):
Buildtree path 'C:/dev/vcpkg/buildtrees/qtwebengine' is too long.
Consider passing --x-buildtrees-root=<shortpath> to vcpkg!
Trying to use 'C:/dev/vcpkg/buildtrees/qtwebengine/../tmp'
Call Stack (most recent call first):
scripts/ports.cmake:147 (include)
CMake Error at ports/qtwebengine/portfile.cmake:90 (message):
Buildtree path is too long.  Build will fail! Pass
--x-buildtrees-root=<shortpath> to vcpkg
Call Stack (most recent call first):
scripts/ports.cmake:147 (include)
error: building qtwebengine:x64-windows failed with: BUILD_FAILED
error: Please ensure you're using the latest port files with 'git pull' and 'vcpkg update'.
Then check for known issues at:
https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+qtwebengine
You can submit a new issue at:
https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[qtwebengine]+Build+error
Include '[qtwebengine] Build error' in your bug report title, the following version information in your bug description, and attach any relevant failure logs from above.
vcpkg-tool version: 2022-11-10-5fdee72bc1fceca198fb1ab7589837206a8b81ba
vcpkg-scripts version: 30a20e8f1 2022-12-07 (11 days ago).
- Attach ***every failure log file*** mentioned in the console output.
The console output by itself is not enough to investigate a build failure.
Some hint to where the log files are located would help... VCPKG creates a LOT of files!
Additional context
The command was run from the 'x64 Native Tools Command Prompt for VS 2022'. It failed on two (2) separate machines. git pull and vcpkg update had been run previously with zero changes, as it is a brand shiny new install on new laptops.
stdout-x64-windows.log
pip-install-virtualenv-x64-windows-out.log
pip-install-packages-x64-windows-out.log

CMake Error at ports/qtwebengine/portfile.cmake:90 (message):
Buildtree path is too long.  Build will fail! Pass
--x-buildtrees-root=<shortpath> to vcpkg

It exactly tells you what the issue is.

Hi Alexander,
How short must it be? My path is C:\dev\vcpkg, 12 characters if you include the drive specification. To use the command line option, I’m assuming my install command would be…
C:\dev\vcpkg> .\vcpkg install qt:x64-windows --x-buildtrees-root=C:\dev\vcpkg
…or would I need to specify the actual buildtrees folder…
C:\dev\vcpkg> .\vcpkg install qt:x64-windows --x-buildtrees-root=C:\dev\vcpkg\buildtrees
…or just the qtwebengine?
C:\dev\vcpkg> .\vcpkg install qtwebengine:x64-windows --x-buildtrees-root=C:\dev\vcpkg\buildtrees
Your help in this matter is greatly appreciated, as vcpkg is all new to me.
Kind regards,
Micheal
From: Alexander Neumann @.>
Sent: Tuesday, 20 December 2022 6:17 PM
To: microsoft/vcpkg @.>
Cc: Knight, Micheal @.>; Author @.>
Subject: Re: [microsoft/vcpkg] [qtdeclarative] Build error (Issue #28451)
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
CMake Error at ports/qtwebengine/portfile.cmake:90 (message):
Buildtree path is too long.  Build will fail! Pass
--x-buildtrees-root= to vcpkg
It exactly tells you what the issue is.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vcpkg/issues/28451#issuecomment-1358985609, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACEJP7W4SVCMGDQR7QRPAX3WOFTOTANCNFSM6AAAAAATD37STM.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>==================================================================== Attention Email Disclaimer Notice - This message is the property of AAM Pty Ltd. The information in this email is confidential and may be legally privileged. It is intended solely for the addressee. Access to this email by anyone else is unauthorised. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it is prohibited and may be unlawful. If you have received this message in error please notify AAM Pty Ltd immediately via email to @.*** This email has been scanned and cleared by Exchange Online Protection, however AAM Pty Ltd does not guarantee this message free of viruses, or interference.

Hi Alexander,
The first option worked for the qtwebengine; thank you.
However, if I try this with the second option to ensure the qt install is complete, I got (a) a page size error on one laptop (4GB page size on this laptop and 16GB on the other), and (b) both laptops now crash when I run the second command, with 16GB page sizes. No error log is produced… the laptops simply choke, then reboot!
One final question… do I need to maintain the C:\b folder, and if so, could you briefly explain why?
Regards,
Micheal
From: Alexander Neumann @.>
Sent: Tuesday, 20 December 2022 11:49 PM
To: microsoft/vcpkg @.>
Cc: Knight, Micheal @.>; Author @.>
Subject: Re: [microsoft/vcpkg] [qtdeclarative] Build error (Issue #28451)
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
.\vcpkg install qtwebengine:x64-windows --x-buildtrees-root=C:\b\
will be enough. No need to build everything there but
.\vcpkg install qt:x64-windows --x-buildtrees-root=C:\b\
will also work.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vcpkg/issues/28451#issuecomment-1359382705, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACEJP7RMPICEJFYS6CGFIETWOG2MLANCNFSM6AAAAAATD37STM.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>==================================================================== Attention Email Disclaimer Notice - This message is the property of AAM Pty Ltd. The information in this email is confidential and may be legally privileged. It is intended solely for the addressee. Access to this email by anyone else is unauthorised. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it is prohibited and may be unlawful. If you have received this message in error please notify AAM Pty Ltd immediately via email to @.*** This email has been scanned and cleared by Exchange Online Protection, however AAM Pty Ltd does not guarantee this message free of viruses, or interference.

do I need to maintain the C:\b folder, and if so, could you briefly explain why?

No the build folder is only required for building. After that you can delete it. There is also the option --clean-after-build
--clean-after-build             Clean buildtrees, packages and downloads after building each
                                 package
 --clean-buildtrees-after-build  Clean buildtrees after building each package
 --clean-packages-after-build    Clean packages after building each package
 --clean-downloads-after-build   Clean downloads after building each package

I got (a) a page size error on one laptop (4GB page size on this laptop and 16GB on the other), and (b) both laptops now crash when I run the second command, with 16GB page sizes.

I wouldn't try to build qtwebengine on a laptop. From the errors you describe I would assume you are running in a hefty RAM deficit and running a pagefile is not enough in this case.

Hi Alexander,
I have no interest in the qtwebengine and would be happy to build qt without it, but building Qt using .\vcpkg install qt:x64-windows always includes it. How do I omit it from the build, if I don’t know what other packages are required?
Regards,
Micheal
From: Alexander Neumann @.>
Sent: Wednesday, 21 December 2022 6:52 PM
To: microsoft/vcpkg @.>
Cc: Knight, Micheal @.>; Author @.>
Subject: Re: [microsoft/vcpkg] [qtdeclarative] Build error (Issue #28451)
CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
do I need to maintain the C:\b folder, and if so, could you briefly explain why?
No the build folder is only required for building. After that you can delete it. There is also the option --clean-after-build
--clean-after-build             Clean buildtrees, packages and downloads after building each
                             package

--clean-buildtrees-after-build  Clean buildtrees after building each package
--clean-packages-after-build    Clean packages after building each package
--clean-downloads-after-build   Clean downloads after building each package
I got (a) a page size error on one laptop (4GB page size on this laptop and 16GB on the other), and (b) both laptops now crash when I run the second command, with 16GB page sizes.
I wouldn't try to build qtwebengine on a laptop. From the errors you describe I would assume you are running in a hefty RAM deficit and running a pagefile is not enough in this case.
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/vcpkg/issues/28451#issuecomment-1361024865, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACEJP7UOLNMT6P54JAKMAG3WOLAJZANCNFSM6AAAAAATD37STM.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>==================================================================== Attention Email Disclaimer Notice - This message is the property of AAM Pty Ltd. The information in this email is confidential and may be legally privileged. It is intended solely for the addressee. Access to this email by anyone else is unauthorised. If you are not the intended recipient, any disclosure, copying, distribution or any action taken or omitted to be taken in reliance on it is prohibited and may be unlawful. If you have received this message in error please notify AAM Pty Ltd immediately via email to @.*** This email has been scanned and cleared by Exchange Online Protection, however AAM Pty Ltd does not guarantee this message free of viruses, or interference.

How do I omit it from the build, if I don’t know what other packages are required?

You should know which parts of Qt you are using. In general you should be able to infer that from the Qt classes/functionality you are using. Maybe just qtbase[gui,widgets] is already enough if you don't use qml stuff or need qtimageformts as a plugin.

@mknigh23, Did @Neumann-A 's answer help you solve this problem?

We hope your question was answered to your satisfaction; if it wasn't, you can reopen with more info.


Add App Service Editor to the Function App Settings

Description
It's very nice Function App Settings have improved and now I can easily access to must touch settings.
It's nice if Function App Settings add link to the App Service Editor. I always use it to edit shared code outside functions.
Current Looking

Yes, I think we could consider a Monaco button.
I do wonder whether the current button layout takes a bit too much space.

Before we do this, we need to come up with a more attractive way of displaying the buttons as the list grows.
What about adding a button to the dev tab? Maybe in the file explorer there?

@christopheranderson will try to come up with a design. Might back this out of RC if we don't like the design.

Sounds nice. Maybe all users imagine App Service Editor for Dev usage:)

FYI - this is the proposed redesign w/ addition of the App Service Editor:

fixed in here https://github.com/projectkudu/AzureFunctionsPortal/pull/742


x/tools/cmd/gopls: builtin errors in Problems view in VSCode

What version of Go are you using (go version)?

$ go version
go version go1.12.5 darwin/amd64

Does this issue reproduce with the latest release?
Yes (tools commit: 2de7f9bf822ce37065c31659ae7187ea9e416dde)
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
...
GOARCH="amd64"
GOOS="darwin"

Editor and extension versions
VSCode: 1.34.0
vscode-go: 0.10.3-beta.4
VSCode settings for vscode-go and gopls
settings.json
    ...
    "go.autocompleteUnimportedPackages": true,
    "go.useCodeSnippetsOnFunctionSuggestWithoutType": true,
    "go.docsTool": "godoc",
    "go.formatTool": "goimports",
    "go.lintTool": "golangci-lint",
    "go.lintFlags": [
        "--config=/my/config/folder/.golangci.yml"
    ],
    "go.buildOnSave": "off",
    "go.vetOnSave": "off",
    "go.lintOnSave": "package",
    "go.coverageDecorator": {
        "type": "gutter"
    },
    "go.coverageOptions": "showUncoveredCodeOnly",
    "go.coverOnTestPackage": true,
    "go.coverOnSingleTest": false,
    "go.testFlags": [
        "-v",
        "-cover"
    ],
    "go.addTags": {
        "tags": "json",
        "promptForTags": false,
        "transform": "snakecase"
    },
    "go.enableCodeLens": {
        "references": false,
        "runtest": true
    },
    "go.useLanguageServer": true,
    "go.languageServerExperimentalFeatures": {
        "diagnostics": true,
        "documentLink": false,
    },
    "gopls": {
        "usePlaceholders": true,
    },
    "[go]": {
        "editor.defaultFormatter": "ms-vscode.Go",
        "editor.snippetSuggestions": "top",
        "editor.formatOnSave": true,
        "editor.codeActionsOnSave": {
            "source.organizeImports": true,
        },
    },
    "files.eol": "\n",
    ...

What did you do?

Just working in any project for some time (usually 5+ minutes is enough).
What did you expect to see?
Only my project's issues.
What did you see instead?
After some time I start to get builtin issues in Problems view:

This should be fixed with https://golang.org/cl/179921.

Seems to be fixed. Thanks!


615: Fix Go Package Scopes

Related to: #615

Codecov Report

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

@@            Coverage Diff            @@
##             master     #620   +/-   ##
=========================================
  Coverage     83.99%   83.99%
  Complexity      883      883
=========================================
  Files           146      146
  Lines          2281     2281
  Branches        307      307
=========================================
  Hits           1916     1916
  Misses          228      228
  Partials        137      137

Impacted Files
Coverage Δ
Complexity Δ

cli/zally/commands/lint.go
80.28% <ø> (ø)
0 <0> (ø)
:arrow_down:

cli/zally/utils/result_printer.go
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

cli/zally/commands/supported_rules.go
90% <ø> (ø)
0 <0> (ø)
:arrow_down:

cli/zally/app.go
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

:+1:

👍


Server log gridview script

I would like to see a generic import of a server log script which can fetch and ogv server logs :)
Requested:

Ability to gather server logs VIA Alias OR Fully Qualified Domain Name and output them via grid view.

Aliases are supported for Get-AspNetEvents.ps1 , for grid view, just pipe
to ogv.
On Tue, Jul 7, 2015, 16:37 M W<EMAIL_ADDRESS>wrote:

I would like to see a generic import of the server log script which can
fetch and ogv server logs :)
Requested:

Ability to gather server logs VIA Alias OR Fully Qualified Domain
Name and output them via grid view.

—
Reply to this email directly or view it on GitHub
https://github.com/brianary/scripts/issues/1.

Seems reasonable enough to me. I appreciate the follow up :)

No problem, thanks for your interest.
On Wed, Jul 8, 2015, 16:00 M W<EMAIL_ADDRESS>wrote:

Seems reasonable enough to me. I appreciate the follow up :)
—
Reply to this email directly or view it on GitHub
https://github.com/brianary/scripts/issues/1#issuecomment-119754389.


WIP:- INKO-363:- Implement Metropol for Corporate with Id Number

Description
Describe the changes made and why they were made.
Ignore if these details are present on the associated Apache Fineract JIRA ticket.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[ ] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[ ] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

Changes nolonger required. We shall go with initial implementation


Update Files.swift

The shareFilesFromDirectory function already sets the content type and length headers on the response. This just adds those headers in the shareFile function as well.

I forgot to mention that you should update the CHANGELOG with your change. You can find it at CHANGELOG.md.


Cannot pass DestinationSpecifier with quotes from TFS to OSX

We want to launch xcode tests via the xcodebuild task, and struggle with the destinationspecifier parameter.
In our case, the platform name is 'iPhone 5', so the destinationspecifier for our command ought to be:
-destination name="iPhone 5",OS="10.2"
How to pass such double quotes from TFS to an OSX agent? All we tried seemed to fail us (" or "" or even '")
Thanks
PS: I first open the issue on the vsts-agent repo, but was redirected here

I have this in my build definition: -destination "platform=iOS Simulator,name=iPhone 6s" and it works. Please try: -destination "name=iPhone 5,OS=10.2" and see if that helps.
The arguments are split on spaces and the quotes indicate it is one argument and should not be split. I agree this is confusing.
Thanks,
Madhuri

Thanks @madhurig, that's exactly what we needed to do. And indeed it's a bit confusing,


Open Multiple Files

It would boost my productivity if I was able to open more than 2 files in multiple panes. Currently, I can only open 2 panes with 2 files in each.

Closing as there isn't any limit imposed by the extension on how many files you can have open - let me know if I've misunderstood.


Incorrect implementation

So, i tried to figure out the best way to make this package work properly.
However, it looks like i have missed something important on the base unit initialization :
For now, a ScaleValue instance is able to change its base scale with methods starting with "from".

We should be able to create a ScaleValue with correct base scale. The created instance shouldn't be able to change its base scale ! The ScaleValue factory should be the only instance able to set a base scale definitely.
The ScaleValue instance may be able to convert himself to another scale from its immutable base scale.
The ScaleValue might be able to build a new ScaleValue with a different base scale using the factory. I still have doubts about this functionnality.
So, i guess that next release will bring some breaking changes.

I think it's ok now


Expose

Check out
A test oriented developer toolbox for building dApps dApps using Helios and vitest.

discord:
Open Source and Open Standards
IOG tech
CDP,

discord:
Open Source and Open Standards
IOG tech
CDP,

https://developers.cardano.org/docs/portal-contribute


Memory leak!!!

Currently there is a memory leak inside the CRUD component.
Because memory leaks aren't really our thing, we would prefer not to have one.
You can watch Kwinten's talk Keep your memor(ies|y) safe! from NgTalks 2018 to learn how to track memory leaks and how to solve them.
Leave a comment if you would like to take this issue.

I created a PR here:


Rename "Library" menu item for clarity

I'm thinking these are installation items. Right? A library is a bit too ambiguous.

@akarasulu for now it conflicts with #181 Let's remove this one from 5.0 and keep it for future versions

Yes, same as #179 here. No need if we go straight into the wizard.


Access to cumulative processing time and fixed processing time from GDScript

Would it be possible to add functions that return "processing" time and "fixed processing" time, which would return the value you would get (in principle, ignoring precision issues) if you added processing or fixed processing deltas to some accumulator every frame.
This would be useful for instance to keep track of cooldowns without active processing, by simply storing the time the cooldown would end in a variable and then comparing that to the current processing time.
I would guess the engine keeps track of it anyway, so it shouldn't be too hard to simply expose it to GDScript.

It's obvious that a singleton with process() could solve that easily. It's ok if the engine can provide it already, but the pause mode makes it a bit more difficult than it sounds because it implies that the nodes being pausable are always the same ones.

Or fixed frame counter, I can see it useful for syncing.
Then if want to pause, the pause method can register frame number of pause and un-pause if needed.

You could just use OS.get_ticks_msec() and a couple of variables in a "Pause" singleton for that.

@RodeoMcCabe That's more complex than what @Zylann proposed, compare:
# Singleton that increments time
var time = 0
func _ready():
    set_process(true)
func _process(delta):
    time += delta

# Singleton that works with get_ticks_msec
var paused_time = 0
func _ready():
    set_process(true)
func _process(delta):
    if get_tree().is_paused():
        paused_time += delta
func get_time():
    return OS.get_ticks_msec() / 1000.0 - paused_time

Just call get_process_time() and get_process_fixed_time()
On Apr 24, 2017 3:52 PM, "Bojidar Marinov"<EMAIL_ADDRESS>wrote:

@RodeoMcCabe https://github.com/RodeoMcCabe That's more complex than
what @Zylann https://github.com/Zylann proposed, compare:
Singleton that increments timevar time = 0func _ready():
set_process(true)func _process(delta):
time += delta

Singleton that works with get_ticks_msecvar paused_time = 0func _ready():
set_process(true)func _process(delta):
if get_tree().is_paused():
    paused_time += deltafunc get_time():
return OS.get_ticks_msec() / 1000.0 - paused_time

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/6999#issuecomment-296675101,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z28hTpw-pAxWoAn0299NYi6yV4MGFks5rzKkygaJpZM4KlWaC
.

@reduz This PR asks for people to be able to use some function to get cumulative process/fixed_process time, and not delta time. So, just like you add up all _fixed_process deltas so far.
(PS. They are actually called get_process_delta_time  and get_process_fixed_delta_time)

@bojidar-bg Sorry, I was responding to eon-s. Something like this (actually you don't need get_ticks_msec if you're just looking for a frame counter):
# Singleton with pause mode set to process (so this singleton is never paused)
var paused_frame = 0
var unpaused_frame = 0
var total_frames = 0
var run_time = 0

func _ready():
    set_process(true)

func _process(delta):
    total_frames += 1
    if not get_tree().is_paused():
        run_time += delta

func pause():
    get_tree().set_paused(true) # can't remember exactly how to pause the tree
    paused_frame = total_frames

func unpause():
    get_tree().set_paused(false)
    unpaused_frame = total frames

Reopening as reverted by https://github.com/godotengine/godot/pull/51979

Shouldn't this just be closed? @reduz already rejected the feature, and I agree with him - it's very easy to just do this in a script.

Apart of syncing this may be probably useful for some deterministic systems (some core ones), but is a (very old) feature request and will need a proper proposal with use cases now, I guess...

Closing per @aaronfranke's comment.


review comments: general

Hi @OverLordGoldDragon
I haven't gotten a chance to look at this in much detail yet, but here are some initial overall comments regarding the code.
General comments

1.) This toolbox reimplements its own CWT. Is it possible to reuse the one already in PyWavelets?
I know there are some numerical limitations to the CWT as currently implemented in PyWavelets (e.g. https://github.com/PyWavelets/pywt/issues/535, https://github.com/PyWavelets/pywt/issues/531). I don't yet know if these are specifically a problem for this application.
Alternatively, would the SciPy implementation work?

2.) PyWavelets uses the Numpydoc for docstrings as described at:https://docs.scipy.org/doc/numpy-1.15.0/docs/howto_document.html

I wouldn't make refactoring the docstrings a first priority though (The same applies for PEP8 issues)

3.) It would be good to make the argument handling more Pythonic.
For example, rather than specify opts=None or opts={} and then parse it with a _get_opts helper, I would explicitly make the signature of synsq_cwt_fwd

def synsq_cwt_fwd(x, t=None, fs=None, nv=32, type='morlet', difftype='direct', gamma=None)```

so that it is easy for the user to determine the names and defaults of all options.
The same applies to the various other functions such as synsq_squeeze that are called within. It is better to explicitly list the arguments by name than hiding them within an opts dictionary.

For example, rather than specify opts=None or opts={} and then parse it with a _get_opts helper, I would explicitly make the signature of synsq_cwt_fwd

That would be very helpful indeed, and much more idiomatic. Note that passing a bunch of parameters around internally can still be done in a grouped fashion by putting them in a dict kwargs = dict(type=type, difftype=difftype, ...) which is then passed around via **kwargs.

@bluenote10 Agreed, changed my mind since last time; opts's getting ditched.


fix pls

from linepy import *
Traceback (most recent call last):
File "", line 1, in
File "linepy_init_.py", line 1, in
from .client import LineClient
File "linepy\client.py", line 4, in
from .models import LineModels
File "linepy\models.py", line 2, in
from .channel import LineChannel
File "linepy\channel.py", line 2, in
from .client import LineClient
ImportError: cannot import name LineClient

I have fixed this error, please update the lib:
pip install linepy --linepy
Thank you


Revert "Fix opencensus agent to be java6 compatible. (#619)"

This reverts commit d2263490e5353b005657bed1fbd60585559a85f2.
I'm reverting this because it seems unrelated to the issue in https://github.com/census-instrumentation/opencensus-java/issues/616#issue-256872886, which is actually about a different code location (line 52).
These "Undefined reference" warnings come from Animal Sniffer. In theory, Animal Sniffer may have seen the class files before they were processed by Retrolambda. However, the gradle task dependency tree (from dorongold/gradle-task-tree) doesn't support this theory.
In any case, we have test coverage for running on Java 6 since https://github.com/census-instrumentation/opencensus-java/pull/638.

Coverage decreased (-0.006%) to 78.042% when pulling 4a592ed0f92a16aa9533ee46e7fd265fb09f6749 on ubschmidt2:java6 into d118ce4c6bcc5d7f94f94b4e5f1b684b4d037b04 on census-instrumentation:master.

Codecov Report

Merging #743 into master will decrease coverage by 0.06%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##             master    #743      +/-   ##
===========================================
- Coverage     76.16%   76.1%   -0.07%
  Complexity      783     783
===========================================
  Files           144     144
  Lines          3617    3616       -1
  Branches        343     344       +1
===========================================
- Hits           2755    2752       -3
  Misses          794     794
- Partials         68      70       +2

Impacted Files
Coverage Δ
Complexity Δ

...in/java/io/opencensus/contrib/agent/Resources.java
50% <100%> (-7.15%)
3 <0> (ø)

...census/implcore/trace/export/SpanExporterImpl.java
90% <0%> (-1.67%)
6% <0%> (ø)

Continue to review full report at Codecov.

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

Sounds reasonable

Try with resources does not work on java6, am I missing something?

Retrolambda rewrites bytecode of try/resources to normal try finally
On 31 Oct 2017 08:18, "Bogdan Drutu"<EMAIL_ADDRESS>wrote:

Try with resources does not work on java6, am I missing something?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/census-instrumentation/opencensus-java/pull/743#issuecomment-340622885,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD61zcynmF0NDt14dFGmk9xectZGbI1ks5sxmdqgaJpZM4QLads
.

Yes, though source/target is 1.7, retrolambda rewrites the classes to 1.6. Try-with-resources works fine (https://github.com/orfjackal/retrolambda#backported-language-features).

PTAL.

I think we should keep the code as 1.6 (without fancy retrolambda). My 2 cents.

Bogdan that's actually a good point. I'd assumed to get more benefit from not restricting the code to 1.6, but only using try-with-resource in this class is not convincing. Thus, happy to just stick with 1.6. Will send a separate PR for that.

Obsoleted by https://github.com/census-instrumentation/opencensus-java/pull/799.


Replace Pplint:variable_scope

We should have a look at symbol table: https://github.com/SonarSource/sonar-java/blob/master/java-squid/src/main/java/org/sonar/java/ast/visitors/SonarSymbolTableVisitor.java. The basis to provide nice check like this one and more.


more usage of getSupportedPlatformInfoFunctions

Description
re #1085

rewrites the following components to use getSupportedPlatformInfoFunctions:

getTotalDiskCapacity*
getLastUpdateTime*
isEmulator*
getCodename*
getPreviewSdkInt*
getType*

Compatibility

OS
Implemented

iOS
✅

Android
✅

Windows
✅

Checklist

[X] I have tested this on a device/simulator for each compatible OS

tested on iOS and Android, but unable to test on Windows

[ ] I added the documentation in README.md
[ ] I mentioned this change in CHANGELOG.md
[ ] I updated the typings files (privateTypes.ts, types.ts)
[ ] I added a sample use of the API (example/App.js)

@mikehardy,
Thoughts on adding the hacktoberfest topic to this repo during October? I'd love to just work on this project and not have to work on other projects that I'm not too interested in to get a tree planted this year. (Purely a selfish request, so totally fine with a "na, I'm good")

awesome! thanks!


Adding the name to the list

Hi There! I am trying to add my name in the list and as such am creating a pull request! If I have done something wrong, feel free to guide me!

I think added another commit. please check it out

Hi Krish , I have added another commit with the required changes!
On Mon, Jan 31, 2022, 1:50 PM Krish Gupta @.***> wrote:

@.**** requested changes on this pull request.
In README.md
https://github.com/EddieHubCommunity/hacktoberfest-practice/pull/867#discussion_r795427022
:

@@ -645,6 +645,7 @@ A GitHub conflict is when people make changes to the same area or line in a file

Sahil Jain
Sahil Khandelwal
Sahil Vaidya

Sahil Maheshwari

I think your name should be one name above
—
Reply to this email directly, view it on GitHub
https://github.com/EddieHubCommunity/hacktoberfest-practice/pull/867#pullrequestreview-867500874,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APMQGSATTG43RJRE3RHAZZTUYZA5RANCNFSM5NFJYWWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***
.com>


How can I use deprecated (v1) features with go module (github.com/plutov/paypal/v3)?

I would like to use /v1/payments/payment in order to use the deprecated express checkout feature from PayPal (Except if I am missing something, there are no ways with v2 to create a fully custom button on the front-end, since they are really trying to force their slow/not customizable importable checkout button on every website now)
I understand I can use go get<EMAIL_ADDRESS>to use the old features but then I can't use go modules. Would it be possible to add go.mod support to this version too?
Thanks!

Hey @adriendomoison sorry I missed this!
you should be able to use any API of paypal's which the module doesnt have support for by using the method in the missing-endpoints section of the readme. Please let us know if this doesnt work for you


Model path does not exist: path/to/llama-2/llama-model.gguf

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[x] I am running the latest code. Development is very rapid so there are no tagged versions as of now.
[x] I carefully followed the README.md.
[x] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).
[x] I reviewed the Discussions, and have a new bug or useful enhancement to share.

Expected Behavior
Please provide a detailed written description of what you were trying to do, and what you expected llama-cpp-python to do.
Current Behavior
Please provide a detailed written description of what llama-cpp-python did, instead.
Environment and Context
Please provide detailed information about your computer setup. This is important in case the issue is not reproducible except for under certain specific conditions.

Physical (or virtual) hardware you are using, e.g. for Linux:

$ lscpu

Operating System, e.g. for Linux:

$ uname -a

SDK version, e.g. for Linux:

$ python3 --version
$ make --version
$ g++ --version

Failure Information (for bugs)
Please help provide information about the failure if this is a bug. If it is not a bug, please remove the rest of this template.
Steps to Reproduce
Please provide detailed steps for reproducing the issue. We are not sitting in front of your screen, so the more detail the better.

step 1
step 2
step 3
etc.

Note: Many issues seem to be regarding functional or performance issues / differences with llama.cpp. In these cases we need to confirm that you're comparing against the version of llama.cpp that was built with your python package, and which parameters you're passing to the context.
Try the following:

git clone https://github.com/abetlen/llama-cpp-python
cd llama-cpp-python
rm -rf _skbuild/ # delete any old builds
python -m pip install .
cd ./vendor/llama.cpp
Follow llama.cpp's instructions to cmake llama.cpp
Run llama.cpp's ./main with the same arguments you previously passed to llama-cpp-python and see if you can reproduce the issue. If you can, log an issue with llama.cpp

Failure Logs
Please include any relevant log snippets or files. If it works under one configuration but not under another, please provide logs for both configurations and their corresponding outputs so it is easy to see where behavior changes.
Also, please try to avoid using screenshots if at all possible. Instead, copy/paste the console output and use Github's markdown to cleanly format your logs for easy readability.
Example environment info:
llama-cpp-python$ git log | head -1
commit 47b0aa6e957b93dbe2c29d53af16fbae2dd628f2

llama-cpp-python$ python3 --version
Python 3.10.10

llama-cpp-python$ pip list | egrep "uvicorn|fastapi|sse-starlette|numpy"
fastapi                  0.95.0
numpy                    1.24.3
sse-starlette            1.3.3
uvicorn                  0.21.1

llama-cpp-python/vendor/llama.cpp$ git log | head -3
commit 66874d4fbcc7866377246efbcee938e8cc9c7d76
Author: Kerfuffle<EMAIL_ADDRESS>Date:   Thu May 25 20:18:01 2023 -0600

I have the same issue on a linux system with a jupyterLab Notebook.
Python 3.11.9
llama_cpp_python 0.2.87
Even the simple example:
model = Llama(model_path="/model/name.gguf")
does not not work with "Value Error: Model path does not exist"

Llama.__init__ just checks the existence of models by os.path.exists method.
How does
os.path.exists("path/to/model.gguf")

turn out to be?

Out of reasons, i do not know, my system think the script in in the model folder.
import pathlib
pathlib.Path().resolve()
Show your path. Maybe that will help vinodhanvino as well.
After i deletet the model part of the path string it is workling.

I was facing similar issue but later realized that the GGUF model was corrupt and had to be re-installed. Now it works. (Install the GGUF model using hugging face cli / through the browser only)


feat: add default config handling (#6)

This pull request implements the default config setup via default-tags and prototype values which was one of the main purpose for providing this extension of Viper.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

113 of 113 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
319

Relevant Lines:
319

💛  - Coveralls


can't reproduce 71% mAP

I cloned your repo and follow the readme.md to train (python train.py --gpus 0,1,2,3 --epoch 0
), but the final max mAP I can get is about 52%, did I missing something? thank you very much if you have any tricks during training!

You will need a pretrained model using imagenet

Hi zhreshold, I also got 52% mAP. I have downloaded the darknet19 pretrained model from the url in the readme. I think this pretrained model path is in the default arguments, right?

I have found the where is the problem, in the original code, the lr_factor is set to 1.0, just use 0.1 can get a good result, sorry for so late to report the solution

@tcye Thank you! Have your reproduced the 71% mAP with lr_factor=0.1 ?

@zhreshold , thanks for your nice work. Is there anything which is different fron the darknet source code. Why the mAP in paper is 78.4% while we get 71.0+%.

I implement YOLO3 with tensorflow can only get 82% mAP without data enhancement and multi-scale training.

@Stinky-Tofu ， Can you share your code?

@Stinky-Tofu ， Can you share your code?

https://github.com/Stinky-Tofu/YOLO_V3


Publish release <IP_ADDRESS> in Hackage

Could you publish release <IP_ADDRESS> in Hackage? :-)
Trying to compile async-extras <IP_ADDRESS> with GHC 7.10.1 gives me the next error, but release <IP_ADDRESS> compiles solves it:
src/Control/Concurrent/Async/Lifted/Extra.hs:83:15:
    Couldn't match type ‘StM m a0’ with ‘StM m a’
    NB: ‘StM’ is a type function, and may not be injective
    The type variable ‘a0’ is ambiguous
    Expected type: Async (StM m a) -> m b -> m (Async (StM m b))
      Actual type: Async (StM m a0) -> m b -> m (Async (StM m b))
    In the ambiguity check for the type signature for ‘withParent’:
      withParent :: forall (m :: * -> *) a b.
                    MonadBaseControl IO m =>
                    Async (StM m a) -> m b -> m (Async (StM m b))
    To defer the ambiguity check to use sites, enable AllowAmbiguousTypes
    In the type signature for ‘withParent’:
      withParent :: MonadBaseControl IO m =>
                    Async (StM m a) -> m b -> m (Async (StM m b))
cabal: Error: some packages failed to install:
async-extras-<IP_ADDRESS> failed during the building phase. The exception was:
ExitFailure 1

+1 I just ran into the error as well.


From private repo

@v-alje : Thanks for your contribution! The author, @linda33wj, has been notified to review your proposed change.


Is it possible to create read coverage density plot

I noticed that this tool could create plot for mutation and junctions. But I want to create plot for read coverage density (Coverage track) plot, can I use this tool?
Thanks.

No, that's not the purpose of this tool.


[BUG] Volume subpath cannot be cleaned up after the node of the workload pod is down and recovered

Describe the bug
After the node back, the old pod should be cleaned up without endless error logs
To Reproduce
Steps to reproduce the behavior:

set up 1 master + 1 worker k3s cluster
Launch a 1-replica deployment on the worker node. The containers of the pod use subpath.
Kill the k3s agent on the worker node and wait for the pod being terminating.
Re-run the k3s agent cmd. The error log will be printed in the worker side:

The deployment YAML file I am using:
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: mysql-pvc
  namespace: default
spec:
  accessModes:
    - ReadWriteOnce
  storageClassName: longhorn
  resources:
    requests:
      storage: 5Gi
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: mysql
  labels:
    app: mysql
  namespace: default
spec:
  selector:
    matchLabels:
      app: mysql
  strategy:
    type: Recreate
  template:
    metadata:
      labels:
        app: mysql
    spec:
      nodeSelector:
        kubernetes.io/hostname: shuo-k3s-01  # worker node
      restartPolicy: Always
      containers:
      - name: mysql
        image: mysql
        command:
        - sh
        - -c
        - "while true; do dd if=/dev/urandom of=/var/lib/mysql/t bs=1M count=1500 oflag=dsync && rm /var/lib/mysql/t; done"
        env:
        - name: MYSQL_ROOT_PASSWORD
          value: "rootpasswd"
        volumeMounts:
        - mountPath: /var/lib/mysql
          name: site-data
          subPath: mysql
      - name: php
        image: php:7.0-apache
        command:
        - sh
        - -c
        - "sleep infinity"
        volumeMounts:
        - mountPath: /var/www/html
          name: site-data
      volumes:
      - name: site-data
        persistentVolumeClaim:
          claimName: mysql-pvc

Expected behavior
No endless log like orphaned pod \"a5224f39-1db5-485c-84a9-b4826883e1f5\" found, but failed to rmdir() subpath at path /var/lib/kubelet/pods/a5224f39-1db5-485c-84a9-b4826883e1f5/volume-subpaths/pvc-78c08abc-6814-4110-9ee3-219cf4ba123b/mysql/0: device or resource busy in k3s/kubelet.
Log
E0928 12:45:41.088557   22535 kubelet_volumes.go:225] "There were many similar errors. Turn up verbosity to see them." err="orphaned pod \"a5224f39-1db5-485c-84a9-b4826883e1f5\" found, but failed to rmdir() subpath at path /var/lib/kubelet/pods/a5224f39-1db5-485c-84a9-b4826883e1f5/volume-subpaths/pvc-78c08abc-6814-4110-9ee3-219cf4ba123b/mysql/0: device or resource busy" numErrs=1
I0928 12:45:43.088096   22535 kubelet_getters.go:300] "Path does not exist" path="/var/lib/kubelet/pods/a5224f39-1db5-485c-84a9-b4826883e1f5/volumes"
I0928 12:45:43.088127   22535 kubelet_getters.go:300] "Path does not exist" path="/var/lib/kubelet/pods/a5224f39-1db5-485c-84a9-b4826883e1f5/volumes"
I0928 12:45:43.088230   22535 kubelet_volumes.go:140] "Cleaned up orphaned pod volumes dir" podUID=a5224f39-1db5-485c-84a9-b4826883e1f5 path="/var/lib/kubelet/pods/a5224f39-1db5-485c-84a9-b4826883e1f5/volumes"

Environment:

Longhorn version: v1.2.1-rc2

Installation method (e.g. Rancher Catalog App/Helm/Kubectl): kubectl

Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: v1.21.4+k3s1

Number of management node in the cluster: 1
Number of worker node in the cluster: 1

Number of Longhorn volumes in the cluster: 1

Sometimes the error logs mentioned above won't be triggered. But the mount path in the pod is unavailable after the node back and the pod restart. e.g.,
$ k exec -it mysql-8495d4d479-gdbnv -- bash
Defaulted container "mysql" out of: mysql, php
root@mysql-8495d4d479-gdbnv:/# ls /var/lib/mysql/
ls: reading directory '/var/lib/mysql/': Input/output error

Workaround: Scale down then re-scale up the workload.

The NodeUnpublishVolume is not called after kubelet restart sometimes, so the volume is not unmounted.
The race condition is resolved in this commit since kubernetes 1.22.0.
https://github.com/kubernetes/kubernetes/commit/3eadd1a9ead7a009a9abfbd603a5efd0560473cc

The commit is not back ported from kubernetes 1.22 to 1.21 or before.
I'll document it on longhorn.io knowledge base.

@khushboo-rancher I know we cover test cases for node down, but do we have this included?

@innobead You're right, we need to explicitly call this case out of kubelet/k3s agent restart.
We have automation around the case where we crash instance manager and remount the pod with subpath https://github.com/longhorn/longhorn-tests/blob/master/manager/integration/tests/test_ha.py#L2047 and this is running fine on k8s and k3s env.
We'll add the test case in manual test case during validation of the issue.

Testing in progress.
Trying to reproduce the issue with k3s "v1.21.7+k3s1" + longhorn master, but not really able to see the race condition.
Rebooting the targeting-node a couple more times still seems holding. Will log more notes here if anything.

TL;DR: This is a Kubernetes problem on cleaning up orphaned pod data under volume path. User can prevent this happen with Kuberenetes v1.22 but not able to resolved existed error automatically, even after upgrading the cluster.

I can now consistently reproduce this on k3s v1.20.14+k3s1 + Longhorn v1.2.2.
Dec 28 15:03:20 c1-worker1 k3s[56124]: I1228 15:03:20.783593   56124 kubelet_getters.go:300] "Path does not exist" path="/var/lib/kubelet/pods/8789bb85-2b5c-4d5f-a0bc-709b467b6626/volumes"
Dec 28 15:03:20 c1-worker1 k3s[56124]: I1228 15:03:20.784091   56124 kubelet_getters.go:300] "Path does not exist" path="/var/lib/kubelet/pods/8789bb85-2b5c-4d5f-a0bc-709b467b6626/volumes"
Dec 28 15:03:20 c1-worker1 k3s[56124]: I1228 15:03:20.784339   56124 kubelet_volumes.go:160] "Cleaned up orphaned pod volumes dir" podUID=8789bb85-2b5c-4d5f-a0bc-709b467b6626 path="/var/lib/kubelet/pods/8789bb85-2b5c-4d5f-a0bc-709b467b6626/volumes"
Dec 28 15:03:20 c1-worker1 k3s[56124]: E1228 15:03:20.784460   56124 kubelet_volumes.go:245] "There were many similar errors. Turn up verbosity to see them." err="orphaned pod \"8789bb85-2b5c-4d5f-a0bc-709b467b6626\" found, but failed to rmdir() subpath at path /var/lib/kubelet/pods/8789bb85-2b5c-4d5f-a0bc-709b467b6626/volume-subpaths/pvc-3b69e9f9-868f-4fd4-a93a-8826569f16c4/mysql/0: device or resource busy" numErrs=1
Dec 28 15:03:21 c1-worker1 k3s[56124]: I1228 15:03:21.047035   56124 reconciler.go:224] "operationExecutor.VerifyControllerAttachedVolume started for volume \"pvc-3b69e9f9-868f-4fd4-a93a-8826569f16c4\" (UniqueName: \"kubernetes.io/csi/driver.longhorn.io^pvc-3b69e9f9-868f-4fd4-a93a-8826569f16c4\") pod \"mysql-5cb7b8fdbd-v6fdv\" (UID: \"bad6e339-5061-4ce0-a62c-7218ecab090f\") "
Dec 28 15:03:21 c1-worker1 k3s[56124]: E1228 15:03:21.047709   56124 nestedpendingoperations.go:335] Operation for "{volumeName:kubernetes.io/csi/driver.longhorn.io^pvc-3b69e9f9-868f-4fd4-a93a-8826569f16c4 podName: nodeName:}" failed. No retries permitted until 2021-12-28 15:03:29.04767865 +0800 CST m=+561.393408566 (durationBeforeRetry 8s). Error: Volume has not been added to the list of VolumesInUse in the node's volume status for volume "pvc-3b69e9f9-868f-4fd4-a93a-8826569f16c4" (UniqueName: "kubernetes.io/csi/driver.longhorn.io^pvc-3b69e9f9-868f-4fd4-a93a-8826569f16c4") pod "mysql-5cb7b8fdbd-v6fdv" (UID: "bad6e339-5061-4ce0-a62c-7218ecab090f")

However, the volume and pods are appear to be healthy, even after adjust the deployment to 3 replicas:
NAME                     READY   STATUS    RESTARTS   AGE
mysql-5cb7b8fdbd-cnnmz   2/2     Running   0          61s
mysql-5cb7b8fdbd-gwpsc   2/2     Running   0          28m
mysql-5cb7b8fdbd-zrpd6   2/2     Running   0          61s
NAME                                       STATE      ROBUSTNESS   SCHEDULED   SIZE         NODE         AGE
pvc-3b69e9f9-868f-4fd4-a93a-8826569f16c4   attached   healthy      True<PHONE_NUMBER>   c1-worker1   37m

Adjusted steps to reproduce the problem:

Create a cluster with kubernetes v1.21 or before
Deploy the example workload, make sure all pods are on the same worker node

label the first worker node
nodeSelector with the label

Stop then start the kubelet agent again with sudo service k3s-agent restart
Watch the k3s-agent log, orphaned pod error message will present

And the workaround dose not needed in Kubernetes v1.22 to avoid the problem.

Moving this back to analysis, since the upstream fix doesn't really fix in some condition (with 5 replicas I can still reproduce this issue fairly easy)

The subpath deletion failure is related to the cleanup mechanism in kubelet.
kubernetes 1.21.6

Pod ADD events handling

kubelet on the restarted worker node received a series of pod ADD events from the server node.
The events are handled by HandlePodAdditions. If a pod is at terminating state and its DeletionTimestamp is set, the containers of the pod is stopped.
In this step, the volumes of the terminated pod are not unmounted.

SyncLoop (housekeeping)

Add the pod the pending termination map

Pods killing goroutine received the the notification from step 2

pk.killPod fails to delete the pod and remove the pod from the  pending termination map.

SyncLoop DELETE

Pod is fully terminated and removed from etcd by kl.statusManager.TerminatePod(pod)

SyncLoop REMOVE

kl.deletePod tries to delete the pod again
The pod is deleted from the pod manager by kl.probeManager.RemovePod

SyncLoop (housekeeping)

HandlePodCleanups cleans up orphaned pod volumes dir
The mounted subpath cannot be deleted by rmdir()

kubernetes 1.23.3
The handling logic in the kubelet should avoid the subpath deletion failure.
kubelet on the restarted worker node received a series of pod ADD events from the server node.
The event is handled by the HandlePodAdditions. The terminating pod is terminated by syncTerminatingPodFn that stops the running containers in the pod and syncTerminatedPodFn that waits for complete unmount of the all volumes by volumeManager.reconcile.
Before unmounting completely, the SyncLoop (housekeeping) does not clean the volumes up, because the terminating pod is still in the pod manager.
Workaround
User can unumount the subpath by manual. Then, the subpath can be deleted by the kubelet successfully.


Release workflow

We need to dev using the pro version of Kendo, but upon release the core version should be the default bundle
This could be a release build step, but basically we need to ensure config.js and package.json are updated to reflect Core rather than Pro libs.
What's the best approach here? Any suggestions?

can't we just keep kendo-core in the config.js of the plugin, and override it in the sample app?

:headdesk: yes we can...


[AF-744] Fix server command with custom path

cc @zendesk/vegemite
Fix server command with custom path.
References

https://zendesk.atlassian.net/browse/AF-744

Risks

[Low] zat server may not work correctly

Any ETA on deployment? v2.0.3 of the gem is still out there.

@bryan-flynn-zd pushed just now: https://rubygems.org/gems/zendesk_apps_tools/versions/2.0.4


Question: Interpretation of outputs in crop_segmentation.ipynb notebook

I cannot understand the purpose of the crop_segmentation.ipynb notebook. I can understand it shows some differences of the lands, as is visible in the last cell. However, I do not understand how the heat map in the last cell output can be interpreted... 🤔  What does the yellow colour mean? And what does the blue colour mean? Can you help me with that, please?
Thank you!
EDIT
I am sorry, I am not able to add any further info to the issue (assignees, label, project, ...).

Hi, @jardabezdek. Thank you for raising this question.
In that example notebook, our goal is to train a crop land segmentation model. Given a set of Sentinel-2 images taken throughout a year, we want to estimate where the regions with crops are.
The maps that you see in the last cell show the prediction of the network (top image) and the ground-truth crop land map (bottom image). These are probability heatmaps, i.e., high values (yellow color) mean that region/pixel was assigned as crop land by the model, whereas low values (blue color) represent regions without crop areas.
We will improve on the description and labels of both maps for the next release.

This was implemented in #14, specifically in https://github.com/microsoft/farmvibes-ai/blob/main/notebooks/crop_segmentation/04_inference.ipynb. I'm closing this issue.
@jardabezdek, please feel free to reopen if you think this could be further improved.


Upgrade K8s to v1.21

Update versions
Changes for https://github.com/kubernetes/kubeadm/issues/2525
Remove deprecated admission plugin

/azp run

/azp run

/azp run

/azp run

/azp run

/azp run

@atsikham Do we need any preflight checks for the repository machines disk space when we are pulling in so many new images? I remember @to-bar was doing somethings for that before. Maybe it needs to be adjusted?

machines disk space when we are pu

I don't see anything related to K8s/repository in preflight checks, but it could be investigated in adiitional task. In fact, would be nice to have such check independently on the number images we have, according to the currently consumed disk space on registry machine.
@przemyslavic could you check amount of used/free disk space after pipelines execution?

@atsikham Do we need any preflight checks for the repository machines disk space when we are pulling in so many new images? I remember @to-bar was doing somethings for that before. Maybe it needs to be adjusted?

I only added removing old image files (*.tar) from epirepo that no longer exist in requirements.txt to free up disk space. However old images are not removed from image registry so we may still have "snowball effect" and something to do with this. In the future we can add size attribute for each image and count how much space is needed to download them and load into registry.

/azp run

Upgrade of HA cluster does not work :cry: It's been hanging for hours on TASK [upgrade : k8s/utils | Drain master or node in preparation for maintenance].

As for disk usage, it is as follows:

apply
Control Plane: 30/32 GB by default

[ubuntu@ec2-1-1-1-1 ~]$ df -h /
Filesystem      Size  Used Avail Use% Mounted on
/dev/nvme0n1p1   30G  4.5G   25G  16% /

Repository: 64 GB by default
df -h[ubuntu@ec2-2-2-2-2 ~]$ df -h /
Filesystem      Size  Used Avail Use% Mounted on
/dev/xvda1       62G   17G   46G  28% /

Single machine installation: 30/32 GB by default
[ubuntu@ec2-1-1-1-1 ~]$ df -h /
Filesystem      Size  Used Avail Use% Mounted on
/dev/nvme0n1p1   30G   19G   11G  64% /

upgrade
Control Plane:

df -[ubuntu@ec2-1-1-1-1 ~]$ df -h /
Filesystem      Size  Used Avail Use% Mounted on
/dev/nvme0n1p1   30G  6.9G   23G  24% /

Repository:
[ubuntu@ec2-2-2-2-2 ~]$ df -h /
Filesystem      Size  Used Avail Use% Mounted on
/dev/xvda1       62G   23G   40G  36% /

Single machine:
[ubuntu@ec2-1-1-1-1 ~]$ df -h /
Filesystem      Size  Used Avail Use% Mounted on
/dev/nvme0n1p1   30G   25G  4.9G  84% /

It doesn't look bad. Only in the case of a single machine installation with a default 30 GB disk, the used space is around 80%.

:heart:


Iman / test configurations added

Changes:

...

Type of change

[ ] Bug fix
[x] New feature
[ ] Update feature
[ ] Refactor code
[ ] Translation to code
[ ] Translation to crowdin
[ ] Script configuration
[ ] Improve performance
[ ] Style only
[ ] Dependency update
[ ] Documentation update
[ ] Release

please resolve the conflicts @iman-fs

@ali-hosseini-fs done


Updated Az commands to support V2.

Description
Updated Az commands to support DataFactory V2 which resolves https://github.com/MicrosoftDocs/azure-docs/issues/37039
Checklist

[x] This pull request was tested on both of:

[x] PowerShell 5.1 (Windows)
[x] PowerShell 6.x (Latest PowerShell)

[x] Scripts do not contain static passwords or other secret tokens.
[x] All Azure resource identifiers which must be universally unique are guaranteed to be so.

Testing information

Platform:
Powershell version: $PSVersionTable.PSVersion
Az version: (Get-InstalledModule -Name Az).Version

@KranthiPakala-MSFT Please confirm a test on PowerShell 6.x and then I will start review. Thanks!

@sptramer I have completed testing PS commands in latest Powershell 6.x and everything is working as expected. Could you please review and approve the changes.

@sptramer The PR itself only updates a few commands. The change you recommended is a bit more intensive. Can we push this incremental change to unblock a few users?

@djpmsft The changes are actually not extensive. I'm pretty sure it's easy to change a write to the absolute path C:\ to the relative path ./. That would be the recommended fix and it is simple enough that it should be made before merge.

@KranthiPakala-MSFT Please change the references C:\ to ./ as requested. This solves two issues:

Makes the script work on all platforms
Avoids writing files in the root of the C: drive on Windows, which will fail unless you are running elevated.

@sdwheeler Thanks for bumping this up.  This is impacting multiple docs where this powershell script is referenced. And the docs have 'C:' everywhere, so I will escalate this to the doc author internally, but will close this PR for now.
@djpmsft for awareness.


Negative numbers

I think you should add the ability to handle negative numbers !

Are negative number common thing in sentences or number words? I don't think so. Open to suggestions though.
Thanks for the suggestion 👍


Long time no see, I wonder what happened? Why rewrite the project?

Question

Hi @hossainemruz
Long time no see, I wonder what happened? Why rewrite the project?

Hi @Chow-Ray, the existing site was getting heavy and harder to maintain as code was more or less unstructured. This was the reason for  the re-write.

Refactoring has greatly improved the performance. Also, current dark mode implementation is terrible. Want to fix that.

Unfortunately, this (#748) happened during the rewrite. Things are good now. Hoping to finish rewriting soon.

I am active again. 😃


Using mongoskin to upload images files to store in GridFS

I am suing mongoskin very much in my application. Recently we decided to use GridFs to store files. I tried to find out the example code using mongoskin however couldn't find any good one. Can anyone please help me on this.

I use gridfs-stream for this task, works.


Reporting an issue in Table3.py

I have made a modification to line 63 of the Table3.py file to address an error related to a missing argument. Previously, the code was as follows:
y = F.Add(INV(inter_value, F), INV(key, F))
I have updated it to:
y = F.Add(INV(inter_value, n, F), INV(key, n, F))
Using this, I am able to generate the accurate Table3 values.

Thank you for catching this, I have fixed this error.

Thank you for catching this, the error is now fixed.


Replace peekalink with OpenGraph Ninja API

https://opengraph.ninja/pricing

Relevant Peekalink code here: https://github.com/jsonhero-io/jsonhero-web/blob/design/better-404/app/services/uriPreview.server.ts#L16

/bounty $500

/attempt #10


Change email in pumprock.net

Hi,
I have an account in pumprock.net but I don't know what email I had put there.
How can I see what email I used?
And how can I change the email?
Thanks,

You can't change the e-mail address at this time, but since you weren't asked for on when creating the account, there's really nothing to change. The web interface doesn't provide a way to either add or change an e-mail address, yet.

Ok, update!
Now you can set or change your e-mail address using the development version of the Dianara client. The only problem is that you'll need to build it yourself from source, unless you wait for the v1.3.1 release =)

@phls Could you change your email with Dianara? If yes, please close this issue.
Thanks for your support!


Issue

If you set the max amount of cells to Infinity in the gameserver.js and .ini, when you hit a virus, it crashes.
In normal MultiOgar this doesn't happen.
Any fix?

If you set the max amount of cells to Infinity

Well there's your problem.

@MastaCoder

In normal MultiOgar this doesn't happen.

xD

that is a problem though...

@freaky767 When you hit a virus it splits you into into the amount of cells defined from the playerMaxCells config. If you set the value of the config to Infinity the server will crash because your cell cannot split into "Infinity" pieces.


Bug report

What happened?
DV code blocks using file.cday, file.mday and this.file.day in Daily Notes not returning data (was working in v0.43)
Confirmed DV date format matches Daily Note date format in DV Options.
Confirmed file.cday and file.mday and this.file.day all working as inline DV queries.
But following code returns no data
DQL
TABLE file.cday as "Created"
WHERE file.cday = this.file.day
TABLE file.mday as "Modified"
WHERE file.mday = this.file.day
JS
No response
Dataview Version
0.4.45
Obsidian Version
0.16.2
OS
MacOS

Have you copied your vault or deleted/created files, or done file syncing? file.ctime and file.mtime are based on the file system created/modified date, which may not at all match the intuitive concept of when a file was "created" or "last modified."

I have done all of those (except copying vault) and did consider that  - but I am pretty certain that is not the issue.
My reasoning for this is that the code worked perfectly in 0.43. They returned the data I expected.  I upgraded to 0.45 and the issue started (no data for those queries) I checked the mday and cday of the files I expected DV to return and they were as expected but still no data.
I then rolled back to 0.43 and the codes started working again as expected.  I am now working on 0.43 again as those code blocks in my Daily Note are important to me and I couldn't figure out another way of doing what they do.
I haven't done any plugin conflict checking - but I have not added or upgraded any plugins since upgrading to v0.45 so I reason that is not a possible cause.  The only thing that changed was DV 0.45 and the problem went away when reverting to 0.43.
A discussion on Discord #dataview indicated that others have had similar issues and reverting back to 0.43 solved them.  One suggestion was that it may be related to this.file.day but I am not a coder so wouldn't know.

@blacksmithgu I think there's something happening with dates in general.
One example:
- [x] task 1 [due:: 2022-09-09]
- [ ] task 2 [due:: 2022-09-09]
- [ ] task 3 [due:: 2022-09-10]

This is a sentence with an inline field with date (fDate:: 2022-09-10)

Query
TASK
FROM "folder"
WHERE due = date(2022-09-09)

or
LIST
WHERE fDate = date(2022-09-10)

There's no results in each query.

Just an extra thought FWIW...I am using periodic notes and thus have disabled Daily Notes plugin.  Not sure if this would have any impact but thought I would mention it in passing.

This is likely due to the timezone handling added in #1405 to fix #1404. Fixing that broke this apparently. I can't quite think of a clean way to bridge the behavior but IMO, from a user perspective, it would make sense to have the following behavior:

If no TZ info attached to datestring, preserve 0.5.43 behavior (pre 1405)
If TZ info attached, parse and attach to the Luxon object.

Any thoughts?

I suppose luxon is no longer properly comparing dates because I've added timezones to all of them? That is tremendously frustrating.

Haha I don't envy you at all, I agree that it's super painful.

Fixed via #1418, at least for file.cday and file.mday - sorry for the regression. Released as 0.5.46.

I should probably swap back to beta releases for these intermediary changes :)


Feature info new layout

This PR changes the feature info popup layout from horizontal table layout to vertical table layout.
I tried to tweak many things and this is what I got.

@manthey, Thanks for the reviewing. 👌


-demo-mode is broken when you have a profile already deployed.

When playing around with things, I noticed -demo-mode is broken when you have a profile deployed.

https://github.com/macadmins/nudge/pull/116 addresses it.


MapRoute missing overload

Hi,
I have noticed that on RouteBuilderExtensions  MapRoute method string[] namespaces overload is missing. Any plans for it?

In vNext, areas are determined by [Area(...)] on your controller, so this feature isn't really relevant anymore. So no, no plans to bring it back :laughing:

But what If I have the controllers on a different assembly how do I achieve that without the namespaces overload?

Instead of using a list of namespaces to group your controllers we recommend using Areas. You can attribute your controllers (regardless of which assembly they are in) with a specific Area and then create a route for that Area.
You can see a test website that shows an example of using Areas in MVC 6 here: https://github.com/aspnet/Mvc/tree/dev/test/WebSites/RoutingWebSite.
Does this work for your scenario?

Thanks for the example.
It is a very good solution but unfortunately it does not work. Reason is I want to keep each route into a different assembly. This is for authorization purposes as the system is huge and several clients need to have only specific dlls.

And I suppose Im not the only one out there having this requirement :)

I think you could still do that with Areas - you could have an Area and route for the controllers in each assembly, right?

Could you give me an example?

I dont think that's a solution really. Areas they have to be marked in each and every one controller. I don't want to use attribute routing. What I want is to map routes according to some parameters. Each route is on a different assembly as I described above.

Nevermind. I used attribute routing for areas only. Created a sample. If you wish I can send it to include it in the examples.

Cool - going to go ahead and close this issue.


Test that persistence is working

In the smoke-test we add a randomly generated file to /var/log and after reboots we check that the file is still there and contains the same data.

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

Comparison is base (22c7e8a) 73.91% compared to head (388a3aa) 73.91%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1919   +/-   ##
=======================================
  Coverage   73.91%   73.91%
=======================================
  Files          71       71
  Lines        7762     7762
=======================================
  Hits         5737     5737
  Misses       1600     1600
  Partials      425      425

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


RFR: Add linux pack requirements to st2 requirements

Since linux is now a system pack, we need it's requirements to be part of system requirements. This is definitely a work around for now. We need to come up with a better way to ship packs with st2.

+1

LGTM, +1


Unit tests for the parser

More extensive unit tests for the default hierarchical parser

Mostly resolved in 1a9de6a, nested types are still pending.


keyword properties don't work in strict mode

Following code triggers a compile error:
"use strict";
var foo = { "enum": 1 };

Obviously another issue with the narcissus formatter. It strips the quotes around enum.

Looks like compiler should accept keywords without quotes. V8 accepts the current keywords unquoted (while, break, function, etc.) but rejects words that are reserved for future use (enum, yield, class). Strange...
I posted on the JSMentor group: http://groups.google.com/group/jsmentors/browse_frm/thread/a3b7eca48ba1e20f?hl=en

Problem is gone with node 0.6.1. Looks like it got fixed by V8 upgrade.


Add the ability to modify app permissions after installation

Hello Eric!
I am building an app where the user will be able to select a number of different actions on his shop which may require different app permissions. So to avoid requesting a bunch of permissions on initial installation, the shop can add or remove accordingly.

Looks great, thanks for the PR!


Support QNX OS without RUSTFLAGS

Removes the need for --cfg mio_unsupported_force_poll_poll.

@AkhilTThomas  any change you could test this without any RUSTFLAGS? I can only run cargo check.

@AkhilTThomas any change you could test this without any RUSTFLAGS? I can only run cargo check.

@SebastianSchildt Could you help out here? my license has expired.

@AkhilTThomas any change you could test this without any RUSTFLAGS? I can only run cargo check.

@SebastianSchildt Could you help out here? my license has expired.

I currently have no usable license as well. Let me ask around

I'm going to merge this so we can add QNX OS to the CI in #1804, @SebastianSchildt if you find someone to run the tests that would be appreciated.


datastore: use transaction object

Fixes #633
The goal of this change is to give more visibility into the operations being performed when using transactions.
To Dos

Automatically rollback if the commit fails

[ ] Review
[ ] Tests

Before
datastore.runInTransaction(function(transaction, done) {
  transaction.save({
    key: key,
    data: { value: true }
  });

  done();
}, function(err) {
  // Where is this error coming from?
});

After
var transaction = datastore.transaction();

transaction.run(function(err) {
  if (err) {
    // Error handling omitted.
  }

  transaction.save({
    key: key,
    data: { value: true }
  });

  transaction.commit(function(err) {
    if (!err) {
      // Transaction committed successfully.
    }
  });
});

The diff scariness doesn't reflect reality... I organized method names alphabetically. The code bodies themselves didn't change, just their location in the file & doc blocks.
@pcostell is this a good change in terms of usability?

Coverage remained the same at 100.0% when pulling c4ce6b9df85c5bef34ae09403f46d1d206da7279 on stephenplusplus:spp--633 into 902d03f93aa0cb84f14fc4165c92faae0af35940 on GoogleCloudPlatform:master.

What happens if there is an error? Do we rollback automatically? Does the user have to do this manually?

Nope, and I'll add a to do item to make sure we rollback if the commit call fails. So, how does this case work:
transaction.commit(function(err) {
  // Behind the scenes, we see that this call fails.
  // So before executing this callback, we should automatically rollback.

  // What happens if the user didn't catch that part in the docs, and tries
  // to rollback themselves?
  if (err) {
    transaction.rollback(...);
  }
});

We just need to make sure to swallow any errors on the automatic rollback -- it will get an error:
INVALID_ARGUMENT: The referenced transaction has expired or is no longer valid.

/cc @jdimond

Added a commit to automatically rollback after a failed commit: https://github.com/GoogleCloudPlatform/gcloud-node/pull/1369/commits/41eec4740f9fc68190690e5fac071f8ea089b850

Changes Unknown when pulling 41eec4740f9fc68190690e5fac071f8ea089b850 on stephenplusplus:spp--633 into ** on GoogleCloudPlatform:master**.

Coverage remained the same at 100.0% when pulling f69cc972053f784e3d428d1dbbf223fb2f648ebe on stephenplusplus:spp--633 into 464ff6c21e48cfde62a3ed1928e1bebe9e99a0db on GoogleCloudPlatform:master.

To verify -- if an error happens during the BeginTransaction call, all attempted calls as part of that transaction will fail, correct?

Are we worried users may try to do something like:
datastore.transaction().run(....) and then use datastore instead of transaction inside that run method?

if an error happens during the BeginTransaction call, all attempted calls as part of that transaction will fail, correct?

var transaction = datastore.transaction();

transaction.run(function(err) {
  if (err) {
    // BeginTransaction failed
  }
});

Calling anything else from transaction will return whatever error the API will.

Are we worried users may try to do something like: datastore.transaction().run(....) and then use datastore instead of transaction inside that run method?

That's a fair concern. We could return the transaction as the second argument to the run callback:
transaction.run(function(err, tx) { /* transaction === tx */ });
datastore.transaction().run(function(err, transaction) {});

Just to be sure -- there are cases where someone might want to start a transaction, and then from outside the transaction execute a query of sorts, right?
If there is no good reason to ever do:
transaction.run( (err) => {
  datastore.run(somethingElse);
  transaction.commit();
});

then we should throw an error if people try to do something like that.... ?

I think I'm a bit confused by your example. run is only a method on a Transaction object, so
datastore.run(somethingElse); would throw.

Sorry - I mean it as an example of "running another query from inside that block". Is there a time when that would make sense? That'd I'd get an entity while inside a transaction block, but not want to use the transaction ?

Gotcha. In terms of a use case, I'm not sure.
I don't think I'd block those operations, though. I'm not sure how, actually... choosing where a function can be executed or not:
transaction.run(function (err) {
  datastore.get(key, function(err, entity) {})
  //        ^-- need to detect this was executed inside of `transaction.run`
})

If this is what we want, maybe @callmehiphop and I can think of something clever. However, I think the user would expect if they have something that works in one place, it would work wherever they choose to put it.

@callmehiphop PTAL

Coverage remained the same at 100.0% when pulling 3f8c632a9f4869fe791326a24f39fe4fda3c663b on stephenplusplus:spp--633 into 50d465b3a960e297abbccf82bd0697f44535f087 on GoogleCloudPlatform:master.

Coverage remained the same at 100.0% when pulling 5aa2c449934ef98c3b54853aeacd1a070239a9c9 on stephenplusplus:spp--633 into 50d465b3a960e297abbccf82bd0697f44535f087 on GoogleCloudPlatform:master.


Is this warning matters? And where does it rise?

UserWarning: Overwriting tiny_vit_5m_224 in registry with tinysam.modeling.tiny_vit_sam.tiny_vit_5m_224. This is because the name being registered conflicts with an existing name. Please check if this is not expected. return register_model(fn_wrapper)

We haven't found that this warning will have any impact.


Change the Chinese punctuation in Example_Delay.md to English punctuation

What is the purpose of the change
Change the Chinese punctuation in Example_Delay.md to English punctuation
Brief changelog
Change the Chinese punctuation in Example_Delay.md to English punctuation
Verifying this change
Change the Chinese punctuation in Example_Delay.md to English punctuation
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.05%) to 47.813% when pulling 7c6a88273b4a629d7ede5a0303db32dfc1e6b433 on xiaoxinglai:develop into 580037fc43d4ffb55bd59b60cfa6e8a9b57e8ca1 on apache:develop.


Overload handling should handle constructor

Problem
https://github.com/dudykr/stc/blob/269f6c07676f728e667ca5fb4c80d7f17f66a98d/crates/stc_ts_type_checker/tests/conformance/types/objectTypeLiteral/constructSignatures/constructSignaturesWithOverloads2.ts#L15-L24
should pass, but stc fails to verify it.
Currently we only check overloads for methods, so overload selection for new fails
Solution
We should handle constructor from extract https://github.com/dudykr/stc/blob/805667830752f229f1674d034e1e9b0eb3fcb076/crates/stc_ts_file_analyzer/src/analyzer/expr/call_new.rs#L1215-L1647

Hi, I would like to take this!

Thanks!

Updated the issue to reflect the real solution

Oh it's a union. namespace is rendered as any in type dump

Ok, so I tried the new real solution by doing something like this:
diff --git a/crates/stc_ts_file_analyzer/src/analyzer/expr/call_new.rs b/crates/stc_ts_file_analyzer/src/analyzer/expr/call_new.rs
index d655cf6e..523e5722 100644
--- a/crates/stc_ts_file_analyzer/src/analyzer/expr/call_new.rs
+++ b/crates/stc_ts_file_analyzer/src/analyzer/expr/call_new.rs
@@ -1515,20 +1515,19 @@ impl Analyzer<'_, '_> {
                 )
             }

-            // Type::Constructor(ty::Constructor {
-            //     ref params,
-            //     ref type_params,
-            //     ref ret_ty,
-            //     ..
-            // }) if kind == ExtractKind::New => self.try_instantiate_simple(
-            //     span,
-            //     ty.span(),
-            //     &ret_ty,
-            //     params,
-            //     type_params.as_ref(),
-            //     args,
-            //     type_args,
-            // ),
+            Type::Constructor(c) if kind == ExtractKind::New => self.get_return_type(
+                span,
+                kind,
+                expr,
+                c.type_params.as_ref().map(|v| &*v.params),
+                &c.params,
+                *c.type_ann.clone(),
+                type_args,
+                args,
+                arg_types,
+                spread_arg_types,
+                type_ann,
+            ),
             Type::Union(..) => {
                 self.get_best_return_type(span, expr, ty.clone(), kind, type_args, args, arg_types, spread_arg_types, type_ann)
             }

However, I don't see any change in the behaviour. In fact, I don't think it was the path taken by the code anyway. ty is Type::Union, so it goes to get_best_return_type() -> extract_callee_candidates(), which does add all 3 CallCandidates of C2.
...am I missing something really obvious here?

@kdy1 I think this issue can be closed since it is fixed by #222?

You are right, thanks!


BMI055 IMU driver reports incorrect temperature

The BMI055 IMU driver is reporting incorrect temperature results.
I've been discussing it on the forum here:
http://discuss.px4.io/t/pixhawk-4-temperature-compensation-failure/10226/10
All testing was performed on a Pixhawk 4 in an industrial environment chamber.
I've inspected the driver source and found that there is indeed a problem with the code that fills the report buffer with the temperature value.  The gyro driver always reports 0, and the accel driver is reporting some value between 23.0 and 23.9 no matter what the actual temperature is.
I've modified the driver code to read the temperature correctly using the slope and offset specified in the BMI055 datasheet.  I'll submit a PR after I've finished testing it.

Thanks for reporting @dmillard-TD. I can quickly review and test once the PR is available.

If possible I'd like to get this into v1.9.0.

PR #11867 submitted.

Fixed in https://github.com/PX4/Firmware/pull/11867


Remove melee pathing nodes for VTOLs

Still having issues where AI can target VTOLs with melee. Return zero path nodes for them when the target is unaffected by pathing per CAC.

Fixed by removing all pathing nodes in melee selection state


Update sensor value checking.

This change is

Reviewed 10 of 10 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

greenpithumb/light_sensor.py, line 27 [r1] (raw file):

        light_level_as_pct = 100 * (
            (light_level - _LIGHT_SENSOR_MIN_VALUE) /
            (adc.PIN_MAX_VALUE - _LIGHT_SENSOR_MIN_VALUE))

This looks a little funny because it's weird to use our own min value, but an external max value. Suggest reverting back to _LIGHT_SENSOR_MAX_VALUE but doing it like
_LIGHT_SENSOR_MAX_VALUE = adc.PIN_MAX_VALUE

greenpithumb/reservoir.py, line 18 [r1] (raw file):

        reservoir_weight_grams = self._weight_sensor.weight()
        if reservoir_weight_grams < self._container_weight:
            raise ValueError()

I think we can go ahead and define a custom exception for this in this PR. If we're touching the line anyway, it's reasonable to also fix issues on that line as long as they're not adding a bunch of complexity to the PR.

Comments from Reviewable

Reviewed 3 of 3 files at r2.
Review status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

Comments from Reviewable

Coverage decreased (-0.4%) to 76.923% when pulling 37441c81cf9ff21f4b3c512af1a85e5ce5ab27cf on check_pin_value_in_adc into f0690a8cdb3184aaa443dd9e2a1d98e110f582c2 on master.


Update README.md

As a User/Market Manager
I need Update the README.md file about instructions
So that they can know what calls are available and what inputs they expect and how to run and test your code so that users will know how it works.
Details and Assumptions

In the README.me file, show all the endpoints and args needed
How to test your code
(optional) Overall workflow if they want to join the team

Acceptance Criteria
Given a user
When they see the readme file
Then how to use your endpoints and reuse your test cases

@brady800526 Should I update the readme file locally and issue a pull request or I just update it directly in GitHub?


Add token overspend and duplication tests

This adds extra tests to check that straightforward attempts to overspend tokens are treated as expected.

Merged 🚀
Thanks @Andrew-128!


CNP validation

One thing I recognized: a CNP with a prefixed letter processed as valid.
Example:
5110102441483     --> Valid (and it actually is)
x5110102441483   --> Valid (but actually it is not)
Thanks for the lib.

fixit

I was about to give you a PR but anyway, thanks for the very quick bugfix :+1: Issue is fixed


SSO issue

Hi there, I am running OKD via helm chart. I tried to authenticate cluster via microsoft authenticator but not able to succeed. As per the docs i did my cnfiguration but afterauthenticating successfully OKD page is refreshing again and again. Please help.
Here is conf- - name: BRIDGE_BASE_ADDRESS
value: https://abc
- name: BRIDGE_K8S_MODE_OFF_CLUSTER_SKIP_VERIFY_TLS
value: "true" # no tls enabled
- name: BRIDGE_USER_AUTH_OIDC_ISSUER_URL
value: https://sts.windows.net/355839ee-fa1f-44d9-b7de-f83955609b52/
- name: BRIDGE_USER_AUTH_OIDC_CLIENT_ID
value: *****************************
- name: BRIDGE_USER_AUTH_OIDC_CLIENT_SECRET
value: ********************************

Hi, did anyone had the same issue or did same deployment of authenticating openshift via azure AD


Check power and auto/remote turn off features on rev1 board

Solder and check both DC/DC converters
Solder and check the auto turn-off feature

Checked on rev.2 they don't work xd. Reasons: unknown


Add ES6 import configuration support

Input
The code looked like this before beautification:
import {ok} from 'lol';

Then become :
import { ok } from 'lol';

Expected to keep "no-space" between braces :
import {ok} from 'lol';

Settings used :
Example:
{
    "brace_style": "collapse,preserve-inline",
}

Can you please consider to implement a new option to config the way objects and ES6 import are beautifulled in javascript ?
https://developer.mozilla.org/docs/Web/JavaScript/Reference/Instructions/import
Settings Example:
{
    "brace_style": "collapse,preserve-inline,nospace",
}

or
{
    "space_in_brace" : false
}

Would you expect in line objects to also have no space?  { a: 1, b: 2 }

Yeah that's the idea but both config are interesting.

I got the same kind of issue with typescript / flow typed javascript :

Both on {} bracket and <> .

Here is the eslint option for this object-curly-spacing : http://eslint.org/docs/rules/object-curly-spacing

@DavidBabel
Can you file the typescript issue separately?

#1147

Thanks!


Update Extension Descriptions in stores to mention open source

The descriptions for the Coil extension listings in Chrome, Microsoft, Firefox, etc. web stores should mention that the extension is open-source and the source code is available at https://github.com/coilhq/web-monetization-projects/tree/master/packages/coil-extension

This appears to be tied to publishing in the Chrome Webstore. Let's remember to close this out once we update the description on the next release

[x] https://chrome.google.com/webstore/detail/coil/locbifcbeldmnphbgkdigjmkbfkhbnca?hl=en
[x] https://microsoftedge.microsoft.com/addons/detail/coil/ljionajlbinlfkdnpkloejeoogfgkojm
[x] https://addons.mozilla.org/en-US/firefox/addon/coil/

Looks all sorted!


Update sprockets requirements

Sprockets 2 does not work with this gem, so make sure to specify that.
One thing that is worthy of discussion is that this makes the requirement for sprockets explicit, whereas before it was "dynamic" to whether sprockets existed or not. Unsure if that was intentional, and if it was then this is probably not going to work.

Hey @hammerdr, thanks for the PR.
The gem also works without sprockets, that's why it's not an explicit dependency. For that reason, I'd rather not merge this.

Sprockets 2 does not work with this gem, so make sure to specify that.

What exactly doesn't work?
It used to work on sprockets 2. Support for sprockets 3 was added in version 1.8.1.

Rails 5.0.x and Sprockets 2.x fails due to this: https://github.com/fphilipe/premailer-rails/issues/211


Is the project alive?

I see that last developer activity was more than 1 month ago, 8 pull requests are waiting something and no comments from the author...
I want to use it in our production development cycle, but that inactivity and author's silence scares me a little.
P.S. also the VS extension is not open-sourced at all.

I expect the project author is just very busy. I'll ping him.
We'd like to open source the VS extension but are waiting for the project system on which it is based to be documented.

Good to hear, because the project and VS extension are really helpful for package creation.

The author is still interested in this project, but very busy at the moment. I am tempted to start merging pull requests into my fork of the project (in fact, I already have) in a more pseudo-official capacity till @terrajobst is back.

@AArnott, how about asking @terrajobst to add you as a collaborator, so you can merge stuff directly into the master ;)

Already done. 😃
Sent from my Windows Phone
-----Original Message-----
From: "Pavol Kovalik"<EMAIL_ADDRESS>Sent: 4/5/2015 3:31 PM
To: "terrajobst/nuproj"<EMAIL_ADDRESS>Cc: "Andrew Arnott"<EMAIL_ADDRESS>Subject: Re: [nuproj] Is the project alive? (#84)
@AArnott, how about asking @terrajobst to add you as a collaborator, so you can merge stuff directly into the master ;)
—
Reply to this email directly or view it on GitHub.

@AArnott Are you starting to handle the outstanding pull requests?

Yes. Immo has so empowered me. But first, we're going to meet once to go
over some pull requests together and make sure I represent his interests
well.
On Tue, Apr 7, 2015 at 8:37 PM williamb1024<EMAIL_ADDRESS>wrote:

@AArnott https://github.com/AArnott Are you starting to handle the
outstanding pull requests?
—
Reply to this email directly or view it on GitHub
https://github.com/nuproj/nuproj/issues/84#issuecomment-90794380.

I see that last developer activity was more than 1 month ago, 8 pull requests are waiting something and no comments from the author...

Sorry for the inactivity. As @AArnott explained I was quite busy at work. As you can see, @AArnott and myself created this organization so that more folks can run this project so we don't bottleneck on a single person, AKA me.
We'll review some of the pending PRs later today.


Remote tests on localhost are not isolated

Running remote tests on playbooks which are delegated to localhost will in fact run the role against the system localhost. There needs to be logic to delegate this properly despite the fact this is a user error.

Note that this could be the solution:

https://docs.ansible.com/ansible/latest/cli/ansible-playbook.html#cmdoption-ansible-playbook-list-hosts

Merged at b33ed5985970c8119e6f9aa77f676bcadc46924f


ValueError: shapes not aligned

I have a dataframe with 522 observations, but when trying to use this I'm getting the following error from line 241:
ValueError: shapes (73, 3869) and (522, 1) not aligned: 3869 (dim 1) != 522 (dim 0)
I understand that the error is that the two matrices are not aligned, but I don't know where the (73, 3869) matrix was generated from?

y has a length of 73, so I assume that is part of the first matrix, if that's any use.

I'm using quarterly data for y and weekly for X, with no NaN values at all.  It appears that the R package supports this, but might this be the issue?

For anyone still having this problem, it's caused by the following:
 def generate_conversion_matrix(self):

        if self.conversion ==  "sum":
            conversion_weights = np.repeat(1, self.fr).reshape((1, self.fr))

If you go from monthly to weekly, this command will make a 1x12 matrix, but it will only make a 1x4 matrix if going from year to quarter. Therefore, the matrix will be way too large. Instead, what we want is a 1x3 matrix and not a 1x12. By changing "self.fr" to 3, it works fine again. This can be done for "average", "first" and "last".
There's probably a better way to do this, but this will fix it for now.

So the problem is, that the author uses the following code:
  fr = len(np.unique(df.grain))
This only works with year to quarter, since there's 4 quarters in a year. However, if you go from quarter to month, it will interpret it as 12 month to a quarter, since there's 12 months in a year. Therefore, the code should instead be:
  fr = df["y"].value_counts().tolist()[0]
This looks at how many times a certain number is repeated in the "y"-column.

Alternative solution to the one suggested by dipti-gupta-2105:
I added a sequence of very small numbers to the "y" column, such as 1e-10, 2e-10,...
This ensures the low frequency data values are unique across periods.
It also doesn't change the functions in the package.

Alternative solution to the one suggested by dipti-gupta-2105:
I added a sequence of very small numbers to the "y" column, such as 1e-10, 2e-10,...
This ensures the low frequency data values are unique across periods.
It also doesn't change the functions in the package.


Problem migrating Iota from Trinity to Firefly using a Nano S

Hi.
I'm trying to migrate my Iota from Trinity to Firefly using my Nano S. Everything went fine, I even saw my balance. But when I tried to migrate the Iota, I got the notification "Something went wrong" with no other explanation. Has anyone else had this problem, and if so, how did you solve it?
Any advice would be greatly appreciated.
Thank you!

Salut jai exactemment le meme probleme que toi , tu as trouvé une solution ?

Unfortunately migrating tokens from the old network using a Ledger Nano in combination with the newest Firefly version 1.3 has some issues atm. If it fails to migrate with the "something went wrong message" do the following:

Downgrade your Firefly version to Version 1.2.
To do this, delete the current Firefly version from your PC, and download version 1.2 from here: https://github.com/iotaledger/firefly/releases/tag/desktop-1.2.0 (scroll down to "assets"), install this Firefly version, and then migrate your tokens with the Nano ledger as described here:
https://blog.iota.org/firefly-token-migration-guide-for-ledger-users/

After this has succeeded, you can update to Firefly version 1.3 which will make staking your IOTA tokens possible.
Do not install Firefly as a "Developer Profile"!

Hi
I have exactly the same problem here:
I installed Firefly 1.3, all fine, it even showed my balance.
Then I installed Firefly 1.2 as you described.
Now I don't even get the balance. It shows "Das dauert ein bisschen. Generiert dein Ledger Adressen?"
I've reinstalled Iota Legacy, doesn't work. Same message.

Can you try to close and reopen the IOTA legacy app on the ledger device? Your ledger needs to display "Generating Address" in this process. If this issue persists please join the IOTA Discord server (https://discord.iota.org/) to get immidiate help.

@AndreMarselous @MAITREIOTA @Marzo2121 Were you able to solve your issue?

Yes, sorry for not replying!
I got help on discord.
Solution:

deactivate firewall
run firefly as admin


Fix KyberSlash vulnerability.

Fix KyberSlash vulnerability.

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

Project coverage is 94.95%. Comparing base (2cb48a3) to head (ee9d003).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
+ Coverage   94.57%   94.95%   +0.38%
==========================================
  Files           8        8
  Lines        1105     1110       +5
  Branches       84       84
==========================================
+ Hits         1045     1054       +9
+ Misses         60       56       -4

Flag
Coverage Δ

unittests
94.95% <100.00%> (+0.38%)
:arrow_up:

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

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

Thank you for the review. I believed that the original modification was largely fine, but following your suggestion, I have decided to perform strict uint32 operations just to be safe.


improve mixin performance by fixing https://code.google.com/p/v8/issu…

…es/detail?id=4165

see https://github.com/jadejs/jade/pull/1982


Remove jim-at-jibba/ariake-vim-colors

This repo has not been updated since 2021-02-23 19:56:14 UTC.
Can @jim-at-jibba confirm whether this repo is still intended for use?

Happy for this I be removed. There are many far better themes.
Thanks


Include review of issues, PRs and release notes in release checklist

Related to Issue #443

Thanks :grin:


Can not install 0.2.0 dvdcli

The command curl -sSL https://dl.bintray.com/emccode/dvdcli/install | sh -s stable always install 0.1.0 but not 0.2.0
root@mesos002:~# curl -sSL https://dl.bintray.com/emccode/dvdcli/install | sh -s stable
(Reading database ... 115972 files and directories currently installed.)
Preparing to unpack dvdcli-latest-x86_64.deb ...
Unpacking dvdcli (0.1.0-1) over (0.1.0-1) ...
Setting up dvdcli (0.1.0-1) ...

dvdcli has been installed to /usr/bin/dvdcli

Binary: /usr/bin/dvdcli
SemVer: 0.1.0
OsArch: Linux-x86_64
Branch: v0.1.0
Commit: 12590b39c7b988bb6cc8408d67c0874e2e686166
Formed: Wed, 07 Oct 2015 04:00:54 CST

Thanks for calling this out. Will check on it.

@jay-lau Sorry for the delay. The 0.2.0 release is in place as expected now.

@clintonskitson just have a try, its working well! Thanks for the quick response, really appreciated!
root@mesos002:~/src/mesos/m3/mesos# curl -sSL https://dl.bintray.com/emccode/dvdcli/install | sh -s stable
(Reading database ... 115972 files and directories currently installed.)
Preparing to unpack dvdcli-latest-x86_64.deb ...
Unpacking dvdcli (0.2.0-1) over (0.1.0-1) ...
Setting up dvdcli (0.2.0-1) ...

dvdcli has been installed to /usr/bin/dvdcli

Binary: /usr/bin/dvdcli
SemVer: 0.2.0
OsArch: Linux-x86_64
Branch: v0.2.0
Commit: b2f436bcd832fed904ac7d9b94cee7174ff59e18
Formed: Thu, 21 Apr 2016 22:43:26 CST

The command curl -sSL https://dl.bintray.com/emccode/dvdcli/install | sh -s stable reports the following error:
sh: line 1: Forbidden!: command not found

Even unable to download the tar file directly. Reports "Forbidden" error.

Is there any other location from where I can install dvdcli?. Please help.


Fix the alert sent if no shared sig algs (1.0.2)

We were sending illegal parameter. This isn't correct. The parameters are
legal, we just don't have an overlap. A more appropriate alert is
handshake failure.
Fixes #2919
This is the 1.0.2 version of #6010.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Pushed. Thanks.


scripts: Use named params for generator options

These changes are in preparation for the upcoming header update.
When calling in the vulkan header python generator the options have
changed in 1.2.142 which causes the options passed by the tools to be
off by one. This ends up setting the emitExtensions to the value of the
sortProcedure and eventually causes the script to fail.
This CL updates the tools to used named parameters for the generator so
that the addition of the genpath does not throw off the parameter
positioning.
Change-Id: I580d5f07ec3ff9e523dbf738561c2e53cfd42796

Man, nothing ever just works anymore, does it.

Haha. Apparently not.

All of the checks are now passing


7408: Adds Barcode Question Type

Superb! Notes from QA:

Remove red line in icon
Fix sms guide for barcode questions
Rebase against the force pushed develop branch (sorry)