[BUG] Credentials could not be decrypted

Describe the bug
Right after I updated the flowise image, this error started to occur.
I've tried everything, looked in the documentation too and nothing.
I deleted the encryption.key, created a new one, even deleted the docker image and also the Flowise folder and the same error continued.
I really don't know what else to do. Not even ChatGPT can help me anymore.
Docker compose logs

ubuntu@typebot-alisson:~/Flowise/docker$ sudo docker compose logs -f
docker-flowise-1  | 2023-09-08 19:36:14 [INFO]: Starting Flowise...
docker-flowise-1  | 2023-09-08 19:36:14 [INFO]: ⚡ [server]: Flowise Server is listening at 3002
docker-flowise-1  | 2023-09-08 19:36:14 [INFO]: 📦 [server]: Data Source has been initialized!
docker-flowise-1  | 2023-09-08 19:36:27 [INFO]: ❌ DELETE /api/v1/chatmessage/ec17aa9c-8ed1-4d50-ba83-96a1021f8004
docker-flowise-1  | 2023-09-08 19:36:32 [INFO]: ⬆ POST /api/v1/chatmessage/ec17aa9c-8ed1-4d50-ba83-96a1021f8004
docker-flowise-1  | 2023-09-08 19:36:32 [INFO]: ⬆ POST /api/v1/internal-prediction/ec17aa9c-8ed1-4d50-ba83-96a1021f8004
docker-flowise-1  | SyntaxError: Unexpected end of JSON input
docker-flowise-1  |     at JSON.parse ()
docker-flowise-1  |     at decryptCredentialData (/usr/local/lib/node_modules/flowise/node_modules/flowise-components/dist/src/utils.js:440:21)
docker-flowise-1  |     at async getCredentialData (/usr/local/lib/node_modules/flowise/node_modules/flowise-components/dist/src/utils.js:463:41)
docker-flowise-1  |     at async OpenAIEmbedding_Embeddings.init (/usr/local/lib/node_modules/flowise/node_modules/flowise-components/dist/nodes/embeddings/OpenAIEmbedding/OpenAIEmbedding.js:57:32)
docker-flowise-1  |     at async buildLangchain (/usr/local/lib/node_modules/flowise/dist/utils/index.js:221:50)
docker-flowise-1  |     at async App.processPrediction (/usr/local/lib/node_modules/flowise/dist/index.js:765:40)
docker-flowise-1  |     at async /usr/local/lib/node_modules/flowise/dist/index.js:558:13
docker-flowise-1  | 2023-09-08 19:36:32 [ERROR]: Error: Credentials could not be decrypted.
docker-flowise-1  | Error: Error: Credentials could not be decrypted.
docker-flowise-1  |     at getCredentialData (/usr/local/lib/node_modules/flowise/node_modules/flowise-components/dist/src/utils.js:467:15)
docker-flowise-1  |     at async OpenAIEmbedding_Embeddings.init (/usr/local/lib/node_modules/flowise/node_modules/flowise-components/dist/nodes/embeddings/OpenAIEmbedding/OpenAIEmbedding.js:57:32)
docker-flowise-1  |     at async buildLangchain (/usr/local/lib/node_modules/flowise/dist/utils/index.js:221:50)
docker-flowise-1  |     at async App.processPrediction (/usr/local/lib/node_modules/flowise/dist/index.js:765:40)
docker-flowise-1  |     at async /usr/local/lib/node_modules/flowise/dist/index.js:558:13
docker-flowise-1  | 2023-09-08 19:36:32 [ERROR]: [server]: Error: Error: Error: Credentials could not be decrypted.
docker-flowise-1  | Error: Error: Error: Credentials could not be decrypted.
docker-flowise-1  |     at buildLangchain (/usr/local/lib/node_modules/flowise/dist/utils/index.js:231:19)
docker-flowise-1  |     at async App.processPrediction (/usr/local/lib/node_modules/flowise/dist/index.js:765:40)
docker-flowise-1  |     at async /usr/local/lib/node_modules/flowise/dist/index.js:558:13

My .env Variable

DATABASE_PATH=/root/.flowise
APIKEY_PATH=/root/.flowise
SECRETKEY_PATH=/home/ubuntu/Flowise/docker/secrets/
LOG_PATH=/root/.flowise/logs

Setup

Installation: docker
Flowise Version 1.3.4
OS: Ubuntu 22.04 VM Oracle ARM
Browser: chrome

just confirming are you using the steps here for your docker setup? are you running locally or deployed on cloud?

Yes, I followed this same tutorial in conjunction with the deployment via AWS, just changing some commands to run on the Oracle Cloud ARM VM.
I was already running flowise even before version 1.3.0 and running some applications with it. But after I updated to this latest version 1.3.4 it started giving this key error.
As I said, I followed all the tutorials, looked here in the repository for answers and spent a few hours in partnership with ChatGPT to help me solve it and I couldn't.
I tried to delete the docker image and also delete the folder, but when I reinstalled the files they were there as if I hadn't deleted them and so I was back in the same place.
A shame, because I was using flowise a lot and even selling applications to my clients.

I've seen these issues before but only saw the issue happening to the openai credentials. So to solve this I simply recreated them. After this I didn't see the issue again.

Update:
I created a new encryption.key, put the passphrase in the .env file and then deleted all the API keys that had balance and recreated new ones.
With that, it worked again!

Added https://github.com/FlowiseAI/Flowise/pull/939 to allow users to specify their own encryption key, this should resolve all errors regarding the credential path


Feature request Remember the viewstate

Remember the viewstate after clickng on a check and then going back.

It seems that we have few duplicated issues for this feature, I'll close this one in favor of https://github.com/sensu/uchiwa/issues/156 so it will be easier to keep everyone informed about the progress.


When API's response is a pure text, it will cause error,Could you please help?

I try to fix this issue for ./lib/node/parsers/json.js:
'use strict';
module.exports = function parseJSON(res, fn) {
res.text = '';
res.setEncoding('utf8');
res.on('data', chunk => {
res.text += chunk;
});
res.on('end', () => {
try {
var body
try {
body = res.text && JSON.parse(res.text);
} catch (e) {
body = res.text;
}
} catch (e) {
var err = e;
// issue #675: return the raw response if the response parsing fails
err.rawResponse = res.text || null;
// issue #876: return the http status code if the response parsing fails
err.statusCode = res.statusCode;
} finally {
fn(err, body);
}
});
};
it works in my local test cases, but it's not ok in the your test, is it ok for my change? If so, Could you please help update your test?If not, Could you please help find another way?Thank you so much!
Here is the test result in your test:
https://travis-ci.org/visionmedia/superagent/builds/487220999?utm_source=github_status&utm_medium=notification

PR welcome


thrift: module integration, samples, and tests

[x] TSC vote to fork thrift
[x] fork https://github.com/apache/thrift into zephyrproject/thrift
[ ] https://github.com/zephyrproject-rtos/docker-image/pull/133
[x] update sample docs to include e.g. using Python on the host

Closes https://github.com/zephyrproject-rtos/zephyr/issues/51620

All samples & tests passing with the addition of the following in apache/thrift/zephyr/module.yml:
name: thrift
build:
  cmake-ext: True
  kconfig-ext: True

#
# Run Test
#
twister --retry-failed 3 -G -T tests/lib/thrift/
ZEPHYR_BASE unset, using "/home/cfriedt/zephyrproject/zephyr"
Renaming output directory to /home/cfriedt/zephyrproject/zephyr/twister-out.2
INFO    - Using Ninja..
INFO    - Zephyr version: zephyr-v3.2.0-3787-gb22347e7b20e
INFO    - Using 'zephyr' toolchain.
INFO    - Selecting default platforms per test case
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /home/cfriedt/zephyrproject/zephyr/twister-out/testplan.json
INFO    - JOBS: 32
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - Total complete:   36/  36  100%  skipped:    0, failed:    0
INFO    - 7 test scenarios (36 test instances) selected, 0 configurations skipped (0 by static filter, 0 at runtime).
INFO    - 36 of 36 test configurations passed (100.00%), 0 failed, 0 skipped with 0 warnings in 235.66 seconds
INFO    - In total 538 test cases were executed, 0 skipped on 7 out of total 535 platforms (1.31%)
INFO    - 36 test configurations executed on platforms, 0 test configurations were only built.
INFO    - Saving reports...
INFO    - Writing JSON report /home/cfriedt/zephyrproject/zephyr/twister-out/twister.json
INFO    - Writing xunit report /home/cfriedt/zephyrproject/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /home/cfriedt/zephyrproject/zephyr/twister-out/twister_report.xml...
INFO    - Run completed

#
# Build samples
#
twister --build-only -G -T samples/modules/thrift/
INFO    - Using Ninja..
INFO    - Zephyr version: zephyr-v3.2.0-3787-gb22347e7b20e
INFO    - Using 'zephyr' toolchain.
INFO    - Selecting default platforms per test case
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /home/cfriedt/zephyrproject/zephyr/twister-out/testplan.json
INFO    - JOBS: 64
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
INFO    - Total complete:    8/   8  100%  skipped:    0, failed:    0
INFO    - 8 test scenarios (8 test instances) selected, 0 configurations skipped (0 by static filter, 0 at runtime).
INFO    - 8 of 8 test configurations passed (100.00%), 0 failed, 0 skipped with 0 warnings in 29.26 seconds
INFO    - 0 test configurations executed on platforms, 8 test configurations were only built.
INFO    - Saving reports...
INFO    - Writing JSON report /home/cfriedt/zephyrproject/zephyr/twister-out/twister.json
INFO    - Writing xunit report /home/cfriedt/zephyrproject/zephyr/twister-out/twister.xml...
INFO    - Writing xunit report /home/cfriedt/zephyrproject/zephyr/twister-out/twister_report.xml...
INFO    - Run completed

@stephanosio - looks like we will need https://github.com/zephyrproject-rtos/docker-image/pull/133 merged in order to see all green here.
Even then, it seems as though checkpatch and other checks are flaking-out on C++ syntax, thnking that some things are comparisions, multiplications, bitwise operators, labels, etc.
I have just re-verified every test and sample configuration is working as expected, and fixed one minor issue in the doc/index.rst.

I might need to add a build-only flag to sample.yaml for CI. Will get to that shortly.

added build_only: true to samples

Without the thrift compiler installed, I get this during a configure:

The warnings and errors look entirely reasonable from my perspective.
Did you follow the instructions in doc/index.rst that specifically say what to install if you want to build the sample?
Several change requests to this PR explicitly specificied not to include the thrift compiler or additional dependencies installed by default in the GSG, so other than documenting how to install the necessary tools and then hoping users read the documentation and install dependencies, there is not a lot I can do.

It failing is fine, it failing because it has tried to execute THRIFT_EXECUTABLE-NOTFOUND as a shell command is not - that is the issue

It failing is fine, it failing because it has tried to execute THRIFT_EXECUTABLE-NOTFOUND as a shell command is not - that is the issue

@nordicjm - That's exactly the same way that any build would fail if were unable to find a specific tool, be it qemu, compiler, linker, etc.
E.g. https://github.com/zephyrproject-rtos/zephyr/search?l=CMake&q=find_program
How would you prefer the build fail?
Perhaps I'm missing something, but to me, fixing this would also mean fixing the same issue in at least 55 other places, which certainly seems our of scope for this PR.

use platform_allow for tests/lib/thrift/ThriftTest instead of integration_platforms
use platform_allow for samples/modules/thrift/ instead of integration_platforms

add braces in case statement where variable is declared in scope (to get rid of llvm error)

mitigate checkpatch warnings in modules/thrift/muzic/inflater.c

Topics for TSC:

exception for merging the module after feature freeze
approve Zlib and MIT licenses used in "muzic" (considered an implementation detail in Thrift module until dedicated compression subsystem is created)

remove extraneous inflater.h

set min_ram for tests/lib/thrift/muzic

Still seeing THRIFT_EXECUTABLE-NOTFOUND. Likely the build container needs to be updated.

How would you prefer the build fail?

Fixed:
find_program(THRIFT_EXECUTABLE thrift)
if(NOT THRIFT_EXECUTABLE)
  message(FATAL_ERROR "The 'thrift' command was not found")
endif()

mark thrift as experimental (CONFIG_EXPERIMENTAL)

rebased

Will take a day to pull out the Zlib related stuff to eliminate license-compatibility issues. That is mainly going to involve manually going over tests and samples again (particularly, host-side server demos). The original ZlibTransport implementation will always be available from https://github.com/zephyrproject-rtos/gsoc-2022-thrift .
Hopefully at some point soon we can design and implement a compression / decompression API for Zephyr that is Apache-2.0 from the start that takes more care in being space-efficient.
Some notes from the TSC meeting on a potential Compression / Decompression subsystem or API:

differentiate clearly between streaming (convolutional) and block codecs
include Zlib compatibility header to "simplify" integration

https://github.com/zephyrproject-rtos/gsoc-2022-thrift/blob/main/lib/thrift/include/zlib.h

should be ready to go
Note: TSC vote is not about reviewing the module, it's about allowing the exception for being past-deadline

https://github.com/zephyrproject-rtos/zephyr/pull/54626, which includes thrift-compiler has been merged.
Rebasing

@cfriedt FYI, this PR is currently pointing to add-zephyr-module-dir branch in the module.

@cfriedt FYI, this PR is currently pointing to add-zephyr-module-dir branch in the module.

Let me update that. Thanks @stephanosio

update west.yml to point to zephyr branch of https://github.com/zephyrproject-rtos/thrift.git

@nordicjm - please re-ack when you have a moment


enable Rack and Puma tests with Ruby 3.3

test Ruby 3.3 with Puma (via the Rack suite) now that puma/puma#3165 has been merged
update add_version to respect any prefix a human maintainer might want to specify for a version, and not just =

resolves #2042
NOTE: this PR can't come out of draft until a Puma version greater than v6.3.0 is relased.


Cannot mutate an existing table when its schema is changed

I get this error in the terminal where space-cloud is running when I mutate an existing table whose schema I changed recently.
2019/10/25 13:21:59 Eventing Error: event could not be updated Error 1054: Unknown column 'remark' in 'field list'
there is no such column as 'remark' in my table. I had to drop my table and re-create it to fix this issue.

Hey @shubham4443 this isn't a schema module problem. It's a problem with the schema used for the events_log table.
@Jayesh333 we need to make sure we both are using the same schema

Fixed in v0.13.0


error "TypeError: Cannot read property 'bottomright' of undefined"

I'm creating an issue, but it is the same as https://github.com/Charmatzis/react-leaflet-google/issues/15 .
There is a similar problem in line: https://github.com/Charmatzis/react-leaflet-google/blob/00bc05fe34c5bca5253aee0f00ade356363a8361/src/leaflet.google.js#L91
The same fix should be applied:
https://github.com/Charmatzis/react-leaflet-google/blob/00bc05fe34c5bca5253aee0f00ade356363a8361/src/leaflet.google.js#L109

hi @Charmatzis sorry to be a pain - is there any chance getting a release out with these fixes in in the next few days - trying to avoid a local fork of this project to get a product release out next week

Any news @Charmatzis ? it's a simple fix

Hi @paulwilton, did you finally succeed to render a GoogleMap Layer with react-leaflet-google?


Wrap images

Hi!
Is your feature request related to a problem? Please describe.
The closing of issue #200 bring us close to cool possibilities, but... There's something still missing. Some email template engines require wrappers. Mautic wants images to be wrapped in a div.
Describe the solution you'd like
Maybe a wrap or parent parameter?
Describe alternatives you've considered
Use mj-raw to create the div.
         <mj-raw>
          <!-- htmlmin:ignore --><div data-slot="image"><!-- htmlmin:ignore -->
        </mj-raw>

Additional context
Is that a bad idea?

mj-raw would produce invalid html here. I don't think there's any way to do so without forking the component here.
It's a really weird very specific issue here :(

Yeah... I spent more time looking at Mautic code. It requires almost all of its components to be wrapped in divs.
Maybe a super-hyper-power RegEx can do that. Thank you!


SeekHead positions are wrong in the example that makes changes to the mkv

Can you provide any more information about what you mean (i.e. which example and what is "wrong")?  None of the examples in this repository explicitly work with SeekHead elements.

The seek head element is a pointer to the start of other segments. If you change a segment and reserialize it the file offsets aren't valid anymore.

Ah, gotcha.  This isn't necessarily an issue with the library code, but an issue with the example usage in the documentation (Example 3) in the sense that it will generate a corrupt webm.  I can take a look at getting that corrected.

Most players will just ignore it and parse the whole file. Currently I'm just filtering the segment so it at least doesn't produce an invalid segment. Not sure if this will require some kind of special handling, probably a two pass approach where the file is serialized, then parsed again but with correct seek head positions.


Removes deprecated filterwarning from pytest.ini

Since we removed the deprecated API this filter warning can be removed as well.

Codecov Report

Merging #2039 (68ca372) into main (4092e46) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2039   +/-   ##
=======================================
  Coverage   74.86%   74.86%
=======================================
  Files          59       59
  Lines        5037     5037
=======================================
  Hits         3771     3771
  Misses       1266     1266

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 4092e46...68ca372. Read the comment docs.


Ajax is not working.

Expected Behavior
Ajax post/get method returns a success message.
POST routes send data securely.
Session/Flash message or data pass and store in session or flash.
Current Behavior
Ajax returns 302 status code and error.
POST routes are returning 419.
if we store any data in session or flash is not get in another place of code.

Most likely you have not disabled CSRF, that is why you are getting 419.
https://github.com/osiset/laravel-shopify/wiki/Installation#csrf

Same issue
When i has upgrade 16.0.0 to latest v(17.1.1) all Ajax request return the error [ {error: "Session token is invalid."}
error: "Session token is invalid."].
@khaiknievel
CSRF already disabled

@beingak
You might want to refer to this #895.

@khaiknievel
I try to this
axios.interceptors.request.use(function (config) {
return utils.getSessionToken(window['app-bridge-app']) // requires a Shopify App Bridge instance
.then((token) => {
console.log('axios token');
// Append your request headers with an authenticated token
config.headers.Authorization = Bearer ${token};
return config;
});
});
But it not working from me
I have simple AJAX request.
$.ajax({
type: 'GET',
dataType: 'JSON',
url: ${window.location.origin}/app/fetch-messages,
headers: {
'Content-Type': 'application/json',
'Accept': 'application/json',
'X-CSRF-Token': csrfToken,
'shop': 'aruntesting1.myshopify.com'
},
success: (data)=> {
if(data.status) $('#dashboard-fetch-message').empty().html(data.html);
},
error: (xhr, textStatus, errorThrown) => console.log(xhr.statusText)
});
Can you please let me know. How i can use this.

Hello,
I have HTML + Laravel (Shopify app).
I am doing get and post Ajax request. which has been request to web.php middleware route. it did return [ 302 ] status.
[ {error: "Session token is invalid."}]
Do can you let me know how can i get the valid session token and how could be use into Ajax request.Please add example.

Might be help #1083

This sounds like an issue with your code setup, not the app so closing as a couple solutions have been provided.


Deno library types are extremely broken

I tried to use the deno library and it complains a ton about type checking errors - I can't even show them all here...

Sadly I can't find a single other deno library that provides openapi schemas 😢

it works with deno and vscode using an older version of zod.
/deno/lib/deps.ts needs to import zod from<EMAIL_ADDRESS>here's an example project that uses an import-map.json: https://github.com/hastebrot/zod-endpoints-playground


update a new image for GAD

a smaller image

updated a smaller image, plz resolved the conflict and delete the image in the root(was a mistake), thanks

@tyranthym I added it here https://github.com/TrustWallet/tokens/commit/21794e87c1ab5aade3aeaa4077e624b31a26bc79
Also, you can use these graphics to say that your ERC20 token fully compatible with Trust Wallet (https://trustwalletapp.com/logo-assets). Tokens will show up automatically once the user receives them.
Feel free to share this on social media and tag Trust Wallet:

Twitter: @trustwalletapp

Facebook: @trustwalletapp


Out-of-sample projections

Any updates on this?

@skn123 unfortunately, no, any specific request?

I think a Laplacian eigenmap OOS would be useful. Without OOS, most of these methods are not useful (other than linear ones)

@skn123 yeah, they are more for exploration without OOS. Can't promise anything but will try to get some time/power to do that.


force release of quota when AW is delete in target namespace

[AllocateForest] Found consumer %v in quota tree but didn't find its matching AppWrapper. Deleting consumer from the tree.
Done, thanks

@asm582 I think the best place to add the logic you are proposing is the
func function.
Do you know what is the root cause of what you are proposing?

I think that the logic for checking if the AW is a namespace does not belong to the qutamanger struct. It should be placed (if needed) a lot higher in the call chain: the Fits method call: quotaFits, preemptAWs, msg := qjm.quotaManager.Fits(qj, aggqj, proposedPreemptions) or inside the method itself.

Good Point, I will move the code higher up

This PR should by replaced by #398 once completed.


add command for formatting a single file

It will be good if the guide mentions a way to format specified files only. This can be done in two ways:

npx --write <pattern to match required files>: npx is a package which comes with npm. It simply executes node packages if found in ./node_modules. If this doesn't work you may install it using sudo npm install -g npx otherwise follow the other method.

./node_modules/prettier/bin-prettier.js --write <pattern to match required files>: This command runs prettier installed in the node_modules.

pattern to match required files examples:

src/Diagram/index.js
static/docs/command-reference/add.md
static/docs/**/*.md

I guess this info would go in the contributing documentation guide. I vote for npx.
p.s. I often git add my changes and then use git diff --staged --name-only | xargs npx prettier --write to lint only the staged files. But that shouldn't be necessary anymore now that we have automatic pretty-quick (kudos @algomaster99 in #386)

We have added a mention already how to run it for a single Markdown file. Do we need anything else here? Add a command for JS?

@shcheklein It's the same. You just need to write js files in <files> instead of md files.

ok, closing it then. Feel free to reopen it if you feel we are missing something here.


Governance section revamp update overview.md of Gov Section

Checklist

[x] I have read the How to Contribute.
[x] I have run yarn build after adding my changes without getting any errors.

Updating documentation or Bugfix
Governance section revamp update overview.md of Gov Section as explained in #1223

@thenic95 please correct me if I'm wrong, but the purpose of this PR is the edits in this single commit https://github.com/cardano-foundation/developer-portal/pull/1237/commits/556ce8cf35e6aa69c57bea5bfbc6a2a3d3327433 (which themselves look good to me), while:

the first of the commits in this PR is duplicated from https://github.com/cardano-foundation/developer-portal/pull/1228: likely because you've edited files in the same branch without rebasing on what you merged into the main repository.
the 2nd and 3rd commits cancel each other out.

This is why the PR is falsely showing a merge conflict right now.  For the continuity of the changes to the upstream cardano-foundation:governance-revamp branch, I think this PR should be closed & then submitted again relative to either:

a new branch based on cardano-foundation:governance-revamp
the old branch, after syncing this branch in your local fork & then git pulling it back to the local git where you're editing these files.

p.s. while I was writing this you have resolved the merge conflict (and probably noted there was nothing to resolve).  I still think the commit history for this PR would be messed up if it were merged as-is and that the PR should be resubmitted. 🤔

Thanks for taking the time to write this @rphair
I will close this PR and create a new one that doesn't have a messed-up commit history.


User info is not valid. (authManagement)

When doing a post request to authManagement with the following data
{
    "action": "sendResetPwd",
    "value"<EMAIL_ADDRESS>}

I get a response of User info is not valid. (authManagement) and I've no idea what is means in order to fix the issue.

The docs (https://docs.feathersjs.com/api/authentication/local-management.html#client) have
// send forgotten password notification
authManagement.create({ action: 'sendResetPwd',
  value: identifyUser, // {email}, {token: verifyToken}
  notifierOptions, // options passed to options.notifier, e.g. {preferredComm: 'email'}
})

You need value: { email<EMAIL_ADDRESS>}.

OK I wasn't familiar with {email} translating to that.

I have similar problem when I'm trying to use verifySignupShort action.
I'm making a request with Postman to '/authmanagement' with these data:
{ 	"action": "verifySignupShort", 	"value": { 		"user": {"id":"9591d3e6-2e8d-44c1-9315-c8b02ca1e339"}, 		"token":"367596"   } }
And i get this response:
{     "name": "BadRequest",     "message": "User info is not valid. (authManagement)",     "code": 400,     "className": "bad-request",     "data": {},     "errors": {         "$className": "badParams"     } }
Am I doing something wrong?

UPD: Sry, found a mistake. I hadn't specified 'identifyUserProps'
I have the same problem. Digging sourcecode told me that function ensureObjPropsValid checks received user for inclusion of props, that are build from identifyUserProps. But I thought this parameter purpose is to keep unique fields for user, not all props of user instance.
Was I wrong? Should I add all the props from my migration to this array when configuring authManagement?

I was passing the whole context results and it wanted only the email
    // user after hook
    create: [
      context => {
        context.app.service('authManagement').create({ action: 'resendVerifySignup', value: {email: context.result.email} })
      },
      verifyHooks.removeVerification()
    ],


[SPARK] support for Spark 3.5

Problem
Recently introduced Spark version 3.5.0 is not supported. Althouhg iceberg and delta do not support this version, we can enable it in our integration tests to make sure it does not cause any problems on platforms like databricks.
Solution

Enable 3.5.0 integration tests in circle ci,
Bump 3.3.2 to 3.3.3

Note: All schema changes require discussion. Please link the issue for context.

[ ] Your change modifies the core OpenLineage model
[ ] Your change modifies one or more OpenLineage facets

If you're contributing a new integration, please specify the scope of the integration and how/where it has been tested (e.g., Apache Spark integration supports S3 and GCS filesystem operations, tested with AWS EMR).
One-line summary:
Checklist

[x] You've signed-off your work
[x] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[x] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] Your comment includes a one-liner for the changelog about the specific purpose of the change (if necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2023 contributors to the OpenLineage project

Codecov Report

Merging #2118 (c459c2a) into main (021092a) will not change coverage.
Report is 5 commits behind head on main.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff            @@
##               main    #2118   +/-   ##
=========================================
  Coverage     81.05%   81.05%
  Complexity      125      125
=========================================
  Files            90       90
  Lines          3784     3784
  Branches         33       33
=========================================
  Hits           3067     3067
  Misses          678      678
  Partials         39       39

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Allow option for developmental releases defined in PEP 440.

Fixes #8. I'm willing to change the name of the option or make any tweaks as necessary.

Coverage remained the same at 100.0% when pulling 5aafadef9170d5d93ab362f7d61a57b038f370a7 on tylercrumpton:master into 72f8f0a7e0121cf5989a2cb00d5e1395e02a0445 on habnabit:master.

Any thoughts on pulling this request in?

Really sorry. I'm trying to schedule time so I can get around to all of my currently-neglected projects.

@habnabit I would also need this feature, can I somehow support?

So sorry I haven't set aside time to review this yet. I'm going to schedule
some for tonight.
On Tue, Feb 7, 2017 at 10:49 AM, Tyler Crumpton<EMAIL_ADDRESS>wrote:

@tylercrumpton commented on this pull request.
In vcversioner.py https://github.com/habnabit/vcversioner/pull/9:

@@ -73,6 +74,14 @@ def find_version(include_dev_version=True, root='%(pwd)s',
the version number will be 1.0.post3. This behavior can be disabled
by setting this parameter to False.

:param use_dev_not_post: PEP 440 states that versions created during

   development cycles should use the ``.dev`` suffix instead of ``.post.``

   This suffix can be used by setting this parameter to ``True``. This

   suffix references a corresponding future release, so vcversioner will

   assume that the future release is a single version higher than the

   current commit. For example, if the current commit is one revision past

   the ``2.0.1`` tag, the computed version will be ``2.0.2dev1``.

@Julian https://github.com/Julian Any thoughts on this? I can add the lambda
current: next_version if that feels required to move forward.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/habnabit/vcversioner/pull/9, or mute the thread
https://github.com/notifications/unsubscribe-auth/AABSsF3Cu46HDKiS7jsYmRJ45guq59h8ks5raLyjgaJpZM4JBvm8
.

@habnabit @Julian Did either of you get a chance to take a look at this? We've been using this option for a while now and love how it has improved our CI process. 😃

The idea is +1 from me I think, though I haven't read PEP 440 close enough to be sure, but it's @habnabit's opinion that matters :) (Sorry for the wait :/)

Hey, just wanted to give a gentle ping 😄  This is still a feature that we use constantly and would love to see it in the official project!

Hey there! Any new thoughts on pulling in something like this?

Coverage remained the same at 100.0% when pulling a30f32e09b1ac08c3b7ccf4a9f1559a1248df381 on tylercrumpton:master into 72f8f0a7e0121cf5989a2cb00d5e1395e02a0445 on habnabit:master.

Coverage remained the same at 100.0% when pulling 204a28dfe00ca5564b0242b39eed8ed4fd1aca74 on tylercrumpton:master into 72f8f0a7e0121cf5989a2cb00d5e1395e02a0445 on habnabit:master.

Ack, sorry for the commit/rebuild noise here! I forgot that this PR was based off of tylercrumpton/vcversioner@master, so a change I was playing with got yanked in by accident. Should be reverted out now, and the PR can be squashed if we still want to pull the dev-version changes in.


Reading and updating the tool verification pin value

On the web app, clicking “Read Sensor” after mounting a tool should return a pin 63 value of 0. This part works fine.
However, when trying to read pin 63 after mounting a tool, the returned value is still 1. After clicking “Read Sensor” on the web app, only then does the code correctly return 0.
How, using python, can I make the exact same effects of the “Read Sensor” button on the web app?
Here’s what I use so far:
state[‘pins’][‘63’][‘value’]
or
bot.state[‘pins’][‘63’][‘value’]
both return either 0.0 or 1.0, but don’t update (if a tool is dismounted, it still reads 0.0 until I click the “Read Sensor” button on the web app).
Any help or clarification on how/when these values update would be great! Thanks.

That is the correct way to retrieve the state of the pin. To read the pin and update the state, use bot.read_pin(63):
https://github.com/FarmBot/farmbot-py/blob/6a860c9d6277ba88bbc79720e1df4c6482dd57c4/README.md?plain=1#L175

A full example:
from farmbot import Farmbot, FarmbotToken

token = FarmbotToken.download_token("email", "password")
fb = Farmbot(token)
class MyHandler():
    def on_connect(self, bot, mqtt_client):
        bot.read_pin(63)
    def on_change(self, bot, state): pass
    def on_response(self, bot, response): pass
    def on_log(self, bot, log): pass
handler = MyHandler()
fb.connect(handler)

If you ever want to know what commands the web app is sending, you can listen via:
import json
import paho.mqtt.client as mqtt
from farmbot import FarmbotToken

token = FarmbotToken.download_token("email", "password")
token = json.loads(token)['token']

def on_connect(client, userdata, flags, rc):
    client.subscribe(f"bot/{token['unencoded']['bot']}/from_clients")

def on_message(client, userdata, msg):
    print(json.dumps(json.loads(msg.payload), indent=4))

client = mqtt.Client()
client.username_pw_set(
    token['unencoded']['bot'],
    password=token['encoded'])
client.connect(token['unencoded']['mqtt'])
client.on_connect = on_connect
client.on_message = on_message
client.loop_forever()

Where both the web app and the example code above return:
{
    "kind": "rpc_request",
    "args": {
        "label": "6e1fc688-6579-45f7-8e93-0db05d9ebe45",
        "priority": 600
    },
    "body": [
        {
            "kind": "read_pin",
            "args": {
                "pin_number": 63,
                "label": "pin63",
                "pin_mode": 0
            }
        }
    ]
}

Gabriel,
Thank you for your help so far. When we run movement commands, the script you have provided works great, we will use it. However, the problem persists. When we use read_pin(63), the script is reading incorrect(not updated?) values.
For example, it reads:

{
    "kind": "rpc_request",
    "args": {
        "label": "4d74c8ae-ff18-48e8-83bb-3ecc9d400e8d",
        "priority": 600
    },
    "body": [
        {
            "kind": "read_pin",
            "args": {
                "pin_number": 63,
                "label": "pin63",
                "pin_mode": 0
            }
        }
    ]
}

When the Web App button "Read Sensor" returns: 1 (No Tool)

Here's an example for viewing the pin value over time. Every state update and pin read executed via external script or web app will print the value.
from farmbot import Farmbot, FarmbotToken

token = FarmbotToken.download_token("email", "password")
fb = Farmbot(token)
class MyHandler():
    def on_connect(self, bot, mqtt_client): pass
    def on_change(self, bot, state):
        print(state['pins']['63']['value'])
    def on_response(self, bot, response): pass
    def on_log(self, bot, log): pass
handler = MyHandler()
fb.connect(handler)

I am sorry to bother you once more. However, when I run this given code, the problem is the same.
The tool verification sensor value is updated only when I click the "Read Sensor" button in the Web App. Even after updating in the Web App, the sensor value does not get updated in the state tree. I have provided a screenshot of what I mean.

{"kind": "read_pin", "args": {"pin_number": 63, "label": "pin63", "pin_mode": 0}} is the command that is sent when bot.read_pin(pin_number=63, pin_mode='digital') is called. pin_mode: 0 corresponds to pin_mode='digital'. The resulting pin value from the read pin command appears in the state tree, which can be viewed via print(state['pins']['63']['value']) and will print as 1.0 or 0.0.
from farmbot import Farmbot, FarmbotToken

token = FarmbotToken.download_token("email", "password")
fb = Farmbot(token)
class MyHandler():
    def on_connect(self, bot, mqtt_client):
        # This is the command request sent:
        bot.read_pin(63)
        print('read pin command sent!')
    def on_change(self, bot, state):
        # This is the resulting value:
        print(f"pin value: {state['pins']['63']['value']}")
    def on_response(self, bot, response): pass
    def on_log(self, bot, log): pass
handler = MyHandler()
fb.connect(handler)

Thank you, that just solved a great deal of issues...
So we must read the pin using on_change(), and update the pin wherever we need to in the script.
I owe you a drink Gabriel


Improved vsync / vblank handling and frame detection

Frame detection currently ignores vsync and is solely based on vblank. This affects display in a numbe of games:

Summer Games, Winter Games (shifted title screen)
James Bond (no display at all)
Enduro (lower screen cut of)
Most tigervision games (lower screen cut off, flicker)

Solved in 3d7ebd62e754d9a4a95fa7b2d28ebdb497a67cf9


Terrain update ii

Seems like rotation for "decoration" in HexGrid.cs isn't set right, missing about 90 degress on the y-axis.

Seems like players get stuck on water tiles if they fall down into one. Players should either be able to leave water tiles, or alternatively sink and die, because no fate is worse than gettin permanently stuck.


feat: new browse()/browseFrom()/browseAll()

browse(query, queryParameters) now has the same signature than
search(). You can use any query and queryParameters.

browseFrom(cursor) can be used as an efficient way to
continue (next page) a previous browse() call. All browse responses now have a cursor property.

browseAll(query, queryParameters) can be used to get all
the content of your index
It returns an EventEmitter.
Available events:

result
end
error (you should listen to it or it will throw)

There's also stop() method on the event emitter so that you can
stop browsing at any point.

fixes #101

review @kokliKo @pixelastic (random!)


fig 1 color boxes

do you know how to make them go "under" the figures?
i think the fig would look better if each step had its own row, whose boundaries were the boxes.
in other words, make the boxes extend the width of the figure, and the panels overlaid on top.
do you know what i mean?

yah I know what you mean.
The easiest way I can think of, is to do it manually via paint.....
I will search and see if this can be achieved in tex.
In either case, I will show you a template figure in a few hours.

hmmm, I cannot find a way to do it in tex......
tried a cheap version in paint first just to separate rows (kind of like a white box..), see figure 2 as a temporary one.
If we are to have a colored box underlying each row, we may need to

further separate each row for better visual?
decide a good color as background, as we already have black/grey/green/blue in the figures (maybe light yellow?)

i don't see fig 2? where is it?

it is on page 4 of overleaf.

yah, let's definitely further separate each row.
i don't think we need a background color.
if we make a 4 row image in paint (or adobe illustrator),
we can just drop the current images in top...
On Thu, Dec 29, 2016 at 6:23 PM, cshen6<EMAIL_ADDRESS>wrote:

it is on page 4 of overleaf.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/MGC/issues/187#issuecomment-269709248, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACjcsxVZGfeYheqS6cQRUXzuAYd9K42ks5rNED3gaJpZM4LXVGT
.

--
the glass is all full:  half water, half air.
neurodata.io, jovo calendar
https://calendar.google.com/calendar/embed?src=joshuav%40gmail.com&ctz=America/New_York
starting in 2017, i will be responding to emails approximately weekly,
so that i can focus more on the people around me.
thank you for your understanding.

ok, no problem!
I will let you know once done; and I will probably put the caption text to a different page.

done, and removed old figure 1 for now.
Let me know anything to improve? maybe put (0) -(3) out of the box?

i don't love it.  i slacked you an image of a drawing.
does it make sense?
the text boxes are, in fact, the steps between the figures, so they should go there.
i think we can do the whole thing in matlab, possibly easier than not in matlab?
(we would just drop the text for "step 0", that is not our method).
what do you think?

I see what you mean now.
So we narrow the text box, make it go under the figure?
And still add a boundary including two figures & the text box for each row?
The steps could be a bit confusing though.
How about make the text box go above the two figures?
And I am not sure if we still have enough page space, once we make the text box vertical, because it is equivalent to add 4 more small panels vertically....

@gkiar - can we discuss design of this figure?
On Fri, Dec 30, 2016 at 7:11 PM, cshen6<EMAIL_ADDRESS>wrote:

I see what you mean now.
So we narrow the text box, make it go under the figure?
And still add a boundary including two figures & the text box for each row?
The steps could be a bit confusing though.
How about make the text box go above the two figures?
And I am not sure if we still have enough page space, once we make the
text box vertical, because it is equivalent to add 4 more small panels
vertically....
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/MGC/issues/187#issuecomment-269837660, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACjcqHeCFUYD5ZT4YKOfTIihab1F_bfks5rNZ3GgaJpZM4LXVGT
.

--
the glass is all full:  half water, half air.
neurodata.io, jovo calendar
https://calendar.google.com/calendar/embed?src=joshuav%40gmail.com&ctz=America/New_York
starting in 2017, i will be responding to emails approximately weekly,
so that i can focus more on the people around me.
thank you for your understanding.

@gkiar - can we discuss design of this figure?
On Fri, Dec 30, 2016 at 7:11 PM, cshen6<EMAIL_ADDRESS>wrote:

I see what you mean now.
So we narrow the text box, make it go under the figure?
And still add a boundary including two figures & the text box for each row?
The steps could be a bit confusing though.
How about make the text box go above the two figures?
And I am not sure if we still have enough page space, once we make the
text box vertical, because it is equivalent to add 4 more small panels
vertically....
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/MGC/issues/187#issuecomment-269837660, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACjcqHeCFUYD5ZT4YKOfTIihab1F_bfks5rNZ3GgaJpZM4LXVGT
.

--
the glass is all full:  half water, half air.
neurodata.io, jovo calendar
https://calendar.google.com/calendar/embed?src=joshuav%40gmail.com&ctz=America/New_York
starting in 2017, i will be responding to emails approximately weekly,
so that i can focus more on the people around me.
thank you for your understanding.


Add option to filter files using glob patterns

Fixes #10
Three new flags were added:

-g/--glob, which allows specifying a glob pattern to use when filtering files
--iglob, which is like --glob, but the pattern is case insensitive
--glob-case-insensitive, which makes all --glob to behave like --iglob

I couldn't think of a good way to test this under tree/test.rs (because the big code change is in cli), so the tests for this feature were added as E2E tests under /tests. They execute erdtree using cargo with the given args, and compare the output (with ansi escapes stripped) against string literals (de-intended using indoc).
Because the glob pattern may be invalid (see https://docs.rs/globset/latest/globset/#syntax), parsing Clargs is now fallible (done by replacing the implementation of From<&Clargs> with TryFrom<&Clargs>.

This looks good! Do tests pass for you locally by chance? It is failing here

Yes, they are passing locally. The globbing seems to work correctly, but the output lists the files in a different order. I added --num-threads 1 and --order filename to try make the order deterministic.

Fantastic, thanks for the feature add! I will include it in the release hopefully later this week.


Empty README on my fork

Overview
TL;DR

Questions
<If relevant, write a list of questions that you would like to discuss related to the changes that you have made.>
Next Steps
<If incomplete, create a task list of items that are still being worked on within the Pull Request.>
Review
<If complete, or ready for :eyes:, use @mentions for quick questions, specific feedback, and progress updates.>

Hi @SASS4! I agree with @MarkelFe. I think that we should probably keep the README.md.


⚠ Bofrost.at has degraded performance

In ce2ddb3, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1497 ms

Resolved: Bofrost.at performance has improved in 84867e3.


community of IIT Guwahati coding club is missing in the list.

-
+

+[__IIT Guwahati coding club__](https://discord.gg/9tSS6PEBpc) [](badges.md#official-identification-badge) [](https://www.iitg.ac.in/stud/gymkhana/technical/home/CodingHome.html)
+Notable Channels: `#friendly`, `#24/7 questions`, `#great community`, `#3500+`,`#good support`, `#iitguwahati`, `#competitive`, `#ds +and algo` \
+Language: English

I already added a pull request!


Apple Homepod does not play track from the beginning.

What version of Music Assistant has the issue?
2.0.4
What version of the Home Assistant Integration have you got installed?
2024.5.1
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Individual audio tracks, announcement tones and TTS messages do not play for the beginning. Instead they are played after a delay and at some point AFTER the track has begun playing.
For example, when using the mass.play_announcement service:
alias: Test MA announcement
sequence:
  - service: mass.play_announcement
    metadata: {}
    data:
      url: http://192.168.1.##:8123/local/audio/sounds/gentle-alarm.mp3
      use_pre_announce: true
      announce_volume: 60
    target:
      entity_id: media_player.kitchen

The sound was not heard. After further testing it became apparent that the audio file was too short to play. When a longer audio track was played it was hear but after some midpoint.
For example, with TTS (modified from the MU documentation page):
service: tts.google_say
data:
  entity_id: media_player.kitchen
  message: This is a test message to see if this is working

The message plays from “to see if this is working”. ie… “this is a test message” isn’t played at all.
Similarly, this file skips the beginning of the audio file.
 service: mass.play_media
metadata: {}
data:
  media_id: good morning voice message
target:
  entity_id: media_player.kitchen

The file is a 252 KB MP3 audio file.
This is the configuration within MA for TTS announcements but the issue is also there for individual sound files..

How to reproduce
Run the code below and note that the audio does not commence at the beginning as expected.
alias: Test MA announcement
sequence:
  - service: mass.play_announcement
    metadata: {}
    data:
      url: http://192.168.1.##:8123/local/audio/sounds/gentle-alarm.mp3
      use_pre_announce: true
      announce_volume: 60
    target:
      entity_id: media_player.kitchen

Note * .. file is too short to be played and ends before the audio is heard.

service: tts.google_say
data:
  entity_id: media_player.kitchen
  message: This is a test message to see if this is working

*Note.. The message plays from “to see if this is working”. ie… “this is a test message” isn’t heard.
 service: mass.play_media
metadata: {}
data:
  media_id: good morning voice message
target:
  entity_id: media_player.kitchen

Note.. the sound file plays AFTER the beginning.

Music Providers
Filesystem (local disk)
Player Providers
Airplay (through music assistant) on an homepod mini.
Following plays the entire TTS message and second sound file through homepod mini via home assistant:
service: tts.google_say
data:
  entity_id: media_player.home_pod_mini_kitchen
  message: This is a test message to see if this is working

service: media_player.play_media
metadata:
  title: gentle-alarm.mp3
  thumbnail: null
  media_class: music
  children_media_class: null
  navigateIds:
    - {}
    - media_content_type: app
      media_content_id: media-source://media_source
    - media_content_type: ""
      media_content_id: media-source://media_source/local/sounds
data:
  media_content_id: media-source://media_source/local/audio/sounds/gentle-alarm.mp3
  media_content_type: audio/mpeg
target:
  entity_id: media_player.home_pod_mini_kitchen

Full log output
MU_log.log
Additional information
I am running home assistant on proxmox on an intel NUC. Not sure that type of installation I have, though I am running the version with the add-ons store (I assume that is home assistant OS).
What version of Home Assistant Core are your running
2024.6.0
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

I am seeing more reports about issues with the HomePod. Can you confirm you can play music ok? Can you confirm if you play announcements to other player types then that works ok? Do you have Airplay players? If so, do they work ok?

Yes I can play music, though I generally listen to streams so the missed start is less of an issue.
I can play messages though the general HA HomePod entities, so that is what I am doing for the time being.
I have two HomePod mini’s and a HomePod. They all seem to be displaying this behaviour.

Please confirm if this is still an issue with 2.1.0

No worries. I'll do a check tomorrow. Have been using chime_tts.

There is still a delay with the alert tone (it plays perhaps the last one third). Also play is not resumed after the announcement has run.
service: mass.play_announcement metadata: {} data:   url: http://<IP_ADDRESS>:8123/local/audio/sounds/call-to-attention.mp3   announce_volume: 100   use_pre_announce: true target:   entity_id: media_player.kitchen
service: tts.google_say data:   entity_id: media_player.kitchen   message: This is a test message to see if this is working   cache: true

Can you double check that media_player.kitchen is the MA version of that device?

Thanks, I double checked and it is the MA entity (I checked via the MA integration). Attached is a screenshot of the logbook entry from within the MA integration together with a screenshot of the entity's announcement setting from within Music Assistant. Thanks a lot for the integration and add-on.

Yeah sorry can't reproduce. It works fine with my Chromecast devices. You will just have to be patient again until others who have a Homepod can have a look.

No problem. Thanks for the integration.

@steveof2620 is this still a problem with 2.2.4?

Hi, I have a similar issue, but with the Apple music provider; local samba share files play fine, but music from Apple Music provider starts playing a good few seconds into the track. This happens from MA's UI, so it's not specific to tracks being played from Home Assistant. I tried the following:

Play to Homepod mini
Play to Sonos speaker (airplay)
Play to Airplay group of speakers (the two above)
Play to Sonos speaker (sonos integration)

Only #4 plays the track from the beginning

Still happening with 2.3.0b25

Still an issue I'm afraid.

Keeping open


Ensure agent runs on Node 14

Fixes #123
Description
A very minor set of changes needed to remove some Node 16 specific syntax.
Will want to give it a bit more soak testing - but unit tests are clear and initial sniff tests using it all work.
Have also added Node 14 to the unit test runner matrix to ensure we don't regress it given we mostly work in a Node 16+ environment.
Checklist

[x] I have read the contribution guidelines
[ ] Suitable unit/system level tests have been added and they pass

/me shakes my fist at eslint and gets to work

The underlying cause of the failure, we believe, is the older version of npm in node 14 not installing peer dependencies properly for some of the latest eslint plugins - so this is only relevant for running the lint checks.
Fixed by disabling linting on the node 14 run - no technical reason to run it on multiple versions of node.


Making the project eligible for hacktoberfest

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Currently, the project is not eligible for the Hacktoberfest, it would be great to include this project with other amazing global open source projects.
Describe the solution you'd like
Add tags like hacktoberfest, hacktoberfest2020 to the repository.
Describe alternatives you've considered
Apparently, there are no alternatives to solve this issue.
Additional context
No

Thanks for the help, I added some tags in the r :+1: o :)

Awesome it looks great 🥇


Apt key can't be added "gpg: no valid OpenPGP data found."

Hi Folks,
Guessing that an issue might be on-going on the infrastructure side, the GPG key can't be retrieved at the moment.
For instance:
$ wget  -qO - https://adoptopenjdk.jfrog.io/adoptopenjdk/api/gpg/key/public | sudo apt-key add -

gpg: no valid OpenPGP data found.

With a get request:
$ http -Fh get https://adoptopenjdk.jfrog.io/adoptopenjdk/api/gpg/key/public

HTTP/1.1 503 Service Temporarily Unavailable
Connection: keep-alive
Content-Length: 190
Content-Type: text/html
Date: Tue, 12 Jan 2021 21:52:13 GMT

Duplicate of https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1830.

Duplicate of https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1830.


Failed to install HTML language server

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
After fresh install I'm getting: "Language server error: HTML
failed to execute npm install:
stdout: ""
stderr: "npm ERR! code 127\nnpm ERR! path /Users/m0xai/Library/Application Support/Zed/languages/vscode-html-language-server/4.6.0/node_modules/core-js\nnpm ERR! command failed\nnpm ERR! command sh -c node -e "try{require('./postinstall')}catch(e){}"\nnpm ERR! sh: node: command not found\n\nnpm ERR!      /Users/m0xai/.npm/_logs/2023-04-09T15_08_37_265Z-debug-0.log\n"
Environment
Zed: v0.80.5 (stable)
OS: macOS 10.15.7
Memory: 8 GiB
Architecture: x86_64
Node: v18.15.0
npm: v18.15.0
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
No response

Quit Zed. Run the following in a terminal of choice:
$ rm -rf ~/Library/Application\ Support/Zed/node
$ rm -rf ~/Library/Application\ Support/Zed/languages/vscode-html-language-server

Open Zed and try again. Does that resolve the issue?

Unfortunately it didn't
Language server error: HTML

failed to execute npm install:
stdout: ""
stderr: "npm ERR! code 127\nnpm ERR! path /Users/m0xai/Library/Application Support/Zed/languages/vscode-html-language-server/node_modules/core-js\nnpm ERR! command failed\nnpm ERR! command sh -c node -e \"try{require('./postinstall')}catch(e){}\"\nnpm ERR! sh: node: command not found\n\nnpm ERR!       This is weird as Zed manages Node for you in recent versions. Did you completely quit Zed before running both commands? Ensure the commands are run outside of Zed, that is, inside some terminal not being Zed's integrated one.
Would you also mind giving me the output from the following commands:
$ ls -la ~/Library/Application\ Support/Zed/node/
$ ls -la ~/Library/Application\ Support/Zed/node/*/bin

@hovsater
So after I quit zed, and ran the commands in your first comment here and then list commands, here is the output:
m0xai:~ % ls -la ~/Library/Application\ Support/Zed/node/
total 0
drwxr-xr-x 3 m0xai staff  96 Apr 14 20:29 .
drwxr-xr-x 5 m0xai staff 160 Apr 14 20:29 ..
drwxr-xr-x 9 m0xai staff 288 Apr 14 20:30 node-v18.15.0-darwin-x64
m0xai:~ % ls -la ~/Library/Application\ Support/Zed/node/*/bin
total 98888
drwxr-xr-x 6 m0xai staff      192 Apr 14 20:30 .
drwxr-xr-x 9 m0xai staff      288 Apr 14 20:30 ..
lrwxr-xr-x 1 m0xai staff       45 Apr 14 20:30 corepack -> ../lib/node_modules/corepack/dist/corepack.js
-rwxr-xr-x 1 m0xai staff 93111376 Mar  5 06:22 node
lrwxr-xr-x 1 m0xai staff       38 Apr 14 20:30 npm -> ../lib/node_modules/npm/bin/npm-cli.js
lrwxr-xr-x 1 m0xai staff       38 Apr 14 20:30 npx -> ../lib/node_modules/npm/bin/npx-cli.js
m0xai:~ %  rm -rf ~/Library/Application\ Support/Zed/node
m0xai:~ % rm -rf ~/Library/Application\ Support/Zed/languages/vscode-html-language-server
m0xai:~ % ls -la ~/Library/Application\ Support/Zed/node/*/bin
zsh: no matches found: /Users/m0xai/Library/Application Support/Zed/node/*/bin
m0xai:~ % ls -la ~/Library/Application\ Support/Zed/node/
ls: cannot access '/Users/m0xai/Library/Application Support/Zed/node/': No such file or directory
m0xai:~ %

And I started zed again, it opened my project, which is written in Angular and Java. When I open a template HTML file, I get again:
Language server error: HTML

failed to execute npm install:
stdout: ""
stderr: "npm ERR! code 127\nnpm ERR! path /Users/m0xai/Library/Application Support/Zed/languages/vscode-html-language-server/node_modules/core-js\nnpm ERR! command failed\nnpm ERR! command sh -c node -e \"try{require('./postinstall')}catch(e){}\"\nnpm ERR! sh: node: command not found\n\nnpm ERR!      /Users/m0xai/.npm/_logs/2023-04-16T16_29_32_051Z-debug-0.log\n

And here are the outputs after I quitted zed:
m0xai:~ % ls -la ~/Library/Application\ Support/Zed/node/*/bin
zsh: no matches found: /Users/m0xai/Library/Application Support/Zed/node/*/bin
m0xai:~ % ls -la ~/Library/Application\ Support/Zed/node/
ls: cannot access '/Users/m0xai/Library/Application Support/Zed/node/': No such file or directory
m0xai:~ % ls -la ~/Library/Application\ Support/Zed/node/*/bin
total 99136
drwxr-xr-x 6 m0xai staff      192 Apr 16 18:29 .
drwxr-xr-x 9 m0xai staff      288 Apr 16 18:29 ..
lrwxr-xr-x 1 m0xai staff       45 Apr 16 18:29 corepack -> ../lib/node_modules/corepack/dist/corepack.js
-rwxr-xr-x 1 m0xai staff 93111376 Mar  5 06:22 node
lrwxr-xr-x 1 m0xai staff       38 Apr 16 18:29 npm -> ../lib/node_modules/npm/bin/npm-cli.js
lrwxr-xr-x 1 m0xai staff       38 Apr 16 18:29 npx -> ../lib/node_modules/npm/bin/npx-cli.js
m0xai:~ % ls -la ~/Library/Application\ Support/Zed/node/
total 0
drwxr-xr-x 3 m0xai staff  96 Apr 16 18:29 .
drwxr-xr-x 5 m0xai staff 160 Apr 16 18:29 ..
drwxr-xr-x 9 m0xai staff 288 Apr 16 18:29 node-v18.15.0-darwin-x64

Typescript Language servers works without problem btw.

Hmm, I can't really re-produce this. I've tried all kinds of variations without replicating this behaviour. Is it still an issue on the latest version of Zed?

It's still an issue.

@hovsater installed Node manually and it works now.

That shouldn't really be needed as Zed ships with its own version of Node. I guess we need to look into this more closely.

I am having the same issue, and using the same fixed @hovsater mentioned, I can't seem to fix this issue. It is the exact same error code and those commands reproduce the same output

Hi,
Same issue here with the same debug logs.
Something weird was that the vscode-html-language-server folder was empty after supposed install.
I do not have Node installed on this computer (aside from zed's)
What worked for me was to add Zed's Node bin directly to my .zshrc :
export PATH=$PATH:/Users/username/Library/Application\ Support/Zed/node/node-v18.15.0-darwin-arm64/bin:$PATH
After that I closed Zed, ran :

$ rm -rf ~/Library/Application\ Support/Zed/node
$ rm -rf ~/Library/Application\ Support/Zed/languages/vscode-html-language-server

and after opening Zed, vscode html language server properly installed.
Environment
Zed: v0.90.2 (stable)
OS: macOS 13.4.0
Memory: 8 GiB
Architecture: aarch64

Same issue, it all just started today.
I have been working fine with all sort of files but now getting it only for HTML files, I've tried all the above and it still does it.

Hmm, I can't really re-produce this. I've tried all kinds of variations without replicating this behavior. Is it still an issue on the latest version of Zed?

DO you need anything from us to be able to troubleshoot?


Fixed a bug that appears when file doesn't include all data

Hi!
I found a bug from makeTreeSummarizedExperimentFromBiom. I have a biom-file and I wanted to create TSE from it. The problem is that it's not include sample metadata, i.e., colData. When TSE object is created NULL leads to an error. I fixed it so that now it is possible to create TSE object if biom-file includes only counts.
I was wondering if this should be constructed like loadMothur, i.e., there could be additional parameters for rowData and colData files.

User could choose if those files are provided.
First, rowData and colData are fetched from the biom-file.
If user has provided additional rowData and colData files, they replace those ones that was fetched from biom-file.
SE is created instead of TSE.

However, I'm not familiar with biom-format. I don't know if the file should normally include colData and rowData, or is my file just an exception and additional parameters are not needed.
-Tuomas

Hi Tuomas,
thanks for catching that and the PR.
I think it is only necessary to expand colData to a DataFrame and not rowData, since rowData = NULL is the default in the SummarizedExperiment constructor TreeSummarizedExperiment() redirects to.
Felix

Now empty data frame is added only to colData


Swagger to SDK Python

Works, example:
https://github.com/Azure/azure-rest-api-specs/pull/12252
that produces:
https://github.com/AzureSDKAutomation/azure-sdk-for-python/pull/3510


'Greeter' is not exported by app\ts\greeter.js

'Greeter' is not exported by app\ts\greeter.js (imported by app\ts\index.ts). For help fixing this error see https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module
export class Greeter {
    greeting: string;
    constructor(message: string) {
        this.greeting = message;
    }
    greet() {
        return "Hello, " + this.greeting;
    }
}

import { Greeter } from './greeter';
days wasted on rollup: too many

I am not sure about .ts stuff, but import { Greeter } from './greeter.js'; should import your class, see example

@kudorgyozo Protip 1:

days wasted on rollup: too many

...is not a good way to ask for help.
Protip 2, include a repro.

No repro, closing


Add bower.json

Adds a bower.json configuration file to the project.
PR in response to reference made by @posva in comment: https://github.com/vuejs/vue/issues/5029#issuecomment-283184944

I don't understand, I told you there's no bower.json on purpose

Derp having a long day.


Commenting is buried a little deep

If I want to make an edit to an entity (gene, variant, evidence) there is a direct edit button. But, to simply leave a comment, I need to go to the entity Talk page page, then to the Comments tab. If there are existing comments it is not super obvious that the empty Title/Comment boxes are for entering a new comment. There should maybe be comment button similar to edit button next to each entity. And, the comment section should have a heading that says something like: Please add a comment with the form below!

Yeah, I agree. Adding a subtle button in the header next to the title that links directly to the comments view sounds like a decent fix. Regarding the comment form layout, I'm going to copy the chat/comment style closer to the one we're using with github right now, and eliminate the title field.

bump

Close in favor of #802


Feature: simple captain app to keep track of the game and submit at end

It would be nice if captains could have an easy way to keep track of all their stats

Moving to discussion for now


OpenId Connect Changes

https://tools.hmcts.net/jira/browse/EM-1875

Just one question actually, why do you have all the userinfo stuff if you dont have a authorities extractor

Removed the UserInfoClient now


Bug found when 'from japronto import Application'

In example/8_html/html.py
With the name "html.py",running this script results "ImportError: cannot import name 'Application'".
When change it's name from "html.py" to "htm.py" or anything else,script runs with no errors.

It should be a namespace problem. html probably is used in another module.
Em 27/04/2017 04:59, "ghosttk"<EMAIL_ADDRESS>escreveu:

In example/8_html/html.py
With the name "html.py",running this script results "ImportError: cannot
import name 'Application'".
When change it's name from "html.py" to "htm.py" or anything else,script
runs with no errors.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/squeaky-pl/japronto/issues/71, or mute the thread
https://github.com/notifications/unsubscribe-auth/AZlo55wB57RT4xboMAaiROIkwcDl3Jv4ks5r0ErbgaJpZM4NJ3ZF
.

This makes sence,I found a python3 buildin module named 'html'.

Fixed as of 866bca3d910f8aee788f6861ec3770b4f09c1ae9

This issue happened with me too. The problem was that my script file was named as japronto.py. Renamed to a different name than the package and it worked.


Scheduler Systemtest Framwork (with k8s)

The PR contains the following:

Creation of scheduler abstraction to plug-in suitable schedulers like swarm , mesos , k8 to contiv system framework
Currently contains environment for  docker and k8s
K8 test environment which brings up k8s nodes with contiv binaries. This also has dev env changes for compiling and  folder sync.

Changes to follow this PR:
L3 test integration
Service test integration
Replication controller changes
Changes to be made:

Ipv6 changes where ever necessary for k8
ACI changes (ACI test cases are commented out - waiting for the changes)
Swarm integration
Mesos integration

LGTM, other than the minor comments.


python -m not finding function for get_read_ctg_map

Hi,
First off, thanks a ton for all your work on FALCON and FALCON_unzip!  I've been using FALCON for about a year now, and the results are great!
--Note--
This may be related to FALCON issue #499: https://github.com/PacificBiosciences/FALCON/issues/499
I've run into a series of issues trying to run FALCON_unzip on one of my assemblies (genome size estimate of between 380 and 430 Mb, but the p_ctg.fa comes out near 580-590 Mb, so I suspect haplotigs are being kept together in p_ctg.fa).
The first error involves a Python exception being thrown (an AttributeError) that the 'module' object does not have attribute 'get_read_ctg_map'.  This seems to trace back to the python -m get_read_ctg_map call in track_reads.sh generated by task_track_reads() in unzip.py.  I've also tried manually calling python -m get_read-ctg_map with and without deleting mypwatcher and other relevant directories and files, basically resetting the FALCON_unzip run to 0.
What did succeed from there was running the DBshow commands manually, and running the (slightly-modified) contents of the generate_read_to_ctg_map() function.
After that, I'm able to run python -m rr_ctg_track and python -m pr_ctg_tracksuccessfully, but then hit another error running python -m fetch_reads complaining about input.fofn not existing as a file.  I used a non-canonical input FOFN name, and included this in my fc_unzip.cfg, but it doesn't seem to be used in task_track_reads() in unzip.py (should be easy to tack into the config[] dictionary in main() in unzip.py, assign it to a variable in task_track_reads(), and then throw that after a --fofn in the script definition for track_reads.sh).
And a third note (not an error, but a suggestion), maybe pypeFLOW already does this, but it'd be good to avoid repeating get_read_ctg_map, rr_ctg_track, pr_ctg_track, and fetch_reads if their output files already exist.  Since I manually ran the steps, as soon as I tried to re-run fc_unzip.py to get to alignment and phasing, FALCON_unzip tried to redo all the steps I had run manually, and failed again on get_read_ctg_map.
Version info:
I tried first with FALCON_unzip c936cb9 on the p_ctg.fa generated by FALCON 0.4.0 (don't remember which commit), and then installed the newest via FALCON-integrate a few days ago and re-ran to see what might be happening.  Newest versions used:
FALCON-integrate: 3b36fd91d36149ec000ab35fbd1e8b7646f8e95c
FALCON: 87b6262607a885979727592c5aa0dad459085f33
FALCON-make: 02ed9dedb0d5bc7bec75bfad8aa0a92629ad0099
FALCON_unzip: c936cb94a63cc763f9733d2f497ccc00a68ba02c
pypeFLOW: bdd41dbc5ac4e4cf1dec1fa57e52850b399d85aa
Install was via PYTHONUSERBASE, I'm using SLURM, and I unset PYTHONPATH and source env.sh prior to running fc_unzip.py each time.  I've tried with Python 2.7.3 and 2.7.12 to see if that makes a difference (maybe they fiddled with the handling of the -m flag?).
Potential source for the first error:
Comparing the scripts getting called with python -m, it looks like get_read_ctg_map.py is the only one that uses any makePypeLocalFile() calls and runs functions via a PypeTask.  Is this even necessary?  It seems that get_read_ctg_map.py isn't terribly computationally intensive compared to the others, so adding extra Pype tasks seems an unnecessary additional complexity.  Attached is an example (and apologies, but I forgot to remove the PypeProcWatcherWorkflow part).
get_read_ctg_map.py.txt
Again, thanks for your time and all your great work!
Best regards,
Patrick Reilly

No, PypeTasks aren't really needed for that script. But it should still run. You have some kind of version mis-match. I see this:
python -m falcon_kit.mains.get_read_ctg_map

You report this:
FALCON_unzip: c936cb9

I do not believe that you are actually using that version.
26034b9f falcon_unzip/unzip.py (Christopher Dunn 2016-11-21  48) python -m falcon_kit.mains.get_read_ctg_map

That commit is included under yours:
* 1820d82 (HEAD, origin/master, origin/HEAD, master) Rm copyrighted script
* c936cb9 Drop FALCON submodule
* b04f104 Drop TaskBase, URL from PypeTask
* 8e4f06e Moved README into wiki, so we do not need to edit the code-tree
*   22c492e Merge branch 'simple' into master
|\
| * d2dd81a Update README.md
| * 1bb0010 Fix some task dependencies
| * a532bb4 unzip works now
| * aa0700b fix read_map dirs
| * 7a10666 Drop setNumThreadAllowed, old pypeflow use
| * e1d73e6 PEP-8 spacing
| * 60d88aa single quotes
| * 26034b9 simpler script def
...

So I think you have an integration problem, which is by far the most difficult thing for us to address remotely.
Note the FALCON-integrate/FALCON-make does not yet install FALCON_unzip. You still have to do that yourself. (I will address that this weekend.)

Sorry, that was a typo on my part, I did run python -m falcon_kit.mains.get_read_ctg_map and it produced the AttributeError as stated above.
Here's the first commit from the output of git log in my FALCON_unzip directory (which I had to run python setup.py install --prefix=[path to fc_env subdirectory of FALCON-integrate folder] in after installing FALCON-integrate, and don't see any errors from python setup.py install):
`$ git log
commit c936cb94a63cc763f9733d2f497ccc00a68ba02c
Author: Christopher Dunn<EMAIL_ADDRESS>Date:   Sun Nov 27 12:19:30 2016 -0600
Drop FALCON submodule

`
Here's the command and output generated by running that manually:
$ python -m falcon_kit.mains.get_read_ctg_map WARNING:pypeflow.simple_pwatcher_bridge:In simple_pwatcher_bridge, pwatcher_impl=<module 'pwatcher.fs_based' from '/Genomics/grid3/users/preilly/bin/FALCON_0.7.0/FALCON-integrate/pypeFLOW/pwatcher/fs_based.pyc'> ERROR:pypeflow.simple_pwatcher_bridge:Task Node(2-asm-falcon/read_maps/dump_rawread_ids) failed with exit-code=256 ERROR:pypeflow.simple_pwatcher_bridge:Some tasks are recently_done but not satisfied: set([Node(2-asm-falcon/read_maps/dump_rawread_ids)]) ERROR:pypeflow.simple_pwatcher_bridge:ready: set([]) submitted: set([Node(2-asm-falcon/read_maps/dump_pread_ids)]) Traceback (most recent call last):   File "/usr/local/python/2.7.12/lib/python2.7/runpy.py", line 174, in _run_module_as_main     "__main__", fname, loader, pkg_name)   File "/usr/local/python/2.7.12/lib/python2.7/runpy.py", line 72, in _run_code     exec code in run_globals   File "/Genomics/grid3/users/preilly/bin/FALCON_0.7.0/FALCON-integrate/FALCON/falcon_kit/mains/get_read_ctg_map.py", line 137, in <module>     main()   File "/Genomics/grid3/users/preilly/bin/FALCON_0.7.0/FALCON-integrate/FALCON/falcon_kit/mains/get_read_ctg_map.py", line 134, in main     get_read_ctg_map(rawread_dir=rawread_dir, pread_dir=pread_dir, asm_dir=asm_dir)   File "/Genomics/grid3/users/preilly/bin/FALCON_0.7.0/FALCON-integrate/FALCON/falcon_kit/mains/get_read_ctg_map.py", line 96, in get_read_ctg_map     wf.refreshTargets() # block   File "/Genomics/grid3/users/preilly/bin/FALCON_0.7.0/FALCON-integrate/pypeFLOW/pypeflow/simple_pwatcher_bridge.py", line 210, in refreshTargets     self._refreshTargets(updateFreq, exitOnFailure)   File "/Genomics/grid3/users/preilly/bin/FALCON_0.7.0/FALCON-integrate/pypeFLOW/pypeflow/simple_pwatcher_bridge.py", line 277, in _refreshTargets     raise Exception(msg) Exception: Some tasks are recently_done but not satisfied: set([Node(2-asm-falcon/read_maps/dump_rawread_ids)])

python setup.py install

It is possible that you system install is not being updated. You might need --force. Or, if you are using a virtualenv, just delete it and re-install everything.
I recommend using pip install --edit. That uses "edit" mode, which means that only a kind of symbolic link (egg-info) is installed. Then, whenever you update pure python code, you do not necessarily need to re-install.

I have reproduced this locally. I see the problem:
$ cat 2-asm-falcon/read_maps/dump_rawread_ids/task.json
{
    "inputs": {
        "rawread_db": "/home/UNIXHOME/cdunn/repo/localhost/unzip/iter/0-rawreads/raw_reads.db"
    },
    "outputs": {
        "rawread_id_file": "rawread_ids"
    },
    "parameters": {},
    "python_function": "__main__.dump_rawread_ids"
}

__main__ needs to be an actual module. Working on it...

Awesome, thanks!

But that's not the only problem. I will push an update to FALCON-integrate in a few minutes...

1.8.5 fixes a couple other things too. Passes Quiver for me now.


Posterior weighted allele counts

See #98 for background
Consider adding an optional field of length "A" (one value for each allele) which contains the dosage of each allele weighted by posterior probability of each genotype for a sample.
This would be interpreted as a floating point best estimate of allele counts in each individual and could be summed over all individuals to to produce an approximation of allele frequencies in the population.
An example of calculating this over a full posterior distribution may look like this:
@njit(cache=True)
def dosage_posterior(posteriors, ploidy, n_alleles):
    n_genotypes = len(posteriors)
    dosage_posterior = np.zeros(n_alleles, dtype=np.float32)
    genotype = np.zeros(ploidy, np.int64)
    for i in range(n_genotypes):
        p = posteriors[i]
        for j in range(ploidy):
            dosage_posterior[genotype[j]] += p
        increment_genotype(genotype)
    return dosage_posterior

But this could be calculated more efficiently directly from the trace of an MCMC.

Best to add two options here:

AFP for allele frequency posteriors which is useful for sgkit
DS for posterior dosage of alternate alleles (drop the reference) which is what Jeff E. group are using in GWASpoly


consul_config: Add "node_id" option

Fixes #418
node_id option is available in Consul 0.7.3+

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@08e704a). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #420   +/-   ##
=========================================
  Coverage          ?   58.38%
=========================================
  Files             ?        7
  Lines             ?      346
  Branches          ?        0
=========================================
  Hits              ?      202
  Misses            ?      144
  Partials          ?        0

Impacted Files
Coverage Δ

libraries/consul_config.rb
77.5% <100%> (ø)

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 08e704a...06d5531. Read the comment docs.


iOS: incorrect layout and tapping on stories does nothing after restore

I originally posted this on #1216 but the changes that closed that issue do not seem to have resolved this one.
After restoring the IOS app directly into the stories view, it seems to restore into a hybrid of the feeds/all stories and folder stories view. In this view, clicking on stories on either side does nothing other than gray them out.
I took a video with the previous version of NewsBlur but the current Git version (27d2384cf15d36768eb940047c002a9494da612b) does the same thing.

Yes I've noticed this too when returning to the app


Seeking clarification about resnet18/densenet121 and the original papers

Hi,
I read the paper "Realtime Multi-Person 2D Pose Estimation using Part Affinity Fields" (https://arxiv.org/pdf/1611.08050.pdf)
and it does not mention about resnet18 or densenet121. I was wondering why this implementation needs pre-trained resnet18 or densenet121 model?
Or is this implementation based on the paper "Simple Baselines for Human Pose Estimation
and Tracking" (https://arxiv.org/pdf/1804.06208.pdf). The code seems to use the idea of Part Affinity Fields as mentioned in the first paper.
Can you please clarify?
Thanks for the great work!
Tareq

Hi @tareq992403 ,
Thanks for reaching out!
You’re correct!  This project is largely a combination of the ideas in each paper, with some other additional architecture tricks applied on the head of the model.
Please let me know if this helps or you have any questions.
Best,
John

Hi @jaybdub,
Thanks for the quick reply. As the code does not have detailed comments, can please share any documentation or blog on this implementation? This will help us to understand the code better.
Thanks,
Tareq

After the backbone pretrained RenNet18, do you feed the features to the two-branch multi-stage CNN for heatmap and PAF generation as shown in Fig. 3 of "Realtime Multi-Person 2D Pose Estimation using Part Affinity Fields" (https://arxiv.org/pdf/1611.08050.pdf)?
I am confused. Can anyone help? I saw several requests about the explanation of this architecture, so this will help many of us.
Thanks, Tareq

@tareq992403 you should read this article https://www.geeksforgeeks.org/openpose-human-pose-estimation-method/

Hi @tareq992403. OpenPose released a newer paper https://arxiv.org/abs/1812.08008. Are there any advantages of the two-branch approach ((https://arxiv.org/pdf/1611.08050.pdf) over the one branch (https://arxiv.org/abs/1812.08008)?


pkg/tar: chroot into rootfs, then extract

Extract manifest first, then chroot into "rootfs" to extract
remaining files.
Assumes normal ACI image layout
Fixes #3565

Please help modifying unit tests.

@squeed PTAL

ok to test

Two things here:

ExtractTarInsecure is an exported function and already has external consumers relying on its unprivileged/insecure behavior. Can we keep the changes in here to a minimum non-breaking delta, and have the rest of the logic in chroot.go?
before chrooting, you should take an fd to current root and restore when done. See here for an existing example, which perhaps can be moved to pkg.

Ok! I will work on using (and modifying) PathWhiteListMap parameter to achieve this. Will it be ok? Because previously, I tried using pwl to extract rootfs/ only but it would only extract the directory (not its contents)

Directly in extractTarCommand? I'm not sure what's your plan for it, as it seems to be a path-list instead of a prefix-list as you'd need. I fear it is quicker/cleaner to share away some of the common logic and then having an additional ChrootExtractTar routine.

Sorry, but I'll be away for month or two. Should this issue be closed? Or open if some one else would like to take it up?

@cshubhamrao no problem, I'm leaving this one open for history tracking. You can restart it once you are back, and other contributors as well are encouraged to jump on it in the meanwhile.

Hi @cshubhamrao, are you still interested in pushing this forward?


Auditing Information for Complicated Pricing Calculation

Once the CCXT plugin is up and running it would be useful to be able to have some kind of paper trail exist on how the price routing occurred.
For instance, how the CCXT converter plugin found the fiat value of BETH, so that it can be available for the report that is generated in RP2. This would entail listing the market, exchange, and candle where the price was retrieved for each 'hop'. Something like the following:

BETH -> ETH, Binance.com, 5 minute candle starting at Jan 4, 2022 at 12:53 UTC.
ETH -> USDT, Binance.com, 1 minute candle starting at Jan 4, 2022 at 12:53 UTC.
USDT -> USD, Kraken.com, 1 minutes candle starting at Jan 4, 2022 at 12:53 UTC.

This would be useful if a detailed audit did occur and the user would have to explain how they came about the pricing. This hop data would need to be stored in what is returned from get_historic_bar_from_native_source() (I think). Maybe add it to HistoricalBar or an extended class for 'routed' pricing?

Completely agree. I think the information should end up in one of the following (perhaps both):

the Notes field;
the log (INFO level).

It may be enough to add an optional Notes field to HistoricalBar, populated with this information. Then the transaction resolver can append the HistoricalBar notes to the transaction ones.

The Notes field is good I think. Users can then optionally pull a more detailed, longer report with this included if anybody needs it, otherwise they can just use the standard report.


Outdated version of black does not support f-strings

This is likely the root cause of #517 and #513. black upstream only recently added support for f-strings (see https://github.com/psf/black/issues/3746), and bug fixes are only available in more recent releases (for example, see https://github.com/psf/black/releases/tag/24.8.0). It would be nice if the bundled version (which is now ~half a year out of date) were also updated.
For others that stumble upon this, I diagnosed this when I noticed the formatter no longer appeared to be working, and saw Oh no! 💥 💔 💥 1 file failed to reformat. in the "Output" panel for the Black Formatter extension, and worked around this by installing a newer version of black and manually setting black-formatter.path.

fixed via https://github.com/microsoft/vscode-black-formatter/pull/540

This should be available in version 2024.3.12751822 of the extension.


Ensure index:only on remote repos find-refs queries

We have index:only when looking for refs in the current repo, but we don't set it when looking at remote repositories.

Seems reasonable. What do you think about ways to test this?

Use nock to mock the API. I'll add tests in another effort. I know there's some lacking coverage in the search code.

Do we really need to mock the whole API? I think the only thing we care to test here is whether we are generating the right query strings. Can we have a test ram some data down into the method and see what falls out—at most maybe we need a little stub to capture the results?
Agreed it can be a separate pass anyway.

Do we really need to mock the whole API?

No, just the sourcegraph.command.executeCommand function, which we already have util functions for (in some other package). The LSP integration tests already do this stubbing.


Converted NER with transformers example to keras_core

Awesome, made the changes accordingly and also ran the code in notebook environment works fine !


support for Update-DbContext command

Here is a typical scenario: SQL Server DB, EF data access layer and Web API JSON web service.
And just one, quite common requirement: some table/entity fields shall not be returned by web service. They are for internal use only so they still must be present in generated entities.
The easiest solution is to decorate entity fields which are to be hidden with attribute like [JsonIgnore].
Since then, any DB changes have to manually reflected in DB context code since Scaffold-DbContext command will wipe out previously set attributes. That is not good.
Another intuitive solution: inherit from generated entity class, add properties which are to be hidden with "new" access modifier (unfortunately generated properties are not virtual) and [JsonIgnore] attribute, use in web service instead.
Obviously, the above will not work. That is for two reasons: just adding such inherited class DbSet<> will cause "Invalid column name 'Discriminator'." exception. Second: DbSet methods will fail during runtime if fed with such inherited object. Why they are defined like:
public abstract class DbSet<TEntity> where TEntity : class
{
   public virtual EntityEntry<TEntity> Add(TEntity entity);

instead of:
   public virtual EntityEntry<T> Add<T>(T entity) where T : class, TEntity;
}

Would not that make programmer's work easier?
The entire solution seems too 'type constrained' for no apparent reason. But perhaps it is too late for such design change.
So please, at least implement Update-DbContext command,

Duplicate of https://github.com/aspnet/EntityFramework/issues/831


error while Generate googleapis gRPC source code

protoc --proto_path=.:/usr/local/include --cpp_out=./gens --grpc_out=./gens --plugin=protoc-gen-grpc=/usr/local/bin/grpc_cpp_plugin google/maps/routes/v1/route_service.proto
[libprotobuf FATAL /home/maybe/grpc/third_party/protobuf/src/google/protobuf/io/printer.cc:142]  Unclosed variable name.
terminate called after throwing an instance of 'google::protobuf::FatalException'
what():   Unclosed variable name.
--grpc_out: protoc-gen-grpc: Plugin killed by signal 6.
Makefile:44: recipe for target 'google/maps/routes/v1/route_service.pb.cc' failed
grpc version: v1.27.3
protoc version: 3.11.2
ubuntu: 18.04

That seems like a bug in gRPC or Protobuf, have you reported the bug with them?
@tmatsuo do we have any examples in this repo that compile all the protos in googleapis?  If so, we should clean that up.

Currently the speech sample generates google/maps/routes/v1/route_service.pb.cc with gRPC 1.26.

now is it solved?

now is it solved?

I think the original bug report was trying to use the grpc plugin distributed with gRPC v1.27.3, which for whatever reason was crashing. Downgrading to v1.26.x seems to work.

grpc version: v1.34 (latest)
protoc version: 3.13.0
ubuntu: 18.04
I am still getting this error while googleapis github repo
Any update???
step:

export LANGUAGE=cpp

git clone https://github.com/googleapis/googleapis.git

cd googleapis

make all

protoc --proto_path=.:/usr/local/.local/include --cpp_out=./gens --grpc_out=./gens --plugin=protoc-gen-grpc=/usr/local/.local/bin/grpc_cpp_plugin google/maps/routes/v1alpha/route_service.proto [libprotobuf FATAL /usr/local/grpc/third_party/protobuf/src/google/protobuf/io/printer.cc:142]  Unclosed variable name. terminate called after throwing an instance of 'google::protobuf::FatalException'   what():   Unclosed variable name.
any help

grpc version: v1.34 (latest)
protoc version: 3.13.0
ubuntu: 18.04
I am still getting this error while googleapis github repo
Any update???

I just compiled a similar configuration successfully using the Dockerfile at the end of this comment.  I suspect you picked a bad version of googleapis or you are missing some build step.

step:

export LANGUAGE=cpp
git clone https://github.com/googleapis/googleapis.git
cd googleapis
make all

I recommend that you do not use the top-level Makefile in that repository. It generates all the .pb.* files, but (a) you rarely need all of them, and (b) it does not produce any useful libraries that you can link against.  It is better to use the libraries in https://github.com/googleapis/google-cloud-cpp If that repository does not generate the protos you need, let us know (file a bug in that repository) in many cases we can add the additional protos. If all fails, create your own Makefile with just the protos you need.

protoc --proto_path=.:/usr/local/.local/include --cpp_out=./gens --grpc_out=./gens --plugin=protoc-gen-grpc=/usr/local/.local/bin/grpc_cpp_plugin google/maps/routes/v1alpha/route_service.proto [libprotobuf FATAL /usr/local/grpc/third_party/protobuf/src/google/protobuf/io/printer.cc:142] Unclosed variable name. terminate called after throwing an instance of 'google::protobuf::FatalException' what(): Unclosed variable name.

Is there a particular reason you need to compile that file?  Can you just skip it, or better yet, compile what you need?

any help

See below.
 I suspect you picked a bad version of googleapis or you are missing some build step.

I was following those steps from https://github.com/GoogleCloudPlatform/cpp-samples/blob/master/speech/api/README.md

I recommend that you do not use the top-level Makefile in that repository. It generates all the .pb.* files, but (a) you rarely need all of them, and (b) it does not produce any useful libraries that you can link against. It is better to use the libraries in https://github.com/googleapis/google-cloud-cpp If that repository does not generate the protos you need, let us know (file a bug in that repository) in many cases we can add the additional protos. If all fails, create your own Makefile with just the protos you need.

I want to achieve speech to text and text to speech in cpp using googleapis. (I am using grpc::GoogleDefaultCredentials())
yes,  don,t need all .pb, this is helpfull. Thanks!
Thanks for docker command
can u suggest anything else for speech to text and text to speech in cpp using GCP??

can u suggest anything else for speech to text and text to speech in cpp using GCP??

I am afraid using the raw gRPC files is the best I can recommend at this time.

This still appears to be an issue with protoc 3.14.0 and grpc 1.35.0 on Ubuntu 20.10
protoc --proto_path=.:/usr/local/include --cpp_out=./gens --grpc_out=./gens --plugin=protoc-gen-grpc=/usr/local/bin/grpc_cpp_plugin google/maps/routes/v1/route_service.proto [libprotobuf FATAL /home/steve/simcortex/grpc/third_party/protobuf/src/google/protobuf/io/printer.cc:142]  Unclosed variable name. terminate called after throwing an instance of 'google::protobuf::FatalException'   what():   Unclosed variable name. --grpc_out: protoc-gen-grpc: Plugin killed by signal 6. make: *** [Makefile:45: google/maps/routes/v1/route_service.pb.cc] Error 1


pull request for e2e test at 12/12/2016 05:15:02

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Problem with DP progress

Describe the bug
Hello, I have a problem with the data points:
spotify-premium.0.player.progress
spotify-premium.0.player.progressMs
spotify-premium.0.player.progressPercentage
They only show a value every 10 seconds. In the time in between there is zero in the DP.
To Reproduce
Steps to reproduce the behavior:

Go to 'Play an Track'

See error in DP

Expected behavior
Permanent display of the playing time
Screenshots & Logfiles
A short Screenrecord to show the Problem.
https://youtu.be/5yiIYjP7KnY
Versions:

Adapter version: 1.2.2
JS-Controller version: 4.0.23
Node version: 14.19.1
Operating system: Debian 11

+1

For your information: In version 1.2.1 the DP spotify-premium.0.player.progressPercentage still works normally. Only from V1.2.2 up to V1.3.1 the progressPercentage has this bug that it jumps back and forth between value and zero (null).

Duplicate #339 with a fix ready :)

fixed with 1.4.0


How to autoreload xresources when I log in Ubuntu

When I log in, some settings change default and I must reload xresources when i turn on ST terminal. How to solve that ???
P/s : actually I don't know what xresources is so I don't know solve that :( Do you have a discord sever? Plz send me, I need your help :<

@phanlong2811 siduck76#5226 is my discord account. To reload xresources , you need to add xrdb merge xresources ( xresources from repo ) , at your autostart options . move the xresources file in your home folder so it gets easy . Idk much gnome!


Add @the-ethan-hunt as collaborator?

Hello @keon,
Just wanted to check if we could add @the-ethan-hunt as a collaborator?
It will allow faster contributions.
He has made ample contributions imho, refer author contributions

@NirantK , I am already a contributor. ( Please see my comments on various issues; I have a contributor tag) 😅 . Maybe you meant a collaborator?

@NirantK , I am already a contributor. ( Please see my comments on various issues; I have a contributor tag) 😅 . Maybe you meant a collaborator?

Pinging @keon for this. :sweat_smile:

Sorry for the delay :(
@the-ethan-hunt has been contributing a lot lately.
Thanks, and I invited you as the collaborator.

Welcome @the-ethan-hunt !

Thank you for the warm welcome @NirantK ! Looking to contribute more to awesome-nlp :tada:


Some 7tv colors showing shadow instead of glow

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
Some random 7tv name colors dont show glow but shows a dropshadow instead.
Most name colors display glow correctly however.
Correct glow in first screenshot from browser, second screenshot from latest stable chatterino 7tv fork.
The one in the screenshot is "Rosé"
Confirmed its still an issue in latest nightly build as of Oct 03
Screenshots

OS and Chatterino Version
Chatterino 7.5.1 (commit b281c60e) built with Qt 6.5.0, MSVC 193833135 Running on Windows 11 Version 23H2, kernel: 10.0.22631

Some random 7tv name colors dont show glow but shows a dropshadow instead.

There are no glows in 7TV paints - only drop shadows.

This is more of an issue with Qt's pixmap filters not working the same as CSS filters, I think.

Yeah sorry thats what i meant, effectively the shadow "looks like" glow


Translation blog Weekly-Kubernetes-Community-Hangout

Blog translation /blog/_posts/2015-08-00-Weekly-Kubernetes-Community-Hangout.md
Chinese tranlation task: https://github.com/k8smeetup/website-tasks/issues/3361

/lgtm
/approve


Update Guillaume's title in the About Us page

Hey @gcharest! I know you don't care about this stuff but I feel like your title on the team page should reflect your recent promotion.
Feel free to close this PR if you'd rather leave your title as is. :)

Hahaha, thanks for thinking about this!! :smile:


Moving AtomicBlocks

To be able to drag and drop atomic blocks around within the editor, it is necessary to have a way to move them within the blockMap. To achieve this goal, this pull request implements two methods in AtomicBlockUtils: moveAtomicBlockBefore and moveAtomicBlockAfter.
I was not completely sure how to name the new EditorChangeType for this action. Feel free to find some better names...
Would be great to see this feature in the master!
Cheers, delijah

I'd say move forward with the DraftInsertionMode approach -- I think it sounds like a good way to do this. :)

Hey!!! I've just rebased this branch with master, obviously github considers this as closed since i had to remove the remote to push it again :( I guess i have to open another pull request...


There is another similar project you may wish to check out.

Hello,
Not sure if this is being actively maintained or not, it doesn't look like it. I just wanted to let you know that this package inspired me to add Lighthouse support to another Ethereum 2.0 Validator Ansible Role.
You can find it here: https://github.com/MaximilianMeister/ethereum-validator
Lighthouse support has not been merged yet but it is coming soon.
We would love to have your assistance over on that project if your still looking to work on this or something similar
Cheers,
BU

Hey, thanks for reaching out! The lack of updates is because everything has been running smoothly, but soon I'll be working on a branch for prater testnet to prep for the merge.


update code to support simple-oauth2 from 2.5.2 to 4.1.0

Bumped simple-oauth2 from 2.5.2 to 4.2.0 in v0.3.1 (https://github.com/DrozmotiX/ioBroker.tado/commit/b9b75c2020e0c8fa0ada8b491a5f4ba25a63446e)


Add about us to navbar or footer

I think it would be nice to add an introduction to the team accessible from either the navbar or the footer.
I only noticed the about section through a hackerrank post, and think it could be great for easier access through the website directly as a link, might be stupid and just could not find it, or you've already discussed this.
These are just some small options and opinions on how it could possibly be implemented:
Navbar
<Link href="/about" legacyBehavior>
    About
</Link>

or in the Footer ( under company )

    <Link href="/about" legacyBehavior>

            About

    </Link>

Hi, I would like to resolve this issue


chore(docs): remove samples

Follow-up on #305 — send cl/520747639 to address sample includes, so do not merge until snippetbot is happy. Thanks!

Submitted cl/520747639 so this is ok to merge whenever snippetbot is happy.


Fix mobile view

Reduce font-size in mobile (100% -> 90%)
Fix top navigation
Fix buttons at the top
Fix footer

lgtm