Would it be possible to use my own entities?

I have my own entities extracted through another model.  How easy would it be to apply DEE to these external entities using this work?  I see that the entities and events share an lstm at the beginning of processing.

Hi, there. Do you mean you want to 1) skip the entity extraction part and use extracted entities directly or 2) add additional entities while training?

First.  Skip entity extraction because I already have my own entities
detected and these entities come from a custom set of entity classes.
-Greg
On Wed, Dec 29, 2021 at 8:00 AM Tong Zhu（朱桐） @.***>
wrote:

Hi, there. Do you mean you want to 1) skip the entity extraction part and
use extracted entities directly or 2) add additional entities while
training?
—
Reply to this email directly, view it on GitHub
https://github.com/Spico197/DocEE/issues/6#issuecomment-1002582875, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ATBXGBNGGPMKPKKDGMY3BEDUTMA6HANCNFSM5K55CAJA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

I see. You can follow the steps as below:

Build your own dataset which includes all the entities extracted by another model.
Change the event template in dee/event_types. Add the customised entity type into common_fields (or FIELDS if you have your own event template).
Train the model. If you don't want to do the NER part again, change the get_mix_loss function in model files.
During evaluation (prediction), make sure you pass the use_gold_span=True when calling model.forward function.

I'd like to close this inactive issue. Feel free to open it if the above discussion does not solve your question.


Add categorical support for pandas and dask backend

Can now discover pandas dataframes with categorical dtype. Also implemented discover for dask.dataframe.DataFrame and dask.dataframe.Series.

One request, please make a whatsnew/0.5.1.txt document and add a bullet point for this PR.
Overall, LGTM.
@llllllllll you've mentioned categorical support in the past, I believe -- can you take a look?

Just the one comment about the tests. Looks good, thank you

Fixed tests and added whatsnew page. Good to merge?

Yup


[promptflow][fundamental] Exclude PF Azure test from current sdk-cli-test workflow

Description
Exclude sdk_cli_azure_test from sdk-cli-test workflow, as we have replay tests and daily live test to cover.
All Promptflow Contribution checklist:

[x] The pull request does not introduce [breaking changes].
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.
[ ] Create an issue and link to the pull request to get dedicated review from promptflow team. Learn more: suggested workflow.

General Guidelines and Best Practices

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

Testing Guidelines

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

#1211 will exclude as what this PR does, close this one.


Use attrs<19.2.0 with pytest

to avoid this error https://github.com/pytest-dev/pytest/issues/3280.

/test


move repo to chihacknight?

Now that we have an org for Chi Hack Night, feel free to move this repo over there if you'd like. @stevevance I believe you have the admin privileges.

@zorostang Can you move this over to the ChiHackNight?


Set proper native codeaction previewer for native profiles

The codeaction issue made me remember that I wanted to PR this

Ty @deathbeam, I wanted to do this as well but it slipped my mind :)


Update Moya.podspec for new major Alamofire version

Hey, awesome! Thanks :) The CI is failing because Travis. Gonna merge!


A way to subscribe to new active window event?

Hi, I was curious if its possible to implement subscribing to active window change event somehow? The thing is I know how to do this on linux, have vague understanding how to make it with winapi, but have any clue how (if at all possible) to do this for OSX.
I'm asking because polling is quite expensive, especially in OSX for some reason, and I can't find any library that could do this for some reason.
Thx.

I don't really have plans to implement subscribing at this point.
How bad is the performance you're seeing on OSX? I just checked and I'm getting results in around 0.2-0.3 ms on linux and windows and closer to 1 ms on a mac while running in a loop which is fine for me.


Docs mention RSA-PKCS1-v1_5, code uses RSA-OAEP

Hi there Charles! Just reading your excellent FluentConf talk and checking out these examples. I notice that although the doc says:

This example uses RSA-PKCS1-v1_5 to encrypt a random 128-bit AES key

But createAndSaveAKeyPair  on https://github.com/infotechinc/public-key-encryption-in-browser/blob/master/pkcrypto.js#L47 uses name: RSA-OAEP.
Also per section 19 of the WebCrypto spec, isn't RSA-PKCS1-v1_5 is used for 'sign' and 'verify' not 'encrypt' and 'decrypt'
Should the doc read the following instead:

This example uses RSA-OAEP to encrypt a random 128-bit AES key

Thanks!

You found two bugs in a single word. The original documentation should have referred to RSAES-PKCS1-v1-5, not RSA-PKCS1-v1_5. That was an RSA encryption algorithm that I used when I first posted this. But it was removed from the Web Cryptography API specification last fall, so I changed to RSA-OAEP and forgot to update the documentation.
Now it's fixed.
Thanks for reading this example. I hope it is useful to you.


Add a new card from the widget

I'd love a "quick add" button in the title-bar of the widget.  It could open a little dialog that lets me add a card (by title only).  The goal would not be to re-create trello's whole "add card" with all the features and additions and things, but just a quick thing for one-off ideas that I would then edit in trello later as I refined them.

Here's what I mean by an add button in the title bar:

Here's my proof-of-concept for the add button.  I actually have it fully coded and working:

Features:

Type in some text and click the "AT END" or "AT TOP" button, and a new card is created in the current list at either the top or the bottom, with the provided text as the title.  The popup closes and the widget is refreshed - There's your new card.
Want to do more that just add a title?  Enable "And open".  After you click the "AT END" or "AT TOP" button, the card is created and then opened either in the Trello app or your browser.  You can add a description, checklists, other users, etc.
Want to add a bunch of cards all at once? Enable the "Multiples" option.  When you click "AT END" or "AT TOP", the dialog isn't closed - Instead it creates the new card and clears the "title" text entry box, ready for another.
Click the "X" button to close the dialog.

An interesting side-effect of this - I had to change the authentication a bit, as to do this kind of thing we need to start requesting write privileges.
I have this implemented and working, too, to a point.  If you upgrade from a version that only has read privs, you need to logout and re-login to enable the new privs.  I Fyou click "add" and try to add a card before you've done this login/logout, the only error you get right now is a toast that tells you to login and logout again... Maybe this can be made better.

I just reworked my change some, so that when you click the "add" button but we don't have write privs yet, it pops up a dialog:

Clicking "Cancel" just closes the dialog, and the widget functions as before (the add button can be hidden via on option screen toggle)
Clicking "OK" invalidates our cached login token and opens up the main activity where clicking the "Login" button will now request write access.

FYI: Work-in-progress branch is here: https://github.com/lack/TrelloWidget/tree/add_button

This is pretty cool.
However,

I would prefer to keep the widget as a read-only thing and not expand the feature set to a point where it's now an app that is being developed in parallel to the official one. (Ideally if the guys at Trello got their act together and created a usable widget instead of what they currently offer, this widget will be "deprecated").
This arguably only saves you the couple of clicks of pressing the title to open the app and creating a card from there.
Plus the widget title is already looking a bit crowded with buttons.

I'll have to say no to this, even though it looks like you've put a decent amount of work into it.


Add Windows support

Add the possibility to install everything on Windows

This is now completed with the latest commit. We now have a fully working one-click installer for Windows with Geth and Teku.


Clean-up/change some Skeleton3D API

Follow up #64802

Function declarations must be removed from the header as well

pose_updated signal is now available to the public

It was done ADD_SIGNAL in public to begin with, but only the signal firing only was TOOLS_ENABLED. If made public, there are enough use cases that it would not make sense to keep this as TOOLS_ENABLED only

Made get_bone_children() and get_parentless_bones() qualifiers to const

Same as other getters

Thanks!


[TEST-PA-659] Test helper methods for scheduling backups

What this PR does / why we need it:
This pull request has been rebased on PR #1164 and resolves the issues pertaining to "ScheduleBackupCreationSingleNS". This PR has been created with the intention of facilitating an easy review process.
Which issue(s) this PR fixes (optional)
Closes NA
Special notes for your reviewer:
Please review the Jenkins build for the refactored "ScheduleBackupCreationSingleNS" test case using the link below
https://jenkins.pwx.dev.purestorage.com/job/Users/job/Sumit/job/Custom-Pipelines/job/px-backup-on-demand-system-test-byoc/162/
Aetos Dashboard: http://aetos.pwx.purestorage.com/resultSet/testSetID/112370

I have implemented the changes suggested by @venkatpx to the test case. Specifically, it now performs a restore of the second schedule backup instead of the first one. Please review the refactored "ScheduleBackupCreationSingleNS" test case using the link below:
https://jenkins.pwx.dev.purestorage.com/job/Users/job/Sumit/job/Custom-Pipelines/job/px-backup-on-demand-system-test-byoc/174/
Aetos Dashboard: http://aetos.pwx.purestorage.com/resultSet/testSetID/113345.
P.S. Please note that there is a significant amount of code redundancy, especially with regards to the backup success checks (PA-650). You might find the log messages grammatically incorrect and inconsistent. I will rectify these issues in a new PR that focuses solely on this particular test case.

Please review Jenkins build for the latest refactored "ScheduleBackupCreationSingleNS" test case using the link below:
https://jenkins.pwx.dev.purestorage.com/job/Users/job/Sumit/job/Custom-Pipelines/job/px-backup-on-demand-system-test-byoc/183/
Aetos Dashboard: http://aetos.pwx.purestorage.com/resultSet/testSetID/115005

@apimpalgaonkar please review 43e640f859ca4b2504c53d4a56f66af97455b1d0

@venkatpx please review 196da33c773921556a58caba5f2b2693c9bf55a0
Latest Jenkins for refactored ScheduleBackupCreationSingleNS
https://jenkins.pwx.dev.purestorage.com/job/Users/job/Sumit/job/Custom-Pipelines/job/px-backup-on-demand-system-test-byoc/239
Aetos Dashboard: http://aetos.pwx.purestorage.com/resultSet/testSetID/118124

@venkatpx please review fd77bb0acea1860bef9c29d91c0f36b237c4c1f1
Latest Jenkins for refactored ScheduleBackupCreationAllNS
https://jenkins.pwx.dev.purestorage.com/job/Users/job/Sumit/job/Custom-Pipelines/job/px-backup-on-demand-system-test-byoc/244
Aetos Dashboard: http://aetos.pwx.purestorage.com/resultSet/testSetID/118243

I created this pull request to test the helper methods used in scheduling backups. I am closing this PR since I have submitted four separate pull requests, each addressing a test case that relies on these helper functions: PR #1191, PR #1193, PR #1194, and PR #1195. Thank you for taking the time to review these changes.


feat: 🎸 update dependencies

main upgrades: cryptography (fixes a vulnerability that made the CI fail), pyarrow 11 -> 12. Also: mypy 1.1.1 ->1.3.0 (should not affect)

Codecov Report
Patch coverage has no change and project coverage change: -0.97 :warning:

Comparison is base (516061a) 90.43% compared to head (fc919c5) 89.47%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1303      +/-   ##
==========================================
- Coverage   90.43%   89.47%   -0.97%
==========================================
  Files         183       92      -91
  Lines       10796     5072    -5724
==========================================
- Hits         9763     4538    -5225
+ Misses       1033      534     -499

Flag
Coverage Δ

jobs_cache_maintenance
99.08% <ø> (ø)

jobs_mongodb_migration
83.49% <ø> (ø)

libs_libcommon
91.61% <ø> (ø)

services_admin
85.99% <ø> (ø)

services_api
?

services_worker
?

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

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

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

OK, the mypy upgrade seems to trigger an error when we use # type: ignore (rightly, because it's ignored)

The two remaining CI errors come from a change in users populated in Hub CI (see https://github.com/huggingface/moon-landing/pull/6502, internal). It will be fixed with https://github.com/huggingface/datasets-server/pull/1298.
I'm rebasing on use-app-token-for-parquet-uploading, and changing the destination branch of this PR to it

Merging


console.clear does not get disabled on Preserve log flag.

according to the documentation

if the Preserve log checkbox is enabled, console.clear() is disabled.

this is a very neat feature, however it does not work, console.clear() always clears the console regardless of the preserve log flag.
i am running console.clear() from javascript, on Chrome Version 58.0.3029.110 (64-bit) from my mac
source:
https://developers.google.com/web/tools/chrome-devtools/console/console-reference
"console.clear()" section.

@Bamieh thanks for raising the issue - the bug is filed on the Chromium project. Please chekc on the process over at https://bugs.chromium.org/p/chromium/issues/detail?id=735440

@kosamari thanks, i did raise the issue there since i wasnt sure where to post it :)


Reaching Out

Hey Alamofire Team:
First off - I know this is completely out of line but I'm hoping someone out of the kindness of their heart will help me out here as I would do the same for anyone else. If someone is willing to help me here then I can promise you that I will not forget about this in the future - so please keep that in mind.
I am hoping someone would be willing to write a quick simple example in Swift 3.0 that uses Alamofire to record a video and then uploads this video once the user clicks "Upload" onto a web server.
If ANYONE on the team or any user would be willing to write a quick example that accomplishes this then I would be forever grateful and I would pay this favour back with interest one day in the future. That's the type of person I am.
Thank you so much all and again I apologize for such an annoying request.
Cheers

I'm closing this. If someone in the community wants to help out @TannenBiff, then feel free to continue the conversation through this issue.


Support multiple stop options for OpenAI models

https://github.com/simonw/llm/blob/898418a518ddcde24a01d31b3da4d5c35890c022/llm/default_plugins/openai_models.py#L107-L110
Ideally this would duplicate the official docs:

Up to 4 sequences where the API will stop generating further tokens.

The trick here is figuring out the syntax, since options can only be specified once as a string. I'll probably do a trick a bit like what I did with logit_bias:
https://github.com/simonw/llm/blob/898418a518ddcde24a01d31b3da4d5c35890c022/llm/default_plugins/openai_models.py#L118-L141

Here's how Replicate partially solved this: https://replicate.com/joehoover/falcon-40b-instruct

Comma delimited string specifying stop sequences. Multi-token stop sequences are supported, but they cannot contain commas.


Store folder display mode

Without this PR, I have to manually change the folder display mode every time I move a message because K9 hides important target folders.
Before this change, the method setFolderTargetMode was only used when de-serializing the preferences. The method was never called from the UI.

Given that the behavior has been like this for ages, I see no reason to change it now

Okay.

there's another place to do the permanent change 'Account settings > Folders > Move/copy destination folders').

I did not know that :+1:


Prepend the current directory to EXEC_LIBS

Previously the relative path is linked and running the test from another place might fail because it can't find the lib.

So I had thought about this also and even made the change. For some reason I do not remember I had to undo the change. I think the problem is that on the Mac this will not work. I do not have a Mac to check things on. Maybe @JunoRavin  can pull your branch and check on the Mac. Shared linking is tricky on Mac as they use a different system.

The more robust solution can be found at https://stackoverflow.com/questions/18136918/how-to-get-current-relative-directory-of-your-makefile


PoolChunkList.allocate(...) should return false without the need to w…

…alk all the contained PoolChunks when the requested capacity is too big.
Motivation:
PoolChunkList.allocate(...) should return false without the need to walk all the contained PoolChunks when the requested capacity is larger then the capacity that can be allocated out of the PoolChunks in respect to the minUsage() and maxUsage() of the PoolChunkList.
Modifications:
Precompute the maximal capacity that can be allocated out of the PoolChunks that are contained in the PoolChunkList and use this to fast return from the allocate(...) method if an allocation capacity larger then that is requested.
Result:
Faster detection of allocations that can not be handled by the PoolChunkList and so faster allocations in general via the PoolArena.

@trustin PTAL

LGTM

Cherry-picked into 4.1 (cfa258e096bf93cceac424c55779cf0e4f1cde66) and 4.0 (87b8bdbbdd33fef3f929aaa70cbb0cf9df3a83d2)

lgtm


az vm run-command invoke --no-wait report unrecognized arguments: --no-wait

az vm run-command invoke --no-wait -g UBUNTU -n UBUNTU --command-id RunShellScript --scripts 'echo $1 $2' --parameters hello world
unrecognized arguments: --no-wait
remove --no-wait cab be work
az vm run-command invoke -g UBUNTU -n UBUNTU --command-id RunShellScript --scripts 'echo $1 $2' --parameters hello world
/ Running ..
az version
{
"azure-cli": "2.38.0",
"azure-cli-core": "2.38.0",
"azure-cli-telemetry": "1.0.6",
"extensions": {
"account": "0.2.3",
"aks-preview": "0.5.100",
"automation": "0.1.1",
"k8s-extension": "1.2.2",
"ssh": "1.1.1"
}
}
btw test in az cli 2.41.0. the issue disappear

@zhoxing-ms for awareness

I am still seeing this on version 2.49

@blueboxes Could you please send the debug log (add --debug parameter) to me? My email address is<EMAIL_ADDRESS>thanks~

Thanks for your quick response, I have sent you an email.

@blueboxes Thank you for providing the debug log. I found that the --no-wait parameter is not supported by the az resource invoke-action command (not az vm run-command invoke) in the log.
If you need the az resource invoke-action command to also support the --no-wait parameter, please help submit a new Github issue to help us follow up on this requirement.

Thanks, good spot. I have found an existing request for this  #21190


Update stdlib.ring

Please correct the comments because my English is very poor.

This branch has conflicts that must be resolved


Refactoring to use pyproject.toml

This PR adapts the code to use pyproject.toml
The phonopy_install has to be done manually

We can maybe ask Jusong, for some help , and/or we can copy the same they do to install the pseudos in aiidalab_qe

Hi @AndresOrtegaGuerrero , everything is fine for me, I just added you a co-author of the code, and added a raise error if in phonopy install, if the code is already stored... I am not sure is the best choice, maybe a warning is enough?


No 1.29.1 support?

Are the servers for 1.29.1 down because it just says that i need to update my Beat Saber version?

The servers have been updated to 1.37+ with the update support for 1.28.0 and 1.29.1 was dropped as the changes to the underlying netcode were too substantial to reasonably maintain support.
You can find the latest version of MpCore for PC Beat Saber here: https://github.com/Goobwabber/MultiplayerCore/releases/latest And the latest version of BeatTogether for PC Beat Saber here: https://github.com/BeatTogether/BeatTogether/releases/latest

yeah just drop support for the version of the game that has the most mods

Hello,
Most mods are now on newer versions. It is not best interest to continue 1.29.1 support. If this was done, the mod would not be timely updated, nor would we be able to provide new features that make the experience better.
It's in your best interest to upgrade. Plain and simple.


Add support for instanced morph target animations

If this PR gets merged into three.js...
https://github.com/mrdoob/three.js/pull/27616
... I would look at adding support to a-frame instanced-mesh.

The functionality was merged in mrdoob/three.js#27669 which supports all types of morph target animations now


HDDS-3049. Fix Replication factor passed in create API doesn't take effect

What changes were proposed in this pull request?
Replication did not take effect when we created the file through o3fs. This PR fix that.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-3049
How was this patch tested?
Ut already exists

Thanks @captainzmc for working on this fix.
Maybe I'm missing something, but with this change files are created with factor=1 by default on a cluster configured with default factor=3. Shouldn't BasicOzoneFileSystem override getDefaultReplication() to return the configured replication factor instead of 1?
Here's how I tested:
$ ozone fs -put /etc/passwd o3fs://bucket1.vol1/passwd
$ ozone sh key info /vol1/bucket1/passwd
{ ... "replicationFactor" : 1, ... }

Hi @adoroszlai Thanks for you reviewing. Yes, use ozone fs or ozone sh is ok. But we test o3fs using org.apache.hadoop.fs.FileSystem API and there were a lot of problems. For more details you can see here.

/retest

@captainzmc , the commit seems not clean, there is file like hadoop-ozone/recon/src/main/resources/webapps/recon/ozone-recon-web/yarn.lock, I'm not sure it's related with this fix or not. Can you double check it?
Also please add new unit tests for replication factor, verify that if the replication factor is supported, new created key will has the specified replication factor, otherwise, it matches the system default replication factor.

Thanks @captainzmc for updating the patch.

Thanks @captainzmc for the contribution and @adoroszlai for review the code.


Govuk update fix confirm page styling

Update package.json
To ensure release is the
latest version

What
Added class "govuk-footer__link" and "govuk-footer__inline-list" to footer support links so it follows design system service


feat(EG-857): implement frontend interface for HealthOmics features

This PR contains the UI changes for introducing HealthOmics. No real functionality has been included as the BE is not finalized.
These changes include:

Lab view

split Pipelines tab into "NextFlow Pipelines" and "HealthOmics Pipelines"

Lab Details view

Add “Enable Seqera Integration” toggle

Move “Workspace ID” and "Personal Access Token" text fields under this toggle, showing only if toggle is enabled

Add “Enable HealthOmics Integration” toggle

Also included:

refactor to replace ref/state-based form UI manipulation with simpler declarative computed logic

update styling of UTabs to size tabs based on content, improving their fit in the space

increase page max width to ~1300px

As discussed in standup, we won't merge this PR prior to the functionality being ready, to keep the app ready for demos. I'll keep it updated until then.


[WIP] Refresh token support

This is the continuation of the PR #5 and aims at supporting refresh-tokens.

Btw. tagging of your versions in git would be nice. So everyone can jump between 1.1 and 2.0 for instance.

@GNSSYSTEMS-tg I've added the tags. Note however that we do not plan to support older branches as long we have not reached a stable version.
I have now basically completed the 2.x release, and you can try the 2.0.0-beta-7 version. I've tested it in some projects already, however the refresh token support has not been tested in production.

Don't worry, neither do I, but thanks for the tags.
I'll have a peak at the current beta.

Ok, so the refreshTokenUrl is missing in the config ;) It does exist within the AuthConfig but it's never initialized nor accessible within JwtAuthConfig.

Ah I see. The OAuth standard would be to have different grant_types in the same token url. I can optionally support configuring a separate refresh url and by default its the obtain token url.

Beta 2.0.0-beta.8 is released which supports a customisable token refresh url.

There is an issue with refreshing after the first refresh:
2017-09-21T14:51:51Z [DEBUG] Obtaining JWT for OAuthGrant.grant_type: password
2017-09-21T14:51:51Z [DEBUG] Token time to live: 69718
>>> logged in
2017-09-21T14:52:01Z [DEBUG] Refreshing JWT.
2017-09-21T14:52:01Z [DEBUG] Obtaining JWT for OAuthGrant.grant_type: refresh_token

And then the logging of the new token ttl is missing, so I suppose the new interval is not created.
For some reason, the observable of the token container is not triggered after refresh...

Looks fine to me.

Crap, there's one more thing...


Tag docker image with sermver

db-sync has a semver 1.4.0
but docker image only has a master version
Why not tag docker image with 1.4.0?

Hi @islishude. It's our intent, but the webhook didn't send the tag event. Looking into this now, and will update you when it's available.

resolved.use commit hash.


Fix lock -r warning being logged to stdout instead of stderr

The issue
In https://github.com/pypa/pipenv/pull/5069, a deprecation message was added when pipenv lock -r is run.
The warning message is written to stdout instead of stderr, breaking scripts that run commands like pipenv lock -r > requirements.txt.
The fix
This PR directs the warning message to stderr instead of stdout.
I also de-dented the message to make it look nicer:
Old (notice the newline at the top):

            Warning: The lock flag -r/--requirements will be deprecated in a future version
            of pipenv in favor of the new requirements command. For more info see
            https://pipenv.pypa.io/en/latest/advanced/#generating-a-requirements-txt
            NOTE: the requirements command parses Pipfile.lock directly without performing any
            locking operations. Updating packages should be done by running pipenv lock

New:
Warning: The lock flag -r/--requirements will be deprecated in a future version
of pipenv in favor of the new requirements command. For more info see
https://pipenv.pypa.io/en/latest/advanced/#generating-a-requirements-txt
NOTE: the requirements command parses Pipfile.lock directly without performing any
locking operations. Updating packages should be done by running pipenv lock.

Thanks @tomarad  -- would you add a bugfix news fragment for this and reference the PR number (unless there is an issue report)?

Done @matteius.

Slightly annoying, this bit me today here - https://github.com/actions/setup-python/issues/398.

Wow, should have realized this. Apologies and thanks for the fix

@ImreC If it makes you feel better I've done the same thing in a recent release in a refactor of a print statement.

@matteius thanks! It is an easy thing to miss I guess. @tomarad if you add a newline in the newsfile, the checks will pass.

hey all, I'm still getting this issue while using AWS CDK. It uses pipenv lock -r to generate a requirements file.
https://github.com/aws/aws-cdk/blob/c3c4a97e65071fcab35212be82dea7b29454786f/packages/%40aws-cdk/aws-lambda-python/lib/packaging.ts#L40
In my logs I can see that pipenv-2022.4.30 is being installed. Does this fix need to be part of a new release? It seems 4.30 is the latest version which I assume doesn't include this fix

The release notes show that this PR (#5091) did not make it into the most recent release (v2022.4.30).
This issue is currently breaking our CI/CD deployments which use the serverless-python-requirements plugin.

@guyc I just tagged a new release for this issue.  2022.5.2

Frist uninstall pipenv:  'pip uninstall pipenv` OR 'brew uninstall pipenv' OR 'sudo uninstall pipenv'
Second install pipenv==2022.5.2:     sudo pip install pipenv==2022.5.2

pipenv: 2022.5.2 works great!! ⚡🚀🤘😃


fix(TextBox): add x value in the stories file for the InlineContentString

Description
With this fix, the InlineContentString will now appear properly in storybook UI.
References
LUI-901

:tada: This PR is included in version @lightningjs/ui-components-v2.5.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


php is not working inside tags

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

VS Code Version: i searched but i dont found it, i install it today
OS Version: windows 11

Steps to Reproduce:

try to use php
2.doest works
look this image:

i tried so many times and dont work, but i have another error : when i run the file and go to chrome, this happens :

esqueci de editar o texto, mas não desativo as extensões

I forgot to edit the text, but I don't disable the extensions

seems the html grammar just doesn't support php

How exactly are you opening the php file in browser? You need a php-capable server software to run php files locally (and probavly rename file to .php or change the server's config).
Also, running of php files has nothing to do with vscode.

How exactly are you opening the php file in browser? You need a php-capable server software to run php files locally (and probavly rename file to .php or change the server's config).
Also, running of php files has nothing to do with vscode.

oh i tried to run with the server wamp and it works, thanks


Executable file on git repo

Hi,
I found that you have one executable file on the repo, which causes issues when updating it from the source.
https://github.com/eddiesigner/liebling/blob/master/locales/zh-hans.json

Hi, thanks for the heads up. It's fixed now on master ;)

Thanks!


list_tables.py does not output if a table is cartodbfied

The script does not show a header on the output and all my tables appear as not cartodbfied.

done :)

Reviewed.
Close the issue to go for carto-python 1.0.0!


add IGAPS dataset

IPHAS dr2 has been superseded by the IGAPS catalog and the survey should be updated
Feature Summary
The IPHAS dr2 catalog has been superseded by IGAPS (http://www.star.ucl.ac.uk/IGAPS/survey.shtml). IGAPS included both IPHAS and UVEX; and contains data in the Ugriz-filters. Especially the U-band data is important for compact object science.
Usage / behavior
This is simply an update of a catalog available for crossmatching
Implementation details
This is simply an update of a catalog. If needed, some catalog columns can be omitted.
Additional context
IGAPS is a Galactic Plane catalog, having this available soon would be nice since the Galactic plane is rising again in the sky

@dmitryduev This fix went in, did it get ingested and we can close?

@mcoughlin started ingesting and noticed the following:

missed this when reviewing, sorry about that...
Would you mind fixing the following:

Properly parse all the id/name strings
Ditch all NaNs before ingesting - it's NoSQL, it's ok not to have all the fields for each document.

Thanks!

@mcoughlin @janvanroestel finished ingesting. Can IPHAS be ditched now?

yes, IPHAS can be dropped.
On 4/23/21 7:04 PM, Dmitry Duev wrote:

@mcoughlin https://github.com/mcoughlin @janvanroestel
https://github.com/janvanroestel finished ingesting. Can IPHAS be
ditched now?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dmitryduev/kowalski/issues/97#issuecomment-826017545,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACSGMR6A4CTXVFKJIHPKFX3TKIRKHANCNFSM422JNE3A.


Priority not included in message

Hi,
I'm trying to send a CAN message with the header '0x08FF3DC8', where the priority is 0x08, the PGN is 0xFF3D, and the source address is 0xC8.
To do this, I'm using the following line of code:
    `ca.send_pgn(data_page=1, pdu_format=0xFF, pdu_specific=0x3D, priority=0x08, data=FF3D_data)`

where ca is the controller application object.  But when I probe the can bus to view the message being sent, the message's PGN shows up as '0x01FF3DC8'.  Is it not possible to specify a message's priority or am I doing something wrong?

Valid priority values are 0 to 7.  A quick Google leads me to this location but there will be others.

@bwelte34375 are you still having trouble with this, or OK now?

@bwelte34375 would you OK with this issue being closed?

Yes.
From: gRant @.>
Sent: Monday, June 10, 2024 2:18 AM
To: juergenH87/python-can-j1939 @.>
Cc: Ben Welte @.>; Mention @.>
Subject: Re: [juergenH87/python-can-j1939] Priority not included in message (Issue #74)
@bwelte34375 would you be OK with this issue being closed? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned. Message ID: <juergenH87/python-can-j1939/issues/74/2157520869@ github. com>
@bwelte34375https://urldefense.com/v3/__https:/github.com/bwelte34375__;!!ICUevlz5aoA!tX7HTjtVJ1dzcd5jDclwGhME7m2ZWfCV2N1CrgNZGL6oeTElCNNcRS9R1zT_I73V7uW-_k4SzKj6ABSHcACT75U$ would you be OK with this issue being closed?
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/juergenH87/python-can-j1939/issues/74*issuecomment-2157520869__;Iw!!ICUevlz5aoA!tX7HTjtVJ1dzcd5jDclwGhME7m2ZWfCV2N1CrgNZGL6oeTElCNNcRS9R1zT_I73V7uW-_k4SzKj6ABSHofGeexk$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/BARRUZZFPRILC4R2PBINAHTZGVHKHAVCNFSM6AAAAABJBWQQGSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJXGUZDAOBWHE__;!!ICUevlz5aoA!tX7HTjtVJ1dzcd5jDclwGhME7m2ZWfCV2N1CrgNZGL6oeTElCNNcRS9R1zT_I73V7uW-_k4SzKj6ABSHlTJnd0o$.
You are receiving this because you were mentioned.Message ID: @.***>


Retry without starting from scratch

For large episodes it can get cumbersome to retry the file from scratch on slow connections, how about just resuming it

#39


ovs-offline: build containers from specified ovs repo

Add build time args to allow for users to specify an
ovs repo and branch to build ovs from via option flags.
Default ovs is changed from:
https://github.com/openvswitch/ovs/archive/${OVS_VERSION}.tar.gz
to:
https://github.com/openvswitch/ovs.git branch-2.15
Signed-off-by: Salvatore Daniele<EMAIL_ADDRESS>

Apologies for the last minute changes. I realized while using the feature that the cache would prevent ovs from being rebuilt when changes were pushed to a branch.
All set for review at your convenience.

Conflicts resolved

LGTM. Thanks


Image transparency missing from one place and omitted from styleguide

Merged in https://github.com/torchbox/wagtail/commit/2d16bf9606f2ad53f5269586e0aa296fdb3d1190

@davecranwell, we probably need to set 1.4 milestone because this PR is already merged to master.


Added import event

Hi,
I've written a simple import plugin here and to make it work, I needed to add an event to the server import logic.  Here is that change, along with two other minor tweaks I needed: delete files on import, and show current plugins.
Feel free to take none, some, or all of these changes if you think they are helpful!  I think they will apply to either master or v3.0.0 branches, but I have tested against master.

There're several problems with unlink:

It won't work most of the time, because users upload with a different user and the php-user doesn't has the required rights to delete the photo. This could lead to confusion as it depends on your installation.
(Empty) folders will stay
You can enter a custom path when importing via Lychee. This could lead to security issues. Strangers with access to your Lychee would be able to delete photos from your server.

I'm not entirely sure about this anymore, but I think the default way to fire a plugin action was:
# Call plugins
$this->plugins(__METHOD__, 0, func_get_args());

As $this isn't available in a static function, the right way of calling the trigger should be something like this:
# Call plugins
self::plugins(__METHOD__, 0, func_get_args());

Could you give this a try?

Is it ok for you if I add FlashAir to the list of Plugins? https://github.com/electerious/Lychee#plugins-and-extensions

I can't use self::plugins as that method (defined in Module.php) uses instance variables (specifically $this->plugins) which are undefined when we don't have an actual object.  This can be fixed by removing the plugins method from Module and pushing the functionality down into the Plugins class itself, possibly by adding a before and after method.  If you like, I can make a PR with some proposed changes for refactoring this part separately.
I also can't use func_get_args() as this returns the arguments that Import::server was originally called with, and I need the possibly updated version of the path variable.
I agree that the unlink fix may be confusing.  How about if I only call it if $path===LYCHEE_UPLOADS?  Then importing from a custom directory won't try to delete the photos.  Another way that might work would be for plugins to be able to maintain state between the before and after triggers, so I could keep a list of the photos I have downloaded and delete them once they are done, but I think this might require bigger changes to how plugins work.
Finally, it is fine if you want to list the plugin, but you might want to wait until it works with your version of Lychee!

Thanks for the explanation! Lets do the plugin call your way, but with a comment about why we are calling the function directly. Maybe it would be good to adjust the Import-Module later, so it works without static functions (just like Album, Photo etc.).
unlink should be removed for now. Maybe I will find a better solution.

I've updated my plugin to build a list of imported files during the before trigger, and then delete them during the after trigger.  This means I don't need to unlink the files in the  Import::server function, so I've removed that part from this PR.  I've also added a comment explaining why the plugin trigger doesn't match the usual form, as you mentioned.
I think that's everything!

Added the plugin to the list: https://github.com/electerious/Lychee/commit/f3ce2de49f70ca24a877fad7af3da5a6759bbc3d


type float doesn't allow 0 or 0.0 on loadJS

I have a model set up with one of the fields like:
{key:'blockStartTime', type: 'float', default: 0},

When I load JSON data from a file using nSQL().loadJS the field above is eliminated from the data if the value is set to either 0 or even 0.0. If I change it to something like 0.001 it works. I also tried changing the default value to 0.0.

Hmmm, I forgot that toFixed() turns a float into a string. D'oh!

Haha that'll do it.
I did notice that the validation functions simply checked if the value in each column is falsey, ie if(value)....  That would definitely cause what you're describing, because 0 and false are both falsey.
So I changed those to if(value === undefined) so falsey like values should be passed through now, just omitting undefined values.
Just pushed the update, thanks for the heads up!
Best,

Thanks, Scott! Yea, I did see where it still fails with 0. Much appreciated. Was actually gonna go in and see if I could fix and contribute. You beat me to it!

Just pulled the latest and that fixed it. Thanks again. Closing.


Collaborators got 403 Forbidden API

In my dashboard, analytics running well.
But in my collaborators' dashboard, they got forbidden message when accessing this API
https://apps.ionic.io/api/v1/app/APP_ID/events

Here is the 403
HTTP 403 Forbidden
Allow: GET, HEAD, OPTIONS
Content-Type: application/json
Vary: Accept

{
    "detail": "Permission denied."
}

Maybe your interceptor overwrite the Authorization token for the Ionic API requests ?
The authorization token for your API (If you have one) and the one for the Ionic API are different.

I don't have any. And if it caused 403(permission denied), why does in my dashboard run well ?
I think it caused by user's permission.
I don't know about collaborator's permission to access Analytic's API.
This is what happened in my app:
Owner -> 200 OK
Collaborator -> 403 (Permission Denied)
Not Login -> 403 (Authentication credentials were not provided)
can the collaborator use analytics for shared app ?
If they can't (or shouldn't), maybe need some additional config for allowing collaborators access the analytic's API. (IMHO)


feat: Add User and Post Models/Schema for our platform in Node.js

@SwanandD121  what are your views on this ??
User Schema Structure:

username: Unique string identifying the user (required).
email: Unique string for user’s email address (required).
password: String for user’s password (required).
profilePicture: String URL of the user’s profile picture (optional, defaults to an empty string).
bio: Short description about the user (optional, max length of 250 characters).
followers: Array of user IDs referencing the users who follow this user.
following: Array of user IDs referencing the users this user follows.
createdAt: Date indicating when the user was created (auto-generated).

Post Schema Structure:

user: Reference to the user ID who created the post (required).
photoURL: String URL of the uploaded photo (required).
description: String providing details about the photo (optional).
tags: Array of strings for categorizing the post (optional).
likes: Number representing the total likes on the post (default is 0).
comments: Array of comment objects, each containing:
userId: ID of the user who commented.
text: Content of the comment.
createdAt: Date of comment creation (auto-generated).
createdAt: Date indicating when the post was created (auto-generated).

Notification Schema
The Notification Schema is designed to manage notifications related to user activities within the platform. It includes the following fields:

user: Reference to the User ID who receives the notification (required).
type: String indicating the type of notification (e.g., "follow", "like", "comment", "mention") (required).
message: String describing the content of the notification (e.g., "User X liked your photo") (required).
isRead: Boolean indicating whether the notification has been read by the user (default is false).
post: Reference to the Post ID related to the notification (optional, required for notifications linked to posts).
createdAt: Date indicating when the notification was created (automatically generated).

This schema ensures users are informed about interactions with their content, enhancing engagement and communication within the platform.
@SwanandD121 please assign this to me

@SwanandD121  what do you think about this please let me know.

@SwanandD121  I think it's time we should start working on backend @SwanandD121  what do you think


feat: add DataDeliveryVote event handler

Add DataDelieveryVote event handler

this code pass the manual test on a normal sellData request
But, there are two problems:

it takes a long time to process the requests (about 30sec), because there are 4 QueryClient requests and they are processed sequentially.
I don't know the exact reason, but the subscriber subscribe DataDelivery event of the endBlocker twice.

I will consider the above two issues and post them as another PR.

Fixed to handle errors and votes correctly

In most cases, we have voted through VOTE_OPTION_NO when it fails
When GetDeal and GetDataSale get error that is not NotFound errors, do not vote and return an error

Left some comments.
For improvements, we can cache the pubkey of seller and buyer (query for the first time, after first time reduce 2 queries)
And, if subscriber subscribes twice, it votes twice as well?

Yes. Good idea.
In general, values that do not change are cached at regular intervals.


Use ruff linter and formatter

Closes #71
Relevant changes are done in README.md, pyproject.toml, .pre-commit-config.yaml, Makefile and .flake8 (deleted). Other changes are due to applying the pre-commit, which sorts the imports: first built-ins, then any installed packages, and finally movement and its modules.
I temporarily disabled "B" and "C" linting rules as they were blocking the push. See #72

@jwallwork23 I didn't have permission to push to movement so I forked it :)


IMU on Android

Hi,
I would like to know when is planned the support of the IMU for the camera D435i on Android ?
Best Regards,
Stephane

@cottev hello,
Delivering IMU capability to android platform is a scheduled feature. I'm sorry but I am not able to provide further details at this time.
Please follow the (bi-weekly) SDK releases and notifications.

Hi,
Thanks for this reply but could you tell me if it is a matter of weeks, months or years ?
We would need this info to choose the approach of our algo based on your camera.
Br,
Stephane

Following up on this issue here. Our team also needs this feature and would be interested in knowing the release date for IMU support on Android.

@dorodnic Our team are also focused on this issue. We are now developing an app on android which needs imu data for VIO. So it would be very important to know how long can you deal with this problem on android. If this can not be done in the next few months, we need to change the hardware.

Same Problem.
I am interesting about the support of IMU on Android. I would like to know when you can release it.

For this problem, I am also head-scratching, waiting for its new progress.

Using your product, same problem, please release ASAP. Thx

Would it be possible to Intel to comment on this topic ?
Br,
Stephane

@dorodnic. Please, would it be possible for you to comment on this issue?
Best Regards, stephane

Hi @cottev
Sorry for the delay. We cannot comment on features then there is significant technical risk for their schedule. That said, we do prioritize features requested by the community - the team has been working on this for last couple of months, it looks good,  now its safe to say it will be merged very soon (week or two), and you can try it right now by pulling #4605

Hi @cottev
Do let us know your feedback based on testing the PR #4605 outlined by dorodnic.
Thanks

Hi,
Sorry for the delay I was in vacation ;-)
It is a great news and I will take a look to this release.
Thanks again for your effort.
Br,
Stephane

Hi,
I have just tested with the release of SDK 2.26.0 with an update of the FW camera ( Signed_Image_UVC_5_11_11_100.bin ).
I can us the gyro and the accel if I enable through the following commands:
config.enable_stream(RS2_STREAM_GYRO);
config.enable_stream(RS2_STREAM_ACCEL);
However as soon as I try to enable the pose the function pipe.start is blocked.
My code of enabling is quite simple:
config.enable_stream(RS2_STREAM_DEPTH, WIDTH, HEIGHT, RS2_FORMAT_Z16, FPS);
config.enable_stream(RS2_STREAM_COLOR, WIDTH, HEIGHT, RS2_FORMAT_RGB8, FPS);
config.enable_stream(RS2_STREAM_POSE);
pipe.start(config);
I have the following message under the logcat on android:
2019-08-26 16:25:29.730 17935-22495/? I/librs: Found HID device: id- gyro_3d
vid- �
pid- :
unique_id- /dev/bus/usb/002/002
path- /dev/bus/usb/002/002
2019-08-26 16:25:29.730 17935-22495/? I/librs: Found HID device: id- accel_3d
vid- �
pid- :
unique_id- /dev/bus/usb/002/002
path- /dev/bus/usb/002/002
2019-08-26 16:25:29.730 17935-22495/? I/librs: Found HID device: id- custom
vid- �
pid- :
unique_id- /dev/bus/usb/002/002
path- /dev/bus/usb/002/002
2019-08-26 16:25:29.731 17935-22495/? I/librs: Found 302 unreachable streams, 302 extrinsics deleted
2019-08-26 16:25:29.783 17935-22495/? W/librs: hwmon command 0x4f failed. Error type: No data to return (-21).
2019-08-26 16:25:29.783 17935-22495/? I/librs: IMU EEPROM Read error: hwmon command 0x4f failed. Error type: No data to return (-21).
2019-08-26 16:25:29.783 17935-22495/? I/librs: Motion Module - no intrinsic calibration, Depth Module V2 intrinsic invalidated : Accel !
2019-08-26 16:25:29.788 17935-22495/? W/librs: null pointer passed for argument "profile"
2019-08-26 16:25:29.788 17935-22495/? E/librs DeviceWatcher: Couldn't resolve requests
2019-08-26 16:25:29.788 17935-22495/? D/librs DeviceWatcher: Device: /dev/bus/usb/002/002 added successfully

Hi,
In fact it is working as the pose is not supported by the camera D435i.
Thanks again,
Stephane

The sensor that i use is D435i. Now I want to extract imu  data on android But I don't know how to do. Who can tell me. Thanks a lot !

@cottev Hello, and now can you get the imu data? Could you share the approach to get it?


expose underlying http request and client

Expose the underlying HTTP request and client, and allow the caller to customize the request details, such as headers or proxy.

LGTM


FastQC threads specified inappropriately

FastQC --threads option specifies the number of files that can be processed in parallel (https://manpages.debian.org/stretch/fastqc/fastqc.1.en.html).
Currently it looks like FastQC processes are launched with the maximum number of threads available, but only ever with two FASTQ files as a maximum.
https://github.com/nf-core/rnaseq/blob/master/main.nf#L846
As a result many threads are reserved that won't be used.  --threads should be specified based on number of read files provided (1 for single end, 2 for paired)

Thanks for reporting this - shouldn't be a big thing to adjust. If you want to, feel free to open a PR against dev to get this resolved :-)

Hi @prmac! Please let me know if you planning to do this otherwise I can help with adding this through a PR :)

Hi @sk-sahu, I'll fix this over the next couple of days.

Fixed in https://github.com/nf-core/rnaseq/commit/077dc289c46ae7a28cd139db8a4366c89c9b5b81 - thanks for reporting!

Am I mistaken, or is this issue was reintroduced in the latest release?


PyData Copenhagen

Site
URL

Youtube channel
https://www.youtube.com/channel/UCphaUFghyyhpd9Fi2nBKsNA/videos

Meetup
https://www.meetup.com/es-ES/PyData-Copenhagen/

This youtube channel only has three videos in it, each of which is under 2 minutes. It looks like they haven't had any activity in months. Do we know anyone at that meetup who has information about whether or not they'll be uploading any content?

@andersbogsnes, sorry to bother (spam) you.
Do you know if PyData Copenhagen is producing video content (in order to index it in Pyvideo)? If so, please, contact with us in this github issue (meanwhile we will assume no content and close the issue).
Thanks in advance.


Add Docker Support on feature/docker-support Branch

#14 #57
Note: Because of the --network host this currently only works with Docker on Linux or WSL, which is why I put it in a separate branch.
This is because using http://host.docker.internal:11434 on my system in llm_config.py did not connect to Ollama, while --network host did (which can only be used on Linux or WSL).
#29 Had a few errors and inefficiencies like downloading requirements.txt every time a change to the codebase was made and using docker-compose up which is not as universal as docker compose up (the former doesn't work on WSL out of the box, for example), but this PR is based on that.

Could you look it over and merge it?


Add MSRV to cargo.toml and README

Goal
Help users know if their Rust version is supported.
Related #285
Changes

Added MSRV to ts-rs/Cargo.toml
Added MSRV to README
Removed the "todo" section from the README, as all of those todos were already done

Checklist

[x] I have followed the steps listed in the Contributing guide.
[x] If necessary, I have added documentation related to the changes made.
[x] I have added or updated the tests related to the changes made.

Sorry about that, I requested the review and then I was like "this... really doesn't need a review, it's just 1 line in Cargo.toml and some text on the README" lol

np!


docs: fix auto-import differences for composables and components folder

🔗 Linked issue
#22056

❓ Type of change

[x] 📖 Documentation (updates to the documentation, readme or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Currently the documentation is not explaining about the differences to disable auto-import for the composables and components folder. Added the information based on @danielroe comments (Thanks for the explanation 😄).

📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

Thank you! Love the quick action to make the docs better :heart:


Warn if a local variable can be made a constant

e.g.
func foo() -> Int {
  var a: Int = 2 // warn: a could be made a let-variable
  return a
}

I am now working on this issue.


Failure while running npm run setup

This is the error I'm getting.
npm ERR! not ok code 0
npm ERR! weird error 3
npm WARN This failure might be due to the use of legacy binary "node"

I tried symlink solution and installing nodejs-legacy package from here: http://stackoverflow.com/questions/21168141/can-not-install-packages-using-node-package-manager-in-ubuntu
But still getting the same issue.

Did you try linking nodejs with /usr/bin/node? It was suggested in this answer and seems to be a good idea.
http://stackoverflow.com/a/21168305/2580798

It is working now. Thanks.


HuggingFaceTextGenInference Streaming does not output

System Info
langchain==0.0.354
text_generation==0.6.1
python:3.10-slim
Who can help?
@agola11 @hwaking
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[X] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[X] Chains
[X] Callbacks/Tracing
[X] Async

Reproduction
Model: TheBloke/Llama-2-7B-Chat-GPTQ, but I've also tried TheBloke/Mistral-7B-OpenOrca-GPTQ
FastAPI example with HuggingFaceTextGenInference streaming:

from fastapi import FastAPI

import langchain
from langchain.llms import HuggingFaceTextGenInference
from langchain.chains import LLMChain
from langchain.prompts import PromptTemplate
from fastapi.responses import StreamingResponse
from pydantic import BaseModel

import os
from fastapi.middleware.cors import CORSMiddleware

app = FastAPI()
langchain.debug = True

# Enable CORS
app.add_middleware(
    CORSMiddleware,
    allow_origins=["*"],  # You can specify the list of allowed origins or use "*" for any origin
    allow_credentials=True,
    allow_methods=["*"],  # You can specify the HTTP methods that are allowed
    allow_headers=["*"],  # You can specify the HTTP headers that are allowed
)

# Configuration for local LLM
ai_url = "http://tgi-ai-server:" + str(os.getenv("AI_PORT", 80)) + "/generate"

# Configure the LLM
llm = HuggingFaceTextGenInference(
    inference_server_url=ai_url,
    max_new_tokens=20,
    streaming=True,
)

template = """
[INST] <<SYS>>
You are a helpful, respectful and honest assistant. Always answer as helpfully as possible, while being safe.  Your answers should not include any harmful, unethical, racist, sexist, toxic, dangerous, or illegal content. Please ensure that your responses are socially unbiased and positive in nature. If a question does not make any sense, or is not factually coherent, explain why instead of answering something not correct. If you don't know the answer to a question, please don't share false information.
<</SYS>>
{prompt}[/INST]
"""
prompt_template = PromptTemplate(
    template=template,
    input_variables= ["prompt"]
)

# Initialize the LLM Chain
llm_chain = LLMChain(llm=llm, prompt=prompt_template)

@app.get("/chat")
async def chat():

    prompt = {"prompt":"What is the Nickelodeon channel?"}
    # Generate the response using the LLM Chain and stream the output
    async def generate():
        for text in llm_chain.run(prompt):
            yield text

    return StreamingResponse(generate(), media_type="text/plain")

# Run the server (if running this script directly)
# Use the command: uvicorn script_name:app --reload
if __name__ == "__main__":
    import uvicorn
    uvicorn.run(app, host="<IP_ADDRESS>", port=8000)

Output:
[chain/start] [1:chain:LLMChain] Entering Chain run with input:
{
  "prompt": "What is the Nickelodeon channel?"
}
[llm/start] [1:chain:LLMChain > 2:llm:HuggingFaceTextGenInference] Entering LLM run with input:
{
  "prompts": [
    "[INST] <<SYS>>\nYou are a helpful, respectful and honest assistant. Always answer as helpfully as possible, while being safe.  Your answers should not include any harmful, unethical, racist, sexist, toxic, dangerous, or illegal content. Please ensure that your responses are socially unbiased and positive in nature. If a question does not make any sense, or is not factually coherent, explain why instead of answering something not correct. If you don't know the answer to a question, please don't share false information.\n<</SYS>>\nWhat is the Nickelodeon channel?[/INST]"
  ]
}
[llm/end] [1:chain:LLMChain > 2:llm:HuggingFaceTextGenInference] [765ms] Exiting LLM run with output:
{
  "generations": [
    [
      {
        "text": "",
        "generation_info": null,
        "type": "Generation"
      }
    ]
  ],
  "llm_output": null,
  "run": null
}
[chain/end] [1:chain:LLMChain] [765ms] Exiting Chain run with output:
{
  "text": ""
}

Expected behavior
HuggingFaceTextGenInference does not return any streaming data. Works fine when streaming=False in parameters.

@dosu-bot
Also streaming via curl from command line works fine:
curl <IP_ADDRESS>:3331/generate_stream \
    -X POST \
    -d '{"inputs":"What is Deep Learning?","parameters":{"max_new_tokens":17}}' \
    -H 'Content-Type: application/json'

Output:
data:{"token":{"id":13,"text":"\n","logprob":-0.3269043,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":13,"text":"\n","logprob":-2.6054688,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":2772,"text":"De","logprob":-0.017089844,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":1022,"text":"ep","logprob":-0.000064492226,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":6509,"text":" learning","logprob":-0.004310608,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":338,"text":" is","logprob":-0.15283203,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":263,"text":" a","logprob":-0.0038261414,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":11306,"text":" subset","logprob":-0.11029053,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":310,"text":" of","logprob":-0.000018119812,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":4933,"text":" machine","logprob":-0.0003349781,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":6509,"text":" learning","logprob":-0.0001887083,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":393,"text":" that","logprob":-0.0065727234,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":20789,"text":" involves","logprob":-0.004333496,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":278,"text":" the","logprob":-0.0018758774,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":671,"text":" use","logprob":-0.00043320656,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":310,"text":" of","logprob":-0.0000046491623,"special":false},"generated_text":null,"details":null}

data:{"token":{"id":23116,"text":" artificial","logprob":-0.14477539,"special":false},"generated_text":"\n\nDeep learning is a subset of machine learning that involves the use of artificial","details":null}

@dosu-bot @hwchase17  Wondering if it might be an issue similar to this:
https://github.com/langchain-ai/langchain/pull/6211/commits

@dosu-bot Your code did not work
@app.get("/chat")
async def chat():

    prompt = {"prompt":"What is the Nickelodeon channel?"}
    # Generate the response using the LLM Chain and stream the output
    async def generate():
        async for text in llm_chain.run(prompt):
            yield text

    return StreamingResponse(generate(), media_type="text/plain")

This is the error I received:
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/starlette/responses.py", line 255, in wrap
    await func()
  File "/usr/local/lib/python3.10/site-packages/starlette/responses.py", line 244, in stream_response
    async for chunk in self.body_iterator:
  File "/app/fastapi_server.py", line 78, in generate
    async for text in llm_chain.run(prompt):
TypeError: 'async for' requires an object with __aiter__ method, got str

Just to close this, it was a mistake on my end. URL should have had '_stream' on the end. wompidy womp
ai_url = "http://tgi-ai-server:" + str(os.getenv("AI_PORT", 80)) + "/generate_stream"

not

ai_url = "http://tgi-ai-server:" + str(os.getenv("AI_PORT", 80)) + "/generate"


assets-7.sqlite' doesn't exist

Dear Zipline Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment

Operating System: Windows 11
Python Version: 3.10.9
Python Bitness: 64
How did you install Zipline: pip
Python packages: pip install zipline-reloaded

Now that you know a little about me, let me tell you about the issue I am
having:
On running the 04_single_factor_zipline.ipynb script, getting the error "assets-7.sqlite' doesn't exist" when executing the line %%zipline --start 2015-1-1 --end 2018-1-1 --output single_factor.pickle --no-benchmark --bundle quandl
What steps have you taken to resolve this already?
Tried running zipline ingest -b quandl
as described in
https://stefan-jansen.github.io/machine-learning-for-trading/installation/

try to re-install latest zipline version there was a recent fix

Note that the conda-forge hasn't yet been updated with these fixes:
https://anaconda.org/conda-forge/zipline-reloaded/files
Does conda-forge need a bit of gentle persuasion to run?

Thanks so much, Richard! Should I try reinstalling zipline-reloaded in a couple of days?
From: Richard Dale @.>
Sent: Friday, July 21, 2023 6:33 AM
To: stefan-jansen/zipline-reloaded @.>
Cc: azsaha123 @.>; Author @.>
Subject: Re: [stefan-jansen/zipline-reloaded] assets-7.sqlite' doesn't exist (Issue #211)
Note that the conda-forge hasn't yet been updated with these fixes:
https://anaconda.org/conda-forge/zipline-reloaded/files
Does conda-forge need a bit of gentle persuasion to run?
—
Reply to this email directly, view it on GitHub https://github.com/stefan-jansen/zipline-reloaded/issues/211#issuecomment-1644710421 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AVV4DEMQUI4OSPVDNIQKJCDXRGW2TANCNFSM6AAAAAA2RMGUYA .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AVV4DEKO5H7N3IROHPGMFVDXRGW2TA5CNFSM6AAAAAA2RMGUYCWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTTCBBFBK.gif Message ID: @.*** @.***> >

conda-forge has been updated last week so closing this. feel free to reopen if issues persist.


Add a ram-size option

Wanted to increase the RAM on the emulator because it seemed to be running out of RAM in one of the runs, so decided to add this as an option as well.
There's no input validation on this field, same as the field for SD Card size.

Thank you for the super fast review!


Add a Dynamic BVH implementation.

-Based on Bullet Dbvh, has style and functional changes.
-Helps achieve efficient pairing
-Needed to optimize rendering
-Needed to optimize physics

I think that this PR should also contains tests, because seems that results will be mostly reproducible and this will prevent from regressions like in LocalVector - https://github.com/godotengine/godot/pull/40913

@qarmin I agree, though its difficult to think how tests for this could be created


[] update to

Library name:
New version number:
Other information that may be useful (release notes, etc...)

Invalid issue.


Replace test meta from +junit to +tests

As per changes in https://github.com/objectionary/eo/issues/1869 for all EO tests replace +junit with +tests.

@mximp the puzzle #151 is still not solved.


chore: bdd-gh-1584191114 to 0.0.1

chore: Promote bdd-gh-1584191114 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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


Translation Issue

Issue Summary
Hi! There is translation issue (typo) on ghost/i18n/locales/tr/comments.json
It must be "Şimdi kayıt ol" instead of "Şimdi kaydıt ol"
Could you please fix it?
Steps to Reproduce
Make it "Şimdi kayıt ol"
Ghost Version
5.65.0
Node.js Version
--
How did you install Ghost?
Ghost.org Pro Membership
Database type
MySQL 5.7
Browser & OS version
No response
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

Will take a look at the PRs soon 🙂


server: validate input

Use deepmap's OapiRequestValidator function to validate input.

 :)

After trying a few things out, it seems we need to specify the relative baseurls in order for this to work properly. Try adding
servers:
- url: "/api/image-builder/v1"
  description: Running locally
- url: "/api/image-builder/v1.0"
  description: Running locally

to the spec between paths: and info:


Update placeholder to required syntax

A name with spaces produces error package.json: Name contains illegal characters
Co-authored-by: Heather Moore<EMAIL_ADDRESS>

:+1:


V6 - changes in array item field 1, do not remove error from field 2

I have an array editor, and a validation function.
When I make changes in 1 field, my validation function runs and based on the value of field 1, the validation for field 2 might change (it's not passwords comparison).
When I make changes to field 1, validation is updated only for that field, but when the validation function removes an error for field 2, that error is not removed in the UI.

On v6, updating a field does not rerender other fields (unlike v6). This might be what's causing your issue. Could you add some code to diagnose what's happening here?

On v6, updating a field does not rerender other fields (unlike v6)

@ooflorent this is the first I'm seeing of this (i'm guessing you meant "unlike v5"). Do you have more info (issues, docs, etc.) I could look into? I feel like I've looked through the docs, especially the migration guide, but I'm not seeing much else on this issue

I understand what @q42jaap means. This is a bug.
It's not particularly related to arrays. If you had a validation function like:
const validate = values => {
  const errors = {}
  if (values.newPassword !== values.confirmNewPassword) {
    errors.confirmNewPassword = 'Must match!'
  }
  return errors
}

And then you edited the newPassword field in such a way that it added or removed the error on confirmNewPassword, the <Field name="confirmNewPassword"/> won't be rerendered....but it should be!
@clayne11 is currently working on some changes (see #908) that will result in the sync validation errors being in the Redux store, which will make fixing this bug trivial.

I'm seeing an issue where other Fields within a FieldArray aren't re-rendered when the field array has new elements inserted/removed. Is this related to this issue or should I make a new one?
Thanks!

Is this related to this issue or should I make a new one?

New one. 😦

This is fixed. I will comment again when the fix is published.

Thanks @erikras ! No pressure, but what's the eta on the new release?

Published in v6.0.0-rc.1.


endUser identifiers as standard XDM fields and corresponding ExpEvent Mixin

Presently, all the identifiers are placed at https://github.com/adobe/xdm/blob/master/schemas/context/enduserids.schema.json
But these are of type identities. There are no schema objects which contain such identifiers together as standard fields rather than as identities. So this PR introduces endUser identifiers as standard XDM fields and also an Experience Event mixin which includes these identifiers
Plus, an attempt has been made to further divide these identifiers on the basis of their types, like personal identifiers (Email, PhoneNumber) , DeviceIdentifiers (Experience Cloud Id, Push token) and Solution identifiers (Adobe solution identifiers like campaign id, analytics id, etc).
Corresponding example files have also been made available as part of the PR.

@lrosenthol @cmathis please resume review

@lrosenthol @cmathis review comments incorporated, please resume review.

@mjasrotia and @prabhum2  - Will you look at this PR and verify that the field names are correct. Should the field names be camel case ecId, gcId, etc.

@rajindal  - I see we have grouped each of the different identifier types into their own datatype, but we have created a single mixins which brings them all in. Do you think it would be beneficial to have three different mixins for each of the different types of identifiers. This way a user can bring in only the types they need to use.
Also, do you every see these new datatypes being used outside of this mixin? If not, then what about moving the identifiers out of a datatype and simply define them in the mixin? Just an idea.
Either way we are planning to have this PR included in the release into INT and Stage next week. Just a few minor details to work out as it's a pain to change later.

@Craig<EMAIL_ADDRESS>I actually created these as separate to have the flexibility of including them in different use case mixins if need be.
Like if a use case comes where someone wants to have device details with device identifiers in a mixin for some use case, then this data model will help.
So, that's why for our use case of having all identifiers in one mixin to be available as standard fields, I created the experienceevent-identifier mixin.
Get Outlook for Androidhttps://aka.ms/ghei36

From: cmathis<EMAIL_ADDRESS>Sent: Thursday, August 13, 2020 10:41:47 PM
To: adobe/xdm<EMAIL_ADDRESS>Cc: Rajat Jindal<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [adobe/xdm] endUser identifiers as standard XDM fields and corresponding ExpEvent Mixin (#937)
@rajindalhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frajindal&data=02|01|rajindal%40adobe.com|bc4453fa71b94829868108d83fabf6cb|fa7b1b5a7b34438794aed2c178decee1|0|0|637329355103258193&sdata=FyI1%2BcnGDRoWxiXXBz6P4TFPINzfciT6i%2BXHdS6DMZk%3D&reserved=0 - I see we have grouped each of the different identifier types into their own datatype, but we have created a single mixins which brings them all in. Do you think it would be beneficial to have three different mixins for each of the different types of identifiers. This way a user can bring in only the types they need to use.
Also, do you every see these new datatypes being used outside of this mixin? If not, then what about moving the identifiers out of a datatype and simply define them in the mixin? Just an idea.
Either way we are planning to have this PR included in the release into INT and Stage next week. Just a few minor details to work out as it's a pain to change later.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fadobe%2Fxdm%2Fpull%2F937%23issuecomment-673599448&data=02|01|rajindal%40adobe.com|bc4453fa71b94829868108d83fabf6cb|fa7b1b5a7b34438794aed2c178decee1|0|0|637329355103268187&sdata=uCESAzuswR%2FUPtDn6hT541nLft3diel%2FbNs5qz8aCUo%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAI5CKGU3VLQPOWYD54XCVJTSAQNFHANCNFSM4P2E25WQ&data=02|01|rajindal%40adobe.com|bc4453fa71b94829868108d83fabf6cb|fa7b1b5a7b34438794aed2c178decee1|0|0|637329355103268187&sdata=04uQNM0jmVEQ0xGSOOJyB43qhF9CQ64xCUAEHlW5J6I%3D&reserved=0.

Sounds good. So it sounds like we are free to include in this PR (or another in the future) a new device-identifiers mixin, solution-idenfiers mixin, personal-identifiers mixin, etc.
This mixin you have included in this PR is more of a grouping all all potential identifiers. Is this right?

Yes, exactly, in my PR, I have grouped all the potential identifiers.
Get Outlook for Androidhttps://aka.ms/ghei36

From: cmathis<EMAIL_ADDRESS>Sent: Thursday, August 13, 2020 11:01:42 PM
To: adobe/xdm<EMAIL_ADDRESS>Cc: Rajat Jindal<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [adobe/xdm] endUser identifiers as standard XDM fields and corresponding ExpEvent Mixin (#937)
Sounds good. So it sounds like we are free to include in this PR (or another in the future) a new device-identifiers mixin, solution-idenfiers mixin, personal-identifiers mixin, etc.
This mixin you have included in this PR is more of a grouping all all potential identifiers. Is this right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fadobe%2Fxdm%2Fpull%2F937%23issuecomment-673609337&data=02|01|rajindal%40adobe.com|1625208a59484cd7105108d83faebea0|fa7b1b5a7b34438794aed2c178decee1|0|0|637329367034336425&sdata=aJaX9u2F6p0LawB5OcYk1lD0tpfgU42rkHcREMoIGww%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAI5CKGUYYB5OL2TN365ZJW3SAQPP5ANCNFSM4P2E25WQ&data=02|01|rajindal%40adobe.com|1625208a59484cd7105108d83faebea0|fa7b1b5a7b34438794aed2c178decee1|0|0|637329367034336425&sdata=svSC7bwILCMpyO2kzjcR3LjPW6INruBc%2FXbrLRKi8Gw%3D&reserved=0.

Okay, we are going to make a few small modifications to this PR.

Convert all the field names to camel case - emailaddressid -> emailAddressId. I know we did no have it this way in the original enduserids mixin, but given that was old and we use camel case in all current field names we will go that way here.
We will create three other mixins for each of the different types of identifiers

@rajindal - Please remind us here why the current enduserids "https://ns.adobe.com/experience/aaid" and other fields could not be used. I know it was because they were of type "identity", but what problem did that cause?

@clubycoder @cmathis
The use case is for a solution like campaign to map its identifier fields like campaign to such standard fields (created in the PR) as part of solution's out of the box mappings , and leave it to the customer to mark those fields as identities using xdm descriptors. Some customers have requirements where they want to decide which field they want to mark as identities, rather than solution deciding for them.
This cannot be fulfilled using endUserIds schema because its already of type identity, it would not be possible to mark it as non-identity.

The one problem with this idea is that the ecid should always have an identity namespace of "ECID", emailaddressid should always have an identity namespace of "EMAIL", etc. All of these proposed solution and device identifiers in this PR have standard identity namespace defined by the system already. It may cause confusion and issues in down stream processing if an incorrect identity namespace is used.
I don't understand the purpose of having a standard "identifier" field not be associated with its "identity" namespace. Can you explain why you would ever do that?

@cmathis ,
In campaign, we want to send certain such potential identity fields like ecid, emailid, campaignid, etc as standard fields to XDM and leave it to the customer to mark those fields as identity on XDM UI using identity descriptors.
For ease of customer, we want to group these fields together so as to make a good experience for the customer, where the customer knows where all standard identifiers are, just like endUser has all them together. So that's why this PR.
I agree the downstream application can get confusion with having different namespace for different field, but I wonder how this is being handled today. the same problem can arise today with someone creating a custom field named ecid and then mark it as identity tagged with EMAIL namespace.
If not considering the custom fields, even the enduser schema where we was the namespace code and the id , there also it can be a case of someone mistakenly passing namespace code as EMAIL for ecid object.

I agree that a user can still send the wrong identity namespace with the current identityMap and enduserids fields so we can drop that argument.
But we are introducing another set of "standard" fields just because Campaign wants to do something differently. The point of a "standard" model is that its used the same across all users of that model. The one and only "standard" approach being used by all existing Adobe solutions today is the identityMap. The enduserids is used in conjunction with that to support query performance, but it's never used without the identityMap being present as well.
So what we are wanting to do in this PR is present a new "standard" which is going to be tricky. It's still a possibility, but it can't be to drive some UI behavior in Campaign.  We will have @khanna-vikas  and a few others weigh in on this subject. Probably best to schedule a meeting to talk more about it.

@cmathis , yes Campaign can have a UI where we present possible identity fields. But the intention was to leverage XDM descriptors . The intention was to map potential identityFields to XDM fields out of the box , and let the customer use the XDM descriptors to select identities .
Since endUserIds cannot be used for this purpose, but a similar grouping of fields is required, from here the identifiers grouping idea came.
The idea came to support Campaign's use case, but it seems like every solution wanting to leverage XDM descriptors this way can use these identifiers.
WDYT?

There is already a standard for handling solution identifiers and ECID under endUserIds and and I'm seeing no reason to change that standard.  Also the cost for changing something so fundamental to our systems would be huge.  The same is true for global gross-customer identifiers like EMAIL and mobile advertising IDs.
On the point of allowing users to decide to not mark ECID or solution identifiers as identifiers, that is something we can't allow because it will break down the semantics and assumptions we rely on in the whole system.  That would also give customers a gun they don't need to shoot themselves in the foot with their implementation.
Campaign needs to follow the existing standards to work with the holistic system.
👎

@rajindal  - We had our normal PR review today and talked about this. For all standard identifiers (aaid, ecid, etc.) we suggest you pass those using the identityMap with the known identity namespace associated. Descriptors are to be used for custom fields where the namespace is not known.
If there is still a reason for AEP to have a standard identity field that should not be consumed by the identity service, then please have the PM of this project talk to the PM of identity and XDM to go over that detail.


Config File Name

Please would you have a default config file name, like phpda.yaml so that it can be added to packages alongside things like phpunit.xml and the like?
That way the command can be run without specifying a config file and look for the local phpda.yaml file.

#35
I've create a pull request for this.


Feature request: colorize the chat window

Chats would be much easier to follow if the the author name of each message were displayed in a unique, persistent color, and if the timestamp were displayed in a color other than the default text color.
Here is a random example of what I mean (from irssi):
https://i.stack.imgur.com/89nsP.png
Thanks!

I see there was some effort done to colorize names in the chat window but was it ever released?

@Zebrazilla it will be available in the upcoming version, you can check it out in the branch v0.4.1, add colorize to name in the slack-term config file to get individualized name colors. e.g.:
{
  "theme": {
    "message": "colorize,fg-bold"
  }
}

I second this, but as of now any changes made to slack-term.json don't seem to take with regards to color, less It's just me  :)


refactor: vite middlewares as named functions

Description
This PR eases debugging of the middlewares by using named functions as middlewares.
Additional context
connect uses the name of the middleware functions for debugging.
When debugging vite with DEBUG=connect:dispatcher, this PR replaces, eg, connect:dispatcher <anonymous>  : /src/main.ts +0ms with connect:dispatcher viteTransformMiddleware  : /src/main.ts +0ms, which is very useful to know which middleware a request goes through.

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[X] Other

Before submitting the PR, please make sure you do the following

[X] Read the Contributing Guidelines.
[X] Read the Pull Request Guidelines and follow the Commit Convention.
[X] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

@Shinigami92 A comment has been added to each middleware.
FYI, I have made other attempts (using ESLint, tests...), not conclusively.


javascript sql batch not working

Hi, I am trying to execute js client-side batch script but its not working correctly
code:
db.language('javascript').command("var r = db.query('select from user');r;").then(function(response){
        console.log(response);
    });

Am i doing it in a wrong way?

Only 'sql' and 'gremlin' are supported languages.
node-orientdb-http doesn't support batches.

does node-orientdb-http support server-side function execution?

I'm afraid not. And since i'm not actively maintaining the this repo, I don't expect these features in the short future. Feel free to fork the repo and add the features that you need.


netatalk: 3.1.11 -> 3.1.12

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/netatalk/versions.
meta.description for netatalk is: '"Apple Filing Protocol Server"'.

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/cnid2_create had a zero exit code or showed the expected version
/nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/macusers passed the binary check.
Warning: no invocation of /nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/asip-status.pl had a zero exit code or showed the expected version
/nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/apple_dump passed the binary check.
Warning: no invocation of /nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/afppasswd had a zero exit code or showed the expected version
/nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/afpldaptest passed the binary check.
Warning: no invocation of /nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/ad had a zero exit code or showed the expected version
/nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/dbd passed the binary check.
/nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/netatalk-config passed the binary check.
Warning: no invocation of /nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/afpstats had a zero exit code or showed the expected version
/nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/afpd passed the binary check.
Warning: no invocation of /nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/cnid_dbd had a zero exit code or showed the expected version
/nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/cnid_metad passed the binary check.
/nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin/netatalk passed the binary check.
8 of 14 passed binary check by having a zero exit code.
2 of 14 passed binary check by having the new version present in output.
found 3.1.12 with grep in /nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12
directory tree listing: https://gist.github.com/2364d46aa87e3957ec21283dcd1589d0
du listing: https://gist.github.com/96af6d397cb7d3912ef762674d760251

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
netatalk

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A netatalk https://github.com/r-ryantm/nixpkgs/archive/b32c357ea9beaa4f999635628745ad82af3cf58e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12
ls -la /nix/store/palg0jci4ixw1gi3qdj2y7x264zks1ln-netatalk-3.1.12/bin

Please update to this latest release as soon as possible as this releases fixes an major security issue (CVE-2018-1160)

Netatalk before 3.1.12 is vulnerable to an out of bounds write in dsi_opensess.c. This is due to lack of bounds checking on attacker controlled data. A remote unauthenticated attacker can leverage this vulnerability to achieve arbitrary code execution.
https://nvd.nist.gov/vuln/detail/CVE-2018-1160

@c0bw3b: you self-assigned this – any more plans about it?

@vcunat thanks for backporting this.
I meant to look at it when assigning it to myself, then got quite busy.


Affiliation of category "Belongs to series" does not work

Categories set to "Belongs to series" are not shown in videos of the same series.

Additional observation: When I try to change the affiliation from "Belongs to series" to "Belongs to the video" and click on save, the window gets close. When I reopen the window "Edit category" the option "Belongs to series" is still activated. Somehow this change does not get saved.

At least this should now be fixed on staging.

Additional observation: When I try to change the affiliation from "Belongs to series" to "Belongs to the video" and click on save, the window gets close. When I reopen the window "Edit category" the option "Belongs to series" is still activated. Somehow this change does not get saved.

At least this should now be fixed on staging.

Thx. I can confirm this part of the fix.

The basic behaviour works now as expected. But there is a new bug which occurs, if I have at least two videos (1 and 2) and if I edit the affiliation setting of category belongig to the series in the video, where I did NOT create it.
For example (see screencast): I create a series-category in video 1 and edit the affiliation setting to "Belongs to the video" in video 2: At first everything seems fine. But when I reload the browser in video 2 all existing labels show up twice.
https://user-images.githubusercontent.com/15246287/150859997-ab6bfe04-1479-4489-91b5-dba51060b2e1.mp4

From what I understand, the idea was that there should still be a difference between the event which created the category in the first place and the events where it shows up because they are part of the series. What now happens is likely that you turn the category into a video-local category on the second video, but when you reload, the series categories are still pushed to the video two again. That is why you end up with the duplicated categories.
I can see that this is confusing, but I also don't quite get what the original intend for this was? Was it that you shouldn't be able to turn series categories local at all, so that they don't suddenly disappear or duplicate? Was it that you wouldn't be able to edit the category on the second video? Was it that you just wouldn't be able to change the categories' affiliation on the second video? Was it that you could turn them local?
So, what I'm asking is, can you describe how you would expect this to behave?

(Our/The) idea was that it does not matter in which event a category belongig to a series is created. Mainly because we do not believe that teachers can recall in which event they have created the category. Therefore you should be able to turn a category belongig to series to a local category (= belonging to a video) in all events.

Is the following an accurate description of what you would expect to happen?
Step 1:

Create category A and labels B and C
Mark as “belongs to series”
Result: Category A shows up on all events of the series, with labels B and C

Step 2:

Go to a random event X of the series
Edit category A
Mark as “belongs to video”
Result: Category A becomes a local category on video X with labels B and C
Result: Other events of that series will not show the category any longer

Step 3:

Go to event X of the series
Edit category A
Mark as “belongs to series”
Result: Category A shows up on all events of the series, with labels B and C

Exactly. Thanks for summing it up in detail.

Bad news: I can't reproduce the issue. I've tries different test systems, different browsers, …
Good news is that I might have found the issue anyway.
If I'm righter than this could be a raise condition happening when saving the category itself takes some time but saving the labels is pretty fast. I hope that https://github.com/opencast/annotation-tool/pull/569 fixes the issue.

Looks fine now, thx!


Factor SwiftProtobuf module name into the namer, add SwiftProtobufMod…

…uleName
This is a change to allow the generator to generate code against a parameter-specified swift-protobuf module. The new parameter is SwiftProtobufModuleName, and the namer holds the knowledge of which module name to use throughout generation now.

I have a few comments, and am looking for feedback! Thanks.

I clearly need to add tests, I'll do that.
I am not fond of the fact that the namer and the module mapper both individually know about the module name, it makes their initialization awkward.
The generator doesn't change the way it generates any code comments, I think that's OK?

Initial thought is should we kill off SwiftProtobufInfo.name to ensure nothing accidentally uses the wrong name in the future? For comments and things where it doesn't really need to be the module name, we could just go and hardcode the value in the strings that need it.
I need to reread the code, but could we make the module namer know the name, and that's it?  The Namer could pass thru the value, but not independently track it.  That might make things a little cleaner?
reservedTypeNames is one of the problem cases as we do need to ensure whatever someone wants gets handled there also.

reservedTypeNames is interesting. I gave that some thought as I grepped through for SwiftProtobufInfo.name, and I am not so sure what I'd want to change. Do you think it would be appropriate to sanitize names differently depending on the parameter?
Removing SwiftProtobufInfo.name is a great idea, I'll do that.

reservedTypeNames - the problem is that code serves as a catch all for ensuring a generated type name doesn't match the module name.  We could punt on that check and if someone does get overlap, oh well; but that's doesn't seem great as it means we generate code that won't compile.
If I'm remember the flow right, we could move the standard library name into reservedTypeNames, but then look at moving sanitizeTypeName onto the Namer to it can use the ivar with the module name directly to check against that as well as reservedTypeNames.
That way we also avoid using SwiftProtobuf as a type name, but also if someone does use a custom name, we'll also ensure that doesn't get used.

What do you think @thomasvl , does there remain anything to address here?

Sorry, last time I looked I thought there were some unresolved comments, but I guess not.
The only things that giving me pause is the cycle between NamingUtils and the Namer.  But since that's all within the plugin library, we can always revisit it in the future.

Yes, I think when you called out the sanitize functions you suggested to move them into the namer. I'll play with that and send a separate change.

Yes, I think when you called out the sanitize functions you suggested to move them into the namer. I'll play with that and send a separate change.

The other option is to just pass the module name thru instead of the whole namer since that's all the sanitize really needs.


Use typeface-roboto instead of google cdn

Less API calls.
Remember to yarn.

Will it increase the bundle Size. Also roboto is usually cached in the device as it is very popular font from google cdn

Indeed it will increase the bundle size, but after all it is still downloaded.
Anyway, the bundle is cached too and it allows the use of the font while offline. And also if I remember correctly the font face are already included.

@SebastianCApostolescu you are correct MUI uses svg version on icons. @JesseRWeigel This change is worth committing.

Thanks @SebastianCApostolescu !


Broken IRC webchat link in the footer

irc: #asciinema on freenode

It should be considered broken because:

freenode has long rotated their servers under "new" administration, so I doubt the channel still exists there;
even if it does, the webchat interface doesn't work with freenode:

I'd suggest replacing it with: #asciinema on libera.chat. I wanted to send a PR but couldn't find the link in this repository, so I guess it's specific to the hosted instance.

This is good idea. I already registered #asciinema on Libera.Chat.

Fixed in fb4496edadc7cccdd3e9ba23ddb4a6126266f836


Origin is freezing

Hello!
I ran "Origin" via "Lutris".
When DXVK enable, games have freezing every two second.
I tried to change version dxvk and add "dxgi.customVendorId = 10DE " in "dxvk.config", but it doesn't help.
Origin_d3d9.log
Origin_d3d11.log
Origin_dxgi.log
about_system.txt

Try disabling D3D9.

Even after hours of play? But this happens even when only Origin is running. I know it because my hard disk loaded every two second and i can hear how it happen

How exactly do you expect me to fix your hard drive problems caused by Origin in DXVK?

i don't think the problem is in my hard drive. When i use proton it's normal, when i turn off dxvk in lutris it's the same

Origin itself does not do any significant amounts of D3D11 rendering.
Just disable DXVK if it doesn't work for you I guess. There's nothing I can fix here, and I never had that problem back in the day when Origin actually worked on my system.


build warning: 'swift-algorithms': found 1 file(s) which are unhandled; explicitly declare them as resources or exclude from the target

warning: 'swift-algorithms': found 1 file(s) which are unhandled; explicitly declare them as resources or exclude from the target
    .build/checkouts/swift-algorithms/Sources/Algorithms/Documentation.docc

Resolved by #240.


Governance. Delegation summary. Delegate voted info

Description
Add delegate Votes on Delegation summary:
https://www.figma.com/design/4vjREBpvZK7M3m1GaoVptq/OpenGov-delegations?node-id=1-4442&t=1RubE1Senh4y0pqw-1

Status verification: ⏳
What has been verified:

AYE\NAY\ABSTAIN state
Different accounts
https://github.com/novasamatech/nova-spektr/issues/2406

Status verification: ✅


Allocations regressed in 5.9

Motivation:
I'm not yet sure why they regressed, but having the CI repeatedly fail because of this isn't helpful. We should investigate the regression.
Modifications:
Raise the allocation limits.
Result:
CI is green again.

Flaky syscall wrapper failure in nightly.

@swift-server-bot test this please


update dependencies

Adding ratatui 0.25.0 compatibility

v3.0.0 is now available on crates.io. It required a major version bump as the latest ratatui you upgraded requires unstable features