PrimInt: to_be_bytes & co

The current to_be functions require unsafe to get a slice out of them; the individual types now (since 1.32) have to_be_bytes that does not have that flaw.
I suggest adding to_be_bytes() as well as to_ne_bytes and to_le_bytes as well as the analogous from_be_bytes, from_ne_bytes and from_le_bytes.
To implement this, PrimInt would need to get an associated type that reflects such a buffer; depending on when this gets implemented and what MSRV is targeted then, that could be a const NBYTES: usize (where to_be_bytes would return [u8; NBYTES]), or a BytesBuffer associated type with the right array lengths. In the latter form, the associated type would be required by the trait to implement at least TryFrom<&[u8]> and AsRef<&[u8]>.
Researching into whether that latter form would be upgradable to a const generic one easily later, I found that PrimInt is not sealed; that didn't sound too bad until I noticed that even a default implementation that uses to_be() and then transmutes around could still not come up with a suitable sized and owned return value short of returning a Box<[u8]> (that would technically satisfy the bounds but be ugly). Worse yet, it'd slam the door on being able to migrate towards returning [u8; NBYTES] later (which otherwise could be API compatible as the new return type does satisfy all bounds previously placed on it).
Is there any evolution mechanism in num-traits through which to_be_bytes() could be added?

I don't know any way to extend PrimInt for this.
There could be a new trait though -- #103 was working on this, but it would need to be rebased and reviewed again. Someone can also come up with a new design, if they prefer.

Is #103 still the current thinking around it ? If yes, i might want to pick it up and try to resolve pending issues. Related question, if OverflowingShl/Shr implementation like this would also be accepted ?

Is this crate simply abandoned? I need the floating point versions of these (to_bits, from_bits specifically, stabilized in 1.20; bytes stabilized in 1.40: https://github.com/rust-lang/rust/commit/7b72c2831c6a50a9e07a3d94ebef5d9f29bfd51e).
I'm going to just work off of @kaidokert's branch in #224 and make a fork because clearly there's something wrong with maintainership when such an important crate hasn't even allowed workflows to run or even acknowledged #224. I think @kaidokert is far nicer than me, to their credit…and therefore didn't raise this, but I will.
I don't hope to need a fork forever—@hauleth and/or @cuviper need to nominate a new maintainer.

I'm sorry, I will try to get to this, but I have a lot on my plate.
There's also nothing stopping this from being implemented as a separate extension trait, or indeed just using a fork.

No problem, glad to know it's not totally dead :-)
I think you should nominate someone but only you can decide that. I'll maintain a fork for a little while until you're less busy as I need some of the features…

Woo this finally landed !!! Thank you so much. 🙏


Rework K8s resource annotations for controlling notifications

Overview
Currently, there are two annotations which control notifications:

botkube.io/disable: true/false - whether to send notifications about a given resource

botkube.io/channel: {channel} - custom channel where the notifications are routed to. But the behavior is inconsistent:
This is supported by Slack and Mattermost only. In case of Mattermost, the annotation value should contain:

for Mattermost: channel ID,
for Slack: channel name.

We should straighten that up.
Acceptance Criteria

Rethink the annotation names and theirs values to allow extensibility

Take into account newer config
support multiple channels across different communications, e.g. botkube.io/communications.slack.channels: "general,random"
Take into consideration whether a target channel should be "authorized", that is, defined inside the static configuration (discussion: https://github.com/kubeshop/botkube/issues/250)
Instead of botkube.io/disable, maybe we should have different annotation to exclude a given resource notification from a given source? As we'll have multiple sources support in #676, it could make sense... 🤔

Support all communication platforms
Update documentation

Reason
Currently, this feature is inconsistent and limited to two communicators. We should aim to have feature parity across all integrations.

As we are going to extract the K8s source as a plugin, such functionality will be hard to keep. As discussed with @pkosiec and @huseyinbabal we are going to remove support Botkube specific K8s annotations:

botkube.io/disable: true/false
botkube.io/channel: {channel}

Instead, you will be able to still annotate your objects and use annotations property under k8s source to filter by.
This will be done as a part of #840


add django run server instructions

These are the instructions to run django server on your local computer.

These docs will probably be updated in the near future.


RPC validate whit source config does not really validate given data

Hi,
the RPC validate with source config does not really seem to validate the given data. I can enter any well formed XML and it is accepted without complains (RPC response is OK).
<validate xmlns="urn:ietf:params:xml:ns:netconf:base:1.0">
  <source>
    <config>
      <what-ever xmlns="urn:what:ever">
        <foo>bar</foo>
        <bar>foo</bar>
      </what-ever>
    </config>
  </source>
</validate>

Actually I have no idea where the bug is located (may be not in netopeer2).

Yes, now the data is validated.
Thanks!


thunderbolt: cleanup

Use placeholder to simplify, as comments request
categorize inputs appropriately

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: thunderbolt
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Attempted: thunderbolt
Partial log (click to expand)
shrinking /nix/store/wlyni0j42mwfsn3vrhyw0jl8pjp8qihn-thunderbolt-0.9.3/bin/tbtadm
shrinking /nix/store/wlyni0j42mwfsn3vrhyw0jl8pjp8qihn-thunderbolt-0.9.3/bin/tbtacl-write
gzipping man pages under /nix/store/wlyni0j42mwfsn3vrhyw0jl8pjp8qihn-thunderbolt-0.9.3/share/man/
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wlyni0j42mwfsn3vrhyw0jl8pjp8qihn-thunderbolt-0.9.3/bin
patching script interpreter paths in /nix/store/wlyni0j42mwfsn3vrhyw0jl8pjp8qihn-thunderbolt-0.9.3
/nix/store/wlyni0j42mwfsn3vrhyw0jl8pjp8qihn-thunderbolt-0.9.3/bin/tbtxdomain: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
/nix/store/wlyni0j42mwfsn3vrhyw0jl8pjp8qihn-thunderbolt-0.9.3/bin/tbtacl: interpreter directive changed from "/bin/sh" to "/nix/store/dsyc1z7ck08ga7l0b1jcxx35wj69qcii-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/wlyni0j42mwfsn3vrhyw0jl8pjp8qihn-thunderbolt-0.9.3...
/nix/store/wlyni0j42mwfsn3vrhyw0jl8pjp8qihn-thunderbolt-0.9.3

Success on x86_64-linux (full log)
Attempted: thunderbolt
Partial log (click to expand)
shrinking /nix/store/hjbpvfhn9cm5wybay8fykw3ma42w5jwp-thunderbolt-0.9.3/bin/tbtadm
shrinking /nix/store/hjbpvfhn9cm5wybay8fykw3ma42w5jwp-thunderbolt-0.9.3/bin/tbtacl-write
gzipping man pages under /nix/store/hjbpvfhn9cm5wybay8fykw3ma42w5jwp-thunderbolt-0.9.3/share/man/
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/hjbpvfhn9cm5wybay8fykw3ma42w5jwp-thunderbolt-0.9.3/bin
patching script interpreter paths in /nix/store/hjbpvfhn9cm5wybay8fykw3ma42w5jwp-thunderbolt-0.9.3
/nix/store/hjbpvfhn9cm5wybay8fykw3ma42w5jwp-thunderbolt-0.9.3/bin/tbtacl: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
/nix/store/hjbpvfhn9cm5wybay8fykw3ma42w5jwp-thunderbolt-0.9.3/bin/tbtxdomain: interpreter directive changed from "/bin/sh" to "/nix/store/r47p5pzx52m3n34vdgqpk5rvqgm0m24m-bash-4.4-p23/bin/sh"
checking for references to /build in /nix/store/hjbpvfhn9cm5wybay8fykw3ma42w5jwp-thunderbolt-0.9.3...
/nix/store/hjbpvfhn9cm5wybay8fykw3ma42w5jwp-thunderbolt-0.9.3


The icon is blurry

The icon look blurry on IntelliJ side bar

linux or mac ?

Looks like they increased it to 22x22 in the new UI, because the guidelines still say 13x13.


fix(TDI-38735): handle error while closing already closed job

What is the current behavior? (You can also link to an open issue here)
When attempting to close job that was closed on Salesforce side - exception was thrown.
https://jira.talendforge.org/browse/TDI-38735
What is the new behavior?
Since job is closed we need to continue processing without any interruption.
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: failure :x:

Pull request test result: failure :x:

Pull request test result: failure :x:

Pull request test result: failure :x:

Pull request test result: failure :x:

Pull request test result: success :star:

Pull request test result: success :star:


Review copy for forecast eval dashboard

Hi @nickreich, I've included screenshots of the "About" copy for the dashboard, and would love your feedback.
Specifically:

[ ] Should we use the term "Forecast Evaluation Working Group"?
[ ] What should go in the "Who We Are" section? (there is a TODO there asking more specific questions)
[ ] How the collaborators should be displayed / should there be a description of what each person provided (and if there is anyone I am missing)
[ ] "Our Mission" section: what to take out / what to add
[ ] Is there a good link for the source for all the forecaster data on the Forecast Hub? ("Sources" section)
[ ] Is the list of terms in the "Terms" section helpful? Accurate? Should anything else go here?
[ ] Anything else that should be included in the "Notes on the Data" section?
[ ] Explanation of scoring methods: accurate? complete enough?
[ ] WIS: Need at least another sentence or two on how this is actually calculated

Text is here: https://docs.google.com/document/d/1gPlQNnOtxFVTl58B6xHuvgeZdkSBF3a5ZIhLQohlWuI/edit?ts=60455031&resourcekey=0-SOtPlqmK0EjspLG4MYv0Fg#


[FLINK-8785][Job-Submission]Handle JobSubmissionExceptions

What is the purpose of the change
We will get an "Internal server error" exception if we submit a jobgraph with a restclusterclient. This PR helps us get more details and causes of the exception, such as "The jobgraph is empty" message.
Brief change log
Add causes and details of an exception which happens in job submission.
Verifying this change
Does this pull request potentially affect one of the following parts:
Documentation

@zentol  ok, I thought that it was a small change before, so you mean we can make more changes on the messages of the exceptions and let them reported more properly ?
I will test all the cases in the JobSubmissionFailsITCase.


ExecuteIndirect DispatchRays

The ID3D12GraphicsCommandList4 documentation states:
typedef struct D3D12_DISPATCH_RAYS_DESC {
  D3D12_GPU_VIRTUAL_ADDRESS_RANGE            RayGenerationShaderRecord;
  D3D12_GPU_VIRTUAL_ADDRESS_RANGE_AND_STRIDE MissShaderTable;
  D3D12_GPU_VIRTUAL_ADDRESS_RANGE_AND_STRIDE HitGroupTable;
  D3D12_GPU_VIRTUAL_ADDRESS_RANGE_AND_STRIDE CallableShaderTable;
  UINT                                       Width;
  UINT                                       Height;
  UINT                                       Depth;
} D3D12_DISPATCH_RAYS_DESC;

void DispatchRays(
  const D3D12_DISPATCH_RAYS_DESC *pDesc
);

This could pose a problem, as it requires you to know how many rays you're going to dispatch on the CPU. This means that you can't use a compute shader that calculates all pixels that require shading and dispatch rays for those (without a CPU stall). I'd request adding functionality for DispatchRays to ExecuteIndirect or adding DispatchRaysIndirect, which takes a GPU buffer and reads 3 uints from it.
Adding this would allow for a lot of optimization, meaning you would only have to dispatch rays that are required (without wasting other RT cores on a raygen shader that doesn't do anything for those threads).
I see that this is already brought up at the DXR roadmap, but I didn't see any issues or forum posts about this. Any idea if this might be implemented and at what time?

Hi,
Thanks for the feedback. This is on our roadmap, but unfortunately not something we could do for the v1 of the DXR API. We don't have a specific date when this will be make into the API to announce at this time. Outside the documentation, Raytracing page on DirectXTech is a good place to keep track of for API specification discussions or provide other API feedback.
Best


Missing Data class move constructor symbol problem for C++ language level C++11 and later problem

Added the destructor to the Data class to prevent the generation of the move constructor, when the C++11 and later compiler is used by the users. Adding the destructor avoid implicit move constructor generation (see https://en.cppreference.com/w/cpp/language/move_constructor).
fixes https://github.com/hazelcast/hazelcast-cpp-client/issues/563

 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.

Since we already started using C++11, this issue is no longer valid. Closing it.


A request

Thank you very much for your open source contribution, which is very helpful for my current work!
But I encountered some problems. In version 1.2, when inferring the 93x480p version with an A800 80G GPU, the memory of one A800 GPU is not enough to support the inference task, and two A800s are needed to infer at the same time. Is this a normal situation?
If this is a normal situation, could you please consider doing some memory optimization work in the future, which would be very helpful for secondary development.
Thank you again for your open source work!

It's not normal, our model can be inferenced in a single GPU.

Could you post more details? Such as scripts and outputs.


fix the typo of lable in statefulset

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.8 Features: set Milestone to 1.8 and Base Branch to release-1.8
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: Please check the “Allow edits from maintainers” box (see image below) to
allow reviewers to fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.

This change is

for approve @Bradamant3

ok,thanks,I have updated it,for approve @ @Bradamant3

PTAL @bradamant3

/assign @chenopis
thanks


Tell search engines to start indexing managed docs

For the initial release, we didn't want search engines indexing our managed docs. Now that we're up and running, it makes sense to have them indexed.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/89c644b5655dec6fb6bbddb986d791c6b952efbb/


Feature/partition heal

This PR implements a heal command on ringpop-admin making use of the public endpoint that is added in ringpop. To prevent merge conflicts it builds on #31 so this PR contain more changes. We shall land #31 before this one but feel free to start reviewing.
I added a tests as well, but since the tests are run against ringpop-node the call to heal returns an error in the test suite for now. When we have partition healing in ringpop-node we can bump the ringpop version it tests against and write a proper test for the heal command.

example output:
$ node ./heal.js <IP_ADDRESS>:3000
Executed heal to 1 targets
 - <IP_ADDRESS>:3003
Executed heal to 1 targets
 - <IP_ADDRESS>:3002
No known partitions left

Looks Good!

LGTM

changed the output slightly to include times:
$ node ./heal.js <IP_ADDRESS>:3000
15:34:45.515 Executed heal to 1 targets
15:34:45.516  - <IP_ADDRESS>:3002
15:34:46.521 Executed heal to 1 targets
15:34:46.521  - <IP_ADDRESS>:3002
15:34:47.523 No known partitions left


wrong initialization in SeparableConv2D

Please make sure that the boxes below are checked before you submit your issue. If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.
Thank you!

[ X ] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/keras-team/keras.git --upgrade --no-deps

[ X ] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[ X ] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ X ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

in SeparableConv2D if I set a constant value as initializer it gives me different values ( e.g.
weight values setting constant initializer to 100:
[array([[[[ 0.10519278]],[[-0.09711087]],[[-0.1125772 ]]],[[[-0.18705219]],[[-0.13889125]], [[ 0.21000636]]],[[[ 0.24378067]], [[ 0.46328533]], [[-0.15319693]]]], dtype=float32), array([[[[-0.15856814]]]], dtype=float32), array([0.], dtype=float32)]
-----------------------------CODE---------------------
a2= SeparableConv2D(1, (3, 3),input_shape=[3, 3, 1], activation='linear', padding='valid', kernel_initializer=keras.initializers.Constant(100), bias_initializer=keras.initializers.Constant(0))
init = tf.global_variables_initializer()
sess.run(init)
print(a2.get_weights())
NB this behavior doesn't seem present in other layers (e.g. Conv2D)

Looks like an inheritance edge case where kernel_initializer / bias_initializer args are being allowed in SeparableConv2D (even though the layer isn't supposed to support these arguments), doing nothing while not raising any error. This should be fixed (by raising the appropriate TypeError).

why not?


API Sign: Change the S3 Provider into a signer

Description
Provides an api endpoint api/sign/s3 Which returns a signed URL.
GitHub Issue
#1607
Type of Change

✅ New feature (non-breaking change which adds functionality)
✅ Breaking change (fix or feature that would cause existing functionality to not work as expected)
✅ Documentation

How have you tested this?
Tested with bugs_testing/sign/s3_workspace.json
Testing Checklist

✅ Existing Tests still pass
✅ Ran locally on my machine

Code Quality Checklist

✅ My code follows the guidelines of XYZ
✅ My code has been commented
✅ Documentation has been updated
✅ New and existing unit tests pass locally with my changes
✅ Main has been merged into this PR

How are requests handled where signing fails? eg. missing resource, missing credentials.

How are requests handled where signing fails? eg. missing resource, missing credentials.

Signing will always succeed even if the credentials are wrong, calling the signed url will return a 403.

I don't understand this bit. Can you comment on this for me to test.
  if (provider[req.params.provider]?.deprecated){
    return await provider[req.params.provider].fn(req, res)
  }

I have updated the S3 module docs with reference to the credentials required from AWS and optional dependencies.

Happy with that! :)

I bumpred the optional dependencies to their current versions 3.691.0
AWS recommends to use ListObjectsV2Command in development. The ListObjectsCommand is only kept for legacy purposes.
I also changed the s3 signer to return the signed URL as a text which it is.

Rather than providing command maps, and action dictionaries to translate commands and properties back and forth it will be much easier to talk directly with the SDK using the commands as documented in the S3 documentation.
https://github.com/GEOLYTIX/xyz/pull/1698/commits/9cce4f17d1e651693a0ff978a04a02bb25db2baf

Don't feel like I'm best placed to review this one so removed my request. If you two @dbauszus-glx and @RobAndrewHurst are happy, that's great with me!


ci: pass credentials to release process

The release process for Maven artifacts fails, if the credentials are missing. Authentication is required to publish them.

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

1.0.0
GitHub release

Your semantic-release bot :package::rocket:


OLD data in the page

This says the SMS outbound does not work in ACS without an inbound SMS message.     In Omnichannel we can do outbound SMS messages

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

ID: d675154d-5fa9-e5da-1628-270c16a02b04
Version Independent ID: d675154d-5fa9-e5da-1628-270c16a02b04
Content: Configure an SMS channel using Azure Communication Services
Content Source: ce/customer-service/configure-sms-channel-acs.md
Service: dynamics-365-customerservice
GitHub Login: @mh-jaya
Microsoft Alias: v-jmh

Thanks for your comment, @bcohen26. This issue has been assigned to the doc author for review.

Hello @bcohen26
Thanks for bringing this up. We have updated the docs and you can find the content here:
Overview of SMS channels
Configure SMS channel using ACS
Thanks,
Jaya


Reorganize react-server-website components

Trying to get things a little more consistently named and organized.

reorganized/renamed Homepage components
moved homepage/HeroSection to a functional component in homepage-body
renamed multipurpose doc-title to page-title
renamed Docco to source-body to align with homepage/docs sections
updated all references

Hoping to get some time soon to start working on unifying code style so we can use the codebase as an example.

Nice. 👍


Fixes #3169 : Track event listeners keyed by type to allow earlier event firing veto

Thank you for the fix.
But doesn't InvocationScopedEventSink.fireOrdered still block on all events if any ordered listener was registered?
Maybe skip events without interested listeners and just consume the queue entry?

Thank you for the fix.
But doesn't InvocationScopedEventSink.fireOrdered still block on all events if any ordered listener was registered?
Maybe skip events without interested listeners and just consume the queue entry?

@SamuelBussmann you're right... we should be able to pull this higher and eliminate a lot more work. Let me move this back to draft and until I get time to rework it.

@SamuelBussmann you're right... we should be able to pull this higher and eliminate a lot more work. Let me move this back to draft and until I get time to rework it.

Just add a filter, whether any listener is interested in this event at all?
And then a further optimization would be to group events by ordered listener to let the other events get past. But this will get tricky fast with Listeners implementing multiple events...


Integrate noremap and map

As discussed in #104, #96, we should implement the easy-to-write macro syntax. The current map and noremap have different features. The map is designed to propagate defined macros to other applications except for win-vind, whereas the noremap effects in win-vind scope only.
However, the map and noremap in the original Vim are used for different purposes with win-vind. It is a factor to misunderstand for Vim users. At this time, this patch allows us to write the following style macros.
noremap j easy_click_left

" a -> move_cursor_down
noremap a j

" a -> easy_click_left
map a j

" only inside {} is actually entered.
noremap a j{This text is inserted}

However, this patch changes the core design and effects .vindrc syntax.

#55

I designed an abstract mapping flow.

In this flow, the main actor is MapSolver, which solve noremap and map recursively and uses it to match the current input. At first glance, the class design is very complex but the main purpose is the latter one. The former uses only at specific times such as :noremap or :source, so it behaves like a constructor.
MapSolver has CmdMatcher consisting of a list of CmdUnitMatcher. Each CmdUnitMatcher has a state to state the matching and the CmdMatcher has matching heads to denote the matching unit. They are alternative classes instead of KeyLogger or KeyLog classes.
When a keyset is inputted, it passes through InputGate and en-queues into Queue. The Queue periodically de-queues a CmdUnit to execute it. CmdUnit derives to three classes, ExternalCmdUnit, InternalCmdUnit, FunctionalCmdUnit. The ExternalCmdUnit, parsed from the syntax like {<s-h>}, executes SendInput of Win32 and creates an internal command unit. The InternalCmdUnit creates only a command unit. The FunctionalCmdUnit, parsed from the syntax such as <easy_click_left>, wraps the conventional BindedFunc to execute each predefined function.
The InternalCmdUnit and the ExternalCmdUnit create the basic CmdUnit to match with global maps. Therefore, the MapSolver also behaves as a function generator. Such MapSolver is passed to CmdUnit.execute() to solve further matching in functions like :wq, :dw, :y$, or etc.
This design is very small and simple rather than conventional KeyLogger, KeyLog, LoggerParser something. In addition to decreasing the number of singletons used as global objects.
I'm implementing it now.

TODO

[x] Implement new mapping parser
[x] Add a new class to solve and map (MapSolver).
[ ] Implement a class to count the prefix number.
[ ] Fix binded functions to fit new mapping system.
[ ] Add a interface into InputGate to register low-level hotkeys from MapSolver.

I eliminated conventional complex window_accel, window_tweight, and window_maxv options and added simpler a window_velocity option.

Most of the refactoring has now been completed, but command mode implementation remaining.
However, since it is difficult to get prefixes in command mode for the system command {mode}map, it is better to seprate each mode by inheriting the class.

I renamed the config cursor_tweight to cursor_resolution.

Removed cursor_maxv, cmd_maxchar, and cmd_maxhist.


Fix dependency resolution for broken build

This is a bit of a workaround for the library change in graphql-language-service-interface now producing esnext output in typescript which breaks webpack.
Looking at bumping to webpack5 but that could be a while, so this is a workaround in the meantime.
You might need to apply this change to your local installation if you run into issues with webpack failing with the following error:
Error: Failed to compile.
/tmp/backstage-e2e-uMeycm/test-app/node_modules/graphql-language-service-interface/esm/GraphQLLanguageService.js 100:23
Module parse failed: Unexpected token (100:23)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|         }
|         let customRules = null;
>         if (extensions?.customValidationRules &&
|             typeof extensions.customValidationRules === 'function') {
|             customRules = extensions.customValidationRules(this._graphQLConfig);

A life-saving solution..


[FR] Improve performance of Bool embeddings

domluna posted on generativeAI Slack a really nice gist using Bool embeddings (held in Int8) + StaticArrays (here).
It seems to provide huge performance benefits compared to my fairly trivial Bool retrieval implementation in RAGTools (here).

playing around with RAG and binary vectors https://huggingface.co/blog/embedding-quantization
the idea of 64 bytes is from this post https://www.mixedbread.ai/blog/binary-mrl
this is brute force parallel implementation for search that assumes the data is stored as binary in byte elements (int8, uint8). so 512 bits is 64 int8 elements, or a 64 element static vector using StaticArrays. For 100M vector dataset I get < 1s comparison time on my M1 macbook air. I’m wondering if there’s anything I can do to make it faster.

It would be excellent to:

[ ] add an extension for StaticArrays.jl
[ ] build a kernel for similarity on top of Bool embeddings in Int8 (see the gist)
[ ] Add the corresponding type <: AbstractSimilarityFinder
[ ] Add relevant tests (+ make sure that the dimension checks still work, since the size(emb,1) would now be 64-times smaller)
[ ] Overload EmbeddingEltype trait (to provide a downstream signal of the embedding type/logic)
[ ] Update any serialize/deserialize functions necessary to be able to store on disk (AIHelpMe uses HDF5, so just test it... eg, BitMatrix wasn't working with HDF5)

Ideally, also integrate into AIHelpMe so everyone downstream can benefit (these TODOs would be transferred to the other repo)

[ ] Add SA.jl as a direct dep of AIHelpMe
[ ] Change all bool knowledge packs to this new implementation
[ ] update artifacts correspondingly

nice!
StaticArrays makes it faster but it's not massive ~2x at most (on 1M vectors), so techinically not explitcitly required:
# no StaicArrays
julia> @b $k_closest_parallel(X1, q1, 10)
21.741 ms (52 allocs: 5.250 KiB)

# q2 is a StaticArray
julia> @b $k_closest_parallel(X1, q2, 10)
12.545 ms (52 allocs: 5.500 KiB)

# q2 is a StaticArray, X2 is a list of static arrays
julia> @b $k_closest_parallel(X2, q2, 10)
9.180 ms (52 allocs: 5.500 KiB)

the cool thing about the binary embeddings is that you can keep everything in memory and you don't need an enourmously powerful computer. for 1 billion rows you would need 64GB instead of 1TB, which greatly decreases costs. Furthermore you can potentially use this as part of a reranking pipeline where you keep a higher dimensional embedding version on disk and then seek the relevant rows from the binary embedding similarity.

function hamming_distance(x1::AbstractArray{T}, x2::AbstractArray{T})::Int where {T<:INT}
    s = 0
    for i in eachindex(x1, x2)
        s += hamming_distance(x1[i], x2[i])
    end
    s
end

changing the sum calc to the above now produces these timings (adding simd or inbounds macros seems to have no effect)
julia> @b k_closest_parallel(X1, q1, 10)
4.710 ms (52 allocs: 5.250 KiB)

julia> @b k_closest_parallel(X1, q2, 10)
4.080 ms (52 allocs: 5.500 KiB)

julia> @b k_closest_parallel(X2, q2, 10)
4.070 ms (52 allocs: 5.500 KiB)

using StaticArrays
using Base.Threads

INT = Union{Int8,UInt8}

function hamming_distance(x1::T, x2::T)::Int where {T<:INT}
    c = 0
    for i = 0:7
        c += ((x1 >> i) & 1) ⊻ ((x2 >> i) & 1)
    end
    return Int(c)
end

function hamming_distance(x1::AbstractArray{T}, x2::AbstractArray{T})::Int where {T<:INT}
    s = 0
    @inbounds @simd for i in eachindex(x1, x2)
        s += hamming_distance(x1[i], x2[i])
    end
    s
end

function k_closest_parallel(
    db::AbstractArray{V},
    query::AbstractVector{T},
    k::Int,
) where {T<:INT,V<:AbstractVector{T}}
    n = length(db)
    t = nthreads()
    task_ranges = [(i:min(i + n ÷ t - 1, n)) for i = 1:n÷t:n]
    tasks = map(task_ranges) do r
        Threads.@spawn k_closest(view(db, r), query, k)
    end
    results = fetch.(tasks)
    sort!(vcat(results...), by = x -> x[1])[1:k]
end

function k_closest(
    db::AbstractVector{V},
    query::AbstractVector{T},
    k::Int,
) where {T<:INT,V<:AbstractVector{T}}
    results = Vector{Pair{Int,Int}}(undef, k)
    m = typemax(Int)
    fill!(results, (m => -1))

    @inbounds for i in eachindex(db)
        d = hamming_distance(db[i], query)
        for j = 1:k
            if d < results[j][1]
                old = results[j]
                results[j] = d => i
                for l = j+1:k-1
                    old, results[l] = results[l], old
                end
                break
            end
        end
    end

    return results
end

the core operation takes 20ns on a static array but when everything is combined we actually get even lower than that on average.
On a 1M vector of where each element is a 64 element vector of Int8
julia> @b k_closest_parallel(X, q, 1)
2.816 ms (50 allocs: 3.547 KiB)

julia> @b k_closest_parallel(X, q, 5)
3.142 ms (52 allocs: 4.516 KiB)

julia> @b k_closest_parallel(X, q, 10)
3.560 ms (52 allocs: 5.500 KiB)

julia> @b k_closest_parallel(X, q, 50)
7.449 ms (54 allocs: 13.938 KiB)

julia> @b k_closest_parallel(X, q, 100)
11.626 ms (54 allocs: 23.781 KiB)

naively looping and doing the distance op 1M times without any additional work would be 20ms but parallelized over 4 cores we're less than that.

I'm actually slightly hesitant to enforce threading under the hood, because:

if we have composite indices (eg, 5 indices with different types of data), it might be more natural to thread over them
we don't actually need the benefits (and associated future complications) - 99% of time spent on GenAI calls, so perhaps we can enjoy the luxury of letting LLVM handle threading and keep it single-threaded on Julia level?

Btw. I'm a bit lost in the references above -- what's your recommendation wrt StaticArrays? Do you think they are a valuable addition or should we keep it simple?

it doesn't seem that StaticArrays is absolutely necessary. The performance will be better but it's not drastically better. For 1M rows, 17.5% faster. Parallel isn't necessary, but it's just a situation where the problem is easily parallelizable (mapreduce pattern) so we do get very close to perfect scaling with cores, i.e., 4 cores makes it ~4x faster, 16 cores - 16x faster.

Closed by https://github.com/svilupp/PromptingTools.jl/pull/152


[Task]: Playground integration test: Changing an example

What needs to happen?
Playground integration test: Changing an example.
Issue Priority
Priority: 3 (nice-to-have improvement)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[X] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue


Need help converting many Dapper rows to List<IDictionary<string, object>>

Hello,
I am trying to convert List of Dapper Rows to List<IDictionary<string, object>>  So while (IDictionary<string, object>)query.Get().FirstOrDefault() works perfectly fine,
(List<IDictionary<string, object>>)query.Get(); fails with an exception "Value cannot be null.
Parameter name: key", for testing purpose two records are returned and both the records are same with no nulls returned from the DB.  What am i doing wrong...

Sorry forgot to mention, I tried multiple approaches
FAILED
 var items = query.Get().Select(r => new { r.Key, r.Value }).ToDictionary(r => r.Key, r => r.Value);
 var Items1 = (from r in query.Get()
                      select new Dictionary<string, object>(r.Key, r.Vaue)).ToList();

So for now I am doing...
List<IDictionary<string, object>> dbItems = new List<IDictionary<string, object>>();
foreach (var dapperRow in query.Get())
{
     dbItems.Add((IDictionary<string, object>)dapperRow);
}

Is there a better way to achieve this.

You can use the QueryFactory.GetDictionnary(Query query) method, check https://github.com/sqlkata/querybuilder/blob/master/SqlKata.Execution/QueryFactory.Extensions.cs#L21-L26

Perfect!! Thanks for your help!!


Bugfix/issue 1451

Description
Thanks for contributing this Pull Request. Make sure that you submit this Pull Request against the develop branch of this repository, add a brief description, and tag the relevant issue(s) and PR(s) below.

Relevant Issues : https://github.com/intuit/karate/issues/1451#issuecomment-774346425
Relevant PRs : (optional)
Type of change :

[ ] New feature
[x] Bug fix for existing feature
[ ] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

opening the fix for now, might add some more unit tests later for matches for contains, contains any etc

@joelpramos

I see the build failed. actually that test looks wrong ! so if you also agree, you can change the test


Interest in adding TagLib library

I was hoping someone could either add support for installing the TagLib library or show me some examples in how to add it myself.
Here's a link to its GitHub repo.

Take a look at our packaging example: https://github.com/Microsoft/vcpkg/blob/master/docs/examples/packaging-zlib.md

It is already ported.

@evpobr , you're right! My apologies, not sure how I missed it.


[doc] consider clarifying the relationship to rpav/c2ffi

here are some questions that i had after reading the readme:

why another project on the same name? doesn't seem to be a fork of it.
what are the planned features that the other c2ffi doesn't support?

Hey @attila-lendvai,
I had it named as something different but then re-wrote it with tests under the name c2ffi as that accurately describes the project. If you have a better name I am open to suggestions.

This project only supports C. The other one apparently supports C++, ObjC, etc. I wrote in the README why only C is supported.
This project fully supports macros objects by parsing via C++ using auto.
This project is intended to be used to generate the .JSON output for each target platform (clang target triple). Then merged together into guaranteed cross platform .JSON output. If it can't, then there is likely something wrong with the C code.
The README contains a brain dump of things which one should do and do not in C for interoperability.
This project is used directly by another project [c2cs](https://github.com/bottlenoselabs/c2cs) to generate C# bindings.

Cheers

thanks for the detailed answer @lithiumtoast !
consider adding some of this to the readme for the possible other wondering hackers... :)
happy hacking!

Thanks @attila-lendvai,
I updated the README with some information.


Same dialog is reused when receiving Subscription-State is terminated;reason=timeout

Here is the NOTIFY informing that our subscription is terminated:
NOTIFY sip:cbsr5umj@ss5fdi3s9p32.invalid;transport=ws SIP/2.0
Via: SIP/2.0/WS <IP_ADDRESS>:5060;branch=z9hG4bK2b37d1c6
Max-Forwards: 70
From: <sip:ID00000002@uep>;tag=as114a0625
To: <sip:6d2b467e-3234-4a6f-81a1-af0edab67638@escaux.com>;tag=6vd6h5hbf8
Contact: <sip:ID00000002@<IP_ADDRESS>:5060;transport=WS>
Call-ID: 9qaqhhdfavpjlj0np4fj
CSeq: 112 NOTIFY
User-Agent: ESCAUX-SIP
Subscription-State: terminated;reason=timeout
Event: presence
Content-Type: application/epresence+json
Content-Length: 0

Here is the SUBSCRIBE  sent back by SIP.js:
SUBSCRIBE sip:ID00000002@uep SIP/2.0
Via: SIP/2.0/WSS ss5fdi3s9p32.invalid;branch=z9hG4bK586217
Max-Forwards: 70
To: <sip:ID00000002@uep>
From: <sip:6d2b467e-3234-4a6f-81a1-af0edab67638@escaux.com>;tag=6vd6h5hbf8
Call-ID: 9qaqhhdfavpjlj0np4fj
CSeq: 2530 SUBSCRIBE
Authorization: Digest algorithm=MD5, username="6d2b467e-3234-4a6f-81a1-af0edab67638", realm="asterisk", nonce="33c25cbe", uri="sip:ID00000002@uep", response="36f2a49c2e05a24c048723e7fdc75033"
Accept: application/epresence+json
Event: presence
Expires: 60
Contact: <sip:cbsr5umj@ss5fdi3s9p32.invalid;transport=ws>
Allow: ACK,CANCEL,BYE,OPTIONS,INFO,NOTIFY,INVITE,MESSAGE
Supported: outbound
User-Agent: Escaux Connect - Development
Content-Length: 0

As you can see the Call-ID and from tag are identical.
The expected outcome would be the creation of a new dialog.

Good catch on this one.  For those looking on, here are some relevant RFC 6665 snippets:
Section 4.4.1 explicitly states that we should be destroying the dialog in this case.

A subscription is destroyed after a notifier sends a NOTIFY request
with a "Subscription-State" of "terminated", or in certain error
situations described elsewhere in this document.  The subscriber will
generally answer such final requests with a 200 (OK) response (unless
a condition warranting an alternate response has arisen).  Except
when the mechanism described in Section 4.5.2 is used, the
destruction of a subscription results in the termination of its
associated dialog.

Section 4.1.3 explains the reasoning behind our immediate resubscription:

timeout:  The subscription has been terminated because it was not
refreshed before it expired.  Clients MAY re-subscribe
immediately.  The "retry-after" parameter has no semantics for
"timeout".  This reason code is also associated with polling of
resource state, as detailed in Section 4.4.3.

So, we are definitely out of compliance with how we are handling this at the moment.  It's a tough one to fix reliably, though, and I want to go about it carefully.  @adaxi, your Pull Request makes an attempt at fixing it, although I am concerned that we are not refreshing all of our state properly when we resubscribe in that case.  It changes one parameter and resubscribes, depending on a lot of code to work that wasn't intended to deal with that sort of situation (a changing this.request).
Reading up on the RFC, the proper thing to do here, to me, seems to be using a completely new Subscription object, building everything from scratch.  The old subscription is terminated, so let's start new.  That has its own issues, though, as it doesn't leave us a lot of places to fix it.  We could either:

Terminate the subscription without reviving it, allowing the application to resubscribe.  This would be an API breaking change, and have to be planned, merged carefully, documented, and communicated accordingly.
Or, we could complicate UA.subscribe with code to deal with timeouts.  This may be okay, but it is certainly unique behavior in that area of the code base.

@adaxi - Is there a specific scenario or presence server that you ran into issues with, or is this just an effort to clean up some non-compliant behavior that could become an issue later?  I'd like to sit on this for a while and gather any feedback I can, both from the other devs and any users who make use of Subscriptions.

I would be in favor of solution that exposes a higher level API. Having timeout errors handled properly by the framework would make SIP.js more reliable and easier to use.
Note that we have intensely been using & testing the PR(s) from @adaxi for the last 2 weeks without noticing anymore problems on an Asterisk 11 communication server.

@wakamoleguy - It is related to #236, before my pull request the timers were always cleared when receiving a NOTIFY, and were added if the NOTIFY contained the Expires time. Since Asterisk does not send the Expires time in the NOTIFY the subscriptions were not refreshed. This caused frequent timeouts, which led me to this issue.
Regarding my pull request, I assumed that it was OK to replace the this.request, since it is  already done in the unsubscribe. As @matthieusieben stated, we have been using this code for the last 2 weeks without issues.

Hi @adaxi,
I recently made some changes to unsubscribe that should make it work properly (as in, actually terminate a subscription). I don't think that had anything to do with this ticket, but is there anything still outstanding here?
Thanks,
James

@james-onsip - I will test this as soon as possible, to check if your changes resolved this issue. As long as a new dialog is created when the subscribe method is called after a timeout it should be fine.

We have tested SIP.js in production with your fixes and everything seems fine.
This issue can be closed.


[Feature Request] Albums within Album

Good day,
Is it possible to have Albums inside Albums to better organization and security/convenience?
Currently, all of my albums are password protected. This means a viewer has to enter a password for each album they want to see (and re-enter that password each time they re-enter the album).
It would be much easier if a viewer could enter a password protected folder and navigate through all sub-albums inside.
is this making sense?
Thank you!

Yes, we are working on a subalbum feature. See the branch feature/subalbums. It's not completely finished yet and you also need to rebuild dist/* first (see docs/Build.md), but you can already give it a try, if you like.

See https://github.com/electerious/Lychee/issues/254 for more information.

Thank you.


wrong redshift in G019

In galaxy_database.csv, G019: Correct Redshift from '1.8' to '0.18'.

Can I be assigned this issue?


OSError: [Errno 30] Read-only file system: '/boot/efi/nixos/vgbwi0dnqv79j3k051myc6vbyqkjwns6-linux-4.9.37-bzImage.efi'

After messing around with the wrapper scripts I use for rebuilding my system, suddenly most of my generations no longer boot, and rebuilds (this time the normal kind without a script) no longer work (for a configuration that built in the past). They yield the following error:
meso% sudo nixos-rebuild switch
building Nix...
building the system configuration...
Traceback (most recent call last):
  File "/nix/store/3qbr1aa4psdiwhsfmmsk6pab82l0a6d5-systemd-boot-builder.py", line 149, in <module>
main()
  File "/nix/store/3qbr1aa4psdiwhsfmmsk6pab82l0a6d5-systemd-boot-builder.py", line 142, in main
remove_old_entries(gens)
  File "/nix/store/3qbr1aa4psdiwhsfmmsk6pab82l0a6d5-systemd-boot-builder.py", line 105, in remove_old_entries
    os.unlink(path)
OSError: [Errno 30] Read-only file system: '/boot/efi/nixos/vgbwi0dnqv79j3k051myc6vbyqkjwns6-linux-4.9.37-bzImage.efi'
warning: error(s) occurred while switching to the new configuration

The last line - OSError: [Errno 30] Read-only file system: '/boot/efi/nixos/vgbwi0dnqv79j3k051myc6vbyqkjwns6-linux-4.9.37-bzImage.efi' is the error I get when trying to boot most of my generations.
 For completeness, they look something like this (update-rebuild is the one I ran just before the trouble)
update-rebuild:  sudo -i nix-channel --update && rebuild-system && clean-system && exit
rebuild-system:  sudo cp -R /my/config/directory/* /etc/nixos && sudo nixos-rebuild switch
clean-system:    nix-collect-garbage --delete-older-than 31d
I used to have the scripts have no sudos in them, but I wanted to install them declaratively and tried this approach.

System: NixOS 17.03.1517.849b5a5193 (Gorilla)
Nix version: 1.11.11
Nixpkgs version: "17.03.1775.56da88a298"
Sandboxing enabled: nope

My hardware-configuration.nix:
# Do not modify this file!  It was generated by ‘nixos-generate-config’
# and may be overwritten by future invocations.  Please make changes
# to /etc/nixos/configuration.nix instead.
{ config, lib, pkgs, ... }:

{
  imports =
    [ <nixpkgs/nixos/modules/installer/scan/not-detected.nix>
    ];

  boot.initrd.availableKernelModules = [ "xhci_pci" "ahci" "nvme" "usb_storage" "sd_mod" "rtsx_pci_sdmmc" ];
  boot.kernelModules = [ "kvm-intel" ];
  boot.extraModulePackages = [ ];

  fileSystems."/" =
    { device = "/dev/disk/by-uuid/5dc6254c-9c02-4695-8f70-7b9d19f929e7";
      fsType = "ext4";
    };

  fileSystems."/boot" =
    { device = "/dev/disk/by-uuid/9E00-97EC";
      fsType = "vfat";
    };

  swapDevices = [ ];

  nix.maxJobs = lib.mkDefault 4;
}

Alright, @grahamc helped me diagnose and resolve the issue. The following is what we did:
1. Check /boot is mounted
We checked that /boot was mounted using mount; in the output, the line:
"/dev/nvme0n1p1 on /boot type vfat (ro,relatime,fmask=0022,dmask=0022,codepage=437,iocharset=iso8859-1,shortname=mixed,errors=remount-ro)"

2. Looked for errors relating to '/boot'
Used dmesg | grep nvme0n1p1 to look for errors related to /boot in the kernel message buffer; this yielded:
[    2.732212] FAT-fs (nvme0n1p1): Volume was not properly unmounted. Some data may be corrupt. Please run fsck.
[ 1535.306323] FAT-fs (nvme0n1p1): error, fat_free_clusters: deleting FAT entry beyond EOF
[ 1535.306325] FAT-fs (nvme0n1p1): Filesystem has been set read-

3. Unmounted '/boot' and ran fsck on it
Unmounted /boot and checked for filesystem errors using the following:
sudo umount /boot
sudo fsck nvme0n1p1

The check found a dirty bit and a mismatch between some sector counts, and I followed the recommended steps to correct it.
4. Remounted '/boot' and rebooted the system
Remounting:
sudo mount /dev/nvme0n1p1 /boot

and voila! Problem solved. Rebooted and now all my generations work again.

Note that the likely culprit of the dirty bit is that my rebuild blackscreened my system, and it seems I went for a hard reboot before the rebuild was finished.

Thanks, that did it for me as well, but your fsck command should be sudo fsck -a /dev/nvme0n1p1


#184581195 buyer should be able to clear their cart

#184581177 Authorization: protected routes (all) should only be accessed by autheticated users
What does this PR do?
This PR ..........
Description of Task to be completed?

............
............
............

How should this be manually tested?

Clone this repo git clone https://github.com/atlp-rwanda/e-comm-team-amigos-bn
Switch your working directory cd e-comm-team-amigos-bn
Create an .env file and update it with .env-example
Install all the dependencies by running npm install
Then run the project by typing npm start

Endpoints to be tested:

...........
...........
...........

Any background context you want to provide?

Have Node Js installed in your local machine
...........

What are the relevant pivotal tracker/Trello stories?
Story ID: ..........
Screenshots (if appropriate)
Questions:

Resolve conflict from this PR

I'm wandering if it could be  possible to delete a single Item from cart?


Update profiles.json

The latest updates on your projects. Learn more about Vercel for Git ↗
Name	Status	Preview	Comments	Updated (UTC)
hacktober2023	❌ Failed (Inspect)			Oct 1, 2023 4:54am

@daksh20031231

In this contributors/profiles.json I have added myself as one of the contributors

Deployment Failure
Reason: JSON Error
FIx:  Missing , in between 2 json objects


E2e tests

@posix4e The github workflow file that runs the E2E test expects the secrets are under the environment called DEV.
If you created those environment secrets under another label, we will have to update that in the workflow file.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 3 committers have signed the CLA.:x: vkolgi:x: vkolagi:x: posix4eYou have signed the CLA already but the status is still pending? Let us recheck it.


Some minor fixes and additions

Fix BigList's default setElements function using the size of the outer array instead of the proper BigArrays.length.
Add an example of intStream usage to overview.html
Update overview.html examples to show off the new of static factory methods.

If I'm understanding correctly, you slightly deviated from the JDK's design in the sense that collector methods are in collection classes, and not in a Collectors class. Am I right?

Correct. I didn't want to add yet another new generated class, and in Guava the collector methods are on the ImmutableX types, so it has precedence.

OK. I see an advantage here in that if you have several implementation of the same idea (e.g., several type of sets) you have a more clear distinction. But this must be documented very well—people will look for IntCollectors. I don't have a good feeling about this. Let me think.

I think the "JDK" way is preferable. For example, having IntCollectors.set() and IntCollectors.unmodifiableSet() as "general purpose". Besides alignment with JDK, you have the freedom to change the backing implementation or even dynamically choose it. FWIW, the "generic" collector could forward to specific stuff like IntOpenHashSet.collector()?

Well, the problem is that, in fact, we cannot really have type-specific collectors unless we wanna add 9³ interfaces, and possibly as many abstract implementations, which I'm not sure I wanna do, as Collectors have three parameters.
The way @techsy730 implemented collection of streams (by methods in the collection classes) certainly reduces flexibility, and breaks partially the fluent API, but at the same time saves us from having a gigantic amount of new interfaces and classes.
I'll document this in the Javadoc, explaining the choice. It is of course a pity, as many functionalities of collectors would be much faster implemented in a primitive way. One possibility might be that of allowing only collectors whose last two parameters are the same (i.e., the data and the accumulator are of the same type). That would be bearable.

Sorry for the necro-post, but the real deal breaker for things like IntCollector is that none of IntStream, LongStream, and DoubleStream have a collect method that takes an equivalent style interface like Collector.
So we would have to either re-implement the primitive Stream types (not happening). Or add such a method to generated subinterface of IntStream et al, that adds that method with a reasonable default implementation.
While this wouldn't be too bad in the sequential() single-threaded case, it would be a nightmare for parallel() streams.


L515 Problem with running the RealSense Viewer

Hello,
i have connected my new L515 with my Comupter and downloaded/installed this:
Intel.RealSense.SDK-WIN10-<IP_ADDRESS>12.exe
After successful installation I want to start the Viewer and that error came up.
I tried the previous Version (Intel.RealSense.SDK-WIN10-<IP_ADDRESS>09) too... but with the same error report.
... when I start the rs-capture the following comes up:

My graphics driver is up-to-date.
Could anybody help me ?
Thanks a lot.

librealsense
2.45.0 RELEASE

OS
Windows

Intel RealSense Viewer / Depth Quality Tool has crashed with the following error message:
Unhandled exception escaping from a worker thread!
Error type: Access Violation!

Hi @Carp-catcher
there is similar issue reported before
https://github.com/IntelRealSense/librealsense/issues/7666
when streaming using viewer, is everything ok and the camera is enumerated as USB 3 device

Hi @Carp-catcher  Do you require further assistance with this case, please?  Thanks!

Hi,
oh sorry, I forgot to answer.
No, thank you.
I don't need further assistance.
Regards
On Sun, Jun 27, 2021 at 12:09 PM MartyG-RealSense @.***>
wrote:

Hi @Carp-catcher https://github.com/Carp-catcher Do you require further
assistance with this case, please? Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/9182#issuecomment-869136773,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUMMXY4ZLWCY63FQ25CQ7B3TU32GTANCNFSM46JUBVZA
.

Thanks very much @Carp-catcher for the update!  I will close this case as you do not require further assistance.


scrollTo bug

window.scrollTo(0, document.body.scrollHeight); don't work because there is two scrolls bar
did u know how we can scroll inside div plz ? or another solution ?

I resolve the problem with :
I don't now if the good way to resolve this


Add package.json

It would be nice to be able to install this project using npm.
Please add package.json - PR here.

As I said in the PR this repo is not intended to be a npm module as it doesn't have any functionality.


Fix minor documentation issue (IDFGH-4354)

Thanks for your contribution.

Thanks for your contribution again, sorry for late reply, changes merged with https://github.com/espressif/esp-idf/commit/6e2dc5d291538a8e6cc50415e2c911ef31048381


[Google cloud] No nodes are available that match all of the predicates: MatchInterPodAffinity (2).

Is this a BUG REPORT or FEATURE REQUEST?
BUG REPORT

What happened:
pvc is not created, Status of pvc replica is pending state
What you expected to happen:
it must be in running state
Anything else we need to know?:
i have created  namespace=test-mongo, where i had applied the sc and i am running demo mongo
Environment:
kubectl get po,pv,pvc,svc,deploy --namespace=test-mongo
NAME                                                                READY     STATUS    RESTARTS   AGE
po/mongo-0                                                          2/2       Running   0          46m
po/pvc-2bdddc35-94ed-11e8-8b97-42010a80008b-ctrl-5f767cb7f5-wvtwv   2/2       Running   0          46m
po/pvc-2bdddc35-94ed-11e8-8b97-42010a80008b-rep-cb96bdb76-5v9gt     0/1       Pending   0          46m
po/pvc-2bdddc35-94ed-11e8-8b97-42010a80008b-rep-cb96bdb76-9nzp2     1/1       Running   0          46m
po/pvc-2bdddc35-94ed-11e8-8b97-42010a80008b-rep-cb96bdb76-9wq4g     1/1       Running   0          46m
NAME                                          CAPACITY   ACCESS MODES   RECLAIM POLICY   STATUS     CLAIM                                         STORAGECLASS
 REASON    AGE
pv/pvc-2577529a-90a1-11e8-8b97-42010a80008b   5G         RWO            Delete           Released   default/mongo-persistent-storage-mongo-0      openebs-mongodb
           5d
pv/pvc-2bdddc35-94ed-11e8-8b97-42010a80008b   5G         RWO            Delete           Bound      test-mongo/mongo-persistent-storage-mongo-0   openebs-mongodb
           46m
pv/pvc-68d269cf-46f1-11e8-b754-42010a800155   5G         RWO            Delete           Bound      default/jenkins-claim                         openebs-standard
           99d
pv/pvc-89b29336-3dd5-11e8-849c-42010a800261   5G         RWO            Delete           Bound      default/demo-vol1-claim                       openebs-percona
           110d
pv/pvc-996add41-0b5d-11e8-91cd-42010a8001fd   6G         RWO            Delete           Bound      default/grafana-claim                         openebs-grafana
           175d
pv/pvc-de690018-9109-11e8-8b97-42010a80008b   5G         RWO            Delete           Released   default/mongo-persistent-storage-mongo-1      openebs-mongodb
           4d
NAME                                   STATUS    VOLUME                                     CAPACITY   ACCESS MODES   STORAGECLASS      AGE
pvc/mongo-persistent-storage-mongo-0   Bound     pvc-2bdddc35-94ed-11e8-8b97-42010a80008b   5G         RWO            openebs-mongodb   46m
NAME                                                    TYPE        CLUSTER-IP      EXTERNAL-IP   PORT(S)             AGE
svc/mongo                                               ClusterIP   <IP_ADDRESS>   <none>        27017/TCP           46m
svc/pvc-2bdddc35-94ed-11e8-8b97-42010a80008b-ctrl-svc   ClusterIP   <IP_ADDRESS>   <none>        3260/TCP,9501/TCP   46m
NAME                                                   DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
deploy/pvc-2bdddc35-94ed-11e8-8b97-42010a80008b-ctrl   1         1         1            1           46m
deploy/pvc-2bdddc35-94ed-11e8-8b97-42010a80008b-rep    3         3         3            2           46m

@inyee786 -- how many nodes are there in your cluster? How is the resource usage?

i have two nodes in my cluster


revise keywords

remove path, 1_010, etc.

stop word, and , the

https://github.com/FancyCoder0/INFOX/commit/d9b6799e9589bf0cb07e93131921ce1513681c70
fixed. need to test for verification.


Add Python 3 support

As discussed in issue #2, here's the pull request that brings in Python 3 compatibility.
Note that I expect TravisCI run to fail on this, since future supports Python 3.3 and later. If 3.2 is required, we could swap the future package for the six package, that one should be compatible with nearly every Python version.

This does fail on 3.2, however our travisci config does not yet fail on unit test errors (which is where this error is presenting). I'm fine dropping 3.2 support, as that's seems to be a common enough choice when trying to make a 2.7 + >=3.3 compatible codebase (for codebases I respect, anyways :smile: , see https://github.com/ipython/ipython/pull/4002
I'm going to do a little more local testing, then i'll merge and fix the travis-ci stuff in post.
Thanks for all the effort here!

:disappointed:
So, my (extremely thorough) tests have produced some less-than-stellar results. They are not conclusive, but more than sufficient to make me do a double take.
My tests consisted of timing the unit test suite for 1k iterations in serial. It's possible that there are deficiencies in the runner or the xml serialization library at play here, so it may be helpful to just create a stress test class to prove out py2 vs py3 performance.
TLDR; the python2 tests were almost identical for master vs this change, but the python3 tests took > 8x as long to run. As a python2 consumer of the lib, that doesn't affect me terribly, but it seems like a real gotcha for python3 devs.

I have committed the change to master that will trigger build failures on unit test failures, and officially drops support for 3.2. Feel free to rebase.

Rebased. :)
The performance issue sounds odd to me, though I haven't thoroughly checked out the code. A quick run with cProfile left me nowhere, I'll try to poke around some more later. If you have any ideas, I'm all ears.

I've been trying to poke around with some flame graphs and PycallGraph, and I think it would help if we swap the python-future for six module quite a bit. Can you share the exact example you used for the 1k runs of the unit test suite? I've got nothing definitive at this time, but it looks a lot better to me.
I can also push this trivial changes to a new branch, say py3-support-six, so you could give it a go as well, if you want.

I've pushed the changes to a new branch, give it a go when you find the time. :)

so, upon further research, this doesn't seem to be an issue in adapt as much as it is in the test framework. These changes do not have a dramatic impact on performance, and I'm happy to merge. Do you have a preference which branch gets pulled in?

Well, after trying both six and future, it seems to me that there are some advantages to using six - the module is smaller (pip install future pulls in ~1,6MB), PyCallGraph generates a smaller image for the same thing (likewise for the flame graph), and runtimes with cProfile seem to be a shade quicker in average. All of this are just "rule of thumb" measures, though. But since we are using just the xrange stuff, it might make sense to switch over to six. And with six, Python 3.2 support should work as well.
I'd love to hear more about your findings with regards to testing framework, if you can spare the time.

Sounds good. You can open a new pr with the new branch, or you can rebase your changes off this branch. I'll leave that up to you.
As for my findings, I don't see a lot of value digging into it very deeply. My  initial "perf test suite" was running the tests 1k times. My new suite is running the single intent parser example 100 times. The disparity between py2 and py3 doesn't appear in the latter test, and I'm happy to chalk it up to the test framework.
Thanks for all your effort!

Rebased, this should be it. With the switch to six module, Python 3.2 support could be reinstated, I think. :)

Thanks for all the effort, this should go into the next release (since we're adding py3 support, i'll make that v0.2.0)

No worries, keep up the good work! :)


Maven repository config variables must specify a username and password

https://nexus.opendaylight.org/content/repositories/public/ and http://nexus.opendaylight.org/content/repositories/opendaylight.snapshot/ are examples of Nexus Maven repositories which host artifacts from a (large) particular open source community, but which does not require authentication.
Currently, setting the Maven repository URL config variable on https://app.dependabot.com/accounts/vorburger/repos (global only; see #503) refuses this with an error saying "Maven repository config variables must specify a username and password", thus apparently insisting that any other Maven repo than central must have both a URL and uid/pwd - which is not, neccessarily, the case.

Sorry about that. I'll get this fixed tomorrow morning - it's a bank holiday here today. 🙂

This should now be fixed. Can you let me know if you have any problems?

I've just been able to set https://nexus.opendaylight.org/content/repositories/opendaylight.snapshot/ as Maven repository URL config variable on https://app.dependabot.com/accounts/vorburger/repos without username and password, so it looks like this works.

@vorburger or @greysteil  Can you explain how you set it? I am looking for the same


When a player join on the server the book it dose not open

book: https://pastebin.com/0EUc1yGG (file name is: update.yml)
config.yml: https://pastebin.com/GtVj2tGP
When a player join on the server the book it dose not open

@TerrorGamesOff What version are you using?

Latest
În mar., 29 oct. 2019 la 21:33, Leonardo Di Gianfelice Stornelli <
<EMAIL_ADDRESS>a scris:

@TerrorGamesOff https://github.com/TerrorGamesOff What version are you
using?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Leonardo-DGS/InteractiveBooks/issues/27?email_source=notifications&email_token=AMLMRW23MXXLLEAUVA7JCITQRCFYXA5CNFSM4JCXF672YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECRZWOI#issuecomment-547593017,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMLMRW5I3D7V74YD72F4NS3QRCFYXANCNFSM4JCXF67Q
.


LeakyReLU uses up too much memory.

Right now I am implementing leaky relus like this tf.maximum(0.1 * x, x).  This works fine except when it comes to memory usage.  Networks which will fit on my GPU when using tf.nn.relu or tf.nn.elu fail when I am using my leaky relu implementation.  I think this is because it needs to store both the intermediate 0.1 * x and x values of the activations to compute the gradients which essentially does the memory usage.  However, I do not think this would be an issue if there were a dedicated tf.nn.leaky_relu.  Can someone consider adding this to a future tensorflow release.

Try this:
def lrelu(x, leak=0.2, name="lrelu"):
     with tf.variable_scope(name):
         f1 = 0.5 * (1 + leak)
         f2 = 0.5 * (1 - leak)
         return f1 * x + f2 * abs(x)

In the future we plan to generate faster, more memory efficient code by automatically fusing operations in some subgraphs, rather than adding lots more specialized Ops.


[DatePicker] Fix cursor pointer of the header

Closes https://github.com/callemall/material-ui/pull/3325. Thanks @CameronFletcher92 for the initial finding.

Thanks @oliviertassinari . TimePicker could user the same TLC. ;-)


Live Demo Stop Working

Issue type
I'm submitting a ...  (check one with "x")

[x ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Issue description
All live demo stop working. Tested access with Safari and Chrome

Steps to reproduce:
Just click in any the link for live demo provided in the home project
Other information:

Duplicate https://github.com/akveo/ngx-admin/issues/5751

Everything is working again. I am closing this issue.


Clean up transactions by making class

Signal felt pretty gnarly, so I cleaned it up a bit my moving some of the logic out into a Transaction class. I also added "states" to Transactions to make it clearer what is going on.

:+1:


Add guide for setting up on wsl

Summary
As part of getting introduced to this project, I set up teiserver on Ubuntu 22.0.4 from scratch. I did not have WSL set up on PC before, so this guide should cover everything needed from complete 0 to a working teiserver.
I took careful notes about all the steps I needed. Many of these are mentioned in the existing setup guides, but glossed over and not explained in enough detail to help someone who's not familiar with the ecosystem or tools to figure out easily.
Test Evidence
Server running locally in WSL:

Webpage after logging in with root@localhost

So in this document you created you already have lots of stuff that's already in the local_setup.md so I recommend we just update the original one but add some spoiler sections that you can fill in. See below for example. Update the below example with any extra comments and rearrange any sections if needed.

This is still a work in progress but loosely the sequence of steps you will need to take to setup a Teiserver is:
Local/Dev
Clone repo
git clone git@github.com:beyond-all-reason/teiserver.git
cd teiserver

Windows-Specific Instructions

Install WSL
Add links to WSL installation and any other comments here

Install Elixir and Erlang
You will need to install:

Elixir/Erlang installed.

You must use the same versions defined in .tools-versions or you will get errors.

SASS
We use sass for our css generation and you'll need to run this to get it started.
mix sass.install

Fake data
Running this:
mix teiserver.fakedata

Will generate a large amount of fake data and setup a root account for you. The account will have full access to everything and the database will be populated with false data as if generated over a period of time to make development and testing easier.
Running it
Standard mode
mix phx.server

Interactive REPL mode
iex -S mix phx.server

If all goes to plan you should be able to access your site locally at http://localhost:4000/.
Libraries you need to get yourself
The site makes liberal use of FontAwesome so if you are using the site you'll need to download it and do the following
fontawesome/css/all.css -> priv/static/css/fontawesome.css
fontawesome/webfonts -> priv/static/webfonts

If you want to use the blog you will also need to place ace.js folder in the same place.
Configuration
Most of the configuration takes place in config/config.exs with the other config files overriding for specific environments. The first block of config.exs contains a bunch of keys and values, which you can update.
Connecting to the spring party of your server locally
telnet localhost 8200
openssl s_client -connect localhost:8201

config/dev.secret.exs
If you want to do things like have a discord bot in development you don't want these details going into git. It is advisable to create a file config/dev.secret.exs where you can put these config details. I would suggest a file like so:
import Config

config :teiserver, Teiserver,
  enable_discord_bridge: true

config :teiserver, DiscordBridgeBot,
  token: "------",
  bot_name: "Teiserver Bridge DEV",
  bridges: [
    {"---", "main"},
    {"---", "promote"},
    {"---", "moderation-reports"},
    {"---", "moderation-actions"}
  ]

# Comment the below block to enable background jobs to take place locally
config :teiserver, Oban,
  queues: false,
  crontab: false

Resetting your user password
When running locally it's likely you won't want to connect the server to an email account, as such password resets need to be done a little differently.
Run your server with iex -S mix phx.server and then once it has started up use the following code to update your password.
user = Teiserver.Repo.get_by(Teiserver.Account.User, email: "root@localhost")
Teiserver.Account.update_user(user, %{"password" => "password"})

Main 3rd party dependencies
The main dependencies of the project are:

Phoenix framework, a web framework with a role similar to Django or Rails.
Ecto, database ORM
Ranch, a tcp server
Oban, a backend job processing framework.

One reason I wanted a separate document is that this guide is very specific to ubuntu 22.0.4 (and with wsl, dunno if that changed anything though) while the other guide is generic and doesn't specify what OS to use, but is lacking specific instructions because of it.
I also wasn't wanting to disrupt the existing docs, but if we want to update that doc instead I can take a look at doing that

People actually tried to setup Teiserver on pure Windows and ran into issues. So my recommendation for all future devs on Windows is to use Linux on Windows.


Hotfix for some missing functions/variables

This PR includes

Added "game 'common'" into fxmanifest.lua. To fix that this resource can't start.
Added the old function back "PatternToRoute" but with some modifications that should fix the variable errors.
Since function "Util:DecodeURI" doesn't exists. Added a function called "decodeURI".

Looks good!   Thank you very much. This was something I had been working on inside of another project so there were a few oversights 😅


UI for importprivkey

A lot of paper wallets are being distributed to nontechnical new users but redeeming a paper Wallet requires a console command which is intimidating and error prone. An import paper wallet dialog that accepts a private key would be a vast improvement.

Any activity on this?
Maybe we should discuss whether to actually import the private key (including all the reindexing) or rather sweep the wallet (creating a transaction that spends all the dogecoin).

Resolved in #1856


localization changes - scenario A

issue: https://github.com/participedia/api/issues/666

when creating a new article save original_language field as the language that the site is being viewed in.
one-off script to create "en" localized_texts records for records that only have another language record saved. i feel like this is just a short-term solution for this bug so  users can read articles that were submitted on the old site in an original_language other than english. ideally, when fetching a case, if there are no localized texts for the language the user is reading in, then we should return the localized_texts for the original_language. i think this needs to be changed in the ${ARTICLE_TYPE}_BY_ID sql files, but i'm not sure how to do that. @dethe what do you think about this interim solution?

todo:

[ ] adding language drop down to entries and email request for translation button (design TBD from jesi)

Yes! I will work on design for the additional drop down placement on the reader as soon as I have my Adobe CC up and running (it just expired, waiting for Amber to green light my renewal at ECUAD). It can wait anyway, this progress is great!


Basic support for v4.1.0

Add support for:

List torrents
Add torrents
Delete torrents
Delete torrents and data files
Pause torrents
Resume torrents
Pause all torrents
Resume all torrents

Things that won't be availabe on this release:

Set torrent priority
Set rate limit
Force recheck
Add Tracker
RSS
Options
Labels
Force recheck
Sequential download
First and last pieces first
Add tracker

Upcoming 4.7.0. release will add support to qBittorrent 4.2.0 (API 2.3.0).  Probably it's compatible with 4.1.x

Version 4.1.9 is supported!!

qBController v4.7.0 fixes it!


Letters in a number are valid?

I am using c# port of this library, but am able to reproduce in the demo here:
+4072d3800439
http://libphonenumber.appspot.com/phonenumberparser?number=%2B4072d3800439
+40721eeasd0
http://libphonenumber.appspot.com/phonenumberparser?number=%2B40721eeasd0
Although this number can be parsed, I wouldn't expect that Result from isValidNumber() would be true

@obkdev
Thank you for using libphonenumber!
We have scenarios where phone numbers are alphanumeric for eg. 1800 MICROSOFT which is 1800 102 1100. Each character corresponds to specific number on the dialpad hence we parse and validate such phone numbers too.

I'll close this as it's not an issue with the library. Please consider joining the discussion group if you'd like to discuss this further.


WIP: t31135461 / Refine jest-worker API

Summary
Node 10 shipped with a "threading API" that uses SharedBuffers to communicate between the main process and its child threads. Being jest-worker a parallelization library, we can take advantage of this API when available. At the same time, we'll decouple our scheduling logic from our communication logic for better re-usability.
Here are some key things:

Decouple the scheduling logic (i.e. everything that's not specifically to send and receive jobs)
Add the option to use "threads" into the library. If they are present, we'll favor them.

Test plan
TBA

Please, take into account that this is a WIP PR. Currently, there are a few parts missing:

Retry in case of a failure
Tests

Ooooooh, nice!

@SimenB you haven't seen the performance part yet 😉
P.S. Added "Usage docs" as a missing part. I think it should come along with this PR to avoid inconsistency

Any news on this one? I'm excited!

@mjesun @Kureev ping 🙂 Would be awesome to land this. Currently landing some breaking changes, so this should be fine to land as part of that if needed

Hi @SimenB! :) Sorry for being silent. I'm working in a different team now, might be someone would like to take it over? IIRC, it's only about tests now. cc @mjesun

@mjesun @rubennorte anyone at FB working on this? If not, anyone able to say what works and what doesn't with this PR so we can land it?

@SimenB AFAIK no one is working on this right now. I might take a look in a few weeks but I don't think it's going to be ready for the next major. Feel free to take over from here if you want to work on this (or anyone else).

Might look into it if you could say some about what's missing. Just getting CI green? More tests?

I merged in master and wrote tests for this on https://github.com/facebook/jest/pull/7408
Separated the PRs to get a clean review 👍

Landed


Preserve closing newline?

Running the beautifier clips the closing newline from the file, which in turn is flagged as a warning by scss-lint.
Any way to get an option to preserve/add closing newline?

+1

Have you tried setting "ensure_newline_at_eof_on_save": true in your user preferences?

You mean in SassBeautify.sublime-settings because the other options are camel-cased? That still does not work.

No I mean in the sublime user settings / preferences.

Greats that works. I wasn't aware of a global config setting for that.

Yes, the preference adds it back in at save. But this sequence happens every day:
1 - run beautify, it removes the newline
2 - sass lint flags the error
3 - save file, adds newline again
It works, but it's annoying.

The beautify plugin should be saving the file after it's beautified, so the linter shouldn't' be complaining, but I'd need to test this to see exactly whats happening. When i get some time I'll have a go with the scss-lint plugin.

Having the newline preserved would be superb. +1

@badsyntax Thanks for the ensure_newline_at_eof_on_save tip!

+1


🐛 Message: no entry found for key (crates/biome_js_semantic/src/semantic_model/scope.rs)

Environment information
CLI:
  Version:                      1.7.2
  Color support:                true

Platform:
  CPU Architecture:             aarch64
  OS:                           macos

Environment:
  BIOME_LOG_DIR:                unset
  NO_COLOR:                     unset
  TERM:                         "xterm-256color"
  JS_RUNTIME_VERSION:           "v20.6.0"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "yarn/3.5.0"

Biome Configuration:
  Status:                       Loaded successfully
  Formatter disabled:           false
  Linter disabled:              false
  Organize imports disabled:    true
  VCS disabled:                 true

Linter:
  Recommended:                  true
  All:                          false
  Rules:                        a11y/useAnchorContent = "off"
                                a11y/useKeyWithClickEvents = "off"
                                a11y/useKeyWithMouseEvents = "off"
                                a11y/useMediaCaption = "off"
                                a11y/useValidAnchor = "off"
                                complexity/noForEach = "off"
                                complexity/useFlatMap = "warn"
                                complexity/useSimplifiedLogicExpression = "off"
                                performance/noAccumulatingSpread = "warn"
                                style/noUnusedTemplateLiteral = "warn"
                                style/noUselessElse = "warn"
                                suspicious/noExplicitAny = "warn"
                                suspicious/noThenProperty = "off"

Workspace:
  Open Documents:               0

What happened?
Biome encountered an unexpected error
This is a bug in Biome, not an error in your code, and we would appreciate it if you could report it to https://github.com/biomejs/biome/issues/ along with the following information to help us fixing the issue:
Source Location: crates/biome_js_semantic/src/semantic_model/scope.rs:115:33
Thread Name: biome::worker_5
Message: no entry found for key
src/client/hooks/useMutationCallback.ts internalError/panic  INTERNAL  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
✖ processing panicked: no entry found for key
⚠ This diagnostic was derived from an internal Biome error. Potential bug, please report it if necessary.
Expected result
It shouldn't crash
Code of Conduct

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

This is a duplica of #2680 and #2659.
The issue is fixed. We need to release a new patch version.
I am keeping the issue open to avoid further duplicates.


Added Custom Price Method Override

Added an override so users can override the name of the price function by adding a custom name in the settings, similarly to how this could be done with the mint function:
       contractMethods: {          mint: "mintQueen",          price: "getPriceQueen"     }

Great addition, thanks! I noticed a bug, so I pushed a fix. I've let Vercel deploy this branch, let's check if it works


Generate languate_id

This pull request implements @bkeepers' idea, generating the language_id from the hash of the language's name. I also added a test to check that all language ids are unsigned 32bit integers.
Supersedes #3247. Fixes #3211.
The language_id is the SHA256 hash of the language name modulo-ed to an unsigned 32bit integer:
Digest::SHA256.hexdigest(language).to_i(16) % (2**30 - 1)

Therefore, as long as the language names are different, the probability of collision is of 0.009561175% if we reach 1,000 languages [1]. This is important as we might not always be able to detect a collision. For instance, if a language entry is removed from Linguist and we later add a new language id that conflicts with the older removed language, the tests won't help detecting it.
The choice to generate the language id from the language name is based on the idea that we probably want two entries with the same name to have the same id (if we remove a language and add it back later). If we do not want that, I could simply change the pull request to generate random ids, which would give us the same collision probability.
I think it's important that we discuss this thoroughly, as we probably want to avoid changing the strategy to generate language ids too often (it's already our second system :stuck_out_tongue_closed_eyes: ).
/cc @bkeepers @arfon @Alhadis @TwP
[1] -  - Details at preshing.com. I took into consideration the fact that we already have 422 fixed language ids.

Weren't we going to use the digest of the entire languages.yml file before the ID is calculated? That's guaranteed to be unique every time, because the hash is being generated based on the entire file's data before the language_id field is inserted.
For instance, let's assume I'm in the middle of adding a new entry:
0x00:
  type: programming
  extensions:
  - ".00"
  tm_scope: none
  ace_mode: text

Prepending that to the current state of the file yields a checksum of 055d502e8b8a4bd1ea4367772cd1e86e:
λ GitHub-Linguist (master): md5 lib/linguist/languages.yml
055d502e8b8a4bd1ea4367772cd1e86e

Now, if I add that checksum to the entry...
0x00:
  type: programming
  extensions:
  - ".00"
  tm_scope: none
  ace_mode: text
  language_id: 055d502e8b8a4bd1ea4367772cd1e86e

... suddenly, the checksum is different:
λ GitHub-Linguist (master): md5 lib/linguist/languages.yml
5f2732b95e6ba0ed1f3d66c97d4be45c
`

Theoretically, that should make it impossible for there to ever be a collision, should it not? The only conflict I can foresee is if two people are adding the same language at the same time, from the same copy of the file. =)

Weren't we going to use the digest of the entire languages.yml file before the ID is calculated? That's guaranteed to be unique every time

No, it's only guaranteed to be different with a probability that depends on the size of the hash function output (output values are guaranteed to be uniformly distributed). That should be the probability I computed above.
If we do not want the language_ids to be the same for two entries with the same language name, then we should simply generate a completely random id. It is simpler and provides us with the same collision probability.

Well then, perhaps just hashing a concatenation of the name and a random ID?

@Alhadis That would give us the same result as generating directly a random id ;)
You seem to think that we should generate a different id every time, even if the languages' names are the same. Why is that? Do you see a case where my proposed method would be an issue?

No, my approach to dealing with numbers I can't grasp is to blast it with as much OTT syrup as possible.
I'll, erm, let you drive... pardon me while I sit in the back seat with all my regular expressions...

@pchaigno I'm happy with the size / possibility space for generating these hashes. If we start seeing too many collisions we can add some entropy into into how we generate these but I don't think it'll be necessary.
On GitHub's side, this change won't have much impact since this field in our database and search it set for 32-bit integers and this change stays within that. However, I'm not familiar enough with linguist's usage outside for GitHub to really say the same for everyone else.
How much of a disturbance will this change be to others? Are there any special considerations we should think about in releasing this change?
CC: @LizzHale @lildude @TwP

If we start seeing too many collisions we can add some entropy into into how we generate these but I don't think it'll be necessary.

Unfortunately there is no way to reduce the probability of collisions other than extending the id bit size since SHA-256 already achieves uniform distribution.

However, I'm not familiar enough with linguist's usage outside for GitHub to really say the same for everyone else.

I don't know of any major project (outside of github.com) which uses Linguist.

How much of a disturbance will this change be to others? Are there any special considerations we should think about in releasing this change?

This change only affects the way ids are generated so I don't think there will be any disturbance; it does not affect the API.


doc: fix typo in PSparseMatrix docstring

Codecov Report
Patch and project coverage have no change.

Comparison is base (725c9e5) 84.69% compared to head (1b80308) 84.69%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #102   +/-   ##
=======================================
  Coverage   84.69%   84.69%
=======================================
  Files          10       10
  Lines        2541     2541
=======================================
  Hits         2152     2152
  Misses        389      389

Impacted Files
Coverage Δ

src/p_sparse_matrix.jl
86.05% <ø> (ø)

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


Battle.net integration not importing Diablo 1

Bug Description
The integration correctly imports all of my games, the only one missing is Diablo 1 (it was added to Battle.net not long ago), I have tried everything.
To Reproduce
No response
Diagnostics ID
35fd0781-df31-4fdc-b424-72780d995b4d
Screenshots
No response

Can you please test it with with this plugin version? Just download and open to install it.
https://playnite.link/download/BattlenetLibrary_Builtin_2_20_test.pext

Support added in 2.21 update


Server Detail on Boot from Volume Instance throws Exception

We are using the icehouse release of openstack. When we boot the node using a volume and use the sdk to get server details , we get the exception
Can not instantiate value of type [simple type, class com.woorea.openstack.nova.model.Image] from JSON String; no single-String constructor/factory method (through reference chain: com.woorea.openstack.nova.model.Servers["servers"]->com.woorea.openstack.nova.model.Server["image"]); nested exception is org.codehaus.jackson.map.JsonMappingException: Can not instantiate value of type [simple type, class com.woorea.openstack.nova.model.Image] from JSON String; no single-String constructor/factory method (through reference chain: com.woorea.openstack.nova.model.Servers["servers"]->com.woorea.openstack.nova.model.Server["image"])
The image is being returned as an empty string as part of the server detail and seems like Json has issues marshalling this ?  Would adding a String constructor be the correct solution for this ?
Thanks

Hi, We are facing similar issue with 3.2.2. snapshot version of the SDK. Could you please share the fix?


Quick Open should be case insensitive

It would be more intuitive to be able to look for a file (even capitalized) by searching for a lowercase string

What behaviour are you currently seeing?
The default for the quick open is to use "smart" case sensitivity, which is if an all lower case string is given, then we do a case insensitive search. Otherwise, if any capitalised letters are given, the search swaps to case sensitive.
There is a config option ("menu.caseSensitive") to change that to simply always be case sensitive or never be case sensitive (set it to true or false).
If my description didn't make sense there, the code for it can be found here : https://github.com/onivim/oni/blob/262098fefd7e82bb69b0b9b68ff66ba6af3d18d8/browser/src/Services/Menu/MenuFilter.ts#L16

@rgehan As @CrossR has already said, unless you type a capitalized character the search should be case insensitive.
I think this is a pretty good default, let us know if you think of a better behaviour.

I'm coming from VSCode so that was a bit disconcerting at first. I guess it does make sense though.
That said, I still found a bug here. If I try to find backers.md (in the oni repo), it properly shows BACKERS.md as the first suggestion, but it is not highlighted. I'll create an issue for that.