Socratic Seminar 16 Reading List (May 2023)

Drop links here!

Da fees are too damn high
https://dune.com/dgtl_assets/bitcoin-ordinals-analysis

https://lists.linuxfoundation.org/pipermail/bitcoin-dev/2023-May/021620.html

I dont think we really talked about this enough
https://zerosync.org/zerosync.pdf

https://payjoin.substack.com/p/interactive-payment-batching-is-better
Interactive payment batching

Ledger recovery system
https://www.nobsbitcoin.com/ledger-to-launch-kyc-cloud-based-recovery-service/

new mempool
https://github.com/bitcoin/bitcoin/issues/27677

If we have time, zkcoin proposal
https://gist.github.com/RobinLinus/d036511015caea5a28514259a1bab119

Tx accelerator!
https://twitter.com/mempool/status/1659619347910803466

TBDxxx is not Ark
https://www.arkpill.me/

Thanks for gathering all these amazing links @0xBEEFCAF3 ! Re: TapAss (lol), it's my understanding this is just Taro, which had to be renamed. I'd put it on the reading list, but only plan to talk about it if someone from the audience brings it up and wants to lead that part of discussion.

https://twitter.com/JackTracante/status/1661114322469134362?s=20 Thread about the Justice Department bringing a case against Roman Sterlingov claiming that he was the operator of a centralized mixing service called Bitcoin Fog.


Indent to same column position when replacing text

One issue that I've seen with almost all of the languages is when text is replacing other text, it is not going to column where the first node remove was, it's always going to column 0 (or 1 if 1 is the start, I can't remember). From checking the function vim.lsp.util.apply_text_edits vs the treesitter node column, I don't know if this is possible but wanted to raise the issue in case I'm missing something. Some formatting will resolve this (like gofmt if that gets integrated), but this would resolve the issue for most languages I believe.

I would LOVE to do this.
I was thinking about creating this during the refactor setup phase.  It determines the indent of current line.  Perhaps some simple line stats class dangling off of the refactor object (which should turn into a class soon)

I was thinking about this as well. I don't think we can depend solely on a formatter for a language like python, where whitespace carries meaning. I suppose we'd need to pass the information about the original ident around, so that the output can be formatted accordingly. Also, worth considering situations where you may want to modify ident levels as you extract functions. Consider:
class PrintingCalculator:
    def add_one_and_print(self, x: int) -> int:
        plus_one = x + 1

        print("---")
        print(f"Result: {plus_one}")
        print("---")

Assuming we'd want to extract all the prints, where should it land?
a) Outside of the class? Need to dedent by one level.
b) Inside the class, but separate method? Need to keep indentation level.
Perhaps I'm missing something, but I'm not sure how this could work w/o keeping hold of the ident level information until the extraction is output. 🤔

For 106 we would have to do the following:

Determine scope of refactor
Determine scope's indenting
Determine outer scope's indenting
Apply constant creation's indenting with scope's indenting level
Apply function creation with outer scope's indenting.

Each of these will have to have their own determination for indenting.  We will also have to use vim's tab settings to determine if we are doing 4 white spaces or a tab.  There is a lot to do on this.

addressing @muchzill4 great points, we will also need to consider pulling functions out as class level vs out side class level.
There is still a lot to be done, and I am certainly not claiming we are close, but I would like to get there!
For part of this we may need to make language specific operations.  You can imagine applying function level protection (private / public / whatever) is going to be a very language specific task.  It will require node traversing at a language level.  Not a huge fan of taht :)

@muchzill4 I was playing around on this with python for adding whitespace to the text during   code generation for 106. The idea I had was for languages where the formatter won't help like you mentioned, but we just always pass in the column of the first node column into code generation. Then let code generation determine if it uses it. Where I got hung up was determining the nodes in the current region to find the first node.

There is something called indentexpr, this could really help!

Given the advances made in indentation in the last year and a bit, I'll close this issue for now - we can open issues for specific indentation cases later on if need be.


Document additional '-- tensor parallel size n' needed in ilab model serve

An extra -- --tensor-parallel-size n (n being the num of gpus) is needed in the ilab model serve command when serving the model. This should be doc'ed in the readme. Also, we would like to point out that everything after that flag is expected to be a vLLM command.
In 0.19.0, the vLLM args should become flags for ilab model serve.

What is the outcome if this doesn't make it to 0.18.0? Trying to understand whether or not this is a true blocker.

@mairin This issue is actually 2 things.

This is a UX improvment. It's to give the user the option to specify --gpus for ilab model serve. Right now to specify that they'd need to edit the config file, and change the vllm_args section to include a specific vLLM flag.

Charlie opened up https://github.com/instructlab/instructlab/pull/1931 to fix that issue. I narrowed down the scope for his PR in https://github.com/instructlab/instructlab/pull/1974. I think we can get 1974 in today.
I think providing just the flag for ilab serve should be a blocker. What Charlie had done by a gpu value into the serving config has bigger scope and I think we could move to the next release.

Upstream Docs for item 1.

CC: @alinaryan @cdoern


Add gstreamer1.0-rtsp and libgstrtspserver-1.0 packages

gstreamer1.0-rtsp:
https://packages.ubuntu.com/bionic/gstreamer1.0-rtsp
https://packages.debian.org/buster/gstreamer1.0-rtsp
(Can't find on fedora, ditto for all below.)
libgstrtspserver-1.0-0:
https://packages.ubuntu.com/bionic/libgstrtspserver-1.0-0
https://packages.debian.org/buster/libgstrtspserver-1.0-0
libgstrtspserver-1.0-0-dbg:
https://packages.ubuntu.com/bionic/libgstrtspserver-1.0-0-dbg
https://packages.debian.org/buster/libgstrtspserver-1.0-0-dbg
libgstrtspserver-1.0-dev:
https://packages.ubuntu.com/bionic/libgstrtspserver-1.0-dev
https://packages.debian.org/buster/libgstrtspserver-1.0-dev
libgstrtspserver-1.0-doc:
https://packages.ubuntu.com/bionic/libgstrtspserver-1.0-doc
https://packages.debian.org/buster/libgstrtspserver-1.0-doc

I think Fedora's gstreamer1-rtsp-server-devel is equivalent to Ubuntu's libgstrtspserver-1.0-dev, based on similar contents / file lists, so I'll add it under the libgstrtspserver-1.0-dev key.


GAP | NSFS | Versioning | List Object Version - Always Returns IsLatest true

Environment info

NooBaa Version: current master (5.14)
Platform: Rancher Desktop 1.19.0 (VM and cluster)

Actual behavior

We always return IsLatest true.

Expected behavior

Only the latest version should return IsLatest true.

Steps to reproduce
General Settings:

Change the configuration of NSFS Versioning to be Enabled in file config.js and change config.NSFS_VERSIONING_ENABLED to true.
Deploy noobaa system on minikube (Based on the instructions here - the steps of ‘Build images’ and ‘Deploy noobaa’).
Note: nb is an alias that runs the local operator from build/_output/bin (alias created by devenv).
Deploy NSFS on minikube (Based on the instructions here).

Steps:

Enabling versioning
aws --endpoint-url=https://localhost:12443 --no-verify-ssl s3api put-bucket-versioning --bucket fs1-jenia-bucket --versioning-configuration Status=Enabled
Put an object twice
 s3-nb-user-1 aws --endpoint-url=https://localhost:12443 --no-verify-ssl s3api put-object --bucket fs1-jenia-bucket --key fish.txt --body <some_txt_file>
 s3-nb-user-1 aws --endpoint-url=https://localhost:12443 --no-verify-ssl s3api put-object --bucket fs1-jenia-bucket --key fish.txt --body <some_txt_file>
List object versions:
aws --endpoint-url=https://localhost:12443 --no-verify-ssl s3api list-object-versions --bucket fs1-jenia-bucket

More information - Screenshots / Logs / Other output
Noobaa output:

AWS output:

@v-harihar knows about this gap, so I edited the title.

Please note that in addition to the object versions also multiple delete markers for a single object show IsLatest as true .
After deleting a single object, re-adding a new version and again deleting it:
...
    "DeleteMarkers": [
        {
            "Owner": {
                "DisplayName": "NooBaa",
                "ID": "123"
            },
            "Key": "key1",
            "VersionId": "mtime-d3lql1gt4ruo-ino-t1g",
            "IsLatest": true,
            "LastModified": "2024-08-21T16:39:42+00:00"
        },
        {
            "Owner": {
                "DisplayName": "NooBaa",
                "ID": "123"
            },
            "Key": "key1",
            "VersionId": "mtime-d3lqj85nwwlc-ino-t1e",
            "IsLatest": true,
            "LastModified": "2024-08-21T16:37:19+00:00"
        }
    ],
...

Closing this issue per Nadav's fix - @nadavMiz @8298

is this backported into 4.17, it is not available in the 4.17-0904 d/s build

@rkomandu
This is the link to check if a fix was backported to 5.17 - https://github.com/noobaa/noobaa-core/commits/5.17/
Search the PR's commit name - NSFS | NC | fix list-objecs-versions issues
and then you can see it was merged on September 4th (you need to hover the backport merge date- Merge pull request https://github.com/noobaa/noobaa-core/pull/8325 from liranmauda/liran-backport-into-5.17)


Swipe gesture support?

Hi there. I've been looking around for an accessible off canvas that also supports swipe gestures, but that seems to be difficult to find. Is there any chance gestures may be added to js-offcanvas at some point? Currently, I'm planning to retrofit accessibility stuff onto Snap.js, which seems to be very good on the gesture front, but really poor with accessibility, but I'd much rather use a solution built from the ground up with accessibility in mind.

Those seem like a good start, but I'm hoping for something more tightly integrated. The thing I really like about Snap.js and similar libraries is the way they slide the menu in real time as you swipe, instead of just popping open or closed.

Same here, would be cool to see support for this included.

Sorry, i can't add Swipe gesture.
Please use a use a external touch-plugin
See: https://github.com/vmitsaras/js-offcanvas/issues/12#issuecomment-361898617


Book Store: Add components

In my bookstore app I have added the following:

Created a directory for components
Added two routes
Added a navigation component with links to the two routes

Please kindly remove/update the Live-demo link so that it navigates to the correct URL
## 🚀 Live Demo

- [Live Demo Link](https://deploy-preview-5--musong-math-magicians.netlify.app/)

Thanks @abdulhamiid for your feedback. I have now added the tiltle and author values as props.


Change related-document (again) to relatedDocument

We should stick to one naming convention in our prismic model fields and pascalCase is nicer for the codebase, can be used as is for variables.
As the field is mega new and barely used, there's still time to change it.

All in prod and applied to prismic model


update package from update site

why not update from help->update on standalone version,that can make update operation more easy!

We have this (or similar) feature in plans.
It will require to add Eclipse p2 support in standalone version. Which means adding pretty many additional plugins.
Probably will try it on EE version first (it is already not-so-lightweight).

P2 update was added in DBeaver 3.7 (EE version).


New object implementation: Connectivity Statistics.

Remove compiler warnings for all object implementations.
Unify memory management using lwm2m_malloc, lwm2m_free, lwm2m_strdup.
additional callback implementation for object 'closeFunc' for memory
management verification.
Resource Id defines for object implementations.
Connectivity Monitoring object bug: Reduce TLV message size of object to
less than 128 Byte to avoid message buffer truncation without blockwise.
Signed-off-by: Joerg Hubschneider<EMAIL_ADDRESS>

Two minor comments, otherwise looks good. Thank you.


saslprep: update to the latest version

Question
Description
Hey, pdfkit team! I'm author of saslprep package. I've seen you are use v1.0.1 which eat A LOT  of resources and also slow. This bug was fixed in https://github.com/reklatsmasters/saslprep/issues/3 for mongodb.
If i understood right, you want to avoid fs operations by code point's loader (saslprep#lib/memory-code-points.js). I think, there is need to define second loader (for the web). Sadly, i don't known how to do that right. PR welcome!

@zesik any reasons you used saslprep 1.0.1 instead of 1.0.2?

@blikblum It's exactly because of what @reklatsmasters mentioned. It runs fine on node, but my tests show browserify had some problems loading 'code-points.mem'. Not sure if this is fixed in newer versions.

And now all versions prior 1.0.3 has a wrong mechanism of getting code points. code-points.mem might be converted into base64 in webpack. I think it should help.

Will look into this

saslprep version 1.0.1 have code points in file code-points.js with size 23 kB.
saslprep version 1.0.2+ have code points  in binary file code-points.mem with size 410 kB.
It is huge increase pdfkit package for browser...

410 kB is prohibitive and gets worse on browser were must be converted to base64
Options:

stick with 1.0.1
try to optimize saslprep for size
look for other dependency

Second options i have an idea what to do

Second options i have an idea what to do

Ok, listen. I may return previous solution, but it will less optimal.

While upstream saslprep package is not updated with a fix, i added a local version optimized for size and memory


First class support for parh based APIs in AuthConfig

Hello
I've created a discussion, https://github.com/Kuadrant/authorino/discussions/485
but not sure which is more preferrable: issue or discussion.
Plz let me know what should be kept.

Hello @eshepelyuk!
Discussions are usually better for topics with multiple of ramification (so people can comment on each thread separately). We don't use them a lot in Authorino, but it's really up to you. While issues tend to be more suitable for feature requests, bug reporting and simple questions, I guess.
In this particular case, I left an answer for you in the discussion (which I've already seen you replied to.) Let me try to elaborate on the problem with supporting path-based lookup there. If needed, we can reopen this issue with a more concrete plan.


Validation Fail

Challenge http://www.freecodecamp.com/challenges/waypoint-create-a-set-of-radio-buttons has an issue. Please describe how to reproduce it, and include links to screen shots if possible.

Both radio inputs do not share the same name value, and should not pass the lesson's validation.

Yea you can also do that one with only one label before them and 1 ending label at the end. Reported it last night as well, but different way to do it. Probably needs to have the test do a label has input . text type check?

Thanks @wcountiss!
I've confirmed this issue:
  indoor<input type=radio name=indoor-outdoor>
  outdoor<input type=radio>

in the form, passes all the tests when number 2 should fail.


The header of the M3 DatePicker is clipped in landscape mode.

If the DatePicker is in landscape mode and it uses DatePickerEntryMode.input, the header will be clipped.
The portrait mode and M2 version work as expected.

Code sample
import 'package:flutter/material.dart';

void main() => runApp(const DatePickerApp());

class DatePickerApp extends StatelessWidget {
  const DatePickerApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData(useMaterial3: true),
      home: const Scaffold(body: Center(child: DatePickers())),
    );
  }
}
class DatePickers extends StatefulWidget {
  const DatePickers({super.key});

  @override
  State<DatePickers> createState() => _DatePickersState();
}

class _DatePickersState extends State<DatePickers> {
  DateTime? selectedDate;
  final DateTime _firstDate = DateTime(DateTime.now().year - 2);
  final DateTime _lastDate = DateTime(DateTime.now().year + 1);

  @override
  Widget build(BuildContext context) {
    return TextButton(
      onPressed: () {
        showDatePicker(
          initialEntryMode: DatePickerEntryMode.input,
          context: context,
          initialDate: selectedDate ?? DateTime.now(),
          firstDate: _firstDate,
          lastDate: _lastDate,
        );
      },
      child: const Text('Show date picker'),
    );
  }
}

The M3 date picker lays out correctly in portrait mode and the spec doesn't even cover landscape mode. Currently the date picker chooses landscape orientation using MediaQuery.orientationOf(context) however it should probably use portrait orientation unconditionally on non-mobile platforms.
Here's the current portrait model input date picker dialog:


Remove misplaced 'ems_ref' from log message

Other methdods in this module iterate over 'ems_ref' of resources, but
SSPS collect everything with a single call to 'connection.ssps'. The log
message was probably copied from the other block without removing the
'ems_ref'.

There are other places in this same file interpolating ems_ref inside a rescue block.


append data with “%25” causes URIError: URI malformed  at decodeURIComponent ()

obsidian://advanced-uri?vault=MyVault&filepath=test.md&heading=test&mode=prepend&data=%25 or any data with “%25” (percent symbol encoded) produces the following error:
plugin:obsidian-advanced-uri:3727 Uncaught (in promise) URIError: URI malformed
    at decodeURIComponent (<anonymous>)
    at AdvancedURI.eval (plugin:obsidian-advanced-uri:3727:69)
    at step (plugin:obsidian-advanced-uri:72:23)
    at Object.eval [as next] (plugin:obsidian-advanced-uri:53:53)
    at eval (plugin:obsidian-advanced-uri:46:71)
    at new Promise (<anonymous>)
    at __awaiter (plugin:obsidian-advanced-uri:42:12)
    at eval (plugin:obsidian-advanced-uri:3717:100)
    at t (app.js:1:1515406)
    at <anonymous>:1:52

Apparently, the data portion of the URI needs to be URLencoded twice, so that a [space] becomes %2520 rather than just %20. Doing this produces the desired functionality.


[6.x] Fixed Application: isRunningInConsole being cached

Application: isRunningInConsole is cached at Application.php#L561.
For cli programs, this cache should not be initialized when loading environment variables.

Breaking change.


[Event Enhancement Request] Sender needs to be added in Codeunit 5802 Events

Sender needs to be added in Codeunit 5802 Events
Three integeration events in codeunit 5802 "Inventory Posting to G/L" need to be enhanced by adding the sender.
These events are:

OnAfterBufferOutputPosting

OnAfterBufferConsumpPosting

OnAfterBufferCapacityPosting
[IntegrationEvent(true, false)]
local procedure OnAfterBufferOutputPosting(var ValueEntry: Record "Value Entry"; var CostToPost: Decimal; var CostToPostACY: Decimal; var ExpCostToPost: Decimal; var ExpCostToPostACY: Decimal)
begin
end;
[IntegrationEvent(true, false)]
local procedure OnAfterBufferConsumpPosting(var TempInvtPostingBuffer: array[4] of Record "Invt. Posting Buffer" temporary; ValueEntry: Record "Value Entry"; var PostBufDimNo: Integer; var CostToPost: Decimal; var CostToPostACY: Decimal)
begin
end;
[IntegrationEvent(true, false)]
local procedure OnAfterBufferCapacityPosting(var ValueEntry: Record "Value Entry"; var CostToPost: Decimal; var CostToPostACY: Decimal)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


add: Sibs & update: Multicert

From the GitHub Action it seems that Multicert's career page changed.
The new "careers" URL is this one https://www.multicert.com/pt/carreiras/, however, inside it we can find the link to the true job listing page (which from my understanding should be the one linked), which redirects to the Sibs's career page (https://sibs-sgps.zohorecruit.eu/careers). Took advantage of this to also add SIBS.
Creating this as a draft PR since I'm not 100% sure if we should maintain both companies.

Seems OK to me keep both SIBS and Multicert given Multicert division has a different focus than the parent company SIBS.


Why cant i use it with github enterprise ?

First of all, Thanks! this is really useful!
I see that this package allows only https://github.com/*,  https://gist.github.com/* origins, but i've a github enterprise and i dont want to do any host mapping to get it working.  I thought i would introduce an option that can be taken from the user, but the below comment stops me doing that 😃

We're happy to receive suggestions and contributions, but be aware this is a highly opinionated project. We're not interested in adding options as it's a slippery slope into adding it for everything. Users will always disagree with something. That being said, we're open to discussing things.

Anyway, i could manually add an origin in the manifest.json and import it into my browser to make it work for now.
Can we take this as an enhancement?

Thanks for opening the issue! However I'm closing this as duplicate of #315. Please subscribe to it and participate in any discussion there. Cheers!

Thanks for the quick response!


Pull request for adding Terrain Classification using DeepViT, EfficientNetb3, YoloV8, DeiT3

Pull Request for DL-Simplified 💡
Issue Title : Terrain Recognition

Info about the related issue (Aim of the project) :
Name: Mohana Krishna
GitHub ID: https://github.com/MohanRocks999
Email ID<EMAIL_ADDRESS>Idenitfy yourself: (Mention in which program you are contributing in. Eg. For a JWOC 2022 participant it's, JWOC Participant) I am Contributing in GSSoC 2024

Closes: Issue number 636 will be closed through PR
Add-ons or changes made 📃
Trained a Image Classification using various deep learning algorithms. This include DeepViT (Not just Vision Transformer),  EfficientNet B3, DeiT3 (Data efficient image transformer) and YoloV8 that classifies images on 5 different types of terrains - coast, mountain, glacier, forest and desert.
Type of change ☑

[ X] New feature - A Terrain Classification using the respective neural network architecture using pytorch.

How Has This Been Tested? ⚙
The Model have been evaluated extensively with a validation dataset comprising of 2000 images (400 images per class) and a test dataset of 500 images.
Checklist: ☑

[ X] My code follows the guidelines of this project.
[ X] I have performed a self-review of my own code.
[X ] I have commented my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[ X] My changes generate no new warnings.
[X ] I have added things that prove my fix is effective or that my feature works.
[ ]X Any dependent changes have been merged and published in downstream modules.

What's the issue number?

The issue number is 636

Hi @MohanRocks999 remove the extra files and follow the project structure for this project.

What files needs to be removed and do you want me to add all the models in one single notebook?

It'll be better to have a single notebook file (.ipynb) where all the models and EDAs can be implemented together.

The changes has been as per your request. check it out.

The requested changes has been done. Please check the changes as soon as you can.


Set up a Timber Slack team or IRC channel

Opening an issue to discuss and help out if necessary.
I think it would be very useful to have somewhere where we can discuss Timber and get and give help. I had a little discussion with @jarednova on Twitter about Slack or IRC, and it seems preferable to get something going on IRC.
What does everyone think?

Hey, why not gitter.im?

@prisis No real reason apart from never having used it, and that I'm already using Slack and IRC, so those were the tools that I thought of.
I personally don't mind what is used, as long as we end up with something :)

+1 for the idea. but i would tend to either the mentioned slack ( where most of the wordpress related conversations happen these day ) or irc. to get another chat app like gitter.im just for a timber channel i am unsure if i would join. but that are only my two cents. ;)

freenode -> #timber

+1

@dinolatoga @daslicht @rpkoller @anugupta @prisis I've got things setup. Email me at<EMAIL_ADDRESS>and I'll invite you to the Slack Team

:)

For those wondering, we've shut down the Slack team in favor of Gitter:
https://gitter.im/timber/timber?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge


Please add // ignore_for_file: unnecessary_new

for actions or remove new from generated code

Ups, seems the analyzer result didn't update properly after all the code changes.


MLIL return statement sometimes eliminates return value

Binary Ninja Version: 1.1.930-dev, 8df66a30
Platform: OS X 10.12
mipsel-libpam.so.0.zip
In the attached MIPS binary, the function pam_authenticate returns an error code, but when going from mapped MLIL to full MLIL, jump($ra) becomes "return" without a value.  Note the function type signature has an int32_t return type, so the MLIL lifting should end with a "return $v0".

I'm having a similar issue (using a custom architecture plugin) where it ignores the size of the return value and just returns one byte (=regsize):
int16_t  mul_u16(char arg1 @ r26, char arg2 @ r27, char arg3 @ r30, char arg4 @ r31)

int16_t  mul_u32(char arg1, char arg2, char arg3, char arg4, char arg5 @ r26, char arg6 @ r27, char arg7 @ r30, char arg8 @ r31)

The mapped MLIL looks correct for me as well.

We're not sure if this was the result of a coincidence or another fix along the way, but it looks like this particular example now behaves properly. If you see other functions that have trouble, you can additionally try analysis.experimental.alternateTypePropagation which may help with some binaries, but may have other negative side effects. Feel free to re-open if you see other examples.


how to handle \G in regexes

the python regex engine doesn't have this -- I imagine I can replace it with \A and do something clever during parsing, but I'm not super sure

via addd5b4cab00740f71d61b1b9e2e0f9e92d37bc8


Select Top 1000

Issue Type: Bug
Steps to Reproduce:

Open Azure Data Studio
Connect to database
right click on one of the tables in the database
select 'SELECT TOP 1000' option
sit back and watch nothing happen:)
On this and the last version of Azure Data Studio (maybe others too I don't know) the 'SELECT TOP 1000' does nothing. No error message, no nothing. This is a crucial feature please fix promptly

Azure Data Studio version: azuredatastudio 1.39.1 (7553f799e175f471b7590302dd65c997b838b29b, 2022-08-30T01:11:13.047Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: No

System Info

Item
Value

CPUs
AMD Ryzen 7 PRO 5850U with Radeon Graphics      (16 x 1896)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
11.83GB (4.97GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Issue currently does not reproduce on our side via a local database, can you please describe the server database configuration you are using?

I am not sure. I work for a medium organization and we are connecting to
Azure. Wouldn’t our configuration be set by IT or those that set the DB up?
On Tue, Sep 6, 2022 at 4:13 PM Alex Ma @.***> wrote:

Issue currently does not reproduce on our side via a local database, can
you please describe the server database configuration you are using?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azuredatastudio/issues/20551#issuecomment-1238704839,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARQ36C2EE6RWKMPLJSV6LFDV466Y3ANCNFSM6AAAAAAQGHEERY
.
You are receiving this because you authored the thread.Message ID:
@.***>

That is correct, they should know what the database type/configuration is.
This issue appears to be similar to this previously submitted issue: https://github.com/microsoft/azuredatastudio/issues/20241
Ask your IT department or those who set the DB up if they are using Azure Synapse or not, we currently have an issue where Select Top 1000 does not work for those kinds of Databases.

I am not sure. I work for a medium organization and we are connecting to Azure. Wouldn’t our configuration be set by IT or those that set the DB up?
…
On Tue, Sep 6, 2022 at 4:13 PM Alex Ma @.> wrote: Issue currently does not reproduce on our side via a local database, can you please describe the server database configuration you are using? — Reply to this email directly, view it on GitHub <#20551 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARQ36C2EE6RWKMPLJSV6LFDV466Y3ANCNFSM6AAAAAAQGHEERY . You are receiving this because you authored the thread.Message ID: @.>

Closing as duplicate

@nathangmahler1987 please try it again when the new version is available in a month or so.

Thanks!
On Fri, Oct 14, 2022 at 8:50 PM Alan Ren @.***> wrote:

@nathangmahler1987 https://github.com/nathangmahler1987 please try it
again when the new version is available in a month or so.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azuredatastudio/issues/20551#issuecomment-1279640931,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARQ36C2G4F6YBAAUMP2HJQLWDILW3ANCNFSM6AAAAAAQGHEERY
.
You are receiving this because you were mentioned.Message ID:
@.***>


fix(file_upload): Fix columnar and Excel upload forms

SUMMARY
One of the form fields we changed when working on CSV forms was the not very descriptive "con" field, which was renamed to "database". Now, we are updating Columnar and Excel forms so they both stop displaying an error message. Additionally, we updated our tests.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
After:

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

QA tested. Fix LGTM!

🏷 preset:2022.47


ComfyUI support?

Is there a way to use this in comfyui?
Really impressed with the prompt following that a user shared into a discord channel.
Also can lora's be created for it? Can it be trained?

Interested as well （づ￣3￣）づ

Made a wrapper that can run the T2I models:
https://github.com/kijai/ComfyUI-LuminaWrapper

Made a wrapper that can run the T2I models:
https://github.com/kijai/ComfyUI-LuminaWrapper

Very cool, though I am on Windows and slightly reluctant to use pre-built wheels (the llmvision* issue has me cautious)
*think that was the name of it.

Made a wrapper that can run the T2I models:
https://github.com/kijai/ComfyUI-LuminaWrapper

Very cool, though I am on Windows and slightly reluctant to use pre-built wheels (the llmvision* issue has me cautious)
*think that was the name of it.

I get that, but it's the same issue with the original code. There's a fallback to SDP attention in the code, but it does not work at all. I built flash_attn on my Windows install and it works fine, took almost an hour though...

I'm having a lot of trouble with flash attention on my windows, any help would be greatly appreciated :)

It now works without flash_attn, it's just much slower and uses twice as much VRAM, but it works.

I need a something to host a Demo Locally on my 4090


[WEB-447] Move research & papers from org to com

@lscheinkman please review

I will be sure to split commits up by content and snaps next time.


Image has no description or it is repeated due to some typo

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://demo.plone.org/content-types'
Scroll down to see the repition

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.

Software (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Volto Version [e.g. 8.5.0]
Plone Version [e.g. 5.2.2]
Plone REST API Version [e.g. 7.0.1]

Additional context
Add any other context about the problem here.

Hey @SuyashShukla0007  i would love to work on this issue!!

This is not really an issue; there are just 2 different images in that folder. Anyone is welcome to improve the title and description of the images. The demo content is in https://github.com/plone/demo.plone.org


Adding TODO headliner after current headline puts new TODO after blank newlines

Describe the bug
First of all, thanks for all the work you're putting in this project. I could not live without it anymore.
When i try to add a new TODO e.g. with <leader>oit, the new todo is added after the empty new lines that follow that TODO instead of right before its content. I think that blank lines are treated as actual content of the previous TODO, even if they're completely empty (no whitespace in)
Steps to reproduce

Go to a TODO that is followed by one or two blank lines
Press oit

Expected behavior
The new TODO should be right after TODO and its content, ignoring whitespace (which can be used as separator between headlines)
Emacs functionality
No response
Minimal init.lua
I'm using
org_blank_before_new_entry = {
    heading = false,
    plain_list_item = false,
  }

Screenshots and recordings
No response
OS / Distro
Arch Linux
Neovim version/commit
0.6.1
Additional context
No response

To be honest, that's the behavior I would expect. If there are newlines after a TODO item, they should belong to that TODO. What else should they belong to? Probably not the following headline?

I usually use them as visual separation between headlines like it is show in the README's screenshots https://user-images.githubusercontent.com/1782860/123550227-e8605800-d76c-11eb-96f6-c0a677d562d4.gif
How do you accomplish that? Do you manually reapply space each time you insert a new TODO item?

Ah I see what you're trying to do! Personally I don't use blanks for visual separation so I haven't ecountered the issue before.
I would maintain that the blanks under a headline belong to that headline, and to achieve what you want should be phrased differently, something like "how to insert a headline such that there's a blank both to the headlines above and below it". I'm curious how emacs org-mode does this, if I find some time I'll research that.

@kofm why do you have heading set to false in in org_blank_before_entry? Try setting it to true, I think that will enable the behavior you want

Thanks guys for your responses.
I just installed and tested using emacs-org and I can reproduce the behaviour I'm expecting.
I don't want to automatically insert a blankline before a new entry but I want the empty newlines following a TODO to not be considered as part of it.
Example:
ATM if i have this situation:
* Heading 1
** TODO First | <-- cursor here
   Text associated with my todo

* Heading 2

if I press (default mapping) <leader>oit, i get this:
* Heading 1
** TODO First
   Text associated with my todo

** TODO new todo
* Heading 2

if instead i press <leader>oiT, i get this:
* Heading 1
** TODO First
** TODO new todo
   Text associated with my todo

* Heading 2

which disrupt any note-clock-etc. under the TODOs
Using emacs-org, in the same first scenario if I use S-M-RET (org-insert-todo-heading) in the same spot, I correctly get:
* Heading 1
** TODO First
   Text associated with my todo
** TODO new todo

* Heading 2

(Reference: https://orgmode.org/manual/TODO-Basics.html#TODO-Basics)
I think the point here is that IF the line does not contain any whitespace it should be ignored and not considered as part of the previous TODO.
Hope this helps

Interesting. Prompted by your comment I just tried it in emacs as well and could reproduce. It's great that we can always cross-check behavior with emacs and just copy it.

@kristijanhusak I think that the idea of @kofm to have the blank lines above a headline be part of that headline, and not the headline above it, would make the most sense. I guess this would have to be changed in the tree-sitter org grammar (@milisims), and then this issue would be automatically solved.
It's quite unintuitive at first. But as far as I can see, blank lines between headings are used mostly/only for visual separation. So making this change would have more advantages: if one does something with a headline (refiling it, moving it), it's practical if the separator whitespace gets moved to the new location as well.

I agree with @milisims . Also, Emacs Org considers empty lines in a same way (not above the headline).

@Maltimore can you confirm that empty lines above a headline are considered part of the headline tree in Emacs? I.e. what happens if you archive / refile such a headline? Do the empty lines get moved as well?


Footer

Create static footer with legal links + infomation like:
https://local.ubs.com/westchester-purchase-ny/Index.htm

Hi kesiah,
I’m working on the static footer with legal links and information as mentioned, but when I try to add it, the footer doesn’t display or disappears. Should I modify footer.html directly, or is it dynamically loaded? Also, do I need to create a new CSS file for styling, or use the existing one? Any guidance would be appreciated!
Thanks!


chore: split communication-related functionality from mempool and mempool-types modules

Stack:

#224 ⬅
#223

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

This change is

Codecov Report
Attention: Patch coverage is 87.23404% with 6 lines in your changes missing coverage. Please review.

Project coverage is 65.72%. Comparing base (600a0cb) to head (3d464d7).

Files
Patch %
Lines

...s/mempool_types/src/mempool_communication_types.rs
71.42%
1 Missing and 5 partials :warning:

Additional details and impacted files
@@                                        Coverage Diff                                        @@
##           pr/Itay-Tsabary-Starkware/tsabary/mempool_types_cleanup/d2d71f66     #224   +/-   ##
=================================================================================================
  Coverage                                                             65.72%   65.72%
=================================================================================================
  Files                                                                    23       24    +1
  Lines                                                                   916      916
  Branches                                                                916      916
=================================================================================================
  Hits                                                                    602      602
  Misses                                                                  288      288
  Partials                                                                 26       26

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Order of List<>   changes when using  @OneToMany relation

Hi,
we are using a one to many relation  from a PersistedCase to a PersistedPerson:
@OneToMany(fetch = FetchType.EAGER, cascade = CascadeType.NONE) @JoinProperty(name = "offers")  private List<PersistedOffer> offers = new ArrayList<>();
Then we add Persons in a specific order and save it. In 20-50% of the cases the order of the Persons lists order has changed  when I look into the DB.
Best
Andreas

Hello @andy-mueller yes there is no guarantee on the order of the elements in the array, you should specify order through a comparator on java side.


Use Ormolu, together with treefmt.

This adds Ormolu as the Haskell source code formatter, replacing Brittany. Brittany is unmaintained since 2022-11-11. This also adds a treefmt configuration file to drive Ormolu. That configuration could be extended in the future to e.g. also format Nix expressions.

I also find the alignment brittany does for imports (including qualified imports) quite nice.
It is a shame that brittany is unmaintained. I wonder how much work it would take to bring it up to speed.

Yeah, I'm not super happy either. I'll still merge the PR, even if we don't run ormolu on our code right away.


Fix the autoupdate

The code is hard to understand. With this pull request, it becomes much harder. See #1934 and please to just say: "It's not my thing ...". :-)


[backport] Optimize RandomState.interval

The backport of #559 .

jenkins, test this please

LGTM!


Ear 943 mbs fixes

What is the context of this PR?
Describe what you have changed and why, link to other PRs or Issues as appropriate.
How to review
Describe the steps required to test the changes (include screenshots if appropriate).
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

Include: furloughed employees in any employment figures.

I went back through and added this.

All banners need to have
Include: furloughed employees in any employment figures.

in them

I went back through and added this.

There's a few questions dotted around that have been set to required. Previously, the business accidentally did this and making them optional was one of the things they wanted to fix in this.
I've asked about if they've all been done and raised that there's still some lurking.


Issue with IoT Edge Quickstart for Windows

On this tutorial, the "Deploy a Module" section doesn't work for me.  I set the name and copy in the URI and click Save as instructed, but nothing happens.  I've tried this over and over again, even in different browsers.  Nothing, including no errors displayed.
Do I need to specify a Container Registry login first?  Please try and reproduce this issue and update the doc if there's something awry.
Thanks!

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

ID: 64575e7c-664c-4d6b-a440-24c91e93def5
Version Independent ID: 17d9481c-a75b-bbfd-1f9a-76fc3d6b2930
Content: Quickstart Azure IoT Edge + Windows
Content Source: articles/iot-edge/quickstart.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

@vfab Thanks for the feedback! We are currently investigating and will update you shortly.

@vfab can you try using CLI instead of Azure Portal as a workaround?
https://docs.microsoft.com/en-us/azure/iot-edge/how-to-deploy-modules-cli
Thank you!

@vfab can you please try to "Deploy a Module" through the portal again? It should now be possible.
Thank you!

I tried it just now and it worked, no problem.  Obviously it's not a
documentation issue!
Thanks!
On Thu, Sep 13, 2018 at 9:42 AM António Sérgio Azevedo <
<EMAIL_ADDRESS>wrote:

@vfab https://github.com/vfab can you please try to "Deploy a Module"
through the portal again? It should now be possible.
Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/14871#issuecomment-421011302,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AO_bmSgN7gUEjdqBN3PCO8imXgELDA06ks5uamDRgaJpZM4WlaiO
.


Bind multiple actions to one key (execute, then abort)

My script is
#/bin/sh

session=$(tmux list-sessions -F "#S" | fzf +s --tac +m --tiebreak=index --toggle-sort=ctrl-r --bind='ctrl-t:execute(tmux kill-session -t {})')
tmux switch-client -t "$session"

I'd like to be able to abort the fzf command after the execute has been done. Is it possible ?
I've tried --bind='ctrl-t:execute(tmux kill-session -t {})'),abort but that doesn't work

How about using --expect=ctrl-t instead? There was a similar request: https://github.com/junegunn/fzf/issues/340

Strangely if I do that, and quit with ctrl-t, the session variable contains a \n at the beginning of the element.
#/bin/sh

session=$(tmux list-sessions -F "#S" | fzf +s --tac +m --tiebreak=index --toggle-sort=ctrl-r --bind='ctrl-t:execute(tmux kill-session -t {})' --expect=ctrl-t)
printf "%s" "$session" > /tmp/tsession
tmux switch-client -t "$session"

If I look at /tmp/session, it contains a "\n" when I have the --expect=ctrl-t and exit fzf with enter, but I don't have a "\n" if I remove the --expect=ctrl-t

It seems that it is the expected behaviour of --expect

--expect=KEY[,..]
Comma-separated list of keys that can be used to complete fzf in  addition  to  the  default
enter  key. When this option is set, fzf will print the name of the key pressed as the first
line of its output (or as the second line if --print-query is also used). The line  will  be
empty if fzf is completed with the default enter key.
e.g. fzf --expect=ctrl-v,ctrl-t,alt-s,f1,f2,~,@

I finally achieved what I want :
#/bin/sh

session=$(tmux list-sessions -F "#S" | fzf --no-sort --tac --tiebreak=index --toggle-sort=ctrl-r --bind='ctrl-t:execute(tmux kill-session -t {})' --expect=ctrl-t)

tmux list-sessions -F "#S" | grep -v "dot"

while read -ra LINES; do
    for i in "${LINES[@]}"; do
        [ "$i" == "ctrl-t" ] && fzf-list-session && return 1
        tmux switch-client -t "$i"
    done
done <<< "$session"

Hmm, you don't want/need execute in this case:
lines="$(seq 100 | fzf --expect=ctrl-t,alt-t)"
key="$(head -1 <<< "$lines")"
rest="$(sed 1d <<< "$lines")"

case "$key" in
  ctrl-t)
    echo "you pressed ctrl-t on $rest"
    ;;
  alt-t)
    echo "you pressed alt-t on $rest"
    ;;
  *)
    echo "enter key or mouse click on $rest"
    ;;
esac

You see the point?

Ok, I've removed ctrl-t from my binding.
Thanks for the tip


Decouple publishing and closing dates on briefs

Can you give a description of what this will do?

Sure. This adds 2 new columns closed_at and questions_closed_at to the brief table.
When a brief is published (there is a publish API that already sets the published_at to now), the closing dates are now calculated and saved into those columns. This allows marketplace admins to adjust those dates if required (eg. for public holidays/shutdown).
In allowing admins to adjust the dates, there are still business rules that validate the interdependency of the three columns closed_at/questions_closed_at/published_at - hence the tests have had to be adjusted to save published_at. This wouldn't happen for real users because they would create the brief over time.


-n/--no-file option for suppressing the # File: lines

a --no-file option to suppress output of the # File:  lines.
Those can have shortcuts of -i for --imports and -n for --no-file - so if you want to apply both (and just get the imports) you can then do this:
symbex -d symbex -in

Split from:

#26

Pretty neat:
symbex -in -d symbex

# from .lib import find_symbol_nodes
def find_symbol_nodes(code: str, filename: str, symbols: Iterable[str]) -> List[Tuple[(AST, Optional[str])]]

# from .lib import code_for_node
def code_for_node(code: str, node: AST, class_name: str, signatures: bool, docstrings: bool) -> Tuple[(str, int)]


Types for messages inside errors and error parsers

Currently the Error enum contains String as the type holding the message for each error. This is obviously really easy to work with since regardless of what we want to say in the message we can always format it into a string. This is obviously not very efficient when our error messages are static as they often are. Currently I am leaning towards replacing the String type with Cow<'static, String, str> inside the Error enum which should at least avoid the allocation for errors with a static string as a message.
Other approaches could be something like allowing functions or boxed closure as "error messages" (calling the function would produce the message) but maybe this is overkill.

Closed due to #17


[Feature] Fix empty grid slots in renderer

There is a visual bug caused by adblockers, and (if this is a common issue) I think it would be a good idea to impliment a fix. Here's my solution, it works pretty well. LMK what you think.

I'm using violentmonkey, i'm not sure they support userstyle

Any ideas how I can fix the skeleton loader, or replace it with a more compatible one?


AVX encoded instructions wrong if register index is greater than 7 (REX)

Will be documented

Fixed by https://github.com/kobalicek/asmjit/commit/3c477177d2bcbbde25f405e30c2ca22393211d3e


:bug: fix reactive tracing

Description
Fix reactive tracing.
Reference
Issues closes #25

Did you take a look at the best practices mentioned in https://medium.com/@aakashkumar2001jha/mastering-reactive-tracing-in-spring-boot-3-a-comprehensive-guide-to-achieving-seamless-29c1229177ca? Are those applicable to our case?

Did you take a look at the best practices mentioned in https://medium.com/@aakashkumar2001jha/mastering-reactive-tracing-in-spring-boot-3-a-comprehensive-guide-to-achieving-seamless-29c1229177ca? Are those applicable to our case?

As far as i understood it they are only for logging and there is nor real benefit for us. But feel free to look over it again maybe you'll see it differently


Raspiraw support

I tried for 2 days now to modify core.py to accept files from raspiraw output but they turn out really blue and has terrible noise. Would it be possible to add support for raspiraw?
raspiraw: https://github.com/6by9/raspiraw
useful info about unpacking raspiraw files: https://www.strollswithmydog.com/open-raspberry-pi-high-quality-camera-raw/

You can refer to this issue here to see how this can be done. You shouldn't need to edit core.py.
#28

Thank you so much! I should've checked the closed issue's, my bad.

Would it be possible to add an example file that can convert using raspiraw files? I tried multiple times with #28 as reference but without any usable result. My dng's were either heavily purple tinted with a lot of noise or straight up black. Any help would be appreciated, thank you!

Would it be possible to add an example file that can convert using raspiraw files? I tried multiple times with #28 as reference but without any usable result. My dng's were either heavily purple tinted with a lot of noise or straight up black. Any help would be appreciated, thank you!

I'll look into creating an example script for this use case, can you send me a folder ( zipped ) of the raspiraw's you wish to convert? The header is also required so we can know whether the image is flipped or not to figure out bayer order.

Thank you very much for looking into it, the zip file can be found here.
Inside you will find a raspiraw raw file created with the following command:
~/raspiraw/raspiraw -t 100 -g 1010 -eus 5000 -hd0 /dev/shm/hd0.32k -o /dev/shm/out.%03d.raw
Included is also the separate header file and raspiraw raw file + header file combined using:
ls /dev/shm/.raw | while read i; do cat /dev/shm/hd0.32k "$i" > "$i".all; done
Also the combined file converted to a tiff using dcraw:
ls /dev/shm/.all | while read i; do ~/dcraw/dcraw -f -o 1 -v  -6 -T -q 3 -W "$i"; done
Finally also a jpg created with raspistill (in case you need a reference):
raspistill -t 100 -o raspistill.jpg -vf -hf
If you need anything else, please let me know. Thank you very much!

Thank you very much for looking into it, the zip file can be found here.
Inside you will find a raspiraw raw file created with the following command:
~/raspiraw/raspiraw -t 100 -g 1010 -eus 5000 -hd0 /dev/shm/hd0.32k -o /dev/shm/out.%03d.raw
Included is also the separate header file and raspiraw raw file + header file combined using:
ls /dev/shm/.raw | while read i; do cat /dev/shm/hd0.32k "$i" > "$i".all; done Also the combined file converted to a tiff using dcraw: ls /dev/shm/.all | while read i; do ~/dcraw/dcraw -f -o 1 -v -6 -T -q 3 -W "$i"; done
Finally also a jpg created with raspistill (in case you need a reference):
raspistill -t 100 -o raspistill.jpg -vf -hf
If you need anything else, please let me know. Thank you very much!

Below is a simple script that given a header file and a .raw file will output a DNG.
It's not perfect and lacks some metadata that the raspistill jpeg+raw contains so I will still advise if you can use that format to do that instead.
`
import ctypes
import sys
import os
import io
from pydng.core import RAW2DNG, DNGTags, Tag
import numpy as np
import struct, collections
hdr = "hd0.32k"
rawFile = 'out.001.raw'
class BroadcomRawHeader(ctypes.Structure):
fields = [
('name',          ctypes.c_char * 32),
('width',         ctypes.c_uint16),
('height',        ctypes.c_uint16),
('padding_right', ctypes.c_uint16),
('padding_down',  ctypes.c_uint16),
('dummy',         ctypes.c_uint32 * 6),
('transform',     ctypes.c_uint16),
('format',        ctypes.c_uint16),
('bayer_order',   ctypes.c_uint8),
('bayer_format',  ctypes.c_uint8),
]
BAYER_ORDER = {
0: [0, 1, 1, 2],
1: [1, 2, 0, 1],
2: [2, 1, 1, 0],
3: [1, 0, 2, 1],
}
file = open(hdr, 'rb')
img = io.BytesIO(file.read())
data = img.getvalue()
assert data[:4] == 'BRCM'.encode("ascii")
header = BroadcomRawHeader.from_buffer_copy(data[176:176 + ctypes.sizeof(BroadcomRawHeader)])
image specs
width = header.width
height = header.height
bpp= 12
load raw data into 16-bit numpy array.
numPixels = width*height
rf = open(rawFile, mode='rb')
img = io.BytesIO(rf.read())
data = img.getvalue()
data = np.frombuffer(data, dtype=np.uint8)
reshape, crop = {
1: ((1952, 3264), (1944, 3240)),
2: ((2480, 4128), (2464, 4100)),
3: ((3040, 6112), (3040, 6084)),                #raspiraw seems to omit rows vs the embedded raw in the jpgs, 3040 instead of 3056.
}[3]
data = data.reshape(reshape)[:crop[0], :crop[1]]
data = data.astype(np.uint16)
shape = data.shape
unpacked_data = np.zeros(
(shape[0], int(shape[1] / 3 * 2)), dtype=np.uint16)
unpacked_data[:, ::2] = (data[:, ::3] << 4) +
(data[:, 2::3] & 0x0F)
unpacked_data[:, 1::2] = (
data[:, 1::3] << 4) + ((data[:, 2::3] >> 4) & 0x0F)
data = unpacked_data
uncalibrated color matrix, just for demo.
ccm1 = [[6759, 10000], [-2379, 10000], [751, 10000],
[-4432, 10000], [13871, 10000], [5465, 10000],
[-401, 10000], [1664, 10000], [7845, 10000]]
ccm2 = [[5603, 10000], [-1351, 10000], [-600, 10000],
[-2872, 10000], [11180, 10000], [2132, 10000],
[600, 10000], [453, 10000], [5821, 10000]]
fm1 = [[7889, 10000], [1273, 10000], [482, 10000],
[2401, 10000], [9705, 10000], [-2106, 10000],
[-26, 10000], [-4406, 10000], [12683, 10000]]
fm2 = [[6591, 10000], [3034, 10000], [18, 10000],
[1991, 10000], [10585, 10000], [-2575, 10000],
[-493, 10000], [-919, 10000], [9663, 10000]]
ci1 = 17
ci2 = 21
set DNG tags.
t = DNGTags()
t.set(Tag.ImageWidth, width)
t.set(Tag.ImageLength, height)
t.set(Tag.TileWidth, width)
t.set(Tag.TileLength, height)
t.set(Tag.Orientation, 1)
t.set(Tag.PhotometricInterpretation, 32803)
t.set(Tag.SamplesPerPixel, 1)
t.set(Tag.BitsPerSample, bpp)
t.set(Tag.CFARepeatPatternDim, [2,2])
t.set(Tag.CFAPattern, BAYER_ORDER[header.bayer_order])
t.set(Tag.BlackLevel, (4096 >> (16 - bpp)))
t.set(Tag.WhiteLevel, ((1 << bpp) -1) )
t.set(Tag.ColorMatrix1, ccm1)
t.set(Tag.ColorMatrix2, ccm2)
t.set(Tag.ForwardMatrix1, fm1)
t.set(Tag.ForwardMatrix2, fm2)
t.set(Tag.CalibrationIlluminant1, ci1)
t.set(Tag.CalibrationIlluminant2, ci2)
t.set(Tag.AsShotNeutral, [[1,1],[1,1],[1,1]])
t.set(Tag.DNGVersion, [1, 4, 0, 0])
t.set(Tag.DNGBackwardVersion, [1, 2, 0, 0])
t.set(Tag.Make, "Camera Brand")
t.set(Tag.Model, "Camera Model")
t.set(Tag.PreviewColorSpace, 2)
save to dng file.
RAW2DNG().convert(data, tags=t, filename="custom", path="")
`

Thank you very much for taking the time! Do you know what header information is missing and causing this heavy noise?

You should use raspistill with the raw option turned on, instead of raspiraw. That's what they are doing in the article you posted.
The jpeg+raw from raspistill gives you extra exif information and makernote metadata that is used in the dng metadata.


Pull kit sensors throw exceptions on fatal errors

This PR adds code to the sensors to throw exceptions on fatal errors, errors that could not otherwise be differentiated from normal operation, and various range and operating errors.
There are still error conditions that can occur that will output error messages without throwing exceptions.  Maybe these can be addressed in the future if that behavior is desired.

Awesome contribution, merged with some changes. Thanks a lot for this!


Can I Fine-Tune the Diarization Model to Recognize a Specific Individual's Voice?

Hi @MahmoudAshraf97
I'm curious to know if it's possible to customize the diarization output. Specifically, can we assign a custom name, such as 'Mr. XYZ', to dialogues spoken by a particular person, while the rest are labeled as 'Person 0', 'Person 1', etc.?
Thanks!

It's doable but not through finetuning, you will use the intermediate embeddings generated from MSDD model and compare them to reference embeddings that you generated to identify which speaker is XYZ


Update overview.md

Added Network Service Mesh in the Supported Service Meshes.
Signed-off-by: Harshini<EMAIL_ADDRESS>

Closes https://github.com/layer5io/meshery-nsm/issues/1.


Adding unit tests for TestPrepareTransferTasksForWorkflowTxn

What changed?
Adding unit tests for TestPrepareTransferTasksForWorkflowTxn

Why?
code coverage increase.

How did you test it?
unit test

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 018e1bb5-1e90-45c8-ad90-2a60be3b917a
Details

0 of 0   changed or added relevant lines in 0 files are covered.
111 unchanged lines in 20 files lost coverage.
Overall coverage increased (+0.03%) to 63.472%

Files with Coverage Reduction
New Missed Lines
%

common/task/weighted_round_robin_task_scheduler.go
2
88.56%

common/task/parallel_task_processor.go
2
93.06%

service/history/task/transfer_active_task_executor.go
2
72.51%

service/matching/taskListManager.go
2
79.7%

common/persistence/statsComputer.go
2
95.71%

common/persistence/historyManager.go
2
66.67%

service/history/task/transfer_standby_task_executor.go
2
86.19%

common/persistence/executionManager.go
2
78.59%

common/membership/hashring.go
2
82.23%

service/history/queue/timer_gate.go
3
95.83%

Totals

Change from base Build 018e1bb3-a46f-4e6f-90af-e244a5deb9e0:
0.03%

Covered Lines:
93049

Relevant Lines:
146598

💛  - Coveralls


Update pycon-2011--ten-years-of-twisted.json

Set the "speakers" field correctly so this shows up on my list of talks.

Thank you for fixing this @glyph!

Hi @glyph , we still have to publish the changes from https://github.com/pyvideo/pyvideo; Will do it today

@glyph , done.

@glyph: I too was confused about this when I first contributed here.
Merging to master on the pyvideo/data repo doesn't do anything to the site. The site gets updated only when master on the pyvideo/pyvideo repo changes (afaik all the static pages get re-generated at that point).
The way to make the latest contents of pyvideo/data to go live is to bump the reference as @redapple did in https://github.com/pyvideo/pyvideo/pull/151
@redapple: Is this actually documented somewhere? Would it make sense to add this information from this comment to CONTRIBUTING.rst?


The new MediaPlayerElement crashes the app when using most of its transport controls

Describe the bug
I tested the new MediaPlayerElement that was just added to WinUI 3 (in the Windows App SDK version 1.2 Preview 1).
On my computer, the volume and seek bar are working. However, all the other buttons (Play/Pause, etc) cause the app to crash and exit with the code 0xc0000409.
Output when I catch the exception in VS:
'KlearTouchApp.MediaPlayer.WinUI.NuGet.exe' (Win32): Loaded 'C:\Windows\System32\mfps.dll'.

onecoreuap\windows\frameworkudk\ntadapter\mediaextensionsntadapter.cpp(37)\Microsoft.Internal.FrameworkUdk.dll!00007FFB4F424036: (caller: 00007FFB4F4847EA) FailFast(1) tid(6048) 80004002 No such interface supported

Unhandled exception at 0x00007FFBFC90B2DC (KernelBase.dll) in KlearTouchApp.MediaPlayer.WinUI.NuGet.exe: Fatal program exit requested.

So I suspect that this is caused by my computer running Windows 10 (20H2) which may be missing an API that the transport controls assume exists.
Steps to reproduce the bug

Download the updated sample in this repo.
Run it, then pick a file to play. It should start auto playing.
When you press on the pause button that is part of the native Transport Controls, the app will crash.

Note that the custom play/pause buttons at the top of the app do work fine. I have experimented with other features, and everything works programmatically.
Expected behavior
The native Transport Controls should work as expected.
NuGet package version
WinUI 3 - Windows App SDK 1.2 Preview 1: 1.2.220902.1-preview1
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
Windows 10 (20H2): Build 19042
Additional context
Using Visual Studio 17.3.0 Preview 1.1.
Update: I quickly installed the .NET SDK 6.0.400. Still crashing.

It looks like your theory is correct that this issue is specific to Win10.

With the release of v1.2, this issue is resolved.


_parse methods don't handle incorrect case of enum values

For example, _parseStatus works fine as long as the status provided in the list is lowercase, but fails (and defaults to unknown) in any other case.

Accepted standards are to use lowercase for the enum itself, so Color.red instead of Color.Red.


Change ctags fork?

The original ctags seems abandoned and has a lot of known, unfixed issues. Have you considered changing to a maintained fork like https://github.com/universal-ctags/ctags ?

I've got a version of symbols-tree-view working with the universal-ctags. It fixes Latex parsing for me and I'm quite happy with it. I've tested it on Linux.
Would you consider a pull request? Or is there another way that you would prefer? Or do you plan on staying with exuberant ctags?

@harmsk Thank you, it's very nice if you could submit a pull request.
I have to stay with the old version ctags because we need darwin/linux/win binary separately...
Maybe we should consider Linux firstly.


chore(ci) switch CI to github actions

No particular preference here, I usually use Coveralls for Lua coverage. Codecov may be better or worse I don't know, but if you want a sample of setting up Coveralls here you are:
https://github.com/sile-typesetter/cassowary.lua/blob/master/.github/workflows/busted.yml#L33
Note the luarocks install and passing the automatic token as repo identifier which is all you need for public repos.

We used to use coveralls in the past. I like it's GUI better than CodeCov (at least so far). But Coveralls has a flaw, it only retains 1 report per commit. That is the reason you quite often in Penlight PR's saw that Coverage failed because it went down. This simply depended on which job was the last one to send in its report.
I have hopes that Codecov does not have that flaw, and it will actually combine the reports per commit into a single report. In the case of Penlight it would mean to get combined coverage for the Windows and Unix based tests (that's where the main difference in coverage is coming from).

I'm pretty sure that was a bug in luacov-coveralls that I fixed a while back. It should be in v0.2.3.

ah... Nice!


Fix Webhooks on withdraw

Update does not work:

create a link with webhook
update it and remove the webhook url

Error:
TypeError: Assignment to constant variable

LGTM. See comments.

@talvasconcelos are we waiting on something for this PR?

closes #1

@talvasconcelos are we waiting on something for this PR?

I don't think so... LGTM!


Installing 1.6.7 Yosemite 10.10.3 fail

I've build it in deploy mode after installation fails, imgs:

Same here.

Same for me.

Third this and not seeing anything immediately relatable in console logs.
Built and installed both dev and dep versions to the same effect.
Checked kext perms - all ok.

Seems the code signing is the issue as indicated by kextutil:
Diagnostics for /System/Library/Extensions/Soundflower.kext:
Code Signing Failure: not code signed
ERROR: invalid signature for com.Cycling74.driver.Soundflower, will not load

One solution, at least temporarily until the code signing is fixed would be to disable kext signing in OS X as follows: (WARNING: know the risks of doing this before doing so, no warranty implied)...
sudo nvram boot-args=kext-dev-mode=1

Then restart your mac for these changes to take effect:
sudo shutdown -r now

Soundflowerbed should now work as expected.

+1
I'll wait for the fix
Thanks

2.0.2b from https://github.com/mattingalls/Soundflower/releases works for me

@kybernetyk and a helpful DMG installer too!
Kind thanks


Why sha256 hash become an issue on release PIN verify function?

can you explain more detail for this?
can't we just hash pin code then send with apdu command?

Because the spec is locked and owned by CHT.

A personal company owned a national card's spec. ridiculous

You can expect if new national ID would be locked into specified vendor or not?!

base on this I hope not.
but .. 英文不足以表達我心中的幹XX心情，雖然我只是要讀健保卡但我覺得這件事真的狗幹扯。

Sorry, that's my bad(Typo for years). The password is not SHA256 hashed. It is AES encrypted.

base on this I hope not.
but .. 英文不足以表達我心中的幹XX心情，雖然我只是要讀健保卡但我覺得這件事真的狗幹扯。

NHI Card has no issue on password as it did not have password.

Yes, I just assume the pin verify is standard command for smart card and search about MOICA card
and I was wrong , But I didn't expect the cause is the AES key owned by CHT

Yes, I just assume the pin verify is standard command for smart card and search about MOICA card
and I was wrong , I didn't expect the cause is the AES key owned by CHT

So was I .


Error during compiling ebpf with chart because of missing openssl headers

Describe the bug
When starting the container with BPF_USE_LOCAL_KERNEL_SOURCES for compilation:

Setting up /usr/src links from host
Running falco-driver-loader with: driver=bpf, compile=yes, download=yes
Mounting debugfs
Found kernel config at /host/boot/config-4.14.198-152.320.amzn2.x86_64
Using downloaded kernel sources for kernel version 4.14.198...
Downloading http://mirrors.edge.kernel.org/pub/linux/kernel/v4.x/linux-4.14.198.tar.gz
Extracting kernel sources
Configuring kernel
scripts/sign-file.c:25:30: fatal error: openssl/opensslv.h: No such file or directory
compilation terminated.
make[1]: *** [scripts/Makefile.host:102: scripts/sign-file] Error 1
make: *** [Makefile:570: scripts] Error 2
Trying to compile the eBPF probe (falco_amazonlinux2_4.14.198-152.320.amzn2.x86_64_1.o)
Skipping download, eBPF probe is already present in /root/.falco/falco_amazonlinux2_4.14.198-152.320.amzn2.x86_64_1.o
eBPF probe located in /root/.falco/falco_amazonlinux2_4.14.198-152.320.amzn2.x86_64_1.o
Success: eBPF probe symlinked to /root/.falco/falco-bpf.o
Sat Jan  9 14:48:32 2021: Falco version 0.26.2 (driver version 2aa88dcf6243982697811df4c1b484bcbe9488a2)

Expected result: compile without errors.
Actual result: compile errors about opensslv.h missing.
Even though this error happens it does start up correctly.
Adding the ssl headers (libssl-dev package) to the container should solve the issue.

Start falco with the helm chart on amazon linux and BPF_USE_LOCAL_KERNEL_SOURCES=1 in daemonset.env .

Falco version:

0.26.2

System info:

Cloud provider or hardware configuration:
Amazon EKS
OS:

amazonlinux2_4.14.198-152.320.amzn2.x86_64

Kernel:

amazonlinux2_4.14.198-152.320.amzn2.x86_64

Installation method:

helm, kubernetes, from source
Additional context

Hi
It startups correctly because the issue seems to be in the "Configuring Kernel" step, then - in the next step -  the compilation works and stores the eBPF probe in /root/.falco/falco_amazonlinux2_4.14.198-152.320.amzn2.x86_64_1.o.
However, this issue is not a helm chart problem, so I'm moving this issue to the main repo.

Hi
It startups correctly because the issue seems to be in the "Configuring Kernel" step, then - in the next step -  the compilation works and stores the eBPF probe in /root/.falco/falco_amazonlinux2_4.14.198-152.320.amzn2.x86_64_1.o.
However, this issue is not a helm chart problem, so I'm moving this issue to the main repo.


Docs continued

Header style still seems to be in flux. I'm not sure if I'm keeping this version either.
Changed comma style for Readerless, based on experience here.

I need to proofread the docs again, but as mentioned in #180, I have to do these doc updates incrementally or nothing will get done. I'm still in the process of moving example usages from the Whirlwind Tour to the API docstrings, so docs are still in an inconsistent state right now. This PR doesn't make any progress on that.


Feature Request: Support Worker Threads using Sharp

@lovell While I have read how Sharp itself would not benefit from worker threads in https://github.com/lovell/sharp/issues/1297, I am using Sharp alongside some CPU-intensive JS programming that certainly would benefit from being in a worker thread.
https://github.com/nodejs/node/issues/21481 explains the issue, and at the bottom is a reference to how it was solved in node-lzo

Is it worth still using nan or shifting to N-API?

Please see #1282 for N-API.

FYI for how iltorb does it: https://github.com/MayhemYDG/iltorb/blob/master/src/iltorb.cc#L7-L19

There's a discussion about adding this logic to nan at https://github.com/nodejs/nan/issues/844

+1. I am still suffering from some non-heap leakage at heroku for over a year now.
Would workers be a benefit for me as they eventually die?
I'm currently spawning a new child_process, which fixes the leak, but the overhead is unbelievable.
The request time went from 140ms to ~1500ms after this workaround.

I can user sharp in Worker Threads （sharp:v0.22.1,node:v10.15.3).
But when I require sharp in mainThread and workerThread both, I still get error: "Module did not self-register."

Commit 946d3c8 makes the switch to use the new NAN_MODULE_WORKER_ENABLED macro. This will be in v0.23.0.
I've flagged this as experimental in the changelog as although both sharp and libvips are very much thread-aware, this probably requires testing at scale before use in production.

sharp v0.23.0 is now available.

Hi,
Would anybody know by any chance if worker threads have their own libuv threadpool? In my tests, worker threads share the exact same sharp job queue, which is limited to 4 concurrent images (without setting UV_THREADPOOL_SIZE) and can even be queried from the main thread (sharp.counters()). Is there any way to give each thread its own set of processing threads? I have a code sample if someone is interested.

Hi,
Would anybody know by any chance if worker threads have their own libuv threadpool? In my tests, worker threads share the exact same sharp job queue, which is limited to 4 concurrent images (without setting UV_THREADPOOL_SIZE) and can even be queried from the main thread (sharp.counters()). Is there any way to give each thread its own set of processing threads? I have a code sample if someone is interested.
My current solution is to spawn more Node.js process, but threads would be lighter. I don't want to bug my user to set UV_THREADPOOL_SIZE either for a CLI tool.

@MarcusCemes worker threads all share the same thread pool, you can't much about it. You could try to use N workers, and in each worker set sharp concurrency to 1, but I don't think the performance would be better.


Amazon.AWSCLI version 2.15.38

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

2024-04-12T23:25:58.2456043Z ##[error]Manifest Validation Failed
2024-04-12T23:26:03.2897062Z ##[section]Finishing: Validate Manifest

(Automated response - build 876.)

Close with reason: Merge Conflicts;


fix: union_distinct shouldn't remove child distinct

Which issue does this PR close?
Closes #.
Rationale for this change
original code will remove distinct when they are children of union.
But we should't remove them, because

distinct can decrease cardinaty/rowcount of union input
sometime distinct can pushdown, such as distinct can pushdown through join.
even if we want to remove distinct, we should create rule instead of eliminate distinct in plan builder

What changes are included in this PR?
union_distinct shouldn't remove child distinct
Are these changes tested?

Are there any user-facing changes?

I will add other rule after this rule, such as MergeUnionRule to combine multiple union


typo with file

Hi @viresh-arm, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@viresh-arm : Thanks for your contribution to the Azure documentation! The author, @kundanap, has been notified to review your proposed change.

#sign-off @viresh-arm thank you for fixing that issue!

Only the author of the article can use the sign off comment to add the ready-to-merge label.


Run AArch64 build during CI

This PR adds six steps to the Jenkins CI pipeline:

Build for AArch64 on Ubuntu 16.04 (Debug)
Build for AArch64 on Ubuntu 16.04 (Release)
Build for AArch64 on Ubuntu 16.04 (RelWIthDebInfo)
Build for AArch64 on Ubuntu 18.04 (Debug)
Build for AArch64 on Ubuntu 18.04 (Release)
Build for AArch64 on Ubuntu 18.04 (RelWIthDebInfo)

These builds are run inside containers running the corresponding oetools-full-* images. Once #1795 is merged, the Ansible scripts used to set up the Docker images will be updated to include AArch64 build toolchains.
This has all been tested outside of the master branch as can be seen here.
Dependencies:

#1795 must be merged;
The Docker image build job must run with TAG_LATEST=TRUE.

bors r+


MSI GE70 issues

I tested this code on MSI GE-70 in a static color mode and found two issues:

The 'green' color is not actually green. It looks more like yellow. Also, original control software do not limits user to a set of colors, it allows selection from full RGB space. Maybe there is a protocol that accepts RGB values?

Hardware backlight button acts like hardware backlight switch. But original driver hooks this button and change backlight layers on press.

Unfortunately, at this time I cannot reverse a keyboard protocol from original Windows drivers. I cannot get these drivers working in VirtualBox.

There is no 'protocol', it is just a HID device which accepts numbers as colors.
The windows software has a key-mapping from the RGB to number values.
Refer to this list for found colors: https://github.com/wearefractal/msi-keyboard/issues/11#issuecomment-72904478
The hardware button is dependent on the version of the keyboard and the MSI motherboard controls. My GS-60 ghost pro only has illumination levels from off to high. My MSI GT-60 had the modes regardless of OS.
Since this is a node module for accessing the interface and allowing customization I would not add the RGB conversion to it directly, but rather to a GUI project.
Would you be interested in creating an electron GUI app for the keyboard? I have been planning to do so, but have not had the time.

When I refered to a 'protocol', I refered to a format of HID reports. Thanks for a list of colors, so some facts that I found on a my laptop:

Green is a color 10, color 4 looks more like yellow on my laptop. Maybe this is keyboard-dependent.

Levels are inverted, so 'high' (0 as defined in constants.json) is most darkest level, and 'light' (3) is most bright.

Red (1) at brightness 1 is red and red (1) at brightness 3 is pink, but red (9) is same color with different brightness. Again, this may be keyboard-dependent.

I will try to reverse color alghorithm from original Steelseries software, but there is no guarantee that I will succeed in it


Gulp + cleaning

Add gulp task
minifier
less
es6

Codecov Report

Merging #247 into master will increase coverage by 1.33%.
The diff coverage is 85.2%.

@@            Coverage Diff             @@
##           master     #247      +/-   ##
==========================================
+ Coverage   83.81%   85.14%   +1.33%
==========================================
  Files          18       18
  Lines        1081      700     -381
  Branches      179      134      -45
==========================================
- Hits          906      596     -310
+ Misses        175      104      -71

Impacted Files
Coverage Δ

src/controller/directory.js
100% <ø> (+16.66%)
:arrow_up:

src/controller/auth.js
100% <ø> (+18.98%)
:arrow_up:

src/controller/logs.js
100% <ø> (+40%)
:arrow_up:

src/database/client.js
100% <ø> (+28.76%)
:arrow_up:

src/controller/torrent.js
100% <ø> (+16.66%)
:arrow_up:

src/worker/filetransfert.js
100% <ø> (+32.25%)
:arrow_up:

src/controller/config.js
100% <ø> (+20%)
:arrow_up:

src/controller/filetransfert.js
100% <ø> (+19.04%)
:arrow_up:

src/worker/config.js
100% <100%> (ø)
:arrow_up:

src/worker/log.js
100% <100%> (ø)
:arrow_up:

... and 20 more

Continue to review full report at Codecov.

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


0.4.7 not working

Hello,
It is not working after update to 0.4.7
I connect sftp to my server.
uploading progress bar is never end.
My atom version is 0.177 and OS is Mac 10.10.2

+1 me too

+1
The upload bar never stops and the remote file gets truncated (zeroed).
Conecting, listing files and downloading is working fine.
Tested on my Mac (Yosemite) and on Windows 7. Both with latest Atom 0.177.

It isn't working for me either.  Just tried again with . 0.4.8 and no luck.

Can you guys open up the console CTRL+ALT+I (on Windows) and post the error here?

+1 me too

Can you guys run the following snippet in your atom's developer tool? Paste the resulting line.
atom.project.remoteftp.connector.put(
    "/path/to/file/you/want/to/upload",
    function () { console.log('Completed', arguments); },
    function () { console.log('Progress', arguments); }
);

This line will try to upload a file and will print some debug in the console.

@mgrenier New plugin version 0.6.0 is working fine. Thank you for the update! :+1:
I'm using Atom 0.182.0 in Windows 7.

@luizpaulo165 @brocknessmonster @theRichu @prugel Is it fixed on your side too?

I do not have this problem. I am using a macbook now.It is working properly.


navbar support for email/password users

add the word "Account" next to avatars for all accounts and remove the GitHub names


fix typo in route53 file for dnsNameservers-zoneNameservers mismatch

What this PR does / why we need it:
It fixes a typo in the code when an error condition becomes true
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

NONE

Thanks @plitha, can you make sure to sign off the commit?


[wip] feat: padding for preventOverflow and flip

(Defaults are for testing - 0 is the new default)

Don't really know what you want the options destructuring etc to look like
I don't know how flip + preventOverflow paddings being different for a single side should work

By the way I have no idea if the types are correct because idk how to use Flow in VSCode 😳

By the way I have no idea if the types are correct because idk how to use Flow in VSCode 😳

You can install this extension https://github.com/flowtype/flow-for-vscode
Or just run yarn flow in the repository root.

There are some flow errors but idk why

I manually merged it and fixed the type errors. Thanks!

You accidentally kept the non-0 default value for padding

Fixed! Thanks!


[FEATURE] Create a pull request for the Clang Tidy

Detailed description
Currently, all of the Clang Tidy changes are being pushed in forks and appearing in other pull requests. However, these changes are not being pushed to the original repository due to the main branch being protected. This can be fixed by creating a pull request (running the action only on this repository) and then the maintainers merging the PR to reflect the changes.
Context
This will make it better for both PR creators and reviewers.

PR creators won't have unwanted changes in their PRs.
Reviewers won't be confused about other changes in PRs.
All changes will be pushed to the original repository, and all missed things will be fixed.

Possible implementation
Create a pull request instead of committing directly.
Additional information
We could also make the check fail if the user has missed anything and let the user fix the errors themselves.
It depends on what you like most. Having a PR created makes it easier and faster. Having the user fix the errors themselves helps them learn.
Let me know your thoughts.

this seems like a good idea


assign layerOptions first

One of the preconditions for entering this block is that newOverlayLayer[name] is not defined.  So, it is a mistake to pass a property (layerOptions) of that object to safeRemoveLayer :bug:

@elesdoar can you review this?

The problem is fixed in the PR #933, You can close it.

Thanks but @elesdoar got to it.

Glad it got resolved!  Thanks @elesdoar

I need to checkout other issues prior to cutting a release. But I'll try to get to it soon.


Update: Adjusted deprecated dockerfile/nodejs to node:0.10.38 trusted

Good call, must've missed this one ;)

@b00giZm No worries, thats community for you!


About data processing pipeline

Will you disclose the thresholds for data filtering in data processing later(ocr,match score)

Not for now since we use different thresholds for different datasets. We are still trying to work out which threshold is better.


Do not push down Sorts if it violates the sort requirements

Which issue does this PR close?
Closes https://github.com/apache/datafusion/issues/11675
Closes https://github.com/apache/datafusion/pull/11493
Rationale for this change
See https://github.com/apache/datafusion/issues/11675 -- the PushDown sort optimizer was incorrectly pushing down sorts in certain cases
This change fixes a problem we were seeing in our tests downstream in influxdb iox
What changes are included in this PR?

Don't push down sorts if doing so would violate the ordering requirements
Add tests

Are these changes tested?
Yes, new unit tests are added
Are there any user-facing changes?
Bug fix

Thank you for the review @mustafasrepo


Images with aria-hidden are getting flagged for missing alt text.

It's my understanding that an img tag with aria-hidden="true" set doesn't need alt text (since it'll be invisible to screen readers anyways). The "Image alt-text" plugin flags such images anyways though.

Yep you're right. We cycle through $("img"), but should be filtering out anything hidden like ADT does.
In reality we should probably just call audit("imagesWithoutAltText") from ADT instead of writing our own logic. We did this to get some extra customizability (showing different annotations for presentational images), but we can probably do that after the fact.

FWIW role="presentation" or alt="" might be your solution here, but it shouldn't be failing regardless.


Translate camera with middle mouse button

Hello,
First may I say, this project is incredible! What you've achieved is very impressive and a pleasure to use.
I wanted to ask if it would be possible to add translation of the camera when holding down the middle mouse button. This would make the experience much more familiar to PyMOL users.
All the best,
Harry

Thanks for the kind words. I'm glad you like it.
I've added support for middle-mouse panning in one of the last commits. Can you test if this does what you want?
Best,
Marco

Apologies for the delay in my response.
I've got the middle mouse panning working in the demo running the serve command, but not when emulating the "setting up a small website" from the documentation. The viewer works but middle mouse panning doesn't; am I missing something? I tried including:
<script type='text/javascript' src='./pv/src/mouse.js'></script>

But this didn't help. I tried using both bower and by cloning the git repository.
Thanks again for your help :).

Hey,
sorry for the late reply. I've had a few very busy days and didn't have time to look into your problem until now. Can you try again? I haven't updated the bio-pv.min.js when I committed the changes, that's why you were most likely using a version that didn't yet have the mouse adjustments. The version which is committed works for me in a small demo.

Hey,
No problem, same here! Am preparing for a presentation in which I intend to use (and advertise :D) PV.
I updated to the latest release and it's working now, thank you so much! Am looking forward to continuing to use PV :)


AMT 12 dropping CIRA when sending command while system is off

Toolkit version: 1.3
AMT device version: 12
Steps to reproduce:

Send AMT command to power off system (either REST call or through UI)
Once device is off, send any other command (get powerstate, refresh device page on the UI and re-pull hardware)
In MPS logs, will see the device disconnect and be removed after several seconds

Sometimes the device will reconnect after a minute or two, other times you need to completely power off the device (pull power or 5 second hold on the power button).  Once you power the device back on, it will reconnect to MPS.
Have tested this on other AMT versions (11.8) and this behavior doesn't happen.

Vyshali needs to test with AMT 12 system and see if issue is reproducible.

Not able to replicate the issue on latest FW systems using v 1.5 release.


Priority #1 - Show Stopper Bug - View Rendering engine binds incorrect data to controls in View after DELETE operation

Rewording this bug completely
There is a problem with the way ASP.NET Core MVC (v 3.0) renders collection items in an editor scenario. This is a serious problem. I wish someone would investigate this on priority and provide a fix.
Here is a scenario:
The model that is being bound to the view has a collection-style property. In my case, it is a List<T>. The properties in the classes are simple types -- string, DateTime and int.
Here are the two classes:
public class Trip {
    public string Name {get; set;}
    public string Description {get; set;}
    public List<Traveller> Travellers {get; set;}
}

public class Traveller {
    public string Nickname {get; set;}
    public string Name {get; set;}
    public DateTime DateOfBirth {get; set;}
    public int Gender {get; set;}
}

Trip is the class I am binding in the view. I have tried two different ways of binding the view.
Method 1: Custom Editor
I created a custom editor. The controls I added were via simple @Html.TextBoxFor().
Method 2: Manual binding
I wrote a for loop (using indexing) and bound controls using the same @Html.TextBoxFor() again.
When binding normally, it looks like the binding happens properly. However, you will see the bug when you try to remove data.
In my case, I have a button on the page that binds to a RemoveTraveller action on the controller. In the action, I can see that the required row is being removed correctly. This is the code:
model.Travellers.RemoveAll(t => t.Nickname.Equals(nickname));

where nickname was the nickname being removed (passed in via asp-route-nickname on the remove button on the view).
THIS is where things get interesting.
After the delete, when the collection is being bound, the data that you actually removed is still shown on the UI. If there was only one element in the List, you would need to delete it TWICE to cause it to be removed from the UI.
VERY IMPORTANT
If you debug the View's binding code, you can SEE that the data being bound is DIFFERENT to the data that you see after the rendering is complete !!!
@Pilchie  - Can I send you more info via e-mail?

I have debugged it even at the HTML side. I have this statement:
@Html.EditorFor(m => m.Travellers)

(Travellers is a List<Traveller>, where Traveller is the equivalent of my Poco1).
Even at that point, in the Immediate Window if I do Model.Travellers[0], I see the right data. But on hitting F5, it renders completely different data!

One more piece of information:
If I use a for (indexed) loop and bind this way:
@Html.TextBoxFor(m => m.Travellers[index].Nickname)

The bug is triggered. But if I use a foreach (iterator) loop and bind using:
<input id="Travellers_@(index)__Nickname"<EMAIL_ADDRESS><EMAIL_ADDRESS>/>

Then it works correctly. So this seems to be something to do with indexed collections?

@Pilchie - Can I send you more info via e-mail?

Of course.  Also tagging @mkArtakMSFT

@mkArtakMSFT - Let me know what info you need.

@sujayvsarma could you share a minimal repro app?

Create a simple ASP.net Core MVC application.. No connections to anything or external packages.
Add one controller and a view. Add the two classes to Models. Bind the view to the Trip class. Hook up add & delete actions in the controller for the Traveller binding.
You should see the bug.
I'll take me 12 hours to send you a repro project. It's 10.30pm here now. About to go to sleep.

I have currently unblocked my team by putting through this workaround. But it is not sustainable.

Attaching the repro application. This is the exact application in which the bug occurs, I have removed portions of the code and system that is out of scope of this issue.
Project: AspNetCore.Issues.13950.zip
How to view the bug:
When you currently F5-run it, you will see a trip planner UI. This UI uses the workaround and will not show you the bug. But you can add/remove rows from the Travellers section on the page and see that it works correctly.
Now, open Views > Trips > PlannerView.cshtml.
Comment lines 56 to 112.
Uncomment lines 114 to 145.
F5-run. Now add and remove travellers. You will see that the remove part does not work correctly and shows the behaviour detailed above. Notice the difference in how the two blocks of code perform their binding (i.e., between 56-112 and 114-145). I believe the key to the root cause of the problem in somewhere there.
Let me know if you need additional details.

@dougbu the problematic case uses EditorFor, so my guess is that removing the item from the model isn't enough since it's still in ModelMetadata. It's basically the same as this: https://github.com/aspnet/Mvc/issues/2509.  That said, ModelState.Remove($"List[{ i }]"); works for one instance, what's the way to do this for a complex type with sub properties.

@sujayvsarma, the answer @pranavkm provided is correct. The view intentionally binds information from the model state to allow round-tripping of values that fail validation. And, the way to prevent that binding is to remove information from both your model and ModelState.
After changing your sample to use a valid namespace, I can't reproduce an issue when using @for or @foreach. Not sure what's happening with @Html.EditorFor(...); it seems to display just a GUID. But, I expect it's possible to make it display elements just deleted from the List<Traveller>. Put another way, the ModelState will still contain information about Travellers[1] after the list is shorted to a single element.
In any case, thank-you for your feedback. We're closing this issue as the questions asked here have been answered.

@dougbu - So this is a "Bug by design" ? Why should developers using the framework have to care whether the data is in the ModelState or not? This is called "Ridiculous Framework Design".
Also, what @pranavkm provided is NOT an answer. It seems to be some thought process. There is no clear instruction or example on how to actually implement this.
How can you close such a bug without a proper way forward?
For your urgent attention @Pilchie -- This goes back to my feedback on your feedback systems!

As I said, the design is intended to handle the normal case of editing after entering invalid data by default (without additional developer thought). Removing data is much less common and requires a small additional step.

@dougbu  Are you trying to tell me that "Delete Item" from data-bound lists is not a common scenario in the wild ? If so, you are absolutely wrong.

No, I'm saying it's less common than editing (or adding) and thus takes an additional gesture

It is not less common. Anywhere there is an "add", there is a "delete" in some way. Even if it is just a soft-delete in the back, it would always be a requirement to remove it from what is shown on the UI. This spans application types from very simple apps to gigantic ERPs.
I fail to understand what kind of a user study gave you the go-ahead to penalize developers that need to do the "remove item" workflow, whether you consider it common or not. To me, as a developer, I put stuff into a Model or remove it from the Model. When I bind that model to data, I expect the engine to show exactly what is in the model. THAT there is another ephermeral thingy called the ModelState (or whatever!) should be transparent to me, it is a vignette of YOUR implementation. Why should I care about ModelState at all as a consumer of that API?

To clarify: Removing data (an item) from a collection while displaying the entire collection is much less common than other create / update / delete operations.

Actually, your application is very unusual in that the data exists only in the view. Data is normally stored somewhere farirly permanent. This means the entire collection is almost never bound in delete operations -- only the id is necessary to update the stored model.

@dougbu - I think you are talking about applications that were being written 5 years ago. Nowadays, it is the age of "database-less applications". Store data only when it needs to be stored. There are way too many problems and costs involved with storing data nowadays, so this is the approach that is being increasingly adopted.
The application I pulled that code extract from is a travel-domain application that has zero databases. It works entirely off web services. The UI the controls were part of is an itinerary builder. Why would I store it anywhere? It will be sent off to the booking engine to create the tickets/etc.

Removing data (an item) from a collection while displaying the entire collection is much less common than other create / update / delete operations.

It most certainly is not. There are hundreds of thousands of applications out there that does this. Perhaps they don't use asp.net because of this very bug.