Open3d vedo conversion

closes #14
closes #22

I'm stuck here because the mesh smoothing in vedo delivers weird results:
This is how it looks at the moment (example notebook):

Compared to the open3d-version:

Tests pass locally:
(bio39) C:\structure\code\napari-process-points-and-surfaces>pytest
================================================= test session starts =================================================
platform win32 -- Python 3.9.15, pytest-7.2.0, pluggy-1.0.0
PyQt5 5.15.7 -- Qt runtime 5.15.6 -- Qt compiled 5.15.6
rootdir: C:\structure\code\napari-process-points-and-surfaces
plugins: anyio-3.6.2, dash-2.7.1, napari-0.4.17, napari-plugin-engine-0.2.0, npe2-0.6.1, qt-4.2.0
collected 5 items

napari_process_points_and_surfaces\_tests\test_function.py .....                                                 [100%]

================================================== warnings summary ===================================================
..\..\..\Users\haase\mambaforge\envs\bio39\lib\site-packages\vispy\app\backends\_qt.py:93
  C:\Users\haase\mambaforge\envs\bio39\lib\site-packages\vispy\app\backends\_qt.py:93: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
    if LooseVersion(QT_VERSION_STR) >= '5.4.0':

..\..\..\Users\haase\AppData\Roaming\Python\Python39\site-packages\setuptools\_distutils\version.py:346
  C:\Users\haase\AppData\Roaming\Python\Python39\site-packages\setuptools\_distutils\version.py:346: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
    other = LooseVersion(other)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:341: DeprecationWarning: nppas.convex_hull() is deprecated. Use nppas.convex_hull_from_surface() instead.
    warnings.warn("nppas.convex_hull() is deprecated. Use nppas.convex_hull_from_surface() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:312: DeprecationWarning: nppas.to_mesh() is deprecated. Use nppas.to_vedo_mesh() instead.
    warnings.warn("nppas.to_mesh() is deprecated. Use nppas.to_vedo_mesh() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py: 13 warnings
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:289: DeprecationWarning: nppas.to_vector_d() is deprecated.
    warnings.warn("nppas.to_vector_d() is deprecated.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:295: DeprecationWarning: nppas.to_vector_i() is deprecated.
    warnings.warn("nppas.to_vector_i() is deprecated.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py: 10 warnings
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:329: DeprecationWarning: nppas.to_surface() is deprecated. Use nppas.to_napari_surface_data() instead.
    warnings.warn("nppas.to_surface() is deprecated. Use nppas.to_napari_surface_data() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py: 23 warnings
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:307: DeprecationWarning: nppas.to_numpy() is deprecated. Use np.asarray() instead.
    warnings.warn("nppas.to_numpy() is deprecated. Use np.asarray() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:361: DeprecationWarning: nppas.filter_smooth_simple() is deprecated. Use nppas.smooth_surface() instead.
    warnings.warn("nppas.filter_smooth_simple() is deprecated. Use nppas.smooth_surface() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:402: DeprecationWarning: nppas.filter_smooth_taubin() is deprecated. Use nppas.smooth_surface() instead.
    warnings.warn("nppas.filter_smooth_taubin() is deprecated. Use nppas.smooth_surface() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:381: DeprecationWarning: nppas.filter_smooth_laplacian() is deprecated. Use nppas.smooth_surface() instead.
    warnings.warn("nppas.filter_smooth_laplacian() is deprecated. Use nppas.smooth_surface() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:446: DeprecationWarning: nppas.simplify_quadric_decimation() is deprecated. Open an issue if you are using this function and/or if you know a good replacement https://github.com/haesleinhuepf/napari-process-points-and-surfaces/issues.
    warnings.warn("nppas.simplify_quadric_decimation() is deprecated. Open an issue if you are using this function and/or if you know a good replacement https://github.com/haesleinhuepf/napari-process-points-and-surfaces/issues.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:422: DeprecationWarning: nppas.simplify_vertex_clustering() is deprecated. Open an issue if you are using this function and/or if you know a good replacement https://github.com/haesleinhuepf/napari-process-points-and-surfaces/issues.
    warnings.warn("nppas.simplify_vertex_clustering() is deprecated. Open an issue if you are using this function and/or if you know a good replacement https://github.com/haesleinhuepf/napari-process-points-and-surfaces/issues.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:467: DeprecationWarning: nppas.subdivide_loop() is deprecated. Use nppas.subdivide_loop_vedo() instead.
    warnings.warn("nppas.subdivide_loop() is deprecated. Use nppas.subdivide_loop_vedo() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:487: DeprecationWarning: nppas.sample_points_uniformly() is deprecated. Use nppas.sample_points_from_surface() instead.
    warnings.warn("nppas.sample_points_uniformly() is deprecated. Use nppas.sample_points_from_surface() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:510: DeprecationWarning: nppas.sample_points_poisson_disk() is deprecated. Use nppas.sample_points_from_surface() instead.
    warnings.warn("nppas.sample_points_poisson_disk() is deprecated. Use nppas.sample_points_from_surface() instead.",

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:527: DeprecationWarning: nppas.voxel_down_sample() is deprecated. Use nppas.subsample_points() instead.
    warnings.warn(

napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:321: DeprecationWarning: nppas.to_point_cloud() is deprecated. Use nppas.to_napari_points_data() instead.
    warnings.warn("nppas.to_point_cloud() is deprecated. Use nppas.to_napari_points_data() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:550: DeprecationWarning: nppas.points_to_convex_hull_surface() is deprecated. Use nppas.create_convex_hull_from_points() instead.
    warnings.warn("nppas.points_to_convex_hull_surface() is deprecated. Use nppas.create_convex_hull_from_points() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:628: DeprecationWarning: nppas.fill_holes() is deprecated. Use nppas.fill_holes_in_surface() instead.
    warnings.warn(

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:597: DeprecationWarning: nppas.surface_from_point_cloud_ball_pivoting() is deprecated. Use nppas.create_convex_hull_from_points() instead.
    warnings.warn("nppas.surface_from_point_cloud_ball_pivoting() is deprecated. Use nppas.create_convex_hull_from_points() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:301: DeprecationWarning: nppas.to_vector_double() is deprecated.
    warnings.warn("nppas.to_vector_double() is deprecated.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_something
  C:\structure\code\napari-process-points-and-surfaces\napari_process_points_and_surfaces\__init__.py:571: DeprecationWarning: nppas.surface_from_point_cloud_alpha_shape() is deprecated. Use nppas.create_convex_hull_from_points() instead.
    warnings.warn("nppas.surface_from_point_cloud_alpha_shape() is deprecated. Use nppas.create_convex_hull_from_points() instead.", DeprecationWarning)

napari_process_points_and_surfaces/_tests/test_function.py::test_curvature
  C:\Users\haase\mambaforge\envs\bio39\lib\site-packages\vedo\pointcloud.py:463: FutureWarning: `rcond` parameter will change to the default of machine precision times ``max(M, N)`` where M and N are the input matrix dimensions.
  To use the future default and silence this warning we advise to pass `rcond=None`, to keep using the old, explicitly pass `rcond=-1`.
    C, residue, rank, _ = np.linalg.lstsq(A, f)  # solve AC=f

napari_process_points_and_surfaces/_tests/test_function.py: 64 warnings
  C:\Users\haase\mambaforge\envs\bio39\lib\site-packages\vedo\colors.py:779: PendingDeprecationWarning: The get_cmap function will be deprecated in a future version. Use ``matplotlib.colormaps[name]`` or ``matplotlib.colormaps.get_cmap(obj)`` instead.
    mp = _cm_mpl.get_cmap(name=name)

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
========================================== 5 passed, 150 warnings in 17.35s ===========================================

Hi @jo-mueller ,
can you do me a favor and try out this branch? Especially in the context of napari-stress it might be important to check if common workflows still work. All menus with open3d in the name were removed and thus, I presume we need to update documentation in napari-stress if those used open3d-nppas-menus.
Thanks!
Best,
Robert

can you do me a favor and try out this branch?

Can do. I'll probably do a napari-stress release next week and make sure that it's not broken by this.

I'll probably do a napari-stress release next week

Please let me know in advance @jo-mueller . I can release nppas then as well and you could link the right version here:
https://github.com/campaslab/napari-stress/blob/97f0e1c3153b5d1ec0997b69d4ec936735783512/setup.cfg#L52

Hi @haesleinhuepf  Looks alright to me. I found a small bug in the remove_duplicate_vertices function (it was missing the return type annotation, PR coming). Once that is merged it should be fine on the napari-stress end.

Codecov Report
Base: 53.55% // Head: 67.45% // Increases project coverage by +13.90% :tada:

Coverage data is based on head (1fff138) compared to base (b1eee05).
Patch coverage: 74.03% of modified lines in pull request are covered.

:exclamation: Current head 1fff138 differs from pull request most recent head 29787dd. Consider uploading reports for the commit 29787dd to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##             main      #23       +/-   ##
===========================================
+ Coverage   53.55%   67.45%   +13.90%
===========================================
  Files           3        8        +5
  Lines         338      845      +507
===========================================
+ Hits          181      570      +389
- Misses        157      275      +118

Impacted Files
Coverage Δ

..._points_and_surfaces/_surface_annotation_widget.py
16.56% <0.00%> (-0.86%)
:arrow_down:

...ari_process_points_and_surfaces/_quantification.py
70.80% <66.15%> (-6.54%)
:arrow_down:

napari_process_points_and_surfaces/_vedo.py
68.75% <68.75%> (ø)

napari_process_points_and_surfaces/__init__.py
82.12% <76.42%> (ø)

...rocess_points_and_surfaces/_tests/test_function.py
100.00% <100.00%> (ø)

..._process_points_and_surfaces/_tests/test_open3d.py
100.00% <100.00%> (ø)

..._points_and_surfaces/_tests/test_vedo_functions.py
100.00% <100.00%> (ø)

napari_process_points_and_surfaces/_utils.py
100.00% <100.00%> (ø)

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

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

Awesome!


Should lock be deleted from dictionary when DeleteDistributedLockRecord is called?

Should the lock be deleted from the the dictionary when DeleteDistributedLockRecord is called?
https://github.com/OrchardCMS/Orchard/blob/1.10.3/src/Orchard/Tasks/Locking/Services/DistributedLockService.cs#L180
I don't understand why you'd want to keep a deleted lock in the dictionary? Currently the issue we're having is when task acquires a lock for say five minutes and then finishes it calls the DeleteDistributedLockRecord and then when a task runs again within the time the lock has been acquired it won't recreate the record in the database and as a result an exception is thrown:
https://github.com/OrchardCMS/Orchard/blob/1.10.3/src/Orchard/Tasks/Locking/Services/DistributedLockService.cs#L185
@DaRosenberg @sebastienros @jtkech any thoughts?

Not so easy for me to remember the details ;)
But looks like you're right, seems that a lock is added once in the dico and then never removed even there is no more related record ...
Maybe remove the lock from the dico before this line
Would you suggest a PR for this?

@jtkech Thanks for getting back to me. Of course I'll create a PR.

Will keep the issue opened once the PR has been approved and merged.

@jtkech I can confirm your suggestion on the line works a treat! Thank you! As promised here is the PR: https://github.com/OrchardCMS/Orchard/pull/8394
So to recap since Orchard uses software locks to ensure the same task doesn't run simultaneously from two different servers.
When a task runs it creates a software lock and adds a record in the locks table and in a dictionary so the lock doesn't have to be reacquired. The problem was that it wasn't deleting it from the dictionary so that meant when the same task ran again it would pick up the lock because it's already been acquired, once the task had finished it attempted to delete the lock from the database but it was already deleted so that's why it was throwing the exception https://github.com/OrchardCMS/Orchard/blob/1.10.3/src/Orchard/Tasks/Locking/Services/DistributedLockService.cs#L185. With the fix once the task is disposed the new line deletes the lock from the dictionary.
Can you please do the honours of a code review? :-)
Also just thinking about the impact of this change? Since we're just deleting the item in the dictionary and we've not touched any of the other logic I can't imagine this having any major issues. Of course the most obvious thing would be to make sure the tasks are running and that the locks are being deleted from the database. Just wondered if you could think of anything else we might need to test?
Many Thanks

Also @jtkech Would I be right in thinking with the current version of Orchard with the bug since the lock gets deleted from the database and not the dictionary it's possible that the same task would continue to run on say machine A whilst on machine B a new lock would be created for the same task and that would also run defeating the purpose of the distributed lock feature.

@imranazad exactly, i will approve your PR


[FLINK-7199] [gelly] Graph simplification does not set parallelism

The Simplify parameter should accept and set the parallelism when calling the Simplify algorithms.
The LocalClusteringCoefficient "count triangles" reduce now uses the assigned ("little") parallelism as this computation is proportional to the number of vertices (the combine computation is proportional to the potentially much larger number of triangles).
The ignored CombineHint on the HITS all-reduces have been removed.

@NicoK thanks for the review! I'll work on refactoring the Runner and adding checks for checking the parallelism. I think we're okay if we whitelist the "large" operators by name.

@NicoK, parallelism tests have been added. There is a small refactor for Runner which needs more extensive rework elsewhere.

@NicoK I have fixed the noted issues and will merge.


Unable to resolve module prop-types

error: bundling: UnableToResolveError: Unable to resolve module prop-types from C:\Users\suhas\Documents\FacebookTest\node_modules\react-native-fbsdk\js\FBLikeView.js: Module does not exist in the module map or in these directories:
C:\Users\suhas\Documents\FacebookTest\node_modules
,   C:\Users\suhas\node_modules

Hey, thanks for reporting this issue!
It looks like your description is missing some necessary information. Can you please add all the details specified in the template? This is necessary for people to be able to understand and reproduce the issue being reported.
I am going to close this, but feel free to open a new issue with the additional information provided. Thanks!


Implement pasting image from clipboard for browsers that supports that

Fixes #3032
Implemented in new issue text area and when adding issue/pr comments

Codecov Report

Merging #5317 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5317      +/-   ##
==========================================
- Coverage   37.37%   37.34%   -0.03%
==========================================
  Files         312      312
  Lines       46379    46379
==========================================
- Hits        17332    17322      -10
- Misses      26567    26573       +6
- Partials     2480     2484       +4

Impacted Files
Coverage Δ

modules/process/manager.go
76.81% <0%> (-4.35%)
:arrow_down:

models/repo_indexer.go
44.49% <0%> (-3.82%)
:arrow_down:

models/repo_list.go
64.55% <0%> (+1.26%)
:arrow_up:

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 4c1f1f9...c9760bc. Read the comment docs.

It seems it's not work for me. Chrome 70.0.3538.77 on macOS

@lunny do you get any error in JavaScript console?

@lafriks no any error on console.

@lunny can you describe steps you did?

@lunny I just tested and it worked on both chrome and safari on Mac. Try pressing Ctrl+F5 or clearing cache, could be browser was using cached javascript version


Support FrameworkBarrier for GangExecution and Add Distributed TensorFlow Training Example

Maybe @sterowang should review logic of barrier.


v0: Monitoring

this pr lays the foundation for a client side monitoring system to evaluate the performance of an individual market aid instance. market aid instances must only utilize provided functionality of the market aid when dealing with the transfer of ERC20s in order to ensure proper accounting through the subgraph. the market aid subgraph can be found here and has been updated to be v2 compatible.
current functionality includes:

tracking historic ERC20 balances on a given market aid instance over set windows of time (6h/12h/24h/48h)
tracking market aid transactions and cumulative gas spend over set windows of time (6h/12h/24h/48h)
tracking market aid performance over set windows of time (6h/12h/24h/48h) - including gas spend

these various metrics are logged to the cli at the moment, with the goal of a client side gui to follow
this pr sets up a graphclient sdk within the monitoring folder that contains a variety of useful queries that can be utilized going forward for increased visibility into market aid activity. there is an associated data file that contains functions for interacting with these queries and handling rate limiting/automated pagination. currently, this monitoring system uses redux to handle state.
to get started, in the root of the project make sure to add an .env file that contains a MONITOR_AID_ADDRESS value corresponding to the address of the market aid you would like to monitor. then, simply run yarn build:monitor - this will handle the construction of the graphclient and running initial metrics in one go. after the graphclient is built, you can simply run yarn monitor for future metrics reporting. if for some reason you need to update the graphclient, or rebuild it, simply run graphclient:build
im excited to continue to iterate on this monitoring front and build out the gui to something that is easily accessible / understandable to any end user, regardless of technical expertise.
going forward, some short term metric goals that i have include:

[ ] expanded insights into outstanding offers and other real-time values
[ ] tracking historic fill and offer data
[ ] increased granularity & visualizations for historical erc20 balance changes & portfolio value

our index.ts needs to be cleaned up a bit to deal with async calls. this should be a pretty quick iteration to improve reporting and should happen before we merge. hope to report back on this front 🔜

updated index.ts to better utilize promises and added column for asset/quote portfolio % value in pnl readout


CODE RUB: ChatCompletionValidationException Class Update v2.10.1

Updated ChatCompletionValidationException class by removing constructors in which the error message was hard coded in. Updated all relevant service files to include message.

Closed Pull Request. Resolved in PR #556.


Getting npm ERR! code ENOLOCAL when using npx create-react-app some-app

Getting below errors when using npx create-react-app some-app:
npm ERR! code ENOLOCAL
npm ERR! Could not install from "B\AppData\Roaming\npm-cache_npx\38676" as it does not contain a package.json file.
It is working after using npm i create-react-app -g however it should work without this global install
Help me if anyone faced the same issue

Try running npm cache clean --force.

@narenbh This is an issue with npx. Your computer name has a space in it. [See this issue](https://github.com/zkat/npx/issues/146) in the npx repo for more info.


Fixed bug with youtube track details parsing

This must have been changed very recently as the JSON format returned by Youtube has changed slightly in structure. Added code to optionally pull "player" and "playerResponse" elements from the results map at the top level first rather than as child values.

Addresses Issue #688

Hey, I know this might sound stupid, but i can't manage to compile this using the gradlew.bat due to a compilation error.
lavaplayer\stream-merger\src\main\java\com\sedmelluq\lava\player\extras\stream\StreamAudioPlayer.java:291: error: constructor TrackStuckEvent in class TrackStuckEvent cannot be applied to given types;       dispatchEvent(new TrackStuckEvent(StreamAudioPlayer.this, track, thresholdMs));
Can you help me with building this?

@Janisbtw Yeah, I couldn't get it to compile without modifying the constructor, and that was the state the code was in when I cloned the repo. I just ignored it as it doesn't seem to affect anything I was using (primarily Youtube). I can look further into a fix for it, but I'd really like this fix to get pulled in first as it fixes completely broken Youtube functionality.

After fixing the constructor and the references in the other files I get this error
`Execution failed for task ':extensions:format-xm:compileJava'.

Could not resolve all files for configuration ':extensions:format-xm:compileClasspath'.
Could not resolve com.github.micromod:ibxm:a73.
Required by:
project :extensions:format-xm
Could not resolve com.github.micromod:ibxm:a73.
Could not get resource 'https://dl.bintray.com/gabixdev/kyoko/com/github/micromod/ibxm/a73/ibxm-a73.pom'.
Could not GET 'https://dl.bintray.com/gabixdev/kyoko/com/github/micromod/ibxm/a73/ibxm-a73.pom'.
The server may not support the client's requested TLS protocol versions: (TLSv1.2, TLSv1.3). You may need to configure the client to allow other protocols to be used. See: https://docs.gradle.org/6.8.2/userguide/build_environment.html#gradle_system_properties
Remote host terminated the handshake`
Do you mind uploading the build to a file hosting service?

I couldn't get the ibxm library compiling either. I was really focused only on the core library.

I couldn't get the ibxm library compiling either. I was really focused only on the core library.

Same, just fixed and builded the core to get it running. But i checked the ibxm lib and it looks like it's hosted on a private server or something?

Bintray has been closed ages ago. There's PR #676 which fixes mod player build breakage.

How do I use this to fix the issue? (dont know how to use pull requests)
I'm using maven to add lavaplayer as a dependency btw

How do I use this to fix the issue? (dont know how to use pull requests) I'm using maven to add lavaplayer as a dependency btw

Use jitpack with the PR author's fork and branch: https://jitpack.io/#idpromnut/lavaplayer/youtube_details_parse_bug-SNAPSHOT
Or Walkyst's fork, which isn't abandoned: https://github.com/Walkyst/lavaplayer-fork

Hello, how do I use this to fix the problem? I have added this as a dependency like
        <dependency>
            <groupId>com.github.idpromnut</groupId>
            <artifactId>lavaplayer</artifactId>
            <version>1.3.78</version>
        </dependency>

but the problem still pops up...

You need their branch as version, not 1.3.78. In this case youtube_details_parse_bug-SNAPSHOT

It works for single video, but still throws null when parsing playlist.

It works for single video, but still throws null when parsing playlist.

Ah, I don't use the playlist functionality, so did not test it. @God0nlyKnows can you grab a network capture of the JSON that is returned when playing a playlist and I can see if I can see what the issue is?

Ah, I don't use the playlist functionality, so did not test it. @God0nlyKnows can you grab a network capture of the JSON that is returned when playing a playlist and I can see if I can see what the issue is?

Hmm.. I've tested it now and it's working fine, probably they changed it again


Change dockerfile to use alpine instead of debian

I mostly did this for fun, but I prefer alpine images as they tend to be smaller.

Thanks for the pull request, @munntjlx! I will take a look at this soon.

Thank you for the PR, @munntjlx!
I just took a look at this, and the alpine-based image does end up significantly smaller than the debian-based one: 57MB vs 216MB, nearly a 4x reduction.
Frustratingly, the scan performance of the alpine-based image is several times slower! Using cpython as a casual benchmark (30GB of blobs), I see this alpine-based Docker image scan in about 100 seconds, whereas the debian-based Docker image takes about 30 seconds — a 3x performance difference! I re-ran this a handful of times to be sure.
I suspect the performance different is somehow due to using musl instead of glibc? Some quick searching online indicates that it may be due to a malloc implementation in alpine having much more contention in multithreaded settings than glibc: https://www.linkedin.com/pulse/testing-alternative-c-memory-allocators-pt-2-musl-mystery-gomes/. This sounds plausible, as Nosey Parker does make heavy use of thread-based parallelism.
Interesting to see that Nosey Parker can build with musl! That may be relevant for one day producing statically-linked binaries (which is trickier than one would like here, due to a number of native-code dependencies).
I would take this PR if the performance did not drop significantly, as the images are many times smaller. But at present, I value the Nosey Parker scan throughput over container size, so won't be merging this back.
It may be possible to sidestep the performance drop by switching Nosey Parker to use an alternative allocator such as mimalloc instead. (Nearly all the allocation that Nosey Parker does is in Rust code, not C++.) But this would be a larger-scale investigation.

I am sad that the performance was so much slower. You CAN build glibc things in alpine, you just have to add the right packages, which most people sometimes forget (theres been a package for glibc stuff for AGES in alpine). I mostly did it for the fun, to see how much smaller the image is. I will probably switch to your default, just that the scans will be so much sadder.

@munntjlx this was running on an x86_64 macOS machine. I'll also try on a big Linux machine.

@munntjlx switching the global allocator in Nosey Parker to use mimalloc does seem to restore the performance for me in the Alpine Docker image (#81). I'm going to do more performance validation with that in the near future, and if that is indeed a win overall, will then merge this PR. It will take a little while, but keep this PR open :)

The main reason I tend to prefer alpine is the smaller base, it does have a bit of strangeness (python for example) but once you get used to working with musl (I am an old hand at openwrt), I find that most things work (with a few exceptions). Thanks for being willing to entertain a MUSL build! The other 'old' complaint among the k8s folks was the lack of support for dns tcp (which has been fixed for about 6 months now), in that MUSL now directly supports TCP based dns requests.

FYI I see similar performance characteristics from the Alpine image on a 32-core Ubuntu 22.04 machine -- a 6x slowdown relative to the glibc-based Docker image (~500MB/s vs ~3GB/s scan throughput).
Similarly, switching the global allocator to mimalloc mostly fixes the slowdown there: (from ~500MB/s up to 2.6GB/s scan throughput within Docker containers).
Let me get the switchover to mimalloc in, and then get back to this PR.

Now that #88 is done can we have a 'separate' dockerfile for alpine?

@munntjlx Yes, a separate Dockerfile.alpine is my thought. Give me a bit and I'll take care of it.

@munntjlx I added the new Alpine-based Dockerfile as Dockerfile.alpine in 6bf13dc1eb2e2bd20a899b1a2150705f02d9870a, now on main.
I'm going to also update the GitHub Actions CI to automatically build both flavors of Dockerfile.

P.S. @munntjlx I did mark you as a co-author on that commit, so you should get credit for it.

Thank you!


fix: truncate only when allowed length exceeded

Thankfully that was caught by the E2E test in the release pipeline: https://github.com/Kong/kubernetes-testing-framework/actions/runs/3891847929/jobs/6644691387 😓

Codecov Report
Base: 6.58% // Head: 6.88% // Increases project coverage by +0.29% :tada:

Coverage data is based on head (672ec7e) compared to base (3b8c1a0).
Patch coverage: 100.00% of modified lines in pull request are covered.

:exclamation: Current head 672ec7e differs from pull request most recent head 0e3438e. Consider uploading reports for the commit 0e3438e to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##            main    #495      +/-   ##
========================================
+ Coverage   6.58%   6.88%   +0.29%
========================================
  Files          8       8
  Lines       1246    1250       +4
========================================
+ Hits          82      86       +4
  Misses      1159    1159
  Partials       5       5

Impacted Files
Coverage Δ

pkg/clusters/types/gke/builder.go
8.33% <100.00%> (+2.08%)
:arrow_up:

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

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


Naviagation issue if Blog URL is set with Upper and Lower case letters

Issue Summary
If blog URL is set with upper and lower case letters (e.g. https://MakeItYour.LIFE) the Naviation settings in the Design menu do not work properly.
Steps to Reproduce

Set blog url in config file to be with multiple cases
Try to create or change a navigation setting
Weird things get appended and replaced

Technical details:

Ghost Version: 1.0.2

Other
Not too serious of a bug. But something that was unexpected for me and could be handled internally I think.

Hmm... I think the only thing to do here is to validate the url is lowercase when entered in ghost config.
Uppercase domains aren't really a thing - I think this is the first of many problems you'd be likely to encounter configuring your blog this way.

Validating this (or forcing it to be lowercase automatically somewhere) would be fine I think.

Closing if favour of https://github.com/TryGhost/Ghost-CLI/issues/399


How to generate the such small ‘.MP4’ file.

It's convient to visualize the segment result with a video, but my video is some big, so i want to konw how to generate '.mp4' file like yours? I'm using 'cv2'.
Thanks for your reply~

i also using opencv video writer to output a video.


Extracted data can now be retrieved in a calllback

Also:

Writing on disk can be avoided.

The API for using returned data is the following:
// Returning extracted data in a callback
var options = {
	source: 'mySwf.swf',
	returnData: true
};

jeff(options, function (error, stats, extractedData) {
	// Uncovering convertion stats
	var nbFilesConverted = stats.files;
	var nbErrors         = stats.errors;

	// Fetching extracted data
	var imageNames = extractedData.imageNames;
	var image      = extractedData.images;
	var data       = extractedData.data;
});

@cstoquer what do you think?

API looks very nice 👍


Some results after filtering seem off

With the filter [1] we can see that the map shows values for Scotland and Northern Ireland but the barchart doesn't.
This is currently happening in dev at 22111164e9c2ed35755bbbdaff8da41db298333f  before merging #192 which is supposed to read the filters from the URL, which means the link below won't work right now but it can be useful to check this issue while reviewing #192 to see if that was the culprit.
[1]
http://localhost:5173/explorer/number/geo/installation_cost?categsGeoSortBy=total&categsStreamgraphsSorting=off&categsTimeGraph=trends&filterQuery=(installation_cost%3A(lte%3A35555.55656155046)%2Cproperty_feature_glazed_area%3A(lte%3A4.176890859817689%2Cgte%3A1.875174509320851)%2Cproperty_feature_number_habitable_rooms%3A(lte%3A21.77348690153568)%2Cproperty_feature_total_floor_area%3A(lte%3A928.6536092633653)%2Cproperty_energy_efficiency_floor%3A!(Unknown%2C'Very+Good'%2C'Very+Poor')%2Cinstallation_date%3A(lte%3A1577836800000%2Cgte%3A1356998400000))&interval=1y&numTimeGraph=percentiles&regionType=itl21_1&stackedBarsExtents=absolute&stringsGeoSortBy=total&stringsStreamgraphsSorting=off&stringsTimeGraph=trends&stringsTopCount=20&trendType=progressive

it doesn't seem to happen anymore, closing


Node LTS Compatibility

Node 6 is currently LTS. Can we make this Node 6 compatible?

It seems like this works fine in Node 6 if we just change the package.json node requirement to >=6. Is that something that is reasonable @hunterloftis ?


Add tutorial explaining how to rebuild preprocessed directories

Here I add an advanced tutorial that details how to rebuild the preprocessed glacier directories from the ground up. Related to https://github.com/OGGM/oggm/issues/1675.
This tutorial not only guides users through each preprocessing level but also advises on how and where personal data can be integrated. Instead of delving into the specifics of each function, it provides links to related notebooks for further exploration.
An area previously unaddressed in our tutorials is the utilization of different climate datasets. I've included a section on this topic in the current tutorial. However, to maintain its concise and overview nature, this section might be better suited for a separate, dedicated tutorial—possibly within the OGGM shop, where there's already a placeholder for climate data discussions.
I welcome any reviews, especially regarding the appropriateness of the links to other tutorials and whether any relevant tutorials have been overlooked.

I have started reviewing the other PR but will get into this one too this week!


Ability to use Basic Auth File

Describe the bug
Trying to use the new --kube-apiserver-arg to enable basic auth in k3s
To Reproduce
sudo k3s server --no-deploy traefik --kube-apiserver-arg basic-auth-file=/home/adrienm/pass.csv --kube-apiserver-arg v=9
Expected behavior
K3S server starting as expected
Error
panic: creating CRD store Unauthorized
goroutine 544 [running]: github.com/rancher/k3s/vendor/github.com/rancher/norman/stor/crd.(*Factory).BatchCreateCRDs.func1(0x400168ac20, 0x4001363e00, 0x3, 0x3, 0x400197ebe0, 0x58c4d20, 0x3747500, 0x4001363fb0, 0x0, 0x0)
/go/src/github.com/rancher/k3s/vendor/github.com/rancher/norman/store/crd/init.go:65 +0x258 created by github.com/rancher/k3s/vendor/github.com/rancher/norman/store/crd.(*Factory).BatchCreateCRDs 	/go/src/github.com/rancher/k3s/vendor/github.com/rancher/norman/store/crd/init.go:50 +0x9c
Full logs
Pastebin code
Without this flag it start as expected.

@adi90x The problem is that the rest of components are using the following kubeconfig /var/lib/rancher/k3s/server/cred/kubeconfig-system.yaml which is generated by k3s and contain different credentials than the one you specified in the basic auth file, this will not work because this changes the way k3s orchestrate the authentication between components.

Ok make senses, at the moment I am adding the content of my file to the one
generated by k3s and reloading the service, will stick to that solution
then.
Le mer. 1 mai 2019 à 19:20, Hussein Galal<EMAIL_ADDRESS>a
écrit :

@adi90x https://github.com/adi90x The problem is that the rest of
components are using the following kubeconfig
/var/lib/rancher/k3s/server/cred/kubeconfig-system.yaml which is
generated by k3s and contain different credentials than the one you
specified in the basic auth file, this will not work because this changes
the way k3s orchestrate the authentication between components.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rancher/k3s/issues/424#issuecomment-488348040, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACVGU5FSDUF6UDOYDKGWLTTPTHGMLANCNFSM4HJSSBSQ
.

Closing due to age


Instructions that were not found in the kit

Hello,
missing instructions, if these instructions were available, it would give extended flexibility to the engine.
arpl
bound
call
clac
cldemote
clflushopt
clts
clwb
cmovcc
cmps
cmpsd
encls
enter
fbld
fbstp
fclex
fcmovcc
fldenv
fnclex
fnclex*
fninit*
fnsave
fnsave*
fnstcw
fnstcw*
fnstenv
fnstenv*
fnstsw
fnstsw*
frstor
fsave
fstenv
fstsw
fxrstor
fxrstor64
fxsave
fxsave64
hlt
ibvpermilps
in
ins
insb
insd
insw
int
int n
int1
int3
into
invd
invept
invlpg
invpcid
invvpid
iret
iretd
iretq
lar
lds
leave
les
lfs
lgdt
lgs
lidt
lldt
lmsw
lods
lodsb
lodsd
lodsq
lodsw
loop
loopcc
loope
loopne
lsl
lss
ltr
mov
movdir64b
movdiri
movq
movs
movsd
nop
out
outs
outsb
outsd
outsw
pabsq
pmaxsq
pmaxuq
pminsq
pminuq
pmovsx
pmovzx
pmullq
pop
popfq
prefetchh
psraq
ptwrite
push
pushfq
rdfsbase
rdgsbase
rdpid
rdpkru
repe
repne
repnz
repz
rsm
scas
setcc
sgdt
sidt
sldt
smsw
stos
str
swapgs
syscall
sysenter
sysexit
sysret
test
tpause
ud
umonitor
umwait
vbroadcast
verr
verw
vextractf32x4
vextracti32x4
vgatherdpd
vgatherdps
vgatherqpd
vgatherqps
vmaskmov
vmcall
vmclear
vmfunc
vmlaunch
vmptrld
vmptrst
vmread
vmresume
vmwrite
vmxoff
vmxon
vpbroadcast
vpbroadcastm
vpgatherdd
vpgatherdq
vpgatherqd
vpgatherqq
vpmaskmov
vshuff32x4
vshuff64x2
vshufi32x4
vshufi64x2
wrfsbase
wrgsbase
wrpkru
xabort
xacquire
xbegin
xchg
xend
xlat
xrelease
xrstor
xrstor64
xrstors
xrstors64
xsave
xsave64
xsavec
xsavec64
xsaveopt
xsaveopt64
xsaves
xsaves64
xsetbv
xtest
xtest

ENCLS
ENCLS[EADD]
ENCLS[EAUG]
ENCLS[EBLOCK]
ENCLS[ECREATE]
ENCLS[EDBGRD]
ENCLS[EDBGWR]
ENCLS[EEXTEND]
ENCLS[EINIT]
ENCLS[ELBUC]
ENCLS[ELDBC]
ENCLS[ELDB]
ENCLS[ELDU]
ENCLS[EMODPR]
ENCLS[EMODT]
ENCLS[EPA]
ENCLS[ERDINFO]
ENCLS[EREMOVE]
ENCLS[ETRACKC]
ENCLS[ETRACK]
ENCLS[EWB]
ENCLU
ENCLU[EACCEPTCOPY]
ENCLU[EACCEPT]
ENCLU[EENTER]
ENCLU[EEXIT]
ENCLU[EGETKEY]
ENCLU[EMODPE]
ENCLU[EREPORT]
ENCLU[ERESUME]
ENCLV
ENCLV[EDECVIRTCHILD]
ENCLV[EINCVIRTCHILD]
ENCLV[ESETCONTEXT]

GETSEC[CAPABILITIES]
GETSEC[ENTERACCS]
GETSEC[EXITAC]
GETSEC[PARAMETERS]
GETSEC[SENTER]
GETSEC[SEXIT]
GETSEC[SMCTRL]
GETSEC[WAKEUP]

INVEPT
INVVPID
VMCALL
VMCLEAR
VMFUNC
VMLAUNCH
VMPTRLD
VMPTRST
VMREAD
VMRESUME
VMWRITE
VMXOFF
VMXON

Thank you for your advice.
I think xbyak is used for user application, so it does not support mnemonics for system-programming.
And Xbyak already supports call, cmps, cmpsd, fcmovcc, pmaxsq, vextractf32x4, vgatherdpd and etc. in your list.
I think again if there are especially mnemonics you want.

Hi herumi,
sad that this is the case, because it can be not only system programming, for example manual exception handling, and how are things with basic instructions like - pushfq, popfq, int, syscall, sysenter, lodsq, lods ... In any case, I would be very happy if the missing instructions were present. The project is very cool! One more question, is there a possibility in the project to output just in time code to the bin file, this is necessary for dynamic code verification, in this case, I could test the entire set of instructions for correct coding.
Thanks for the support.
p.s. In general, the main purpose of assembler is system programming =) One could sketch a small pdf file with a description of how to add your own instructions. That would be very nice!

I see, I'll support pushfq, popfq, int, syscall, sysenter, lodsq, lods ... at first.

is there a possibility in the project to output just in time code to the bin file

How about saving (code.getCode(), code.getSize()) for CodeGenerator code?

I'm now appending some mnemonics at dev branch.
Please tell me high priority list if you have. And do you need mnemonics which requires level 0 privilege?

Hello,
was away these days, some instructions exist, they are simply encoded differently, for example, as SETAE r / m8 * SETBE r / m8 *, but not all, I re-checked the list ..
A list of instructions that are exactly required for daily work, priority from top to bottom ..
pushfq
popfq
enter
leave
int
test
syscall
sysenter
sysexit
sysret
stosq
loop
loopcc
loope
loopne
movsq
nop
pabsq
pmaxsq
pmaxuq
pminsq
pminuq
pmullq

//And do you need mnemonics which requires level 0 privilege?
no, not necessarily =)

//How about saving (code.getCode(), code.getSize()) for CodeGenerator code?
Thank you, I think this is what I need!

The following mnemonics are already supported.
pushfq
popfq
int       ; use int_ instead of int
syscall
sysenter
sysexit
sysret
stosq
movsq
nop
pabsq
pmaxsq
pmaxuq
pminsq
pminuq
pmullq

The following mnemonics are old and very slow.
eg. dec ecx / jnz lp is faster than loop
Could you tell me the reason why you want to use them?
enter
leave

loop
loopcc
loope
loopne

These instructions are often used in textbooks and examples.
In non-speed-critical procedures or code generators.
leave is often used without enter.
Tribute to the past =)

But in general I completely agree! =) The main goal is educational ..

I see. I've appended those mnemonics at dev.
If you have other mnemonics you want, please show me them.

you have other mnemonics you want, please show me them.

fbld fbstp fclex fldenv fnclex fnsave fnstcw fnstenv fnstsw frstor fsave fstenv fstsw fxrstor fxsave hlt invd invlpg invpcid lds - LDS/LES/LFS/LGS/LSS—Load Far Pointer REP/REPE/REPZ/REPNE/REPNZ—Repeat String Operation Prefix lods

hlt invd invlpg invpcid requires privilege level 0.
Do you need them really?

//And do you need mnemonics which requires level 0 privilege?
no, not necessarily =)

hlt invd invlpg invpcid require privilege level 0.
Do you need them really?

Hello,
no, I wrote it automatically =) Thank you so much for the work! I will test it with multifunctional code generation!
Regards


Corrected dead mentioning of Signal.Channel

Removed dead mentioning of Signal.Channel in the documentation Graphics.Input.Field. Just followed the steps here:
https://github.com/elm-lang/core/issues/231#issuecomment-97546945

Great, thank you!

Thanks, was about to submit a PR myself!


issue with arangoimp --file : unexpected identifier

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[ ] 2.8
[ x] 3.1.11
[ ] self-compiled devel branch

On this operating system:

[ ] DCOS on

[ ] AWS
[ ] Azure
[ ] own infrastructure

[ ] Linux

[ ] Debian .deb
[ ] Ubuntu .deb
[ ] SUSE   .rpm
[ ] RedHat .rpm
[ ] Fedora .rpm
[ ] Gentoo
[ ] docker - official docker library
[ ] other:

[ ] Windows, version:
[x ] MacOS, version:

this is an AQL-related issue:
[ ] I'm using graph features
I'm issuing AQL via:

[ ] web interface with this browser:     running on this OS:
[x ] arangosh
[ ] this Driver:

I've run db._explain("<my aql query>") and it didn't shed more light on this.
The AQL query in question is:
error when using arangimp
**<IP_ADDRESS>:8529@EDW> arangoimp --file "data.csv" --type csv --collection "xxx_data"
JavaScript exception: SyntaxError: Unexpected identifier
!arangoimp --file "data.csv" --type csv --collection "xxx_data"
!            ^^^^
stacktrace: SyntaxError: Unexpected identifier
arangoimp does not understand --file identifier**
The issue can be reproduced using this dataset:
Please provide a way to create the dataset to run the above query on; either by a gist with an arangodump, or `db.collection.save({my: "values"}) statements. If it can be reproduced with one of the ArangoDB example datasets, it's a plus.
Foxx
this is a web interface-related issue:
I'm using the web interface with this browser:     running on this OS:

[ ] authentication is enabled?
[ ] using the cluster?
[ ] _system database?

These are the steps to reproduce:

open the browser on http://<IP_ADDRESS>:8529
log in as ...
use database [ ] _system [ ] other:
click to ...
...

The following problem occurs: [Screenshot?]
Instead I would be expecting:
this is an installation-related issue:
Describe which steps you carried out, what you expected to happen and what actually happened.

Looking at the invocation and the error message
<IP_ADDRESS>:8529@EDW> arangoimp --file "data.csv" --type csv --collection "xxx_data"
JavaScript exception: SyntaxError: Unexpected identifier
!arangoimp --file "data.csv" --type csv --collection "xxx_data"
! ^^^^
stacktrace: SyntaxError: Unexpected identifier

it seems that you try to invoke arangoimp from inside the ArangoShell (arangosh). That will not work.
arangosh is a JavaScript shell, so you can run JavaScript commands and programs in it, for example:
db._create("test");
db.test.insert([ { _key: "foo" }, { _key: "bar" }, { _key: "baz" } ]);

The line
arangoimp --file "data.csv" --type csv --collection "xxx_data"

is no valid JavaScript, thus the error message.
arangoimp is a separate binary that is shipped with ArangoDB in most distributions. The arangoimp invocation command should thus be entered and run in bash or another Unix shell (or even MS-DOS).


Link to AATL trust list

By doing some simple research the link to the AATL from Adobe seems to be: http://trustlist.adobe.com/tl12.acrobatsecuritysettings
This is a signed PDF file with an XML attachment that represents the trust list.
Is this link left intentionally?

Thanks for your contribution, this link was not left out intentionally, we are collecting the information and relying on contribution such as those by yourself.
Would like to create a PR to update https://github.com/pkic/ltl/blob/main/data/adobe.yaml ?
See also: https://github.com/pkic/ltl/blob/main/.github/CONTRIBUTING.md

Okay, just to be sure I contacted Adobe regarding this link and that's what they replied:

Hi Jan,
the AATL is not available for download from non-Adobe software. Doing so would infringe Adobe’s IP as it is copyrighted material.
A link to the AATL is missing in the PKI Consortium website because the AATL is not open sourced.
As said before, the AATL is a restricted service only intended to be distributed as part of the Adobe Acrobat and Acrobat Reader software, more or less like a regular product update.
Kind regards

...so, we can close the issue (for now). Maybe Adobe will change their mind in the future.


Add support for topLevelAwait option in experiments

What problem does this feature solve?
Adds compatibility to topLevelAwait option from wepback
What does the proposed API of configuration look like?
const config = {
  experiments: {
    topLevelAwait: true,
  },
};

This issue blocks migration of following Webpack tests

https://github.com/webpack/webpack/blob/main/test/configCases/graph/issue-11770/webpack.config.js

duplicate of #3889


[MRESOLVER-283] Shared executor

Now that we have shutdown hook, and context parameters, this is possible.
Simplest possible but shared executor implementation.

https://issues.apache.org/jira/browse/MRESOLVER-283

@caiwei-ebay ping

@cstamas
The PR looks good to me. Thanks for the quick fix.
My main concern is about how to tell user pom/metadata.xml is downloaded in parallel now with BF collector.
As the special param "aether.dependencyCollector.bf.threads" has to be removed using SharedExecutor, could you put some lines to explain the parallel download behavior in param "aether.dependencyCollector.impl" where you've introduced BF and DF.  With such information, user may try the BF and feedback so we can discuss whether BF could be the default based on the feedbacks.

Reworked to:

have split pools (service can decide using service "key")
pools are stored in session, so each session has own pool(s)
this merely reflects what was before, but all is centralized.

Sharing executors must be done with caution.
Is there any way that the executor may be running out of threads to service a call ? For example, resolving multiple files in parallel, each file being resolved in its own thread, then downloading dependencies and running out of threads ?
I think using a more general ForkJoinPool which supports work stealing would be safer to avoid running into such problems.

I kinda agree with @gnodet  here, and will "tone down" this PR to NOT SHARE executors, merely will introduce this facade but instances will be handled as before (as even now, it is not 100% what it was before: collector had per arg instance, now shared). And I cannot certainly asnwer the question "Is there any way that the executor may be running out of threads to service a call".
OTOH, I disagree with forkjoinpool: all these threads will do (probably blocking, maybe inifinite) HTTP IO, not to mention the refactoring needed to use them (most of "client" code remained unchanged here, using plain Runnable and Callable).

Hence, this PR will change to merely "centralize executor creation and (probably) use ResolverExecutor shim instead directly exposing ExecutorService iface"...

OTOH, I disagree with forkjoinpool: all these threads will do (probably blocking, maybe inifinite) HTTP IO, not to mention the refactoring needed to use them (most of "client" code remained unchanged here, using plain Runnable and Callable).

ForkJoinPool is capable of handling blocking operations, just like a standard ThreadPoolExecutor. The only problem is when you don't use your own instance and reuse the system provided instance with blocking operations.  The only difference between a ForkJoinPool and a ThreadPoolExecutor is that the former has more features such as thread affinity, work stealing and fork/join operations.
I think a single thread pool is a good idea, but it has to be implemented in a correct way to avoid any thread exhaustion. I.e. use a single fork/join pool and use the appropriate methods to dispatch all the work, which should ensure proper ordering of tasks execution.  I've done a similar thing in https://github.com/apache/maven/pull/803 (which I need to finish).

Ok, now I feel I should not push this anymore, as I agree now with use of forkJoinPool. Hence, will close this PR, as that change would be bigger, but I want 1.9.0 out as soon as possible....


Fix warning when validating GET requests (#13)

Closes #13.

Note: Build is currently broken due to 2FA in UK Sandbox, see #18.


Bring back "Per-category Display" from v1.9.4

Describe your suggested feature
I don't know if this is already planned to be brought back after completing shifting to Mihon
If Category 1 >Display >Cover only grid // then it should only affect Category 1 only and not all other categories

Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extension's repository.
[X] I have updated the app to version 1.10.2.
[X] I will fill out all of the requested information in this form.

No plans to bring it back


Do we have any implementation demo anywhere, all the examples I am seeing are just retrieving data in one shot with parent child relationship and loading them at once. I wish to make an API call to get child elements, any help is appreciated thank you :)

I was struggling with this too but I have found a solution:
// NavTree object
$scope.files = [];
var tree;
$scope.my_tree = tree = {};

// Load root folder
Factory.method().then(function (response) {
	$scope.dropbox = response.data;
});

/**
 * Load branch
 * @param  {string} branch
 */
$scope.loadBranch = function (branch) {
	Factory.method().then(function (response) {
		// Add branches
		response.forEach(function (value, key) {
			tree.add_branch(branch, value);
		});
	});
}

And your HTML:
<abn-tree tree-data="files"
	icon-leaf="fa fa-file"
	icon-expand="fa fa-folder"
	icon-collapse="fa fa-folder-open"
	on-select="loadBranch(branch)"
	tree-control="my_tree"
></abn-tree>

The implementation of the tree variable is a bit weird but at least it works this way. Only thing I'm still struggling with is that you can't let navtree know whether it's a file or a directory programmatically. Apparently this plugin checks if there is a children key in the object if true a folder icon is assigned. Problem is that when you use lazy loading the children key is never available. Also sending an empty children key isn't working because you'll see an empty branch.
If you find a solution for this please contact me.

I figure out this problem by adding children to the branch when it is clicked.the solution could click this page. http://blog.csdn.net/baidu_35407267/article/details/52457779

thank you @guguji5. I have taken your solution and translated to english and adapted to the sample test tree. In this case, I have added the i_click function to the Venezuela branch under South America
                      label: 'South America',
                      children: [
                        {
                            label: 'Venezuela',
                            noLeaf: true,
                            //children: ['Caracas', 'Maracaibo'],
                            i_click: pull_data
                        }, {
                            label: 'Brazil',
                            children: ['Sao Paulo', 'Rio de Janeiro']
                        }, {
                            label: 'Argentina',
                            children: ['Buenos Aires', 'Cordoba']
                        }
                      ]

Within the abn_tree_directive.js, this is the change to add i_click(row) to the ng-click event replacing the original a.branch.expanded = !a.branch.expanded on line 11 next to the image  element's ng-click
              template: "
\n
 {{ row.label }}
\n
",
Then add the i_click function to abn_tree_derective.js links (I put this right after scope.user_clicks_branch function definition
                  scope.i_click = function (a) {
                      a.branch.expanded = !a.branch.expanded; // Expand or collapse this branch
                      scope.user_clicks_branch(a.branch); // get the corresponding node to the plus sign, because the current state may not pick your point + number of nodes and this could create a node error
                      if (a.branch.i_click) {
                          // This is the way to call the definition of the controller, if the incoming data has i_click field, it is executed
                          a.branch.i_click(a);
                      }
                  };

In the controller code for the tree control add the function pull_data which the Venezuela i_click value was assigned. This is the code that will execute when the Venezuela branch is expanded. Note: you will need to have this function defined before assigning it to the Venezuela branch to avoid the error of pull_data function is not defined.
                // The following is the event triggered by the plus sign, first request data, loop, and add to the current node below.
                //For testing purposes, I commented out the http promise (the url end point @guguji5 provided is not responding) and hardcoded a sample response for an http promise call
                pull_data = function (a) {
                    if (a.branch.children.length < 1) {
                        //hardcode sample response for what would be an http promise returning data asynchronously
                        response = [
                                    { label: 'misko', isLeaf: 1 },
                                    { label: 'Thomas', isLeaf: 1 },
                                    { label: 'Linda', isLeaf: 1 },
                        ];

                        //$http.post('http://pings.iok.la:35612/oapm/sys/org/treelist', { "body": { "parentid": /a.branch.id } }).success(function (response) {
                        var b = tree.get_selected_branch();
                        angular.forEach(response, function (value, key) {
                            if (value.isleaf == 0) {
                                value.noLeaf = true;
                            } else {
                                value.noLeaf = false;
                            }
                            value.i_click = pull_data;
                            //value.onSelect = last_selected;
                            return tree.add_branch(b, value)
                        });
                        //})
                    }
                };

Result is

For anyone interested here is the same pull_data function using an $http post promise to retrieve data asynchronously
                // The following is the event triggered by the plus sign, first request data, loop, and add to the current node below.
                pull_data = function (a) {
                    if (a.branch.children.length < 1) {
                        //response = [
                        //            { label: 'misko', isLeaf: 1 },
                        //            { label: 'Thomas', isLeaf: 1 },
                        //            { label: 'Linda', isLeaf: 1 },
                        //];
                        var b = tree.get_selected_branch();
                        $http({
                            url: "\\TFVC\\TestTreeData",
                            method: "POST",
                            data: { branchId: a.branch.id }
                        }).then(function mySuccess (response) {

                            angular.forEach(response.data, function (value, key) {
                                if (value.isleaf == 0) {
                                    value.noLeaf = true;
                                } else {
                                    value.noLeaf = false;
                                }
                                value.i_click = pull_data;
                                //value.onSelect = last_selected;
                                return tree.add_branch(b, value)
                            });
                        }, function myError(response) {
                            var errvalue={
                                              label: "Error: " + response.statusText,
                                              noLeaf: false
                                          };
                            return tree.add_branch(b, errvalue);
                        });
                    }
                };

Controller action to retrieve data is
        [HttpPost]
        public JsonResult TestTreeData(string branchId)
        {
            var returnData = new List<ChildData>();
            returnData.Add(new ChildData() { label = "misko", isLeaf = 1 });
            returnData.Add(new ChildData() { label = "Thomas", isLeaf = 1 });
            returnData.Add(new ChildData() { label = "Linda", isLeaf = 1 });

            return Json(returnData);
        }

public class ChildData
{
    public string label { get; set; }
    public int isLeaf { get; set; }
}

@pneborg I am so sorry to see your message just now,my project is to heavy to have little time this week.is that you have figured it out?

Hi @guguji5 Thank you for providing your initial answer. It was great to have this question resolved. All I did was translate it and set up an example. 👍


Cosmic/dw 8242/integrate mt pelerin fia

Description
Added Mt Pelerin fiat ramp to the list of added ramps, notably, this one does not have KYC for non us customers. Users will be able to select a currency and get redirected to the ramp.
Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

Risk

Higher risk
Testing

Go to the ramps and select Mt Pelerin, and fill out the information and hit the continue button.
Screenshots (if applicable)

closing in favor of https://github.com/shapeshift/web/pull/2529


Fix crafting steel tiles requiring silk

About the PR

I think the title says all that needs to be said.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: PixelTK

fix: Nanotrasen have fixed their accidental breakage of time space continuum, making sure that steel tiles do indeed take steel to create.

bro, just make cl like fixed *bug name*😭😭😭

No

bro, just make cl like fixed *bug name*😭😭😭


Renaming Process

Checklist:

[x] rename this package
[x] open PR to registry to change the old Tapir URL to the new Mooncake one - https://github.com/JuliaRegistries/General/pull/115400
[x] rename everything in this package + declare a breaking change - #256
[x] register package under new name
[ ] create new ADType which uses Mooncake, and have it use a generic Mooncake config object, so that we can change the config arguments if necessary
[ ] uncomment various references to ADTypes.Mooncake throughout this package (ctrl-f for AutoMooncake to find them all), and re-enable tests (these had to be commented out during the transition period)

Note that the new AutoMooncake struct cannot be something like
struct AutoMooncake{C<:Mooncake.Config}
    config::C
end

because that would force ADTypes to depend on Mooncake. However you can copy what we did for other backends and leave the type parameter free, while specifying exactly what it should be in the docstring.


No readme info in upm default branch

Currently to support upm we had to update the default branches to the upm specific one, unfortunately this also caused us to lose the nice readme landing page from dev/master.

So do we transfer the ReadMe to the UPM branch as well, or is this likely to have knock on issues.
Alternative would be to have a simple link follow to the main XRTK Core project Wiki (although for extensions that could get messy)

We do. I've already fixed it in the CI/CD workflows.
They copy the files and place them in the correct location at build time, then commits them back to the branch.


Crash when compiled with recent nightly

Since some time back, alacritty crashes when compiled with nightly (and works fine with stable). I reported the issue to rustc and it turned out to be an UB in a dependency.
I updated the Cargo.lock in a local checkout. And the Illegal Instruction disappeared… to be replaced by SEGFAULT. It is probably some other UB in another dependency (I hear quite few of them surfaced after some recent change), but I didn't find out which one. I'm not sure if there are any steps to be taken at the alacritty side, except watching out for updates of the dependencies, but I'm reporting it anyway, to be known.
The backtrace I get now is quite useless:
(gdb) bt
Python Exception <class 'gdb.error'> Dwarf Error: Cannot find DIE at 0xb4 referenced from DIE at 0x1e47b5 [in module /home/vorner/.cargo/bin/alacritty]:
#0  0x0000000000000000 in ?? ()

Which operating system does the issue occur on?
Linux
If on linux, are you using X11 or Wayland?
X11

Are you using the latest nightly? If so, please post the modified Cargo.lock that you're using.

It's this one (simply run cargo update): https://github.com/vorner/alacritty/blob/master/Cargo.lock
It was the latest nightly at the time of the report. Now I updated and wait for the compilation to finish. I believe the crash would happen only on a release build, but once the release compilation finishes, I'll check that theory.

Yes, it happens only on release build, debug build works. Which supports the suspicion there's one of the newly surfaced UBs in one of the dependencies.

Hi, this has been debugged already: https://github.com/rust-lang/rust/issues/46158
There was UB in x11-dl apparently, is it possible to update the dependency version?

No, that's not it. As I wrote above, I tried compiling with updated dependencies. There must be more than this one UB somewhere, the original error disappeared (illegal instruction) and was replaced by a different one (segfault).

Hm weird, I'm not getting any other error running an optimized build after  cargo update here.

I just tried again. I updated nightly, did cargo update, checked Cargo.lock links to 2.17.0 version, removed the whole target directory just to make sure and did CARGO_INCREMENTAL=0 cargo install -f. It still crashes:
$ alacritty
Segmentation fault (core dumped)

This is all I manage to get out of gdb (any ideas for more?):
Thread 1 "alacritty" received signal SIGSEGV, Segmentation fault.
0x0000000000000000 in ?? ()
(gdb) bt
Python Exception <class 'gdb.error'> Dwarf Error: Cannot find DIE at 0xb4 referenced from DIE at 0x20953b [in module /home/vorner/.cargo/bin/alacritty]:
#0  0x0000000000000000 in ?? ()
(gdb) info threads
  Id   Target Id         Frame
* 1    Thread 0x7ffff7f80e00 (LWP 25889) "alacritty" 0x0000000000000000 in ?? ()
  2    Thread 0x7fffd65ff700 (LWP 25896) "radeon_cs:0" 0x00007ffff70ba926 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib64/libpthread.so.0
  3    Thread 0x7fffd5bf2700 (LWP 25897) "disk_cache:0" 0x00007ffff70ba926 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib64/libpthread.so.0
(gdb) thread 2
[Switching to thread 2 (Thread 0x7fffd65ff700 (LWP 25896))]
#0  0x00007ffff70ba926 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib64/libpthread.so.0
(gdb) bt
#0  0x00007ffff70ba926 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib64/libpthread.so.0
#1  0x00007fffe9e1710b in ?? () from /usr/lib64/dri/r600_dri.so
#2  0x00007fffe9e17017 in ?? () from /usr/lib64/dri/r600_dri.so
#3  0x00007ffff70b390a in start_thread () from /lib64/libpthread.so.0
#4  0x00007ffff6bd479f in clone () from /lib64/libc.so.6
(gdb) thread 3
[Switching to thread 3 (Thread 0x7fffd5bf2700 (LWP 25897))]
#0  0x00007ffff70ba926 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib64/libpthread.so.0
(gdb) bt
#0  0x00007ffff70ba926 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib64/libpthread.so.0
#1  0x00007fffe9e1710b in ?? () from /usr/lib64/dri/r600_dri.so
#2  0x00007fffe9e17017 in ?? () from /usr/lib64/dri/r600_dri.so
#3  0x00007ffff70b390a in start_thread () from /lib64/libpthread.so.0
#4  0x00007ffff6bd479f in clone () from /lib64/libc.so.6

Any ideas what to try? Or any „stale“ cache or something I might have somewhere?
An empty window appears just before the crash.
I'm adding an strace log in case it is of any use to anybody. But I have no idea what to try next ‒ if there are any things I might experiment with, I'd be glad to hear them.
strace.log

Hmm.  Do you override RUSTFLAGS env var somewhere?
You could try to play around with the [profile.release] block in Cargo.toml and see if removing lto or forcing different opt-level (opt-level = {0,1,2,3} changes something.

Sometimes I do set RUSTC_FLAGS (I think this is the correct one) to -C target-cpu=native. But I unset the variable for all the above experiments. So unless cargo looks somewhere and reuses something from a cache I don't know about, it shouldn't be influenced.
lto seems to have no effect. It crashes with opt-level 2 and 3, works fine with opt-level 1 (and, assuming, 0, as that one is used in debug).

I forgot about the sanitizers!
https://users.rust-lang.org/t/howto-sanitize-your-rust-code/9378/2
ASAN_OPTIONS=detect_odr_violation=0  RUSTFLAGS="-Z sanitizer=address"    cargo run --release  --target x86_64-unknown-linux-gnu if you are on linux

I tried playing with the sanitizers a bit. This is getting interesting :-|.

It doesn't crash with address sanitizer.
It is killed with the memory sanitizer on something (logs are below).
It crashes with segfault with leak sanitizer.
It doesn't crash, but reports some errors with the thread sanitizer.

Is it possible the sanitizer runtime prevents some optimisations in some cases?
sanitizers.txt

From what I have seen, some optimizations might be disabled by the sanitizers because they are made impossible due to the way the sanitizers instruments the code. It might also happen that optimizations change sanitization (make some bugs harder to reproduce or mess with stacktraces).

I get illegal instruction if I compile with nightly and I get a segfault if I do cargo update first.
Program received signal SIGSEGV, Segmentation fault.
0x0000000000000000 in ?? ()
(gdb) bt
#0  0x0000000000000000 in ?? ()
#1  0x00005555557d3a5a in glutin::api::glx::ffi::glx_extra::{{impl}}::SwapIntervalEXT
    (interval=1, self=<optimized out>, dpy=<optimized out>, drawable=<optimized out>)
    at /home/niklas/git/alacritty/target/release/build/glutin-cdc7f667a938aca8/out/glx_extra_bindings.rs:591
#2  glutin::api::glx::{{impl}}::finish (self=..., window=<optimized out>)
    at /home/niklas/.cargo/registry/src/github.com-1ecc6299db9ec823/glutin-0.9.2/src/api/glx/mod.rs:247
#3  glutin::platform::platform::x11::{{impl}}::new (window_builder=...,
    events_loop=<optimized out>, pf_reqs=<optimized out>, gl_attr=<optimized out>)
    at /home/niklas/.cargo/registry/src/github.com-1ecc6299db9ec823/glutin-0.9.2/src/platform/linux/x11.rs:223
#4  0x00005555557d479d in glutin::platform::platform::{{impl}}::new (
    window_builder=..., events_loop=<optimized out>, pf_reqs=<optimized out>,
    gl_attr=<optimized out>)
    at /home/niklas/.cargo/registry/src/github.com-1ecc6299db9ec823/glutin-0.9.2/src/platform/linux/mod.rs:52
#5  glutin::{{impl}}::new (window_builder=..., context_builder=...,
    events_loop=0x7fffffff99b0)
    at /home/niklas/.cargo/registry/src/github.com-1ecc6299db9ec823/glutin-0.9.2/src/lib.rs:322
#6  0x000055555570273a in alacritty::window::{{impl}}::new (title=...)
    at src/window.rs:196
#7  0x00005555556f214d in alacritty::display::{{impl}}::new (config=0x7fffffffd788,
    options=0x7fffffffd348) at src/display.rs:140
#8  0x000055555564e23d in alacritty::run (config=..., options=0x7fffffffd348)
    at src/main.rs:95
#9  0x000055555564db71 in alacritty::main () at src/main.rs:45
#10 0x000055555584f98e in panic_unwind::__rust_maybe_catch_panic ()
    at /checkout/src/libpanic_unwind/lib.rs:101
#11 std::panicking::try<(),closure> () at /checkout/src/libstd/panicking.rs:459
#12 std::panic::catch_unwind<closure,()> () at /checkout/src/libstd/panic.rs:365
#13 std::rt::lang_start () at /checkout/src/libstd/rt.rs:58
#14 0x00007ffff6afe830 in __libc_start_main (main=0x555555651230 <main>, argc=1,
    argv=0x7fffffffdc98, init=<optimized out>, fini=<optimized out>,
    rtld_fini=<optimized out>, stack_end=0x7fffffffdc88) at ../csu/libc-start.c:291
#15 0x000055555557ce09 in _start ()

I've just updated to the latest nightly and tested my master and it seems to work.
Rust: rustc 1.24.0-nightly (cddc4a62d 2017-12-26)
Alacritty: db4e6510384b25ea04c2abd9d311f8633aca621c


bug: bad argument #1 to 'get_line_col' (string expected, got nil)

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.11.0-dev-130+gefa45832e
Operating system/version
Ubuntu 22.04 lts
Describe the bug
Lazy version: 12.1.0

Extremely verbose output in notify of the above. 1 or 2 notifications of the same message at a time

^output of Notifications
Steps To Reproduce

In a file (ts file in my case), escape from insert to normal mode after making changes floods my screen with these notifs

Expected Behavior
No big red notifications after file changes
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  "folke/LazyVim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

I've tried replacing typescript-tools with the lazyvim typescript extra but still get the same errors

Same issue here, with several file types.
Neovim 0.10.0

Please update trouble. should be fixed


Prove nonimplications from equation 3342

null
Closes #662

Nice simplification from polynomials to Option (ℕ × ℕ).  The operations I had of multiplication by 2 and multiplication by t indeed could just as well have been the two standard shifts on Option (ℕ × ℕ).


How to Get Rid of Highlight.js?

As you can see here, highlightjs adds about half a megabyte to the bundle when vue-simple-markdown is used.
Would be great if there’d be a second build without it, or if the highlight library would need to be injected into it by the user explicitly.

+1 for this. I am not actually using any code/syntax highlighting so ideally I would be able to dump Highlight.js. it's currently tripling the size of my vendor bundle... :(

Is there any way or response ? Me too not using highlight :+1:

@ByamB4 I personally ended-up using marked instead 🤷


ResourceQuota used percentage is wrong if total expressed with "k" suffix

Describe the bug
The used percentage of resourcequota is wrong in case the total is in terms of thousands, and the used is actual value.
eg: if the total is 5k and the used is 5, it reports 100% used.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
The usage percent computation should take the suffix into account. 5/5k is 0.1%
Screenshots

Environment (please complete the following information):

Lens Version: 3.5.0
OS: [e.g. OSX]
Installation method (e.g. snap or AppImage in Linux): homebrew

related to #465


Update feature_request.md

Added [FEATURE] as start of title for all feature request issues. So all issues following the issue template would have either [FEATURE], [DOC] or [BUG] and we can easily see issues that were opened without templates and request user to use the templates.
Alternatively we could use [ENH] as a abbreviation which is e.g. used by pandas.

Yes let's use three-letter abbreviations, i.e. [ENH], we already use [DOC]


Update discussion-group.md

Fixing time zone chaos and updating the dates for future ones.

/fix:format


Examples in readme don't return the stream

None of the examples in the readme return the stream. This could cause problems for beginners.

Thanks for the report @callumlocke. I'm in transit ATM but PRs are welcome.


ion2-calendar is not working in ios device and safari brwoser

ion2-calendar is not working in ios device and safari brwoser

@anshu300 this is probably going to need more details since it works for me in Safari. Is it safari for Os X or ios? Any errors in the console? Any other hints?


chore: attempt to fix the version problems of replicationgroups (#1)

Description of your changes

Fixes #2087
I have:

[x] Read and followed Crossplane's [contribution process].
[x] Run make reviewable test to ensure this PR is ready for review. <--- Doesn't exist and the crossplane contribution guide seems specific for the crossplane core

@MisterMX - Linting fixed


CVSB-19004

Signed-off-by: Tanio Artino<EMAIL_ADDRESS>

merged to test project settings and scaffolding scripts in pipeline

merged to test project settings and scaffolding scripts in pipeline

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


how does getItemHeight  work?

Can we have an example of how getItemHeight works?
the doc feels unclear to me;
if my section header is 44px
and each of my list items are 40px
what should I put in getItemHeight ?

Yes, the doc is unclear. You can return the number like this:
getItemHeight={() => 40}

also set the sectionHeaderHeight:
sectionHeaderHeight={44}

same issue on #65


cleanup specification of storageclasses and persistence

Rename persistence variables in values.yaml to better reflect
what they actually mean. Update docs to match.
Fixes #376.

This works for me on docker-for-mac 2.0.0 and on an IKS (1.11) cluster.

This works for me on docker-for-mac 18.06, yet it doesn't work for IKS (1.10.8).

Sorry for my misleading messages. Finally I verified this PR worked well for IKS 10.8, just with more patience. The pvc binding may take 1-2 minutes. If it shows pending, it doesn't mean it fails. Just waiting for 2 minutes, it will work fine.
Thanks, David. After you remove document about the additional configuration for Docker for Mac, I'm OK to merge this PR.

i'm going to merge #380 first, then reconcile the changed variable names and enhance the docs as discussed with daisy.  Please hold off merging until I remove the wip label from this PR.

ready to be merged.


kie-server - jbpm - avoid npe and provide proper response on get task…

… if task does not exist
@cristianonicolai one more please :)

+1


Update .eslintrc

[x] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

What's the context behind these changes? Was there and open issue that you were trying to address? Has this been discussed previously?


feat: recommend node compat on builtin use

TODO:

[ ] Figure out if the example node app is in the right place

Closes #989

The windows CI issue is because of https://github.com/cloudflare/wrangler2/pull/977, investigating that separately


On demand windows environments

What
Derive a plan to help create on-demand windows environments to avoid CI time sinks with platform-specific fails

Unfortunately we're unlikely to have time to schedule this.


Shared system memory is not used in Tensorflow

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): 2.4
Are you willing to contribute it (Yes/No): ?

Describe the feature and the current behavior/state.
When using large models and/or large batch sizes which would not fit into the GPUs VRAM, one might expect that the system RAM would be used in addition to the VRAM to avoid OOM, similar to how TF works with nvidia's UVM. However, this is not the case and the program crashes with an OOM error. It seems that ROCm already has support for unified memory, but tensorflow-rocm just doesn't make use of it.
Will this change the current api? How?
No
Who will benefit with this feature?
Anyone, especially in a situation as explained in 'Other Info'.
Any Other info.
ROCm version: 4.0.1
GPU: Vega FE (gfx900)
When using batches with varying dimensions as e.g. in sequential models, a few outlier batches with particularly long sequences can lead to an unexpected OOM crash after hours of training. When using unified memory, such situations can be avoided without having to resort to small batch sizes and accepting underutilization of resources. The performance penalty of using unified memory would only affect these few outlier batches, and the performance benefit of larger batch sizes would outweigh this cost because the majority of batches still fit into the VRAM.

@deven-amd can you help look at this issue?

Is there progress? This should be helpful for training models on consumer-grade platforms.


Box scores for 2nd week of 2-week playoff matchup are not pulled

Using league.box_scores(14) (week 14 being the most recently completed week, which was week 2 of a 2 week playoff matchup) gives me the following traceback.  This issue was discovered in the previous version and persists after upgrading to 0.9.1.
  File "<ipython-input-2-bf17d8a7832e>", line 9, in <module>
    for box in league.box_scores(week):

  File "C:\Users\mtdic\Anaconda3\envs\nfl\lib\site-packages\espn_api\football\league.py", line 209, in box_scores
    box_data = [BoxScore(matchup, pro_schedule, positional_rankings, week, self.year) for matchup in schedule]

  File "C:\Users\mtdic\Anaconda3\envs\nfl\lib\site-packages\espn_api\football\league.py", line 209, in <listcomp>
    box_data = [BoxScore(matchup, pro_schedule, positional_rankings, week, self.year) for matchup in schedule]

  File "C:\Users\mtdic\Anaconda3\envs\nfl\lib\site-packages\espn_api\football\box_score.py", line 7, in __init__
    self.home_score = round(data['home']['rosterForCurrentScoringPeriod']['appliedStatTotal'], 2)

KeyError: 'rosterForCurrentScoringPeriod'

Strange, I am playing in a league that is in the 2nd week of 2-week playoff as well and not running into this problem.
Could you tell me what league.currentMatchupPeriod and league.scoringPeriodId returns from your league object?

Hmm, that is interesting.  Here's what I get:
In [1]: league.currentMatchupPeriod
Out[1]: 14

In [2]: league.scoringPeriodId
Out[2]: 15

Okay that looks correct and is what my league is returning. I think the only way to find the issue with the data is if you could run the package in debug mode like league = League(league_id=123, year=2020, debug=True) and then post the output of the data when calling league.box_scores().
When you initliazte the league in debug mode it will print out a lot of data but you can ignore that. Once you call box_scores it will print out the response data on a line with ESPN API Response:. That will help me figure out why in the box score a matchup does not have rosterForCurrentScoringPeriod.

I might not need the data. I have an idea on what the issue could be. When getting box score data I am using the current week for scoringPeriodId and filterMatchupPeriodIds and in your league case those are different, matchupPeriod = 14 but scoringPeriodId = 15.
This shouldn't be to hard to check and the settings has a matchupPeriods object that maps matchup periods to scoring periods. I am going to try to get this fixed in the next day or two so that it will work for this weekend.

So which is proper to be passed to box_scores as the week parameter?
Also, can confirm using latest version and a league from 2019 with a 2 week playoff that league.box_score breaks once week equals a week of the playoffs.
This may all be moot if your described method above works, but wanted to give an additional test data point.
league = League(league_id=164483, year=2019)

#success
box_scores = league.box_scores(week=13)

#failure (week 15 and 16 as well) KeyError: 'rosterForCurrentScoringPeriod'
box_scores = league.box_scores(week=14)

Here's the data that came back.  Hope it helps.  Thank you for the quick response and working on this.
league_box_score_output.txt

Thanks for the data! So the current week is still fine to pass but I will now need to use the league settings and check the corresponding matchup period to the scoring period. I should be able to work on this tonight/tomorrow.

Okay, so I believe it is fixed and will be in v0.9.2, one thing to note is the points for the home and away team will be total points from both weeks.


Haskell on Windows

I am experiencing issues using beautify with haskell. I cannot use it. It tells me that haskell-stylish (or other way around) is not installed. So I tried it and it gave me an error. I need Cygwin/MinGw to install a dependency called old-times which needs that. Is someone else having the exact same problem. Or is there maybe another way without installing Cygwin? :/

I am not a Windows or Haskell user, so I will not be much help. You should ask your questions regarding stylish-haskell installation over at https://github.com/jaspervdj/stylish-haskell#installation
Maybe @bobjflong  has some suggestions, too.


Fixes a few AATH failures

Codecov Report

Merging #1897 (f1f5086) into main (d407c48) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1897   +/-   ##
=======================================
  Coverage   93.79%   93.79%
=======================================
  Files         539      539
  Lines       34084    34086    +2
=======================================
+ Hits        31970    31972    +2
  Misses       2114     2114

Helpful to know which ones are fixed :-).  I'll check it out on main and this branch.


Cannot Load Images from /static/images

Home Assistant release (hass --version):
57.2
Python release (python3 --version):
3.5.3
Component/platform:
Presence Detection
Description of problem:
Can no longer load custom image files from /static/images it shows 404 not found this has always worked in older versions and it 56 but it seems broken now in 57+
Expected:
Images to be loaded and displayed.
Problem-relevant configuration.yaml entries and steps to reproduce:
Matthews-S8-Plus:
  hide_if_away: false
  mac: Redacted
  name: Matthew
  picture: /static/images/Matthew.jpg
  track: true
  vendor: SAMSUNG ELECTRO-MECHANICS(THAILAND)

Traceback (if applicable):

Additional info:

Ok thanks ill do that instead


Unpack: Checksum implimentation

Avoids the expensive conversion to .json if the file hasn't changed since the last unpack.
Sha1 is used to determine change, with an sqlite database holding the file data for comparison.
At this stage tested function for the most basic case (Creating a brand new unpack, and re-trying the same unpack). Requires testing for various combinations of files but wanted to get it out here in case there's broader changes to coding structure desired.
Also cleaned up a bunch of unused imports, only because my software kept bugging me about them.

I like that this handles deleted items. However, if you choose not to re-add a file deleted from the source directory, it is removed from the database. That means the file will be automatically re-added next time the module is unpacked. Could we instead flag the file as deleted and then remove the flag if the user decides to restore it?

How would we tell if the user decided to restore it?
I did think of the situation posed. However that would require the user to

Have a source (usually via unpacking first)
Delete one of the source files.
unpack the package again, triggering the question to which they must confirm the deletion
WITHOUT packing, they would have to unpack the same package a second time.

If the user packed between 3 and 4, there would be no problem because it would now be deleted from the package. It would be unusual for a user to double-unpack a file without packing in-between.
Alternatively, we add a forth column to the sqlite database "deleted" - but I'm unsure in that scenario how the user informs nasher that the file is to be restored. Worst case scenario:

Have a source (usually via unpacking first)
Delete one of the source files.
unpack the package again, triggering the question to which they must confirm the deletion
Packs the package (doing everything right)
... some time passes...
The user re-uses the name of the file deleted at item 2. The database has it marked as 'deleted' and so nasher ignores it/does not unpack it.
The user packs their package. The package is now without the new file from item 6, and the user may be totally unaware of why nasher has ignored that file.

In summary, unless you propose an elegant solution to "when to restore", I thought to stick to a safer method, which re-adds an excess file a user might have wanted it deleted. The alternative is risking nasher deleting a file that the user did not want deleted - a harder issue to resolve.

You're right, it is an edge case, though I can see reasons why such a thing might be desirable (e.g., you unpack, change a file, then want to revert to the module's version, so you unpack again). There's no reason that should stop this from being merged, though.
Thanks for the good work!


Script parsing is giving wrong address

I am getting very strange results when trying to parse scripts.  Here is an example:
`//Get Data from DB
SqlCommand cmd = new SqlCommand();
cmd.CommandText = "SELECT OutputScript FROM [BTC].[dbo].[TransactionOutput] WHERE TransactionOutputId = 5";
cmd.Connection = conn;
byte[] arr = (byte[])cmd.ExecuteScalar();
//Instantiate Script object
NBitcoin.Script script = new Script(arr);
//Get the address
BitcoinAddress ba2 = script.PaymentScript.GetDestinationAddress(Network.Main);`
The result I get in ba2?  "3P25dtEAem88Ek5FdA5otddoDb2FjrxgCY"
The raw script (as my SQL query above suggests) corresponds with the fifth transaction output in the blockchain (which is the output in block #5 / block hash: 000000009B7262315DBF071787AD3656097B892ABFFD1F95A1A022F896F533FC
The hex represenatation of the output script I'm passing in is:
0x410456579536D150FBCE94EE62B47DB2CA43AF0A730A0467BA55C79E2A7EC9CE4AD297E35CDBB8E42A4643A60EEF7C9ABEE2F5822F86B1DA242D9C2301C431FACFD8AC
I a truly confused as to why I would get 3P25dtEAem88Ek5FdA5otddoDb2FjrxgCY as the public address.  This is a bogus result (compare to: https://blockexplorer.com/block/000000009B7262315DBF071787AD3656097B892ABFFD1F95A1A022F896F533FC).

script.PaymentScript.GetDestinationAddress(Network.Main); is giving you P2SH version of the script.
script.GetDestinationAddress(Network.Main); will give you the expected result.

Hi Nicolas,
Thank you very much for your response.
I tried what you suggested as below:
//Get Data from DB
SqlCommand cmd = new SqlCommand();
cmd.CommandText = "SELECT OutputScript FROM [BTC].[dbo].[TransactionOutput] WHERE TransactionOutputId = 5";
cmd.Connection = conn;
byte[] arr = (byte[])cmd.ExecuteScalar();

//Instantiate Script object
NBitcoin.Script script = new Script(arr);

//Address
BitcoinAddress addr = script.GetDestinationAddress(Network.Main);

When I step through the debugger, I get:
script == {0456579536d150fbce94ee62b47db2ca43af0a730a0467ba55c79e2a7ec9ce4ad297e35cdbb8e42a4643a60eef7c9abee2f5822f86b1da242d9c2301c431facfd8 OP_CHECKSIG}
This seems right -- I get the same results if I run this through an old Python script I have.
However, when I step through addr = script.GetDestinationAddress(Network.Main);  I get 'null' as the result instead of an address.  Given that the return type of GetDestinationAddress() is a BitcoinAddress, I would expect the result of the method is the address.  Am I doing something else wrong?

0456579536d150fbce94ee62b47db2ca43af0a730a0467ba55c79e2a7ec9ce4ad297e35cdbb8e42a4643a60eef7c9abee2f5822f86b1da242d9c2301c431facfd8 OP_CHECKSIG is an old style scriptPubKey which can't be represented as an address.
To understand why it is a lie, read https://programmingblockchain.gitbooks.io/programmingblockchain/content/

Thank you much for your help.  I have started reading your book, as well.  It looks to be a great resource!  Many thanks.