PostConfirmation failed with RangeError: Maximum call stack size exceeded

Describe the bug
After update the PostComirmation lambda function by adding permission to existing DynamoDB, I got "RangeError: Maximum call stack size exceeded".
Amplify CLI Version
4.18.1
To Reproduce

Create PostComfirmation lambda function for adding user to a user group:

> amplify update auth
…
Do you want to enable any of the following capabilities? Add User to Group
…
? Do you want to configure Lambda Triggers for Cognito? Yes
? Which triggers do you want to enable for Cognito   Post Confirmation
? What functionality do you want to use for Post Confirmation Add User To Group

Push the update to cloud and make sure it works
> amplify push
Add permission to PostCormfirmation lambda function for access an existing DynamoDB:

>amplify update function
? Please select the Lambda Function you would want to update xxxxxxxPostConfirmation
? Do you want to update permissions granted to this Lambda function to perform on other resources in your project? Yes
? Select the category storage
? Storage has 5 resources in this project. Select the one you would like your Lambda to access <target DynamoDB name>
? Select the operations you want to permit for <targetDynamoDB name> read, update
? Do you want to invoke this function on a recurring schedule? No
? Do you want to edit the local lambda function now? No
Successfully updated resource

Push the update to cloud
> amplify push
From user sign-up GUI, create a new account.
After confirmation I got "RangeError: Maximum call stack size exceeded"
In CloudWatch logging:

2020-04-24T16:35:12.511Z 109c27df-a040-4856-8e55-9d859d556541 ERROR Invoke Error
{
    "errorType": "RangeError",
    "errorMessage": "Maximum call stack size exceeded",
    "stack": [
        "RangeError: Maximum call stack size exceeded",
        "    at Object.hasOwnProperty (<anonymous>)",
        "    at Function.NativeModule.exists (internal/bootstrap/loaders.js:199:33)",
        "    at Function.NativeModule.nonInternalExists (internal/bootstrap/loaders.js:217:27)",
        "    at Function.Module._resolveLookupPaths (internal/modules/cjs/loader.js:456:20)",
        "    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:629:20)",
        "    at Function.Module._load (internal/modules/cjs/loader.js:562:25)",
        "    at Module.require (internal/modules/cjs/loader.js:692:17)",
        "    at require (internal/modules/cjs/helpers.js:25:18)",
        "    at exports.handler (/var/task/index.js:15:25)",
        "    at exports.handler (/var/task/index.js:16:5)"
    ]
}

Expected behavior
It should run the confirmation success without error because there is no any customized code was added to the PostConfirm lambda function.
Desktop (please complete the following information):

OS: Windows 10
Node Version: v12.13.1

@SwaySway
How is the work going?
Thank you.

@jingxizhang was the lambda customized? Would it be possible to share the lambda function code?

Hi Yuth,
The lambda function is auto generated by Amplify CLI. I did not modify it.

@jingxizhang this is a bug in Amplify CLIs function update flow. When the update flow overwrites <proj-root>/amplify/backend/function/<fn-name>/parameters.json and <proj-root>/amplify/backend/function/<fn-name>/function-parameters.json. Till we fix this you could work around this by adding the following to
--- a/amplify/backend/function/fn-name/function-parameters.json
+++ b/amplify/backend/function/fn-name/function-parameters.json
@@ -1,4 +1,8 @@
 {
+    "trigger": true,
+    "modules": [
+        "add-to-group"
+    ],
     "parentResource": "postconf4167fde9",
     "functionName": "postconf4167fde9PostConfirmation",
     "resourceName": "postconf4167fde9PostConfirmation",
diff --git a/amplify/backend/function/fn-name/parameters.json b/amplify/backend/function/fn-name/parameters.json
index 13624db..3a915e2 100644
--- a/amplify/backend/function/postconf4167fde9PostConfirmation/parameters.json
+++ b/amplify/backend/function/postconf4167fde9PostConfirmation/parameters.json
@@ -1,3 +1,4 @@
 {
+    "modules": "add-to-group",
     "resourceName": "postconf4167fde9PostConfirmation"
 }


—
Reply to this email directly or view it on GitHub
https://github.com/Bimde/Blackjack-Server/pull/27#event-485200889.


Update stix2 Package Structure

Closes #91

Codecov Report

Merging #93 into master will decrease coverage by <.01%.
The diff coverage is 98.07%.

@@            Coverage Diff             @@
##           master      #93      +/-   ##
==========================================
- Coverage   96.55%   96.54%   -0.01%
==========================================
  Files          53       54       +1
  Lines        4875     4894      +19
==========================================
+ Hits         4707     4725      +18
- Misses        168      169       +1

Impacted Files
Coverage Δ

stix2/test/test_properties.py
100% <100%> (ø)
:arrow_up:

stix2/v20/common.py
98.57% <100%> (ø)

stix2/v20/__init__.py
100% <100%> (ø)

stix2/v20/sro.py
100% <100%> (ø)

stix2/v20/observables.py
99.31% <100%> (ø)

stix2/v20/sdo.py
98.75% <100%> (ø)

stix2/__init__.py
100% <100%> (ø)
:arrow_up:

stix2/test/test_bundle.py
100% <100%> (ø)
:arrow_up:

stix2/core.py
98.48% <95.45%> (-1.52%)
:arrow_down:

Continue to review full report at Codecov.

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

I am going to write something for documentation. We should make it clear that using just doing import stix2 will change your behavior with every major release.
I could also modify the settings in the packaging to remove the tests from the packages distributed officially through PyPI.

Also, using python setup.py sdist' I managed to remove the testdir from the package, but I cannot get it to work with wheelpython setup.py bdist_wheel`.... I will push what I have.

The latest commits should address all of the aforementioned concerns.


CB-15579 - Add support for FreeIPA upscale/downscale

CB-15579 - Add support for FreeIPA upscale/downscale
This commit contains
- API layer
- Controller
- validations for
- desired number of instances present
- instances being not available
- correct initial instance number
- stack status
- configurable permitted upgrade paths

do you plan to add e2e/integration tests for this?

do you plan to add e2e/integration tests for this?

@horadla23
Yes, this will be it: https://github.com/hortonworks/cloudbreak/pull/12093/files @gergopapi2 is working on it now.


⚠ Bofrost.lu has degraded performance

In cbc322e, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1978 ms

Resolved: Bofrost.lu performance has improved in 24ef916 after 6 minutes.


Access to graphs and nodes from the outside

@vismate To be able to draw a graph and its nodes on screen with the correct information and to later be able to manipulate them, there needs to be a way to access a graph's nodes, which are, at the moment, private. Please implement getters and setters in the Port, Node and Graph classes.

Okay, I'll go and implement getters soon. It'll be done by around 3, if you want to work on it today.

Okay, thanks for the heads-up, I will.

looks good, thank you.

Error: Graph's "nodes" field not initialized in constructor. Please fix.

Also, could you please do add/remove element operations for array fields instead of just a setter?

As we just merged the branch, we'll have to do it later. Is it urgent or can it wait?

Yes, it's not that important right now. In fact I'll close this issue and open another one for that.


Add EuiHealth component to DetectorDetails

Signed-off-by: Sinisa Andric<EMAIL_ADDRESS>Description
Add EuiHealth instead of colored label for Detector status. EuiHealth is a consistent means of reflecting the entity state. It should replace custom colored label.
Issues Resolved
https://github.com/opensearch-project/security-analytics-dashboards-plugin/issues/143
Check List

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Hi Sinisa, if you pull main and update your PRs, the cypress tests will pass now.


Ubuntu 16+ and Node.js 8+

Can you install and use this out of date software on latest version of Ubuntu and Node.js? Seems to be dependencies missing and broken.

+1


IGAPP-509: Tooltips are opened on click

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.
Opening tooltips on click seems fine for me, so I just fixed the weird clipping

I had to pass through direction through a lot of components... Changes which are relevant are only in Toolbar and CSS rules.

I think I'll try to find another solution using JS. Maybe I can figure out whether there is more free space on the left or the right in the DOM.

Can't we just access the state directly in Tooltip? E.g. with the useSelector hook? https://react-redux.js.org/api/hooks#useselector

That would violate our rules of components vs containers. This is indeed still a good rule. You would not expect to create a whole redux store to render a Tooltip.

replaced by https://github.com/Integreat/integreat-app/pull/256

@maxammann can we delete the branch?


Internet via proxy

I run DCOS on coreOS and i just try Beta-Kubernetes on DCOS 1.10.0
I use proxy to connect to Internet.
All the tasks are in a running state, but when i try to install a new pod i get the following error
createPodSandbox for pod "redis-master-1405623842-zt2hq_default(97f75286-972a-11e7-937b-e41f13303b18)" failed: rpc error: code = 2 desc = unable to pull sandbox image
"gcr.io/google_containers/pause-amd64:3.0": Error response from daemon: {"message":"Get https://gcr.io/v1/_ping: dial tcp <IP_ADDRESS>:443: i/o timeout"}
I tried to pull gcr.io/google_containers/pause-amd64:3.0 from command line on my coreos host and it works.
Can we use this framework with a proxy to access Internet ?

Hi @ezreal92170, this sounds like a routing or firewall issue.  The proxy should not matter, as long as the kubelet container has a route to the Internet.  If you enter the container, can you ping the GCR IP?
nsenter -t <KUBELET PID> -m -n -p -w

Just to clarify,  in my environment i don't have direct access to Internet. I can only use http and https protocol to access the internet and only through a "corporate" proxy.
In this error message
Error response from daemon: {"message":"Get https://gcr.io/v1/_ping: dial tcp <IP_ADDRESS>:443: i/o timeout"}
we see that the daemon try direct connect to the Internet (ignoring my docker core os host configuration) because dns resolution happened. The direct connection will always fail in my environnement, i must to try to configure a http proxy for the daemon

Thanks for the details. We are currently in the process of "air gapping" the framework. It will be runnable behind the firewall soon.

Thanks, if you have some clue to modify inside container any configuration before a new released i can do some test.

This will be fixed with a new release to happen next week.

thx i will be happy to test that.

Can you try the new release and report back?

Closing. Please report back if you encounter any issues.


Write a PHP program to find largest and smallest numbers in an array

Description
Write a PHP program to find largest and smallest numbers in an array
Input  : [1, 2, 3, 4, 5]
Output : 5, 1

How to contribute

Save the solution in program/find_largest_and_smallest_numbers_in_an_array.php file
Add find_largest_and_smallest_numbers_in_an_array.php file in find-largest-and-smallest-numbers-in-an-array folder

!assign


Foresee a conversion for the old configuration file

When moving to https://github.com/cudeso/misp2sentinel/blob/upload_indicators_api/config.py.default
Take into account the older configuration settings, so that existing installs remain functional

Fixed with https://github.com/cudeso/misp2sentinel/commit/04d7a9b7d3add59946324746e2f1a443187dda29#diff-1c0e4a21a58677b4e286a6f25349efff5e7abd3eb74253e3d2dc1e8dc3ff4ebbR156


Remove duplication of apollo client files

The apollo client files have a large amount of duplicated code. Each one has subtle differences, generally in the cache policy. When making a change to one (for example to make the websocket connection work), these changes have to be made in many different places. We should have a tiny lib to unify apollo client setup and invert any dependencies, such as cache policies, that are app specific as well as providing config options for anything that requires it.

this has been done


Can't install extension ClearURLs

https://chrome.google.com/webstore/detail/clearurls/lckanjgmijmafbedllaakclkaicjfmnk
https://microsoftedge.microsoft.com/addons/detail/clearurls/mdkdmaickkfdekbjdoojfalpbkgaddei
[Variable $LIMIT$ used but not defined.].

https://gitlab.com/KevinRoebert/ClearUrls/-/issues/771
https://gitlab.com/KevinRoebert/ClearUrls

Thanks for reporting this issue @c2xusnpq6. The extensions engineering team has filed an internal bug and it trying to repro the error. I'll go ahead and close this issue now. Thanks.


Issues with macro hygiene at 0.13.0-alpha.3

I noticed a small issue: when using 0.13.0-alpha.3 code with the non-standard package name
metrics1 = { package = "metrics", version = "0.13.0-alpha" }

the counter! (and, presumably, other macros too) can't find metrics crate. Since those are reexported from the metrics crate, I suppose the idea was to bind to the exact (metrics) crate that does the reexport, regardless of the said crate's name.
This might result in a serious problem for binaries using libraries that have adopted the metrics crate if there are multiple versions in use.

This can't be fixed, it's a generic issue with proc_macros


ValueError: You selected an invalid strategy name: strategy='dp'

Traceback (most recent call last):
File "/content/ChordMixer/trainer.py", line 394, in cli_main
trainer = Trainer(
File "/usr/local/lib/python3.10/dist-packages/pytorch_lightning/utilities/argparse.py", line 69, in insert_env_defaults
return fn(self, **kwargs)
File "/usr/local/lib/python3.10/dist-packages/pytorch_lightning/trainer/trainer.py", line 393, in init
self._accelerator_connector = _AcceleratorConnector(
File "/usr/local/lib/python3.10/dist-packages/pytorch_lightning/trainer/connectors/accelerator_connector.py", line 140, in init
self._check_config_and_set_final_flags(
File "/usr/local/lib/python3.10/dist-packages/pytorch_lightning/trainer/connectors/accelerator_connector.py", line 209, in _check_config_and_set_final_flags
raise ValueError(
ValueError: You selected an invalid strategy name: strategy='dp'. It must be either a string or an instance of pytorch_lightning.strategies.Strategy. Example choices: auto, ddp, ddp_spawn, deepspeed, ... Find a complete list of options in our documentation at https://lightning.ai/

Hi, this happens when using the new version of pytroch_lightning. Please use pytorch-lightning==1.9.0

Thank you for your help.  I was able to get this up and running using a Google Colab notebook: https://colab.research.google.com/drive/1Kpa43G5UUoWnTakEwgr6D64I2HzozUKf?usp=sharing


Add checker for elixir's credo.

Why:

Credo is a nice static code analysis tool for elixir.

This change addresses the need by:

Add credo.vim to the elixir folder.

Side effects:

Credo can be used with Syntastic when enabled.

This is my first PR for syntastic or vimscript so I am very open to suggestions on how to improve this PR.

Actually, elixir is one of the checkers I'm considering for removal after the next release.  It produces unparseable error messages, and generally it has oddities that make it unsuitable for use with syntastic.  It's also going to break when syntastic switches to async checking.  Credo has better error messages, but it still adds its own oddities to those of Elixir.  Among other things it insists to run in UTF-8 locales, and it produces useless binary characters in its output.  Vim doesn't cope well with that.  If you insist to make a syntastic checker out of it you should consider making it an external checker.
Specific notes about your code:

it doesn't have a license
g:syntastic_elixir_credo_all is pointless, options can be added via g:syntastic_elixir_credo_args
running mix credo --version is probably a better validity check than mix credo help
Credo returns errors encoded as UTF-8 regardless of the current locale, so you need to convert them
your errorformat should match the letters [F], [D], etc. rather than the arrows
syntastic can't handle informational messages (%I etc.); this is a historic limitation
you should probably mark Credo messages as "style".

This is all great feedback! Thank you for your time @lcd047.
The limitations of the elixir checker is why I wrote the credo checker. To be very clear, are you saying that you do not want to merge such a PR even if I address your comments?
What do you mean by?

you should probably mark Credo messages as "style".

The limitations of the elixir checker is why I wrote the credo checker.

As far as I can tell they have pretty different scopes.  Can Credo deal gracefully with syntax errors?

To be very clear, are you saying that you do not want to merge such a PR even if I address your comments?

I'll merge it if you get it to an useable form and if you insist.  Just beware that it's going to break pretty soon, and that I'll also remove it if I decide to remove the elixir checker. shrug

What do you mean by?

you should probably mark Credo messages as "style".

Please see the FAQ.

SInce the future of the elixir checker is unsure, I've decided to give neomake a try. Thank you so much for all your hard work @lcd047. Syntastic has been a faithful friend for many years, but neomake and neovim have async checkers as well as makers for credo.


Update ctf_setup.sh | netcat not installed on linux instance

netcat not installed on Linux instance which prevents the [nc -l -p] command from executing

@rishabkumar7 this seems to address the issue discussed in the discord

https://github.com/learntocloud/ltc-linux-challenge/pull/7

we've updated the guide to instruct instead of applying the exact change, thank you

@rishabkumar7 yes I agree, I was thinking the same thing, and shared another approach with Gwyneth. It gives everyone the opportunity to also learn.
@madebygps great work Gwyneth

Cool, yeah I included it in the hint section, here is the PR - https://github.com/learntocloud/ltc-linux-challenge/pull/7

Cool, yeah I included it in the hint section, here is the PR - #7

This is wonderful, great work! @rishabkumar7

@rishabkumar7 check this out

@Ayolowo wait, what is that output of?

@rishabkumar7 the output was gotten from executing the port_8080_service.sh file within the Linux instance, I did that because I could not find any process running on port 8080. The output shows there's an issue with netcat - nc, not netstat.

@Ayolowo oh good catch, testing something, will merge this once tests complete.

@rishabkumar7 cool! great work.

thanks @Ayolowo for covering other clouds as well. LGTM!

you are most welcome @rishabkumar7


Keys table added

Notes for Reviewers

Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit fcf572b597c52164aba2c6f5a9db8ba0da15c89c at: https://66261741427b5f728530f75f--meshery-docs-preview.netlify.app

@iArchitSharma what other pages link to this page? Is it tagged with frontmatter such that it shows up in the main index page?


More examples

Client/Server examples are great. They demonstrate that the project works, I can wireshark the traffic, see the quinn API, etc. But they are not so great for newbies when I want as less context as possible to see how to use quinn.
I was playing around with the library and made a couple of examples to get a better understanding. Seems like a valuable asset to share :)

TBH I do have a concern that these three examples between each other duplicate quite a lot of code even though the variations among them are conceptually pretty minor. Maybe it would be nicer to demonstrate some other usage types within the same file?

TBH I do have a concern that these three examples between each other duplicate quite a lot of code even though the variations among them are conceptually pretty minor. Maybe it would be nicer to demonstrate some other usage types within the same file?

Makes sense. What about extracting common code to a separate module thouh? :thinking:

Separate module works for me, I don't know whether/how the Cargo examples structure facilitates that.

I just pushed the changes, see if that looks better :)
I like it this way: now there is a common.rs module that exposes commonly used building blocks and then each example composes them in a specific usable way.

Looks pretty good to me. @Ralith, do you want to take a look?

I addressed all your comments and pushed the changes.

Regarding cargo run --example insecure_connection, I'm actually not that sure anymore. I tried running without the feature flag and it works for me. To be honest I'm a bit lost because those functions should only be available when dangerous_configuration feature is on: https://github.com/ctz/rustls/blob/v/0.15.0/src/client/mod.rs#L221
@Ralith does cargo run --example insecure_connection work for you?

It looks like it coincidentally works when you invoke cargo from the workspace root, rather than the quinn subdir, because the interop workspace crate incidentally enables the required feature. I'm not a huge fan of relying on that, or of the resulting breakage of cargo test from inside the quinn subdir.

It looks like it coincidentally works when you invoke cargo from the workspace root, rather than the quinn subdir, because the interop workspace crate incidentally enables the required feature. I'm not a huge fan of relying on that, or of the resulting breakage of cargo test from inside the quinn subdir.

good catch, thanks, will fix this

Pushed the changes ;)

@Ralith I addressed all your comments ;)


Improved sample processing

Need to adapt the suite to be able to create multiple output files, udp/tcp connections, cea708 support, multiprogram support, ... This requires a big processor change.

Closed by c395e77c7e467b8bb15c99b5adfe8e5b61dbc5ee


feat: adds a default warning for 422 and 302 responses

Reason:

IBM Handbook says 400 response should be used instead of 422
IBM Handbook prefers 303 and 307 responses over 302

Changes:

added warning for 422 response codes
added warning for 302 response codes
added ibm_status_code_guidelines config option with default warning

Tests

added a test to ensure warning issued for 422 response
added a test to ensure warning issued for 302 response

:tada: This PR is included in version 0.26.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


update to sqlx 8.1 (waiting for sqlx PR)

Update the crate sqlx.

We should wait to this PR to be approved
https://github.com/launchbadge/sqlx/pull/3608

We should wait for this changes to be released #3608

@pxp9 looking great. can you please fix clippy?


cors headers

how to add cors headers similar to haproxy's rspadd .. etc
This would be much needed and common feature.
Need a quick fix to add (cors) headers to outgoing messages .. for now.
for ex: in haproxy's backend we add ,..
rspadd Access-Control-Allow-Origin:
rspadd Access-Control-Headers:\
Restrictions on frontends can be added later.

Hi @navinSing, this is certainly an interesting feature. As Traefik uses negroni to manage network middlewares, we could use https://github.com/rs/cors to make it work.
This is not in my top priority for now, would you be interested to make a PR on that?

Hi @emilevauge
can you please give an example to make it work .. rs/cors , or a possible way to intergrate the same for now, as this is blocker for me(any example for the workaround will be helpful); else would have to switch to some other frameworks.

sorry still not sure how to do it .. like rspadd Access-Control-Allow-Origin:\ etc ..
in both frontends and backends

Hi @navinSing, as I said you can't do it right now, some dev is needed to make it work.

any rough timelines..
don't have much hands on with golang ..

Somewhat related, being able to modify (add/edit/remove) headers in the responses returned from backends would be nice (used for other things too). There is already a broader issue on this open: https://github.com/containous/traefik/issues/30
Just trying to help with some issue-gardening. 🌳 🍃 🌷

@navinSing practically, it is up to your backend to add the CORS headers to the response and to tune the security more than to the reverse proxy.

@navinSing There are certainly situations where CORS from the backend is impractical.
We are in the process of dividing our backend from a monolith to many microservices. Together they expose one API. We do not want each of the services to decide the CORS header. We want one unified CORS header for the whole API – and that would fit nicely in our edge proxy.

We have >100 services under one API and would also prefer to manage headers on the loadbalancer itself.

there is a lot of security related headers, not just CORS : https://blog.appcanary.com/2017/http-security-headers.html

Any movement on this issue.?
I feel allot of people use traefik as an API gateway, having forward proxy features such as CORS is such a huge win in these cases. Draining the need for each service to manage it independently.

You can track those PR #1292 #1236

Closed by #1236.


Safekeeper sets Postgres timeline ID to 0

I've hit one problem that's potentially unsolvable without backwards incompatibilities: PostgreSQL relies on pg's TimeLineID always being >0, i.e. the initial timeline ID of a freshly initialized cluster is 1. The Safekeeper initializes this to 0. As a result, I cannot start nodes in replication mode, because the safekeeper has a lower timeline identifier than that what postgresql has.
I could hack around that in the WAL receiver infrastructure in PG, but I don't think that is smart. Instead, I could replace all occurances of TLI == 0 with TLI == 1, which I think is a more ergonomic option.
Originally posted by @MMeent in https://github.com/zenithdb/zenith/issues/769#issuecomment-957866095

@arssher, can you take a look at this please? Or reassign to someone.
@MMeent, can you elaborate, where does the safekeeper set the postgres timeline id? Is it present in the streaming replication messages, or in the WAL page headers, or where?

According to further research on how walkeeper gets the TLI, it receives it from the greeting message; which contains the value of ThisTimeLineID (see xlog.h), which is uninitialized for both --sync-walkeepers and when the walsender starts up.

Changing PG timeline is not really supported, and it is not even clear what meaning it would have, so let's just set it to 1:
https://github.com/zenithdb/zenith/pull/829
I haven't removed protocol message field, it is harmless and probably we'll get back to this.

According to further research on how walkeeper gets the TLI ... which is uninitialized

Interesting, I haven't realized ThisTimelineID lives uninitialized until first RecoveryInProgress call. walproposer side should also be cleaned up.


fix: deepcopy not working on older neovim versions

vim.deepcopy function doesn't have the second argument on older neovim versions (0.9.5) and thus the plugin is not working on them. The simplest fix is to simply remove the second argument, sacrificing a minimal performance gain for wider support. From the testing that I did on my side, the plugin works normally with this change

Thanks!


ts-jest version 26.4.2 broke coverage

🐛 Bug Report
Upgrading ts-jest to 26.4.2 broke the coverage files using default coverage provider(babel).
jest.config.js
module.exports = {
  roots: ['<rootDir>/src'],
  collectCoverageFrom: [
    '<rootDir>/src/**/*.ts',
    '!<rootDir>/src/**/protocols/**',
    '!<rootDir>/src/**/*.spec.ts',
  ],
  coverageDirectory: 'coverage',
  testEnvironment: 'node',
  transform: {
    '.+\\.ts$': 'ts-jest',
  },
  moduleNameMapper: {
    '^@domain/(.*)$': '<rootDir>/src/domain/$1',
  },
};

Downgrading ts-jest to 26.4.1 solves the issue.
To Reproduce
Run jest with coverage option with default coverage provider.
npx jest --coverage

or
yarn jest --coverage

Expected behavior
Show files coverage.
Debug log:
> jest --coverage

 PASS  src/domain/use_cases/Tools/CreateTools.spec.ts
 PASS  src/domain/entities/Tool.spec.ts
----------|---------|----------|---------|---------|-------------------
File      | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
----------|---------|----------|---------|---------|-------------------
All files |       0 |        0 |       0 |       0 |
----------|---------|----------|---------|---------|-------------------

Test Suites: 2 passed, 2 total
Tests:       10 passed, 10 total
Snapshots:   0 total
Time:        1.554 s

System:
    OS: Ubuntu 20
    Node version: 12.18.1(LTS)

Npm packages:
    jest: 26.6.1
    ts-jest: 26.4.2
    typescript: 4.0.3
    babel(optional):

Hi, would you please provide a repo to reproduce the issue ?

Sure.
https://github.com/ygortgaleno/bossabox-backend/tree/ts-jest-coverage-broken

@ahnpnl - I have a project where the change from 26.4.1 to 26.4.2 breaks the tests because 26.4.2 is not "finding" ambient types. i.e. I have types declared in modules without export and those are (obviously) ambient types and 26.4.2 is not picking them up.
This PR has the errors: https://github.com/guyellis/plant/pull/4180
I think that this is the same problem that the OP has mentioned. If it's different I'm happy to open a different issue.

@ygortgaleno I have checked out your project but I see coverage works for me on the branch ts-jest-coverage-broken.
@guyellis I think your issue is a different one, but I know what causes the issue.

@ahnpnl - would you like me to open a new issue or will you take it from here?

yes, I think it's better to have a new issue. Your issue is about ambient types.

Tracking Ambient Type issue here.

Here when I run with ts-jest 26.4.2

But I added an action for target branch and coverage was ok, I think is something with my environment, so I will close this issue, thanks for help.

@ahnpnl @ygortgaleno
I have the exact same problem, Jest Coverage is broken with latest version of "ts-jest": "^26.4.2". I was about to open a new issue on Jest GitHub but it took me an hour to figure out that it was actually ts-jest that broke it. Going back to version 26.4.1 fixes it (even with latest versions of Jest).
For a repro, you can refer to 1 of my open source lib Slickgrid-Universal
Here's my package.json and here's my jest.config.js
So this issue is valid and should be reopened, it is a regression on my side (I actually have 2 open source libs that encountered the exact same problem).

hi @ghiscoding , I've checked out your repo but I don't know what are the steps to reproduce. Would you please give some instructions ?

@ahnpnl you can run the Jest test with yarn test or npm test which calls npx jest --runInBand --coverage=true --config ./test/jest.config.js

I actually had to run build first to be able to run tests. I tried with 26.4.3 (prepare to release) and I got this

I guess this proves that 26.4.3 will solve the issue ?

I'm working on a branch (not pushed) which I have updated to the following
  "devDependencies": {
    "@types/jest": "^26.0.15",
    "jest": "^26.6.1",
    "jest-cli": "^26.6.1",
    "jest-environment-jsdom": "^26.6.1",
    "jest-extended": "^0.11.5",
    "jest-junit": "^12.0.0",
    "ts-jest": "^26.4.2"

and I don't see anything showing up in the Coverage, while going back to 26.4.1 does show up.

would you please push that branch ?

Ok sure, here's the PR of the new branch, do you want me to merge it? It's missing the unit tests, but I can add them later if you want it merged. CircleCI is running now, I'll get the result in few minutes, I just went back to 26.4.2 and I see 2 files (instead of a hundred)

I'll report back once CircleCI is over

I cannot reproduce either. My local shows exactly like your CI

I have 2 other small changes in webpack config, let me undo them and see if that makes a difference.... nope still the same I don't see the full result. I don't understand... let me delete my node_modules and re-install, that sometimes fixes these kind of issues

@ahnpnl
I deleted the node_modules, deleted the jest-coverage report folder and re-ran the Jest tests but still it's incomplete. It now shows 1 more file (now I have 2/4 files that I've changed in the PR). Anything else I can try to troubleshoot this problem? I could go back to previous version to fix it temporary but I'd like to be at the latest as much as possible (even more so on a patch version).

yes that is expected.
I just published 26.4.3-beta.0 at tag next. You can also try with this next tag.

I am using Mac. Is your issue about missing coverage of files or the coverage is not correct ?

@ahnpnl
I did the Jest clear cache with npx jest --clearCache and now it works, and then I wanted to see what happened if I run it twice and it provides full test coverage both times. I find that strange that I never had to do that before the new version but now it seems that I need to clear Jest before running a full test coverage (I don't mind adding it to my script, I'm already running runInBand and I don't think clearing Jest would slow it down that much, I just wonder why I have to clear it now when I've never had to do that before).
I think installed 26.4.3-beta.0 as you suggested, ran the full test coverage and it works too. So I'm not sure if it's just because of that Jest clear cache that everything started to work? but I only cleared once and now it works every time I run the coverage... never thought of doing a clear cache (rarely ever have to do that).

Is your issue about missing coverage of files or the coverage is not correct ?

I'm expecting a full test coverage report of all files, I typically run this every time I'm adding new features so that I can see which lines are missing unit tests. However the latest version was only show 1 or 2 files of coverage, not the entire hundred files which was totally unexpected.
So it looks like I'm back in business, even if I find that strange since I had the same issues on 2 different computers with 2 different projects.


DOC: Could better documentation and instructions and add support for usage in Github actions.

Like the title says could there be more documentation on usage. Also adding support for GitHub actions will be great. It will become more accessible too.

@ShantanuDash you know, I work on this project for free in my spare time, when I should be working extra hours in a paid job to cover rent/food/family etc. We are already using github actions
if you need better documentation, please tell me about WHAT you want better documentation on
and best of all , you can even make a pull request to start better documentation, dont just think I will magically solve everything


is it possible to run it in headless mode

It would be great if we could run it in headless mode.

+1

Check out alertzmanager.io


check if google maps script is already loaded

Hello, first of all thanks for your work.
Added simple check if google maps script is already loaded on the page.
Issue occurs if you use this wrapper in multiple pages with vue router or nuxt and wrapper tries to load google maps it each time its mounted.

great! nice catch!, I'm publishing this in a couple of minutes. Thank you.


#006 Release Additional Functionalities

[x]  Update the version ****
[x]  Push/release the changes to the PyPI and do the first test in any other environment

Done

https://pypi.org/project/pyGeckoCrypto/


Fix double urlencode

Double encoding caused some issues, for example, spaces in the title would appear as +'s when decoded.

It's correct that title is encoded separatly ("doubled"). Please read this discussion:
https://github.com/PHPGangsta/GoogleAuthenticator/pull/15


kubectl-proxy changes http-header-keys to capitalized

What happened:
When I am using kubectl proxy, the past proxy http request header keys are converted to initial capital.

What you expected to happen:
HTTP/1.1
According to RFC 7230, section 3.2:

Each header field consists of a case-insensitive field name followed by a colon (":"), optional leading whitespace, the field value, and optional trailing whitespace.

HTTP/2.0
Quoting RFC 7540, section 8.1.2:

Just as in HTTP/1.x, header field names are strings of ASCII
characters that are compared in a case-insensitive fashion.

How to reproduce it (as minimally and precisely as possible):
Use kubectl proxy and log all headers in svc-pods
Anything else we need to know?:
Happened both in aks and local k3s.
I'm not sure if this is a bug or intended?
Environment:
aks
Client Version: v1.29.4
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.28.9

local
Client Version: v1.29.4
Kustomize Version: v5.0.4-0.20230601165947-6ce0bf390ce3
Server Version: v1.29.3+orb1

Since the headers are case-insensitive, does it matter that these headers are capitalized or not?. Wouldn't eventually all be considered as same?.

Since the headers are case-insensitive, does it matter that these headers are capitalized or not?. Wouldn't eventually all be considered as same?.
Thanks for your reply. I'm not thinking it a serious problem, but want to know that why kubectl proxy did that capitalizing


Updated readme with instructions for analysis mode (Issue #171)

@SebastianZimmeck, I added a small paragraph on how the Analysis Mode works, let me know if you want me to change anything.

Excellent!


Validate memory heap information: device local bit

The AMD beta driver has been returning a memory type specified as DEVICE_LOCAL but which references a heap which does not have the VK_MEMORY_HEAP_DEVICE_LOCAL_BIT set. This is in violation of the specification and should be tested, but doesn't appear to be at this time.

Thanks for the suggestion. We will take a look at adding a test for this.

This should be checked by vulkan-cts-1.0.2 release.


Wire UI gestures to API calls

Drag and drop

WE DID IT

https://www.youtube.com/watch?v=csBaUsAySSQ

IT'S BEAUTIFUL

WORKING FLOWCHART


[chore] Update typescript to 5.3.3

Seeing if we can update typescript with a smaller blast radius than https://github.com/lit/lit/pull/4520

Closing in favor of https://github.com/lit/lit/pull/4520


Built in bake button

Description
I read the documentation here: https://terrain3d.readthedocs.io/en/stable/docs/navigation.html#baking-a-nav-mesh
and it says that the built in bake button shouldn't be used, because of https://github.com/godotengine/godot-proposals/issues/5138.
This proposal was already implemented. Should this notice be removed?

That PR implemented a new API. We aren't using that API. So, the nav region bake button shouldn't be used until someone implements the callbacks.

Oh okay, thanks for the clarification


Does ng-sortable deal with isolate scope?

Hey so I have an issue with what I think is an isolate scope issue.  I have a unordered list inside of a ui-bootstrap accordion.  I try to drag something horizontally from 'Available Columns' to Custom Template.  It gives me a weird error about:

Uncaught TypeError: this.parent.modelValue.indexOf is not a function
apply @ ng-sortable.js:306drag
End @ ng-sortable.js:983n.event.dispatch
@ jquery-2.1.4.min.js:3r.handle
@ jquery-2.1.4.min.js:3

I think it has something to do with the scoping, but I am not completely sure.  This is happening with ui-bootstrap-0.14.3-tpls, jquery 2.1.4, angular.js 1.4.7, and ng-sortable most current as of 11-10-2015.
This, by the way, only happens when i have things in both tableItems and columns.  If I have only tableItems (left un-ordered list) or columns (right un-ordered list) it doesn't do anything.  No error.  No moving of the card.  It acts like it is completely ignored.
Any help would be greatly appreciated.

Available Columns

                <uib-accordion close-others="false">
                    <uib-accordion-group ng-repeat="table in $storage.tables">
                        <uib-accordion-heading>
                            {{table}}
                        </uib-accordion-heading>

{{item.table}}

{{item.column}}

                    </uib-accordion-group>
                </uib-accordion>

Custom Template

{{item.table}}

{{item.column}}

                                Remove

If I have only tableItems (left un-ordered list) or columns (right un-ordered list) it doesn't do anything. No error. No moving of the card. It acts like it is completely ignored.

Hi I see a 'no-drag' attribute set on 'sortable-item' so you wont get the drag working.

Yea so I remove that attribute and still see a problem.
When I drag from one column to the other column I get an error:
ng-sortable.js:983  dragItemInfo.apply();
ng-sortable.js:306
      if(this.parent.modelValue.indexOf(this.source.modelValue) < 0) {
            this.parent.insertItem(this.index, this.source.modelValue);
      }

this.parent.modelValue points to an object, not an array/list.  Erroring on indexOf.  Is there something I am missing in the above HTML?  Like do I need a special ng-model value to make this work?  I am going to try it without the ui-bootstrap accordions and see if it changes how it behaves.
Thanks for the reply and help!

Okay so I figured out the model at the scope of as-sortable needs to be a list.  I was being dumb and referencing a model that held the list not the list itself fixing that fixed the issue.


bump deps

I had always assumed that deps should be bumped to the version that was actually tested.  I will push a new commit in a minute updating pify only.

Requested changes completed.

There's a reason we have semver ;).


If alluxio can support index file based distributedLoad CLI

Is your feature request related to a problem? Please describe.
Given a UFS is mounted, we only want to distributedLoad a subset of files which are listed in a index file.
Describe the solution you'd like
an index file can be provided in distributedLoad CLI to specify files being loaded.
Describe alternatives you've considered
N/A
Urgency
Medium.
Additional context

@chaowangnk1 If I understand correctly, you have a file with a bunch of names you want loaded. Would having a script that parses that and call distload for each file work and you are looking for a more convenient API? Or are you looking for a batch distload (ie. decrease the number of RPCs)?

@calvinjia yes, in one scenario, I have a file with list of files I want to load.
It would be great if we could have a convenient CLI to load files based on the index file, in one CLI run. for example, to extend CLI dtsirbutedLoad to support index file based distload.

Got it, thanks for clarifying. Would something like distributedLoad -f <FileName> work?

Definitely works for me.

@jenoudet could you take a look at this issue?

@calvinjia @chaowangnk1 should the index file live on Alluxio or is it a local file ?

@jenoudet let's start with a local file.


fix: report relevant error on invalid jsons

Fixes https://github.com/netlify-labs/netlify-plugin-lighthouse/issues/79

:tada: This PR is included in version 1.4.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Shields of Ecuador

Ecuador identifica sus carreteras con dos discos que parecen mucho similares con el escudo de autopistas interestatales de los EE.UU. Las carreteras primarias o arteriales tienen un disco azúl y los secundarias o colectoras tienen un disco verde. En OSM, estas rutas tienen las etiquetas network=EC:national y network=EC:provincial, respectivamente. Los números rutales tienen el prefijo “E”, entonces los números pueden ser más largos que aún el número interestatal más largo.
Ecuador marks its roads with two shields that look strikingly similar to the Interstate Highway shield in the U.S. Primary or arterial highways get a blue shield, while secondary or collector highways get a green shield. In OSM, these routes are tagged network=EC:national and network=EC:provincial, respectively. Route numbers in the shield are prefixed with “E”, so the numbers can get longer than even the longest Interstate route number.

network=EC:provincial is incorrect on the collector highways, which are still part of the national route network. In fact, Guayas province has its own provincial primary highways: #457.

He pedido en el chat OSM Latinoamericana; un usuario allá he reenviado el mensaje a Telegram de MappingEcuador.
Asked in the OSM Latin America Telegram chat; a user there forwarded the message to MappingEcuador Telegram.

A general strike in Ecuador (archive) has been going on for the last couple of weeks, so I imagine most Ecuadorians are quite busy at the moment and this issue may be on holld until the political situation is better.

I have just come across this issue. There are 2 types of shields that are standardized for the national road network of Ecuador. One is blue for the arterial roads and the other is green for the collector roads. The nomenclature always starts with a capital "E". From west to east and in a north-south direction begin with E5, E15, E25, E35 and E45. From north to south and in east-west direction E10, E20, E30, E40 and E50. All others are variations of these 10. The standard of the shield can be found in this link https://drive.google.com/file/d/1f2Adspg5LoIhqOkXEEXdqJIw-mhaZcHw/view

Hi @fmvaldezg! Glad to have more help with route shields.
As it is right now, route=road relations in Ecuador are mapped completely and consistently on OSM, but they're tagged with network=* values that appear to be misnomers (EC:national for arterials, EC:provincial for collectors) applied by corporate contributors without consulting locals. There's enough information in these tags for Americana to display correct shields for arterial and collector routes, but before they can be added, should Ecuador stick with this tagging scheme, or retag them all to something more aligned with official nomenclature?


Refactor predict_weight function

I was thinking maybe we could re-factor predict_weight and close https://github.com/rust-bitcoin/rust-bitcoin/pull/1793.

Definitely agreed. This turns a no-allocation API into an unnecessary-allocation API, and reduces its flexibility.

@Kixunil @apoelstra I updated the function signature to:
pub fn predict_weight<I, O>(inputs: &mut I, output_script_lens: &mut O) -> Weight
where
    I: Iterator<Item = InputWeightPrediction> + Clone,
    O: Iterator<Item = usize> + Clone,
    I::Item: Borrow<InputWeightPrediction>,
    O::Item: Borrow<usize>,
{

That just a mutable ref is passed and a second heap allocation is avoided.  What do you think?

That just passes a mutable ref and a second heap allocation is avoided. What do you think?

I'm unclear on the benefit here ... this still makes the API harder to use and less flexible.

I was hoping to make the fold that does three things more readable.  However I didn't think that type signature would need to be modified so much.

Iterators in rust are much different than other languages I'm learning.  When an iterator is used it is consumed and theres no way to unwind it which is why the signature must be cloneable and mutable so that the same iterator can be copied and consumed again.

Anyway, since this seems like a failed attempt to refactor this section in hopes of making it more readable it can be closed

I guess better just learn the combinators. Perhaps change fold to for if people really dislike this a lot but not sure if it'll help much.

Iterators ought to perform better than for loops. It would be nice for this code path to have some benchmarks.

I think there wouldn't be a difference in this case. It applies when calling collect but in this case we're doing trivial math operations.

I meant that generally iterators are easier for the compiler to reason about than something as generalized as a for loop.  I'm not sure if the rust compiler actually makes that type of optimization yet.


Re-add editor guide link

Following feedback from @Scotchester. Follow-up from #9479.
This re-adds the link to the placeholder page, so people can still locate the guide in our navigation. We will need to cherry-pick this to stable/4.1.x after merge.

Cherry-picked on stable/4.1.x.


$0 behaves differently depending on its content

All snippets are provided in the vscode format

The following snippet makes the last node of the snippet where the $0 is.
${1:var} = $0;

The following do the same, after jumping to $0, luasnip.jump(1) will no longer take effect, nvim will keep select mode no matter how luasnip.jump(1) is called.
${1:var} = ${0:nil};

The problem occurs in the following snippet. Ideally, it should behave all the same as the last one, i.e. after jumping to $0, luasnip.jump(1) should not take effect, but the actual situation is different, the calling luasnip.jump(1) again will move the cursor to the position before semicolon and enter insert mode.
${1:var} = ${0:$TM_FILENAME_BASE};

I've tested the same cases in vscode, luasnip behaves differently with vscode.

Hey :)
IIRC I implemented this because there were some issues with the $0-node in luasnip, in that it couldn't actually contain text.
Since then there have been quite a few changes, and I'll reevaluate whether this still makes sense :)
(But it'll take some time, I don't think this is pressing)


Godlike faces bug (i can't confirm that fake rate is the problem)

The godlike faces don't show when outside the the level, let me show you an example

Also i disabled texture loader by geode team because that's the problem to the pink box thingy

Caused by GodlikeFaces (Will be fixed AdyaGMD/GodLikeFaces#5 if Adya actually becomes active, because it's becoming quite tiring waiting)


installation: should pywebpack v1.0.3 be pinned here?

Left-over from #113

From Alex:

Theoretically if we follow the "coordinator" modules paradigm we shouldn't, i.e. tests will get the latest since it's a dependency of Flask-WebpackExt. The fact that users of invenio-assets have to upgrade their dependencies is though another issue which can e.g. be addressed in the cookiecutter

It is true that pip will install the latest, so it should be ok, unless someone installs a version before and then the theming will not work. We can probably close this. @slint?


:heavy_minus_sign: Removing "react-hot-loader" package

This package is deprecated and there's no plans to support it for React 18.
https://github.com/gaearon/react-hot-loader/issues/1808#issuecomment-1279728183
What do you think if we remove it?
The webpack hot module replacement works well even without "react-hot-loader".
The problem I'm trying to solve is this error:

Also I wanna update some dependencies to resolve all similar dependency errors.

Also, this package wasn't used 😀
We don't use it in the webpack config
https://github.com/microlinkhq/react-json-view/blob/master/webpack/webpack.config.js#L37

This is a good catch, thanks for it 🙂


run by tag in tabledisplay

def _run_by_tag(self, tag):
    arguments = dict(target_name='beakerx.tag.run')
    comm = Comm(**arguments)
    msg = {'runByTag': tag}
    state = {'state': msg}
    comm.send(data=state, buffers=[])

this was relying on beakerx - we have to fix this

@ildipo as we discussed I've removed this in https://github.com/twosigma/beakerx_tabledisplay/pull/3


Add runs flag for compiler optimiser on test cli command

resolve #170

Codecov Report

Merging #206 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #206      +/-   ##
==========================================
+ Coverage   98.57%   98.57%   +<.01%
==========================================
  Files          73       73
  Lines        2876     2877       +1
==========================================
+ Hits         2835     2836       +1
  Misses         41       41

Impacted Files
Coverage Δ

cli-commands/commands.js
99.44% <100%> (ø)
:arrow_up:

cli-commands/etherlime-test/etherlime-test.js
100% <100%> (ø)
:arrow_up:

cli-commands/etherlime-test/test.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Update:(libsinsp/parsers): extend parseres to support pidfd.

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area libsinsp
/area tests
Does this PR require a change in the driver versions?
What this PR does / why we need it:
This PR extends parsers to support syscalls like pidfd_open and pidfd_getfd
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
Update:(libsinsp/parsers): extend parseres to support pidfd

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Rohith-Raju
Once this PR has been reviewed and has the lgtm label, please assign lucaguerra for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Please double check driver/SCHEMA_VERSION file. See versioning.

Driver SCHEMA_VERSION checks failure is a false positive here.

New changes are detected. LGTM label has been removed.

LGTM label has been added.  Git tree hash: b84fa507b44e1ce129a25a73f1ee4832fb828c53

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, Rohith-Raju
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, Rohith-Raju
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


mysql + cannot change /var/lib/mysql/ ownership

Info:

Docker version: 17.03.1-ce, build c6d412e
Laradock commit:  18b6c1b90785329ad4bff28ed5f6dcc298d2ad69
System info: Mac
System info disto/version: macOs Sierra 10.12.5

Issue:
mysql_1                | chown: changing ownership of '/var/lib/mysql/': Operation not permitted

Expected behavior:
mysql container should successfully start.

Reproduce:
Use d4m-nfs (https://github.com/IFSight/d4m-nfs)
Use LaraDock in multi site way, direcory structure:

BASE_DIR_PATH/laradock
BASE_DIR_PATH/projects
BASE_DIR_PATH/data

Warning:
BASE_DIR_PATH/data/mysql MUST exist since /tmp is the only directory shared by docker (see https://github.com/IFSight/d4m-nfs)
Adjust .env file as follows:
APPLICATION=../projects
DATA_SAVE_PATH=../data
PHP_VERSION=56

Relevant Code:
# start d4m-nfs
/path/to/d4m-nfs/d4m-nfs.sh

# rebuild everything
docker-compose build --no-cache nginx mysql phpmyadmin workspace php-fpm applications

# start relevant containers
docker-compose up nginx mysql phpmyadmin

Specifying the user in the docker-compose.yml file worked for me.
In the MySQL / MariaDB container section add: user: mysql

@JohnAlonso Thx, it works!!!!

Specifying the user in the docker-compose.yml file worked for me.
In the MySQL / MariaDB container section add: user: mysql

thank you! you save my day.
your answer helps me to resolve a similar issue on Kubernetes.
https://kubernetes.io/docs/tasks/configure-pod-container/security-context/
Once again, thank you!

Specifying the user in the docker-compose.yml file worked for me.
In the MySQL / MariaDB container section add: user: mysql

Thank you! Your solution worked for me.


Marlowe page: add security review

Not to get in the habit of adding these links one at a time (https://github.com/cardano-foundation/developer-portal/pull/1095, https://github.com/cardano-foundation/developer-portal/pull/1078) but I believe this blog entry today, which developed from the recently linked Tweag audit, also serves as a substantial documentation page.
@joseph-fajen @bwbush please confirm if you think this link is presented appropriately or not.

@rphair I agree, I think this is a significant blog post and it will be a useful reference point going forward. It's worth calling it out. :)


Login button broken

Describe the bug
Login button does nothing. Cannot click the login but the clear data button is working.
To Reproduce
Go to apps
Click on greenlight
Click on login
Expected behavior
Click login button to proceed to xbox login
Environment
- OS: steamos
- Device: steam deck 256gb
- Greenlight version: v2.1.1

Anything else?
No response
New issue

[X] I confirm that this is a new issue and not a duplicate

If the authentication window does not pop up then it is most likely because it fails to retrieve a device token. This usually happens when the time and date of the device is configured incorrectly. Can you make sure the time and date is synced with the internet time and then try it again?

I was able to sign in but only through my mobil Hotspot feature. Will not work on wifi. And when connecting to console, it just endlessly loops.
Sent from my U.S.Cellular© Smartphone
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: UnknownSKL @.>
Sent: Thursday, February 8, 2024 7:58:58 PM
To: unknownskl/greenlight @.>
Cc: Kingspade420 @.>; Author @.>
Subject: Re: [unknownskl/greenlight] Login button broken (Issue #1126)
If the authentication window does not pop up then it is most likely because it fails to retrieve a device token. This usually happens when the time and date of the device is configured incorrectly. Can you make sure the time and date is synced with the internet time and then try it again?
—
Reply to this email directly, view it on GitHubhttps://github.com/unknownskl/greenlight/issues/1126#issuecomment-1935224441, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BFZTDLOOEMGBJQUQOD2YZFDYSV7GFAVCNFSM6AAAAABCWCGPQCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZVGIZDINBUGE.
You are receiving this because you authored the thread.Message ID: @.***>


A little strange to include CubicBool in ConsIndShockSetup

Would it not make more sense to include CubicBool only in ConsIndShockSolver? Outside of ConsIndShockSolver, it's assigned in the __init__ of ConsIndShockSetup and used here https://github.com/econ-ark/HARK/blob/7afa2c18f67c7e534fa51abbb4a2f74540086296/HARK/ConsumptionSaving/ConsIndShockModel.py#L695 . Would it not make more sense to move this to the initialization in ConsIndShockSolver? This way, there will be no CubicBool parameter in a ConsIndShockSolverBasic at all.
If not, should it be possible to use cubic interpolation in the ConsIndShockSolver as well?

This is part of a more general, longstanding problem with the solvers: The
ridiculous propagation of methods that store particular solver inputs
attributes of self.  That is, I wrote terrible init methods that do
simply:

call the method of the parent solver class
store 1-3 extra inputs as elements of self

This was poor form and stemmed from ignorance of Python.  The
assignParameters (or call) method of HARKobject was the first step in
fixing this, as the init method of the solvers could then simply do
self(*args) (or self.assignParameters(*args) for clarity) to make all the
inputs accessible, and never have to fiddle with input names.  Or in this
very bad case, store inputs that literally aren't used!
So yes, this is definitely important to fix.
On Mon, Feb 25, 2019 at 5:41 AM Patrick Kofod Mogensen <
<EMAIL_ADDRESS>wrote:

Would it not make more sense to include CubicBool only in
ConsIndShockSolver? Outside of ConsIndShockSolver, it's assigned in the
init of ConsIndShockSetup and used here
https://github.com/econ-ark/HARK/blob/7afa2c18f67c7e534fa51abbb4a2f74540086296/HARK/ConsumptionSaving/ConsIndShockModel.py#L695
. Would it not make more sense to move this to the initialization in
ConsIndShockSolver? This way, there will be no CubicBool parameter in a
ConsIndShockSolverBasic at all.
If not, should it be possible to use cubic interpolation in the
ConsIndShockSolver as well?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/228, or mute the thread
https://github.com/notifications/unsubscribe-auth/ANUQFQ1hiROGo_JBVpDI5Tup20QTQOGmks5vQ73MgaJpZM4bPlDl
.

Or in this very bad case, store inputs that literally aren't used!

Yes, this is exactly what I was focused on. It creates a sort of reverse inheritance where ConsIndShkSetup begins to setup things that are only relevant for some children - those children should just do that work instead.
I was wondering if it made sense for me to do the cubic-part, but if you're planning an overhaul, I may just step on your toes.

Regardless of who does it, I think it should be done whole hog: Go down the
line of solvers and delete the "store one input at a time" init
methods, and replace it with a single simple one at the top of the class
tree.  I don't think it's worth fixing this one extra weird case on its own.
On Mon, Feb 25, 2019 at 8:58 AM Patrick Kofod Mogensen <
<EMAIL_ADDRESS>wrote:

Or in this very bad case, store inputs that literally aren't used!
Yes, this is exactly what I was focused on. It creates a sort of reverse
inheritance where ConsIndShkSetup begins to setup things that are only
relevant for some children - those children should just do that work
instead.
I was wondering if it made sense for me to do the cubic-part, but if
you're planning an overhaul, I may just step on your toes.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/228#issuecomment-467020634, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANUQFQazSWFwZDZDcTdAo-GcAS__WHOhks5vQ-wZgaJpZM4bPlDl
.

I'll also mention here that I think either PKM or CDC agreed with my proposal to simplify the inheritance structure in ConsIndShock; there are too many "middlemen", which I've never liked.

OK -- what's the next step on this?
On Thu, Feb 28, 2019 at 8:57 AM Matthew N. White<EMAIL_ADDRESS>wrote:

I'll also mention here that I think either PKM or CDC agreed with my
proposal to simplify the inheritance structure in ConsIndShock; there are
too many "middlemen", which I've never liked.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/228#issuecomment-468281708, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABQlf1Xy7dliRW5lKVe7LnvdDbZPnZcyks5vR-BjgaJpZM4bPlDl
.

--

Chris Carroll

I suspect this might be something that ends up in HARK 2.0.  So much inherits from ConsIndShock that it might take a lot of surgery to change this.

Now I'm imagining Patrick as the nurse who wipes the sweat from my brow as I focus on carefully slicing the patient on the table.

I'm here for you every step of the way!
(btw, yes I agreed with you on the simplification of the inheritance structure)

For someone new to HARK who wants to try tackling this:

We have a bunch of code in init methods of solver classes (or methods called by init) that does nothing more than store arguments as (identically named) attributes of self.

This starts with HARK.ConsumptionSaving.ConsIndShockModel.ConsPerfForesightSolver, but there is a long tree of inheritance beyond that.  As models get more and more complex, there's a long chain of calls to assignParameters in various (grand)parent classes.

This is silly and unwieldy.  We should instead have one place where all solver arguments are stored as attributes of self, and then all subclasses of the highest level solver execute the parent's init method.

Essentially, the init method of ConsPerfForesightSolver should: 1) Get the names of [this method]'s arguments; 2) store them all as attributes of self.

Then, all classes that inherit (directly or indirectly) from ConsPerfForesightSolver (i.e. basically every class that ends in Solver) should run super().init(args).

There might be a bunch of small fixes as well, if some solvers do any work in assignParameters other than simply store arguments.  But this is a start!


:h hotpot-cache no longer works

This comes from the README:

This allows the user to easily view the result of the compilation for debugging. See :h hotpot-cache.

I just tried to use :h hotpot-cache but the following error appears: E149: Sorry, no help for hotpot-cache.

The docs are auto generated now so it's under hotpot.api.cache, thanks for the heads up.


CI fails on the majority of PRs at the moment

It seems that the rv32i test times out for most pushes. If we can't speed it up, would it make sense to turn this test off so that we don't have to keep hitting "re-submit"?
CI latency is currently ~1 hour if all is well, but this more than doubles if you're unlucky.
@imphil as resident CI guru!

Yes, let's disable them. We didn't have a single issue caught by them for a long time now, so doing that shouldn't have much influence on the quality of results.
I'll do a PR shortly.


bump to 0.93.0

this PR

bumps the dependencies to 0.93.0
fixes the code
bumps the plugin and the package to 0.93.0

Summary by CodeRabbit

New Features

Updated dependency versions for enhanced functionality and compatibility.

Bug Fixes

Adjusted clipboard operations to improve memory efficiency.

Refactor

Modified JSON handling to simplify code and improve performance by removing redundant cases.

Thank you for PR

happy to help, this is so helpful to share snippets from the REPL :pray:


[Solved]

Hi all,
I try to using command alert as below,
alert:
-command
command: ["/opt/alert/action.sh", "%(timestamp)s","%(user)s","%(ip)s","%(id)s","%(count)s"]

but got the error as follows,
ERROR:root:Error while running alert command: Error while running command /opt/alert/action.sh 2016-09-07 11:14:16.819 guest <IP_ADDRESS> 1/0/3 5: [Errno 8] Exec format error
If i'm wrong is it possible to explain how i can do this ?
Thanks by advance,

How to solve it

Probably missing https://en.wikipedia.org/wiki/Shebang_(Unix)


converted delimiter from String to char

Pull Request Test Coverage Report for Build 1128

4 of 4 (100.0%)  changed or added relevant lines in 4 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 88.081%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/com/salesforce/dynamodbv2/mt/mappers/sharedtable/impl/FieldPrefixFunction.java
3
89.66%

Totals

Change from base Build 1125:
0.0%

Covered Lines:
2143

Relevant Lines:
2433

💛  - Coveralls


Not possible to use --line-ending parameter

ApiDoc supports --line-ending param, but it is not availbale in the laravel wrapper.
https://github.com/apidoc/apidoc/issues/190

Don't know when this will get to the roadmap, so PRs welcome.

This is not a Laravel wrapper of that. This is a Laravel wrapper of Documentarian. And there are no known issues with line endings.


Make logging enablement simpler in Azure Web Apps

Today, when git deploying an app to Azure, you end up with this in web.config:
<aspNetCore processPath="dotnet" arguments=".\AspNetCoreRC2.dll" stdoutLogEnabled="false" stdoutLogFile=".\logs\stdout" forwardWindowsAuthToken="false" />

To enable logging (including streaming), you need to change it to:
<aspNetCore processPath="dotnet" arguments=".\AspNetCoreRC2.dll" stdoutLogEnabled="true" stdoutLogFile="%home%\LogFiles\dotnetcore" forwardWindowsAuthToken="false" />

But Azure Web App already has an existing mechanism to enable logging, and it would be good to just honor it. It works as follows:

there is a file called settings.json in %home%\site\diagnostics
it contains something like this: {"AzureDriveEnabled":true,"AzureDriveTraceLevel":"Error"}
if AzureDriveEnabled is true, it logs to files in %home%\LogFiles\Application

I guess this is already tracked by https://github.com/aspnet/IISIntegration/issues/167, though I probably have more info here.

This should be in IISIntegration linked to https://github.com/aspnet/IISIntegration/issues/167


"Cannot find a definition for job named build" error calling workflow: 'build' when Implementing CircleCI "Trigger Build" webhook,

When configuring CircleCI Trigger Build webhook via Contentful Dashboard at /settings/webhooks, job fails as it requires a job that wraps all jobs called build. Is there no way to configure this webhook to trigger all jobs in .circleci/config.yaml? We'd prefer to keep our jobs separate.

Hey @AlexandraKlein this repo exists as a example. I expect you'd need to modify it to meet your needs so please feel free to adjust or change it as nessesary.
In this example I don't actually use a webhook from Contentful to fire off a build, I end up using a push to GitHub to trigger a content migration. The logic being that if the content model is changing, the code should have also changed at the same time. Specifically using a contentful webhook to trigger a build seems like a bad pattern to me because you can't guarantee that code deployed is ready to handle whatever content model changes that you'd be sending up.
In terms of getting a webhook to trigger a job I'd encourage you to take a look at the circle docs. They have support for running jobs via and API (https://circleci.com/docs/2.0/api-job-trigger/). You could set up a lambda function that is called by a webhook on Contentful and fires off an API call to circleCI trigger whatever jobs you need if that's appropriate.

Hi @Shy,
I opened up an issue with Contentful Support and they encouraged me to open an issue here.
My problem is with the CircleCI Build webhook template in the Contentful Dashboard (/settings/webhooks).  It does seem to allow me to set which jobs (all jobs?) I'd like to run from my config.yaml when that webhook is triggered. I only get the error Cannot find a definition for job named build, which tells me that webhook is looking for a job named build by default. I do not have a job named build. I have many jobs. And I'd like them all to be triggered, as is needed to deploy my site. Is there any way to tell Contentful which jobs to run? Or must a wrap my "jobs" in a job called build for this to execute? If so, that seems like an oversight on Contentful's part.

@AlexandraKlein I understand now. It looks like support pointed you to the wrong place. This repo and our circleci webhook template are totally different things. That said I'm reaching out internally and I'll do my best to get back to you with a real answer.
In the interim, you can create custom webhooks that don't use the templates at all. You can customize those to whatever your needs are. You can also modify the payload of the template webhook in the webhook settings after it's created. I'm not sure if changing the payload or building the webhook from scratch will be enough to let you specific which jobs it would need to run though. I haven't used circle ci's webhook functions before so I'd need to read there docs to find out.

Thanks, @Shy ! Yeah, unfortunately the add webhook feature that doesn't use a template still does not have the option to specify job names from the config.yaml. I really appreciate you looking at this!

#70329 is my Contentful Support request number. (https://support.contentful.com/hc/requests/70329)


[REQUEST] A personalized stdout of progress

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2<IP_ADDRESS>. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2<IP_ADDRESS>

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
$ youtube-dl -v <your command line>
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue required an account credentials please provide them or explain how one can obtain them.
Hi I'm using the youtube-dl as part of an other process and I searched if the command line has any option to print the output progress at a diserd format but, there's no one. So I'm wondering if it would be possible to add an option to choose a desired output format of process for get the % of downloaded, totalsize, downloading current speed, etc into personalized form to pass to any external program for parse just like making string.split(" ") to choose the desired column.
Thanks in advance

There is a mechanism to reach progress data via progress hooks available from python code https://github.com/rg3/youtube-dl#embedding-youtube-dl. You can write a wrapper script in python that will translate it in any format you like. Is that enough?

Well since I'm not using python, I cannot make any wrapper python based. I'm using nodejs to invoke a child process that call youtube-dl. So I pick up the output and I parse it, at the moment with a regular expression, but I would pass as command line argument to youtube-dl soem option to format the output to jut pick up and no make any kind of regular expression, for example just like blikd command that you can choose what data you whant to show and in which format.

Could you give more details, for example how to specify the format and what the output should look like?
For me such a feature sounds highly application-specific but not for generic uses, so youtube-dl may not implement it. Writing a Python wrapper is never difficult. Feel free to ask questions if something goes wrong.

Well actually there's a -o option to format the final file downloaded which hace many predefined values such %(id), %(title),  %(ext), etc... So what I say is specify, say -O or --output or -stdout, whatever you want, a template format to style the output, selecting a specified variables like %(download_speed),  %(eta),  %(progress), anything else, for format the output, overriding the predefined which actually is showing [download]    blablabla...
And some other kinda debugging words also separated with more than 1 space. I want format the output to just make a simple split by space or just pick up an specified field or the fields that I want.
One example could be: youtube-dl -O "%(download_speed) %(eta)"

Came here to request the same feature as OP. I'm honestly surprised this hasn't gained any traction. Plenty of other applications (UI specifically) wrap youtube-dl and could improve if this option was available. Even outputting bytes instead of a formatted percentage (progress) or MiB (file size) would be useful at this juncture.


Refactor mistral to support new CANCELLED status

Nice, LGTM :+1:

@m4dcoder what's happening with this PR - do we still need / want it?

Is this related to PR #3125 at all?

None what so ever

@m4dcoder Is this PR still active (aka needed / wanted)?

This PR is still active. Once our mistral fork catches up w/ upstream mistral, I will refactor this branch.

Current coverage is 77.44% (diff: 100%)

Merging #2916 into master will increase coverage by 0.06%

@@             master      #2916   diff @@
==========================================
  Files           431        431
  Lines         22236      22260    +24
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          17207      17239    +32
+ Misses         5029       5021     -8
  Partials          0          0

Diff Coverage
File Path

•••••••••• 100%
st2common/st2common/util/workflow/mistral.py

Powered by Codecov. Last update 4817949...913ed0b


[No QA] Remove Login With Validate Code logic we are no longer using

Details
Remove this unused code.
HOLD on

[ ] https://github.com/Expensify/App/pull/5396

Fixed Issues
$ https://github.com/Expensify/Expensify/issues/174109
Tests
N/A
QA Steps
N/A
Tested On

[ ] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots
N/A

This is no longer needed, will be taken care of here: https://github.com/Expensify/App/pull/5578


Strapi-Docker error connecting to Mongo-DB

Hi,
I hope you can help me.
I am trying different times now to establish a Strapi Test-Project using a running Mongo-DB.
I can access the MongoDB with the user rights are given in the environment vars (you can see below the existing MongoDB-Users as well as the existing "StrapiDB" on an internal network).
The environment parameters you can see below as well - I thought I should give all the information so you can decide what is important.
But it will not go forward as Strapi is not able to connect to the Mongo-DB, see the following:
Using strapi 3.0.0-beta.19.3
No project found at /srv/app. Creating a new strapi project
Creating a new Strapi application at /srv/app.
Creating a project from the database CLI arguments.
Creating files.
Dependencies installed successfully.
Your application was created at /srv/app.

Available commands in your project:
yarn ......

Starting your app...
Building your admin UI with development configuration ...

✔ Webpack
Compiled successfully in 16.19s
[2020-03-30T10:53:53.091Z] debug ⛔ Server wasn't able to start properly.
[2020-03-30T10:53:53.092Z] error Error connecting to the Mongo database.
                           Server selection timed out after 30000 ms

What I am doing wrong???
Maybe it's a small issue..., but as I am trying already different times without success, I am a little bit down now ;-)
Thanks in advance for any advise, below all relevant docker information:

Hi @jochenthomas
Have you tried setting the authenticationDatabase option to admin?

Hi thanks for your answer.
YES, as you can see in the second picture --> last Env_vars --> DATABASE_AUTHENTICATION_DATABASE = admin
Also in options I have set
      "options": {
        "ssl": false,
        "debug": true

Where I can find the logfile, maybe there is a more detailed explanation?

What I meant is that you need to add this variable to your database.json file too :)
{
  "connections": {
    "default": {
      //...
      "options": {
         "authenticationDatabase": "admin"
       }
    }
}

Haha,
yes I just did this, but Strapi shuts down again with the same error-message...
{
  "defaultConnection": "default",
  "connections": {
    "default": {
      "connector": "mongoose",
      "settings": {
        "client": "mongo",
        "host": "<IP_ADDRESS>",
        "port": "27017",
        "database": "StrapiDB",
        "username": "StrapiDB_dbOwner",
        "password": "pass"
      },
      "options": {
        "ssl": false,
        "authenticationDatabase": "admin",
        "debug": true
      }
    }
  }
}

I honestly don't see anything special to add to your config.
If your code should be using the env vars you should check this doc: https://strapi.io/documentation/3.0.0-beta.x/concepts/configurations.html#dynamic-configurations

Hi, I have seen this too...
But I don't see any advantage in using dynamic configs.
Because I changed the database.json directly.
So what would be possible steps to solve this issue?
I guess, the first point would be to evaluate the log files:
Where exactly I can find the database log file?
What is the result of activating "debug: true"?

I wasn't sure you were setting the right params in the file that's why i suggested using the env var to be sure it is the same values.
debug is an option to see the sql queries made with the sql connector. This will be useless with mongo.
You can try running your code with a DEBUG=* env var to see debug info for all the libs. This might give your more infos. I don't see what the issue could be if all the infromations are right to be honest 🤔

Is it possible that it might be a network thing maybe ?

In this case, it would not be possible to connect from another device (Notebook) with MongoDB-Tool (noSQLBooster) ....

Hi,
I created the container again and made a mistake by defining the env-vars (DATABASE-USERNAM instead of -NAME), so I got the following error message.
✔ Webpack
  Compiled successfully in 16.18s

[2020-03-31T11:50:41.022Z] error An index error happened, it wasn't applied.
        - command createIndexes requires authentication
[2020-03-31T11:50:41.023Z] error An index error happened, it wasn't applied.
       - command createIndexes requires authentication
[2020-03-31T11:50:41.023Z] error An index error happened, it wasn't applied.
        - command createIndexes requires authentication
[2020-03-31T11:50:41.025Z] debug ⛔ Server wasn't able to start properly.
[2020-03-31T11:50:41.025Z] error MongoError: command find requires authentication
    at Connection.<anonymous> (/srv/app/node_modules/mongodb/lib/core/connection/pool.js:460:61)
    at Connection.emit (events.js:210:5)
    at processMessage (/srv/app/node_modules/mongodb/lib/core/connection/connection.js:368:10)
    at Socket.<anonymous> (/srv/app/node_modules/mongodb/lib/core/connection/connection.js:537:15)
    at Socket.emit (events.js:210:5)
    at addChunk (_stream_readable.js:308:12)
    at readableAddChunk (_stream_readable.js:289:11)
    at Socket.Readable.push (_stream_readable.js:223:10)
    at TCP.onStreamRead (internal/stream_base_commons.js:182:23)

This shows me, that the DB connection is OK.
So what could be the issue here?

AS I told you there is nothing specific to strapi about debug files. You will need to look at your mongo logs to see if the connection is even going to the mongo instance.
What version of mongo are you using ??

Hi,
unfortunately, reading the guide gives no clear answers to that issue.
Luckily, after useless hours, I found the reason  - but would have expected clarification would come from the project owner.
If you follow the instruction (e.g. editing the database.json) please be aware that the username and password you are defining are NOT only for the requested project-database!!!
The user must have nearly superuser rights or more specific read/write rights to ANY database.
Only incidentally, I saw the advice related to Atlas DB:
    Under User Privileges ensure Read and write to any database is selected. Then click Add User to save.

So I am a little bit irritated: An enterprise-grade solution that exposes the whole MongoDB to a single user, even only a single project-database is necessary?
However I am currently in test-mode, so will go further to check out...

Hi @jochenthomas,  Strapi uses mongoose behinds the scenes and we do not do anything special with it. Strapi do run index creation at startup to match with the schema (e.g unique indexes). This explains the error in your last message.
As your error was abut connecting to the DB I was only focused on this and not the fact that you fixed the connection and had a new error :)
Hope this answers why you need more user rights enabled.
I'm going to close this issue as it wasn't coming from the docker image.


Multiple GPU Support

Description
We follow this idea and spawn one learner on each available GPU. Then we use Torch DDP to average the gradients like we already do when using multiple nodes.
For starters we are not using policy workers on multiple GPUs, as suggested here:

To take full advantage of this, we also need to support policy workers on multiple GPUs. This requires exchanging the parameter vectors between learner and policy worker through CPU memory, rather than shared GPU memory. This can be a step 1 of the implementation.

The only advantage of this would be to save some memory (and maybe some time transferring the gradients through DDP), by not storing the model multiple times per node.
Tasks

[ ] Add host list as parameter for multi-sample-factory
[ ] Initialize num_policies learner worker on each GPU
[ ] Initialize policy_workers_per_policy * num_policies  on each GPU
[ ] Initialize one SharedBuffer per GPU
[ ] Add each GPU separately to the DDP host list
[ ] Assign actor workers a fixed SharedBuffer to write to

With commit<PHONE_NUMBER>0201919b415467fa6c750ad962d727 we get a deadlock, because the learners are initialized synchronously, resulting in the learner 0 waiting for the DDP initialization of the following learners.
Log output
^[[36m[2021-12-14 10:52:34,672][17487] Default env families supported: ['doom_*', 'atari_*', 'dmlab_*', 'mujoco_*', 'MiniGrid*', 'unity_*']^[[0m
^[[36m[2021-12-14 10:52:34,671][32665] Default env families supported: ['doom_*', 'atari_*', 'dmlab_*', 'mujoco_*', 'MiniGrid*', 'unity_*']^[[0m
^[[36m[2021-12-14 10:52:35,366][17487] Env registry entry created: unity_^[[0m
^[[36m[2021-12-14 10:52:35,366][32665] Env registry entry created: unity_^[[0m
^[[33m[2021-12-14 10:52:35,470][17487] Saved parameter configuration for experiment saving_training_iss26 not found!^[[0m
^[[33m[2021-12-14 10:52:35,470][17487] Starting experiment from scratch!^[[0m
^[[33m[2021-12-14 10:52:35,470][32665] Saved parameter configuration for experiment saving_training_iss26 not found!^[[0m
^[[33m[2021-12-14 10:52:35,470][32665] Starting experiment from scratch!^[[0m
^[[36m[2021-12-14 10:52:37,505][32665] Queried available GPUs: 0,1
^[[0m
^[[36m[2021-12-14 10:52:37,505][17487] Queried available GPUs: 0,1
^[[0m
[INFO] Connected to Unity environment with package version 2.0.0-pre.3 and communication version 1.5.0
[INFO] Connected to Unity environment with package version 2.0.0-pre.3 and communication version 1.5.0
[INFO] Connected new brain: GoalKeeping?team=0
[INFO] Connected new brain: GoalKeeping?team=0
[WARNING] The environment contains multiple observations. You must define allow_multiple_obs=True to receive them all. Otherwise, only the first visual observation (or vector observation ifthere are no visual observations) will be provi$
[WARNING] The environment contains multiple observations. You must define allow_multiple_obs=True to receive them all. Otherwise, only the first visual observation (or vector observation ifthere are no visual observations) will be provi$
/work/grudelpg/envs/multi-sample-factory-env/lib/python3.9/site-packages/gym/logger.py:34: UserWarning: ^[[33mWARN: Box bound precision lowered by casting to float32^[[0m
  warnings.warn(colorize("%s: %s" % ("WARN", msg % args), "yellow"))
/work/grudelpg/envs/multi-sample-factory-env/lib/python3.9/site-packages/gym/logger.py:34: UserWarning: ^[[33mWARN: Box bound precision lowered by casting to float32^[[0m
  warnings.warn(colorize("%s: %s" % ("WARN", msg % args), "yellow"))
^[[37m^[[01m[2021-12-14 10:52:43,084][17487] Using a total of 240 trajectory buffers^[[0m
^[[36m[2021-12-14 10:52:43,085][17487] Allocating shared memory for trajectories^[[0m
^[[37m^[[01m[2021-12-14 10:52:43,085][32665] Using a total of 240 trajectory buffers^[[0m
^[[36m[2021-12-14 10:52:43,085][32665] Allocating shared memory for trajectories^[[0m
^[[37m^[[01m[2021-12-14 10:52:44,738][32665] Initializing learners...^[[0m
^[[37m^[[01m[2021-12-14 10:52:44,739][32665] Initializing the learner 0 for policy 0^[[0m
^[[37m^[[01m[2021-12-14 10:52:44,752][00322] Set environment var CUDA_VISIBLE_DEVICES to '0' for learner process 0^[[0m
^[[37m^[[01m[2021-12-14 10:52:44,770][17487] Initializing learners...^[[0m
^[[37m^[[01m[2021-12-14 10:52:44,771][17487] Initializing the learner 0 for policy 0^[[0m
^[[37m^[[01m[2021-12-14 10:52:44,783][17610] Set environment var CUDA_VISIBLE_DEVICES to '0' for learner process 0^[[0m
^[[36m[2021-12-14 10:52:44,790][00322] Visible devices: 1^[[0m
^[[37m^[[01m[2021-12-14 10:52:44,793][00322] Starting seed is not provided^[[0m
^[[36m[2021-12-14 10:52:44,820][17610] Visible devices: 1^[[0m
^[[37m^[[01m[2021-12-14 10:52:44,822][17610] Starting seed is not provided^[[0m
^[[37m^[[01m[2021-12-14 10:52:45,871][00322] Waiting for the learner to initialize...^[[0m

Currently our models are all initialized randomly. As stated here, torch DDP

performs an all-reduce step on gradients and assumes that they will be modified by the optimizer in all processes in the same way.

We need a method, which initializes the models in a deterministic way (because we don't want full models to be send from one note to another). Methods, which initialize models deterministically are e.g. torch.nn.init.constant_, torch.nn.init.ones_, torch.nn.init.zeros_, torch.nn.init.eye_.  To fix this, we could do one of the following things:

Add an additional option for the parameter --policy_initialization , which initializes models in a deterministic way. This method has the disadvantage, that users of MSF must set this option by hand, when using multiple nodes. This is prone to be forgotten.
Check that --num_policies is greater than 1 and --with_pbt is false in this method. If this is the case, then apply deterministc intitialization.


Compiling in a single binary file, currently there is libnut.node, libnut1.node and libnut2.node

Short summary
Is there a way we can compile in a single binary file, like: libnut.node would be libnut.node, libnut1.node and libnut2.node combined?
Detailed question
We are having an issue while signing our electron app, because libnut1 and libnut2 don't have types they can't be signed.

Hi @augustnmonteiro 👋
these binaries are platform specific, so having one single uniform binary is not possible.
I'm not 100% sure what "they don't have types" means, do you have a repo I can check out?

Hi @s1hofmann
so, libnut.node is the windows one?
can I just delete the other 2?
Our application needs to be signed using signtool.exe and it signs all the binaries that are included in the electron app.
it signes libnut.node fine, but it doesn't sign the other two, and because of that the build breaks

so, libnut.node is the windows one?

One of them for sure.
I don't know how your build process looks like and how binaries are processed, but one of the *.node files is targeting Windows.
The other two are built for macOS and Linux.

@s1hofmann I found a fix, thank you!
If someone else has the same problem this is how I solved it.
the file @nut-tree/libnut/index.js looks like this:
const libnut = (() => {
	switch (process.platform) {
		case 'win32':
			return require(`@nut-tree/libnut-win32`);
		case 'linux':
			return require(`@nut-tree/libnut-linux`);
		case 'darwin':
			return require(`@nut-tree/libnut-darwin`);
	}
})();

module.exports = libnut;

I created a script to remove the darwin and linux version
const fs = require("fs");
const file =<EMAIL_ADDRESS>
let text = fs.readFileSync(file, 'utf-8');

text = text.replace("return require(`@nut-tree/libnut-linux`);", "return;");
text = text.replace("return require(`@nut-tree/libnut-darwin`);", "return;");

fs.writeFileSync(file, text);

I run this right after npm install
In our case we only need the app to work with windows, if you need your app to work with other os you probably need a different solution


How to use this in Swift?

How can I use this library in Swift?

Hi @heshanlk !
I don't have the time to re-write it to Swift at the moment, but you can use a bridging header to use Objective C code in your Swift project.
Look at this answer on SO: http://stackoverflow.com/a/24005242/598812


Revert "feat(stream): add Left + Right, Semi + Anti streaming hash join"

Reverts singularity-data/risingwave#2387
q20 is failing in main CI. I will fix this.

Not necessary now. I disabled tpch q20 and increase timeout. #2424


Fireball not working

I cant get the Fireball Spell to cast, it just says that Something went wrong. Console says its the LargeFireball entity cannot be found. Just wanted to know if this was an issue or Its just an issue i have.

Sorry for the late response, but this was an issue in recent builds but should be fixed now.


just use aSceneReference then error

just use aSceneReference then error
UnityException: Calls to "AssetDatabase.LoadAssetAtPath" are restricted during domain backup. Assets may not be loaded while domain backup is running, as this will change the underlying state.
UnityEditor.AssetDatabase.LoadAssetAtPath[T] (System.String assetPath) (at <599499f170ad48e4856b76baa93bb22f>:0)
ScriptableObjectArchitecture.SceneInfo.get_Scene () (at Assets/ThirdParty/SO Architecture/Variables/SceneVariable.cs:80)
ScriptableObjectArchitecture.SceneInfo.OnBeforeSerialize () (at Assets/ThirdParty/SO Architecture/Variables/SceneVariable.cs:107)

try to change the Scene return in SceneInfo class to this:
    #if UNITY_EDITOR
    private UnityEditor.SceneAsset _cacheScene;
    internal UnityEditor.SceneAsset Scene
    {
        get {
            try
            {
                var scene = UnityEditor.AssetDatabase.LoadAssetAtPath<UnityEditor.SceneAsset>(_sceneName);
                _cacheScene = scene;
                return _cacheScene;
            }
            catch (Exception)
            {
                return _cacheScene;
            }
        }
    }
    #endif


Milestone: mempool and block producer

This issue will be used to trace the dev process of mempool & block producer.
The goal is to run the very first testnet of opentron.🚀
Update 2021/03/01
Since the producer is the most complex part, the milestone is divided into:

[x] executor for all type of transactions
[x] a state-db implementation: use cargo run -- dev to test against it
[ ] refactor channel protocol to handle transaction broadcast
[ ] impl Manager::produceBlock (in manager crate)
[x] impl consensus base(code in base/*.java)

most of the code logic is implemented by manager crate

[ ] impl DPoS mining (logic in dpos/*.java files)

[x] DposSlot  - merge logic into Manager
[x] IncentiveManager
[x] MaintenanceManager
[ ] StateManager
[ ] DposService
[ ] DposTask - rewrite using producer_service crate

To clarify:
The top-level logic of a FullNode is: https://github.com/opentron/opentron/blob/master/docs/txpool.md

RpcApiService(including HTTP/gRPC) is responsible for accepting and validating new transactions(broadcast)
TronNetService is the gossip layer to broadcast a transaction to the whole network
ConsensusService is responsible for producing blocks

Use dposService.getBlockHandle().produce(miner, pTime, timeout) to produce a block.

Thanks @andelf. Merged #59.

Update 2021/03/03
To test the slot timing, now the node generates empty blocks.

Update 2021/03/05
Broadcast channels

Outgoing block broadcast(from block producer)
Outgoing transaction broadcast(from API)
Incoming transaction(from gossip, to mempool)
Incoming block(to consensus handler, handle forks and states)

Update 2021/03/08

massive refactor
impl basic broadcastTransaction graphql API

Update 2021/03/14

Testing against transactions and block producer.
Add a new cli to send transfers.

Update 2021/03/15
Use cli to create witness, vote...
wallet-cli 0.1.0
OpenTron Developers<EMAIL_ADDRESS>Opentron command line util

USAGE:
    cli [FLAGS] [OPTIONS] <SUBCOMMAND>

FLAGS:
    -d, --dont-broadcast    Don't broadcast transaction to the network (just print to stdout)
    -h, --help              Prints help information
    -s, --skip-sign         Skip actual sign process
    -V, --version           Prints version information

OPTIONS:
    -k, --account <account>                 The account address used for signing
    -x, --expiration <expiration>           Set the time in seconds before a transaction expires
        --fee-limit <fee-limit>             Maximum value of TRX allowed consumed
        --memo <memo>                       The memo for the transaction
    -K, --private-key <private-key>         The private key used for signing [env:
                                            OPENTRON_CLI_PRIVATE_KEY=]
        --rpc-url <http://[HOST]:[PORT]>    Raw RPC address to connect [env: OPENTRON_CLI_RPC_URL=]  [default:
                                            http://<IP_ADDRESS>:3000]

SUBCOMMANDS:
    account     Account management
    asset       TRC10 asset management
    contract    Smart Contract
    exchange    On-chain DEX
    market      New On-chain Echange
    proposal    Proposal management for active witnesses
    system      Chain governance and resource management
    transfer    Transfer TRX tokens
    witness     Witness management


Support lxml 3.4.4 (Latest)

Hey there,
I have found this library to be quite useful. Through, its compatibility is limited to 3.3.6 version of lxml. Will see if I can help in any way...
Has there already been issue found between 3.3 and 3.4 ?

Fixed by #8. Thank you a lot for your contribution.