kotlin: refactor public functionality as precursor to L7 filters

Context
This is a mirror of the iOS interface refactors done in https://github.com/lyft/envoy-mobile/pull/858. Additionally, it introduces a set of mocks to match the iOS mocks introduced in https://github.com/lyft/envoy-mobile/pull/900.
Resolves https://github.com/lyft/envoy-mobile/issues/905 and resolves https://github.com/lyft/envoy-mobile/issues/907.
Goals

Unify the existing response handler and stream emitter types into a single "stream" type that will make it easier to communicate with an underlying filter manager for both outbound requests and inbound responses
Provide guard rails that prevent the consumer from making mistakes, such as setting callback handlers after starting a stream and thus missing updates
Provide a clear interface to the user for creating and starting streams
Focus on the basics (bi-directional streaming support), which will allow for other wrappers to be written in the future (i.e., GRPC, unary, Rx, etc.)
Update interfaces to utilize the new wrapper types introduced on master (ResponseHeaders, RequestHeaders, etc.)

Types

StreamClient: The publicly exposed interface of EnvoyEngine. Provides a function for starting a new stream using Envoy Mobile, and is built using StreamClientBuilder
StreamPrototype: Type that allows users to set up a stream (providing callbacks, etc.) prior to starting it
Stream: Produced by calling start() on a StreamPrototype, and allows the consumer to send data over the stream and eventually close it
GRPCClient / GRPCStreamPrototype / GRPCStream: Types wrapping the above 3 types, converting the existing gRPC types to utilize the new functionality

Tests

I went through all of the Swift tests and audited the Kotlin tests to:

Add/update to ensure that we now have 1:1 parity on tests between the two languages (even the test names are the same!)
Fixed a bug I noticed while adding the tests where the retryable status code enum couldn't be properly serialized to headers and deserialized from those same headers
Migrate them to the new interfaces

Sample apps have also been updated (CI validates via "hello world")

Notes

I noticed some Kotlin types that were unnecessarily public. Those have been made internal in this PR
Logic and tests were added for setting headers with restricted prefixes (: and x-envoy-mobile), for parity with iOS
Documentation will be updated separately in https://github.com/lyft/envoy-mobile/issues/906

Signed-off-by: Michael Rebello<EMAIL_ADDRESS>

Thanks for the review @johnpaulmckean! I addressed your comments


how to get config value from the local file?

I define a config file about database setting. follow example code in URL:
https://go-chassis.readthedocs.io/en/latest/user-guides/dynamic-conf.html
	func main() {
	    chassis.RegisterSchema("rest", &schemas.RestFulHello{})
	    chassis.RegisterSchema("rest", &schemas.RestFulMessage{})
	    //start all server you register in server/schemas.
	    if err := chassis.Init(); err != nil {
		lager.Logger.Error("Init failed.", err)
		return
	    }

	    //test code
	    dbCfgFile := fileutil.GetConfDir() + "\\db.yaml"
	    archaius.AddFile(dbCfgFile)
	    dbAddr:=archaius.GetString("dbconf.conn1.addr", "")
	    fmt.Println(dbAddr)

	    chassis.Run()
        }

but I can't get value of dbAddr, it's empty.
Then I check code of https://github.com/go-chassis/go-archaius/blob/master/examples/main.go, it's ok.
but I find code
        ConfigFactory.DeInit()

in sample code, and this code will clean all config item about FileSource. Is there any problem with this behavior?
Later, I just use object of NewYamlConfigurationSource to get config value, like this:
	//test
	dbCfgFile := fileutil.GetConfDir() + "\\db.yaml"
	fSource := filesource.NewYamlConfigurationSource()
	fSource.AddFileSource(dbCfgFile, 0)
	dbAddr, _:=fSource.GetConfigurationByKey("dbconf.conn1.addr")

	fmt.Println("---------------- DB Addr:", dbAddr)

It's work, and get real value of dbAddr.

@fzxdev
For AddFile() function events were not handled properly.
            dbCfgFile := fileutil.GetConfDir() + "\\db.yaml"
	    archaius.AddFile(dbCfgFile)
	    dbAddr:=archaius.GetString("dbconf.conn1.addr", "")
	    fmt.Println(dbAddr)

Thats why above code was not giving proper output.
I have changed the logic and raised PR
https://github.com/go-chassis/go-archaius/pull/35
Once this is merged you can check again with latest code.
I have added testcase scenario in https://github.com/go-chassis/go-chassis/pull/388
Thanks for noticing this issue.

@fzxdev
For AddFile() function events were not handled properly.
            dbCfgFile := fileutil.GetConfDir() + "\\db.yaml"
	    archaius.AddFile(dbCfgFile)
	    dbAddr:=archaius.GetString("dbconf.conn1.addr", "")
	    fmt.Println(dbAddr)

Thats why above code was not giving proper output.
I have changed the logic and raised PR
https://github.com/go-chassis/go-archaius/pull/35
Once this is merged you can check again with latest code.
I have added testcase scenario in https://github.com/go-chassis/go-chassis/pull/388
Thanks for noticing this issue.


strongswan: 5.9.8 -> 5.9.10

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/strongswan/strongswan/releases.
meta.description for strongswan is: OpenSource IPsec-based VPN Solution
meta.homepage for strongswan is: https://www.strongswan.org
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.9.9 with grep in /nix/store/65w2f81jx73rh0lpjspfnrvdbcr5blw0-strongswan-5.9.9
found 5.9.9 in filename of file in /nix/store/65w2f81jx73rh0lpjspfnrvdbcr5blw0-strongswan-5.9.9

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
gnome.networkmanager-l2tp
networkmanager-l2tp
networkmanager_strongswan
strongswan
strongswanNM
strongswanTNC

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/65w2f81jx73rh0lpjspfnrvdbcr5blw0-strongswan-5.9.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A strongswan https://github.com/r-ryantm/nixpkgs/archive/af2cb943cdfb9e4f7a08b08d0180f9db6c133593.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/65w2f81jx73rh0lpjspfnrvdbcr5blw0-strongswan-5.9.9
ls -la /nix/store/65w2f81jx73rh0lpjspfnrvdbcr5blw0-strongswan-5.9.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

5 packages built:

networkmanager-l2tp (gnome.networkmanager-l2tp)
networkmanager_strongswan
strongswan
strongswanNM
strongswanTNC

Maintainer pings

Result of nixpkgs-review pr 209695 run on x86_64-linux 1

5 packages built:

networkmanager-l2tp (gnome.networkmanager-l2tp)
networkmanager_strongswan
strongswan
strongswanNM
strongswanTNC

As noted in the comments, the file nixos/modules/services/networking/strongswan-swanctl/swanctl-params.nix needs to get updated as well

As noted in the comments, the file nixos/modules/services/networking/strongswan-swanctl/swanctl-params.nix needs to get updated as well

@onny would you like to do that? Obvious @r-ryantm the bot cannot do this himself.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/strongswan/strongswan/releases.
meta.description for strongswan is: OpenSource IPsec-based VPN Solution
meta.homepage for strongswan is: https://www.strongswan.org
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.9.10 with grep in /nix/store/lsggzsrcj7xyskrhhvhyh484ab7q2drj-strongswan-5.9.10
found 5.9.10 in filename of file in /nix/store/lsggzsrcj7xyskrhhvhyh484ab7q2drj-strongswan-5.9.10

Rebuild report (if merged into master) (click to expand)

6 total rebuild path(s)

6 package rebuild(s)

First fifty rebuilds by attrpath
gnome.networkmanager-l2tp
networkmanager-l2tp
networkmanager_strongswan
strongswan
strongswanNM
strongswanTNC

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/lsggzsrcj7xyskrhhvhyh484ab7q2drj-strongswan-5.9.10 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A strongswan https://github.com/r-ryantm/nixpkgs/archive/a86756d8d986354bbc75afe6b304c717a22b87f3.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/lsggzsrcj7xyskrhhvhyh484ab7q2drj-strongswan-5.9.10
ls -la /nix/store/lsggzsrcj7xyskrhhvhyh484ab7q2drj-strongswan-5.9.10/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

5 packages built:

networkmanager-l2tp (gnome.networkmanager-l2tp)
networkmanager_strongswan
strongswan
strongswanNM
strongswanTNC

Maintainer pings

Done in #228225.


general cleanup of the complex deletion code; still need to add dialo…

…g functionality

Okay, this removes a bunch of the code complexity that got introduced, still need to add dialog but I'll make another ticket for that.


ENH: add visualization scripts

@sthaar mentioned that @dgmets has some visualization scripts, might be nice to add (if in pure Python) or adapt (e.g. if in Matlab)

As discussed today, @yangzheng-121 and I wrote some simple code to output onsets and offsets to a csv file: https://github.com/sthaar/audio-analyses/blob/main/songdkl_csv_onset-offset_share.py
and to show spectrograms of the syllable segmentation based on this csv file (credits to @yangzheng-121): https://github.com/sthaar/audio-analyses/blob/main/get_spectrogram_with_lines_clean.ipynb
I'm sure you can implement it in a much better way! But it works for now for testing purposes


chore: Lerna-Lite 2.0 publish & version are becoming optional commands

same as https://github.com/vuejs/language-tools/pull/2597
in the next Lerna-Lite v2.0.0, I made version and publish as optional commands since it was requested by some users. So Lerna-Lite is now entirely modular and the user will need to manually install the command(s) he want to use (plus the CLI)

Oh i missed it. LGTM, thanks!


Mock IQ Backend to generate different results across different qubits

Currently, mock backends use a single seed for every random draw. For the discriminator, it would be useful if the mock IQ backend can support different seeding and IQ cluster/width for different qubits so that different qubit data is distinguishable when running parallel experiments.

@gadial Do you intend to work on this issue?

As far as I see the mock IQ backend doesn't support multiple qubits. So it's not only a matter of different clusters to different qubits, but more basically a matter of multiple qubits at all.

Done in https://github.com/Qiskit/qiskit-experiments/pull/607.


Error after upgrading the theme

Hello,
I followed this guide to upgrade my theme by replacing my current themes/academic folder with the latest version. However, after the upgrade, when running hugo -t academic I got

ERROR 2018/02/22 00:48:38 Failed to render "theme/section/talk.html": runtime error: index out of range

Could you please help? I'm using Hugo 0.30.2 under Linux Mint 18.3.
Thank you very much in advance!

I have upgraded Hugo to v0.36.1 but the problem remains :(

I replaced my old content/talk with exampleSite/content/talk and now it can build without error. I'm having some display problems, though. I will try replacing old files with new files to see.

I updated my config.toml with the new one. Problem solve!

Yes I ran into this challenge too. There is a mis-match somewhere between the config file and the talk templates. I had to pull out some custom template content but like you eventually got it to work.

The update instructions can be found in the release notes on the documentation website that are relevant for your update path.

@HughP can you elaborate further?
I dont use the talk widget and removed it from /content/home however I still get this error. I had diff'd in the changes to the exampleSite/config.toml from the update guide and cant seem the tackle this issue.


Redirect logs to stdout if ENV variable is present

By default Rails writes its logs to a specific file, which is convenient if you only have one log file to tail. When you start scaling to multiple instances running your app, finding a single request or failure across multiple files becomes much harder. Storing logs on disk can also take down a server if the hard drive fills up.
12 Factor recommend to redirect all logs to stdout -> https://www.12factor.net/logs . I propose to add ENV variable that would add such ability.

@skatkov Fully agree with you and 12 Factor app philosophy
We redirect all logs to stdout by default https://github.com/rubykube/peatio/blob/master/config/environments/shared/logger.rb#L12-L16

@ysv that's great.
Sorry, I totally missed this initialiser ! Closing this Pull Request :)


Enable easy configuration of encrypted PostgreSQL connections with new optional DB_SSLMODE (defaults to current value of "disable") following values allowed by PostgreSQL

…PostgreSQL connections, adding a new environment variable DB_SSLMODE with notes about use of existing PGSSLROOTCERT and PGSSLCRL environment variables that may be needed.  Default for sslmode in connection string remains current value of disabled

Summary

This pull request enables quick configuration of encrypted PostgreSQL connections when setting up Docker containers.  The need for this arises when using a hybrid configuration - application and web servers running in local containers while the database is hosted remotely in a separate server or on some cloud-based Database-as-a-Service.  Using a new environment variable DB_SSLMODE, the code to build the default connection string (MM_SQLSETTINGS_DATASOURCE) now supports parameterization of the sslmode parameter (currently hardcoded to a value of disabled)
Ticket Link

This is an unsolicited contribution (I was unable to find a corresponding JIRA ticket for this) arising from my particular deployment challenge.  In a particular circumstance, I have an environment where the application and web code runs in Docker containers while the database is hosted in a DaaS configuration remotely.  That provider requires an encrypted connection (which is of course good practice anyway) when connecting remotely.  PostgreSQL has several parameters to support this behavior, most notably including sslmode, sslrootcert and sslcrl.  Best practice for remote databases today is to use verify-ca or verify-full, though require is often used as well.  The current code in entrypoint.sh hardcodes the sslmode parameter to disable.  This is quite reasonable behavior in the main case where all components run in the same container, or perhaps even in the same Docker network.
My goals were twofold: support configuration of the sslmode parameter and support the associated configuration data such as the root CA file.  I initially considered simply overwriting the derived variable MM_SQLSETTINGS_DATASOURCE, on the premise that the case might not be used all that frequently.  I decided to propose a new environment variable instead (DB_SSLMODE) for two reasons:

The use case of a remote database seems plausible for other users, and a simple replacement of the hardcoded value disable for sslmode makes the least change possible to implement the functionality
The convention, at least in the Docker configuration, seems to be that the other variables support the automatic building of MM_SQLSETTINGS_DATASOURCE for PostgreSQL applications, with the documentation recommending override of this variable for MySQL use cases.  Therefore, it seemed consistent to extend this and simply make the existing "build" of the variable support this configurable mode for sslmode

Having decided to do this, I was faced with a problem (that incidentally would have arisen even if overriding the MM_SQLSETTINGS_DATASOURCE variable while keeping a URL format).  Once one elects to use a mode such as verify-ca or verify-full in PostgreSQL, one must provide the root CA file.  Depending on the application, other data such as a certificate revocation list (sslcrl) may be needed also.  The URL format for connection strings with PostgreSQL does not always work with these variables, even if one URL-encodes the filenames.  I considered changing the default format of the connection string in MM_SQLSETTINGS_DATASOURCE to a parameterized value, but then concluded this would create unnecessary change for users that did not need this new configuration behavior.
Faced with the practical decision to keep the connection string format in current URL format without always having good support for the supplemental configuration information like sslrootcert, I considered creating several new environment variables to handle cases when a user wanted (for example) to specify a CA file in a non-standard location.  However, PostgreSQL has built-in support for environment variables PGSSLROOTCERT and PGSSLCRL.  So, anyone wishing to use an encrypted connection can supply these environment variables into a Docker container just as one would add any other.
In summary, I feel the approach of adding parameterization for the sslmode value in the MM_SQLSETTINGS_DATASOURCE build-up in entrypoint.sh represents the least overall change while providing the behavior required:

because the entrypoint.sh initializes the newly proposed variable DB_SSLMODE to the previously hard-coded value of default, existing applications should run unchanged
the syntax for building up MM_SQLSETTINGS_DATASOURCE remains the same, minimizing the possibility that subtle issues might arise for some users if the connection string build-up was made using the variable method
the convention of having the entrypoint.sh build-up of MM_SQLSETTINGS_DATASOURCE remains the same for PostgreSQL connections, while users still have ability to override completely
there is no unnecessary duplication with additional environment variables, instead leveraging the PostgreSQL-supported variables for specifying CA files and certificate revocation lists when required.

The PostgreSQL reference for encrypted connections can be found here - see sections 33.18.1 and 33.18.2.

Hello @3leapsdet,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Thanks for catching this - I had not intended that echo to remain!  Will
update shortly.
On Thu, Dec 10, 2020 at 8:31 AM Carlos Tadeu Panato Junior <
<EMAIL_ADDRESS>wrote:

@cpanato requested changes on this pull request.
thanks for your PR, just a small request change
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mattermost/mattermost-docker/pull/506#pullrequestreview-549196138,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APKMI5OXCBX7RGQW7UHJBU3SUDER7ANCNFSM4UOS4IUQ
.


Message from string doesn't work

Hey
I want to use siwe for one of my project. However i am having issue. When I create SiweMessage always when i use with string. Even tho the string version is created with SiweMessage a little snippet to reproduce the issue.
from eth_account import Account
from siwe.siwe import SiweMessage

account = Account.create()
test_message = {
        "domain": "test",
        "address": account.address,
        "uri": "test",
        "version": "1",
        "chain_id": "1",
    }
message = SiweMessage(test_message)
signature = account.sign_message(
        messages.encode_defunct(text=message.prepare_message())
    ).signature

# Fails here
message_from_string = SiweMessage(message.prepare_message())

I simply forgot to add the statement, but the error messages can be improved to direct in the better direction


refactor to permit HistoryProviders and in memory history + dart_dev

Issue

Changes
Source:
Tests:
Areas of Regression

Testing

Code Review
@maxwellpeterson-wf
@evanweible-wf
@dustinlessard-wf
@jayudey-wf

TODO - there are a couple of skipped tests to figure out...

First pass, this looks good. Only a few minor things.

@evanweible-wf all comments addressed

+1

+1

+1

@jayudey-wf ready for merge and tag as 0.7.0

(will get travis turned on for this next)

QA +1
verified passing tests and example is working as expected
Merging into master


ionic build fails without @angular/tsc-wrapped - 3.0.0

Short description of the problem:
$ ionic build
Error: Cannot find module '@angular/tsc-wrapped/src/tsc'
    at Function.Module._resolveFilename (module.js:470:15)
    at Function.Module._load (module.js:418:25)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (.../node_modules/@ionic/app-scripts/dist/aot/aot-compiler.js:6:13)
    at Module._compile (module.js:571:32)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)

Steps to reproduce:

update ionic-app-scripts to 3.0.0
update ionic-angular to 3.7.1
update webpack.config.js (link) as suggested from changelog breaking changes

$ ionic info

cli packages: (.../node_modules)

    @ionic/cli-utils  : 1.12.0
    ionic (Ionic CLI) : 3.12.0

global packages:

    cordova (Cordova CLI) : 7.0.1

local packages:

    @ionic/app-scripts : 3.0.0
    Cordova Platforms  : android 6.3.0 ios 4.5.0
    Ionic Framework    : ionic-angular 3.7.1

System:

    Android SDK Tools : 26.1.1
    ios-deploy        : 1.8.6
    ios-sim           : 5.0.8
    Node              : v6.11.3
    npm               : 5.4.2
    OS                : macOS Sierra
    Xcode             : Xcode 9.0 Build version 9A235

Misc:

    backend : legacy

Which @ionic/app-scripts version are you using?
3.0.0.
Workaround
Installing @angular/tsc-wrapped in the mobile app project fixes the issue

@gianpaj,
Interesting. @angular/tsc-wrapped is a private dependency for angular, so you shouldn't have to install it. That is very interesting. Could you try blowing away your lock file and trying again?
Thanks,
Dan

I removed the package-lock.json file, ran npm i but ionic build still fails with the same err msg.
Here is the diff between the package-lock json before and after (if helps the diagnostic):
https://gist.github.com/gianpaj/581130392676756339962a1c88f306a7

@gianpaj I realize how nuanced these issues can be so YMMV, but FWIW I had the same error and noticed npm displaying a warning regarding an unmet peer dependency on<EMAIL_ADDRESS>by tsc-wrapped so I ran npm install<EMAIL_ADDRESS>--save-dev and that resolved the problem for me

Hmm, okay I am not sure. This seems like a one-off thing. Anyone have any information on how to solve? I don't think this is too widespread or there would be many more reports.
Thanks,
Dan

Yeah not sure what happened. I had other issues with cordova plugins so I reverted back a bunch of changes in package.json and config.xml and now i have a semi-stable build and app.
I'll reopen this if I see this again once I upgrade back to 3.0.0
Cheers


Lint recipe for version 0.2.1

Initial PR for this feedstock to get it onto Zeus.
This is a dependency for django 4 (ticket DSNC-4153).
Package Info
Home: https://github.com/pganssle/zoneinfo
Source: https://github.com/pganssle/zoneinfo/archive/refs/tags/0.2.1.tar.gz
Documentation: https://zoneinfo.readthedocs.io
Actions

Add URL to licemse
Remove localization of the doc URL
Remove python 3.9 support since upstream only supports up to 3.8
Fix build requirements

we need to verify the build number. I could be missing some details, but do not seem to have a build 5 of backports.zoneinfo in defaults.

This package is brand-new for us, but conda-forge had build number 5. So, what's our policy here?

we need to verify the build number. I could be missing some details, but do not seem to have a build 5 of backports.zoneinfo in defaults.

This package is brand-new for us, but conda-forge had build number 5. So, what's our policy here?

Normally we look at the build number based on what we have on defaults, since this is going to be the first file on defaults, then the build number will be set to zero.

we need to verify the build number. I could be missing some details, but do not seem to have a build 5 of backports.zoneinfo in defaults.

This package is brand-new for us, but conda-forge had build number 5. So, what's our policy here?

Normally we look at the build number based on what we have on defaults, since this is going to be the first file on defaults, then the build number will be set to zero.

Thank you! I set the number to zero


fix: re-add HOME to paths in updater

I'm not sure if this is the ideal solution, because there's a few cases where we actually wouldn't have $HOME (EnvSubdir or the user actually setting $CARGO_HOME)... ideally we check for both forms.

obsoleted by #928


MAUI iOS configure errors

Hi,
In my MAUI application targeting net6.0-ios15.4 I configure the Firebase App the following way in the AppDelegate
public bool FinishedLaunching(UIKit.UIApplication application, NSDictionary launchOptions)
{
   Firebase.Core.App.Configure(new Firebase.Core.Options("googleAppId", "gcmSenderId") { ... });

   Crashlytics.SharedInstance.Init();
   Crashlytics.SharedInstance.SetCrashlyticsCollectionEnabled(true);
   Crashlytics.SharedInstance.SendUnsentReports();
   return base.FinishedLaunching(application, launchOptions);
}

But It seems that I can't instantiate the Firebase.Core.Options class because I get the following exception:
[0:] An error occurred: 'Could not create an native instance of the type 'Firebase.Core.Options': the native class hasn't been loaded.
It is possible to ignore this condition by setting ObjCRuntime.Class.ThrowOnInitFailure to false.'. Callstack: '   at Foundation.NSObject.InitializeObject(Boolean alloced)
   at Foundation.NSObject..ctor(NSObjectFlag x)
   at Firebase.Core.Options..ctor(String googleAppId, String gcmSenderId)
   at MauiApp5.AppDelegate.FinishedLaunching(UIApplication application, NSDictionary launchOptions) in C:\Source\MauiApp5\MauiApp5\Platforms\iOS\AppDelegate.cs:line 14
   at UIKit.UIApplication.Main(String[] args, Type principalClass, Type delegateClass)
   at MauiApp5.Program.Main(String[] args) in C:\Source\MauiApp5\MauiApp5\Platforms\iOS\Program.cs:line 22
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)'

As a workaround, I've also tried to initialize the Firebase App using a GoogleService-Info.plist file in the Platforms/iOS folder and adding the following to the csproj:
<ItemGroup>
  <BundleResource Include="Platforms\iOS\GoogleService-Info.plist" Link="GoogleService-Info.plist" />
</ItemGroup>

I've verified that the plist file does get included in the ipa output and the Firebase.Core.App.Configure call runs without errors. But when I'm trying to call Crashlytics.SharedInstance.Init() I get a NullReferenceException on the SharedInstance. Also after the Firebase.Core.App.Configure call the Firebase.Core.App.DefaultInstance stays null.
public bool FinishedLaunching(UIKit.UIApplication application, NSDictionary launchOptions)
{
   Firebase.Core.App.Configure();

   // both Firebase.Core.App.DefaultInstance and Crashlytics.SharedInstance are null here!
   Crashlytics.SharedInstance.Init();
   Crashlytics.SharedInstance.SetCrashlyticsCollectionEnabled(true);
   Crashlytics.SharedInstance.SendUnsentReports();
   return base.FinishedLaunching(application, launchOptions);
}

I've tested it in an empty MAUI project adding only the following nugets:

Xamarin.Firebase.iOS.Core (<IP_ADDRESS>)
Xamarin.Firebase.iOS.Crashlytics (<IP_ADDRESS>)

Anyone else experiencing a similar issue?

Hi, I am facing the similar issue with Auth.
                        ///DefaultInstance is null.
                        Firebase.Auth.Auth instance = Firebase.Auth.Auth.DefaultInstance;

MAUI and with Xamarin.Firebase.iOS.Auth <IP_ADDRESS>

Hot Restart is supports only dynamic libraries: https://learn.microsoft.com/en-us/dotnet/maui/deployment/hot-restart#prevent-code-from-executing
I think thats the problem.

Same problem here.
Is there any fix yet?

Same problem here.
Is there any fix yet?

Workaround: Try to deploy by connect with mac. For GoogleService-Info.plist file use the @balazs-adam's workaround.

Yes, for now I'd recommend using the HOTRESTART preprocessor to conditionally initialize the FirebaseApp only when Hot Restart is not enabled.


Shadcn/ui cli should NOT replace tailwindcss config file

Why does it replace ALL of the configs????????

The init command is meant for a new project. Let me see how we can improve it so that it updates instead of replace.
I'll file this under enhancement to the CLI and take a look at it.

I actually came through google to this comment. I was also surprised that it replaces the file without a warning. Upserting wold be great or at least give a warning that it will happen.

@shadcn What do you think is a good approach for now? What is the expected behavior?
I think before write any file, just check if already exist, and if yes exist throw a new interaction to the user explaining all the files will be replaced to user decide to go on or not.
If I understand well the code below, the files write are:

tailwind.config.js
tsconfig.json (this is not replaced)
components.json
@/lib/utils.ts
app/globals.css

https://github.com/shadcn-ui/ui/blob/c21ecfb665214e18cd5914ea319f925cd676e786/packages/cli/src/commands/init.ts#L205-L279
Example of message, default response is Not:
`You already have config files at this folder, this command will overwrite the files:
${highlight(
  existingFiles
)} Proceed? > (y/N)
`

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Mango using Partitioned parquet ADAM

Replaces #358
works with ADAM: #1911
Example mango run
mango-submit --master yarn --num-executors 10 --executor-cores 8 --executor-memory 10g --driver-memory 20g  -- /home/eecs/akmorrow/builds/hg19.2bit -genes http://www.biodalliance.org/datasets/ensGene.bb -reads hdfs://{headnodepath}/user/jpaschall/mango1/HG00096_Jan30_v2.adam -show_genotypes

The problem with "chr" fix was resolved by querying both with and without "chr" prefix

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/575/
Test FAILed.

More recent test of this Mango code with partitioned dataset of  the large Platinum genome files can be done with this command
mango-submit --master yarn --num-executors 10 --executor-cores 4 --executor-memory 20g --driver-memory 20g  -- /home/eecs/akmorrow/builds/hg19.2bit -genes http://www.biodalliance.org/datasets/ensGene.bb -reads hdfs://amp-bdg-master.amplab.net:8020/user/jpaschall/feb16_work/NA12877_S1.bam.partitioned.v3.adam,hdfs://amp-bdg-master.amplab.net:8020/user/jpaschall/feb16_work/NA12890_S1.bam.partitioned.v3.adam,hdfs://amp-bdg-master.amplab.net:8020/user/jpaschall/feb16_work/NA12889_S1.bam.partitioned.v3.adam -show_genotypes -parquetIsBinned

The "warm-up period", done upfront when using the -parquetIsBinned flag is still 30-45 sec one time, after that response time is 1-3 seconds when jumping around locations in genome browser,

This is replaced by #370


Creating bots

What about creating a bot that would just send chat invite whatever i am typing a message to him

Check out smartersmarterchild for an example of how to implement a bot

If you have more questions like these, please join our discord


When running copyWith, not all types are full move the the copyWith method

I have the code:
class CommandContext<Result> {
  CommandContext({
    required this.invocation,
    required this.rest,
    required this.current,
    required this.result,
  });

  final List<String> invocation;
  final List<String> rest;
  final Command<Result> current;
  final Result result;
}

And when I execute Generate copyWith, this is generated in the above class.
CommandContext copyWith({
  List? invocation,
  List? rest,
  Command? current,
  Result? result,
}) {
  return CommandContext(
    invocation: invocation ?? this.invocation,
    rest: rest ?? this.rest,
    current: current ?? this.current,
    result: result ?? this.result,
  );
}

It is missing the type in the <> for invocation, rest, and current

Indeed there is a problem, I'm gonna fix that soon.

It has been fixed in the 2.1.1 released this morning.
Thanks for your help 🙏

Thank you!


Rounding is broken for negative values

Hi,
I've recently started using NBi, and I'm really impressed. Great work! However, today I had to learn the hard way that rounding fails for negative values. I've add some test cases to demonstrate. Please see PR attached.
Thanks,
Mathias

Just to clarify: By "failing" I mean that applying NBi rounding to negative values actually makes them positive (round -0.1234 with step 0.01 gives +0.12).

Just to be sure that there is no misunderstanding, your tests are defined with
<column ... rounding-style="floor" />

and not with
<column ... tolerance="0.001" />

See doc at http://www.nbi.io/docs/compare-tolerances-roundings/
I'll check this but it should berelatively easy to fix.

Yes, I am talking about roundings, not tolerances.(BTW, I temporarily solved that problem for my test cases by applying the rounding at the data source level, i.e. before any values get to NBi - but, obviously, it would be nice to be able to use that feature as it is intended.)

You'll find a bug fix at https://ci.appveyor.com/project/CdricLCharlier/nbi/build/1.13.0-ci.42/artifacts

Thanks for sorting this out so quickly! Really appreciated.


Deprecate iris.util.approx_equal

🚀 Pull Request
Description

As per #4394
Will conflict with #4513 so will need a rebase once that's merged

Consult Iris pull request check list

But I would like to be sure @lbdreyer has had a chance to comment, before merging.

I have no objections to this being removed. It doesn't make sense to live in Iris' codebase and I can't find it being using anywhere within Iris


Azure Identity - handling DefaultAzureCredential's accessToken expiry

Query/Question
Does Azure Java SDK handles accessToken & refreshToken expiry or is it my task to handle expiry? If it is my task, how to do it?
Why is this not a Bug or a feature Request?
Setup (please complete the following information if applicable):

OS: Windows10
IDE: IntelliJ
Library/Libraries: com.azure:azure-identity:1.3.6

I use DefaultAzureCredential, as it works locally and also on cloud when ManagedIdentity is needed for authenticating to KeyVault.
I intend to deploy a function app to Azure Cloud, where I have the following static constructor of my class containing the functions:
DefaultAzureCredential credential = new DefaultAzureCredentialBuilder().build();
CertificateClient certificateClient = new CertificateClientBuilder()
             .vaultUrl(KEY_VAULT_URI)
             .credential(credential)
             .buildClient();

SecretClient secretClient = new SecretClientBuilder()
             .vaultUrl(KEY_VAULT_URI)
             .credential(credential)
             .buildClient();

service = new Service(certificateClient, secretClient);

I would like to create the service only once with certificateClient and secretClient.
Then if accessToken expires, get new accessToken.

@g2vinay could you please follow up with @nalesnyik on their question regarding Access tokens and Refresh tokens


Support mapping team-members to SCIM id's in order to align with userIds used in auth

Fixes #111.
Currently a draft as docs are needed to add, and also there is a snag with app-based auth when using the v4 client (https://github.com/palantir/go-githubapp/issues/104).

@sabre1041 PTAL.
This makes it possible to map teams from github, resolving the userId to the IDP-identity. This way the github-teams member uids will align with the id used in the IDP when using AAD as IDP. Use-case outlined in #111.
Note that this introduces a couple of fields in the CRD when using GH as provider.

@sabre1041 PTAL?

LGTM

Awesome - when (ca) will the next release be?


[Question]: APIConnectionError: Connection error

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I want to run the following starter code with llama-index:
`

import os
from silly import  no_ssl_verification
with no_ssl_verification():
os.environ["HUGGINGFACEHUB_API_TOKEN"] = "my_token"
from llama_index import VectorStoreIndex, SimpleDirectoryReader
documents = SimpleDirectoryReader("./docs/examples/data/paul_graham").load_data()
index = VectorStoreIndex.from_documents(documents)
query_engine = index.as_query_engine()
response = query_engine.query("What did the author do growing up?")
print(response)

`
I get the following error:
raise APIConnectionError(request=request) from err openai.APIConnectionError: Connection error.

@dosu-bot
I didn't use any database connection. I just do starter task named "starter.py" at the documentation.
raise APIConnectionError(request=request) from err openai.APIConnectionError: Connection error.

"Connection error" you're encountering is likely due to incorrect or missing environment variables related to the Azure OpenAI API." according to this statement, what can i do? i have llama-index project folder taken with git clone (from https://github.com/run-llama/llama_index)

@4entertainment openai is having some API issues lately. It will probably be a lately shakey for the next day or two


[FEATURE REQUEST] Allow auditd rules to be configurable for Filebeat

Is your feature request related to a problem? Please describe.
Currently auditd rules for Filebeat or not configurable.
Describe the solution you'd like
Allow auditd rules to be configurable for Filebeat
Describe alternatives you've considered
None
Additional context
https://github.com/epiphany-platform/epiphany/blob/develop/ansible/playbooks/roles/filebeat/templates/audit-epi.rules.j2

DoD checklist

Changelog

[ ] updated
[ ] not needed

COMPONENTS.md

[ ] updated
[ ] not needed

Schema

[ ] updated
[ ] not needed

Backport tasks

[ ] created
[ ] not needed

Documentation

[ ] added
[ ] updated
[ ] not needed

[ ] Feature has automated tests
[ ] Automated tests passed (QA pipelines)

[ ] apply
[ ] upgrade
[ ] backup/restore

[ ] Idempotency tested
[ ] All conversations in PR resolved
[ ] Solution meets requirements and is done according to design doc
[ ] Usage compliant with license

@rafzei and @to-bar please have look here if that is not already implemented.

@rafzei and @to-bar please have look here if that is not already implemented.

Still valid. Audit rules are hard-coded in audit-epi.rules.j2.

Not going todo anymore feature requests for Epiphany.


cannot insert custom data on drag-and-drop of vscode.TreeItem to editor

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.88
OS Version: Windows 11

Steps to Reproduce:

I need to drag a vscode.TreeItem (node) on the editor and need to insert some data  based upon node which is dragged.
To achieve this, I have followed sample - https://github.com/microsoft/vscode-extension-samples/blob/main/tree-view-sample/src/testViewDragAndDrop.ts
This sample inserts content of resourceUri property on vscode.TreeItem prefix with \

On handleDrag() method if we specify file uri to 'text/uri-list' MIME type (as shown in below code snippet) then drag and drop on editor opens new file as specified in file URI and
If we press shift key and drop then it inserts file uri path i.e. c:\office.txt rather than inserting content of file.

public async handleDrag(source: Node[], treeDataTransfer: vscode.DataTransfer, token: vscode.CancellationToken): Promise<void> {
		treeDataTransfer.set('application/vnd.code.tree.testViewDragAndDrop', new vscode.DataTransferItem(source));
		let uri = vscode.Uri.parse("file:///C:/office/1.txt");
		let dataTransferItem = new vscode.DataTransferItem(uri.toString());
		 treeDataTransfer.set('text/uri-list', dataTransferItem);
	}

To insert custom data i have implemented one more interface vscode.DocumentDropEditProvider (code snippet shows). Can this be used to insert data for drag and drop of vscode.treeview node to editor? Documentation says it is used to drag and drop files

class FileNameListOnDropProvider implements vscode.DocumentDropEditProvider {
	async provideDocumentDropEdits(
		_document: vscode.TextDocument,
		_position: vscode.Position,
		dataTransfer: vscode.DataTransfer,
		token: vscode.CancellationToken
	): Promise<vscode.DocumentDropEdit | undefined> {
		// Check the data transfer to see if we have dropped a list of uris
		const dataTransferItem = dataTransfer.get(uriListMime);
		if (!dataTransferItem) {
			return undefined;
		}

		// 'text/uri-list' contains a list of uris separated by new lines.
		// Parse this to an array of uris.
		const urlList = await dataTransferItem.asString();
		if (token.isCancellationRequested) {
			return undefined;
		}

		const uris: vscode.Uri[] = [];
		for (const resource of urlList.split('\n')) {
			try {
				uris.push(vscode.Uri.parse(resource));
			} catch {
				// noop
			}
		}

		if (!uris.length) {
			return undefined;
		}

		// Build a snippet to insert
		const snippet = new vscode.SnippetString();
		uris.forEach((uri, index) => {
			const name = path.basename(uri.path);
			snippet.appendText(`${index + 1}. ${name}`);
			snippet.appendTabstop();

			if (index <= uris.length - 1 && uris.length > 1) {
				snippet.appendText('\n');
			}
		});

		//return new vscode.DocumentDropEdit(snippet);
		return new vscode.DocumentDropEdit(`testing drag
		n drop` );
	}
}

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.88.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I am using latest vscode vesrion 1.88.0

Not currently supported, but its a cool idea.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

For drag and drop of vscode.TreeItem on editor , Can we implement  vscode.DocumentDropEditProvider in order to insert custom data based on vscode.TreeItem?

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


NA values are generated with wrong probability

I believe the sign here should be reversed:
https://github.com/wesm/bitmaps-vs-sentinels/blob/master/benchmark.cc#L104

Ah, nvm -- I didn't notice that you encode bit 1 as an indicator of a valid value, rather than indicator of an NA.


ThreadLocal for React class

Hi,
as discussed I'm sending the pull request for issue#3 (https://github.com/winterbe/spring-react-example/issues/3)
commit f4407f5 contains the discussed change (using ThreadLocal)
commit dcbfaab increases version of spring boot, adds remote debug port (5005) and support for spring-loaded (allows much better hotswap when debugging/developing)

Thanks for your contribution.


Feat closeonpartnerclose fixes#610

WARN: Despite the title this PullRequest does not handle a user-close of a connection yet (# 569)
fixes #610 -> GUI should react to a partner close:
Testing:

Create new need

Connect with the DebugBot Request

type "close"
result -> you should receive a message that will close this connection (message should disappear from the gui)

Create new need

Close Post
result -> you should receive messages that close the respective connections -> the messages are marked as #Closed in the state (visible via console)

For the record - the case of the partner deactivating her need (causing a connection close) is not handled, either (FromSystem -> FromExternal) - but with your description of the PR that was kind of expected


fix: avoid using deprecated jQuery functions to prepare for jQuery 4 …

…(#779)

some DOM check functions were rewritten to use native DOM API
jquery 3 dependency was removed from package.json

Afaik based on https://blog.jquery.com/2024/02/06/jquery-4-0-0-beta/ invocation of $.each is not deprecated.
I tried to analyze gmail.js and as far as I can see, there is only $.isArray() invocation need to be replaced.
Would it make sense to simplify this PR so it only replaces $.isArray with Array.isArray?
With such change gmail.js fine with jQuery 4 beta in my local env.

VERY nice work in this PR. This is taking Gmail.js in a general direction I'm very much agreement about, and I'm glad to see someone having the initiative to make it happen. Thank you VERY much!
I've left some comments in the PR code, which I think is worth considering or addressing, but have no doubt, that this is a PR I would like to see landed.
Let's do this!

Btw, regarding the trusted types, as I can understand from the jQuery changes, users of jQuery need to provide "trusted" HTML, i.e. ´$(el).html(trustedHtmlHereInChromeOnly)´, jQuery will not do it automatically.
So it needs to be done on the gmail.js side.

Btw, regarding the trusted types, as I can understand from the jQuery changes, users of jQuery need to provide "trusted" HTML, i.e. $(el).html(trustedHtmlHereInChromeOnly), jQuery will not do it automatically.
So it needs to be done on the gmail.js side.

And it needs to be dispatched like that only when in Chrome... Sounds like a new internal helper-method?
And then if we implement that anyway... We might as well not use jQuery for the task at all?

Regarding TrustedHTML support, I believe this could be done like:
const rawHTMLPolicy = window.trustedTypes?.createPolicy('forceInner', {
	createHTML: input => input
});
...
$(el).html(rawHTMLPolicy ? rawHTMLPolicy.createHTML(html) : html)

Simply amazing. Very nice!
One slight nitpick left (which is my fault, really)... But apart from that, this is looking great.
That said, this version may not be a drop-in replacement for existing versions without risk of breaking... So based on that I guess this should be a 2.0 release (following SemVer conventions).
Do you want me to push a non-breaking fix for the reply-menu thing we just merged first, or should we take them both in one go? I'm open to both options.
Also: We definitely need a changelog-item for this. You fix? :smile:

Also: Thinking about loud the elimination of jQuery as a dependency completely (and thus from our API), I think it could be achieved in a fairly non-breaking way, by accepting an "output-transformer" type parameter.
Said in TypeScript, roughly like this:
type Transformer<TElem extends HTMLElement, TResult> = (el: TElem) => TResult;
declare class Gmail<Transformer> { ... };

const gmailNoJquery = new Gmail(); // implied identity transformer (i => i)
const element = gmailNoJquery.dom.compose().$el; // HTMLElement

const gmailJquery = new Gmail(el => $(el));
const element = gmailNoJquery.dom.compose().$el; // JQuery

That exact code might not compile, but you get the general idea.
Basically jQuery is something you need to bring if you want it, and if you do, you get to keep 100% the API you used to have.

Hello @josteink,
I believe that this version does not introduce any breaking API changes, for now it would operate as before - jQuery wrappers would be in place, and if consumer has jQuery object in global scope, it would be used. The only change is that jQuery-less mode is made explicit, but I think it's not a frequent use case.
Let's keep 2.0 for the version with element transformers, or even jQuery-less. 🙂 For now I'd say that this PR is mostly a hotfix for jQuery 4 compatibility.
I've updated constructor argument check order and added a CHANGELOG entry.

Regarding your idea with transformers - let's continue in issue #592 discussion. In general, I think that transformers for input and output and for single/multiple elements would be needed - i.e. dom.inbox_content() returns single element, while dom.inboxes() returns multiple. Also, check.is_peoplekit_compose(el) is assumed to expect el as a single element, while you can pass multiple elements wrapped to $. 🙂 At some point I could think that gmail-js 2 should be a complete jQuery-less spinoff, while gmail-js 1 with jQuery support could stay in security/hotfix maintenance mode for a transition period.

I've made some very small ammendments based on comments from @huksley. I've also tested some of the changes which I can test, and they seem good.
The ones I couldn't test was because of TrustedHTML now being enabled on all accounts I've tried, and that breaking the changed code.
Based on that, I think it is safe to merge this, publishing, and then we can start working on solving the bigger issues.
Thanks for all the work so far, guys!


lax.while_loop with multiple conditions

Description
   def cond_fun(t):
        return t < 10 # & True

    def body_fun(t):
        t = t + 1
        return t

    t = lax.while_loop(cond_fun, body_fun, 0)
    print(t)

Hello i need some assistance please.  When running the above code i get the value of 10. However the moment i uncomment the True value. i get Zero.
Is this a normal behavior ? does jax support multiple condition in the condition function ? I am interested in enforcing multiple conditions . Thank you.
System info (python version, jaxlib version, accelerator, etc.)
python 3.8.13
jaxlib 0.4.10+cuda12.cudnn88
acceleraror Nvidia 3090

You can string together multiple conditions using the bitwise_and operator, &:
def cond_fun(t):
    return (t < 10) & True

Note the parentheses, which are not optional in this case: because of Python's operator precedence rules, t < 10 & True is equivalent to t < (10 & True), which is why you were getting unexpected results.


Devfile run command not started if odo push is run after odo push --debug

/kind bug

What versions of software are you using?
Operating System:
Output of odo version:
How did you run odo exactly?
$ odo push --debug
Validation
 ✓  Validating the devfile [47060ns]

Creating Kubernetes resources for component nodejs
 ✓  Waiting for component to start [1ms]

Applying URL changes
 ✓  URLs are synced with the cluster, no changes are required.

Syncing to component nodejs
 ✓  Checking file changes for pushing [493558ns]
 ✓  Syncing files to the component [135ms]

Executing devfile commands for component nodejs
 ✓  Executing install command "npm install", if not running [1s]
 ✓  Executing debug command "npm run debug", if not running [1s]

$ odo push
Validation
 ✓  Validating the devfile [22081ns]

Creating Kubernetes resources for component nodejs
 ✓  Waiting for component to start [2ms]

Applying URL changes
 ✓  URLs are synced with the cluster, no changes are required.

Syncing to component nodejs
 ✓  Checking file changes for pushing [634649ns]
 ✓  No file changes detected, skipping build. Use the '-f' flag to force the build.

Pushing devfile component nodejs
 ✓  Changes successfully pushed to component

if you check the logs, debug command is running not the run command
$ odo log -f
Debugger listening on ws://<IP_ADDRESS>:5858/d6d8cdc8-d412-4f8d-8a29-bf016c22a36d
For help, see: https://nodejs.org/en/docs/inspector
time="2020-08-19T14:10:16Z" level=debug msg="wait program exit" program=debugrun

Actual behavior
odo push should run the run command not debug command
Expected behavior
odo push should stop the debug command and start run command, if run after odo push --debug.
Any logs, error output, etc?

/area devfile


Regarding the issue of zone_spec.json

Hello, I am very interested in this project. What information is in the zone_spec.json file? Is it the array size of tmf8820？

Sorry for the delay, I wasn't notified of this for some reason.
zone_spec.json describes the field-of-view of each of the 9 "zones" in the TMF8820, which is why there are 9 items in the main array. For each zone, it contains:
center: the angular x and y position of the center of the zone, in radians
width: the angular field of view of the zone in the x direction, in radians
height: the angular field of view of the zone in the y direction, in radians
To convert an angular coordinate into a unit vector pointing in the given direction, use this function: https://github.com/uwgraphics/ProximityPlanarRecovery/blob/e8c22d1754edeca12a59c5b54451093a399457e2/util.py#L31
the _px fields describe the pixel position of the zones, following the convention on page 23 of the datasheet. I have a script elsewhere that uses the formula on the same page to derive the angular coordinates from that.

Feel free to comment or reach out if you have other questions.

Feel free to comment or reach out if you have other questions.

Thank you very much for your answer. I have understood the meaning of the parameters in zone_spec.json. After reading your paper, I am very interested in the direct_method. Is the 'center' parameter in zone_spec iterated from the collected dataset or calculated through a formula?

The "center" parameter comes straight from page 23 of the datasheet. For the direct method, we tweak the center location by optimizing over some large dataset (this is described in the paper). You can see in the direct method demo code where we multiply the center location by a factor before using it: https://github.com/uwgraphics/ProximityPlanarRecovery/blob/e8c22d1754edeca12a59c5b54451093a399457e2/direct_method.py#L24. There is a different factor for zones on the edge vs on the corner.


Update error condition for unauthorized error

New scenario when the secret is mistyped when setting up cluster using parameter --aad-server-app-secret

@SenthuranSivananthan : Thanks for your contribution! The author, @iainfoulds, has been notified to review your proposed change.

Thanks, @SenthuranSivananthan
#sign-off


AseCommandBuilder throwing MissingPrimaryKeyException exception

Hi everyone, and sorry for my english in advance....
I work for a financial institution and we have a system developed in VB.NET .NET Framework 4.6.1 working with Sybase ASE 12.5.4 (Adaptive Server Enterprise/12.5.4/EBF 13387/P/NT).
I've been trying out this library and now it seems pretty stable, especially on managing asynchronous processing, so I'm planning to replace the current Sybase.AdoNet45.AseClient.dll with this one.
But I came to an exception which I did not have on Sybase.AdoNet45.AseClient.dll. This happens when using a AseDataAdapter and AseCommandBuilder to INSERT/UPDATE a table wich has no primary key, but does have a unique index.
Debugging the soruce, it ends in this method: GetSchemaTableWithKeyInfo:
throw new MissingPrimaryKeyException("Cannot generate SQL statements if there is no unique column in the source command.");
The table script which I intend to INSERT is this:
CREATE TABLE dbo.ODEBIN  (
ODB_FECNEG	datetime NOT NULL,
ODB_NROLOG	decimal(12,0) NOT NULL,
ODB_TIPOBJ	varchar(5) NOT NULL,
ODB_IDEBIN	varchar(22) NOT NULL,
ODB_TIPMOV	varchar(1) NOT NULL,
ODB_FECORI	datetime NOT NULL,
ODB_TIMORI	decimal(6,0) NOT NULL,
ODB_CODIMP	varchar(1) NOT NULL,
ODB_ORITRX	varchar(40) NOT NULL,
ODB_ORITER	varchar(10) NOT NULL,
ODB_ORIADI	varchar(10) NOT NULL,
ODB_CBU   	char(22) NOT NULL,
ODB_CUENTA	decimal(15,0) NOT NULL,
ODB_ORDEN 	decimal(15,0) NOT NULL,
ODB_TIPCTA	decimal(5,0) NOT NULL,
ODB_SUCURS	decimal(5,0) NOT NULL,
ODB_MONEDA	decimal(5,0) NOT NULL,
ODB_CUIT  	decimal(11,0) NOT NULL,
ODB_IMPORT	decimal(13,2) NOT NULL,
ODB_FECPRO	datetime NOT NULL,
ODB_ESTADO	smallint NOT NULL
)
ON [default]
GO
CREATE UNIQUE NONCLUSTERED INDEX X_ODB_NRLOG
ON dbo.ODEBIN(ODB_NROLOG)
ON [default]
GO
Creating primary keys is not an option. Do you think you can reproduce my case ? Or should I provide more information ?
Thanks in advance
Best regards

Below is a sample code to reproduce the case. Create the following table in the database:
CREATE TABLE TBTEST(Field1 INT,
                    Field2 VARCHAR(20))

CREATE UNIQUE INDEX IXTB_TEST ON TBTEST(Field1)

The following VB.NET code produces the exception:

     Private Sub Button1_Click(sender As Object, e As EventArgs) Handles Button1.Click

        Dim oCommand As New AseCommand

        Dim dsTemp As DataSet
        Dim oRow As DataRow

        Dim daTemp As AseDataAdapter
        Dim cbTemp As AseCommandBuilder

        Try

            oCommand.Connection = oConn
            oCommand.CommandText = "SELECT * FROM TBTEST WHERE 1 = 0"
            daTemp = New AseDataAdapter(oCommand)

            dsTemp = New DataSet
            daTemp.Fill(dsTemp)
            cbTemp = New AseCommandBuilder(daTemp)

            oRow = dsTemp.Tables(0).NewRow
            oRow("Field1") = 1
            oRow("Field2") = "Hello world"
            dsTemp.Tables(0).Rows.Add(oRow)

            daTemp.Update(dsTemp)
            dsTemp.AcceptChanges()

        Catch ex As Exception

            MessageBox.Show(ex.Message)

        End Try

    End Sub

Wonder why we'd need to enforce primary key or unique index at the driver level? The driver shouldn't mandate those.

Really don't know why.... But if intended behaviour is like Sybase.AdoNet45.AseClient.dll it should be fixed.
The exception has logic if the table has no indexes, but this is not the case...
Thanks in advance.


Add quick layperson explanation to each reason

Could we create a short description of what each reason means? Some (like bad ASN, bad NS) can be cryptic to those who don't live around smokey. Just a little editable text field that'd show up on reasons, and maybe on hover for the reasons list on a post.

Gonna reopen this, since the reason I stuck this here was because of this (mod only). We really should have reason descriptions on the post#show (I think) view.


Not yet release to Crates.io

https://crates.io/crates/horaedbproto doesn't contain the 2.0.0 release.
cc @jiacai2050 @tanruixiang

@jiacai2050 This crates still contains information from ceresdb and I don't have access to make changes, can you help with the process of posting to crates?

https://crates.io/crates/horaedbproto/2.0.0 Released


Toggle dragData to true/false

Hi Christoph,
I am looking for an option to toggle the dragging feature. I placed a button for toggle dragData property to true or false and updated the chart but it doesn't work.
By default, it considers the value set on page load. I can't change it afterwards.
Could you please provide me with a solution?

Hi @krishaws,
What you‘re after sounds perfectly doable. However, could you set up a Fiddle/Plunker/etc. with what you got so far, then I can have a look.
Best
Christoph

Hi Christoph,
I couldn't make it in Fiddle, but I have placed the source files in my S3.
Demo: http://dbrestorefroms3.s3-website.eu-central-1.amazonaws.com/
Source: http://dbrestorefroms3.s3-website.eu-central-1.amazonaws.com/sourcecode.zip
Its a Angular project, once downloaded, you need to run "npm install".
You can find the code inside app.components.ts
I combined two plugins here, one for zoom and one for drag(yours).
I have a use case of showing a long chart with scroll and the user need to select a particular range and drag data points within the same range at once.
see this link http://dbrestorefroms3.s3-website.eu-central-1.amazonaws.com/graph.gif - I need to mark a specific range and drag a single data point. But I need all the data points within this range to be dragged. This is my prime use case.
Feel free to ask me if anything is unclear. Your support would be highly appreciated.

Alrigth,
I'm sorry that I won't dig through the code, but the following should get you what you want:
Instead of switching the dragData option to true and false and updating the chart (wich won't work due to the way the update function is working), you could simply return false when you want to disable the dragging in the onDragStart function.
  dragData: true,
  onDragStart: function (e) {
    if (dragDisabled) return false
    // other code in the onDragStart function
  },

Now you can set-up a button that controls the value of the dragDisabled variable. In very basic Vanilla-JS, it may look like this
var dragDisabled = false
var button = document.getElementById("toggleDragButton");
button.onclick = function () {
  dragDisabled = dragDisabled ? false : true
  button.innerText = dragDisabled ? "Drag: Off" : "Drag: On"
};

and that's it.
Obviously this is not production ready code, but I hope it shows how it can be done.
Here's a fiddle of the above put together in one piece:
https://jsfiddle.net/db8myruh/10/
Best,
Christoph

Thanks for this. it helps me to toggle the drag feature on or off by clicking a button. Highly appreciated. However, it doesn't help fix my problem. Because the chartjs-plugin-zoom acts weird (drag selection is not working anymore i when I set dragData to true).
I know your plugin is an awesome tool. If you have any idea, please let me know. Thanks for your time.

Hey again @krishaws,
sorry for taking me so long. I had another look at it your issue today, mainly focusing on the compatibility of the plugin-zoom and the dragData one. As I was fiddling around, I realised that the toughest part of using these two plugins in combination actually lies in the UI design.
For instance, if a user wanted to zoom in on a particular part of the chart, and then drag one of the data points. At first, the user might want to drag the entire chart, so that the data point is in the center, and the pan to zoom. Now what happens if the user accidentally starts dragging right on the data point. How would you determine what the user intents to do? Dragging the data point? Or dragging the entire field of view? I realised that this has a lot of potential for bad UI design -> leading to frustrated users.
So, my current opinion on this is to EITHER allow panning/zooming OR dragging a specific data point. To demonstrate this, I updated the previous fiddle,
https://jsfiddle.net/ehyz5vj9/3/
which allows toggling between the zooming and dragging features.
However, I'm not a UI designer, and if your or anyone else has a better idea of how to combine these features it would definitely be interesting to hear about!
Anyways, as the fiddles demonstrated how to toggle dragData, and how to combine the two plugins, may I close this issue then?

Thanks a lot for your timely help. Although the use case is outside the scope of this plugin. this workaround helped me a lot. Appreciation for your effort.

I know that this ticket is closed, but if found another workaround for this issue, so i wanted to share it :).
In the plugin "chartjs-plugin-zoom", you can add a function to 'mode' properties, so I used it like that:
pan: {
	enabled: true,
	mode: function({ chart }) {
    if(dragPointsEnable)
      return '';
    return 'xy';
  },
}

In this manner, I disable the panning when I want the user to drag points in my graph.


Ultimate Car Mod Not Fully Compatible

Issue Description:
The Ultimate Car Mod (UCM) is not fully compatible with Ender IO (EIO).  I did not see any errors in the output log, and there was no crash.

EIO's conduits and machines do not work on UCM's machines, even when the hopper does.
I can't get any RF Energy to transfer between EIO's blocks and UCM's blocks.

It is possible to use EIO's fluid tanks to transfer fluid in & out of UCM's tanks. However, to get any fluid out of UCM's tanks and into any of UCM's machines, I would have to use UCM's fluid extractor and pipes.
What happens:

I placed the creative capacitor bank next to the Oil Mixer and set all sides to 'push'. There was no RF transfer.
Item conduits won't connect to the Oil Mixer or Blast Furnace.

(if i add anything more here it would be redundant elsewhere)
What you expected to happen:

RF transfer between creative power bank and Oil Mixer.
conduits to connect to machines (i.e. energy conduit to generator)

Steps to reproduce:

try to attach an energy conduit to the Generator, Oil Mixer, Blast Furnace, or Backmix Reactor. It wont work.
try to attach a fluid conduit to the Generator, Oil Mixer, Blast Furnace, Backmix Reactor, Tank, Gas Station, or Split Tank. it wont work.
try to use an item conduit or the item buffer with the Blast Furnace or Oil Mixer. It won't work, but a vanilla hopper will.

Affected Versions (Do not use "latest"):

EnderIO: 5.0.33
EnderCore: 1.12.2-0.5.39
Minecraft: 1.12.2
Forge: <IP_ADDRESS>60
SpongeForge? no (what even is that?)
Optifine? no
Single Player and/or Server? Single Player only (I don't have a server to test it with.)

They most likely aren't using capabilities at all or correctly. Log the issue with them.


[macOS] The position of the SVG image in the menu is slightly off-center.

code:

MenuAction(
                    title: 'with svg',
                    image: MenuImage.withImage((theme, devicePixelRatio) async {
                      final size = theme.size!;
                      final color = theme.color!;

                      const rawSvg =
                          '''<svg width="24" height="24" viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg"><rect height="24" width="24" fill="currentColor"/></svg>''';

                      final PictureInfo pictureInfo = await vg.loadPicture(
                          SvgStringLoader(
                            rawSvg,
                            theme: SvgTheme(
                              fontSize: size,
                              currentColor: color,
                            ),
                          ),
                          null);

                      final image = pictureInfo.picture.toImage(
                        (size * devicePixelRatio).round(),
                        (size * devicePixelRatio).round(),
                      );

                      pictureInfo.picture.dispose();
                      return image;
                    }),
                    callback: () {})

This happens because the SVG is rendered to 24x24 pixels and then you call picture.toImage(32, 32), which does not stretch the image, but it pads it with empty data.
You need some other way to render the SVG for target resolution (32x32 pixels in this case).

This happens because the SVG is rendered to 24x24 pixels and then you call picture.toImage(32, 32), which does not stretch the image, but it pads it with empty data.↳
You need some other way to render the SVG for target resolution (32x32 pixels in this case).↳

I understand what you mean, but I'm a little unsure how to do it. Do you know how to correctly set different sizes of SVGs to a menu image?


Modify String Pattern for Test Discovery

@jelizaga3-gatech commented the below here:

By convention, Visual Studio identifies tests as methods whose names start with test.

Is it possible to modify the process that Visual Studio identifies the string pattern to discover tests? I.e. see python doc here: https://docs.python.org/3/library/unittest.html#test-discovery
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.
ID: b2cc4efe-a1a2-b461-baca-d8acf4957382
Version Independent ID: cc2c9b90-a9b9-29eb-89c2-c9f09521d9e0
Content: Unit test Python code - Visual Studio
Content Source: docs/python/unit-testing-python-in-visual-studio.md
Product: visual-studio-windows
Technology: vs-python
GitHub Login: @JoshuaPartlow
Microsoft Alias: joshuapa

@bschnurr will be able to answer this. He's on leave for a couple of weeks, so he won't be able to respond till he gets back.

This is not possible right now. We are working on pytest support which will have additional features that may support this.
You can follow the progress in #1847

Related to https://github.com/microsoft/PTVS/issues/5425

this can now be customized in new test adapter, in project settings.


Expand the support for tokens to env and other actions

I would very much like to be able to update my environment settings from my build server using the modulus cli. TOKENS seems only supported in the deploy action.
What I would like to do is something like this.
export MODULUS_TOKEN=xxx
modulus env set "tada" "todo"
without the modulus cli asking me about the project name and just rely on the token set.

hi @LarsBuur sorry for the big delay! so just to catch up, currently you are still asked to choose the project when setting  your environment variables. if this is still an issue for you please re-open this issue and we'll look into it. thank you!


demo app shows how to include java sample app as lib

@best-nazar no description at all?
Where is the source of the java files that are packaged inside java-springboot.jar.

"Where is the source of the java files that are packaged inside java-springboot.jar" ....

All Java sample App is a source (like at GitHub).
"this is not a separate app sample, this should be a change set"...
As I understand, I should rebuild all Java sample App to separate Auth sequence and other logic, and replace existent folder.

So,if I understood You right. I'll rebuild the project.  And I will commit changes, so you can check it. If all fine, I'll replace java sample app folder with new branch.


plastanium bridge bug

Describe the bug
Power nodes connects to plastanium bridge even though it doesn't require power
How To Reproduce
put plastanium bridge near a power node or connect it manually
System infomation

OS: Windows 10
Mindustry v146 steam build
Mod Version: 1.7.3

Do you have a picture of it, idk what you just saying, in my mod there is no build name "plastanium bridge"


Owntracks MQTT fails on 102.1

Home Assistant release with the issue:

102.1
Last working Home Assistant release (if known):
101.3
Operating environment (Hass.io/Docker/Windows/etc.):

Hassio, Rpi4
Integration:

Owntracks via Mqtt, Mqtt is fine for all other events
Description of problem:
No Owntracks connection (app iPhone, 6s and 11 pro) appearing since updating  MQTT HA instance to 102.1
in the Owntracks app:
idle Der Vorgang konnte nicht abgeschlossen werden.
(OSStatus-fehler -9807.) {
  "_kCFStreamErrorCodeKey"="-9807";
  "_kCFStreamErrorDomainKey"= 3;
}

downgrading to 101.3 restored (both) my Owntrack devices and all sensors and device_trackers in HA.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
for all sensors:
  - platform: mqtt
    state_topic: !secret owntracks_mqtt_topic
    name: 'Ot Battery'
    value_template: '{{value_json.batt}}'
    unit_of_measurement: '%'
    device_class: battery

  - platform: mqtt
    state_topic: !secret owntracks_mqtt_topic
    name: 'Ot Longitude'
    icon: mdi:map-marker-radius
    value_template: '{{value_json.lon | round(3)}}'

Traceback (if applicable):
idle Der Vorgang konnte nicht abgeschlossen werden.
(OSStatus-fehler -9807.) {
  "_kCFStreamErrorCodeKey"="-9807";
  "_kCFStreamErrorDomainKey"= 3;
}

Additional information:
posted in https://github.com/home-assistant/home-assistant/issues/28464 but that is really about another issue I am not experiencing. SO hope this is a valid reason for the separate issue.

has this been acknowledged yet? Downgraded for the time being, but would really like to go back to 102....

You didn't mention the MQTT Broker that you use.

Sorry bout that, using the HA Mosquito broker.
I've been able to update to 102.2, which seems to have restored the correct MQTT/Owntracks connection.
Please close if desired, would be interested to understand if this was a temporary fluke, or something has changed in HA between the version regarding this connection.

That's weird. I don't see any MQTT related fixes in 0.102.2: https://www.home-assistant.io/blog/2019/11/20/release-102/#release-01022---november-24
Will close this, please re-open if it persists.


[SDMMC] ESP32-S3-N8R8 fails to start a microSD card in SDMMC_FREQ_HIGHSPEED mode (IDFGH-6901)

Environment

Module or chip used:  [ESP32-S3-WROOM-1-N8R8]
IDF version (run git describe --tags to find it): v5.0-dev-1867-g5faf116d26
Build System:         [idf.py]
Compiler version (run xtensa-esp32-elf-gcc --version to find it): xtensa-esp32-elf-gcc (crosstool-NG esp-2021r2-patch3) 8.4.0
Operating System:     [macOS]
Using an IDE?: [Yes (please give details)]: VSCode
Power Supply:         [external 3.3V]

Problem Description
ESP32-S3-WROOM-1-N8R8 module has trouble with microSD cards at 40MHz / 4-line mode. I have tried both an ESP32-S3-DEVKITC-1-N8R8 and an ESP32-S3-WROOM-1-N8R8 module on a printed circuit board.
The PCB is populated with 10KOhm pull-ups on CMD and D0-D3 lines (and I also tried adding an extra pull-up on the CLK line).
The Devkit was tested with just the internal weak pull-ups enabled.
In both cases, and after trying numerous brands, sizes and speed-rated microSD cards, I have found that they can rarely start at 40MHz (SDMMC_FREQ_HIGHSPEED) mode, and when they do, there are block read errors observed.
Expected Behavior
Expected the microSD card to start in HIGHSPEED mode.
Actual Behavior
Most of the times, the card fails to start in 40MHz mode with the following errors:
E (1037) sdmmc_common: sdmmc_init_ocr: send_op_cond (1) returned 0x107
E (1037) vfs_fat_sdmmc: sdmmc_card_init failed (0x107).

After resetting, it can sometimes start at 40MHz, but when proceeding to read data, block read errors eventually occur:
E (3340) sdmmc_cmd: sdmmc_read_sectors_dma: sdmmc_send_cmd returned 0x109
E (3340) diskio_sdmmc: sdmmc_read_blocks failed (265)
E (3342) sdmmc_req: handle_idle_state_events unhandled: 00000008 00000000

Starting the cards at 20MHz mode works fine with no errors.
Steps to reproduce

Setup an ESP32S3-N8R8 module or devkit with the following pin assignments:

const gpio_num_t kSDSCK_Pin = GPIO_NUM_8;
const gpio_num_t kSDCMD_Pin = GPIO_NUM_9;
const gpio_num_t kSDD0_Pin = GPIO_NUM_18;
const gpio_num_t kSDD1_Pin = GPIO_NUM_7;
const gpio_num_t kSDD2_Pin = GPIO_NUM_17;
const gpio_num_t kSDD3_Pin = GPIO_NUM_16;

Attach 10k pull-up resistors to all lines above except for SCK (CLK)
Compile and run the example code below
SD card fails to start at 40MHz, or when it starts, read errors occur

Code to reproduce this issue
      sdmmc_host_t host = SDMMC_HOST_DEFAULT();
      host.max_freq_khz = SDMMC_FREQ_HIGHSPEED;

      sdmmc_slot_config_t slot_config = {
        .clk    = kSDSCK_Pin,
        .cmd    = kSDCMD_Pin,
        .d0     = kSDD0_Pin,
        .d1     = kSDD1_Pin,
        .d2     = kSDD2_Pin,
        .d3     = kSDD3_Pin,
        .d4     = GPIO_NUM_NC,
        .d5     = GPIO_NUM_NC,
        .d6     = GPIO_NUM_NC,
        .d7     = GPIO_NUM_NC,
        .cd     = SDMMC_SLOT_NO_CD,
        .wp     = SDMMC_SLOT_NO_WP,
        .width  = 4,
        .flags  = SDMMC_SLOT_FLAG_INTERNAL_PULLUP
      };

      esp_vfs_fat_sdmmc_mount_config_t mount_config = {
        .format_if_mount_failed = false,
        .max_files = 5,
        .allocation_unit_size = 16 * 1024
      };

      sdmmc_card_t* card;
      esp_err_t mountRes = esp_vfs_fat_sdmmc_mount("/sdcard", &host, &slot_config, &mount_config, &card);
      sdmmc_card_print_info(stdout, card);

Debug Logs
I (957) gpio: GPIO[8]| InputEn: 0| OutputEn: 0| OpenDrain: 0| Pullup: 1| Pulldown: 0| Intr:0
I (963) gpio: GPIO[9]| InputEn: 0| OutputEn: 0| OpenDrain: 0| Pullup: 1| Pulldown: 0| Intr:0
I (973) gpio: GPIO[18]| InputEn: 0| OutputEn: 0| OpenDrain: 0| Pullup: 1| Pulldown: 0| Intr:0
I (983) gpio: GPIO[7]| InputEn: 0| OutputEn: 0| OpenDrain: 0| Pullup: 1| Pulldown: 0| Intr:0
I (991) gpio: GPIO[17]| InputEn: 0| OutputEn: 0| OpenDrain: 0| Pullup: 1| Pulldown: 0| Intr:0
I (1001) gpio: GPIO[16]| InputEn: 0| OutputEn: 1| OpenDrain: 0| Pullup: 0| Pulldown: 0| Intr:0
E (1037) sdmmc_common: sdmmc_init_ocr: send_op_cond (1) returned 0x107
E (1037) vfs_fat_sdmmc: sdmmc_card_init failed (0x107).

Other items if possible

[x] sdkconfig file (attach the sdkconfig file from your project folder)
sdkconfig.zip

Thanks @igrr, I've applied the patch and I can confirm that it now consistently starts at 40MHz and no read errors are observed (I did not test extensively however).
Type: SDHC/SDXC
Speed: 40 MHz
Size: 15193MB
CSD: ver=2, sector_size=512, capacity=31116288 read_bl_len=9
SSR: bus_width=4```

Hey all, I think I might have a similar issue.  I'm using an ESP32-S3 as well (specifically the ESP32-S3-DevKitC-1) and can't get it to talk to a microSD card.  Card's connected over the following pins:

MOSI: GPIO11
MISO: GPIO13
SCLK: GPIO12
CS: GPIO10
The SD card breakout board includes 10kohm pullup resistors on each of those lines, which seems to be necessary for this particular devkit.

I'm getting the same error message that ThanosSlopoudis is in the initial issue, however, applying Igor's patch did not fix the issue for me.  The only difference I've noticed is that I haven't yet tried SDMMC_FREQ_HIGHSPEED, only SDMMC_FREQ_PROBING and the default, but I'll give that a try as well.
esp-idf version: 5963de1c plus Igor's patch from above
Full code:
 39 esp_err_t autoconfigure_usd_card(void) {
 40     esp_err_t err;
 41 #if USDLOG_USE_ADV_MOUNT // false
 42     err = configure_usd_card();
 43 #else
 44     ESP_LOGW(SDTAG, "proceeding without SD card error checking!");
 45     setup_spi();
 46     esp_vfs_fat_sdmmc_mount_config_t mount_config = {
 47 #if USDLOG_FORMAT_IF_MOUNT_FAIL
 48         .format_if_mount_failed = true,
 49 #else
 50         .format_if_mount_failed = false,
 51 #endif
 52         .max_files = MAX_OPEN_FILES,
 53         .allocation_unit_size = ALLOC_UNIT_SIZE
 54     };
 55     // do not init -- output pointer argument
 56     sdmmc_card_t *card;
 57     const char mountpoint[] = MOUNTPOINT;
 58     sdspi_device_config_t slot_config = {
 59         .host_id = SPI2_HOST,
 60         .gpio_cs = USDLOG_PIN_CS,
 61         .gpio_cd = SDSPI_SLOT_NO_CD,
 62         .gpio_wp = SDSPI_SLOT_NO_WP,
 63         .gpio_int = SDSPI_SLOT_NO_INT
 64     };
 65     sdmmc_host_t host = SDSPI_HOST_DEFAULT();
 66     slot_config.host_id = host.slot;
 67
 68     err = esp_vfs_fat_sdspi_mount(
 69             mountpoint,
 70             &host,
 71             &slot_config,
 72             &mount_config,
 73             &card);
 74 #endif
 75     if (err == ESP_OK) {
 76         sdmmc_card_print_info(stdout, card);
 77     }
 78     return err;
 79 }

(the whole #if USDLOG_USE_ADV_MOUNT thing is to allow development to continue while using the all-in-one example function while also working on the production-safe reimplementation of it, as suggested in the API reference.  I'm getting the exact same behavior in both methods right now.)
sdkconfig.txt
Apologies if my code makes no sense -- it's seen better days :)

@mishaturnbull I'm sorry, this issue isn't related to the problem you are seeing. Your code uses sdspi driver while this issue is about sdmmc. My patch won't help because it modifies the sdmmc driver.
Please try to run examples/storage/sd_card/sdspi from the master branch, adjusting the pins in menuconfig for your hardware. If that doesn't work, please open new issue and post the details, including console output at debug log level. Thanks!

Hi @igrr
In my Arduino code following the SDMMC test example; https://github.com/espressif/arduino-esp32/blob/master/libraries/SD_MMC/examples/SDMMC_Test/SDMMC_Test.ino
for our custom designed ESP32S3Wroom1u with 16 MB of flash and OPI PSRAM
I have  added the following to set pins
if(!SD_MMC.setPins(47, 48, 39, 40, 0, 38)){   // added setting pins and testing if SDMMCFS class bool setPins() returns False when setting pins
Serial.println("Setting Pins Failed");
return;
}
#define SOC_SDMMC_USE_GPIO_MATRIX;
#define BOARD_MAX_SDMMC_FREQ SDMMC_FREQ_DEFAULT
But for
if(!SD_MMC.begin()){
Serial.println("Card Mount Failed");
return;
}
I get the following error:

E (212) sdmmc_common: sdmmc_init_ocr: send_op_cond (1) returned 0x107
E (212) vfs_fat_sdmmc: sdmmc_card_init failed (0x107).
[   349][E][SD_MMC.cpp:138] begin(): Failed to initialize the card (0x107). Make sure SD card lines have pull-up resistors in place.
SD Card Mount Failed
Although we have added external pull-up resistors.
Any comments on how to fix this issue?
Thanks

@Paryavi It seems that the card init fails at a very early stage. I think it's unrelated to the issue of high-speed mode being discussed here.
Please check the connections carefully, and use an oscilloscope or a logic analyzer to check if the card responds on the D0 line. You can open the issue in https://github.com/espressif/arduino-esp32 for further support.

I ran into this issue as well with my PCB with a N32R8V module. The patch posted above fixed it and it now seems to mount every time.

@ThanosSiopoudis @ntwallace Thank you for reporting the issue! Could you please try to apply the attached patch and see if it improves the situation?
0001-Draft-sdmmc-fix-unreasonable-i-o-clock-phase-setting.patch.txt

@igrr Can this patch be applied to Stable 4.4.2?  Facing the same issue with ESP32-S3-N8.   Compiling with latest master is ok, compiling with 4.4.2 can only mount if 20 MHz and always fails on 40 MHz.   Using SDMMC/4-bit.  We must use the 4.4.2 as the ESP-ADF is not yet fully supported with v5.1 latest (any time frame for v5 to support ADF? or patches?)
Thanks

E (212) sdmmc_common: sdmmc_init_ocr: send_op_cond (1) returned 0x107

Hi @Paryavi ,
I'm having the same problem on a ESP32-S3-WROOM-1U-N16R8. Did you manage to fix it?
Set custom pins according to schematic and function contructor:
bool set_pins = SD_MMC.setPins(SDMMC_CLK, SDMMC_CMD,\                                  SDMMC_DAT_0, SDMMC_DAT_1,\                                  SDMMC_DAT_2, SDMMC_DAT_3);
Schematic is as follows:

Applying the patch didn't work. Microsd formatted as fat32 and exfat, same negative result. Microsd_det works correctly. Speed is 10kHz.
Thanks in advance.


Enzyme Internal Error: configured enzyme adapter did not inherit from the EnzymeAdapter base class

Has anyone seen this error: Enzyme Internal Error: configured enzyme adapter did not inherit from the EnzymeAdapter base class
Using Enzyme version 3.11.0 and here's my simple setup:
import { configure } from 'enzyme';
import Adapter from '@wojtekmaj/enzyme-adapter-react-17';

configure({ adapter: new Adapter() });

Update- downgrading to Enzyme 3.10.0 seems to have fixed this. Curious if it's my setup or a widespread issue.

Given that this package has been downloaded 8.1 million times already I think I would have known about this if it was widespread.
My best guess is that you may have had multiple versions of Enzyme installed, and as the result the Enzyme @wojtekmaj/enzyme-adapter-react-17 inherited from was not the same one that checked the inheritance, causing said error.

Fair enough, I caught the issue here as well and verified that I only had one version of Enzyme in the package-lock, but I don't doubt it's something I'm doing. Thanks for the quick response 👍

I'll be happy to help out if you need, but I'll need a repro repo to work with.


ssh_box: Shutdown container when fail to start ssh session

I have seen this many times. Although the chance is low, but sometimes OpenDevin fails to start a ssh session, and exits without shutting down containers, leaving them as zombies. This PR makes sure the close method is called before it shuts down.

@rbren I remember you also saw zombie sandboxes a few times before - hope this would solve the problem.


cmd/go: unable to get golang.org/x/... packages

I am unable to get packages hosted in go.googlesource.com, I can't even clone the repositories by hand, getting the same error.
What version of Go are you using (go version)?
go version go1.10.2 linux/amd64
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/sanz/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/sanz/go"
GORACE=""
GOROOT="/home/linuxbrew/.linuxbrew/Cellar/go/1.10.2/libexec"
GOTMPDIR=""
GOTOOLDIR="/home/linuxbrew/.linuxbrew/Cellar/go/1.10.2/libexec/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc-5"
CXX="g++-5"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build001979355=/tmp/go-build -gno-record-gcc-switches"
What did you do?
$ go get -u golang.org/x/crypto/

What did you expect to see?
I expected to get crypto installed
What did you see instead?
$ go get -u golang.org/x/crypto/
# cd .; git clone https://go.googlesource.com/crypto /home/sanz/go/src/golang.org/x/crypto
Cloning into '/home/sanz/go/src/golang.org/x/crypto'...
fatal: unable to access 'https://go.googlesource.com/crypto/': Unrecognized content encoding type. libcurl understands identity content encodings.
package golang.org/x/crypto: exit status 128

@agnivade have the latest curl and git versions, no proxies. I can clone git repositories from github, but not from go.googlesource.com:
$ curl --version
curl 7.60.0 (x86_64-pc-linux-gnu) libcurl/7.60.0 OpenSSL/1.0.2o
Release-Date: 2018-05-16
Protocols: dict file ftp ftps gopher http https imap imaps pop3 pop3s rtsp smb smbs smtp smtps telnet tftp
Features: AsynchDNS IPv6 Largefile NTLM NTLM_WB SSL TLS-SRP UnixSockets HTTPS-proxy

$ git --version
git version 2.17.0

Other people in my company are getting the same issue since Sunday

Does the issue only occur inside your company? You may need to talk to your network administrators and check that they are not filtering your traffic.

@davecheney I am working remotely, we are in different countries/buildings. I am not filtering my traffice :)

Output of plain curl
$ curl https://golang.org/x/crypto
<!DOCTYPE html>

<meta http-equiv="Content-Type" content="text/html; charset=utf-8"/>
<meta name="go-import" content="golang.org/x/crypto git https://go.googlesource.com/crypto">
<meta name="go-source" content="golang.org/x/crypto https://github.com/golang/crypto/ https://github.com/golang/crypto/tree/master{/dir} https://github.com/golang/crypto/blob/master{/dir}/{file}#L{line}">
<meta http-equiv="refresh" content="0; url=https://godoc.org/golang.org/x/crypto">

Nothing to see here; move along.

``

@GonchuB what was the fix for you?

Some keg (using linuxbrew) installed git and curl as a dependency, so those binaries were being used instead of the system ones. Apparently they were not working correctly (it is odd, because they worked correctly with github repos and go packages) but not with go.googlesource ones. Uninstalled git and curl kegs and issue was solved

I am glad to hear that you found the solution!

Wow, weird indeed.

@GonchuB Thank you for saving me hours of banging my head against the wall! Just had (and resolved) the same issue.


Very strange: some css classes are removed some are not

For example I have non existing css class regexp-main. It's not deleted from outfile.
When I rename it to regexp-mama it gets deleted.
Some more examples of classes which aren't deleted:
Being deleted | Not being deleted
regexp-colxumn | regexp-column
non-existing-clxass | non-existing-class

Yes, I'm also seeing a few classes being left behind at random.
But I haven't seen anything removed that should not be, which is good.

I added a pullrequest to fix this problem by giving the opportunity to add custom regex to filter a wider range of classes https://github.com/purifycss/purifycss/pull/190
@deser in your case, you probably have some other classes with the word main in it. That is why it is not getting filtered. It interferes with some of the restrictions that are made in the purify plugin:
 classes that are added together
    ✕ can find .added-together (5ms)
    ✕ can find .array-joined (1ms)

this is a function so you can combine classes by php / javascript. For example
<?php echo 'regexp-'.'main'; ?>

I have same problems. My non-removed classes are:
i3
i5
i7
fb

and many others

👀


log-file option, status display

No real issue, just a picky wish.

when using the --log-file switch, I would like to see the messages on screen, too.  Screen is for now, log file is "for ever", for search.
it might be a bit nicer to vertically align the "msg=" string on screen and log when using --verbose.

What do you think?

Hi @gitgrub.
Whilst I always try satisfy every request, there are limits ;-)

when using the --log-file switch, I would like to see the messages on screen, too. Screen is for now, log file is "for ever", for search.

No sorry, the whole point of the --log-file is to prevent the console from "getting spammed", and of course to keep a more permanent and easily accessible record (log file). If you wish to see a "live version" you can always tail -f <logfile> which should give you realtime events.

it might be a bit nicer to vertically align the "msg=" string on screen and log when using --verbose.

Whilst I agree with you from a cosmetic point of view, it's not supported. The log format is determined by logrus which states "the output is compatible with the logfmt format" - so clearly this is a defined format, I suspect to allow parsers to process the data, and of course because log files are not designed to be pretty.
Sorry, I can't really help you here...


chore: Remove old inbox and AROptionsBidManagement feature flag

The type of this PR is: chore

This PR resolves CX-2289
Description
Remove old inbox and AROptionsBidManagement feature flag.
PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests/stories for my changes, or my changes don't require testing/stories, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[ ] I have added a changelog entry below or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Remove old inbox and AROptionsBidManagement feature flag - ole

This PR contains the following changes:

Dev changes (Remove old inbox and AROptionsBidManagement feature flag - ole)

  Generated by :no_entry_sign: dangerJS against 2e111023f9d3867e18d85989f01128d31a715a43

#squashongreen


Package FLOSS standalone binary for OSX

We should be good to go on this now.

travis has an osx build environment, so we may be able to use that to do nightly builds


Update readme.md

Has been added a new category (Redux) to the "Front-end Development" section. It's the very famous and popular approach for building an application like in Flux of Facebook.

Good potential, but the readme is super messy and could use a lot of cleanup. The examples entries could use a description, on the last one you've just included a bunch of links, which seems very lazy. Below the examples, there's suddenly a lot of entries with huge headers. Just generally very hard to read.

Please read the contributing guidelines and the awesome manifest.

Going to pass on this for now as it needs a lot of work. Ping me here in a month and I'll reconsider.


just a function to list the queue

Hey @mmatera, I think this is a nice feature to add! I have two minor suggestions:

Could you move the import subprocess line to the top of base.py underneath import os
Could you rename list_queue to get_queue

If you don't have the time, I'd be happy to make these changes myself. Thanks!

Awesome, looks good

Adds feature associated with issue #11


Update calc.py

[+] Show the calculation options in a neater way
[+] Print the equations and answers together for an easier to understand UI

I checked the code. Good work on the UI here!
Just one change I would suggest is that we should keep the option of continuing calculations for the user. In your program only one calculation can be performed, after which the program ends. Could please make it so that the user can choose when to stop? Thanks!

Hi! Thank you so much for the feedback; it’s motivating! 😄
I’ve fixed the issue & added extra stuff.

Hi! Thank you so much for the feedback; it’s motivating! 😄
I’ve fixed the issue & added extra stuff.

Thank you!! 😃


Header not included in swagger

This is how I created the docs page.
	const spec = await generateOpenApiDocument(openApiRouter, {
		title: 'ZSA OpenAPI',
		version: '1.0.0',
		baseUrl: env.NEXTAUTH_URL,
		securitySchemes: {
			auth: {
				type: 'apiKey',
				in: 'header',
				name: 'Authorization',
				description: 'Bearer token for authentication',
			},
		},
	})

Hi, can you make sure you have protect: true on your route? I think if that is not on then these won't get passed through.

Thank you :)


feat: onboard directus

Summary

Onboard directus https://github.com/pluralsh/plural-artifacts/issues/536

Test Plan

Deployed helm chart + terraform locally

Next

Set docker image from plural registry
Configure postgres backend
Configure file storage backend (s3, gcs, blob)
Configure OIDC

Checklist

[ ]  No images hosted from dockerhub
[ ]  Are dashboards present to understand the health of the application.  There must be at least 1 of these

[ ]  all databases should have dashboards
[ ]  ideally also have at least cpu/mem utilization dashboards for webserver tier of the app
[ ]  you can use plural from-grafana to convert a grafana dashboard found via google to our CRD

[ ]  Are scaling runbooks present

[ ]  all databases must have scaling runbooks
[ ]  you can use the charts in pluralsh/module-library to accelerate this

[ ]  do you need to add config overlays?

[ ]  inputing secrets
[ ]  configuring autoscaling

[ ]  If there’s a web-facing component to the app, we need to support OIDC authentication and setting up private networks if no authentication option is viable
[x]  All major clouds must be supported

[x]  Azure
[x]  AWS
[x]  GCP

We should definitely use the postgres operator to set up a proper db here as well, you should be able to see a lot of prior art using the module-library postgres chart that wraps a lot of the helpful resources together for it

you can see some postgres related templating here: https://github.com/pluralsh/plural-artifacts/pull/635/files
jupyterhub was weird because we had to feed it the entire postgres url, which isn't needed for directus, looks like you can just wire up these env vars: https://docs.directus.io/self-hosted/config-options.html to give it the right db configuration.  You'll almost certainly need to add DB_SSL__REJECT_UNAUTHORIZED=false too as an env var
it looks like directus does require s3/gcs/etc so might want to revert the kube terraform module and go back to the standard cloud specific ones too


Add Glide custom component support

This PR adds a new kind of GlideImageLoader class called GlideCustomImageLoader which allow the consumer of the lib to specify a concrete GlideModule associated with a previously registered glide component.
That way, the lib is able to load resources as the client specifies, in the example added we request an specific image size by appending two parameters at the end of url.
Regarding implementation, I've opted to make a new class instead of adding code to the original one (that was my first approach) and messing the code, hope it's the less invasive as possible.

Thanks for your contribution, I'll check it later.

And could you please rebase to the latest master branch? Thanks!

@Piasy I've done the changes you requested 👍

@SmasSive Could you please also add description about how to implement custom image request url and more customization, by utilizing classes added by this PR, into the README?

@Piasy done! I'm not Shakespeare so... Feel free to modify the README as you want!

v1.5.2 is published, check it out! Thanks for your contribution!

GREAT! A pleasure!

Hey @Piasy ! Looks like the module GlideImageViewFactory is not well published: http://dl.bintray.com/piasy/maven/com/github/piasy/GlideImageViewFactory/1.5.2/

It's strange that bintray didn't publish it automatically, I've published it manually, please try again.

Ok, now it's working! Thanks!