Warning panel - Align text to top of icon if over one line

What is the context of this PR?
Fixes #765
How to review
Adjust screen size on warning panel example and observe the vertical alignment of the text move from centre (one line) to top (two lines or more).

Seems to be aligning with the top of the exclamation mark but not the icon

Seems to be aligning with the top of the exclamation mark but not the icon

I had this fixed already. There was no need for a new PR.


Add more datasets during Hacktoberfest

Welcome Hacktoberfest-ers!
If you know of any JSON datasets and need more Pull Requests for the limited edition GitHub/@DigitalOcean t-shirt then look no further. If you don't know of any JSON datasets, all good, you can find many using your favorite search engine or just look through the current list and find missing/broken endpoints.
ProTips:
📖  Make sure to read the contributing guidelines
🐦  Tell your friends about this issue on Twitter
ℹ  Learn more about Hacktoberfest here: https://hacktoberfest.digitalocean.com

I would to collab

What is JSON? I read about it here: https://en.wikipedia.org/wiki/JSON#Example :)

Hey everyone! 😄
Added Github Meta JSON in PR #33

Nice we got featured in the last Hacktoberfest newsletter!:

Great job everyone. =)

Just saw you in the news letter and thought I would add what I thought of.  Just submitted pull request for a movie section.
Great list everyone!

Hockey stick star growth since Hacktoberfest

What about different versions of games? For example, there are a hundred different versions of Monopoly. Should those be submitted or not?

https://github.com/jdorfman/awesome-json-datasets/pull/156

Huge thanks to all that participated in this years Hacktoberfest! I will be merging the open PRs ASAP so don't think I am just ignoring yours 😉
Have a great Halloween! 🎃 👻


goploop install failed

Hello!
I try to install this pacage on openvz node, it have all from - https://github.com/kolyshkin/goploop#prerequisites .
But when I try to install this lib it have error -
go get github.com/kolyshkin/goploop
# pkg-config --cflags --static ploop
Package ploop was not found in the pkg-config search path.
Perhaps you should add the directory containing `ploop.pc'
to the PKG_CONFIG_PATH environment variable
No package 'ploop' found
pkg-config: exit status 1

But I dont see ploop.pc file and etc -
rpm -ql  ploop ploop-lib ploop-devel
/sbin/mount.ploop
/sbin/umount.ploop
/usr/sbin/ploop
/usr/sbin/ploop-balloon
/usr/share/man/man8/ploop.8.gz
/usr/lib/tmpfiles.d/ploop.conf
/usr/lib64/libploop.so.1
/usr/lib64/libploop.so.1.13.2
/var/lock/ploop
/usr/include/ploop
/usr/include/ploop/dynload.h
/usr/include/ploop/libploop.h
/usr/include/ploop/ploop1_image.h
/usr/include/ploop/ploop_if.h
/usr/lib64/libploop.a
/usr/lib64/libploop.so

On node  -

Centos 6
kernel  -2.6.32-042stab108.7
vzctl version 4.9.4
ploop version  -

rpm -qa ploop*
ploop-1.13.2-1.x86_64
ploop-devel-1.13.2-1.x86_64
ploop-lib-1.13.2-1.x86_64

go version - 1.5

.pc package for ploop is only added in the latest version -- please upgrade ploop to 1.14.1


Ignore Speedwire-Packets with no measured values

I have changed the speedwire.py  so that only the packets with measured values are processed.
should fix https://github.com/kellerza/hassio-sma-em/issues/27

Thanks @littleyoda!
Let's only change dev to start with. For now I'll merge so that I can test it locally (you will have to delete/re-install to get an update)
Not sure we should log this if it happens every minute?


Highlight error for function named 'is'

VSCode Version: 1.3.1
tsc: 2.0.0
OS Version: Windows 2012 R2

This issue was moved to Microsoft/TypeScript-TmLanguage#229


Rewrite osTicket frontend in React

Summary
Make osTicket frontend modern using React
Intended Outcome
osTicket current UI is straight outta 2000s - the goal is to have a modern and responsive UI.

When should be ready this new UI ?

@leguennecj
We have no set release date at this time. You will see the progress here as it progresses.
Cheers.

Hi team, we are looking to implement osTicket in our company and we are trying to find is there is any 'estimated' release date for the new version. We tested and even purchased the osticket awesome theme, but it still does not feel up to the level we need. I know you mentioned back in early November that no date have been released, but is there any indication if the release will be winter, spring of summer 2022? Unfortunately, at present time we are looking to implement Zoho desk. I fear once implemented, we will never come back.... Thanks for any update, if possible... M

And please don’t open-source it. Thanks.
Craig

And please don’t open-source it. Thanks.


Move DBnull scrubbing from C# type to PoweShell

I have moved from using a C# type to handle DBNulls to using PowerShell. I used the same logic as the C# function. I mainly did this because C# types do not work in AWS lambda functions. But, I believe it is more readable for PowerShell folks. I believe this also cleaned up the code.

Hiyo!
I like the idea, particularly for environments that preclude using C# this way.  That said, two suggestions!

It might be worth adding a new -As option.  I'm bad at naming, but for example, PSObjectViaPS or some other variation.  This way the function would continue behaving exactly as before, with flexibility for folks in more restricted environments
Look into optimizing your code as much as possible - given how often this code will run for certain returned data, any small differences can add up, quickly.  If you kept the Add-Member methodology, for example, you could swap $PSObject | Add-Member -MemberType NoteProperty -Name $column.ColumnName -Value $value for Add-Member -InputObject $PSObject -MemberType NoteProperty -Name $column.ColumnName -Value $value to avoid the costs that come with pipeline use

Cheers!


Problem to randomize YKW rom

Hi, I seen your YKW Randomizer and I wanted to try it but I don't have the "right rom". Could you can send me the link of your Discord Channel please ? Like this, I will be able to randomize the YKW rom.

Hi. You should read the README.md (available on the first page : https://github.com/Tiniifan/YokaiWatch1Randomizer) before opening an issue because the discord link is written in the README.md

Excuse me. I didn't Seen !😅😭
Le lun. 12 déc. 2022, 12:17, Tinifan @.***> a écrit :

Hi. You should read the README.md (available on the first page :
https://github.com/Tiniifan/YokaiWatch1Randomizer) before opening an
issue because the discord link is written in the README.md
—
Reply to this email directly, view it on GitHub
https://github.com/Tiniifan/YokaiWatch1Randomizer/issues/1#issuecomment-1346301082,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4V7TEEKVQZQ7BUFVZY3Q2DWM4CSXANCNFSM6AAAAAAS3A473E
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Sa me met cela quand je fais le tuto jusqu'au randomizer


BUG: Dropdown for adding users show deleted users as 'Anonymous'

My database contains lots of deleted users.
Is it in fact necessary to retain deleted users at all?

This is part is still based on original project. The users should not be visible to anyone, they should not be able to login. But kept in database because they may have other related data (ie. added comments, issues, ...etc).

Ok.  I suggest we simply delete them then - along with all references as owners or commentors and display missing references as "Anonymous" instead.

Not sure I understand.

I think deleted users should be deleted for good and not marked as deleted.
With them you should also remove the references to those users.

Then you loose everything they have done. We have developers joining the company. Work on issues make comments and upload files .... When these developers leaves the company, if we delete everything they have done, then we loose all the info.
You need the database records, it just need to be not selectable.

When logged in as manager and assigning users to a project I see lots of anonymous entries.
I do think a manager should always see a complete list of names without any reference to the public status of a user.

Looks good now. Thanks and closed.


Hook up Travis CI.

Soon.

Is this dependent on #15?

There's some challenges running the tests because the binary has to be run. I haven't used Travis much before so I'll need to research if that's even something possible.
A good first step is just making sure everything builds on our desired platforms for each commit.

It may be a good idea to add an after_success hook which builds the documentation and pushes it to GitHub Pages so users can browse it online instead of building locally. I do something similar for most of my projects.

@Michael-F-Bryan I like this idea. I'll have to check on the status of our travis work on Monday.

I created initial version of Travis config which works for Linux builds, see #187. Probably OS X support will be introduced later.
From my point of view the real problem of Travis is outdated build environment (here we choose between Ubuntu Trusty (12.04) and some even older version). That cause a lot of problems for modern C++ (that could be clearly seen in the pull-request). So I am not sure that tests and SC2 can be used inside a Travis Virtual Machine without additional magic.

I implemented os x support too. Feel free to reuse the code:
https://github.com/alkurbatov/suvorov-bot/blob/master/.travis.yml

@AnthonyBrunasso Travis build already exists.
https://travis-ci.org/Blizzard/s2client-api


Please tag 0.43.1

Version 0.43.1 was published to crates.io as part of #7994 but there is no corresponding tag. Thanks in advance!

Commit 5b129c4fed8a5cf1e8834c49add647740f536259 for reference.

Apparently this commit is dangling so you'll have to git fetch origin 5b129c4fed8a5cf1e8834c49add647740f536259 directly

The commit appears to be the tip of the rust-1.42.0 branch?

Anyway, tagged 0.43.1 at that commit, and pushed; closing.

Thanks! Yes my mistake, I forgot about side branches when pulling.


AttributeError: 'NoneType' object has no attribute 'events'

Hi I am not using docker, getting below error. Can you please take a look.
(dagda) ubuntu@ubuntu-Latitude-E6420:~/dagda/dagda$ python3 dagda.py start
<2019-07-06 10:46:02,709>   <docker_driver> <init:39>
<2019-07-06 10:46:02,778>    main:46 <'NoneType' object has no attribute 'events'>
Traceback (most recent call last):
File "dagda.py", line 35, in main
r = execute_dagda_cmd(cmd=cmd, args=parsed_args)
File "/home/ubuntu/dagda/dagda/cli/dagda_cli.py", line 43, in execute_dagda_cmd
ds.run()
File "/home/ubuntu/dagda/dagda/api/dagda_server.py", line 93, in run
docker_daemon_events_monitor.run()
File "/home/ubuntu/dagda/dagda/analysis/runtime/docker_events_monitor.py", line 45, in run
for event in self.docker_driver.docker_events():
File "/home/ubuntu/dagda/dagda/driver/docker_driver.py", line 114, in docker_events
return self.cli.events()
AttributeError: 'NoneType' object has no attribute 'events'
Serving on http://localhost:5000
<2019-07-06 10:46:02,836>   <dagda_server> run:104
<2019-07-06 10:46:02,837>   <dagda_server> run:105

Hi @ranvijay-sachan ,
Sometimes this error occurs when docker version and the pyhon library for docker don't match.
I suppose you have the last version of Dagda and its dependencies but, what are your os and docker version?

Hi @eliasgranderubio thanks for the reply.
I am using dagda.py 0.8.0 , Docker version 18.09.7, build 2d0083d and os ubuntu 16.04.
Do i need to change anything?


Discriminators do not obey writeOnly

I wanted to use a discriminator as part of a POST call, but not include the discriminator property as part of the response.  I added writeOnly to everywhere it was legal to do so, but the property still shows up.  This is also true for the properties on the discriminated object which are also marked readonly.
In the below screenshot, optionsType is a writeOnly property.

Here's a reduced test case:
openapi: 3.0.2

info:
  title: Discriminators do not obey `writeOnly`
  version: v1
servers:
  - url: http://localhost

paths:
  /v1/items:
    post:
      summary: Create an item
      requestBody:
        required: true
        content:
          application/json:
            schema:
              $ref: "#/components/schemas/Item"
      responses:
        200:
          description: OK
          content:
            application/json:
              schema:
                $ref: "#/components/schemas/Item"
components:
  schemas:
    Options1:
      type: object
      allOf:
        - $ref: '#/components/schemas/Item'
        - properties:
            options1Property:
              type: boolean
              writeOnly: true
              default: true
    Options2:
      type: object
      allOf:
        - $ref: '#/components/schemas/Item'
        - properties:
            options2Property:
              type: boolean
              writeOnly: true
              default: false
    Item:
      type: object
      discriminator:
        propertyName: optionType
        mapping:
          Options1: "#/components/schemas/Options1"
          Options2: "#/components/schemas/Options2"
      properties:
        id:
          type: integer
          example: 2194
          readOnly: true
        optionType:
          type: string
          writeOnly: true
          enum:
          - Options1
          - Options2

EDIT: Forgot to mention I'm using the official CDN hosted version of v2.0.0-rc.4

Seems like property used as discriminator cannot have modifiers such as readOnly or writeOnly.
As specifications says:

While composition offers model extensibility, it does not imply a hierarchy between the models. To support polymorphism, the OpenAPI Specification adds the discriminator field. When used, the discriminator will be the name of the property that decides which schema definition validates the structure of the model. As such, the discriminator field MUST be a required field.

So if you're using discriminator, it should be presented in your schema.

Thanks for the explanation.  What's interesting is that none of the validators/linters seem to complain about this, but that's a completely separate issue.
Sorry for the noise.

Awesome, no worries.
Thanks, @neluzhin!


Fix Uncaught ReferenceError: videojs is not defined

Remove $1 in order to fix #1

Resolved by @zarqman #13


Spaceship Game using Pygame

Describe the solution you'd like
This script will let the user play a multiplayer spaceship game built by using Pygame library of Python
I am GSSoC 21 paricipant .
Kindly assign @avinashkranjan

@iamakkkhil assigned to you

@iamakkkhil updates on this?

@iamakkkhil updates on this?

Almost done with the game will put the PR today itself


translate suggest-improvements into chinese

I saw that suggest-improvements.md has not been translated to Chinese, so i want to try to translate it.
if you find anything wrong or weird, please let me know.

@lexmovrx Please include the original English text in comments for review and maintenance. Thanks.
/hold

@tengqm ok, https://kubernetes.io/docs/contribute/suggest-improvements/

@lichao42 Please see how other translation embeds the original English text. E.g. https://raw.githubusercontent.com/kubernetes/website/master/content/zh/docs/tasks/access-kubernetes-api/configure-aggregation-layer.md

/check-cla

@lichao42 Please see how other translation embeds the original English text. E.g. https://raw.githubusercontent.com/kubernetes/website/master/content/zh/docs/tasks/access-kubernetes-api/configure-aggregation-layer.md

oh, i see.

@tengqm Can I close this PR? I'll revise the document and resubmit the PR request

/check-cla
@tengqm Hi, I just submitted a pull request, please review.

/lgtm
/check-cla
Don't know why you lose your CLA signing status.

/approve

/hold cancel

/approve
CLA status checking is flaky ...

/check-cla

/check-cla

I'm so sorry, I changed my email and username,so cla check failed.


Encryption

I am very much interested in adding encryption to my MongoDB database and I was wondering if there was a way for me to easily do so with mongoist.
EDIT: Even exposing a concept like setters/getters would work for me.

I suppose I could do this myself at the document creation stage. Hmm...


Timestamp in GPRMC vs {fixTime} in forward.url

hi,
I see the following gprmc statement in the forward.url
$GPRMC,052714.000,A,0119.7256,N,10352.6082,E,0.00,0.00,030416
but then I also pass the {fixTime} in the forward url then the value received for the same sentence is
fixTime=1459661234000 (actually the fixTime value seems strange)
Shouldn't the fixTime be the same as that received in the GPRMC? Or does it convert that time into Server time zone?
Cheers,
M.M

It should be. Why do you think it's different? Please provide actual URL with both parameters at the same time.

The values above are actual values taken from a single/same url. Anyway here's another complete forward url - as seen on the receiving end
/insertposdata?id=359710034734114&code=0x0000&gprmc=$GPRMC,064614.000,A,0119.7256,N,10352.6082,E,0.00,0.00,030416,,*04&address=11+Tannery+Rd%2C+Singapore%2C+SG&latitude=1.32876&longitude=103.87680333333333&fixTime=1459665974000&statusCode=0xF841
Thanks,
M.M

Everything works as intended. Note that fixTime is in unix timestamp format.

oic...thanks :+1:


fix formatting for sql queries

@timsander1 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


update to admin, maintainer, and member organization user roles

#3742
Old user roles:

Members: Can create and update organizations and collections
Maintainers: Member permissions and can add, remove and edit the roles of other users.

New user roles:

Members: No permissions
Maintainers: Can create and update organizations and collections
Admins: Maintainer permissions and can add, remove and edit the roles of other users.

Looks good, is there an associated issue for updating the documentation?

Good point, just made one #3765

Following up on Andrew's question about a doc PR, does there need to be one for the UI, or should that just work?

Yeah, I have one here https://github.com/dockstore/dockstore-ui2/pull/1059


Themes, Styles and Case

Features

Themes (default: Hexylamine) added
Formats (default: ascii) added
One could choose between lower-case and upper-case hex-values now

Other

More Documentation and limited visibility of certain things
Introduced more lints, including clippy, and fixed resulting warnings
Increased test-coverage with help of tarpulin (it says 88.66%, but is actually higher)

Extended Binary Coded Decimal Interchange Code added. Test Coverage is now 91.75%

Thank you very much for your contribution.
I didn't have a detailed look, but these are a lot of changes. In general, I would be great to split this into several PRs. Better yet, possible changes should be discussed in an issue first, in order to avoid unnecessary work for you.
Now that everything is here already, we can also try to review it in total. But I need some help in making sense of all of this.
For example. Why did you introduce look up tables? It certainly doesn't make the code any easier. Looking at benchmark results, it also doesn't seem to make the code faster. Quite the contrary: this version of hexyl is about 4 times slower:

Command
Mean [ms]
Min [ms]
Max [ms]
Relative

./hexyl-master ./hexyl-master
249.5 ± 3.0
243.9
253.7
1.00

./hexyl-sivizius ./hexyl-master
1022.4 ± 6.7
1014.8
1036.4
4.10 ± 0.06

The initial reason for these commits was to introduce formats, e.g. for ELF, PNG, and themes (colour-schemes, but maybe more).
In the master-version, the lookup-tables were generated at runtime at lib.rs:172 with the output-sequence (ansi-sequence + character/hexadecimal value) for each input-byte. This is fine for ASCII, but with EBCDIC, the characters had do be remapped to ASCII and do keep both ascii.rs and ebcdic.rs similar, I used a constant lookup-table there too. There is a unit-test with the old generator-code to check the correctness of the ascii-lookup-table, but I somehow forgot to check the lookup-tables for the hexadecimal values.
These lookup-table could in theory boost the performance for small inputs (but it does not matter there) and is insignificant faster for big inputs (because the initialisation-phase is short in comparison to the runtime), but performance was not the reason I introduced constant lookup-tables instead of generated.
To support context-sensitive formats like ELF (which was actually the main reason I started to work on this project), where bytes have different colours depending on context, I introduced this an abstraction-layer: Instead of byte→sequence, bytes are mapped to categories and the bytes are coloured depending on category and theme. Because one could only one theme each run, the theme, which is a structure, gets converted to an lookup-table category→ansi_term::Style.
Now the performance-issue: At the moment the paint_byte-method gets called for every byte and not just 256-times in the initialisation-phase. This of course make this slower for inputs bigger than 256 bytes, but I am already working on it:
For primitive formats like ASCII and EBCDIC, the old lookup-tables for each byte could be generated using the constant lookup-table for the characters and categories with the lookup-table returned by applying the to_colors-method on the used theme. An optimisation I see for ELF is to paint every byte for every category or at least every used category, beforehand and use a two-dimensional lookup-table or a one-dimensional by with index (category<<8+byte).
Anyway: Yes, I know, this is not optimal yet, but this is WIP.
I implemented these features in parallel and they are somewhat depended on each other, and it was actually easier just to implement it that to open an issue first, but if you like to have the discussion on the feature-part (issue) and the implementation-of-the-feature-part (here) separate, we could open an issue.

Yes, I think it would be best to discuss each feature first. In general, I would like to keep hexyl simple and focused on it's current task. That being said, if there is a large support for a particular proposed feature, I'm happy to consider adding it.
I'm closing this PR for now, but happy to reopen later if it should become relevant.

Gotta say, theme support would be really cool, it was the first thing I looked up after using it. I want a rainbow theme!


CSS line break doesn't work in split equations?

Hi everyone,
I have this split equation:
$$\begin{split} err &= \left(\int_{-\infty}^\infty (p(x) - f(x))^2 dx\right)^{0.5}\\&=\left(\textstyle\sum_{i=0}^{2^n-1} \displaystyle\int_{x \isin D_i} (p_i - f(x))^2dx + \int_{x\notin D} f(x)^2dx\right)^{0.5}\end{split}$$

It renders fine, but when applying the CSS for linebreak, the equation doesn't wrap. Is it to do with split?
.katex-display > .katex { white-space: normal }

@mazayadigital I think the problem is just that the line separator should be \\, not \. See the rendered math.

Line breaks only happen automatically at the top level. You won't get line breaks inside \left(...\right).  (In LaTeX or KaTeX.) 🙁

I had a look at this as an exercise.
mazayadigital's equation can wrap, and looks as follows:

The trickery is due to splitting the expression at the "+" sign and doing a post-processing step that aligns the wrapped portion of the equation in an appropriate place. I'm also using ASCIIMath input, but I think the same effect would be possible with LaTeX input.
I put this and some other wrapping ideas on this demo page:
https://bourne2learn.com/math/katex/katex-wrap-examples.php
(View it on a phone.)
I know this doesn't fit with KaTeX's philosophy of LaTeX purity, but it works for me since it's readable on both phone and desktop.
Regards
Murray


Firestore rules failing with custom claims within tokens

Xcode version: 9.2
Firebase SDK version: 4.11
Firebase Component: Auth / Firestore
Component version: 4.5 / 0.10.4

I have a system where the user is able to 'step up' their credentials by approving a code emailed / sent via sms. The endpoint is a http cloud function with code that ultimately creates a claim using:
let claim = new Claim()
claim.deviceApproved = deviceApproved
claim.ssnApproved = ssnApproved
claim.ownershipApproved = ownershipApproved
return this.admin.auth().setCustomUserClaims(userKey, claim)

Once the claim claim is set, the iOS app refreshes the token using getIDTokenForcingRefresh.
I can confirm he claim is coming back and has the relevant credentials. Now both read and writes to the database that use these credentials sometimes fail, with no apparent reason as to why.
I've included an example read, but various writes do the same thing.
Using Charles proxy I can see the requests being made to get the id token and can confirm relevant claims are included there.
Rules are included below for reference.
Steps to reproduce:
I am finding it hard to create an example where I can get this to reoccur, unfortunately trying to simplify the app for demo I have been unable to recreate the issue, yet it persists in the main app.
Relevant Code:
Rules:
service cloud.firestore {
  match /databases/{database}/documents {

    function notAnonymous() {
      return request.auth.uid != null
    }

    function signedIn(uid) {
      return request.auth.uid != null && request.auth.uid == uid;
    }

    function deviceApproved(uid) {
      return signedIn(uid) && request.auth.token.deviceApproved == true;
    }

    function deviceOwner(uid) {
      return signedIn(uid) && deviceApproved(uid) && request.auth.token.ownershipApproved == true;
    }

    match /settings/public {
      allow read;
    }

    match /stocks/{stockId} {
      allow read: if notAnonymous();
    }

    match /bonds/{bondId} {
      allow read: if notAnonymous();
    }

    match /users/{uid} {
      allow read, write: if signedIn(uid);

      match /device/token {
        allow read, write: if signedIn(uid)
      }

      match /accounts/{accountId=**} {
        allow read: if deviceApproved(uid);
        allow write: if deviceOwner(uid);
      }
    }

  }
}

One of the reads that fails. Note I am using RxSwift, but the listen is just a wrapper. This sometimes throws.
func observeOutstandingTransferTotal(withUserKey userKey: String, andAccountKey accountKey: String) -> Observable<Double> {
    return Firestore.firestore()
        .collection("users").document(userKey)
        .collection("accounts").document(accountKey)
        .collection("current").document("pending")
        .rx.listen()
        .flatMapLatest({ (snapshot) -> Observable<Double> in
            guard let data = snapshot.data(),
                let total = data["total"] as? Double
                else { return Observable.just(0) }

            return Observable.just(total)
        })
        .catchError({ (error) -> Observable<Double> in
            throw TransferError.failedObservingPendingTransfer
        })
}

This morning the following path cannot be written to:
/databases/(default)/documents/users/f6F2Nd6aeubS0IYtb909RrBNj5B3/accounts/ce8213ac-15ea-4086-80be-d87c8855b1d9

Perhaps I am doing something wrong in the rules? I can't see anything and have confirmed the claims are correct.

The iOS client treats custom claims exactly the same as any other auth token so this is likely an issue with your rules. This probably isn't the best place for rules questions and without a more complete example (including all of the data that your rules depend on) it will probably be difficult for somebody to help you.  If you're still stuck, perhaps post on the google-cloud-firsetore-discuss google group with a full example or as much detail as you can provide.  Thanks, and sorry I can't help you here!


How to style the component?

I wish to change the border color and border-radius of the component but was not able to. When I add a new className to classes, your class .iciELI takes precedence over any class I wish to write to add extra CSS.

it's not possible as the class is added after the styled component one.

if two classes are in separate files, import the priority file second
So, probably, the issue is that both library styles and user styles are imported randomly, so styles are applied randomly as well


sqlitestudio: init at 3.4.4

Description of changes
Adds SQLiteStudio, a free, open source, multi-platform SQLite database manager.
License: GPLv3
Build instructions
AUR PKGBUILD
I'm a nix novice and this is my first-time contribution to Nixpkgs. I've read the contribution guide and tried my best to follow, please let me know if something is wrong or needs to be changed. Thank you!

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

SQLiteStudio also provides official plugins. Should I package them separately (in another package or in another PR), or include them in the base package?

@ofborg build sqlitestudio


[HUDI-4491] Re-enable TestHoodieFlinkQuickstart

Change Logs
Added dependency flink-metrics-dropwizard to hudi-examples/hudi-examples-flink
Impact
no
Risk level (write none, low medium or high below)
no
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change. If not, put "none".

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

The tests passed: https://dev.azure.com/apache-hudi-ci-org/apache-hudi-ci/_build/results?buildId=24025&view=results


Correct defaultTo, head, path, nth, last

This PR corrects the typings to more accurately reflect Ramda behavior. In particular, these typings do not behave appropriately in all contexts with strictNullChecks enabled.
As a minimal example:
import * as R from './index';

const addOne = (
  n: number,
): number => n + 1;

const x: number | null = ((): number | null => 3)();
const defaultedValue = R.defaultTo(4, x);

const result = addOne(defaultedValue);

yields:
toy.ts(10,23): error TS2345: Argument of type 'number | null' is not assignable to parameter of type 'number'.
  Type 'null' is not assignable to type 'number'.

We correct this by specifying that the second argument to defaultTo can be null or undefined but that it only returns U or T.
The other changes reflect the fact that Ramda may return undefined for those functions in some contexts.
I did not use the scripts to fix these errors, but I attempted to modify it post-fact in order to reflect these changes. Please let me know if I did so incorrectly.

Thank you for the PR! Having it reflected in the script is already more than I'd ask for :), I'd still need to finish that generation...
First of all, sorry for the bug in defaultTo, that definitely looks good. For a sec I wondered if it'd no longer accept arguments that wouldn't offer null types as well, but guess not! So all good there.
For the path typings based on in, I'm under the impression the in checks would already check to ensure the input object would have this path available -- did I overlook something there?
I'm under the impression that'd also go for the tuple-based head typings. If we're guaranteed to have an array of that size, then I suppose that should suffice to exclude the possibility of an empty result (except for empty tuple []).
On inference-less path and the other three returning | undefined, you're not wrong, but to the user I guess both options here come at their own cost -- either unsafety, or having to add defaults anywhere even when they'd 'know' the operation would succeed. Now, Sanctuary managed to address this elegantly by supplying both a head as well as an unsafeHead, so it's a bit unfortunate we don't have that privilege here. I guess that'd make for a breaking change (if only we had our own semver rather than tying to Ramda's versioning), but I'll concede the safety should outweigh that convenience.

I'm under the impression that'd also go for the tuple-based head typings. If we're guaranteed to have an array of that size, then I suppose that should suffice to exclude the possibility of an empty result (except for empty tuple []).

For the path typings based on in, I'm under the impression the in checks would already check to ensure the input object would have this path available -- did I overlook something there?

You are correct on both counts here, my mistake!

On inference-less path and the other three returning | undefined, you're not wrong, but to the user I guess both options here come at their own cost -- either unsafety, or having to add defaults anywhere even when they'd 'know' the operation would succeed.

Yes, there seems a need for some sort of NonEmptyList<T> to handle this case.

but I'll concede the safety should outweigh the convenience here.

I agree.

Thanks for making the adjustments.
I'd definitely be curious as to whether you'd have specific ideas on how a NonEmptyList option might work out. I'm imagining it'd consider type casts on the input list, though an alternative could be to cast the output (could be less desirable if you'd need the interference to get the type?).
I think at the moment internal types had not been successfully exported yet, but it seems like an interesting idea.
That said, I guess right now we might as well just merge this. Thanks for your contributions! :)


Link fixes for Text Generation using FNet example

For the Text Generation using FNet example broken colab and github source links have been fixed.
cc: @fchollet

Thanks for the PR! Does the ipynb file also need an update?

No, the .ipynb file does not contain any notebook or GitHub source link.


Adding support for makeCommand and installPerequisites

These changes are for Kompose pr-building in Fabric8cd environment here http://jenkins.cd.k8s.fabric8.io/job/kubernetes-incubator/.
Here is the Jenkinsfile for Kompose, once this PR is merged I will point to fabric8 pipeline library
Let me your suggestions on this ?
cc : @rawlingsj

[merge]


Multiple renderers causing duplicate elements on print overlay

Describe the bug
When utilizing multiple map renderers such as SVG and Canvas, the print overlay shows duplicates for records tied to the non-default map renderer.
To Reproduce
Steps to reproduce the behavior:

Create simple map with default options (uses default SVG renderer)
Create new canvas renderer with L.canvas();
Add object to map using the canvas renderer.
Click the print button using any mode
Duplicate copies of the canvas rendered object are shown in the print overlay and print preview

Expected behavior
There should only be one copy of each layer showing in the print overlay and print preview
Example
https://jsfiddle.net/shannonthurston_nuvolo/9hn73j84/
Additional context
I experimented with the code and found that when the layers are cloned for the print overlay the 'renderer' option isn't getting tied to the cloned renderer(L.SVG or L.Canvas), but instead is being left to the original renderer from the underlying map and that is what seems to be causing the issue of duplication.  I can make the problem go away partially by modifying the builder functions to set the renderer option to null, but ideally the cloned layers should link to the cloned renderer.

Hi Igor,
I appreciate the fast response to fix this.  While I've been incorporating this into a more complex build, I found that the problem still exists if the user prints once then prints a second time and zooms on the manualMode preview page.  The problem seems to be that the layers are getting registered every time the print command runs and that seems to change the base renderer again so there are duplicate layers generated each time the print button is pressed.
If you use the same example I gave above, you can see it showing duplicates after printing once, then printing again and zooming out or in with your mouse.  The desired effect would be to allow the user to zoom in or out to resize the map on the manual mode preview without duplicates, if that would be possible.  Thank you!

Hi there.
Yes, you are right, I missed that case.
Added new version with a proper behavior.
Regards,
Igor


Support for multiple secrets manager secrets

Context
https://github.com/buildkite/buildkite-agent-metrics/pull/227 added support for Clusters (@DrJosh9000), with agent registration tokens provided as a comma separated list.
The assumption for secrets manager secrets was that it would be a single secret with a comma separated list of values. This is not ideal, as it means a new secret needs to be created just for this lambda. If instead a comma separated list of secrets is used, we can keep the registration tokens in their own secrets which is referenced in multiple places.
Change
This PR supports multiple secrets provided to BUILDKITE_AGENT_SECRETS_MANAGER_SECRET_ID (comma-seperated) and queries secrets manager for each value.
Testing
I've built the lambda zip locally and tested on AWS with 2 agent registration tokens across 2 clusters.

hey @DrJosh9000 @triarius are you able to take a look at this?
seems I am unable to add reviewers


Update dependencies

Includes coverage packages in project dependencies and updates the lockfiles.

@stephenworsley I'd appreciate your thoughts on why the changed dependencies have caused regressions. I've tried re-running the benchmarking job but they're still there.

For time_make_grid, my guess would be something to do with cartopy running these lines:
https://github.com/SciTools-incubator/iris-esmf-regrid/blob/82c494875894b7e54fd8c6c2f6707b917d26eac2/esmf_regrid/_esmf_sdo.py#L157-L158
Cartopy has gone up a version here, as has Proj. I'm not sure about the other slowdowns.

I'm not sure about the other slowdowns.

I don't suppose they might also depend on the same stuff as time_make_grid? Which, as you say, we already understand (and have merged on main).


Installing dependency from modal on plugin page should refresh plugins list

Instructions

Install a plugin with a dependency
On Plugins > Installed Plugins, click the link of a dependent plugin in the "Requires: {Plugin}" list
Click "Install Now" in the modal to install the plugin

Actual Outcome
The dependent plugin is installed but the plugin is not listed in the list of plugins until the page is refreshed
Expected Outcome
The dependent plugin is listed in the list of plugins and the plugin with dependencies has its options updated to reflect the installed dependency

You should try installing the dependencies from the Dependencies tab in the Install Plugins page.

I tried that too and it worked fine.
This mostly feels like a bug because of some of the other ways that the messaging and display don't always give a clear forward path for acting on a dependency.


HotBit listing

https://hotbit.zendesk.com/hc/en-us/article_attachments/360029189313/Token_Listing_Information_and_Requirement_new_version_.xlsx

#
Token Listing   Information

2
Project   Name

3
Token   Name (Abbr.)

4
Email   Address（business）

Email Address（receive the daily market report）

5
Project   Website

6
CoinMarketCap   Link

7
Telegram   Link

Twitter Link

LinkedIn Link

Reddit Link

Bitcointalk Link

Medium Link

Youtube Link

Facebook Link

Discord Link

Steemit Link

Slack Link

Github Link

Whitepaper

Other Links

8
The   scale and cost of private equity financing(price&amount of Seed/Private   Sale)

The scale and cost of public   financing(price&amount of Pre-/ Public Sale)

9
General   Descriptions*Information about the date of ICO, total amount in circulation,   whether it has been listed on other exchanges, etc..

General Descriptions*Information about the date   of STO, total amount in circulation ,cooperated exchanges & etc..(STO   only)

General Descriptions of Local Law ,Regulations   & Restrictions(STO only)

10
Logo   of your token*     500px*500px, PNG or JPG with transparent background     Image formats like .jpg, .png, .gif, .bmp, .psd, .tiff are supported.

11
Smart   contract address of the token *

12
blockchain/platform*

13
Coin Accuracy(own   mainnet project only)

source code address(own mainnet project only)

wallet program(own mainnet project only)

API Doucuments(own mainnet project only)

14
Other   Contact Information

* means required   information

cost prohibitive


ShortTagFixer enhancement

Nice optimization. :+1:
Not sure if this belongs in 1.6 though, since it is not a bug fix, technically.

:+1:
target was set into 1.6

Thank you @gharlan.


Print output in reverse order

This prints output in reverse order in order to show user most relevant pods first.

This is also related to CocoaPods/CocoaPods#4473, which improves sort algorithm.

Spec for printing in the correct order?

Spec added.


Update supporting offer type

Hello, I'm member of ASMS Japan team.
I modified line 15.
I checked ASMS internally and confirmed that CSP Azure plan subscription can also use this API now and customer would like to update this page.
Please also update this page "https://docs.microsoft.com/en-us/rest/api/consumption/" since there is same description.

@hinukaims : Thanks for your contribution! The author(s) have been notified to review your proposed change.

The following users don't have valid GitHub IDs, or are not collaborators on this repo: microsoftdocs/public-repo-pr-review-team

#sign-off

@hinukaims I'm updating https://docs.microsoft.com/en-us/rest/api/consumption/ as you requested with https://github.com/Azure/azure-docs-rest-apis/pull/4610

@Court72 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi all,
Thank you for review and update.

@bandersmsft
My modification is half-backed.
Sorry, I only confirmed that CSP Azure plan subscriptions can use Usage Details API.
Would you clarify if CSP Azure plan subscriptions can also use other APIs (Marketplace Charges, Reservation Recommendations, Reservation Details, Reservation Summaries)?
If CSP Azure plan subscription can also use other APIs, please also modify line 18 as below.
[Now] Line 17 and 18:
17  Azure Consumption APIs provide access to:
18    - Enterprise and Web Direct Customers
[My proposal] Line 17 and 18:
17  Azure Consumption APIs provide access to:
18    - Enterprise, Web Direct Customers and CSP Azure plan Customers
In addition, please also modify https://docs.microsoft.com/en-us/rest/api/consumption/ since there is same description as you know.
If I need to create new request for this topic, please let me know.


Weights for model without Leffa Loss

Hi,
Thank you for sharing the code and the weights. I saw that you released the weights for the model trained on VITON-HD and DressCode. Is it possible to share the weights for the models trained without the leffa loss, used for the ablation study?
Thank in advance for your help.
Best

Thank you for your interest.
Currently, we do not plan to release the intermediate models, but we will assess the situation and decide in the future.
However, if you wish to conduct ablation studies quickly, you can try replicating our model structure and following the training methods provided in the paper. This process should not be very difficult.


added az.compare

Description
Replaced modelwise elpd computation by az.compare.
Checklist

[x] Tests covering the new functionality have been added
[x] Documentation has been updated OR the change is too minor to be documented
[x] Changes are listed in the CHANGELOG.md OR changes are insignificant

oops, I am late to the party.


[AVM Question/Feedback]: Consumption of child modules?

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Description
Are there any ideas around the ability to consume child modules?
I read here in the FAQ that currently this is not possible.
I would love to see the ability to use child modules.
Why?
Most of my deployments are grouped in modules per workload with a resource group scope. I think this is fairly common.
Whenever a workload needs to deploy something in another scope, say a container in a 'central storage  account' or a role assignment somewhere , I need to add a separate module in my source tree just to add this child resource.
For example: in my deployment of appXYZ.bicep to rg-appXYZ, I need to add a container to a central storage account in centralRG.
For that I need to do this:
appXYZ.bicep:
// add a container to the central storage account
module blobcontainer '../modules/blobcontainer.bicep' = {
  name: workloadName
  scope: centralRG
  param: {
    .....
  }

So I need to create the blobcontainer.bicep module just to change the deployment scope to another resource group.
Instead of creating the module, I would love the ability to do something like this:
appXYZ.bicep:
// add a container to the central storage account
module blobcontainer 'br/public:avm/res/storage/storage-account:0.8.0/blobservices/container' = {
  name: workloadName
  scope: centralRG
  param: {
    ......
  }

Appreciate any thoughts on this!

Hi @joostm1, thanks for opening this issue.
Please note that this topic was recently discussed in the following issues:

https://github.com/Azure/Azure-Verified-Modules/issues/578
https://github.com/Azure/Azure-Verified-Modules/issues/747

Please let us know if you have any additional questions.
CC: @AlexanderSehr

Thanks for tagging me @matebarabas. Yes I'd recommend to take a look at the linked issues. I would have loved to carry that functionality (we already implement in the CARML CI) over to the Public-Bicep-Registry, but unfortunately it's technically much more complex. However, the modules are still setup in a way that they could be published - and there is an intend to ensure compatiblity withthe CARML CI once the migration of the modules concluded. So at least for inner-source purposes we could already ensure that child-modules would be published alongside their parent.
When we'd be able to do the same in the public registry is unclear as of this point. But as there is (unsuprisingly) definitely demand, I'm sure we'll get there.

Thanks for sharing your insights @matebarabas @AlexanderSehr, much appreciated.
My summary is that this won't happen any time soon. Glad to see you recognize the need for this though.

This is the only thing holding us back from migrating away from CARML. So would be a nice feature. We have a big environment to setup, lot's of different parts that depend on each-other. So only having the "root" modules isn't going to cut it for us.

This is the only thing holding us back from migrating away from CARML. So would be a nice feature. We have a big environment to setup, lot's of different parts that depend on each-other. So only having the "root" modules isn't going to cut it for us.

Could not agree more...


The message send hook are called very slowly

Environment

OS: ubuntu 16.04
Erlang/OTP: 18
EMQ: 2.0.6

Description
I want to register the message.publish hook to send the message to python using thrift, but I found that the hook was called very slowly. With the emqtt-benchmark in a short time made a million messages, but this hook was only called tens of thousands of times. I would like to know this is a design problem or set up the problem, what I can do to solve this problem ?

😂

@primantok Really cool issue😂

@emqplus Hi Feng Lee, I would like to pushlish messages via thrift. Do you have any suggestions for me?


fix/delete & set permissionsSet and its tests

@bochaco , looks like the last failing test is because the AuthGranted contains an optional BootstrapConfig
/// safeclientlibs:/safe_core/src/ipc/resp/ffi.rs:L27
/// ----------------
/// It represents the authentication response.
#[repr(C)]
#[derive(Clone)]
pub struct AuthGranted {
    /// The access keys.
    pub app_keys: AppKeys,
    /// Access container
    pub access_container: AccessContInfo,

    /// Crust's bootstrap config
    pub bootstrap_config_ptr: *mut u8,
    /// `bootstrap_config`'s length
    pub bootstrap_config_len: usize,
    /// Used by Rust memory allocator
    pub bootstrap_config_cap: usize,
}

Which our current test doesn't contain. I am okay with either: disabling the test here and send a second PR to fix it, or fix it within this PR.
Fixing it means updating our JS-Struct and getting a newly generated response from authenticator (ask @shankar2105 ) to put into the test.

I'll send a separate PR for that then, so we can have this merged today.


[Question]: Github static gh-pages

Is there a way to prevent Google analytics being blocked by ad blockers for static gh-pages without requiring a server-side solution?

Hello! Thanks for your question.
Currently, you still need to run the back end for it. While quite soon (in a month), we're launching DataUnlocker. With DataUnlocker, you won't need a back end anymore, but you will still need a control over the domain you host your website on (i.e. you will need to add a DNS record to your domain). If your website is hosted on GitHub but has a custom domain which you own, that's perfectly fine for DataUnlocker. If not - please let us know - we have a plan to eventually make it possible even without the custom domain (and for free), so we can prioritize it accordingly.
Anyway, you can subscribe on the landing page to get an email notification once DataUnlocker goes live. Our pricing is very affordable: if it's a small website with a few users, you won't need to ever pay for it. Beyond that, it's a cheap pay-as-you-go solution.


Using GraphQL Yoga with Express and Sentry - Middleware has not enough arguments

Describe the bug
As createYoga returns a middleware that only has one argument, Sentry Express Integration fails.
See https://github.com/getsentry/sentry-javascript/issues/10932
Error: Express middleware takes 2-4 arguments. Got: 1
    at wrap (/workspace/node_modules/@sentry-internal/src/node/integrations/express.ts:216:46)
    at /workspace/node_modules/@sentry-internal/src/node/integrations/express.ts:234:46
    at Array.map (<anonymous>)
    at wrapMiddlewareArgs (/workspace/node_modules/@sentry-internal/src/node/integrations/express.ts:232:46)
    at Function.router.<computed> [as use] (/workspace/node_modules/@sentry-internal/src/node/integrations/express.ts:257:46)
    at Object.<anonymous> (/workspace/src/main.ts:36:5)
    at Module._compile (node:internal/modules/cjs/loader:1376:14)
    at Module._compile (/workspace/node_modules/source-map-support/source-map-support.js:568:25)
    at Module.m._compile (/tmp/ts-node-dev-hook-32544876058001004.js:69:33)
    at Module._extensions..js (node:internal/modules/cjs/loader:1435:10)

Your Example Website or App
https://codesandbox.io/p/devbox/ancient-grass-gxfm76?file=%2Fsrc%2Fmain.ts&layout=%257B%2522sidebarPanel%2522%253A%2522EXPLORER%2522%252C%2522rootPanelGroup%2522%253A%257B%2522direction%2522%253A%2522horizontal%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522id%2522%253A%2522ROOT_LAYOUT%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522cltel4ywy00083b6fen893duf%2522%252C%2522sizes%2522%253A%255B70%252C30%255D%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522EDITOR%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522id%2522%253A%2522cltel4ywy00033b6feavp0323%2522%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522SHELLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522id%2522%253A%2522cltel4ywy00053b6f8me5cky1%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522DEVTOOLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522id%2522%253A%2522cltel4ywy00073b6f5hxv0ihd%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%252C%2522sizes%2522%253A%255B50%252C50%255D%257D%252C%2522tabbedPanels%2522%253A%257B%2522cltel4ywy00033b6feavp0323%2522%253A%257B%2522id%2522%253A%2522cltel4ywy00033b6feavp0323%2522%252C%2522activeTabId%2522%253A%2522cltel4ywy00023b6ffgj6adq1%2522%252C%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522cltel4ywy00023b6ffgj6adq1%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522FILE%2522%252C%2522filepath%2522%253A%2522%252Fsrc%252Fmain.ts%2522%252C%2522state%2522%253A%2522IDLE%2522%257D%252C%257B%2522type%2522%253A%2522FILE%2522%252C%2522filepath%2522%253A%2522%252Fnode_modules%252Fresolve%252Ftest%252Fresolver%252Fmultirepo%252Fpackages%252Fpackage-a%252Findex.js%2522%252C%2522id%2522%253A%2522cltel8yh0007f3b6f7mq2zmta%2522%252C%2522mode%2522%253A%2522temporary%2522%252C%2522state%2522%253A%2522IDLE%2522%257D%255D%257D%252C%2522cltel4ywy00073b6f5hxv0ihd%2522%253A%257B%2522id%2522%253A%2522cltel4ywy00073b6f5hxv0ihd%2522%252C%2522tabs%2522%253A%255B%255D%257D%252C%2522cltel4ywy00053b6f8me5cky1%2522%253A%257B%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522cltel4ywy00043b6ftpnligye%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522TASK_LOG%2522%252C%2522taskId%2522%253A%2522dev%2522%257D%255D%252C%2522id%2522%253A%2522cltel4ywy00053b6f8me5cky1%2522%252C%2522activeTabId%2522%253A%2522cltel4ywy00043b6ftpnligye%2522%257D%257D%252C%2522showDevtools%2522%253Atrue%252C%2522showShells%2522%253Atrue%252C%2522showSidebar%2522%253Atrue%252C%2522sidebarPanelSize%2522%253A15%257D
Steps to Reproduce the Bug or Issue
Run the example. It crashes immediately.
Expected behavior
No crash. Working example.
Screenshots or Videos
No response
Platform

OS: Linux
NodeJS: 20.11
@graphql-yoga/* version: 5.1.1

Additional context
No response

Hi! The error here comes from Sentry trying to wrap all middlewares (not sure why they do this instead of just adding their own middleware, but they have probably a good reason to do so)
The problem is that they try to detect arrity of the middleware function using fn.length, but Yoga instance is not a classic function. It is a generic request handler, so it doesn't have this length parameter.
A workaround is probably to add a function instead of giving the yoga instance directly to app.use:
const yoga = createYoga({...})
app.use(yoga.graphqlEndpoint, (req, res) => yoga(req, res))

You can see it in action here : https://codesandbox.io/p/devbox/unruffled-goldstine-h4pggr?file=/src/main.ts:41,1&workspaceId=65cb9b58-f829-4669-8aa5-9bdf9f94c245

Thanks @EmrysMyrddin  this worked.

Cool!
Sorry to not offer an actual fix... But where to fix this issue is not very clear.
We can't actually add a length ourself to make Sentry detection working, because it's at risk of breaking another tool without noticing.
In the other hand, I understand Sentry also can't change there logic at risk of also breaking compatibility with other Express middleware.
Sadly, this little workaround in user land is probably the best solution we have.


Add files via upload

To add our University

@mmasias I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host uneatlantico.uneatlantico.es not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Not able to use .env file with docker-compose

BUG REPORT INFORMATION
Output of docker version:
Client:
 Version:      1.10.1
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   9e83765
 Built:        Thu Feb 11 19:14:21 2016
 OS/Arch:      darwin/amd64

Server:
 Version:      1.11.2
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   b9f10c9
 Built:        Wed Jun  1 21:20:08 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 3
 Running: 3
 Paused: 0
 Stopped: 0
Images: 91
Server Version: 1.11.2
Storage Driver: aufs
 Root Dir: /mnt/sda1/var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 140
 Dirperm1 Supported: true
Logging Driver: json-file
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.4.12-boot2docker
Operating System: Boot2Docker 1.11.2 (TCL 7.1); HEAD : a6645c3 - Wed Jun  1 22:59:51 UTC 2016
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.955 GiB
Name: default
ID: ANEF:4T6C:5IHT:I5PC:YUFB:SVE6:GCL4:AUHM:HQVM:WOHH:ZOHX:UMYG
Debug mode (server): true
 File Descriptors: 30
 Goroutines: 56
 System Time: 2016-08-22T10:16:30.301647434Z
 EventsListeners: 0
 Init SHA1:
 Init Path:
 Docker Root Dir: /mnt/sda1/var/lib/docker
Labels:
 provider=virtualbox

Steps to reproduce the issue:
Follow the documentation to create an .env file and use it with docker-compose: https://docs.docker.com/compose/environment-variables/#the-env-file
Describe the results you received:
docker-compose config displays: WARNING: The TAG variable is not set. Defaulting to a blank string.
Describe the results you expected:
I wanted to use the .env file to define the ports for each service. So each developer can use the same docker-compose.yml file and have its own not versioned .env file where he defines the ports that are available for him.
Additional information you deem important (e.g. issue happens only occasionally):

I found the solution in another site. If this can help someone, it's just a version problem:
http://stackoverflow.com/questions/39076871/not-able-to-use-env-file-with-docker-compose


[@kadena/transfer] Module explorer

Is this PR completed? I noticed the code viewer is displayed after the list (which can be really big sometimes)

@nil-amrutlal-dept not quite. I just learned that the code editor should render the contract in JavaScript, which should be updatable and executable. The UX is indeed not optimal with the editor below the list. I would prefer a detail view on a page that is separate from the list page.

@nil-amrutlal-dept can you please help me get this PR merged after all?

Note: when we have a design for this page, the affected pages should be changed accordingly. Appart from that, everything looks good


Cosmos DB RequestOptions PartitionKey will not serialize with JsonConvert, Always NULL

Trying to serialize RequestOptions to send to Azure Functions but the JsonConvert.SerializeObject always produces a null on the PartitionKey object. That seems to be a bug. RequestOptions is sealed so my only work around is to manually construct my own class. That seems extreme when it is needed for just about everything in Cosmos. The only way to pass parameters to Azure Functions is by serialization.
`		public static void Main(string[] args)
{
var partitionKeyValue = "39393939";	// Some partition key in Cosmos DB collection
var requestOptions = new RequestOptions
{
PartitionKey = new PartitionKey(partitionKeyValue)
};
		var data = JsonConvert.SerializeObject(requestOptions);

		// send data to Azure Functions
		var requestOptions1 = JsonConvert.DeserializeObject<RequestOptions>(data);
	}

`

@laultman try using the new v3 sdk.


Agents must be in specific order in config file

In the configuration file, agents must be put in a specific order (ie the NavAgent must be before the agent that is trying to follow it, because the FollowTask will look for the NavAgent when it is initialized)
The better way to do this would for the FollowTask to have a null reference to the target, and every .tick() it would look for the target if that reference is null, and set it when it finds it. That way the order can be more flexible.

Fixed in  #164

This is done?


Can I implement

Description
As an [Actor], I want [feature] so that [why].
Acceptance Criteria

[Criteria 1]

Steps to Reproduce (Bug)

[Step 1]
[Step 2]
[Step ...]

Expected behavior: [What you expect to happen]
Actual behavior: [What actually happens]
Frequency: [What percentage of the time does it occur?]
Versions (Add all that apply)

Software version: [besu --version]
Java version: [java -version]
OS Name & Version: [cat /etc/*release]
Kernel Version: [uname -a]
Virtual Machine software & version: [vmware -v]
Docker Version: [docker version]
Cloud VM, type, size: [Amazon Web Services I3-large]

Smart contract information (If you're reporting an issue arising from deploying or calling a smart contract, please supply related information)

Solidity version [solc --version]
Repo with minimal set of deployable/reproducible contract code - please provide a link
Please include specifics on how you are deploying/calling the contract
Have you reproduced the issue on other eth clients

Additional Information (Add any of the following or anything else that may be relevant)

Besu setup info - genesis file, config options
System info - memory, CPU

I am  newbie at hyperledger Besu. I wanna implement  ERC-721 tokenization for private network using hyperledger Besu. But I cannot find any documentation to implement. Is it possible or not. ? If possible , how can I implement?


Issues with V-USB implementation in schematics

Hi brickbots,
You did a fantastic work here and I would really like to try get this thing to work.
Right now I have some issues with the keyboard. I have flashed usbaspLoader modified to work with Atmega328P used in Plaid keyboard. Everything goes well, I flashed your qmk config on the keyboard but i constantly get some issues with USB resets and HID errors. The strange thing is that this heppens only with the firmware flashed and not while in bootloader.
Other strange thing is that the keyboard works sometimes on my computer while getting HID errors but never on Rasberry Pi 4.
What I tried so far:

Changing different fuses on the chip
Replacing the chip with new one (tought this could be the issue because I heated the first one badly)
Built another new keyboard with 68R instead of 74R
Changed USB cable

I am using male USB cable, it is not yet soldered on the Pi, could this maybe related to the errors?
I am stuck and I would be thankful if you have some suggestions for diagnosing this issue.

Hi @sini6a !  Sorry to hear about the bumps you've run into here.  From the photos your build looks very clean and nice and I don't see anything out of place.
As a next step, I would suggest making your USB 'cable' as short as possible.  Normal USB cables are generally well shielded and constructed to prevent interference.  Using bare wires like the HandiPi does, means that the run needs to be as short as possible.  Also try to make the two data lines the same length, within reason.
I had some similar issues with a longer test cable I created and they went away when I made the cable much shorter.  Hopefully this will help you as well!

Just looking a bit closer at your photos... the capacitor in the upper-left (C1) is specified as an electrolytic.  Not sure exactly what the type/value is of the component you've used, but this is part of the power filter/smoothing and it could conceivable lead to some voltage fluctuations if it's not quite right.
I'd definitely suspect the wire/cable length first, but this might be another thing to try if you are still having issues.

Thank you for fast response @brickbots!
The cable length is 15cm. I will try and cut them to half of the current length.
As of the capacitor I think you mean C3 which is located on the upper left corner of the image. It is a 4.7uF ceramic capacitor. I will attach more specification below of the order.
I also noticed that without 1n4148s I got it to work on Raspberry but it was lagging and often stuck on keypress. I think this could be related to USB resets.
I am trying your suggestions first and I will keep you updated.
Thank you for your time 😊

Hi Hi!
Yes, I did mean C3, thank you :-). I've never seen a solid electrolytic before, I'm used to the cylindrical aluminium ones, pretty neat!   That should totally work fine, so hopefully it's just the cable.
It's interesting that it worked without the diodes.... are you referring to D37/D38?  If so, those are not general purpose signal diodes, they are zener diodes to assure that the USB signal lines are 3.6v rather than 5.  My first prototypes without them worked on some devices, but not others, depending on how tolerant the host was of the over-voltage USB data lines.

Hello again 🙂
I was reffering to

Hello again 🙂
Actually I was referring to small signal diodes that should be on the keys. I removed them all and will test without them first. I got them from another supplier and thought that maybe they did some mistake about the model but I will investigate this further when my other order arrives.
Those two diodes sit in place. I will send more specification about them below.
I am still not able to report anything because I was busy these days. Will do it in the weekend latest and report back.
I really appreciate your help and the time spend here trying to resolve the issue. 😊

Those are definitely the correct Zener diodes!  Good luck with your experiments over the weekend!

Hi again!
No luck so far. I cut the cable half of the length, same problem.
I tried measuring the voltage on D lines, I am not sure what values should I read but in one of the lines (D+) I get 2.30 idle and 1.20V on key pressed. Other line (D-) has 5.20V idle and 3.20V on key press.
Is it allowed for this line to go over 3V?
Thank you 😊

Okay I have update:
On R2 I installed one more resistor with value of 1.74kOhm and it works like charm.
I don’t have enough technical knowledge to describe why is it so, maybe someone can explain.
One question that I have right now is does this means that I need to put 3.24kOhm instead of 1.5kOhm?
Thank you so much for your help! 😊

Hi Hi!
So glad it's working, but it's definitely a bit mysterious... R2 pulls the D- line high, indicating that the usb connection is low-speed and 1.5k is the recommended value here.
Adding resistors in parallel, as the photo indicates, actually produces a lower resistance than either resistor's value.  In this case ~0.8k.... even stranger!   Might be interesting to test the combined resistance without the circuit powered.
As for the D-/D+ voltages, these will be tricky to measure without a good oscilloscope as they will be shifting between low and high states very quickly.  However, it should generally not read above 3.6v.    I'd be curious to know the voltages across the two Zener diodes if you can power the board but leave the signal lines unconnected.
All that aside, if it works, it works!  Glad you worked through the problem and got it sorted :-)

Hi!
Yes, that was the solution in my case, been testing for a whole day and didn’t get a single USB reset.
Really strange because all schematics on the internet shows 1k5 or 1k7 resistance on the line.
I am traveling this week but I would be happy to do some more tests and report back as soon as I get back. It would be interesting to learn why this board acts like this.
I will close the thread as solved for now. Huge thanks for your help and keep up the good work. This device is brilliant. 😊


I have this error that apparently comes from the Akairo Discord Inhibitors file TypeError: Cannot read property 'id' of null

I have tried to put conditionals to avoid that error continues to appear but it does not work
Nor do I know what inhibitor it is, it no longer tells me where the problem is, it only tells me where it originates from (source code of discord-akairo).
Any solution?
I don't know what else to do

Error*

2021-02-14 17:19 ERROR (node:101) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'id' of null
    at CommandHandler.runAllTypeInhibitors (/home/container/kaede/node_modules/discord-akairo/src/struct/commands/CommandHandler.js:567:55)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async CommandHandler.handle (/home/container/kaede/node_modules/discord-akairo/src/struct/commands/CommandHandler.js:346:17)
2021-02-14 17:19 ERROR (node:101) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 352)

This is the general structure of my bot's inhibitors

export default class NameInhibitor extends Inhibitor {
  constructor() {
    super('...', {
      reason: '...',
      type: 'post',
      category: '...',
      priority: 5,
    });
  }

  async exec(message: Message, command: Command): Promise<boolean> {
    // .....
  }
}

seems like a problem on your side, It is looking for <File(Inhibitor|Command|Listener)>.id which according to error implies that  is somehow null.

seems like a problem on your side, It is looking for <File(Inhibitor|Command|Listener)>.id which according to error implies that is somehow null.

Hmm, it's not a problem on my side because if you look at the Discord-Akairo source code and look for that line of code that gives an error about getting the user ID
I am not looking for any user ID in that inhibitor
https://github.com/discord-akairo/discord-akairo/blob/29810b9e5d5cd804c649948ad60e5f2ac3897141/src/struct/commands/CommandHandler.js#L567:55

} else if (this.blockClient && message.author.id === this.client.user.id) {

The line does not seem to error, works for me.

} else if (this.blockClient && message.author.id === this.client.user.id) {

The line does not seem to error, works for me.

It gives me an error how can you see, the error does not mark what is in my code, it marks what is in that file

That is a common mistake you see, for example,
(node:9880) UnhandledPromiseRejectionWarning: AbortError: The user aborted a request.
    at RequestHandler.execute (/home/pi/node_modules/discord.js/src/rest/RequestHandler.js:107:21)
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:9880) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)
(node:9880) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

The above problem does not mean there is a problem with discord.js RequestHandler, it is just a mistake you made.

I see, but what explanation would that error I get?

Like i said, something wrong with your file structure I see, (maybe you forgot to export an extended class)

Like i said, something wrong with your file structure I see, (maybe you forgot to export an extended class)

not really, that file all it does is look for a document in Mongoose and validate with a condition, and that's it

Deprecated package.


Add Damian Kolakowski to stakeholders.

I maintain the following projects:
https://github.com/httpswift/swifter
https://github.com/glock45/swiftX

Merging. Welcome aboard Damian.


Add FaceEmo

https://github.com/suzuryg/face-emo is a NDMF and MA based tool for creating face animation based on

I believe i saw it somewhere, but forgot about it, i'll try and add it. Thank you. 👍


Added keycode 162

This closes #232, closes #233

thanks :) Ill add you ad a contributor if you want to wade through all the old issues :D

Sure, I would be very happy to help you in any way possible.


Revert "sched/group: move task group into task_tcb_s to improve performance"

Reverts apache/nuttx#11832 since there are some issues.
See https://github.com/apache/nuttx/pull/11832 for the details.

Let me close this PR since https://github.com/apache/nuttx/pull/11885 fixes the issues.


Created the folder and the files and added content to the files. (#32)

@anubhav1206 can u send screen shots so i can verify and merge the coee.

@anubhav1206 can u send screen shots so i can verify and merge the coee.

Sure

Thanks for your contribution @anubhav1206. Make sure you 🌟 the repo and follow us on Linkedin

Thank you!


wis2box not starting well

Hello This is David from Tanzania Meteorological Authority,  I have another issue in my local wis2box test machine which i am working on ...after i started the wis2box nginx is not running as shown in the output bellow. what might be the cause for this
tma@TMA-WIS2BOX:~/wis2box-1.0b4$ python3 wis2box-ctl.py restart
/usr/lib/python3/dist-packages/paramiko/transport.py:236: CryptographyDeprecationWarning: Blowfish has been deprecated
"class": algorithms.Blowfish,
Stopping wis2box-ui    ... done
Stopping wis2box-api   ... done
Stopping wis2box-minio ... done
Stopping elasticsearch ... done
Stopping grafana       ... done
Stopping mosquitto     ... done
Stopping loki          ... done
Stopping prometheus    ... done
Removing nginx         ... done
Removing wis2box-ui    ... done
Removing wis2box-api   ... done
Removing wis2box-minio ... done
Removing elasticsearch ... done
Removing grafana       ... done
Removing mosquitto     ... done
Removing loki          ... done
Removing prometheus    ... done
Removing network wis2box_project_default
Removing network wis2box_project_vpcbr
/usr/lib/python3/dist-packages/paramiko/transport.py:236: CryptographyDeprecationWarning: Blowfish has been deprecated
"class": algorithms.Blowfish,
Creating network "wis2box_project_default" with the default driver
Creating network "wis2box_project_vpcbr" with driver "bridge"
Creating prometheus    ... done
Creating loki          ... done
Creating elasticsearch ... done
Creating grafana       ... done
Creating mosquitto     ... done
Creating wis2box-minio ... done
Creating wis2box-api   ... done
Creating wis2box-ui    ... done
Creating nginx         ...
Creating nginx         ... error
ERROR: for nginx  Cannot start service web-proxy: driver failed programming external connectivity on endpoint nginx (1b1cd57a926604305dd40f2ff948cfb7c2c4671bfef3126173fd1f59908803bb): Bind for <IP_ADDRESS>:80 failed: port is already allocated
Creating wis2box-management ... done
Creating wis2box-auth           ... done
Creating cadvisor               ... done
Creating mqtt_metrics_collector ... done
ERROR: for web-proxy  Cannot start service web-proxy: driver failed programming external connectivity on endpoint nginx (1b1cd57a926604305dd40f2ff948cfb7c2c4671bfef3126173fd1f59908803bb): Bind for <IP_ADDRESS>:80 failed: port is already allocated
ERROR: Encountered errors while bringing up the project.
tma@TMA-WIS2BOX:~/wis2box-1.0b4$

i am using ubuntu 22.04.1 and docker vesion is 24.0.5,  their is no other applications like webserver as the machine is only dedicated to wis2box
tma@TMA-WIS2BOX:~/wis2box-1.0b4$ docker version
Client: Docker Engine - Community
Version:           24.0.5
API version:       1.43
Go version:        go1.20.6
Git commit:        ced0996
Built:             Fri Jul 21 20:35:18 2023
OS/Arch:           linux/amd64
Context:           default
Server: Docker Engine - Community
Engine:
Version:          24.0.5
API version:      1.43 (minimum version 1.12)
Go version:       go1.20.6
Git commit:       a61e2b4
Built:            Fri Jul 21 20:35:18 2023
OS/Arch:          linux/amd64
Experimental:     false
containerd:
Version:          1.6.22
GitCommit:        8165feabfdfe38c65b599c4993d227328c231fca
runc:
Version:          1.1.8
GitCommit:        v1.1.8-0-g82f18fe
docker-init:
Version:          0.19.0
GitCommit:        de40ad0
tma@TMA-WIS2BOX:~/wis2box-1.0b4$ uname -a
Linux TMA-WIS2BOX 6.2.0-31-generic #31~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Wed Aug 16 13:45:26 UTC 2 x86_64 x86_64 x86_64 GNU/Linux
tma@TMA-WIS2BOX:~/wis2box-1.0b4$


Document the behavior of adding etcd nodes while member of current cluster is unhealthy

etcd prevents adding a new etcd members when one or more etcd nodes in the current cluster is unhealthy, this behavior can be overriden by passing  --strict-reconfig-check=false to etcd, but this can cause etcd to enter a quorum loss situation and breaks the cluster, we should document that when one or more etcd is down, user should either fix the broken node or remove it before adding new ones.
related issue: https://github.com/rancher/rancher/issues/20956

Closing this issue because this PR was merged https://github.com/rancher/docs/pull/1849


#491 testgen: make Precommit default vote type

Trivial changes to make Precommit default for votes

[X] Referenced an issue explaining the need for the change
[X] Updated all relevant documentation in docs
[X] Updated all code comments where relevant
[X] Wrote tests
[ ] Updated CHANGELOG.md

What do you think of having a ‘—vote-type=precommit|prevote’ flag instead?

@romac Thanks for the suggestion, but I think the current solution is better. With what there is currently, it's
Rust API:
let vote = Vote::new(...)  // if default Precommit is fine
let vote = Vote::new(...).prevote(true)  // if Prevote desired

CLI:
tendermint-testgen --stdin vote # if Precommit is fine
tendermint-testgen --stdin vote --prevote # if Prevote desired

With your proposal it would be
Rust API:
let vote = Vote::new(...)  // if default Precommit is fine
let vote = Vote::new(...).vote_type(Prevote)  // if Prevote desired : this either needs a separate def, or parsing a string

CLI:
tendermint-testgen --stdin vote # if Precommit is fine
tendermint-testgen --stdin vote --vote-type=prevote # if Prevote desired: requires parsing a string

In short, if there is an obvious default, there is no need to introduce additional overhead of parsing a choice. I hope this is convincing:)

It turned out that more changes are necessary for testgen, so I will add more commits here

I don't feel strongly about it. The one argument I can raise for the solution I proposed is that it would scale to cases where there are 3 or more choices, which can then not be expressed as a boolean flag/parameter. But in this case, if there will only ever be these two options I guess the current solution is fine as well :)

OK, then let's leave it as is for now, and convert to another form if ever the third choice arises:)


Add db index for reward smesher ID

For the API we need to know which smesher and coinbase received each reward (https://github.com/spacemeshos/api/blob/ff4a9a6b913a23a83cf07a0a7b4b7c16605ce9c0/proto/spacemesh/v1/types.proto#L100-L107). Right now, we can find rewards by account that received the reward but there is no smesher ID or coinbase attached to it:
https://github.com/spacemeshos/go-spacemesh/blob/61eafeeb2a9e3ec785113cf8284eec8bf8ef6ee9/mesh/meshdb.go#L436
Similar to #2064

specifically the coinbase is known, because that's how we retrieve the reward in the first place, so we could just fill that in

The account that received the reward is the coinbase - we may want to rename this, depending how we ultimately design incentives and rewards (as smoothing makes the concept of a "coinbase" account seem slightly off) - but, as Noam pointed out, we do have this data and indeed this is how it's currently indexed. We do not currently store SmesherId in the database. So the scope of this issue is:

add SmesherId to the Reward type (other code may need to change; be sure to update existing tests and/or add a test for this)
make sure it's returned by meshdb.GetRewards
surface it in the API code

The account that received the reward is the coinbase - we may want to rename this, depending how we ultimately design incentives and rewards (as smoothing makes the concept of a "coinbase" account seem slightly off) - but, as Noam pointed out, we do have this data and indeed this is how it's currently indexed. We do not currently store SmesherId in the database. So the scope of this issue is:

add SmesherId to the Reward type (other code may need to change; be sure to update existing tests and/or add a test for this)
make sure it's returned by meshdb.GetRewards
surface it in the API code

Currently, the database does not take into account that a single Coinbase could be associated with multiple SmesherIDs. When writing the transaction rewards to the database, the key is of the form reward_<coinbase>_<layerID>, and the reward struct is for a specific Coinbase and SmesherID. Since multiple SmesherIDs could share a Coinbase, we cannot differentiate between different rewards with the same Coinbase but different SmesherIDs with this current key-scheme. Therefore, we are extending the primary database key to be of the form reward_<coinbase>_<smesherID>_<layerID>. In order to perform efficient look-ups of rewards by SmesherIDs, we will have to add a secondary key of the form reward_<smesherID>_<coinbase>_<layerID> that points to the primary key, and then the lookup becomes a two-step process, but we can do this via the prefix-matching that LevelDB offers.

@avive, API-related question regarding database indices and sort order of rewards: Sid pointed out that, if we refactor the DB index for reward along the lines described above, we can no longer guarantee that rewards (per-smesher, or per-coinbase) will be returned from the API in order of layer. They'll be returned in order of (smesherID, layer) (when querying by coinbase) or (coinbase, layer) (when querying by smesherID). In other words, if one smesher has rewards accumulated to multiple coinbases, the rewards will come out of the database grouped by coinbase; and if one coinbase has accumulated rewards from multiple smeshers, they'll come out of the database grouped by smesher.
The question is, how important is it that we be able to return rewards (by coinbase, or by smesher) strictly sorted by layer?

I think it is okay and clients can sort the data as they like once they have it quite easily. However, doesn't all of this pending on whether we are going to have epoch-wide rewards smoothing? if there's smoothing then the reward amount is per epoch and not tied to specific layers. So, for example, if I smeshed in 3 layers in an epoch and got 1 SMH reward for that epoch then can we assume 1/3 SMH per layer?

You're right, but there's still a layer associated with each reward - in the case of epoch smoothing, it would become the layer at which the reward is calculated, or paid, or vests, something along those lines. We have yet to figure out exactly how this will work, but I think the question I asked here remains the same. We'll just make sure this is clearly documented.

Why not make it reward_<coinbase>_<layerID>_<smesherID> and reward_<smesherID>_<layerID>_<coinbase>?
Or, if we only have endpoints for rewards by coinbase or by smesher and there's no need for internal differentiation (other than by layer) then we can just have reward_<coinbase>_<layerID> and reward_<smesher_id>_<layerID> with a sum inside.


[Feature][Connectors-v2-Paimon] Adaptation Paimon 0.6 Version

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
paimon has now been updated to version 0.6, adding many new features and actually reaching production availability. Currently the bundle of seatunnel connectors for paimon uses 0.4, so I want to upgrade it.
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

It has been merged, closing issues


Test Failures when Integration Tests Run in GitHub

20 out of 21 integrated  tests failed in GitHub but it worked in local.
Steps to Reproduce the behavior
Steps should be reported as following example:

Tests implemented in 'Integrated Test' File
When Integrated Test ran in local, all testing passed.
The error occurred when committed to GitHub.

Expected Behavior
All Integrated Testing ( Total 21 Tests ) should passed successfully.
Actual Behavior
The following error message appeared.
Error: Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.19.1:test (default-test) on project devops: There are test failures.
Only 1 out of 21 tests passed and all other tests failed.
Test Results failed because all test results are null.
Screenshots/Recordings
The detail report can be seen in 36dde79 unit-testing  branch.
Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version/Commit: 36dde79 unit-testing

Additional Information
Potential Error Could be Database Connection Error.

The Root Cause of all tests failed and Error message "Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.19.1:test (default-test) on project devops: There are test failures." is because database connection does not work in GitHub.
This error is fixed by correcting database driver connection configuration in App.java File.


⚠ PrivateHD has degraded performance

In 050bf14, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 2267 ms

Resolved: PrivateHD performance has improved in d381db0.


GNU Arm Embedded Toolchain 8-2018-q4-major can't produce hex files on Windows

EDIT: Fixed in toolchain release 8-2019-q3-update Released: July 10, 2019
Describe the bug
The latest release of "GNU ARM Embedded Toolchain" fails when trying to create a hex file on the Windows platform.
To Reproduce

Build a sample that creates a hex file (for instance the hello world sample with nrf52_pca10040) with the environment described below.
See error

Expected behavior
No build failure.
Actual behavior
Error:
arm-none-eabi-objcopy.exe: zephyr.hex 64-bit address 0x4b4fa30004000 out of range for Intel Hex file
Impact
Unable to use the latest release of gnuarmemb on Windows.
Environment:

Only affects Windows
Use exactly GNU Arm Embedded Toolchain 8-2018-q4-major

Additional context
This has been independently reported upstream by another user:
https://bugs.launchpad.net/gcc-arm-embedded/+bug/1810274

Hello,
I am trying to compile and debug my code in VS Code for the STM32 F401RE MCU. When I attempt to build my project, I am getting the following error: "C:/Program Files (x86)/GNU Tools ARM Embedded/8 2018-q4-major/bin/arm-none-eabi-objcopy:build/vscode2.hex: bad value". What would you suggest doing in this case?
Thank you very much!

This issue and the PR's related to this, are very confusing, to say the least.

Has this actually been fixed?
Can we still use GNU ARM Embedded to compile or do we need to change?

This was fixed with the release of
GNU Arm Embedded Toolchain Version 8-2019-q3-update Released: July 10, 2019

@SebastianBoe
Since you seem to be from Trondheim, you may be working with Nordic Semi. If that is so, I would like you to tell them to update all their getting started and setup pages, which still refer users to use the older tool-chain before this bug...

Thank you for the heads up, will pass it along.


Consider adding option to bail on error

In my usage of hipSYCL I'm seldomly wait()-ing on tasks to finish, and instead repeatedly query the command_execution_status on an event to see if a particular command has finished. This currently presents a problem whenever something unexpected happens, such as an out-of-bounds memory access within a CUDA kernel, which will manifest as CUDA error 700 on the host. I then basically get pages upon pages of this:

I've worked around this by adding a std::terminate in the error handler, but that got me thinking that maybe a canonical solution should exist for this problem. For example, an environment variable such as HIPSYCL_BAIL_ON_ERROR or HIPSYCL_EXIT_ON_ERROR.

The default asynchronous error handler already terminates in accordance with SYCL 2020:
https://github.com/illuhad/hipSYCL/blob/0217b30a90b7c5889712bf7790b6be692c7f5bc4/include/hipSYCL/glue/error.hpp#L82
By not calling queue::wait_and_throw() or event::wait_and_throw(), or queue::throw_asynchronous() you are asserting that you do not want that behavior imho. I'm not sure adding another user-facing mechanism for error handling is the right approach.
I think the main issue here is that hipSYCL does not yet clear the CUDA error state. Because of this, a single error usually means that all subsequent operations fail as well, even if they are unrelated.

queue::throw_asynchronous()

Huh, TIL. Using wait_and_throw is not an option, but periodically calling throw_asynchronous might be. This just has to be a cheap operation then. I checked and it looks like hipSYCL is always creating two std::vectors and locking a mutex in async_error_list::pop_each_error. Both should be reasonably cheap I think (if the mutex is contended I will abort anyway), but I will look into this some more. Thanks for the time being and sorry for the noise!


why scoop cleanup not working in WingetUI <IP_ADDRESS>

I disabled it because it was a time-consuming operation (it would delay the launch for a while), and since it is not strictly required in order to have uncached apps and manifests, I didn't thought it was needed.
Anyway, i'll add a switch on the settings


Implement new filters into communities and users

I think this might replace the search all posts feature as described in https://github.com/kyle1373/Archiverse/issues/30, since this would be scoped down further. Perhaps create a composite index with community id, game id, NNID, title ' ' text, etc

Archiverse has been acquired by @PretendoNetwork. Please redirect this issue there: https://github.com/pretendonetwork/archiverse


fix Issue 16439 - Non-typesafe variadic functions can never be @safe

LGTM. I assume a compiler fix is coming too? :)
@WalterBright / @andralex can I get merge rights for dlang.org? There's plenty of PRs to review & merge in the repo.

@AndrejMitrovic yes, I've set you up.

Thanks. :o)


Nightly GitHub Action failure

Our nightly GitHub Action for the container image fails since yesterday with:
#34 16.10 Copying rrdp_notification.c
#34 16.11 Copying rrdp_snapshot.c
#34 16.11 Copying rrdp_util.c
#34 16.12 Copying rsync.c
#34 16.12 Copying tal.c
#34 16.12 Copying validate.c
#34 16.12 Copying version.h
#34 16.13 Copying x509.c
#34 16.14 Applying patch patches/0001-Allow-overriding-default-user-and-file-locations.patch
#34 16.15 1 out of 6 hunks FAILED -- saving rejects to file main.c.rej

Should be fixed in HEAD aka a9253f3a738989a5b42335d4597cc033ecbea633

Confirming successful run with commit a9253f3a738989a5b42335d4597cc033ecbea633 - thank you.


Separate local and system parameter names.

Resolves #1694.

@peterNordin This should now work with hierarchical system parameters. I also added unit test for HString::startsWith function. Can it be merged, or does it require more work?

I will test this again

Also I tried to do  self.B+100 to assign mass inside a Mass component, but it did not work. I guess this is because numhop support for self. is still missing ?

I made a PR #1899 with a bunch of tests that capture various usage of system parameters, if you re-base on this branch (now or after it is merged) we can see what things will break.
Preferably nothing should break, but for edge-cases a warning message may be sufficient, but who looks at those. If I have time I can look into fixing some of these issues, especially those regarding numhop scripts.

Also, since the tests do not test GUI specific parts of the functionality. It is a good idea to open the "unittestmodel" in hopsangui manually and try to simulate it. It really does not catch all possible errors, but at least it will catch problems with up-converting of the model.