Error : tronpy.exceptions.AddressNotFound: account not found on-chain

Got error  tronpy.exceptions.AddressNotFound: account not found on-chain When I try to get Balance of address.
Code :
from tronpy import Tron

from pprint import pprint

client = Tron(network='nile')

address=client.generate_address()['base58check_address']

pprint(client.get_account_balance(address))

Traceback :
Traceback (most recent call last):
  File "tron.py", line 9, in <module>
    pprint(client.get_account_balance(address))
  File "G:\BOT\tipbot\env\lib\site-packages\tronpy\tron.py", line 517, in get_account_balance
    info = self.get_account(addr)
  File "G:\BOT\tipbot\env\lib\site-packages\tronpy\tron.py", line 501, in get_account
    raise AddressNotFound("account not found on-chain")
tronpy.exceptions.AddressNotFound: account not found on-chain

Got error tronpy.exceptions.AddressNotFound: account not found on-chain When I try to get Balance of address.
Code :
from tronpy import Tron

from pprint import pprint

client = Tron(network='nile')

address=client.generate_address()['base58check_address']

pprint(client.get_account_balance(address))

Traceback :
Traceback (most recent call last): File "tron.py", line 9, in <module> pprint(client.get_account_balance(address)) File "G:\BOT\tipbot\env\lib\site-packages\tronpy\tron.py", line 517, in get_account_balance info = self.get_account(addr) File "G:\BOT\tipbot\env\lib\site-packages\tronpy\tron.py", line 501, in get_account raise AddressNotFound("account not found on-chain") tronpy.exceptions.AddressNotFound: account not found on-chain

You must use a faucet http://nileex.io/join/getJoinPage to get coin and activate your account on-chain.

You must use a faucet to get TRX and activate your account on-chian.
Faucet Page: https://nileex.io/join/getJoinPage


Upgrade the version of isort

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
https://github.com/open-mmlab/mmflow/pull/87
Upgrade the version of isort.
Modification

Remove seed-isort-config in pre-commit hook
Upgrade isort to v5.10.1 (latest version)
Replace known_standard_library with extra_standard_library

BC-breaking (Optional)
Does the modification introduce changes that break the back-compatibility of the downstream repos?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist

Pre-commit or other linting tools are used to fix the potential lint issues.
The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
If the modification has potential influence on downstream projects, this PR should be tested with downstream projects, like MMDet or MMCls.
The documentation has been modified accordingly, like docstring or example tutorials.

Need to pass lint before merging.


UI-Error: when the user send long messages , then the timestamp did not show properly in ChatActivity

Expected Behavior:
Message size should not impact the time Stamp.

@Ijaiswalshivam Assign me


fix: wording translation

Pull Request Checklist

[x] Description: Briefly describe the changes in this pull request.
[x] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[ ] Documentation: Have you updated relevant documentation?
[ ] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?

Description
new wording fix translated
Changelog Entry
Fixed

new wording fix translated

Changed

new wording fix translated (desktop notifications to notifications)

Thanks!


Add a Clearing site data feature

Draft until BCD fixes have been merged:

https://github.com/mdn/browser-compat-data/pull/24821
https://github.com/mdn/browser-compat-data/pull/24822

So, Clear-Site-Data is not fully interoperable. https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Clear-Site-Data#browser_compatibility. We could divide this in separate features:

C-S-D: storage and cookies (the only parts that would be baseline)
C-S-D: cache
C-S-D: execution contexts
C-S-D: clientHints

or we leave it as one feature and have a baseline false until C-S-D is more interoperable.

I've pinned the wildcard support. I think that makes the most sense if we don't want to split out anything here for now.


Translation Script - Doorway

Doorway has called out several places where we're missing translations.Can we use a script to output all the places where we're missing translations in Doorway so we can be sure we aren't missing translations anywhere?

https://docs.google.com/spreadsheets/d/1gtsSeMC2UJlJJS9N4GLesUVsy-nY1R_tS4RUAKym9nw/edit#gid=0 -- emily did this!


Form Element Issue

Challenge Create a Form Element has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.81 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

<form action="/submit-cat-photo">
<input type="text" placeholder="cat photo URL"/>
</form>

Hi @lindsaylove, what's your issue?

I have the same exact issue, it doesn't proceed when I add the last
<form action="/submit-cat-photo">
<input type="text" placeholder="cat photo URL">
</form>

and when I press "Run Test" nothing happens. it doen't go any further than that

It seems the issue has been resolved. I was able to successfully "Run Tests" with the following code, which was previously showing as incorrect:

@lindsaylove thanks yeah I just checked it again and its resolved now.

I'm having this issue now, was the text posted above correct? That's what I've been trying to use.

@simeiste Yup, it is correct. For me, I tried it next day again and it worked :) so I would recommend you to that too :)

Have the same problem, will try again tomorrow  :smiley:

Having the same issue using Chrome 58.0.3029.96 (64-bit). I was able to get around it by loading Firefox and submitting the same code.

Had the same issue and clearing browser cache for this page helped.

The page is working now, I tried refreshing the page and giving it a few minutes. I'm not sure what fixed it but thanks for the suggestions!

Just for anybody who is looking for help with this issue...
As far as I know, Chrome 57 will not work. You need to update or use another browser.
If Chrome 58 freezes the code, the way to fix it is to make sure you have typed in a full <form></form>  element and refresh the page.


Indicate fix for stale opencl program cache in error message

Better error message when system update breaks OpenCL program caching  (#2588)

What is -30 in OpenCL terms?

CL_INVALID_VALUE

Code
OpenCL Error Flag
Function(s)
Description

-30
CL_INVALID_VALUE
clGetDeviceIDs, clCreateContext
This depends on the function: two or more coupled parameters had errors.

I'm also open to just throwing the cache warning on all error codes, but since I don't have a good understanding of them, I was just maximally specific (I know that -30 can get triggered by a stale cache/system upgrade).
When I get a chance, I'll push a patch with the version stamp on the disktensor that is a better fix.

just use cl.CL_INVALID_VALUE?

Yes, that's better.  Made the change.

Closing this since #2618 is a better fix


Break private key into Shamir secrets

We'll eventually support multisig name ownership, but before we do so, I'd like to add support for breaking the master private key into Shamir secrets, which can be distributed across multiple (air-gapped) devices.  I'd make it easy to do automatically, with clear instructions on what to do and how to re-combine them.  I think this would get us a good chunk of the benefits of multisig, without having to make any potentially-consensus-breaking changes.

Oh I thought we already supported multi-sig. Is that not the case?
If the name is owned by an arbitrary scriptPubKey then all ownership types, including multi-sig, would be supported.

Oh I thought we already supported multi-sig. Is that not the case?

The tooling doesn't support it--i.e. the CLI doesn't set up multisig, and neither the CLI nor the server generate multisig scripts.  I also can't say for sure that the server will accept multisig addresses, since we haven't tried it yet (still need to move blockstack-integration-tests over to using -regtest).

OK can we set aside some time to try? I have a hunch that it will work. As long as the consensus hashes are calculated from the scriptPubKey's (and not public keys) then it shouldn't be consensus breaking. Would you say that is a valid assumption?

I have the same hunch, but I just want to be sure.  Tracking the details here:  https://github.com/blockstack/blockstack-integration-tests/issues/1

Also, I agree that integrating Shamir's Secret Sharing could be a good first step, since we already have our own library for it and it should be relatively simple to plug in.
In addition, we should set up a time schedule on our roadmap for supporting multi-sig.
In my view, multi-sig is much more useful than Shamir's Secret Sharing (think about multiple devices and the security implications of communicating shares to one device) and is among the top priorities for new updates to the Blockstack core.
This will let us do multi-sig not only in the CLI but also in the desktop app, which will be an important feature to add in one of the earlier releases.

@jcnelson ok cool, I'll watch that thread as well.

We're just going to do straight-up multisig.


Remove Discord bot and also add catch for legi key not being present (oops)

This removes the use of py-cord as a dependency, removes the cogs cache from gitignore (since it's no longer relevant and should be safely removable hopefully), and adds a small check for the existence of the environment variable so the program exits abruptly to avoid making erroneous API calls if there's no key and triggering anti-spam behaviors.

I'm not ready to yeet this just yet... I might want to use some of this for the search feature I have in mind


Update example/customshortcut readme.md

[ ] update how to run
[ ] add demo footage

:octocat: cibot: Thank you for posting issue #436. The person in charge will reply soon.

resolved with #474


[SMALLFIX] Remove the upgradeJournal commands

What changes are proposed in this pull request?
Remove the upgradeJournal commands.
Why are the changes needed?
Remove the upgradeJournal commands.
Does this PR introduce any user facing changes?
NA

Remove this since it is for "From 1.x to 2.x"

alluxio-bot, merge this please


feat: add the universty named kiet (.kiet) into the edu directory

KIET Website Link
Address:  KIET, PAF Airmen Academy, PAF Colony Paf Colony Korangi Creek, Karachi, Karachi City, Sindh 75190
County: Pakistan
Undergrad Programs: https://kiet.edu.pk/programs/?ungrad
proof for the domain emails, attachment contains the admission dept email with valid .kiet.edu.pk  domain:

@neerajparkashsharma First stage of verification passed, official domain for Karachi Institute of Economics & Technology (KIET) is kiet.edu.pk as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@neerajparkashsharma Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


Handle split headers in Http(Request|Response).aggregate()

Motivation:
Some HTTP/2 implementations can send additional headers after sending a
non-informational HEADERS frame.
HttpRequest.aggregate() and HttpResponse.aggregate() will not merge the
split headers correctly, making AggregatedHttpMessage.informationals()
contain split headers and headers() and trailingHeaders() omit some
headers.
Modifications:

Merge the split HTTP headers correctly.

Result:

Fixes #1014 for HttpRequest.aggregate() and HttpResponse.aggregate()

/cc @anuraaga @kojilin Just in case you are interested

Thanks, @minwoox !


Add option to mutate agent spec

Right now we keep adding stuff to our configuration, but it'd be nice if we could provide user with ability to override anything in the spec.
I'd expect logic similar to Kustomize, where we have our definition, then user can define any key,value and it overrides ours.

Won't do this since it'll cover up whatever issue it was that made the user change the agent spec in the first place.


C3 resize fix, 2nd try

ping @h-kataria, @dclarizio
redo changes in #8061 w/o breaking the forms

verified forms in UI.


Continue to support older net-ssh while fixing 4.2 deprecation

In #197, we bumped our dependency on net-ssh to ~> 4.2 to address a deprecation of the paranoid option made in net-ssh 4.2.0. Unfortunately, this is causing users of newer InSpec versions to not be able to use InSpec with Chef v12 due to net-ssh v3 in use in that version.
This change reverts the net-ssh pin to be looser and dynamically chooses the connection option to use depending on the version of net-ssh in use. This will avoid the deprecation warning and allow us to support newer net-ssh versions.

🥇


Add back in OBJCUWP_IN_BUILD or OBJCUWP_INCLUDE_ONLY guard to #pragma includes for projections

this guard was removed because it was thought to be superfluous in CR. Turns out I'm not as crazy as I thought and did have a good reason for it. When a different Projections binary is being built and including a .h, it will try to link against that Projections binary that may not be built yet. This is normally handled via a ProjectReference but these references are the minset of actual deps and headers can be included for other, non dep forming, reasons. Currently the build simply works by happenstance as it is referencing the checked in binaries from prebuilts but if anyone needs to change the layout / structure of modules again it will explode spectacularly.

may want to look at this with the work #2687

Closing; @mukhole told me that this is "No longer needed, the dependency is taken care of at project level."


API call to get all accounts whitelisted by a particular account

Currently white lists are tracked on the account that is being whitelisted rather than the accounts that re doing the whitelisting.   We need to maintain a reverse index to facilitate a user interface that enables exchanges to enumerate their white list.
For now the exchange user interface can simply have a form that allows them to check to see if an account is white listed and either add/remove them.

Superseded by: #228


Add generic lens constructor for spherical singlets

Currently we have some smart lens builders with validators for the "planar" and "bi" lenses, four of the six fundamental spherical lenses. The meniscus lens are more dificult to validate because to degeneracy issues. Instead it would be better to have a general purpose lens constructor that can work for any spherical lens and performs simpler validation.

The generic lens factory functions for the generic lens geometry should live inside raysect primitives. They could be inherited from Intersection or some other CSG geometry primitive.

Agreed the Alex will do a prototype of the interface he wants. This could be pseudocode. Either of us can then finish the code based on the specified interface. Needs to support both types of initialisation, through base properties (i.e. width, height, radius) and optical properties (i.e. focal length, aperture stop).

I've build a Biconvex lens primitive to act as a template for lens/optical primitives that are constructed using CSG primitives. This uses a new EncapsulatedPrimitive class to hide the CSG primitives "inside" the lens primitive (it holds it's own scene-graph internally - note: this scene-graph can only have a single primitive directly attached to the root node). The lens code is now relatively simple to build, just set up the CSG hierarchy and attach.
Alternative initialisations can be added as class methods, these will be custom for each lens type. The class methods should just calculate the appropriate lens dimensions and build the lens by instancing the class.
See commit 63e292bada8b4f2f3cb8ad299205850169267228

premature closure

Closed by 8640d0cccc14a0fe6bd236cc62870ac00e395a65.


eslint does not work unless I terminate all nodejs processes

Hello
After searching over the internet for days now, and finding many posts either here or other places I still haven't manage to make VSCode run eslint on the editor.
Suddenly I noticed something by accident, and I would like to have your opinion
ESLint is installed globally and in the project using npm. As soon as I open VSCode and I go ti OUTPUT > ESLINT I see the following output
[Info  - 4:11:31 PM] ESLint server stopped.
[Info  - 4:11:32 PM] ESLint server running in node v12.4.0
[Info  - 4:11:32 PM] ESLint server is running.

If I then open a CMD window with elevated permissions and I run the following command
taskkill /f /im node.exe
Then the VSCode Output shows this message
[Info  - 4:13:36 PM] ESLint library loaded from: c:\Users\Username\Desktop\development\project1\node_modules\eslint\lib\api.js
And ESLint suddenly starts workings on VScode
So my question is from where does VSCode tries to load ESLInt Library? And is there a way to specify that? I searched the settings and found nothing.
PS: Same problem (ESLint not working) happening on both Windows and MacOS

The eslint library is loaded from the opened workspace folder.
can you please provide me with a GitHub repository I can clone that demos what you are experiencing.

Sure this is a test repo  that experience the same behavior. Its a simple test project since its happening for all projects.
[1] I open VSCode and this is what I see in the Output > ESlint
[2] Then I open PowerShell, kill all node processes and this is what I see. Then VScode starts working and underlining errors

@christos312 can you please add a package.json to the repository that list your eslint version and plugins you are using. This makes it self contained and ensures we are seeing the same behaviour.

Sorry for that, I have added the package.json

This works for me without any problems.

Anything that you think could be special for you?

So you got the message
ESLint library loaded from: c:\Users\Username\Desktop\development\project1\node_modules\eslint\lib\api.js
as soon as you opened VSCode?

Yes I did.

Then something is wrong with my system :/
I will have to check it. Thanks for taking the time to check this

OK. Please ping if you find something. I will then reopen the issue.


Fix docs

Small pr but I caught the error so I thought I might as well

Looks good. I should really fix so I don't have any extra space on line endings. Thanks!

No problem


Support Navigation Server

Likely needs to just support collision. Perhaps temporarily baking a very large collision, long enough to bake navigation, then it can revert to a smaller size. Devmar had a video about it for his clipmap terrain.
https://www.youtube.com/watch?v=YiBrZMkCkLs

Update:
@slashscreen has been working on experimenting with the nav server. They've setup a repo with a makeshift nav baker
https://github.com/SlashScreen/terrain3d_nav_bake
Read discord conversation starting here
https://discord.com/channels/691957978680786944/1130291534802202735/1136232663989100554
The current primary navigation server core developer was asked about using it on large terrains and this was the conversation:

Slashscreen currrently has a nav baker that can bake a large mesh in about 30 seconds and reports this information:
The general low-level process is as follows:

Geometry is parsed from the scene and put into a NavigationMeshSourceGeometryData3D object. This is done on the main thread, since the scene tree is not thread safe.
The geometry data and a Navmesh is fed into the NavigationMeshGenerator singleton to bake. This can be done multi-threaded, like I did, and/or a separate thread in the background.
The resulting Navmesh can then be put into the scene, usually by creating a NavigationRegion3D, but it looks like you may be able to add a mesh directly to the server without creating a new node.

Theoretically, as you generate chunks for the collision, you could upload them to the Geometry Data directly, as it has an add_mesh function, and bake that in a separate thread. (A navmesh can be limited to a certain AABB, so you don't have to re-bake the entire thing.)

I have also found that the slowest part of the process is parsing the scene tree, rather than baking. I like to lazy-load things, so how I would go about parsing the scene tree is to cast a big box on the chunk you wish to bake, grab all of the colliders hit, and add those to the Geometry Data too before baking, to allow for, say, buildings or whatever. To me it seems like doing that every so often won't be super intensive.

should i polish the baker tool and put it in a PR?

Sure, thanks. That will give us rudimentary support for now. Also please give me a writeup on how it works and how to use with the Nav Server that that I can put in the wiki. You can write it all as a comment on the PR with markdown and pictures and I can copy it over.
Place it in the addon tools directory next to the importer.
Don't make it a plugin, just a standalone scene with a terrain3d node and your other nodes so they can load in the storage, bake, and save it out for use in other scenes like how the importer is set up.
Is it usable by a new user?
Does it support disconnected regions / islands?
How big of a space have you tested? Can it do 16 regions (4k)?
I'm going to release 0.8.2 this weekend, maybe tomorrow. I'm still waiting on two pending PRs. Maybe you can get it in by then as an experimental feature.

I've prototyped threaded runtime navmesh baking on my hacks branch here.
@SlashScreen's comments and research helped me an unbelievable amount.
The general design is:

Parse source geometry upon entering the scene. The length of time this takes depends on the complexity of the scene. This has to happen on the main thread, but (hopefully?) this can be hidden with a loading screen. Crucially, it does not include terrain geometry.
Center an AABB on the player as they move around the scene. Generate geometry for the terrain within this AABB and add it to the source geometry. This happens on a thread, which is safe if we assume terrain is not modified at runtime.
Bake the source geometry (which includes the initial parsed geometry + the new terrain geometry) to a navmesh on the worker thread.
Back on the main thread, load the new navmesh into the nav server.

The demo plays without hitching, so it seems to work OK, but I'm unsure how well it would work in more production-realistic scenarios. I don't have any suitable scenes for testing it on. Is the initial pause caused by geometry parsing acceptable in realistic scenarios?
Not sure how well it performs on low-end / console devices. Can the threaded baking actually keep up with moving players in complex scenes?
https://github.com/TokisanGames/Terrain3D/assets/165720/28f0eee8-c4df-47e7-9b41-14cb02995819

@tcoxon This is great and something we can start with. Ultimately, I'd like to move things towards more dynamic generation like this. e.g. Collision should have another mode where it only generates around the player in-game.
For a navigation mesh, I really like this approach where it is paintable. I'm allocating space in the control map for it. We can move towards this direction over time. I haven't played with navigation at all yet, so I don't know how complicated it is, or how it performs. Though we will need it for Out of the Ashes.
https://www.youtube.com/watch?v=_QzzaajTHnw
Code
https://github.com/mohsenph69/Godot-MTerrain-plugin/tree/master/gdextension/src/navmesh
What FPS do you normally get in the demo without any navigation mesh generation?

Center an AABB on the player as they move around the scene.

Of course it needs to be large enough to encompass any enemies that can see the player so they can navigate to them.

The demo plays without hitching, so it seems to work OK, but I'm unsure how well it would work in more production-realistic scenarios.

We'll find out as it gets tested.

Is the initial pause caused by geometry parsing acceptable in realistic scenarios?

Is this on startup of the scene? How long is it?

@tcoxon super glad to hear my research helped. This demo looks good here; does it already support scene geometry? Realtime threaded baking is something a lot of games do, so I don't think it will be a problem in most cases (in my own game, the player moves rather slowly). Just in case, for times when the player would be whipping around the map like, for example, flying in a spaceship or a broomstick or something, you could add a way to temporarily turn off generation.
@TokisanGames I know you've been using heightmaps for collision, but perhaps this could be used instead?

@TokisanGames

For a navigation mesh, I really like this approach where it is paintable. I'm allocating space in the control map for it. We can move towards this direction over time.

Oh this is cool! I considered using this approach--glad to see others are already doing it. The two approaches could be combined quite well, and it would be simple to add the control map check to the prototype code.

What FPS do you normally get in the demo without any navigation mesh generation?

I did some non-scientific testing right now. While not recording my screen, I get 350-370 FPS without navigation, about 340-360 with threaded navigation baking.

Is this on startup of the scene? How long is it?

Basically, 'source geometry parsing' takes place at runtime, in the scene's _ready. It's literally microseconds in the demo project, but it's very dependent on what other geometry (besides the terrain) is in the scene. I have one other small scene I can test with where it takes 9ms. It really needs testing with a more realistic & complex project.
@SlashScreen

This demo looks good here; does it already support scene geometry?

You mean like other objects in the scene besides the terrain itself? Yes, that's what the 'parse source geometry' on scene startup is for.

Realtime threaded baking is something a lot of games do, so I don't think it will be a problem in most cases (in my own game, the player moves rather slowly).

I've seen that, but I've been burned in the past when I've thought things that work well in other engines can apply as easily to Godot. 😆
There's a bunch of copying and vertex duplication Godot has to do to get its data into the format Recast wants for baking. Whereas Unreal for example might have everything in the correct form from the start.

@tcoxon Oh, my bad, I got some stuff in my head mixed up. What does it consider "source geometry"? Is it dependent on heirarchy? In my case, I have a ProtonScatter node that creates a load of trees on _ready(), and I would need the baker to parse the geometry after it's done placing the trees.

@SlashScreen No worries, I'm probably just not explaining myself very well. Source geometry is stuff in the scene tree. What exactly counts depends on the current settings of the navmesh resource. The prototype parses all that source geometry in the scene tree once, in _ready.

In my case, I have a ProtonScatter node that creates a load of trees on _ready(), and I would need the baker to parse the geometry after it's done placing the trees.

Good point! The baker script should probably wait one frame before parsing scene geometry.
@TokisanGames I thought a bit more about realtime navmesh baking over the weekend. Not sure how you feel about this, but I am starting to think fully-featured realtime navmesh baking might go beyond the scope of a terrain plugin. The exact feature, latency, and performance requirements are going to vary a lot from project to project. I think it does  sense to provide some navmesh features for users:

Paintable navigability and static in-editor navmesh baking (as slow as that may be)
A function on Terrain3D exposed to GDScript that can be used to generate source geometry for threaded baking (like in the prototype)
An example script showing how to use this function to bake navmeshes at runtime

But going beyond that will probably mean you'll get inundated with endless contradictory issue reports about the latency and memory usage of realtime navmesh baking, as well as requests to add support for any number of other engine plugins, etc. It might be better to leave that up to users. What do you think?

I am starting to think fully-featured realtime navmesh baking might go beyond the scope of a terrain plugin.

I don't think we need continuous, real-time baking at runtime, unless we're going to only bake a small area around the player.
For a level like the demo, I don't want enemies to go everywhere in the 1k x 3k terrain. If I did, I wouldn't need navigation. I would only want them to stay on the path. The navigable area is quite small and should be bakeable and storable while working in the editor. I don't think a computer is going to automatically figure out this small area either. But if it's paintable, it should be pretty easy to be efficient. And even for larger areas it should be fine as the walkable areas will still be surrounded by cliffs, rivers, and mountains that devs won't want their units traversing.

I think it does sense to provide some navmesh features for users:

Paintable navigability and static in-editor navmesh baking (as slow as that may be)

Yes. In editor baking of the area painted by the gamedev should be sufficient for +80% of projects including OOTA.

A function on Terrain3D exposed to GDScript that can be used to generate source geometry for threaded baking (like in the prototype)

So they can regenerate it on the fly during game time if they want? Yes that's good. We should always support code based game-time generation of terrain. Though no guarantees on generation performance. They can use Voxel Tools for a dynamic terrain.

An example script showing how to use this function to bake navmeshes at runtime

Yes, that can go in the CodeGenerated.tscn demo.

But going beyond that will probably mean you'll get inundated with endless contradictory issue reports

I appreciate your thoughts. I think we're on the same page.

@tcoxon Paintable area with indication is now done in 0.9 as of 529d66a. You can now generate wherever there is navigation and not a hole: eg (control >>1u & 0x1u) && ! (control >>2u & 0x1u).

Completed in #253 thanks to @tcoxon and @SlashScreen


fix: Vuex Store is not updating in the Dev Tools

🐛 The bug
Vuex store is never updated in the Vue/Vuex Dev Tools.
mutations, actions, getters all work. The pages are reactive, and work correctly.
However "Load state" in the Vue/Vuex devtools show the state as its initial state
🛠 To reproduce
Go to the "Normal Usage" example
https://typed-vuex.roe.dev/build
Open the Web Browser section into a new window.
Open Vue Dev tools, go to the Vuex tab, and click "Load State"
🌈 Expected behaviour
I would expect the submodule firstName to be set to "John" after the 500ms timeout.
I would expect the email to be<EMAIL_ADDRESS>after clicking the button.
However it only shows initial state
ℹ Additional context
I'm just getting started with Typescript in Vue/Vuex/Nuxt. I initially found this in my simple experimentation, however confirming this on the given example indicates its either an issue with my dev tools, the dev tools, or the package.

@Towerful This is unfortunately an issue with the dev tools rather than this package. (duplicate of #108)

Ah, gotcha. I had read over that, but missed the "New Vuex Backend -> Enable" switch.
This fixed it for me, so thanks for the pointer!
I don't think I saw that in the docs, either. Might be worth adding a note to the "Getting Started" section about this


Add api REST response time metric

Motivation
Our beacon node - validator client interaction is problematic causing frequent timeouts on the validator client requests.
Description
This PR adds metrics to know which routes are slow and by how much. It adds a histogram metric labeled by operationId. If many routes are hit, the metrics can get very verbose. If so, the buckets can be reduced to one or use multiple gauges.
It also:

Normalize naming of rest files and functions to the operationId in the eth2 api specs
Add missing route /states/:stateId/root

this is nice to have. Due to the nature of async/await, I'm afraid the metric is not accurate if epoch transition or any heavy operations happen randomly at any api calls.

this is nice to have. Due to the nature of async/await, I'm afraid the metric is not accurate if epoch transition or any heavy operations happen randomly at any api calls.

Right, but what we want to track and minimize is the total async time to fullfill a duties request for example. The validator doesn't care about the internal queue system of the beacon node, it just needs to get the responses under x seconds


btn-raised  removal

.btn-raised  no longer looks so modern and requires a lot of output CSS bloating the build. consder removal

Another thought is I would like "capsule" buttons aka "pills"
https://stackoverflow.com/questions/29966499/border-radius-in-percentage-and-pixels-px-or-em

This is done in following (but not yet publish via bumped npm or documented):
[button-variants-100-and-custom-props-135 7acc631d] Adds Button variant capsule. Removes variant 'btn-raised'. Removes -from and -to color tokens as we are no longer needing (were used for raised buttons)


check master node availability status before cluster recovery

Describe the Bug
A clear and concise description of what the bug is.
check master node availability status before cluster recovery
Versions Used
Kubernetes:
Kubeclipper: master
Environment
How many nodes and their hardware configuration:
For example: CentOS 7.5 / 3 masters:  8cpu/8g; 3 nodes: 8cpu/16g
(and other info are welcomed to help us debugging)
How To Reproduce
Steps to reproduce the behavior:

Go to 'create 3 master cluster。backup cluster'
Click on 'close one of the master node api'
Scroll down to 'recovery cluster'
See error：etcd recovery failed

Expected behavior
A clear and concise description of what you expected to happen.

/reopen


[Discussion] Site Extensions are moving to nuget.org

Discussion thread for https://github.com/Azure/app-service-announcements/issues/87

What's the best way to validate we've migrated successfully? I can see both the (unlisted) siteextensions.net version (1.0.0) and the nuget.org version (1.0.2) listed at the moment when adding a site extension to an Azure Web App.
https://www.siteextensions.net/packages/RemoveCustomHeaders/
https://www.nuget.org/packages/RemoveCustomHeaders

@tjrobinson looks like it was because the indexer on siteextensions.net was behind. I rebuilt it and that solved it. Presumably it would have self-solved eventually.
As an aside, that's a good example of why we don't want to manage our own NuGet gallery, and instead let the NuGet team do that :)

Is there any way we can do a rename as packages migrate? My packages have naming conflicts that didn't matter on their own index, but will matter on the main nuget.org site.

@zooba Darn, I didn't think about that possibility. Are the ones on nuget.org also owned by you?
It doesn't look like NuGet supports any package rename mechanism, so I don't have a great idea here. Worst case, users will need a one-time update of their ARM template, though I understand that's not ideal.

@davidebbo Some of the nuget ones are owned by me, but they're fundamentally different packages. The existing ones are generic Python packages, while the siteextensions are only good for app service.
I think I'll add an azure- or appservice- prefix to them. LMK if you have any preference

@zooba It'd go with azureappservice- to make it really clear what they're about.

The search functionality in the KUDU gallery doesnt work when the packages are moved to nuget.org.
Also the interface in the portal.azure is quite bad. No search functionality and no version is shown when the packages are listed (only when they are installed can you see it, unless I'm missing something).

@sjkp can you give a specific example of a search that doesn't work? e.g. I tried searching for "python" using Kudu UI, and it returned the Python site extensions, which are all on nuget.org now.
As for the Portal interface being limited, I agree, but this is not specifically affected by this transition.

@davidebbo i'm searching for letsencrypt one of the tags in my package. But it doesnt show up.

Actually if I search for python I also gets nothing.

This is the system info from the web app i tried at ( but I tried a few)
System up time: 19.05:14:45.4210000
OS version: Microsoft Windows NT 10.0.14393.0
64 bit system: True
64 bit process: False
Processor count: 1
Machine name: RD0003FF693AA5
Instance id: 6de56f945ea1f3491c519b8821ae011ec275ee3dd341679345f24d11dc42aeeb
Short instance id: 6de56f

@sjkp it's really quite odd that we don't see the same thing when both searching for 'python'! If you can create a dummy test app where you see this and share its name, I'll try it myself on the very same app.
As for letsencrypt, it does find it for me when searching for 'encrypt', but not 'letsencrypt'. I'm not entirely sure why.

Do you support signed nuget packages?

Once the extension has been moved, how would users update their ARM templates to reflect the new location at https://www.nuget.org/api/v2/ ? Currently I'm getting a BadRequest error.
      "resources": [
        {
          "apiVersion": "2015-08-01",
          "name": "NewRelic.Azure.WebSites",
          "type": "siteextensions",
          "dependsOn": [
            "[resourceId('Microsoft.Web/Sites', variables('webSiteName'))]"
          ],
          "properties": {
          }
        }
      ],

@ridomin that has not come up, and frankly I'm not sure. We're using some older version of the NuGet library to manage the package. You will need to try it.
@neilpdx this change only affects Publishers and not extension Consumers. However, in the case of NewRelic, the package ID was already in use for something else on NuGet.org, so they had to rename it from NewRelic.Azure.WebSites to NewRelic.Azure.WebSites.Extension. Please give that a try.


Removed blocking conditional on ack Task before execution

The execution of a polled task was conditioned by the return of self.taskClient.ackTask which was required to be True. This was never happening because task Ack has been made a Nop and the python client has been left behind (as per discussion on gitter community).
I don't know whether ack TAsk should be removed altogether but in the meanwhile consider removing the if.
Please consider this as a followup of a discussion on gitter.im:

Hi all. I started evaluating Conductor with current docker images (server,ui,dynomite,elasticsearch). I am trying to write a complete rest client starting from python example. After polling succesfully for a task, the ack request returns Always false thus preventing execution. Now I forced the condition to true in the if stmt checking the ack outcome and everything seems to work. What should I check to understand what happens and why ack returns true even of task Is polled correctly? Thank you.

Hey @dcore94 , Ack has been made no-op in one of the efforts, and Python client might be left behind, where it is still checking for the ack response to be true, while the client can safely ignore it. Please send us a PR if you could triangulate the issue and find a resolution.

Pull Request Test Coverage Report for Build 4247

0 of 0   changed or added relevant lines in 0 files are covered.
19 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.09%) to 69.451%

Files with Coverage Reduction
New Missed Lines
%

mysql-persistence/src/main/java/com/netflix/conductor/dao/mysql/MySQLBaseDAO.java
3
69.32%

cassandra-persistence/src/main/java/com/netflix/conductor/dao/cassandra/CassandraEventHandlerDAO.java
8
68.42%

cassandra-persistence/src/main/java/com/netflix/conductor/dao/cassandra/CassandraMetadataDAO.java
8
56.48%

Totals

Change from base Build 4239:
-0.09%

Covered Lines:
12404

Relevant Lines:
17860

💛  - Coveralls

@dcore94 Thanks for the contribution. I'm merging, but feel free to get rid of ack altogether in any of the future changes, if possible. It is not a deal breaker for now.


Implement Slack Webhook Analyzer

There should be a new analyzer to find pastes which (probably) contain slack webhook urls.
To solve this issue there needs to be a new analyzer added in the analyzers directory. The analyzer must follow the example of the other analyzers in this directory. Don't forget to add the analyzer to the analyzer package file.
The most simple example for such an analyzer is the AlwaysTrueAnalyzer which always returns True.
This analyzer should extend the RegexAnalyzer. I expect unit tests to be part of the PR so that you can show off that the given regex works. Please add at least 5 positive and 5 negative tests. Creating uint tests is not hard - see this unit test file for example. If there are questions, don't hesitate to contact me.
You can use the regex used in the shhgit repository: https://github.com/eth0izzle/shhgit/blob/f9b4febcd6ec6c1d509b28efbad6dc1ca9d17837/config.yaml#L354-L356

Dibs, please :)

Implemented in #135


Check for duplicate label names in remote read

In #5731 a bug was discovered where there would sometimes be a timeseries from local storage and a timeseries from a remoteRead that were duplicates of eachother. This happened in cases where a timeseries coming from a remoteRead had duplicate label names. This caused mergeSeriesSet to not merge the remoteRead timeseries with the local storage timeseries.
This PR fixes this issue by adding a function to check for and remove duplicate label names coming from a remoteRead. There are also tests that ensure this is really happening.
CC @brian-brazil

Thanks!


✨ Add timeout while evaluating CEL expression

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
TODOs:

[ ] squash commits
[ ] include documentation
[ ] add unit tests

NOTE it need rebase after we merge https://github.com/SovereignCloudStack/cluster-stack-operator/pull/61 and https://github.com/SovereignCloudStack/cluster-stack-operator/pull/65

can be merged now

since Cluster API always retries upgrades independent of whether the hook server fails one upgrade, we should not include timeouts - they would not have any effect anyway.
We need to make sure to build cluster stacks that work and as such they should successfully upgrade.
We do not expect an upgrade to fail because of no reason and therefore we do not need timeouts. Cluster API does not allow roll-backs anyway


🐛 pkg/server: ensure that home workspace handler gets authz with audit …

…logging

Summary
This resolves https://github.com/kcp-dev/kcp/issues/2627
This resolves https://github.com/kcp-dev/kcp/issues/2627
This resolves https://github.com/kcp-dev/kcp/issues/2627
This resolves https://github.com/kcp-dev/kcp/issues/2627
Related issue(s)
Fixes https://github.com/kcp-dev/kcp/issues/2627

/lgtm
/approve
thanks for the comment!

/approve

/lgtm


Support case-insensitive (configurable) option name parsing

Hi! I'm considering to contribute to this project, and I'd like to know whether this issue is still opened to PR. It seems that this has not been implemented yet. If the issue is still valid, have you got any advice on how to implement this?

Hi, thank you for considering contributing!
Yes, PRs are welcome, especially ones with tests and documentation.  :-)
About this particular feature, I haven’t analyzed what would be involved.
I was also planning to add support for abbreviated options and commands in the near future. That feature and this feature both increase the probability that options are ambiguous, and we need to be careful about that.
For example, picocli currently throws an InitializationException if a command has two options with the same name. If case-insensitive mode is enabled, this exception should also be thrown if two option names only differ in their (upper/lower) case.
Also, the ParseResult class allows applications to query if an option was specified on the command line (and what its value was). If case-insensitive mode is enabled, should this query also become case-insensitive?
There may be other areas where the expected behaviour needs to be clarified.

Hello, @remkop ! I am evaluating different approaches to implement case-insensitive support and would love to hear some feedbacks.

The least modification way seems to be add setCaseInsensitive and isCaseInsensitive methods to the CommandSpec class, since the class holds the map from string to OptionSpec / CommandLine, it would be relatively easy to replace LinkedHashMap to a simple case aware LinkedMap implementation. It would only require minimum changes to the current code, and will be fully backward compatible. It may not be the most intuitive, and it only allows case-(in)sensitive be changed at (sub)command level, not per option. (but is that really needed?)

We can also add the aforementioned methods to both OptionSpec and CommandSpec for individual control over each option / (sub)command. However, is it reasonable to have both case-sensitive and case-insensitive options in one command? This is the most flexible solution, but it would probably be harder to implement and possibly slow at runtime.

ParserSpec seems to be another place for such options to be toggled at a parser level, but users will be unable to toggle for each command, thus not flexible.

Hello @ifsheldon and @NewbieOrange, I noticed you both submitted a pull request for the same issue. First, thank you both for helping making picocli a better library! 👍
I would like to make this feature work as follows:

separate support for making options and subcommands case-insensitive (no special treatment needed for mixins)
enabling case-insensitive mode should be on a per-command basis (not per-option)
when options are case-insensitive, then negatable options and posix (single character) options also become case-insensitive
when options are case-insensitive, the ParseResult methods like hasMatchedOption(char), hasMatchedOption(String), matchedOption and matchedOptionValue should also be case-insensitive
if this information is stored in ParserSpec, we need setters like ParserSpec::optionsCaseInsensitive(boolean) and ParserSpec::subcommandsCaseInsensitive(boolean), and corresponding getters. Similarly if this information is stored in CommandSpec (or in the custom maps), then these accessors could be on CommandSpec (that would also be fine)
the CommandLine facade needs getters and setters for these properties, which are then applied to all subcommands, similar to  CommandLine::setCaseInsensitiveEnumValuesAllowed

Both proposals are part of the way there, but not fully.
@ifsheldon and @NewbieOrange,  how would you like to proceed? One idea is that you collaborate on a solution together, on a branch. Another option is that you work on different parts of the solution, and contribute each part separately. And there may be other ways to do this.
Thoughts?

Hi @remkop , thanks for your feedbacks!
Since ParseResult.matchedOption() calls static OptionSpec findOption(char shortName, Iterable<OptionSpec> options) in CommandSpec class, which does not pass any information about a CommandSpec instance, and findOption is a static method, it seems difficult to get any information about the case-insensitivity.
Also, since ParseResult is mainly used by the developers (not the application users), and the query name is written in code, is it necessary to support case-insensitive in such query? (Is there any use case where the query is generated at runtime?)

Hi @NewbieOrange, here is an example:
Suppose that a case-insensitive command has an option @Option(names = "--option"). The end user specified --OPTION=123 on the command line.
The application may have code like String value = parseResult.matchedOptionValue("--option", -1). This should return value 123.
Implementation-wise, we can modify the static OptionSpec findOption(char shortName, Iterable<OptionSpec> options) method to take an additional boolean, and pass this.commandSpec().caseInsensitiveOptions() to that method.

Currently my PR should have parseResult.matchedOptionValue("--option", -1) return 123, I was thinking you asked parseResult.matchedOptionValue("--OPTION", -1) to also return 123. :)
I have already updated my fork to match all the features you have mentioned, if there is anything missing, please let me know!

Indeed we are working together to contribute. Since @NewbieOrange has proposed a complete solution, it is unnecessary to propose another one, so it's fine that you just check and merge his branch.

@ifsheldon Understood. I agree, your PR is already helping clarify the requirements and improving the solution. Many thanks! 👍

@NewbieOrange I guess it makes sense for applications to query the ParseResult only with the option name that was used in the option definition. I will think about this some more.
Different topic: I don't think that the convenience methods CommandSpec::caseInsensitive and CommandLine::setCaseInsensitive are needed. I prefer that applications explicitly specify both setCaseInsensitiveOptions and setCaseInsensitiveCommands.
Also, did you get a chance to add tests for case insensitive subcommands (and sub-subcommands)?

@remkop The convenience methods are removed and tests for subcommands are added!


Update GSON proto type wrapper. [phab:D1123]

Created by fromberger at 2016-10-27 17:51:43:
We have been using an older version of this wrapper, and it no longer
works with the current Guava release candidates. Update to the latest
from GitHub and drop the now-obsolete patch for enum support.

Fix usage of the ProtoTypeAdaptor type.

shahms accepted this revision at 2016-10-27 17:54:59

fromberger closed via commit ab55ccce6755ae6fb8bee77d093b98398b77cf5c at 2016-10-27 17:56:11


Enable buildifer checks

Run buildifer on bazel files and fixe issues found.

Jenkins job manager/presubmit passed

Jenkins job manager/presubmit passed


Bugfix: console history submission was broken.

cf. https://github.com/jupyter/jupyterlab/issues/118

Thanks!


XOXO Comics won't load the comics' pages

Source information
1.4.10
Source language
English
Source new URL
https://xoxocomic.com/
Other details
The site is still up but the comics pages won't load. When I open a page in webview I get a "page not found" error, so the URLs/links for the comics have changed. If I enter their website on a browser and search for the comic, I can read it normally.
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

There is an issue and the comics won't load. The pages point to broken links.

open a "bug report" instead of "domain change"

Before that, check WebView for errors with said chapters. From a quick test, seems like it's fine with the few I tried. Maybe internet-related issue or the source is missing the chapters/having issues themselves. If you do make a new issue request, list the problematic entries & chapters, otherwise, come to the Keiyoushi Discord repo-support channel if you have other questions/issues.

It was a cache issue, I apologize. It is fixed. Thanks guys!


Provide collection usage stats

Curators are asking for these (for the first time). Ideally they could log in and look this up themselves.
Data desired: Pageview trend per collection.
Acceptance criteria: Figure out and document how to get this data from matomo.

Do we want to go further than google analytics for this?

I don't know. The use case it that curators were asking for useage stats about 'their collections' over past year. So I can imagine wanting filter by slug, and maybe by curator, and having at least 12 months of data at any time. Can we get this from GA? Also wondering if @kayiwa might have an opinion as he starts to work on logging.

Are people still using PUDL? We should look at PUDL stats and other stats people are using.

@hackmastera an idea for this. I will see if I can get some decent feedback and suggestions from the Communications and Exhibitions folks in EOG.

@elrayle mentioned to me that she just implemented an analytics page for Cornell (one page per exhibit). She sent this along as the best starting point https://wiki.lyrasis.org/display/SPOTLIGHT/Analytics+Dashboard


Pandas Performance suggestion

Hello, i am working with your awesome paper and awesome project. And I think I can improve your performance when working ratings data in pandas.
According to https://realpython.com/fast-flexible-pandas/
The code with long loops and dict management can be speed up by replacing:
ratings = pd.read_csv('../data/ml-20m/ratings.csv')
users = np.array(list(set(ratings['userId'])))
ratings_by_user = dict([(i, pd.DataFrame()) for i in users])
for u in tqdm(list(users)):
        ratings_by_user[u] = ratings.loc[ratings['userId'] == u].sort_values(by='timestamp').reset_index(drop=True)

import pickle

ratings_by_user = pickle.load(open('../data/rbu.pkl', 'rb'))

for user in tqdm(ratings_by_user.keys()):
    ratings_by_user[user] = ratings_by_user[user].drop(['timestamp', 'userId'], axis=1)
    ratings_by_user[user]['rating'] = ratings_by_user[user]['rating'].apply(lambda i: 2*(i-2.5))

to_delete = []
for user in tqdm(list(ratings_by_user.keys())):
    ratings_by_user[user]['rating'] = ratings_by_user[user]['rating'].apply(lambda i: 2*(i-2.5))
    ratings_by_user[user] = ratings_by_user[user][ratings_by_user[user][:,1] >= 0]
    ratings_by_user[user] = ratings_by_user[user].values
    # specify your frame size here! 1 end choice + # of movies to be fed into the model
    if len(ratings[i]) < 11:
        to_delete.append(ratings_by_user[user])

for i in to_delete:
    del ratings_by_user[user]

class ML20mDataset(Dataset):
    def __init__(self):
        self.set_dataset(1)

    def set_dataset(self, u):
        self.user = u
        self.dataset = ratings[u]

    def __len__(self):
        return max(len(self.dataset) - frame_size, 0)

    def __getitem__(self, idx):
        ratings = self.dataset[idx:frame_size+idx+1]
        movie_chosen = ratings[:, 0][-1]
        films_watched = ratings[:, 0][:-1]

        films_lookup = torch.stack([movies[id_to_index[i]] for i in ratings[:, 0]])

        state = films_lookup[:-1].to(cuda).float()
        next_state = films_lookup[1:].to(cuda).float()

        rewards = torch.tensor(ratings[:, 1][:frame_size]).to(cuda).float()
        next_rewards = torch.tensor(ratings[:, 1][1:frame_size+1]).to(cuda).float()

        action = films_lookup[-1].to(cuda)

        reward = torch.tensor(ratings[:, 1][-1].tolist()).to(cuda).float()
        done = torch.tensor(idx == self.__len__() - 1).to(cuda).float()

        state = (state, rewards)
        next_state = (next_state, next_rewards)

        return state, action, reward, next_state, done

with arrow processing when working with pandas df but still yield corresponding result

        print("Start update ratings %s" % (datetime.datetime.now(),))
        train_df = train_df.copy()
        train_df["rating"] = train_df['rating'].apply(lambda i: 2 * (i - 2.5))
        train_df = train_df[train_df["rating"] >= 0]
        users = train_df[["user","item"]].groupby(["user"]).size()
        users = users[users >= self.frame_size + 1]
        train_df = train_df[train_df["user"].isin(users.index)]
        user_rated = train_df.sort_values(by=["user", 'timestamp']).drop(columns=["timestamp"]).set_index("user")

        self.user_rated = user_rated

        print("Done update ratings %s" % (datetime.datetime.now(),))

        batch_bar = tqdm(total=self.get_total_user())

        for user, df in self.user_rated.groupby(level=0):
            batch_bar.update(1)
            size = max(len(df) - self.frame_size, 0)
            for idx in range(0, size):
                if np.random.rand() > 0.2:  # intake percents
                    continue
                ratings = df[idx:self.frame_size + idx + 1]
                ratings = ratings[["item", "rating"]].values

                movie_chosen = ratings[:, 0][-1]
                films_watched = ratings[:, 0][:-1]
                ...

Above process took
Start update ratings 2019-04-09 16:44:01.957343
Done update ratings 2019-04-09 16:44:24.071254
Instead of manually looping over each element to get it's index
id_to_index = dict([(u, i) for i, u in enumerate(pd.read_csv('../data/ml-20m/movies.csv')['movieId'].values)])

You can use
movies_series = pd.read_csv('../data/ml-20m/movies.csv')["movieId"].reset_index(drop=True)
movies_series = pd.Series(movies_series.index.values, index=movies_series)
id_to_index = movies_series.to_dict()

This is my first issues on github, and i want to contribute to your awesome project.

Thanks, haven't start doing the data loader yet, the one I did is deprecated. But I will make sure I have used your suggestions! The update will probably come out today. Thanks again, I'd been running into major performance issues

Tho line  train_df = train_df[train_df["rating"] >= 0] is wrong because it neglects all the negative rating reviews which are really helpful for the critic. DDPG is supposed to work with negative rewards and learn 'what not to do' from them

You are right, I have double checked the original paper and found the difference. I will come back with new code as I am implementing that paper myself soon.
On paper they also use epsilon greedy to generate random action, and they re-rank all remaining items in each step and select the highest one.
Though because of fixed N, I have to make sure user begin state include his K positive item, not sure his first N positive item based on timestamp or just random N positive item from his session.

Observe current state s t = f(Ht), where
Ht = {i1 , ..., in }

Done! I have also started working on an asynchronous version but it doesn't seem to be functioning properly just yet. If you need code it is located in the notes under Async Dataloader. I used aiostream with async generators and streams for synchronizing. Everything is just copy paste. My vision is that the ratings need to be split by users and fed into async workers that will yield the mini-batches into the stream


Refactored GSON 2.7 for Schedules Direct

I just need to check with some folks internally to see if this needs to go in the third_party directory or not since it is Google code, but from a different project.

I can just move it and save you the effort. It might make it easier to understand what's going on down the road too.

I already asked....your call, I'll get a response but it might not be until
Monday.
Jeff Kardatzke
Sent from my Android
On Sep 17, 2016 2:34 PM, "Joseph Shuttlesworth"<EMAIL_ADDRESS>wrote:

I can just move it and save you the effort. It might make it easier to
understand what's going on down the road too.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/pull/175#issuecomment-247809485, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANEIDNzgY8giE8jbJEqAC-lCyVDnDm9rks5qrFztgaJpZM4J_tY9
.

I moved it. Let me know if you need further details in the folder or what other files need to be updated for legal reasons.

So will we also need to alter the plugin repo to remove the gson.jar with this change? (I'm reading this as pulling GSON into the core, right?)

It's refactored into another package, so it will not conflict with GSON if someone else imports a different version of GSON for a plugin.

Got ya.. if I had thought for more than a couple seconds on it I would have realized that.. ;-)


Adjust sometimes-failing test to account for stubbed PPM discount rates

Description
In #5699, the discount rates were hard-coded to account for the fact we aren't loading BVS rates anymore.  The test adjusted in this PR would dynamically pick one of two TSPP records (whichever's ID came first alphabetically) to use for the target of the test assertions.  But now that we've hard-coded the rates, the test would only pass if one of the two were picked.  This PR hard-codes the IDs in the test itself to make it always pick one of the two TSPP records as the target.
Reviewer Notes
We may not ultimately need this test after the PPM rate engine is reworked in the future, but I didn't want to assume that yet.
Setup
make server_test

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 393fbd3af53416b35623757672d04584acbc7a24

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 393fbd3af53416b35623757672d04584acbc7a24


maxBytesPerPartition not working as stated in the documentation

Describe the bug
I have set the value of maxBytesPerPartition to 1 and when i produce a message as big as 126KB, the message still get consumed, wherein the documentation it is stated that
The maximum amount of data per-partition the server will return. This size must be at least as large as the maximum message size the server allows or else it is possible for the producer to send messages larger than the consumer can fetch. If that happens, the consumer can get stuck trying to fetch a large message on a certain partition
which means the consumer should be stuck and the message should not be consumed.
To Reproduce
you can try setting this value to 1 and produce a very big message.
Expected behavior
the consumer should be stuck and no message should be consumed.
Observed behavior
the message still get consumed.
Environment:

OS: Mac OS 14.2.1
KafkaJS: version 1.13.0
Kafka version: 3.1.0
NodeJS version: 14.18.0

@abhinavgautam07 I found this issue when I encountered something similar. I do think that this has to do with the api version this package supports as described in my new issue #1693


HMR not working

I downloaded the project and restored the dependencies in VS 2017.  When i run the project using VS2017 by F5, it is running in the browser successfully.  But when i edit the html and save, it is not reloading the changes in the browser.
I am getting the below message in browser console.
Failed to load resource: net::ERR_CONNECTION_RESET
Pls help me to resolve this.

@mageshbala90 can you try removing wwwroot/dist folder and run again to see if it makes any different?
I think you have done npm install without any errors?

Hi  @asadsahi ,
The issue is the visual studio solution is closed in the background due to an exception in common.es5.js.  This causes the iis to stop running and produces the "Failed to load resource: net::ERR_CONNECTION_RESET" error.
**Unhandled exception at line 18448, column 5 in http://localhost:5000/dist/vendor.dll.js?v=rYKpEjxEyf-G0grd3roPYLNmfrTdUgY0t6weJb0GNS4
0x800a138f - JavaScript runtime error: Unable to get property 'apply' of undefined or null reference**
Please help me to resolve this error.
Thanks

Hi @asadsahi ,
I forgot to uncomment polyfills.. It is working fine now.   I can customize it as my needs.  Thanks a lot for your contribution...

@mageshbala90 Glad you resolved it.
thanks for kind words. :)


Implement open science badges

As described here:
http://journals.plos.org/plosone/article?id=10.1371/journal.pone.0125208
So, basically just 3 images, and a boolean field to toggle them on each publication.

I realize that with the journal being mandatory open data, such badges make little sense. There are a few cases where they would not get tagged on, e.g. theoretical papers or papers that rely on unshareable data (e.g. using private information, but not possible to anonymize).

The more important reason for implementing them is this:
http://emilkirkegaard.dk/en/?p=6233
To be eligible for what counts as an open science journal for the purpose of the preregistration challenge, one must meet certain criteria. One of these is having the badges. Since it's just a matter of having 3 boolean fields more for the Publication class and a if statement in the template for the badges, this is pretty easy to do.


ANSI escape code support

I maintain various command-line tools that produce coloured output with ANSI escape codes. Every time I change the output format, I manually update various terminal screenshots on the project READMEs. I've been looking into using carbon to automate this process (and produce much nicer screenshots!).
This may be considered outside the scope for carbon, but it'd be really great to be able to generate images of files containing arbitrary ANSI escapes (rather than doing syntax highlighting on the file). This would allow using carbon to generate screenshots of terminal interactions as well as code snippets.
What do you think?

I'm open to supporting it!

@mfix22 @CraigFe Integrating something like this: https://github.com/rburns/ansi-to-html seems like the way to go.

Hey @mfix22  👋
I wanted to contribute to the project (as a long time user of the app) and saw this issue open and labeled as a good first issue.
Since I didn't make any PR to carbon previously, I would probably need some guidance on how to start with this one.
Can you confirm that issue is still open (with no work being done) and is up for grabs? 🙂
If so, can you please provide more info on where one could start in order to successfully implement this?
Many thanks! 🎉

Can you assign this to me ?

Hey @mfix22 👋
I wanted to contribute to the project (as a long time user of the app) and saw this issue open and labeled as a good first issue. Since I didn't make any PR to carbon previously, I would probably need some guidance on how to start with this one. Can you confirm that issue is still open (with no work being done) and is up for grabs? 🙂
If so, can you please provide more info on where one could start in order to successfully implement this?
Many thanks! 🎉

@ognjenjevremovic feel free to open a PR to implement this feature 👍


How to style the validation text

Is there a way to style the validation not seeing that is possible and the validator only tacks a string

I see its not possible
style: TextStyle(
                        color: Colors.red[800],
                        fontSize: 12.5,
                      ),

Can we get similar thing on MultiSelectDialogField ? As of now it has hardcoded value of Colors.red.shade800.withOpacity(0.6) as validation text color


Is faster-whisper on an old CUDNN Version?

I received the following error when using faster-whisper
Could not load library libcudnn_ops_infer.so.8. Error: libcudnn_ops_infer.so.8: cannot open shared object file: No such file or directory
Aborted (core dumped)

Other issues pointed me to install libcudnn8 via the apt-get package manager. The thing is, I have installed cudnn-9.2 as per Nvidia's official instructions. (https://developer.nvidia.com/cudnn-downloads?target_os=Linux&target_arch=x86_64&Distribution=Ubuntu&target_version=22.04&target_type=deb_local)
After viewing the cudnn.so files ls /usr/lib/x86_64-linux-gnu | grep libcudnn, I noticed cudnn was correctly installed but there is no libcudnn_ops_infer.so.8 or libcudnn_ops_infer.so.9 file within the directory.
Is it best to delete the 9.2 install and revert back to cudnn 8, or wait for ctranslate2 to update to the latest cudnn?

@psych0v0yager , hello. According to GPU requirements:

Note: Version 9+ of nvidia-cudnn-cu12 appears to cause issues due its reliance on cuDNN 9 (Faster-Whisper does not currently support cuDNN 9). Ensure your version of the Python package is for cuDNN 8.

Therefor, you should back to use cudnn 8. To install libcudnn_ops_infer.so.8:
sudo apt-get install libcudnn8 libcudnn8-dev


added PageBindings that references GetPageRoute in  [dependencies] method

this adds abstract class PageBindings extends Bindings that references the GetPageRoute object in its void dependencies([GetPageRoute? page]);
useful in navigation 2.0 when using GetRouterOutlet with initialRoute that requires parameters (Since the route isn't added to the history stack, we can't access its parameters from the router delegate).

Nice, but I think GetRouterOutlet not being added to the stack history is the real issue here.

it shouldn't tho, since the actual route being displayed is the initial route, and it just delegates the view part to a different route

But if we navigate with the button, the url displayed is from the initial route, that seems inconsistent to me. It's actually the only thing stopping me from migrating the current app of the company I work to nav 2, and that's sweet, because nav1 doesn't display urls correctly.
It also seems to bother other users, and cause bugs in the rest of the framework, such as the lack of removal of controllers linked to the initial route.
https://github.com/jonataslaw/getx/issues/1700
IMO this behavior is inconsistent, and unless explicitly stated, the home page should be added to history.
It also seems to bother other users, and cause bugs in the rest of the framework, such as the lack of removal of controllers linked to the initial route.

I don't think that's a problem.
the user should have complete control over what to display based on the current navigation stack.
the developer can even completely ignore initial route and use emptyWidget builder instead from pick pages.
also I am using the current nav 2 system in more than 5 production apps with all sorts of cases, and I never notice any problems with disposing controllers and such.

I also added a reply on #1700 on how to add the initial route to the stack


add missing Option to avatar

This Option is present in crates.io's definition of EncodableUser, so I think it should be in Cargo too.

r? @huonw
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+ fc68cf5

:hourglass: Testing commit fc68cf5 with merge 425fd85...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


REPL directory doesn't show current directory

On VS 2017 with RTVS 0.6
I changed it to a different folder using the select working directory button (last one on the toolbar) and the combo box doesn't get updated.

Fallout from filtering setwd activities

#2312

https://github.com/Microsoft/RTVS/commit/1e553fc7584a9c6be107baf4d4e76debe5610218


Singletons are getting disposed

I have a problem with registered entries as singleton.
If I add a registration as a singleton (with IDisposable as interface), these objects are getting disposed on request ending in aspnetcore, but still exists in the container on the next request, but disposed off course.
Is this normal? I would think if you register a singleton, It would not get disposed on request scope?
ex: container.Configure(x => x.For(serviceType).Use(implementationType).Singleton());
Thanks,
Koen

combining singletone and idisposable is considered as a bad idea :)

Hmm. I'll have a look. It sounds really weird that a singleton should be disposed when a nested container is disposed.


chore(node_framework): Emit logs on context/resource access

What ❔

Adds more logs when the node is being configured. We can certainly improve it further (e.g. print task name in resource-related logs or use prettier type output for Unique), but I think it's a good starting point.

Why ❔

Logs are helpful for debugging, especially in dynamic systems like the new framework.

New logs look roughly as follows:

I wonder what will be the impact on logs

@EmilLuta Next to none. 1. These logs are printed once at node launch, and most of impact comes from stuff like logging in API request handlers. 2. For one-shot machines you can always configure logging as zksync_node_framework=warn so that info level logs do not appear.


[Vue warn]: Unknown custom element:

Hi everyone....
I need your help please with this stuff:
When i try to use a special field any one between the five fields, I got :
[Vue warn]: Unknown custom element: <vuetable-field-{{the field name here}}> in my console..
Here are the examples:
-[Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.
-[Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.
-[Vue warn]: Unknown custom element:  - did you register the component correctly? For recursive components, make sure to provide the "name" option.
here is my code:
`
		<vuetable-pagination-info ref="paginationInfoTop"></vuetable-pagination-info>

		<vuetable-pagination ref="paginationTop"
			@vuetable-pagination:change-page="onChangePage"
		></vuetable-pagination>

	<vue-table
		ref="vuetable"
 		api-url="https://vuetable.ratiw.net/api/users"
		:per-page="10"
		:fields="field"
		pagination-path=""
		@vuetable:pagination-data="onPaginationData"
		:css="css"
	>
	</vue-table>

		<vuetable-pagination-info ref="paginationInfo"></vuetable-pagination-info>

		<vuetable-pagination ref="pagination"
			@vuetable-pagination:change-page="onChangePage"
		></vuetable-pagination>

`

I had the same problem.
The problem was that I had installed the version
npm install vuetable-2 @ next --save
And I was using the old doc instead of the one corresponding to my installed version
https://www.vuetable.com/
For the costomized fields I have to add this object in my fields table
 {           name: 'actions',           title: 'Actions',           titleClass: 'text-center',           dataClass: 'text-center',           width: '15% '         }
and

 ........

inside the vuetable tag


Fix mono build properly!

Fixes the mistake I made in #17603 to make it require Python >= 3.6.

Alright I just tried with 2.7, 3.5 and 3.6 and all of them worked. Seems good now.


Map Background

The background image for the map seems to be drawn over the hud overlay although the had is drawn after the map.

fixed with c57426e252059da409d3f071d77011ceb94e7a1b
fix is somewhat buggy because batch.begin is made twice, once in the stage and once for the map sprite, but it is what it is


coq-lsp: init at 0.1.3+v8.16

Description of changes

Add https://github.com/ejgallego/coq-lsp
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

It seems that version 0.1.4 is out, right?

Thank you for reviewing. I moved the expression to coq-packages.nix and updated to 0.1.4.
Tomorrow I will test if it's still working, I'm a bit busy with work.

Hi, coq-lsp 0.1.5 has been released. Would we be able to bump the version here?

Also we have a flake capable of building that has been merged, does that help things?

Hi @Alizter I'd suggest packaging 0.1.6 which is just out.
coq-lsp is a standard dune package (an vscode extension) so as long as Coq kinda works things should be fine, it also obeys COQLIB and COQPATH if Nix is using them.

Thank you for your reviews. I updated the expression to 0.1.6+8.16 and tested it works correctly.

Great @marsam , thank you very much!
We've detected a bug specific to 8.16 support (we usually test in 8.17), it is a small bug but annoying, we put a new https://github.com/ejgallego/coq-lsp/releases/tag/<IP_ADDRESS>%2B8.16 tag with the 8.16 specific fix.

Updated to <IP_ADDRESS>+8.16


changed  https://github.com/github link

Issue being fixed or feature implemented
The github links in the Readme containing the various repository is misleading. If you do not look at it you will think the links are all private or does not exist. One can simply pass it or think its private source.

What was done?
The link was changed from https://github.com/github/dashevo/android-dpp.git to https://github.com/dashevo/android-dpp.git

Thanks for fixing this typo, we will merge this in a few days.


replace component with script setup

✔ Deploy Preview for timber-grader-polecat-68568 ready!
🔨 Explore the source changes: e3211f0f7e69521e705e1b2915a1f4531d1c502b
🔍 Inspect the deploy log: https://app.netlify.com/sites/timber-grader-polecat-68568/deploys/61ff5c5f4bfa2a0008bec304
😎 Browse the preview: https://deploy-preview-464--timber-grader-polecat-68568.netlify.app/


Fix #72: chunked buffer for SimpleCharStream

Reworked SimpleCharStream in order to avoid O(N^2) complexity for buffer expansion (see issue #72)
Tested the code both on JavaCC and CelestaSQL test set, verified in profiler.

I suggest adding this as a ne class - ChunkedSimpleCharStream so we don't destabilize existing users - especially backup is a tricky method an we don't have good tests for that.

Hello @new-javacc ! I'm glad to see the recent activity in the project and see my PRs being merged.
Concerning the new class for ChunckedSimpleCharStream. If we do this, what do you think is going to be the way to switch between a SimpleCharStream and ChunckedSimpleCharStream?  Now we have a  JAVA_UNICODE_ESCAPE boolean option that switches between JavaCharStream and SimpleCharStream. Do we need another option to make use of 'chunked' version? If set to true, it will be incosistent with JAVA_UNICODE_ESCAPE  set to true.
Concerning the tests. If we add this class as a new standard CharStream implementation, then it will be missing any tests at all. For now, it's being tested by existing tests, and I can give my evidence that it works on my SQL project. Maybe other guys, big guys like e. g. Apache Calcite, could try to build their systems using the chunked version and report if it works for them (@vlsi, do you think it's a feasible idea?)
Maybe it's worth the risk trying to substitute the standard SimpleCharStream with the chunked version?..

I think Travis CI job can be added, and it might be configured to execute Apache Calcite's tests when using new JavaCC.
Current Calcite tests take ~15 minutes, and master is almost always buildable so I guess we can just run them in full (e.g. excluding checkstyle).
What do you think?

Do we need another option to make use of 'chunked' version?

Isn't "chunked" always better?

Isn't "chunked" always better?

I believe it is! The @new-javacc's point  (if I get it right) is that the new version may contain subtle bugs that are not catchable by the current test set, so it's better to have an option for users to fallback to the old version. In my opinion, this will only bring more chaos to the JavaCC project.

I think Travis CI job can be added, and it might be configured to execute Apache Calcite's tests when using new JavaCC.

Looks like an easy thing to do. @new-javacc , if Calcite builds with the chunked version of SimpleCharStream, will it be a sufficient evidence for you? 😃

A less intrusive approach to this issue would be to change the buffer growth strategy. For small tokens, there is no effect -  the buffer is still a char[] and the javacc code behaves as it does now. Only ExpandBuff changes where the new buffers are created.
The current reallocation strategy in JavaCC is char[] newbuffer = new char[bufsize + 2048];
Following the example of java.util.ArrayList.newCapacity:
   int newSize = (bufsize == 2048) ? bufsize+2048 : (bufsize+(bufsize>>1));
   char[] newbuffer = new char[newSize];

which goes 2048, 4096 as before then increases faster.
For a 1Mb buffer, the linear resize copying is 267,911,168 chars.
For a growth of 1.5, the copying is 2,385,340 chars.

Frankly speaking, I think the either way works, and the current approach avoids copies altogether.

@vlsi - yes - but there was a concern in this thread that the complexity of the change to the chunked array may have unexpected effects.
Changing just the allocation only has any impact when an allocation occurs at all.
I hope this simple approach will mitigate the effect and that people will try it out in their systems.

I would like to see perf tests for normal cases (like the java grammar in the examples folder) to make sure there is no overhead for the most common cases where ExpandBuf should be called extremely rarely.

That's the problem - we have none! So you need to create some for this "improvement" to show it actually is one that does not impact the normal 99.99% cases that should not/rarely call expandbuf.
Start with the Java grammar, create a gigantic file (may be concatenate one of the generated parsers which is huge - some 100 times) and see how tokenizing that works with and without your change.

@new-javacc , I suggested adding JMH for benchmarking 2+ years ago: https://github.com/javacc/javacc/issues/151.
Would you please do so?

I don't want to do anything that's not core (and no charstream is not
core) lol! I'm lazy and also super oldschool in terms of memory
management etc and focus only on performance - which is a hallmark of
Javacc.
That's one of the reasons for me to separate out the codegenerators so I
don't own them at all! They are like reference implemnentation and if
someone finds it lacking, then can add a custom code generator and fix
it for themselves or for the community as well!
Anyway since you are the one trying ot say the old perf is bad, it's on
you to prove it and do whatever is necessary for that.

@new-javacc https://github.com/new-javacc , I suggested adding JMH
for benchmarking 2+ years ago: #151
https://github.com/javacc/javacc/issues/151.
Would you please do so?
—
Reply to this email directly, view it on GitHub
https://github.com/javacc/javacc/pull/85#issuecomment-1179163984, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE2B3LJBJCGFOO7TUVSTZL3VTBIODANCNFSM4HAUNPEQ.
You are receiving this because you were mentioned.Message ID:
@.***>

Would it make sense to make the buffer size grow exponentially instead of linearly? Then chunking won't be needed and hopefully the change will be deemed safe, without the need for more flags. The ArrayList approach proposed here   uses exponential growth too.


change sans-serif references to fonts

@vinniefalco would like to see any references to "Noto Sans", sans-serif fonts changed to "Noto Sans", serif so any cases where the fonts are not displaying correctly would be more obvious.
Requesting a change to have serif in staging and sans-serif in production, so it's obvious in staging if the fonts aren't loading and has a gentle fallback in production.
Additionally, add display:swap to the font-face tags to remove loading delay.

we already have font-display: swap : https://github.com/boostorg/website-v2/blob/develop/static/css/fonts.css#L7


feat: update testify version to v1.8.0

update stretchr/testify version to v1.8.0 to fix CVE-2022-28948
Signed-off-by: Valina Li<EMAIL_ADDRESS>

@vli11 thanks for the upgrade. The fails test because the go.sum entry is missing. Could you please do a go mod tidy from the same directory?


Reference container when rendering MyLoginXComponent in lightbox demo

Fixes #146

Can we make this render into 'body'? Ideally lightbox-style components should be rendered directly into the body so they can show a background overlay over the whole page.
Thanks for taking the time! :)


DB builder error at step 11-- assertion error?

Hi there! Thank you for making this awesome annotation tool, I am really excited to try it out on my metatranscriptomes.
I have been working on installing the database on my space within my university's Linux cluster environment. I had to restart and run some steps individually during steps 1-10 because we are supposed to download files and run jobs on different host machines, but everything seemed to work up until step 11. Can you give me any tips for addressing this error? I found a past issue  that looks similar, but it looks like it was a small program bug that had been fixed.
Thanks again!!
(microbeannotator) [grifnata@congo microbeannotator]$ microbeannotator_db_builder -d MicrobeAnnotator_DB -m diamond -t 40 --step 11
2022-08-22 12:01:01,503 [INFO]: This is MicrobeAnnotator v2.0.5
2022-08-22 12:01:01,503 [INFO]: I will download and format the databases I use.
2022-08-22 12:01:01,503 [INFO]: Creating database folders
2022-08-22 12:01:01,503 [INFO]: Step 11
Traceback (most recent call last):
File "/nfs5/Lab/grifnata/conda/envs/microbeannotator/bin/microbeannotator_db_builder", line 445, in
main()
File "/nfs5/Lab/grifnata/conda/envs/microbeannotator/bin/microbeannotator_db_builder", line 437, in main
single_step, aspera, keep_temp, bin_path)
File "/nfs5/Lab/grifnata/conda/envs/microbeannotator/bin/microbeannotator_db_builder", line 206, in database_duilder
viral_list = refseq.search_all_files(temp_gb_dir / "viral")
File "/nfs5/Lab/grifnata/conda/envs/microbeannotator/lib/python3.7/site-packages/microbeannotator/database/refseq_data_downloader.py", line 293, in search_all_files
assert(dirpath.is_dir())
AssertionError

I figured out this issue-- it seems to be because when running steps individually, any temporary files created during the step are deleted at the end. In this case a temporary file created during step 10 was required for step 11.  I fixed this by re-running step 10 with the --keep flag to avoid deleting the temporary file used in step 11.


Fix: Fix false positive with empty array variable in require-meta-has-suggestions rule

If the suggestions array is initialized as an empty array variable, we need to assume that someone may have pushed suggestion objects to it, so we should consider that a usage of suggestions.
Previously, this rule incorrectly assumed that using const suggest = []; ... context.report({suggest,...}); meant the rule was not using suggestions.
Real world example here: https://github.com/eslint/eslint/blob/master/lib/rules/no-nonoctal-decimal-escape.js#L91

released in<EMAIL_ADDRESS>

Thanks!


int parses to instant

hi
thanks for the lib
i tried  to parse  the below but the newerThan field is being parsed to Instant instead of int
{
specs: [
{
from<EMAIL_ADDRESS>s3Bucket: "xxxx",
jobs: [
{
description: "xxxx",
s3Destination: "xxxx",
folder: "Inbox",
filter: {
combiner: "AND",
//in days
newerThan: 30,
subject: "test subject 9",
"from": "xxx.co.uk"
},
onFailure: {
copyTo: "failure_archive",
delete: false
},
onSuccess: {
copyTo: "success_archive",
delete: false
}
}
]
}
]
}

Yes, ints are parsed as Instants by default. I will change this in the next version, that was actually not intended to be true by default.
For now, you can add the following to your main or in a static{} block:
JSONOptions.setDefaultOptions(JSONOptions.builder().parseInstants(false).build());

Yes, ints are parsed as Instants by default. I will change this in the next version, that was actually not intended to be true by default. For now, you can add the following to your main or in a static{} block:
JSONOptions.setDefaultOptions(JSONOptions.builder().parseInstants(false).build());

thank you


Assert thrown in glfwGetJoystickAxes()

I'm experiencing this in all three examples when running a debug build on Windows 10.
In i_ctx_create() you malloc the struct, but don't clear the fields to zero so joy_exists and joystick_id consistently have the values 205 and 52685 respectively. Next in i_joy_create(input_ctx, 0) the if (!ctx->joy_exists) isn't true because of the garbage value. Finally, in i_ctx_update(), the if (ctx->joy_exists) passes and glfwGetJoystickAxes() throws an assert because the jid exceeds GLFW_JOYSTICK_LAST.

I've updated the input polling, can you check if this still is present on your machine in the latest commit 91fd57d7c23e57ca9ce87fcb18fabef74882a08c?

Fixed

Glad to hear! Thanks for taking the time to test & report this!


Upgrade dependencies & raise minimum Node.js version

I've made several upgrades to madge related dependencies in the last few months. The result should be that madge is better aligned with the behaviour of other project and allow deduplication with their dependencies.
Here are some highlights though likely incomplete:

PostCSS 8 (https://github.com/joscha/node-detective-postcss/pull/76)
Smaller dependencies for module-lookup-amd (https://github.com/dependents/node-module-lookup-amd/pull/29)
Remove dependency on make-dir (https://github.com/webforge-labs/requirejs-config-file/pull/12)
enhanced-resolve v5 - matching Webpack 5 - with support for exports in package.json (https://github.com/dependents/node-filing-cabinet/pull/94)
@typescript-eslint/typescript-estree v4 (https://github.com/pahen/detective-typescript/pull/30)

Awesome job! I'll merge and create a new major release of Madge since it contains some breaking changes.

Awesome job! I'll merge and create a new major release of Madge since it contains some breaking changes.


Update fs2-core, fs2-io, ... to 3.2.9

Updates

co.fs2:fs2-core
co.fs2:fs2-io
co.fs2:fs2-reactive-streams

from 3.2.7 to 3.2.9.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "co.fs2" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #506.


Running only one test at a time for development

It would be nice to be able to run just one test at a time to help focus and minimize clutter in the output during debugging.
There are two approaches to this, test.only or specifying test name in the command args either by string or regex.
Examples:

test.only() in Vitest
cargo test [name]

Hey there 👋🏻 , this is something that's been on our list too. We will look into it, but as we have mentioned in other issues, it may take some time due to other priority higher tasks we have on our hands right now.

@dimitrovmaksim Would you accept an external PR that adds this?

Hey @hasparus, all contributions are welcome. We are planing some changes to the repos and want to start fresh, that's why I'm closing all issues here.


Improve diagnostics aggregation method

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I've agreed with the maintainers that I can plan this task.

Description
Recently MRM supports the multiple behavior that are emergency stop and comfortable stop and will plan to support pull over in https://github.com/autowarefoundation/autoware.universe/issues/3221. Therefore, the system must select an appropriate MRM from the diagnostics.
Current diagnostics do not have well-structured diagnostic trees, making it difficult to select a appropriate MRM. Autonomous driving systems are complex, so diagnostic dependencies are directed acyclic graph (DAG) rather than tree.
For users, there is the problem that it is difficult to understand the error that prevents the switch to automatic driving, and thare are false positives. Also, I feel that terms such as SPF/LF/SF are a little different from their original meanings, so I would like to improve these as well.
Purpose

Select appropriate MRM when detecting errors.
Filter errors appropriately according to vehicle status and user interests.
Supports DAG diagnostics dependencies.
Supports diagnostics for redundant system.

Possible approaches
The diagram below is the sample of an improved diagnostic structure. Each block in the diagram has the same information as DiagnosticStatus.msg with extra "links" field for DAG. The mechanism is almost the same as diagnostic_aggregator. So we need to extend it to support DAG, or create a similar implementation.

Error classification
Now the system_error_monitor classifies each status as either SPF/LF/SF/NF. However, the severity of the impact depends on the functionality that requires it. For example, consider a situation where the system has LiDAR and radar, and only LiDAR is broken. The object detection can work if either is fine, so that is not an error. But the object prediction, which needs the exact position of an object, requires LiDAR, so that is an error.
So instead of categorizing each error, consider the impact on the functionality that each error affects. In this case it is sufficient to create a DiagnosticStatus for that functionality and determine its level. In conclusion, remove the classification (SPF/LF/SF/NF), use the error level (ERROR/WARN/OK) instead.
Select MRM
Create a DiagnosticStatus for the functionality for each MRM and set the diagnostic dependencies appropriately for that MRM to work. Then the system can easily find the currently available MRMs by checking at their error levels.
Also, create a status for normal functionality such as autonomous driving and remote driving in the same way. Then simply prioritize these functionality and select the one with the highest priority.
Error report
Create a root (it's not a tree, but a node that can reach all other nodes) DiagnosticsStatus for error reporting. It dynamically determines the level of error according to the user's interests and vehicle conditions. For example:

Ignore most errors during initialization.
Ignore errors of autonomous driving while stopped.
Notify errors of autonomous driving as warning while preparing for autonomous driving.

Backward compatibility
You can create messages compatible with /hazard_status by assigning error categories as follows:

Report OK
Report WARN
Report ERROR

Status OK
NF
-
-

Status WARN
SF
LF
SPF

Status ERROR
SF
LF
SPF

Definition of done

[ ] create aggregator for DAG.
[ ] create new diagnostics config.

It was not possible to support graph structures without changing the plugin interface for diagnostic_aggregator. It seems that AnalyzerGroup cannot share child Analyzer with others.

I'll change the Autoware diagnostics as follows.

Replace diagnostic_aggregator and system_error_monitor with system_diagnostic_graph node.
Repalce diagnostics_agg and hazard_status with diagnostics_graph and diagnostics_summary topic.

diagnostics_summary is similar to emergency, emergency_holding fields in hazard_status
diagnostics_graph is similar to nf, sf, lf, spf fields in hazard_status or diagnostics_agg

PR

https://github.com/tier4/tier4_autoware_msgs/pull/96
https://github.com/autowarefoundation/autoware.universe/pull/4722

Add diagnostic graph aggregator

https://github.com/tier4/tier4_autoware_msgs/pull/96
https://github.com/autowarefoundation/autoware.universe/pull/4722
https://github.com/autowarefoundation/autoware.universe/pull/5340
https://github.com/autowarefoundation/autoware.universe/pull/5447
https://github.com/autowarefoundation/autoware.universe/pull/5816
https://github.com/autowarefoundation/autoware.universe/pull/5827

Package creation is complete. We will integrate with MRM in https://github.com/orgs/autowarefoundation/discussions/4176.


Reverting because this was committed twice and is causing another build break.

@compnerd @vedantk @dcci @JDevlieghere


Design on-boarding screen using on-chain funds

If the dlc-channel API allows for it, we can use the same flow as before, i.e. let the user deposit a certain amount where we provide money in the channel aswell

Related to #1793

I'm closing this. We should create a ticket about how to on-board users into DLC Channels


Implement a POST API call which will take confirmation from the my-site

[ ] Implement a POST API call which will take the boolean ie, isAuthorized .
[ ] If isAuthorized, then the backend should generate the access token and save it for now.
[ ] if !isAuthorized, then it wont generate the access token.

not needed


Release object_store YYYY (next release after0.5.2)

Follow on from https://github.com/apache/arrow-rs/issues/3229

Planned Release Candidate: ~TBD~ 2023-01-06
Planned Release and Publish to crates.io: ~TBD~ 2023-01-09

Items:

[ ] Update changelog and readme:
[ ] Create release candidate:
[ ] Release candidate approved:
[ ] Release to crates.io:

cc @tustvold -- any thoughts on other features we would want to get in?

I'm not aware of anything outstanding, it's probably about time to make a release 👍

unless I hear otherwise I'll plan to cut a release later this week / weekend and release next week

Changelog in https://github.com/apache/arrow-rs/pull/3457

Release thread - https://lists.apache.org/thread/01p1zbpfblw91137oh3hsc9379oczg8t