refund advances

Reticketed from #3675.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/26543937-refund-advances?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F85909&utm_medium=issues&utm_source=github).

@whit537, following on from my earlier questions, it's clear that we would no longer have the scenario of charging a user and then immediately having to refund the same user. And possibly, the extent of the 'small balance' situation I'd been thinking of will have diminished after a few weeks of charging in arrears.
There's something else I'd like to clarify. Would refunds be made as payments back to the same credit card? Or to the paypal account of the user? If it's the latter, what do we do in the case of users who don't have paypal accounts? e.g. someone who is only giving and hasn't bothered to set up a paypal account?

Would refunds be made as payments back to the same credit card?

Yes. Balanced and Braintree have refund APIs that work even if the card was cancelled—though almost certainly not if the entire bank/card account was closed, a situation we'll almost certainly run into.

Having a spot of trouble. When running make test, things getting stuck at recreating public schema .... Doesn't seem to have anything to do with my code changes - trying with a fresh checkout doesn't help. Wonder if my machine's got corrupted.
Also, a couple of questions. Found a MINIMUM_CREDIT value in exchanges.py. What is this meant for? And do Braintree levy a fee on credits as well?

I've moved beyond the 'corruption' (was my code after all :-(), but now facing a seemingly intractable problem. I think possibly something to do with my (lack of) understanding of threaded_map/thread pool usage. Trying to figure it out.

@rorepo Possibly related to https://github.com/gratipay/gratipay.com/issues/3789?

I've moved beyond the 'corruption'

Cool. :)

I think possibly something to do with my (lack of) understanding of threaded_map/thread pool usage. Trying to figure it out.

How can I help? :-)

Found a MINIMUM_CREDIT value in exchanges.py. What is this meant for?

This is vestigial from when we did bank payouts via Balanced. It can probably be removed now.

And do Braintree levy a fee on credits as well?

Braintree doesn't provide credits (payouts). The payout route we offer is PayPal.

it's clear that we would no longer have the scenario of charging a user and then immediately having to refund the same user.

Actually, if a payout fails, we do want to immediately refund all of the cards the money came in on. No?

Sorry, @whit537, looks like we got our wires crossed a bit.

Braintree doesn't provide credits (payouts). The payout route we offer is PayPal.

However, this earlier exchange suggests otherwise:

Would refunds be made as payments back to the same credit card?
Yes. Balanced and Braintree have refund APIs that work even if the card was cancelled—though almost certainly not if the entire bank/card account was closed, a situation we'll almost certainly run into.

To help things along :-), I've been trying credits with Braintree and as far as I can see, it was working (i.e. didn't get any errors - I expect we'd need to confirm at the Braintree end as well).
So now, where does this leave us - Braintree or Paypal? :-)
If it's going to be Paypal, my above problem may no longer be relevant.

However, this earlier exchange suggests otherwise:

Ah, yes. That's a subtle difference: Braintree does provide a refund API, but technically that's a refund of a payin, which is not quite the same as a payout.

So now, where does this leave us - Braintree or Paypal? :-)

Braintree for refunds! And Balanced, too ...

I've been trying credits with Braintree and as far as I can see, it was working (i.e. didn't get any errors - I expect we'd need to confirm at the Braintree end as well).

Cool. Was that using a sandbox or production? If a sandbox (as I hope ;), do you have access to the Braintree dashboard for the sandbox?

Well, must be a sandbox - defaults.env says BRAINTREE_SANDBOX_MODE=true. Besides, I'm sure Braintree would've objected to transactions in the name of 'obama', 'picard', etc., if it had been otherwise :-)
How do I access the dashboard?

@rorepo I think your best bet is to sign up for your own account at https://sandbox.braintreegateway.com/login and set the relevant BRAINTREE_ keys in your local.env to point to your own sandbox.

I got things working up to the point where the refund was being made and the participant balance  correspondingly being brought down to 0. But there was one problem remaining - check_db. The expected balance was the (-ve of the) figure I had directly inserted into the participant balance in order to create the excess balance situation. I suspect this was because there was no corresponding payments record.
At this point, I thought I'd try my own Braintree sandbox credentials. Things didn't go well. Braintree returned an error, complaining that the custom field participant_id was invalid. Removing that didn't help either, so I put it back and reverted back to the old credentials. Now I get an authorization error. Haven't been able to move beyond that.
P.S. I hadn't used local.env to put in my credentials - made the changes in test.env.

Braintree returned an error, complaining that the custom field participant_id was invalid.

Ah, okay. You would need to configure that under "Settings > Processing"

If you're able to get a sandbox working, we should add docs to the README.

P.S. I hadn't used local.env to put in my credentials - made the changes in test.env.

Okay. The former is for the web app (make run), the latter for the test suite.

You might consider creating a tests/local.env file instead of modifying tests/test.env, since the latter is in source control and the former is .gitignored.

Turning to this.

Who do we refund, and how much?

I'm thinking about this in the context of 1.0 refund as well (#3539). I think we want to think in terms of "generations" of money. Given:

Week 1—We charge Alice $10 and Alice gives $2 to Bob.
Week 2—Alice gives $2 to Bob, and Bob gives $0.50 to Carl.
Week 3—Alice gives $2 to Bob, and Bob gives $0.50 to Carl.

The goal on this ticket is to refund $6 to Alice: that's first-generation money, because it's still sitting in the balance of the account by which it first entered the system.
The goal over on #3539 is to refund—back to Alice—the $1 that Carl is holding, and the $3 that Bob is holding.

We need a list of accounts holding first-generation money.
And for that I believe we're going to need to backfill the status and route columns of exchanges first: #2779.

Who do we refund, and how much?

Actually, the question is, which transactions do we refund, and how much? That's why #2779 is prerequisite.

Tryin' to eek something out here ...

OMG! Had to take a break and looks like I missed out on some serious fun. Looked at #3810 and my head's spinning.
Had got a bit lost at the end here in 3763, looked like I was missing some historical knowledge. Now I suppose that's just academic.
@whit537, any reason why this one hasn't actually been closed? #3810 does mention that it closes this one.

Gonna clean up and merge #3810, and then write a blog post.

And yeah, there was some serious fun on #3810 over the weekend. Landed it an hour before the Balanced cutoff. :scream_cat:

Actually, we should refund advances on Braintree before closing this.

In that case, there are two things I'd like some input on, both probably to do with historical knowledge I'm missing:

The example you'd given earlier:

Week 1—We charge Alice $10 and Alice gives $2 to Bob.
Week 2—Alice gives $2 to Bob, and Bob gives $0.50 to Carl.
Week 3—Alice gives $2 to Bob, and Bob gives $0.50 to Carl.

How is it that all the $6 that Alice gave out has to be refunded? I was thinking of this as Alice giving to a team owner (Bob), who in turn distributed this to a team member (or the team member 'taking' it).

I'm afraid I'm rather lost on this one as well:

And for that I believe we're going to need to backfill the status and route columns of exchanges first: #2779.

How is it that all the $6 that Alice gave out has to be refunded? I was thinking of this as Alice giving to a team owner (Bob), who in turn distributed this to a team member (or the team member 'taking' it).

Under Gratipay 1.0, ~users gave directly to each other.

I'm afraid I'm rather lost on this one as well:

We needed a connection between individual credit card transactions in our database and in Balanced's database. The status and route columns were actually just part of the picture: we needed a new ref column to link the individual transactions (route just links to the payment method).

I'm still unclear, I'm afraid. Users giving directly to each other is fine, but my main confusion is that I thought it was only 'excess', i.e. whatever's not been distributed out to end recipients, that needed to be refunded.
And about the ref column - how would we go about filling this in?

And about the ref column - how would we go about filling this in?

We'll have to use logs from our various payment providers and compare them with our records in exchanges. This comes under #2779.

Users giving directly to each other is fine, but my main confusion is that I thought it was only 'excess', i.e. whatever's not been distributed out to end recipients, that needed to be refunded.

Correct. We only refunded the user's current balance, but the way refunds work is you have to perform them against a specific transaction. You can refund less than the original transaction amount. We wanted to refund Alice the $4 she's left with, and the way we do that is to refund here $4 against the $10 transaction where we originally charged her.
Anyway this is done in #3810. :-)


Correction TypeError: Cannot read property 'IsValidated' of undefined

I do not know if it's the best way, it's the first time I do this so apologize for any lack of professionalism

You can check https://github.com/newbreedofgeek/react-stepzilla/pull/149 for reference.


MC Launcher crashes upon Play

Minecraft Version
1.18.2
Describe the Bug
i wish i had a crashlog for you, i do however, have a manifest for you at the very least, so you can.Compare for the future if anybody else mentions that this crashes their game, maybe some mods will line up to tell whats wrong, but yeah it wont even boot up past the launcher.
Your mod looks super cool and I'm really kind of disappointed it doesn't run. For me. However, here's hoping that something changes in the future and it does work. That would be awesome.
I hope this helps to some degree.
Reproduction Steps

Install mod.
Open Minecraft Launcher.
Press play.
Profit- i mean crash.

MultiMC
Minecraft 1.18.2
Forge 40.2.0
LWJGL 3.2.2
create-1.18.2-0.5.0.i
create-confectionery1.18.2_v1.0.8
create_central_kitchen-1.18.2-for-create-0.5.0.i-1.3.2
create_enchantment_industry-1.18.2-for-create-0.5.0.i-1.2.0
createaddition-1.18.2-20230426a
flywheel-forge-1.18.2-0.6.8.a
Farmer's Delight 1.18.2-1.2.0
I had more mods installed, but pared it down to make troubleshooting a bit easier.
crash-2023-05-18_14.10.17-fml.txt
Another crash log, this one with only Create, Flywheel, and Central Kitchen installed:
crash-2023-05-18_14.12.36-fml.txt
I hope that this is helpful to you.

Would have to provide the common config file since there seems to be a problem with it
Also:
If Minecraft crashes like that you can usually still provide the debug.log file
(Location depends on the launcher it seems - if you ultimately start it from the vanilla launcher (which gets opened by the Curse launcher e. g.) you can also go into the settings and set it so that the console (log) window gets opened and provide that)

Sorry about the delay. Having the mod generate a fresh config file actually seems to have fixed the issue.
The old config file:
create_central_kitchen-common.toml.txt
Debug Log:
debug.log
Installation Information:
Minecraft 1.18.2
Forge 40.2.4
LWJGL 3.2.2
AutoRegLib-1.7-53.jar
FarmersDelight-1.18.2-1.2.0.jar
FramedBlocks-5.11.4.jar
ImmersiveEngineering-1.18.2-8.4.0-161.jar
Mekanism-1.18.2-<IP_ADDRESS>5.jar
MekanismAdditions-1.18.2-<IP_ADDRESS>5.jar
MekanismGenerators-1.18.2-<IP_ADDRESS>5.jar
Patchouli-1.18.2-71.1.jar
Quark-3.2-358.jar
QuarkOddities-1.18.jar
StorageDrawers-1.18.2-10.2.1.jar
SuperCircuitMaker2-0.2.9+1.18.2.jar
TKLib-0.0.23+1.18.2.jar
antiqueatlas-7.1.1-forge-mc1.18.2.jar
architectury-4.11.93-forge.jar
cloth-config-6.3.81-forge.jar
create-1.18.2-0.5.1.b.jar
create-confectionery1.18.2_v1.0.9.jar
create_central_kitchen-1.18.2-for-create-0.5.1.b-1.3.7.jar
create_enchantment_industry-1.18.2-for-create-0.5.1.b-1.2.4.b.jar
createaddition-1.18.2-20230527a.jar
flywheel-forge-1.18.2-0.6.8.a.jar
hexerei-0.2.2.jar
jei-1.18.2-forge-<IP_ADDRESS>4.jar
mutil-1.18.2-4.5.0.jar
pneumaticcraft-repressurized-1.18.2-3.6.0-19.jar
tetra-1.18.2-4.10.1.jar

I'm not sure why, how or when it is happening but this error seems to be caused by this entry in the common config file not having a # in front of it (indicating that it is a comment): [@cui:RequiresReload:both]
If you add a # before that line it should work without having to delete the config

A nightconfig problem I think. RaymondBlaze says he has a workaround but he's busy now.

Just want add on that I just had this issue on my server. I has been working well but I had the same issue. weird thing is this happen also while the server was running fully and my server stop working without anything showing in the console. This happen just randomly


Python Postcommits timing out on postCommitIT

Test suites failing:
https://ci-beam.apache.org/job/beam_PostCommit_Python38/
https://ci-beam.apache.org/job/beam_PostCommit_Python37/
https://ci-beam.apache.org/job/beam_PostCommit_Python36/
They seem to be hanging on postCommitIT for some reason, with the following logs repeating:
Source

17:21:55 WARNING:root:Make sure that locally built Python SDK docker image has Python 3.8 interpreter.
17:21:55
WARNING:apache_beam.runners.dataflow.dataflow_runner:Typical end users should not use this worker jar
feature. It can only be used when FnAPI is enabled.
17:21:55 WARNING:apache_beam.options.pipeline_options:Discarding
unparseable args: ['--sleep_secs=20', '--kms_key_name=projects/apache-beam-testing/locations/global/keyRings/beam-it/cryptoKeys/test']
17:22:03
WARNING:apache_beam.options.pipeline_options:Discarding unparseable args: ['--sleep_secs=20', '--kms_key_name=projects/apache-beam-testing/locations/global/keyRings/beam-it/cryptoKeys/test']
17:22:03
WARNING:root:Make sure that locally built Python SDK docker image has Python 3.8 interpreter.
17:22:59
WARNING:apache_beam.runners.dataflow.dataflow_runner:Typical end users should not use this worker jar
feature. It can only be used when FnAPI is enabled.
17:22:59 WARNING:apache_beam.options.pipeline_options:Discarding
unparseable args: ['--sleep_secs=20', '--kms_key_name=projects/apache-beam-testing/locations/global/keyRings/beam-it/cryptoKeys/test']
17:23:08
WARNING:apache_beam.options.pipeline_options:Discarding unparseable args: ['--sleep_secs=20', '--kms_key_name=projects/apache-beam-testing/locations/global/keyRings/beam-it/cryptoKeys/test']

The root cause seems to be PR #14481.
Imported from Jira BEAM-12249. Original Jira may contain additional context.
Reported by: danoliveira.

Currently on hold as GitHub migration involves splitting postcommit tests. Downgrade to P2 for tracking if it still occurs when GitHub migration completed.

.remove-labels P1
.add-labels P2


Add tab completion

Add tab completion for Bash and Zsh users. Update README with instructions on getting going with tab completion.
:bookmark_tabs:
Closes #119.

I'll give this a thorough review first thing tomorrow!

@chrisbodhi can you fix the item that @LinusU mentioned and be sure to sign the CLA: http://dojofoundation.org/about/claForm
Once you've done both things, just ping me here :)

@rwaldron Added the newline & I already signed the CLA when I submitted another PR a few months ago. Should be good to go, but let me know if there's anything else you need from me!

Thanks for the reminder—this is good to go!

:clap: Thanks for contributing again @chrisbodhi! I'm excited to use this.


custom dEntities no longer get recognised by scriptname after chunk unloads

- spawn e@scriptname <location>
on scriptname damaged:
- narrate "oink"

you get the idea here
When the chunk gets unloaded, and you load it back in to go to the entity, it no longer gets recognised in those events when the entity is visibly still there.
Tested on dev 390.
Tested on multiple servers besides my own infamous one (don't blame my server mcmonkey!).

mcmonkey pls

Can you give me a better way to test this so I can know for sure a fix works?

I.. can't replicate this

@Fortifier42 I couldn't either, thus my request for a better replication method, assuming he's doing something differenty that we're not thinking of. It could be another issue with just his server :o

"Tested on multiple servers besides my own infamous one (don't blame my server mcmonkey!)."

nice reading m8

How about a proper replication method?

http://mcmonkey.org/haste/22023 these are some snippets of script that it happens with

Try setting the other stuff such as name in the script itself, not in the entity.

why would i

replicated with
  type: entity
  entity_type: e@skeleton```

@mcmonkey4eva if you need to test a fix i'll happily do it for you as i already can replicate the problem

Fixed in 1.14+ by the tracker rewrite in https://github.com/DenizenScript/Denizen-For-Bukkit/commit/7872f0e8853ee24ebf00f4f1ab07de589a6a2d99


Custom content-cta should show free public preview

Issue Summary
When I create a custom view for content-cta it seems to not respect the free preview divider in the Ghost.
Steps to Reproduce

Create a page or post with content before and after a Public preview block, set post access to members only, and publish to confirm that the preview content displays.
Create a custom CTA in ./partials/content-cta.hbs as per the documentation. Restart Ghost if necessary.

The content before Public preview should be visible, but the entire post body is hidden.
Ghost Version
4.34.0
Node.js Version
?
How did you install Ghost?
Cloudron / locally with node
Database type
Other
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

@matthanley Ok thanks for letting me know, and I can confirm that it works. I would suggest adding this to the documentation page as it isn't written anywhere.


Prevent VS warning for empty attribute

Fixes https://github.com/aspnet/AspNetCore-ManualTests/issues/39.

/cc @Pilchie for 3.1-preview3

Approved for 3.1.0-Preview3


Change augmented image tracking state

Hi, I want to make some changes about this function:
https://github.com/googlecodelabs/arcore-augmentedimage-intro/blob/master/arcore-android-sdk-1.18.1/samples/augmented_image_java/app/src/main/java/com/google/ar/core/examples/java/augmentedimage/AugmentedImageActivity.java#L304

I want to change the "Tracking State" of augmented image to "Paused" when the tracking method is "LAST_KNOWN_POSE", because I want to re-detect and re-draw again when image is out of camera. I don't want the code to remember the position.
How to change the "Tracking State"?

Is it okay to re-detect and re-draw again when changing "Tracking State" to "STOPPED"?
I'm not clear about the document of ARcore on public static final TrackingState STOPPED from this link:
https://developers.google.com/ar/reference/java/com/google/ar/core/TrackingState

Thank you for your replying!

No, there's no way to change the image's TrackingState. You can implement this logic inside the "case TRACKING" section.
Stopped means the "current detected image" is no longer tracked. But if you move the device or image so that it re-enters the view, it will be re-detected.

Thank you for your hint! @kai-dream
I have successfully made it work!
I have other questions to ask you, too:

Is is possible to make .obj file touchable to show a pop-up view after the .obj file shows on the screen? If not, how about changing .obj file to other type of files such as .jpg or .png file?

I'm curious about those magic numbers you set for augmented images in draw function!

such as maze_edge_size:
https://github.com/googlecodelabs/arcore-augmentedimage-intro/blob/master/arcore-android-sdk-1.18.1/samples/augmented_image_java/app/src/main/java/com/google/ar/core/examples/java/augmentedimage/rendering/AugmentedImageRenderer.java#L69
and -251.3f, 0.0f, 129.0f in Pose.makeTranslation:
https://github.com/googlecodelabs/arcore-augmentedimage-intro/blob/master/arcore-android-sdk-1.18.1/samples/augmented_image_java/app/src/main/java/com/google/ar/core/examples/java/augmentedimage/rendering/AugmentedImageRenderer.java#L83
How do you adjust the magic number to make the GreenMaze attached on images? Do you have any formula to follow?
Thank you for your reply as always!

Is is possible to make .obj file touchable to show a pop-up view after the .obj file shows on the screen? If not, how about changing .obj file to other type of files such as .jpg or .png file?

I'm not sure I understand the "touchable" feature. Are you saying to allow user to click on the object? If so that is a Computer graphics question, and there're several existing solutions to workaround that. Change *.obj to *.jpg or *.png, I am not sure I quite understand what do you want to achieve with this too. *.ojb file is a 3d asset file, *.jpg and *.png are 2D image files, they are very different from each other.

Those magic numbers are the size of the actual maze in its obj file. You can open the maze *.obj file using a text editor to calculate its size (using this formular: max(x) - min(x) = size (x)). This obj file is not properly adjusted, it's center is not at (0, 0, 0), what's why I need to make a adjustment using (-251.3, 0.0, 129.0), this is not necessary if your *.obj file is centered at (0, 0, 0).

Yes! I want to allow user to click the object. After clicking the object, it will show some information about the image which rendered the object.  Such as information about Earth in the sample code.
What should I do if I want to allow user to click the 3D asset files?
How about rendering 2D images files and allow users to click them?
Could you give me some suggestions on these two different files, respectively?

I try to read the text of *.obj file. There are some different types of data in the file: v, vt, vn, f...  Should I just focus on the coordinate x of type v to calculate the size?
How do you calculate the center? By any other software?

I try to replace the maze obj file other obj file. However, the color of file is so weird. I try to replace .png with .mtl file in this line:

https://github.com/googlecodelabs/arcore-augmentedimage-intro/blob/master/arcore-android-sdk-1.18.1/samples/augmented_image_java/app/src/main/java/com/google/ar/core/examples/java/augmentedimage/rendering/AugmentedImageRenderer.java#L50
But it failed, how to do adjust the code to get the original color of *.obj file?
I am very grateful if you have free time to help me figure out my questions!

Thank you for giving further information! I'm sorry that I ask too many unrelated questions. I will think about other solutions to solve my problem. Thank you so much!


The word system renders as sys_t_e_m (version 1.7.1)

The word system renders as sys_t_e_m.  I have observed this in the preview window, HTML output, and PDF output.
Environment: Windows 10, AsciidocFX 1.7.1.

Ops, it is my mistake. I'll resolve it today.

Hi, please re-install https://github.com/asciidocfx/AsciidocFX/releases/tag/v1.7.1, it is fixed.

I have confirmed that the issue has been resolved.  Thanks so much for the quick response!


[repo] chore: Update html images to root readme

#minor
Related to #1057, in a way.
Simple image updates

Closing this to re-do after #1084 is updated. 👍🏼


Add prop to allow reversal of the order of toast items. For instance NEWEST first e.t.c

https://github.com/jossmac/react-toast-notifications/blob/469bd6ac4c29bed458191b193f465179139547ac/src/ToastProvider.js#L116

#173

See here #173


Flipper Isssue

Prerequisites

[X] I have searched the repository’s issues and Kinde community to ensure my issue isn’t a duplicate
[X] I have read the contributing guidelines
[X] I agree to the terms within the code of conduct

Describe the issue

Facing that issue Flipper after installing the latest 0.7 kinde react native starter on ios version
Starter kit URL
https://github.com/kinde-starter-kits/kinde-react-native-starter-kit-0-7x
Operating system(s)
iOS
Operating system version(s)
macOS Sonoma 14.5 Beta
Further environment details
react-native version - 0.76
node version - v21.7.3
Reproducible test case URL
No response
Additional information
No response

#include <__functional/function.h>
Add this line on top of FlipperTransportTypes.h class


Setup Private Link for FHIR Infrastructure on non-CDC subscription

Brian to setup another resource group for this.

Resource group is called "cdc-dex-fhir-private-rg-eastus"


Ast separation review

Looks fine, if it makes you happy. Still, I doubt anyone's using it directly.

By trying a snapshot version, I had a lot of compilation errors because of SourceMapper that is being used on any custom validation using AstNodeViolation .

I incorporated a couple of your changes in my branch before I realized that you're actually PR-ing to it, so you'll need to rebase.

I incorporated a couple of your changes in my branch before I realized that you're actually PR-ing to it, so you'll need to rebase.

You could aslo remove some commits in your branch, force-push, and then merge this one.

@performantdata I'll create another PR.

@performantdata I won't have time today. If you want, you can cherry-pick some commits, and just create your own with copy/pasta.


[WIP] API version update, fix file creation.

Related UI changes: https://github.com/groupdocs-total/GroupDocs.Total-Angular/pull/194.

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1
- Added 2

Clones removed
==============
+ src/Products/Editor/Controllers/EditorApiController.cs  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Clones removed
==============
+ src/Products/Editor/Controllers/EditorApiController.cs  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Clones removed
==============
+ src/Products/Editor/Controllers/EditorApiController.cs  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Clones removed
==============
+ src/Products/Editor/Controllers/EditorApiController.cs  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Clones removed
==============
+ src/Products/Editor/Controllers/EditorApiController.cs  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Clones removed
==============
+ src/Products/Editor/Controllers/EditorApiController.cs  -2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 1

Clones removed
==============
+ src/Products/Editor/Controllers/EditorApiController.cs  -2

See the complete overview on Codacy


UCP: Migrate scalar function Uncompress from TiDB

Description
Port the scalar function Uncompress from TiDB to coprocessor.
Score

50

Mentor(s)

@sticnarf

Recommended Skills

Rust programming

Learning Materials
Already implemented expressions ported from TiDB

https://github.com/tikv/tikv/tree/master/components/tidb_query/src/rpn_expr)
https://github.com/tikv/tikv/tree/master/components/tidb_query/src/expr)

/pick-up-challenge

/give-up-challenge

This issue is addressed as a whole in #7807


Direct write to backend texture from crate::host::RgbaFrameBuffer

Instead copying of buffer from RgbaFrameBuffer to SDL texture, it can be implemented such way that rustzx_core will directly write to the sdl texture instead. Required SDL context could be passed via AppHost (however, additional abstraction is required to separate host::RgbaFrameBuffer module from backend (sdl)

Actually, now I think that this is pretty questionable for now. Closing.


signVerify when legitimate signers, replaceRecentBlockhash when not signing to avoid AlreadyProcessed error

Two links that can explain a bit why I did it like that. I don't think it is perfect but at least for the general case, it should do.
People who want to simulate with full signature should just do a preflight in most cases.
https://github.com/solana-labs/solana-web3.js/blob/e89abb0969d138e83fabd18330f7b33b8d88573b/src/connection.ts#L3432-L3500
https://docs.solana.com/developing/clients/jsonrpc-api#simulatetransaction

Closing as stale.

fud


Errors

When running carbon tracker like this:
tracker = CarbonTracker(epochs=100)
correct = 0
wrong = 0
tracker.epoch_start()
for i in range(100):
true_label = id2label[val_labels[i].item()]
img = T.ToPILImage()(val_features[i].squeeze())
inputs = processor(images=img, return_tensors="pt").to(device)
pixel_values = inputs.pixel_values
outputs = model(pixel_values)
logits = outputs.logits
logits.shape
prediction = logits.argmax(-1)
prediction_str = model.config.id2label[prediction.item()]
display(img)
print("Predicted class:", prediction_str)
print("True class:", true_label)
if true_label == prediction_str:
correct += 1
print("correct")
else:
wrong += 1
print("wrong")
tracker.epoch_end()
tracker.stop()
On an Intel(R) Core(TM) i9-10900KF CPU @ 3.70GHz  with 16 GB RAM, NVIDIA GeForce GTX 980 Ti and Windows 10 in Jupyter Notebook, I get the following output from CarbonTracker:
CarbonTracker:
Actual consumption for 1 epoch(s):
Time:	0:00:05
Energy:	0.000152 kWh
CO2eq:	0.006405 g
This is equivalent to:
0.000060 km travelled by car
CarbonTracker: INFO - Traceback (most recent call last):
File "C:\Users\user\anaconda3\envs\gpu2\lib\site-packages\requests\models.py", line 971, in json
return complexjson.loads(self.text, **kwargs)
File "C:\Users\user\anaconda3\envs\gpu2\lib\json_init_.py", line 346, in loads
return _default_decoder.decode(s)
File "C:\Users\user\anaconda3\envs\gpu2\lib\json\decoder.py", line 337, in decode
obj, end = self.raw_decode(s, idx=_w(s, 0).end())
File "C:\Users\user\anaconda3\envs\gpu2\lib\json\decoder.py", line 355, in raw_decode
raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\user\anaconda3\envs\gpu2\lib\site-packages\carbontracker\emissions\intensity\intensity.py", line 117, in carbon_intensity
carbon_intensity = fetcher.carbon_intensity(g_location, time_dur)
File "C:\Users\user\anaconda3\envs\gpu2\lib\site-packages\carbontracker\emissions\intensity\fetchers\energidataservice.py", line 21, in carbon_intensity
ci = self._emission_prognosis(time_dur=time_dur)
File "C:\Users\user\anaconda3\envs\gpu2\lib\site-packages\carbontracker\emissions\intensity\fetchers\energidataservice.py", line 48, in _emission_prognosis
raise exceptions.CarbonIntensityFetcherError(response.json())
File "C:\Users\user\anaconda3\envs\gpu2\lib\site-packages\requests\models.py", line 975, in json
raise RequestsJSONDecodeError(e.msg, e.doc, e.pos)
requests.exceptions.JSONDecodeError: Expecting value: line 1 column 1 (char 0)
CarbonTracker:
Predicted consumption for 100 epoch(s):
Time:	0:08:26
Energy:	0.015250 kWh
CO2eq:	1.462028 g
This is equivalent to:
0.013600 km travelled by car
CarbonTracker: Finished monitoring.

Looks like the energiservice api which we use to query the carbon emissions changed their string format. We will release a bug fix. For now, if you change the query method in energidataservice.py in your local installation it should work. You can find this in your installation here C:\Users\user\anaconda3\envs\gpu2\lib\site-packages\carbontracker\emissions\intensity\intensity.py
    def _emission_prognosis(self, time_dur):
        from_str, to_str = self._interval(time_dur=time_dur)
        from_str = from_str.replace(' ','T')
        to_str = from_str.replace(' ','T')
        url = "https://api.energidataservice.dk/dataset/CO2Emis?start=" + from_str + "&end=" + to_str + "&limit=4"
        response = requests.get(url)
        if not response.ok:
            raise exceptions.CarbonIntensityFetcherError(response.json())
        data = response.json()["records"]
        carbon_intensities = [record["CO2Emission"] for record in data]
        return np.mean(carbon_intensities)

The above function is found in carbontracker/emissions/intensity/fetchers/energidataservice.py. The above path is incorrect.

This issue has been addressed in Release v1.2.0. We've phased out support for the energidataserviceapi and transitioned to the ElectricityMaps API. This change ensures better consistency and resolves the timeout issues previously encountered with consecutive requests. Thank you for bringing this to our attention.


[C++20] Failed to build with error C2039: 'unique': is not a member of 'std::shared_ptrtd::SqliteConnectionSafe'

Summary
Tdlib failed to build with error C2039: 'unique': is not a member of 'std::shared_ptr<td::SqliteConnectionSafe>' on /std:c++20 option.
This function was deprecated in C++17 and removed in C++20 because use_count is only an approximation in multithreaded environment.
https://en.cppreference.com/w/cpp/memory/shared_ptr/unique
https://learn.microsoft.com/en-us/cpp/standard-library/shared-ptr-class?view=msvc-170#unique
Additional Option
set _CL_=/std:c++20

Error Info
TdDb.cpp
td\telegram\TdDb.cpp(277,11): error C2039: 'unique': is not a member of 'std::shared_ptr<td::SqliteConnectionSafe>'
td/telegram/TdDb.h(119,8): message : see declaration of 'std::shared_ptr<td::SqliteConnectionSafe>'
td\telegram\TdDb.cpp(320,3): error C2039: 'unique': is not a member of 'std::shared_ptr<td::BinlogKeyValue<td::ConcurrentBinlog>>'
td/telegram/TdDb.h(180,8): message : see declaration of 'std::shared_ptr<td::BinlogKeyValue<td::ConcurrentBinlog>>'
td\telegram\TdDb.cpp(322,3): error C2039: 'unique': is not a member of 'std::shared_ptr<td::BinlogKeyValue<td::ConcurrentBinlog>>'
td/telegram/TdDb.h(180,8): message : see declaration of 'std::shared_ptr<td::BinlogKeyValue<td::ConcurrentBinlog>>'

Related Code
https://github.com/tdlib/td/blob/a7f8ff6eefbb4951d62b9e29bc4f7b95a28a46f0/td/telegram/TdDb.cpp#L277
https://github.com/tdlib/td/blob/a7f8ff6eefbb4951d62b9e29bc4f7b95a28a46f0/td/telegram/TdDb.cpp#L320
https://github.com/tdlib/td/blob/a7f8ff6eefbb4951d62b9e29bc4f7b95a28a46f0/td/telegram/TdDb.cpp#L322

TDLib has multiple incompatibilities with C++20 and currently is supposed to be built with a C++14 compiler as specified in CMakeLists.txt.

This function was deprecated in C++17 and removed in C++20 because use_count is only an approximation in multithreaded environment.

And our use case was perfectly legitimate, because we just check that only the current thread owns the shared pointer before completely destroying it. It was a sanity check, which is now harder to implement in C++20.

TDLib has multiple incompatibilities with C++20 and currently is supposed to be built with a C++14 compiler as specified in CMakeLists.txt.
...

Thanks for your reply and provide the information.


Add cookie support

#2 support

Thank you @recep. Can you move cookie test codes to another test function named TestSetCookie?


add garadener-metrics dashboard

Implement the task of Grafana Dashboard for Gardener Usage Exporter by adding new dashboard (see example )

/retest

/retest


update types rejection handled

fixes https://github.com/microsoft/TypeScript/issues/34511

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: vkrbt sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi, I noticed that all check were passed, could this code be merged? I need this fix in my code
Thanks a lot!

@orta?

👍 yep, looks like it's correct to the spec

I think it could be too late to get it into the 3.7 RC though


Doesn't mention tuple member identifier

Even if it's not supported (I do not possess this knowledge), it should be mentioned as not supported.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: b402d2e0-a53a-b61a-4da8-b5ee766d9301
Version Independent ID: 612695df-5934-d967-d709-500ce02b1395
Content: .NET Naming Conventions For EditorConfig files - Visual Studio
Content Source: docs/ide/editorconfig-naming-conventions.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @gewarren
Microsoft Alias: gewarren

@mikadumont or @sharwell Can you confirm if naming conventions can target tuple member names?

It's not currently supported. The request for it is https://github.com/dotnet/roslyn/issues/38705.

@Logerfo I went ahead and made that update!


Ensuring correct file names on Digi app exports

During post processing of digi app exports, a file was discovered with the name 'NHMD_Herba_20231117_15_30_RL_original.csv', but containing pinned insect data.
We want to ensure the correct file naming based on the collection, in this example:
'NHMD_PinnedInsects_20231117_15_30_RL_original.csv'
Suggestions to avoid this error in the future:

Create subfolders in 0.ForChecking, named "PinnedInsects", "Herbarium" and any future collections.
Create a "dummy" file, to copy the name from, i.e. "NHMD_PinnedInsects_YYYYMMDD_HH_MM_XYZ_original.csv" and "NHMD_Herba_YYYYMMDD_HH_MM_XYZ_original.csv"
Make an extra effort to check that the file name is correct, when moving files from 0ForChecking to 1.Exported files from App.

Possible issues:
We must make sure this change in structure folder does not inhibit Bhupjits quest to automate the post processing.

@AstridBVW DOes the new folder structure help with the above issue? Is there anything specific we need to communicate with digitisers to help prevent this or any suggestions?

@PipBrewer Yes, the new folder structure should solve this issue. I will make sure that all digitisers are informed when the new folder structure is fully implemented and how to use the new structure to ensure correct naming of export files.


Unable to connect on PostgreSQL via pgbouncer.

Hi,
I'm trying to connect commafeed on postgresql via pgbouncer but I'm getting this error in pgbouncer

2014-03-10 21:13:59.385 5773 WARNING C-0x1be8bb0: (nodb)/(nouser)@<IP_ADDRESS>:38733 unsupported startup parameter: extra_float_digits=2
2014-03-10 21:13:59.385 5773 LOG C-0x1be8bb0: (nodb)/(nouser)@<IP_ADDRESS>:38733 closing because: Unsupported startup parameter: extra_float_digits (age=0)
2014-03-10 21:13:59.385 5773 WARNING C-0x1be8bb0: (nodb)/(nouser)@<IP_ADDRESS>:38733 Pooler Error: Unsupported startup parameter: extra_float_digits

Commafeeds logs show this :
https://gist.github.com/moumoul/9473401
My database configuration file is like this:
        <Resource id="PostgreSQL" type="DataSource">
                JdbcDriver org.postgresql.Driver
        JdbcUrl jdbc:postgresql://localhost:6432/commafeed
                UserName xxxxxxxxxxxxx
                Password xxxxxxxxxxxx
                MaxActive 5
        </Resource>

This configuration file is working fine when I'm directly connect to PostgreSQL.

So to save the next guy from clicking the link above, the "fix" is to add this to the config.ini under the [pgbouncer] section:
ignore_startup_parameters = extra_float_digits

An explantion why this happens and what it does and why it is not the default would be nice to get a better understanding :)

To save the next next guy some more clicks:
http://lists.pgfoundry.org/pipermail/pgbouncer-general/2008-November/000307.html

There are few parameters that pgbouncer can keep track of.
Like client_encoding.  Because postgres reports back chenges
of that parameter.  The 'extra_float_digits' is not one of them.
Thus, in transaction pooling mode, there is no way to give
consistent picture of that parameter to clients.
Giving hard-error instead of ignoring by default is delierate,
so admin can decide if the parameter is actually important
for application logic.  If it is then you should make
if default from postgres config.

So to save the next guy from clicking the link above, the "fix" is to add this to the config.ini under the [pgbouncer] section:
ignore_startup_parameters = extra_float_digits

I'm getting the below error even after adding the below line under [pgbouncer] in .ini file.
ignore_startup_parameters = extra_float_digits

unsupported startup parameter: options=-c extra_float_digits=2
closing because: unsupported startup parameter: options (age=1s)
pooler error: unsupported startup parameter: options

Can someone please advise?

@Preetham-Srinivasan You probably had ignore_startup_parameters assigned twice. Search the file and ensure it is only assigned once.

@Preetham-Srinivasan I believe you want ignore_startup_parameters = options in your case


Extension readme in product does not support markdown reference links

Markdown:
| Project | License | Reason
|---------|---------|--------
| [pako]  | [MIT][pako_MIT] and [Zlib][pako_Zlib] | Used to decompress png datastreams

[pako]: https://www.npmjs.com/package/pako
[pako_MIT]: https://github.com/nodeca/pako/blob/master/LICENSE
[pako_Zlib]: https://github.com/nodeca/pako/blob/master/lib/zlib/README

Editor:

In extension view:

I can obviously workaround this but it's pretty ugly:

I'm not sure if the marketplace supports this or not as I didn't publish it.

Looks like a potential bug with marked:  demo

Looks like this is being tracked by https://github.com/markedjs/marked/issues/2217
Will keep this open to track picking up new marked version


Add last sync status metric

shouldn't be 0 -> ok, 1 -> not ok?

Actually, with Boolean metrics values are 1 for ok (true), 0 for not ok (false).
This is useful, since 0 computes to false and 1 to true and it works well with the bool modifier.
a good example is the builtin up metric, value (1) means UP, value (0) means DOWN.

constant instead of the values?


Fixtures.zip

While importing the tests into the geth repository I noticed that the release only provides the linux version not a windows version for the fixtures.
Additionally it would be great to modify the naming of the fixtures to include the version number.


Ability to choose model from PowerToys run

Thanks for this great plugin! It would be great if it was possible to choose the model from the command line, with something like this:
llm > llama3 > my llm prompt

Hi thanks for the feedback!! I really appreciate it.
can you try it and give feedback. I implemented the ability to chose the model. The initialisation will take some time so you might initially get a timeout but subsequent queries will work perfectly. I am currently loading all the models from Ollama Library and parsing the webpage to generate the list of models supported by Ollama. I was unable to find a faster option.
LocalLLM-v1.0.2.zip

Wow, didn't expect such rapid action! It seems to work:

Could you specify the model in use next to the "LLM Response" header?

Hi I was actually thinking of implementing what you were talking about some time ago but tbh had completely forgotten about this. So when I got this issue I actually had the exact changes mapped out which weren't much. You raising this issue helped a lot because I came back to this project. Anyways here is the updated version Try it out and I will release it in a day or 2 after doing some testing.
LocalLLM-v1.0.3.zip

This is looking good!

Further improvements for consideration:

Could you autosuggest the engines already installed in Ollama once "llm > " has been input?
Can you make the response pane bigger to accommodate longer responses, and make sure there's text wrapping?
Is there a way (spinning hourglass or similar) you could indicate that the prompt has been sent and being processed by Ollama?

I am currently loading all the models from Ollama Library and parsing the webpage to generate the list of models supported by Ollama. I was unable to find a faster option

Have you looked into the list_models endpoint?
https://hexdocs.pm/ollama/0.3.0/Ollama.API.html#list_models/1

This is looking good!
Further improvements for consideration:

Could you autosuggest the engines already installed in Ollama once "llm > " has been input?
Can you make the response pane bigger to accommodate longer responses, and make sure there's text wrapping?
Is there a way (spinning hourglass or similar) you could indicate that the prompt has been sent and being processed by Ollama?

Hi if it’s possible could you open up a separate issue with your suggestions so I would be able to look at them  and open it up for others to possibly work on. I was planning on the auto suggestions feature next

I am currently loading all the models from Ollama Library and parsing the webpage to generate the list of models supported by Ollama. I was unable to find a faster option

Have you looked into the list_models endpoint? https://github.com/ollama/ollama/blob/main/docs%2Fapi.md#list-local-models
You can know which models are already running in memory from this: https://github.com/ollama/ollama/blob/main/docs%2Fapi.md#list-running-models

I did see them but it doesn’t give the global models just local models installed in your local ollama. I thought global model support would be good but I might change it for local model support if u believe it would help.

This is looking good!
Further improvements for consideration:

Could you autosuggest the engines already installed in Ollama once "llm > " has been input?
Can you make the response pane bigger to accommodate longer responses, and make sure there's text wrapping?
Is there a way (spinning hourglass or similar) you could indicate that the prompt has been sent and being processed by Ollama?

Hi if it’s possible could you open up a separate issue with your suggestions so I would be able to look at them and open it up for others to possibly work on. I was planning on the auto suggestions feature next

Done: https://github.com/Darkdriller/PowerToys-Run-LocalLLm/issues/4 https://github.com/Darkdriller/PowerToys-Run-LocalLLm/issues/5 https://github.com/Darkdriller/PowerToys-Run-LocalLLm/issues/6

This is looking good!
Further improvements for consideration:

Could you autosuggest the engines already installed in Ollama once "llm > " has been input?
Can you make the response pane bigger to accommodate longer responses, and make sure there's text wrapping?
Is there a way (spinning hourglass or similar) you could indicate that the prompt has been sent and being processed by Ollama?

Hi if it’s possible could you open up a separate issue with your suggestions so I would be able to look at them and open it up for others to possibly work on. I was planning on the auto suggestions feature next

Done: https://github.com/Darkdriller/PowerToys-Run-LocalLLm/issues/4 https://github.com/Darkdriller/PowerToys-Run-LocalLLm/issues/6 https://github.com/Darkdriller/PowerToys-Run-LocalLLm/issues/6

Hi if it’s possible could you open up a separate issue with your suggestions so I would be able to look at them and open it up for others to possibly work on. I was planning on the auto suggestions feature next

Done: #4 #5 #6

Thanks for the feedback I will try to add this functionality soon.


Create a contact info block

Add an intro about yourself: who are you, what is your strength, why are you here.


Explicitly ignore data-* attributes without warning

When parsing SVGs the parser will flag data-*-attributes as unknown properties.
e.g. an SVG like this:
<svg data-name="Ebene 1" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 633.1 221"></svg>

will throw this warning:
    Ignoring data-name attribute(s) in svg

However, the data-*-attributes are known non-UI related attributes and there has no effect, throwing an error will result in noise in the warnings.
This issue can be considered as a request to support data-*-attributes in the sense of simply ignoring them similar to how its done for xmlns

Published as 1.1.9


Make GOlr loading steps in AmiGO's gulpfile.js more robust to upstream ontology outages

Occasionally, during GOlr loads on Jenkins, there will be periods of time where the upstream ontologies timeout or otherwise fail due to somewhat mysterious circumstances. Manually probing after such fails almost always seems to indicate that things are fine. We are unsure of the exact reason, or if there is only a single cause for these various fails.
To better protect ourselves, and to give better speed (always nice) we will look at creating an ontology catalog on the fly that we can check and use. Ideally, it looks something like:
## TODO: Purge previous catalog generation runs
## TODO: Generate list of ontology files
## Using the above step, loop through (or similar):
owltools AN_ONTOLOGYFILE.owl  --slurp-import-closure -d mirror -c mycat.xml
## TODO: Then run check and load commands, using the specified catalog, something like:
owltools --catalog-xml mycat.xml THE_REST_OF_THE_COMMANDS_1
owltools --catalog-xml mycat.xml THE_REST_OF_THE_COMMANDS_2

As we are still loading on Jenkins, this should be rolled out to the 2.4.x and master branches, and then triggered in the Jenkins jobs.
Tagging @cmungall
We've noticed this periodically, but the immediate urgency here if from https://github.com/geneontology/noctua/issues/511

Note that this is a temporary fix until the new pipeline and loads are running off of skyhook catalogs.


Training Modules

Add training modules:

Multiple "levels" of curated assignments
Include standard editing, splitting merging, claiming, etc.
Need gold standard masks, can compute dice score. Don't advance until meet dice score threshold
Earn badges for each level
When finished, eligible to do "real" work.

@hyi @talafar Tyring to capture our meeting here:

Single training volume
Only keep ids we want for specific tasks
Tala supplies metadata per id to Hong: id, task type, description, difficulty, etc.
Hong supplies assignments in order as users complete/request new

First version of this implemented


[i2c,rtl] Compress FMT FIFO entries

When moving all i2c FIFOs into a single SRAM, which is one option being explored for https://github.com/lowRISC/opentitan/issues/22159, the FMT FIFO is problematic as it has more bits per entry (13) than all the other FIFOs (at most 10). The current entry encoding, where each flag has its own bit

Bits
Type
Reset
Name
Description

12
wo
0x0
NAKOK
Do not signal an exception if the current byte is not ACK'd

11
wo
0x0
RCONT
Do not NACK the last byte read, let the read operation continue

10
wo
0x0
READB
Read BYTE bytes from I2C. (256 if BYTE==0)

9
wo
0x0
STOP
Issue a STOP condition after this operation

8
wo
0x0
START
Issue a START condition before transmitting BYTE.

7:0
wo
0x0
FBYTE
Format Byte. Directly transmitted if no flags are set.

might be a waste of space. (I think the CSR format is fine and doesn't have to change, but the encoding for storage in the FIFO can be compacted.)
Could a more compact encoding, such as the following, work?

wdata[9:8]
wdata[7:0]
effect

2'b00
fbyte
write fbyte

2'b01
fbyte
write fbyte w/o exception if not ACK'd

2'b10
fbyte
read fbyte bytes

2'b11
{8'h00, 8'h01, 8'h02}
depending on wdata[7:0], generate a Start (8'h00) or Stop (8'h01) condition or continue reading after having received all bytes (8'h02, current RCONT flag)

(If compressing FMT FIFO entries to 10 bit is impossible, we could still use a 10-bit-wide deep SRAM plus 3x64 FFs 'on the side' to complement the FMT data.)

@marnovandermaas @hcallahan-lowrisc @a-will WDYT?

(I think the CSR format is fine and doesn't have to change, but the encoding for storage in the FIFO can be compacted.)

I'll have to disagree there: We break the ABI if Start / Stop symbols require a separate entry in the FMT FIFO, and that will necessitate a CSR format change. Currently, the control symbols are part of the same entry as data, so a write to the CSR always consumes 1 item of the FIFO.

Could a more compact encoding, such as the following, work?

wdata[9:8]
wdata[7:0]
effect

2'b00
fbyte
write fbyte

2'b01
fbyte
write fbyte w/o exception if not ACK'd

2'b10
fbyte
read fbyte bytes

2'b11
{8'h00, 8'h01, 8'h02}
depending on wdata[7:0], generate a Start (8'h00) or Stop (8'h01) condition or continue reading after having received all bytes (8'h02, current RCONT flag)

I think an encoding close to that could work, though I can't say I understand why we even have the NAKOK bit. INTR_ENABLE already handles whether interrupts are generated, and otherwise, the spec is quite clear about what a NACK means. If we receive a NACK, the transfer must end.
In any case, if we leave NAKOK alone, I'd at least invert the handling of RCONT:

wdata[9:8]
wdata[7:0]
effect

2'b00
fbyte
write fbyte

2'b01
fbyte
write fbyte w/o exception if not ACK'd

2'b10
fbyte
read fbyte bytes and ACK all bytes (RCONT = 1)

2'b11
{8'h00, 8'h01}
depending on wdata[7:0], generate a Start (8'h00) or Stop (8'h01) condition, and if a read is ongoing, read and NACK one last byte prior to the transfer-ending Start or Stop

We could split the read transfer-ending values into two separate values (8'h02, 8'h03) from the write transfer-ending values, if desired.

Thanks for your feedback. Let's hold this for now: With #22216 the SRAM that now backs all FIFOs is 13 bit wide. If we don't need to further optimize the area, we'll close this issue (or at least defer it to a future revision).

I think the i2csvg examples are mostly well behaved, but things like the "NACKOK" are needed to manage some of the many random devices (sensors etc) that in practice do strange things (like always NACK the last byte they accept).
The I2C spec is one that has many imitations because of history "A two wire protocol that..." and many of them do not actually match the spec or invent their own protocols above the byte-level spec. SMBus cleaned things up a lot (but for many years had the max 100kHz limitation) but there are still lots of not-quite-spec devices.
You should look hard at a lot of example devices before compressing the flags.

This isn't planned for now --> potential future release and backlog


Payment method paypal is not working for android

When i select payment method PayPal that is not working for android where payment method credit or debit card is working.

Do you have PayPal enabled in you Braintree account?

Thanks @Deeeej , that seems right as I came with the same solution in iOS.
Apparently yo need to add a new URL type in XCode for allowing the app navigating to Paypal's.
https://developers.braintreepayments.com/guides/paypal/client-side/ios/v4
Hope this helps as well!

@Amarjit1234 Just found this:

important

Your app's URL scheme must begin with your app's package ID and end with .braintree. For example, if the Package ID is com.your-company.your-app, then your URL scheme should be com.your-company.your-app.braintree. ${applicationId} is automatically applied with your app's package when using Gradle.
Note: The scheme you define must use all lowercase letters. If your package contains underscores, the underscores should be removed when specifying the scheme in your Android Manifest.

In my case, this The scheme you define must use all lowercase letters was causing the issue in Android. So, make sure yor APP_ID is all lowercase.
Hope this helps.


"Plus" button covers the last row to edit

It can be hard to edit the ssh list in some cases. See the last one, living room, in my list which is covered by the “+” button.

Originally posted by @mbierman in https://github.com/isontheline/pro.webssh.net/issues/272#issuecomment-890545795

Hello @mbierman 👋
Enhancement available inside WebSSH 15.1 - Cactus (Opuntia)

Nice! 🙏🏻


if IE LT 8 / if IE LT 9 Bug in ie(all)

Using the Merced theme as a base with Mura 5.6.4814.
In IE (all versions) keep getting:
 (DISPLAYED AT TOP OF PAGE) &
 (DISPLAYED AT THE BOTTOM)
According to Mura forum user 'inge'
Refer to...
http://muracms.com/forum/messages.cfm?threadid=468BCDF4-B90E-4F9C-B1B02316D155B101
this is due to a syntax error - code should read <!--[if lt IE 8]>. Tested this out using a static version of the page and does fix the issue.
Tracked it down to:
cffunction name="$renderIeConditional" access="private" returntype="string" output="false" hint="I wrap an html include string with IE conditional statements when necessary, returning the result"
in requirements\org\cfstatic\util\Base.cfc
Thanks

This is a documentation issue for CfStatic and a code issue for the Mura theme. The CfStatic wiki documentation was giving bad examples, have fixed that.
Code fix should be made in the IE only mura theme files (css and js) so that they read @ie LT IE 8 and @ie LT IE 9

Thanks for the response...
Not really a developer so am a bit cloudy on what i need to change and where.
Can you be more specific re the file I need to amend.
Many thanks...

I've not had a chance to look at Mura before so I've no idea what files will need amending. This issue should be taken up by either the mura author, or the author of the theme that you are using. Failing that, you could do a global search for @ie among css and js files to identify the areas that need fixing.

Fantastic!
Now sorted...
Had to change:

@ie IE LT 8  - to -  * @ie LT IE 8 in the ie.css file (Your Theme Path\css\ie\lte7\ie.css) - and
@ie IE LT 9  - to - * @ie LT IE 9 in the html5.js file (Your Theme Path\js\ie\lte8\html5.js)

Thanks again for your help and advice!

Excellent, glad to be of help. Thanks for raising the issue here too.


[BUG] Kafka config bootstrap_server to be changed to bootstrap_servers config/autoload/kafka.php

Description:
Kafka config bootstrap_servers is being used inside the code but in the documentation just bootstrap_server is given.
For Example:
class: Hyperf\Kafka\Producer
$producerConfig->setBootstrapServers($config['bootstrap_servers']);

@limingxinleo could you close this issue?


Terminal suggest: Environment variable completion doesn't work good when the variable is the first part

Repro:

Windows/pwsh
Suggest enabled
Open pwsh
Type $env, 🐛  suggestions don't show
Press ctrl+space, 🐛  suggestions still don't show

I still get the bug in step 4.
For step 5, I'm unsure whether I'm seeing the right suggestions:


lock files never need to be corrupted

The error message here suggests that "kombustion.lock may need to be corrupted", which sounds incorrect.

Pull Request Test Coverage Report for Build 449

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.313%

Totals

Change from base Build 442:
0.0%

Covered Lines:
661

Relevant Lines:
1600

💛  - Coveralls

Pull Request Test Coverage Report for Build 447

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.313%

Totals

Change from base Build 442:
0.0%

Covered Lines:
661

Relevant Lines:
1600

💛  - Coveralls


python310Packages.plexapi: 4.11.2 -> 4.12.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.plexapi is: "Python bindings for the Plex API"
meta.homepage for python310Packages.plexapi is: "https://github.com/pkkid/python-plexapi"
meta.changelog for python310Packages.plexapi is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 4.12.0 with grep in /nix/store/i64192ljqj4ykx994ig21xhasfsdwmkc-python3.10-plexapi-4.12.0
found 4.12.0 in filename of file in /nix/store/i64192ljqj4ykx994ig21xhasfsdwmkc-python3.10-plexapi-4.12.0

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

8 total rebuild path(s)

6 package rebuild(s)

6 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
home-assistant-component-tests.cast
home-assistant-component-tests.plex
home-assistant-component-tests.sonos
python310Packages.plexapi
python39Packages.plexapi
tauon

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/i64192ljqj4ykx994ig21xhasfsdwmkc-python3.10-plexapi-4.12.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.plexapi https://github.com/r-ryantm/nixpkgs/archive/63c871907375170d97e686d83b4360374231a374.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/i64192ljqj4ykx994ig21xhasfsdwmkc-python3.10-plexapi-4.12.0
ls -la /nix/store/i64192ljqj4ykx994ig21xhasfsdwmkc-python3.10-plexapi-4.12.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

6 packages built:

home-assistant-component-tests.cast
home-assistant-component-tests.plex
home-assistant-component-tests.sonos
python310Packages.plexapi
python39Packages.plexapi
tauon

Maintainer pings
cc @colemickens for testing.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/why-is-the-r-ryantm-bot-using-refs-tags-in-the-version/20519/1


ci: Working on ci/cd

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


chore(release): publish branch main

:robot: I have created a release beep boop
@cpro-js/vite-ui5-common-config: 0.1.3
0.1.3 (2024-04-16)
Features

common-config: new package for common configuration (0d3aa8a)
readme (24f2542)

Dependencies

The following workspace dependencies were updated

dependencies

@cpro-js/vite-ui5-integration-plugin bumped from 0.1.2 to 0.1.3
@cpro-js/vite-ui5-mockserver-plugin bumped from 0.1.2 to 0.1.3
@cpro-js/vite-ui5-resources-proxy-plugin bumped from 0.1.2 to 0.1.3

@cpro-js/vite-ui5-integration-plugin: 0.1.3
0.1.3 (2024-04-16)
Miscellaneous Chores

@cpro-js/vite-ui5-integration-plugin: Synchronize Vite versions

@cpro-js/vite-ui5-mockserver-plugin: 0.1.3
0.1.3 (2024-04-16)
Miscellaneous Chores

@cpro-js/vite-ui5-mockserver-plugin: Synchronize Vite versions

@cpro-js/vite-ui5-resources-proxy-plugin: 0.1.3
0.1.3 (2024-04-16)
Miscellaneous Chores

@cpro-js/vite-ui5-resources-proxy-plugin: Synchronize Vite versions

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/cpro-js/ui5-vite/releases/tag/%40cpro-js/vite-ui5-common-config-v0.1.3 :sunflower:

:robot: Release is at https://github.com/cpro-js/ui5-vite/releases/tag/%40cpro-js/vite-ui5-integration-plugin-v0.1.3 :sunflower:

:robot: Release is at https://github.com/cpro-js/ui5-vite/releases/tag/%40cpro-js/vite-ui5-mockserver-plugin-v0.1.3 :sunflower:


Drop Unique Index throwing error when not using Cascade

Describe the problem
Our docs say that when you create a unique Index and you wish to drop it, you can do so without using CASCADE  as of 19.2 (https://www.cockroachlabs.com/docs/v19.2/drop-index.html).
However if you create a table with a unique index you are not able to drop it unless you use cascade:
root@localhost:26261/defaultdb> CREATE TABLE logon (
    login_id  INT PRIMARY KEY,
    customer_id   INT,
    logon_date    TIMESTAMP,
    UNIQUE index (customer_id, logon_date)
  );
CREATE TABLE

Time: 8.262ms

root@localhost:26261/defaultdb> show indexes from logon;
  table_name |            index_name            | non_unique | seq_in_index | column_name | direction | storing | implicit
+------------+----------------------------------+------------+--------------+-------------+-----------+---------+----------+
  logon      | primary                          |   false    |            1 | login_id    | ASC       |  false  |  false
  logon      | logon_customer_id_logon_date_key |   false    |            1 | customer_id | ASC       |  false  |  false
  logon      | logon_customer_id_logon_date_key |   false    |            2 | logon_date  | ASC       |  false  |  false
  logon      | logon_customer_id_logon_date_key |   false    |            3 | login_id    | ASC       |  false  |   true
(4 rows)

Time: 3.335ms

root@localhost:26261/defaultdb> drop index logon_customer_id_logon_date_key;
pq: index "logon_customer_id_logon_date_key" is in use as unique constraint (use CASCADE if you really want to drop it)

Is this a bug or expected behaviour?
Environment:

CockroachDB version v19.2.2

Thanks, @roncrdb. I've transferred the issue to the docs repo.


Chorth

I'm trying to get Sporth in ChucK working but I'm stuck at:
[test.ck]:line(2): undefined type 'Sporth'...
[test.ck]:line(2): ... in declaration ...
I am using chuck version <IP_ADDRESS> and the latest Sporth/Chorch source all on linux.

@ziqbal  Ah. It looks like I was manually copying chuck header files, and now they are too stale. I have updated those files, and some portions of Chorth as well. Things compile and run now on my OSX device, but I have yet to test it on Linux. Can you check and see if things work?

@PaulBatchelor Update works! Thank you for the fast response and fix.


Implement source namespace matching

It would be useful to be able to route on source namespace additional to source labels. The implementation is essentially identical to for source labels.

@jackkleeman We need some test cases

@jackkleeman: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
b7c224c2c53b809433d942439d0122a9ae54bf51
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Non-Root workaround for broken stdlibs

Ok. This is going to be painful (for others to watch), but:
If we can't fix the system Python - and I don't want to compile my own - then here's what I'm going to try:

download the original cPython source packages from github, version specific to whatever python version we are running, extract it in /tmp
copy lib/venv, lib/... to ~/.appenv/py/lib
include that in the pythonpath

Hopefully any venvs we create will properly inherit this ... I'll try this out.

Ok, including it in the pythonpath isn't sufficient. So it turns out that I will

create venv without pip
download original python source
copy distutils and pipenv over

We might want to cache this in the future and maybe even have a workaround if venv should be missing. But anyway. This allows non-root bootstrapping with a missing distutils/ensurepip for now.


Integrated Marshal for client send. Fixed some style.

As soon as @huyang1988 fixes a couple things, demarshal_client should be up. You can see the code now at PR #47

Tests failed due to missing make target
cd client && make test
make[1]: Entering directory `/home/travis/build/gwAdvNet2015/gw-kv-store/client'
make[1]: *** No rule to make target `test'.  Stop.
make[1]: Leaving directory `/home/travis/build/gwAdvNet2015/gw-kv-store/client'
make: *** [test_client] Error 2

This is something you can fix in conjunction with https://github.com/gwAdvNet2015/gw-kv-store/issues/50#issuecomment-94621946

@sacul29 @kstats @jdk514 Has any more progress been made on this? @phil-lopreiato recently merged #55 which should have completely resolved your issues.

Moved to #58


Fix SQLAlchemyMd5Cache race condition

If the SQLAlchemyMd5Cache is shared among multiple processes, it is possible to encounter a race condition during the cache update.

we can merge as soon as tests pass

thanks @pprados


Popup has no tabId so can't message between it and background

Sending a message from popup to background doesn't seem to work for me.
I am using this package in a Vue project.
//popup.ts

import { createApp } from 'vue'
import App from './Popup.vue'
import '../styles'
import { sendMessage } from 'webext-bridge/popup'

(async () => {
    const resBG = await sendMessage('test-message', { text: "Popup" }, 'background')
    console.log(`[popup] Got ${ resBG }`);

    const resCS = await sendMessage('test-message', { text: "Popup" }, 'content-script')
    console.log(`[popup] Got ${ resCS }`);
})()

const app = createApp(App)
app.mount('#app')

Background receives this message but throws an error:
        Uncaught (in promise) TypeError: When sending messages from background page, use @tabId syntax to target specific tab
Background looks like this:
import { onMessage } from 'webext-bridge/background'
onMessage('test-message', async (message) => {
  const { data, sender } = message
  console.log(`[background] Got test message from ${ sender.context }@${ sender.tabId }. Data: ${ JSON.stringify(data) } `);
  return `Hello`

})

that console message shows:
[background] Got test message from popup@NaN. Data: [object Object]
Content script does not receive anything from the popup.
Is it possible to communicate with the popup in this way or have I got it wrong?
Thanks

Closing this as I realise it is basically the same as #37


Provide timestamp of next connection attempt

I'd like to inform the user that there's a connectivity problem, and when the next attempt will be to restore the connection. To do that, I'd have to know when the websocket will attempt to reconnect (e.g. through a timestamp property on the ReconnectingWebSocket object that tells when the next connection attempt will be).

Interesting. Maybe a better idea would be to accept via constructor a function getReconnectingDelay(retryNumber), so you can have full control of that.
Would that fit your needs?


Auto updates never apply on macOS

Issue Type: Bug
I've had repeated difficulty getting VSCode updates to apply through the in-editor update system. For the last few updates, I've had to resort to manually downloading and updating the new version.
Steps:

When a new update is available (as it is currently, since I'm on 1.46.0), open Code and use "Check for updates..." from the menubar.
Update gets downloaded and the "Restart Visual Studio Code to apply the latest update." notification pops up in the lower right corner.
Click "Update Now".
VSCode exits but does not restart.
Manually restart VSCode, and note that it is still running the same version as before (1.46.0 currently).

These steps are 100% repeatable for me. There aren't any errors presented at any step in this process, the update just never happens. I don't see any errors like "Cannot update while running on a read-only volume. The application is on a read-only volume." that others have reporter on similar auto-updater issues. I've also tried the "xattr -dr com.apple.quarantine ~/Library/Caches/com.microsoft.VSCode.ShipIt* /Applications/Visual\ Studio\ Code.app/" workaround mentioned in other similar issues with no effect. This is on macOS 10.15.5.
VS Code version: Code 1.46.0 (a5d1cc28bb5da32ec67e86cc50f84c67cc690321, 2020-06-10T08:59:06.977Z)
OS version: Darwin x64 19.5.0
Remote OS version: Linux x64 5.2.17-1rodete3-amd64

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz (8 x 2800)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
4, 3, 3

Memory (System)
16.00GB (0.50GB free)

Process Argv

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: desk

OS
Linux x64 5.2.17-1rodete3-amd64

CPUs
Intel(R) Xeon(R) CPU E5-2690 v4 @ 2.60GHz (56 x 1197)

Memory (System)
125.89GB (12.18GB free)

VM
0%

Extensions (20)

Extension
Author (truncated)
Version

theme-monokai-pro-vscode
mon
1.1.17

vscode-json5
mrm
1.0.0

remote-containers
ms-
0.122.1

remote-ssh
ms-
0.51.0

remote-ssh-edit
ms-
0.51.0

remote-wsl
ms-
0.44.4

vscode-remote-extensionpack
ms-
0.20.0

vscode-gn
npc
1.1.0

vim
vsc
1.14.5

material-theme
zhu
3.8.2

githistory
don
0.6.5

nasc-touchbar
fel
1.12.0

language-fidl
fuc
0.3.1

Go
gol
0.14.4

vscode-clangd
llv
0.1.6

rust-analyzer
mat
0.2.216

git-graph
mhu
1.24.0

rewrap
stk
1.12.0

gitblame
wad
4.0.1

clang-format
xav
1.9.0

(2 theme extensions excluded)

Logs with both of those options attached.
code-update-verbose.log

Thanks for the log, seems like the download was fine.
2020-06-25 09:12:11.808 Electron[50053:5893268] Download completed to: file:///Users/bryanhenry/Library/Caches/com.microsoft.VSCode.ShipIt/update.Ja0xMA8/VSCode-darwin-stable.zip
[main 2020-06-25T16:12:13.816Z] update#setState ready

Can you attach the ShipIt_stderr.log, ShipIt_stdout.log under Library/Caches/com.microsoft.VSCode.ShipIt/

ShipIt_stdout.log is empty so it won't attach. ShipIt_stderr.log only seems to contain old logs from last year.
ShipIt_stderr.log

On a whim I moved the old ShipIt_stdout.log and ShipIt_stderr.log out of that directory, because I noticed they were owned by root rather than my user and file permissions were set to 0644 (user writable only). After a few seconds VSCode opened with the update applied with no other action from me; I guess it was trying repeatedly in the background to apply the update?).
So the log files not being user-writable were preventing the update from occurring? I'm not sure how the owner of those files got changed...I certainly don't think that's something I did.

Oh nice!

After a few seconds VSCode opened with the update applied with no other action from me; I guess it was trying repeatedly in the background to apply the update?

The updating task is handled by https://github.com/Squirrel/Squirrel.Mac framework, and once an update is downloaded it will try to apply it whenever the application quits. So yeah it must have been re-trying it.

So the log files not being user-writable were preventing the update from occurring?

This is new, I would expect squirrel to throw some sort of error for this on the handlers we listen to. I will try to setup a minimal example and confirm the issue.

I'm not sure how the owner of those files got changed

Maybe shipit was owned by root previously ?

Should have better error signals for this case of failure once https://github.com/electron/electron/pull/26358 lands in upstream.

I am also facing the issue on Mac. I got this alert while while click on Check for Updates -

I have downloaded this install - https://code.visualstudio.com/docs/?dv=darwinarm64 for MacBook Pro M1 Max.
While downloading, it's showing .zip but after downloaded, showing as Visual Studio Code.app
When I click on this file, it's auto started the VS code, not showing any options to install it using drag&drop in Application folder.

Closing as upstream issue has been resolved.


Append promise reaction values in a gc safe manner

Fixes #4096.

Can't really add a test case for this as it depends on garbage collection.


Tabs not showing properly.

As you can see here: http://prntscr.com/64m3ci
Doesn't properly display the tabs and there is always a red bar over it, even I open more it will display said bar over all tabs at the same time.
Cheers.

Thanks for the heads up @DarthSuu !
Get the latest update and the tabs should look fine :+1:

I just installed it and got that bug.

Yeah, the Tabs extension was updated 16 hours ago which broke it in the theme. I just made the update on my end so it should be fixed when you update or re-install the theme.

I installed the theme a few minutes before your extension.

Are you still getting this issue?
When you look at your installed extensions, do you see an update next to the Stripper theme?

Sweet, the update warning just came up. Thanks! All fine and dandy now.


Line-Height by Percentage

Hey Zell.
I need a "negative" line-height. I tried using a percentage within the map:
  null: (ms(0), 80%),
  small: (ms(0), 80%),
  medium: (ms(0), 80%),
  large: (ms(0), 80%)

That doesn't seem to work.
Also tried using a fraction in a vr().
.my_DOM_el  {
  line-height: vr(0.5);
}

I am probably missing something. Can you help?

line-height is programmed as a unitless value in Typi. Try this:
null: (ms(0), -0.5),

Perfect, my man. TY.


Add exclude from cascade in TransientSessionTicket

The JpaTicketRegistry tries to traverse ticketGrantingTicket attribute using HQL. TransientSessionTicket has always ticketGrantingTicket constant, and is implemented as a getter returning null and therefore does not have HQL equivalent. It could be changed in JpaTicketRegistryTicketCatalogConfiguration as the bug seems to be just in JPA registry but the core package seems more adecuate.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Where could I add tests?

Thank you for the pull request. Make sure the cla is signed too.
I am a bit confused; what problem are you trying to solve? You are proving a description for a state of affairs but I am not sure where the problem lies. Are you getting an error somewhere?
Please start with a test case likely in the JPA ticket registry module that demonstrates a failure or a symptom and work your way up.

Hi mmoayyed,
Given there are logged in users, it happens whenever one of them tries to log out or whenever their tickets expire.
The exception is this one:
2020-06-28 10:23:28,255 ERROR [org.apache.catalina.core.ContainerBase.[Tomcat].[localhost].[/cas].[dispatcherServlet]] - <Servlet.service() for servlet [dispatcherServlet] in context with path [/cas] threw exception [Request processing failed; nested exception is org.springframework.webflow.execution.ActionExecutionException: Exception thrown executing org.apereo.cas.web.flow.logout.TerminateSessionAction@782e0844 in state 'terminateSession' of flow 'logout' -- action execution attributes were 'map[[empty]]'] with root cause>
org.springframework.transaction.UnexpectedRollbackException: Transaction silently rolled back because it has been marked as rollback-only
	at org.springframework.transaction.support.AbstractPlatformTransactionManager.processCommit(AbstractPlatformTransactionManager.java:752) ~[spring-tx-5.2.6.RELEASE.jar!/:5.2.6.RELEASE]
	at org.springframework.transaction.support.AbstractPlatformTransactionManager.commit(AbstractPlatformTransactionManager.java:711) ~[spring-tx-5.2.6.RELEASE.jar!/:5.2.6.RELEASE]
	at org.springframework.transaction.interceptor.TransactionAspectSupport.commitTransactionAfterReturning(TransactionAspectSupport.java:631) ~[spring-tx-5.2.6.RELEASE.jar!/:5.2.6.RELEASE]
	at org.springframework.transaction.interceptor.TransactionAspectSupport.invokeWithinTransaction(TransactionAspectSupport.java:385) ~[spring-tx-5.2.6.RELEASE.jar!/:5.2.6.RELEASE]
	at org.springframework.transaction.interceptor.TransactionInterceptor.invoke(TransactionInterceptor.java:118) ~[spring-tx-5.2.6.RELEASE.jar!/:5.2.6.RELEASE]
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186) ~[spring-aop-5.2.6.RELEASE.jar!/:5.2.6.RELEASE]
	at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:749) ~[spring-aop-5.2.6.RELEASE.jar!/:5.2.6.RELEASE]
	at org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(ExposeInvocationInterceptor.java:95) ~[spring-aop-5.2.6.RELEASE.jar!/:5.2.6.RELEASE]
	at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186) ~[spring-aop-5.2.6.RELEASE.jar!/:5.2.6.RELEASE]
	at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:749) ~[spring-aop-5.2.6.RELEASE.jar!/:5.2.6.RELEASE]
	at org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:691) ~[spring-aop-5.2.6.RELEASE.jar!/:5.2.6.RELEASE]
	at org.apereo.cas.DefaultCentralAuthenticationService$$EnhancerBySpringCGLIB$$4692783a.destroyTicketGrantingTicket(<generated>) ~[cas-server-core-6.2.0-SNAPSHOT.jar!/:6.2.0-SNAPSHOT]
[catalita.out.txt](https://github.com/apereo/cas/files/4841919/catalita.out.txt)

	at org.apereo.cas.web.flow.logout.TerminateSessionAction.terminate(TerminateSessionAction.java:101) ~[cas-server-support-actions-6.2.0-SNAPSHOT.jar!/:6.2.0-SNAPSHOT]
	at org.apereo.cas.web.flow.logout.TerminateSessionAction.doExecute(TerminateSessionAction.java:82) ~[cas-server-support-actions-6.2.0-SNAPSHOT.jar!/:6.2.0-SNAPSHOT]
	at org.springframework.webflow.action.AbstractAction.execute(AbstractAction.java:188) ~[spring-webflow-2.5.1.RELEASE.jar!/:2.5.1.RELEASE]
Caused by: org.hibernate.QueryException: could not resolve property: ticketGrantingTicket of: org.apereo.cas.ticket.TransientSessionTicketImpl [DELETE FROM org.apereo.cas.ticket.TransientSessionTicketImpl s WHERE s.ticketGrantingTicket.id = :id]
[...]

I attach the full stacktrace.
Hope it helps!

Debugging the error, it is thrown by the the destroyTicketGrantingTicket() method in the JpaTicketRegistry because the Jpa implementation expects ticketGrantingTicket attribute to be there but the TransientSessionTicket doen't declare it.
I suppose that the TransientSessionTicket never is owned by a TGT so the fault is in the TicketDefinition.
The solution I propose within the pull request is to declare the fact in the TicketDefinition.


Mixtral nf4 performance 2x slower than expected

System Info
Latest Lorax version
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[ ] My own modifications

Reproduction
Compare mistral-7b nf4 perf to mixtral nf4 perf
Expected behavior
On a single a6000 using bitsandbytes nf4 quantization I'm seeing 17ms per token with Mistral-7b and 80ms per token with Mixtral.
My expectation is that Mixtral should have the performance of roughly 2x the 7b model (around 40ms per token). Current perf levels seem to negate the advantage over a large non-MOE model.

Hey @timohear, thanks for reporting. I can definitely take a look some time this week. There are some differences in how Mixtral is implemented Mistral that might be contributing to the differences in perf, but some more thorough benchmarking on our side to see where the bottlenecks are coming from would be a good next step.


SSL FULL verify seed connection, but only CA verify for cluster node connections

Feature Request
For cluster-mode connections, fully verify certificates and hostnames for seed connections, but only verify certificates (without hostname verification) for discovered cluster node connections where IP addresses are used.
Is your feature request related to a problem? Please describe
When using redis instances provided by cloud providers in cluster mode with SSL enabled, the cloud providers typically provide a certificate that only matches the hostname, and does not contain subject alternative names for the IP addresses in the cluster.
When using SslVerifyMode.FULL, the seed connection will work properly because it is typically made using hostnames, which match the certificate hostname.   However, after lettuce discovers all the cluster nodes, and attempts connections to individual cluster nodes, lettuce uses IP addresses, and those connections will fail hostname verification.   Therefore, currently SslVerifyMode.FULL cannot be used.
In this case, you currently have to use SslVerifyMode.CA.   But that means that the hostname is not verified for the seed connection.
Describe the solution you'd like
I would like an SslVerifyMode that fully verifies the certificate (signature and hostname) on cluster seed connections, but only verifies the certificate (without verifying hostnames) on cluster node connections when IP addresses are used.
A simple implementation would be to have an SslVerifyMode that only performs hostname verification on connections where an actual hostname is used, but does not perform hostname verification where only an ip address is used (which would be the case for cluster node connections by ip).  In this case, the connection logic would not know if the connections is a "seed" connection or a specific node connection.  It only needs to know the destination.
Describe alternatives you've considered
Using SslVerify.CA will allow all connections to be established, but it does not perform hostname verification on the seed connections as I would like.
Teachability, Documentation, Adoption, Migration Strategy
SslVerifyMode.FULL_IF_HOSTNAME  (naming is hard.  better names welcome) - Fully verifies the certificate and its hostname when a connection is made using a hostname, but only verify the certificate (without hostname verification) when a connection is made using an IP address.  This is useful when using cluster mode, and the SSL certificate does not contain subject alternative names for IP addresses.

Weakening SSL verification is almost a guarantee for a future security incident. Right now, you can provide your own TrustManagerFactory via SSLOptions to tune how hosts are verified.
The effect you see on cloud providers exposing only certain hostnames is a lack of specification of how SSL in a Cluster actually should work. Part of the problem is that Redis is originally based off IP addresses and host names were introduced later on.

Weakening SSL verification is almost a guarantee for a future security incident.

To be fair we already provide an unsecure solution :

In this case, you currently have to use SslVerifyMode.CA. But that means that the hostname is not verified for the seed connection.

So the proposition is to tighten the security by verifying at least the seed node.
@mp911de do you think we should instead remove  SslVerifyMode.CA entirely?

SslVerifyMode.CA is already a compromise introduced to address the missing SAN/IPs in certificates. It is however a quasi-standard that many other networked services (e.g. Postgres) make use of. There are many more verification options (e.g., comparing fingerprints). With the ability to configure a custom TrustManagerFactory I think we're good for the majority of users and for the remaining group of folks, there seems to be no common ground for an abstraction beyond our customizer.

Having thought about it some more I agree with @mp911de that it doesn't make a lot of sense to implement this now.


fix the wrist joints list for Xsens/iFeel joint retargeting

To resolve the problem in https://github.com/robotology/walking-controllers/pull/87#issuecomment-872133999

thanks @S-Dafarra . Merging!


Scrolling to the bottom is not possible in certain conditions

Hello Laurent,
I have an issue with lrStickyHeader/stStickyHeader when the table size is a little bit bigger than the size of the viewport.
When the sticky header appears, it is removed, making the page scroll up again.
See http://run.plnkr.co/M9d8ptMcl3ivpJLZ/ with a viewport size of 800x600 and scroll to the bottom of the page to see the undesired effect.
Regards,
cmorgant.

@cmorgant The plunkr link doesn't exist anymore.

@lorenzofox3 Hi Laurent,
I have the same issue and I understand the cause. In my case, the issue manifests itself when scrolling slowly. When I scroll quickly it does not occur. I have traced the issue as below....
Start scenario: lr-sticky-header is not applied

Scroll down offsetTop = 449, window.scrollY = 454
The test to see if lr-sticky-header should be applied is conducted
offetTop - window.scrollY < 0 the result is -6 (line 53 of lrStickyHeader.js)
timeout is set and lr-sticky-header is applied.
Checking window.scrollY the value is now 417
Test is immediately performed to see if lr-sticky-header should be removed.
this.threshold - window.scrollY > 0 this.threshold = 449, winow.scrollY = 417, result = 32
lr-sticky-header is removed.
As soon as it is removed window.scrollY reads 454
This repeats adding and removing the class and you are unable to scroll down.

CAUSE:
this.thead.style.position = 'initial';
this.thead.style.position = 'fixed';
When these lines are applied through the chrome console they toggle window.scrollY between 417 and 454 respectively, this behaviour is only apparent when scrolling slowly.
When scrolling quickly the Y scroll value large enough to not cause the lr-sticky-header class to be immediately removed.
I believe the problem to be that when positioning the thead as fixed you are removing it from the flow of the page and so reducing the length of the page. When you set it back to initial it is reintroduced and the page lengthens.
I think the best way to address this is to read this.thead.clientHeight before changing the position and factor it into the calculation. What are your thoughts?


Support copying file/header/block links to clipboard

If this can be supported, one can paste the link to other external apps and open the desired Obsidian note/header/block there.

By header, do you mean the section under the current heading, or the text of the heading itself?

By header, do you mean the section under the current heading, or the text of the heading itself?

Originally, I was thinking about the link to the text of the heading itself, i.e. the link to "Header" in the example below. On a second thought, the text/section under the current heading may also be useful, i.e. all the text below "Header" in the example below.
## Header

Text.

- [ ] Task

Therefore, there can be different combinations for copying file/header/block links:

Only copy the link to the file, header (link to the text of the header) or block
Copy the link and the associated text to the file, header or block, i.e.

File link plus the content of the file
Header link plus the text under the header
Block link plus the text of the block

Only copy the text of the file (file content), header (text under the header) or block