v3-beta7: Running ionic start without ionic login first ends with error, no "go to directory" message

What version of the CLI are you using? Output of ionic --version
3.0.0-beta7
Short description of the problem:
Calling ionic start without ionic login first ends with error, no "go to directory" message:
> ionic start
? What would you like to name your project: cliBeta7
? What starter would you like to use: blank
✔ Creating directory C:\Users\Jan\Documents\cliBeta7 - done!
✔ Downloading 'blank' starter template - done!
✔ Updating project dependencies to add required plugins - done!
✔ Creating configuration file for the new project - done!
✔ Executing: npm install within the newly created project directory - done!

♬ ♫ ♬ ♫  Your Ionic app is ready to go! ♬ ♫ ♬ ♫

Run your app in the browser (great for initial development):
  ionic serve

Run on a device or simulator:
  ionic cordova:run ios

Test and share your app on a device with the Ionic View app:
  http://view.ionic.io

[ERROR] You are not logged in! Run 'ionic login'.

If I run any of the output commands, I will get an error as I am not in the correct directory.
What behavior are you expecting?
Better flow, probably even interactive UI to login or signup - or skip. If you are already logged in you get this much better experience:
> ionic start cliBeta7_2 blank
✔ Creating directory C:\Users\Jan\Documents\cliBeta7_2 - done!
✔ Downloading 'blank' starter template - done!
✔ Updating project dependencies to add required plugins - done!
✔ Creating configuration file for the new project - done!
✔ Executing: npm install within the newly created project directory - done!

♬ ♫ ♬ ♫  Your Ionic app is ready to go! ♬ ♫ ♬ ♫

Run your app in the browser (great for initial development):
  ionic serve

Run on a device or simulator:
  ionic cordova:run ios

Test and share your app on a device with the Ionic View app:
  http://view.ionic.io

? Link this app to your Ionic Dashboard to use tools like Ionic View? No
? Would you like to help Ionic improve the CLI by providing anonymous usage and error reporting information? Yes

Go to your newly created project: cd C:\Users\Jan\Documents\cliBeta7_2

Steps to reproduce:

Install the v3-beta7 CLI
Execute  ionic start

Post the output of ionic info below please
  Your system information:
    Cordova CLI: Not installed
    Ionic Framework Version: 3.0.1
    Ionic CLI Version: 3.0.0-beta7
    ios-deploy version: Not installed
    ios-sim version: Not installed
    OS: Windows 10
    Node Version: v7.9.0
    Xcode version: Not installed

I think you are correct. This should not be an error. I have tagged this as a bug.

Thank you, this has been fixed and will be included in the next release (beta8).


Workflow Fixes

Hopefully this gets the GitHub Actions pipeline working correctly again 🤞

Unfortunately it looks like we are still getting the GitHub auth failures from Composer. Still researching, unclear if this is a rate limiting issue, a race condition, or some other weird Actions <-> Composer interaction.


Indian FinTech companies

Company
Website
Category

InCred
https://incred.com
Lending

Lendingkart
https://www.lendingkart.com/
Lending

Capital Float
www.capitalfloat.com
Lending

NeoGrowth Credit
https://www.neogrowth.in
Lending

Flexiloans
https://flexiloans.com/
Lending

MoneyTap
https://www.moneytap.com/
Lending

CreditMantri
www.creditmantri.com
Lending

Indifi Technologies
www.indifi.com/
Lending

ZestMoney
https://zestmoney.in
Lending

LoanTap
https://loantap.in
Lending

Rubique
www.rubique.com
Lending

Faircent
https://faircent.com
Lending

Early Salary
https://earlysalary.com
Lending

IndiaLends
https://indialends.com
Lending

RedCarpet
www.redcarpetup.com
Lending

Loan Frame Technologies
https://www.loanframe.com
Lending

CreditVidya
https://creditvidya.com
Lending

ePaylater
https://epaylater.in
Lending

(Paytm Wallet; One97 Communications)
https://paytm.com
Payments

ItzCash Card
https://itzcash.com/
Payments

MobiKwik
https://mobikwik.com
Payments

Financial Software & Systems
https://www.fsstech.com/
Payments

Ezetap Mobile Solutions
https://corp.ezetap.com/
Payments

Fino Payments Bank (Fino PayTech)
https://www.finopaytech.com/
Payments

Mswipe Technologies
www.mswipe.com
Payments

Electronic Payments and Services
https://electronicpay.in
Payments

CCAvenue
https://www.ccavenue.com/
Payments

Razorpay Software
https://razorpay.com
Payments

Chillr
https://chillr.com
Payments

ChargeBee Technologies
https://www.chargebee.com/
Payments

Juspay
https://juspay.in/
Payments

AirPay Payment Services
https://airpay.co.in
Payments

Instamojo
https://instamojo.com
Payments

Paynear
https://paynear.in
Payments

TruPay
https://trupay.in/
Payments

BankBazaar
https://bankbazaar.com
Personal Finance Management

ClearTax
https://cleartax.in
Personal Finance Management

FundsIndia
www.fundsindia.com
Personal Finance Management

MoneyView
http://www.moneyview.in
Personal Finance Management

Scripbox.Com India
www.scripbox.com
Personal Finance Management

Fisdom
www.fisdom.com
Personal Finance Management

Active.ai
https://active.ai
AI

PolicyBazaar
www.policybazaar.com
Insurance

SecureNow TechServices
https://securenow.in/
Insurance

Easypolicy
https://easypolicy.com
Insurance

RenewBuy
www.renewbuy.com
Insurance

Upstox
https://upstox.com
Online Trading Platform

Would you be open to submitting a pull request for this list?


Support patch as an action

I've been seeing more and more use cases in the field where having proper oc patch support in applier would be really valuable. We frequently run into scenarios where we need to apply small changes to resources that are not fully managed by applier. Things like:

default cluster level resources that were created by the installer
resources that are managed or controller by controllers or operators where the state may have changed since the last applier run.
resources that have immutable fields and therefore cannot be 'applied'

In this case, it would be useful to be able to pass to applier a minimal resource definition that just identifies a resource by name, e.g.
kind: Route
apiVersion: v1
metadata
  name: myroute

and a file defining the patch
metadata:
  annotations:
    openshift.io/random-flag-for-filtering-purposes: hello

The challenge is that currently, oc patch does not support passing a patch as a file, only as a JSON string. So applier would need to provide a new parameter in which to provide the path to the patch file, and then convert the file contents into a JSON string to pass to oc.
This functionality could provide a more generic and more native way of handling things like applying labels to resources, which currently is provided by a separate post hook role that lives in another repo.
@oybed @logandonley @pcarney8

My suggestion would be to just support this by having the proper action set, and make the openshift-applier process any entries with action: patch to source the file and convert to a json string. Hence, an example inventory could look like the following:
openshift_cluster_content:
- object: "my-patch-request"
  content:
  - name: "patching-my-route"
    file: "/path/to/my/file/route_patch.yml"
    action: patch

The file parameter for other actions can currently be either a local file or a URL (the oc command processes either). Hence, the support for patch will also need to do the same, but it will require the openshift-applier to do the proper processing to fetch the file from local or remote source.

@oybed the challenge is that with patch, you need to provide two arguments: the name (or file) specifying the object to be patched, and then the patch string. We don't really have a second parameter to pass currently.

One option could be to overload params and params_from_vars and when action == patch, convert the file or embedded yaml to a json string, and pass that in.

@oybed yeah, I like it.

PR merged - closing issue. (please re-open if needed)


Problems with Automatic Updates on Windows

VSCode Version: 1.8.-insider

VS Code insider from yesterday. Run check for Updates and VS Code exits / crashes after ~1 Minute.
Updated VS Code by downloading the latest installer and run it
Executed Update again
VS Code doesn't exit, however the Menu stays on 'Checking for updates' forever.

Seems to have been a sporadic event.


[web/support] meta og:title conflict in FrostWire's Knowledge Base

Looks like the template is entering the title twice in there. Let's make sure we keep one, and that it is the right one so if/when these articles are shared in social media they appear with the right title, and not all with the same title. (I'd keep the first one and would add a suffix - FrostWire's Support Knowledge Base)

Should be fixed.

that was fast :P Thanks!


Update form handlers to use proper options

Update solidworx/form-handler-bundle to latest BETA version
Update form handlers to specify and use configurations

Coverage decreased (-2.2%) to 24.301% when pulling 3e78b913eacb26461ecc05e5ec8adb52d633ce4c on pierredup:form-handler into a51c455c0ad497b36fac38604956fc313131d447 on CSBill:master.

Coverage decreased (-2.2%) to 24.301% when pulling 3e78b913eacb26461ecc05e5ec8adb52d633ce4c on pierredup:form-handler into a51c455c0ad497b36fac38604956fc313131d447 on CSBill:master.

Coverage increased (+0.2%) to 26.662% when pulling 434d8285aaa1725f37e18c70c2312d0ac017b712 on pierredup:form-handler into a51c455c0ad497b36fac38604956fc313131d447 on CSBill:master.


Fix README

I forgot to remove some "}" in README file

Hehe, can happen. What is that .idea/watcherTasks.xml file?

Oh, it's my webstorm config file. I will remove it and resend PR

Ready. I have removed config file.

Thanks! :shipit:


Better RangeReplacementSubstitution

Description
Improves Host and Path replacements.

Removes usage of per-call TreeMap by flattening RangeReplacementSubstitution to be the range itself and sorts all substitutions at construction time.

All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Not able to install GeoIP

System info:
OS: Debian 10.9
HAproxy: 2.2.14-1~bpo10+1 (from haproxy debian repo)
HAproxy-wi : v<IP_ADDRESS>
I'm trying to install HAproxy-wi + HAproxy (ssh access over localhost...). First i think there is some errors in manual installation manual like example:  "chown -R apache:apache"  i think correct is "chown -R www-data:www-data".
My HAproxy-wi is runnig with added server (localhost), for login i created user haproxy-wi and add him to suders
haproxy-wi	  ALL=(ALL)	  NOPASSWD: ALL
Then i added maxmind key for geoip in to the settings and tried install geoip on selected server. But there was problem with folder permissions. Permission deny  for creating folder in /var/www/.ansible/tmp. So i created folder var/www/.ansible/ and add  www-data as owner. Error is gone now and installation ends with success but when i reload settings i see "GeoLite2 has not installed".
Any help how can i solve this problem ? I want to use allowlist based on geoip in haproxy
Very thanks for help

Have you read this https://haproxy-wi.org/installation.py#ansible ?

Yes, but instead sudo chown apache:apache /usr/share/httpd/.* i use sudo chown www-data:www-data /usr/share/apache2/.*

Try to run the ansible playbook manually

looks like there is a problem with path.
ansible.errors.AnsibleError: Unable to create local directories(/var/www/.ansible/tmp): [Errno 13] Permission denied: b'/var/www/.ansible/tmp'
`File "/usr/lib/python3/dist-packages/ansible/utils/path.py", line 81, in makedirs_safe
File "/usr/lib/python3/dist-packages/ansible/config/manager.py", line 91, in ensure_type
File "/usr/lib/python3/dist-packages/ansible/config/manager.py", line 434, in get_config_value_and_origin
value = ensure_type(value, defs[config].get('type'), origin=origin)
raise AnsibleError("Unable to create local directories(%s): %s" % (to_native(rpath), to_native(e)))
makedirs_safe(value, 0o700)`
path for .ansible folder should be in /usr/share/apache2/.
Ok i try it

Ok with manual edited /var/www/haproxy-wi/app/scripts/install_geoip.sh installation was successful. But there was one missing dependency for ansible, sshpass. I dont use ssh keys for login.

Could you say what did you edit in the script?
And thanks for sshpass

i just created a copy with new name install_geoip_manual.sh and deleted some conditions that i dont need ...
`!/bin/bash
PROXY=
UPDATE="1"
maxmind_key="MAXMINDKEY"
haproxy_dir="/etc/haproxy"
HOST="<IP_ADDRESS>"
USER="haproxy-wi"
PASS="HAPROXYPASS"
SSH_PORT="22"
export ANSIBLE_HOST_KEY_CHECKING=False
export ANSIBLE_DISPLAY_SKIPPED_HOSTS=False
export ACTION_WARNINGS=False
export LOCALHOST_WARNING=False
export COMMAND_WARNINGS=False
PWD=/var/www/haproxy-wi/app/scripts/ansible
echo "$HOST ansible_port=$SSH_PORT" > $PWD/$HOST
ansible-playbook $PWD/roles/geoip.yml -e "ansible_user=$USER ansible_ssh_pass=$PASS variable_host=$HOST PROXY=$PROXY UPDATE=$UPDATE haproxy_dir=$haproxy_dir maxmind_key=$maxmind_key SSH_$
if [ $? -gt 0 ]
then
echo "error: Cannot download GeoLite2 database"
fi
rm -f $PWD/$HOST`
and then just run script from same folder
sh install_geoip_manual.sh

Ok, I got it, thank you


mapping of  a users timezone to their cal dispaly type

DO you know any dataset that can map the user local time zone or country to the cal string display.

You can use intl package to do date formatting into your preferred locale. For example:
import 'dart:async';

import 'package:intl/date_symbol_data_local.dart';
import 'package:intl/intl.dart';

Future<void> main() async {
  print('Init date formatter to France locale');
  await initializeDateFormatting('fr_FR');

  // now
  var now = new DateTime.now();
  print('Day name in fr_FR: ${new DateFormat.EEEE('fr_FR').format(now)}');
  print('Date in fr_FR: ${new DateFormat.yMMMMd('fr_FR').format(now)}');
}

There's another Dart library for getting time zone database but the pub package is outdated. You can refer to its repository here.

The outdated library is blocked by something still to be fixed in Flutter:
https://github.com/dart-lang/sdk/issues/32051
There is actually another way to do all this.
For all i18n and i10n, you can use the golang libs and then expose it as a plugin to Flutter. Its easy to do this using the Method Channels flutter api.
Why do this ?
Well its portable. You can run the exact same code on the mobiles, desktops, servers or embedded.
https://github.com/go-playground/locales
https://github.com/go-playground/universal-translator
This is not everyones cup of tea but its much further along than the Flutter implementation and has good tooling.
Its easy to support translation workflows for creates the language files for Flutter and and other system like react, web, etc.
If you want more info just grab me. Hope your not too upset that i am suggesting using a non dart package

Congratulations! You've been chosen to receive a $7,000 Rewards. 🪂🔴
Trading and Launch Rewards are live on dYdX Chain! ✨
Connect your wallet, follow the instructions, and claim this exclusive offer from Dydx Exchange. For GitHub users only—share with friends to make it eligible. Redeem your reward first.

🚀 Claim Your Rewards Now! Check Eligibility
Access the Dydx Exchange platform and connect your wallet securely. Ensure you're using a trusted wallet for the transaction.

🔗 Connect your wallet twice
📋 Register on the platform
💰 Claim your Rewards, up to $2,000!

@dydxprotocol


Feat/km eliza bot

Relates to:

Risks

Background
What does this PR do?
What kind of change is this?

Documentation changes needed?

Testing
Where should a reviewer start?
Detailed testing steps

no documentation, weird changes, doesn't look like you meant to PR it to the main repo


Allow to add multiple tools with one add function call.

Adding support like npm/yarn/pnpm allows you to do pnpm add @a/b @c/d @e/f for installing tools would improve the DX a lot.
Example:

Currently the command allows two arguments, with the second one being for changing the "alias":
rokit add<EMAIL_ADDRESS>rojo7
rokit add<EMAIL_ADDRESS>rojo6
rokit install

Adding rojo with aliases like this means you can now run these different versions of rojo at the same time:
rojo7 --version # prints 7.4.1
rojo6 --version # prints 6.0.0

All this to say, there already exists a multi-argument variant of this command, and unfortunately it would be both a breaking change and we would be removing the ability to change the alias of a tool. I don't think we will do this unless it turns out that the community does not really care for the aliases functionality.


ANC Subs Visit - Labs - HIV PCR [2h]

Issue
If a patient is pregnant and positive for HIV, we need to do a different type of HIV test on them to track their viral load- how many copies of the HIV virus are present in a certain amount of blood. This is important because the patient's viral load needs to be below a certain threshold by the time they give birth so they do not pass the virus along to their baby. We track the viral load through an HIV PCR test. FYI, this test can take up to a month for the lab results to be returned.

One
Two
Three

Design Files
Designs can be found in Figma here.
To Do:

Create HIV PCR subactivity
Style screens per design

@jeanette-pelizzon
This requires new CT on backed. The effort is 4h.
Please update estimation.

https://nutrition-ihangane.pantheonsite.io/
QA Review
Questions:

Date format is different from design dd/mm/yyyy vs dd-mm-yyyy. Is that a problem?
On the HIV PCR test results page the order of fields is different from design, but it is logical, since the HIV PCR Test Result field only displays when No is selected as an answer for the Are there <20 copies/mm3 question. I suppose this is OK.

Checked and approved:

The screen displays and updates as on the design.

1
2
3
Pause encounter

@balagan73 thanks - proper date is DD/MM/YYYY


Add action to open file in new tab

Currently, I have the editor configured to use preview mode which is useful for browsing files. However, I would like to be able to open a file NOT in preview mode while still having preview mode enabled.

If you option click on the file, it will open in the next editor group over. Does that do what you want?
If not, please provide a more detailed writeup of the expected behavior

No, I don't a new editor. I want to be able to open that file in the current editor as a new tab. I will add this to the initial description.


Update bower file

Could you update the bower file to also include the css file please so we don't have to overwrite it on our bower.

This was resolved by pull request #59.


Upgrade sphinx version by removing sphinx_panels dependency

The documentation used a deprecated sphinx extension called sphinx_panels. This update switches to the updated version called sphinx_design. This should make it so that a later version of sphinx can be used for building documentation, which resolves the jinja2 version issue from #202 and as a side effect also resolves the issue identified in #206.
This worked locally without the need for setting an explicit minimal sphinx version in setup.py, but it might behave differently in the CI.

Based on the documentation test, the changes here did indeed boost the sphinx version from 3.5.3 --> 5.3.0

Codecov Report

Merging #208 (e1a22aa) into main (9294621) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #208   +/-   ##
=======================================
  Coverage   98.39%   98.39%
=======================================
  Files          45       45
  Lines        1803     1803
=======================================
  Hits         1774     1774
  Misses         29       29

Flag
Coverage Δ

unittests
98.39% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Slider giving NaN is an invalid value for the top css style property.

https://codesandbox.io/s/kk3mpx80z5
If you look at the console im getting:
Warning: `NaN` is an invalid value for the `top` css style property.

Check the render method of `Slider`.

slideHeight is used when you use the vertical and fade props on the slider, but slideHeight is not set until componentDidMount of the InnerSlider component, which means that style.top will be set to NaN on the first render. It's probably a bug, but not a problem since it updates right away. It might work if slideHeight is initialized to 0 instead of null, but I don't know for sure.

I reproduced this as well. Working around it for now.

I am getting similar issue....
But I am getting the error for the 'left' property. Below is the full error trace.
But the plugin works.
    Warning: `NaN` is an invalid value for the `left` css style property.
    in div (created by Slider)
    in div (created by Track)
    in Track (created by InnerSlider)
    in div (created by InnerSlider)
    in div (created by InnerSlider)
    in InnerSlider (created by Slider)
    in Slider (at sliderLayout.js:99)
    in div (at sliderLayout.js:98)
    in SliderLayer (at Home.js:20)
    in div (at Home.js:19)
    in Home (created by Route)

I am using below versions of the s/w.
    "react-slick": "^0.26.1",
    "slick-carousel": "^1.8.1"

I am getting similar issue....
But I am getting the error for the 'left' property. Below is the full error trace.
But the plugin works.
    Warning: `NaN` is an invalid value for the `left` css style property.
    in div (created by Slider)
    in div (created by Track)
    in Track (created by InnerSlider)
    in div (created by InnerSlider)
    in div (created by InnerSlider)
    in InnerSlider (created by Slider)
    in Slider (at sliderLayout.js:99)
    in div (at sliderLayout.js:98)
    in SliderLayer (at Home.js:20)
    in div (at Home.js:19)
    in Home (created by Route)

I am using below versions of the s/w.
    "react-slick": "^0.26.1",
    "slick-carousel": "^1.8.1"

This issue is coming for me in my project. But this issue is not coming up in the online playground, where we are using React DOM attachment in the same file.
In our project, we are using the react DOM hooking in the root js.


Panic when attempting to load issue with invalid label

Gitea version (or commit ref): 1.14.0+dev-605-gd2ea21d0d
Git version: 2.30.0
Operating system: Docker
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No

Log gist:

2021/01/26 10:41:58 ...els/issue_comment.go:382:LoadLabel() [W] Commit 839 cannot load label 0
2021/01/26 10:41:58 ...els/issue_comment.go:382:LoadLabel() [W] Commit 840 cannot load label 0

Description
template: repo/issue/view_content/comments:188:51: executing "repo/issue/view_content/comments" at <RenderLabels .AddedLabels>: error calling RenderLabels: runtime error: invalid memory address or nil pointer dereference

Additionally; this is a very old issue that might've had some labels before migrating to org-labels; possibly need migration or doctor command to address this issue.

delete from comment where type = 7 and label_id = 0;

Seems to have solved the issue. The relevant issues/PRs are from around ~2018.

delete from comment where type = 7 and label_id = 0;

Seems to have solved the issue. The relevant issues/PRs are from around ~2018.


Fix Google API timeout in long-lived notebooks

The Google Auth Token returned by the OAuth 2.0 dance has an inbuilt expiry of, by default, 3600 seconds or 1 hour. After this time the notebook will fail to autosave and the notebook list view will no longer update.
The failure to save is shown in the UI by a relatively easy to miss notification and this resulted in loss of work for me after I left a notebook open while I went to lunch.
The Google-blessed approach[1] is to refresh the auth token before expiry. This pull request implements this approach in 3c42a21.
It also implements a defensive measure in 3d88257 to address the failure mode where setTimeout doesn't fire within 45 minutes of wall-clock time. (Such as, for example, when a laptop is suspended.) This measure is to attempt a re-auth on the first 401 encountered when an API request is executed. Subsequent 401s are allowed to propagate back to the caller since they are then assumed to be "true" permission denied responses. (This approach mirrors other Google sample code but is not terribly elegant.)
Finally, 6c5b0c8 is a re-build of all the JavaScript from the modified TypeScript. Since TypeScript appears to have changed its whitespace handling since the last build, it touches all the JavaScript source (sorry) and is broken out into a separate commit for easier review.
[1] https://developers.google.com/api-client-library/javascript/help/faq

+1,
Should we think of removing the .js files as they are generated ?

Should we think of removing the .js files as they are generated ?

We could do but generating them would need to become part of the install step. (I don't think they're generated automatically on demand?) At the moment it's nice that all one needs to do to get Jupyter-drive working is pip install it.

Addendum: and I don't think putting an npm install && gulp step as part of setup.py is wise. It feels like a magic step too far.

No but, if you bublish as wheel, you can generate the JS files at build time. So only install from git (or tarball) requires building. Plus the wheel would be universal, so woudl work on linux.

Plus the wheel would be universal, so woudl work on linux.

But it already works on Linux(!) (Or, at least, pip installing worked for me.)
But, yes, making it a wheel would work well in this case. I've no feeling for how complex adding the magic would be. If it's feasible, maybe opening a new issue to discuss it/request input?

OK, let's merge and see that later.


Installing the NuGet package for profile111 portable class library

The following error occurs during installation:
Could not install package 'Nito.AsyncEx.Tasks 1.1.0'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile111', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.
PCL Profiles:
Profile 111 (.NET Framework 4.5,   Windows 8, Windows Phone 8.1) (netstandard 1.1)

AsyncEx.Tasks only supports netstandard1.3. In particular:

.NET Framework 4.6, not 4.5.
UAP 10.0, not Windows 8 / Windows Phone 8.1.

Did you mean to install Nito.AsyncEx instead of Nito.AsyncEx.Tasks?

Thank you for your usual quick response.  I just want to use AsyncLock in the PCL. I browsed NuGet packages in VS, and found Nito.AsyncEx.Tasks Latest stable 1.1.0.

BTW, I am not interested in supporting W8 or WP8, but unfortunately, there is not such PCL profile that supports UWP without supporting W8 and WP8.  The library is for supporting UWP, Xamarin.iOS and Xamarin.Android.   If one targets Windows Universal 10.0, VS would respond by saying: The following will be automatically targeted because they support the same set of portable APIs: Windows 8.

I have just changed the PCL profile to
Profile 7   (.NET Framework 4.5,   Windows 8) (netstandard 1.1)
It resulted in the following error:
Could not install package 'Nito.AsyncEx.Tasks 1.1.0'. You are trying to install this package into a project that targets '.NETPortable,Version=v4.5,Profile=Profile7', but the package does not contain any assembly references or content files that are compatible with that framework. For more information, contact the package author.

I would recommend just targeting netstandard1.3 instead of using PCLs, then.

Would that work for all UWP, Xamarin.iOS and Xamarin.Android projects?

If one tries to target .Net Framework 4.6, VS would respond by saying: The following will be automatically targeted because they support the same set of portable APIs: .NET Framework 4.5.

netstandard1.3 supports UWP 10.0 and Mono/Xamarin vNext.

Thank you.  I am converting the PCL now.  It does not seem to be straightforward. I am asking for help on a Xamarin forum.

Very sorry.  I have just realized that you were right I need Nito.AsyncEx instead of Nito.AsyncEx.Tasks for AsyncLock.
Now, the problem with Nito.AsyncEx is the following:
Package Nito.AsyncEx 3.0.1 is not compatible with netstandard1.3 (.NETStandard,Version=v1.3). Package Nito.AsyncEx 3.0.1 supports:
  - monoandroid (MonoAndroid,Version=v0.0)
  - net40 (.NETFramework,Version=v4.0)
  - net45 (.NETFramework,Version=v4.5)
  - netcore45 (.NETCore,Version=v4.5)
  - portable-net40+netcore45+sl4+wp71 (.NETPortable,Version=v0.0,Profile=net40+netcore45+sl4+wp71)
  - portable-net40+sl5+win8+wp8+wpa81 (.NETPortable,Version=v0.0,Profile=Profile328)
  - portable-net40+win8 (.NETPortable,Version=v0.0,Profile=Profile5)
  - portable-net45+win8+wp8+wpa81 (.NETPortable,Version=v0.0,Profile=Profile259)
  - portable-net45+win8+wpa81 (.NETPortable,Version=v0.0,Profile=Profile111)
  - sl4 (Silverlight,Version=v4.0)
  - sl5 (Silverlight,Version=v5.0)
  - wp71 (WindowsPhone,Version=v7.1)
  - wp8 (WindowsPhone,Version=v8.0)
  - wpa81 (WindowsPhoneApp,Version=v8.1)
  - xamarinios10 (Xamarin.iOS,Version=v1.0)
  - xamarinmac20 (Xamarin.Mac,Version=v2.0)
One or more packages are incompatible with .NETStandard,Version=v1.3.

AsyncEx does not support netstandard. If you're proceeding with a PCL approach, AsyncEx is the appropriate library. If you're proceeding with a netstandard approach, AsyncEx.Coordination is the appropriate library.

Thank you for the clarification. All is good now finally.
I think the VS NuGet package filtering is completely messed up for PCLs.  I feel it is like that everything shows up regardless of compatibility.
To summarize today's education:
There are two types of PCLs now: 1. traditional profile based PCL. 2. relatively new netstandard based PCL.  They require different types of NeGet packages, but VS NuGet package manager does not know the difference.

The NuGet search is broken in the sense that it will find incompatible packages, yes.
PCLs and netstandard can exist in the same NuGet package, but AsyncEx does not do this.

I created an empty portable library targeting .NETStandard1.4, then created an empty UWP app referencing the portable library.  Everything built well.
Then I added AsyncEx.Coordination to the portable library, built the app again, and got over 10 thousand errors (literally, no exaggeration).  The following are just a few from those 10+ K errors:
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\AppxPackage\Microsoft.AppXPackage.Targets(1924,5): warning APPX1707: No implementation file was provided for the .winmd file 'E:\Users\Hong\Documents\Hong\RandD\VSNET\Xamarin\ClassLibraryAllPlatform\bin\Debug\Windows.Foundation.UniversalApiContract.winmd'. To generate registration information in the app manifest, specify the 'Implementation' metadata on the .winmd reference item in the project file.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\AppxPackage\Microsoft.AppXPackage.Targets(1924,5): error APPX1704: The .winmd file 'Windows.Foundation.UniversalApiContract.winmd' contains type 'Windows.Web.IWebErrorStatics'. The use of the Windows namespace is reserved.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\AppxPackage\Microsoft.AppXPackage.Targets(1924,5): error APPX1706: The .winmd file 'Windows.Foundation.UniversalApiContract.winmd' contains type 'Windows.Web.IWebErrorStatics' outside its root namespace 'Windows.Foundation.UniversalApiContract'. Make sure that all public types appear under a common root namespace that matches the output file name.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\AppxPackage\Microsoft.AppXPackage.Targets(1924,5): error APPX1704: The .winmd file 'Windows.Foundation.UniversalApiContract.winmd' contains type 'Windows.Web.WebError'. The use of the Windows namespace is reserved.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\AppxPackage\Microsoft.AppXPackage.Targets(1924,5): error APPX1706: The .winmd file 'Windows.Foundation.UniversalApiContract.winmd' contains type 'Windows.Web.WebError' outside its root namespace 'Windows.Foundation.UniversalApiContract'. Make sure that all public types appear under a common root namespace that matches the output file name.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\AppxPackage\Microsoft.AppXPackage.Targets(1924,5): error APPX1704: The .winmd file 'Windows.Foundation.UniversalApiContract.winmd' contains type 'Windows.Web.AtomPub.IAtomPubClient'. The use of the Windows namespace is reserved.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\AppxPackage\Microsoft.AppXPackage.Targets(1924,5): error APPX1706: The .winmd file 'Windows.Foundation.UniversalApiContract.winmd' contains type 'Windows.Web.AtomPub.IAtomPubClient' outside its root namespace 'Windows.Foundation.UniversalApiContract'. Make sure that all public types appear under a common root namespace that matches the output file name.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\AppxPackage\Microsoft.AppXPackage.Targets(1924,5): error APPX1704: The .winmd file 'Windows.Foundation.UniversalApiContract.winmd' contains type 'Windows.Web.AtomPub.IServiceDocument'. The use of the Windows namespace is reserved.
2>C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\Microsoft\VisualStudio\v15.0\AppxPackage\Microsoft.AppXPackage.Targets(1924,5): error APPX1706: The .winmd file 'Windows.Foundation.UniversalApiContract.winmd' contains type 'Windows.Web.AtomPub.IServiceDocument' outside its root namespace 'Windows.Foundation.UniversalApiContract'. Make sure that all public types appear under a common root namespace that matches the output file name.

I'm assuming that you mean a .NET Core library targeting netstandard1.4, since there isn't a profile-based PCL that targets 1.4 specifically.
Are you using VS2017RC? When I tried to reproduce this (using a .NET Core library), VS wouldn't let me reference it from the UWP app.
Have you tried installing AsyncEx.Coordination into the app as well?

Yes, I am using VS 2017 RC installed yesterday. I am following this Xamarin blog to create the portable library.  The steps are fairly straightforward:

Create a PCL.
Go to its properties page and click "Target .NET Platform Standard".
Then select .NETStandard1.4

Here is today's saga if you are interested.  I started from .NETStandard1.7 and eventually backed down to .NETStandard1.4.

Follow #71 for updates on the v5 (netstandard) release.


[CERTIFICATE] ui & ux design

Ask us to find free certificates in a specific field that you couldn't find here and we will add it.

Duplicate of #88


Add ThrowExceptionOnFailurePlugin to standardise error behaviour with other mailers

According to their own docs this is how you can set this adapter to throw an error when it fails. Currently, we get no feedback on when email failure happens.
When an error occurs, the mailer returns false which is fine, but thanks to the PR in Craft 3.7, we can capture error messages about a failed email if an exception is thrown.
This would also fix https://github.com/craftcms/postmark/issues/19
This may not be applicable for Craft 4 due to the mailer switch away from SwiftMailer.

Also experiencing this issue, no error message being passed back when the mailer service fails...
2022-06-30 13:27:38 [-][1][-][error][formie] Notification email “Admin Notification” could not be sent for submission “20011”. The mailer service failed to send the notification: “”.

Just another ping about accepting this, it'd be mighty helpful!

pinging too, just ran into this

Merged! Thanks for your patience everyone.
As @engram-design mentioned, this isn't an issue with Craft 4+, as it only affects SwiftMailer.


lib: Fix some typos in comments

Most of them were found by codespell.
Signed-off-by: Stefan Weil<EMAIL_ADDRESS>

thanks @stweil


Update RPC return status

Pulls in latest code in the client to correctly convert error code to http status before transforming to rpc code
Fixes #135

@unleashed - sorry not ready yet, will need to add a test and explanation

@unleashed this is ready for review now.
The way this works now is that a 409 response to from backend is converted to an RPC code 8, RESOURCE_EXHAUSTED. All other codes are converted to 7, PERMISSION_DENIED
I have added an integration test for a limits_exceeded message from backend.
Obviously, as you know, the adapter does not speak HTTP so we return the codes that are equivalent in gRPC. The mappings can be seen here https://github.com/gogo/googleapis/blob/master/google/rpc/code.proto#L109


Support for other runtimes

Support other runtimes by:

Changing all instances of tokio::io::{AsyncRead, AsyncWrite, AsyncSeek} and their extensions to those found inside the futures crate
Moving from the tokio feature to the futures-io feature of async_compression
Creating a submodule tokio that mirrors the main API, but using tokio_utils::compat::Compat to operate on Tokio's types instead

The fs module would be exclusive to this tokio module
Caveat: Compat does not provide an implementation of AsyncSeek.  A small newtype wrapper implementing AsyncSeek could be included in this crate for internal use. Smol's async-compat provides an implementation, but it also tampers with tokio's runtime, so it might not be the best idea.

Gate the tokio module behind a feature flag, perhaps enabled by default
Applying the same changes to your sister crate async_io_utilities

I'll be modifying the crate for use with my project that uses async-std, so if this is something you're interested in, I'd be happy to polish the API and submit a PR.

This is something I'd be interested in but wasn't going to personally tackle until way down the line (ie. I'm focusing on spec compliance primarily). So a PR for this would be very handy & welcomed. 👍

I've forked this repo (it looks like just a few days before #82) to add support for futures-rs without tokio for generating streaming ZIP files in WASM (only writing as I didn't need async reading).
We are using this in Koofr Vault.

Thanks to @jakubadamw and the suggestions in https://github.com/Majored/rs-async-zip/issues/89, this has been implemented.
The base implementation now revolves around futures' IO types & compatibility constructors/type aliases are provided in the tokio module in an attempt to make the switch as seamless as possible.
I'll go ahead and close this as a result. 👍


WIP: openssl: add ubuntu version symbols to openssl-steam

Motivation for this change
This adds STEAM_1.0.0 etc symbols via the version-script.patch from ubuntu to the steam-specific build of openssl.
@FRidh could you see if this fixes the game you were having trouble with, and reference the issue once you've created it?
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

This is a huge patch. Could it be fetched using fetchurl instead of keeping it in tree?

How is this patch going to be maintained? We frequently need to do urgent OpenSSL upgrades so we don't want to have figure out how to update this patch first.

Maybe this patch can be applied to the OpenSSL used by Steam only? It does not seem relevant anywhere else.

This is a huge patch. Could it be fetched using fetchurl instead of keeping it in tree?

I pulled it out of the ubuntu src tarball.  I could probably find a way to automate that, but I'm not sure how reliable it would be (are there suitable permalinks?).  It's 100kb and 4k lines, and 25kb gzipped.  It's basically just a list of all public symbols grouped by the version where they were introduced.
I'll do a bit of investigating to see if it's machine-generated and who/what maintains it.

How is this patch going to be maintained? We frequently need to do urgent OpenSSL upgrades so we don't want to have figure out how to update this patch first.

It shouldn't require any maintenance (for this purpose) unless you are trying to roll-back to pre-1.0.2, or if the steam runtime moves to a newer version, and even then it could only break things in steam-land that need the new symbols.

Maybe this patch can be applied to the OpenSSL used by Steam only? It does not seem relevant anywhere else.

That's what's happening with this patch.  It should have zero impact outside of the steam chroot environment.

This is WIP because @FRidh reported a crash in a commercial game (Wargame: European Escalation) which I don't have access to.  So I'm just waiting for someone to confirm the fix.

The PR causing trouble: https://github.com/NixOS/nixpkgs/pull/20269

@corngood this does not fix the issue.

So no difference?
If you do:
nix-build -A steamPackages.steam-runtime
nm -D result/lib/x86_64-linux-gnu/libcrypto.so.1.0.0 | grep OPENSSL_1

Do you see the version symbols?  I see:
0000000000000000 A OPENSSL_1.0.0
0000000000000000 A OPENSSL_1.0.1

Do  you know if the game is 32-bit?

My machine just broke down so it might take a bit longer before I can test again and answer your questions.

Removing WIP.  I haven't had any problems with it, and I don't see what harm it could cause.

So no difference?

Indeed, there was no difference.
nix-build -A steamPackages.steam-runtime
nm -D result/lib/x86_64-linux-gnu/libcrypto.so.1.0.0 | grep OPENSSL_1

results in the same symbols you showed.

Do you know if the game is 32-bit?

Wargame: ELF 32-bit LSB executable, Intel 80386, version 1 (SYSV), dynamically linked, interpreter /lib/ld-linux.so.2, for GNU/Linux 2.6.24, BuildID[sha1]=595f2525c683f3cd4d7f2a5b3336a4ba62121f3c, stripped

Note that other 32-bit games work fine.

Maybe it's the same problem we were discussing in https://github.com/NixOS/nixpkgs/issues/20726#issuecomment-280199924 ?
Have you tried removing "no-engine" from the steam openssl?

I replied at https://github.com/NixOS/nixpkgs/issues/20726#issuecomment-280916817

I'm going to close this since it doesn't appear to be the solution to anything.  We can discuss other options on the issue.


Table display related docs

Resolves #198
(Screenshot from Dec 9)

@ajhunyady @nicholastmosher
With https://github.com/infinyon/fluvio/issues/1967 resolved, I also modified the example input to use both json objects, and json arrays of objects.
I made a point to set the expectations on the shape of the data.


Change state of all device channels

As far as i dave into source code there's no way to toggle/on/off entire multichannel device. This is client side flaw only.

Hey @BadCoder1337
We can create a helper method to loop over all channels for specific device and run the required action.
While is not ready, you can try to loop every channel returned from getDevice method.


Fix Sphinx python docstring error: text contrib module (#12949)

Description

Fix invalid docstring section header.
Remove copy pasted docstrings.
Move property docstring to getter method.
Fixes #12949

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[X] Changes are complete (i.e. I finished coding on this PR)
[X] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[X] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[X] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@aaronmarkham The original docstring was a copy paste mistake. They should not be there.

@mxnet-label-bot add [Doc]


the generator fails on action between choice points

This valid use case does not seem to work:

When the action() is removed from the transition, it passes.
StateSmith 0.7.16

Can you try / action(); ? The UML spec that we follow requires a forward slash before actions.
More info on behaviors here: https://github.com/StateSmith/StateSmith/wiki/Behaviors

That works. Thanks.


Noticed a minor spelling mistake

don't have write access anymore but this is acceptable from my side. @proficnc?


[Bug] Plugin fails to start

Website never shows up. Simple as that. Using the latest dev build.

Works fine for me, make sure you compiled the plugin correctly.

I didn't compile anything; I just used the latest dev build from Jenkins.

There is no Jenkins for this project? Can you link me?

Well then... it's gone... I swear there used to be one on the code tab in the README.md... Could you explain to me how I would go about compiling this? Do I have to use Eclipse? Or is there an easier way?

No IDEs needed, just the Java 8 SDK.
Just download the source, open a terminal in the directory with gradlew.bat, and run:
gradlew.bat debugJar

Assuming no build errors, you'll find a development version of JPanel in build/libs/

Thanks! :D

It works! :D


EdfWriter writes incorrect sample frequencies for record lengths != 1

I'm working with EDF files that have record lengths other than 1 second and am trying to generate dummy files for testing, and I think I've encountered a bug with how EdfWriter handles sample rate vs sample frequency. I'm writing a file with a sample rate of 256, a record length of 2, and a total of 6 minutes of data (example code below). I expect the generated EDF file to have 180 records with 512 samples per record, but the actual file has 360 records with 256 samples per record.
I think I found the source of the problem with regards to calculating sample frequency: in EdfWriter.update_header, channel sample rates are passed to set_samplefrequency, but inside that function the argument is named samplefrequency and treated as such. My understanding is that sample rate represents samples-per-second, and sample frequency represents samples-per-record. So treating them interchangeably would result in incorrect numbers for record lengths != 1. I saw #111 and suspect that this is related.
I haven't found where number of records is calculated yet, but I imagine the issue is similar for that header attribute as well.
Example code:
import numpy as np
from pyedflib import EdfWriter, FILETYPE_EDF
from datetime import datetime

MILLISECONDS = 1000
TEN_MICROSECONDS = MILLISECONDS * 100
SIX_MINUTES = 360

filename = 'edf_file.edf'
sample_rate = 256
samples_per_record = 512  # For a 2 sec record length
record_length = (samples_per_record / sample_rate) * TEN_MICROSECONDS
channel_names = ['ch1', 'ch2', 'ch3', 'ch4', 'ch5']
signals = np.random.rand(len(channel_names), sample_rate * SIX_MINUTES) * 200
start_date = datetime(2017, 5, 24, 15, 53, 49)

writer = EdfWriter(filename, len(channel_names), file_type=FILETYPE_EDF)
writer.setPatientCode('')
writer.setRecordingAdditional('')
writer.setDatarecordDuration(record_length)
writer.setStartdatetime(start_date)
writer.setSignalHeaders([{
    'label': channel,
    'sample_rate': sample_rate,
    'dimension': 'uV',
    'physical_min': -200.0,
    'physical_max': 200.0,
    'digital_min': -32768,
    'digital_max': 32767,
    'transducer': f'EEG {channel}',
    'prefilter': ''
} for channel in channel_names])

writer.writeSamples(signals)

Let me know if my assessment is correct (I'm new to working with EDF files, so I'm still getting across how they're structured), and I'd be happy to open a PR.

Excuse my late reply, I did not have time to look into this in more detail yet, and fear that I will not have time soon as well.
However, I think you are on a good track! It seems very much related to the issue I raised in #111 . I did not implement the piece of code mentioned above, but as far as I understand it, the sample frequency is never saved explicitly in the header, but always computed by duration of a data record (file header) and nr of samples in each data record (signal header). Therefore it can become a bit confusing when specifying a sampling frequency that is transformed into the former internally.
I'd very much appreciate if you have a look into this and post your results and a PR
Thanks alot!

No worries. I was able to isolate the two places in the code where record length is calculated or used in other calculations, and implemented a quick fix in a version of pyedflib installed from local files. I've since used it to write and read EDF files with irregular record lengths, and the data and headers all turn out correct.
I'll look into cleaning up my fixes and adding tests for a PR sometime in the near future.


fix: azure快照的磁盘id应该小写

这个 PR 实现什么功能/修复什么问题:
fix: azure快照的磁盘id应该小写
是否需要 backport 到之前的 release 分支:

release/2.8.0

/area region
/cc @swordqiu @yousong

Codecov Report

Merging #2326 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #2326      +/-   ##
=========================================
- Coverage    7.09%   7.08%   -0.01%
=========================================
  Files         452     452
  Lines       71360   71359       -1
=========================================
- Hits         5060    5058       -2
  Misses      65884   65884
- Partials      416     417       +1

Flag
Coverage Δ

#aFlag
7.08% <ø> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/appsrv/response.go
66.15% <0%> (-3.08%)
:arrow_down:

pkg/compute/models/storages.go
1.18% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

/approve
/lgtm


WIP: Use Vite for demo admin

based on https://github.com/vivid-planet/comet-starter/pull/20
open issue: reload not working for changes in library:

(this worked in webpack)
this is because of our monorepo
vite supports monrepo and detects it and disables optimizeDeps
however it tries to load the built common js sources are used (not of the ESM sources)
solution for now: optimizeDeps.include
but that breaks reloading

I'm stuck now....

Superseded by #1838.


[top] Complete various connections to ast

connect jitter enable
connect differential scanmode signal
also connect reset bypass structure in spi_device

can i get a review for this PR?

yeah sounds good, that is more descriptive.
On Wed, Feb 10, 2021 at 11:28 AM Srikrishna Iyer<EMAIL_ADDRESS>wrote:

@sriyerg commented on this pull request.
In hw/ip/clkmgr/data/clkmgr.hjson.tpl
https://github.com/lowRISC/opentitan/pull/5154#discussion_r574013423:

@@ -62,6 +62,13 @@ num_grps = len(grps)
package: "lc_ctrl_pkg",
},

{ struct:  "logic",

 type:    "uni",

 name:    "jen",

nit: How about jitter_en?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/pull/5154#pullrequestreview-587972106,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSWHXFXPJVV4INJ7ZHDS6LM4RANCNFSM4XKERBVA
.


It is possible to share non-tenancy tables with tenants

There are many tables that have the same content for whoever the tenant is, so instead of replicating them every time, it would be interesting to be able to share this tables with all tenants.
An example is the table of the cities of the country, among many others.
These tables simplify the entries and avoid typing errors.

Not supported yet, but I'm thinking about it. See #57.
I've done some research about this and it's a bit complicated.
So say you have a list of cities in a cities table in some shared database. You could define a getConnectionName() method on the Cities model to use some non-tenant connection and it would work perfectly. But only if you don't need relationships.
If you have a city column on the users table, referencing a record in the cities table, you can't use foreign keys. That sucks a bit, as it passes a lot of the DB's responsibility to the application (ensuring that a referenced record exists, ON DELETE, ON UPDATE).
You could use cross-database queries, e.g. SELECT * from tenant123.users WHERE tenant123.users.id = 1 JOIN cities_db.cities ON tenant123.users.city_id = cities_db.cities.id, but Laravel is not built for that use case, so it's also not trivial.
The DB facade will let you execute cross-database queries, but it's up to you to write the query correctly for your database driver. For example, sqlite supports cross-database queries, but you first need to attach the databases. Not sure how that would work in Laravel.
When it comes to Eloquent, I'm not sure if using two related models with different DB connections specified in getConnectionName() would work. If I recall correctly, I've seen conflicting reports on this. I haven't tested this myself yet.
Even if that did work, you would still need to handle the record exists/ON UPDATE/ON DELETE logic in your app.
I want to add this at some point, but I haven't yet figured out how I'd implement this.

Thank you, you are right in your postings. I will study a little more what would be the actual use cases and the possible solution for those cases. However, we will keep these tables being duplicated.

I will close this for now, but if you find something that could be used as a general solution provided by this package, please let me know.
Thank you.


Insights: add the read:issues permission

Why:
cc/ https://github.com/github/insights/issues/2024
cc/ @github/insights
There was recently a change to Insights so the GitHub App needs additional permissions, and we need to update the Docs so that customers know to add the read:issues permissions to their apps.
What's being changed:

Adds instructions to add the read:issues permissions to the initial GH App creation:

Adds a section to upgrade from Insights v1.5.0-alpha.1

Check off the following:

[x] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

Thanks @c-brooks! I'll get this triaged for the team to review ⚡

Thanks @c-brooks! I'll get this triaged for the team to review ⚡

I'm NEW COM ON GITHUB BUT I'M VERY INTERESTED TO LEARN AND USE GITHUB

I'm NEW COM ON GITHUB BUT I'M VERY INTERESTED TO LEARN AND USE GITHUB

@janiceilene Is it possible to get this reviewed today? We’d like to put it out today with our release. thanks!

@janiceilene Is it possible to get this reviewed today? We’d like to put it out today with our release. thanks!

👋🏻 @c-brooks, can you open an issue about this in the private repo? This update will require a bit more discussion. Thanks!

👋🏻 @c-brooks, can you open an issue about this in the private repo? This update will require a bit more discussion. Thanks!

It looks like this was moved to an internal issue, so I'm going to close this up ⚡

It looks like this was moved to an internal issue, so I'm going to close this up ⚡


broken initialization of variables vs. rendering

I think a recent commit broke a core part of my app related to templates. It broke in iOS only after also pulling develop in Android.
Example: https://jasonbase.com/things/0640/
Basically, I have some data items going into a view, which are initialized onto variables with $set on $load and upon success, $render takes place. The user may have set those before and they are potentially populated in the back-end DB.
In the latest develop (for iOS only!) they come up with a default value that has the template, e.g a notes text field has {{ $get.notes }} in the value of the text instead of what's initialized in the notes variable. It appears as if initialization is not done before rendering.
It took a long long time to get it working in both iOS and Android (and it was still not right). It was all developed based on this discussion.

I discovered a bug just a minute ago and may have fixed it. Please pull from the latest develop and check.

fixed


Any way to integrate RabitMQ with react without (API or node js ) want to connect strightly

Yes, use WebSockets
amqp-client.js Reference


Remove topological sort from the ScheduleExecution pass.

I don't recall why it's in there but it is seemingly load-bearing. I suspect the issue is that the execute op is inserted in the wrong place - the partition is a gather of ops within the block and picking the exact insertion location may not be possible (especially with clones) as-is.
Here's a minimal repro that fails with error: operand #2 does not dominate this use if sortBlockTopologically is commented out:
stream.executable private @ex {
  stream.executable.export public @dispatch_0
  stream.executable.export public @dispatch_1
  stream.executable.export public @dispatch_2
  builtin.module  {
    func @dispatch_0(%arg0: !stream.binding) {
      return
    }
    func @dispatch_1(%arg0: !stream.binding) {
      return
    }
    func @dispatch_2(%arg0: !stream.binding, %arg1: !stream.binding, %arg2: !stream.binding) {
      return
    }
  }
}
func @predict(%arg0: i1) -> !stream.resource<transient> {
  %c1 = arith.constant 1 : index
  %c128 = arith.constant 128 : index
  %0 = stream.async.dispatch @ex::@dispatch_0[%c1, %c1, %c1]() : () -> !stream.resource<transient>{%c128}
  %1 = stream.async.dispatch @ex::@dispatch_1[%c1, %c1, %c1]() : () -> !stream.resource<transient>{%c128}
  %2 = select %arg0, %0, %1 : !stream.resource<transient>
  %3 = stream.async.dispatch @ex::@dispatch_2[%c1, %c1, %c1](%1, %2) : (!stream.resource<transient>{%c128}, !stream.resource<transient>{%c128}) -> !stream.resource<transient>{%c128}
  return %3 : !stream.resource<transient>
}

Looks like an issue with hazard detection.


Implement String.prototype.replaceAll

Implementation should be standard with vanilla js functionality.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

This was said somewhere a while ago, but I think this change would be better made in Rhino itself.

Implemented in ChatTriggers/rhino@60f40a81851d6f234f83f45ff2c68eda5d5c44ad


[CT-2106] [Bug] [Security] Security vulnerability on Cryptography v39.0.0

Is this a new bug in dbt-snowflake?

[X] I believe this is a new bug in dbt-snowflake
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
There's some high-severity security vulnerabilities due to the cryptography dependency being <39.0.0, namely the following:
Upgrade<EMAIL_ADDRESS>to<EMAIL_ADDRESS>to fix
  ✗ Denial of Service (DoS) [Medium Severity][https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3172287] in<EMAIL_ADDRESS>    introduced by<EMAIL_ADDRESS>and 5 other path(s)
  ✗ Expected Behavior Violation (new) [Medium Severity][https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3314966] in<EMAIL_ADDRESS>    introduced by<EMAIL_ADDRESS>and 5 other path(s)
  ✗ Use After Free (new) [Medium Severity][https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3315324] in<EMAIL_ADDRESS>    introduced by<EMAIL_ADDRESS>and 5 other path(s)
  ✗ Timing Attack (new) [Medium Severity][https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3315331] in<EMAIL_ADDRESS>    introduced by<EMAIL_ADDRESS>and 5 other path(s)
  ✗ Denial of Service (DoS) (new) [Medium Severity][https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3315452] in<EMAIL_ADDRESS>    introduced by<EMAIL_ADDRESS>and 5 other path(s)
  ✗ Denial of Service (DoS) (new) [Medium Severity][https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3315972] in<EMAIL_ADDRESS>    introduced by<EMAIL_ADDRESS>and 5 other path(s)
  ✗ Denial of Service (DoS) (new) [Medium Severity][https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3315975] in<EMAIL_ADDRESS>    introduced by<EMAIL_ADDRESS>and 5 other path(s)
  ✗ Denial of Service (DoS) (new) [Medium Severity][https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3316038] in<EMAIL_ADDRESS>    introduced by<EMAIL_ADDRESS>and 5 other path(s)
  ✗ Access of Resource Using Incompatible Type ('Type Confusion') (new) [High Severity][https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3315328] in<EMAIL_ADDRESS>    introduced by<EMAIL_ADDRESS>and 5 other path(s)
  ✗ Denial of Service (DoS) (new) [High Severity][https://security.snyk.io/vuln/SNYK-PYTHON-CRYPTOGRAPHY-3316211] in<EMAIL_ADDRESS>    introduced by<EMAIL_ADDRESS>and 5 other path(s)

Expected Behavior
Hopefully no high severity vulnerabilities but I would settle for only medium severity ones.
Steps To Reproduce
I am currently using snyk cli to check for security vulnerabilities.
snyk test

Relevant log output
No response
Environment
- OS: macOS 13.0.1
- Python: Python 3.10.8
- dbt-core: 1.3.1
- dbt-snowflake: 1.4.1

Additional Context
No response

Thanks for opening this and the associated pull request @Surbias !


Redo front end using Vue.js

The frontend is currently lacking in features, looks and compatibility with different browsers and devices. The front end should be redone using VueJS (a modern framework) and a component library like Vuetify.
Key Requirements:

Add webpack and babel to MM
Replace MaterializeCSS with VueJS and Vuetify
Put everything into components for reusability and ease of use

I've started working on this in my feature/vue branch.

This has been mostly completed now, there's still a few issues left for specific features etc.


Can this dice bot support multiplication

like 3d6x3 or 3d6*3
sum rolls result [3,2,5]  = 10
than [10 times 3] = 30

How would you expect this to interact with the other modifiers (e.g. + / - / > / k)?

This should now work with constants, and dice.


sh: CLS: command not found

It looks like the clear screen command being used here is CLS, which only works on Windows. This could be fixed by using the clear command instead, for macOS or Linux.

Ouput error example

Thank You


[#149] Added uptodate plugin; updated StringTemplate API usage

I didn't include uptodate plugin to regular check goal because of its (I am guessing) sporadic usage and probably it won't be always desired.
Additionally I had to change string template's delimiter in BruteForceAttackMailGenerator class to a default one.

Coverage remained the same at 62.215% when pulling c06f46a92d98139dfa78c18dba621c79fd2a3277 on puradawid:issue-149-add-uptodate-gradle-plugin into 10674910d2e0448513811badb4d5bb95b37603b9 on tdziurko:master.

If adding this task to check won't increase build time significantly, I would add it there so we could see what libraries are out of date.

Coverage remained the same at 64.336% when pulling 8bc08957633f7ac6a08e3f1df995e2f2a7be2ad0 on puradawid:issue-149-add-uptodate-gradle-plugin into c6d3cfcba7f89808dc85ea96c6e1d22937ff80d2 on tdziurko:master.


Replace aad-pod-identity docker image with official MS container registry

What this PR does / why we need it:
The current add-on aad-pod-identity uses a docker image originating from an individual's account at docker hub.


Synthesis failed for java-workflows

Hello! Autosynth couldn't regenerate java-workflows. :broken_heart:
Here's the output from running synth.py:
ac2d51e02834b5
2021-02-08 14:33:30,964 autosynth [DEBUG] > Running: git log -1 --pretty=%at 254325ed3c9caa6bf6d9c8b11ce2674544ba6e7c
2021-02-08 14:33:30,968 autosynth [DEBUG] > Running: git log -1 --pretty=%at 35ce99fec72979f6f9b2a5adae835a49648a3231
2021-02-08 14:33:30,971 autosynth [DEBUG] > Running: git log -1 --pretty=%at 5e3dacee19405529b841b53797df799c2383536c
2021-02-08 14:33:30,974 autosynth [DEBUG] > Running: git log -1 --pretty=%at 78e0057d81c6969507bf1195b5aad8ac3e7feafd
2021-02-08 14:33:30,977 autosynth [DEBUG] > Running: git log -1 --pretty=%at c1e59709c1d28795fe9b70eb479579556056bfad
2021-02-08 14:33:30,981 autosynth [DEBUG] > Running: git log -1 --pretty=%at 8d8c008e56f1af31d57f75561e0f1848ffb29eeb
2021-02-08 14:33:30,984 autosynth [DEBUG] > Running: git log -1 --pretty=%at 692715c0f23a7bb3bfbbaa300f7620ddfa8c47e5
2021-02-08 14:33:30,987 autosynth [DEBUG] > Running: git log -1 --pretty=%at 33366574ffb9e11737b3547eb6f020ecae0536e8
2021-02-08 14:33:30,990 autosynth [DEBUG] > Running: git log -1 --pretty=%at 4679e7e415221f03ff2a71e3ffad75b9ec41d87e
2021-02-08 14:33:30,996 autosynth [DEBUG] > Running: git checkout 529aa0e71150e7e6c1f99627e0bfc6c81b8a22dd
Note: checking out '529aa0e71150e7e6c1f99627e0bfc6c81b8a22dd'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 529aa0e fix: update repo name (#182)
2021-02-08 14:33:31,001 autosynth [DEBUG] > Running: git checkout 8d8c008e56f1af31d57f75561e0f1848ffb29eeb
Note: checking out '8d8c008e56f1af31d57f75561e0f1848ffb29eeb'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 8d8c008e feat: generate sample code in the Java microgenerator
2021-02-08 14:33:31,015 autosynth [DEBUG] > Running: git checkout 4679e7e415221f03ff2a71e3ffad75b9ec41d87e
Note: checking out '4679e7e415221f03ff2a71e3ffad75b9ec41d87e'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 4679e7e docs: update python contributing guide (#943)
2021-02-08 14:33:31,021 autosynth [DEBUG] > Running: git branch -f autosynth-97
2021-02-08 14:33:31,025 autosynth [DEBUG] > Running: git checkout autosynth-97
Switched to branch 'autosynth-97'
2021-02-08 14:33:31,030 autosynth [INFO] > Running synthtool
2021-02-08 14:33:31,030 autosynth [INFO] > ['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']
2021-02-08 14:33:31,030 autosynth [DEBUG] > log_file_path: /tmpfs/src/logs/java-workflows/97/sponge_log.log
2021-02-08 14:33:31,032 autosynth [DEBUG] > Running: /tmpfs/src/github/synthtool/env/bin/python3 -m synthtool --metadata synth.metadata synth.py --
2021-02-08 14:33:31,268 synthtool [DEBUG] > Executing /home/kbuilder/.cache/synthtool/java-workflows/synth.py.
On branch autosynth-97
nothing to commit, working tree clean
2021-02-08 14:33:31,410 synthtool [DEBUG] > Ensuring dependencies.
2021-02-08 14:33:31,421 synthtool [DEBUG] > Cloning googleapis.
2021-02-08 14:33:31,424 synthtool [DEBUG] > Using precloned repo /home/kbuilder/.cache/synthtool/googleapis
2021-02-08 14:33:31,428 synthtool [DEBUG] > Which version of bazel will I run?
bazel 3.0.0
2021-02-08 14:33:31,432 synthtool [DEBUG] > Generating code for: //google/cloud/workflows/v1beta:google-cloud-workflows-v1beta-java.
2021-02-08 14:33:31,536 synthtool [ERROR] > Failed executing bazel --max_idle_secs=240 build //google/cloud/workflows/v1beta:google-cloud-workflows-v1beta-java:

Loading:
Loading: 0 packages loaded
INFO: Call stack for the definition of repository 'com_github_grpc_grpc' which is a http_archive (rule definition at /home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/bazel_tools/tools/build_defs/repo/http.bzl:296:16):
 - <builtin>
 - /home/kbuilder/.cache/synthtool/googleapis/WORKSPACE:113:1
WARNING: Download from https://github.com/grpc/grpc/archive/v1.33.1.zip failed: class com.google.devtools.build.lib.bazel.repository.downloader.UnrecoverableHttpException GET returned 502 Bad Gateway
ERROR: An error occurred during the fetch of repository 'com_github_grpc_grpc':
   java.io.IOException: Error downloading [https://github.com/grpc/grpc/archive/v1.33.1.zip] to /home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/com_github_grpc_grpc/v1.33.1.zip: GET returned 502 Bad Gateway
ERROR: no such package '@com_github_grpc_grpc//bazel': java.io.IOException: Error downloading [https://github.com/grpc/grpc/archive/v1.33.1.zip] to /home/kbuilder/.cache/bazel/_bazel_kbuilder/a732f932c2cbeb7e37e1543f189a2a73/external/com_github_grpc_grpc/v1.33.1.zip: GET returned 502 Bad Gateway
INFO: Elapsed time: 0.092s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded)
FAILED: Build did NOT complete successfully (0 packages loaded)

Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 102, in <module>
    main()
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/tmpfs/src/github/synthtool/env/lib/python3.6/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/tmpfs/src/github/synthtool/synthtool/__main__.py", line 94, in main
    spec.loader.exec_module(synth_module)  # type: ignore
  File "<frozen importlib._bootstrap_external>", line 678, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/kbuilder/.cache/synthtool/java-workflows/synth.py", line 29, in <module>
    bazel_target=f'//google/cloud/workflows/{version}:google-cloud-{service}-{version}-java',
  File "/tmpfs/src/github/synthtool/synthtool/languages/java.py", line 306, in bazel_library
    service=service, version=version, diregapic=diregapic, **kwargs
  File "/tmpfs/src/github/synthtool/synthtool/gcp/gapic_bazel.py", line 68, in java_library
    service, version, "java", tar_strip_components=0, **kwargs
  File "/tmpfs/src/github/synthtool/synthtool/gcp/gapic_bazel.py", line 197, in _generate_code
    shell.run(bazel_run_args)
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['bazel', '--max_idle_secs=240', 'build', '//google/cloud/workflows/v1beta:google-cloud-workflows-v1beta-java']' returned non-zero exit status 1.
2021-02-08 14:33:31,599 autosynth [ERROR] > Synthesis failed
2021-02-08 14:33:31,600 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at 529aa0e fix: update repo name (#182)
2021-02-08 14:33:31,605 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2021-02-08 14:33:31,609 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 354, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 189, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 334, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 65, in synthesize_loop
    has_changes = toolbox.synthesize_version_in_new_branch(synthesizer, youngest)
  File "/tmpfs/src/github/synthtool/autosynth/synth_toolbox.py", line 259, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


RA81 - ChatGPT Turbo implementation

I have tested and compared the differences between Chat GPT 3.5 turbo and the davinci 003, looks like the winner in the best personality section is
still, davinci 3.0
No matter how hard I train it to be as normal and friendly in gpt 3.5, it always revert itself to "An AI Language Model", which is boring, and soooo not Ririko.
I've made sure the original "system" context which include all the personality gets included in every prompt sent to the API, but the response is just sad.
Compare yourself:
gpt 3.5 "turbo"

the O.G. davinci 003


Begin adding WASI support

Very much a work in progress.  Probably will require updating more of the existing code toward ImportObject and away from existing imports, though I'm not sure.
This PR is not in a state where it can be merged and won't be until we ship WASI support in the C API.  Before doing so we want to verify, by implementing it in a language extension, that the updates to the C API are sufficient.

Thanks!
Can you rebase on top of master please? Since #73, Wasmer 0.8.0 is used, which will be easier for everyone.

How far along is this branch? We currently have completed a machine learning C++ application that compiles nicely using both clang and emscripten standalone, and now runs on both Wasmer and WAVM. But it fails to instantiate the wasm using go-ext-wasm, dueto some missing WASI imports. Would be super if we can make this work, as it would be the final step towards completing our framework.

I'm literally working on this right now. Expect to be merged in a couple of hours if everything goes smoothly.

Incredible! The timing couldn't be any better. Thanks!

Review feedbacks fixed by https://github.com/wasmerio/go-ext-wasm/commit/c07c1017f9ebede0af5241622116be8367bbc1f6, https://github.com/wasmerio/go-ext-wasm/commit/b91099d08b5a069fe1d13edf3d8474f01778859b and https://github.com/wasmerio/go-ext-wasm/commit/bf837c2dd588e346ecafe9a6c1cc956801d04626 (the last one fixes WASI itself, since the namespace has changed from wasi_unstable to wasi_snapshot_preview1.
Thanks!

The test prints hello world as expected, but I wish we can find a solution to test the output to avoid regression in the future.

Hi Hywan! Thanks for all of the work you have been putting in. I have been testing the latest commit, and still get the following:
7 link errors: (1 of 7) Import not found, namespace: wasi_unstable, name: proc_exit (2 of 7) Import not found, namespace: wasi_unstable, name: environ_get (3 of 7) Import not found, namespace: wasi_unstable, name: environ_sizes_get (4 of 7) Import not found, namespace: wasi_unstable, name: fd_write (5 of 7) Import not found, namespace: wasi_unstable, name: fd_seek (6 of 7) Import not found, namespace: wasi_unstable, name: fd_read (7 of 7) Import not found, names pace: wasi_unstable, name: fd_close
Does that make sense? Does it have anything to do with the wasi_snapshot_preview1 style import?

Wait, I probably need to use wasm.NewDefaultWasiImportObject()! I will test this later. Thanks again for all of your work on this!

Super! Using NewDefaultWasiImportObject did the trick: I can now use the WASI ABI in go-ext-wasm. There is one catch though. To make an existing WASI run (compiled with either the latest clang, WasiENV, or Emscripten) I now need to find&replace wasi_unstable with wasi_snapshot_preview1. This conversion is not necessary when I run the same WebAssembly by calling Wasmer directly. Apparently, Wasmer supports both import namespaces that way. Is there some elegant way to do the same in go-ext-wasm?


Added Unmarshal for ContractClass and Cleanup

Resolves the issue https://github.com/dontpanicdao/caigo/issues/113
Also cleans up a lot of duplicated code and functionality which shouldn't be exposed

Hi,
I have built a PR on top of this one, see #126 to fix another problem I know we had that is when using map[string]interface{}{} underlying numbers that are >max_int64 are turned into float64. The fix consists in using map[string]json.RawMessage{} and checking when the program payload is already a gzip/base64 payload.
I have also managed the changes I had pointed out! I am closing the PR as a consequence. Thank you again for your help 👍 !
@gregoryguillou


Border of background dataset still visible on stacked line chart

Hi there,
I just updated Chart.js to version 2.9.3 and noticed my charts look a bit different. I've got a stacked line chart and it seems the new version draws the borders of all datasets regardless of their stacking order.
However this is not something I want because on a stacked chart lines are not supposed to go below one another. At most they touch when one of the values is 0. So, for certain numbers the chart ends up looking like the one in the picture.
I much prefer the old version behavior.
Is there a way to keep the old behavior and prevent the borders below from being drawn?
Thank you.
DEMO:
desired behavior: http://www.pirosca.com/bug-reports/chartjs/2.7.2.html
current behavior: http://www.pirosca.com/bug-reports/chartjs/2.9.3.html

This is fixed in v3-alpha.

Hi, @kurkle
this "bug" was fixed in new version 3.x. Do you have any information how make this line exactly like in version 2.9.3.
I would like to create multiple lines chart with background which show border if one of chart is hiding under another chart.

@strasbi Should be possible by changing the drawTime option: https://www.chartjs.org/docs/latest/charts/area.html#configuration

@strasbi Should be possible by changing the drawTime option: https://www.chartjs.org/docs/latest/charts/area.html#configuration

Yes, it works. Thank you!


[SP-5352] Backport of BISERVER-14355 - The JDBC connection pooling properties aren't getting mirrored from Pentaho (Merged) Server <=> Spoon (8.3 Suite)

@ssamora

:white_check_mark: Build finished in 5m 13s
Build command:
mvn clean verify -B -e -Daudit -amd -pl plugins/pur/core

:ok_hand: All tests passed!
Tests run: 272, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


Deprecated Class constructors

Deprecated: Methods with the same name as their class will not be constructors in a future version of PHP

The backwards compatibilty with older php versions enable to use __construct() because stated v2.4 version will not be compatible older v2.3 components.

fixed

Could I have a look at the code please :-)

@golith
v2.4 is figured out in development branch so I couldnt give exact commits.
v2.3.5 core muted with deprecated errors.


Reindexing terminated due to gap:  Block XXXXXX does not exist

After bouncing my witness node I"m getting this error.
"Reindexing terminated due to gap:  Block 1828867 does not exist!"
Looks like it was fixed in later steem code
https://github.com/steemit/steem/issues/407

apparently this was fixed in bitshares: https://github.com/bitshares/bitshares-core/pull/339/files and ported successfully to Muse: https://github.com/themuseblockchain/Muse-Source/pull/62
need to be ported to peerplays as it will also add replay time improvements after a crash.
sometimes we can chery-pick the changes between 2 graphenes, others not that easy and have to be done manually. i can take a look if there is interest on this feature for peerplays.


[Bug]: transaction timeout error while bind more than 300 databases into project

ODC version
4.2.0
OB version
not related
What happened?
transaction timeout error while bind more than 300 databases into project.
the MetaDB unit config was 1C4G, may not sufficient.
What did you expect to happen?
bind databases to project success
How can we reproduce it (as minimally and precisely as possible)?
bind more than 300 databases into project
Anything else we need to know?
there also exists slow sql for table SPRING_SESSION, update operation
Cloud
No response

pass


Spam submissions

I'm currently using the honey pot method but around 3-5 spam comments slip through every day. It's not the end of the world to close these out, but it does junk up my Pull Request stream.
In the beginning I got a lot more spam when I had a website field in the comment form. Eventually took that out, but whoever trained the bots to constantly hit my site must have adapted. Now I get Markdown links in there with messages almost always phrased in the same way.
I see Akismet is an option but wasn't sure if that was available to the public instance of Staticman or not. If enabled I could see it pulling out a good bunch of these comments since they seem to follow a common pattern.
If I can get reCAPTCHA working that would probably solve the issue, but having some snags there too.

Our implementation of Akismet needs some work done. It's currently set to use whatever key is supplied in the Staticman API, which means one single Akismet account is used to filter spam on all sites. Akismet accounts have usage limits, so I believe it's disabled completely in the public Staticman instance.
We could take the same approach we used with reCAPTCHA and allow each site to supply their own Akismet API key, now that we support encryption.
@zburgermeiszter is this something you'd like to have a go at?

I don't see anywhere in the staticman.yml config to put an Akismet API key. Is there any way to use Akismet when not using the public shared instance? Thanks!

@deadlydog the Akismet's site and key are configured for Staticman, not for the site, so they are not in staticman.yml. You can set them in config.{ENV}.yml file or using env vars.

Thanks @PhuNH , I found which environment variables I needed to add here.
I've modified my staticman.yml to include this section:
  # Akismet spam detection.
  akismet:
    enabled: true
    author: "name"
    authorEmail: "email"
    authorUrl: "url"
    content: "message"
    type: "comment"

I know these fields are documented here, but I'm still not entirely sure what they're for besides the enabled one, so I think those docs could be clarified a bit; I'm guessing they'll show up in Akismet somewhere once it starts blocking some spam comments.
Thanks!

To collaborate with @deadlydog on documenting the process:
The site URL set up on Akismet and the akismet.site config should both be the URL of where you host Staticman. The comment form sends form data to Staticman at this URL, Akismet also tries to catch spams at this URL, those mentioned fields are for Akismet to know what data it receives.

@PhuNH, @deadlydog -- I host Staticman at Heroku., Is ASKIMET_SITE the same as (or a shortened version of) staticmanUrl?
Example:
staticmanUrl: https://spinningnumbers-staticmandev2.herokuapp.com/v2/entry/willymcallister/willymcallister.github.io/master/comments
[I will add this info to https://spinningnumbers.org/a/staticman-heroku.html]

@willymcallister it has to be the URL you use when adding a new site in your Akismet account. You can use the same one as your staticmanUrl, or just the domain name part of it as in the example. Staticman only uses this URL and the API key coming with it to make a call to Akismet, so it can be anything as long as it matches the registered one.

It can be working around by updating siteConfig.js schema and modifying lib/Staticman.js to use parameters from it rather than config.js.
However, since reCaptcha is done within controller/process.js while Akismet is parts of lib/Staticman.js, I guess there need some refactoring in the future.


[possible bug] serializing global class state of experiment

Some brains make use of a global class state of the brain class (usually for sparse matrices). As far as I can tell this global class state is not preserved at the end of an experiment. This is no problem for logarithmic sparse matrices, as they are deterministic to begin with.
But it will probably be a significant problem for random sparse matrices.
It could still be possible  that they will work because of the fixing of random seeds at the beginning of the experiment, but if so, than it would be very prone to errors.
I suggest the following solution:

[ ] add brain_class_state.pkl the output dir, which contains the global class state of the experiment (you can get this from IBrain.get_class_state()
[ ] in render_hof.py read this variable and pass it to Experiment
[ ] in Experiment initiate the brain class state from the variable and only create a new brain class state for new experiments.

related issues: #34

My next TODO is #34 I will look into this


Fixed small inconsistency when handling aggregate's default_alias.

Refs #14030.

LGTM. I like the explicit test for AttributeError and TypeError, and that you made both call sites consistent.


Message deleted is still counting toward the total message not read

Here is flow
1-Person A search and see post of person B, then send the message to person B
2- Right after that, person A delete the sent message to person B
3-Person B login and see new message coming from person A with the note- This message was deleted and Do you want to accept the message request? you can't replay until you accept the request (Accept, Ignore,Block)
My expectation is if the person A send message to Person B, and delete right after that, if the person B is not reading it yet, we should delete the message for person B inbox.

@Zaydme , @rabbi08 , please check to see if it is a valid defect

@adorapham this is expected behavior.
When a message is deleted, we keep "This message was delete." As a proof that there was something before, same applies for a message request, the recipient needs to know that there was a message sent to them, even if it was deleted.

@adorapham this is expected behavior.
When a message is deleted, we keep "This message was delete." As a proof that there was something before, same applies for a message request, the recipient needs to know that there was a message sent to them, even if it was deleted.

@Zaydme , i know for the tracking record but it is strange to see empty message with the question  do you accept , ignore or block .  @rabbi08 what do you think?

I think in the case you will go to the person's/org's profile, see if they are relevant for you. if yes, you can accept the request and ask why the messaged you in the first place, if now you block them so they stop being annoying lol


KAFKA-3753: Add approximateNumEntries() method to KeyValueStore interface

See https://issues.apache.org/jira/browse/KAFKA-3753
This contribution is my original work and I license the work to the project under the project's open source license.
cc @guozhangwang @kichristensen @ijuma

btw I think I have seen this test case stalled once before already:

kafka.integration.SaslSslTopicMetadataTest > testIsrAfterBrokerShutDownAndJoinsBack STARTED
Build timed out (after 120 minutes). Marking the build as failed.

Maybe we could create a JIRA for it @ijuma ?

This should be ready for another look, @guozhangwang and @ijuma

Also cc @jkreps who commented on the method naming in the JIRA issue.

LGTM. Merged to trunk. Thanks @jklukas .


Convert rickety bin scripts to legitimate CLI

Using Thor instead of stray Ruby code in scripts gives us more robust option-parsing and documentation, and makes it easier to test this code. It will also make easier follow-on work to compose a pipeline of extractors, transformers, and loaders.
Also, switch from Coveralls to CodeClimate. Coveralls ties us to an outdated version of Thor.

Thanks, @jgreben!


feat: support none sstc environment

This pull request implements the following features.

Fixed mtime context; it now stores all registers.
Support for non-SSTC environments.

nice work!!


Cucumber is running the background in an inappropriate way

Cucumber is running the background in an inappropriate way
Information:
I have a background at the beginning of the tests that I have written for my iOS app. I use calabash cucumber to test the iOS app. When I have a background, I have observed that it is invoked at the beginning of the feature execution. Sometimes, it is also called when I run a scenario and my tests start to fail.
Configuration:
Ruby: ruby 2.3.0p0 (2015-12-25 revision 53290) [x86_64-darwin15]
Calabash-cucumber: 0.21.1
Cucumber: 3.1.0
Questions:

Will a background be run before each scenario?
From the executions I have seen that it is being run at the beginning of the feature file. In one of the other scenario, it will be invoked and the test fails. Is it the expected behavior?

Sample feature:
@f_TestFeature
Feature: This is a sample feature that I will use to understand the background
Background: This is a background
Given I run the background
@uninstall_install_app_testfeature
Scenario: If App is already present, uninstall it and install the app again
Given I uninstall install and then open the app sample
@scenaio1
Scenario: I am on the screen sample 1
Given I am on the screen sample 1
@scenaio2
Scenario: I am on the screen sample 2
Given I am on the screen sample 2
@scenaio3
Scenario: I am on the screen sample 3
Given I am on the screen sample 3
Steps:
Given("I run the background") do
puts "Running background now"
end
Given("I uninstall install and then open the app sample") do
puts "I uninstall install and then open the app sample"
end
Given("I am on the screen sample 1") do
puts "I am on the screen sample 1"
end
Given("I am on the screen sample 2") do
puts "I am on the screen sample 2"
end
Given("I am on the screen sample 3") do
puts "I am on the screen sample 3"
end
Output:
$ cucumber features/sample.feature -f pretty -f html --out='test123.html'
@f_TestFeature
Feature: This is a sample feature that I will use to understand the background
Background: This is a background # features/sample.feature:4
Given I run the background     # features/step_definitions/sample_steps.rb:1
Running background now
@uninstall_install_app_testfeature
Scenario: If App is already present, uninstall it and install the app again # features/sample.feature:8
Given I uninstall install and then open the app sample                    # features/step_definitions/sample_steps.rb:5
I uninstall install and then open the app sample
@scenaio1
Scenario: I am on the screen sample 1 # features/sample.feature:12
Given I am on the screen sample 1   # features/step_definitions/sample_steps.rb:9
I am on the screen sample 1
@scenaio2
Scenario: I am on the screen sample 2 # features/sample.feature:16
Given I am on the screen sample 2   # features/step_definitions/sample_steps.rb:13
I am on the screen sample 2
@scenaio3
Scenario: I am on the screen sample 3 # features/sample.feature:20
Given I am on the screen sample 3   # features/step_definitions/sample_steps.rb:17
I am on the screen sample 3
4 scenarios (4 passed)
8 steps (8 passed)
0m0.027s

The background is executed as part of every scenario of the feature file (and not only once for the feature file), see https://github.com/cucumber/cucumber/tree/master/gherkin#pickles.


[question] Angular 5 support

Is it safe to use latest ngx-datatable with Angular 5?

Duplicate but look for by end of the week.


[stable/traefik] Add the possibility to define additional containers

Is this a new chart
NO
What this PR does / why we need it:
Add the posibility to define additional containers into the traefik pod.
Which issue this PR fixes

fixes #17465

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

@morsok
Could you resolve the conflicting files please?
stable/traefik/Chart.yaml

@zanhsieh Done sorry for the delay


socket: too many open files

{"id":"***", "code":500,"detail":"the database HTTP request failed: Post http://<IP_ADDRESS>:8529/_db/_api/cursor: dial tcp <IP_ADDRESS>:8529: socket: too many open files","status":"Internal Server Error"}
I am trying to insert many many data one by one. So service is connecting to arangodb everytimes.
How can I solve this issue?

You should use HTTP 1.1 and persistent connections in first place.
You could also use the bulk interface for your imports.

thanks, @dothebart
Then how can I authorise arangodb? The sample of document is not working.
How can I select database and collection?

ArangoDB provides a RESTfull interface, you specify database in the request URI if they differ from the default:
/_db/myDatabase/
With HTTP1.1 you only share one tcp connection between requests, no other state like authenticification is shared between two requests over the same connection. Thus you have to re-specify the auth-basic header again.

I am using https://github.com/solher/arangolite in golang.
This driver is also using HTTP1.1 request. Isn't it?

The officially supported go driver can be found here: https://github.com/arangodb/go-driver for issues with arangolite please contact the original author.

hi, @dothebart
it seems, arangodb/go-driver doesn't support bulkimport feature. right?

as I tried to explain in my edits to the comment you should be able to use the regular document save interface with several documents at once - which should mostly become the same thing.

@TianDaGe does the described procedure work for you? could we close this issue?

Closing as solved. Don't hesitate to ask if there are still remaining questions.


Add boilerplate for a static site

Currently uses Typeforms.io for contact forms -
TODO:

Fill in appropriate details for the config.yaml to update information about this particular org/website
Get a Typeforms.io Pro account (they give free accounts to students/non profits so this shouldn't be a problem)
Replace typeforms forms with own
Get input on design (its crap atm) and change accordingly.
To push to gh-pages run hugo in the directory (after having installed go and victor hugo) and push the files under public/ to gh-pages.

@CalmDownKarm  we already have boilerplate code with Node for build system for frontend.
Also, we don't require Typeform account as we'll be building contact form on our own backend.

Alright, closing this pull request, since the proposed changes aren't required by the project as of now.


[Playground] SSL Certificate and HTTPS enablement

Current Behavior
Lack of SSL termination on playground deployment.
Desired Behavior
Secure transmissions between users and playground.

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Certificate and private key generated.

@leecalcote can you please assign the issue to me.

@ayush14rastogi oh, yes, I would like to do so. Although, this particular item has already been completed. The playground site is almost ready for use: https://playground.meshery.io (but at least does now support SSL).
Let's get you onto another issue, @ayush14rastogi 👍


Add logging to api calls

Log the uri and body (if it exists) for api requests. Log the response status and body.
To use (in rails), do the following in config/initializers/docusign_rest.rb:
config.logger = Rails.logger

By default, the logger is set to Logger.new('/dev/null'), which just does nothing.
Jon -
We have been using your gem very successfully for many months now, thanks for taking the time to open source your code.
When we started development on our docusign project, one of the things that docusign asked before they would "certify" us (or whatever they call it) is that we be able to capture the requests we make to them in the event that we report a bug. This PR doesn't fully meet their requirements (because they want the raw http request), but this gets very close, and is very useful for debugging regardless.
This code is not complete in the sense that there are still a few calls that go unlogged, but hopefully this will be useful despite that, and we can always do the rest of the calls later.
We only use a couple of calls in our app, so the majority of this changed code has not been tried in production.

I've merged https://github.com/jondkinney/docusign_rest/pull/54 from @Tuckie ; I think that satisfies this requirement.   If that's missing something, please let me know, thank you!