Add directory watching option to cli

Resolves #79
This PR adds a --watch option to the command line interface.  When passed into either graph codegen or graph build a watcher will be initialized on the working directory that detects file changes and triggers a restart of the respective process.  The watcher for codegen differs in that it ignores changes to typescript files (*.ts, *.types.ts).

The watcher now parses the subgraph and watches the relevant files.

codegen: ABI's, subgraph manifest
build: ABI's, mappings, graphQL schema, subgraph manifest

If a change is detected to the subgraph manifest it is parsed again and any newly added files will be watched.

@fordN There is one more thing missing in this PR: when there is e.g. a build error while watching, we should print the errors but keep watching. Because errors will happen and people will want to recover from them without having to restart watching.

@fordN Comments should be addressed now.


Send data to kafka streams

Add a module for mage to be able to send data to kafka.

[ ] test
[ ] return the same data to the user
[ ] think about unnamed arguments

Closing stale PR.


Update localization.ts

Added german translation (#1).
Should english be on top because it uses the default strings, or should the languages be in alphabetical order?

Thank you very much!!!

should the languages be in alphabetical order?

I think that would make sense.


Exercise <IP_ADDRESS>: improve answer. Seperating the pieces before and after the apostrophe.

The answer could be improved by really seperating out the pieces before and after the apostrophe with str_match for example.

@jrnold
Hi I wanted to open a new issue because of this, but I willl first try to reach out by commenting on this issue.
The subsection to which this exercise is related to introduced str_match() and str_match_all(). Thus, the answer that the book wants to be given is certainly not a two-step processes which does not involve either function, but rather as @neander09 suggested the use of str_match_all().
I used this code and it did the trick:
str_match_all(sentences, "([A-Za-z]+)'([A-Za-z]+)")


Try to resolve AS info for IP addresses without PTR

Currently, if an IP address of a hop does not have a reverse record, then its AS info also won't be looked up. This seems like a pretty strange limitation.
Would it be sensible to enable AS info lookup for all IP addresses, regardless of whether PTR exists?

Thanks @hexchain, I agree that it makes sense to try and lookup AS info for all IP addresses.
I'll include this in the next release, if you want to try it out now you can build from https://github.com/fujiapple852/trippy/pull/480

Aside: Trippy looks up additional data about each AS, specifically:
pub struct AsInfo {
    pub asn: String,
    pub prefix: String,
    pub cc: String,
    pub registry: String,
    pub allocated: String,
    pub name: String,
}

Currently only asn (i.e. AS15169) is displayed, but we could display and/all of the other fields.  Most useful is name (i.e. "GOOGLE, US") but this is often quite long so perhaps not well suited to be displayed in Trippy.

@hexchain 0.8.0 which includes this change has been released: https://github.com/fujiapple852/trippy/releases/tag/0.8.0


2.5 UI Notices for Deprecated Features

Provide UI notifications in Rancher 2.5 to users that these features are being deprecated.
The features and functionality are replaced with new versions/implementations of the tools and users will need to migrate before upgrading from 2.5 to the minor release in the future.

[x] Rancher Monitoring - Ember UI based
[x] Alerting - Ember UI
[x] Notifiers - Ember UI
[x] Rancher Logging - Ember UI based
[x] CIS Benchmark 1.4 -  Ember UI based
[x] Istio 1.5 - Newer versions available in 2.5.
[x] Rancher MultiCluster Apps - Being replaced by Fleet GitOps
[x] Rancher Pipelines - Being replaced by Fleet GitOps

Need the actual docs location for the deprecation notice to be added.
Since we're moving from info to warning of the deprecation, I've update the banner class to bg-warning.

Monitoring, Alerting . Notifiers , logging and CIS benchmark have the deprecation message. The link to docs-
https://rancher.com/docs/rancher/v2.x/en/reference/deprecation is not available yet.
Restore/Backup is now renamed as Rancher Backups
MultiCluster Apps  and Pipelines changes are tracked in  https://github.com/rancher/rancher/issues/29179


Unzoom without padding

Hello,
Is it possible to unzoom without padding?
img_trans_train = iaa.Sequential([
        iaa.Affine(scale=(0.32, 1)),
        iaa.CropToFixedSize(width=1920, height=1280),
    ])

My original images are 6000x4000. If a scale of 0.32 is applied, the images become 1920x1280 so CropToFixedSize shouldn’t do anything. My problem is " iaa.Affine” does add padding to keep the image at the same size. As a result, CropToFixedSize can contain padding.

Using Resize fixed the problem
img_trans_train = iaa.Sequential([
        iaa.Resize((0.32, 1)),
        iaa.CropToFixedSize(width=1920, height=1280),
    ])


@page :first does not work in combined mode

Hi,
I would like to style my first page differently. Unfortunately, the css declaration in a @page :first pseudo element affect every page.
I guess it's an issue related to #36 .
Could you do something about it ?

Unfortunately I don't see a way to fix this easily.., when #35 is implemented, I will expose the page number so per page styling could be done.


Request: show backtrace for GC allocation warnings when using debug builds

As mentioned in #2104, warnings from Boehm GC might appear randomly on bigger programs.
It will be great that the hook provided by @asterite in that issue could be turned on when compiling with debug information, or a similar option (say --debug-gc), to allow better debugging of applications.
Thank you.

+1 here :)


SAA-1814: add a job to fix Risley unpaid activity. This is for a single activity only and runs via two separate cron jobs

This is set to run in preprod only to check it works. It does the following

Deallocate prisoners from an activity
Remove the activity pay and set the activity to unpaid
Reallocate the prisoners for a day ahead

Codecov Report
Attention: Patch coverage is 13.63636% with 57 lines in your changes missing coverage. Please review.

Project coverage is 83.74%. Comparing base (e5b2be3) to head (22f9d20).

Files
Patch %
Lines

.../hmppsactivitiesmanagementapi/job/FixZeroPayJob.kt
3.03%
32 Missing :warning:

...sactivitiesmanagementapi/service/DataFixService.kt
0.00%
15 Missing :warning:

...tiesmanagementapi/resource/JobTriggerController.kt
14.28%
6 Missing :warning:

...esmanagementapi/service/ActivityScheduleService.kt
66.66%
1 Missing and 1 partial :warning:

...activitiesmanagementapi/service/ActivityService.kt
50.00%
0 Missing and 1 partial :warning:

...hmppsactivitiesmanagementapi/util/SecurityUtils.kt
50.00%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #992      +/-   ##
============================================
- Coverage     84.11%   83.74%   -0.38%
- Complexity     3720     3724       +4
============================================
  Files           380      382       +2
  Lines         12342    12408      +66
  Branches       1084     1088       +4
============================================
+ Hits          10382    10391       +9
- Misses         1464     1518      +54
- Partials        496      499       +3

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


[Kucoin] NoSuchMethod error on Kucoin SDK

java.lang.NoSuchMethodError: okhttp3.HttpUrl.get(Ljava/lang/String;)Lokhttp3/HttpUrl;
at retrofit2.Retrofit$Builder.baseUrl(Retrofit.java:458)
at com.kucoin.sdk.factory.RetrofitFactory.getPublicRetorfit(RetrofitFactory.java:33)
at com.kucoin.sdk.rest.impl.retrofit.PublicRetrofitAPIImpl.getAPIImpl(PublicRetrofitAPIImpl.java:28)
at com.kucoin.sdk.rest.adapter.SymbolAPIAdaptor.getSymbols(SymbolAPIAdaptor.java:27)
at org.knowm.xchange.kucoin.KucoinMarketDataServiceRaw.lambda$getKucoinSymbols$2(KucoinMarketDataServiceRaw.java:31)
at org.knowm.xchange.kucoin.KucoinExceptionClassifier.classifyingExceptions(KucoinExceptionClassifier.java:17)
at org.knowm.xchange.kucoin.KucoinMarketDataServiceRaw.getKucoinSymbols(KucoinMarketDataServiceRaw.java:31)
at org.knowm.xchange.kucoin.KucoinExchange.remoteInit(KucoinExchange.java:72)
at org.knowm.xchange.BaseExchange.applySpecification(BaseExchange.java:115)
at org.knowm.xchange.kucoin.KucoinExchange.applySpecification(KucoinExchange.java:39)

Hmm, strange. I'll try to replicate this.

I need to remove the dependence on the external SDK anyway, so may just go down that route if the cause isn't obvious.

@trixprod, I just did the following:

Pulled the latest code on the develop branch
Opened xchange-examples in my IDE
Fully refreshed all Maven dependencies
Ran org.knowm.xchange.examples.kucoin.marketdata.KucoinMarketDataDemo

Everything worked fine.
Can you post some code that causes this error?

Could you also check which version of okhttp3 you have on your classpath?

I will try with the SDKless version and update the ticker status in few days.

Could you also check which version of okhttp3 you have on your classpath?

My okhttp3 version is 3.8.1


add autoconfiguration skip config

fix https://github.com/koupleless/koupleless/issues/186 https://github.com/koupleless/koupleless/issues/41

Summary by CodeRabbit

New Features

Introduced mechanisms to enhance configuration flexibility during application startup, including filtering of auto-configuration classes and skipping specific initializers based on configurations.
Added utility methods for better handling and formatting of property values and determining module types.

Tests

Added tests for new functionalities related to application context setup and initializer skipping logic.

Refactor

Improved the application's runtime context management by introducing a method to retrieve the master BizRuntimeContext.

@coderabbitai review


Blazor CascadingParameter design has multiple flaws

The Blazor CascadingParameter design has multiple flaws:

A child component has to know that its parameter will be used in a cascading manner and has to decorate the parameter with [CascadingParameter]. Why not just come up with a design so that a child component does not have to know whether or not it will be used in a cascading manner? This would be a much better SOC design; only the parent should have to know how the child's parameters will be used. For children, properties should simply use [Parameter] everywhere, and a proper cascading design should allow for cascading usage/specification (or not) at the parent (higher) level only.

The [CascadingParameter] attribute uses a string to identify the parameter to cascade! This is a horrible design pattern. We have all the type-safety, compile-time checks, and reflection of .NET at our disposal now, and someone chose to use a string value to represent the identifier of the parameter. Why? a) What if I want to change the string identifier of the parameter (answer: I have to do a fuzzy search through all the code to try to make the change, which puts us right back to the Javascript world, ugh). b) What if I have two different types of child components that for some reason decided to use the same identifiers for their cascading parameters? Such a poor design choice.

Can we please come up with a better design choice for cascading parameters in Blazor?

Hello @velocitytoo ... We only handle the docs on this repo. Please open this for the product unit at ...
https://github.com/dotnet/aspnetcore/issues


Add prisma-docs-search extension

Description

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Thanks for the help with this PR @pernielsentikaer! 🙏
We've shared the news of the extension from our Prisma accounts, Raycast makes some cool stuff possible :)
https://x.com/prisma/status/1842193695158022604


Update AngularJS to 1.6

Confirmed with AngularJS 1.6 in a work project and the demo here. Unit tests continue to pass.

Awesome, thanks for doing that!


feat: support json ser/de for datafile

Why we don't do normal serialization/deserialization like other structs?

Have integrate the avro and json ser/de and added related test.
And we should pass the type to convert on_disk::DataFile to in_memory::DataFile.
pub fn data_file_from_json(
    value: serde_json::Value,
    partition_type: types::Any,
)

There are some place may need to refine. We can check the interface first and if it looks well, I will refine them later.


Cannot read property 'init' of undefined

https://ellie-app.com/48tX8QLg8Yya1
Looks like it doesn't support module names other than Main

Line 10 in your HTML should be ``

Line 10 in your HTML should be
   var app = Elm.Cube.init({ node: document.querySelector('main') })

https://ellie-app.com/4hbS8N9sfZba1


Switching between Workspaces shortcuts not working

Bug description:
Switching between Workspaces shortcuts not working
Steps to reproduce:

Go to add new service
Add a custom website
Enter the url "https://miro.com"
Create a free account, and make a new board

Expected behavior
When you have any Miro board loaded, you are unable to use shortcuts to switch between workspaces
Environment:

Operating System: Windows 10
Ferdi Version: 5.6.0-beta.5
Server: Ferdi

Miro.com shortcut's seem to be conflicting with Ferdi, https://help.miro.com/hc/en-us/articles/360017731033-Shortcuts-and-Hotkeys
I think Ferdi should force the shortcuts not the other way around


[SPARK-36086][SQL][3.0] CollapseProject project replace alias should use origin column name

What changes were proposed in this pull request?
For added UT, without this patch will failed as below
[info] - SHOW TABLES V2: SPARK-36086: CollapseProject project replace alias should use origin column name *** FAILED *** (4 seconds, 935 milliseconds)
[info]   java.lang.RuntimeException: After applying rule org.apache.spark.sql.catalyst.optimizer.CollapseProject in batch Operator Optimization before Inferring Filters, the structural integrity of the plan is broken.
[info]   at org.apache.spark.sql.errors.QueryExecutionErrors$.structuralIntegrityIsBrokenAfterApplyingRuleError(QueryExecutionErrors.scala:1217)
[info]   at org.apache.spark.sql.catalyst.rules.RuleExecutor.$anonfun$execute$2(RuleExecutor.scala:229)
[info]   at scala.collection.LinearSeqOptimized.foldLeft(LinearSeqOptimized.scala:126)
[info]   at scala.collection.LinearSeqOptimized.foldLeft$(LinearSeqOptimized.scala:122)
[info]   at scala.collection.immutable.List.foldLeft(List.scala:91)
[info]   at org.apache.spark.sql.catalyst.rules.RuleExecutor.$anonfun$execute$1(RuleExecutor.scala:208)
[info]   at org.apache.spark.sql.catalyst.rules.RuleExecutor.$anonfun$execute$1$adapted(RuleExecutor.scala:200)
[info]   at scala.collection.immutable.List.foreach(List.scala:431)
[info]   at org.apache.spark.sql.catalyst.rules.RuleExecutor.execute(RuleExecutor.scala:200)
[info]   at org.apache.spark.sql.catalyst.rules.RuleExecutor.$anonfun$executeAndTrack$1(RuleExecutor.scala:179)
[info]   at org.apache.spark.sql.catalyst.QueryPlanningTracker$.withTracker(QueryPlanningTracker.scala:88)

CollapseProject project replace alias should use origin column name
Why are the changes needed?
Fix bug
Does this PR introduce any user-facing change?
No
How was this patch tested?
Added UT

FYI @cloud-fan

Test build #142211 has started for PR 33686 at commit 00a69fd.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46724/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46724/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46724/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142211/

retest this please

Test build #142232 has started for PR 33686 at commit 00a69fd.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46744/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46744/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46744/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142232/

is it a real failure?
DataSourceV2ScanExecRedactionSuite.SPARK-30362: test input metrics for DSV2

is it a real failure?
DataSourceV2ScanExecRedactionSuite.SPARK-30362: test input metrics for DSV2

Test in local, it's ok.

Test build #142259 has started for PR 33686 at commit f039a98.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46766/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/46766/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/46766/

Test build #142259 has finished for PR 33686 at commit f039a98.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/142259/

thanks, merging to 3.0!


Stripe api

Description
Added Stripe for easy payments. Payment options are the following 2.99 for one month, 9.99 for 6 months and 19.99 for a full year of unlimited alerts.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
All transactions were tested locally and monitored through stripes dashboard.
Checklist:

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

I'm getting the following error when try to start up the react server:
./src/components/Billing/Billing.js
Module not found: Can't resolve 'reactstrap' in '/Users/nikhil/lambdaschool/labs7/ksl-alerts/CS10-ksl-alerts/client/src/components/Billing'

It doesn't look like reactstrap is in the client's package.json so you may have installed it globally on your environment.


can't create related lead from leads editable subgrid on account form

Help Wanted
Hi,
I am trying to access the new button on a related leads subgrid on an account form.
So far I have implemented this:
                **//Open the Tab Leads and Opportunities and create a new Lead**
                xrmBrowser.Entity.ExpandTab("Leads and Opportunities");
                **//Click the lookup add button**
                xrmBrowser.Entity.ClickSubgridAddButton("Leads");

                **//Select the lookup on the subgrid**
                xrmBrowser.Entity.SelectSubgridLookup("Leads");

Now I want to access the new button in the right lower corner, but can't find any function that allows me to do that.
I can invoke a new lead but this creates a NON related lead.

Thanks in advance!
Eduard Dalm

This has been replaced by the Unified Interface. The new APIs will be:
xrmApp.Entity.SubGrid.ClickCommand(string subgridSchemaName, string commandButton, string subCommandButtonIfRequired);


Is it possible to use AirPlayRoutePickerView() with other video player

Hello,
Thanks to make this plugin I really need this. But I want to know Is it possible to use AirPlayRoutePickerView() with other video player (like flick_video_player) or if possible please add video controllers and screen orientation functionality.

Hi, theoretically it should be, as it is an independent widget that can be integrated anywhere, but it still needs to be tested in such cases. Do let me know if it works for you or not and I will be available to improve it if there is an issue.
Regarding further support of video controllers and orientation functionality, please give me time, I work on these over the weekends only, but I will try to check it out during the week as well. In the meantime, if you want to contribute, please feel free to open a PR and I will review that.
Thank you for your comment.

I tried to integrate it independently and it worked 1st time but 2nd time when i played my video app app crashed with any logs.

Can you share the logs or the PR, so that I can review and fix it?
It will be difficult to fix without reproducing.

I tried with xCode and got this
Fatal error: Unexpectedly found nil while unwrapping an Optional value: file /path/flutter_sdk/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_to_airplay-1.0.2/ios/Classes/FlutterAVPlayer.swift, line 26

Okay, let me fix it.
Can you share the widget code as well? how you are creating FlutterAVPlayer widget.

I used flick_video_player because I need to play video in landscape mode

I just pushed a fix to the master branch. I will submit a new version to pub.dev in a few minutes.

Any update on above? As I am still facing the same issue.
Fatal error: Unexpectedly found nil while unwrapping an Optional value

I have found a workaround but you can find a proper solution to it, I was going through the code and using breakpoints realized that the first time I build AirPlayRoutePickerView() Widget it builds it properly and when I try to build it second time it was trying to build FlutterAVPlayer() Widget instead and it was null so the app was crashing. So in the below file you see
if viewId == 1 it build the AirPlayRoutePickerView() else it would build FlutterAVPlayer() Widget and when I try to load the page second time the viewId is incremented, for now I have changed it to if viewId >= 1 and it works for my purpose but you can implement a proper fix for it.
Thank you
Jigar Kotak
//
//  SharePlatformViewFactory.swift
//  flutter_to_airplay
//
//  Created by Junaid Rehmat on 22/08/2020.
//
import Foundation
import Flutter
class SharePlatformViewFactory: NSObject, FlutterPlatformViewFactory {
var _messenger : FlutterBinaryMessenger
init(messenger: FlutterBinaryMessenger & NSObjectProtocol) {
    _messenger = messenger
}

func create(withFrame frame: CGRect, viewIdentifier viewId: Int64, arguments args: Any?) -> FlutterPlatformView {
    if viewId == 1 {
        let pickerView = FlutterRoutePickerView(frame: frame, viewIdentifier: viewId, arguments: args as! Dictionary<String, Any>, binaryMessenger: _messenger)
        return pickerView
    }
    else
    {
        let pickerView = FlutterAVPlayer(frame: frame, viewIdentifier: viewId, arguments: args as! Dictionary<String, Any>, binaryMessenger: _messenger)
        return pickerView
    }
}

func createArgsCodec() -> FlutterMessageCodec & NSObjectProtocol {
    return FlutterStandardMessageCodec.sharedInstance()
}

}

Thank you @expertdevs and @JigarKotak7. I have published version 1.0.4 and that will fix all of the crash issues. Please feel free to comment if you notice any further crashes. I will try to be more responsive now.


Do verify with Google id_token for every single requets

A dedicated endpoint to exchange the google id token with out own JWT (JSON Web Token) would be the better to do. Reduces network utilization and better response times.

Fixed with this pull request


[$250] Android - add attachment drawer animation is janky

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Navigate to any chat
Click on the "+" in the compose box
try to add attachment

Expected Result:
Transition should be smooth
Actual Result:

When you tap the + button in the compose box
The gray overlay flashes, leaving some space at the bottom that isn't covered by the overlay
The tray showing up in two "steps" instead of a smooth slide up

Workaround:
unknown
Platform:

Where is this issue occurring?

Android

Version Number: 1.2.0-0
Reproducible in staging?: Y
Reproducible in production?: needs repro
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/190671691-54019588-d8a0-4211-a549-556eea7c39da.mp4
Expensify/Expensify Issue URL:
Issue reported by: @alex-mechler @cead22
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1662396446776909
View all open jobs on GitHub

Good candidate for external!

Job posting: https://www.upwork.com/jobs/~0199b6f2d618d1d175

May I fix this issue?
I have no upwork account so I cannot make proposal but I can fix this

@Smart133594 because we issue payment via Upwork, an Upwork account is required.

doubled price: https://www.upwork.com/jobs/~0199b6f2d618d1d175

@NicMendonca Mind double the price again? Reasoning is that this is tied up with a roadmap item, so I want to ensure that the price is in line with our internal priority for it as well. Thanks!

Proposal
The issue here is with timing and easing of the transition for backdrop and bottom modal.
You see, the default value for speed according to docs is 300
With that in mind we have two options, we either slow down the backdrop fade in appearing, or we speed up the bottom modal appearing.
For me personally, slowing down could give a bit of a feel that app is slower
So I would choose we go with speeding up the modal appearance.
How can we do that:
We would need to pass a property called animationInTiming from popover menu(where this Is called) to Base Modal component
I'd recommend that we go with value of 100, because it gives a nice feel that animation is not too fast and satisfies our needs
Here is a video
https://user-images.githubusercontent.com/70144379/194364229-e91903b4-fbcd-4528-8d73-60ea3a0694cf.mov

doubled price: https://www.upwork.com/jobs/~0199b6f2d618d1d175
bump @mananjadhav on the proposal above ☝

@Uros787 Are you able to reproduce this at your end? And can you confirm that after your fix it is working fine?
Also I am not able to figure out what parts are you going to change. I can see you've linked two files ReportActionCompose and BaseModal, but can you post which lines of code will change and what props will be updated?

Yes sir, I was able to reproduce it on Android and the solution is working smoothly.
The solution consists of changes in 4 files since we are going to send a prop down from ReportActionCompose to BasePopoverMenu.
And also we would need to change popoverPropTypes, and popoverMenuPropTypes

@mananjadhav any feedback on @Uros787 update?

Thanks for the bump @NicMendonca .
@Uros787 I am fine with the animation going as fast as 100ms.
Let's get an input from @Expensify/design to see what they think about this.

Looks good, but I'm curious how this compares with other drawer speeds (e.g. opening up the menu from the big green plus)?

@mananjadhav ☝

Thanks for the bump @michelle-thompson. Sorry I didn't get a notification.  Other drawers should be at 300 with the default value, and the Skeleton UI animation is at 100.

Cool, I think what we have in the video above would be good to go then.

Okay thanks for the comment @shawnborton. @Uros787 Your proposal here https://github.com/Expensify/App/issues/11059#issuecomment-1273160490 looks fine. While it is missing some details on the actual changes, the expected behavior looks good and we can work out the kinks in the PR.
@shawnborton You being the CME, I guess you'll have to assign the job.
🎀 👀 🎀
C+ reviewed

@NicMendonca all you!

@Uros787 the job posting is here, thanks!

Hi, only upwork is possible for your job?
Directly hire impossible?
On Fri, Oct 21, 2022 at 8:44 AM Uroš Stojanović @.***>
wrote:

Thanks, i’ve submitted a proposal already on upwork
—
Reply to this email directly, view it on GitHub
https://github.com/Expensify/App/issues/11059#issuecomment-1286913519,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQP5L72TLAT6PUK7FV5OMI3WEKF4XANCNFSM6AAAAAAQONMOVY
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Smart133594 Thanks for the comment, but we make our Payments via Upwork. So while the proposal review, and PR etc. is all done via Github, the payment happens only via Upwork only! Please check the our contributing guidelines here

Thanks but if I have no upwork account and I want also work on this.
How can I? May I use other man's account?
On Fri, Oct 21, 2022 at 9:50 AM Manan @.***> wrote:

@Smart133594 https://github.com/Smart133594 Thanks for the comment, but
we make our Payments via Upwork. So while the proposal review, and PR etc.
is all done via Github, the payment happens only via Upwork only! Please
check the our contributing guidelines
https://github.com/Expensify/App/blob/main/contributingGuides/CONTRIBUTING.md
here
—
Reply to this email directly, view it on GitHub
https://github.com/Expensify/App/issues/11059#issuecomment-1286993764,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQP5L7334XVVA6CQLBQJCALWEKNTVANCNFSM6AAAAAAQONMOVY
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Uros787 hired on upwork 😊

If you have other tasks, could you contact me?
On Mon, Oct 24, 2022 at 9:20 AM Nicole Mendonca @.***>
wrote:

@Uros787 https://github.com/Uros787 hired on upwork 😊
—
Reply to this email directly, view it on GitHub
https://github.com/Expensify/App/issues/11059#issuecomment-1289025500,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQP5L744VD5Y7T4QSI4ASADWE2ELVANCNFSM6AAAAAAQONMOVY
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'll do the second round of review by tomorrow.

@mananjadhav any updates here? Thanks!

Sorry for the delay here, travelling due to festive week. But I'll posted a comment on the PR. Once done I'll test the changes.

PR is merged. I am not sure what needs to be done for the BugZero Checklist. @NicMendonca @shawnborton  ?

@mananjadhav we're discussing internally here, so I'll come back with more info asap!

@NicMendonca  @miljakljajic  This is due payment but I am not sure what happened here with the tags and reassignment?

Thank you, Manan. Uros was already hired for the job, so he has been paid. Awaiting your approval of the offer and I'll pay you for your contribution immediately.

Applied @miljakljajic. Any action pending from my side here @miljakljajic?

Thank you - job is now closed, both of you have been paid.


chore: increase lint timeout

Today I faced multiple times with timeout exceeded error on ci action. Increasing timeout helps to avoid this issue.

Codecov Report

Merging #2917 (b54efef) into main (33f5cd3) will decrease coverage by 1.45%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2917      +/-   ##
==========================================
- Coverage   51.72%   50.28%   -1.45%
==========================================
  Files         167      167
  Lines       10801    10801
==========================================
- Hits         5587     5431     -156
- Misses       4719     4883     +164
+ Partials      495      487       -8

see 6 files with indirect coverage changes


Use new account in integration tests

What does this PR do?
use the new account id + migrate integration tests from sa-east-1 to eu-west-1
Motivation
account change
Testing Guidelines
CI must pass
Additional Notes

Types of changes

[ ] Bug fix
[ ] New feature
[ ] Breaking change
[x] Misc (docs, refactoring, dependency upgrade, etc.)

Checklist

[ ] This PR's description is comprehensive
[ ] This PR contains breaking changes that are documented in the description
[ ] This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
[ ] This PR impacts documentation, and it has been updated (or a ticket has been logged)
[ ] This PR's changes are covered by the automated tests
[ ] This PR collects user input/sensitive content into Datadog

Codecov Report

Merging #119 (4472296) into main (0ffadcc) will increase coverage by 1.28%.
The diff coverage is n/a.

:exclamation: Current head 4472296 differs from pull request most recent head 0e8462e. Consider uploading reports for the commit 0e8462e to get more accurate results

@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
+ Coverage   83.23%   84.51%   +1.28%
==========================================
  Files          13       13
  Lines         847      788      -59
==========================================
- Hits          705      666      -39
+ Misses        113       96      -17
+ Partials       29       26       -3

Impacted Files
Coverage Δ

ddlambda.go
78.57% <0.00%> (-1.05%)
:arrow_down:

internal/metrics/kms_decrypter.go
68.96% <0.00%> (-1.04%)
:arrow_down:

internal/wrapper/wrap_handler.go
97.91% <0.00%> (-0.03%)
:arrow_down:

internal/metrics/processor.go
100.00% <0.00%> (+2.29%)
:arrow_up:

internal/trace/listener.go
58.06% <0.00%> (+3.13%)
:arrow_up:

internal/extension/extension.go
90.90% <0.00%> (+10.64%)
:arrow_up:

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


Improve automatic dark mode support

re-opening #3903

oops, I just realized that the changes from #3903 were already committed.

oops, I just realized that the changes from #3903 were already committed.


Turn GitHub's native "Delete branch" & "Delete fork" toggle into 2 single-click links

As suggested in https://github.com/sindresorhus/refined-github/pull/1791#issuecomment-470229212, #1786 might have been implemented prematurely, because GitHub's design requires more clicks than the removed delete-fork-link.
Having no working experience with this addon's notions, please let me summarize @bfred-it's reply there as an FR here:

"make both buttons visible"
"unhide the Delete Fork button via CSS only"
"make it look like the old text link"

I agree, it should have been two buttons natively.

Thank you :-)


Testing on new data!

Hey! @cardwing
How to test the pre-trained model on other test data (apart from CULane or any other datasets mentioned in this repo)?
Are the ground truth labels required??

@sicarioakki I think anwsers can be found in ENet-Label-Torch/testLane.lua. GT lables are not required.


Classifiers issue

This project
https://github.com/druid-io/tranquility/
fails with:

[error] 2 not found
[error]   https://repo1.maven.org/maven2/org/apache/flink/flink-core/1.0.3/flink-core-1.0.3.test-jar
[error]   https://repo1.maven.org/maven2/org/apache/flink/flink-runtime_2.10/1.0.3/flink-runtime_2.10-1.0.3.test-jar

when resolving artifacts with "tests" classifiers.
My intuition in that, the token "test-jar" if deduced from the pom file ... my less than 2 cents.
net.virtualvoid.sbt.graph.Plugin.graphSettings must be changed to net.virtualvoid.sbt.graph.DependencyGraphSettings.graphSettings
PS. The project build without coursier.

Running into a similar issue where the artifact type seems to be set to test-jar in the pom.  For example, the pom for hadoop-yarn-server-resourcemanager-2.7.1 has dependencies like:
{code}

org.apache.zookeeper
zookeeper
test-jar

{code}
There seems to be some description of these "default" types (and there associated artifact handlers) here:
https://github.com/apache/maven/blob/c023e58104b71e27def0caa034d39ab0fa0373b6/maven-core/src/site/apt/artifact-handlers.apt
I've also put together a barebones SBT project to help illustrate this issue here: https://github.com/matt-martin/coursier-zoo-test

I think the following map needs to be updated to include the test-jar key:
https://github.com/alexarchambault/coursier/blob/c5fb28f982ff03eef386958f79954b846187acd7/core/shared/src/main/scala/coursier/maven/MavenSource.scala#L157
And probably some changes to the artifacts method above that to select the appropriate classifier, etc based on the artifact type.


The "Superkudzu" Incident...

This post may only be for documentation, since I think this could have been a one-time thing, but i think its good to have documentation anyways.
Yesterday on Lizard, A kudzu appeared in sci while working on Barratry. Now, Unless I've been misinformed by a mod, I'm 90% sure that Kudzu is supposed to slow down when it gets to a certain size...

The "Superkudzu" just kept spreading and spreading, never slowing down. I'm a hundred percent sure that this was a bug, because not even ERT and Goat teams could take it out. In fact, the goats couldn't even EAT the kudzu, because I think the abnormal... kudzu... npc... grow... I dunno. The goat AI broke because of Superkudzu. That's all I know. The Superkudzu grew to such extent that it took over ALL of the outside spacewalk , including singulo.  Only a few survived to tell the tale of Superkudzu...
Reproduction
Unknown
Screenshots
If you want to see the replay, it is round number 31913. Download it Here. (The kudzu starts at Minute 18 Second 4.)
Additional context
shitsci

Dupe https://github.com/space-wizards/space-station-14/issues/18021

mb


x/leverage: Support Lending Assets

Summary
Have the ability for x/leverage to support users lending allow-listed tokens where a minted "u" token is derived of equal amount.
Note, without an oracle in place, all integration tests should be done with borrowing and lending with internal tokens.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

With the exception of CLI support, which has its own issue (#94), this is complete.


[BEAM-11219][Website revamp] Development of All about Apache Beam component

Implemented the "All about Apache Beam component"!

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

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

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Review: @TheNeuralBit


Adds first set of travis tests.

This just adds a short list of travis tests to kick off the CI environment.
@madopal could you review? Thanks!

:lgtm:

Awesomesauce. Thanks!


Volume Keys Navigation

Is it possible to implement volume keys actions such as thread scrolling (as PgUp/PgDn style or Next/Previous post) and Next/Previous file in Gallery view?

It's possible, but for what purpose? All smartphones has a convenient touch screen that allows you to scroll as you wish.
I don't think it's necessary. This pattern is quite unusual for android applications.


Popup not working in django1.9

In foreign key field the edit button is not working and + button opens it in the next page (not to the popup). So can I have a patch or any solution for 1.9?

Yeah, django 1.9 is possibly broken because templates have changed. And possibly it was me breaking popups for a good cause though :) https://github.com/django/django/commit/0decef337f7f00d73b380531ac22cbc843d6e806

Thanks for quick reply, it works for me...


Scroll position lost in one tab after going to another tab

SQL Operations Studio Version:
0.31.4 (Mac)
Steps to Reproduce:

Have 2 tabs open
Run SQL statement that produces many resulting rows
Scroll down the result set and highlight a row
Go to the other tab
Come back to original tab
The result set is back at the top again

The same thing happens in the file editor too. Visual Studio Code doesn't do this at least in the file editor.

This is fixed in the August Preview Insiders Build 0.32.3. (results grid and Editor) I have confirmed this on Windows only.  Should be included in the official August Preview release.

See #1744 for the main issue that tracked this


Persist database facts (in a database :)), add creation endpoint

ORM research
https://diesel.rs/guides/getting-started.html -> lots of stars, no async. Has a notion of migrations and models. Has its own query language, so can claim to be typesafe. No async.
https://www.sea-ql.org/sea-orm-tutorial/ch00-00-introduction.html -> Less stars but popular. Similar functionality to Diesel, but async.
https://github.com/launchbadge/sqlx#usage -> Async, supports several DBs (no one supports DuckDB, but this is for stacks' own models, not for the managed databases). Not an ORM --- you can provide a struct into which to load data from a SQL query, but you have to manage selecting the appropriate columns, etc. Migration support is built-in. This article introduced me to sqlx and has a nice discussion with example queries and how migrations work: https://www.reddit.com/r/rust/comments/n8cu8z/what_ormalike_library_are_you_using_with_rust_for/
Current leaning: sqlx and seeing how much boilerplate that entails. I'd still like to have clean models.rs structs, so let's see how far I can get without a formal ORM.

Running with this example to set up project: https://github.com/launchbadge/sqlx/tree/main/examples/sqlite/todos

Next steps

[ ] Install sqlx-cli: https://crates.io/crates/sqlx-cli
[ ] export DATABASE_URL="sqlite:test.db"
[ ] sqlx db create
[ ] sqlx migrate run

Next steps

[ ] Document installation (cargo install sqlx-cli, DATABASE_URL, sqlx db create/migrate run)
[ ] Create stacks_db module with metadata/persistence
[ ] Create stacks_db::create_database
[ ] Add create HTTP endpoint

Next steps

[ ] Add tests for create_database
[ ] Recreate the same create_database logic for database_owner
[ ] Read database owner in create_database from path
[ ] Read db_type from creation endpoint headers
[ ] Document installation (cargo install sqlx-cli, create DB/user in Postgres, DATABASE_URL, sqlx db create/migrate run)

Next steps

[ ] Add tests for create_owner, create_database, query
[ ] Read db_type from creation endpoint headers
[ ] Switch to a more inclusive form than owner
[ ] Document installation (cargo install sqlx-cli, create DB/user in Postgres, DATABASE_URL, sqlx db create/migrate run)


Adjust options when DOCKER_TLS_VERIFY is enabled

@joelwurtz , can you look into this?
The DockerClient::createFromEnv not producing correct connection param.
Current output is:
Array
(
    [remote_socket] => tcp://<IP_ADDRESS>:2376
    [ssl] => Array
        (
            [cafile] => /Users/smacalalag/.docker/machine/machines/default/ca.pem
            [local_cert] => /Users/smacalalag/.docker/machine/machines/default/cert.pem
            [local_pk] => /Users/smacalalag/.docker/machine/machines/default/key.pem
            [peer_name] => boot2docker
        )

)

Should be:
Array
(
    [remote_socket] => tcp://<IP_ADDRESS>:2376
    [ssl] => 1
    [stream_context_options] => Array
        (
            [ssl] => Array
                (
                    [cafile] => /Users/smacalalag/.docker/machine/machines/default/ca.pem
                    [local_cert] => /Users/smacalalag/.docker/machine/machines/default/cert.pem
                    [local_pk] => /Users/smacalalag/.docker/machine/machines/default/key.pem
                    [peer_name] => boot2docker
                )

        )

)

Also, If peer_name is not set, then the name is guessed based on the hostname used when opening the stream. So, I made changes on how the peer_name should be added.

:+1: Thanks !


Wfnews 814

Add fireCentreName attribute and update api to fetch and orderBy it. I have reviewed with Sean to verify the validation

You're missing mappings in PublishedIncidentResourceFactory, which would prevent the new attribute from ever being mapped? Make sure you test API changes, there are Docker scripts set up to assist with this if you don't have a local debug environment for java APIs.


Different zmqpath ports in install dokumentations

As new user I would like to follow the default configuration. Bitcoin.conf and lnd.conf contain different zmqpath ports. Which one is the recommended default port?
Port 28332: https://github.com/lightningnetwork/lnd/blob/47329c9267f03887a2f1596a99f23446dc7d4176/docs/INSTALL.md#L215
Port 18501 https://github.com/lightningnetwork/lnd/blob/0c1b619e53ac88ad53f3a88e5d1129c25ddeaf13/sample-lnd.conf#L190

Per 47329c9, 28332 is recommended. A fix for the sample config is a part of #911.


Tor Hidden Services v3

So is it safe to say that new TOR v3 nodes will be live soon?
On Sun, Feb 17, 2019, 16:43 Marvin Piekarek<EMAIL_ADDRESS>wrote:

[image: image]
https://user-images.githubusercontent.com/7430964/52912125-577c3680-32ad-11e9-9afb-b62dc2b0beb4.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vergecurrency/VERGE/pull/872#issuecomment-464443348,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjzlvMhwSU2tRewv3IR_QD2HYoGwzhAaks5vOTk-gaJpZM4a_fMj
.

-Remove my laziness
Hmm will add it to my to-do list 😅
On Sun, Feb 17, 2019, 16:43 Marvin Piekarek<EMAIL_ADDRESS>wrote:

[image: image]
https://user-images.githubusercontent.com/7430964/52912125-577c3680-32ad-11e9-9afb-b62dc2b0beb4.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vergecurrency/VERGE/pull/872#issuecomment-464443348,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjzlvMhwSU2tRewv3IR_QD2HYoGwzhAaks5vOTk-gaJpZM4a_fMj
.

Well @karanpatel863, once 5.0.0 is released we will by default start v3 hidden services since they are more secure to most network based attacks.

@karanpatel863 there are some v3 nodes up already but this commit allows them from within

also I'll be pushing the patch to remove rust shortly

Awesome. Will wait for 5.0 to try TOR v3 default nodes by myself.
On Sun, Feb 17, 2019, 19:06 sunerok<EMAIL_ADDRESS>wrote:

@karanpatel863 https://github.com/karanpatel863 there are some v3 nodes
up already but this commit allows them from within
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vergecurrency/VERGE/pull/872#issuecomment-464457968,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjzlvK2NxHjctuLocpBrO9RwA5XtSeBlks5vOVrLgaJpZM4a_fMj
.

In the meantime justin and marpme please have a look at dandelion++
researched by giulia fanti & team. Dandelion++ is a lightweight and
straightforward network layer solution with formally guaranteed anonymity
that can easily be implemented with existing cryptocurrencies.
Currently it is being considered to be added via BIP along with taproot in
bitcoin.
I am attaching arxiv research papers containing dandelion and then updated
dandelion++. Kindly have a look & you will find it interesting.
On Sun, Feb 17, 2019, 19:07 sunerok<EMAIL_ADDRESS>wrote:

also I'll be pushing the patch to remove rust shortly
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/vergecurrency/VERGE/pull/872#issuecomment-464458115,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AjzlvN4ZVoSLOEdUvmyA-qfjRt-tuCGbks5vOVsJgaJpZM4a_fMj
.

1701.04439(1).PDF
1805.11060.pdf


Improve assertMacroExpansion highlight verification

Adds a new initializer to DiagnosticSpec which takes an optional array of highlights instead of a single optional highlight string. Updates existing initializer to map the string into an array of one.
Updates assertMacroExpansion to check each highlight in the diagnostic spec against the actual produced diagnostic individually. Additionally changes the behavior to drop leading and trailing trivial when performing the highlight comparison to better match the Swift compiler's behavior.

This is behavior breaking iff:

you use explicit highlight checking (it seems like most users don't) and

your explicit highlights had trailing space to match the misreported highlights from swift-syntax or
your diagnostics emit multiple highlights, which now need to be passed as an array instead of string with odd concatenation

IMO this should be considered a bug fix because the current highlight checking doesn't actually allow the user to be confident in their highlight assertion.

@swift-ci please test

@swift-ci Please test Windows

The linux error seems unrelated to me:
error: multiple targets named '_InstructionCounter' in: 'swift-format', 'swift-syntax'; consider using the `moduleAliases` parameter in manifest to provide unique names

Yes, fixed by https://github.com/apple/swift-format/pull/632.

@swift-ci Please test

@swift-ci please test

@ahoppen could you take another look when you have some time?

@swift-ci please test

@swift-ci please test


Add full LLVM 18 support

Some methods and enums got renamed from LLVM 17 -> 18, etc.
Also some adjustment to warning messages about version mixing between clang and llvm.

There are some CI failures that are unrelated to this patch, I'm addressing those separately.
It's the Macos-14 CI that is building against LLVM 18.1 to test this.
Macos-13 would be too, but it is failing for entirely different reasons -- because building OIIO is failing with clang 18. That's already fixed and this failure will go away when we push out a new "release" branch on May 1.


fix: The extract method logs output only when self.verbose is set to True

@lu4nx Thanks for the contribution. Your full request has been accepted and has been merged.


cannot detect and python interpreters

Environment data

VS Code version: 1.33.1
Extension version (available under the Extensions sidebar): 2019.4.11881
OS and version: macOS Mojave 10.14.4
Python version (& distribution if applicable, e.g. Anaconda):
Python 3.7.3 (default, Mar 27 2019, 09:23:15)
[Clang 10.0.1 (clang-1<IP_ADDRESS>)] on darwin
3.7 installed via brew, but same issue with 3.6 or 3.7 also installed from python.org and including virtual environments
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Relevant/affected Python packages and their versions: Core python

Expected behaviour
installed python interpreter(venv or not) should be detected
Actual behaviour
no python is detected at all and putting the pythonPath or venvPath in settings.json does not allow the interpreter to be detected
Steps to reproduce:

Install Python 3.6 or 3.7 via brew or dmg from python.org
Create a venv(or not) with python -m venv or virtaulenv
Install VS Code
Install python extension
Try to set the interpreter(none is found and 'hardcoding' it does not seem to have any effect)

Logs
Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)
None.

Output from Console under the Developer Tools panel (toggle Developer Tools on under Help; turn on source maps to make any tracebacks be useful by running Enable source map support for extension debugging)
Python Extension: Failed to parse interpreter information for '/Users/XXXX/.venv/bin/python' with JSON Invalid -W option ignored: invalid action: '"once'
{"versionInfo": [3, 7, 3, "final"], "sysPrefix": "/Users/XXXX/Projects/YYYY/.venv", "version": "3.7.3 (default, Mar 27 2019, 09:23:15) \n[Clang 10.0.1 (clang-1<IP_ADDRESS>)]", "is64Bit": true}

SyntaxError: Unexpected token I in JSON at position 0
	at JSON.parse (<anonymous>)
	at p.getInterpreterInformation (/Users/XXXX/.vscode/extensions/ms-python.python-2019.4.11881/out/client/extension.js:83:293720)

t.log @ console.ts:134

There are similar errors for all installed versions of python(system, virtualenvs, gcc compiled, clang compiled, 2.7.X, 3.6.X, 3.7X)

This has been resolved in the latest update, please update your extension.
Duplicate of #5437

I updated to 11987 and I have the exact same behavior.
From the logs, #5437 seems to be a different issue.

While i look at the issue, can you please try rollbacking to the previous version as a temporary solution and see if that fixes the issue.

You can right-click the Python extension in the extension tab and select a different version to install

Hmm... well perhaps it is fixed.
Here's the sequence of events I went through, in case you care:
prelude) I had previously used VS Code w/ Python(unknown extension version) without issue, but the hard drive failed.

Setting up a new instance with latest version as of two days ago(11881 maybe), which is where I first discovered it. Thought it was my issue so I wiped everything relating to python and started over.
Definitely an extension bug as initially filed(11881).
Updated to 11987, reloaded, no change
back to 3.6558, reloaded, no change,
back to 6352, reloaded, no change,
back to 2019.1.0, reloaded, fixed!
Then you asked if a rollback would fix. I check the installed version and it's 11987 which I definitely did not intend to do.

So I'm not sure what happened when I initially updated to 11987 or how 2019.1.0 automatically jumped back to 11987, but it's working now.
Sorry for the extra work. If I run into the issue again, I'll get back to you.

No problem. Can you please answer a few things,

Updated to 11987, reloaded, no change

What steps did you use to initially update to 11987? What steps did you use to rollback to 6352?

back to 6352, reloaded, no change,

When you were rollbacking to previous version, did you check the installed version? Was it the one you intended?

No problem. Can you please answer a few things,

Updated to 11987, reloaded, no change

What steps did you use to initially update to 11987? What steps did you use to rollback to 6352?
View->Extensions->Python->Gear icon->"Install another version" for both.

back to 6352, reloaded, no change,

When you were rollbacking to previous version, did you check the installed version? Was it the one you intended?

I did not really check in between, I guess. I naively assumed no errors meant it went through without a problem.
I also realize I don't know Code's extension updating policy default. I usually use PyCharm, but it's os heavy, I've been trying out Code. Does VS Code automatically update extensions to the latest version on a restart/reload? That's probably why rolling back didn't fix the issue... but not really why when I first went to 11987, it didn't seem to fix things.
I suspect PEBKAC.

Does VS Code automatically update extensions to the latest version on a restart/reload?

Yes, but users can disable it with a setting. See this.
Note even if you rollback, you will have to reload the window to use that version. Maybe you didn't reload, due to which you were not able to use the latest version in the first place.

...and it's back to not working.
It was working before lunch(working all yesterday afternoon and this morning) but when I came back  now it's saying to select an interpreter. Although this time at least I can see all of the expected interpreters, but selecting them has no real effect
You need to select a Python interpreter before you start debugging. Tip: click on "Select Python Interpreter" in the status bar.
and the same error messages in Developer tools:
Python Extension: Failed to parse interpreter information for '/Users/XXX/PycharmProjects/TraceUpdater/venv/bin/python' with JSON Invalid -W option ignored: invalid action: '"once' {"versionInfo": [3, 7, 3, "final"], "sysPrefix": "/Users/XXX/PycharmProjects/TraceUpdater/venv", "version": "3.7.3 (default, Mar 27 2019, 09:23:15) \n[Clang 10.0.1 (clang-1<IP_ADDRESS>)]", "is64Bit": true} SyntaxError: Unexpected token I in JSON at position 0 	at JSON.parse (<anonymous>) 	at p.getInterpreterInformation (/Users/XXX/.vscode/extensions/ms-python.python-2019.4.11987/out/client/extension.js:83:293720)
It's Version: 2019.4.11987
As far as I know, nothing changed on this system between leaving for lunch and coming back(...and yet...)

...and it's back to not working.
It was working before lunch(working all yesterday afternoon and this morning) but when I came back now it's saying to select an interpreter. Although this time at least I can see all of the expected interpreters, but selecting them has no real effect
You need to select a Python interpreter before you start debugging. Tip: click on "Select Python Interpreter" in the status bar.
and the same error messages in Developer tools:
Python Extension: Failed to parse interpreter information for '/Users/XXX/PycharmProjects/TraceUpdater/venv/bin/python' with JSON Invalid -W option ignored: invalid action: '"once' {"versionInfo": [3, 7, 3, "final"], "sysPrefix": "/Users/XXX/PycharmProjects/TraceUpdater/venv", "version": "3.7.3 (default, Mar 27 2019, 09:23:15) \n[Clang 10.0.1 (clang-1<IP_ADDRESS>)]", "is64Bit": true} SyntaxError: Unexpected token I in JSON at position 0 at JSON.parse (<anonymous>) at p.getInterpreterInformation (/Users/XXX/.vscode/extensions/ms-python.python-2019.4.11987/out/client/extension.js:83:293720)
It's Version: 2019.4.11987
As far as I know, nothing changed on this system between leaving for lunch and coming back(...and yet...)

Does reinstalling and reloading the windows help?
Unfortunately i can't reproduce this flaky behavior. Can you pleasing try recreating the virtual environment/ selecting a different interpreter?

Reinstalling/reloading doesn't help.
I didn't re-create the venv, but I tried the system installed interpreters(apple and brew) and other venvs (mostly from the brew install) and they all give the same result. All the interpreters seem to work fine in PyCharm and from the command line.
What is the core error? is the "Unexpected token I in JSON..." bit or is it the "invalid action 'once'"? and what is the command(I assume it's a command line) that it's trying to run?

This is the error block
try {
    json = JSON.parse(jsonValue);
} catch (ex) {
    traceError(`Failed to parse interpreter information for '${this.pythonPath}' with JSON ${jsonValue}`, ex);
    return;
}

Error equals SyntaxError: Unexpected token I in JSON at position 0 at JSON.parse, because the json value it's trying to parse is,
Invalid -W option ignored: invalid action: '"once' {"versionInfo": [3, 7, 3, "final"], "sysPrefix": "/Users/XXX/PycharmProjects/TraceUpdater/venv", "version": "3.7.3 (default, Mar 27 2019, 09:23:15) \n[Clang 10.0.1 (clang-1<IP_ADDRESS>)]", "is64Bit": true}
jsonValue here is the output of the running the file,
import json
import sys

obj = {}
obj["versionInfo"] = sys.version_info[:4]
obj["sysPrefix"] = sys.prefix
obj["version"] = sys.version
obj["is64Bit"] = sys.maxsize > 2**32

print(json.dumps(obj))

Ideally the output should be,
{"versionInfo": [3, 7, 3, "final"], "sysPrefix": "/Users/XXX/PycharmProjects/TraceUpdater/venv", "version": "3.7.3 (default, Mar 27 2019, 09:23:15) \n[Clang 10.0.1 (clang-1<IP_ADDRESS>)]", "is64Bit": true}

Well this works without reinstalling json:
➜  XXXX git:(branch) ✗ . ./venv/bin/activate
(venv) ➜  XXXX git:(branch) ✗ which python
/Users/XXXX/venv/bin/python
(venv) ➜  XXXX git:(branch) ✗ python
Python 3.7.3 (default, Mar 27 2019, 09:23:15)
[Clang 10.0.1 (clang-1<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.`
>>> import json
>>> import sys
>>>
>>> obj = {}
>>> obj["versionInfo"] = sys.version_info[:4]
>>> obj["sysPrefix"] = sys.prefix
>>> obj["version"] = sys.version
>>> obj["is64Bit"] = sys.maxsize > 2**32
>>>
>>> print(json.dumps(obj))
{"versionInfo": [3, 7, 3, "final"], "sysPrefix": "/Users/XXXX/venv", "version": "3.7.3 (default, Mar 27 2019, 09:23:15) \n[Clang 10.0.1 (clang-1<IP_ADDRESS>)]", "is64Bit": true}

I'm not sure how to go about reinstalling json... isn't that just part of the python std libs? Unless you mean reinstall python from scratch.
I'll try a few more things next week, but I suspect that this is just my own issue, not a tool bug.  Thanks for the help!

Anytime.
I am not sure if it's the part of std libs, best way to go about it is reinstalling python from scratch (maybe trying a different python version), and installing json if it's not available.
I suspect it's a PEBKAC as well.

Well, I completely formatted my HD, reloaded Mojave, re-installed python3 with brew(brew install python), created a new venv python -m venv ./venv and...
exact same issue!

@mrbell321
Please could run the following in the terminal and provide the output:
/Users/mrbell/PycharmProjects/TraceUpdater/.venv/bin/python /Users/XXXX/.vscode/extensions/ms-python.python-2019.4.11881/pythonFiles/interpreterInfo.py
Note: The version number could be different (assuming the extension has auto updated).

Slightly different paths on the new system so:
(venv) ➜  which python /XXXX/venv/bin/python (venv) ➜  python ~/.vscode/extensions/ms-python.python-2019.4.11987/pythonFiles/interpreterInfo.py  {"versionInfo": [3, 7, 3, "final"], "sysPrefix": "/XXXX/venv", "version": "3.7.3 (default, Mar 27 2019, 09:23:15) \n[Clang 10.0.1 (clang-1<IP_ADDRESS>)]", "is64Bit": true}
and I don't have anything defined for PYTHONWARNINGS

I installed Python 3.7.3 from here https://www.python.org/downloads/ and I get the same error for the version installed in Frameworks.

I installed Python 3.7.3 from here https://www.python.org/downloads/ and I get the same error for the version installed in Frameworks.

Did you try installing a different version of Python? The reason i am asking this is to make sure that this is not an issue we face with 3.7.3.

Same problem with 3.6.8.
Python Extension: Failed to parse interpreter information for '/XXX/PROJECT/venv/bin/python' with JSON Invalid -W option ignored: invalid action: '"once'
{"versionInfo": [3, 6, 8, "final"], "sysPrefix": "/XXX/PROJECT/venv", "version": "3.6.8 (v3.6.8:3c6b436a57, Dec 24 2018, 02:04:31) \n[GCC 4.2.1 Compatible Apple LLVM 6.0 (clang-600.0.57)]", "is64Bit": true} SyntaxError: Unexpected token I in JSON at position 0
	at JSON.parse (<anonymous>)
	at p.getInterpreterInformation (/XXX/.vscode/extensions/ms-python.python-2019.4.12954/out/client/extension.js:83:293720)

But if you look closely, that's reporting 3.6.8 in my venv. I didn't create a 3.6.8 venv, I just wanted to know if the vscode would pick up the system installed 3.6.8 interpreter.
Apparently a difference between the venv module and virualenv is that venv uses symlinks by default and virtualenv uses copies. I've been doing python for years and I've never noticed before(although, I guess the venv module is relatively new).
So I deleted the existing venv-module-created-venv-folder and suddenly vscode can see all of the interpreters on the system. I created a new venv folder with virtualenv and vscode can also see all of the interpreters that way.
I recreated the venv folder with the venv module and all of the interpreters are gone again.
It seems there's a bug w/ vscode-python and the venv module which manages on break ALL interpreters.

Good observation. Can you please clarify the minimal steps to reproduce this for macOS?
Doe just creating a venv folder with the venv module breaks the interpreter search for you?

Hmm... to get a clean set of minimal steps, I cannot reproduce the issue entirely. Instead, only the venv(symlink) interpreter reports the error in developer tools, but the others show up as cached and are still selectable and useable.  This is how I tested that:

Create a new project folder.
Opened project folder in vscode
Cmd-P > 'Select Python Interpreter'

/usr/bin (2.7, OSX default)
/Library/Frameworks/... (3.7, the version I installed, I think, via the dmg)
/usr/local/bin (3.7, the version installed via brew, I'm pretty sure)

Create a venv folder with python3 -m venv venv
Restart vscode
Cmd-P > 'Select Python Interpreter'
Result -> All system installed interpreters are available but not the venv.
The venv/bin/python reports the error from earlier in Dev Tools
Delete and re-create venv with virtualenv -p python3 venv
Restart vscode
Cmd-P > 'Select Python Interpreter'
Result -> System Interpreters remain available AND the venv interpreter is now available.
Delete and re-create venv with -m venv
Restart vscode
Cmd-P > 'Select Python Interpreter'
Result -> System Interpreters remain available AND the venv interpreter is gone again with the error in Dev Tools.

Last week, approximately the same process would lose all interpreters when using -m venv so I'll have to do some digging to find out what changed between then and now...

@mrbell321
I'm sorry for the inconvenience this issue has caused and the time taken to get to the bottom of this.
I'd like to try something, something we probably should have done early on:

Open the file /Users/***/.vscode/extensions/ms-python.python-2019.***/pythonFiles/interpreterInfo.py
Edit the file (contents provided further below)
Run VS Code, and when it fails, then get the contents of the log.log file and upload it here.

Here's what I'd like to get out of this:

Why is the code failing
Have a look at any arguments passed to the above python file
Have a look at the environment variables when the python file is running
Dump all exceptions

** New contents for interpreterInfo.py:
# Copyright (c) Microsoft Corporation. All rights reserved.
# Licensed under the MIT License.

import json
import os
import sys
import traceback

obj = {}
obj["versionInfo"] = sys.version_info[:4]
obj["sysPrefix"] = sys.prefix
obj["version"] = sys.version
obj["is64Bit"] = sys.maxsize > 2 ** 32

log_file = os.path.join(os.path.dirname(__file__), "log.log")
with open(log_file, "a") as fp:
    try:
        fp.write(str(sys.argv))
        fp.write(os.linesep)
    except Exception:
        fp.write("1")
        fp.write(os.linesep)
        traceback.print_exc(file=fp)
    try:
        fp.write(json.dumps(obj))
        fp.write(os.linesep)
    except Exception:
        fp.write("2")
        fp.write(os.linesep)
        traceback.print_exc(file=fp)

    try:
        fp.write(json.dumps(dict(os.environ)))
        fp.write(os.linesep)
    except Exception:
        fp.write("3")
        fp.write(os.linesep)
        traceback.print_exc(file=fp)

print(json.dumps(obj))

Please review the environment variables in the log.log file before uploading the contents

So I ran this twice, this is the 2nd result(I accidentally overwrote the first try). Interestingly(or irritatingly) the results I saw were different. Here was my workflow:

Launch vscode to see if there was an update
close vscode
Update interpreterInfo.py as per previous post
Launch vscode
Select interpreter.
The first run showed the system installed interpreters and even a virtualenv from another project, but not the currently open folder.
Copy log.log to my desktop for archival purposes.

I then deleted the ~/.vscode/extentions/ms-python.../log.log and ran thru the process a second time.
That time, no interpreter was found at all.
My downfall was that I copied the file again, overwriting the original. D'oh!
Anyway, this is the log that I did manage to capture:
log.log
I'll see if I can make it behave the other way again...

Thanks a lot for cooperating. However looking at the log,

The code seems to run just fine, for .venv as well, which should imply that you should not be getting the earlier error.

['~/.vscode/extensions/ms-python.python-2019.4.12954/pythonFiles/interpreterInfo.py']
{"versionInfo": [3, 7, 3, "final"], "sysPrefix": "~/code/myProj/.venv/bin/..", "version": "3.7.3 (v3.7.3:ef4ec6ed12, Mar 25 2019, 16:52:21) \n[Clang 6.0 (clang-600.0.57)]", "is64Bit": true}
{"PYTHONPATH": "$PTYHONPATH:", "PYTHONWARNINGS": "\"once", "TMPDIR": "/var/folders/3m/6s58jygj3jvbp4fvvw8w8hfh0000gn/T/", "__CF_USER_TEXT_ENCODING": "0x1F5:0x0:0x0", "SHELL": "/bin/zsh", "HOME": "~", "Apple_PubSub_Socket_Render": "/private/tmp/com.apple.launchd.mgXUBGPOCl/Render", "SSH_AUTH_SOCK": "/private/tmp/com.apple.launchd.CWFwMMcgzl/Listeners", "LOGNAME": "dumbguy", "XPC_SERVICE_NAME": "com.microsoft.VSCode.2256", "USER": "dumbguy", "XPC_FLAGS": "0x0", "VSCODE_NLS_CONFIG": "{\"locale\":\"en-us\",\"availableLanguages\":{},\"_languagePackSupport\":true}", "VSCODE_NODE_CACHED_DATA_DIR": "~/Library/Application Support/Code/CachedData/51b0b28134d51361cf996d2f0a1c698247aeabd8", "VSCODE_LOGS": "~/Library/Application Support/Code/logs/20190513T154742", "VSCODE_IPC_HOOK": "~/Library/Application Support/Code/1.33.1-main.sock", "VSCODE_PID": "47580", "LC_CTYPE": "UTF-8", "LESS": "-R", "LSCOLORS": "Gxfxcxdxbxegedabagacad", "OLDPWD": "/", "PAGER": "less", "PWD": "/", "SHLVL": "0", "ZSH": "~/.oh-my-zsh", "_": "/Applications/Visual Studio Code.app/Contents/MacOS/Electron", "AMD_ENTRYPOINT": "vs/workbench/services/extensions/node/extensionHostProcess", "PIPE_LOGGING": "true", "VERBOSE_LOGGING": "true", "VSCODE_IPC_HOOK_EXTHOST": "/var/folders/3m/6s58jygj3jvbp4fvvw8w8hfh0000gn/T/vscode-ipc-004f15a6-995a-4dde-b10e-161aa78a92a6.sock", "VSCODE_HANDLES_UNCAUGHT_ERRORS": "true", "VSCODE_LOG_STACK": "false", "VSCODE_PREVENT_FOREIGN_INSPECT": "true", "ELECTRON_RUN_AS_NODE": "1", "APPLICATION_INSIGHTS_NO_DIAGNOSTIC_CHANNEL": "true", "PATH": "/Library/Frameworks/Python.framework/Versions/3.6/bin:/Library/Frameworks/Python.framework/Versions/3.7/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin", "PYTHONUNBUFFERED": "1", "PYTHONIOENCODING": "utf-8"}

Can you please confirm that you see the earlier error, and please provide the log file just after you see this error,
Python Extension: Failed to parse interpreter information for '/Users/XXXX/venv/bin/python' with JSON Invalid -W option ignored: invalid action: '"once' {"versionInfo": [3, 7, 3, "final"], "sysPrefix": "/Users/XXXX/venv", "version": "3.7.3 (default, Mar 27 2019, 09:23:15) \n[Clang 10.0.1 (clang-1<IP_ADDRESS>)]", "is64Bit": true} SyntaxError: Unexpected token I in JSON at position 0 at JSON.parse (<anonymous>) at p.getInterpreterInformation (/Users/XXX/.vscode/extensions/ms-python.python-2019.4.11987/out/client/extension.js:83:293720)

** Please try this as the new content for interpreterInfo.py:
# Copyright (c) Microsoft Corporation. All rights reserved.
# Licensed under the MIT License.

import json
import os
import sys
import traceback

obj = {}
obj["versionInfo"] = sys.version_info[:4]
obj["sysPrefix"] = sys.prefix
obj["version"] = sys.version
obj["is64Bit"] = sys.maxsize > 2 ** 32

log_file = os.path.join(os.path.dirname(__file__), "log.log")
with open(log_file, "a") as fp:
    try:
        fp.write(str(sys.argv))
        fp.write(os.linesep)
    except Exception:
        fp.write("1")
        fp.write(os.linesep)
        traceback.print_exc(file=fp)
    try:
        a = json.dumps(obj)
        fp.write(a)
        fp.write(os.linesep)
    except Exception:
        fp.write("2")
        fp.write(os.linesep)
        traceback.print_exc(file=fp)

    try:
        fp.write(json.dumps(dict(os.environ)))
        fp.write(os.linesep)
    except Exception:
        fp.write("3")
        fp.write(os.linesep)
        traceback.print_exc(file=fp)

    try:
        print(a)
    except Exception:
        fp.write("4")
        fp.write(os.linesep)
        traceback.print_exc(file=fp)

print(a)

Closing as no further response was provided.


openzwave & pythonPackages.python_openzwave: init at 20180404 and 0.4.4

Motivation for this change
OpenZWave can be used with a wide range of USB-Sticks used to controll Z-Wave devices.
Python_OpenZWave can be used with homeassistant that is already packaged to be able to control and script Z-Wave devices.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

cc @dotlambda @f-breidenstein maybe?

@dotlambda /etc/openzwave contains a bunch of XML files that are configuration for OpenZWave. It's needed to be able to use OpenZWave at all. All utilities and libs in all languages depends on it.

@dotlambda As for your other question:

Did you test that this actually works with Home Assistant?

Yes, I have it running here on my desktop as well as a raspberry pi. I had to change one little thing in the home-assistant derivation (for some depenedency), and had to add some configuration myself to the config file.
Diff of hame-assistant:
diff --git a/pkgs/servers/home-assistant/default.nix b/pkgs/servers/home-assistant/default.nix
index 75f8b309a40..e1814317463 100644
--- a/pkgs/servers/home-assistant/default.nix
+++ b/pkgs/servers/home-assistant/default.nix
@@ -72,6 +72,7 @@ in with py.pkgs; buildPythonApplication rec {
   propagatedBuildInputs = [
     # From setup.py
     requests pyyaml pytz pip jinja2 voluptuous typing aiohttp async-timeout astral certifi attrs
+    pydispatcher
     # From http, frontend and recorder components
     sqlalchemy aiohttp-cors hass-frontend
   ] ++ componentBuildInputs ++ extraBuildInputs;

configuration:
  # Enable Home Assistant, open port and add the hass user to the dialout group
  services.home-assistant.enable = true;
  services.home-assistant.package = pkgs.home-assistant.override {
    extraPackages = ps: [ pkgs.python36Packages.python_openzwave ];
  };

  networking.firewall.allowedTCPPorts = [ 8123 ];
  users.extraUsers.hass.extraGroups = [ "dialout" ];

Yaml for home-assistant:
zwave:
  usb_path: /dev/serial/by-id/usb-0658_0200-if00
  config_path: /nix/store/dlgdch7iwdd6ky4z1b4wpzrn1mg8w4i5-openzwave-2018-04-04/etc/openzwave/

extraPackages = ps: [ pkgs.python36Packages.python_openzwave ];

That shoud be extraPackages = ps: with ps; [ python_openzwave pydispatcher ];.
However, after running pkgs/servers/home-assistant/parse-requirements.py, these dependencies will be added automatically when using the services.home-assistant.config option or extraComponents = [ "zwave" ].

You need to copy exactly that: extraPackages = ps: with ps; [ python_openzwave pydispatcher ];

Oh, yeah, that did work :)

How about so?
  installFlags = [
    "PREFIX="
    "pkgconfigdir=lib/pkgconfig"
    "sysconfigdir=etc/openzwave"
  ];

  preInstall = ''
    installFlagsArray+=("DESTDIR=$out")
  '';

@etu Please also try fixing the Fontconfig error: Cannot load default config file errors.
Maybe @peterhoeg or @FRidh want to have a look at this? I'm certainly not an expert when it comes to C libraries.

@etu I added a commit which should fix extraComponents = [ "zwave" ];.

@dotlambda Re fontconfig: Here I have a build log of openzwave and python_openzwave: https://gist.github.com/etu/6bdba2f432a839566e6806b5d72e3773 -- which warnings do you refer to?

@dotlambda
I've copied the files of that commit to my own nixpkgs and tried to change the config to:
  # Enable Home Assistant, open port and add the hass user to the dialout group
  services.home-assistant.enable = true;
  services.home-assistant.package = pkgs.home-assistant.override {
    extraComponents = [ "zwave" ];
  };

  networking.firewall.allowedTCPPorts = [ 8123 ];
  users.extraUsers.hass.extraGroups = [ "dialout" ];

That resulted in a python error log from home-assistant that complaints about a missing python module named openzwave. So I'm not sure I trust extraComponents at all :p

Maybe that error only appears when enabling sandboxing.
log.txt

@dotlambda
This patch does get rid of the fontconfig warnings, but I'm not sure where to put the environment variables because preInstall feels like the wrong place.
diff --git a/pkgs/development/libraries/openzwave/default.nix b/pkgs/development/libraries/openzwave/default.nix
index 0d5feae2419..3c18849f1c5 100644
--- a/pkgs/development/libraries/openzwave/default.nix
+++ b/pkgs/development/libraries/openzwave/default.nix
@@ -1,5 +1,5 @@
 { stdenv, fetchFromGitHub
-, doxygen, libxml2, git, which, pkgconfig, graphviz-nox
+, doxygen, libxml2, git, which, pkgconfig, graphviz-nox, fontconfig
 , udev }:

 let
@@ -15,7 +15,7 @@ in stdenv.mkDerivation rec {
     sha256 = "0yby8ygzjn5zp5vhysxaadbzysqanwd2zakz379299qs454pr2h9";
   };

-  nativeBuildInputs = [ doxygen libxml2 git which pkgconfig graphviz-nox ];
+  nativeBuildInputs = [ doxygen libxml2 git which pkgconfig graphviz-nox fontconfig ];
   buildInputs = [ udev ];

   installFlags = [
@@ -26,6 +26,9 @@ in stdenv.mkDerivation rec {

   preInstall = ''
     installFlagsArray+=("DESTDIR=$out")
+
+    export FONTCONFIG_FILE=${fontconfig.out}/etc/fonts/fonts.conf
+    export FONTCONFIG_PATH=${fontconfig.out}/etc/fonts/
   '';

   postPatch = ''

I will try your change.

You can make these a part of the Nix expression, i.e.
preInstall = ''
  installFlagsArray+=("DESTDIR=$out")'
'';

FONTCONFIG_FILE = "${fontconfig.out}/etc/fonts/fonts.conf";
FONTCONFIG_PATH = "${fontconfig.out}/etc/fonts/";

@dotlambda Oh, ok. Rebased and pushed.
I've tested that change in home-assistant/default.nix and no such luck. Now it's missing pydispatcher instead :confused:

@dotlambda I've just confirmed that my patch here: https://github.com/NixOS/nixpkgs/pull/38577/files#diff-588a05cff755242c814f80436a1184faR23
Works properly in home-assistant. So one doesn't need to configure that. That makes everything better I'd say :smile:

Sorry about being late to the party. Please don't merge this yet, I have a few comments.

@peterhoeg Oh, nice. I'm gonna test through it all tonight. You can push this to my branch if you want :)

Pushed. You probably want to squash those commits into the relevant commits on your side.

@peterhoeg git is used by openzwave build to find version-number from it's git-repo. So without it I got a warning that it doesn't exist. I don't think it matters very much for the actual usage though.

Correct, you get the warning but $src is not a checked out git repo but simply a directory, so it cannot do anything there anyway.

@peterhoeg I've rebased and squashed your changed. And I'll fixed some dependency issues for the CLI tools for the python library and have pushed it. I hope it looks good now :)

Looks good. @dotlambda/@fridh do you want this to go via staging?

I manually checked that it won't create merge conflict, so I guess merging into master is fine.

Thanks a lot @etu!


Remove unused, conflicting dependencies

Thanks for this example project. A few things I noticed when setting this up myself:

Mocha & Chai are included in the deps, though they are not used anywhere. Rather, Jasmine is used as the test framework.
Karma Babel preprocessor is included as a dep and the settings block is in your karma config file, but the preprocessor isn't actually used anywhere. It's a little confusing for someone who is also trying to get this configured w/ Babel. I think by removing the unused dep, it will be more obvious that the webpack preprocessor is a way to transform JS w/ Babel.
There is a .setup.js file that I believe is not needed for this example project. It looks like it should be used in combination w/ JSDom given this documentation which I don't believe is necessary here since Karma has both the Chrome & Phantom JS launchers (and JSDom isn't even in the package.json file.

There are actually more dependencies listed in the package.json file which aren't used in this repo though I know their use case if there was actually a webpack config, so maybe it makes sense to also either remove those or add a webpack config which utilizes them.
So this PR cleans that up a bit. Let me know your thoughts.

Do the tests still pass without the .setup.js?  I think the load of it is hidden away a bit in karma.conf.js.  Perhaps we need simple travis build on this proj that run npm test =)

Yep, sure do. Tried both PhantomJS & Chrome.
I'll remove the browserify deps as well.

Done. Not sure I have permissions to setup travis for this repo otherwise I would help.

Bump. Any additional feedback here?

I suppose this can be closed ;)


[BUG] missing dependency "packaging"

Describe the bug
After fresh install - there seems to be a missing dependency on packaging.
 File "/csse/users/yya164/Documents/taichi-splatting/taichi_splatting/data_types.py", line 4, in <module>
    from tensordict import tensorclass
  File "/local/miniforge3/envs/torch/lib/python3.11/site-packages/tensordict/__init__.py", line 6, in <module>
    from tensordict._lazy import LazyStackedTensorDict
  File "/local/miniforge3/envs/torch/lib/python3.11/site-packages/tensordict/_lazy.py", line 25, in <module>
    from tensordict._td import _SubTensorDict, _TensorDictKeysView, TensorDict
  File "/local/miniforge3/envs/torch/lib/python3.11/site-packages/tensordict/_td.py", line 25, in <module>
    from tensordict.base import (
  File "/local/miniforge3/envs/torch/lib/python3.11/site-packages/tensordict/base.py", line 40, in <module>
    from tensordict.utils import (
  File "/local/miniforge3/envs/torch/lib/python3.11/site-packages/tensordict/utils.py", line 44, in <module>
    from packaging.version import parse

Thanks for reporting! I'll add packaging to the tensordict deps!


Add more events and send methods, some fixes

Will add several more I think and then it will be good to go
On a separate note, on every deprecation message we need to write something "will be removed in version 1.3" or smth. It's bad to keep a spaghetti of old and new code in the same plate.

Nice work!
IMO uid's should be integers, and not strings.

Well I think in this way - are they manipulated in a mathematical way? We never need to see if one ID should be bigger or smaller. So they should be strings. Furthermore, FB  mostly sends them as strings, not ints.


Clarify that pause happens with any replica count

Based on the discussion in the issue  I think that pause happens no matter of "paused-replicas" value?
https://github.com/kedacore/keda/issues/944
When I read it I was not sure what happens if value is >0
@zroubalik @aryan9600 @tomkerkhove @JorTurFer

Provide a description of what has been changed
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

Fixes #

Can you fix DCO please?

@tomkerkhove I did steps form docs for DCO hopefully it worked https://github.com/kedacore/keda/blob/main/CONTRIBUTING.md#i-didnt-sign-my-commit-now-what

It certainly did, thank you!


[client] Issue 3218: Support specifying multiple hosts in pulsar service url and web url

Motivation
Sometimes people doesn't have DNS for brokers. It might be good for people to specify a list of brokers as bootstrap brokers.
Changes
This PR introduce ServiceURI and ServiceNameResolver for supporting specifying multiple hosts in pulsar service url and
web url.
Master Issue: #3218

retest this please

run java8 tests

@jiazhai good point. I will add documentation later. because currently this feature only available in pulsar client (through http and binary lookup). However there are a few more changes in the pipeline:

make pulsar cli tool works with multiple hosts. since it is jersey based, I haven't figured out how to do that.

I will probably document it only after all changes are in. otherwise it will cause confusion in the documentation.

@merlimat @codelipenghui can you guys help review this?

@BewareMyPower  : can you please thorw some light on whether this change supports broker URLs from same cluster only OR it can support broker URLs form multiple clusters

It can only support broker URLs from the same cluster.


How get status_code

Thank, all work fine, but how i can get status_code on my request? The responce.status_code dont use in code. Can you help me plz.

I found colution. Need use function HttpQueryInfoA (wininet.h) with HTTP_QUERY_STATUS_CODE and conесе it to responce.status_code. Now all work good and can to use status code of answer.

I found colution. Need use function HttpQueryInfoA (wininet.h) with HTTP_QUERY_STATUS_CODE and conесе it to responce.status_code. Now all work good and can to use status code of answer.


Problem with showing fa-ir language characters in keywords

return SEO::analyze('https://www.lahzino.ir')  returns:
Malformed UTF-8 characters, possibly incorrectly encoded.
dd(SEO::analyze('https://www.lahzino.ir')) :
"keywords" => array:[       b"ø" => 410       b"ù" => 245       b"ûœ" => 35       b"ûœø" => 25       b"øªø" => 23       b"ú" => 20       b"ûœù" => 18       b"øªù" => 18]

I will look into this asap.

I will look into this asap.

thanks, I checked source code, there is problem with str_word_count and strtolower functions in findKeywords method. persian or Arabic language hasn't got uppercase or lowercase characters.
mb_str_word_count instead of str_word_count

I will look into this asap.

thanks, I checked source code, there is a problem with str_word_count and strtolower functions in findKeywords method. Persian or Arabic language hasn't got uppercase or lowercase characters.
mb_str_word_count instead of str_word_count.

Thank you for your pull request.


chore(deps): update terraform-ibm-modules/common-pipeline-assets action to v1.22.4

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/common-pipeline-assets
action
patch
v1.22.3 -> v1.22.4

Release Notes

terraform-ibm-modules/common-pipeline-assets (terraform-ibm-modules/common-pipeline-assets)
v1.22.4
Compare Source
Bug Fixes

added support for trailing spaces in pipeline run command (#699) (215fa1a), closes #701

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

:tada: This issue has been resolved in version 1.35.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add Project Types to Settings, Add .nuproj to Default Project Types

Thanks for the pull request! I apologise for how it's taken me to look at this, better late than never I guess. I'll merge this in now and get a new package issued over the weekend after I've tested it - looks like there's a couple of minor issues but that's it.


Include WebClient.Builder in HTTP Client Integration docs

Added WebClient.Builder as a supported bean
Updated verbiage since a WebClient can only be created by using one of the various factory methods and a WebClient.Builder can be created either by new or various factory methods.

Fixes https://github.com/spring-cloud/spring-cloud-sleuth/issues/1716

Thank you for this contritbution @ciscoo !


IKEv2 with macOS 13.1

IKEv2 EAP username/pass connection from macOS 13.1 to Linux Ubuntu 22.04 fails with
pfkey_sa: NAT-T: type=UDP encap (2) sport=4500 dport=4500
pfkey_sa: unsupported integrity algorithm HMAC_SHA2_256_128
ikev2_childsa_enable: failed to load CHILD SA spi 0xd316e0d9
spi=0xd0f5d9995c2f573f: ikev2_childsa_delete: deleted CHILD SA spi 0xd316e0d9
spi=0xd0f5d9995c2f573f: ikev2_childsa_delete: deleted CHILD SA spi 0x0c4a3c5f
ikev2_resp_recv: failed to send auth response
spi=0xd0f5d9995c2f573f: sa_state: EAP_VALID -> CLOSED from 80.xxx.xxx.117:4500 to 92.xxx.xxx.23:4500 policy 'IKEv2 RoadWarrior'
ikev2_recv: closing SA
spi=0xd0f5d9995c2f573f: sa_free: IKE_AUTH failed

7.1 and 7.0 versions of openiked crashed with segfault.
In version 7.2 the iked daemon doesn't crash, the macOS connection stays UP (the panel says "connected"), but as seen from the logs above server-side is actually down, and the traffic is not flowing.
Is HMAC_SHA2_256_128 support in linux/openssl broken?

Linux does a weird thing where it truncates HMAC_SHA2_256_128 to 96 bits by default when using the old pfkey kernel interface (which we do). This makes it incompatible with other implementations which is why it is (or at least i thought it is) disabled by default there. I wonder why it still works for you. Do you have an esp auth hmac-sha2-256 line in your config file?

yes, I have
        ikesa auth hmac-sha2-256 enc aes-256 prf hmac-sha2-256 group modp2048  \
        childsa auth hmac-sha2-256 enc aes-256 group modp2048 \

without these lines doesn't come up macOS-side.
Putting the lines and seeing "connected" in OSX doesn't oviously let you ping the local-side autoassigned ip address.
That's weird and at least some kind of bug of macOS.
In iOS without the lines the tunnel comes up with aes-128 and sha1 childsa  (which is apparently not allowed on macOS) .

I see, i wonder where our defaults are incompatible. It depends on what macos is sending as default proposal but you could try replacing hmac-sha2-256 with hmac-sha2-384 and see if that works. I think we should try to catch that linux error earlier and try to find a way to make it all work out of the box with macos (if that is possible)

It seems that last macOS 13.1 (but not iOS) wants only ESP childsa INTEGR=HMAC_SHA2_256_128
So I went to the AppConfigurator app for Mac, created a custom profile, loaded into the OS and after that was able to connect forcing hmac-sha2-384 client-side.
The following linux Ubuntu 22.04 server configuration in a policy permits macOS 13.1, last Windows11 (Version 10.0.22000 Build 22000) and iOS 16.1.2 clients to connect to iked in IKEv2 ms-chap. Windows 11 is picky about having a trusted public certificate in /etc/iked/certs matching srcid (I use one from letsencrypt), in macOS and iOS you can eventually override it with Apple Configurator.
ikev2 "winmacRW" quick passive tunnel esp inet \
        from any to dynamic \
        local any peer any \
        ikesa auth hmac-sha2-384 auth hmac-sha2-256 enc aes-256 enc aes-128 prf hmac-sha2-384 prf hmac-sha2-256 group ecp384 group modp2048 group modp1024 \
        childsa auth hmac-sha2-384 auth hmac-sha2-256 auth hmac-sha1 enc aes-256 enc aes-128 group ecp384 group modp2048 group modp1024 noesn \
        srcid "xxxxx" \
        eap "mschap-v2" \
        config address <IP_ADDRESS>/24 \
        config name-server <IP_ADDRESS> \
        config protected-subnet <IP_ADDRESS>/0 \
        tag "$name-$id"

For mac and iOS, you should create and load a custom profile with Apple Configurator (only available on mac) to enforce hmac-sha2-384. As a side note, the created Apple Configurator .mobileconfig file, contains an error and is not digested by default by macOS and iOS. You should edit it manually eliminating the spurious DNS directive entries in the first few rows, right before the VPN conf payload.


choose specific time to sleep when using decorator routines.routine

Current explanation about wait_first in doc is ambiguous.
https://github.com/TwitchIO/TwitchIO/blob/14b0880d9c22f6fc599f23ddbcb08e1aa8c962c4/twitchio/ext/routines/__init__.py#L364-L366
It says it waits for specific time so, this could be understood as it's requiring int or float.
https://github.com/TwitchIO/TwitchIO/blob/14b0880d9c22f6fc599f23ddbcb08e1aa8c962c4/twitchio/ext/routines/__init__.py#L390-L407
But it actually waits for time to  wait before the next iteration of the routine. ↑
https://github.com/TwitchIO/TwitchIO/blob/14b0880d9c22f6fc599f23ddbcb08e1aa8c962c4/twitchio/ext/routines/__init__.py#L288-L289
delta passed to Routine is used wait for next iteration as the code shows ↑

proposal
add wait_for keyword parameter requiring int or float to wait for specific time
@routines.routine(seconds=600,wait_for=20)
async def do_this_job_every_20_sec(self):
    await do_something( ... )
    ...

for example wait_for key word will be used for jobs that has time out like DB connections
that should run first in the place when the app started and  then run after every 10 minutes

The type hint shows if it takes a value it has to be a bool so it is not ambiguous as to whether it could be an int or a float.
The argument name of wait_first implies the question "Would you like to wait first before the first iteration", where specified time is the routine loop interval.
The docs can be updated to reflect this better.

@chillymosh  I haven't thought using @before.routine  thanks ,my bad

Just for someone else like me

Closing as not an issue.


ISSUE with Secured SSL SITE

My site already has Secured SSL, but however i have encountered the issue while uploading the dwg files below
"Exception when calling BucketsApi->createBucket: API call to https://developer.api.autodesk.com/oss/v2/buckets failed: SSL certificate problem: self signed certificate in certificate chain"
Please let me know the solution ASAP for this above menioned issue.
thanks and regards,
Pankaj

Sounds related to this issue: https://github.com/guzzle/guzzle/issues/1855
A workaround is to disable SSL certificate validation at https://github.com/Autodesk-Forge/core-php-client/blob/3d55f599a5f90b8710057ee051a1061b236933fd/lib/Auth/TokenFetcher.php#L52 like the below according to the GuzzleHttp API documentation:
if ($httpClient === null) {
    $httpClient = new GuzzleClient([
      'verify' => false
    ]);
}

You can do this change in the vendor/autodesk/core/lib/Auth/TokenFetcher.php folder of his PHP project. If it doesn't work, you need to check your php.ini and find if there are any special SSL settings that exist (see ref).


Add EightShapes Specs to the allowlist

Duplicates ❌

[X] I have searched the existing Figma plugins requests.

Figma plugin/widget name
EightShapes Specs
Figma plugin/widget URL
https://www.figma.com/community/plugin/1205622541257680763/EightShapes-Specs
Why do you need this Figma plugin/widget?
Because it can really help us (better than Redlines). 🤓😍

Approved :)


When a proxy fails, the password should not be dumped to the screen

https://github.com/replicatedhq/kURL/blob/a18c2e2a488c5f28dc6504b84f38bf73f0c605a8/scripts/common/proxy.sh#L68
Ideally, HTTPS_PROXY=https://USERNAME:PASSWORD@<IP_ADDRESS>:8080/ would be redacted to not show the password.

Hi @laverya,
This one is very old. Things that were not get done in the latest 2 yrs probably will not be done soon :-)
So, would still that a need? Should we kept this one open or can we close this RFE?