[FEATURE REQUEST] Save State by Hotkeys

Hello, I've noticed this emulator is capable of save states but the way it does it is not one I personally like much, just one hotkey to save and you switch savestates with hotkeys too.
Is there any way I could request to make it different, so it's Hotkey-based instead?
What I mean is, F1 will load Save State 1, meanwhile Shift+F1 will save it there.
I know the emulator is already preconfigured to have F1 and F2 as Soft & Hard Reset respectively, as well as that some may prefer this method instead.
No problem, I am sure making this into an option, even if it means you have to take your time to configure it, would be a great idea itself.
I would use PCSX2 for reference on how to do these hotkey-based Save States.

I'll need to think about how to handle this in the hotkey code (it's frankly a bit of a mess right now) but I'm willing add an option for this.
The main thing I'll need to figure out is how to handle key combinations in a clean way - right now those aren't supported at all in the hotkey/input code.

Hello,
Chiming in here, i like the way it's implemented in jgenesis, i have personnaly mapped F1 and F2 to save and load state and f3 / f4 to increment/decrement slot.
Then with joy2key I have mapped these to gamepad combo which allow me to use entirely my gamepad without keyboard.
What is would rather see is the ability to set combo buttons of gamepad for these functions instead of being keyboard only, that would avoid having to use joy2key.

It took a few tries to get this working decently, but I've added support for key/button combinations, mapping hotkeys to gamepad buttons (and mouse buttons), and hotkeys for saving/loading specific slots. I haven't merged these changes yet because I still need to do a bit more testing, but a current build is available here:
https://github.com/jsgroth/jgenesis/actions/runs/11752065124?pr=137
If you test with this build I recommend using a different folder or backing up the config file, because input and hotkey configuration is stored in an incompatible format to previous versions (including the current latest release version).
There is a little weirdness when the same key is used as both an individual mapping and as part of a key combination, as in the example of mapping F1 to load state and mapping Shift+F1 to save state. As currently implemented, if you press and hold Shift and then press F1, only the Shift+F1 mapping will trigger (save state). This is based purely on combination length which seemed like the most intuitive way to handle this, but there might be a better solution.


Fix the email validation error message on the recovery page #2130

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Screenshots

Original
Updated

Checklist

[x] 🔽 My branch is up-to-date with "development" branch
[ ] ✅All tests passed locally
[ ] ✨My changes working with up-to-date admin and back-end part locally, like charm
[ ] 🔗 Link pull request to issue

It seems you repeat yourself with font properties...


Proper fix for change in re.split in Python 3.7

Compare with commit 9c906b4

Out of interest, why isn't Python 3.5 still in the build matrix?

Thanks! I don't remember what happened with python 3.5


⚠ Bofrost.be has degraded performance

In 6972132, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2550 ms

Resolved: Bofrost.be performance has improved in 1bfc431.


Jenkins Server Sync

sir, I getting problem with my jenkins server. While I'm creating the projects in Jenkins those are not synching to my Jenkins Machine . Can you please guide me to resolve this issue.

please check your console log it will display where is target file and your war package
otherwise please send me the jobs console log

please send us detailed log

Daybuild-gol #2 Console output.pdf

/var/lib/jenkins/workspace/Daybuild-gol/gameoflifeweb/target/gameoflife.war
above location you have your war file

yha! I know  that but my projects(Workspace) are not shown in my  ubuntu machine


Clean up QubitManager types

This is a follow-up to #833, where some internal types in the Qubit Manager were updated to QubitTypeId that didn't need to be. See comments there for context.

These fixes made in response to Dmitry's feedback on #833. One thing to note is that changing these internal counts to use int32_t means they will not be able to accommodate the highest qubit id of type intptr_t. It's incredibly unlikely we will hit a problem with the difference between the maximum values of these types, but it's worth calling out as a place where there is a disconnect between two indirectly linked types.

(Sorry if my comment adds even more ambiguity ;-) )
Looking at #833 and this PR I feel that in a broad sense the qubit identifier and the number of (supported) qubits are different concepts, and it is reasonable for those concepts to use different types (similar to how in the operating systems we have a concept of a file descriptor (handle) and a number of files that can be opened at the same time - completely different types are used).
Another illustration, as has been mentioned earlier, could be a long-running quantum computing cloud service, that allocates and deallocates qubits endlessly. And upon every allocation the qubit id increments until the type overflows, and id enumeration starts over. Whereas the number of qubits used at the same time can be relatively small, e.g. 1000.
We may want to consider a new type to express the number of qubits, e.g. MaxQubitQuantity (in common case not equal to QubitIdType and not equal to int32_t, but in particular case can be an alias of any of those types or of the other type, e.g. uint16_t). After this introduction of a new type we will be able to change that type easily, if needed, with a small fix in one place.


[Feature request] Ability to show latest git commit info/message

Hi, thanks for your awesome work. I'm trying to get the statusline showing the latest git commit information (author, date, message) I tried to find around here as well as Google but has no luck. Do you have any plan to implement it or know how to work around?
Sorry if this question already asked / duplicated.

You can read from command git log -1 (latest commit information).
You can extract the information you want using git pretty formats.
For example, git log -1 --pretty="%cN, %ar, %s" will output something like You, 12 days ago, Some progress on feature. Note that you most likely want to limit the number of chars.
For the implementation itself, you can read command's output using io.popen. Here is an example.

Thank you, I will give it a try.

I found out a way,  using the coc-git (I know it's not "lua-qualified" but it works for me at least for now), I can reach the b:coc_git_blame and assign it back to galaxyline is so smooth:
        provider = function()
            return vim.b.coc_git_blame or 'nothing'
        end,


HacktoberFest19

Name the program which you are adding
I am Adding the suffix array creation based on a string in C++
Programming Language you will code
C++
YOU ARE
a student
GITHUB USERNAME
fazd

Please send the PR.


Port codec-haproxy to Buffer API

Motivation:
Helps to have a pipeline working only with Buffer API
Modifications:

Port codec-haproxy to Buffer API
Adapt the junit tests

Result:
codec-haproxy now is using the Buffer API

@violetagg Thanks!

Yeah, let's go with a copy in HAProxyMessage. Tried a few things myself, but wasn't able to come up with something better.

yeah I also tried without copy but with no success :(


Fixing import/from colors on .tsx files

This closes https://github.com/sdras/night-owl-vscode-theme/issues/78

nice, thanks!


Added rtu.txt and reverted accidental deletions

This pull request adds the domain rtu.ac.in for Rajasthan Technical University to the swot repository. The file rtu.txt has been placed under the appropriate directory structure lib/domains/in/ac/rtu/. This addition will allow students and staff from RTU to utilize the services that rely on the swot repository for domain verification.
I also reverted an accidental deletion of Compiler.kt and Swot.kt files, ensuring that no critical files were removed in this process.

@Husain-Bohra desktop.ini I am sorry, I have to close the request without merging, because you made a mistake in a request and did not specify any filepath. Please ensure that you are familiar with this repository rules located at the bottom of this page and make a new correct request. Thank you for your understanding!


Blue bottom bar - add number of lines when selecting text

It's a very small feature but would be usefull. When selecting text you can see the amount of characters selected. Can you add number of selected lines too? Something like (200 selected in 5 lines)

See https://github.com/microsoft/vscode/issues/45726 and others.  That link mentions an extension that will show the number of selected lines.

Duplicate of #45726


Add op2 map writer and refactor OP2 Map code

I'd like to continue working on and improving the OP2 Map code, but these changes are significant enough that a merge into master would be smart.

No worries. I already know what caused the conflicts and figured it was coming before pushing. I've got some experience fixing merge conflicts in Mercurial and want to try it out here. It is more being caused by working in MapHeader.h in 2 different branches and deleting MapHeader.cpp in the master branch while it exists in this branch than moving the files. However, you are right it would have been cleaner if I had separated renaming the directory into a separate commit.


Inline editting/Add/Remove for w2ui url based grid

Hi Vitmalina,
First of all, I wanted to thank for this wonderfull library and congratulate you for this huge work and effort !
I'm Rihab, a python dev and I'm now working on a project in which I wated to use the w2ui library.
I started by showing and sorting grids using url method but now I'm facing a problem becuse I can't find a way to add/remove/inline edit records when using url method to load the grid?
I wish you could help me to find the right way to do so.
Looking forward to seeing your answer.
Best regard,
Rihab.

There are 2 python server implementations right here: https://github.com/vitmalina/w2ui/tree/master/server/python
They both handle get / save / delete requests.
Note: they are not yet updated to work with w2ui 1.5 - See this issue for hints on how 1.4 and 1.5 differ.
About inline editing: you need to keep in mind that you don't get a single record, but an array of records.
If your url is pointing to a JSON file you still need to write the backend to handle saving/updating the records.
If you are stuck and have a specific programming question, you should visit stackoverflow instead.

Hi mpf82 !
Thank's a lot for your answer.
I'll see the python server ssolutions you suggested to me.
For my url returns an ' application/json' content-type which is the result of an SQL request for a data base  using flask API RESTful.
I'm really new in this field and if I got a stack I will go to stackoverflow as I always did.
Thank a lot again for your answer.
Best reagrds.
Rihab.


README.md: Fix the Matrix address

There is a plumbed room at #feather:monero.social, while #_oftc_#fether:matrix.org is a portal room. This is problematic because it results in two Matrix rooms bridged to the same IRC. #feather:monero.social has significantly more users, so switch to that in the README to avoid new users joining the portal room.

Thanks.


Create reference table of issues reported by pkilint

It would be nice to have a reference table which summarizes the information about the issues discovered by pkilint, something similar to this page for pylint.
I found some CSV files in the repository which provide these details, but I am not sure whether this information is comprehensive, and I think it should reside in a more prominent place.

I think of an approach that is a minor and natural extension of what is already available.

Keep these severity, code, description tuples in the repository itself, rather than on a separate site. Thus, the docs are close to the code and are easier to keep in sync.
Keep the current CSV format. Github renders it as a table; it requires horizontal scrolling but it is not too bad, since the main concern is to have the info.
Define a naming and location convention for these CSV files.

My personal preference is to keep all the codes in a single table, rather than in linter-specific ones, because this way I don't need to have N windows open and go through all of them whenever I'm looking for something. This can be done in a later stage, where the CSV files are transformed into HTML by a script invoked in the CI pipeline.


about .env file in the README

readme says:

Configure umami
Create an .env file with the following

DATABASE_URL=(connection url)
HASH_SALT=(any random string)

So... create an .env file where? with what name?

.env is the complete filename for your environment file. Check https://docs.docker.com/compose/env-file/ for more

.env is the complete filename for your environment file. Check https://docs.docker.com/compose/env-file/ for more

it works thanks! It was kind of confusing because create an .env file sounds like you have to create a file with that extension, not name


Go to definition

TODO:

[x] Dart tests for the different symbol types/ReferenceKinds.
[x] Server capability (declare it in initialize response, add handler).
[x] VS Code test.

Fixes #9

Bound to be some cases that I missed during testing, but it seems to be working well enough to move on to the next one.


Mod data for option + control

Sometimes QLab places multiple shortcuts on the same key with different labels. They are accessible with the mods option and control, but they then create conflicts that are conflicting on the shortcut mapper.
E.g.
Enter Edit Mode	⇧⌘[
Enter Show Mode	⇧⌘]

Are not place correctly on Danish keyboard as they are hidden with the option modifier.
However the following works:
SHIFT + OPTION + COMMAND + 8 = Enter Edit Mode
SHIFT + OPTION + COMMAND + 9 = Enter Show Mode

Potentially adding mod data for option + control could solve the issue.

Expand and collapse are overlapping with other shortcuts on English layouts.


Tooltip is overflowed

Link to file.

I don't know of a way to solve that. Since the HTML for the tooltip belongs to the <a> element and that is hierarchically inferior to the top bar there's no z-index that will solve it (this way of saying is probably not the best, but you get the analogy, I'm no CSS expert to know exactly why it will not work).
Maybe there's a way to fix make it work, but I'm not aware of it. Do you know how can we make this work?
Perhaps making the tooltip appear on the bottom of the link instead of on the right? Do you feel that's a good idea? Or making the whole tooltip text fit in a single line, that could work too.

Perhaps making the tooltip appear on the bottom of the link instead of on the right? Do you feel that's a good idea?

Tooltip will be overflowed if link will be on bottom of source code :)

Or making the whole tooltip text fit in a single line, that could work too.

Tooltip will be overflowed on narrow displays :)

I think tooltip must be created via JS, not CSS, and have position: absolute with top and left based on <a> position relative to the document.

Will a new version be published in Chrome Web Store? :roll_eyes:

I'm sorry. I really need some help to get things going here.
I must port this code that is currently running in free backend providers so it'll run on your computer, then I can release a 2.0.0 to Chrome Web Store.
Or you could migrate to Firefox in the meantime :P

Okay, thank you, don't worry! We're with you ✊

Or you could migrate to Firefox in the meantime :P

I tried, I didn't like, thank you 😁


{{ format-ignore-start }} causes weird issues

When ignoring a piece of Blade template and then running the formatter, I get the following error:
SyntaxError: Unexpected closing tag "div". It may happen when the tag has already been closed by another tag. For more info see https://www.w3.org/TR/html5/syntax.html#closing-elements-that-have-implied-end-tags (172:9)
[error]   170 |
[error]   171 | </Bvquium45fhI5Hw4B06nwmucmWxtkrsEmVaqZKZerQmrEcsaB>
[error] > 172 |

[error]       |         ^^^^^^
[error]   173 |     </BRx4jNgACUUiqJ9hvs1kvS7ZY6JX8daTucyyjUK5j0EZZv9fumHVUdQa19bj7p686KtcjVG7M4QUk0lkUk92T43W8H4soVgCw8TlLsbNPHmBtvDMDsUHRWVxGPAubvQzwjsts01bwrqiM6ibHJ5wKgL9j3jYGeSsDyot8zb1Ib0U01y12TlB>
[error]   174 |
[error]   175 |

Edit: probably my fault as I added the comments around some attributes on a div, while it should wrap the entire div. However, when I wrap the entire div, its contents get replaced by <BoRWTjjG3HrujYEnB />.

Would you be able to share more of the template that is causing the issue? thanks!

Yeah it's inside a conditional. This is a minimal example which reproduces the issue:
@if (true)
    {{-- format-ignore-start --}}

    {{-- format-ignore-end --}}
@endif

Resolved in 1.3.0 for this situation 🙂


add rockspec

add rockspec: sqlite is a dependency but I chose not to expose it
hack to fix tests: wouldn't it be simpler to use the envvar in all cases ?

Currently the ci tests failing for two reason:

upper/lower type names: Had this issue for awhile, I guess we need to make sure that is "always" lower case.
test/auto/stmt_spec.lua:96: types no longer return in ci, maybe this should just be ignored/commented out

I tried to make luarocks test succeeds and it was failing because of that code: sqlite_clib_path is not set in the minimal_test.vim.

In that case, how about if not vim, try to read path from env var? or regardless if not set then read from env var. That way users may have the freedom to pick either one.

unrelated but would also be nice to have a vim.health script to check sqlite is correctly loaded.

Definitely it would, can you open an issue so we can later implement

this should be more acceptable.

upper/lower type names: Had this issue for awhile, I guess we need to make sure that is "always" lower case.
test/auto/stmt_spec.lua:96: types no longer return in ci, maybe this should just be ignored/commented out

sry I didn't check those, could you rerun the workflow so I have a look please ?

LGTM, I will fix test later, in another pr along with ci issues

I've added a makefile target to run stylua locally so that I could fix the issues. The problem is that the stylua version I use locally may be different than the one on the CI. Since you are a nix user, would you mind if I added a flake just for development ? to get lualint/stylint and correctly export LIBSQLITE ?

I've added a makefile target to run stylua locally so that I could fix the issues. The problem is that the stylua version I use locally may be different than the one on the CI. Since you are a nix user, would you mind if I added a flake just for development ? to get lualint/stylint and correctly export LIBSQLITE ?

Not at all. I like where this is going.

I've added the flake, running nix develop followed by luarocks test seems to pass. I wouldn't mind this being merged if only so that the github workflows get run automatically in followup PRs ^^'


Conversion To Swift

@Skornos Any plan to convert it to swift?

Hey, currently our focus is on different parts of the project.

OK Thanks


UnitTests errors

Hi,
I managed to build ARMNN and ran  UnitTests. There are 4 failures detected and below is the log. I reviewed the whole build procedure again throughly couldn't find any issue. Moreover, I don't think these errors are  related to the way the libraries are built.
Before debugging further, thought of checking if something else is going wrong.
Or any pointers for further debugging would be great.
Thank you.
There is no change in any of the source files. I'm just following the build instructions from here and running the UnitTests on and Android board (Firefly RK3399) with Android 7.1.2.
D_LIBRARY_PATH=/data/local/tmp ./UnitTests                                    <
Running 626 test cases...
unknown location(0): fatal error: in "Compute_ArmComputeNeon/MultiplicationBroadcast1Element": std::runtime_error: in validate_arguments src/core/NEON/kernels/NEPixelWiseMultiplicationKernel.cpp:62: Tensors have different shapes
/home/san/san/armnn/src/armnn/backends/test/ArmComputeNeon.cpp(245): last checkpoint: "MultiplicationBroadcast1Element" entry.
unknown location(0): fatal error: in "Compute_ArmComputeNeon/MultiplicationBroadcast1DVector": std::runtime_error: in validate_arguments src/core/NEON/kernels/NEPixelWiseMultiplicationKernel.cpp:62: Tensors have different shapes
/home/san/san/armnn/src/armnn/backends/test/ArmComputeNeon.cpp(246): last checkpoint: "MultiplicationBroadcast1DVector" entry.
unknown location(0): fatal error: in "TensorflowParser/ParseMultiplicationBroadcast4D1D": armnn::Exception: LoadNetwork failed
/home/san/san/armnn/src/armnnTfParser/test/Multiplication.cpp(141): last checkpoint: "ParseMultiplicationBroadcast4D1D" fixture entry.
unknown location(0): fatal error: in "TensorflowParser/ParseMultiplicationBroadcast1D4D": armnn::Exception: LoadNetwork failed
/home/san/san/armnn/src/armnnTfParser/test/Multiplication.cpp(159): last checkpoint: "ParseMultiplicationBroadcast1D4D" fixture entry.
*** 4 failures are detected in the test module "UnitTests"
Thanks,
San

Correct.. :) I'm using v18.03.
Will move to v18.05  and try again. There are very few commits between v18.03 and v18.05 though.

Moving ACL to v18.05 solved the above errors.
Thanks a lot!!


Actions self-hosted  run locally

Description
Can Gitea Actions only work with Docker, or is it possible to run tasks locally in the context of the runner, similar to GitHub Actions?
I use this example https://gitea.com/action-examples/CPlusPlus
push to my self-hosted gitea
and try use Gitea actions
I download https://dl.gitea.com/act_runner/0.2.11/act_runner-0.2.11-windows-amd64.exe
and rename to act_runner
C:\bin\gitea>act_runner generate-config > config.yaml
C:\bin\gitea>act_runner --config config.yaml register

After that i have in my Gitea Runners Management offline runner
C:\bin\gitea>act_runner daemon --config config.yaml
time="2024-11-14T11:12:23+01:00" level=info msg="Starting runner daemon"
Error: daemon Docker Engine socket not found and docker_host config was invalid

C:\bin\gitea>act_runner exec --config config.yaml
time="2024-11-14T10:40:10+01:00" level=info msg="Using default workflow event: push"
time="2024-11-14T10:40:10+01:00" level=info msg="Planning jobs for event: push"
time="2024-11-14T10:40:10+01:00" level=info msg="cache handler listens on: http://<IP_ADDRESS>:4816"

In github action runner is running and wait for jobs ,
Gitea Version
1.22.3
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
Windows 10
How are you running Gitea?
self-hosted over docker
Database
SQLite

I found solution:
replace in config.yaml default labels to
  labels:
    - local
    - windows


AutoGPT Email Plugin Issue: Emails Going to Drafts and Unable to Read Emails

Dear Riensen
I hope this message finds you well. I am writing to request assistance with an issue I've encountered while using the email plugin for AutoGPT. I have been following the provided guide to set up and use the plugin, but I am experiencing difficulties when attempting to send emails and read the received emails.
When I use the "send_email" command, I receive the following error message:
Command send_email returned: Email went to [Gmail]/Drafts!
Instead of sending the email as intended, it is being saved to the Drafts folder in Gmail. Additionally, the plugin is unable to read the received emails.
I have double-checked the guide and followed the instructions carefully, but the problem persists. I would appreciate any guidance or suggestions you can provide to help me resolve this issue. Please let me know if you require any further information or logs to better understand the problem.
Thank you in advance for your assistance.
Sincerely,

Hi @Ismatpg
It seems like you activated the option EMAIL_DRAFT_MODE_WITH_FOLDER as described in https://github.com/riensen/Auto-GPT-Email-Plugin#9-add-email-configuration-settings
If you want to send emails, then just delete the key from the .env

Thanks!


Update rest-client to 2.x + bring in changes from forks

Includes the ssl cipher error fix https://github.com/strixleviathan/victor-ops-client/commit/d6a6c82300248c0065e511150f3434646717741a as well as some changes from @dhulihan's fork to remove awesome_print. Still need to test this in a live environment...

Confirmed working under Ruby 2.5.1.


[Dockerfile] Clean up apt cache, make image smaller

This will make the image smaller! (From 438MB to 421MB)

Thanks @PeterDaveHello !


Energy card not working with Draconic evolution energy core

Though given the capacity capable i would not be surprised if that was intentional rather then a bug...its ....a big number

I guess this is duplicating #973

Can you test out if it works if you use the latest McJtyLib from here:
http://jenkins.k-4u.nl/job/McJtyLib 1.11/lastSuccessfulBuild/artifact/build/libs/mcjtylib-1.1x-2.3.6-SNAPSHOT.jar
I implemented support but was not able to test it. Thanks

Can confirm it is working on Pylon (not the core - but honestly, I expected Pylon). Now it's not reporting internal buffer (like One Probe did), but real core energy. Kudos!

I can also confirm, RF Monitor is working and detecting correct buffer from Pylon.


🐛 FIX(Recipes/Remix): explicit exports to fix paths links/metas

This PR assures that the routing catch is working
I noticed a bug in previous implementations ( using export * from  ) where remix was not able to detect the functions in all $puckPaths  so I just exported them explicitly and tested all sites and they are working by this PR

Hey @ahmedrowaihi thanks for sharing the PR. I worked on the Remix recipe and the bug must have slipped through.
Do you mind sharing the minimum steps to reproduce?

Thanks @ahmedrowaihi. As @EarthlingDavey said, please can you provide steps to reproduce? It worked during my testing of #227, but it's possible something was missed.

Is it possibe that the bug was happening when you were on a nested route like /slash1/slash2/edit. The Remix recipe doesn't support nested routes (yet).

Hi @ahmedrowaihi ok, I understand.
For some more context... the Remix recipe was released only a few days ago, so there may be other undiscovered bugs too. The good news is that I doubt it will have made it to production anywhere - so we can take our time to discover and apply fixes.
If you hit any other bugs, I'm sure other Issues or PRs will be welcome. Also, you can tag me in Discord or GitHub, and I'll try and make a timely reply.
Thanks for your support on the project, I look forward to any further contributions :) And, have a great weekend!


[Feature Request] Dotted lines around "Unknown" when frigate connection

Now that we have frigate connections properly denoted under "Types" for wormholes, it'd be nice to display the dotted line for frigate holes around "Unknown" destination.
This serves as intel as to why that connection wasn't mapped, and someone can bring the right ship to map it.

@porowns I´m not sure what exactly you mean?
Do you mean the dotted lines (top/bottom) in the "leads to" sig. table column ?
So in this example the frig wh L005 signature is mapped to the C2 but not linked in the sig. table. And you want the dotted borders around the unknown "leads to" cell?

Yeah, I think it'd be useful for explaining why some signatures aren't mapped (e.g someone needs an astero)

After checking the existing code base, it turned out to be more complex than expected.
I had to implement this a bit different:
The "type" column now shows the "size" ('S', 'M', 'L', 'XL') label.


Explicitly check if tooltip callback is undefined before ignoring value

The current implementation incorrectly dismisses values returned from tooltip callback methods if they are falsy rather than explicitly checking that they are missing. See the following example:
...
label: (tooltipItem) => {
  return 0; // Assume a more realistic situation resulting in the label being 0.
},

With this fix we will instead check for undefined or null values, and otherwise use the return value.

In this case the ESLint suggestion pointed out by codeclimate would actually cause us to exclude null from the check 😒. Is the preference of this project to just do two checks in this situation?

The label callback should be returning strings, so in your case I would expect '0' as the return value.

@etimberg Is this documented anywhere? It seems quite odd to me to not accept numbers when the majority of the time, that label will be a number.

In the doc files it's mentioned in https://github.com/chartjs/Chart.js/blob/master/docs/configuration/tooltip.md#tooltip-callbacks

Alright, well i think it's confusing at best that the implementation essentially works perfectly fine with any truthy value that can be meaningfully coerced to a string, but then quietly fails with falsy values. Seems a little arbitrary to me and i'm certain less diligent users of the library won't really notice this behaviour if they happen not to test out falsy values. And on top of all that, it's fairly trivial to change the behaviour. That's just my two cents.

Rebased and switched to the helper :)

Not sure why tests are failing

Oh, i think i know what's happening. This change not only makes it so that the falsy value of 0 are not ignored, it also does the same for all falsy values, including the empty string. The tests seem to rely on the empty string (the default value for the title) being ignored.
In my view the empty string is a valid value just as 0 is, but i'm not sure if it actually makes a difference to what is displayed in the end anyway?

If I'm not wrong, considering empty string valid means that if a callback returns '', an empty line will be displayed but right now this empty line is skipped, so that would be a breaking change for users who used to return empty string to remove a tooltip entry (though I'm not sure why it would override the callback to return an empty string).
The document is not very explicit on how to remove/skip a line, but simply says:

"All functions must return either a string or an array of strings"

It should also add: "... or undefined/null to display no text"
I would personally consider empty string valid (and so generate an empty line) but maybe skipping empty string is safer to prevent breaking too many projects. Or should we promote this breaking change and a bug fix, IDK?! @etimberg

I agree with your thoughts here. I suppose it depends on what real world usage is like out there, there may well be users who have callbacks that fallback to returning the empty string when some conditions are not met, i'm not sure.
Taking a look at your releases, it seems that the versioning model is not semver. What is the versioning model?

We use semver (MAJOR.MINOR.PATCH)

In terms of the callbacks. I always used returning an empty string to mean: show the grid line but not label. Returning null/undefined meant hide the grid line and label.


Inner method triggers 'public method must have explicit type'

def outerMethod():String={
  def innerMethod()="foo"
  "bar"
}

gets a warning that innerMethod is a public method that needs an explicit type.
I'd argue that innerMethod is not public.

In fact the second part is a duplicate of #33 so I'm now closing this issue.


Crash

Describe your bug here.
The engine crashes when I try to add ghost animation
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
I did not

Right now, there isn't a thing such as Ghost Moving Animation in Psych Engine, only hardcoded, Ghost Animation in the Character Editor is only used better visual for adjusting character position


macOS version on launch shows "To use Java command-line tools you need to install a JDK"

System information:

Operating system (distribution) and version:  macOS 10.15.4 with all updates
DBeaver version:  7.0.5
Additional extensions

Connection specification:

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

Describe the problem you're observing:
Whenever I try to run DBeaver, I get the following dialog box:

To be clear, app never runs.
Steps to reproduce, if exist:
Download the .DMG and mount.
Window show usual application and alias to /Applications.
Drag/drop application icon to /Applications alias.
Double-click DBeaver.app in /Applications folder.
I figured you included the necessary JRE in the macOS bundle as some other apps do, since the main GitHub page states

...
Running
DBeaver requires Java (JRE) 1.8+ to run.

Windows and MacOS X installers include JRE so just use them and don't think about internals.
On Linux you may need to install Java manually (usually by running sudo apt-get install openjdk-11-jdk or something similar).
If you don't use installer (on Windows or Mac OS X) you may need to download Java (JDK) from Adopt OpenJDK web site.
...

Odder still is that I DO have Java installed, specifically Oracle JRE version 8 update 251.  So not sure what's going on here.  But dialog box does indicate the app is trying to do something which requires the JDK vs. the JRE.
Include any warning/errors/backtraces from the logs

Not possible as can't get to Window menu since app won't start, and there's no ~/Library/DBeaverData/ let alone subdirectories therein.

from the download page "If you are on Mac OS X and you do NOT use PKG installer then you will need to setup Java."
Just choose the "installer" option instead of dmg.

"from the download page"... which page would that be exactly?
And again, I DO have Java setup.  It's just Oracle's Java 8 update 251 JRE.
Looking around, there's what is listed here that I referenced earlier:
https://github.com/dbeaver/dbeaver
Then there's the official site's download page:
https://dbeaver.io/download/
But I think the confusion may well be that the former page indicates that

DBeaver requires Java (JRE) 1.8+ to run.

and also says

Windows and MacOS X installers include JRE so just use them and don't think about internals.

The implication is that you only need the JRE.
But I suspect this is not the case.  The fact that on the latter page the following appears under the "Install:" section tips the hand:

Java notes: if you use Windows or MacOS installer then you don’t need to install Java. Otherwise you will need to setup Java separately. Java 1.8+ must be the default Java version. To validate this run “java -version” in terminal window.

THAT, if true, requires the JDK.
[I didn't put much stock in references to Adopt OpenJDK where mentioned, as anyone who wishes to run the JRE on macOS but doesn't have/want to use the Oracle JRE has no choice with Adopt OpenJDK as they don't offer an installer with just the JRE.  All they have is the full JDK which then includes the JRE, which is all most folks likely need anyway.  But it's looking like here that DBeaver requires the JDK.  And if true, this should be documented as such.
I had just figured since a .DMG file was offered with the usual drag/drop, combined with the bit about "Windows and MacOS X installers include JRE so just use them and don't think about internals", that maybe the DBeaver devs had done similar to what HelpSystems/Dartware did with their InterMapper software, where they bundled in the necessary Java runtime within the macOS application bundle itself, making it fully self-contained.  Guessing that's simply not the case here.
As a side note, you may also wish to clean up the docs to not only reflect that DBeaver requires the JDK, but also stop mixing the version numbering between saying it requires 1.8+ and then in other places referencing Java 8 (see first link above as you'll see both on the same page between the Running and Building sections).  Just call it Java 8.  It will create less confusion.

your second link is the download page, and the first link of the mac os section is the one dubbed as "installer" which should be a PKG file.
That said, I don't use MacOS nor I'm a dbeaver developer, just a user trying to help based on what I've seen in many similar issues in the past.
Also I don't see how your quote shows dbeaver needs a jdk to run instead of just a jre. The java program is part of the jre, not the jdk.

Ah, @p91paul , that explains it.  Since you're not a macOS user, allow me to clarify.  On macOS, the Oracle JRE does NOT provide the java utility.  Seems odd I know, but that's the reality.  Hence my opening an issue for the developers, as the documentation indicates all that is needed is the JRE (and specifically in reference to macOS).  And that, as I've noted, I have installed.
Also, again since you're not a macOS user, you may not know this either.  The AdoptOpenJDK project does a fantastic job taking Open JDK, the open-source implementation of the Java platform, and bundling it up into an easy to consume macOS installer.  That installer, however, is for the full JDK.  They don't even offer a JRE-only installer.  And whereas the Oracle JRE provides a System Preference (think Windows Control Panel or the various control panels in Gnome/KDE/etc.) which then gets you to the Java control panel (what you see if, for example, you go into the Windows Control Panel and double-click the Java icon), AdoptOpenJDK installs "under the hood" and you need to use a shell to do things with it.
My suspicion is the developers have been building the project using AdoptOpenJDK, so they may not realize that the project does, in fact, require the JDK and not just the JRE on macOS.  It's no big deal, but the docs as written may confuse new users.  It's one of the unfortunate side effects of using a language that doesn't compile straight to the binary of the OS, that you need to be clear what the requirements are just in order for your application to run.  (This would apply to any dynamic scripting language as well such as Perl, Python, Ruby, etc., so not restricted to Java, mind you).

Oracle JRE does NOT provide the java utility. Seems odd I know, but that's the reality

WHAT?? so what does it offer, an empty zip file? the JRE is basically the java program and its dependencies, I wouldn't know what to do with a JRE without java.

Well, we need JDK on MacOS to run. JRE is not enough.
I'll fix instructions on download page (by replacing JRE on JDK everywhere).
In fact JRE is a historical term, there is no such thing in the OpenJDK.

Still not working for me - I have installed OpenJDK and dbeaver from homebrew. Java version:
java --version
openjdk 14.0.1 2020-04-14
OpenJDK Runtime Environment (build 14.0.1+14)
OpenJDK 64-Bit Server VM (build 14.0.1+14, mixed mode, sharing)

Is there a workaround for this? What should I be doing differently?
Thanks in advance


Where do the campaign dates come from?

Hi there. I'm trying to validate that I'm using all of the information you've provided correctly in our application. The first report we've ingested is the Tick group. I see two campaigns in the data, and they match up with the specific attack patterns used for each one just like I see on the Playbook Viewer page. However, the data file names one campaign "January 2018" but on the Viewer it says, "December 2017 to January 2018". Where do those dates come from? Are you using the "first_seen" and "last_seen" dates and extrapolating?
"December 2017 to January 2018"
{
      "type": "campaign",
      "id": "campaign--be7cda7f-8a8a-4bf6-9ba3-5c713664ecdf",
      "created": "2018-07-11T20:25:11.667Z",
      "modified": "2018-07-23T17:49:05.500Z",
      "name": "January 2018",
      "description": "\nTick group targeted a specific type of secure USB drive [snip]...",
      "first_seen": "2018-01-01",
      "last_seen": "2018-01-12"
    }

"October 2016 to June 2017"
{
     "type": "campaign",
     "id": "campaign--358b5915-2a63-4406-828f-890558e27a1f",
     "created": "2018-08-03T13:20:50.755Z",
     "modified": "2018-08-06T18:11:18.234Z",
     "name": "July 2017",
     "description": " The “Tick” group has conducted cyber espionage attacks [snip]... ",
     "first_seen": "2016-11-01",
     "last_seen": "2017-07-01"
   }

The campaign dates come from the Playbook JSON.
The dates are read in from the Playbook JSON and converted to Date objects in the Viewer using JavaScript's Date(...) constructor, which is most likely displaying the date in local time instead of UTC. Depending on the timezone and time of day it is possible this causes an off by one (day) error. However, since the first_seen and last_seen are frequently the first and last day of the month respectively, this one day change throws off the month in the Playbook Viewer.
I have merged changes that should help to correct this.
If there are any more discrepancies between the dates in the Playbook JSON and the dates in the Playbook Viewer, go with the dates in the JSON, and please let us know.

Fantastic. I saw the update today and the months match up to the first_ and last_seen properties now. Thanks!


Customer UI: cancellation option

How to demo:
Customer can select option for item or room cancellation
Changes to transaction history should be shown when chosen
Order total should decrease

Hours worked: 15mins
Added the option to cancel items and a room for the customer UI


build problem.

❓ Questions and Help
unable to execute '/usr/local/cuda:/bin/nvcc': No such file or directory
error: command '/usr/local/cuda:/bin/nvcc' failed with exit status 1
thanks for u code. need your help

Hi,
Make sure that you have the cuda toolkit and the cuda drivers properly installed in your system. You are missing the nvcc compiler, which is the nvidia compiler for cuda programs.

jiangtao@jiangtao:~$ nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2017 NVIDIA Corporation
Built on Fri_Sep__1_21:08:03_CDT_2017
Cuda compilation tools, release 9.0, V9.0.176
thanks for your help. does it means i already have nvcc?

running build
running build_py
running build_ext
/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/utils/cpp_extension.py:182: UserWarning: Error checking compiler version for g++: a bytes-like object is required, not 'str'
warnings.warn('Error checking compiler version for {}: {}'.format(compiler, error))
building 'maskrcnn_benchmark.C' extension
gcc -pthread -B /home/jiangtao/anaconda2/envs/Pytorch1.0/compiler_compat -Wl,--sysroot=/ -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -DWITH_CUDA -I/home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/torch/csrc/api/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/TH -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/THC -I/usr/local/cuda:/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/include/python3.6m -c /home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc/vision.cpp -o build/temp.linux-x86_64-3.6/home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc/vision.o -DTORCH_API_INCLUDE_EXTENSION_H -DTORCH_EXTENSION_NAME=C -D_GLIBCXX_USE_CXX11_ABI=0 -std=c++11
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++
gcc -pthread -B /home/jiangtao/anaconda2/envs/Pytorch1.0/compiler_compat -Wl,--sysroot=/ -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -DWITH_CUDA -I/home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/torch/csrc/api/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/TH -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/THC -I/usr/local/cuda:/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/include/python3.6m -c /home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc/cpu/ROIAlign_cpu.cpp -o build/temp.linux-x86_64-3.6/home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc/cpu/ROIAlign_cpu.o -DTORCH_API_INCLUDE_EXTENSION_H -DTORCH_EXTENSION_NAME=C -D_GLIBCXX_USE_CXX11_ABI=0 -std=c++11
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++
gcc -pthread -B /home/jiangtao/anaconda2/envs/Pytorch1.0/compiler_compat -Wl,--sysroot=/ -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -DWITH_CUDA -I/home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/torch/csrc/api/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/TH -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/THC -I/usr/local/cuda:/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/include/python3.6m -c /home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc/cpu/nms_cpu.cpp -o build/temp.linux-x86_64-3.6/home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc/cpu/nms_cpu.o -DTORCH_API_INCLUDE_EXTENSION_H -DTORCH_EXTENSION_NAME=C -D_GLIBCXX_USE_CXX11_ABI=0 -std=c++11
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++
/usr/local/cuda:/bin/nvcc -DWITH_CUDA -I/home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/torch/csrc/api/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/TH -I/home/jiangtao/anaconda2/envs/Pytorch1.0/lib/python3.6/site-packages/torch/lib/include/THC -I/usr/local/cuda:/include -I/home/jiangtao/anaconda2/envs/Pytorch1.0/include/python3.6m -c /home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc/cuda/ROIPool_cuda.cu -o build/temp.linux-x86_64-3.6/home/jiangtao/github/maskrcnn-benchmark/maskrcnn_benchmark/csrc/cuda/ROIPool_cuda.o -D__CUDA_NO_HALF_OPERATORS -D__CUDA_NO_HALF_CONVERSIONS -D__CUDA_NO_HALF2_OPERATORS__ --compiler-options '-fPIC' -DCUDA_HAS_FP16=1 -D__CUDA_NO_HALF_OPERATORS__ -D__CUDA_NO_HALF_CONVERSIONS__ -D__CUDA_NO_HALF2_OPERATORS__ -DTORCH_API_INCLUDE_EXTENSION_H -DTORCH_EXTENSION_NAME=_C -D_GLIBCXX_USE_CXX11_ABI=0 -std=c++11
unable to execute '/usr/local/cuda:/bin/nvcc': No such file or directory
error: command '/usr/local/cuda:/bin/nvcc' failed with exit status 1

I think I see the issue:
your CUDA_HOME environment variable has a trailing : in the end /usr/local/cuda: instead of /usr/local/cuda.
Can you try modifying it to match what I mentioned?

I think I see the issue:
your CUDA_HOME environment variable has a trailing : in the end /usr/local/cuda: instead of /usr/local/cuda.
Can you try modifying it to match what I mentioned?
thanks for your help sincere.
i have changed
export LD_LIBRARY_PATH=/usr/local/cuda/lib64:$LD_LIBRARY_PATH
export CUDA_HOME=/usr/local/cuda:$CUDA_HOME
to
export LD_LIBRARY_PATH=/usr/local/cuda/lib64
export CUDA_HOME=/usr/local/cuda. build completed.

Great!
I'm closing the issue, but let us know if you face other problems.


Add golangci-lint and fix the issues

Signed-off-by: aisuko<EMAIL_ADDRESS>Signed off: yes
fixes: #82

woooo! this is great thank you!
I don't understand why CircleCI isn't running this. I think I messed up the configuration so that it doesn't run for PRs. Hold on...

ok, i think i know what the issue is -- I'm going to merge this and fix it up afterwards. Thanks again @Aisuko !


Could not load the Qt platform plugin "xcb"

使用train_Rotnet.py训练完epoch15后返回的信息，这是报错吗？要安装qt platform然后重新训练吗？

环境：
WSL2 Ubuntu22.04
Python 3.10.9
cuda 11.8
tensorflow 2.12.0
pytorch 2.01

qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.
Available platform plugins are: eglfs, minimal, minimalegl, offscreen, vnc, webgl, xcb.
Aborted

追问：有什么办法能继续训练模型吗？这个刚出来的模型不完整报错。

        Unexpected key(s) in state_dict: "model", "lrm".

使用train_Rotnet.py训练完epoch15后返回的信息，这是报错吗？要安装qt platform然后重新训练吗？

环境：
WSL2 Ubuntu22.04
Python 3.10.9
cuda 11.8
tensorflow 2.12.0
pytorch 2.01

qt.qpa.plugin: Could not load the Qt platform plugin "xcb" in "" even though it was found.
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.
Available platform plugins are: eglfs, minimal, minimalegl, offscreen, vnc, webgl, xcb.
Aborted

这个问题已解决，缺少依赖

怎么解决的呀？

Maybe help
PyQt5：解决qt.qpa.plugin: Could not load the Qt platform plugin "xcb"错误
https://github.com/NVlabs/instant-ngp/discussions/300


(Not an issue) Will it be on sale?

@whid-injector I really like this project. This BadUSB is very compact and discrete. I would like to buy one. Will it be on sale soon?

Yes it will definitely be for sale worldwide!
Right now we are ending the beta test phase and in 2/4 weeks should start the production. :smile:


[Bug]: Split SEQUENTIAL flag lost when project files reloaded

What happened?
When I open a file (via menu open project) where a node is set with split and sequential, the sequential option is lost.
this happens in the subgraph node and the Ollama node (those are where I have tested it)
What was the expected functionality?
Expected function is that the sequential flag you be maintained across save / load
Describe your environment
MacOS 14.2.1, Rivet 1.7.4
Relevant log output
No response
Relevant screenshots
No response
Code of Conduct

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

Fixed in d8c09633c56224f5cdb3aacf88850e21404c2318


Error after adding neovim-flake to my inputs.

⚠ Please verify that this bug has NOT been reported before.

[X] I checked all existing issues and didn't find a similar issue

Description
Adding neovim-flake into my configuration caused this error.
error:
       … while evaluating a branch condition

         at /nix/store/233086pcwz4qx09l1xgdlzn2yh1p04zy-source/flake.nix:38:15:

           37|             (name: value:
           38|               if isAttrs value then lhs.${name} or { } // value
             |               ^
           39|               else if isList value then lhs.${name} or [ ] ++ value

       … while calling the 'isAttrs' builtin

         at /nix/store/233086pcwz4qx09l1xgdlzn2yh1p04zy-source/flake.nix:38:18:

           37|             (name: value:
           38|               if isAttrs value then lhs.${name} or { } // value
             |                  ^
           39|               else if isList value then lhs.${name} or [ ] ++ value

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: getting status of '/nix/store/mdi3hh7gwk41hgkbhliyhc1ira77b2gi-source/default.nix': No such file or directory

NOTE I have only added the Neovim-flake URL and the nixpkgs.follows attribute to my inputs, and I haven't done anything more than that.
👟 Reproduction steps

Add neovim-flake url and the nixpkgs.follows attribute in the inputs.
run sudo nixos-rebuild switch --flake .#

👀 Expected behavior
It should build my configuration because the only thing I did was add the neovim-flake to my inputs.
😓 Actual Behavior
It throws an error.
💻 Metadata

system: "x86_64-linux"  - host os: Linux 6.2.11-xanmod1, NixOS, 23.05 (Stoat), 23.05.20230418.555daa9  - multi-user?: yes  - sandbox: yes  - version: nix-env (Nix) 2.15.0  - nixpkgs: /nix/store/qj565ha0ix6l30118j5i3j6vmhnb1n3y-source

📝 Relevant log output
Check the description.

You'll find that I also use this flake in my system flake - which seems to work fine.
How did you add it to your flake, is your flake public?

I've just rebuilt successfully after updating the flake input, the flake seems to be working fine.

Update, I am able to reproduce the error on your flake and only yours. Perhaps it is a problem with how digga approaches flake inputs?

Hmm, that's really weird. I guess it's time to ditch digga :joy:


Enhancement: allow configuring ACS app-only auth in SPO

Extend the spo tenant set command with the DisableCustomAppAuthentication true|false option to allow admins to configure if ACS-based app-only auth should be enabled or not.
For the implementation see the Set-SPOTenant cmdlet

Hey @waldekmastykarz  - Do you mind assigning this to me if you guys are fine with that.? I can take this up.

Woohoo! Appreciate your help @arjunumenon! 👏

Hello @waldekmastykarz - Just a quick confirmation before I start implementing this command.
Currently the command set for spo tenant follows  spo tenant settings set. Do you think we need to update the specs so that the command follows the consistency.
Pardon me If I am being particular about that, though it is self-explanatory, just wanted to confirm on that. 🤔

Good catch @arjunumenon and my bad. I'll update the spec to point to spo tenant settings set which is the right command to extend.


How to identify whether the left hand or the right hand is operating

Describe the problem
In MRTK 2.6, is it possible to identify whether it is a left-hand click operation or a right-hand click operation?
I want to grab objects with both hands in HoloLens 2 at the same time, and obtain the data of the objects separately.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

I want to grab objects with both hands in HoloLens 2 at the same time

Are you using an MRTK component for this or looking for how to get this info from MRTK events?


Fix#10219

#10219

The new implementation is causing a dtype mismatch in the CompileAndCheck test when X64=True, which I don't entirely understand. It may be indicative of some larger type promotion bug.

The new implementation is causing a dtype mismatch in the CompileAndCheck test when X64=True, which I don't entirely understand. It may be indicative of some larger type promotion bug.

Would you investigate this type promotion problem?

Ah, I see, the issue is that you use the raw input x in the expression x / 2, so when x is a numpy array it follows numpy-style promotion and promotes to float64. You can fix this by calling x = jnp.asarray(x) at the start of the function, or alternatively by calling lax functions directly in place of Python arithmetic operators.

Ah, it is really subtle! Calling lax functions can be not suitable since lax won't do type promotion, and need a manually promoted literal 2 (or 0.5 if use mul).  In addition,lax.div(1, 2) == 0 since 1 and 2 are int32.
I have fixed it with
def logpdf(x):
  # convert to JAX before using python arithmetic op
  # ensure correct type promotion behavior
  # see https://github.com/google/jax/pull/10220#issuecomment-1095444492
  half_x = jnp.multiply(x, 0.5)
  return -2 * jnp.logaddexp(half_x, -half_x)

Calling lax functions can be not suitable

Right, you'll see elsewhere that we use lax._const for this reason: https://github.com/google/jax/blob/9455254b9fd456fe0ca00851f5fd4ebfdf04aa5b/jax/_src/scipy/stats/poisson.py#L26-L31
I think that would be preferred here over introducing a jnp dependence.

Right, but
from jax._src.lax.lax import _const
assert _const(1, 0.5) == 0

Do we need a _promote_args_inexact here?

Done.
def logpdf(x):
  x = _promote_args_inexact("logistic.logpdf", x)
  two = _lax_const(x, 2)
  half_x = lax.div(x, two)
  return lax.mul(lax.neg(two), jnp.logaddexp(half_x, lax.neg(half_x)))

I make an empty force-push to re-run lint and typecheck which seems to fail with juptext internal bug
Traceback (most recent call last):
  File "/home/runner/.cache/pre-commit/repobvvyoahl/py_env-python3.8/bin/jupytext", line 8, in <module>
    sys.exit(jupytext())
  File "/home/runner/.cache/pre-commit/repobvvyoahl/py_env-python3.8/lib/python3.8/site-packages/jupytext/cli.py", line [489](https://github.com/google/jax/runs/5996732829?check_suite_focus=true#step:5:489), in jupytext
    exit_code += jupytext_single_file(nb_file, args, log)
  File "/home/runner/.cache/pre-commit/repobvvyoahl/py_env-python3.8/lib/python3.8/site-packages/jupytext/cli.py", line 557, in jupytext_single_file
    notebook = read(nb_file, fmt=fmt, config=config)
  File "/home/runner/.cache/pre-commit/repobvvyoahl/py_env-python3.8/lib/python3.8/site-packages/jupytext/jupytext.py", line 409, in read
    return read(stream, as_version=as_version, fmt=fmt, config=config, **kwargs)
  File "/home/runner/.cache/pre-commit/repobvvyoahl/py_env-python3.8/lib/python3.8/site-packages/jupytext/jupytext.py", line 418, in read
    return reads(fp.read(), fmt, config=config, **kwargs)
  File "/home/runner/.cache/pre-commit/repobvvyoahl/py_env-python3.8/lib/python3.8/site-packages/jupytext/jupytext.py", line 370, in reads
    notebook = reader.reads(text, **kwargs)
  File "/home/runner/.cache/pre-commit/repobvvyoahl/py_env-python3.8/lib/python3.8/site-packages/jupytext/jupytext.py", line 178, in reads
    return new_notebook(cells=cells, metadata=metadata)
  File "/home/runner/.cache/pre-commit/repobvvyoahl/py_env-python3.8/lib/python3.8/site-packages/nbformat/v4/nbbase.py", line 164, in new_notebook
    validate(nb)
  File "/home/runner/.cache/pre-commit/repobvvyoahl/py_env-python3.8/lib/python3.8/site-packages/nbformat/v4/nbbase.py", line 39, in validate
    return validate(node, ref=ref, version=nbformat)
  File "/home/runner/.cache/pre-commit/repobvvyoahl/py_env-python3.8/lib/python3.8/site-packages/nbformat/validator.py", line 281, in validate
    raise ValidationError("Non-unique cell id '{}' detected. Corrected to '{}'.".format(cell_id, cell['id']))
jsonschema.exceptions.ValidationError: Non-unique cell id '1022fbd7' detected. Corrected to 'b1d2593d'.

Error: The process '/opt/hostedtoolcache/Python/3.8.12/x64/bin/pre-commit' failed with exit code 1


Feature/cn-language

Description
Add traditional Chinese support, fixing issue #351

hi @ai-entrepreneur I don't have your contact information but wanted to reach out to you because you did our previous zh-Hant translation. There are some updates that are not there in the translation yet, would you like to do them? Let me know if you're interested (we're doing it using another service now, not directly as a PR here on github).


Fish Bot

Fish Bot Stopped Random in a Non Shiny Encounter.
Can you Fix that please.
https://imgur.com/a/tKRc0

When do you start the bot

the first text after I used the Rod
Whit i Start it whit Cursor on Use in Show Only Nothing Bite.
Other Bot´s works all Perfectly

Should hover over USE. Which route?

Ecruteak City


Institutionalize the fact that FD Github organization is software hub available for independent developers to have project on without loosing control/ownership?

I think we already have some projects that were created by independent developers and moved to the FD org. De facto we treat this project as independent (the core team doesn't have the power to make decisions there) but de jure this agreement doesn't exist yet anywhere neither as some kind of license nor even as a document/disclaimer.
Currently, it's very actual because of the dataflows situation (as far as I understand the situation).
@lwinfree @akariv @rufuspollock WDYT? Does it make sense to think in this direction? If yes, what can we do (licenses/public agreement/etc)?

@roll can you clarify a bit more. I think a given project under FD can have a lead developer / maintainer who largely owns that code whilst being under FD. At the same time the idea would be that that project is consistent with the ethos and approach (and branding) of FD etc.
At same time I don't FD per se is a software hub for "independent developers" to come and just do stuff.

@rufuspollock
Probably that's a bad wording from my side and your statement is more correct:

I think a given project under FD can have a lead developer / maintainer who largely owns that code whilst being under FD. At the same time, the idea would be that that project is consistent with the ethos and approach (and branding) of FD etc.

One thing which I think is crucial - we need an option when this developer/maintainer owns that code without largely or other limitations. Meaning that he has the last word on anything related to this project. If FD and this developer run into insoluble disagreement the project can leave the organization.

At same time I don't FD per se is a software hub for "independent developers" to come and just do stuff.
Sure, what I meant is more like Apache Incubator - https://incubator.apache.org/

One thing which I think is crucial - we need an option when this developer/maintainer owns that code without largely or other limitations. Meaning that he has the last word on anything related to this project. If FD and this developer run into insoluble disagreement the project can leave the organization.

Yes, i think that generally sounds good - and would flow from your incubator point.

Sure, what I meant is more like Apache Incubator (for a subset of projects) - https://incubator.apache.org/

@roll love it 😄 👍 👍
The one point here for me is that (I think) FD should aim to a be more coherent than Apache - not only in how the components fit together but in their look and feel.

I think the software page on the site has achieved this goal.

@lwinfree
Also, we might need some kind of public agreement for people moving their projects to FD. I think it can be a part of the government work.


Bundle with zero-config microbundle

WIP

This change is

Closing as it does not solve the problem with codesandbox.


STORM-773: STORM-772: Transactional test timeout too low on restart, and InterruptedException

So after much searching and tracing through logs, with some added logs in the CoordinatedBolt I found out that the CoordinatedBolt was timing out the batch in a few cases, if the batch took longer then 300ms to complete because the timeout is set to 30 seconds by default and 10 seconds of simulated time equals 100ms of wall time.  When this would happen the bolts would be confused and the batch would never be fully acked.  I am not sure why the coordinator/spout was not getting a timeout and replaying the batch in simulated time, but because it is a simulated time issue, and only really shows up on this one test, I decided to increase the timeout.  If others think we should dig deeper and understand why the replay is not happening I am happy to hand the JIRA over to them.

+1

+1


feat: add support-bundle-node-collection-timeout config

Summary
PR Checklist

Is this a multi-tenancy feature/bug?

[ ] Yes, the relevant RBAC changes are at:

Do we need to backport changes to the old Rancher UI, such as RKE1?

[ ] Yes, the relevant PR is at:

Are backend engineers aware of UI changes?

[ ] Yes, the backend owner is:

Fixes https://github.com/harvester/harvester/issues/1646

Occurred changes and/or fixed issues

Add new suppport-bundle-node-collection-timeout setting.
Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

This one will also be backported to v1.3.

Does support-bundle-node-collection-timeout setting allow negative integer or given string ?
Since support-bundle-timeout has webhook validation for negative integer or given string

@a110605 you need to update harvester-webhook image as well.

@a110605 Thanks for testing that part. Already created a PR for that https://github.com/harvester/harvester/pull/5969, and tested it. You could use jk82421/harvester-webhook:v<IP_ADDRESS> to test it first.

@a110605 Thanks for testing that part. Already created a PR for that harvester/harvester#5969, and tested it. You could use jk82421/harvester-webhook:v<IP_ADDRESS> to test it first.

Pull your image looks good on my local.

@mergify backport release-harvester-v1.3


[nodejs] Add cycles 13, 8

From #525
Readd from https://github.com/endoflife-date/endoflife.date/commit/f3cf737e2a92d9914eaf5693530f09fef7bf3082#diff-252b7c5f491ceacafa0a6744a80913e26befedc4461b3d019af0eab91ddd2db0

This is ready for review. The other older version can be handled by a separate PR. I will create one if I have the time.


401 Client Error: Unauthorized for url

Environment

Qiskit version: 1.0
Python version: 3.11
Operating system: linux, mac, ...

What is happening?
All login attempts get rejected with:
qiskit_ibm_runtime.exceptions.IBMNotAuthorizedError: '401 Client Error: Unauthorized for url: https://api.quantum.ibm.com/runtime/jobs. {"errors":[{"code":1201,"message":"Invalid credentials.","solution":"Verify your credentials and try again.","more_info":"https://cloud.ibm.com/apidocs/quantum-computing#error-handling"}],"trace":"..."}'

The job shows up shortly on the dashboard, but that crashes as well after that.
How can we reproduce the issue?
from qiskit import QuantumCircuit
from qiskit_ibm_runtime import QiskitRuntimeService, Session, Sampler
import os
num_qubits = 32
qc = QuantumCircuit(num_qubits)
qc.h(0)
qc.cx(0, 1)
qc.measure_all()
_service = QiskitRuntimeService(
    channel="ibm_quantum",
    instance="ibm-q/open/main",
    token="your token",
)
_session = Session(service=_service, backend="ibmq_qasm_simulator")
sampler = Sampler(session=_session)
job = sampler.run(circuits=qc)
job.result()

What should happen?
Login should work?
Any suggestions?
No response

Please can you open these issues with qiskit-ibm-runtime against https://github.com/Qiskit/qiskit-ibm-runtime?  This repo is for the qiskit Python package, and qiskit-ibm-runtime is separate.

Done, can now be found at: https://github.com/Qiskit/qiskit-ibm-runtime/issues/1491


raise ValueError('app_id is required') ValueError: app_id is required

Hallo
I try tu use you code example
from symbl.utils.Threads import Thread
import symbl
import time

# here are all events supported by Telephony API, you just need to uncomment the event which you would like to use
events = {
    'transcript_response': lambda transcript: print('printing the transcript response ', str(transcript))
    ,'message_response': lambda message: print('printing the message response ', str(message))
    #,'insight_response': lambda insight: print('printing the insight response ', str(insight))
    #,'topic_response': lambda topic: print('printing the topic response ', str(topic))
}

sip_uri =<EMAIL_ADDRESS>connection_object = symbl.Telephony.start_sip(uri=sip_uri)

connection_object.subscribe(events)

normally i make this call with linphone with standard Setting and I coukd calling the client

when i use your library in python to do same thing the error coming :
raise ValueError('app_id is required') ValueError: app_id is required
i make a symbl.conf file and add the information
but i don#t sure what is the standard app_id and app_secret wich linephone used
I try : app_id ='admin' app_secret='administrator' come othe problem
HTTP response headers: HTTPHeaderDict({'Server': 'openresty/<IP_ADDRESS>', 'Date': 'Thu, 11 May 2023 07:07:38 GMT', 'Content-Type': 'application/json; charset=utf-8', 'Content-Length': '101', 'Connection': 'keep-alive', 'X-Powered-By': 'Express', 'Access-Control-Allow-Origin': '*', 'Access-Control-Allow-Headers': 'Origin, X-Requested-With, Content-Type, Accept', 'ETag': 'W/"65-Or/vqA393O1grUI2+x/wDmq6zkI"'})
HTTP response body: {"message":"Incorrect 'appId' and 'appSecret' provided. Please provide valid details and try again."}

The app_id and app_secret are the login creds for the Symbl Platform. You need to sign up on the platform, https://platform.symbl.ai/, and as soon as you login, you should see the app_id and app_secret on the main page.
You can find more info in the README under configuration:
https://github.com/symblai/symbl-python-sdk#configuration


increase readability

Before: http://imageshack.us/photo/my-images/443/beforep.png/
After: http://imageshack.us/photo/my-images/337/aftero.png/
I find the changed version much more readable. Any objections?

Less loaded and more readable indeed. +1

Replacing the vertical bars with small bullets is a good typographical solution. There is one catch however: the colons for terrain and difficulty ratings now visually blend with the bullets, thus causing more strain on the reader trying to separate the fields. I would like to suggest to remove the colons completely, as "T 1.5" also is clear and clearly readable without blending into the field separating bullets.

+1

I assume the dot instead of pipe is accepted.
Just for your viewing pleasure I added 3 additional screen shots detailed geocrashers comment:
colons replaced by blanks: http://imageshack.us/photo/my-images/408/colonsremoved.png/
colons/blanks removed completely: http://imageshack.us/photo/my-images/259/spaceremoved.png/
colons/blanks removed and fractions shortened if possible: http://imageshack.us/photo/my-images/502/shortenedfloat.png/
Next round of votes please. :)

colonsremoved

colonsremoved

I would suggest to add a dot between the D and T field (forgot to mention that before and now it becomes clear to me as I watch the mockups). To me, these are two fields and should be separated in the same way the other fields are. Other than that I like all variants, with variant #1 in slight preference.

+1 for a dot between the D and T field. then I'd prefer colons replaced by blanks

You are right. Personally I found the version with blanks a bit difficult before (because the T letter was blending with the D number), but with an additional bullet it becomes very easy to read. I've committed the version without colons, but added bullet now, just so you can evaluate the readability by yourself.
If anyone objects or sees further improvements, just reopen.

But this is the common format for coordinates. We should follow conventions.

@Bananeweizen: I agree to that, so having the bullet inside the app would be good for readability.

@Bananeweizen: I agree to that, so having the bullet inside the app would be good for readability.

I have to work on the coordinate-formatting nevertheless and it's on my internal todo-list. ;)


Test, document and publish Spark clients support matrix

Dimensions (at least):

lakeFSFS / metadata
Spark
Hadoop
Scala (metadata client only)

duplicate of https://github.com/treeverse/lakeFS/issues/2800


co and yield* when a promise is rejected ?

I just faced an issue mixing generators and yield*, co, and node.
It seemed to be the right place to start, but I'm not really sure this is a problem with co.
Let me explain:
I usually use this code to make the node task fail whenever an error occurs in a generators chain handled by co, because of Promise specification, where errors are muted if not handled:
process.on('unhandledRejection', (reason, p) => {
  throw reason;
});

I also find yield* a cleaner way to yield the values returned by an array of promises, instead of Promise.all().
At first, I thought that it pretty did the same thing as Promise.all(), but when one of the promises is rejected, then I had the following error:
/somewhere/yield.js:14
  throw reason;
  ^
unhandled TypeError: Cannot read property 'done' of undefined
    at next (/somewhere/node_modules/co/index.js:98:14)
    at onRejected (/somewhere/node_modules/co/index.js:85:7)
    ....

Here is the full code:
https://gist.github.com/rehia/6b13bd8202a37fbb58c82cbdd9df9b83
I use Node 6.3.1 and co 4.6.0.
http://node.green/#generators tells that yield* is fully supported by Node...
If you have any thoughts on this, I'll be glad to know :)

hi all, any idea about this ?
would love to share about this !

You can only use yield* to delegate to other generators in co. The reason is that the throw call (made by co because of the rejected promise) is propagated to the array iterator and this iterator does not implement a throw method.
See https://bugs.chromium.org/p/v8/issues/detail?id=3484
The solution to your problem is just to use yield array

the same to me , how to reject a error with co?  since only yield is used ,is  there anyway to do this


--quiet doesn't print url for --scan

Expected Behavior
--scan url should print when it's requested
./gradlew --scan --quiet (ok, actually I have it set in properties)
org.gradle.logging.level=quiet

Welcome to Gradle 8.6.

To run a build, run gradlew <task> ...

To see a list of available tasks, run gradlew tasks

To see more detail about a task, run gradlew help --task <task>

To see a list of command-line options, run gradlew --help

For more detail on using Gradle, see https://docs.gradle.org/8.6/userguide/command_line_interface.html

For troubleshooting, visit https://help.gradle.org

./gradlew --scan --warn

Welcome to Gradle 8.6.

To run a build, run gradlew <task> ...

To see a list of available tasks, run gradlew tasks

To see more detail about a task, run gradlew help --task <task>

To see a list of command-line options, run gradlew --help

For more detail on using Gradle, see https://docs.gradle.org/8.6/userguide/command_line_interface.html

For troubleshooting, visit https://help.gradle.org

Publishing build scan...
https://gradle.com/s/aa3mbcfxphb7c

Configuration cache entry reused.

Current Behavior (optional)
No response
Context
------------------------------------------------------------
Gradle 8.6
------------------------------------------------------------

Build time:   2024-02-02 16:47:16 UTC
Revision:     d55c486870a0dc6f6278f53d21381396d0741c6e

Kotlin:       1.9.20
Groovy:       3.0.17
Ant:          Apache Ant(TM) version 1.10.13 compiled on January 4 2023
JVM:          21.0.2 (Eclipse Adoptium 21.0.2+13-LTS)
OS:           Linux 6.6.10-1-MANJARO amd64

The issue is in the backlog of the relevant team and is prioritized by them.

Not printing the "publishing build scan" output with --quiet is by design: https://github.com/gradle/gradle/issues/5043
If you want a build scan URL even with quiet mode you need to register a callback as described here.

Giving a quick glance at that. I think there's a design problem. I would suggest that gradle needs to adopt the concept of git's porcelain in which they have options for machine parse-able output which has a stable api, and a less stable and prettier output meant for human consumption. Although please don't adopt --porcelain as the option for the machine parse-able output, that was a terrible decision.
I don't think log levels should be used to dictate machine parse-able output. There are actually standards for parse-able logs too. Although I get the impression they're doing more than that.
That's my 2 cents, add an option for machine parseable and an option for not, keep the current default, and eventually switch it.

What is it you want to parse from the output? The Build Scan URL? This callback is exactly meant for that.

I'm not parsing anything, I just want to be able to click the link without a lot of extra noise from gradle. gradle itself is more noisy when running on the cli than I'd like, it obscures the actual important messaging. That's why I suggested a machine parse-able output for people that are parsing. As a user writing a callback for this is extreme. Arguably it should be the people who want machine parse-able output writing callbacks (if anyone).

Basically but I feel like that's exactly what the -- scan parameter should do. How do you even get the scan URL if it's not printed out? Is it actually persisted to the disk somewhere? I've never seen a way to list all my build scans. I don't think they're even associated with me in a way that that would work.


Require assistance, Code doesnt seem to work

I am trying to get this working for a project i have for a restricted network. User creates an excel with 1500 items on it, i need that to become a HTML table so i can transfer it to a SharePoint list. The issue is i cannot get your code to work on Fiddle to even begin to move it the isolated network. Not to mention i cannot bring the xlsx.min.js file over due to all the special characters.
JSfidd: https://jsfiddle.net/xpvt214o/819660/

You can check the developer tools to see the errors.
In this case, it's because web workers were enabled but the worker scripts could not be found.  Uncheck the "Use Web Workers" box.
Additionally, if you are doing this in the web browser, there are bundled files in the dist directory.  Use https://unpkg.com/xlsx/dist/xlsx.full.min.js

I think i fixed it by removing huge chunks of the code.
"You can check the developer tools to see the errors." i linked the JSfiddle with the assumption you would
"Ke" is not a constructor


'Option' is not exported from '@material-tailwind/react'.

When trying to export Option it gives the error saying 'Option' is not exported from '@material-tailwind/react'.
If you think this Issue should be worked upon then I would like to  create a PR.

This is the same error that prevent me to use the latest version of this lib

Hey guys,
I've tested the latest version of the library and this issue is not true for it, please update to v1.2.4 to get the latest updates.
Cheers,
Sajad


Design gas selection

From #1721
AC:

[ ] have a design for gas assignment to transactions

https://github.com/cosmos/voyager/issues/1721#issuecomment-466364606
https://zpl.io/blPdgg1


Switch attribute access syntax

As we don't match any operation name for attribute access it is checked last. Makes no sense as it is probably most frequently used.

Now use {* get attribute *}. To keep consistency in template operations


Why isn't the StructureComparer class static?

It might be a good idea to make it static. Looking at the tests and the codebase, the class is always initialized with empty args.
That way, it becomes a bit easier to use it(just a one-liner).
Cheers :P

I think it makes sense. This library should be used in unit testing projects. Thus, there will not be IoC's problems or other concerns due to the use of a static class.
I'll make that change and see how it looks.
Thanks.

Done https://github.com/jeduardocosta/StructureComparer/commit/8086453eabbc074b05b23613aa7b5ec93bfdf5e0


Can't exit html5 fullscreen by keyboard

In most browsers, users can exit html5 fullscreen by pressing 'ESC' key, however Electron can't. I suggest Electron to also do this by default.

This behavior is aligned with Chrome's. In Chrome browser, user presses F11 to enter the full screen and represses F11 to exit(The Esc key makes no sense).

No, I'm talking about html5 fullscreen, not F11-fullscreen. Chrome also exit html5 fullscreen via Esc.

Oops...I misunderstand it.

Is possible to add a option to restore old behavior?


fix(CIP-145): updates from forum discussion

Updates based on this forum discussion.
cc @m0ar

In general, these are the most important features from my perspective:

All known commits, pruned or not, are pinned by the node
A new node can learn about all known commits, pruned or not, from other nodes

The reason is that a historical shuffle should not prevent resolution of a once-valid commit, which we should be able to implement in the client using these two properties. Otherwise, there is an avenue for abusing late publish as an undo button. We would like to be able to rely heavily on deterministic resolution of state, and this is OK as long as the commits are preserved and communicated regardless of a consensus change.
If I understand this CIP correctly, I think the late publishing attack would be equated to just adding a new commit on the tip, which is anyway possible to do for the controller. What I'm not sure about though is what the merge of a divergent branch means for the state of the stream.

In general, these are the most important features from my perspective:

All known commits, pruned or not, are pinned by the node
A new node can learn about all known commits, pruned or not, from other nodes

The reason is that a historical shuffle should not prevent resolution of a once-valid commit, which we should be able to implement in the client using these two properties. Otherwise, there is an avenue for abusing late publish as an undo button. We would like to be able to rely heavily on deterministic resolution of state, and this is OK as long as the commits are preserved and communicated regardless of a consensus change.
If I understand this CIP correctly, I think the scope of a late publishing attack would be equated to just adding a new commit on the tip, which is anyway possible to do for the controller. What I'm not sure about though is what the merge of a divergent branch means for the state of the stream, but I'm not sure if this is relevant in this context.

The two big advantages of this CIP are:

The tip event is a discovery mechanism for and commitment to not only the prev-chain but the prev-DAG.
You can at any point in time account for which branches are pruned and which have been merged.


applying vue.js work (Wuseok/absoulte-ui) to absolute

if @Wuseok will do the same work, I will make a plan to do co-work.
merging https://github.com/Wuseok/absoulte-ui
find out why merging absolte-ui not easy and list-up

I'm closing this issue..


Solid start!

Hi Jason,
Just wanted to say 'thank you' for your effort in setting up a reliable queue library. The NodeJS ecosystem is surprisingly limited when it comes to solid low-level libraries (e.g. networking, database drivers, etc), and yours will be a most welcome addition. Thanks!

Hey @eugene1g, thanks for the appreciation! Hopefully, this library can be useful for you and others.

Can you add to readme brief about project history, why it was born, what problems it does solve better than alternatives, what were the inspirations when developing this library.
Thanks ahead.

Hey @glensc, great idea :) I'll add those topics to the README. Thanks!

@jasrusable please post to this thread when updating something (i've subscribed to the issue)

@jasrusable Thanks for creating this library. It looks very promising.
Your advertisings in bee-queue repo caught my attention. I like that it's written in typescript, very well documented and that you have clear goals of making things better than other not-really-production-ready packages.
I am also curious about your opinion on redis v5.0+ streams. (re: https://github.com/bee-queue/bee-queue/pull/112#issuecomment-634226074) I've found https://github.com/mugli/orkid-node where they went for redis streams, but the development seems to be stalled there.

@jasrusable Thanks for creating this library. It looks very promising.
Your advertisings in bee-queue repo caught my attention. I like that it's written in typescript, very well documented and that you have clear goals of making things better than other not-really-production-ready packages.
I am also curious about your opinion on redis v5.0+ streams. (re: bee-queue/bee-queue#112 (comment)) I've found https://github.com/mugli/orkid-node where they went for redis streams, but the development seems to be stalled there.

Thanks for your comment :)
Regarding my thoughts on steams:
In general, I think Redis streams are great and that they are certainly a very useful primitive to have on Redis.
In terms of the benefit of steams to Conveyor, two main things:

I think streams would be able to reduce some of the complexity around the way in which tasks are acquired by workers but performance-wise and reliability-wise I don't see much benefit. I do think that the reduced complexity may justify changing the current list + brpoplpush implementation to streams at some point.

The distributed communication between workers, managers and listeners is currently implemented using Redis pub/sub and so is not currently reliable (in the distributed system sense). Switching this over to streams would be very beneficial in that the communication could then be fully reliable.

The only downsides to streams I see in terms of Conveyor is that of having to force users to use Redis >= 5, which will obviously become less of a concern over time as people migrate to newer Redis versions but should still be considered for now. In any case, I will most probably be migrating at least the communication systems over to streams in the medium term to take advantage of the reliability it offers.


Add optional animation completion closure

useful to do some stuff at the end of the animation, like remove the view for example

Yes, I will try to do this. I'll let you in touch.
Le mar. 23 mars 2021 à 18:50, Mattt @.***> a écrit :

@.**** commented on this pull request.
In Sources/ConfettiView/ConfettiView.swift
https://github.com/NSHipster/ConfettiView/pull/13#discussion_r599802075:

@@ -30,7 +33,10 @@ public final class ConfettiView: UIView {
3.0 seconds by default.
*/
public func emit(with contents: [Content],

                for duration: TimeInterval = 3.0) {

                for duration: TimeInterval = 3.0,

                completion: ((ConfettiView) -> Void)? = nil) {

   self.completion = completion

@chepiok https://github.com/chepiok A possible solution to this just
randomly occurred to me. What if we subclass CAKeyframeAnimation and add
completion as a property? Then in animationDidStop, we could do something
like this:
public func animationDidStop(_ animation: CAAnimation, finished flag: Bool) {
if let layer = animation.value(forKey: kAnimationLayerKey) as? CALayer {
layer.removeAllAnimations()
layer.removeFromSuperlayer()
self.completion?(self)
}
if let animation = animation as? Animation /* i.e. ConfettiView.Animation */ {
    animation.completion?()
}

}
Would you be interested in / available to rewriting your PR following that
approach?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NSHipster/ConfettiView/pull/13#discussion_r599802075,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARQE7QXMBN4FDXX4EJAWM3TFDIFDANCNFSM4ZBEAPLA
.

I didn’t succeeded in making it work. Try attached.

Le 23 mars 2021 à 18:50, Mattt @.***> a écrit :
@mattt commented on this pull request.
In Sources/ConfettiView/ConfettiView.swift https://github.com/NSHipster/ConfettiView/pull/13#discussion_r599802075:

@@ -30,7 +33,10 @@ public final class ConfettiView: UIView {
3.0 seconds by default.
*/
public func emit(with contents: [Content],

                for duration: TimeInterval = 3.0) {

                for duration: TimeInterval = 3.0,

                completion: ((ConfettiView) -> Void)? = nil) {

   self.completion = completion

@chepiok https://github.com/chepiok A possible solution to this just randomly occurred to me. What if we subclass CAKeyframeAnimation and add completion as a property? Then in animationDidStop, we could do something like this:
public func animationDidStop(_ animation: CAAnimation, finished flag: Bool) {
if let layer = animation.value(forKey: kAnimationLayerKey) as? CALayer {
layer.removeAllAnimations()
layer.removeFromSuperlayer()
self.completion?(self)
}
if let animation = animation as? Animation /* i.e. ConfettiView.Animation */ {
    animation.completion?()
}

}
Would you be interested in / available to rewriting your PR following that approach?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/NSHipster/ConfettiView/pull/13#discussion_r599802075, or unsubscribe https://github.com/notifications/unsubscribe-auth/AARQE7QXMBN4FDXX4EJAWM3TFDIFDANCNFSM4ZBEAPLA.

@chepiok Thanks for giving this a try. I was able to get this to work by putting completion on Layer (the Animation approach I suggested didn't work as expected, so apologies for the bad advice there). Can you check out dadf0fb and confirm that it's working for you?

work fine.

Le 29 mars 2021 à 13:47, Mattt @.***> a écrit :
@chepiok https://github.com/chepiok Thanks for giving this a try. I was able to get this to work by putting completion on Layer (the Animation approach I suggested didn't work as expected, so apologies for the bad advice there). Can you check out dadf0fb https://github.com/NSHipster/ConfettiView/commit/dadf0fbf834e386b1d80f0ea4eec8fa958495d3e and confirm that it's working for you?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/NSHipster/ConfettiView/pull/13#issuecomment-809313737, or unsubscribe https://github.com/notifications/unsubscribe-auth/AARQE7UA6AIRDNFMSFFAGCDTGBSG5ANCNFSM4ZBEAPLA.

Thanks for suggesting this feature and for all of your help, @chepiok. This is now merged and will be included in the next release (0.3.0).


Added InfluxDB content

Added overview page and  getting started content
What changed, and why it matters
Part of the migration of help article - #120
Fixes #83 #92

I do not see the Concepts button in the landing page of Influxdb. I think something went wrong there.

You are right; not sure what happened here. What is the best way to fix this? Create a new issue?


Rule request: replacing unwanted comment substrings with valid ones

Suppose your company hates when people call it "XBox", and wants to replace that with "Xbox":
/**
 * Something something XBox something.
 */
export const value = 7;

Perhaps it would be useful to have a TSLint rule to flag barred comment values, and suggest replacing them?
/**
 * Something something XBox something.
                    // ^^^^ Replace with: "Xbox"
 */
export const value = 7;

Some other useful things:

Replacing " his " with " their " (for fixing common gender neutrality typos)
Replacing words commonly flagged by tools such as PoliCheck

Edit: prototyped this internally; using tsutils.forEachComment, it's relatively straightforward.

💀 It's time! 💀
TSLint is being deprecated; therefore, it and tslint-microsoft-contrib are no longer accepting pull requests for major new changes or features. See #4534. 😱
If you'd like to see this change implemented, you have two choices:

Recommended: Check if this is available in ESLint + typescript-eslint ✅
Not Recommended: Fork TSLint or tslint-microsoft-contrib locally 🤷♂

👋 It was a pleasure open sourcing with you!
If you believe this message was posted here in error, please comment so we can re-open the issue!


Overhaul Formula/Cask JSON generation

Take a slightly hackier but more reliable method of setting up $HOMEBREW_PREFIX, $HOME and --appdir.

Use constants for placeholders
Monkeypatch to set HOMEBREW_PREFIX consistently to placeholder
Use environment variable to set Dir.home consistently to placeholder
Use appdir short-circuit to set Cask#appdir consistently to placeholder
Use Cask.generating_hash! to enable "generating mode" with these patches
Fix Formula#caveats from JSON

See https://github.com/Homebrew/formulae.brew.sh/pull/744 for the formulae.brew.sh side.
Fixes #14505
Fixes #14595

Review period skipped due to critical label.

It would be nice to see some regression tests here.

Added.

And like @Bo98 I'm a bit confused about this only being enabled by default in Cask#to_h when we use Cask#to_hash_with_variations to generate the API json.

Removed from both, will put only in https://github.com/Homebrew/formulae.brew.sh/pull/744 instead.


Add instructions to install dev version

Adds instructions to install the dev version everywhere in vscode.

:+1:


versioning issue

@mefellows I see that you approved the pull request.  While I appreciate it, their seems to be a problem when actually installing the plugin using the vagrant plugin install command.  When installing the plugin, none of the changes that I made are displayed in the powershell file template.  From what I can tell, it looks like you have yet to upload the updated plugin to the ruby gems repository.  That being the case, what do you/I need to get the latest plugin pushed to the ruby gems repository?

or you can just give me the rights needed to upload the updated plugin, @mefellows .  The improvements i made are useless until the updated gem is pushed to the gems repository

Ah, sorry. I have a number of OSS projects and most of them have an automatic step during the build process to push to the relevant repository (in this case, rubygems).
I can add you as an owner if you can provide your email address to me (and have signed up to Rubygems). Email address is on my bio page.

I understand.  Everyone is busy.  Thank you.


Make equals version of code and code server

Repaginate nexts codes pt-bra partir de |4.14.1 to 1.79.2.
You to has access to server github than make change to all versions of code old.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

With different versions like this it's hard to know which vscode connects to the vscode server.

The vscode-dev-containers version number you see there is not the version number for the VS Code server but rather the version number for the Dev Containers extension. Since the Dev Containers extension ships totally independent of VS Code, it can follow its own versioning.


Add datasource configuration page

#28

Codecov Report
Base: 73.84% // Head: 73.93% // Increases project coverage by +0.08% :tada:

Coverage data is based on head (950ecc4) compared to base (7589f80).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #32      +/-   ##
============================================
+ Coverage     73.84%   73.93%   +0.08%
- Complexity     1430     1432       +2
============================================
  Files           154      154
  Lines          5888     5888
  Branches        670      670
============================================
+ Hits           4348     4353       +5
+ Misses         1260     1256       -4
+ Partials        280      279       -1

Flag
Coverage Δ

unittests
73.93% <ø> (+0.08%)
:arrow_up:

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

Impacted Files
Coverage Δ

...ain/java/com/hufudb/onedb/mpc/elgamal/Elgamal.java
89.61% <0.00%> (+6.49%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


when dyups and health check module together use, ngx_shmtx_lock block…

… too long time and cpu full load, cause health check timeout, 502

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


tt_lib.tensor.complex_conj operation fails with low PCC error [Wormhole]

Describe the bug
tt_lib.tensor.complex_conj operation breaks with low PCC error in some test cases.
To Reproduce
Steps to reproduce the behavior:

Run unit test test_complex_conj.py  using this command:
pytest tests/tt_eager/python_api_testing/non_working_unit_tests/grayskull/test_complex_conj.py

Expected behavior
There are eight test cases presented in the unit test tests/tt_eager/python_api_testing/non_working_unit_tests/grayskull/test_complex_conj.py and they all are expected to fail with low PCC error.
For example, one of the tests is expected to fail with this result:
Max ATOL Delta: 199.0, Max RTOL Delta: 36912.34765625, PCC: 0.1610435240628609, Equal check failed
Note
Unit test for complex_real for wormhole has the same code as unit test for GS, so to avoid code duplication we use the same unit to replicate errors on WH.
Getting Additional info for the operation under test and its behavior
To get additional information and results for different combinations of input shapes, types, layouts and memory configs for which this operation was tested you can also run locally sweeps for tt_lib.tensor.complex_conj and check the results. To do this you should:

Follow the Getting Started page to setup the repo, environment variables and python-env
Activate source build/python_env/bin/activate
Run sweeps by using python tests/tt_eager/python_api_testing/sweep_tests/run_pytorch_test.py -i tests/tt_eager/python_api_testing/sweep_tests/test_configs/ci_sweep_tests/broken_ wormhole/pytorch_complex_conj_test.yaml -o ./result-sweeps
After the run is completed all test sweeps results should be available inside specified output directory (in this case ./result-sweeps). There you will find complex_conj_sweep.csv which holds all executed sweeps, among which you can also find the ones that failed and were recreated by the unit test, which you can get by searching unique data_seed field.

Merged to main