[core library] Iterable method first throws StateError, will returning null be more run time safe?

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
  /// Returns the first element.
  ///
  /// Throws a [StateError] if `this` is empty.
  /// Otherwise returns the first element in the iteration order,
  /// equivalent to `this.elementAt(0)`.
  E get first {
    Iterator<E> it = iterator;
    if (!it.moveNext()) {
      throw IterableElementError.noElement();
    }
    return it.current;
  }

Proposal
I would like to request a proposal of adding an additional method i.e firstOrNull, which would return null in case the iterable is empty
/// Returns the first element or null if this is empty.
/// Otherwise returns the first element in the iteration order,
/// equivalent to this.elementAt(0).
  E? get firstOrNull {
    Iterator<E> it = iterator;
    if (!it.moveNext()) {
      return null;
    }
    return it.current;
  }

as with this I won't have to worry about handling the StateError, now my only question is that will it be helpful to any other developer?

Hi @prashant-kellton2, this is a dart language/sdk issue and should be reported on their repo. See https://github.com/dart-lang/language/issues/836 & https://github.com/dart-lang/sdk/issues/42947 for similar issues around this topic.
Closing from here as this is out of scope for the flutter repo.
Thank you


[489] Rubocop ToDo: Rename Accessor Methods

Context
Clear up of Rubocop to do: https://trello.com/c/UhJu4WFi/489-clear-rubocoptodo-part-2
Changes proposed in this pull request
Renaming methods to comply with linter
Guidance to review
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Clear rubocop_todo part 2


Scrolling issue (Android)

Hello,
I've used very first example from docs but manipulating container to become 50% of screen width and center either with flex or marginLeft would make dropdown scrollable only on left side of it.
I've tried downgrading the package to 2.5.2 but that does not help.

I was having a similar issue using the MultiSelect component but for Android only.
Problem
Looks like a left view responsible for detecting the scroll-touching events is floating in the left part of the screen. You can check this behavior on the print screen below (I've added a red background to make it clear):

Solution
To solve this I removed the left prop from a <View> inside the lib component and put this prop into another child <View>. Below is the result after the solution is applied:

I don't have time now to fork the repository and create a PR, but I've created a patch to solve this issue and also this other one.
Here is the patch:
react-native-element-dropdown+2.9.0.patch

At me scrolling on Android desn't work as well


Binds

Edge Jump not working on Toggle, Slow Walk both working Toggle and Hold.
There is 64 number in the middle of my screen. Idk why

Edge Jump not working on Toggle, Slow Walk both working Toggle and Hold.
There is 64 number in the middle of my screen. Idk why

done

my bad, both works actually.
i'm stupid the key conflicted with others.
and for the number on screen i just removed this code


Add new scene import option to import as Skeleton

Adds a bool import option nodes/import_as_skeleton_bones.
This is supported in all FBX or GLTF document based formats. It is especially useful for retargeting and importing animations.
Examples:
Before:

After:

Fixes #81935
Fixes #62275 (if not already fixed by ufbx)
May mitigate problems such as #78028, though this is really an export bug
I suspect there are more issues

I love this so much! At some point I've spent countless hours trying to import an "empty nodes" animation from an FBX. Blender wasn't much help either, even with a "Better FBX" plugin. I think having this in core will save many users from a lot of headache.

Thanks!


Confirmable can set confirmation_sent_at without being sent

A before_create in Confirmable https://github.com/plataformatec/devise/blob/master/lib/devise/models/confirmable.rb#L36 calls generate_confirmation_token, which as a side effect sets confirmation_sent_at without sending the confirmation https://github.com/plataformatec/devise/blob/master/lib/devise/models/confirmable.rb#L224 .
An after_create in Confirmable https://github.com/plataformatec/devise/blob/master/lib/devise/models/confirmable.rb#L37 calls send_on_create_confirmation_instructions if: send_confirmation_notification? . send_confirmation_notification? could have @skip_confirmation_notification set to true, yet we still have a confirmation_sent_at value from the generate_confirmation_token in before_create .
The behavior I expect is, confirmation_sent_at only gets set when the confirmation is sent.
Maybe confirmation_token_generated_at would be a more appropriate attribute to store the token generation time. This would be in line with the comment @ https://github.com/plataformatec/devise/blob/master/lib/devise/models/confirmable.rb#L223 .

Yes, this issue has been opened in the past but we can't fix it without being very, very backwards incompatible. :( It is indeed poorly name. Can you send a pull request that at least documents this behaviour and explain why it stays as is?

will do.

https://github.com/plataformatec/devise/pull/3592

After documenting the Confirmable columns, it didn't seem necessary to explain why the confirmation_sent_at behavior won't be changed; it is what it is. Let me if you'd like a specific addition.


Error when submitting an answer

When I have submitted an answer, I got the following error:

But the answer was correctly saved.

Thank you for submitting the issue.
To help me identify the issue:

What was the question you were answering ?
Was there already an answer which you were improving, or was this the first answer ?


can this server  push message to client ?

I want to make a Tcp Server that can push message to client, and  I'm worry about how to save and mark every SocketChannel of client.
does your code have this function ? thanks.

Hi,
I am not sure if this design has that ability already, but I have made a
more advanced version of a TCP server which can do that -
based on exactly the same design. It is definitely possible to add.
The extended design assigns a socket ID to each socket, and inserts them
into a Map. Then each socket can be identified via that
id (just an incrementing long).  Also, the messages that are read from a
socket contains a reference to the socket (or socket ID)
of the socket they came from, so a reply can be sent back to the correct
socket.

Jakob Jenkov

On Sun, Jul 23, 2017 at 11:53 AM, mmmmar<EMAIL_ADDRESS>wrote:

I want to make a Tcp Server that can push message to client, and I'm worry
about how to save and mark every SocketChannel of client.
does your code have this function ? thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jjenkov/java-nio-server/issues/4, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADb7WaTagmqWiBloOJC7mEHCc1dBowy2ks5sQxgTgaJpZM4OgaMr
.

--
Jakob Jenkov
<EMAIL_ADDRESS>Founder, CEO
Jenkov Aps
http://jenkov.com
http://tutorials.jenkov.com
http://feeds2.feedburner.com/jenkov-com

@jjenkov yeah.  I have the same idea. I think it may be difficult to remove socket from the map when the socket disconnects with server,  so the heartbeat may be need to detect that.

Actually, this server already assigns an ID to the sockets and puts them into a Map. It also removes them again when the sockets are closed. Look at this code from the SocketProcessor:
    if(socket.endOfStreamReached){
        System.out.println("Socket closed: " + socket.socketId);
        this.socketMap.remove(socket.socketId);
        key.attach(null);
        key.cancel();
        key.channel().close();
    }


Add nodeselector for ansible jobs

This way we can run the jobs only on nodes where required NAD networks are available.
jira: OSPRH-10939

/test openstack-operator-build-deploy-kuttl

recheck

/test openstack-operator-build-deploy-kuttl

/cherry-pick 18.0-fr1


How do you  change the color palette of a .bmp and save a new copy?

Expected Behavior
I would expect that after running this code, test-new.bmp would be different and probably more red than the original image.
import jimp from 'jimp';

const image = await jimp.read('test.bmp');

image.bitmap.palette = image.bitmap.palette.map(c => {
    c.red = 200;
    return c;
})

const copy = await image.writeAsync('test-new.bmp');

Current Behavior
It looks identical to the original image.
Steps to Reproduce
Use the code sample above with a test.bmp in the same folder.
Context

Jimp Version: "^0.16.1"
Operating System: Windows
Node version: 14.15.4

It seems the .write method only reads from the buffer,  ie. image.bitmap.data. I don't see any way to set the color palette so that it updates the buffer.

If you use a black color at your font, you can use this method to change the color for what you want:
textImg.color([{ apply: 'xor', params: [color] }]);

What I would do is write the text to another jimp image of the same size, modify the color of that image, then blit it on top of your background


Calendar Selection Style similar to iPhone Expedia app

How can I modify the calendar class to allow user to choose the dates within Range.There is a nice intuitive way to allow user to choose the dates in between.
How this can be achieved using the Tapku library. It has really got such a smooth animation using CALayer class.
Thanks,

https://github.com/phaibin/Kal
I want to use this Calendar but given code, I'm not getting how to use in my app because this code is not in running mode so that I can check what is the flow of code.
Please help guys.


Enable Flow as a type checker

Check for existing issues

[X] Completed

Describe the feature
Currently, it looks like you only support TypeScript as a type checker, but there are a large number of projects that still use Flow. I myself work on a large React Native project and the speed of this code editor could really speed up development time.
If applicable, add mockups / screenshots to help present your vision of the feature

Flow has a built-in language server protocol support, so I guess it should be relatively easy to implement.
https://github.com/flow/flow-for-vscode/blob/c0438955bdf4d9ce98f614dd6edae088692cf040/lib/FlowLanguageClient/FlowLanguageClient.js#L145


Category detail sidebar

Is your feature request related to a problem? Please describe.
Opposed to Article detail, the category one does not have any sidebar. It was left out since category does not have many information to display like an Article.
Describe the solution you'd like
However a Category have language version too and they are not  displayed in detail. Also since there is no sidebar, an Edit button has not been added too.
Describe alternatives you've considered
It may not be a formal sidebar, something could  work too, it just have to be as ergonomic as the sidebar is.

Done in 0.6.0-pre.11, now Category detail have a sidebar to display possible available language, translate button and edit button


Only deactivate this route's shortcuts

Fixes issue #2 -- When leaving a route, only delete the shortcuts added by this route.
This allows shortcuts to be on routes that are not leaves.
All I do here is keep track of the name of the route that added the shortcut. When we deactivate a route, I delete all shortcuts that have that route name. This leverages Ember's constraint that all routes have unique names.

LGTM


Review Axiell Collections CMS

Part of https://github.com/wellcomecollection/platform/issues/5786
In order to better understand potential suppliers for a CMS, we should look at possible suppliers with a view to understanding risks and opportunities in relation to integration with the catalogue pipeline.
Filling out this Notion template.
RFI information available here.

Yes the above is basically it - The most pertinent information is contained in this folder
The spreadsheet questionnaire contains the responses to the scenarios we included in the RFI. The supplementary information has background stuff about the supplier.
Beyond this it would be worth looking at whether there is any documentation online, whether there are any obvious/active developer user groups etc.
All of the above limited to a few hours of investigation.

Had a look through the notion page. Looks like a decent overview given the available information.

Closing as done.


add support for golang 1.18 and 1.19

What problem are you facing?
panic error during make generate
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x50 pc=0x1294e56]

goroutine 1 [running]:
github.com/crossplane/terrajet/pkg/types.init.0()
    /home/rspazzol/git/crossplane-github-provider/.work/pkg/pkg/mod/github.com/crossplane/terrajet@v0.4.0-rc.0.0.20220510203225-5e7094f2ea5c/pkg/types/reference.go:73 +0x116
exit status 2
apis/generate.go:33: running "go": exit status 1
06:50:29 [FAIL]
make[1]: *** [build/makelib/golang.mk:271: go.generate] Error 1
make: *** [build/makelib/common.mk:419: generate] Error 2

How could Template Provider help solve your problem?

this error does not happen with go 1.17, it seems to happen with following versions of go 1.18 and 1.19.
template provider should be tested with those versions of go, or upgraded to them.

I also ran into this problem. https://github.com/crossplane/terrajet/pull/291 seems to correct it.
After that, I ran into another Go 1.18 problem:
ERRO Running error: buildir: failed to load package goarch: could not load export data: cannot import "internal/goarch" (unknown iexport format version 2), export data is newer version - update tool

This is resolved by adding GOLANGCILINT_VERSION ?= 1.48.0 to the Makefile (the version from upbound/build is older)

I'm also getting this error:
/bin/bash: line 1: /home/rspazzol/git/provider-jet-github/.cache/tools/linux_x86_64/goimports: No such file or directory
make[1]: *** [build/makelib/golang.mk:271: go.generate] Error 127
make: *** [build/makelib/common.mk:419: generate] Error 2

the make file should download the needed tools automatically but it doesn't for reasons I can't understand.
running go install golang.org/x/tools/cmd/goimports@latest does not seem to fix the issue.


ISSUE_TEMPLATE.md: Add mention of online formatter

People can try out https://black.now.sh/?version=master to test against master. That should make issue reporting easier.
See https://github.com/jpadilla/black-playground/issues/6#issuecomment-416088863. Thanks @jpadilla!

Pull Request Test Coverage Report for Build 676

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 96.109%

Files with Coverage Reduction
New Missed Lines
%

tests/test_black.py
1
97.77%

Totals

Change from base Build 675:
0.0%

Covered Lines:
2717

Relevant Lines:
2827

💛  - Coveralls

Nice, thanks!


Support more databases for backend store

Hi,
In addition to mysql, do we have plan to support other databases such as mongo as the backend data store?
Thanks

Perhaps something lightweight like sqlite would be beneficial?


Pages overlap too much

Reported in #1

I think that the problem is that if just one waypoint is outside of the pages, then the size of every cluster is reduced.
My current thinking is that once only a few waypoints are missed, then the process of starting again with more clusters should be abandoned and a new approach should be applied focusing on the waypoints that are missed.
I propose using the same algorithm, but apply it only to the missed waypoints

I implemented a simple, quick and dirty version of my proposal.
The results look very hopeful - the overlaps are greatly reduced for almost every page and the number of pages required is reduce by 50% or so.
I will work on implementing this properly.

I think the results are beginning to look reasonable

What do you think?

I have added the pseudo code for the two pass algorithm to the landing page
https://github.com/JamesBremner/Mapify/blob/main/README.md#algorithm

It does look a lot better. I was going to try a different approach, a modified k-means. After each iteration, in the calculation of the centroid, give greater value to points that are of that cluster but are outside of the page. Hopefully it moves the cluster towards points that are located outside of the page.

Hopefully it moves the cluster towards points that are located outside of the page

But it will also move the cluster so that some points that were covered will be uncovered.

If you want to pursue your own development, you should open a new issue for it and either fork the repo or create your own development branch.

@DM-UK I haven't heard from you in a day or so.  In the meantime I have worked on app usability, for details see https://github.com/JamesBremner/Mapify#gui

Hopefully it moves the cluster towards points that are located outside of the page

But it will also move the cluster so that some points that were covered will be uncovered.

I thought that if a cluster shifted towards an uncovered point, any points that it lost would be picked up by a neighboring cluster. But i'm not sure I've got the algorithm totally clear in my head yet (I'm not a professional developer, just a self taught hobbyist).
Do you have any suggestions to minimize overlap instead?
Here is a screenshot of a not quite perfect, but pretty good generation. This is just default k-means (with manual k number input) but shows under the right (rng) circumstances a good fit is possible.

I am curious:  what code are you running?    I support code that I have written or code from a pull request that I have accepted.  I cannot support code that I have never seen!
Here is a screenshot of the Mapify application

Unfortunately i'm a Java guy and have not used your app. I'm posting here as per your suggestion on stackoverflow. Apologies, if it's the wrong place.  Somewhere our discussion turned from algorithm specific to code specific.

When you accepted the invitation to join this repo, I assumed that you wanted to use this app, or develop based on the code in this repo.
Please remove the screenshot - it has nothing to do with this app.  You may want to invite me to join YOUR repo, so that I might comment on your code.

I have been working on a new algorithm where the first pass adds a new adjacent page to the last page which optimizes the number of new points covered without allowing any overlap.
This, of course, means that not all waypoints will be covered.  The missing waypoints are added in subsequent passes using the clustering algorithm.
First results look good, so I will close this issue.  The new algorithm is discussed in #3


About Pagination system

Describe the problem
Hello guys, When there are too many users, it will not fit on the screen. I think a pagination structure can be made. For example, when it exceeds 6 people, it needs to be posted on the 2nd page, in this way, the cost can be reduced and it can be more efficient.
Describe the proposed solution
We added manually but you should do this for library
Alternatives considered
No response
Importance
would make my life easier
Additional Information
No response

Hey @abbaslanbay, thank you for your suggestion!
Depending on your version, you may not have noticed, but we've added pagination with the usePagination hook to the GridLayout. Currently, the maximum number of visible participants is determined solely by the width of the GridLayout, but we plan to add some more customisation options (like maximum visible participants) to make it easier for developers to modify it.

Hi @Ocupe thanks for your response. However I didn't understant how can i use pagination system
For example that code
 <LiveKitRoom           token={token}           serverUrl={liveKitUrl}           options={roomOptions}           video={userChoices.videoEnabled}           audio={userChoices.audioEnabled}           onDisconnected={onLeave}         >           <VideoConference chatMessageFormatter={formatChatMessageLinks} />         </LiveKitRoom>
    Can you send me sample code

@abbaslanbay The VideoConference component is what we call a prefab. Prefabs are ready-made components that are built using other, more low-level components that you can use as a drop-in solution. They are opinionated and not as flexible/customizable as the lower level components. If you like the general direction of the VideoConferenccomponent, but want to make some changes to it that are beyond the options given to you by the properties. Then you can simply create a customVideoConferenc` component.
You would start by copying and pasting the existing implementation of the VideoConferenc component into a file, for example MyCustomVideoConferenc. MyCustomVideoConferencis now a drop in replacement forVideoConferencand if you replace it, nothing should have changed. With this setup you now have access to the implementation ofMyCustomVideoConferenc`, you can start modifying the implementation to your liking.
You can now drill down further and replace the GridLayout component in your MyCustomVideoConferenc component with your own implementation just like you just did with the VideoConferenc. If you do this, you will have all the freedom to build the implementation of a grid layout that you want.
We are still working on our documentation, but there is also a short guide on how to implement a custom component. https://github.com/livekit/components-js/blob/main/docs/alpha-docs/guides/custom-components.md

@abbaslanbay if there's additional things you'd like to see around the new pagination system, feel free to open another issue, I will close this one as done as we have pagination built in in the GridLayout component now.


[📚] [analytics] No replacement information for FirebaseAnalyticsObserver

With release of Firebase Analytics 9, FirebaseAnalyticsObserver class previously used to monitor navigation was removed. I failed to find any information what's the correct approach to observe navigation events and send them to Firebase Analytics service. I think it would be good to provide information what should be done to continue tracking screen changes in latest versions of the library

Maybe following something like that https://newbedev.com/flutter-how-the-navigatorobserver-class-works... It's weird to remove an API without explicit documentation about the new best way :/

Hey guys, after internal discussions, we've decided to add it back in. Apologies for the misunderstanding. PR to follow.

To resolve the issue, I have copied the FirebaseAnalyticsObserver class from the package firebase_analytics inside my project.


Declarative DOM event listeners broken for non-bubbling events on repeated elements

In Marko 3.12.1 and Marko-Widgets 6.4.1 looped elements with custom event handlers in a template are getting assigned the same id and in some cases only firing the event on the first element in the group.
A temporary workaround for me has been assigning the status variable on the for loop as follows:

New version published with fix<EMAIL_ADDRESS>


Ganache Block Generator

This PR adds a test-suite that can be used to generate blocks in Ganache. When run it guarantees that Ganache has at least 25 blocks of history, this is required so that the local tests that depend on Oracle can pass. The generator only needs to be run once anytime Ganache is reset to 0. You can save the state of Ganache after running the generator, and reset to that state instead, removing the need to run the generator again.

Closing as we are restoring the method that was initially done in #547 to resolve the issue.


Remove system-ui from font stack due to i18n issues

On Windows, system-ui causes all sorts of issues. This PR removes it for now, reverting #21356 and closing #22328.

system-ui seems to have made it back into the latest release
https://github.com/twbs/bootstrap/blob/72d3b6efc4b92e83a4abca6f5bc0cd7e6fd25931/scss/_variables.scss#L606

That's a good thing: the i18n concern around Chinese is unfounded. system-ui is either same as, or much better than (Microsoft Yahei UI) what you'd get with a bare sans-serif (SimSun). The "same as" case occurs only on Windows 7 and older -- check their 2024 market share.

It’s still a cross-platform concern. On Linux with no desktop environment, there is not even a way to set that.
Please do not use system-ui.


now alias is quite inconsistent in error messaging

Please take a look at this picture.

I have a Free plan, but also I was able to buy a domain 4 months ago. Your pricing page is saying - NO DOMAINS. How come I bought a custom domain on a Free plan?

First now alias attempt results in DNS Configured! Verifying propagation… and then Error!

Second attempt - Error! Please make sure that your nameservers point to zeit.world.. However, they do point.

Third attempt - Error! Custom domains are only enabled for premium accounts.. Why you are telling me that information so late?

Looks like I have to buy "Premium".
It would be good if you could check plan before aliasing a custom domain. Thank you.

Agreed. Any help is welcome!

Closing due to age, and the fact that custom domains are no longer a premium feature. Please open a new issue if there are still inconsistencies!


[Core] C++ register convenience signatures

Closes #853. Similar to recent work on resolve, we wish to make our users' lives easier by having overloads of register_ that model common usage scenarios. In particular, single-element and/or exception-less workflows.
So add these conveniences to the c++ API, with Python to follow in future work.
Description
Closes # (issue)

[x] I have updated the release notes.
~- [ ] I have updated all relevant user documentation.~

Reviewer Notes
Similar to resolve, although the signature is slightly different.
Have attempted to stick to standards established in https://github.com/OpenAssetIO/OpenAssetIO/pull/877 re: minor naming things.
The pybind declaration is already in the fully expanded form for register, so no need to amend it :)

Converted to "Draft" (in lieu of a "blocked" state) since there are changes in #877 that would be good to roll in.  In particular, making use of a re-usable fixture in Manager tests.

Converted to "Draft" (in lieu of a "blocked" state) since there are changes in #877 that would be good to roll in. In particular, making use of a re-usable fixture in Manager tests.

Is it worth spinning that out into a separate PR that updates resolve, then rebasing these onto that, or just re-jigging post #887?

Converted to "Draft" (in lieu of a "blocked" state) since there are changes in #877 that would be good to roll in. In particular, making use of a re-usable fixture in Manager tests.

Is it worth spinning that out into a separate PR that updates resolve, then rebasing these onto that, or just re-jigging post #887?

Good point, a separate PR is slightly more admin, but probably the right thing to do.

Converted to "Draft" (in lieu of a "blocked" state) since there are changes in #877 that would be good to roll in. In particular, making use of a re-usable fixture in Manager tests.

Is it worth spinning that out into a separate PR that updates resolve, then rebasing these onto that, or just re-jigging post #887?

Good point, a separate PR is slightly more admin, but probably the right thing to do.

Added https://github.com/OpenAssetIO/OpenAssetIO/pull/884

Converted to "Draft" (in lieu of a "blocked" state) since there are changes in #877 that would be good to roll in. In particular, making use of a re-usable fixture in Manager tests.

Is it worth spinning that out into a separate PR that updates resolve, then rebasing these onto that, or just re-jigging post #887?

Good point, a separate PR is slightly more admin, but probably the right thing to do.

Added #884

Rebased (and autosquashed) on main, making use of the new fixture.


Update discv5

For some reason, our libp2p ts declaration is erroring after this update, not why :(
https://github.com/ChainSafe/lodestar/pull/1105/checks?check_run_id=843676201#step:5:381
The latest version of discv5 is a small bugfix that doesn't effect the typescript interface.
libp2p-ts has also been updated to this latest version of discv5.

For some reason, our libp2p ts declaration is erroring after this update, not why :(
https://github.com/ChainSafe/lodestar/pull/1105/checks?check_run_id=843676201#step:5:381
The latest version of discv5 is a small bugfix that doesn't effect the typescript interface.
libp2p-ts has also been updated to this latest version of discv5.

Seems like, there is some disagreement in discovery options. If I remmber correctly, all discovery modules options needs to be merged as well

libp2p-ts hadn't updated in yarn.lock -_-


v0.14.2 uses marked with moderate vulnerability Regular Expression Denial of Service

When running npm audit while using version 0.14.2 shows a vulnerability with the marked package. This is fixed in version >=0.6.2
Vulnerability:
Regular Expression Denial of Service
https://nodesecurity.io/advisories/812

Thanks for the report however I believe this is a duplicate of #1009. It should be fixed in the next release.


Add offset for TextureProgress progress texture

Closes https://github.com/godotengine/godot-proposals/issues/823

If I have to be entirely honest, I think TextureProgress kind of became far too bloated with functions to the point we will never make everyone happy and eventually an add-on may be  better fit. I understand why this is needed and it makes sense, though, so I would favor merging it.

Need to fix the XML doc to pass CI

Thanks!

Cherry-picked for 3.4.


Resolve Trait.gwas and Trait.qtlStudy

Single-purpose: populate the Trait.gwas and Trait.qtlStudy fields (finishing off Trait and resolving #4).
This PR should be merged AFTER #19 as I did this edit on top of that branch. (I think it would merge fine either way but this is the order I did things.)

Ah, right, this includes those other commits since that PR is still pending. It diffs against main of course. Anyway, this is an add-on PR.


20221004 AnsiString compile errors

I'm guessing this was caused by merging Sergey's recent updates.
I will make an effort to fix the problem.

Yes, this was my fault. @robnagler -- it would be helpful if you could work with me on this to help me understand, so I don't do this again.

I'll take a look.

Need to create methods that do this:
static AnsiString GetFileName(AnsiString &F)   //Gets the file name from path
{
  return F.SubString(F.LastDelimiter("/\\")+1,F.Length());
}
//---------------------------------------------------------------------------
static AnsiString GetFileCaption(AnsiString &F)   //Gets the file name without extension
{
  return F.SubString(0,F.LastDelimiter("/.")-1);
}

Substring and LastDelimiter are not in our mock AnsiString.cpp so need to either create them or create these file manipulation functions.

@bruhwiler are Sergey's newest changes available on some branch?

Yes. The easiest way to see the changes (I think) is too look at file differences from the recently closed (and subsequently reverted) PR, https://github.com/radiasoft/rslinac/pull/109/files

kutsaev_2021b

Opened https://github.com/radiasoft/rslinac/pull/116 to merge changes into kutsaev_2021b


chore: switched buildspec to publish to npm

PR switches CI process to publish releases to npm instead of GitHub packages

Does the change effect our internal CI process or is it only the open-source community tools?

Now that this project is open-sourced and available to the public we are no longer going to publish the release to a private github package. This PR publishes it to npm for the public.


Added keywords grammar

Part of issue #84.
The links to this new page (instead of linking to the original grammar keywords section) will be corrected in each corresponding section pull request.

Rendered here.

Thank you!


gha: merge artifacts in net-perf-gke workflow

The net-perf-gke workflow is currently broken because multiple matrix entries attempt to upload the same artifact. Let's fix this by using a different name for each of them, and adding a step which takes care of merging the artifacts at the end.
Fixes: a23106d28619 (".github/workflows: report which features are being used in CI")

/net-perf-gke

/test


refactor: remove obkv dependency

Fixes #29 by removing all macros altogether

See if https://github.com/meilisearch/milli/pull/707 gets merged before merging/completing this. Might be able to remove the macro I added

This PR is all set to go; just waiting on a new milli version with the all_obkv_to_json in it.


HIVE-26464: New credential provider for replicating to the cloud

What changes were proposed in this pull request?
HIVE-26464
Why are the changes needed?
It is needed for on-prem to cloud replication
Does this PR introduce any user-facing change?
No
How was this patch tested?

I've tested it on a real cloud environment, replicated a test database from on-prem to AWS
Added new JUnit tests

There's a Javadoc generation failure and few test failures. The timeout test is an unreliable one, so it's not relevant.
My +1 is valid when the Javadoc issue is resolved, and the TestCliDriver[hybridgrace_hashjoin_2] – org.apache.hadoop.hive.cli.TestMiniTezCliDriver failure is resolved.


Failed to find ChromeDriver

I believe this isn't due to a configuration error on my end, but it can't find ChromeDriver today (2024/4/25). Thank you for maintaining.

I think you are referring to this issue which was just fixed https://github.com/jpjacobpadilla/Google-Colab-Selenium/issues/6


TypeError: 'type' object is not subscriptable in python 3.8

After install 0.2.1, from taskgroup import TaskGroup as AsyncioTaskGroup will fail with this error:
(py38) ➜  ~ python3
Python 3.8.19 (default, Mar 20 2024, 19:58:24)
[GCC 11.2.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from taskgroup import TaskGroup as AsyncioTaskGroup
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/yajieyan/miniconda3/envs/py38/lib/python3.8/site-packages/taskgroup/__init__.py", line 9, in <module>
    from .runners import run, Runner
  File "/home/yajieyan/miniconda3/envs/py38/lib/python3.8/site-packages/taskgroup/runners.py", line 22, in <module>
    from .tasks import task_factory as _task_factory
  File "/home/yajieyan/miniconda3/envs/py38/lib/python3.8/site-packages/taskgroup/tasks.py", line 18, in <module>
    _TaskYieldType: TypeAlias = Optional[asyncio.Future[object]]
TypeError: 'type' object is not subscriptable

Instead version 0.0.0a4 works.


fix name property of wandb logger return project name bug

What does this PR do?
Fixes #12157
Does your PR introduce any breaking changes? If yes, please list them.
None
Before submitting

[x] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

May I ask why is this useful fix not being merged :(

Another PR has been made which does the same thing #12604


Play on CPU?

HI! So, I have a pretty beefy computer in terms of everything but the GPU, which is an AMD card. I have 24 gigs of ram and a pretty powerful CPU though.
Would it be possible to modify the code to run on CPU? (and possibly modify scripts to run on Windows without needing Bash?)
I find it annoying whenever I get an OOM error in Colab, so it'd be great if I could run it locally.
Edit: Time is not an issue for me, I just want to run it locally.

have you tried installing regular tensorflow?

I have it currently running with CPU fine.

I have it currently running with CPU fine.

Ohh, I totally forgot that TF bundles the CPU and GPU versions together in one package now.
Thanks!


Do you have document on how to use Cosette with Calcite RelNode in Java?

The process of converting from Calcite RelNode to xxx.cos file is done manually. The process is documented at https://github.com/uwdb/Cosette/blob/master/examples/calcite/calcite_test.md.
Processed cases are stored in https://github.com/uwdb/Cosette/blob/master/examples/calcite/calcite_tests.json.
These processed cases are then later converted to xxx.cos file using the script https://github.com/uwdb/Cosette/blob/master/gen_cos_files.py.
Let me know if you have any other questions.


chore: Allow GET requests from omniauth

So this works! I removed the lines from volt's initializers/omniauth.rb file and stuck them in here.
I think next steps would be to upgrade this gem in kinetic and artsy-auth, then upgrade those gems in affected repos.

I think we can bump the version here (and maybe even mark it as a security update so dependabot will submit PRs automatically in client repos). We probably don't need to update kinetic or artsy-auth unless there is version requirement mismatch.

On 2nd thought, you could avoid thorny version constraints by just leaving this to consuming apps:
OmniAuth.config.allowed_request_methods = %i[post get] if OmniAuth.config.respond_to?(:allowed_request_methods=)
OmniAuth.config.silence_get_warning = true if OmniAuth.config.respond_to?(:silence_get_warning=)

@starsirius having a hard time figuring out how to mark this version as a security update for dependabot, could you point me in the right direction?

I haven't done this before but I think we will need to PR to ruby-advisory-db (which is where dependabot is sourcing from) for omniauth-artsy. I'm happy to pair on this and try it out!


Fix less tailer URLs for large files

Several log links didn't take file size into account when they should have.
cc - @ssalinas

🚢


UI improvements related to transactions with bad nonce

I accidentally experienced an unusual behavior due to bad timing, that could be very confusing to an average user that didn't realize what was going on in the background, and I think some change in the wallet (maybe also in the explorer) could help improve that user experience.
Here's what happened: I submitted a transaction and waited a while. Then, I got impatient and increased the fee. Between the last time the wallet refreshed and when I submitted the RBF transaction, the original transaction was confirmed. This leaves my RBF transaction pending forever, since it now has an invalid nonce. It would be great if the wallet could detect this scenario and report when a transaction that is pending actually has a bad nonce (and consequently should not be confirmed). Similarly, the explorer could add some UI to show this case.
Correctly, if I try to increase the fee on that transaction again, the wallet will report that it is a bad nonce and refuse to submit an RBF transaction.

Oops. Thanks @andresgalante

Thanks @obycode, that is a valid point and can confuse the user.
What happens is that there are some pending transactions shown in our activity list that will never be mined and should never be acted upon (no increase by fee) and this could be confusing.
When we know the transaction cannot confirm, we could:

change the color/text of this forever 'pending' transaction
remove the 'Increase fee' button in that case.


feat: Argument override

Context
Advanced users want to be able to pass arguments to FFmpeg directly.
Objective

Add a args option that overrides all synthetic options created by simple-thumbnail

References

Original PR by @cmd430: https://github.com/ScottyFillups/simple-thumbnail/pull/58

:tada: This PR is included in version 1.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[WIP] Update moment-strftime to 0.2.0

We've been using my fork of moment-strftime because 0.1.5 had a dependency conflict with patternfly.
This got fixed in benjaminoakes/moment-strftime#19 - changing back to use upstream.
WIP until 0.2.0 actually gets released..

0.2.0 is the latest of 7 releases

:smile:


changed url reference and resource

change 1: call login endpoint for getting marketplace token not marketplaceapi.
change 2: the resource in that call should be 62d94f6c-d599-489b-a797-3e10e42fbe22

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

@qianw211 Can you please follow up your communication with this PR instead?

@santhoshb-msft Could you please just keep the 1st change, as the 2nd isn't correct.  Thanks!
change 1: call login endpoint for getting marketplace token not marketplaceapi.
change 2: the resource in that call should be 62d94f6c-d599-489b-a797-3e10e42fbe22

change 2: the resource in that call should be 62d94f6c-d599-489b-a797-3e10e42fbe22
@qianw211  This is what all my ISV's currently use and only works with this. Is this changed? This is would be worst nightmare and would be a breaking change for all the partners using this already. Could you please double confirm on this?

@santhoshb-msft The old resource will still work but all new ISVs should use the new resource.  And the same is true of ISV is updating the code. No breaking changes just a forward-facing update.

@qianw211 awesome that is a great insight, I will update the PR changes accordingly! Much appreciated!

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

#sign-off


Dataframe display: displaying the row index

When displaying pandas data frames on the dashboard, I would like to have the row index (same as the column index that already appears):

Yeah, it's a problem indeed. We could at least send the index over as a column when we find it.

The index is already sent to the frontend because we use the tight orientation during serialization:
>>> df.to_dict('tight')
{'index': ['row1', 'row2'], 'columns': ['col1', 'col2'],
 'data': [[1, 0.5], [2, 0.75]], 'index_names': [None], 'column_names': [None]}


feat: add vpc_owner_id to outputs

Description
this will just add owner_id output to module
Motivation and Context
I use owner_id/Account ID in different policies and building dynamic variables, for example
codebuild
  buildspec = templatefile("files/buildspec.yml", {
    aws_region   = var.region,
    ecr_registry = "${module.vpc.owner_id}.dkr.ecr.${var.region}.amazonaws.com/${module.label.id}-ecr"
  })

or iam policy
"Resource": [
                "arn:aws:ecr:${var.region}:${module.vpc.owner_id}:repository/ecr-${var.plat}/influxdb",

Breaking Changes
none
How Has This Been Tested?
tested calling from other resources and modules as in example above

Thanks @vmalinics0 for this addition!
v2.38.0 has been just released.


Provides ember g command for building an application adapter

this is coming in ts-ember-cli-addon-refactor
#120


Proper systemerror follow-up

Adds a custom ErrNoSuchContract error, and map it to the corresponding wasmvmtypes error, as discussed in #701.
Needs feedback on registering the error, proper error names / types, and extending this mechanism to other errors.

I am discussing with @maurolacy how to make this work with wasmvm systemerror without circular dependency.

I am discussing with @maurolacy how to make this work with wasmvm systemerror without circular dependency.

Right now he converts the standard type to the expected wasmvmtype so that ToSystemError works properly.
I think any other approach that would make ToSystemError conscious of this custom type and type conversion would require at least as much code and likely be more fragile.
The only thing I could think of to make this more extensible (if we do need to use this multiple places in the codebase) is to pull out these lines into a MapErr function.


Added C++11 CXX flag and changed the output path for Windows

Added Flags
Also changed the output path for Windows, since (CMake) adds the Release and Debug folder into the plugins folder which ends up with sampctl not being able to find it.
Removed includes for removed libraries (natives-plugin)

I still feel we should provide what is needed to get this boilerplate building as it is, the changes should then be changed by the developer so providing -std=c++11 just for the project to build as is.


\r\n or \n

CNV files can be created with breaking line pattern: "\r\n" or "\n"
I left some parts of the parsing rule without the optional \r
Thanks to Hotin David for the example file

Resolved at 0.6.2


Error while starting Settings Sync: No authentication provider 'github' is currently registered

On Mac, when I try to sync settings, which requires login first, I get this error: Error while starting Settings Sync: No authentication provider 'github' is currently registered. Which is quite similar to this one. expect @carlocardella seems already get the Github logged in.
Both of the auto-redirect and manually copy paste generated token did not work.
I've already checked Keychain access, seems like there is nothing related to vscode.
Any idea on how to solve this? please.

Originally posted by @PulsGarney in https://github.com/microsoft/vscode/issue_comments/705649913

I have the same issue at Linux

I have the same issue at Linux

I have solved .I have created token via my github profile and I have copied its and pasted to vs code's token area

Same here.
I also got

I have this issue on Clear Linux with Visual Studio Code stable.

I have the same issue at windows 10, but I tried a few more times, and it worked miraculously. maybe it was a problem with the previous link redirection

I have this issue on Fedora with 1.50.1

I had the same issue with Ubuntu after install vscode via snap. Copy the token and remove the trailing white space at start and final from before use it worked for me.

Same here.
I also got

had never seen this before, could be another issue

Somehow I manage to login with the latest release now, with the exact procedure I did before.
I think it is fixed on Mac now. 👍

Can others who are seeing this issue (specifically, "No authentication provider 'github' is currently registered", not the 801 error) try using the insider's edition of VSCode? Do you see the problem there as well?

Me too

I had the same issue with Ubuntu after install vscode via snap. Copy the token and remove the trailing white space at start and final from before use it worked for me.

This worked for me.

I am tentatively closing this since it seems people have been able to sign in. The original problem in this issue is specifically the "No authentication provider 'github' is currently registered" error, I believe I have fixed it. The 801 error is tracked here: https://github.com/microsoft/vscode/issues/108304

I still have the same issue:
Version: 1.51.0
Commit: fcac248b077b55bae4ba5bab613fd6e9156c2f0c
Date: 2020-11-05T18:16:10.374Z
Electron: 9.3.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 5.4.0-52-generic

Hello, experiencing the same issue on MacOS, even creating a custom token does not work.

I see the same error on a fresh install of the latest .deb, version 1.51.1, on Ubuntu 18.04.
https://github.com/microsoft/vscode/issues/108304 can be fixed by:
rm -rf ~/.vscode ~/.vscode-server ~/.config/Code


build failed when building React app

What did you do?
build prometheus in a centos-like distro
What did you expect to see?
#make build
cd web/ui/react-app && yarn --frozen-lockfile
yarn install v1.22.4
[1/4] Resolving packages...
[2/4] Fetching packages...
info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>[4/4] Building fresh packages...
Done in 19.90s.

building React app
building React app
yarn run v1.22.4
$ react-scripts build
/home/jinnan.wjn/web/ui/react-app/node_modules/react-scripts/config/webpack.config.js:305
...(isEnvProductionProfile && {
^^^

SyntaxError: Unexpected token ...
at createScript (vm.js:56:10)
at Object.runInThisContext (vm.js:97:10)
at Module._compile (module.js:549:28)
at Object.Module._extensions..js (module.js:586:10)
at Module.load (module.js:494:32)
at tryModuleLoad (module.js:453:12)
at Function.Module._load (module.js:445:3)
at Module.require (module.js:504:17)
at require (internal/module.js:20:19)
at Object. (/home/jinnan.wjn/web/ui/react-app/node_modules/react-scripts/scripts/build.js:38:23)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
make: *** [web/ui/static/react] Error 1

seems like build environment issue


Add option to restore recent monitors

It would be cool to have a "recently used" menu to restore closed monitors^.

I have implemented monitor saves in recent commits, you can give it a try.

If I didn't miss anything, then that is just restoring monitors from the last run. Which is really awesome in it's own! But what I'm suggesting is a menu to restore widgets that the user closed. I sometimes close widgets by accident, and especially if it was based on a complex expression it would be nice to easily restore it.

The suggestion reminds me the fact that it lacks ability to remember user input histories on disk, which is important.

I'll add a window to track input expression histories which you can copy and even edit, this should be a more general feature.

Sounds good to me! Not sure how much effort it would be, but making it multi-select would make it easy to select a bunch of expressions and create monitors for all of them

This is the result, https://github.com/Ark2000/PankuConsole/issues/68
I'm pretty sure it will be very useful.

That is really awesome! The only thing I would like to have would be a quick way to create monitors based on them. Maybe next to the "Copy" button there could be a "Monitor" button. That way it would be quick to create monitors without first going to the console and pasting the expressions. But that's just a minor convenience nitpick, it's already super useful


[improve][broker] Use HierarchyTopicPolicies to update topic-level subscribeRate

Motivation
Use HierarchyTopicPolicies to update topic-level subscribeRate
Modifications
Use updateSubscribeRateLimiter() to update topic-level subscribeRate, instead of initializeTopicSubscribeRateLimiterIfNeeded.   After this Modification,  broker-level, ns-level and topic-level subscribeRate updating all use the same method updateSubscribeRateLimiter()
Verifying this change

[x] Make sure that the change passes the CI checks.

This change is already covered by existing tests, such as (please describe tests).

org.apache.pulsar.broker.admin.TopicPoliciesTest#testDisableSubscribeRate

Documentation

[x] no-need-doc
(Please explain why)

@Jason918  @gaozhangmin  @codelipenghui  PTAL


Add documentation on how to use postgres storage in Node.JS

PR #699 adds support for using a postgres wallet type in Node.JS. It would be nice to add some documentation on how to install the postgres plugin on different platforms, and also on how to use it with AFJ.
@sairanjitAW is this something you're willing to take a look at?

@TimoGlastra Yes. I will add documentation for the same.

Awesome, thanks!

Yes, I was able to add for Linux and Mac but for windows, I was not able to prepare as of now. I will add the PR for that today itself.


Fix broken(?) link on Type Compatibility page

Add /docs prefix to link href because without it I'm getting "The resource you are looking for has been removed, had its name changed, or is temporarily unavailable.". (I'm assuming this isn't a "temporarily unavailable" situation, but if it is then disregard.)

Yeah, this change is the right one - thanks!


0.1.8 build number 1

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

requirements: host: python >= 3.6 should not contain a space between relational operator and the version, i.e. python >=3.6
requirements: host: numpy >= 1.17.4 should not contain a space between relational operator and the version, i.e. numpy >=1.17.4
requirements: run: python >= 3.6 should not contain a space between relational operator and the version, i.e. python >=3.6
requirements: run: numpy >= 1.17.4 should not contain a space between relational operator and the version, i.e. numpy >=1.17.4
Non noarch packages should have python requirement without any version constraints.
Non noarch packages should have python requirement without any version constraints.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


SuperclassMemberReferencesCrossModule test fails

https://github.com/Microsoft/python-language-server/pull/96


Linestring unexpected behaviour

Hello all,
I'm trying to simulate a cluster. I got 2 (or more) markers on same coordinate and on mouseover I show them a little bit separated, joined by linestrings.
[Live Demo] (http://plnkr.co/edit/CuDYJe?p=preview)
So, steps to reproduce:

Mouseover on marker
zoom in and zoom out

several times

Unexpected

Expected

Thanks for your work.

I can't reproduce. It seems like your plunker link is different from your screenshots, since it looks different, don't expand on mouseover (but on clicks) and only expands to one direction.

You're right. Fixed now. Same link, plz.

You forgot to remove/clear the old linestrings in the multiLineString geometry, so the old lines were always kept even when zoom changed.
Try adding multiLineString.setCoordinates([]) to line 205: http://plnkr.co/edit/uA75itjtIISpwKiVc0C4?p=preview

@alvinlindstam, you're right again. Solved.
Thank you man.


[MODFISTO-481/MODFIN-371] - Improved cashBalance, available and unavailable calculations

Purpose
Need to change cashbalance, avaliable, and unavaliable calculations after credited feature
Approach
available = totalFunding - encumbered - awaitingPayment - expended + credited
unavaliable = max(encumbered + awaitingPayment + expended - credited, 0)
cashBalance is similar with available

@azizbekxm You should check out this PR to test results. The mod-finance tests should pass at this point, and possibly some others.

image passed for this tests https://github.com/folio-org/folio-integration-tests/pull/1422 passed this pr


update two model with single mutation function

Which Category is your question related to?
Amplify api resolvers
What AWS Services are you utilizing?
Amplify, GraphQL,DynamoDb
Provide additional details e.g. code snippets
I have following models:
 type Service @model {
     id: ID!
     reviewsCount: Int!
     reviews : [ServiceReview] @connection(name: "ServiceReviewsCon")
 }

 type ServiceReview @model{
     id: ID!
     comment:String!
     service:Service  @connection(name: "ServiceReviewsCon")
 }

How can I increase reviewsCount whenever I create a new ServiceReview?
I think writing resolver in this case is not going to work because what I need is changing two tables at once.. am I right ?! If not can you point me in the right direction for solving it?

This is not supported out of the box but you can write a pipeline resolver to implement this. You may also call a lambda function from an AppSync mutation and have that lambda function implement the logic.

@mikeparisstuff  I think amplify does not support pipeline resolver yet . Am I right ?
With lambda solution, should I do both insert and  update in the lambda ? Is it possible to insert data using default query and just update other table with lambda? ( Currently, I have lambda with my logic and I'm trying to edit the Stack part of amplify's GraphQL. but I can't find out how to do it  )
Unfortunately, I'm very lost here, shouldn't this kind of tasks be much more easier ?

@uchar You can use custom stacks to implement pipeline resolvers within Amplify. These docs discuss adding a custom resolver (https://aws-amplify.github.io/docs/cli/graphql#add-a-custom-resolver-that-targets-a-dynamodb-table-from-model) and you can follow a similar pattern but replace it with a pipeline resolver.


[Registration Module Frontend] Prepare Sign up and Login Pages with Validation

Sign up Page
The sign up pages should contain a form collecting details including

[x] Full Name
[x] Email
[x] Phone Number
[x] Password
[x] Confirm Password

Login Page
Prepare Login page according to the UI design by Shammah
BOTH SIGN UP AND LOGIN PAGE SHOULD BE RESPONSIVE AND MOBILE FIRST

done


add URL Dependencies support for module federation(consume shared module)

For module federation shared, without requiredVersion configured, it will infer version base on package.json, as we can see in ConsumeSharedPlugin.js.
But when we get requiredVersion from package.json, it just get dep / dev / peer / optional dependencies value, without consideration of some URL Dependencies cases.
For example, "isarray": "git+https://github.com/juliangruber/isarray.git#v2.0.3", it's real version should be v2.0.3, but we can not get that.
This pr will add a good support for URL Dependencies with versions.
What kind of change does this PR introduce?
Feature, add URL Dependencies support for module federation.
Did you add tests for your changes?
Already did.
Does this PR introduce a breaking change?
No.
What needs to be documented once your changes are merged?
Nothing.

@alexander-akait  Sorry but I do not get you, maybe I should provide a demo package or add some test cases?

https://github.com/npm/hosted-git-info

@alexander-akait Thx, I know, I will add handler for more edge cases, and it will be more simple than hosted-git-info.

@alexander-akait Already add more edge cases handlers according to hosted-git-info. And tests can cover all the cases provided by hosted-git-info.

@alexander-akait Changes intergration and project intergration checks couldn't pass, maybe that's because I fetched some commits from main branch? Could you help giving some clue?

Hi, great work, we will need support for this at my company too, but we have self hosted version of bitbucket at bitbucket.companyName.com, from a quick look at code it looks like it wont be supported.
Therefore, could you support adding custom parsers into extractCommithashByDomain via webpack options?


Error on Start up of HA 2024.6.0

Just updated HA Core to: 2024.6.0
Getting this error on start-up
2024-06-06 14:52:35.986 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration givenergy_local which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2024-06-06 14:52:54.980 ERROR (MainThread) [homeassistant.setup] Setup failed for custom integration 'givenergy_local': Requirements for givenergy_local not found: ['pydantic==1.10.12'].

Apologies - Duplicate of #83

Duplicate of #83


minor styling updates to the compliance table styles

Coverage remained the same at 27.615% when pulling 98690a22e88c3ac74fd0af5f98324c2864739889 on theseyi:styling-updates into b3256082c20e977099e93bf327192c037e21ea16 on linkedin:master.

Codecov Report

Merging #1229 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1229   +/-   ##
======================================
  Coverage    8.24%   8.24%
======================================
  Files          60      60
  Lines        2280    2280
  Branches      483     483
======================================
  Hits          188     188
  Misses       2092    2092

Continue to review full report at Codecov.

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


Add post

Please add this post:
Post URL: https://kholdstare.github.io/technical/2020/05/26/faster-integer-parsing.html
Author Name: Ivan Tham
Categories: Performance
Optional:
Blog RSS/Atom/JSON Feed: https://rust-malaysia.github.io/feed.xml
Providing a feed will help me discover new posts in the future.
If you shared your post on Twitter and/or the Mastodon/the Fediverse include links below:
Tweet URL:
Mastodon URL:

Thanks! Posted.


All activity elapsed/moving time now showing in seconds

Not 100% sure when this changed and things its been ok since 2024.12.1 but not something I appear to have the ability to change at the moment without possibly invoking some sort of template

not sure if that has anything to do with issue #188 and any Strava API change, especially with all their upcoming 3rd party changes
For clarity, just did the recent 2024.12.3 update and still the same

for clarity, the total move distances hasn't changed either and not added to that issue

This would be a different issue to the API issue.
It will have been introduced by a core change.
The time has always been stored in seconds and HA was displaying this in a human readable format automagically.
I will see if there is a change I can make to force the human readable formatting again on that sensor

I'm sure you already have seen this but still the same in 2024.12.4

Hi @jouster1974
I have opened a bug with HA core project https://github.com/home-assistant/frontend/issues/23338 as the entities we have in ha_strava are configured correctly, according to the documentation, to show up in human readable format on the HA frontend.

Hi @jouster1974
This change was made by @piitaya here https://github.com/home-assistant/frontend/pull/23025. To help get this change rolled back or improved to use the human formatting by default, please add your comments to this PR and the bug https://github.com/home-assistant/frontend/issues/23338.
This change has been made in a way in core that I cannot revert in the integration.
Thank you

Hi @jouster1974
This change was made by @piitaya here home-assistant/frontend#23025. To help get this change rolled back or improved to use the human formatting by default, please add your comments to this PR and the bug home-assistant/frontend#23338.
This change has been made in a way in core that I cannot revert in the integration.
Thank you

this has been logged as requested.....for info, my YTD run and ride distance seems to be going backwards and now shows as almost zero for both and ALL distances are as before far away from where they should be

I assume this is still a Strava API issue (related to their API changes) but thought I would mention as I saw a new update today


meta-balena-genericx86: Set linux-yocto preferred version to 5.10.43

Set the preferred kernel version to the newest version available.
Change-type: patch
Signed-off-by: Alex Gonzalez<EMAIL_ADDRESS>

@floion this assumes we want the newest linux-yocto available for all DTs that don't specify their own preferred versions, currently:
conf/machine/surface-pro-6.conf
8:PREFERRED_VERSION_linux-yocto = "5.0%"

conf/machine/surface-go.conf
8:PREFERRED_VERSION_linux-yocto = "5.8%"

It looks like some of the required PR checks were removed from this repo?


Messages with attachments get on top regardless of date

Expected behavior
Messages should be sorted by date regardless of anything else.
Actual behavior
Messages with attachments are sorted by date at the top, and then the rest of the messages are sorted by date below those.
Steps to reproduce
1.Open K-9 Mail
2.Click on folder view
3.Enter any folder
Environment
K-9 Mail version: 5.600
Android version: 7.0
(Security patch: 5th August 2017)
(Kernel version: 3.18.35)
Account type (IMAP, POP3, WebDAV/Exchange): IMAP (Gmail accounts)

Sounds like you accidentally changed the sort order to sort by attachments.
Go to the message list screen, click on the icon with three horizontal lines getting shorter towards the bottom and select "Date".


Enhancements of Chinese and Japanese Citekey Generation

Jurism version: 5.0.93 (latest)
BBT version: 5.4.29
Support log ID: Z3F7ALHY-refs-euc
Overview
Hopefully this isn't too many things in one issue, but since these feature enhancements I am requesting are all pretty similar, I thought I would just put them all here.
First, a quick review (sorry if this is all old news, just want to make sure we're on the same page because working with logographic writing is difficult to wrap your head around if you don't speak a language that uses it):
Citekey generation for Chinese and Japanese is not trivial for two reasons: 1) there is no whitespace separation for words so the computer cannot tell where they start or stop without some natural language processing, 2) the readings of logograms (i.e. "Chinese characters", hanzi or kanji) depend on context. Logograms write meaningful parts of words. Sometimes one logogram=one word, but more often in modern Chinese and Japanese, a word will be written with 2 or more logograms. You have to know what word these logograms are transcribing first before you can reliably create a romanization for them. To get around these difficulties, generating citekeys for each should be a two-step process:

Segment sentences-like-strings (like titles)  by word.  For example, in js, you'd use jieba for Chinese, and one of the Kuroshiro analyzers like kuromoji, mecab, yahooWebApi, like you are doing now.

Then feed the segmented words to a romanization method that can intelligently predict the proper reading of the logogram by the context of the word using a dictionary. Right now, you have have the transliteration package doing this for Chinese and Kuroshiro doing it for Japanese.

Chinese
Both steps (1) and (2) are separate and accessible through filters in citekeys (:jieba, :transliterate/clean) for Chinese. However, step (2) is not useful because the transliteration package does not predict romanizations for Chinese based on words. It just picks the commonest reading for each character. This generates a lot of incorrect outputs. E.g.:

朝 alone has the readings "zhao" or "chao", but in the word 朝陽 "chaoyang" it is always read "chao." Transliterate incorrectly yields "ZhaoYang."
的 can be read "de" or "di", but in the word 目的 "mudi" it is always read "di." Transliterate gives "MuDe"

To make step (2) more useful, I recommend using the pinyin package. https://www.npmjs.com/package/pinyin. The documentation is in Chinese, but I would be happy to help you get it working.
Japanese
Steps (1) and (2) are not separable for Japanese. The way BBT uses kuroshiro now, it segments and outputs romaji in one call. It would be nice if there were two filters so that the use could stop at step (1) and choose a space separated string of Japanese text for titles (like you can currently do for Chinese with jieba), or have it give space separated romaji like it does now. For just segmentation, this could probably be done with kuromoji alone. Again, I'd be happy to help with Japanese language documentation and troubleshooting.
One more nitpick is that it seems like the kuromoji analyzer performs rather poorly with proper nouns like surnames compared to mecab (no benchmark to back that up, just my impression). Switching to mecab would probably hog even more memory and be difficult to implement, but it might be worth considering since getting proper readings of surnames it important for citekeys.
Summary
Here's what I propose:

Use the pinyin package instead of transliterate to handle romanization of Chinese.
Allow users to be able to select japanese words in titles by white space inserted with a sentence segmenter like kuromoji (basically have a filter for Japanese that works like the Chinese jieba filter).
Maybe think about using a different analyzer for Kuroshiro to improve romanization of Japanese surnames.

And one last P.S., per my previous issue, Chinese and Japanese romanization needs to be handled separately (maybe be looking at the item.language?). Right now, if you have the Kuroshiro option checked it overwrites everything so you can't generate citekeys properlyfor Chinese and Japanese items at the same time.

I'm open to trying this, and indeed I'll need your help reading these API documents.

1. Use the pinyin package instead of transliterate to handle romanization of Chinese.

If this is an pure-javascript package, no problem. I can only use pure-javascript packages.

2. Allow users to be able to select japanese words in titles by white space inserted with a sentence segmenter like kuromoji (basically have a filter for Japanese that works like the Chinese jieba filter).

I don't currently understand what change is being requested here.

3. Maybe think about using a different analyzer for Kuroshiro to improve romanization of Japanese surnames.

Since I don't speak these languages at all, you're going to have to do the thinking on this one; ISTR @duncdrum also has some expertise here.
WRT mecab, I can't find the mecab package on npmjs; I did find https://taku910.github.io/mecab/, but from the few English instructions I see there, this seems to be a C library with a javascript wrapper; I can only use pure-javascript libraries, so if it's this mecab, that would seem to be impossible.

And one last P.S., per my previous issue, Chinese and Japanese romanization needs to be handled separately (maybe be looking at the item.language?). Right now, if you have the Kuroshiro option checked it overwrites everything so you can't generate citekeys properlyfor Chinese and Japanese items at the same time.

Separating on language would be easy -- I could technically make jieba a no-op if the language was not set to Chinese, but I think it would be cleaner to have a pseudo function like [language=zh] that would abort unless the language was set to something deemed to be Chinese, and then you could use the | syntax to just have separate patterns per language. But if you think no-op would be easier, I can consider that. Separating on language would have the added benefit that for existing users, the key generation would be more stable.

When using the kuromoji tokenizer, すもももももももものうち tokenizes into the array below. What fields of the array elements are the tokens of the tokenized すもももももももものうち? surface_form?
[
  {
    word_id: 415760,
    word_type: 'KNOWN',
    word_position: 1,
    surface_form: 'すもも',
    pos: '名詞',
    pos_detail_1: '一般',
    pos_detail_2: '*',
    pos_detail_3: '*',
    conjugated_type: '*',
    conjugated_form: '*',
    basic_form: 'すもも',
    reading: 'スモモ',
    pronunciation: 'スモモ'
  },
  {
    word_id: 93220,
    word_type: 'KNOWN',
    word_position: 4,
    surface_form: 'も',
    pos: '助詞',
    pos_detail_1: '係助詞',
    pos_detail_2: '*',
    pos_detail_3: '*',
    conjugated_type: '*',
    conjugated_form: '*',
    basic_form: 'も',
    reading: 'モ',
    pronunciation: 'モ'
  },
  {
    word_id: 1614710,
    word_type: 'KNOWN',
    word_position: 5,
    surface_form: 'もも',
    pos: '名詞',
    pos_detail_1: '一般',
    pos_detail_2: '*',
    pos_detail_3: '*',
    conjugated_type: '*',
    conjugated_form: '*',
    basic_form: 'もも',
    reading: 'モモ',
    pronunciation: 'モモ'
  },
  {
    word_id: 93220,
    word_type: 'KNOWN',
    word_position: 7,
    surface_form: 'も',
    pos: '助詞',
    pos_detail_1: '係助詞',
    pos_detail_2: '*',
    pos_detail_3: '*',
    conjugated_type: '*',
    conjugated_form: '*',
    basic_form: 'も',
    reading: 'モ',
    pronunciation: 'モ'
  },
  {
    word_id: 1614710,
    word_type: 'KNOWN',
    word_position: 8,
    surface_form: 'もも',
    pos: '名詞',
    pos_detail_1: '一般',
    pos_detail_2: '*',
    pos_detail_3: '*',
    conjugated_type: '*',
    conjugated_form: '*',
    basic_form: 'もも',
    reading: 'モモ',
    pronunciation: 'モモ'
  },
  {
    word_id: 93100,
    word_type: 'KNOWN',
    word_position: 10,
    surface_form: 'の',
    pos: '助詞',
    pos_detail_1: '連体化',
    pos_detail_2: '*',
    pos_detail_3: '*',
    conjugated_type: '*',
    conjugated_form: '*',
    basic_form: 'の',
    reading: 'ノ',
    pronunciation: 'ノ'
  },
  {
    word_id: 62510,
    word_type: 'KNOWN',
    word_position: 11,
    surface_form: 'うち',
    pos: '名詞',
    pos_detail_1: '非自立',
    pos_detail_2: '副詞可能',
    pos_detail_3: '*',
    conjugated_type: '*',
    conjugated_form: '*',
    basic_form: 'うち',
    reading: 'ウチ',
    pronunciation: 'ウチ'
  }
]

@jtcarlyle work on translitation/segmentation for ja/zh continues here.

This build introduces a few changes:

:transliterate by default does only non-ja/zh aware transliteration
:transliterate=ja/:transliterate=japanese will apply kuroshiro when it is on. Before the debug build here, this would be the same as :transliterate
:transliterate=zh/:transliterate=chinese will apply pinyin
(unrelated to the current issue, but it was added here) :transliterate=de/transliterate=german will replace a few german characters with soundalikes
[language=<de/german/ja/japanese/zh/chinese>] will test whether the language in the language matches the given language or subvariants of the given language, and will abort to the next pattern if not.
:kuromoji applies kuromoji word segmentation if kuroshiro support is on.

You have to have the language field of the item set to trigger any of these. You could now eg use
[language=ja] [Authors:kuromoji:select=1,2:transliterate=ja][Title:kuromoji:skipwords:select=1,2:nopunct:transliterate=ja][year] | [language=zh] [Authors:jieba:select=1,2:transliterate=zh][Title:jieba:skipwords:select=1,2:nopunct:transliterate=zh][year] | [authEtAl][shorttitle2_2][year]

@duncdrum, @bokamm this is unfortunately a breaking change. :transliterate previously blindly applied kuroshiro when it was on, regardless of what language was set on the item, naively assuming that kuroshiro would leave anything non-Japanese untouched, not knowing the characters it supports is shared between other languages. The current setup puts the work of determining what translit to apply on the user. I see no other safe way. Existing citekeys generated under the old behavior will not change unless you edit the item, or manually refresh the key.

I really need feedback to be able to proceed on this one.

Hallo! Most of my entries do have the language set to "ja", so it should be no problem at my end. I am on research currently with not much time to test anything. I installed 5.5.1 and everything seems to work fine. Can test more next week, if necessary.
Thank you for your great work!


Route should implicitly run decodeURI on pathnames to match browserHistory behavior

Version
4.1.1
Test Case
https://codesandbox.io/s/26WgmOzM
As you can see, the third link (ふが) that has its to argument set with the result of a decodeURI call will correctly detect as active, but the second link, while it will correctly navigate, will never be detected as active. In both cases, the browser's native location.pathname will be URL encoded, but the Router's history.location.pathname will not be URL encoded.
The ultimate cause is that the history package always implicitly calls decodeURI on the pathnames it generates internally, so the extra URL encoding present on the  URLs that do not go through decodeURI prevent them from ever matching in matchPath. The offending line is https://github.com/ReactTraining/history/blob/8d98aec/modules/LocationUtils.js#L37.
This does not affect plain ASCII or latin-1/cp1252 URLs, but it affects any URL in a non-western-european language. decodeURI will preserve dangerous escapes (e.g. %2f for escaping a /) so it should be safe to always implicitly call it.
Steps to reproduce
Use encodeURIComponent (or even a path-to-regexp compiled URL, which uses encodeURIComponent internally) to generate a URL that contains URL encoding, and use it as a Route's path or NavLink to argument.
Expected Behavior
A NavLink that points to a URL with an encodeURIComponent component to always match after it is clicked.
Actual Behavior
A NavLink (or Route) that has any URI encoded characters beyond the minimum necessary will never match.

Location objects are expected to have non-encoded pathnames. I updated your sandbox to remove the encodeURI/decodeURI calls and it works fine without them: https://codesandbox.io/s/wLp37MpR

The problem is easier to occur in practice when untrusted input may be used for the path.
Perhaps it should be recommended in the docs to pass such paths to decodeURI then.

If you open up a PR that updates the <NavLink> documentation to mention that the pathname passed to to should not be encoded, I'm sure it would be appreciated.


chore: change wallet.rs revision to cache outputs

Summary
Previously outputs were not cached, causing nodes to be bombarded with participation-related API requests from all of the Firefly instances.
Changelog
- Change wallet-rs revision to b2cc0b6018325839998c2fc53d86577c62522118
- Change lock files

Relevant Issues
None
Type of Change

[ ] Breaking - any change that would cause existing functionality to not work as expected
[x] Chore - refactoring, build scripts or anything else that isn't user-facing
[ ] Docs - changes to the documentation
[ ] Fix - a change which fixes an issue
[ ] New - a change which implements a new feature
[ ] Update - a change which updates existing functionality

Testing
Platforms

Desktop

[ ] MacOS
[x] Linux
[ ] Windows

Mobile

[ ] iOS
[ ] Android

Instructions
None
Checklist

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added or modified tests that prove my changes work as intended
[x] I have verified that new and existing unit tests pass locally with my changes
[x] I have verified that my latest changes pass CI workflows for testing and linting
[x] I have made corresponding changes to the documentation

Don't merge this unless it has been tested by alpha team

Spoke to thoralf and this is not to do with staking, but more voting so we can wait to merge this until after staking release, and until its merged in develop of wallet.rs

This sounds good - let's revisit it when we begin to test voting :+1:

Already merged in a different version


[bugfix] Pulsar Stream metadata missing

Pulsar stream metadata was not being written to Pinot records due to a missing implementation in the PulsarMessageBatch class.
bugfix
@navina

Codecov Report

Merging #11542 (c80f3cb) into master (be496bc) will decrease coverage by 48.52%.
The diff coverage is 33.33%.

@@              Coverage Diff              @@
##             master   #11542       +/-   ##
=============================================
- Coverage     63.02%   14.50%   -48.52%
+ Complexity     1106      201      -905
=============================================
  Files          2320     2320
  Lines        124660   124661        +1
  Branches      19032    19032
=============================================
- Hits          78566    18085    -60481
- Misses        40496   105044    +64548
+ Partials       5598     1532     -4066

Flag
Coverage Δ

integration
?

integration1
?

integration2
?

java-11
?

java-17
?

java-20
14.50% <33.33%> (-48.39%)
:arrow_down:

temurin
14.50% <33.33%> (-48.52%)
:arrow_down:

unittests
14.50% <33.33%> (-48.52%)
:arrow_down:

unittests1
?

unittests2
14.50% <33.33%> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

...not/plugin/stream/kinesis/KinesisRecordsBatch.java
94.11% <0.00%> (-5.89%)
:arrow_down:

...java/org/apache/pinot/spi/stream/MessageBatch.java
0.00% <ø> (-70.00%)
:arrow_down:

...pinot/plugin/stream/pulsar/PulsarMessageBatch.java
26.66% <40.00%> (+0.74%)
:arrow_up:

... and 1496 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Some ClusterPositions are negative after commit if a select query is performed before

OrientDB Version: 3.0.0-SNAPSHOT
Java Version: 1.8.0_131
OS: Windows 10
Expected behavior
All clusterPositions become positive after commit.
Actual behavior
Some clusterPositions remain negative after commit, if a select query is performed before (try to comment/uncomment line 128).
Steps to reproduce

Add some number of vertexes and edges in a loop.
Perform some select operation inside the loop, when some vertexes and edges are added.
Commit.
Check vertexes clusterPositions - some of them are negative.

Please find code example by link:
https://github.com/arkolesnik/orientdb-automation-transactions/blob/temp/src/test/java/TransactionsTest.java
Vertexes before commit:

Vertexes after commit:

Hi,
This should be already fixed in the last develop

Hi @tglman,
Still being reproduced for me.
(git pull and mvn clean install were done)

Hi,
I tried to reproduce this again with the last orientdb 3.0 snapshot, I got the "temp" branch where some assert were failing but going on debug the id where all valid, it would be cool to have an assert on the rid so we can detect directly the case.
Regards

@tglman we have one https://github.com/arkolesnik/orientdb-automation-transactions/blob/temp/src/test/java/TransactionsTest.java#L139

@tglman so you are not able to reproduce it anymore?

Hi,
I think we can close this now.
Regards


do not overwrite unchanged cached Credentials (#1168)

fixes #1168

This is my first time coding in a real rust project. I didn't even know Arc<T> before this day.
If something goes wrong I'm sorry :rofl:

Looks good to me, thanks!

@jokester @roderickvd this PR does not work. The logic is not correct and can't be correct anyway as the problem lies elsewhere.
The issue is that right now, no matter what, librespot will reconnect to your account every time you use Session::connect.
This is problematic because

This might trigger stuff on Spotify's side
Indeed, it does, as if done too many times, Spotify will force you a reset password

A token refresh if the current session dies would be the way. Still, the issue remains that the stored auth_data isn't being reused at all. Cached but never used.

Could you use Session.is_invalid() to check if the session is dead? or what is that for? Not fluent enough in Rust.

@Laiteux You may want to open a new issue, as this PR is closed and will unlikely get any attention from devs.


Adding Progress Bar in QuizApp Project

Hello! I am Siddhant Chadha and currently a participant of GSSOC'21. I was going through the QuizApp project and want to add a progress bar made using CSS and javascript which would indicate the current progress of the user. It would look something like this.

Please assign this issue to me. Looking forward to contribute.
Thanks!

yea @SiddhantChadha go ahead :)

@ridsuteri @harshgupta20 please check my PR and let me know if it is merged or not.


Coveralls client only exposes coveralls command entry point

When performing a pip install of coveralls, a command-line entry point is made for coveralls. This conflicts with other coveralls clients (such as cpp-coveralls) that also create a command-line entry point named coveralls. It would make sense to expose an additional, unique command-line entry point coveralls-python that can be used exclusively for referring to this package.

That seems reasonable to me -- I'd be happy to accept a PR which adds the secondary command line entry beside the first.

python -m coveralls already seems to exist and work, so to me this may not be necessary given you can just use that if the binary name is used by some other thing.

@Julian That seems like a good approach to me as well, thanks for pointing that out.
I'm going to close this since that's an appropriate work-around.


Add Data Type functions to Data Provider API

Added data types to data provider api. Before, data providers had get, set, notifyUpdate, and subscribe to serve and control updates to one data type. Now, data providers expose multiple sets of functions, one per data type they serve. Essentially, the functions do the same thing, but they are scoped with one extra distinction from just a selector (the data type). For example, the Sneeze Board could have Sneezes and Users, so it would have two sets of functions:

getSneeze, setSneeze, notifyUpdateSneeze, and subscribeSneeze
getUser, setUser, notifyUpdateUser, and subscribeUser

set and notifyUpdate methods now have more complex return types. They can return '*' to update all data types related to this data provider, a string of one data type name to update that data type, an array of data type names to update those data types, truthy (other than string or array) to update just the specific data type for this method (e.g. returning true from setSneeze would update all Sneeze subscribers only), or falsy to update nothing.
Now, instead of just using useData hook, you use useData.<data_type> like useData.Verse. To specify types, you can get the data types from the provider and index into the proper type like useData.Verse<QuickVerseDataTypes['Verse']>
Relates to #109

This change is

I just finished updating the Sneezeboard data engine to match these changes. I really like this second way. I think it makes the data types much more intuitive and the code more readable. The process felt more natural and naming variables was a lot easier as I didn't need to think of a word that could be a sneeze or a user. Understanding the context token and some other changes to activate was probably the hardest part, but the examples helped and I was able to figure it out. The documentation and examples are helpful and I was able to get through the process without asking questions.


Knowing which component triggered onEnter

The onEnter hook is called with just the routeState, which doesn't have any information about which component actually triggered the call. Let's say we have these routes:
Route(
  { path: '/', component: App },
  Route({ path: 'foo', component: Foo, onEnter: handleEnter }),
  Route({ path: 'bar', component: Bar, onEnter: handleEnter })
)

In handleEnter, I don't have any obvious way to figure out if foo or bar triggered it. I could look at the last component in the components array of routeState, but this could be called anywhere from a deep tree. It seems like onEnter is meant to be used when the user wants to hardcode specific actions to run for a specific route component, but not for generic actions.
Actually, there does seem to be a way to do this that is not mentioned in the docs: the this value of the hook will be the component instance. So I can actually do this.component to get the component class, and call static methods on it. Is this intended? I would rather it be more explicit and just pass in the component that triggered it as the second argument.
If it is intended, it should certainly be documented.
Why am I doing this? This is for automatically fetching needed data generically.

@jlongster What component instance? You mean the class? These hooks run before we call createElement on anything.

@jlongster what if you bind some kind of id to your handleEnter function?

onEnter won't have an instance since its called before rendering anything, there is no component instance yet.

I would rather it be more explicit and just pass in the component that triggered it as the second argument.

There can be multiple components, or no component at all for a route, so this is the route, which gives you access to everything about it. You can either put your static methods on this.component or the route itself.
This hopefully illustrates how you could do it:
https://github.com/ReactTraining/advanced-react/blob/master/subjects/routing/solution.js
https://github.com/ReactTraining/advanced-react/blob/master/subjects/routing/lib/AsyncProps.js

@ryanflorence
You beat me to it. We should document this on API.md though - I don't see that part documented anywhere.
There's a hole here when using async components, though - you'd only see getComponent(s) on the route object and not the component itself. You can still identify the route, but you wouldn't necessarily be able to call anything on the component without jumping through hoops; conceptually this is a similar issue to why I added the RoutingContext customization for react-router-relay to support roughly the same thing..

If I had this onEnter function:
  function onEnter() {
    console.log(this);
  };

I get this in the console:

I am not interested in the user's component. I'm talking about the Route component. A Route component has to exist; I have to create one to give an onEnter hook to!

Isn't that what you're looking for? Note that <Route> components are purely for ease of configuration - we convert them into plain route objects like the one you see above for routing purposes.

Yeah, it is but it's not at all in the documentation. I didn't know if that was an accident or intended. I feel like it would be better to pass that object as the second parameter to onEnter.
@ryanflorence AsyncProps is cool, but if you are already using a central store we can just throw data in there. I'm experimenting with just hooking up automatic data fetching at the routing layer and not even having to wrap components. It feels nice because how you fetch data is different in the context anyway (server/client etc).

Sounds like @ryanflorence confirmed that it's intended, not sure if you want to add it to the docs or keep it an undocumented feature. Not sure I understand the RoutingContext stuff you mentioned above @taion, sounds like a more complex use case than I'm familiar with. If there are problems with this it can just be an undocumented feature.

Has to be intentional because why else would someone have written this code: https://github.com/rackt/react-router/blob/v1.0.0-rc4/modules/TransitionUtils.js#L5? XD
Would you be willing to submit a quick PR to write this up for our docs?

The RoutingContext stuff is for the case where I need access to the actual component class associated with the route object when doing data fetching on something like an onEnter hook when the route object uses an async component; this is necessary for Relay because the fragments live on the component class, but you can't just do route.component if the components are async.

@jlongster Route({ onEnter: createEnterHook(specificLoaderFunction) }) ?

@taion done! #2501
@ryanflorence Not sure what you mean? I'm actually playing with creating a higher-order Route component that automatically adds an onEnter hook that when run will fetch the data for that component.

want to hop on discord?

longsterRoute({ load: someSpecificLoadFn })
under the hood creates:
Route({ onEnter: createLoader(props.load) })
You know at the time of route definition your loader dependency, you don't need to store it on the route and then look it up again later.


66 run tests on pr

Codecov Report

Merging #67 (63ddce5) into master (f9c6d82) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #67   +/-   ##
=======================================
  Coverage   70.05%   70.05%
=======================================
  Files          21       21
  Lines        1022     1022
=======================================
  Hits          716      716
  Misses        306      306

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


ERRORCODE=-4214, SQLSTATE=28000

I keep getting,
[jcc][t4][2013][11249][4.26.14] Connection authorization failure occurred.  Reason: User ID or Password invalid. ERRORCODE=-4214, SQLSTATE=28000
I have reset my password via ITIM.
Thank you for reporting an issue.
IMPORTANT -  before creating a new issue please look around:

DBeaver documentation: https://github.com/dbeaver/dbeaver/wiki
and
open issues in Github tracker: https://github.com/dbeaver/dbeaver/issues

If you cannot find a similar problem, then create a new issue. Short tips about new issues can be found here: https://github.com/dbeaver/dbeaver/wiki/Posting-issues
Please, do not create issue duplicates. If you find the same or similar issue, just add a comment or vote for this feature. It helps us to track the most popular requests and fix them faster.
Please fill in as much of the template as possible.
-->
System information:

Operating system (distribution) and version
DBeaver version
Java version
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

hi @karen-handcock ,
what is your db? could you check if issue reproducible with another credentials?

this is what I see
Host: <IP_ADDRESS>
Database: WEB
Port: 3705
Driver name: DB2 LUW

the message says that your userid or password is wrong. This could also mean that you’re not authorized to access this database. Talk to your DBA department

Hi. I have tried to talk with DB2 support. they told me to reach out to DBeaver team.
I also spoke with ITIM team, they say password & Id is correct.
I'm going around in circles.

hi @karen-handcock , is it still actual?

there is no update on issue for a long time. If it is still actual for you - feel free to reopen the ticket with additional info.
BR

What was the issue?  I am experiencing the same with the Eclipse plugin of DBeaver for a DB2 LUW JDBC connection.   Same credentials used from other apps and connects fine, but with same credentials and properties for the DB2 LUW connection under the DBeaver plugin and fails stating that the following:
[jcc][t4][2013][11249][3.68.61] Connection authorization failure occurred.  Reason: User ID or Password invalid. ERRORCODE=-4214, SQLSTATE=28000
[jcc][t4][2013][11249][3.68.61] Connection authorization failure occurred.  Reason: User ID or Password invalid. ERRORCODE=-4214, SQLSTATE=28000
Eclipse
Version: 2021-06 (4.20.0)
Build id: 20210612-2011
DBeaver Plugin
Version: 21.3.3
I can create a SQL Server connection fine in the plugin but not DB2
I attached the trace output.
trace_sds_3.log


Disable filtering out CONST, IMPLEMENTS, ABSTRACT annotations.

It turns out that dartanalyze will complain about unused imports if
these annotations are no longer present. Without real type information
it's not possible to detect those cases reliably, so it's easier for now
to just include the annotations in the output.
Long term, we might either want to ignore those warnings or use the type
information from TS on what's actually used or isn't.

Otherwise LGTM


Bugs/missing features in pkg_filegroup-using pkg_rpm

I will soon be submitting draft PR that will provide a pkg_filegroup-supporting pkg_rpm rule, based on the one in pkg/experimental/rpm.bzl, first created in #129.  All issues related to it are then inherited, but not immediately fixed to allow for swifter review.
This issue tracks known issue with this rule, including:

[x] TreeArtifact installation is broken if done in the "wrong" order (#292)
[x] It uses the deprecated outputs parameter, like the existing spec-file based pkg_rpm rule (#283)
[x] It does not use the rpmbuild toolchain (#299)
[x] It does not support SOURCE_DATE_EPOCH (#178)
[x] The test cases are linux-only (#255)

All known issues resolved; resolving this ticket (yay!).


Patch *.la and *.pc files when they exist

Fix #560
This patches all files in lib/pkgconfig/.pc and lib/.la, on all targets, when those files exist. This will only apply to Linux and Mac hosts; I'm not sure that these files exist on Windows.
Before making this change, you could run this to count unpatched instances of "/Users/qt/work/install":
$ python -m aqt install-qt mac android 5.15.2 android
$ grep "/Users/qt/work/install" 5.15.2/android/**/* 2>/dev/null | wc -l
779

After making this change, the number of unpatched instances of "/Users/qt/work/install" drops from 779 to 13:

5/13 remaining unpatched files are binary shared objects; we probably should not touch those.
8/13 remaining unpatched instances are in the lib/pkgconfig/*.pc files; here's an example in Qt5OpenGLExtensions_x86.pc: Libs.private: /Users/qt/work/install/lib/libQt5Gui_x86.so /Users/qt/work/install/lib/libQt5Core_x86.so -lGLESv2  -llog -lz -lm -ldl -lc

I'm not sure what to do about the remaining unpatched .pc files yet.

It is difficult for us to define what is perfect or required because we don't know these specifications.
I think the best of our doing is to change a issue based on user feedback, one by one, step by step.

I'm not sure what to do about the remaining unpatched .pc files yet.

We are not sure what to do. When we find more, we can change more.


[dv,cosim] Implement internal interrupts in co-sim

Add support to cosim for internal interrupts (documented at https://ibex-core.readthedocs.io/en/latest/03_reference/exception_interrupts.html#internal-interrupts)

estimate 8
remaining 2022-10-06 4
remaining 2022-10-14 2

remaining 2022-10-14 2

@GregAC to add write suppression on ECC failures on load to co-sim environment

I have a set of commits on top of Canberk's work in https://github.com/lowRISC/ibex/pull/1811 but no PR yet. Once that is merged this will be complete.

Completed by https://github.com/lowRISC/ibex/pull/1907


add dockerfile and instructions

@paulczar Thank you for signing the Contributor License Agreement!

@paulczar thanks for the PR. While we have instructions to run docker for services required by petclinic (e.g a database), I am not keen to add instruction to run petclinic with docker as they are much easier ways to do so.