python3Packages.pynetdicom: init at 1.5.5

Motivation for this change
Python implementation of the DICOM networking protocol
https://github.com/pydicom/pynetdicom
Related to #81418
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109448 run on x86_64-darwin 1

3 packages failed to build and are new build failures:

python37Packages.pynetdicom: log https://termbin.com/cxvd
python38Packages.pynetdicom: log https://termbin.com/194u
python39Packages.pynetdicom: log https://termbin.com/dm99

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109448 run on x86_64-darwin 1

3 packages failed to build and are new build failures:

python37Packages.pynetdicom: log https://termbin.com/cxvd
python38Packages.pynetdicom: log https://termbin.com/194u
python39Packages.pynetdicom: log https://termbin.com/dm99

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109448 run on x86_64-linux 1

3 packages built:

python37Packages.pynetdicom
python38Packages.pynetdicom
python39Packages.pynetdicom

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109448 run on x86_64-linux 1

3 packages built:

python37Packages.pynetdicom
python38Packages.pynetdicom
python39Packages.pynetdicom

Probably needs __darwinAlllowLocalNetworking = true;.

Probably needs __darwinAlllowLocalNetworking = true;.

Thanks, I added it but unfortunately I have no way to verify if it's working.

Result of nixpkgs-review pr 109448 run on x86_64-linux 1

3 packages built:

python37Packages.pynetdicom
python38Packages.pynetdicom
python39Packages.pynetdicom

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109448 run on x86_64-darwin 1

3 packages failed to build and are new build failures:

python37Packages.pynetdicom: log https://termbin.com/akhb
python38Packages.pynetdicom: log https://termbin.com/ljs4
python39Packages.pynetdicom: log https://termbin.com/10ld

That helped. We're down to two failing tests.

I excluded the two failing tests for darwin.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109448 run on x86_64-darwin 1

2 packages failed to build and already failed to build on hydra master:

python38Packages.pynetdicom: plain log  |  pretty log
python39Packages.pynetdicom: log was empty

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109448 run on x86_64-linux 1

1 package failed to build and are new build failure:

python38Packages.pynetdicom: plain log  |  pretty log

1 package built:

python39Packages.pynetdicom


Fix NPE when signing publication URLs

Publication URLs might be null. Signing them leads to a NPE.
Your pull request should…

[ ] have a concise title
[ ] close an accompanying issue if one exists
[ ] be against the correct branch
[ ] include migration scripts and documentation, if appropriate
[ ] pass automated tests
[ ] have a clean commit history
[ ] have proper commit messages (title and body) for all commits

Is this going to be a problem for the External API as well? It can sign publication URLs since #3002.

I tried testing if the external api has problems but could not reproduce it with querying one of the /api/events//{eventId}/publications/{publicationId}?sign={sign} Endpoints.. But I am not even sure if that would cause it to fail in the first place.
I found this line in your code @KatrinIhler https://github.com/opencast/opencast/pull/3002/files#diff-606d2a3ea1a0603656eb303a059677b604f3bb095b7f81ec97519869f5a751d0R1620 which I would assume prevents this from being a problem, looks kind of similiar to the fix Matthias provided.

@mliradelc: I wondered as well and looked at the event endpoints. The getSignedUrl method actually has a null check so it should not be affected.


Added a checklist PULL_REQUEST_TEMPLATE.md

Summary
Hey guy I have added a checklist to make contributors more aware of CLA and Contributing guidelines. With this checklist, Kong contributors can review all the necessary requirements and reviewer can see whether the checklist is complete, prioritise which PRs to review first (as the number of checklist items appears next to PR on the PR page), and can double check the CLA.
Preview

Checklist:

[x] I have signed the Kong Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the guidelines in CONTRIBUTING.MD document.
[x] All new and existing tests passed.

I haven't edited any code or any file other than PULL_REQUEST_TEMPLATE.MD and I'm not sure why the Travis CI build failed.

Hey guys, I have just edited the PULL_REQUEST_TEMPLATE.MD file and I'm not sure why the Travis CI build is failing. This change affects GitHub and does not affect the codebase.

Hi,
This checklist includes items that are redundant with the usual PR flow: CI results and CLA signing are already automated as part of the PR's UI, and the CONTRIBUTING.md file is already mentioned in the PR template. Thank you, but this checklist does not seem necessary, unless maybe if revised to be more relevant to this project's contribution workflow.

Hi @thibaultcha,
If there any revisions, then please let me know.

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

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

Closing this due to lack of activity/interest for the time being.


Add tag for v0.0.3

Please could you add a tag for v0.0.3 so that I can get the latest fixes via bower.

Sorry, forgot to push the tag.  You should be good to go.


Don't use Merge.ensureOnHandler in pre Union Merge

For the merge which might be necessary before the union merge, we don't need to
use the Merge.ensureOnHandler method because execution on handler is not
required. However, the union merge afterwards will be performed on the handler.

Codecov Report

Merging #6644 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #6644      +/-   ##
============================================
+ Coverage     83.03%   83.05%   +0.01%
- Complexity    13371    13372       +1
============================================
  Files          1523     1523
  Lines         49698    49714      +16
  Branches       5938     5938
============================================
+ Hits          41269    41289      +20
+ Misses         5602     5594       -8
- Partials       2827     2831       +4

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/io/crate/planner/operators/Union.java
90% <100%> (+2.16%)
18 <0> (ø)
:arrow_down:

.../main/java/io/crate/analyze/AnalyzedStatement.java
0% <0%> (-100%)
0% <0%> (-1%)

...r/transport/executionphases/SetBucketCallback.java
66.66% <0%> (-33.34%)
2% <0%> (-1%)

...o/crate/analyze/SafeExpressionToStringVisitor.java
83.33% <0%> (-16.67%)
7% <0%> (-1%)

...rc/main/java/io/crate/analyze/UnboundAnalyzer.java
76.31% <0%> (-10.53%)
2% <0%> (ø)

...ate/operation/merge/RamAccountingPageIterator.java
94.44% <0%> (-5.56%)
8% <0%> (-1%)

...otocols/postgres/RetryOnFailureResultReceiver.java
88.46% <0%> (-3.85%)
10% <0%> (-1%)

...io/crate/planner/operators/GroupHashAggregate.java
90.74% <0%> (-3.71%)
19% <0%> (-1%)

.../operation/collect/sources/ShardCollectSource.java
82.82% <0%> (-3.54%)
29% <0%> (-3%)

...rc/main/java/io/crate/blob/BlobTransferTarget.java
81.09% <0%> (-2.44%)
25% <0%> (ø)

... and 21 more

Continue to review full report at Codecov.

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

We decided to replace this patch with a comment in the addMergePhase method to state that the union merge will always be performed on the handler node which means that we don't really gain much from doing the pre-union merge on a different node.

Opening a new PR for the comment.


Wrong postgresql error messages because of switch to jackc/pgx

Description
jackc/pgx behaves differently compared to the previously used driver lib/pq. Before upgrading a DB error was reported like:
pq: duplicate key value violates unique constraint ...
After upgrading the same error is reported like:
failed to close statement: ERROR: current transaction is aborted, commands ignored until end of transaction block (SQLSTATE 25P02)
This makes it very hard to find the cause of the error.
Steps to Reproduce the Problem
For example: create the same object (model with same uuid) twice
Expected Behavior
Error should be: pq: duplicate key value violates unique constraint ...
Actual Behavior
Error is: `failed to close statement: ERROR: current transaction is aborted, commands ignored until end of transaction block (SQLSTATE 25P02)``
Info
Pop version v5.2.0

Maybe the causing error should be reported and not the error of stmt.Close:
https://github.com/gobuffalo/pop/blob/4196733eb565453ad7d66b39e7c49162391bcd44/dialect_common.go#L86-L92

Fixed with #569.


updated coffee to 1.9

Meh - why does this break? On my windows machine, I get about 11-13 breaking tests no matter whether I choose coffee 1.7.1, 1.8.0 or 1.9.0

Now I have tested this more extensively on Ubuntu, where 1.8 works fine, but 1.9 introduces issues related to indentation - I am afraid I do not yet have a good enough grasp of this library's inner workings to understand why. Could someone help me take a look?

:+1:

@clutchski any pointers?

This seems to be a bug in coffeescript introduced in 1.9. See https://github.com/jashkenas/coffeescript/issues/3822.

@alubbe I'm the current maintainer of CoffeeLint. I haven't had much time to look at this, the upgrade just hasn't been a big priority for me. As far as I can tell you only need 1.9 if you're using generators.
I see that CoffeeScript issue was opened by @swang, one of the top CoffeeLint contributors. I think we'll probably wait for an upstream fix, hopefully in 1.9.1.

I'm slowly working through the problems I've seen. The above mentioned issue has been fixed, however some code handling for operator spacing is not yet fixed. I've submitted a PR (merged) jashkenas/coffeescript#3830 and also submitted another issue with extra/duplicate string interpolation tags jashkenas/coffeescript#3831. Once these issues get fixed I'll put another run-through of the tests. These should be the last issues before we can implement linting for 1.9.x

thanks for your effort @swang!

You might be interested in that since jashkenas/coffeescript#3840 strings and regexes with interpolations produce different tokens than before:

Strings: Not wrapped in ( and ) anymore, but STRING_STARTand STRING_END. No more .stringEnd property.
Regexes: Now wrapped in REGEX_START and REGEX_END. No more .regexEnd property.

Ah thanks for the heads up! I actually had a rule fix based off stringEnd. Will take a look at it later tonight.


How do I install this?

Hi,
I don't understand how to install this,
Thanks

You pick the version you want from here
For example the default version here
And copy the contents of that file into your webstore's theme section.


false positive

Please don't block nameservers like "ns[1-4].msft.net" . These are among others for Microsoft domains responsible.
And why do you block "us.gov" ? Makes also no sense to me...

This list contains the NSA Blocklist therefore us.gov is blocked. Please whitelist that if needed.
I'll consider the handling "ns[1-4].msft.net" domains.


Currency showing fixes.

Always display the first group.
Use precision when displaying fractional parts.
Zero showing with fractional.
Fixed loss digits on fractional part.

@dkulyk, thanks for the PR!
Soon, the changes will be published in a stable version.

Released 5.0.2 version


Style sheet does not support font face

Hello,
it seems to me that the @font-face is not supported during pdf export..
I have created a custom style sheet with font definition as seen below, but I was not able to make it work. The file "Rajdhani-Regular.ttf" is placed in the same directory as css.
I had to install the font in my windows 10 machine. Then it worked. Same behaviour observed in iOS.
Is there something wrong?
Thanks for your help. Marek
@font-face {
font-family: "RajdhaniRegular";
src: url(Rajdhani-Regular.ttf) format("truetype");
}
html, body {
background: #FFF;
font-family: "RajdhaniRegular", Rajdhani, Courier, freesans, sans-serif;
font-size: 10px;
line-height: 1.4;
color: #333;
word-wrap: break-word;
text-align: justify;
hyphens: auto;
}

Currently, paths in CSS are not resolved. I solve this in v2. In v2, the paths within CSS @import and url(*) are resolved and converted into base64 data uri's for maximum compatibility.

Hello, sorry, but I will no longer maintain the project.
This project is archived because the owner no longer uses the Atom Editor.
It would be very good to find a new owner for this project, who will take care of updating and expanding the Atom package. A new version based on Chrome Headless has already been prepared in a feature branch.


JAX Concretization error when jitting sinkhorn with non-static tau_a, tau_b

Describe the bug
Attempting to jit the sinkhorn algorithm for non-static tau_a or tau_b (i.e. unbalanced OT) throws a concretization error, as the solver uses the two parameters to assess if the problem is balanced (via self.tau_a == 1.0 and self.tau_b == 1.0, see the line here).
To Reproduce
Steps to reproduce the behavior:
import ott, jax

def mysink(x, y, tau_a):
    geom = ott.geometry.pointcloud.PointCloud(x,y)
    prob = ott.problems.linear.linear_problem.LinearProblem(geom=geom,tau_a=tau_a)
    return ott.solvers.linear.sinkhorn.Sinkhorn()(prob).reg_ot_cost

x = jax.random.uniform(jax.random.PRNGKey(0), (12,3))
y = jax.random.uniform(jax.random.PRNGKey(1), (16,3))

# this works fine, but recompiles at every call
mysink(x,y,1.0)

# this will cause a concretization error when the solver calls ot_prob.is_balanced
jax.jit(mysink)(x,y,1.0)

# a workaround is having tau_a as a static argument, but causes recompilation for different values
jax.jit(mysink, static_argnums=(2,))(x,y,1.0)

Full Stacktrace

    Click to expand

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<stdin>", line 4, in mysink
  File "/Users/ahaegele/Projects/ott/src/ott/solvers/linear/sinkhorn.py", line 833, in __call__
    return run(ot_prob, self, (init_dual_a, init_dual_b))
  File "/Users/ahaegele/Projects/ott/src/ott/solvers/linear/sinkhorn.py", line 1086, in run
    out = iter_fun(ot_prob, solver, init)
  File "/Users/ahaegele/Projects/ott/src/ott/solvers/linear/sinkhorn.py", line 1123, in iterations
    state = fix_point(
  File "/Users/ahaegele/Projects/ott/src/ott/math/fixed_point_loop.py", line 92, in fixpoint_iter
    _, state = jax.lax.while_loop(max_cond_fn, unrolled_body_fn, (0, state))
  File "/Users/ahaegele/Projects/ott/src/ott/math/fixed_point_loop.py", line 80, in unrolled_body_fn
    iteration_state, _ = jax.lax.scan(
  File "/Users/ahaegele/Projects/ott/src/ott/math/fixed_point_loop.py", line 76, in one_iteration
    state = body_fn(iteration, constants, state, compute_error)
  File "/Users/ahaegele/Projects/ott/src/ott/solvers/linear/sinkhorn.py", line 1111, in body_fn
    return solver.one_iteration(ot_prob, state, iteration, compute_error)
  File "/Users/ahaegele/Projects/ott/src/ott/solvers/linear/sinkhorn.py", line 946, in one_iteration
    err = jax.lax.cond(
  File "/Users/ahaegele/Projects/ott/src/ott/solvers/linear/sinkhorn.py", line 948, in <lambda>
    lambda state, prob: state.solution_error(
  File "/Users/ahaegele/Projects/ott/src/ott/solvers/linear/sinkhorn.py", line 79, in solution_error
    return solution_error(
  File "/Users/ahaegele/Projects/ott/src/ott/solvers/linear/sinkhorn.py", line 167, in solution_error
    if ot_prob.is_balanced and not parallel_dual_updates:
  File "/Users/ahaegele/Projects/ott/src/ott/problems/linear/linear_problem.py", line 79, in is_balanced
    return self.tau_a == 1.0 and self.tau_b == 1.0
jax.errors.ConcretizationTypeError: Abstract tracer value encountered where concrete value is expected: traced array with shape bool[].
The problem arose with the `bool` function.
The error occurred while tracing the function <lambda> at /Users/ahaegele/Projects/ott/src/ott/solvers/linear/sinkhorn.py:948 for cond. This value became a tracer due to JAX operations on these lines:

  operation a:bool[] = eq b c
    from line /Users/ahaegele/Projects/ott/src/ott/problems/linear/linear_problem.py:79 (is_balanced)

See https://jax.readthedocs.io/en/latest/errors.html#jax.errors.ConcretizationTypeError

Expected behavior
Generally, I do not think it should be needed to have tau_a or tau_b be static. The is_balanced seems to be mostly used as a way to speed up some computations (but maybe there are some places where there is a more intricate switch between the problems, e.g. GW). Having to define tau_a as a static argument causes recompilation every time the parameter is changed.
Additional context
Another fix could be to have the user define a boolean argument is_balanced=True to the problem such that no dynamic check is be needed. This would only cause compilation for True and False, if needed.
I'd be happy to look into this myself and possibly open a PR depending on what you think.
Thanks!

Currently, tau_{a,b} cannot be traced. Is there any reason you want these to be traced, i.e., you wanna differentiate w.r.t. to tau_{a,b}?

Having to define tau_a as a static argument causes recompilation every time the parameter is changed.

How often do you need to recompile in your use-case?

Another fix could be to have the user define a boolean argument is_balanced=True to the problem such that no dynamic check is be needed. This would only cause compilation for True and False, if needed.

I feel this would be really error-prone; what's possible would be to allow tau_{a,b} to be both static or dynamic (checking this at compile time with jax) and then use either Python if or jax.lax.cond (this is more complicated that it being just static/dynamic).

In my use case, I do not actually want to differentiate w.r.t. tau_{a,b}. I was investigating the effect of the two variables on the result of Sinkhorn; trying to use jit and different values led me to the problem above.
I agree, the jax.lax.cond way is probably the most rigorous one. I just thought this would require a more involved rewrite and might (unnecessarily?) complicate the backend.
In any case, this is not a very pressing issue. Maybe we can keep it in mind and introduce the dynamic version when there is more need for it in the future.

I think it's good to keep in mind that the tau_a and tau_b will need to be kept static, maybe it's simpler this way for now. I can imagine use cases where one might want to differentiate against them though... we can revisit this then


Add a changelog

This PR fixes #29 by adding a changelog. A few notes:

Following the Keep a Changelog format with a few deviations

Rather than the default Keep a Changelog categories, follow AppDaemons's simpler categories of Features, Fixes, and Breaking Changes.

Explicitly call out Semantic Versioning
Only retroactively document back to v2.0.0. It feels like diminishing returns on the time invested past that.

As always, open to suggestions and changes :).

😍
Not sure if this is going to work, but I'll try to automatically resolve the associated issue: closes #29

(oh you already had done the 'fixes #29' trick, I hadn't noticed 😁)


Update ASIC numbers in introduction documentation

https://ibex-core--997.org.readthedocs.build/en/997/introduction.html#asic-synthesis has some ASIC numbers which are mostly outdated and don't match the ones we have in the README in the repository. Make these number consistent, or remove them.

This was fixed by #1065


More logging on updates

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 4 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 29.134%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

background/refreshrelease.go
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

background/refreshrelease.go
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
791

Relevant Lines:
2715

💛  - Coveralls


Fixed a few issues reported by "golint"

Summary
Fixed reported issues for add/subtract by 1, error formatting and improper casing:

Add / Subtract by 1 are replaced with increment and decrement respectively
Changed a single usage of errors.New(fmt.Sprintf(...)) to fmt.Errorf(...)
Changed variable names containing "Ip" and "Id" to have them in upper case

Reported issues fixed in this Pull Request
Add / Sybtract by 1:

hack/netpol/pkg/main/main.go:192:4: should replace failCount += 1 with failCount++
pkg/agent/cniserver/pod_configuration.go:152:6: should replace count += 1 with count++
pkg/agent/openflow/network_policy.go:824:3: should replace nClause += 1 with nClause++
pkg/agent/openflow/network_policy.go:828:3: should replace nClause += 1 with nClause++
pkg/agent/openflow/network_policy.go:832:3: should replace nClause += 1 with nClause++
pkg/agent/openflow/network_policy_test.go:295:4: should replace count += 1 with count++
test/e2e/framework.go:170:4: should replace workerIdx += 1 with workerIdx++
pkg/agent/openflow/network_policy.go:611:4: should replace expectedConjunctiveActions -= 1 with expectedConjunctiveActions--
pkg/agent/openflow/network_policy.go:618:3: should replace expectedDenyAllRules -= 1 with expectedDenyAllRules--

errors.New(fmt.Sprintf(...)) usage:

test/e2e/providers/remote.go:41:15: should replace errors.New(fmt.Sprintf(...)) with fmt.Errorf(...)

Improper casing for Ip and Id:

pkg/agent/cniserver/server_test.go:68:14: var containerId should be containerID
pkg/agent/cniserver/server_test.go:310:2: var containerIp should be containerIP
pkg/agent/cniserver/server_test.go:467:2: var containerId should be containerID
pkg/agent/cniserver/server_test.go:487:2: var newId should be newID
pkg/controller/networkpolicy/networkpolicy_controller_test.go:1556:2: var expIpNet should be expIPNet
test/e2e/basic_test.go:450:9: range var externalId should be externalID
test/e2e/basic_test.go:451:3: var externalIdArray should be externalIDArray

LGTM. Thanks @theshubhamp for the cleanup!
/test-all

The failure was not related, tracked via #481.
/test-e2e

Thanks @tnqn! Only folks with write access can merge, can you do that please ?

Sure @theshubhamp , I was waiting for CI reports as well as other reviews. Since it is clean enough, I will just go ahead.


Spotify playback issues

What version of Music Assistant has the issue?
2.3.0b17
What version of the Home Assistant Integration have you got installed?
No response
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
When attempting to play any song on spotify, it instantly skips to the next one until there is nothing left in the queue.
How to reproduce
Install beta or non beta version using the addon store in home assistant, link spotify and a player the attempt to play spotify.
Music Providers
SPOTIFY
Soundcloud works but attempting to skip 30seconds forward results in the song being restarted.
Player Providers
Can only test snapcast. Unsure if happens with other providers.
Full log output
music-assistant.log
Additional information
Running on VMWARE WORKSTATION PRO 17.5.2 build-23775571
Host: Windows 11 23H2 build 22635.4145
What version of Home Assistant Core are your running
2024.8.3
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Alternative

Duplicate https://github.com/music-assistant/hass-music-assistant/issues/2860


Adding more types.

为大多数类型为 any 的值赋予一个可识别的类型。
Assign a recognized type to the value of most types that are any.

@GongCx-Github Please make sure you remove any code style changes

@m4tt72 Sorry to reply so late. I will remove the code style changes and add improvements to the handler of commands in another pull qequest, so I will close this pr first.


Use map exporter for visualization backup

The current visualization backup exports the vizjson on visualization deletion. How about changing it so it saves the map export instead? This includes a lot more information and opens the possibility for restorations (albeit with different ids).
What do you think? @juanignaciosl @gfiorav

I'd certainly go for it, but the ids might be an issue. Exporter removed ids on purpose, but for a plain backup keeping them might make sense.

Closing automatically because of inactivity. If anybody considers that this still important, please reopen and it'll be prioritized.


Feature/effective transient cleanup

Ticket:
Issue
#2280
Solution
Change the delete query in lib/Loader.php to include timeout rows
Impact
That query will just delete more rows so it might be slower.
Usage Changes
Maybe someone is clearing those rows by themselves that is unnecessary from now on.
Considerations
I have not found any considerations but I'm sure you'll find when reviewing
Testing
I had some problems seting up the testing environment. Also I'm not sure if you're doing database tests or if you want them.

Codecov Report

Merging #2281 into master will decrease coverage by 1.67%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #2281      +/-   ##
============================================
- Coverage     94.66%   92.99%   -1.68%
+ Complexity     1587     1552      -35
============================================
  Files            50       50
  Lines          3862     3838      -24
============================================
- Hits           3656     3569      -87
- Misses          206      269      +63

Impacted Files
Coverage Δ
Complexity Δ

lib/Loader.php
95.15% <100.00%> (-0.59%)
68.00 <0.00> (ø)

lib/Site.php
49.42% <0.00%> (-37.94%)
23.00% <0.00%> (ø%)

lib/Cache/Cleaner.php
66.66% <0.00%> (-24.64%)
6.00% <0.00%> (ø%)

lib/Image/Operation/Resize.php
87.96% <0.00%> (-10.19%)
33.00% <0.00%> (ø%)

lib/Timber.php
88.78% <0.00%> (-6.55%)
34.00% <0.00%> (ø%)

lib/Twig.php
96.68% <0.00%> (-2.81%)
55.00% <0.00%> (-32.00%)

lib/Core.php
84.61% <0.00%> (-2.57%)
24.00% <0.00%> (ø%)

lib/Post.php
94.84% <0.00%> (-0.97%)
245.00% <0.00%> (-3.00%)

lib/ImageHelper.php
95.69% <0.00%> (-0.36%)
89.00% <0.00%> (ø%)

... and 5 more

Continue to review full report at Codecov.

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

Coverage decreased (-2.2%) to 91.543% when pulling 920bcfcb5ddea2413dea0565c53f095159d066ad on opengeekv2:feature/effective-transient-cleanup into 5b23f146e510a35b2deae645fc63d1bc8a4e3545 on timber:master.

Coverage decreased (-2.2%) to 91.543% when pulling 920bcfcb5ddea2413dea0565c53f095159d066ad on opengeekv2:feature/effective-transient-cleanup into 5b23f146e510a35b2deae645fc63d1bc8a4e3545 on timber:master.


Where to put this example?

This is a follow-up from the discussion which took place in the PR Issue #29.
I've completed a new example file and am trying to decide where this fits in the dm_usecases set.
I think it'd fit nicely into the "combined_data" case, whose primary goal appears to be in the AssociatedData relation (with WebEndpoint).  This case covers the other 2 flavors, and could fit into the case description.. data obtained from cross match (Source, Detections with assoc. LightCurves) can be combined with similar from other catalogs to generate combined LightCurves.
My 'implementation' of this is merely to summarize the content, show that everything is accessible.
thoughts?
The example illustrates:

Multiple table usage

TimeSeries in Master table, with data from Detection table.

Different flavors of AssociatedData

VOModelInstance and AssociatedMangoInstance

Annotation to different models

The same data is annotated to difference model 'root' elements (mango:Source and cube:SparseCube).  This would be a bit more clear if the TimeSeries relation was a reference.

Raw file:  (modification of standard_properties case - Chandra file)

Master Source table: (one record per source)

identifier, position, significance, extent, variability

Detections table: (one per observation)

identifier, position, time, flux, hardness_ratios

These are mapped to model instances as:

Mango instance - Master Source
Source.identifier         => name
Property (meas:Position)  => (gal_l,gal_b)
Property (meas:Generic)   => significance
Property (mango:FLAG)     => extent_flag
Property (mango:FLAG)     => var_inter_hard_flag
AssocData (mango:Source)  => Detections
AssocData (cube:SparseCube)=> TimeSeries

Mango instance - Detection
Source.identifier         => name
Property (meas:Time)      => o.gti_obs
Property (mango:Photometry)=>o.flux_aper_b
Property (mango:HR)       => o.hard_hs|lolim|hilim
Property (mango:HR)       => o.hard_ms|lolim|hilim
Property (mango:HR)       => o.hard_hm|lolim|hilim

Cube - Light Curve of Detection data for each source
Observable (meas:Time)    => o.gti_obs
Observable (mango:Photometry)=>o.flux_aper_b

combined data is the good place for your dataset.
Please put the raw VOTable  in raw_data  and update the README


Windows 10 Driver for Pixel kit

Trying to connect Pixel kit to Window 10 machine. Device Manager shows the Pixel kit as FT231X USB UART for which there are no drivers available.  Doing a little digging, it appears I should be installign drivers for the Ftdi chip from here.
https://www.ftdichip.com/Drivers/VCP.htm
Can anyone confirm if this is accurate or if there's another method I should be using for loading the drivers?
[update]  I was able to successfully install the drivers above and Windows 10 now sees the device as a "USB Serial Converter"
thanks!

Have you tried to install Kano Code as described on before you start requirements?

Ouch. Is my face red?  ( I think we can close this for now )

Don't be ashamed, @netmanchris! It's a silly step that can be skipped if you are willing to install the FTDI drivers yourself. I just added there because teachers and beginners (and myself most of the times) tend to have difficulties installing drivers and Kano Code does it automagically :D
Feel free to close the issue or add any more questions.

Hey @netmanchris, do you believe this issue is solved? :)

Yes - Sorry!  closing now.


Why do the RefSeq genes say there is no corresponding PATRIC feature, when there is?

Look at the two feature pages for Rv0667.  From the PATRIC side, I can see that it has a peg id.  If I click on the RefSeq id (which users would be inclined to do, as it has more meaning for them than the peg id), they would see that there is no corresponding PATRIC feature.

This is fixed on beta. Same issue to #594
https://www.beta.patricbrc.org/portal/portal/patric/Feature?cType=feature&cId=RefSeq.83332.12.NC_000962.CDS.759807.763325.fwd

Actually, this is another problem. This is due to the fact that subquery to look up PATRIC feature does not work.
https://www.patricbrc.org/api/genome_feature/?and(eq(pos_group,"NC_000962%3A2727339%3A%2B"),eq(feature_type,CDS),eq(annotation,PATRIC))
returns null even though there is a corresponding PATRIC feature.
http://harad.vbi.vt.edu:8983/solr/genome_feature/select?q=pos_group:"NC_000962:2727339:%2B"+AND+feature_type:CDS+AND+annotation:PATRIC&indent=true

Dustin, fixed data api call and now it seems working fine on beta.
https://www.beta.patricbrc.org/portal/portal/patric/Feature?cType=feature&cId=RefSeq.83332.12.NC_000962.CDS.759807.763325.fwd
This will be deployed to production at the bug-fix release.

I see this issue again on production.

Fixed on beta.


Init Containers

I am having an issue whenever a user is created the same pod gets initialized to the user instance and so I am unable to create a new pod for the particular user. I need different pods for different users.
There was no limit range for containers. The requirement is I need to assign container to the pods.

/kind support
/retitle Advice on using containers

@dolamaiti, GitHub is not the right place for support requests.
If you're still looking for help, check Server Fault.
You can also post your question on the Kubernetes Slack or the Discuss Kubernetes forum.
/close


train.py file not found in git repository

I want to train the BERT+Transformer model, but cannot locate train.py file in the repository. Neither am I able to locate preprocess.py to preprocess the dataset.

Solved! All code is located in the 'src' directory


simpleaichat is in active development: A note on update cadence

There have been a few issues asking if the project is dead so I'm pinning an issue here on the matter:
simpleaichat is in active development, and it is not dead or abandoned. A low amount of new commits does not imply the project is not being maintained. Any breaking issues are fixed immediately and rapidly, and there haven't been issues upstream lately.
However, simpleaichat is designed to be simple, and will not add unnecessary features for the sake of adding unnecessary features like other AI projects do to show proof of relevancy.
The primary motivator for new features is from my personal experience and needs: that way I'm positive others will get value from it. I still encourage and appreciate suggestions and bug fixes! If you are unsure if a pull request would be a significant value-add, please file a GitHub Issue first.

Thank you.

simpleaichat is designed to be simple, and will not add unnecessary features for the sake of adding unnecessary features like other AI projects do to show proof of relevancy.

Appreciate your coding philosophy.
For those who read this issue later, it is recommended to take a look at this post: Please Don't Ask if an Open Source Project is Dead | Max Woolf's Blog


Karuulm lighting

Adds dynamic lighting to the Karuulm sulphur pools, vents and furnace - both outside and inside the volcano

Will post some pics but i am looking into the scene/environment lighting as a whole to make them look at their best

Rebased


Updated project admin pages

Breaking the admin page up into separate pages, organized by tabs.  This is prep for "Bot Secrets".

Coverage decreased (-0.3%) to 94.269% when pulling 215d66fd19f745539bc46372577dde2c488176ae on cody-admin-page-sections into edab4f7bf5e8fd97e912c00a2f707519adc94d0e on main.


Superpoint generation

Hello @drprojects,
Can the superpoint generation be replaced with a custom superpoint generation method, and then use your proposed superpoint_transformer for semantic segmentation?

Yes, you can use another partition strategy. We are biased towards the formulation we peopose in the paper because it offers certain properties we judge desirable:

no prior on the number of superpoints
can be adapted to pretty much any pointwise features
encourages superpoint contours with "simple" shapes
does not produce non-connex superpoints

If you have another partition method, you can use it instead of our cut-pursuit formulation, as long as you express it in the expected format. For that you will need to get familiar with our data formats.
Howecer, note that one of the main contributions of our Superpoint Transformer paper is the design of a hierarchical superpoint partition. So your partition algorithm will also need to be hierarchical. See our ICCV'23 paper for more details.


Backport "Fix an infinite read loop with SRV record resolution on windows" to 1.36.x

Backports #25672

Merge?

interop: https://github.com/grpc/grpc/issues/23055 and https://github.com/grpc/grpc/issues/25660


add hanging-punctuation

https://caniuse.com/?search=hanging-punctuation
https://chriscoyier.net/2023/11/27/the-hanging-punctuation-property-in-css/
for blockquote and related elements

nice! like adding this to normalize is what you're thinking?
blockquote {
  hanging-punctuation: first last;

  &::before {
    content: open-quote;
  }

  &::after {
    content: close-quote;
  }
}


Plot graphs using relative time

Problem description
Sometimes it is preferable to plot lightcurves using relative time (from the start of observation).
It'd be great if the supplied plotting methods (plot(), scatter(), etc.) allow such options.
Example
# get the lightcurves for a given target
lcf_coll = lk.search.search_lightcurvefile(target='TIC232612416', mission='TESS').download_all()

lc = lcf_coll[0]

# new parameter for relative time
lc14.PDCSAP_FLUX.plot(use_relative_time=True)

The output would be something like this:

Expected behavior

Environment

lightkurve version (e.g. 1.0b6): 1.3.0

Thanks for the suggestion @orionlee!  :+1:
I guess what you describe is possible right now by doing this:
lc.time -= np.min(lc.time)
lc.plot()

Some questions that pop up in my brain:

Is this a sufficiently common operation to warrant a dedicated keyword?
Is it OK to encourage users to plot light curves this way? (information is removed)
Can you list a few specific examples where this would clearly be handy?

(PS: We always try to come up with arguments against adding a new feature, because new features require extra testing+maintenance, require users to read more docs, and are difficult to remove once added.  Of course we do want to add features if they are useful, so please do argue in favor if you think this keyword would make Lightkurve better!)

I started using lightkurve in conjunction with Planet Hunters TESS, so the use cases might be slightly different from those of the professionals.

Planet Hunters TESS displays lightcurve in relative time (example). So it'd be more convenient to plot graphs in the same way for comparison: less mental conversion.
Sometimes I plot the lightcurves of same target from multiple sectors, as multiple subplots stacked vertically. I find using relative time is more convenient in such case.

Regarding information loss: a mitigation could be displaying the start time as part of x-axis title.

If it's decided the use cases warrant some support from lightkurve, an alternative to adding new parameters to different charting functions would be to add a function to LightCurve.use_relative_time(enabled) .  However, I fear it might have some unintended side effects, e.g., should setting to relative impact non-charting functions?

I'm inclined to close this issue for now because we are running out of person-power to support new features, and I believe we support plots in relative time via e.g.:
lc.time -= np.min(lc.time)
lc.plot()

Please do re-open this issue if anyone feels strongly that a change in Lightkurve is needed to accommodate different X axes labels. Thanks everyone! :+1:


Added a from_read() method to RWops

The ownership is a little squirrelly but this seems the best way, as far as I can tell.  The other option seems to be to make the RWops object actually hold on to a buffer.  Or just force everyone to implement this from scratch, but the lifetime implications actually get sort of tricky so it'd be nice to have a version for anyone to use.
Addresses issue #535.

Honestly it would probably be better to hook the read/seek/etc methods into the RWops function pointers directly somehow... Hmmm.

Thanks!


Updated README to clarify purpose and key features of Classiq.

PR Description
This pull request enhances the README for the Classiq repository by improving formatting, adding clarifications, and rewording sections for better readability and coherence. The changes focus on:

Improving the clarity of example explanations and steps for using the SDK and platform.

Fixing minor formatting issues such as consistent use of headings, bullet points, and code blocks.

Enhancing the flow of the documentation to better guide users through the process of creating and running quantum programs.

Some notes

[x] Please make sure that you placed the files in an appropriate folder

[x] And that the files have indicative names.

[x] Please note that Classiq runs automatic code linting, which may minorly alter some files.

[x] If you're familiar with pre-commit, you may run pre-commit install, and then at each commit, your files will be altered in a similar way

@Smoothengineer, thanks for helping us improve our README.md file!
I used your original idea and attention to detail and slightly refined it.
Please let me know when you have your thoughts on my suggestions.

Hello again @Smoothengineer!
I just merged your changes.
Thanks for the contribution!


[stable/kubernetes-dashboard] #7516 Allow HTTP connection

What this PR does / why we need it:
This PR allows service Kubernetes dashboard with unencrypted HTTP connections via a values flag. This allows it to be served from behind an ingress controller/proxy that do not support self signed certificates and that handle their own authentication/authorization.
We add a enableInsecureLogin flag, defaulting to false for backward compatibility/default security. Internally, we hard code the port to the default 9090 for HTTP, and expect the user to separately override the service.externalPort value if they want it to be served externally on port 80 instead of 443 when the enableInsecureLogin flag is true.
Which issue this PR fixes

fixes #7516

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/ok-to-test

I've manually merged the changes and they work great ! Thanks @mbarrien for the work. I hope it gets merged soon.

This looks great! Thank you !

/assign prydonius

Hello,
Could you rebase one last time and fix the conflicts?

/assign

@desaintmartin rebased and ready for approval

You also need to update the ingress to use either 443 or 80 depending on the case.

Just tested and it requires two changes : the one from Notes I just wrote and :
You also need to update the ingress to use either 443 or 80 depending on the case. (edit: I misread, the service port does not change. Isn't it misleading that a port using 443 serves unencrypted content?)

The problem is the 443 is coming from values.yaml, which to my knowledge doesn't support templating as described in https://github.com/helm/helm/issues/2492. Either I change out values.yaml in a non-backwards compatible way (need some concept of a null default value with logic in 2 places to default to the right port number?), or add a new .Values.service.insecureExternalPort and add different if/else logic. Both seem ugly. Preferences?

I understand. Let's keep it like this and one can still change the externalPort if he wants to.

/lgtm


Fix typos in README

pls let me fix hhhhh

@iunary

Blastoff.zone Airdrop: Win 0.1 ETH + $100 Bonus!

Hello crypto community! Blastoff.zone is offering an exciting Airdrop for the first 10 users! You can win 0.1 ETH plus a $100 bonus just by participating and connecting your wallet.
How to participate:

Click Connect Wallet to access the Blastoff.zone website.
Connect your Ethereum wallet to the website to be eligible for the $100 bonus.
Use the "Claim Now" method to receive your tokens.

Don't miss out on this amazing opportunity! Hurry and claim your prize before the 0.1 ETH runs out.
Remember, tokens will be distributed on a first-come, first-served basis. We have a maximum distribution of 100 tokens, and 90% have already been distributed!
maciej-spec, @malik224, @rheyjohn10, @Jircs1, @neilturner77, @JnTxxxY, @andylim690, @cprkrn, @KINGDRAGON69, @immense055, @cheetooooo, @DerionFaleneva, @ed1868, @blockchain-tech-developer, @ncross42, @gofunbox, @aliscco, @Tuolde, @finleyexp, @mustafat0k

Blastoff.zone Airdrop: Win 0.1 ETH + $100 Bonus!

Hello crypto community! Blastoff.zone is offering an exciting Airdrop for the first 10 users! You can win 0.1 ETH plus a $100 bonus just by participating and connecting your wallet.
How to participate:

Click Connect Wallet to access the Blastoff.zone website.
Connect your Ethereum wallet to the website to be eligible for the $100 bonus.
Use the "Claim Now" method to receive your tokens.

Don't miss out on this amazing opportunity! Hurry and claim your prize before the 0.1 ETH runs out.
Remember, tokens will be distributed on a first-come, first-served basis. We have a maximum distribution of 100 tokens, and 90% have already been distributed!
star1015, @Njabulo67, @VALERY222, @JJ81, @Kropo7ov, @eledir, @blockexp, @poyie979, @Eliatan444, @blueafoo, @MiracleZhang, @jianxiaoBai, @abuzant, @dhnbid8, @marcusscorpius, @ribook24, @nerdfiles, @faghani, @damboscolo, @Korotaew


chore：add more unit tests

@ethe

I don't know why this error is reported. Does the test file need a main method? I can execute it locally.

@ethe Hi

I don't know why this error is reported. Does the test file need a main method? I can execute it locally.

Hey @caicancai
each trybuild test file need a main function, cuz trybuild will compile each file as a independent project.
If the main function is missing, a compile error occurs by rustc.

@Kilerd Thanks for your answer. I just started learning Rust and it's not very clear. I will fix it later.

And I think the folder test is only for the compile test, not for unit tests. probably you need to move your test under src folder ?

Perhaps the examples file is a good place to do this.

it make sense, then I think we need to regroup the testfiles in test folder, and separate them into multiple sub folder, like integration_test and compile_test, to clarify.

it make sense, then I think we need to regroup test files in test folder, and separate them into multiple sub folder, like integration_test and compile_test, for clarification.

yes, good idea

Close this PR first, discuss where the unit test should be placed first, I will open an issue


fix(build): respect the entry of index path

fix #1190

@aleclarson You can choose Squash and merge button when merge pr, that only create one commit on master.

@underfin Yes. The commits weren't all related to the PR title, so I rebased instead.


Problems on GZIP content encoding with SOAP

Describe the bug
First, I'm trying CastleMock and it looks very promising for what we need.
I wanted to record/forward a soap clients requests and record them for mocking later. It works partially, because the response gets recorded but on the client, receiveing the response from CastleMock , I get an GZIP decompression error.
The magic number in GZip header is not correct. Make sure you are passing in a GZip stream. ---> System.IO.InvalidDataException: The magic number in GZip header is not correct. Make sure you are passing in a GZip stream.
If I edit the recorded response (mock) and delete the "Content-Encoding = Gzip" Header, and use the mock, it is working.
Even I'm not a developer, it sounds very similar to this one ERR_CONTENT_DECODING_FAILED gzip encoding which was solved for REST.
I'm sorry that I can't give source code snippets or WSDL, but I'm not allowed to by company confidential notice.
I'm using the latest CastleMock version 1.51 as docker container, running on Docker for Windows.
Thank you for all the work in CastleMock .

Hi @roemsi.
Thank you so much for reporting this issue Great report! I will take a look at it as soon as I can. I will have time over during Christmas, so I hope to be able to solve a couple of issues during that time. I will keep you updated.
Best regards,
Karl Dahlgren

Hi @roemsi.
I have added the same solution that we have applied to the REST implementation. It should hopefully do the trick. The fix will be part of the next release, which I hope can be released before the end of the year.
It would be very much appreciated if you could test and verify that the fix worked.
Thank you.
Best regards,
Karl Dahlgren

Hi @karldahlgren,
Wow, that was fast. Many thanks. I'll test it as soon as the version is ready (optimum as docker image), and of course I will give feedback.
Cheers,
roemsi

Hi Karl,
First sorry for the really late reply.
I tested it with 1.55 sadly I run in the same issue. Maybe it will be a good idea to have an option to manipulate outgoing http header in general, but for all status (forwarded and recording too) and all requests.
It could be like the "add header UI" but for a whole project, and instead of the trash can, a check mark ,if this one should be deleted.
Workflow:
You add a header name --> it appears in the table. Next to the name in the table, is a checkbox. If it is checked, castle is looking for this Header Name in the outgoing http header, and removes it.
Cheers,
Christian


Feature.implement lb ru tests 62

@mattgreene
What issues does this address?
Fixes #43 #62
...
What's this change do?
I added testing of list, update, and show for each of loadbalancer, listener, pool, member, and healthmonitor.    Unexpected behavior in the update invocation for healthmonitor forced me to temporarily comment out update and show test components for it.
Root cause analysis of the healthmonitor update failure is my highest priority.
Where should the reviewer start?
Run the test suite, and examine the logic in test_loadbalancer_CLUDS
Any background context?
CLUDS
C.reate
L.ist
U.pdate
D.elete
S.how
NOTE:   The neutron client supports one additional untested function for loadbalancer, that mentions agent a functional test of the method should be prioritized.

Make sure you file a git issue or make some note so the commented-out healthmonitor update test doesn't fall through the cracks.   We need to figure out if this is a product or test issue, in case we need file an issue for DE to tackle.


gemini dont work with several cores

I have a problem of multicore loading with gemini 0.20.1, grabix 0.1.8 and python 2.7.
With 4 cores the gemini load command don't work :
./tools/bin/gemini load --cores 4 -t VEP -v result.vcf.gz test_db.db
the error message :
Indexing /home/aurelien.beliard/varapp/result.vcf.gz with grabix. Loading 286164 variants. Breaking /home/aurelien.beliard/varapp/result.vcf.gz into 4 chunks. Loading chunk 0. Loading chunk 1. /bin/sh: 1: gemini: not found Loading chunk 2. /bin/sh: 1: gemini: not found Loading chunk 3. /bin/sh: 1: gemini: not found /bin/sh: 1: gemini: not found Traceback (most recent call last):   File "./tools/bin/gemini", line 7, in <module>     gemini_main.main()   File "/home/aurelien.beliard/varapp/data/anaconda/lib/python2.7/site-packages/gemini/gemini_main.py", line 1248, in main     args.func(parser, args)   File "/home/aurelien.beliard/varapp/data/anaconda/lib/python2.7/site-packages/gemini/gemini_main.py", line 204, in load_fn     gemini_load.load(parser, args)   File "/home/aurelien.beliard/varapp/data/anaconda/lib/python2.7/site-packages/gemini/gemini_load.py", line 49, in load     load_multicore(args)   File "/home/aurelien.beliard/varapp/data/anaconda/lib/python2.7/site-packages/gemini/gemini_load.py", line 93, in load_multicore     chunks = load_chunks_multicore(grabix_file, args)   File "/home/aurelien.beliard/varapp/data/anaconda/lib/python2.7/site-packages/gemini/gemini_load.py", line 264, in load_chunks_multicore     wait_until_finished(procs)   File "/home/aurelien.beliard/varapp/data/anaconda/lib/python2.7/site-packages/gemini/gemini_load.py", line 359, in wait_until_finished     raise ValueError("Processing failed on GEMINI chunk load") ValueError: Processing failed on GEMINI chunk load
while with one core :
./tools/bin/gemini load --cores 1  -t VEP -v result.vcf.gz test_db.db
it work perfectly

Make sure that gemini is on your $PATH it looks like that is not the case since you are using /tools/bin/gemini

oh ok i didn't add it to the the path. I don't understand why i can't use it without add it to the path but it work now.
thanks for your help


panic: Bad file number when opening file with Go 1.21 wasip1

Describe the bug
I consistently get a panic when trying to open any file using a wasm binary built with gotip.
Specifically, this only starts occuring after this commit was introduced in to wazero main: https://github.com/tetratelabs/wazero/commit/53ce5eea830b3100af7ec8caa13bb38310c0afbf#diff-4fb3ce1d291b912fd6aa5e7f790040234a92683e313d1c757c0fce7be9e43320L397-L400
It does not fail in the parent.
To Reproduce
Take the following Go code and compile it with gotip to wasm and wasip1 platform/architecture:
package main

import (
	"fmt"
	"os"
)

func main() {
	_, err := os.Open(".")
	if err != nil {
		panic(err)
	}
}

GOOS=wasip1 GOARCH=wasm gotip build -o open.wasm main.go

The run with wazero run:
wazero run --hostlogging=all open.wasm

Expected behavior
No error to be produced.
Screenshots
--> ._rt0_wasm_wasip1()
	==> wasi_snapshot_preview1.random_get(buf=1191392,buf_len=32)
	<== errno=ESUCCESS
	==> wasi_snapshot_preview1.random_get(buf=1191064,buf_len=8)
	<== errno=ESUCCESS
	==> wasi_snapshot_preview1.clock_time_get(id=monotonic,precision=0)
	<== (timestamp=1625363670137041,errno=ESUCCESS)
	==> wasi_snapshot_preview1.args_sizes_get(result.argc=1210940,result.argv_len=1210936)
	<== errno=ESUCCESS
	==> wasi_snapshot_preview1.args_get(argv=21020680,argv_buf=21020688)
	<== errno=ESUCCESS
	==> wasi_snapshot_preview1.environ_sizes_get(result.environc=1210928,result.environv_len=1210932)
	<== errno=ESUCCESS
	==> wasi_snapshot_preview1.clock_time_get(id=monotonic,precision=0)
	<== (timestamp=1625363674881875,errno=ESUCCESS)
	==> wasi_snapshot_preview1.clock_time_get(id=monotonic,precision=0)
	<== (timestamp=1625363674888833,errno=ESUCCESS)
	==> wasi_snapshot_preview1.clock_time_get(id=monotonic,precision=0)
	<== (timestamp=1625363675064875,errno=ESUCCESS)
	==> wasi_snapshot_preview1.fd_fdstat_get(fd=0)
	<== (stat={filetype=CHARACTER_DEVICE,fdflags=,fs_rights_base=FD_DATASYNC|FD_READ|FDSTAT_SET_FLAGS|FD_SYNC|FD_WRITE|FD_ADVISE|FD_ALLOCATE,fs_rights_inheriting=},errno=ESUCCESS)
	==> wasi_snapshot_preview1.fd_fdstat_set_flags(fd=0,flags=NONBLOCK)
	<== errno=ESUCCESS
	==> wasi_snapshot_preview1.fd_fdstat_get(fd=1)
	<== (stat={filetype=CHARACTER_DEVICE,fdflags=,fs_rights_base=FD_DATASYNC|FD_READ|FDSTAT_SET_FLAGS|FD_SYNC|FD_WRITE|FD_ADVISE|FD_ALLOCATE,fs_rights_inheriting=},errno=ESUCCESS)
	==> wasi_snapshot_preview1.fd_fdstat_set_flags(fd=1,flags=NONBLOCK)
	<== errno=ESUCCESS
	==> wasi_snapshot_preview1.fd_fdstat_get(fd=2)
	<== (stat={filetype=CHARACTER_DEVICE,fdflags=,fs_rights_base=FD_DATASYNC|FD_READ|FDSTAT_SET_FLAGS|FD_SYNC|FD_WRITE|FD_ADVISE|FD_ALLOCATE,fs_rights_inheriting=},errno=ESUCCESS)
	==> wasi_snapshot_preview1.fd_fdstat_set_flags(fd=2,flags=NONBLOCK)
	<== errno=ESUCCESS
	==> wasi_snapshot_preview1.fd_prestat_get(fd=3)
	<== (prestat=,errno=EBADF)
	==> wasi_snapshot_preview1.clock_time_get(id=monotonic,precision=0)
	<== (timestamp=1625363675474833,errno=ESUCCESS)
	==> wasi_snapshot_preview1.fd_fdstat_get(fd=0)
	<== (stat={filetype=CHARACTER_DEVICE,fdflags=NONBLOCK,fs_rights_base=FD_DATASYNC|FD_READ|FDSTAT_SET_FLAGS|FD_SYNC|FD_WRITE|FD_ADVISE|FD_ALLOCATE,fs_rights_inheriting=},errno=ESUCCESS)
	==> wasi_snapshot_preview1.fd_fdstat_get(fd=1)
	<== (stat={filetype=CHARACTER_DEVICE,fdflags=NONBLOCK,fs_rights_base=FD_DATASYNC|FD_READ|FDSTAT_SET_FLAGS|FD_SYNC|FD_WRITE|FD_ADVISE|FD_ALLOCATE,fs_rights_inheriting=},errno=ESUCCESS)
	==> wasi_snapshot_preview1.fd_fdstat_get(fd=2)
	<== (stat={filetype=CHARACTER_DEVICE,fdflags=NONBLOCK,fs_rights_base=FD_DATASYNC|FD_READ|FDSTAT_SET_FLAGS|FD_SYNC|FD_WRITE|FD_ADVISE|FD_ALLOCATE,fs_rights_inheriting=},errno=ESUCCESS)
	==> wasi_snapshot_preview1.path_open(fd=-1,dirflags=SYMLINK_FOLLOW,path=.,oflags=,fs_rights_base=FD_READ|FD_SEEK|FDSTAT_SET_FLAGS|FD_SYNC|FD_TELL|FD_ADVISE|PATH_CREATE_DIRECTORY|PATH_CREATE_FILE|PATH_LINK_SOURCE|PATH_LINK_TARGET|PATH_OPEN|FD_READDIR|PATH_READLINK|PATH_RENAME_SOURCE|PATH_RENAME_TARGET|PATH_FILESTAT_GET|PATH_FILESTAT_SET_TIMES|FD_FILESTAT_GET|FD_FILESTAT_SET_SIZE|FD_FILESTAT_SET_TIMES|PATH_SYMLINK|PATH_REMOVE_DIRECTORY|PATH_UNLINK_FILE|POLL_FD_READWRITE,fs_rights_inheriting=FD_DATASYNC|FD_READ|FD_SEEK|FDSTAT_SET_FLAGS|FD_SYNC|FD_TELL|FD_WRITE|FD_ADVISE|FD_ALLOCATE|PATH_CREATE_DIRECTORY|PATH_CREATE_FILE|PATH_LINK_SOURCE|PATH_LINK_TARGET|PATH_OPEN|FD_READDIR|PATH_READLINK|PATH_RENAME_SOURCE|PATH_RENAME_TARGET|PATH_FILESTAT_GET|PATH_FILESTAT_SET_SIZE|PATH_FILESTAT_SET_TIMES|FD_FILESTAT_GET|FD_FILESTAT_SET_SIZE|FD_FILESTAT_SET_TIMES|PATH_SYMLINK|PATH_REMOVE_DIRECTORY|PATH_UNLINK_FILE|POLL_FD_READWRITE,fdflags=)
	<== (opened_fd=,errno=EBADF)
	==> wasi_snapshot_preview1.poll_oneoff(in=1210824,out=1210792,nsubscriptions=1)
	<== (nevents=1,errno=ESUCCESS)
	==> wasi_snapshot_preview1.poll_oneoff(in=1210824,out=1210792,nsubscriptions=1)
	<== (nevents=1,errno=ESUCCESS)
panic: open .: Bad file number

goroutine 1 [running]:
main.main()
	/Users/georgemac/github/georgemac/wasm-invoke/cmd/walk/main.go:11 +0x14
	==> wasi_snapshot_preview1.proc_exit(rval=2)

Environment (please complete the relevant information):

Go version: gotip for wasm and Go 1.20 for building wazero
wazero Version: specifically 53ce5eea830b3100af7ec8caa13bb38310c0afbf, before that it works
Host architecture: Mac Silicon

Additional context
I added print statements to wazero and found the difference which occurs is that:
fd_prestat_dir_name(fd=3)

This used to write into memory the filename / and now it writes zero bytes for the name.

I think this may be intended behavior. If you are running the binary via wazero run open.wasm and you want to interact with the file system, then you will need to -mount a valid path. The following runs successfully:
wazero run --hostlogging=all --mount=.:. open.wasm

Interesting, I got away with this before that release. This issues exists too if you try and use the wasi example code:
https://github.com/tetratelabs/wazero/blob/main/imports/wasi_snapshot_preview1/example/cat.go
But obviously slot in wasm from Go wasip1 there instead. Is there a WithMount? or similar to get that working.

The docs state for WithFS that it should be for the / path:
	// WithFS is a convenience that calls WithFSConfig with an FSConfig of the
	// input for the root ("/") guest path.
	WithFS([fs](https://pkg.go.dev/io/fs).[FS](https://pkg.go.dev/io/fs#FS)) [ModuleConfig](https://pkg.go.dev/github.com/tetratelabs/wazero#ModuleConfig)

Just for clarification, I only used wazero run here to simplify the reproduction. This fails for me in code that uses WithFS.

WithFSConfig :)

Nice, that sounds promising. And you are right, --mount=. and --mount.:/ both work for me.
Knowing that I can work around it with your suggestion there is helpful too. Thanks!

this patch should solve your issue https://github.com/tetratelabs/wazero/pull/1565


Measure Returning Value of First Row in all rows

Hi, This visual is really helpful for my requirement. But I am getting same measure value for all rows when I drag measure to rDataTable and slice outside this visual. If I add the same column again in the rDataTable, it's working fine. Could you check provide your comments or give me source code to make changes.

Hi! @ctspradeep able to find solution?


Close Jedis connection after set or get

I don't know why JReJSON always close Jedis connection after action set or get?
So i can not use jedis conn any more when i use JedisPool

I share you puzzlement and would be glad to accept a pull request that removes the offending conn.close(); lines from the client's methods.

JReJSON 1.1.0 is using Jedis Pool and doesn't require a connection


Unable to scroll after update

Hi,
I have a MathView with a fixed height that can't seem to be scrolled after the latest update. I suspect it is due to the following snippet:
    // disable touch event on MathView
    @Override
    public boolean onTouchEvent(MotionEvent event) {
        return false;
    }

I'm using a ScrollView to wrap the MathView (set to WRAP_CONTENT) which works fine, but it'll be great if the MathView can handle scroll events too.
Thank you very much.

Hi @eric-tong , I am sorry for your inconvenience because of my mistake.
The snippet is a try for fixing issue #12. But apparently it introduced this bug. I am learning Android's input touch event system, and hopefully an update in a few days will fix this bug. Before then, please keep using the previous version.

Hi @kexanie, no worries and thank you for the quick response :)


module to convert sizes

Describe the problem you are trying to solve
In many configurations we have to specify sizes, they are (usually) defined as a number representing bytes, but it would be nice to have some more expressive ways.
Describe the solution you'd like
A module specifying functions to generate different sizes for example size::mb(10) + size::mb(0.5)  for 10mb and 5kb and enable this for the common sizes. This can fully be implemented in tremorscript

this is a good crate, but it doesn't yet have support for the information unit: https://github.com/paholg/dimensioned/issues/37

that's a nice one OTOH it'd be really nice to have something in tremorscript native just for it being tremor native :D

Would it make sense for the module to expose functions like size::kibi(2) (will return 2048), size::kilo(1) (will return 1000), etc.? Correct me if I'm wrong but it's not possible to wrap numbers into user-defined types right now, so size::MB would still have to return unitless value, so the fact that it's bytes has to be kept in the comments/head of the reader anyway

Created a draft with my solution, happy to discuss further before I finish it up :)
https://github.com/tremor-rs/tremor-runtime/pull/1321


Data Table - Filtering support for Kino.Explorer

Working on final touch-ups
https://user-images.githubusercontent.com/5660941/208582993-1f4f8e6a-e49d-45f8-b780-409165e70212.mp4

To reduce the amount of buttons on the interface, what do you think about "Filters:" starting with a value called "None", similar to Sort? If you want to cancel the filter, you can revert it to "None". Initially it will look like this:

Notice I aligned the select to the right  (I think we should align all of them to the right).
Then, once you click a filter, we will show this and input with the button inside the input:

But probably on the right side?

Another nitpick: I also think we are using the Inter font too much here. In this dropdown, I would perhaps use the regular font (like we do in Kino.VegaLite dropdown). I would perhaps also use regular font on the title, subtitle, "Next", "Prev", and  "Show".

Ok! I agree with both your suggestions.

Also, sorting is reflected unless I open the select a second time:
https://user-images.githubusercontent.com/17034772/208931752-8a52af46-cc20-4a1b-beac-16691c2839b7.mov

Suggestions + fixes + cumulative filtering + prefill for filter and sort
https://user-images.githubusercontent.com/5660941/209412987-a29ac056-972a-4c73-8503-aeb2355d1935.mp4

One last nitpick: can you make it so the v of the select and the input touch the red line:

Then we can ship it :)


Update German translation & Add better translation_keys for suggested loads

Sorry was not intending to request yet

Current code only shows 3 values with the same keys

(When program which supports washing & drying is selected)

(When drying program is selected)
I can't find a program which only supports washing, but it demonstrates my pull request.

I also updated the German translations as good as possible as German is my native language & I'm fairly good in English too.
Other translations are generated tho.


Created images not working with UIViewContentModeScaleAspectFit

The created image gets streched even when using UIViewContentModeScaleAspectFit on a UIImageView.
Fixed it by adding another conversion step (CIImage -> CGImage -> UIImage).
https://github.com/Querschlag/QRCode/commit/f603d590bda07ac81f9e3cdd09ebef6d53853e41
Update: This also fixes image preview in the debugger.

Cool, please open a pull request if you want to have this merged.

Merged


[Menu] Add tests for handleEnter()

[x] PR has tests / docs demo, and is linted.
[x] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[x] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

@oliviertassinari made the relevant changes

+0.17% Thanks!


Modal scroll by touchpad/trackpad does not work on IE11

In IE 11 scrolling a modal that is higher than the viewport does not work. It does work when using the up and down keys. Is it reproducable by me with Windows 7 / IE 11 on the modal live example at http://getbootstrap.com/javascript/#modals.

Unable to reproduce in Win 7 IE11 on Sauce Labs:
https://saucelabs.com/beta/tests/35711af9201a43e0a4346aced99fbbb5/watch

Thank you for testing. As the video looked like a mouse wheel was used, I tested that on my laptop. Result: Mouse wheel works (as in the video), touchpad does not. Sorry for not being detailed enough.

I think touchpads just emit wheel events like mice do, so the difference you're describing would presumably be due to a bug in either IE11 itself or your touchpad drivers. Assuming that's the case, there's unlikely to be a way to workaround it on our end.
@twbs/team (or anyone else reading this): Anyone happen to have a Windows laptop handy (with Win 7 if possible)?

@patrickhlauke Do you happen to have a Windows laptop w/ IE11?

FWIW, I've tried on my Win7/IE11 laptop with (Synaptics) touchpad, and the scrolling seems to work as expected.

Going to assume the problem is specific to @jhyle's particular setup then.
We'll keep an eye out for any future reports of the same problem though.


Add feedback message to the Success dialog, under the animated icon, saying “Your changes have been saved.”.

"success" dialog: under the animated icon, saying “_Your changes have been saved.+”.
“progress” dialog, under spinner, saying “Loading content type schema…”
“error” dialog, under the icon, saying “Failed to save the content type schema”


Can we have a release 0.1.5 on pypi?

@msom I can provide you the right to push to the PyPi repository. Just give me your username of the platform.
Unfortunately I cannot maintain the library anymore as I'm not working with Cisco Prime Infrastructure.

The username is inshsr.
If you like, you can also transfer the project to us (https://github.com/HSRNetwork) - we are a Swiss University and probably able to maintain the library for a long time...

I set you as owner of the repository on PyPi. Unfortunatly I cannot transfer the project because you already have a fork of mine (github forbid that). I guess if you delete your fork I would be able to transfer it.

Ok, I deleted the fork!

Thanks, but still not able to transfer the project. Now Gihub is saying that I cannot create public repository on HSRNetwork. I added @msom as a project collaborator. Could you try to transfer it on your own (Settings > Options > Danger Zone).
If you still can't I guess I would need rights to create public repository on HSRNetwork temporarily.

Seems like I don't have the rights for the settings page. An alternative option would be to transfer the repo to my user account and then I transfer it to the organization?

I just transfered the repository to your own namespace @msom. Hope the library could live on your side 😃

Great! I was able to transfer it further. Thanks!


Fixed - RedissonConnection add lpos command

Hello. I found RedissonConnection does not override default implimentation lpos method in class org.springframework.data.redis.connection.DefaultedRedisConnection, which
will cause StackOverflowError when calling "connection.lpos()".
I tried to fix it in this PR and tested on my computer.
Thanks for your time.

Thanks for contribution!


accept persistent instructions

i would like some instructions to remain persistent, especially when i tell copilot to remember the instructions. it seems that copilot forgets instructions i tell it to always remember, sometimes even within the same session.
example: when providing me suggestions to modify my code, retain all the comments that were in the code snippet i gave you.

+1. Examples where it could be useful for me:
"Always generate code with indents as tabs, not spaces",
"Don't modify my code, only show references and examples, unless I ask explicitly for code",
...

https://github.com/microsoft/vscode-copilot-release/issues/563
Also see github.copilot.chat.experimental.codeGeneration.instructions (currently in Insiders/pre-release)


test(routes): GET /accounts with invalid session

closes hoodiehq/camp#62

Coverage remained the same at 93.391% when pulling 0afa14264797c3b963fbf4e260afb800e59f73b0 on simonyangme:test-invalid-session into e3b60d184b3668d676cffee7f0b82b79ad37ce21 on hoodiehq:master.


s_server: Do not use SSL_sendfile when KTLS is not being used

Fix a bug in openssl s_server -WWW where it would attempt to invoke SSL_sendfile if -ktls -sendfile was passed on the command line, even if KTLS has not actually been enabled, for example because it is not supported by the host. Since SSL_sendfile is only supported when KTLS is actually being used, this resulted in a failure to serve requests.
Fixes #17503.
Taking advice on how to test this, as most usage of s_server in the test suite seems to be incidental via TLSProxy rather than testing of s_server itself.

Updated to add a warning when sendfile was requested but a request was served without it.

There is a "get out clause" in the test policy which says we don't have to write tests "where writing the test would result in disproportionately more effort than writing the code being tested"

IMO this is well within this clause. Writing a completely new test recipe that would not be trivial for this few line fix would be disproportionately more effort. However! We should at least create an issue to add tests for s_server/s_client apps.

Updated.

Yes, OK for 3.0 as well.


Tokenizer

This PR adds the following functionalities:

train our own tokenizer using HF's Tokenizer library
batched tokenisation in the collate_fn
correct decoding

Refer readme to see how to run it.
The get_tokenizers function will return two tokenizers -- one for each language.
cc: @ariG23498

Agree on the single file. Will move tokenizer.py to wmt_data_utils.py.

@ariG23498 the PR is ready for review.
from llm import get_dataset, get_tokenizers, get_dataloader

# Load the dataset
ds = get_dataset(name="de-en")

# Get tokenizers
# Path is where you have put the trained tokenizers
tokenizers = get_tokenizers("de-en", path="../data")

# Get dataloader
train_dl = get_dataloader(ds["train"], batch_size=2, source_lang="de", target_lang="en", tokenizer_path="../data")

To train the tokenizer you can run this command from the root of your repo:
python src/llm/wmt_data_utils.py
If you want to experiment: python src/llm/wmt_data_utils.py -h but the defaults are good for now and I have tested them to work well.


cuda-modules: fix sort predicate stability

Incorrect sorting predicate was found as part of
https://github.com/NixOS/nix/issues/12106 where nix was crashing on the code like:
$ nix eval --expr 'builtins.sort (a: b: true) [ 1 2 3 ]'
...
Aborted (core dumped)

Note: the crash happens here because sorting predicate does not implement isLess and triggers assertion failures for std::stable_sort that backs builtins.sort.
THe change restore isLess semantic for preferable.

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Looks good, but off the top of my head I can't immediately tell why Ofborg failed with
       error: attribute 'lib' missing

       at /ofborg/checkout/2/repo/38dca4e3aa6bca43ea96d2fcc04e8229/mr-est/ofborg-evaluator-4/pkgs/development/python-modules/torch/default.nix:369:30:

          368|       export CUDNN_INCLUDE_DIR=${lib.getLib cudnn}/include
          369|       export CUDNN_LIB_DIR=${cudnn.lib}/lib

...a riddle for the next morning


decomp: eye-h and projectile-h

Pull Request Test Coverage Report for Build 884094108

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 69.762%

Totals

Change from base Build 880326930:
0.0%

Covered Lines:
35870

Relevant Lines:
51418

💛  - Coveralls


Support for Cocoapods

Hi czechboy0,
is it possible to support cocoapods as package manager too?
That would be great.
Seppelandrio

Hi @seppelandrio,
I'm not planning to do that at this moment, as the version of Swift required by Redbird changes with Swift 3 getting finalized, but with CocoaPods most people use the standard Xcode Swift version.
If you'd like to use Redbird in your project, manually clone it (and also Socks, its dependency) locally. I'll consider adding CocoaPods support when Swift 3 is released.

Hi,
now Swift 3 has been officially released. Can you please add CocoaPods? It would help me a lot.
Greets.
Seppelandrio


The selector of the component "Component" should be named undefined

I am getting an undefined naming error on the Angular Component selector.

ERROR: src/app/client/account/components/settings/accountAccess/accountAccess.component.ts[4, 13]: The selector of the component "AccountAccessComponent" should be named undefined (https://angular.io/styleguide#style-05-02)

Although name is ok, seems like codelyzer has a problem finding the name_1 parameter in
codelyzer/docs/dist/src/selectorNameBase.js
var error = sprintf_js_1.sprintf(_this.rule.getStyleFailure(), className, name_1);

Here is my code:
import {Component} from '@angular/core';

@Component({
  selector: 'client-account-access',
  templateUrl: './accountAccess.html'
})
export class AccountAccessComponent {
}

I am using:
"@angular/core": "^4.4.4",
"@angular/cli": "1.0.1",
"tslint": "~5.8.0",
"codelyzer": "~4.0.1",

Found the problem, it was because I updated the packages but not my tslint.json.
Changed in  tslint.json
"component-selector": [true, "element", "kebab-case"]
to
"component-selector": [true, "element", ["client"], "kebab-case"]
and it worked.


Shift + Spacebar should take back to the previous slide.

A nice-to-have feature.

Hi @rushiagr , in an effort to clear up older issues/PRs we are pinging back to know if you are still tracking this request.
To give a little bit of context, recently a decision was made in the project to make the development more active and the first task is to clear up older issues like this one to see if the OP is still interested in keep it going.

I was just a normal passers-by for ImpressJS project. I am not an expert in JS and won't be able to contribute to this project in any other form in the near future. Please feel free to take a decision on this issue according to your will :)

@rushiagr Thanks for the response!
In this specific case is there any reason for using shift + space other than "nice to have"?
"Nice to haves" usually lead into bloat due to the low usage in the wild. Also, be aware that we should respect the Principle of the Least Astonishment so that the shortcuts do not have to be checked by the user in the documentation. To see if we are respecting good design principles we should copy the keyboard pattern from most similar tools from other technologies.
So, for example, do you have any evidence that this shortcut exists in other presentation tools or websites to justify it's inclusion? More than one popular example, if possible, will help a lot. we already have plans to support shift + tab for consistency with the tab shortcut, is there any reason that justify creating a new one?

I tried google slides and LibreOffice Impress (a Powerpoint alternative), and both of them don't have this 'shift+space' shortcut.
I am used to using 'space' to go one page down on a web page in a browser, and 'shift+space' to go one page up. I think that is where the idea to submit this issue come from. I still think that if 'space' is provided for 'next slide', similarly 'shift+space' should be provided for 'previous slide' on all presentation softwares for user experience (and to honour the principle of least astonishment, as per my opinion ), though I understand I might be in minority :)

if 'space' is provided for 'next slide', similarly 'shift+space' should be provided for 'previous slide' on all presentation softwares for user experience

That's definitely a very good argument in favor of this feature.
What I see is that impress.js is using web navigation standards instead of focusing only in presentation standards. I can't give an example right now, but we might in the future have a situation where two key combinations does one thing for web navigation and another thing for presentation, this can prevent us from supporting both. Maybe I am overthinking this, but it is good to keep this potential problem in mind.
Also, in the future, we might split the keyboard navigation in a separate module in order to separate the concerns, that might allow the developer to attach web or presentation navigation at will. For now it seems reasonable to be consistent for web navigation too, or at least be consistent with the keys that are already mapped.

Actually, after testing on many browsers, shift+space have the same beahviors, so we could implement this feature as "mixin" between web and presentation standards :smile:

#706 fixed this


HPCC-27627 Fix warnings in the nlp engine

Signed-off-by: Gavin Halliday<EMAIL_ADDRESS>

@dehilsterlexis

@ghalliday these corrections look legit. These changes should go in the nlp-engine and then the nlp-engine submodule updated in the HPCC repository.

@ghalliday you can close this without merging. I have merged these changes into the nlp-engine code. You can now update this submodule to the latest version. https://github.com/VisualText/nlp-engine/pull/186

Closed - merged separately into the main repo


C#: Add Ide Connection library and server for the editor

This will be used for communicating between the Godot editor and external IDEs/editors, for things like opening files, triggering hot-reload and running the game with a debugger attached.

Thanks!


Failed to install rpm from internet on CentOS8 4.18.0-348.7.1.el8_5.x86_64

The commond [yum install  -y createrepo e2fsprogs-devel genisoimage git libtool-ltdl-devel libyaml-devel json-c-devel redhat-lsb-core wget python3-pylint python3-psutil modulemd-tools libcurl-devel ganglia-devel gtk2-devel iptables-devel iproute-devel libatasmart-devel libdbi-devel libcap-devel libesmtp-devel libgcrypt-devel libmemcached-devel libmicrohttpd-devel libmnl-devel libnotify-devel libpcap-devel libssh2-devel libxml2-devel libvirt-devel lm_sensors-devel lua-devel mosquitto-devel net-snmp-devel OpenIPMI-devel openldap-devel perl-ExtUtils-Embed postgresql-devel python-devel qpid-proton-c-devel riemann-c-client-devel rrdtool-devel systemd-devel uthash-devel xfsprogs-devel yajl-devel zeromq-devel] is executed, the setup shell will crash. I find in this OS version, the package "python-devel" cause terrible probles。
I think you can replace "python-devel" with "python36-devel", and "postgresql-devel" with "libpq-devel".
Meanwhile, the paramenter "--enablerepo=powertools" could be supplemented for "yum" command, therewise, some packages can't be installed.
In addition, in file "pybuild/coral_build.py:333",  the "createrepo" will cause some error to crash program, because the result of  "rpm -q createrepo" mean createrepo not be installed. So, I sovle it via changing "createrepo" to ""createrepo_c".
Please check it and recover soon.

Thanks for the advice. A patch is coming and it is a little bit more complex since both RHEL7 and RHEL8 need to be supported.

Fixed in https://github.com/LiXi-storage/barreleye/commit/2662b0e86c2ed44f3864689a43f77bfb18f067bf


Need way to distinguish actual before/after calls from 'new-child' before/after calls

Looking at the code (and example tests), the current semantics for async_wrap seem to be that the before/after hooks are called from two places

before and after the application of the async callback (i.e. the intuitive place): https://github.com/iojs/io.js/blob/0af4c9ea7434e4f505dbe071357e4bc3b4ab2a8a/src/async-wrap.cc#L112
when a child async object is being created, the parent's before and after hooks get called. https://github.com/iojs/io.js/blob/v1.x/src/async-wrap-inl.h#L35-L37

I cannot seem to figure out a way to distinguish between these two cases. When the hook gets called, I am not sure if the event is actually getting processed or a child event got instantiated. Am I missing something?
Would it make sense to have 5 hooks: init, before, after, child_before, child_after? @trevnorris
Here's a sample program: https://gist.github.com/ofrobots/59bb54ef677834443672
The before/after hooks for context with uid 1 get called multiple times.
EDIT: Here's a better test-cased based on the blog post:  https://gist.github.com/ofrobots/d725e7c2aa5fdafa8a13

The simplest solution is to pass an argument to the callback. On the list is a plan to pass the PROVIDER type to the callbacks, which could be easily overloaded so you could detect this case as well.

What if the PROVIDER value is the same?
I'm curious what the intended purpose is to having those before/after hooks around child async creation. Can those not be named differently? Do those even need to be there? What would they be used for?

I see value in being able to string together the parent-child relationship to show causality. But some people might not care. If these are separate hooks, then people could opt out of these extra events and avoid unnecessary callbacks.

Can that not be done by storing a context id in the before handler and checking that value in the child init handlers?

Yes, you can establish parent-child relationship that way if the parent's hooks get called around child's init. In the example I linked to above, the parent's (1) before hook isn't applied (for real) until after child's (2) init.

I think that is mostly a perception issue. Nothing is truly "async" until it jumps the barrier into C++, at which point you no longer have anything that could occur between the init and before events.
I think patching of abstractions over truly async things should simply acknowledge that they themselves are not actually async and simply establish the boundary linking through the thing they abstract.

@Qard

Nothing is truly "async" until it jumps the barrier into C++

The problem is when the starting point is C++. The code is to cover cases where a connection is received, and you want to be able to trace the connection back to the server which received it. Since the connection comes from C++ it has no stack above it so there is no way to store any properties elsewhere which would let you know where it's coming from.

Ah, yes. That is an interesting case.
Ultimately what we want is linking between the call and the callback, but async_wrap provides linking between resource creation and the callback. This style is useful for potentially tracking and cleaning up resources, but doesn't necessarily get us the JS -> C++ -> JS boundary linking we want without the extra weirdness of the before/after events getting triggered around child inits.
I would definitely suggest using different named events for those to not be ambiguous. I would also suggest placing triggers that wrap the sync execution of calls into C++ and around the sync execution of JS callbacks. This encapsulates the sync boundary jumps from JS into C++ (calls) and from C++ into JS (callbacks). Then you just need a mechanism of linking the callbacks to their originating calls.

I'm trying to recall why I'm allowing the user to handle this case, as I'm sure there was a reason. ATM seems I could have handled that automatically instead. I'll investigate.

@Qard The calling of before/after in init has been fixed by https://github.com/nodejs/io.js/commit/7dde95a8bd.

@ofrobots The init callback in the most recent API is function init(provider). A possibly more elegant solution is function init(provider, parent = null), where the parent object is passed, if there is one. Thoughts?

@trevnorris Yes I saw those changes and have been prototyping with them already, thanks. :+1: on the parent object being passed to the init function if available.

@ofrobots @trevnorris This issue can be closed, right?


Thousands of /cfs/servertime calls interrupted by client

I'm getting bombarded with "client request interrupted" warnings (it's an error code H27 on Heroku, HTTP status code 499 "closed by client while the server is still processing its request, making server unable to send a status code back") on my server - one warning every 1-2 seconds for each client experiencing this issue.  Typically it occurs for up to an hour at a time per client, so I am getting thousands of these things.
The call that is causing this problem is method=GET path="/cfs/servertime"
It is always preceded by one or a few socks calls e.g. method=GET path="/sockjs/info?cb=j3e3_z7aeu" calls which usually return fine (status code 200), although sometimes the socks call also returns error 499.
My hunch is that the client would be experiencing some sort of CFS issue at startup, stalled, is never really able to access the CFS collections and just left the app trying again and again. Code it be related to hot code push...?
Anybody else experienced the same issue?
Ideas on how to investigate further?

I should add I'm using gridfs.

I am experiencing something similar when I deploy to Heroku. Mine is on the server side "Server Request Interrupted".
I get a NoSuchKey error when I log the readStream.on 'error'. I was up all night trying to figure it out. It worked on meteor servers and locally, but not on heroku for some reason. Did you ever figure it out @Alveoli ?

Hi @travisdahl it's been such a long time, I've upgraded to Meteor 1.2 and rewritten that part of the code a few times.  The error came out of the wash... so all I offer you is the assurance that there's light at the end of the tunnel ;)


Add Icons for language buttons

Right now, there are no icons for the buttons. It would be better if monotone icons are used before the name of the language.
Please consider using simpleicons.org

@max-programming  Hi.... I would like to work on this

@max-programming  HI I would like to work on this

Sure @Anuran12 go ahead

hi bro I would like to complete this issue

@max-programming I would like to work on this Issue

Sure @Alone-Y154

I've started working on it, Will raise a PR once I've finished it

Hi @max-programming , I have raised the PR🙂

is this issue open?

Can You Please Assign Me This Project I Would be So Much Glad To Contribute In This project

Actually PR have be raised for this Issue awaiting for approval🙂, Thank You

I hope this message finds you well! I'm eager to contribute more to our project. plz assign this issue to me i will handle it

Actually @Alone-Y154 has already taken this up


Nesting inside a positioned element causes incorrect corrective values

Given two divs, one at the top level and another nested in a div, with the following rules and structure:

Div two will only be positioned properly if Div container is unpositioned (static). With relative or absolute positioning applied to Div container, Div two will have a top value of -200, when in fact 0 would be correct. The discrepancy varies as you might expect with the height of Div one.
It's possible I'm misunderstanding the intended behaviour.
Update: If absolute positioning is applied to Div one and Div two from the start there is no discrepancy, which leads me to think the issue involves stale positioning information from before absolute positioning is applied being used after.

Just a hunch, but since adding the top constraint causes the two previously statically positioned divs to become absolutely positioned, I'm guessing the calculation for how to adjust Div two's position across the nesting is based on the positions of the Divs prior to the absolute positioning being applied.

Indeed, this happens.
It does measurement & styling in two separate steps. So when it measures things, it doesnt know yet that something will be taken out of the page flow.
Then it validates those values. Quite counter-intuitively it does not do a validation pass if nothing in the document is listening intrinsic values. This is an incorrect assumption - I'll fix it.
Using something like the following doesnt let the first frame to use stale positioning information by triggering another remeasurement pass.
  a == #two[computed-width]; // make something trigger validation pass
  #container {
    position: relative;
  }

Though generally, position is completely ignored and top/left value should be considered absolute to the document. Adding position: relative on the element helps to maintain multiple global rectangles in the document (there's a mini-guide to this in docs)

I was exploring how well GSS can coexist with plain jane HTML/CSS, as I'm intending to use it as the basis for an extension to Ember.js playing nicely is important. I don't know how often this precise scenario would actually happen, but it's certainly not out of the realm of possibility.

I was exploring how well GSS can coexist with plain jane HTML/CSS, as I'm intending to use it as the basis for an extension to Ember.js and playing nicely is important. I don't know how often this precise scenario would actually happen, but it's certainly not out of the realm of possibility.

Sure. Another possible suggestion could be to set your main blocks be positioned to absolute value from the start. To take them out of the flow by the time first measurements happen.
You can also bind to intrinsic-y/computed-y of elements which should not take the anchor element out of the flow. But it's a bit undertested area at the moment (there's at least one bug issued)

And with a JS app there will be changes that should flush out any stale values. I'm not gonna worry about it until it proves itself to be a real issue for my project. Setting absolute from the start with CSS is a fix, but it's also the sort of arcane implementation detail I need to shield from end users. Complete transparency is the goal. This is, to be fair, a contrived example.

I agree with you, we'll ensure another measurement pass when switching position style of elements, or something like that.


improve filter predicate validation

Issues

Changelog
❗Upgrade module to use go 1.22
Add predicate validation to correct errors with predicates before sending API requests

[ ] List your changes here
[ ] Make a changie entry

Tophatting
# In progress...

While i'm not against this - it now becomes a burden to maintain the bespoke logic the monolith has over in a separate codebase.
Why is relying on the API errors not good enough? Is that really worth the support burden of this logic when 6-12 months from now we have to change this because they've added new stuff? or they have removed one of these validations but our tools still reject it outright? (my gut says the is no value great enough for those edgecases but i leave that decision to you @davidbloss since you maintain the code more then me now)

It's the lesser of two evils really. The API errors themselves aren't always clear.
When trying to create a filter with a predicate that matches another filter, the error the API returns is
Unable to create filter, got error: OpsLevel API Errors:
      - '' Invalid filter ID

Similarly, providing a different invalid predicate returns
Unable to create filter, got error: OpsLevel API Errors:
      - 'value' Invalid global id 'HI'

The API error multiple customers have complained the most about is this one:
Unable to create filter, got error: OpsLevel API Errors:
      - 'type' invalid type for 'repository_ids' predicate'

☝ there is no mention of the predicate that caused the error or indication of what predicate type should be used.
Long term I hope for a better solution on the API side but for now I think the added burden is worth it.


Add support for fullscreen mode

Currently the only option that maintains aspect ratio is default.

What prompted bringing back the calls to /stream? It was working on my device before with just this CSS rule:
#remote-screen img {
  max-width: 100%;
}

#remote-screen[fullscreen="true"] > img {
  margin: auto;
  height: 100%;
  max-height: 100vh;
  max-width: 100vw;
}

Hmm you are right, I do not have the usb hdmi capture device so I am replacing the stream with images with different dimensions to simulate. I tested before I split off the fullscreen true/false. No worries cleaned up again.

I have one more commit to send but I will wait for your feedback first in case anything else needs fixing.
I forgot to remove the unused code (2 lines):
const remoteScreenImg = document.getElementById("remote-screen-img");

Actually I am still seeing an issue when i reduce the size of the browser when the "letter boxes" are on the top and bottom of the screen.

Alright, there is still an issue with the mouse when the top and bottom are grey.

Awesome, thanks for adding this!


gta 4 does not lauch

Software information gta 4
without the d3d9.dll it works fine but as soon as i place it in the game folder and try to lauch the game it shows the following error:
the exception unknown software exception occurred in the application at location
System information

GPU: amd a6 7310 with amd r4 igpu
Driver: adrenaline 17.7
DXVK version: 2.2

Log files
info:  Game: LaunchGTAIV.exe
info:  DXVK: v2.2
info:  Found config file: dxvk.conf
info:  Vulkan: Found vkGetInstanceProcAddr in vulkan-1.dll @ 0x7386bad0
info:  Built-in extension providers:
info:    Win32 WSI
info:    OpenVR
info:    OpenXR
info:  OpenVR: could not open registry key, status 2
info:  OpenVR: Failed to locate module
info:  Enabled instance extensions:
info:    VK_KHR_surface
info:    VK_KHR_win32_surface

The Windows driver for that GPU probably doesn't support the required Vulkan extensions and features.

is there anything i can do plz help. here is my gpu driver info :
Radeon Settings Version - 2017.1026.251.3303
Driver Packaging Version - 17.10.3901.1004-171026a-319526C-HP
Provider - Advanced Micro Devices, Inc.
2D Driver Version - <IP_ADDRESS>9
Direct3D® Version - <IP_ADDRESS>61
OpenGL® Version - <IP_ADDRESS>76
OpenCL™ Version - <IP_ADDRESS>5
AMD Mantle Version - <IP_ADDRESS>9
AMD Mantle API Version - 102400
AMD Audio Driver Version - <IP_ADDRESS>
Vulkan™ Driver Version - 1.5.0
Vulkan™ API Version - 1.0.39

Vulkan™ API Version - 1.0.39

no.

i downgraded my driver so dxvk can work it was not working before as well with the updated driver

Your GPU driver needs to support Vulkan 1.3 to be able to run dxvk 2.0 and above, but your GPU isn't receiving new features anyway from amd.
If you fully upgraded your driver you might be able to run dzvk 1.10.3 but I can't promise it will be a good time.