[Question] Role/scope based result in transformers?

While using Fractal, I want to base my output on the given scopes or roles from logged in users. My question is, can this be done inside the transformer? Of course, in my workflow it can be done, but I'm not sure if it follows the 'rules' of Fractal.
Example code would be:
$data = [
    'id' => $entity->get('id'),
    'name' => $entity->get('name')
];

if($this->user('role') === 'moderator') {
    $data['status'] = $entity->get('status');
}

Thanks in advance!

I'm doing the exact same thing.

In my application-scenario I would use different Transformers in my Controller. E.g., there would be a ProductTransformer_Moderator and a ProductTransformer_User.
Things could get a bit nasty if you would output different strcutures depending on Roles and/or Permissions, so it may be easier to just have different Transformers.. What do you think?

Have the same issue... Is there any beautiful solution?

@johannesschobel , I used this approach and mixed in some advanced transformation with includes, which adds some nice extra flexibility to my APIs endpoints.
BUT.. I am have trouble making this work with collection, which contains multiple types of items (models) and therefor should delegate the transformation of each of the items to a different transformer.
To continue with the user example, when including a collection of users with different roles, how do I tell fractal to use a different transformer based on the user's role, and make sure nested includes are passed through?
I have been stuck on this problem for days, and can't find a solution. Thanks for anybody's help!

@yoannisj , what exactly do you mean by

collections, which contains multiple types of items
?

Do you mean something like this:
$data = [
   $book1,
   $author1,
   $book2,
   $car,
   $user1,
   $user2,
   // ...
];

Are you actually mixing different resources in one response? if yes, i don't think that this is the desired behaviour or how you should implement it.. Usually you have one "root" resource (e.g., determined by the URL, /api/books) and the other resources should be "embedded" (e.g., through the includesX methods..
Hope this helps

@johannesschobel, I am building an API for a website which contains listing/index pages, including a list of entries (a.k.a. posts). The endpoint to that page /api/pages/some-listing-page?include=entries returns something like so:
$data = [
    "title" => "Some listing page",
    "url" =>"https://www.example.com/some-listing-page",
    "postDate" => "2019-05-08T12h30"
    "intro" => "Lorem ipsum dolor…",
    "entries" => [
        $newsItem, $article, $article, $event, $newsItem, $article, /* ... */
    ],
    /* ... */
];

The "root" transformer for that page loads the data for the "entries" key, through an includeEntries method, which returns a fractal CollectionResource. But, the data-structure of the entry items in that collection can differ quite a lot from one entry type to another, some having their own nested includes (for example an article entry could respond to "?include=entries.writers", and an event entry could respond to another"?include=entries.location").
The examples I have seen in the documentation, seem to suggest that I can only pass a single transformer class to the collection returned in the page transformer's includeEntries method. Something like:
public function includeEntries( $page )
{
    $entries = new Query('entries')->parentPage($page)->fetch();

    return $this->collection($entries, new EntryTransformer());
}

But that would mean, that the EntryTransformer needs to implement the data transformation logic for all the different entry types, and all the possible nested includes in one single transformer class, which gets messy very quickly.
Ideally, there would be a way to tell Fractal to:

a) use a different transformer class for each type of entry included in the  $entries collection (i.e. NewsEntryTransformer, ArticleEntryTransformer, …)

b) pass the nested include queries from the endpoint url to all the entry transformers (in such way that "?include=entries.writers" would call ArticleEntryTransformer::includeWriters()` for example)

Is that possible, and how can I achieve such behaviour? Maybe the answer is to rely on a common PolyEntryTransformer class, which gets passed to the included collection of entries, and then instanciates a sub-transformer for each entry, while passing in the nested include scope? How would that look like?
Sorry, this starts to look actually more like a support ticket, than an actual issue for the source code. I would be happy to re-iterate my question in a more appropriate channel if you can point me to one.
Thanks a lot for the help!

Seems to be a duplicate of #244


Add support for arbitrary-precision numbers

Using rug

Things to do:

format floats without exponent suffix (https://gitlab.com/tspiteri/rug/-/issues/18)
format floats as ints if we can convert them

ptal @lukebhan

Note: this would fail on "100/0" for example. We can fix it in a later pass.


SQLAlchemy mapper error after latest migration

Server returns HTTP 500 after latest deployment.
Bug found during local testing.
Issue with missing FK in Issues table

Fixed with 250f82e237b89c72fbcfeb2c69393416fa832a64


websocket communication error / Disconnected

Note: for support questions, please join our Discord server
My setup is:
gekko installed on a vps, nginx, lets encrypt and basic authentication (using the detailed description on the website). The setup was working fine with version 0.5.6, but does not work anymore with 0.5.7.

I'm submitting a ...
[X] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Action taken (what you did)
I updated from version 0.5.6 to 0.5.7 (I also tried a clean "install" with 0.5.7 with the same effect).

Expected result (what you hoped would happen)
Gekko should be accessible via the webinterface.

Actual result (unexpected outcome)
Gekko shows the "Disconnected"-message and cannot be used at all (Testet with Chrome and Safari).

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)

I get two messages. First on the console on startup via 'node gekko --config config.js --ui':
(node:30763) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Exited with code 3
(node:30763) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

And the second in the js console of the browser:
WebSocket connection to 'ws://localhost:3000/api/' failed: Error in connection establishment: net::ERR_CONNECTION_REFUSED
OPTIONS http://localhost:3000/api/imports net::ERR_CONNECTION_REFUSED
OPTIONS http://localhost:3000/api/gekkos net::ERR_CONNECTION_REFUSED
OPTIONS http://localhost:3000/api/gekkos net::ERR_CONNECTION_REFUSED
OPTIONS http://localhost:3000/api/apiKeys net::ERR_CONNECTION_REFUSED
OPTIONS http://localhost:3000/api/exchanges net::ERR_CONNECTION_REFUSED
Uncaught TypeError: Cannot read property 'forEach' of undefined

and after that a lot more:
WebSocket connection to 'ws://localhost:3000/api/' failed: Error in connection establishment: net::ERR_CONNECTION_REFUSED

@askmike Did you change something regarding the websocket communication? Maybe something only affecting the hosted / nginx setup?

I think both issues can be fixed by properly configuring the UIconfig.js:
the first warning says that Gekko could not open your browser (which is
okay since a VPS does not have one).
The browser errors indicate that your UIconfig is configured so that it
tries to connect to localhost:3000 to access the server (but as you stated
you are running it externally on a VPS).
For both issues: please see the installing Gekko on a server doc page.
On 20 Sep 2017 11:34, "1024kilobyte"<EMAIL_ADDRESS>wrote:

Note: for support questions, please join our Discord server
https://discord.gg/26wMygt
My setup is:
gekko installed on a vps, nginx, lets encrypt and basic authentication
(using the detailed description on the website). The setup was working fine
with version 0.5.6, but does not work anymore with 0.5.7.

I'm submitting a ...
[X] bug report
[ ] feature request
[ ] question about the decisions made in the repository
Action taken (what you did)
I updated from version 0.5.6 to 0.5.7 (I also tried a clean "install"
with 0.5.7 with the same effect).
Expected result (what you hoped would happen)
Gekko should be accessible via the webinterface.
Actual result (unexpected outcome)
Gekko shows the "Disconnected"-message and cannot be used at all
(Testet with Chrome and Safari).
Other information (e.g. detailed explanation, stacktraces, related
issues, suggestions how to fix, links for us to have context, eg.
stackoverflow, etc)
I get two messages. First on the console on startup via 'node gekko
--config config.js --ui':
(node:30763) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Exited with code 3
(node:30763) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
And the second in the js console of the browser:
WebSocket connection to 'ws://localhost:3000/api/' failed: Error in connection establishment: net::ERR_CONNECTION_REFUSED
OPTIONS http://localhost:3000/api/imports net::ERR_CONNECTION_REFUSED
OPTIONS http://localhost:3000/api/gekkos net::ERR_CONNECTION_REFUSED
OPTIONS http://localhost:3000/api/gekkos net::ERR_CONNECTION_REFUSED
OPTIONS http://localhost:3000/api/apiKeys net::ERR_CONNECTION_REFUSED
OPTIONS http://localhost:3000/api/exchanges net::ERR_CONNECTION_REFUSED
Uncaught TypeError: Cannot read property 'forEach' of undefined
and after that a lot more:
WebSocket connection to 'ws://localhost:3000/api/' failed: Error in connection establishment: net::ERR_CONNECTION_REFUSED
@askmike https://github.com/askmike Did you change something regarding
the websocket communication? Maybe something only affecting the hosted /
nginx setup?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/1138, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA7MDznlGRZcMTw_-AFwrjV0PPLln0Zqks5skNwKgaJpZM4PdmG8
.

Sorry for that. I saved and restored all settings from 0.5.6 EXCEPT the UIconfig. My bad!
Thanks for the quick reply!


g

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture script
You can also obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the current behavior
Describe the expected behavior
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Closing this issue due to lack of information to work with. Please update the issue with all the information asked by the template and we will reopen it. Thanks!


what hell is going on ?

Environment Versions

OS Type

Node version: $ node --version

http-server version: $ http-server --version
I can`t use http-server --version!!!

Steps to reproduce

$ http-server
i got this!

also i can`t use old!

I'm getting garbage as well on macOS (using Alacritty) and node v16.13.1 (LTS)

Same results in Terminal. This is using a new M1 Pro.

Zooming in...there are words in here like "TESTING"...I highlighted the word in the middle.

Same with 13.0.2 it's likely one of the dependencies

Before you report an issue you may want to check it's already reported or addressed by a PR. See #781, #782, #783, #784, #785, #786

Oh! thanks guys! I`m reading for #294 , and looking for how to fix it. I will close this issues. @Soni96pl @zbynek

Thanks for the heads-up. Just saw the article https://www.bleepingcomputer.com/news/security/dev-corrupts-npm-libs-colors-and-faker-breaking-thousands-of-apps/


barcodes detector

Are there any current effort or plans to implement the interface to the barcodes detector?

Hey @eboily
You can use this plugin for barcode scanning.
I would prefer not to make this plugin too bulky by implementing functionality that already exists.
Sorry I could not be of more help.
Cheers,
Chris
Neutrinos


Add new tests for alerting dashboards

Description
Add new tests for alerting dashboards for composite workflows and monitor dashboards
Issues Resolved
N/A
Check List

[x] 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.

Please add backport 2.x and backport 2.10 labels

Locally this was passing without security. Will try setup with security and see if it passes locally

Apologies for it being really manual but if you add them here, you can utilize: https://opensearch-project.github.io/opensearch-dashboards-functional-test/site/ to discover the test results


Restrict the preview1 implementation of fd_read to one iovec

In the wasi-common implementation of fd_read for preview1, switch to only reading the first iovec given in a vectored read. This avoids issues with Wiggle's runtime borrow-checker implemented in #8277, which would cause the reads to unconditionally fail when two mutable borrows were taken out on the same memory.
While it's not as efficient to perform a single read at a time, it is also valid, as read never promises to read all of the vectors provided.
Co-authored-by: Nick Fitzgerald<EMAIL_ADDRESS>

I was initially confused reading over this trying to piece together how this wasn't covered by https://github.com/bytecodealliance/wasmtime/pull/8353 but that was fd_pread rather than fd_read.
Mind adding a test for this? Also, can you update the filter here to be the same as https://github.com/bytecodealliance/wasmtime/pull/8353 where it searches for the first non-empty buffer rather than the first buffer?

I added a test, and verified that it failed without the fix to fd_read. (It's the same as the pread/pwrite test, but using fd_seek to polyfill their behavior.)


add webpage required files

Codecov Report

Merging #70 (0329483) into dev (484a6d1) will not change coverage.
The diff coverage is 85.71%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##              dev      #70   +/-   ##
=======================================
  Coverage   59.81%   59.81%
=======================================
  Files           8        8
  Lines         321      321
=======================================
  Hits          192      192
  Misses        129      129

Flag
Coverage Δ

unittests
59.81% <85.71%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

spacebench/datamaster.py
100.00% <ø> (ø)

spacebench/eval.py
17.85% <ø> (ø)

spacebench/env.py
74.13% <85.71%> (ø)


Fixed emote modifier split

This fixes emotes with ':' in their names (for plugins adding custom emotes).
However, it is still not possible to apply a modifier for emotes starting and ending with a ':' (":hap::spin"), for discord put ":hap:" and "spin" in a different html tag.

Added in 1.59


[FEATURE] Typings

In raising this pull request, I confirm the following (please check boxes):

[x] I have read and understood the contributors guide.
[x] I have checked that another pull request for this purpose does not exist.
[x] I have considered, and confirmed that this submission will be valuable to others.
[x] I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
[x] I give this submission freely, and claim no ownership to its content.

My familiarity with the project is as follows (check one):

[ ] I have never used the project.
[ ] I have used the project briefly.
[ ] I have used the project extensively, but have not contributed previously.
[x] I am an active contributor to the project.

Added typings to two functions.

Make sure that tests are passing

rip ded :(


Post via Postman response 401

Hi,
i try your example with cake 4.2.4 and running the server.
When i send the post request via Postmann with the user credentials in the body as raw or form-data, i get 401 back.
Any sugesstions to fix this?
Thanks for the example and greetings

Can you please share some screenshots or something?


Python extension squashes notebooks to single line

VSCode Version: 1.39.1
OS Version: Ubuntu

Steps to Reproduce:

Create a notebook in Jupyter / JupyterLab
Open the notebook in VS Code
Save the notebook

After creating the notebook in Jupyter, it has the following formatting:
{
 "cells": [
 {
   "cell_type":
      ...

after editing and saving in VS code, it gets squashed into long single line:
{"cells": [{"cell_type": ...

Expected outcome: keep the initial formatting. This increases readability when using code diff tools (e.g. git diff).

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

It must be an extension that causes this. The built-in JSON formatter does not put everything on a line. Maybe it's another formatter?
Note that you can enable format on save using this:
"[json]": { "editor.formatOnSave": false } (just for JSON if that's what you want)

Please reopen if this is not caused by an extension.


Support Accepted stage on UpdateWorkflowExecution

Is your feature request related to a problem? Please describe.
Currently, you can only synchronously wait on update result when calling UpdateWorkflowExecution.
There are many cases where you want the update to start long running async operations and you don't want to return from UpdateWorkflowExecution as soon as update is accepted.

Thank you for your feature request!
We understand the need for asynchronous waiting on UpdateWorkflowExecution results.  Currently, Temporal does not support this directly.
As a workaround, you can consider using a separate workflow to handle the long-running operations. You can trigger this workflow from the UpdateWorkflowExecution call and poll its status asynchronously.
Let us know if you have any further questions.

Thank you for the feature request! We understand the need for asynchronous behavior when updating workflow executions.
Currently, UpdateWorkflowExecution is synchronous and returns as soon as the update is accepted. This means that any long-running operations started within the UpdateWorkflowExecution call will not be able to complete before the call returns.
As a workaround, you can consider using a separate workflow to handle the long-running operations. You can trigger this workflow from within your UpdateWorkflowExecution call and then return immediately. This will allow your UpdateWorkflowExecution call to complete quickly, while still allowing the long-running operations to run asynchronously in the background.
We appreciate your suggestion and will consider it for future feature development.

Thank you for your feature request! This is a great idea, and we appreciate you bringing it to our attention.
Currently, Temporal doesn't directly support asynchronous waiting on UpdateWorkflowExecution results. However, you can achieve a similar effect by using a combination of:

Workflows:  Create a dedicated workflow that performs the long-running operations you want to start after the update.
Signals: Signal this workflow from within your UpdateWorkflowExecution handler, passing the necessary information for the long-running operations.

This approach allows you to return from UpdateWorkflowExecution quickly, while still initiating the desired asynchronous actions.
We hope this workaround helps! We're always looking for feedback on ways to improve Temporal.

Thank you for your feature request!  We understand the need for asynchronous behavior with UpdateWorkflowExecution.
Currently, UpdateWorkflowExecution is synchronous, meaning the operation must complete before the workflow can proceed.
Here's a possible workaround:

Use a separate workflow to perform the long-running asynchronous operation after the UpdateWorkflowExecution call. This workflow can be initiated by a signal sent after the UpdateWorkflowExecution is completed.

For more information on signal workflows, please refer to the Temporal documentation: https://docs.temporal.io/docs/typescript/workflows/signals


Please confirm that you have:

[X] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
Theme
Expected behavior
The CLI should have been upgrade to CLI3.X however, when I input my upgrade comman, I get an error the "Missing script: 'shopify'"
I have quoted steps that I followed in the CLI below
`→ C:\Users\User› shopify version

A new version of Shopify CLI is available! You have version 2.16.1 and the latest version is 2.36.0.
To upgrade, follow the instructions for the package manager you’re using:
https://shopify.dev/tools/cli/troubleshooting#upgrade-shopify-cli

2.16.1
→ C:\Users\User› npm run shopify upgrade
npm ERR! Missing script: "shopify"
npm ERR!
npm ERR! To see a list of scripts, run:
npm ERR!   npm run
npm ERR! A complete log of this run can be found in: C:\Users\User\AppData\Local\npm-cache_logs\2023-08-20T10_14_41_363Z-debug-0.log
→ C:\Users\User› npm install -g @shopify/cli @shopify/theme
changed 887 packages in 41s
104 packages are looking for funding
run npm fund for details
→ C:\Users\User› shopify version

A new version of Shopify CLI is available! You have version 2.16.1 and the latest version is 2.36.0.
To upgrade, follow the instructions for the package manager you’re using:
https://shopify.dev/tools/cli/troubleshooting#upgrade-shopify-cli

2.16.1
→ C:\Users\User› npm run shopify upgrade
npm ERR! Missing script: "shopify"
npm ERR!
npm ERR! To see a list of scripts, run:
npm ERR!   npm run
npm ERR! A complete log of this run can be found in: C:\Users\User\AppData\Local\npm-cache_logs\2023-08-20T10_17_22_279Z-debug-0.log
→ C:\Users\User›`
Actual behavior
The CLI should be upgraded to 3.X as these same commands/steps to upgrade works on my friends laptop.
Reproduction steps
`→ C:\Users\User› shopify version

A new version of Shopify CLI is available! You have version 2.16.1 and the latest version is 2.36.0.
To upgrade, follow the instructions for the package manager you’re using:
https://shopify.dev/tools/cli/troubleshooting#upgrade-shopify-cli

2.16.1
→ C:\Users\User› npm run shopify upgrade
npm ERR! Missing script: "shopify"
npm ERR!
npm ERR! To see a list of scripts, run:
npm ERR!   npm run
npm ERR! A complete log of this run can be found in: C:\Users\User\AppData\Local\npm-cache_logs\2023-08-20T10_14_41_363Z-debug-0.log
→ C:\Users\User› npm install -g @shopify/cli @shopify/theme
changed 887 packages in 41s
104 packages are looking for funding
run npm fund for details
→ C:\Users\User› shopify version

A new version of Shopify CLI is available! You have version 2.16.1 and the latest version is 2.36.0.
To upgrade, follow the instructions for the package manager you’re using:
https://shopify.dev/tools/cli/troubleshooting#upgrade-shopify-cli

2.16.1
→ C:\Users\User› npm run shopify upgrade
npm ERR! Missing script: "shopify"
npm ERR!
npm ERR! To see a list of scripts, run:
npm ERR!   npm run
npm ERR! A complete log of this run can be found in: C:\Users\User\AppData\Local\npm-cache_logs\2023-08-20T10_17_22_279Z-debug-0.log
→ C:\Users\User›`
Operating System
Windows10
Shopify CLI version (check your project's package.json if you're not sure)
2.16.1
Shell
Powershell
Node version (run node -v if you're not sure)
18.17.1
What language and version are you using in your application?
Node 18.17.1

Hi! I think there is a bit of confusion between cli versions and how to upgrade.

Your original CLI installation is version 2.x (which is deprecated)
The upgrade docs you link are for apps, not for themes. This is the one for themes
But what you actually need is to migrate to CLI 3.x (not just upgrade)
These are the docs you should follow to migrate to CLI 3.x

Basically you run npm install -g @shopify/cli @shopify/theme to install CLI 3.x for themes.
Beware of having CLI 2.x and 3.x installed at the same time, they might conflict, you should uninstall CLI 2.x first.
Hope this helps, i'm going to close this issue as this is not a bug on the CLI. But feel free to ask any question about migration process.

Hi! I think there is a bit of confusion between cli versions and how to upgrade.

Your original CLI installation is version 2.x (which is deprecated)
The upgrade docs you link are for apps, not for themes. This is the one for themes
But what you actually need is to migrate to CLI 3.x (not just upgrade)
These are the docs you should follow to migrate to CLI 3.x

Basically you run npm install -g @shopify/cli @shopify/theme to install CLI 3.x for themes.
Beware of having CLI 2.x and 3.x installed at the same time, they might conflict, you should uninstall CLI 2.x first.
Hope this helps, i'm going to close this issue as this is not a bug on the CLI. But feel free to ask any question about migration process.

Thank you very much @isaacroldan. It worked.


[ERROR] Born effective charge not Hermitian

Hi,
I was trying to include LOTO to phonon in extract_forceconstant. The symmetry of the material I am studying is low, and the Born effective charge tensor is not symmetryic. TDEP showed the following error:
... Born charges not Hermitian, error:   8.2344767095669359E-007
... DISPERSIONS MIGHT BE IRREGULAR!
... FIXME!
This error causes incorrect phonon dispersion.
But some previous discussion indicates that the Born effective charge needs not be Hermitian. http://www.democritos.it/pipermail/pw_forum/2006-January/003511.html
Could you resolve this issue? The log files can be found in the zip file attached.
extract_forceconstants_rc5_skip_400_loto.zip

Hi @kli103 ,
right now, the only reliable workaround is to use tdep_simplify_bec from https://github.com/flokno/tools.tdep via
tdep_simplify_bec infile.lotosplitting

and use the resulting outfile.lotosplitting. This will symmetrize the BEC and make them effectively scalar. Unfortunately there is currently no better solution.


Unnecessary checks

The check() function runs everytime I register a new selector.
I see that's needed to detect for elements already present at the page, but why do you need to run through all listeners, instead of just the selector being registered? Running through the other listeners seems to be unnecessary, and could have a noticeable perform impact as the listeners list grows.

I made the change. Thanks for the heads up!


Why do you not maintain these popular projects anymore?

@harthur  I was looking at your profile and saw this:

Why do you not maintain these projects anymore? If you don't have time to do that, why do you not search somebody to keep them alive?

@harthur - If you would like to move this repo into an organization, I would be happy to maintain it.

@nickpoorman I was going to suggest that. :smile: :+1:

That would be great. I am using this software in a project thesis because it is just the simplest solution out here :)

@harthur any response?

:disappointed:

how about forking the repos and creating an organisation for maintenance during her absence?

I like that idea @nickpoorman :+1:

@FranzSkuffka I am not for forking and maintaining. It's much harder to maintain your own fork because most of the people will find this popular repository.
I also vote for moving these projects in an organization, like @nickpoorman suggested. :dizzy:

I also agree, it would be nice to get an response from @harthur

She have switched jobs to something secret so maybe she isn't even allowed to reply here: https://harthur.wordpress.com

@irony Oh, interesting read! :book:

Any update on this, guys? It would be awesome if this would be moved into an organization.

I just emailed @harthur! Hope she will reply me and save these nice goodies. :grin: :gift: :pray:

Thanks @IonicaBizau let us know how it goes.

I would also like this!

Any update on this?

No response from the author yet... :disappointed:

There's an opportunity here for someone to take this over. Preferably someone who understands the codebase and is willing to move the project forward.

Yes but as @IonicaBizau said that's not easy to fork and maintain because of popularity of THIS repository.

Fair point - I agree that moving this project to an org would be the best solution - it's a shame the owner isn't responding the messages. :disappointed:

@michaelpittino I contacted GitHub support–maybe they will be able to help us since @harthur had a talk at Passion Projects. Hopefully they will have a solution. :pray:

From GitHub support:

Unfortunately we don't have any contact details for Heather Arthur from her Passion Projects appearance. Hopefully she'll reply to your email if she's able to help.

Let's start a petition at change.org. :-)

yeah, I think 50 people will sign, but I'm down.

Why not just create a fork, move development there, keep an eye on this repo and point people to the fork whenever they open issues or PR:s here and then if/when the owner of this repo gets back to you then this repo can be updated to point to that repo.

+1

I also agree with @voxpelli

I did contact @harthur about being a maintainer, no reply.  I did read her website, and why she ended up doing so, and totally respect it.  I would really like to continue in her absence, hoping the best for her.  There is a risk that she doesn't like any of the changes we do, but there is also the chance we may end up improving these fantastic libraries.  That being said, there are enough of us and these libraries are useful enough I went ahead and created: https://github.com/harthur-org and forked all libraries.  If you'd like access, open a pull request.

@robertleeplummerjr That's cool! :+1:
I would still recommend having real source repositories, not forks (that means: delete forks, create repos, git push --all from local clones of @harthur's repos). I do not like forks because they have limitations like:

they are not searchable
commits in forks are not counted in the contributions calendar (big thing, right? :joy:)

Instead we can explicitly state it's the repos are a continuation of the original projects.

I agree with @IonicaBizau, that makes perfect sense.

@ilanbiala, What if @harthur decides to re-maintain?  We open a single pull request, and merge, and then the org can go away.
This just happened: https://github.com/harthur-org/brain/commit/1668939d7f73c22afa85d0957a189045ad6e940c

@robertleeplummerjr Note we can always make pull requests between source repositories (not only between forks)—I still think we should convert them in source repositories, not forking. :grin:

Give me a few, I'll work it out.

Note we can always make pull requests between source repositories

Well, actually that's wrong: I mean, we can synchronize the source repositories anyways: when the original owner is back and wants to maintain the source repo, we fork it, synchronize the fork (git pull ...) and then creating a PR on GitHub. :grin:

I get what you intended to say.  I do it all the time for private repos based off of public ones.  Don't know why I didn't think about it, probably an A.D.D. moment.

Done for brain, and: https://github.com/harthur-org/brain/commit/388e5ea1840a0659156d560f3bb411acdf1ed535

@robertleeplummerjr :cool: We now can start merging these PRs from this very repository in the @harthur-org account (by git pulling). :sparkles:

@IonicaBizau I'd like to make you a fellow maintainer since you started this issue.  Seems only right.  Just pushed unit tests for the likely method.

@robertleeplummerjr Thanks! :+1: Happy to help in my spare time. Not sure, but I may use this module in one of projects. :smile:

:fist: :fire:

Added you to the "with great power comes great responsibility" team.

@robertleeplummerjr Thanks! I'm not evil. :four_leaf_clover:

I know, the name of the team is a reminder for me :stuck_out_tongue:

:joy:

I love that the community has started maintaining this project. One reason this project is so useful to people, is the simplicity of the code. There's no underlying BLAS library being called to do "magic" behind the scenes. In the spirt of this, I hope when these pull requests start being merged you keep that in mind, and document any new features added. Such as the "likely", which in my opinion should be an a separate util lib and not in the lib/neuralnetwork.js source (especially since convolutional neural networks are better suited for that task).

@nickpoorman I agree! :+1: Maybe for such things, open issues in the new repository. It's easier to track. :smile:

Well stated @nickpoorman.

How would you guys feel about a :+1: policy?  Where at least one person that has contributed and that is not the committer must :+1: the PR moving forward.  When the :+1: is landed, we then can merge the PR.

Obviously harthur is the maintainer of the npm packages aswell. I have had experience with the npm removing coffeescript (which was not coffee-script) upon my request, because it was a confusing, unnecessarily publicated package.
We could either a.) ask at npmjs.org for permission to be added as package maintainers or b.) publish the packages with a prefix like harthur-org-brain.

@robertleeplummerjr I agree with you on establishing a consensus-based policy. However I would extend the pull-request cycle by 2 days for example after the :+1:. During these two days anyone can throw in a veto, delay the merge by another day when a valid argument was given.

@FranzSkuffka I always had good experiences with the guys from npm support—they transferred quite few packages on my account (mostly they were unmaintained/obsolete/bad quality/etc packages), but always they did a great job. I'd say we can give a try to ask them. Then we will get some traffic from the npm package. :joy:

@IonicaBizau agreed, would be great to get some traffic from there!
btw, WTF did you do to your commit graph :laughing:

when one has the capability to rewrite git history...

@FranzSkuffka Check this out. :joy:
Pleeeease, without the f word!

ask at npmjs.org for permission to be added as package maintainers

I contacted npm support and they did talk to @harthur. She wants to keep the npm packages in this state, so, we need to apply plan b): to publish the packages under different names. :disappointed:
/cc @FranzSkuffka

Ok. Let's think of a good prefix for the package names.
Sample:
contrib-brain
community-brain
com-brain
...

Ill throw out:
js-brain
org-brain

Total bike-shedding here, but I feel obligated to mention a few common fork naming conventions:
brain-ng
brain-plus

Cool - I didn't know there was a naming convention for it.
Where is the easiest place to start an online poll?

@ghpabs You could try http://strawpoll.me/

My ideas are:

brain.js
brain2

:grin:

Thanks @SoullessWaffle.
See poll here.
Ps. It's okay if you guys decide to add more options to the poll though.

Results as of Apr 8, 2016, 10:43 AM GMT+1

brain.js 10 votes (45%)
js-brain 4 votes (18%)
brain-plus 2 votes (9%)
brain2 2 votes (9%)
contrib-brain 1 vote (5%)
community-brain 1 vote (5%)
org-brain 1 vote (5%)
brain-ng 1 vote (5%)
com-brain 0 votes (0%)

Here's two more ideas:
brain-next
brain++ (as in c++; brain incremented by one)

@SoullessWaffle I don't think + can be used in the package names.

harthur-brain could be a good choice SEO-wise

@IonicaBizau you're right, perhaps we could use brainpp

Can we decide on the date by which everyone should vote.

The poll has been open for just over a week now.
Updated results:

brain.js 13 votes (46%)
js-brain 4 votes (14%)
brain2 4 votes (14%)
community-brain 2 votes (7%)
brain-plus 2 votes (7%)
contrib-brain 1 vote (4%)
org-brain 1 vote (4%)
brain-ng 1 vote (4%)
com-brain 0 votes (0%)

28 total votes
@IonicaBizau I guess you could call brain.js the winner?

I npm published it under the brain.js name on npm. :rocket:
If any of you wants to have superpowers on npm, feel free to :bell: me and I will npm owner add you. :grin:  Also, if you want to contribute to the project we can add you as member in the @harthur-org organization.
/cc https://github.com/harthur-org/brain.js/issues/4

Thank you @IonicaBizau

Devs, I believe a "huzza" is in order.

Huzza!!!!

Great job guys. ;)

In the past I have used Brain.js and some other JavaScript based machine learning libraries but unfortunately I have found them not matching my needs. That's why, working on my personal projects, I have developed the idea of creating a different approach by myself. So, sorry to be here to talk about another project but I would really like to receive some opinions and suggestions from experienced people that, being into this specific field, could help me to set useful and shared expectations. By the way, thanks for the great support given to Brain.js so far.

Can you please open a new issue, that way this issue stays focused?

https://medium.com/@heatherarthur/what-happens-when-you-retire-from-github-979fb6b143b3
Basically, he writes: This is hard work for him, A one with limited resources to maintain his repositories for others. He is overwhelmed by a large number of people contacting him. He has no resources for this. He dislikes this. So he quit GitHub. he  "declared bankruptcy" on time devoted to Github, and He regrets sharing his repositories because it took so much time from his life. He expected people would fork and the code would be maintained by users themselves. but many good wishing people fall into his open door smashing him because they are just oo much for him.

expected people would fork and the code would be maintained by users themselves

And that is exactly what happened.

what happened to @harthur ? lol

She's alive and well - machine-learning-engineering. I think there was more than one issue motivating her to abandon github. But I don't want to seed any rumors here. You better google any information up yourself.

https://medium.com/@heatherarthur/what-happens-when-you-retire-from-github-979fb6b143b3

@FranzSkuffka But her profile says deceased https://github.com/harthur what happened to her. It saddens me to see a github profile like that.

Wow - that must be very recent. The last post on medium was Feb 24.

I still understand why she left the Github, someone can explain?

Why do you care?
On Mon, Mar 26, 2018, 10:20 Juan<EMAIL_ADDRESS>wrote:

I still understand why she left the Github, someone can explain?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/harthur/brain/issues/72#issuecomment-376182507, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHw_2fezhbAWXBSQyl39iCoX1QjSRLQuks5tiPkjgaJpZM4GN3uJ
.

@AlibekJ my English is not so good, but here we go, I care because she's contribute a lot with the community and she just disappeared, why? Some problem with her career, or personal? Is it something we can help with? Is just empathy with the next!

https://harthur.wordpress.com/2013/01/24/771/

This is one of the most fascinating issues I have ever scrolled through. I love coders and the code community. Way to go for pulling together, people!

Exactly my thought.
I'm surprised in why she didn't set maintainer permissions for those who contacted about that but at least BrainJS is a community thing.

Consider this: https://github.com/rejuvenate/rejuvenate


CosmosDB Emulator AllowNetworkAccess disables Data Explorer

Describe the bug
Following the official docs located here It seems to disable the data explorer. We are seeing that changing the key doesnt appear to change the key on the data explorer and so all the request are failing with authentication errors. A work around appears to be to start cosmos with the /Key equal to the default key. That appears to allow it t
To Reproduce
Change /AllowNetworkAccess and /Key then try opening data explorer
Expected behavior
That data explorer will open and allow you to view data
Actual behavior
Data explorer opens and gives authentication errors
Environment summary
SDK Version: <IP_ADDRESS>
OS Version Windows

@Arcalise08 This repo is for the V3 .NET SDK and does not handle Emulator related problems. Please follow the steps at https://docs.microsoft.com/azure/cosmos-db/troubleshoot-local-emulator and you can file a support ticket if needed as instructed on that guide.


Profile view bookmarked content doesn't show filled in bookmarks

and likely authored content isn't showing bookmarks as well.

There's something weird going on with authored content too (same case shows up multiple times)

Fixed in PR #533


ruamel.yaml et al.

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (d904d2e) 92.25% compared to head (1b76dec) 92.25%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #90   +/-   ##
=======================================
  Coverage   92.25%   92.25%
=======================================
  Files          47       47
  Lines        8908     8908
=======================================
  Hits         8218     8218
  Misses        690      690

Impacted Files
Coverage Δ

faucet/__main__.py
85.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add Roman-Rubtsov-source's file

I've added new file,  propose to consider this and add file to main branch

It's my first merge pull request


Bug: Markdown mode with multiline code-block = invalid line numbers indicator

expected behavior

bug

solution: uncomment line 100

Line numbers are a known issue and we have open issue to refactor the code block, in general, including fixing the line numbers. Closing this as a duplicate.


Added Jetbrains to Company list

Hi @mattjegan,please review my PR

We aim for making this list more awesome. Can you provide why the participation of JetBrains is awesome?
Please also make your contribution even more awesome by spelling "JetBrains" correctly.

JetBrains supports non-commercial open source projects by providing core project contributors with a set of best-in-class developer tools free of charge.
It is built on the IntelliJ Platform and includes its sources. JetBrains released both under Apache License 2.0.In 2010, Android support became a part of the Community Edition,and two years later Google announced its Android Studio, the IDE for mobile development on Android platform built on the Community Edition of IntelliJ IDEA and an official alternative to Eclipse Android Developer Tool.In June 2015, it was announced that the support of Eclipse ADT would be discontinued making Android Studio the official tool for Android App development.

Thank you for the text proposal. The first sentence is a copy'n'paste from https://www.jetbrains.com/community/opensource/.
I would like to one sentence after the JetBrains entry.
Maybe:

JetBrains offers a wide range of developer IDEs. For Hacktoberfest, it makes them available for free for open source contributors independently of the contributor being a permanent contributor to an open source project (which is covered by their Free Licenses Program)

@mattjegan,I did copy 'n' past so that user come to know exactly what JetBrains says about it to the world and i didn't want anyone to mis-interpret my words in any manner.
Thank you :)

So where was i supposed to add about JetBrains?I didn't found any suitable file or directory to do so!

I fixed it by myself and committed. Thank you for the proposal. This is a good start for hacktoberfest. Please contribute to other larger OSS projects (being used by many people).


MCedit <IP_ADDRESS> Win64 WONT START...

When I run it, it opens cmd that obliges me to press enter, when I press enter, another cmd window opens and it closes.
when i open mcedit.exe :

after pressing enter:

please help

Could you give us your system specs?

I suspect a wrong OpengGL version, or a wrong path for MCEdit installation.
AppData folder is generally not a good place to install programs. Even administrators can't read/write files easely in there.
In addition of sending your system specs, can you copy mcedit folder elsewhere (eg. C:\Users\James) and reopen the program?

According to http://newsgroups.derkeiler.com/Archive/Comp/comp.graphics.api.opengl/2007-03/msg00109.html it seems to have been introduced in OpenGL 1.4.

Since there's been no response for 12 days, going to assume either solved or attempted to run below system requirements.


No package autocomplete for NPM dev dependency

I installed a package in my NPM dev dependencies, but when I start writing the name of a method in the package, no autocomplete for that method is shown. "Quick Fix" also fails to suggest importing the method.
If I move the package to the dependencies, and reinstall the modules, to update package-lock.json, VSCode will then correctly find the package in the autocomplete and "quick fix" menu

Please provide specific steps to reproduce the issue

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


fix: tailwind intelliSense on vscode

Why
just add a tailwind config to support tailwind css intelliSense on vscode.

I can confirm that adding this to the root fixed the issue for me!


Misc fixes

Description:
Just a couple of minor updates to improve readability:

Remove unused Teardown import.

Merge __tryOrUnSub and __tryOrSetError in a single method called _try that's easier to understand
and eliminates repetition.

Remove the redundant !this.isStopped checks inside the SafeSubscriber's
handlers, the parent Subscriber is already doing that for us.

Remove the observerOrNext !== emptyObserver check inside the SafeSubscriber constructor, the marginal (if any) performance gain doesn't justify the cognitive overhead of a three-layered conditional.

Change the format of arguments to align in the same indentation as the methods bodies, this way the eye doesn't have to jump around diagonally between arguments and the body and has the collateral benefit or reducing line length.

Thanks for the effort you've put into this. However, IMO, these changes cannot be merged. Readability is not the most important metric for the code in this particular file. The code in that file was recently reviewed and changed by Benedikt Meurer with a view to improving performance.
And, on a general note, if some portion of the PR addresses only reformatting, those changes should be separated into individual commits - to make reviewing easier.
ATM, I am reluctant to review this. In future, you might want to open an issue check whether or not the changes you are proposing are the sorts of changes that are likely to be merged.
Again, thanks for the effort you've put into this. I hope this won't discourage you from contributing.

Understandable, although these changes are related to form only, the code is functionally the same except for the case of emptyObserver in the SafeSubscriber constructor, which in itself shouldn't even be there since SafeSubscriber never receives an emptyObserver in the first place, and even if a user went through the hassle of importing it from the Subscriber file, it wouldn't make sense to do so since observers are automatically converted to instances of SafeSubscriber internally.
This file in particular is filled with defensive checks like this one that made it really hard for me as an outsider to even understand what exactly are the cases it covers, even though the underlying logic is really simple; so, would you reconsider reviewing the proposed _try method if I break up the reasoning behind it like I just explained the reasons behind the removal of the emptyObserver case?

Regarding the use of arguments.length, I would be reluctant of accept a change of that nature, as the effected behaviour will be essentially unchanged. And the performance impact of using arguments.length is something I would need to research. Although, I suspect others would already know the answer to this.
I don't want to discourage you from contributing; I want the opposite, really. And I'm pleased to see that you have taken such an interested in the inner workings of RxJS.
I do have some suggestions:

Please separate any changes into individual PRs - unless the changes are directly related.
Doing so makes reviewing and merging much simpler. I would rather see half a dozen small PRs than one large one. It's entirely possible that some changes might be welcomed and others not.

Ideally, PRs should be accompanied by issues. Opening an issue first provides the opportunity to discuss the bug that has been found or the changes that are being proposed. And talking about it first is generally preferable. It can avoid work being done on a change that is unlikely to be merged and it avoids the situation in which a maintainer has to tell a contributor that the change they have proposed isn't wanted. No one likes doing that.

The other thing that I am considering - that you might or might not be aware of - is that the Subscriber implementation is going to be changing. So the code for which you are proposing changes is not going to be around in the long term.

I see, thanks for the feedback I appreciate it, and yes, I read about MutabeSubscriber in your Rxjs 8 design document but still didn't want to miss the chance of discussing the current implementation with the maintainers, anyway, I'll open up an issue to discuss other aspects about the current Subscriber that are related to behavior.


Network Protocol Error with End Crystal

With the ExplosiveFeature enabled, if you try to place an End Crystal on a block of Obsidian or Bedrock, you get kicked for "Network Protocol Error" and can't join the server back unless you restart it.
I also tried this on a simple Minestom server with minimal to none configuration and it happens too.
I momentarily found a work around to be able to still use tnt and respawn anchor features: I rewrote the VanillaExplosiveFeature class and remove the PlayerUseItemOnBlockEvent listener and everything seems to work fine (a part from crystals obviously).

As a temporary workaround it would also be possible to cancel CrystalPlaceEvent instead of rewriting the feature.
Will look at a fix soon.

The issue is due to the CrystalEntity extending LivingEntity when it should only extend Entity.
Client crash report:
java.lang.IllegalStateException: Server tried to update attributes of a non-living entity (actually: EndCrystal['End Crystal'/3, l='ClientLevel', x=27.50, y=158.00, z=-71.50])
	at Genesis//net.minecraft.client.multiplayer.ClientPacketListener.handleUpdateAttributes(SourceFile:2097)
	at Genesis//net.minecraft.network.protocol.game.ClientboundUpdateAttributesPacket.handle(SourceFile:51)
	at Genesis//net.minecraft.network.protocol.game.ClientboundUpdateAttributesPacket.handle(SourceFile:20)
	at Genesis//net.minecraft.network.protocol.PacketUtils.lambda$ensureRunningOnSameThread$0(SourceFile:27)
	at Genesis//net.minecraft.util.thread.BlockableEventLoop.doRunTask(SourceFile:162)
	at Genesis//net.minecraft.util.thread.ReentrantBlockableEventLoop.doRunTask(SourceFile:23)
	at Genesis//net.minecraft.util.thread.BlockableEventLoop.pollTask(SourceFile:136)
	at Genesis//net.minecraft.util.thread.BlockableEventLoop.runAllTasks(SourceFile:121)
	at Genesis//net.minecraft.client.Minecraft.runTick(SourceFile:1240)
	at Genesis//net.minecraft.client.Minecraft.run(SourceFile:882)
	at Genesis//net.minecraft.client.main.Main.main(SourceFile:256)
	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(Unknown Source)
	at java.base/java.lang.reflect.Method.invoke(Unknown Source)
	at com.moonsworth.lunar.genesis.ClientGameBootstrap.apply(Unknown Source)
	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(Unknown Source)
	at java.base/java.lang.reflect.Method.invoke(Unknown Source)
	at com.moonsworth.lunar.genesis.COHORCHOCHOOOCHRHIHIRIICHIHIHO.RROCOCCHRORIORIHOROCORCIHHHICC(Unknown Source)
	at com.moonsworth.lunar.genesis.Genesis.run(Unknown Source)
	at com.moonsworth.lunar.genesis.Genesis.main(Unknown Source)

One simple way to fix this would be to just cancel any EntityAttributesPacket where the entity is an instance of CrystalEntity, although this is sort of a hacky fix. It does work though, I tested it.
This code stops the error.
		MinecraftServer.getGlobalEventHandler().addListener(PlayerPacketOutEvent.class, event -> {
			if (event.getPacket() instanceof EntityAttributesPacket entityAttributesPacket) {
				if (instance.getEntityById(entityAttributesPacket.entityId()) instanceof CrystalEntity) {
					event.setCancelled(true);
				}
			}
		});


Issue #19028 Replace com.jcraft:jsch with com.github.mwiede:jsch

I accidentally deleted the repo - I've recreated the branch here: https://github.com/aSemy/gradle/tree/issues/19028-jsch-update
Signed-off-by:

Fixes #19028
Context

Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

I've made a second attempt: https://github.com/gradle/gradle/pull/20387
I'll close this PR to keep things focused.


Add react-native field to package.json

With this change you can import from native.js by
import {observe} from 'mobx-react'

React Native packager will automatically use correct file.

Great! Thanks


IGNITE-14452 : Add checking of the iptables settings applied.

Sometimes, we lack settings of iptables for unknows reason. Let's monitor this issue.

Migrated to https://github.com/apache/ignite/pull/8954


Not able to run the Flask API

I have followed the steps and set up the environment on Ubuntu 18.04. But while running it facing the following issue.
/home/jeeva/.local/lib/python3.7/site-packages/pydub/utils.py:170: RuntimeWarning: Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work
  warn("Couldn't find ffmpeg or avconv - defaulting to ffmpeg, but may not work", RuntimeWarning)

Killed```

I believe it is an issue with memory limitation but not sure. Can you please help us with this issue and also specify the hardware requirements for CPU only and with GPU?

I have backtracked the code and found that in support.py, the 'load_model' function has a while loop that is not ending and consuming the whole memory.
def load_model(mdl_path):
    while True:
        try:
            mdl, cfg, task = fairseq.checkpoint_utils.load_model_ensemble_and_task([mdl_path])
            print("Successfully loaded model "+mdl_path)
            break

        except:
            m = torch.load(mdl_path)
            m['cfg']['task']['_name'] = 'audio_finetuning'
            torch.save(m,mdl_path)
    return mdl[0].eval(), task.target_dictionary

if I remove the while loop and have only one try and except block like below,
def load_model(mdl_path):
    print("Loading model....")
    # while True:
    try:
        mdl, cfg, task = fairseq.checkpoint_utils.load_model_ensemble_and_task([mdl_path])
        print("Successfully loaded model "+mdl_path)
        #break
    except:
        m = torch.load(mdl_path)
        m['cfg']['task']['_name'] = 'audio_finetuning'
        torch.save(m,mdl_path)
        mdl, cfg, task = fairseq.checkpoint_utils.load_model_ensemble_and_task([mdl_path])
        print("Successfully loaded model "+mdl_path)
    return mdl[0].eval(), task.target_dictionary

then I am getting the following error
Loading model....
tcmalloc: large alloc<PHONE_NUMBER> bytes == 0x55a21f34c000 @  0x7f6d74ea0b6b 0x7f6d74ec0379 0x7f6c65a04cde 0x7f6c65a06452 0x7f6cb8bec571 0x7f6d614130b6 0x7f6d60b84aa8 0x55a1cbe35098 0x55a1cbea84d9 0x55a1cbea2ced 0x55a1cbe35bda 0x55a1cbea3915 0x55a1cbea2ced 0x55a1cbe357f3 0x55a1cbe352f9 0x55a1cbf7c35d 0x55a1cbeeba0b 0x55a1cbe343a1 0x55a1cbf25e1d 0x55a1cbea7e99 0x55a1cbea2ced 0x55a1cbd74e2b 0x55a1cbea4fe4 0x55a1cbea29ee 0x55a1cbe35bda 0x55a1cbea4737 0x55a1cbea29ee 0x55a1cbe35bda 0x55a1cbea3915 0x55a1cbea29ee 0x55a1cbe35bda
tcmalloc: large alloc<PHONE_NUMBER> bytes == 0x55a26a6f4000 @  0x7f6d74ea0b6b 0x7f6d74ec0379 0x7f6c65a04cde 0x7f6c65a06452 0x7f6cb8bec571 0x7f6d614130b6 0x7f6d60b84aa8 0x55a1cbe35098 0x55a1cbea84d9 0x55a1cbea2ced 0x55a1cbe35bda 0x55a1cbea3915 0x55a1cbea2ced 0x55a1cbe357f3 0x55a1cbe352f9 0x55a1cbf7c35d 0x55a1cbeeba0b 0x55a1cbe343a1 0x55a1cbf25e1d 0x55a1cbea7e99 0x55a1cbea2ced 0x55a1cbd74e2b 0x55a1cbea4fe4 0x55a1cbea29ee 0x55a1cbe35bda 0x55a1cbea4737 0x55a1cbea29ee 0x55a1cbe35bda 0x55a1cbea3915 0x55a1cbea29ee 0x55a1cbe35bda
Traceback (most recent call last):
  File "flask_api.py", line 37, in <module>
    model,dictionary = load_model(m['model_path'])
  File "/content/pratilipi/speech-api/app/support.py", line 246, in load_model
    mdl, cfg, task = fairseq.checkpoint_utils.load_model_ensemble_and_task([mdl_path])
  File "/content/pratilipi/speech-api/app/fairseq/checkpoint_utils.py", line 470, in load_model_ensemble_and_task
    model = task.build_model(cfg.model, from_checkpoint=True)
  File "/content/pratilipi/speech-api/app/fairseq/tasks/audio_finetuning.py", line 193, in build_model
    model = super().build_model(model_cfg, from_checkpoint)
  File "/content/pratilipi/speech-api/app/fairseq/tasks/audio_pretraining.py", line 197, in build_model
    model = super().build_model(model_cfg, from_checkpoint)
  File "/content/pratilipi/speech-api/app/fairseq/tasks/fairseq_task.py", line 335, in build_model
    model = models.build_model(cfg, self, from_checkpoint)
  File "/content/pratilipi/speech-api/app/fairseq/models/__init__.py", line 106, in build_model
    return model.build_model(cfg, task)
  File "/content/pratilipi/speech-api/app/fairseq/models/wav2vec/wav2vec2_asr.py", line 212, in build_model
    w2v_encoder = Wav2VecEncoder(cfg, len(task.target_dictionary))
  File "/content/pratilipi/speech-api/app/fairseq/models/wav2vec/wav2vec2_asr.py", line 410, in __init__
    task = tasks.setup_task(w2v_args.task)
  File "/content/pratilipi/speech-api/app/fairseq/tasks/__init__.py", line 39, in setup_task
    cfg = merge_with_parent(dc(), cfg)
  File "/content/pratilipi/speech-api/app/fairseq/dataclass/utils.py", line 490, in merge_with_parent
    merged_cfg = OmegaConf.merge(dc, cfg)
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/omegaconf.py", line 321, in merge
    target.merge_with(*others[1:])
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/basecontainer.py", line 331, in merge_with
    self._format_and_raise(key=None, value=None, cause=e)
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/base.py", line 101, in _format_and_raise
    type_override=type_override,
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/_utils.py", line 629, in format_and_raise
    _raise(ex, cause)
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/_utils.py", line 610, in _raise
    raise ex  # set end OC_CAUSE=1 for full backtrace
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/basecontainer.py", line 329, in merge_with
    self._merge_with(*others)
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/basecontainer.py", line 347, in _merge_with
    BaseContainer._map_merge(self, other)
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/basecontainer.py", line 314, in _map_merge
    dest[key] = src._get_node(key)
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/dictconfig.py", line 259, in __setitem__
    key=key, value=value, type_override=ConfigKeyError, cause=e
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/base.py", line 101, in _format_and_raise
    type_override=type_override,
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/_utils.py", line 629, in format_and_raise
    _raise(ex, cause)
  File "/usr/local/lib/python3.7/dist-packages/omegaconf/_utils.py", line 610, in _raise
    raise ex  # set end OC_CAUSE=1 for full backtrace
omegaconf.errors.ConfigKeyError: Key 'eval_wer' not in 'AudioPretrainingConfig'
	full_key: eval_wer
	reference_type=Optional[AudioPretrainingConfig]
	object_type=AudioPretrainingConfig

Any help on this setup is much appreciated. Thanks!

Hi. Thanks for pointing out.
Can you go to this file and paste the function here?
File "/content/pratilipi/speech-api/app/fairseq/dataclass/utils.py", line 490, in merge_with_parent
    merged_cfg = OmegaConf.merge(dc, cfg)

There is one argument in the function "merge_with_parent" named as remove_missing. Set it to True and try again with your process.


Costume Word list/dictionary

Some of the words are not there. Is there way to add them to dictionary? Is it possible to use my own costume dictionary/word list than the default present one? I don't want to bother you or myself by telling each time I find a new word. ;p

You'd need to take the words from here --> https://skribbler.herokuapp.com/api/words , just copy the entire page really, then add the missing words in the same format and rehost this elsewhere. You can change the page it looks at to get the words in the script itself


Add C# support for Android in Godot 4.x

Godot version
4.0.beta1
System information
Windows 11
Issue description
In Godot 4.0.beta1 mono version, I am unable to export projects to android due to a missing export template for android. On all download sources, it shows windows and linux templates, but no android templates.
When I compare this to the non-mono version, I see android templates.
I have looked at all iterations of godot 4.0 beta here and see later beta versions also don't have android templates for the mono build.
I searched for an issue tracking this, but I was only able to find a closed issue for when godot originally added C# support for android https://github.com/godotengine/godot/issues/20267.
Is C# support for android planned for godot 4.0? If so, is there a different issue I should reference for this issue?
Steps to reproduce

Open Godot 4.0.beta1
Select Project -> Install Android Build Template
Select Manage Templates
Select Download and Install
Close Template Manager
Select Project -> Export
Select Add -> Android
Observe error message No export template found at the expected path:

Minimal reproduction project
Not including minimal reproduction project as this should be reproducible in an empty project. Let me know if you need one anyways.

C# support for Android, iOS and Web is not planned for 4.0, it will be worked later on for 4.x releases (as soon as possible, but for 4.0 we focus on the desktop platform and the overall feature set and UX).

What are the blockers for mono on Android? I would be interested in trying

What are the blockers for net6 on Android? I would be interested in trying

I have no idea what I am talking about here, but my guess is that Xamarin won't work with .NET 5+. So Android export would have to be based on MAUI, which needs to get Vulcan support, which might take a long time.

What's the status of this after the upgrade to NET 7.0?

Its 4.0 Stabls and the support is not there...

@Jonak-Adipta-Kalita yes, as was expected

C# support for Android, iOS and Web is not planned for 4.0, it will be worked later on for 4.x releases

@Jonak-Adipta-Kalita yes, as was expected

C# support for Android, iOS and Web is not planned for 4.0, it will be worked later on for 4.x releases

Yeah, I saw on there blog post.

Waiting for Android export template of Godot 4 mono version!

Still waiting the android export for C#

Please don't comment if you don't have meaningful information to add. We know that C# support for Android is missing, we don't need reminders.
If you want to show your support of the issue, add a :+1: reaction to the first post.

Hi. Regarding this: is there an estimate of when will Godot 4.x support C# for Android and iOS? Even a rough estimate would be appreciated. Are we talking about ~2 months? ~6 months? ~1 year?
Thanks!

I wouldn't expect any less than 6 months to a year.

Hi, I want to know this issue is possible for a new contributor? if yes, please tell me how I should fix it?

@behroozbc Adding C# support in Android is a VERY complex task - not a first issue definitely.
Look for things tagged good first issue

@behroozbc Adding C# support in Android is a VERY complex task - not a first issue definitely.
Look for things tagged good first issue

Is it gonna happen or not

I know I'm not a contributor, but

Please don't comment if you don't have meaningful information to add. We know that C# support for Android is missing, we don't need reminders.
If you want to show your support of the issue, add a +1 reaction to the first post.

Just a heads up, the Godot site is currently wrong until this issue is resolved. It shows you can use C# for your Godot games. While you can, this lacks an important asterix advising of this problem, and should possibly be updated until this issue is resolved.

The square below it advertises cross-platform ability, which is not possible when using C# in 4.0. The website should probably be updated.

Just a heads up, the Godot site is currently wrong until this issue is resolved. It shows you can use C# for your Godot games. While you can, this lacks an important asterix advising of this problem, and should possibly be updated until this issue is resolved.

This is being addressed by https://github.com/godotengine/godot-website/pull/671.

For 4.0.* works. Download the templates for non-mono, go to the downloaded template folder and copy all the android files to the mono templates folder

For 4.0.* works. Download the templates for non-mono, go to the downloaded template folder and copy all the android files to the mono templates folder

That's not going to work, unless you're not using any C# code, in which case there's no reason to use the Mono version.

Download the templates for non-mono
where can i download this template?

People, I'm sorry. I was wrong, it won't work.

For 4.0.* works. Download the templates for non-mono, go to the downloaded template folder and copy all the android files to the mono templates folder

That's not going to work, unless you're not using any C# code, in which case there's no reason to use the Mono version.

People, I'm sorry. I was wrong, it won't work.

C# support for Android was implemented by https://github.com/godotengine/godot/pull/73257 and will be available in Godot 4.2, due for release in November 2023.
You can already test it in the latest dev snapshots (the current one, and first one with the Android export template, is 4.2 dev 4).
There might still be bugs with the implementation, so if you run into any issue when testing, please open a dedicated bug report, so we can make sure to iron out problems before the 4.2 release.
No ETA yet for iOS and Web support, but this is a top priority, we're currently assessing what it will take to get those.

4.2 Do I need to configure anything to be able to select C# in dev 4?
C# cannot be select.

@gtk2k
https://godotengine.org/article/dev-snapshot-godot-4-2-dev-4/#downloads
You need to download the mono version (it's not renamed yet, it's actually .net)

Yours is the standard version.
You can ask questions on other platforms first next time, and if it might be a bug then report it on GitHub.

Thanks

Amazing stuff guys! Thanks!!

No ETA yet for iOS and Web support, but this is a top priority, we're currently assessing what it will take to get those.

Is there an open issue that we can follow just so we can get notified whenever these are available on Godot 4? I couldn't find any. If not, is it okay to open one just for tracking that?

iOS support was implemented already in #82729.


Improve MyPy support

Set setting in ini file
Use warn_unused_ignores mad warn_redundant_casts to remove no longer needed workarounds
Unpin Hypothesis which is now fixed. Fixes #1163
Rather than modifiying pyspcm source code ignore it from mypy

They are all the places that mypy detects. With the setting in this ini file it should raise if there are unneeded casts or ignores


Add takeover time to claim resolution

The Issue
To support the app in showing users how long they need to wait for their vanity claim resolution (https://github.com/lbryio/lbry-app/issues/633#issuecomment-345821486), the daemon should return this information with the resolve call.
Grin's old code for takeover time:
// the following is based on https://lbry.io/faq/claimtrie-implementation
const lastTakeoverHeight = claimsForName['nLastTakeoverHeight'],
maxDelay = 4032, // 7 days of blocks at 2.5min per block
activationDelay = Math.min(maxDelay, Math.floor((claimBlockHeight - lastTakeoverHeight) / 32)),
takeoverHeight = claimBlockHeight + activationDelay,
secondsPerBlock = 161, // in theory this should be 150, but in practice its closer to 161
takeoverTime = Date.now() + ((takeoverHeight - currentHeight) * secondsPerBlock * 1000);

System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

keep in mind, this is an approximate takeover time,  and should be presented as such

@tzarebczan Is this still an open issue?

Yes, needs to be prioritized


Update CHANGELOG.md

Description
Fixed issue #

Added yourNewMethodName() that allows ...
Compatibility

OS
Implemented

iOS
✅❌

Android
✅❌

Windows
✅❌

Checklist

[x] I added the documentation in README.md
[x] I mentioned this change in CHANGELOG.md
[ ] I updated the typings files (deviceinfo.d.ts, deviceinfo.js.flow)
[ ] I updated the dummy web/test polyfill (default/index.js)
[ ] I added a sample use of the API (example/App.js)
[ ] I have tested this on a device/simulator for each compatible OS

That change was small enough and changelog-only so I'm not going to treat it as a formal release, but I really appreciate the doc-fix PR - thanks


feature: dotenv support - allow to change the used OpenSearch version (ddev 1.23.5)

Add the following variables:

OPENSEARCH_TAG -> defines the tag of the used opensearch image
INSTALL_PLUGIN_ANALYSIS_PHONETIC: boolean var
INSTALL_PLUGIN_ANALYSIS_ICU: boolean var

docker-compose is now passing the args to the used Dockerfile
The Issue
Solve #6
How This PR Solves The Issue

Add args to Dockerfile
Define used build-args in docker-compose.yml

Manual Testing Instructions
Use new ddev 1.23.5 and define a dotenv file.
ddev dotenv set .ddev/.env.opensearch --opensearch-tag=2.15.0 --opensearch-dashboards-tag=2.15.0 --install-plugin-analytics-phoenetic=false --install-plugin-analytics-icu=false
ddev add-on get https://github.com/cmuench/ddev-opensearch/tarball/feature/dotenv-support

Related Issue Link(s)

#3 -> This PR allow to define the used OpenSearch version

Release/Deployment Notes
It should not have affects.

Thank you, and the release test can also be updated to:
# bats test_tags=release
@test "install from release" {
  set -eu -o pipefail
  cd ${TESTDIR} || ( printf "unable to cd to ${TESTDIR}\n" && exit 1 )
  echo "# ddev add-on get ${ADDON_PATH} with project ${PROJNAME} in ${TESTDIR} ($(pwd))" >&3
  ddev add-on get ${ADDON_PATH}
  health_checks
}

bats test_tags=release ensures that it will not try to install the released version add-on from PR

Thank you, and the release test can also be updated to:
# bats test_tags=release
@test "install from release" {
  set -eu -o pipefail
  cd ${TESTDIR} || ( printf "unable to cd to ${TESTDIR}\n" && exit 1 )
  echo "# ddev add-on get ${ADDON_PATH} with project ${PROJNAME} in ${TESTDIR} ($(pwd))" >&3
  ddev add-on get ${ADDON_PATH}
  health_checks
}

bats test_tags=release ensures that it will not try to install the released version add-on from PR

Sorry. Forgot it... One second.

Great, thank you!

Thanks for reviewing it.


Add GuildMember.guildOwner

Returns a boolean value of whether the guild member is the owner of the guild.

I think this is pretty useless, you could just do guild.ownerID === member.id

I think it's a nice semantic change, though.


Add -q, --quiet option

The quiet flag will suppress the info line output for each command. It will still print outputs for operations that have outputs, such as list-databases.
Do we need a -qq and --quiet=2 to supporess all output? (dont think so).

Note: This issue has been migrated to https://relationalai.atlassian.net/browse/RAI-7199.
This link is only accessible to employees of RelationalAI.