fix falcosidekick statsd config in secret

What type of PR is this?
/kind bug
/kind chart-release
Any specific area of the project related to this PR?
/area falcosidekick-chart
What this PR does / why we need it:
Which issue(s) this PR fixes:
The statsd config options not being correctly set in the secret.
It also adds a config checksum to the deployment pods, so that on config change falco sidekick will restart.
Checklist

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] CHANGELOG.md updated

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

c506436 fix falcosidekick statsd config in secret

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign leodido after the PR has been reviewed.
You can assign the PR to them by writing /assign @leodido in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @willejs! It looks like this is your first PR to falcosecurity/charts 🎉

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

c506436 fix falcosidekick statsd config in secret
e888b13 bump chart version

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

c53db6c fix whitespace.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@Issif I have fixed your comment. Could we get a merge please 🙂

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 04fdf224c090c54d1ecc6b7431a63ba54d03c618


add icon to sort airlines by creation date (id) for admin

sort by airlines id which is implicitly by creation date.
limit the feature to admin

Why does this feature need to be restricted by admin?

LGTM thanks! there could be a very minor issue that the top right corner will lose its rounded edge. But it's due to my original hacky way of adding a hidden column
Anyway, i will merge and see if i can correct that


Add ze_device_type_t type” and “ze_device_property_flags_t flags” members to part of Sysman Device structure

with zesInit() introduced, sysman is independent of core for getting device
As we are going to get rid of the below core structure in major version
ze_device_properties_t core
[out] Core device properties
https://spec.oneapi.io/level-zero/latest/sysman/api.html#_CPPv423zes_device_properties_t
GeoPM uses the ze_device_type_t type” and “ze_device_property_flags_t flags” members  from above structure. Hence we need to add these two structures part of Sysman own structure same like UUID https://github.com/oneapi-src/level-zero-spec/issues/111
please tag it to spec1.7 update

@jandres742  please tag to spec 1.7 and make it important to update part of upcoming spec change  @wdamon-intel   please tag to spec1.8

@sanilkumar0: please take a look if this can be done.


redwoodjs server configuration request

Language server
No response
Requested feature
i would like to request an adding of redwoodjs server configuration to nvim-lspconfig
Other clients which have this feature
No response

Thanks for your report. The configs here are best-effort:

It is hoped that these configurations serve as a "source of truth", but they are strictly best effort. If something doesn't work, these configs are useful as a starting point, which you can adjust to fit your environment.

If you have an improvement or fix to one of the configs here, send a PR!
If you have a specific question we'll try to answer it (keeping in mind the configs in this repo are best-effort and unsupported).
If you found a bug in the Nvim core vim.lsp module (not part of this repo), the best way to get it fixed is to report to Nvim (not nvim-lspconfig) with steps in a to reproduce it without the particular LSP server and other factors particular to your environment. Or better, by adding a failing test case to lsp_spec.lua, which has code to setup a fake LSP server to simulate various scenarios.


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1276 ms

Resolved: Bofrost.be performance has improved in e5ecb49 after 41 minutes.


Made changes to LAMMPS dihedral table potential header so that LAMMPS…

… knows it is in radians and also so that LAMMPS calculates forces using NOF option; made changes to table_to_tab.pl to remove r2d scaling for LAMMPS dihedrals since it is not needed for dihedrals

I think it is a bit inconsistent to use different units for angle and dihedral potentials in LAMMPS, but if the makes things easier....

Is the table begin und end fixes for dihredrals?
See: https://github.com/votca/csg/pull/202/files#diff-ecda9b07eec4fc37bda4749e4c3ff896R88

We can change it to degrees.  LAMMPS's tabulated dihedral is in the
USER-MISC package so isn't in the main package.  So it's somewhat
inconsistent with the tabulated angle.  Unlike the angle, however, it will
accept degrees or radians.
It seemed like VOTCA was originally scaling to degrees but then radians was
getting written on the steps after the scale and for the potential.
LAMMPS, however, without specifying "RADIANS" in the header assumes it is
degrees so there was an issue.
On Sat, Mar 4, 2017 at 10:36 PM, Christoph Junghans <
<EMAIL_ADDRESS>wrote:

I think it is a bit inconsistent to use different units for angle and
dihedral potentials in LAMMPS, but if the makes things easier....
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/votca/csg/pull/202#issuecomment-284202778, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AYJucjNZS6Qkoh5CEiHpY854gM19LV2Rks5rii24gaJpZM4MTLuy
.

LAMMPS has a check for angle overlap, so you can't specify -180 and 180
degrees.  It may be in some cases you might want to change the upper and
lower bounds?
Do you have any thoughts on this?
This is from the lammps page:
http://lammps.sandia.gov/doc/dihedral_table.html
"Dihedral table splines are cyclic. There is no discontinuity at 180
degrees (or at any other angle). Although in the examples above, the angles
range from -180 to 180 degrees, in general, the first angle in the list can
have any value (positive, zero, or negative). However the range of angles
represented in the table must be strictly less than 360 degrees (2pi
radians) to avoid angle overlap. (You may not supply entries in the table
for both 180 and -180, for example.) If the user’s table covers only a
narrow range of dihedral angles, strange numerical behavior can occur in
the large remaining gap."
On Sat, Mar 4, 2017 at 10:38 PM, Christoph Junghans <
<EMAIL_ADDRESS>wrote:

Is the table begin und end fixes for dihredrals?
See: https://github.com/votca/csg/pull/202/files#diff-
ecda9b07eec4fc37bda4749e4c3ff896R88
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/votca/csg/pull/202#issuecomment-284202847, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AYJucnE75P9v8NFolgC_kEi6kYz20ZXWks5rii4pgaJpZM4MTLuy
.

"...on the steps after the scale and for the potential."
I meant
"...on the steps after the scale and for the lammps table."
On Sat, Mar 4, 2017 at 11:16 PM, Joshua Moore<EMAIL_ADDRESS>wrote:

We can change it to degrees.  LAMMPS's tabulated dihedral is in the
USER-MISC package so isn't in the main package.  So it's somewhat
inconsistent with the tabulated angle.  Unlike the angle, however, it will
accept degrees or radians.
It seemed like VOTCA was originally scaling to degrees but then radians
was getting written on the steps after the scale and for the potential.
LAMMPS, however, without specifying "RADIANS" in the header assumes it is
degrees so there was an issue.
On Sat, Mar 4, 2017 at 10:36 PM, Christoph Junghans <
<EMAIL_ADDRESS>wrote:

I think it is a bit inconsistent to use different units for angle and
dihedral potentials in LAMMPS, but if the makes things easier....
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/votca/csg/pull/202#issuecomment-284202778, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AYJucjNZS6Qkoh5CEiHpY854gM19LV2Rks5rii24gaJpZM4MTLuy
.

2017-03-04 21:17 GMT-07:00 jdmoore2004<EMAIL_ADDRESS>
We can change it to degrees. LAMMPS's tabulated dihedral is in the
USER-MISC package so isn't in the main package. So it's somewhat
inconsistent with the tabulated angle. Unlike the angle, however, it will
accept degrees or radians.
It seemed like VOTCA was originally scaling to degrees but then radians was
getting written on the steps after the scale and for the potential.
LAMMPS, however, without specifying "RADIANS" in the header assumes it is
degrees so there was an issue.

I see, for dihedrals we should have multiplied table_begin and table_end
with r2d as well.

On Sat, Mar 4, 2017 at 10:36 PM, Christoph Junghans <
<EMAIL_ADDRESS>wrote:

I think it is a bit inconsistent to use different units for angle and
dihedral potentials in LAMMPS, but if the makes things easier....
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/votca/csg/pull/202#issuecomment-284202778, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/
AYJucjNZS6Qkoh5CEiHpY854gM19LV2Rks5rii24gaJpZM4MTLuy>
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/votca/csg/pull/202#issuecomment-284204273, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAa0u4v0UkcRA1m1m_ZISKiDvqxt8x-iks5rijc9gaJpZM4MTLuy
.

--
Christoph Junghans
Web: http://www.compphys.de

I like the idea of not doing a conversion, where it is not really necessary!
Can you please update the help of the script accordingly:
https://github.com/jdmoore2004/csg/blob/3b159663055a1ccab9c6bc4c0c9a87d1862bc0cb/share/scripts/inverse/potential_to_lammps.sh#L29
mentioning that degree are only used for angle potentials and then I will merge this.

Sorry, I didn't get back to making the correction to the help of the
script.  Was planning to do that tonight.
Did you already do, or do you want me to do that?
Thanks.
Josh
On Mon, Mar 6, 2017 at 7:18 PM, Christoph Junghans<EMAIL_ADDRESS>
wrote:

Merged #202 https://github.com/votca/csg/pull/202.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/votca/csg/pull/202#event-988742934, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AYJucrxPX-k8ww-Gzrp-1g2pYpZs3rEoks5rjKJdgaJpZM4MTLuy
.

It is done!


Fix flake8-return warnings

Summary of changes
Fix flake8-return issues reported by ruff 0.1.9.
Pull Request Checklist

[ ] Changes have tests
[ ] News fragment added in newsfragments/.
(See documentation for details)

Thank you very much @DimitriPapadopoulos .
Should flake8-return be added to the ruff configuration as well after these fixes?

Yes, it could be added. Some rules should be disabled though. I have not fixed such issues:
if ...:
    return ...
else:
    return ...

I find the symmetry makes them more readable than:
if ...:
    return ...
return ...

Thank you very much @DimitriPapadopoulos, let's merge this for now...
Later we can add the flake8-return.

Some rules should be disabled though.

I am not sure about this. I am more keen to keep the linter configuration as dry as possible, ideally just working out of the box.


load_weights does not consider custom model(layer)

System information

Have I written custom code (as opposed to using example directory):  No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  Ubuntu 18.04
TensorFlow backend (yes / no):  yes
TensorFlow version:  1.13
Keras version:  Tensorflow included keras
Python version:  3.7
CUDA/cuDNN version:  10
GPU model and memory:  Titan RTX(24GB), 64GB RAM

I failed to load the official Resnet h5 weights with self implemented Resnet using the keras.model.
I found the behaviour of load_weights in network.py does not considered the sub tf.keras.model.
For example, the Resnet has a ConvBlock and a IdentityBlock. Instead of use a function to define them, I used keras.model to define them, so it followed the latest standard.
Instead of simply pass the self.layers to load_weights_from_hdf5_group_by_name.  The load_weights function is network.py should collect all layers inside the sub model, then pass to load_weights_from_hdf5_group_by_name.

Suggested naive fix
`
def get_all_layers (model : tf.keras.Model):

    layers = []

    for layer in model.layers:
        if isinstance(layer, tf.keras.Model):
            layers.extend(get_all_layers(layer))
        else:
            layers.append(layer)

    return layers

`


I want to add the simsimd to xmake's package management tool and have encountered some error.

xmake-repo simsimd pr
There are some error with the simsimd package during the testing and compilation phase.
I don't understand SIMD-related knowledge very well.
How should I resolve these errors?
iphoneos | android v8a r22：
simsimd/spatial.h:350:16: error: assigning to 'int32x4_t' (vector of 4 'int32_t' values) from incompatible type 'int'
        ab_vec = vdotq_s32(ab_vec, a_vec, b_vec);
               ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/runner/.xmake/packages/s/simsimd/v3.9.0/181e36b5210a41de9a550584e58f8699/include/simsimd/spatial.h:351:16: error: assigning to 'int32x4_t' (vector of 4 'int32_t' values) from incompatible type 'int'
        a2_vec = vdotq_s32(a2_vec, a_vec, a_vec);
               ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/Users/runner/.xmake/packages/s/simsimd/v3.9.0/181e36b5210a41de9a550584e58f8699/include/simsimd/spatial.h:352:16: error: assigning to 'int32x4_t' (vector of 4 'int32_t' values) from incompatible type 'int'
        b2_vec = vdotq_s32(b2_vec, b_vec, b_vec);
               ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

ubuntu:
simsimd/types.h:122:9: error: ‘_Float16’ is not supported on this target
  122 | typedef _Float16 simsimd_f16_t;
      |         ^~~~~~~~

Hi @Seedking! Thanks for your effort! It would great to have SimSIMD in xmake!
The 2 issues your are seeing are unrelated to each other.

For the second issue, there are a couple of ways to fix it. Either avoid native f16 support entirely, by overriding SIMSIMD_NATIVE_F16=0. The second option would be to find a better way to infer native f16 support here:
https://github.com/ashvardanian/SimSIMD/blob/3e51bacb1b74a7ed3fa7bb037162276047192bab/include/simsimd/types.h#L167-L194
It would be amazing if you could submit a patch for the f16 resolution here. It would help me a lot, and will be immediately used in USearch and several databases.

For the first issue, I suspect something is wrong with the following Clang/GCC __attribute__:
https://github.com/ashvardanian/SimSIMD/blob/3e51bacb1b74a7ed3fa7bb037162276047192bab/include/simsimd/spatial.h#L303-L305
It works fine in my pipelines, so can you please share the hardware you are running on and the compiler version.

I have added it to xmake-repo. https://github.com/xmake-io/xmake-repo/pull/3409

thank you answer! i will try to add all your repo to xmake


Update and expand security policy

Proposed changes
Opened on my fork just to gather comments and adjust.  This PR will be closed in favor of a PR to the main repo.
If this pull request addresses an issue on GitHub, make sure to reference that
issue using one of the
supported keywords.
Before creating a pull request, make sure to comply with
Contributing Guidelines.

Yesterday I had a discussion with Jordan about the following. Probably we need to establish a threshold for Availability-based issues. We can discuss what kind of threshold could that be (an issue should not be triggered by a local file or the attacker should be very lucky to trigger the issue, etc), but the simplest way is a score-based one. For example, we may avoid issuing Availability CVEs for low and medium scores and only issue them for high and critical. An example could be the latest mp4 CVE, which we could avoid in this case.

I'm closing this PR.  We have a new version that I will open a new PR for.


[HârnMaster 3] styling - Dark mode for roll templates

Use new css class to target separate dark mode styling. Also minor changes to light mode style to add a border and bring it more in line with solarized.

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details
Changes / Description

Title is descriptive: add styling for dark mode roll templates

Character Sheet Info Roll20 Internal Use only.


How can I have an open domain?

Hi.
I recently created a page on GitHub pages and would like to have an open domain. How do I do it? How can I have an open domain? Please see this repo: https://github.com/villenmarx/villenmarx.github.io. Is it possible to have a custom domain like .is-cool.dev, something like that: https://villenmarx.is-cool.dev ?

Yes, it is. Just follow the instructions on the README to get started.


Allow setting the user which we drop permissions to using suid.

What do these changes do?
We may want a different user than nobody when dropping permissions -- for example, if we intend to connect to a PostgreSQL database which does ident authentication.
Are there changes in behavior for the user?
No behaviour change.
Checklist

[x] I think the code is well written
[x] Unit tests for the changes exist
[ ] tox testenvs have been executed in the following environments:

[x] Linux (Ubuntu 18.04, Ubuntu 20.04, Arch): {py36,py37,py38,py39}-{nocov,cov,diffcov}, qa, docs
[ ] Windows (7, 10): {py36,py37,py38,py39}-{nocov,cov,diffcov}
[ ] WSL 1.0 (Ubuntu 18.04): {py36,py37,py38,py39}-{nocov,cov,diffcov}, pypy3-{nocov,cov}, qa, docs
[ ] FreeBSD (12.2, 12.1, 11.4): {py36,pypy3}-{nocov,cov,diffcov}, qa
[ ] Cygwin: py36-{nocov,cov,diffcov}, qa, docs

[x] Documentation reflects the changes
[x] Add a news fragment into the NEWS.rst file

Thanks for the review!  I've repushed with your documentation change.

I do have a slight concern that there might be a better way to have the other connections made and then drop privileges to nobody... but I don't think that's necessarily a blocker here.

I don't think there is -- neither root nor nobody can auth, so both making the connections before or after calling suid(nobody) isn't feasible.


pull request for e2e test at 11/05/2016 21:26:03

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Mangaworld

Checklist:

[x] Updated source's version for individual source changes
[x] Updated all sources' versions for template changes
[x] Set appropriate nsfw value
[ ] Did not change id even if a source's name or language were changed
[x] Tested the modifications by running it on the simulator or a test device

you need to fix the conflicts before this can be merged, try pulling main

fixed by #229


Tracking issue: deny missing docs, one crate at a time

Docs are great! However, it's easy to let them atrophy.
As Bevy grows and stabilizes, we want to be able to turn on #![warn(missing_docs)] (which will cause CI to fail if violated). This is helpful because it helpful for new users and contributors, and ensures that our code base's documentation state doesn't regress.
However, not everything is stable or well documented enough to do so already.
We should turn this on one crate at a time, once it hits 100% coverage. If there's a crate
Let's check the current doc coverage.

Set the following environment variable (on Windows, I used the $Env: syntax): RUSTDOCFLAGS="-Z unstable-options --show-coverage"
Run cargo +nightly doc --workspace --all-features --no-deps

Run on 2023-03-14:
Crate | Docs Coverage

[x] bevy_a11y | 100.0%
[x] bevy_animation | 100.0%
[ ] bevy_utils | 53.3%
[x] bevy_tasks | 100.0% (#3509)
[ ] bevy_derive | 33%
[ ] bevy_macro_utils | 28.6%
[x] bevy_math | 100.0% (#3503 | #4591)
[x] bevy_app | 100.0% (#3539)
[ ] bevy_ecs_macros | 40.0%
[ ] bevy_dynamic_plugin | 42.9%
[ ] bevy_window | 97.8% (#4333)
[x] bevy_crevice | 100% (Not our problem)
[x] bevy_log | 100.0%
[x] bevy_transform | 100.0%
[x] bevy_core | 100.0%
[ ] bevy_reflect| 61.7%
[ ] bevy_input | 90.6% (Claimed by @KDecay)
[ ] bevy_gilrs | 0%
[ ] bevy_diagnostic | 50%
[ ] bevy_winit | 56%
[x] bevy_audio | 100.0%
[ ] bevy_scene | 22.1%
[x] bevy_asset | 100.0% (https://github.com/bevyengine/bevy/pull/3536)
[ ] bevy_core_pipeline | 3.4%
[ ] bevy_gltf |16.7%
[ ] bevy_sprite | 30.8%
[ ] bevy_ecs | 79.5%
[x] bevy_dylib | 100% (warning not yet enabled) (#3515)
[ ] bevy_text | 35.8%
[ ] errors | 66.7%
[x] bevy_internal | 100.0% (#3514)
[ ] bevy_pbr | 37.4%
[ ] bevy_ui | 74.8%
[ ] bevy_render | 48.2%
[ ] bevy_core_pipeline | 22.3%
[ ] bevy_derive | 57.1%
[ ] bevy_encase_derive | 0%
[x] bevy_hierarchy | 100%
[x] bevy_log | 100%
[ ] bevy_mikktspace | 90%
[x] bevy_ptr | 100%
[ ] bevy_reflect_derive| 88.9%
[ ] bevy_render_macros | 25%
[ ] bevy_time | 78.8%
[ ] bevy_utils_macros | 0%

So, the steps to tackle this are simple enough.

Pick a crate that is small, or nearly documented.
Make a PR to finish documenting the crate. If the crate is large, this needs to be split across several PRs to make reviewing easier.
Once you've reached 100% coverage, add #!warn(missing_docs) to the start of that crate's lib.rs file.
Repeat 20 or so times.

If you're looking for good candidates, I think the low-hanging fruit here is:

bevy_utils
bevy_tasks
bevy_math
bevy_app
bevy_dynamic_plugin
bevy_log
bevy_core
bevy_gilrs
bevy_transform
bevy_internal`

plus the crates that are already at 100% coverage. Feel free to ping me if you would like review or help understanding the code.

Does it make sense to add that to bevy_crevice since it's really on the crevice maintainers to keep their docs up to date?

Does it make sense to add that to bevy_crevice since it's really on the crevice maintainers to keep their docs up to date?

Yeah, I agree with you, since we're basically just mirroring it.

I'm going to document the bevy_math crate and will link the PR here once I'm done.

Mentioned this on Discord, but I'll take  on bevy_tasks.

I'm going to add the warning to bevy_dylib and update the documentation if needed.

I'm going to add the warning to bevy_dylib and update the documentation if needed.

I'll be adding the warning to bevy_internal and adding missing docs

I'll be adding the warning to bevy_internal and adding missing docs

I'll be looking at bevy_transform next

I'll be looking at bevy_transform next

I'll take a look at bevy_ui but I make no promises, I think there are parts of this crate I cannot explain yet

I'll take a look at bevy_ui but I make no promises, I think there are parts of this crate I cannot explain yet

I'll take a look at bevy_ui but I make no promises, I think there are parts of this crate I cannot explain yet

Sounds good, it's a big crate so even partial progress there will be very helpful.

I'll take a look at bevy_ui but I make no promises, I think there are parts of this crate I cannot explain yet

Sounds good, it's a big crate so even partial progress there will be very helpful.

For the people with ongoing doc PRs (@Sheepyhead, @james7132, @KDecay, @NiklasEi I think?), could you check your PR with the clippy lint clippy::doc_markdown?
It should be enabled soon, there was a first pass to fix issues but it's not yet finished, check #3457 for more details. It could be painful to re-introduce issues.

... could you check your PR with the clippy lint clippy::doc_markdown?

bevy_dylib
I just checked this for my PR #3515 that updated the documentation of bevy_dylib.
I ran the following command without any errors:
cargo clippy -p bevy_dylib --no-deps --all-targets --all-features -- -D warnings -A clippy::type_complexity -W clippy::doc_markdown
bevy_math / bevy_ui
I also have the PR #3503 that is not completely done yet and updates the documentation of bevy_math and parts of bevy_ui. I will run the following commands when the PR is ready from my end:
cargo clippy -p bevy_math --no-deps --all-targets --all-features -- -D warnings -A clippy::type_complexity -W clippy::doc_markdown
cargo clippy -p bevy_ui --no-deps --all-targets --all-features -- -D warnings -A clippy::type_complexity -W clippy::doc_markdown

could you check your PR with the clippy lint clippy::doc_markdown

For the record I find this lint extremely obvoxious, as it assumes anything in a doc with CamelCase is supposed to be backticked

Updated my PRs to pass the docs_markdown lints.

Updated my PRs to pass the docs_markdown lints.

I have also done this

I'm going to take care of bevy_input.

@alice-i-cecile could you update the PR description?

I'm documenting bevy_window. I'll link a PR as soon as I'm done.

Uh oh... For some reason, GitHub will close this issue when my PR merges, but we haven't documented all of bevy yet!

Uh oh... For some reason, GitHub will close this issue when my PR merges, but we haven't documented all of bevy yet!

Doesn't look like it right now; it doesn't show up in the Linked Issues tab. IIRC "Fixes" and "Closes" are both magic words that trigger that. In any case, we can just reopen this if it gets closed by mistake :)

Okay, Thanks!

there is a missing "unsafe" doc in DynamicPluginExt: https://github.com/bevyengine/bevy/blob/520e413c219494b3158ecbdf2b69a786a8469819/crates/bevy_dynamic_plugin/src/loader.rs#LL36C17-L36C17

Updating the table we have on the current status of this issue:

Crate
Doc Coverage

bevy_a11y
100%

bevy_animation
100%

bevy_app
100%

bevy_asset
100%

bevy_audio
100%

bevy_core_pipeline
22.3%

bevy_core
100%

bevy_derive
57.1%

bevy_diagnostic
50%

bevy_dylib
100%

bevy_dynamic_plugin
42.9%

bevy_ecs_macros
40%

bevy_ecs
79.5%

bevy_encase_derive
0%

bevy_gilrs
0%

bevy_gltf
16.7%

bevy_hierarchy
100%

bevy_input
90.6%

bevy_internal
100%

bevy_log
100%

bevy_macro_utils
28.6%

bevy_math
100%

bevy_mikktspace
90%

bevy_pbr
37.4%

bevy_ptr
100%

bevy_reflect_derive
88.9%

bevy_reflect
61.7%

bevy_render_macros
25%

bevy_render
48.2%

bevy_scene
22.1%

bevy_sprite
30.8%

bevy_tasks
100%

bevy_text
35.8%

bevy_time
78.8%

bevy_transform
100%

bevy_ui
74.8%

bevy_utils_macros
0%

bevy_utils
100%

bevy_window
97.8%

bevy_winit
56%

Taking the first crate on the list currently missing documentation, bevy_core_pipeline, my best judgement (based mostly on those who originally checked in the code in its present location) is that those most familiar with the code, and thus best situated to add the requested documentation, are the following:
@cart --
crates/bevy_core_pipeline/src/blit/mod.rs
crates/bevy_core_pipeline/src/clear_color.rs
crates/bevy_core_pipeline/src/core_2d/camera_2d.rs
crates/bevy_core_pipeline/src/core_2d/main_pass_2d_node.rs
crates/bevy_core_pipeline/src/core_2d/mod.rs
crates/bevy_core_pipeline/src/core_3d/camera_3d.rs
crates/bevy_core_pipeline/src/core_3d/main_opaque_pass_3d_node.rs
crates/bevy_core_pipeline/src/core_3d/main_transparent_pass_3d_node.rs
crates/bevy_core_pipeline/src/core_3d/mod.rs
crates/bevy_core_pipeline/src/lib.rs
crates/bevy_core_pipeline/src/msaa_writeback.rs
@JMS55 --
crates/bevy_core_pipeline/src/bloom/mod.rs
crates/bevy_core_pipeline/src/bloom/settings.rs
crates/bevy_core_pipeline/src/taa/mod.rs
@Elabajaba --
crates/bevy_core_pipeline/src/contrast_adaptive_sharpening/mod.rs
crates/bevy_core_pipeline/src/contrast_adaptive_sharpening/node.rs
@jakobhellermann --
crates/bevy_core_pipeline/src/fullscreen_vertex_shader/mod.rs
crates/bevy_core_pipeline/src/tonemapping/mod.rs
crates/bevy_core_pipeline/src/tonemapping/node.rs
crates/bevy_core_pipeline/src/upscaling/mod.rs
crates/bevy_core_pipeline/src/upscaling/node.rs
@DGriffin91 --
crates/bevy_core_pipeline/src/fxaa/mod.rs
crates/bevy_core_pipeline/src/fxaa/node.rs
@IceSentry --
crates/bevy_core_pipeline/src/prepass/mod.rs
crates/bevy_core_pipeline/src/prepass/node.rs
My apologies if any my guesses were wrong.

My plan is to get to renderer documentation (bevy_render, bevy_core_pipeline, bevy_pbr) sometime soon. I have a couple of open PRs I want to resolve first though.

As I started taking a shot a completing the docs for some of the crates, I thought it would be good to have some updated info on the global documentation progress. I only added PR links for those that added the warn(missing_doc) attribute once a crate doc was complete. Here's what I got:

[x] bevy_a11y | 100.0% | (https://github.com/bevyengine/bevy/pull/6874)
[x] bevy_animation | 100.0% | (https://github.com/bevyengine/bevy/pull/4375)
[x] bevy_utils | 100.0% (https://github.com/bevyengine/bevy/pull/6897)
[x] bevy_tasks | 100.0% (https://github.com/bevyengine/bevy/pull/3509)
[ ] bevy_derive | 57.1%
[ ] bevy_macro_utils | 35.7%
[x] bevy_math | 100.0% (https://github.com/bevyengine/bevy/pull/3503 | https://github.com/bevyengine/bevy/pull/4591)
[x] bevy_app | 100.0% (https://github.com/bevyengine/bevy/pull/3539)
[ ] bevy_ecs_macros | 36.4%
[ ] bevy_dynamic_plugin | 42.9%
[x] bevy_window | 100.0% (https://github.com/bevyengine/bevy/pull/4333) | (https://github.com/bevyengine/bevy/pull/9933)
[x] bevy_crevice | 100% (Not our problem)
[x] bevy_log | 100.0% | (https://github.com/bevyengine/bevy/pull/3520)
[x] bevy_transform | 100.0% | (https://github.com/bevyengine/bevy/pull/3516)
[x] bevy_core | 100.0% | (https://github.com/bevyengine/bevy/pull/3521)
[ ] bevy_reflect| 71.1%
[x] bevy_input | 100.0% | (https://github.com/bevyengine/bevy/pull/9467)
[ ] bevy_gilrs | 33.3%
[ ] bevy_diagnostic | 52.2%
[x] bevy_winit | 100.0% | (https://github.com/bevyengine/bevy/pull/8115)
[x] bevy_audio | 100.0% | (https://github.com/bevyengine/bevy/pull/3510)
[ ] bevy_scene | 22.1%
[ ] bevy_asset | 64.5% | (seems like this one regressed from 100%)
[ ] bevy_asset_macros | 0.0%
[ ] bevy_core_pipeline | 26.3%
[ ] bevy_gltf | 14.9%
[ ] bevy_sprite | 28.8%
[ ] bevy_ecs | 100.0% | (https://github.com/bevyengine/bevy/pull/8731)
[ ] bevy_dylib | 100% (warning not yet enabled) (https://github.com/bevyengine/bevy/pull/3515)
[ ] bevy_text | 34.9%
[ ] errors | 66.7%
[x] bevy_internal | 100.0% (https://github.com/bevyengine/bevy/pull/3514)
[ ] bevy_pbr | 39.2%
[ ] bevy_ui | 79.9%
[ ] bevy_render | 51.2%
[ ] bevy_core_pipeline | 22.3%
[ ] bevy_derive | 57.1%
[ ] bevy_encase_derive | 0%
[x] bevy_hierarchy | 100% | (https://github.com/bevyengine/bevy/pull/4168)
[ ] bevy_log | 100%
[ ] bevy_mikktspace | 90%
[ ] bevy_ptr | 100% | (https://github.com/bevyengine/bevy/pull/4760)
[ ] bevy_reflect_derive| 90.9%
[ ] bevy_render_macros | 25%
[x] bevy_time | 100.0% | (https://github.com/bevyengine/bevy/pull/9428)
[ ] bevy_utils_macros | 50.0%
[ ] bevy_gizmos | 100.0% | (https://github.com/bevyengine/bevy/pull/8186)


Fix #33 bad merge

fixes #33 bad merge

@yvasiyarov here you go, merge at will!

@bolshoy done :-)


Start new UI, add session variables for temporary API portion

Reference: https://github.com/nexB/vulnerablecode/issues/798
Signed-off-by: John M. Horan<EMAIL_ADDRESS>

Here is some comments on the current state of this PR... The goal would be to have a minimally new UI working so we can release it in verision 30.0:
On the landing page
remove template in navbar
third-party JS

Remove https://ajax.googleapis.com/ajax/libs/jquery/3.6.0/jquery.min.js https://github.com/nexB/vulnerablecode/blob/441852f29a3d43b5ccf898ebdb1c405cb6e402d5/vulnerabilities/templates/vulnerability.html#L515 we cannot source JS from third-party sites. We would instead keep a Jquery under version control https://github.com/nexB/scancode.io/tree/main/scancodeio/static

Package search

Replace HTML by proper link in the packages search results list page
In the package detail view hide everything after fixing vulnerabilities

Vulnerabilities search

Replace HTML by proper link in the vulnerabilities search results list page

In the vulnerabilities detail view hide everything after fixed package

hide everything after aliases on the essnetial tabs
Populate Affected Packages et Fixed Packages

for later:

limit searches to a minimum number of characters

Here are a few more suggestions:

for the package details, we could have no tabs and everything in one view

for the vulnerabilities details, a more compact view that would also display affected and fixed packages.

The first few affected packages and first few fixed packages would be always listed on the main tab.
There would be a ... (dot dot dot) linking to the tab with the full table
Full details would be in the tabs.

The count of references, and other items would be added to the right of the tab title inside parenthesis as in References (23)

Need to fix the sorting for all lists in the UI.

Change documentation link in navbar to the RTD.

@tdruez Re my using too many style attributes, I'll do my best to replace these with Bulma classes as you requested, but it will take a fair amount of time to figure out what the Bulma version is of a specific display style I've fine-tuned, and not all likely are covered by the class choices provided by Bulma.  Working on this now.

[ ] Fix the labels and wording case consistency


Context_recall returns only nan when using gpt-4-0125-preview and gpt-4-1106-preview

Describe the bug
When using gpt-4-0125-preview and gpt-4-1106-preview as llm I obtain only nan values for the context_recall metrics.
Ragas version: 0.1.5
Python version: 3.11.7
Code to Reproduce
result_ragas = evaluate(
    data,
    metrics=[
        context_recall
    ],
    llm=ChatOpenAI(model_name="gpt-4-0125-preview", temperature=0.0)
)

same issue for me!

The problem comes from the parsing of the output. gpt-4-0125-preview always responds with:
{"classification": []}
Changing this line in _context_recall.py
response = response if isinstance(response, list) else [response]
with
response = response['classification']
solves the problem

The problem comes from the parsing of the output. gpt-4-0125-preview always responds with: {"classification": []}
Changing this line in _context_recall.py response = response if isinstance(response, list) else [response] with response = response['classification'] solves the problem

no, it can not be solved, because the dict generated by GPT always not strictly follow the example, and randomly generated, so the context recall and context precision should not be generated in this manner


NovelLife SE issue with "always off"

I installed the firmware via https://www.nixies.us/projects/elekstubeips-clock/elekstube-ips-v1-2/, and everything went smoothly. Unfortunately, the function always stays in the "off state," meaning that no matter what I set, it only shows what I configured as the "off state" in the web interface (off, dim, matrix).
It’s possible that I have a hardware issue. Could someone configure the firmware for me without this "display on" setting so that the display stays constantly bright?

Please post screen shots of all the web pages in the GUI.

It's behaving as if it thinks the power button is being held down. There could be one of two things wrong here:

The polarity of the power button isn't what I think it is. If this is the case then if you just long press the power button the display should come to life (except if you have the 'off state' set to dim, in which case it'll stay dim, so I recommend you set the off state to blank before you try this, that's a bug). If this works you are all set. Just remember to do this if you power cycle the clock. Though I would probably release a version that fixes this.
The power button is being read as 'pressed' no matter what state it is actually in, in that case read on:

I tried a test where I unplugged my clock, held the power button down and then plugged the clock back in. The display came up until it had initialized and then went off.
Try this:

Set the 'off state' to blank
Set the 'Display on' control to have the clock off for whatever time it is now where you are. In my testing (where I hold the power button down), this first attempt to change the on/off times fails - the control jumps back to whatever it was. If that happens, do it again and it should take.
Set the 'Display on' control to have the display on.

If I am correct, at this point the clock should turn on.
Let me know if either of these work or not.

The Novellife SE has no power button—or at least, I wouldn’t know where to find it on my version :-)
I tried the method you described, and lo and behold: it works and has been running without issues for the last two hours. Perfect, thank you !

Hello again @hthiel75!
I am curious now, which version of the NovelLife clock do you own?
Can you make a photo and upload it?
I have the following: NovelLife: SE - Spectrum-V2.0
It has not Buttons, just the APDS9960 gesture sensor in front.

And I also had some trouble to get out of matrix rain screensafer. I needed to switch the settings a bit arround, to get the clock to show the undimmed digits of the actual time.
So maybe it will also work for you:
Screenshot of my settings:

I switched to "six" first, then "date" and back to "time", then "Dim" and to "Blank", then I turned the "wheel" until it went "gray" nearly completly, like 23-24, then I turned it back, until it went blue again and shows 0-24.
That makes the clock at least show the time in undimmed digits for me...
So, the question is, what are you hoping to achieve with your clock?

Now I'm curious. These are the clocks with gesture control. Does using gestures do anything with my code?

No, the gesture sensor is not doing anything, as long it is not initalized.
But there are (or was) two kinds of Novellife clocks on the markets.
Novellife Pro (with buttons), see https://www.novellife.net/productinfo/941234.html
and
Novellife SE (without buttons, with gesture sensor), see https://www.novellife.net/productinfo/941237.html
Different hardware revision of the SE exists, like "Spectrum-V2.0" or "V2022-Gesture-V2.1", but they seem do be compatible with each other.
The Pro version is long time out of sale, I saw it available last time in Dezember 2023. So I guess it is irrelevant, as long as nobody demands support for it :)
The problem here can be, that you are using the "default" button pin layout. Some of the pins are used differently on the SE (34 and 35), so I prefered to "kill" the button pins by setting them to an always high pin (GPIO 3 i.e.). I did this in my implementation for the gesture sensor support on the ElekstubeHAX project.
I modifed your code in the fork a bit and it is nearly perfectly working now.
see https://github.com/Martinius79/EleksTubeIPS
I modfied the GLOBAL_DEFINES.h
  // No Buttons on SE verion -- gesture sensor not included in code
  #define BUTTON_LEFT_PIN (3)
  #define BUTTON_MODE_PIN (3)
  #define BUTTON_RIGHT_PIN (3)
  #define BUTTON_POWER_PIN (3)

The only problem I have now, is that the "night time dimming" is not working as expected. If I switch to "Dim" instead of "Blank" or "Matrix", the displays stay dimmed all the time. The set time range seems to be ignored. For the other two modes, it is working.
I wasn't able to watch in your code that deep in the moment to check, what is going on there...
And I also noticed, that with the actual espressif framework and the actual PIO espressif integration, the code consumes to much DRAM...I have no idea, how to fix this... :(

Thanks for doing that. I'll change the implementation to disable the inputs - or produce two binaries.
The Dim settings has a bug, which I will fix at some point.
It sounds like you got it running, so I'm not sure what you mean when you say it consumes too much DRAM - that implies that something doesn't work?

It happens, if you use the actual versions of the PIO espressif imtegration.
platformio.ini:
...
[env]
platform = espressif32@ ^6.9.0
board = esp32dev
framework = arduino
platform_packages = framework-arduinoespressif32 @ 3.20017.0
...

---/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: .pio/build/elekstubev1/firmware.elf section `.dram0.bss' will not fit in region `dram0_0_seg'
---/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: DRAM segment data does not fit.
---/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: DRAM segment data does not fit.
---/.platformio/packages/toolchain-xtensa-esp32/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: region `dram0_0_seg' overflowed by 864 bytes
collect2.exe: error: ld returned 1 exit status
*** [.pio\build\elekstubev1\firmware.elf] Error 1

I forcibly deactivated the weather stuff to be able to build it with that version of the framework.
BTW: I am really impressed with what you are doing with the ESP32! I wish I had your coding skills for microcontrollers and C++ :)

Thank you all for the effort you put in. Thanks to the workaround, I now have a perfectly functioning clock.

v1.6.0 should work without this workaround


OnJoin callback isn't fired on windows 7.

Assuming the problem is related to windows 7 at least.
While I receive the callback on my main PC (windows 10), I don't receive it on my laptop (windows 7).
It isn't being ignored or anything, the api is simply never receiving the call. It DOES receive the ready() callback though, which is odd.

Yeah, I can confirm... nobody on windows 7 is getting join callbacks, but can host fine. Regardless of settings.

OK, I really appreciate your diligence in these issues. This and other Windows 7 oddities are the next thing I'll tackle (praying that the Ask to Join bug is fixed with a recent update I mentioned in that issue).

So I did some debugging on my own, and was not able to reproduce. That doesn't mean that this isn't an issue, but I'll need to do more digging.
I tried to repro using a couple live, free-to-play games: Holodrive and Unturned. In both cases, a user on Windows 7 was able to join a game created by me on Windows 10.
If your users see an issue with either of those games as well, that would be an excellent point of reference for my debugging.

Try this. You'll need your own copy of Duke Nukem 3D's duke3d.grp though.
https://gitlab.com/mossj32/EDuke32-OldMP_Maintenance/tags/eduke32-oldmp_release_12

@msciotti You mention "other Windows 7 oddities", can you elaborate on this? Are they separate issues not related to this particular issue? I'm asking because I'm also having some issues, unsure if they're Windows 7 specific or not.

Mostly outlined in this thread, from Strikerman noting that some Windows 7 users aren't showing Rich Presence status, some aren't getting join callbacks. Some get joincallbacks if the game is running, but not from a cold start, etc.
I have a Windows 7 machine getting delivered on Thursday for testing purposes so I can try and put this to bed once and for all (previously been relying on less than ideal VMs).
If you have other explicit issues, please open issues here with repro steps. The heart of this is figuring out whether these are platform issues, SDK issues, or individual implementation quirks.

I will open a separate issue for that. I did some research I don't believe my problem to be a Windows 7 specific one.

OK, back with some news. We seem to have a bug in our native code for launching games that is specific to Windows 7. I have successfully reproduced the issue on Windows 7 100% of the time.
If you are a user on Windows 7, and you are trying to interact with a join invitation when the game is closed on your computer, the game will launch, but the ACTIVITY_JOIN event will never be received.
If you already have the game open, the failure case in launching the game never occurs, and you successfully receive the join event.
We will get on a fix for this, and hopefully can mark this issue as resolved.

I'm sure I've said these exact words before, but I think this is fixed now. The latest version of our Canary client should no longer eat events because game launches """"failed"""". @GamerUntouch or @StrikerMan780, could either of you confirm with a user on your end for me? And I can mark this resolved.

Haven't heard back and can no longer reproduce locally, so will close this.

I'm on Windows 7, and not getting join callbacks on the Canary build while the game is already open. It loads infinitely after clicking the Join button:

No callback seems to be fired in the game. In the Discord console, I only see received/emitted messages with SET_ACTIVITY when I click the button.

OK, few questions:

What does the SET_ACTIVITY payload look like from the hosting user?

Though I'm 1000% sure you already are, you're running Discord_RunCallbacks() to actually have them fire, yes? I know it's a trite question, but hey, cover all your bases lol

Does the callback fire if the game is closed when you hit the button? That is, will the game open and the callback fire? Or it just opens and does nothing?

3b. If it opens and does nothing, do you see any sort of console error like "Game Launch failed, exited with status code 1"?

Here's what it looks like for the hosting user:

Yes, I'm running the callbacks, of course :D

Yes, it does open the game, but of course it launches it directly from Steam (which does not have rich presence in yet) rather than my development build. (I guess I can avoid that by manually registering it with Discord using a non-Steam command, which I can test tomorrow.)

@codecat Uhhh 🤔 Are you rolling your own lib from the master branch, or using one of the releases? Maybe I broke something...

So, master was broken, as was the latest release. Rolling a new one right now. If you were using one of those, apologies for the headscratching @codecat.
I wasn't properly subscribing to the RPC events on Init(), but it will now.

I'm using the master branch, yeah. Thanks for the fix! I'll test it out tomorrow!

That fixed it, thanks! :D


tensorflow 1.13.1 model and triton server 20.03

Trtion server is 20.03.
I want to use tensorflow-tensorRT optimization by setting config.pbtxt like
optimization { execution_accelerators {
gpu_execution_accelerator : [ {
name : "tensorrt"
parameters { key: "precision_mode" value: "FP16" }}]
}}
But Our models are trained in tensorflow 1.13.1 and does it match triton sever 20.03?
I need to train model in high tensorflow version? Thanks.

What happened when you did that? It is best to have TF match versions but is sometimes not required because TF has fairly good backwards compatbility. If you can't move your TF forward to the 20.03 version, you can try to find a Triton that uses TF 1.13.1. See the Triton release notes.

Closing, reopen if still having a problem.


Simplify the trayicon menu building

I'm closing this since I don't think my refactor here would be benefitial. I feel this is more "clever" than it is "helpful" from a maintenance point of view.
However I think much of the codebase is already in excellent shape until more future changes adds more code-smells I can easily sweep. 😎


Changing pseudo headers from bold to heading tags

Updating headers as required at #7.

Closing in favor of the PR I'm about to submit, which does a massive overhaul on these docs. Thanks for the submission, though, @saulotoledo !


CAMEL-19841 - Upgrade Testcontainers to version 1.19.0

Description

Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

/component-test camel-kafka

Locally I tested aws, gcp and azure

Locally I tested aws, gcp and azure

All my machines are busy, so I couldn't test it manually ... but I think it's gonna be fine.

Let's merge and check ci tomorrow


Why couldn't (or shouldn't) buffers clean themselves?

Hi,
I'm writing a wrapper for alure in Cython for convenience purposes, and I'm wondering which would be the better way to clean up buffers' cache:

Let buffers clean themselves.  I notice that the API only expose Context::removeBuffer but any other means for buffer to do this.  I can just let the context be an attribute of the wrapped buffer, but I want to know if that introduce any risk, i.e. why the C++ API chose not to do it.
Let the context track every created buffer.  ContextImpl seems to already have mBuffers, is it safe to just call cleanup on all of them upon context destruction?  I don't think this is a good idea though since used buffers will hold the cache for no reason.

Have a Merry Christmas or other holiday of your choice.

The reason buffers aren't cleaned up automatically with context destruction is because there's no guarantee the context is current, which it would need to be to delete the AL buffers. In fact, I think the context can't be current for it to be destroyed. So for Alure to do that, it would need to transparently manage the current context itself, changing it based on what object the caller is using or deleting. This can get complicated when there's multiple contexts and multiple threads being used simultaneously.
More generally, it's also a good idea for an app to properly manage and clean up its buffers when no longer needed, rather than relying on context destruction to clean them up. Given a game or other long-running app, you don't want to be creating/caching buffers as needed but never deleting them until device shutdown. That can easily waste a lot of memory.

Thanks for the prompt and detailed explanation.  I still have some questions though:

[the context] would need to be [current] to delete the AL buffers

I have not seen that mentioned elsewhere.  Does this rule apply to other objects like Source?  Or more generally, what's the role of having multiple contexts in one device?

Does this rule apply to other objects like Source?

Yes, just like OpenAL itself. With OpenAL, the current context determines which context and device things like sources and buffers belong to. Alure's objects are essentially wrappers around OpenAL objects, so to delete or otherwise modify a Buffer or Source from Alure, the proper Context from Alure needs to be current for the OpenAL context to be current.

Or more generally, what's the role of having multiple contexts in one device?

One use-case could be to share a device with two independent mixes. For instance if you have the main game sound on the main context, but also want to play something else on the same device without worrying about changes to the main context interfering. Another use-case could be something like split-screen, where you're listening from two separate perspectives at once.

Thank you, I think I get it now.


[Help] dbt-project-dir is not recognizing any file path

Passing the --manifest argument allows all the check hooks to pass with flying colors. I'd love to use the dbt-project-dir but no combination of pointing to the manifest.json file seems to work.
full manifest argument
args: ["--manifest","src/waterworks/dbt/target/manifest.json"]
I'd love an example of how to properly path the dbt root directly since it's not at the root level. I can't seem to find a single example

Did you try adding it to the .dbt-checkpoint file?

Yes, trying to figure how what path it needs? My root is in
src/waterworks/dbt
On Fri, Oct 27, 2023, 3:55 PM Noel Gomez @.***> wrote:

Did you try adding it to the .dbt-checkpoint file?
—
Reply to this email directly, view it on GitHub
https://github.com/dbt-checkpoint/dbt-checkpoint/issues/159#issuecomment-1783592692,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY3HJJKY4RNKSC6LHIBSMFTYBQ3UXAVCNFSM6AAAAAA6TN5HUWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGU4TENRZGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

like this
https://github.com/datacoves/balboa/blob/main/.dbt-checkpoint.yaml#L3
change transform to src/waterworks/dbt

Thanks, I'll check it out!
On Sat, Oct 28, 2023, 7:15 AM Noel Gomez @.***> wrote:

like this
https://github.com/datacoves/balboa/blob/main/.dbt-checkpoint.yaml#L3
change transform to src/waterworks/dbt
—
Reply to this email directly, view it on GitHub
https://github.com/dbt-checkpoint/dbt-checkpoint/issues/159#issuecomment-1783825506,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AY3HJJO7TBWPJXZJHFXEFYDYBUHPLAVCNFSM6AAAAAA6TN5HUWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTHAZDKNJQGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Here is my .dbt-checkpoint.yaml file:
version: 1
disable-tracking: true
dbt-project-dir: src/waterworks/dbt

Here's the error I am getting, I wonder if I need .dbt-checkpoint.yaml in a dbt run?
17:40:04  Running with dbt=1.6.6
17:40:04  Encountered an error:
Runtime Error
  No dbt_project.yml found at expected path **C:\Users\**\Documents\GitHub\**\dbt_project.yml**
  Verify that each entry within packages.yml (and their transitive dependencies) contains a file named dbt_project.yml

Correct Path = C:\Users*\Documents\GitHub*\src\waterworks\dbt\dbt_project.yml

Well, I can get it to work and pass when I manually run pre-commit run. But when I actually submit a commit via VSCode I get the following errors for dbt-test & dbt docs generate
All check hooks pass with flying colors:
Check the script does not contain a semicolon............................Passed
Check the script has not table name......................................Passed
Check the script has existing refs and sources...........................Passed
Check the model has description..........................................Passed
Check the model has properties file......................................Passed

Traceback (most recent call last):
  File "C:\Users\AdrianBogle\AppData\Local\Programs\Python\Python38\lib\runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\AdrianBogle\AppData\Local\Programs\Python\Python38\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\AdrianBogle\.cache\pre-commit\repobc10wcfr\py_env-python3.8\Scripts\dbt-docs-generate.EXE\__main__.py", line 7, in <module>
  File "C:\Users\AdrianBogle\.cache\pre-commit\repobc10wcfr\py_env-python3.8\lib\site-packages\dbt_checkpoint\dbt_docs_generate.py", line 37, in main
    return run_dbt_cmd(cmd)
  File "C:\Users\AdrianBogle\.cache\pre-commit\repobc10wcfr\py_env-python3.8\lib\site-packages\dbt_checkpoint\utils.py", line 341, in run_dbt_cmd
    output = cmd_output(*list(filter(None, cmd)), expected_code=0)
  File "C:\Users\AdrianBogle\.cache\pre-commit\repobc10wcfr\py_env-python3.8\lib\site-packages\dbt_checkpoint\utils.py", line 103, in cmd_output
    proc = subprocess.Popen(cmd, **kwargs)
  File "C:\Users\AdrianBogle\AppData\Local\Programs\Python\Python38\lib\subprocess.py", line 858, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "C:\Users\AdrianBogle\AppData\Local\Programs\Python\Python38\lib\subprocess.py", line 1311, in _execute_child
    hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] The system cannot find the file specified
Executing cmd: `dbt docs generate --no-compile --project-dir src/waterworks/dbt`

Traceback (most recent call last):
  File "C:\Users\AdrianBogle\AppData\Local\Programs\Python\Python38\lib\runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\AdrianBogle\AppData\Local\Programs\Python\Python38\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\AdrianBogle\.cache\pre-commit\repobc10wcfr\py_env-python3.8\Scripts\dbt-test.EXE\__main__.py", line 7, in <module>
  File "C:\Users\AdrianBogle\.cache\pre-commit\repobc10wcfr\py_env-python3.8\lib\site-packages\dbt_checkpoint\dbt_test.py", line 57, in main
    return run_dbt_cmd(cmd)
  File "C:\Users\AdrianBogle\.cache\pre-commit\repobc10wcfr\py_env-python3.8\lib\site-packages\dbt_checkpoint\utils.py", line 341, in run_dbt_cmd
    output = cmd_output(*list(filter(None, cmd)), expected_code=0)
  File "C:\Users\AdrianBogle\.cache\pre-commit\repobc10wcfr\py_env-python3.8\lib\site-packages\dbt_checkpoint\utils.py", line 103, in cmd_output
    proc = subprocess.Popen(cmd, **kwargs)
  File "C:\Users\AdrianBogle\AppData\Local\Programs\Python\Python38\lib\subprocess.py", line 858, in __init__
    self._execute_child(args, executable, preexec_fn, close_fds,
  File "C:\Users\AdrianBogle\AppData\Local\Programs\Python\Python38\lib\subprocess.py", line 1311, in _execute_child
    hp, ht, pid, tid = _winapi.CreateProcess(executable, args,
FileNotFoundError: [WinError 2] The system cannot find the file specified
Executing cmd: `dbt test -m account_dimensions_monthly+ --project-dir src/waterworks/dbt`


remove -Zmiri-panic-on-unsupported flag

Fixes https://github.com/rust-lang/miri/issues/3952, see that issue for discussion.

nextest is the better option for "continue running other tests if one test failed"

Nextest does not and cannot support doctests, so this would mean we don't have a way to continue running if one doctest fails.

Cargo/rustdoc already supports continuing if one doctest fails, I think?
It also runs all doctests concurrently. That's why in miri-test-libstd, doctests are so much faster than unit/integration tests. ;)

Ah of course it does, this is probably the same reason to get a single-threaded doctest run you need -j1 plus --test-threads=1.

So if all use of this flag can be replaced by nextest, why should we keep it at all? I don't think I've seen this actually happen, but injecting new unwinding sites into a bunch of unsafe code may cause UB that's only there because of unwinding that can only happen with Miri.

All right, I now made this just entirely remove the flag. @rust-lang/miri any objections?

No objections.

@bors r=RalfJung,saethlin,oli-obk

:pushpin: Commit 2b3deec2e606f080f1ecff90c6bf049b29716408 has been approved by RalfJung,saethlin,oli-obk
It is now in the queue for this repository.

:hourglass: Testing commit 2b3deec2e606f080f1ecff90c6bf049b29716408 with merge bdc100f917b0eef1984afbeba466d9216a811352...

:sunny: Test successful - checks-actions
Approved by: RalfJung,saethlin,oli-obk
Pushing bdc100f917b0eef1984afbeba466d9216a811352 to master...


Caffe2 support

Do Netron support visualization of caffe2's detectron models which are stored in pkl format. Thanks in Advance.

Did you try the converter or get it fixed?

Hi. I tried the converter. I was able to convert the detectron model from pkl to pb format. But when Iam trying to serve the pb model, I am getting the following error
"From localhost:8080
Error loading TensorFlow model. invalid wire type 4 at offset 3"
Please help me out. Thanks

Per Caffe2 convention model, the file needs to be called xxxx_predict_net.pb to be detected as a Caffe2 model. Otherwise it will be assume that it is in TensorFlow or ONNX protobuf format.

Thanks @lutzroeder . Iam bale to visualize the detectron model with above steps. Thanks for the quick response and thanks for helping me out.

Duplicate of #1116


Data disk mounting doesn't behave well with automatic OS image upgrades

With automatic OS image upgrades when the OS image is refreshed the data drives will be rediscovered by the new OS and not necessarily mount to the same drive letters as provided in the original powershell script.
This can cause any code depending on certain drive letters to fail. Can there be some warning here that this script may have compatibility issues with automatic OS image upgrades?

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

ID: 4dd4a5e2-c630-2702-ac7c-15004b541161
Version Independent ID: f64d7493-0278-de95-a16e-c11369f74c55
Content: Attach a data disk to a Windows VM in Azure by using PowerShell
Content Source: articles/virtual-machines/windows/attach-disk-ps.md
Service: virtual-machines-windows
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

@johnthcall Thanks for reaching out. We will review and provide an update as appropriate.

@johnthcall Thanks for the feedback, I am assigning the issue to the author for updating the document as required.

@roygara Can you please add your comments and update the document as required.

@johnthcall Thanks for reaching out, we appreciate it!
I'd like to confirm the behavior here before I reach out to the product team: For clarity's sake, you're referring to this specific segment of the script, right?
`$disks = Get-Disk | Where partitionstyle -eq 'raw' | sort number
$letters = 70..89 | ForEach-Object { [char]$_ }
$count = 0
$labels = "data1","data2"

foreach ($disk in $disks) {
    $driveLetter = $letters[$count].ToString()
    $disk |
    Initialize-Disk -PartitionStyle MBR -PassThru |
    New-Partition -UseMaximumSize -DriveLetter $driveLetter |
    Format-Volume -FileSystem NTFS -NewFileSystemLabel $labels[$count] -Confirm:$false -Force
$count++
}`

In other words, New-Partition -UseMaximumSize -DriveLetter $driveLetter doesn't assign a permanent drive letter?

@roygara Yes I am referring to that section of the script. The New-Partition does assign that letter and if you reboot one of the vmss instances it will come up with that same letter. However if a new vm image is applied to the scale set the new vm image no longer has the mapping and the drives (which were already formatted) will be mapped to the first available drive letters.

D should NOT have been "automatically" selected as a temp drive (T or X would be more appropriate) -- why - because 98% of non-cloud based servers out there use D for data or other uses other than temp storage.   Those of us that have tried to "lift and shift" our servers (which have scripts that reference drive paths) to Azure have been killed by this "featured"!
Just a "real-world" opinion.
[End of Rant]

@johnthcall Keeping you updated: I reached out to the team when you confirmed that a few weeks ago and they've got some ideas they want to try out. I've yet to hear back anything else, though. I'll update here when that changes.

#assign: @MicahMcKittrick-MSFT

We are currently cleaning up old issues and closing out items that are greater than 90 days old. If an issue is still present, please open a new feedback item on the document so we can prioritize correctly.
#please-close

Should this be closed?  Can someone confirm -- Can we create a VM and specify temp storage as something other than "D", i.e. "X" or "T" and if we then update the VM to a different service level, will temp remain at something other than "D"?   I have my doubts.


Implement a JSON serializer

We should now have enough material to try and implement a first serializer.
The main idea is to implement a function like the following:
def jsonSerializer[A](schema: Schema[A])(implicit prims: ToJson[Prim[A]]): A => JSON = ???

Where ToJson is a simple typeclass needed to convert the primitive types of the module to JSON.
The actual JSON representation isn't very important for now, we can just define type JSON = String for the moment.

Done in #26


Is there a more optimized way to complete training? Every game has to be learned for so long, people learn to master it early

Is there a more optimized way to complete training? Every game has to be learned for so long, people learn to master it early

Hello,
I am not sure about what you mean by "more optimized way" but if the training is taking too long you can try tuning the HP, we have not tuned all the configurations of the games implemented, for some it’s just a starting point.
Hope it helps, I am closing this for inactivity but feel free to reopen for any thoughts on this question.


docs: "getting started with development" -instructions

Currently docs are mostly written for end-users that use reana-client to interact with REANA cloud.
There is a getting started with development section, but it focuses on bringing up REANA cluster.
It might be beneficial to add more detailed development instructions or direct the user to REANA cluster documentation and write development instructions there.

Closed by https://github.com/reanahub/reana/pull/135 (introduces Makefile and usage instructiongs).


Add support for LOCATE scalar function

ElasticSearch supports LOCATE, but not with a negative position as Exasol does.
Perhaps we can write an adaptation for it.
https://docs.exasol.com/sql_references/functions/alphabeticallistfunctions/locate.htm#Locate
https://www.elastic.co/guide/en/elasticsearch/reference/current/sql-functions-string.html#sql-functions-string-locate

Shelved.


[MINOR] Set connection settings for maven to avoid build flakiness

Change Logs
Update the options passed in to maven install commands to help with connection issues seen in recent builds. I found the options in this StackOverflow post https://stackoverflow.com/questions/55899091/maven-retry-dependency-download-if-failed
Impact
Improve build stability
Risk level (write none, low medium or high below)
None
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

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

Contributor's checklist

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

Maven builds pass in CI.  Going to merge this PR now to improve CI stability.


Snyk vulnerability with Lodash in devDependencies

Your project uses Snyk which is at version 1.198.0 which is prior to the lodash vulnerability. Snyk warned about lodash itself in your examples (https://github.com/jariz/gatsby-plugin-s3/pull/149), but not in your root project.

Thank you!


Merge release + build into one process

I forgot that actions cannot trigger other actions. So of course this didn't work. I've merged the "create release" + "build/deploy release to the archive" into one process.

thanks @hexylena!


Checkbox

@deafjam what is this issue created for?

@deafjam is there any specific feature you want to include?

@deafjam update to the latest version 6.0 and you can use checkbox tableviewcell style.
selectionMenu.cellSelectionStyle = .checkbox


Extract script from upload-release-asset workflow

We can avoid having a scripting code inline by creating a separate script and combining it with upload-release-asset. This keeps the yml file clean and isolated scripts can be tested easier locally.
Originally posted by @davidgamez in https://github.com/MobilityData/gtfs-validator/pull/1529#discussion_r1258709037
Needs #1529 to be merged.

[ ] Investigate alternatives to upload arbitrary assets to a GitHub release
[ ] Share with the team and Implement it


Revert "Pause wallet connexion"

Reverts w3hc/strat-podcast-ui#8

Ledger connector incident: https://twitter.com/Ledger/status/1735326240658100414


Error 401 Unauthorized when accessing backend over https

I'm getting an error 401 unauthorized when accessing the backend over https, however it loads fine over http.

I noticed this behaviour only for sites on my organization account. It works fine for sites on my personal account. I was able to reproduce this error multiple times on my organization account.
Update: this appear to only be a problem in Chrome, it works fine in Firefox. Please let met know if I you require additional information.

Try this:

Log out (dropdown in the top right)
Close the tab
Open new tab and load the CMS

Thank you @erquhart your solution works fine!


Fix the 1 week ahead problem for airingScheduleQuery

The query to get the airingSchedule is limited to airing schedules between time of the query and 1 week ahead to avoid getting several airingSchedules of one anime and to avoid filtering and dealing with pages.
This leads to no information if one of the following is true

Anime is not yet released and release start is more than one week ahead
Anime is on hiatus due some event and next episode is more than one week ahead

The wanted way would be to get the next airingSchedule of each anime, independend of it's airingAt time. I see two ways to solve this right now:

Get all airingSchedules in the future and pick the first one. Has the problem that a lot of unneeded data is sent and the query is paginate, making it difficult to find the right one and resulting in several calls
Find a possibility to filter in the query itself by saying "get this mediaId once, and than the next one", similar to:  http://graphql.org/learn/pagination/#pagination-and-edges. Would be the preferred way of course.

Fixed with v3.0.0


Swift 3 migration

Hopefully not too difficult, haven't looked into it yet.

Hello Ash, do you have any news?

I have not, I'll try to get to it this week.

For those having a problem converting functions with protocol conformance, here is an example for the TableViewCell.swift:
func prepareCollectionView(_ delegate: UICollectionViewDelegate, _ dataSource: UICollectionViewDataSource, forRow row: Int) {
//Or func prepareCollectionView(_ delegateAndDataSource: UICollectionViewDelegate & UICollectionViewDataSource, forRow row: Int) {
    collectionView.delegate = delegate
    collectionView.dataSource = dataSource
    collectionView.tag = row
    collectionView.setContentOffset(collectionView.contentOffset, animated:false) // Stops collection view if it was scrolling.
    collectionView.reloadData()
}

Hi, i have an question: if the collection view cell numberOfItemsInSection are different, and when i scroll collection view  will crash due to Index out of range.
I found the problem that when i scroll the tableview the collection view cell index path updated to the bottom one, but the collection view i scrolled is top one and it does't remember the numberOfItemsInSection, so it will crash due to Index out of range .
Do you have any idea how to solve this problem?
Thanks


Perform code base move for Data Conservancy repos

Follow up to #208. Now that the IP review is complete, open a ticket with Eclipse (similar to https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/1221) to transition the two Data Conservancy repos to eclipse-pass

Repo
Link

pass-fcrepo-jms
https://github.com/DataConservancy/pass-fcrepo-jms

pass-fcrepo-jsonld
https://github.com/DataConservancy/pass-fcrepo-jsonld

A ticket has been created in the Eclipse Helpdesk to handle this https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/1267
However, I don't have the proper permissions to add the eclipsewebmaster user to these repos so they can do the transfer. Currently, Bill, Mark, Jeff (and Jason) are able to do this.
Once this user is added, I was to update the Helpdesk ticket letting them know they had proper permissions to proceed.

I added eclipsewebmaster as admin to the two repos and made a note of this in the Eclipse ticket.


Android App Crashes

I have had about six cases where the Android app crashes. I have a Samsung S3 with Android version 4.4.2. This happened twice on my way to work this morning which is about a 1/2 hour, 15 mile drive. The app shut down after about five to 10 minutes of recording.  It also happened a couple of times while walking with the app after only a few photos were taken.

Hi @MikeTho16,
Sorry you are experiencing crashes. These are logged automatically. So we should be able to act on those as they come in. Thanks for reporting and being specific around device model and Android version.

Martijn Thanks for the response. Let me know if there is any specific testing I can do to help you track this down.  My OSV username is tekim.

Hi MikeTho16, could you tell us if you saw the typical "app has crashed" popup, or did the app restart shortly after the crash? Thanks.

@kalmanbencze Most of the time it simply exits without a message.  Sometimes there is a popup stating that the app has crashed (next time it happens I will attempt to write down the exact wording) and asking if I would like to report it.  I usually click yes, but I think a couple times in my haste I clicked the other option.  The app stopped (with no message) three times on my way to work this morning.  Also, it crashed - with a popup - last night while uploading photos, although I think all photos were uploaded.

This bug report should probably be moved to the android repo @mvexel

This issue was moved to openstreetview/android#9


H2 and MapDB persistance classes in 0.11

I'm using moquette-broker embedded in a project, but it seems the jar file (https://jcenter.bintray.com/io/moquette/moquette-broker/0.11/:moquette-broker-0.11.jar) does not contain the H2 or MapDB persistence classes. Are those available as a separate dependency? I've not found them on bintray.

As described in docs you should include the bintray jcenter repos in your maven pom.xml
<repositories>   <repository>     <id>bintray</id>     <url>https://jcenter.bintray.com</url>     <releases>       <enabled>true</enabled>     </releases>     <snapshots>       <enabled>false</enabled>     </snapshots>   </repository> </repositories>
And then you have to include the H2  https://dl.bintray.com/andsel/maven/io/moquette/moquette-h2-storage/0.11/
OR the MapDB https://dl.bintray.com/andsel/maven/io/moquette/moquette-mapdb-storage/0.11/

That's the problem.
https://dl.bintray.com/andsel/maven
is not
https://jcenter.bintray.com
So adding the jcenter repository, as documented, just gives me a "Could not resolve dependencies", since these two are not in the jcenter repository. You probably should update the docs to point to your personal bintray repository, or get these two jars added to the jcenter repository.


Error in log Unable to open mgnt_socket

Describe the issue here.
I had installed the last batmon HA 1.78 in my campervan HA.
The HA runs on a raspberry PI 3b
I used two Supervolt batteries.
I observer this error messages during startup the addon on the log.
13:27:50 INFO [main] Power cycle bluetooth hardware
Unable to open mgmt_socket
Unable to open mgmt_socket
13:27:54 INFO [bt] BT Discovery:
After some time, the communication is running.

Include exact BMS model name
Query HW and SW numbers from the BMS app
Enable verbose_log

Paste log output between BEGIN and END:
BEGIN

END

Hi.
no answer or comment?
The error is still visible.
BR
Andreas

I'm also getting Unable to open mgmt_socket — none of the bluetoothctl commands seem to do anything.
I think this used to work like a week ago, but rebooting the system doesn't fix it, and neither does unplugging and replugging the BT dongle.


Error on the home page after loggin out as a system admin

Describe the bug
I got an error on the home page after loggin out as a system admin
To Reproduce Steps to reproduce the behavior:
I don't exactly. I was using Chrome, this time. I logged out for the user dashboard and then clicked on the lamp on the left corner. I scrolled down and saw the error.
OPTIONAL: Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: macOS 10.14.6 (18G103), French

Browser Chrome
Version 78.0.3904.97

Additional context
Add any other context about the problem here.

I could not be able to reproduce the error....


Unlinked Phone Number

We’ll solve this by putting the link HTML code in the content directly, so we don’t have to rely on our automatic “that looks like a phone number” code which fails in this case

I confirm this has been fixed!


CudaFDKConeBeamReconstructionFilter Execute not working, driver problem?

Hi,
I'm trying to use SimpleRTK to make a 3D reconstruction of cone beam projections. I'm using PyCharm (SimpleRTK was successfully builded and the package installed on Python). I succeeded with the CPU methods and am now trying to use the same code with the GPU using methods to make it faster (for example CudaFDKConeBeamReconstructionFilter instead of FDKConeBeamReconstructionFilter ). I received an error:
Traceback (most recent call last):
File "C:\RootReferences\OrsQtPlateform\python\OrsPythonPlugins\OrsReconstruction\mainform.py", line 155, in on_PushButton_apply_clicked
srtkImage = feldkamp.Execute(imageRecons, srtkData)
File "C:\Anaconda3\Lib\site-packages\SimpleRTK\SimpleRTK.py", line 7972, in Execute
return _SimpleRTK.CudaFDKConeBeamReconstructionFilter_Execute(self, *args)
RuntimeError: Exception thrown in SimpleRTK CudaFDKConeBeamReconstructionFilter_Execute: C:\Users\admin\Desktop\RTK-1.4.0\utilities\ITKCudaCommon\include\itkCudaDataManager.h:54:
Cuda Error : CUDA driver version is insufficient for CUDA runtime version.
I am relatively new to programming and can't seem to understand this error. Do you know what I should do to make the methods work? Is it a problem in my code or really a driver problem? I can publish a bit of my code if it helps.
Thank you,
Emmanuelle Richer

The problem was my Nvidia driver version, I updated it and now everything works!


[T4A5][F12-A3]Joel Lim Jing

Ready for Review

@funnieguy
Some comments added. Please ack & close the PR after reading comments.

ack


Add support for annotations to wkg oci push

This adds support for annotations to wkg oci push. Annotations can be added via a new --annotation key=value flag which can be repeated multiple times to add multiple annotations. For example this can be used to add a standard OCI description to the OCI image of the wasi:filesystem spec:
wkg oci push \
    ghcr.io/yoshuawuyts/wasi/filesystem:0.2.0 \
    ./webassembly_wasi_filesystem.wasm \
    --annotation org.opencontainers.image.description="the WASI filesystem spec"

Thanks!

tidied it up; I think this should be good now!


[ds8k] create pvc

This pr:

Add a DS8KArrayMediator to talk with ds8k storage system thru REST APIs.
Implement three methods create_volume, get_volume and validate_supported_capabilities to be able to create ds8k pvc.
Implement delete_volume

Note that

The value of pool field in storageClass should be a pool_id, since pool name is not unique in ds8k.
The max volume name length is 16 in ds8k, so it will be shortened when creating a pvc.

This change is

@oriyarde  @igorgonibm  thank you for your review, it really helps. But there are too many comments, if I missed anything, please let me know.

@igorgonibm ,  I have updated the code, you can check the source code, I don't know why you can not see some of the changes, maybe there are some problems in your browser or github config.
Btw, if you think the "Resolved conversations" are actually not resolved, you can mark it as "Unresolved" again so I can see it, thanks.

@igorgonibm please approve this pr if you have no further comments, thanks.

@oriyarde please approve this pr if you have no further comments, thanks.

@oriyarde please approve this pr if you have no further comments, thanks.

@27149chen please address all unresolved conversations (including outdated ones)

@oriyarde please approve this pr if you have no further comments, thanks.

@27149chen please address all unresolved conversations (including outdated ones)

I think there is only 1 open conversation left which I opened

@27149chen could we delete this branch?


The balance view needs more info

1- locked needs to have a link to locked balance docs
2- when you open the profile manager there needs to be references for the information of the locked balance as well

1- locked needs to have a link to locked balance docs

was added and now it redirects to https://manual.grid.tf/tfchain/tfchain.html?highlight=locked#contract-locking

however the balance need some padding because now its almost clipped by the taskbar above it.

2- when you open the profile manager there needs to be references for the information of the locked balance as well

Nothing was added in the profile manager.

After the changes with the locked tokens link, i believe it's safe to close the issue


[firebase_ml_vision] Custom model request

Hi,
I want to use my custom models on Firebase mlkit with flutter. Is there any plan to release this feature?
Thanks.

Duplicate of #401


Wrong example for prune command

--monthly=-1 option is probably missing **keep-**monthly prefix or this option it is not documented.
Also -1 option is not documented. Probably means "infinite number of backups".
# Keep 7 end of day, 4 additional end of week archives,
# and an end of month archive for every month:
$ attic prune /data/myrepo --keep-daily=7 --keep-weekly=4 --monthly=-1

# Keep all backups in the last 10 days, 4 additional end of week archives,
# and an end of month archive for every month:
$ attic prune /data/myrepo --keep-within=10d --keep-weekly=4 --monthly=-1

--monthly was the old way to specify (and is deprecated), --keep-monthly is the new way. so yes, this is a small doc bug.
-1 is documented:
"Specifying a negative number of archives to keep means that there is no limit."

I overlooked the info about negative number. I was looking for a value -1 :)
Thanks

@petacz maybe reopen until the wrong example is fixed.
There is https://github.com/jborg/attic/pull/308 but until it is merged into master (or fixed in other ways), it is not fixed.


Include a hie.yaml in the starter projects

It would be nice if the starter projects included a hie.yaml so that editors with Haskell support don't give warnings about defaulting to implicit-hie when the file is missing.

Closed by https://github.com/clash-lang/stack-templates/pull/23


Use CSS to massively speed up animation

The abstract is set out here. I've not looked at the viability of this for C3 yet, but Kahn Academy have got major performance enhancement using this method. Leveraging CSS animation to get full use out of the browser GPU rendering and other optimisations, while working with SVG.
This is related to #172, but it is a specific idea, so I thought it should be looked at as a specific issue, and then either adopted or closed by itself.
I might add, that I have had some serious issues with C3 and performance, in data heavy apps. While this isn't a problem for drawing an individual chart on a page. Pages with multiple complex charts need all the help they can get.
Anyway, hopefully this suggestions is interesting and educational, even if it isn't practical for C3 :-D

Thank you so much. I guess this is a good solution for bar animations. :+1:


There's a typo causing showAddPassControllerFromURL to crash

In index.js, line 20, _showAddPassControllerFromString should be _showAddPassControllerFromURL.

I'm on holidays right now so I can't change it, but feel free to do a PR!
Or ask @ jgfidelis who missed this call on a rename :)

@jgfidelis


Code Action - Create function

tried to look here and couldn't find it. Can't think that it wasn't mentioned before, so please forgive me if I just couldn't find it.
I would really like the capability to have the option to use a function and, if it's to available yet, have the option to create a function declaration based on the usage.
So, if I use:
a := SomeFunction(1,2);
And do right click - create definition, I would come up with
local procedure SomeFunction(int a; int b) : int;
begin
end;
same also when I highlight some code, I'd like to be able to refactor it into a new function. basically, the nice little helpers that you start to like when developing in other languages/ full VS

Thanks. What you describe are all good candidates for 'Code Actions' in VS Code. We haven't gotten to implementing them yet but it is something we want to do at some point.

I am searching for exactly the same features. Is there any news?

Hi! Please go ahead and post this to our Ideas forum at https://aka.ms/BusinessCentralideas, or vote up the idea if its already there. We're constantly monitoring top Ideas and will consider them for a future release.


Doesn't install phantomjs dependency (on DynamicScraper only)

It's a similar issue as this.
Tried doing npm install phantomjs and then using scraperjs, here's what happened:
var scraperjs = require('scraperjs');

// Static: runs just fine
scraperjs.StaticScraper.create('https://news.ycombinator.com/')
.scrape(function($) {
 return $(".title a").map(function() {
     return $(this).text();
 }).get();
}, function(res) {
 console.log(res);
})

// Dynamic: complaints about "phantomjs-node: You don't have 'phantomjs' installed"
scraperjs.DynamicScraper.create('https://news.ycombinator.com/')
.scrape(function($) {
 return $(".title a").map(function() {
     return $(this).text();
 }).get();
}, function(res) {
 console.log(res);
})

Your phantomjs installation is local. Can you install it globally and then try?
npm install -g phantomjs

Hi, I just that too. Still the same issue.
What is a proper way to install phantomjs? I've just read that it's not a proper Node module.

Alright solved it. It wasn't made clear how should phantomJS be installed. I wrote a little troubleshoot-doc for my own project:
PhantomJS is a dependency used the scraperjs web-scraper. It is not a Node module, thus its binaries must be downloaded from its site, and the path be exported using PATH on the terminal. Open up your bash profile with nano
nano ~/.bash_profile

and add the path for PhantomJS to it.
### for phantomjs
export PATH=/usr/local/phantomjs/bin:$PATH

If you like to code and run from the Sublime console, modify its Node build system to use shell_cmd instead of cmd, so that it runs the shell terminal from the console. Here the complete Node.sublime-build file:
{
	"shell_cmd": "node \"${file}\"",
	"selector": "source.js",
	"env": {
		"PATH":"/usr/local/phantomjs/bin"
	}
}

Yes, PhantomJS & CasperJS are not node modules, but they are installable using npm (I installed it in my machine with just npm).
npm is a package manager and is not limited to installing just node modules, if I'm not wrong.
Anyways, I'm glad that your issue is solved.


Speed up find_texture_pos_for_glyph()

See #67520 . This does NOT fix this issue, but it does help a bit. Indexing into the ptr() is about 4x faster than the Vector[] lookup.

Thanks!


Battery percent

Hi egzumer,
Someone's pointed out that the full charge lithium battery voltage is 8.4V (which is correct). But your little percent function is saying 8.15V is 100% full.
I've done a battery cal using a DMM, so the voltage the radio is saying is now correct (it was actually quite a bit out), that's assuming the battery on the back doesn't have any series diodes to the two gold terminals at the bottom of the battery (where I measure the battery voltage from).

Ciao 1of11 sul tuo firmware, l'icone della batteria rimane ferma ? le tacche non scendono?

This is on purpose not to rise question "why my radio doesn't charge to 100%?", and to have it showing nice 100% after USB charging finishes. The USB charger charges to 8.1-8.2 so it will never charge to the real 100%, but charging it to 8.4 from 8.15 adds very little to the charge state. The error it causes happens only at the highest state of charge and is minimal.
The battery terminals at the back show the real battery voltage, these are shorted to those on the radio side.

Ciao 1of11 sul tuo firmware, l'icone della batteria rimane ferma ? le tacche non scendono?

The battery notches are calculated differently, they use original implementation and their levels are written in the EEPROM. That is why they are different than percentage.

I'm open to changing this to the real state of charge of 8.4. I can do the modification. Just note the above. Also to add to that, barely any charger will charge the battery to 8.4V really. It usually charges until current drops to e.g. 50mA and switches of, immediately after that the voltage at the battery terminals drops to well below 8.4

No it's OK, you're totally right about the last couple 100 mV, leave as is - well for me anyway. I spose if the muggles gang together and send a written protest in, then we can still ignore em ;)

No it's OK, you're totally right about the last couple 100 mV, leave as is - well for me anyway. I spose if the muggles gang together and send a written protest in, then we can still ignore em ;)

LOL yes just ignore them please....


Implementation of Authorization store hashed secret

In this issue we will update the Token Servies to enable storing hashed secrets and not storing them in plain text.
Also, with this issue AuthorizationValidationExtensions should be updated to enable hashed tokens.

Merged with v3/main


Binding of isaac : Rebirth

From GOG, the save path I have is
<winDocument>/My Games/Binding of Isaac Repentance (Galaxy)

which is missing in the manifest.

Hi! In cases like this, you can add the missing paths on PCGamingWiki, and Ludusavi will automatically pick it up. I've updated the article for this game, so you'll see it recognized in Ludusavi soon :)


Genome-wide significance of summary stats SNPs

Hi,
Thank you for the amazing tool. I have just tested it on some samples and looks great. However, I was wondering why all SNPs from summary stats, irrespective of whether they pass the genome-wide p-value are used? How do we interpret the results and would this not affect our findings? The results don't look as good when I use only SNPs from summary stats that have p<5x10-8. Any suggestions regarding this?
Thanks and regards,
Prakrithi

Hi Prakrithi,
Thank you for your interest in our work. SNPs that do not pass the significance threshold (i.e., 5e-8) are not necessarily uninformative, as their individual effects may be too small to meet the stringent significance criteria in GWAS. gsMap performs partitioned heritability enrichment analysis, and our previous work has demonstrated that SNPs with minor effects (that do not pass the threshold) contribute significantly to the heritability of complex traits. For a more detailed explanation, please refer to the following link to our previous work. https://www.nature.com/articles/ng.608
Best regards,
Liyang

Thank you for the prompt response. I tried the tool with all SNPs and genome-wide SNPs from a summary stats and different genes and cell types show up as top hits. So was wondering what would be the best thing to do. I'll have a look at the paper you sent and decide what works best and get back to you if I need help.
Thanks again!
Prakrithi

No problem :smile:. Please feel free to raise an issue if you encounter any difficulties during gsMap analysis.


Bug in routing

Laravel Version: 5.4.19
PHP Version: 7.1

Description:
This issue is a bit related to the many previous discussions about leading/trailing slashes but differs. The bug even affects laravel.com. If the developer doesn't pay attention he might be a victim of duplicate content.
It's best explained by an example:
Route::get('foo', function () {
    return "Hello World";
});

Because the matching does care about leading/trailing slashes the following URLs works:
/foo
/foo/
The real issue is that the following example also works:
/////foo/////
So you can navigate to e.g. https://laravel.com////docs/5.4/routing/////
I would except the router to be more strict. I know an exact match implementation will be a breaking change so I suggest an opt-in solution by a config setting like router.strict=true|false.
By modifying a single line in the prefix method in \Illuminate\Routing\Router and a single line in the matches method in \Illuminate\Routing\Matching\UriValidator the bug is solved.
prefix method: return $this->getLastGroupPrefix() . $uri;
matches method: $path = $request->getPathInfo();
It also solves the long discussion about trailing slashes as it will be the developers choice per route:
Route::get('/foo', ...) will only be matched by /foo
Route::get('/foo/', ...) will only be matched by /foo/
My suggestions aren't really tested but just some quick thoughts to start a discussion.

Having two routes for /foo and /foo/ seems a bit too much for me...
IMO we could do the modifications you suggested and maybe create a check, if the url ends with / then create a 301 redirect to the same url without the leading /.
We should probably provide a way to disable this redirect however IMO that's the most common usecase...

My intention was not to support both /foo and /foo/ but just one of them. It was just an example to show how the new matching method would work.
I understand that some people might want to match /foo and /foo/ from a single route which I why I suggest a strict and non-strict matching implementation.
Personally, I would except a route to match exactly that I typed.

@fernandobandeira with redirects you could end up with TOO_MANY_REDIRECTS easily, make named route to one with trailing slash and you are in trouble (of course its up to programmer to make things right and this would hopefully never leave local development).
I don't see anything wrong with /////foo/////, users that want to tinker with URL will still tinker with it, maybe you can detect such thing and throttle it to "scare tinkers away" (I know dumb but it might work in some cases)

I solved my issues with a simple middleware with parameters specifying how trailing and duplicated slashes should be handled. Works much better and easier implemented!
I needed another solution as my suggested modifications to the router breaks route parameters.

Currently, I configure how slashes should be handled for each route group.
It would be nice if my middleware could choose the action based on the typed route. If I typed /foo/ then it new it should force the trailing slash.
But unfortunately, the router trims the slashes when adding the route which means my middleware doesn't know what I original typed.
I suggest a property on the route class and a method returning the original name - before trimming and compiling the route.
What are your thoughts?

I think your use-case (or whatever we are talking about) is really out of the reality.
If user is changing URL on purpose, you can not change that, just ignore it.

My concerns are SEO related. I'm not sure but these different URLs for the same content might be seen as Duplicate Content.
One with evil intentions might index these different URLs to harm your entire business. Personally, I do not want to take these risks.
It might sound as a user-specific case, but I bet it is not. Unfortunately, I can not prove the risk, but better safe than sorry.

Given that that's your concern, you can rest assured that many major websites (including this one) have the same pattern matching w/o any noticeable consequences. It would take a lot of effort on someone's part to try and ding your ranking with this, should it have any merit. I think it's safe to say, there are easier ways to hurt your business.

Personally, I do not want to take these risks.
Then don't. Include your own routing or verify the segments don't have multiple slashes etc and you win.

This shouldn't be messed with at a framework level.

I am curios how u done setting on  https://laravel.com////docs/5.4/routing///// I want my web to work the same result as this too.


Deadlink to https://flame-engine.org/docs/effects.md

Description
At the bottom of https://flame-engine.org/docs/components.md it links to https://flame-engine.org/docs/effects.md but it's deadlink. I believe it should point to https://github.com/flame-engine/flame/blob/master/doc/effects.md

@renancaraujo is this fixed on the new website?

@renancaraujo is this fixed on the new website?

I can help test the new site if there is a staging instances?

The link now works, but it doesn't set the scroll position to the top of the page seemingly.

@renancaraujo can you take a look on the anchor issue?

Tested on the new site and it is working now. Closing this.