Deploy 091 - Upgrade OETHVaultCore

Contracts

OETHVaultCore: 0x8f371d8e65f35914cdb8dd58b997411871dabb37

Governance

Proposal:<PHONE_NUMBER>1544237322939434422056971914944279457140328550595948923573494383344
Proposal Tx: 0xf3fa5f5bca75f0dc10e712d8296ef095be14b16d7b67d478c21fad38e4b574f8

If you made a contract change, make sure to complete the checklist below before merging it in master.
Refer to our documentation for more details about contract security best practices.
Contract change checklist:

[ ] Code reviewed by 2 reviewers.
[ ] Copy & paste code review security checklist below this checklist.
[ ] Unit tests pass
[ ] Slither tests pass with no warning
[ ] Echidna tests pass if PR includes changes to OUSD contract (not automated, run manually on local)

I have verified the following

The original PR to master only has one approver but I've since review it. https://github.com/OriginProtocol/origin-dollar/pull/1995
The proposal is upgrading the proxy to the correct implementation address.
The proposal calls cacheWETHAssetIndex on the Vault
The OETH Vault proxy address in the proposal is correct
The deployed OETHVaultCore implementation contract matched the source code on this shah/deploy-091 branch.

sol2uml diff 0x8f371d8e65f35914cdb8dd58b997411871dabb37 .,node_modules
Compared the "OETHVaultCore" contract with address 0x8f371d8e65F35914CDb8Dd58B997411871dABb37 on mainnet
to local files under folders ".,node_modules"

match   @openzeppelin/contracts/token/ERC20/IERC20.sol
match   @openzeppelin/contracts/token/ERC20/utils/SafeERC20.sol
match   @openzeppelin/contracts/utils/Address.sol
match   @openzeppelin/contracts/utils/math/SafeMath.sol
match   contracts/governance/Governable.sol
match   contracts/interfaces/IBasicToken.sol
match   contracts/interfaces/IGetExchangeRateToken.sol
match   contracts/interfaces/IOracle.sol
match   contracts/interfaces/IStrategy.sol
match   contracts/token/OUSD.sol
match   contracts/utils/Helpers.sol
match   contracts/utils/Initializable.sol
match   contracts/utils/InitializableERC20Detailed.sol
match   contracts/utils/StableMath.sol
match   contracts/vault/OETHVaultCore.sol
match   contracts/vault/VaultCore.sol
match   contracts/vault/VaultInitializer.sol
match   contracts/vault/VaultStorage.sol

I've verified that the deployed contract address 0x8f371d8e65f35914cdb8dd58b997411871dabb37 matches the sources in git: npx sol2uml diff 0x8f371d8e65f35914cdb8dd58b997411871dabb37 .,node_modules
there are 2 proposals created for this deploy, the one with id<PHONE_NUMBER>1544237322939434422056971914944279457140328550595948923573494383344 should be passed
the proposal upgrades the OETHVaultCore of the OETHVaultProxy (addresses correct), and calls cacheWETHAssetIndex on the OETHVaultProxy

Source code verification:
npx sol2uml diff 0x8f371d8e65f35914cdb8dd58b997411871dabb37 .,node_modules
Compared the "OETHVaultCore" contract with address 0x8f371d8e65F35914CDb8Dd58B997411871dABb37 on mainnet
to local files under folders ".,node_modules"

match   @openzeppelin/contracts/token/ERC20/IERC20.sol
match   @openzeppelin/contracts/token/ERC20/utils/SafeERC20.sol
match   @openzeppelin/contracts/utils/Address.sol
match   @openzeppelin/contracts/utils/math/SafeMath.sol
match   contracts/governance/Governable.sol
match   contracts/interfaces/IBasicToken.sol
match   contracts/interfaces/IGetExchangeRateToken.sol
match   contracts/interfaces/IOracle.sol
match   contracts/interfaces/IStrategy.sol
match   contracts/token/OUSD.sol
match   contracts/utils/Helpers.sol
match   contracts/utils/Initializable.sol
match   contracts/utils/InitializableERC20Detailed.sol
match   contracts/utils/StableMath.sol
match   contracts/vault/OETHVaultCore.sol
match   contracts/vault/VaultCore.sol
match   contracts/vault/VaultInitializer.sol
match   contracts/vault/VaultStorage.sol


Errors when running dbt deps

I tried to follow the instructions here to set up the demo but after running dbt deps I run into some errors:
CaughtMacroErrorWithNodeError
Compilation Error in seed terminology__icd_10_cm (seeds/terminology/terminology__icd_10_cm.csv)
  macro 'dbt_macro__snowflake__load_seed' takes not more than 1 argument(s)

  > in macro load_seed (macros/load_seed.sql)
  > called by seed terminology__icd_10_cm (seeds/terminology/terminology__icd_10_cm.csv)
CaughtMacroErrorWithNodeError

it seems that there has been some changes in the tuva project macros and the post hooks no longer work?
Changing the version in dbt_packages.yml to:
packages:
  - package: tuva-health/the_tuva_project
    version: [">=0.3.0","<=0.3.16"]

solves the problem. I'm using dbt cloud v1.6

Forcing version 0.3.16 seems to create problems when running dbt build
15:34:45 Database Error in seed terminology__provider (seeds/terminology/terminology__provider.csv)
  100080 (22000): Number of columns in file (17) does not match that of the corresponding table (15), use file format option error_on_column_count_mismatch=false to ignore this error
    File 'provider_data/provider.csv_0_1_1.csv.gz', line 2, character 1
    Row 1 starts at line 1, column "PROVIDER"[17]
    If you would like to continue loading when an error is encountered, use other values such as 'SKIP_FILE' or 'CONTINUE' for the ON_ERROR option. For more information on loading options, please run 'info loading_data' in a SQL client.
  compiled Code at target/run/the_tuva_project/seeds/terminology/terminology__provider.csv

Hi @Gonzalo933 thank you so much for reporting this bug. I'm looking into it now.

@Gonzalo933 I just pushed a bug fix. Let me know if you are still running into issues with this.

This issue is fixed! Thank you.
I keep running into the other issue when loading one of the seeds. I can open a new issue with it
13:17:04 Database Error in seed terminology__provider (seeds/terminology/terminology__provider.csv)
  100080 (22000): Number of columns in file (17) does not match that of the corresponding table (15), use file format option error_on_column_count_mismatch=false to ignore this error
    File 'provider_data/provider.csv_0_1_0.csv.gz', line 2, character 1
    Row 1 starts at line 1, column "PROVIDER"[17]
    If you would like to continue loading when an error is encountered, use other values such as 'SKIP_FILE' or 'CONTINUE' for the ON_ERROR option. For more information on loading options, please run 'info loading_data' in a SQL client.
  compiled Code at target/run/the_tuva_project/seeds/terminology/terminology__provider.csv

@Gonzalo933 we recently made some data model changes to the Provider seed. Make sure you are running with --full-refresh. That should solve the issue. Let us know if it doesn't!


[BUG]Disabled datasource should be greyed out out removed from datasources list in query workbench.

When a datasource is disabled, user shouldn't even be allowed to go to this page and see the error.
it should be disabled from selection in the below list itself.

Catch All Triage - 1 2 3 4 5 6


ValueError: Cannot convert non-finite values (NA or inf) to integer

I'll fix this.
Traceback (most recent call last):
  File "/home/kfuku/.pyenv/versions/miniconda3-4.3.30/bin/amalgkit", line 326, in <module>
    args.handler(args)
  File "/home/kfuku/.pyenv/versions/miniconda3-4.3.30/bin/amalgkit", line 33, in command_getfastq
    getfastq_main(args)
  File "/home/kfuku/.pyenv/versions/miniconda3-4.3.30/lib/python3.8/site-packages/amalgkit/getfastq.py", line 663, in getfastq_main
    total_sra_bp = int(metadata.df['total_bases'].astype(int).sum())
  File "/home/kfuku/.pyenv/versions/miniconda3-4.3.30/lib/python3.8/site-packages/pandas/core/generic.py", line 5874, in astype
    new_data = self._mgr.astype(dtype=dtype, copy=copy, errors=errors)
  File "/home/kfuku/.pyenv/versions/miniconda3-4.3.30/lib/python3.8/site-packages/pandas/core/internals/managers.py", line 631, in astype
    return self.apply("astype", dtype=dtype, copy=copy, errors=errors)
  File "/home/kfuku/.pyenv/versions/miniconda3-4.3.30/lib/python3.8/site-packages/pandas/core/internals/managers.py", line 427, in apply
    applied = getattr(b, f)(**kwargs)
  File "/home/kfuku/.pyenv/versions/miniconda3-4.3.30/lib/python3.8/site-packages/pandas/core/internals/blocks.py", line 673, in astype
    values = astype_nansafe(vals1d, dtype, copy=True)
  File "/home/kfuku/.pyenv/versions/miniconda3-4.3.30/lib/python3.8/site-packages/pandas/core/dtypes/cast.py", line 1068, in astype_nansafe
    raise ValueError("Cannot convert non-finite values (NA or inf) to integer")
ValueError: Cannot convert non-finite values (NA or inf) to integer

This error was caused originally by a truncated metadata row, in which important info such as total_bases are missing.

fixed in https://github.com/kfuku52/amalgkit/commit/427c483f75aacfb497a8fbeaaa6b5643a49b4c16


Environment Detection

Summary
firmware/utilities/build includes a shell script that detects the build environment. It works in both Windows and Linux.
Test Plan

Build from firmware/ec. Verify that firmware/utilities/build/build_ec_cfg.txt is created and firmware builds with no new issues. The text file should be created even if the build fails.
Build from firmware/host. Verify that firmware/utilities/build/build_host_cfg.txt is created and host builds with no new issues. The text file should be created even if the build fails.
Build from firmware/ec/test.  Verify that firmware/utilities/build/build_ut_cfg.txt is created and firmware builds with no new issues. The text file should be created even if the build fails.

Issues
Resolved issue #171

Can you add it to the unit tests? Have it grab the gcc version, and if possible the Unity version located in $(UNITY_ROOT)/release/version.info?
Can you also add a comment in the bug_repot.md template referring to this cfg output and asking for it to be attached?

build_ec_cfg.txt
build_host_cfg.txt
build_ut_cfg.txt
Sample configuration files attached.

I am not sure if this PR is the best place to suggest this comment .Currently the firmware directory is a bit hardcoded as below .
OCWARE_DIR ?= $(HOME)/OpenCellular/firmware/ec
There are times where we might be working on multiple issues at the same time . This restricts the build environment . Can we have
OCWARE_DIR ?= $(shell pwd)

If TIRTOS_DIR , XDCTOOLS_DIR and TOOLCHAIN is not present in the static path provided in the makefile can we add intelligence into this script to detect and update into this new path into the makefile.

Not sure what this suggestion means.  The script is already detecting if TIRTOS_DIR and TOOLCHAIN are valid paths (see lines 12 and 17 in envDetect.sh).  As for UNITY_ROOT, the build will just fail if you don't have Unity.

I am not sure if this PR is the best place to suggest this comment .Currently the firmware directory is a bit hardcoded as below .
OCWARE_DIR ?= $(HOME)/OpenCellular/firmware/ec
There are times where we might be working on multiple issues at the same time . This restricts the build environment . Can we have
OCWARE_DIR ?= $(shell pwd)

We can do it in this PR.

If TIRTOS_DIR , XDCTOOLS_DIR and TOOLCHAIN is not present in the static path provided in the makefile can we add intelligence into this script to detect and update into this new path into the makefile.

Not sure what this suggestion means. The script is already detecting if TIRTOS_DIR and TOOLCHAIN are valid paths (see lines 12 and 17 in envDetect.sh). As for UNITY_ROOT, the build will just fail if you don't have Unity.

Will line if [ -d "$2" ]; search TIRTOS directory into HOME path or it can be from any path ?

If you had several toolchains and used find to search for them, how would you know which one you intend to build with?  The makefile always knows (or the build fails).  This is why the makefile passes the location of these directories to the script.

Are there outstanding issues with this PR?


Compatibility with ubuntu

To ensure compatibility with arduino on ubuntu, please change:
#include <arduino.h>
to
#include <Arduino.h>
in ens210.cpp line 8

Sorry for the late fix; I didn't know github featured issue submission.
Thanks for the comment.


Istio CA Service definition

First draft of Istio CA service definition.

@wattli @myidpt @lookuptable Please take a look and comment :-)

Jenkins job auth/presubmit passed

Jenkins job auth/presubmit passed

Jenkins job auth/presubmit passed

Jenkins job auth/presubmit passed

Jenkins job auth/presubmit passed

Jenkins job auth/presubmit passed

@geeknoid , this protobuf is used by istio node agent calling istio CA (they are both istio auth components). Should we put this in istio/api or this repo?

If the node agent and Istio CA are in different repos, then put the API in
istio/api. If they're both in the same repo, then we consider the interface
to be private and so can go in the repo itself.
On Fri, Jun 30, 2017 at 1:16 PM, Tao Li<EMAIL_ADDRESS>wrote:

@geeknoid https://github.com/geeknoid , this protobuf is used by istio
node agent calling istio CA (they are both istio auth components). Should
we put this in istio/api or this repo?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/auth/pull/164#issuecomment-312363499, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AVucHQJqwihFXMwPdEjvE6WsYJhU45JPks5sJVe7gaJpZM4OJ7oD
.

Jenkins job auth/presubmit passed


feat(linter) eslint-plugin-next no-css-tags

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

main

PR #1931

PR #1932

PR #1933

PR #1934

PR #1935

PR #1936

PR #1937   👈

This stack of pull requests is managed by Graphite.

Merge activity

Jan 7, 11:38 PM: @Boshen merged this pull request with Graphite.


Ability to render a Starlark dict into JSON or YAML for templating use cases

Background & motivation
My config files depend on arrays of services: https://github.com/galenmarchetti/basic-service-package/blob/main/service-b-config.json.tmpl <--- for this demo its very important to have it, and I've had this use case in non-demo (but non-kurtosis-things) before as well. In order to render the array I have to do write my own JSON rendering logic in starlark
Desired behaviour
I'd love to have a utility to render a starlark dict to either JSON or YAML so i can use it cleanly in my service config files
How important is this to you?
Painful; the lack of this feature makes using Kurtosis frictionful.

Talked with Galen about this in a user experience session on Friday; notes forthcoming

The render-to-JSON/YAML part is actually very possible; the bit that's missing is a way to write that JSON/YAML to a files artifact

@galenmarchetti does this package solve your pain point? https://bazel.build/rules/lib/core/json


Is make_timestampz(microseconds) deprecated?

From the documentation we have the following table:

make_timestamptz(microseconds)
The timestamp with time zone for the given µs since the epoch
make_timestamptz(1667810584123456)
2022-11-07 16:43:04.123456-08

However when I try to follow the example:
v0.8.0 e8e4cea5ec
Enter ".help" for usage hints.
Connected to a transient in-memory database.
Use ".open FILENAME" to reopen on a persistent database.
D SELECT make_timestamptz(1667810584123456);
Run Time (s): real 0.001 user 0.000361 sys 0.000144
Error: Binder Error: No function matches the given name and argument types 'make_timestamptz(BIGINT)'. You might need to add explicit type casts.
	Candidate functions:
	make_timestamptz(BIGINT, BIGINT, BIGINT, BIGINT, BIGINT, DOUBLE) -> TIMESTAMP WITH TIME ZONE
	make_timestamptz(BIGINT, BIGINT, BIGINT, BIGINT, BIGINT, DOUBLE, VARCHAR) -> TIMESTAMP WITH TIME ZONE

LINE 1: SELECT make_timestamptz(1667810584123456);
               ^

Looks like the documentation is a bit out of date here - as the error message indicates, the function signature has changed, but it still exists.
v0.8.1 6536a77232
Enter ".help" for usage hints.
Connected to a transient in-memory database.
Use ".open FILENAME" to reopen on a persistent database.
⚫◗ set timezone = 'UTC'; -- for reproducible results
⚫◗ select make_timestamptz(9999, 1, 1, 1, 1, 1, 1);
┌──────────────────────────────────────────┐
│ make_timestamptz(9999, 1, 1, 1, 1, 1, 1) │
│         timestamp with time zone         │
├──────────────────────────────────────────┤
│ 9999-01-01 01:01:01+00                   │
└──────────────────────────────────────────┘
⚫◗ select make_timestamptz(9999, 1, 1, 1, 1, 1, 'Australia/Sydney');
┌───────────────────────────────────────────────────────────┐
│ make_timestamptz(9999, 1, 1, 1, 1, 1, 'Australia/Sydney') │
│                 timestamp with time zone                  │
├───────────────────────────────────────────────────────────┤
│ 9998-12-31 14:01:01+00                                    │
└───────────────────────────────────────────────────────────┘

Actually, it looks like the make_timestamptz(US) version is newer than the latest release version (https://github.com/duckdb/duckdb/pull/7597), so should be available in the next release

Thanks @Mause ! That is great news - felt like I found a little easter egg!

Tripped over the same issue.
@Mause Would it be possible to add "available since version x.y" to the reference manual? It could save us some trouble in the future...

Tripped over the same issue.
@Mause Would it be possible to add "available since version x.y" to the reference manual? It could save us some trouble in the future...

I was actually thinking about that this morning, and automating it using the function docs rework I'm working on right now


CC and BCC for MIME types

It seems that it is not possible to have CC and BCC in an email with HTML body, for example, since you have to explicitly set "to" field after mailComposer builds the message:
mailComposer.buildMessage(function(mailBuildError, messageSource) {
    var dataToSend = {
      to: recipients[0],
      message: messageSource
    };

    mgSender.messages().sendMime(dataToSend, function (error, response) {
    ...

[This] (http://blog.mailgun.com/mailgun-api-2-0-forget-mime/) might help other users.

Here is how I did it using request:
function sendEmail(event, content) {
  if (!process.env.MAILGUN_API_KEY) {
    winston.error('Mailgun API key must be set as an environment variable');
  }

  var noReplyAddr =<EMAIL_ADDRESS>  var form = {
    from: 'Company Name ' + noReplyAddr,
    to: recipients[0],
    subject: 'my subject',
    text: text,
    html: html
  };
  if (recipients.length > 1) {
    form.bcc = recipients.slice(1);
  }

  var deferred = Q.defer();

  function callback(error, response) {
    if (error) {
      console.log("Mailgun ERROR:", error);
      deferred.reject(error);
    } else {
      console.log("Mailgun SUCCESS.");
      deferred.resolve(response);
    }
  }

  request.post({
        url: myMailgunUrl,
        form: httpForm
      },callback)
      .auth('api', process.env.MAILGUN_API_KEY, false);

  return deferred.promise;
}

Hello, I am not sure I understand completely. In first comment you're sending to mime endpoint, in the comment above you seem to be sending to the normal api endpoint (I think)? This worked fine for me:
var params = {
  to<EMAIL_ADDRESS>  bcc<EMAIL_ADDRESS>  from<EMAIL_ADDRESS>  subject: 'testing stuff again',
  text: 'testing message for mailgun',
  html: 'HTML version of body'
};

mailgun.messages().send(params, function (err, body) {
  if (err) console.dir(err);
});

When you deal with Mailgun's HTTP API, you don't need to worry about mime type, as those are handled by Mailgun's backend. You can see from my httpForm that I have a HTML attribute (which is not possible without having mime type set properly). You can have attachments and other stuff as well, and Mailgun server composes and handles mime type properly.

I am still not sure what's the issue here. Multiple examples were given without full explanation that I understand on what exactly the issue is or how to reproduce it. I have shown working sample code that does exactly what I understood the issue is. Also the library has been updated to version 3 of the Mailgun API. Without a reproducible sample source code that demonstrates the problem I am not sure what I can do. Closing due to lack of clarification, activity and clear sample source. I am free to reopening if more information is given.

@bojand This should be reopened. I have the same problem, You HAVE to set the 'to' field or no emails will be sent. But I ONLY want to send BCC emails! What do I do here, except looping the send function?
Additionally defining all recipients in 'to' as explained in #9 doesn't work for me either.

Please try the latest release: 0.7.4. If that doesn't work, please explain exactly the issue, add sample source code to reproduce it and demonstrate the problem. Please see my last comment in the thread. Without sample code and an actual identifiable bug there is little I can do.

For those still running into this (like I just did):
As per the Mailgun documentation, you have to add the BCC addresses to the TO addresses. However Mailgun will remove them from the TO address list and your BCC recipients will not see any other BCC addresses.
If you use mailcomposer, make sure you don't add the BCC field because then your BCC recipients will see all addresses.

It seems possible some users might have made the mistake of assuming that mail channels use the recipients in the message headers - they don't. Routing information is handled entirely separately from the message content (headers and body).
The recipient(s) of an email do not need to be named in the message headers at all, though it is conventional to do so. If they did the Bcc feature could never work, since the addresses would have to be part of the message and so wouldn't be hidden! Particularly in the case of spam, recipients and senders in message headers can differ from the addresses used to for routing. Thats why the latter have to be supplied to the API request as separate parameters, rather than being implied from the message content.


[WIP] Add officehome dataset

Done:

Fetch zip from fig share + unzip it
Parse CSV
Change OfficeHomeDomain values
Create Test
FInd a way to not parse the csv for each new domain (too time intensive)

Merge 1st PR #197 to make the tests work


Active date is not highlighted when using keyboard

Steps to reproduce

activate the date picker
use keyboard to select different date

Expected behavior

the new date is highlighted

Actual behavior

no date is hightlighted

Fixed in #91 . Thanks @DenysVuika .


Release to GitHub, Native binaries on CI, README overhaul

When using RMF-Codegen in documenation we found it's minimal documentation and not only slightly hidden release artifact on Bintray hinders adoption and proper usage.
Here's how the releases would look like: https://github.com/nkuehn/rmf-codegen/releases
This

Overhauls the README and adds the CLI help output to the README so you can see all the glory without having to download it
Adds an additional release CI workflow that builds a JAR and Mac / Linux native binaries and publishes these as a GitHub release, which has the advantage to have a browseable UI

General hints:

windows binaries not built, there's some problem with the build which (I think) should only be tackled after the native builds are error free on the other platforms.
the linked javascript / NPM package is no merged and published yet in the docs kit (PR https://github.com/commercetools/commercetools-docs-kit/pull/760 ) - not an issue since we don't have a lot of external contributors yet.
java 11 used for the binary builds because the windows dependencies are there by default which they aren't for Java 8 (plus, a newer VM anyways).  The JAR and general implementation stays on java 8 though.

Cool 👍 In my blog https://katmatt.github.io/posts/graalvm-cross-compile-gh-actions/ I describe that updating to a newer java version actually fixed my issue with the windows build 😄

@katmatt I totally forgot about your blog post - could have saved some time.
Still don't know why windows fails though  even with Java 11.  It could be specific to rmf-codegent, too.  The fact that it works for you points a bit towards that.

@katmatt I totally forgot about your blog post - could have saved some time.
Still don't know why windows fails though even with Java 11. It could be specific to rmf-codegent, too. The fact that it works for you points a bit towards that.

May be you're right, but my expectation would be that if it compiles on other platform that it then should work on windows too. And then it would point towards a tooling problem.  And anyway we don't need a windows native build since everyone is using a Mac and our CI systems use linux.

Ready now.  The two release workflows are separate again to not trouble the Bintray release because something with the binary builds has issues.


Update with new PowerShell cmds storage-service-encryption-customer-managed-keys.md

Step 4 : Added PowerShell commands to associate an existing storage account with a KeyVault Key to be used for SSE with CMK.
storageAccount = Get-AzureRmStorageAccount -ResourceGroupName "RGSSECMK" -AccountName "storagetousecmk"
$keyVault = Get-AzureRmKeyVault -VaultName "cmkwithsettings"
$key = Get-AzureKeyVaultKey -VaultName $keyVault.VaultName -Name "keytoencrypt"
Set-AzureRmKeyVaultAccessPolicy -VaultName $keyVault.VaultName -ObjectId $storageAccount.Identity.PrincipalId -PermissionsToKeys wrapkey,unwrapkey,get
Set-AzureRmStorageAccount -ResourceGroupName $storageAccount.ResourceGroupName -AccountName $storageAccount.StorageAccountName -EnableEncryptionService "Blob" -KeyvaultEncryption -KeyName $key.Name -KeyVersion $key.Version -KeyVaultUri $keyVault.VaultUri

@AngshumanNayakMSFT : Thanks for your contribution to the Azure documentation! The author, @lakasa, has been notified to review your proposed change.

@AngshumanNayakMSFT - Thank you for your contribution.
Would you mind making the names of the resource group, storage account, etc., non-specific? For example, replace
"RGSSECMK"
with

and so on.

@AngshumanNayakMSFT : Thanks for your contribution to the Azure documentation! The author, @lakasa, has been notified to review your proposed change.

@AngshumanNayakMSFT - Looks good, thanks.
#sign-off

@tamram: I'm sorry - only the author of this article, @lakasa, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Bolt.New subscription missing

Describe the bug
I've already sent a direct message to the team, but have not received a response. My subscription is missing from bolt.new and I can no longer use my credits. It's now been this way for two days. I'm all for giving room to fix, but it would be great to obtain help as a paying customer. This is in addition to history loss as reported by other users.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-c5dnrd
Steps to reproduce

Go to bolt.new.
Sign In.
Go to My Subscription.

My tokens are missing, so I can no longer generate new content. History is not working and the subscription is missing entirely from the management page.
Expected behavior
I should be on the Pro 50 plan.
Screen Recording / Screenshot
No response
Platform

OS: macOS
Browser: Chrome

Additional context
No response

@dsudduth just reached out via email!


Add React v18 strict mode support

Hi everyone, thanks for making this awesome library!
The goal of this PR is to add strict mode support for React v18
React v18 in strict mode runs useEffect twice in dev mode, as documented in ReactJS blog
For this reason, the cleanup function is called after the initial mount of the component, which does call/clear this.unsubs, but the scrolling element this.scrollElement remains set.
When useEffect runs the second time, the _willUpdate if check
if (this.scrollElement !== scrollElement) {...

evals as false, and prevents new subs from being set, resulting in broken virtualisation behaviour, as noted in issue #322
This PR aims to solve the issue by enhancing the cleanup function so the aforementioned if block evals to true on subsequent re-renders so the new subs are properly set.
Here is the relevant react-virtual code for context: https://github.com/TanStack/virtual/blob/beta/packages/react-virtual/src/index.tsx#L39-L45

With your current suggested changes _willUpdate will always unsubscribe and resubscribe to the listeners everytime it runs even when the scrollElement ref remains the same.

I don't understand why this would happen as _willUpdate has a guard against unnecessary cleanup (line 298) in case the scroll element didn't change:
https://github.com/TanStack/virtual/blob/f41852b8f011e968eddd0413e43ab8fd5c6c3866/packages/virtual-core/src/index.ts#L295-L301
The _didMount cleanup function will run twice:

after the initial mount (due to React v18 strict mode)
on unmount

https://github.com/TanStack/virtual/blob/f41852b8f011e968eddd0413e43ab8fd5c6c3866/packages/virtual-core/src/index.ts#L289-L293
Other than these two cleanup runs, the this.scrollElement reference should be stable and should not lead to _willUpdate doing unnecessary unsubscribe/resubscribe runs.

This breaks scrolling FYI. I had to go back to beta 6 to scrolldown properly.

@Tav0 I cannot reproduce your issue, please create a sandbox. Maybe based on this example:
https://codesandbox.io/s/react-virtual-table-example-7v4vbc?file=/src/main.tsx
Thanks.


Training from

Hi, thanks a lot for offering AdaGram.

How can one train a model from many different files? Does one have to provide a single pre-procesed file with all text appended together?

Also, do you offer some clustering algorithm inside AdaGram, like word2vec does?

Thanks

@glicerico out of interest, which clustering algorithms does word2vec provide? do you mean pre-processing that finds collocations, or something else?

@lopuhin No, I mean clustering after word embedding. They provide K-means activated by a flag when running the code. You can see an example in their demo-classes.sh
From their code website:
Word clustering
The word vectors can be also used for deriving word classes from huge data sets. This is achieved by performing K-means clustering on top of the word vectors. The script that demonstrates this is ./demo-classes.sh. The output is a vocabulary file with words and their corresponding class IDs, such as:
carnivores 234 carnivorous 234 cetaceans 234 cormorant 234 coyotes 234 crocodile 234 crocodiles 234 crustaceans 234 cultivated 234 danios 234 . . . acceptance 412 argue 412 argues 412 arguing 412 argument 412 arguments 412 belief 412 believe 412 challenge 412 claim 412

So @lopuhin , I'm guessing you don't know how to train a model using different files?

@glicerico I'm not an AdaGram.jl developer, but I'm pretty sure that if you want to use the command line interface, you must have a single file.
And I don't think AdaGram.jl has any clustering algorithms, but you can extract the sense vectors and do clustering yourself.

@lopuhin I just submitted a pull request that adds clustering functionality to AdaGram
https://github.com/sbos/AdaGram.jl/pull/22

Hi @glicerico,
The code we provide supports only files as inputs, but it should be fairly simple to adapt it for generic streams.
If you manage to implement this, please do send a pull request!
@lopuhin Thanks for the pull request, I will try to review it soon. I unfortunately don't have much time to support AdaGram as it was a research project during my PhD.

@sbos thanks! For the record, it's @glicerico 's pull request :)
And thanks for putting the code on github, it's great and really useful!

@lopuhin you are welcome
@glicerico Sorry for confusing your authorship! What version of julia you are using BTW?

@sbos No worries. I'm using Julia v0.4.5.
My first question is related to being able to "add learning" to a model. Say that you train a model with some corpus, and later you want to add another corpus to it without reprocessing the first one. But I think I see this is not possible from the way the algorithm itself works.

@glicerico Well, the training algorithm is based on stochastic gradient optimization which means that you could continue training on a different file.
But you might need to do something against catastrophic forgetting and generally think about adaptive learning rates for different words.


react-dates v16.3: ejected create-react-app with CSS Modules enabled adds different local identifiers than SingleDatePicker's

Please see the attached screenshots for the easiest reference to my issue.
I have a project built using the latest create-react-app and of course react-dates. I ejected create-react-app so that I could enable CSS Modules and some other goodies, but after doing so the _datepicker.css file ends up with different class names.
The screenshots show CSS Modules successfully getting imported as well as the final class names, along with how SingleDatePicker's class name gets rendered. The issue is visible in the 2 webpack screenshots where I found the problem - any plans to allow adding local Identifiers to classes? Or is there a better way to resolve this? I was hoping there would be some way to enforce class name inheritance from CSS Modules

Hi @mrgrotts, I think that this is actually possible but you'd be the first to use the feature! :) It would require your own theme/interface registration instead of using the initialize script.
Here is what you would need to do instead of calling import 'react-dates/initialize';:
import CSSInterface, { registerCSSInterfaceNamespace } from 'react-with-styles-interface-css';
import registerInterfaceWithDefaultTheme from 'react-dates/lib/utils/registerInterfaceWithDefaultTheme';

registerCSSInterfaceNamespace('datepicker');
registerInterfaceWithDefaultTheme(CSSInterface);

After you do this, you should be able to import and use your react-dates components as needed.
In theory, that should prefix all the classnames in the components with whatever string you provide (https://github.com/airbnb/react-with-styles-interface-css#interface). Let me know how it goes! If it doesn't work as is, we may need to go back and do some fixes to react-with-styles-interface-css.

Well that sounds exciting! I will give this a try when I get home. Thank you for the help either way :)

I gave this solution a try but no luck. I npm installed react-with-styles-interface-css and added the lines above in. Webpack compiles successfully and launches but the app crashes in browser right away with this error:
bundle.js:51353 Uncaught TypeError: Object(...) is not a function     at Object../src/App.js (bundle.js:51353)     at __webpack_require__ (bundle.js:708)     at fn (bundle.js:113)     at Object../src/index.js (bundle.js:58102)     at __webpack_require__ (bundle.js:708)     at fn (bundle.js:113)     at Object.0 (bundle.js:59552)     at __webpack_require__ (bundle.js:708)     at bundle.js:806     at bundle.js:809
I fooled around for a bit but wasn't able to find the exact problem. I did notice 2 things in react-with-styles-interface-css that caught my eye:
dist/index.js : 21 --> namespace is set to an empty string on sharedState, from the GlobalCache. Right after this all styles are created in to an array of classes using the updated namespace - isn't it now an empty string?
dist/index.js : 40 --> this is where the suffix is coming from, and you can see the namespace's length must be 0 because the output from this function is what's seen in the screenshot I shared originally.

I was facing the same issue. For other people who end up here needing a quick (and dirty) fix:
You can use the "Without Webpack" section of the README to configure these styles to be added directly in your <head> in which case Webpack won't change your classnames.
This approach comes with its cons, so hopefully other more maintainable approaches can still be found!

I faced the same problem. My fix was to make a clause in the loaders specifically for node_modules css to avoid them being loaded as css modules:
{
        test: /node_modules\/.*\.css$/,
        use: [
          {
            loader: 'style-loader',
          },
          {
            loader: 'css-loader',
            options: {
              sourceMap: true,
            },
          },
        ],
      },
      {
        test: /\.css$/,
        exclude: /node_modules/,
        use: [
          {
            loader: 'style-loader',
          },
          {
            loader: 'css-loader',
            options: {
              modules: true,
              sourceMap: true,
              importLoaders: 1,
              localIdentName: '[name]__[local]__[hash:base64:5]',
            },
          },
        ],
      },

@philippefutureboy thanks a lot for your solution it works fine for UNIX. If someone is using Windows you should change this line
test: /node_modules\/.*\.css$/,
on this one
test: /node_modules[\\\/].*\.css$/,

@philippefutureboy thank you very much it is working but if I do it my global styles are not being imported. Any ideas?
import React from 'react';
import ReactDOM from 'react-dom';
import './index.css';
import App from './App';
import registerServiceWorker from './registerServiceWorker';

ReactDOM.render(<App />, document.getElementById('root'));
registerServiceWorker();

{
						test: /node_modules\/.*\.css$/,
						use: [{
								loader: 'style-loader',
							},
							{
								loader: 'css-loader',
								options: {
									sourceMap: true,
								},
							},
						],
					}, {
						test: /\.scss/,
						exclude: /node_modules/,
						use: [{
								loader: 'style-loader',
							},
							{
								loader: 'css-loader',
								options: {
									modules: true,
									sourceMap: true,
									importLoaders: 1,
									localIdentName: '[name]__[local]__[hash:base64:5]',
								},
							},
							'sass-loader'
						],
					},

@gdi-user02 The file you are importing in App.js is a .css file. You seem to have forgotten to add a config block for .css files in your config.

@philippefutureboy  thank you for quick response, any idea how can achieve that? Not much experience with Webpack.

@gdi-user02 You could try
{
  test: /node_modules\/.*\.css$/,
  use: [
    {
      loader: 'style-loader',
    },
    {
      loader: 'css-loader',
      options: {
        sourceMap: true,
      },
    },
  ],
},
{
- test: /\.scss/,
+ test: /\.(scss|css)$/,
  exclude: /node_modules/,
  use: [
    {
      loader: 'style-loader',
    },
    {
      loader: 'css-loader',
      options: {
        modules: true,
        sourceMap: true,
        importLoaders: 1,
        localIdentName: '[name]__[local]__[hash:base64:5]',
      },
    },
    'sass-loader',
  ],
},

In which case this config will match .scss files as well as .css files. style-loader and css-loader, defined in the use array, should be enough to handle css files.

@Jokinen that wouldn't work given that @gdi-user02 intends to use CSS as global CSS. Instead, I'd suggest something like this:
        test: /node_modules\/.*\.css$/,
        use: [
          {
            loader: 'style-loader',
          },
          {
            loader: 'css-loader',
            options: {
              sourceMap: true,
            },
          },
        ],
      },
      {
        test: /\.css$/,
        exclude: /node_modules/,
        use: [
          {
            loader: 'style-loader',
          },
          {
            loader: 'css-loader',
            options: {
              modules: true,
              sourceMap: true,
              importLoaders: 1,
              localIdentName: '[name]__[local]__[hash:base64:5]',
            },
          },
        ],
      },
  {
    test: /\.scss/,
    exclude: /node_modules/,
    use: [
      {
        loader: 'style-loader',
      },
      {
        loader: 'css-loader',
        options: {
          modules: true,
          sourceMap: true,
          importLoaders: 1,
          localIdentName: '[name]__[local]__[hash:base64:5]',
        },
      },
      'sass-loader'
    ],
  },

I think it would be good if another cssmodules css file was generated along with the the regular css file. in the new generated cssmodules file, all classes should be global, this will make the life of those who use css modules easier.
It won't be hard to generate such a file using the buildCSS script https://github.com/airbnb/react-dates/blob/master/scripts/buildCSS.js, all what needs to be done is prefix all classes with :global or put all the classes into one huge :global block (:global {})

I think it would be good if another cssmodules css file was generated along with the the regular css file. in the new generated cssmodules file, all classes should be global, this will make the life of those who use css modules easier.

Is there any chance that this may happen? Am also running into the same issue and will use a workaround otherwise, but it would be nice to not have to :)


adding mqtt for stand-alone mode

This pull request will add the ability to run this addon standalone for Home Assistant Container installations.

I am not sure about the "addon: 8cd50eef_ha-sip-next" changes in the readme which came from the other pull request.

This is done automatically by the build.sh build-next script. we need to redo those!

Can you rebase a last time?

Not sure what todo next. What do you mean by rebase?

Rebase to the next branch, or merge next into your branch. There are some conflicts which don't allow to merge this branch into my.

hopefully I did everything right..

That does not look too well. If you add me to your fork of the repo, I could try to correct that?

I merged your contributions to the next branch! If you want, you can re-check if everything is correct. I'll create a release in the next few days! Thanks again for your work!

I still have a question: is it possible to write the payload to mqtt.publish as yaml also? For the stdin method the yaml is automatically converted to json when sending it. Would be easier to read.

And would you like to be mentioned in the release notes?

I merged your contributions to the next branch! If you want, you can re-check if everything is correct. I'll create a release in the next few days! Thanks again for your work!

The next branch works great! I tried the run-local and standalone version.
Thank you very much for creating this sip addon and making the standalone addition possible!
Sorry for my bad git skills though. I guess I need to improve those!

And would you like to be mentioned in the release notes?

Sure. :)

Thanks for testing!
The git situation was not that easy to solve, because i rewrote history to have a nice commit history. My bad!
And I got you covered: https://github.com/arnonym/ha-plugins/blob/next/ha-sip/CHANGELOG.md#37
If you want you can make a post to https://community.home-assistant.io/t/home-assistant-sip-gateway/333694 after the release, because they were a few people asking for running ha-sip outside of home-assistant OS.

I've just released 3.7! :partying_face:

Left a quick note in the thread you mentioned! 😉

😆 - I used the search function! 😉 But then I also read through the post.. just to find some other use cases!


Installation fail

Hi,
when I try to install the plugin, i receive this Ruby error:
$ vagrant plugin install vagrant-ova-plugin-0.1.0.gem
Installing the 'vagrant-ova-plugin-0.1.0.gem' plugin. This can take a few minutes...
/opt/vagrant/embedded/lib/ruby/2.2.0/rubygems/package.rb:522:in `rescue in verify': No such file or directory @ rb_sysopen - vagrant-ova-plugin-0.1.0.gem (Gem::Package::FormatError)
    from /opt/vagrant/embedded/lib/ruby/2.2.0/rubygems/package.rb:500:in `verify'
    from /opt/vagrant/embedded/lib/ruby/2.2.0/rubygems/package.rb:484:in `spec'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/bundler.rb:134:in `block in install_local'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/bundler.rb:294:in `block in with_isolated_gem'
    from /opt/vagrant/embedded/lib/ruby/2.2.0/rubygems/user_interaction.rb:45:in `use_ui'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/bundler.rb:293:in `with_isolated_gem'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/bundler.rb:131:in `install_local'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/plugin/manager.rb:47:in `install_plugin'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/plugins/commands/plugin/action/install_gem.rb:37:in `call'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/action/warden.rb:34:in `call'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/action/builder.rb:116:in `call'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/action/runner.rb:66:in `block in run'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/util/busy.rb:19:in `busy'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/action/runner.rb:66:in `run'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/plugins/commands/plugin/command/base.rb:14:in `action'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/plugins/commands/plugin/command/install.rb:32:in `block in execute'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/plugins/commands/plugin/command/install.rb:31:in `each'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/plugins/commands/plugin/command/install.rb:31:in `execute'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/plugins/commands/plugin/command/root.rb:56:in `execute'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/cli.rb:42:in `execute'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/lib/vagrant/environment.rb:308:in `cli'
    from /opt/vagrant/embedded/gems/gems/vagrant-1.8.6/bin/vagrant:174:in `<main>'

Currently using Vagrant 1.8.6.
Tried using Ruby (installed with rvm) ruby-1.9.3-p551 [ x86_64 ] and ruby-2.2.1 [ x86_64 ]
Any clues?
thanks,
A.

Hi @apiraino!
I've also faced this problem, and it's not related to the plugin. This might be caused by some issue on debian vagrant package.
Could you try purging the Debian vagrant package and installing from https://www.vagrantup.com/downloads.html?
Let me know if it works for you!
Thanks,
Marcos

Hi,
thanks for the reply. I'm already using the vanilla package from vagrantup.com. I've updated to Vagrant 1.9.2 and after some tricks (involving a plugin repair), the installation was successful.
However I still cannot use the plugin:
$ vagrant ova --output /tmp/example.ova
Including Minitar via the Archive::Tar namespace is deprecated.
/home/jman/.vagrant.d/gems/2.2.5/gems/vagrant-ova-0.1.0/lib/vagrant-ova/plugin.rb:34:in `execute': undefined method `name' for nil:NilClass (NoMethodError)
	from /opt/vagrant/embedded/gems/gems/vagrant-1.9.2/lib/vagrant/cli.rb:42:in `execute'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.9.2/lib/vagrant/environment.rb:308:in `cli'
	from /opt/vagrant/embedded/gems/gems/vagrant-1.9.2/bin/vagrant:127:in `<main>'


Getting form details through angular frontend

My angular side code is:
<form aria-labelledby="title"  ngNoForm  [action]="url" method="POST">
        <input type="text" name="email" value="" nbInput fullWidth placeholder="Email Address" >
        <input type="text" name="password" value="" nbInput fullWidth placeholder="Password" >
        <button nbButton fullWidth status="primary" class="mbt" (click)="login()">Login</button >
</form>

The url will be directed to "http://<IP_ADDRESS>:8000/getLogin"
Now, I want to fetch details in the fastapi route, but I am unable to fetch. Can anyone answer?
I tried below code:
from fastapi import FastAPI
app = FastAPI()

class userForm(BaseModel):
    email: str = Form(...)
    password: str = Form(...)

@app.post('/getLogin' )
def getLogin(request: Request,userForm: userForm ):
    email = userForm.email
    password = userForm.password
    print(email,password)
    return json.dumps({"reaction":12})

You can see I am getting password and email below but how to get that in my code.

You currently have it setup so the data needs to be in the body as json
If you go to http://<IP_ADDRESS>:8000/docs you should be able to see the api and try it out
I have it working using postman and it looks like:

Yes, there I am able to get data, but how to get in the code?

Not sure what you mean
If sending via postman body returns {"reaction": 12} then doing print(userForm.password) should print to console
If the postman example works but angular doesn't you need to look how the angular request looks and if it sends the data in the body or the header (or if it sends any data at all)

Not sure what you mean
If sending via postman body returns {"reaction": 12} then doing print(userForm.password) should print to console
If the postman example works but angular doesn't you need to look how the angular request looks and if it sends the data in the body or the header (or if it sends any data at all)

Actually , I  have written the above code which works normally, but isnt working with angular

@ArcLightSlavik
I believe you are not sending JSON data to your API

@ArcLightSlavik
I believe you are not sending JSON data to your API

it's a form, you will get the details in the post request anyway

@AshishGusain17
I replicated your issue. If you don't specify an enctype in a form it will have a default value of application/x-www-form-urlencoded

@MacMacky @ArcLightSlavik Hey, I solved that.
This one is a new issue. I have thus far created access_token for each user loggingIn. Now, what can be done for creating sessions for each user to store more data??
Below code created access_token for user loggedIn
@app.post("/login_basic")
async def login_basic(request:Request, username: str = Form(...), password: str = Form(...) ):
        access_token_expires = timedelta(minutes=ACCESS_TOKEN_EXPIRE_MINUTES)
        access_token = create_access_token(
            data={"sub": username}, expires_delta=access_token_expires
        )
        token = jsonable_encoder(access_token)
        response = RedirectResponse(url="/loggedIn")
        response.set_cookie(
            "Authorization",
            value=f"Bearer {token}",
            httponly=True,
            max_age=1800
        )
        return response

Thanks for the help here @ArcLightSlavik and @MacMacky ! ☕
And thanks for coming back to close the issue @AshishGusain17 🍰


False positive for UniqueValidationWithoutIndex when using 'where'

I am using paranoia to soft delete records. I want to allow a specific column of those records to be unique among not deleted entries. I can define the schema like so:
  create_table "categories", force: :cascade do |t|
    t.string "name"
    t.integer "client_id"
    t.datetime "deleted_at"
   # ....
    t.index ["client_id"], name: "index_categories_on_client_id"
    t.index ["name"], name: "index_categories_on_name", where: "deleted_at IS NULL"
  end

In the model, I have the following:
class Category < ApplicationRecord
  acts_as_paranoid

  belongs_to :client

  validates :name, presence: true, uniqueness: { scope: :client_id }
  # ...
end

Expected behavior
Rubocop-rails does not complain about this.
Actual behavior
It does complain with:
app/models/category.rb:17:3: C: Rails/UniqueValidationWithoutIndex: Uniqueness validation should be with a unique index.
  validates :name, presence: true, uniqueness: { scope: :client_id }
  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

RuboCop version
1.4.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.1.1, running on ruby 2.5.7 x86_64-darwin19)
  - rubocop-faker 1.1.0
  - rubocop-performance 1.9.0
  - rubocop-rails 2.8.1
  - rubocop-rspec 2.0.0

This is expected. Since the uniqueness validation is on two columns, the cop expects for a combined unique index to be present on [:name, :client_id]. Refer these specs.

Oh, my bad. I was not paying attention. Thought it was due to the where attribute.


Random Posting in Newsfeed

When we post it will be posted in any order of post feed

As far as posts and comments are concerned when we feed the information inside the database we use the date and time stamp to paste it. And when we fetch it is a built-in property by Firebase to resolve this issue by the command 👍🏼 .orderBy("datePublished", descending: true)


Message.attachments is empty

Hi everyone,
I am very happy with your powerful library, but I have an issue. I want to implement a bot for my mathematic-prof. and I need to catch the attachment in the message the user will send.
I saw you implemented the property attachment in the Class Message, but the property is everytime the same:
{
  "_array": null,
  "_keyArray": null
}

This is the view in Discord (The text is the 'optional' comment):

This is my Code:
var Discord = require("discord.js");
var bot = new Discord.Client();

bot.login("*******************************************");

bot.on('ready', () => {
	console.log('I am ready!');
});

bot.on("message", msg => {
	var prefix = "!";
	var args = stringArgv.parseArgsStringToArgv(msg.content);
	var cmd = args[0] || "";
	args = args.filter(a => a !== cmd);

	if (msg.author.bot) return;
	if (!msg.content.startsWith(prefix)) return;

	if(cmd.startsWith(prefix + "att")){
		console.log(JSON.stringify(msg.attachments, null, 2));
	}
});

darkson95

https://discord.js.org/#!/docs/tag/master/class/Message?scrollto=embeds

The Property embeds is unfortunately an empty Array.

i believe this should answer your question

To expand on Gus's comment, message.attachments is a collection mapped by attachment ID. Because attachments can be circular, it can be hard to use JSON.stringify on them, I suggest grabbing the values you need from an attachment and JSON.stringifying them instead.


is there any way to skip the prompt for save?

After call
html2pdf().from(element).set({
margin: 0,
filename: 'report' + '.pdf',
...
}).save();
the file will be saved directly with name as report.pdf.
Is there any way to achieve that? thanks

Hi, you mean without asking where to save the file? I believe that would just depend on browser settings, not something we could control.

Yes.
Actually when we download something from a webpage it will  always be downloaded directly in download directory, right?
whether save() can be implemented like that? directly saved to download directory without requiring user to do a click.

As mentioned, this is browser-dependant and not in our control. For instance, if you try .save() with Chrome and disable the setting "Ask where to save each file before downloading", it will automatically download with no prompt. There is no way to bypass this (and there shouldn't be) - users have control over how their browsers interact with files.
Closing.

yes you are right. thank you very much for your help.


ts-standard parser error: the file does not match your project config

What version of this package are you using?
1.4.0
What operating system, Node.js, and npm version?
Windows 10, Node: v12.8.1, npm: 6.14.5
What happened?
I currently have a yarn workspace monorepo setup. When running ts-standard I get the following error for any file I open in my react project
Parsing error: "parserOptions.project" has been set for @typescript-eslint/parser.
The file does not match your project config: ..\..\..\..\..\..\c%3A\Users\[redacted]\packages\web\src\App.tsx.
The file must be included in at least one of the projects provided.

ts-standard seems to work fine on its own via CLI.
What did you expect to happen?
Are you willing to submit a pull request to fix this bug?
If I an understand what is causing it I'd be happy to help fix it.

This seems to be fixed in the release/v1.5.0 branch. I manual packaged it and installed it from that branch and things are working smoothly so far.

I'm having the same issue, I can confirm that using the extension packaged from https://github.com/standard/vscode-standardjs/tree/release/v1.5.0 fix it.
Thanks @bmight


[Feature request] Update to 1.17.4

Idea
Looks like Makemkv updated on 5/29 to 1.17.4.
Could this be updated?

Super new to docker so forgive me. I pulled the latest from image dockerhub and it looks to be the 1.17.3 version. Not sure how to pull the new version if not from dockerhub. I see in this repo you already updated to .4 just not sure how to get it.

Super new to docker so forgive me. I pulled the latest image from dockerhub and it looks to be the 1.17.3 version. Not sure how to pull the new version if not from dockerhub. I see in this repo you already updated to .4 just not sure how to get it.

I'm having the same issue.  Just pulled from repo, still have 1.17.3.

A new release of the image has not been done yet (see https://github.com/jlesage/docker-makemkv/releases).
This should be done today.

New release now available.


Persona interviews

generate scripts
Interview
extract insights

AI assisted user interview script for Deepracer organizer

Interview Script for Identifying Pain Points and Gain Points in Managing Cloud Infrastructure for AWS DeepRacer Competition
Introduction
Interviewer: Good [morning/afternoon], and thank you for taking the time to speak with us today. My name is [Your Name], and I am [Your Position]. We are conducting interviews to understand the challenges and opportunities in managing the operations and finances of cloud infrastructure for competitions like AWS DeepRacer. Your insights will help us develop an automated solution to better support your needs. Could you please introduce yourself and provide a brief overview of your role?
Interviewee: [Interviewee introduces themselves and their role]
General Questions

Interviewer: Can you describe your primary responsibilities in managing the cloud infrastructure for the AWS DeepRacer competition?
Interviewer: How long have you been involved in managing cloud infrastructure for such competitions?

Pain Points

Interviewer: What are the most significant challenges you face in managing the cloud infrastructure for the competition?

Follow-up: Can you provide specific examples of these challenges?

Interviewer: How do you currently monitor and manage cloud costs for the competition?

Follow-up: What difficulties do you encounter in this process?

Interviewer: Are there any specific tasks that are particularly time-consuming or prone to errors?

Follow-up: How do these tasks impact your overall workflow?

Interviewer: How do you handle unexpected issues or outages in the cloud infrastructure during the competition?

Follow-up: What are the common issues you face, and how do you resolve them?

Gain Points

Interviewer: What tools or processes have you found to be most effective in managing the cloud infrastructure and costs?

Follow-up: Why do you find these tools or processes effective?

Interviewer: Are there any aspects of your current workflow that you believe are particularly efficient or beneficial?

Follow-up: How do these aspects contribute to the success of the competition?

Interviewer: Can you describe any recent improvements or innovations in your cloud management practices that have had a positive impact?

Follow-up: What was the impact of these improvements?

Automation and Solutions

Interviewer: If you could automate any part of your current workflow, what would it be and why?
Interviewer: What features or capabilities would you like to see in an automated solution to help manage cloud infrastructure and costs more effectively?
Interviewer: How do you envision an ideal automated solution supporting your role and responsibilities?

Closing

Interviewer: Is there anything else you would like to share about your experience in managing cloud infrastructure for the AWS DeepRacer competition?
Interviewer: Thank you for your time and valuable insights. Your feedback is crucial in helping us develop a solution that meets your needs. If we have any follow-up questions, may we contact you?

Interviewee: [Interviewee responds]
Interviewer: Thank you once again, and have a great day!

This script aims to gather detailed information about the pain points and gain points experienced by administrators, which will be instrumental in designing an automated solution tailored to their needs.


Error: Invalid IV length on decrypt

following the guide I:

created a KMS key
used aws-cli to create a data-key
encrypted secrets cryptex encrypt secret
added secret blob to cryptex.json
attempt to decrypt secret blob

$ cryptex decrypt mysecret
Uh-oh, we have a problem:
Error: Invalid IV length
    at Error (native)
    at new Decipheriv (crypto.js:242:16)
    at Object.Decipheriv (crypto.js:239:12)
    at AES256.decrypt (/Users/scott/code/crypt-test/node_modules/cryptex/src/lib/SymmetricAlgo.js:22:29)
    at Promise.resolve.then.then (/Users/scott/code/crypt-test/node_modules/cryptex/src/index.js:51:30)
    at process._tickDomainCallback (internal/process/next_tick.js:135:7)

Same error on Node 8 and Node 6.
Maybe my KMS key isn't set up right, not sure, any help would be appreciated though.

Hey Scott! I’m so sorry, that’s a typo in the guide you read that I can no longer fix. Replace decrypt in that final command with getSecret and you’re all set. The decrypt command is actually for decrypting data passed in on the command line, rather than decrypting stored secrets.

That works! Thanks a lot. I guess I should have looked more carefully at the README. Thanks again for the quick response!

Not your fault at all! Enjoy the lib :)


Google data studio topic

The advanced workshop needs some finessing still, but all the google data studio materials, as presented by Darshik, are now here

I got bored with the IMDB data, so changed it to the SA Open Cities data. There is a report containing visualisation of the data, so if you think this will be a problem, we can either download some of the datasets or link to city of cape town open data (https://web1.capetown.gov.za/web1/OpenDataPortal/AllDatasets). It's just a nice website with open data to know about.


always destroy socket provider on disconnect, and fix type error

A new version of the TypeScript compiler highlighted this type error:
https://github.com/uProxy/uproxy-networking/blob/dev/src/net/tcp.ts#L61
Fixing it, I realised two things:

close throws an error when the socket is already closed
we have only been destroying the provider when we explicitly call close

I believe it's more appropriate to destroy the provider once we receive an onDisconnect event. This may fix:
https://github.com/uProxy/uproxy/issues/1303
@willscott could you also take a tiny quick look at this?

Thanks for looking at this!

PTAL!

src/net/tcp.ts, line 67 [r4] (raw file):
How about something like
before();
return f().then((val:T) => {
  after()
  return val;
}, (e:Error) => {
  after();
  throw e;
};

That would avoid the ungainly new Promise construction.

src/net/tcp.ts, line 131 [r4] (raw file):
I think this needs a comment to explain that its purpose is to drive the count to -1, which is a special condition indicating termination.

src/net/tcp.ts, line 399 [r4] (raw file):
Binding this to undefined seems pretty scary to me.

src/net/tcp.ts, line 428 [r4] (raw file):
Same here

src/net/tcp.ts, line 453 [r4] (raw file):
Copy and paste makes me sad.  What if the wrapper were its own class, and then Connection and Server each had one?

Comments from the review on Reviewable.io

Thanks @bemasc, PTAL!

src/integration-tests/tcp/freedom-module.ts, line 62 [r1] (raw file):
Done.

src/net/tcp.ts, line 51 [r1] (raw file):
Done.

src/net/tcp.ts, line 67 [r4] (raw file):
Right. I did this so any error raised by after()  would propagate asynchronously but just didn't realised what I had didn't work...changed now. before() will just raise eceptions synchronously.

src/net/tcp.ts, line 399 [r4] (raw file):
Ah, yes. Probably only works due to how the socket provider is written internally. Fixed, thanks!

src/net/tcp.ts, line 453 [r4] (raw file):
Right. I've extracted this to a separate file, with a test. It's a little tricky because only standalone functions can be parameterised...but I think this is about as clean as I can get it.

Comments from the review on Reviewable.io

src/net/counter.ts, line 10 [r6] (raw file):
I think this would be more attractive as a method of Counter.  Then .before and .after could be private.

src/net/counter.ts, line 39 [r6] (raw file):
I think this pattern needs a comment to explain how it relates to the this.counter_ < 0 check.  Alternatively, maybe a comment on counter_ to explain its semantics.

src/net/tcp.ts, line 416 [r6] (raw file):
Do you want to wait for this.counter_.onceDestroyed here?

Comments from the review on Reviewable.io

src/net/tcp.ts, line 416 [r6] (raw file):
YES. Excellent point. Done, for server and client sockets. I like it much better now. Thanks.

src/net/counter.ts, line 10 [r6] (raw file):
So, I agree -- however, I wanted type safety on the return type and TypeScript only supports this on standalone functions.
I don't think there's any way to move this method to Counter without making the return type any, which I think would be worse.

src/net/counter.ts, line 39 [r6] (raw file):
Done.

Comments from the review on Reviewable.io

src/net/counter.ts, line 10 [r6] (raw file):
Just make Counter<T> parametric?

Comments from the review on Reviewable.io

src/net/counter.ts, line 10 [r6] (raw file):
Oh, I see; it's used to wrap several different types.  Hmm...

Comments from the review on Reviewable.io

src/net/counter.ts, line 10 [r6] (raw file):
Right. It works great but only on a standalone function.

Comments from the review on Reviewable.io

src/net/counter.ts, line 10 [r6] (raw file):
I just tried this locally.  I think you can do
public wrap = <T>(f:() => Promise<T>) : Promise<T> => {

Comments from the review on Reviewable.io

src/net/counter.ts, line 10 [r6] (raw file):
Oh! You know, I tried something very similar and couldn't get the magic incantation right. Many thanks, fixed.

Comments from the review on Reviewable.io

:+1:

Comments from the review on Reviewable.io


Use org gpg keys for publishing

We added a gpg key and passphrase to the org so we don't have to create new key pairs for each provider repo we want to publish to the registry. PR updates the current secrets to point to the org secrest

Glad that you guys are still maintaining this 💯


i18n(de): Added german translation for concepts/why-astro

What kind of changes does this PR include?

Translated content

Description
Adding a german translation for concepts/why-astro

LGTM!


Removing/resetting routes

When using Swifter for unit tests, it would be nice to be able to reset all routes in tearDown(). Occasionally a request will fail with error The network connection was lost, likely due to stopping/starting several servers very quickly that all try to bind to 8080. Being able to keep an instance running but resetting the routes should make this error less likely.

Or you can add support for multiple ports on server start, for example, I use this in my iOS UI-tests for parallel runs:
class HTTPDynamicStubs {

    var server = HttpServer()
    var port: UInt16 = 8080

    ...

    func setUp() {
        setupInitialStubs()

        while port != 8090 {
            do {
                try server.start(port)
                print("Server started on port \(port)")
            } catch {
                port += 1
                print("Failure, trying new port \(port)")
                continue
            }
            break
        }
    }

What about using port: 0 (gives an auto-assigned port), then get the port try server.port()


I need quality legit config

Hello to everyone. Do you have a quality legit config that you use?

It's a personal preference, I'd recommend making one yourself.

yes ı know but I want to use different people's settings

@furkansrby Here is my config. If its good or decent enough please let me know. I'll try to improve mine. https://anonfile.com/33m8d2I5n9/Legit_V1-Brian_Vu

depends on rank really.
silver to gnm: use about 5 fov with 10-15 smoothing, about 60 x and 70 y rcs, aimlock, closest bone.
mg-dmg: use 1-2 fov with 3-4 smothing, about 80-90 x and y rcs, aimlock, target neck, chest, or head (pref chest).
le-ge: use 1-2 fov with 2 smoothing, 90 x and y rcs, aimlock, target neck or head.
reasoning:
on lower ranks people tend to just rush out on to sites or basically anywhere together, you don't want your low smoothing and high fov to just lock on to peoples bodies.
on higher ranks people tend to make strategic moves, such as sending a single person a, or baiting teammates on to sites which makes a small fov and low smoothing perfect because it will just appear as a small flick, which for a higher rank player is not hard to do.

@linkupbiz True.

what does aim lock actually do?

My legit config:

RCS x : 85 / RCS y : 0.
Visible charm only.
ESP Weapons < 25 meters away.
Bomb timer.

i actually dont use aimbot on my legit ._.

https://www.mediafire.com/file/dx3ytmp0s2avqcq/Prime/file

thanks for the cool cfgs guys :)

Somebody have a "rage" config?

https://drive.google.com/open?id=1FWB7zA8R-eMW2Z_AX6j10Je54OqgPiUb
Maybe not the best legit config, no aimbot, but hey, at least it's there. Been using this config for quite a while. I'd say 2 months.


Align README with the one in the template repo

Applicable spec: n/a
Overview
Align README with the one in the template repo
Rationale
to ensure a consistent user experience.
Checklist

[x] The charm style guide was applied
[x] The contributing guide was applied
[x] The changes are compliant with ISD054 - Managing Charm Complexity
[x] The documentation is generated using src-docs
[x] The documentation for charmhub is updated
[x] The PR is tagged with appropriate label (urgent, trivial, complex)

If you would like to temporarily override the statsd-exporter version inside the rock, you can add this section to the end of rockcraft.yaml. I will create a pull request in Rockcraft shortly.
    flask-framework/statsd-exporter:
      source-tag: v0.28.0

https://github.com/canonical/rockcraft/pull/770

Thanks @weiiwang01 , I have expected it but still tried it, unfortunately the latest version does not fix the issue as the go library is still outdated.


WebClient should auto-populate content-length header if not explicitly set

As seen in the following code, I have to explicitly set the content-length header when sending requests with WebClient, or else I get 411 Content-Length Required responses. Luckily this particular request doesn't have a body, but if it did, it would be difficult to determine the actual payload length using this builder-style pattern to construct requests.
    private static void getToken(String code) {
        logger.info("Getting new Google Drive access token");
        JacksonSupportShim.registerDeserializer(OAuthToken.class);
        WebClientRequestBuilder requestBuilder = webClient.post()
                .uri(GoogleCredential.getTokenUri())
                .contentType(MediaType.APPLICATION_FORM_URLENCODED)
                .queryParam("client_id", GoogleCredential.getClientId())
                .queryParam("client_secret", GoogleCredential.getClientSecret())
                .queryParam("code", code)
                .queryParam("grant_type", "authorization_code")
                .queryParam("redirect_uri", GoogleOAuthService.getRedirectUri());
        requestBuilder
                .headers().contentLength(0L);
        requestBuilder
                .readerContext().registerReader(JacksonSupportShim.get().readerInstance());
        try {
            token = requestBuilder.request(OAuthToken.class).get();
            logger.info("Google Drive access token retrieved");
        } catch (InterruptedException | ExecutionException e) {
            fatal(logger, e);
        }
    }

Content-Lenght should be set by webclient only in the following cases:

there is no entity and the HTTP method may have an entity - set to 0
there is a small entity, that fits into one "chunk" (our representation of byte buffer) - set to the length of that buffer
otherwise we always use chunked encoding, which (by HTTP specification) does not use a content length (as it may not be known)

The last case is quite important - when we send bytes using publishers, we do not know the entity size in advance, and we will not buffer it (as this is a dangerous operation - what if the entity is 1 GB big?). So we use chunked encoding to send the entity as it comes. In these cases the content length cannot be computed - and it should either be provided by the user (such as when sending a file - you can get the file length, configure Content-Length and then send it).
There is an issue right now that Content-Length is removed when using chunked encoding, which should not happen (it may still be ignored by the server, but we should send it if configured).

The 411 Content-Length Required - that is also a question of the remote server and application implemented on it. The specification requires Content-Length when not using chunked encoding only (which we actually use in most cases).


docs(sql): remove whitespace from connection string

@LayZeeDK : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@LayZeeDK,
Thank you for this contribution to our docs.
PR approved.

#sign-off


discontinue the package

update the package to clarify that it has been discontinued

😝
On Tue, Dec 8, 2020 at 8:06 AM Devon Carew<EMAIL_ADDRESS>wrote:

@devoncarew commented on this pull request.
In .gitignore
https://github.com/dart-lang/tflite_native/pull/50#discussion_r538541347
:

@@ -1,3 +1,4 @@

.dart_tool
+.idea/
I'm landing this as is Kevin, but am enclosing this gift to make up for
it: 🎁
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/tflite_native/pull/50#discussion_r538541347,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAEFCXGMCVD33ZLSTTZELLSTZFGVANCNFSM4URKHAKA
.


refactor(CategoryTheory/Idempotents): replacing equalities of functors by isomorphisms

In the context of idempotent completion of categories and the Dold-Kan equivalence, constructing some isomorphisms was very slow in Lean 3 but has been much faster in Lean 4, while using equalities of functors in Lean 3 was fast, but in Lean 4 it became very slow. In this PR, we switch to using mostly isomorphisms of functors: this also became necessary in order to make the refactor #8531 possible.

!bench

!bench

I think your motivation for this PR should be recorded in the commit history. Do you agree? If so, please move it above the --- fold.

Thanks for the suggestion!
bors merge


Add support for customizable normalization logic

FEATURE REQUEST

Is there an open issue addressing this request? If it does, please add a "+1" reaction to the
existing issue, otherwise proceed to step 2.
No.

Describe the feature you are requesting, as well as the possible use case(s) for it.
One should be able to provide a transformation script to the normalizer. Normalizer will load this script in runtime and apply it to the received messages. This script should return a transformed message and a list of subjects to which these should be published. In combination with #802 this will provide full support for custom message formats.

Indicate the importance of this feature to you (must-have, should-have, nice-to-have).
If we're going to introduce #802, this is a must-have.

For scripting, we can use tengo.

This is good idea, but in order to avoid code complication, Normalizer will stay as is - as Writers understand only SenML.
On the other hand this idea can be followed in a new microservice that would have a role of some Rules Engine.


Update build tool to 1.5.7

The build tool here: http://popcorn-js.chrisdecairos.ca/
Looks to be using 1.5.0, and we should update it to 1.5.7.

1.5.10 please?
Also on the website the complete build is v1.5.6.
http://popcornjs.org/download


Instruction for non hassio HA

Could you please share instruction how to instal in no hassio home assistant? I do have HA in docker on my synology. HASSIO is not an option that would work for me.

I would love this also. I have HA installed in docker on a rpi4

Me too!

I have got a synology ds1815+, too. I managed to use Home Assistant Supervised mit hacs. So ist no problem to use this add-on.
Home Assistant (official docker version) does not support other add-ons.

Use this tutorial to install Home Assistant Supervisor in Docker/Container Manager/Portainer
https://mariushosting.com/how-to-install-home-assistant-supervisor-on-your-synology-nas/


Production hashed images not handled

Hi @Log1x,
Thanks for making this package - it's very helpful :)
As mentioned in #9, if Sage's yarn build:production has been run, the svg_image function doesn't find the hashed file name.
The problem is on this line:
https://github.com/Log1x/blade-svg-sage/blob/e764ca0a2733b5c3e437ee84dcac7bbc52409a77/src/helpers.php#L29
Specifically, the call to sage('assets')->get($icon), where $icon is just the icon name (eg. star). If you pass a simple string like 'star', Sage will never find any hashed assets, so it passes the original string back, which works in development (the filename hasn't changed) but breaks in production because the name has been hashed.
Assuming my SVG path is 'images/svg', when I call @icon('star'), the sage('assets')->get() function should be given the value images/svg/star.svg so it can correctly return the hashed filename (if one exists).
This is simple enough but due to the way the configuration is stored by the BladeSvg\SvgFactory class (it's private) and the way your package uses a filter, I couldn't see a very elegant way of fixing this without making too many changes to the packages.
Maybe you can suggest a better way but for now here's my ugly hack to get it to work properly:
function svg_image($icon, $class = '', $attrs = [])
{
    $extension = '.svg'; // Extension of SVG files
    $base_dist_path = get_dist_path(''); // Base path to "dist" folder
    $svg_path = apply_filters('bladesvg_image_path', get_dist_path('images/svg/icons')); // SVG path that has been set
    $relative_path = trailingslashit(str_replace($base_dist_path, '', $svg_path)); // The relative path inside the dist folder (needed for sage('assets')->get())
    $relative_icon_path = $relative_path . $icon . $extension;

    // Convert the final asset name back to original format (no extension or extra path added)
    // It's possible that we might use @icon('subdir/name') so we can't use the basename() function
    $icon = str_replace($extension, '', sage('assets')->get($relative_icon_path));
    $icon = str_replace($relative_path, '', $icon);

    return sage(\BladeSvg\SvgFactory::class)->svg($icon, $class, $attrs);
}

What do you think? I hope someone can find a more elegant solution but either way, it would be good to have a fix merged because the package is unusable in production without it.
@strarsis: this might interest you too.

I'm having this exact same issue.

@hybridvision: What about normalizing the path first (resolving all relative segments (./, ../)) using realpath.

@strarsis: normally there shouldn't be any relative segments like ../ in the path because you pass a path that is relative only to the bladesvg_image_path - e.g @svg('icons/myicon'). Using realpath gives the full absolute file system path but all we want is the path of the hashed file relative to bladesvg_image_path, without an extension.
Maybe you could refactor to use realpath but I don't see how it solves the specific problem I had...
Do you see an issue with the solution I proposed in my PR?

@hybridvision: When there are no other side effects then this is a nice fix.
Though I prefer existing path manipulation methods, Ideally a method can be found that just handles the redundant ./ correctly.

I completely agree about using the standard PHP path manipulation functions whenever possible.
The first thing I tried was incorporating pathinfo($icon_path_rel, PATHINFO_DIRNAME); but then I needed to concatenate that with the filename and add a slash between them... In many cases using something like @svg('mySVGfile') (no subdirectory reference), the returned dirname is simply ., so you'd end up with ./ at the beginning of the string. I could have added checks for this case but it would make the code way more convoluted...
It's a relatively small detail so I think my solution is robust and clean enough :)

@Log1x: Would it be possible to merge the PR from @hybridvision?


popup toolbar width CSS issue

Hi! We saw this on MapKnitter and found a fix:

This CSS seems to fix it:
.leaflet-toolbar-icon {
  box-sizing: initial;
}

See here:

https://github.com/publiclab/Leaflet.DistortableImage/commit/13a40ac6590ca80b6abc4a3aed960a0f17c445d4
I can open a PR in a moment.

https://github.com/Leaflet/Leaflet.toolbar/pull/53 - thanks again!!! 🎉

Fixed in #53 - thanks, Jeff!


Update stable version in repo from 2.0.2 to 2.1.0

Think about update stable version to 2.1.0
Version: stable=2.0.2 (241 days old) => latest=2.1.0 (30 days old)
Installs: stable=1474 (50.74%), latest=147 (5.06%), total=2905
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 2.1.0 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 2.1.0 but the current stable version is already 2.1.0.
So this issue should be closed.
@mcm1957 for evidence


Fixed typo in sql-database-connect-query-nodejs.md

Fixed small code typo in sql-database-connect-query-nodejs.md by adding 'var' to the request variable so that the code would run without any errors.

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


imp: linting

Just some basic linting. At some point adding (something like) Pre-Commit might be nice.

This I formatted with by using Conform in neovim (LazyVim) cf.
For coding on my phone I use Termux->ssh->tmux->(neo)vim. Unexpected Keyboard (on F-droid) is great for coding.

Does Conform just run pylint with no args? Also, I'm leaning more towards separating the RAG (and therefore python dependent) stuff out into a separate repo - I think that was a good idea. Doing it right will be somewhat complicated and this repo is already getting large. If you're mostly interested in collaborating on RAG things, we should hold off on PR's here since it's likely this is all going away.
I've always wanted to try out coding on the phone but the keyboard really put me off - do you use a more coding friendly keyboard?


Error Analysis UI & Template support for Object Detection

This PR establishes template support for EA on OD scenario.

Description

Checklist

[x] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #2203 (c756056) into main (34610c0) will increase coverage by 3.77%.
 Consider uploading reports for the commit c491625 to get more accurate results

@@            Coverage Diff             @@
##             main    #2203      +/-   ##
==========================================
+ Coverage   88.06%   91.84%   +3.77%
==========================================
  Files         129       35      -94
  Lines        7316      699    -6617
==========================================
- Hits         6443      642    -5801
+ Misses        873       57     -816

Flag
Coverage Δ

unittests
91.84% <ø> (+3.77%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 94 files with indirect coverage changes


rdar://problem/43884430 Allow C code to link NSCFConstantString

When C code links against Foundation, it can't link to the NSCFConstantString because the symbol isn't exposed by the library. Using this attribute makes it visible, but not usable from Swift.

@swift-ci please test


how do i use this program?

how do i go about 'running' this program.
sorry for being a complete noobie D:
(win 10)

Hey,
All you need a text document of words you want to check named "words" in the same directory as the application. Run the application and you're good.

How exactly would I 'run' the application? i've just got the source code ( i think ? )
license file
main.cpp
readme.md
etc etc
there's not an executable or anything like that? did i download the incorrect thing or is there something i completely missing.
thanks for your reply !

Whoops! Let me compile it for ya and send it over
Sent from my Samsung Galaxy smartphone.
-------- Original message --------From: 5peep<EMAIL_ADDRESS>Date: 5/26/18  19:52  (GMT-05:00) To: 18/Steam-ID-Checker<EMAIL_ADDRESS>Cc: Krystian S<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [18/Steam-ID-Checker] how do i use this program? (#1)
How exactly would I 'run' the application? i've just got the source code ( i think ? )
license file
main.cpp
readme.md
etc etc
there's not an executable or anything like that? did i download the incorrect thing or is there something i completely missing.
thanks for your reply !
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
{"@context":"http://schema.org","@type":"EmailMessage","potentialAction":{"@type":"ViewAction","target":"https://github.com/18/Steam-ID-Checker/issues/1#issuecomment-392295247","url":"https://github.com/18/Steam-ID-Checker/issues/1#issuecomment-392295247","name":"View Issue"},"description":"View this Issue on GitHub","publisher":{"@type":"Organization","name":"GitHub","url":"https://github.com"}}
{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/18/Steam-ID-Checker","title":"18/Steam-ID-Checker","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/18/Steam-ID-Checker"}},"updates":{"snippets":[{"icon":"PERSON","message":"@5peep in #1: How exactly would I 'run' the application? i've just got the source code ( i think ? ) \r\nlicense file\r\nmain.cpp\r\nreadme.md \r\netc etc\r\n\r\nthere's not an executable or anything like that? did i download the incorrect thing or is there something i completely missing. \r\n\r\nthanks for your reply !  "}],"action":{"name":"View Issue","url":"https://github.com/18/Steam-ID-Checker/issues/1#issuecomment-392295247"}}}
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "37567f93-e2a7-4e2a-ad37-a9160fc62647",
"title": "Re: [18/Steam-ID-Checker] how do i use this program?  (#1)",
"sections": [
{
"text": "",
"activityTitle": "5peep",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@5peep",
"facts": [
]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "IssueComment",\n"repositoryFullName": "18/Steam-ID-Checker",\n"issueId": 1,\n"IssueComment": "{{IssueComment.value}}"\n}"
}
]
},
{
"name": "Close issue",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "IssueClose",\n"repositoryFullName": "18/Steam-ID-Checker",\n"issueId": 1\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/18/Steam-ID-Checker/issues/1#issuecomment-392295247"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n"commandName": "MuteNotification",\n"threadId": 339488507\n}"
}
],
"themeColor": "26292E"
}

That would be great, thank you!

https://file.town/download/qspw3akr5tptmmcm6cu5315i5 Here is the exe, enjoy :)

Thanks, I've tested it and the output seems to be blank,
It saves a html file which links to the steam profile of the first word in the list?

Hey,
What list of words are you using? If the word is taken, it isn't written to
the output.
On Sat, May 26, 2018 at 8:07 PM, 5peep<EMAIL_ADDRESS>wrote:

Thanks, I've tested it and the output seems to be blank,
It saves a html file which links to the steam profile of the first word in
the list?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/18/Steam-ID-Checker/issues/1#issuecomment-392295884,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AP2-NpIgkIqgo8H3V_3VdVnWM49PpKI4ks5t2e49gaJpZM4UPC77
.

https://file.town/download/qcxmenqjk1yjtodyypzzheoit Here is a version that will write to the file real time. Use this instead.

I tested this one out and it seems to be a bit weird. the words i used were;
midmorning
midflight
midfielders
dog
cat
sheep
fly
the first three not being taken currently whereas the last four are.
the output file simply contained
"ID: fly"
and saved a html file of the steam community profile of id/fly

Nah, its fine, I need to fix it anyways :D
On Sat, May 26, 2018 at 8:21 PM, 5peep<EMAIL_ADDRESS>wrote:

Thank you, sorry for being a pain in the ass!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/18/Steam-ID-Checker/issues/1#issuecomment-392296424,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AP2-NrfcgBXSgbf0PY4xVMN2j_dJL4q8ks5t2fGUgaJpZM4UPC77
.

https://file.town/download/srm3cordyvabihqib0kpptkkf Sorry it took so long, made a silly little mistake

You didn't take long at all haha,
It's working great. Thanks a bunch. :-)


View-level resolves

I don't think we ever explicitly re-integrated support for them, but I realize they're pretty important re: the goal of efficient laziness. How'd you wanna tackle it? Just instantiate a new Resolvable with the ResolveContext of the enclosing state? /cc @christopherthielen

That's the general idea, yes.
I've never liked view-level resolves..  They seem to go against thinking in states.  How do you use them?

So far I gen haven't, but the idea is if you have a state with multiple views, one doesn't wait for the other's resolves in order to start rendering, which is a good idea, but probably not something that could have been taken advantage of before anyway.


Replace gst caps with mime type. Add new create media source API

Summary: Replace gst caps with mime type. Add new API to create audio media source with codec specific configuration
Type: Feature
Test Plan: Unit Tests, Manual Tests
Dependencies and Impacts: Small

 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.


Angular 5 support?

Could the @angular/core and zone.js peer dependencies be upgraded to versions used in angular 5?

I'm not sure if this project is still maintained. As a workaround, I've published another npm repository here. If the owner comes back, they are more than welcomed to merge this, but until then I think you can use that repo. I'll try upgrading to Angular 5.0.0 today or sometime soon.
You can add it like this in package.json:
"ng2-pop-over": "git+https://github.com/SpamExperts/ng2-pop-over.git#v1.0.1"
See here for more details.


Handling of "0000-00-00" and null for dates

See also https://www.php.net/manual/en/datetimeimmutable.createfromformat.php using ! or |  for dates only?


With the necessary systemd bump in place, we can undo the revert.
Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

This currently can't be tested due to https://github.com/NixOS/nixpkgs/issues/95163.

Rebased on latest master (now containing the fix for #95163).
It seems the bump to systemd 245.7 containing the necessary fixes for linux 5.8 wasn't part of the previous staging next round, so we'd need to wait for the next round (which hasn't opened yet) to have landed in master.
Then, this can be rebased to latest master, and we can run nixosTests.latestKernel.login.

What's the latest on this?

@colemickens It's dependent on systemd 245.7 which as flokli said missed the last round of staging. See https://github.com/NixOS/nixpkgs/pulls?q=is%3Apr+is%3Aopen+staging+next to check the status of staging. As you can see in the commits in #95492, the latest round of staging, systemd 245.7 is pending merge. Once that PR is merged, this PR can be merged, and then it will get to nixos-unstable after the tests pass. So it could be a while...
@flokli also, it appears @vcunat recently committed the contents of this PR to the staging next PR. So can this be closed?

Oh, it didn't occur to me there could be a PR for this.


[Integration][ADO] Allow project specific webhook creation for project admins

Description
The genesis of this change is documented in this slack thread
What

Users encounter a 400 error when attempting to create subscriptions (webhooks) due to insufficient permissions to edit the subscription.
Modify the webhook creation process to include the projectId in the payload.

Why

The current setup assumes an organisation token when creating webhooks. If a project-scoped token is supplied, the craetion fails

How

When the user holds a project administrator role, and the PAT is scoped only to that specific project, it is necessary to explicitly define the projectId when creating a webhook.

{
  "publisherId": "tfs",
  "eventType": "git.pullrequest.updated",
  "consumerId": "webHooks",
  "consumerActionId": "httpRequest",
  "consumerInputs": {
    "url": "http://<IP_ADDRESS>/ado-port/integration/webhook"
  },
  "publisherInputs": {
    "projectId": "4901a905-d07b-4027-b7cc-577d2b6a0e31"
  }
}

Type of change
Please leave one option from the following and delete the rest:

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] New Integration (non-breaking change which adds a new integration)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Non-breaking change (fix of existing functionality that will not change current behavior)
[ ] Documentation (added/updated documentation)

Screenshots
Include screenshots from your environment showing how the resources of the integration will look.
API Documentation
Provide links to the API documentation used for this integration.

will open a PR for the docs change


Add ga4gh-common to constraints.txt

Ah, so this is a problem worth noting... if a downstream repo (like the server, here) tries to do development with some commits of upstream repos corresponding to releases (i.e. that have their dependencies pinned) then this problem -- one package looking for an exact version of a ga4gh library that we only have a dev version of and therefore failing -- can result.  Solution: remove all pins for ga4gh packages in upstream ga4gh repos and re-download packages.

This now depends on fixing https://github.com/ga4gh/schemas/issues/797


mean after apply_along_axis dimension issue

What happened:
Performed operation similar to below and an exception was raised just like below.
What you expected to happen:
No exception
Minimal Complete Verifiable Example:
import dask.array as da
import numpy as np

t = da.from_array([[0]]) # make 2d array
s = da.apply_along_axis(np.sum, 1, t, shape=(1,)) # use apply_along_axis with shape(1,)
#s = da.apply_along_axis(np.sum, 1, t) # we are ok if we do not specify a shape
print(s.compute()) # you can do this one in your head: [0]
print(da.mean(s.ravel()).compute()) # ravel first and we are ok. returns 0.0
assert da.mean(s).compute() == 0.0 # Raise IndexError: tuple index out of range, code seems confused about how many dimensions there are

Anything else we need to know?:
Environment:

Dask version: 2.19.0
Python version: 3.6.10
Operating System: Windows/RHEL 7
Install method (conda, pip, source): conda

The problem seems to be that I got confused about how shapes work. In the example above the correct shape is () and  not (1,).


Add pre-commit configuration, upgrade to Python 3.9+ syntax, remove trailing whitespaces

This commit adds a pre-commit configuration, which allows running a few tools automatically both locally and in CI.
Currently:

It adds pyupgrade
It check for trailing whitespaces and if toml and yaml syntax is correct
Updates itself monthly (using pre-commit.ci)

For automatically running pre-commit in CI, I would recommend enabling it on https://pre-commit.ci/.

Thanks for merging!
Feel free to use Squash and merge merge method for PRs with a single commit, or in this case, the Rebase and merge method for PRs with multiple commits. That results in a cleaner commit history. See the GitHub docs for more details.

For automatically running pre-commit in CI, I would recommend enabling it on https://pre-commit.ci/.

Please don’t forget to consider this :)

@toruseo have you thought about enabling pre-commit.ci?

I will look at it. Please give me some time as it may change my workflow a lot.

Thanks! What do you think would change? With my experience with pre-commit, it’s typically enable-and-forget. It’s made to make your life easier :)

Sorry for the delay in addressing this matter. I haven't had the time to learn how to use pre-commit (at least I understand that while it makes code management easier, it does not change the simulation functionality). Given that situation, keeping a pre-commit file in the repo could be confusing, so I've decided to remove it for the time being. I have added the file to .gitignore, so I hope it will not change your workflow.
I really appreciate your contributions and suggestions. Please feel free to propose new ideas!