Add osx

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Selectors are suggested to take a <two spaces>#<one space>[<expression>] form. See lines [37]

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).

@conda-forge-admin, please lint

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Hi @ramtingh,
I appreciate all the work you have put in so far, and wondered if there has been any progress on this? I'm using VMTK for multiple projects of my own and am very eager to see it being released on conda-forge.
Tell me if I can assist in any way. What seems to be the problem when deploying to osx?

Windows/linux builds are already available on conda-forge. I'm not quite sure what is going with osx, it compiles correctly and then runs into segmentation fault when testing. I don't have much experience with osx so I haven't gotten around to figuring out why that is.

@ramtingh I see. I am mostly working on osx at the moment, although my experience with compilation and segfaults on mac is lacking. Do the logs give any insight; any indication at what point during testing it fails?
Also, are you able to re-run the build on Azure? Seems like the previous error logs have been deprecated.

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@hkjeldsberg  no particular error that I can see, just "Segmentation fault: 11" when trying to import anything from vtkvmtk ( along with a lot of ELF warnings that as far as I know can be ignored). At some point I will need to pinpoint exactly what script is causing that

@ramtingh Yeah, I also think the ELF warnings can be ignored. Seems like the segfault occurs during the call to from vmtk.vtkvmtkCommonPython import * based on the logs. Would be interesting to investigate this further.
I can try to build this locally (osx) and see if I'm able to reproduce the error, or if it is something Azure specific.

Update: the build fails locally at the same point. Here is the last output before segfault:
import: 'vmtk'
+ python importtests.py
Fatal Python error: Segmentation fault

Current thread 0x0000000109284600 (most recent call first):
  File "<frozen importlib._bootstrap>", line 219 in _call_with_frames_removed
  File "<frozen importlib._bootstrap_external>", line 1043 in create_module
  File "<frozen importlib._bootstrap>", line 583 in module_from_spec
  File "<frozen importlib._bootstrap>", line 670 in _load_unlocked
  File "<frozen importlib._bootstrap>", line 967 in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 983 in _find_and_load
  File "importtests.py", line 12 in <module>

Note that line 12 in my importtests.py file is where from vmtk.vtkvmtkCommonPython import * is called.

@hkjeldsberg Thanks! If you get a chance can you also try it without the patch I added? (I assume remove line 13/14 in meta.yaml)
The code base itself works fine in Slicer's osx builds, so presumably the only possible error would be in the recipe/patch I've added.

@ramtingh I tried without the patch locally, but it resulted in termination errors during compilation. Seems like your patch is necessary for it to build VMTK successfully.
Here is the last output before during compilation:
[301/598] Python Wrapping - generating vtkvmtkPolyDataLaplaceBeltramiStencilPython.cxx
[302/598] Python Wrapping - generating vtkvmtkPolyDataFELaplaceBeltramiStencilPython.cxx
[303/598] Building CXX object vtkVmtk/Utilities/vtkvmtkITK/CMakeFiles/vtkvmtkITK.dir/vtkvmtkITKArchetypeImageSeriesScalarReader.cxx.o
[304/598] Building CXX object vtkVmtk/Utilities/vtkvmtkITK/CMakeFiles/vtkvmtkITK.dir/vtkvmtkITKImageWriter.cxx.o
ninja: build stopped: subcommand failed.
Traceback (most recent call last):
  File "/Users/henriakj/miniconda3/bin/conda-build", line 11, in <module>
    sys.exit(main())
  File "/Users/henriakj/miniconda3/lib/python3.7/site-packages/conda_build/cli/main_build.py", line 488, in main
    execute(sys.argv[1:])
  File "/Users/henriakj/miniconda3/lib/python3.7/site-packages/conda_build/cli/main_build.py", line 479, in execute
    verify=args.verify, variants=args.variants, cache_dir=args.cache_dir)
  File "/Users/henriakj/miniconda3/lib/python3.7/site-packages/conda_build/api.py", line 195, in build
    variants=variants
  File "/Users/henriakj/miniconda3/lib/python3.7/site-packages/conda_build/build.py", line 3093, in build_tree
    notest=notest,
  File "/Users/henriakj/miniconda3/lib/python3.7/site-packages/conda_build/build.py", line 2212, in build
    cwd=src_dir, stats=build_stats)
  File "/Users/henriakj/miniconda3/lib/python3.7/site-packages/conda_build/utils.py", line 410, in check_call_env
    return _func_defaulting_env_to_os_environ('call', *popenargs, **kwargs)
  File "/Users/henriakj/miniconda3/lib/python3.7/site-packages/conda_build/utils.py", line 390, in _func_defaulting_env_to_os_environ
    raise subprocess.CalledProcessError(proc.returncode, _args)
subprocess.CalledProcessError: Command '['/bin/bash', '-o', 'errexit', '/Users/henriakj/miniconda3/conda-bld/vmtk_1646634849188/work/conda_build.sh']' returned non-zero exit status 1.

@hkjeldsberg Fair enough, thanks for the help. Yeah I think that patch is necessary with conda compilers otherwise it overrides important settings. I will be able to look into this in more detail over the next couple of weeks, until then I'd appreciate any ideas of what might be going on.
Similar problems on windows ended up being slightly different hdf5 settings between the vtk and itk packages used, but as far as I've checked that shouldn't be a problem with the current versions

I also would like this to work on macOS. I've never tried to use the conda-forge stuff before, so apologies for any misunderstandings there.
I've built this branch using the build-locally.py script. When I test I get:
$ python3.7 -c 'import vmtk.vmtkcenterlines'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/Users/rnash2/work/hemelb-dev/vmtk-feedstock/miniforge3/conda-bld/debug_1649756879502/_h_env/lib/python3.7/site-packages/vmtk/vmtkcenterlines.py", line 20, in <module>
    from vmtk import vtkvmtk
  File "/Users/rnash2/work/hemelb-dev/vmtk-feedstock/miniforge3/conda-bld/debug_1649756879502/_h_env/lib/python3.7/site-packages/vmtk/vtkvmtk.py", line 9, in <module>
    from .vtkvmtkCommonPython import *
ImportError: dlopen(/Users/rnash2/work/hemelb-dev/vmtk-feedstock/miniforge3/conda-bld/debug_1649756879502/_h_env/lib/python3.7/site-packages/vmtk/vtkvmtkCommonPython.so, 2): Library not loaded: libvtkvmtkCommon.dylib
  Referenced from: /Users/rnash2/work/hemelb-dev/vmtk-feedstock/miniforge3/conda-bld/debug_1649756879502/_h_env/lib/python3.7/site-packages/vmtk/vtkvmtkCommonPython.so
  Reason: image not found

Inspecting the dylib shows:
otool -l /Users/rnash2/work/hemelb-dev/vmtk-feedstock/miniforge3/conda-bld/debug_1649756879502/_h_env/lib/python3.7/site-packages/vmtk/vtkvmtkCommonPython.so
<snip />
Load command 9
          cmd LC_LOAD_DYLIB
      cmdsize 48
         name libvtkvmtkCommon.dylib (offset 24)
   time stamp 2 Thu Jan  1 01:00:02 1970
      current version 0.0.0
compatibility version 0.0.0
Load command 10
          cmd LC_LOAD_DYLIB
      cmdsize 56
         name @rpath/libpython3.7m.dylib (offset 24)
   time stamp 2 Thu Jan  1 01:00:02 1970
      current version 3.7.0
compatibility version 3.7.0
<snip />
Load command 14
          cmd LC_RPATH
      cmdsize 112
         path /Users/rnash2/work/hemelb-dev/vmtk-feedstock/miniforge3/conda-bld/debug_1649756879502/_h_env/lib (offset 12)
<snip />

which reveals that the VMTK dylib has not been linked correctly: load command 9 should point to @rpath/libvtkvmtkCommon.dylib. I'll see if I can tweak the CMake flags to fix this.

Setting -DCMAKE_MACOSX_RPATH:BOOL=ON reproduces the segfault on import. Debugger shows it's in the module init function. I'm rebuilding with CMAKE_BUILD_TYPE=Debug while I get lunch....

I had another look at this just now. The problem is that the vtkMacroKitPythonWrap macro (in a .cmake of the same name) downloaded from https://github.com/Slicer/vtkAddon incorrectly links against the python library:
target_link_libraries(${MY_KIT_NAME}Python
      PRIVATE
        ${MY_KIT_NAME}
        ${VTK_PYTHON_LIBRARIES} #error
        VTK::WrappingPythonCore
        VTK::Python
        )

Any link that is needed should be picked up through the link against VTK::Python (assuming its INTERFACE_LINK_LIBRARIES property is correctly set - VTK is usually quite good about that)
Since this is an external dependency, I will make a PR there. When accepted (I realise that Slicer may have requirements making this harder...) we can update VMTK's CMake/CMakeLists.txt to point to that and then this recipe should work as-is.

Thanks! this looks very promising. That is the exact opposite of what I was expecting; I thought as the code works fine for slicer builds it couldn't be anything upstream related, but I don't have nearly enough osx/cmake experience to figure out what is going on.
I guess there's three levels this could be fixed at, either the slicer repository, patched in vmtk's git repository or as a patch in this repository (as @rupertnash has already added). @lassoan I think that depends on your preference.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@rupertnash In the interest of not delaying this too much, I'm thinking of merging it as is so osx packages are available on conda-forge and once vtkAddon is updated it can be fixed for future builds. I can't think of any issues that can cause (besides some technical debt I'd have to fix later), do you think I should wait?

Aye, you're probably right. Suggest creating an issue here to track the TODO. Thanks!

@rupertnash  Fair enough, I appreciate all the work you've done to make this work! I'll merge this for now


Several pages use an old jQuery version that has known security issues

Outdated library in api.dartlang.org/stable/1.20.1/dart-html/dart-html-library.html
https://code.jquery.com/jquery-2.1.4.min.js
Library: jquery
Version: 2.1.4.min
Outdated library in events.dartlang.org/2016/summit/
Library: jquery
Version: 2.2.4
We should update to jQuery 3 to resolve this.

Later versions no longer use this jquery version.


[Feature][API]  The result of the assertion cannot be changed. The only difference is the judgment condition.

Purpose of the pull request
[The result of the assertion cannot be changed. The only difference is the judgment condition.]
Brief change log
#10231
Verify this pull request
Manually verified the change by testing locally and unit test;

Codecov Report

Merging #10261 (7ce07f7) into dev (58afe5d) will increase coverage by 0.20%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##                dev   #10261      +/-   ##
============================================
+ Coverage     40.96%   41.16%   +0.20%
- Complexity     4734     4759      +25
============================================
  Files           854      855       +1
  Lines         34565    34618      +53
  Branches       3819     3826       +7
============================================
+ Hits          14158    14252      +94
+ Misses        19050    18988      -62
- Partials       1357     1378      +21

Impacted Files
Coverage Δ

...lphinscheduler/common/enums/AuthorizationType.java
0.00% <0.00%> (ø)

...permission/ResourcePermissionCheckServiceImpl.java
24.00% <24.00%> (ø)

...hinscheduler/api/service/impl/BaseServiceImpl.java
90.69% <50.00%> (-4.18%)
:arrow_down:

...scheduler/api/service/impl/ProjectServiceImpl.java
62.96% <66.66%> (-14.17%)
:arrow_down:

...r/plugin/task/sqoop/parameter/SqoopParameters.java
53.33% <0.00%> (-1.34%)
:arrow_down:

...inscheduler/server/worker/config/WorkerConfig.java
76.47% <0.00%> (-0.95%)
:arrow_down:

.../server/worker/processor/TaskExecuteProcessor.java
0.00% <0.00%> (ø)

...e/dolphinscheduler/remote/NettyRemotingClient.java
53.52% <0.00%> (+0.70%)
:arrow_up:

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
55.35% <0.00%> (+7.14%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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

+1


Start button padding incorrect on smaller viewports

Issue found while visiting the Scottish Government's Device Lab

As seen on https://www.gov.uk/employer-recruitment-incentive-scotland

The business support pages were moved last year to government-frontend and are now using the button component. This is fixed now!


Wrong cursor position in the live editor

Describe the bug
Currently, the cursor position is not aligning with the line where it should be.
Steps to reproduce

Go to any page where a live editor is available. (Suppose https://nextui.org/docs/components/checkbox)
Click on the live editor window

Expected behaviour
The cursor position should align with the clicked line
Screenshots

Environment / Context

All

Comment
I would like to work on that issue

Thank you for reporting it @piall, go ahead all yours, thanks for your help! 🙋🏻♂

My pleasure :smiley: . Is there any pre-requisite of environment setup?

Just be sure you have yarn installed because the project has built over yarn workspaces and be sure you have a node version >=12.13.0
Steps to start the docs project, Execute in the root of the project:

yarn
yarn dev:docs

You will find the source code of the docs in -> packages/docs
Let me know if you need any help

Thank you :raised_hands:

Hi! @piall any updates about this? 🙌

I am busy with other stuff currently. I couldn't start work. :sweat: . I will try to give time on it today.


Migrate from glide to dep

We might consider using dep instead of glide for dependencies. Perhaps that could be tackled, when #762 and #695 are implemented.

This was added with #871


Reshape tool in paint editor can't add points to some sides of shapes

Expected Behavior
Tapping any side of a shape (Circle, Square, Triangle, etc.) should add a point.
Actual Behavior
Some segments of shapes will not receive a new point if you tap it.

On a newly drawn square  it is the top and right segments that can't receive points.
On a newly drawn circle it is the upper left and upper right segments.
On a triangle it is the upper left and upper right, but if you tap the bottom, the first time, it adds a point on the upper right side.  A second tap on the bottom segment adds a point on the bottom segment.  The behavior of adding points to a triangle becomes less predictable after the second new point.

Steps to Reproduce
Open the paint editor
Draw a new shape
Try to add points to the various sides of the shape (See above)
Device and Operating System
iPad mini 5 running iOS 10.14.  Not reproducible on older ipads running older versions of iOS

We could only see this on one of our test devices, so it's hard to know the impact of this issue.  If anyone can reproduce this on other devices it would be useful for us to know the model and os versions.

Closing out since this is fixed in #504


[Bug] Indexing stops for the specific subgraphs

Bug report

Indexing stops for the specific subgraphs

We are using graph node for our L2 network using Optimism. But sometimes, the below error occurs and indexing is stopped for the specific subgraphs.
I have never seen the error. How can I do to resolve the error?
You can see the indexing status using below query in here (https://graph-node.titan-sepolia.tokamak.network/graphql/playground)
{
  indexingStatuses(subgraphs: ["QmNa9YZ58oSRqjtFRNxufXpNTvJ5dRZAMn4nisRAPtL17y"]) {
    subgraph
    synced
    health
    paused
    node
    entityCount
    fatalError {
      handler
      message
      deterministic
      block {
        hash
        number
      }
    }
    nonFatalErrors {
      handler
      message
      deterministic
      block {
        hash
        number
      }
    }
    chains {
      chainHeadBlock {
        number
      }
      earliestBlock {
        number
      }
      latestBlock {
        number
      }
    }
  }
}

Thank you very much.
Relevant log output
Block stream produced a non-fatal error, error: block stream error missing field `block`, sgd: 37, subgraph_id: QmNa9YZ58oSRqjtFRNxufXpNTvJ5dRZAMn4nisRAPtL17y, component: SubgraphInstanceManager

IPFS hash
QmNa9YZ58oSRqjtFRNxufXpNTvJ5dRZAMn4nisRAPtL17y
Subgraph name or link to explorer
https://graph-node.titan-sepolia.tokamak.network/subgraphs/id/QmNYauo5HEKRrQuetjAzPbLE57tcsSu5swzwcPHM6d5PyC
Some information to help us out

[ ] Tick this box if this bug is caused by a regression found in the latest release.
[ ] Tick this box if this bug is specific to the hosted service.
[X] I have searched the issue tracker to make sure this issue is not a duplicate.

OS information
None

It is resolved after flushing data and restarting graph-node.


Update package template, remove astropy helpers.

Hello @samaloney! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file radiospectra/spectrogram.py:

Line 830:13: E128 continuation line under-indented for visual indent

Thanks for doing this @samaloney but your time should never have been wasted doing this.

Thanks @samaloney, I am sorry it took this long for someone to merge this.


Not working at all

Hey guys, I just installed the plugin and cannot get it to work. Typing the : and a few letters on a layer name isn't triggering the dropdown. Done some basic troubleshooting to rule out issues with Sketch, trashed the prefs file, removed other plugins. No luck.
Sketch Version 58 (84663)

@firsttimecommenterof99
Which macOS version do you use? If you use macOS 10.15 Catalina then it doesn't work without notarization process on plugin itself.

Mojave, most recent update, 10.14.6

@firsttimecommenterof99 Okay, now I have no idea why this plugin does not work on your machine.

@griffin-stewie Discovered the issue by accident, the dropdown doesn't show on a secondary display


Fix cos cors

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Feature Request]: Beacon monitoring permanent notification choice.

Summary
When using BeaconMonitoring a permanent notification shows up. You can choose the title and the message of the notification but your unable to remove the notification no matter what. It also results in a 1 number character always being on the app icon so users keep thinking there's an important notification even though there isn't and where I work a customer is dissatisfied by that at least.
So as an update I would like if you can choose whether the notification needs to be permanent.
API Changes
In the class: ShinyAndroidForegroundService
protected virtual void SendNotification()
{
this.builder ??= new NotificationCompat.Builder(this.Context.AppContext)
.SetChannelId(NotificationChannelId)
.SetSmallIcon(this.AndroidNotifications.GetSmallIconResource(null))
.SetOngoing(true); // HERE YOU WOULD SHOULD BE ABLE TO CHOOSE THE BOOL VARIABLE INSTEAD OF
IT BEING HARCODED
        this.builder
            .SetProgress(
                this.Service.Total,
                this.Service.Progress,
                this.Service.IsIndeterministic
            )
            .SetContentTitle(this.Service.Title ?? "Shiny Service")
            .SetTicker("..")
            .SetContentText(this.Service.Message ?? "Shiny service is continuing to process data in the background");

        if (this.notificationId == null)
        {
            this.notificationId = ++idCount;
            this.StartForeground(this.notificationId.Value, this.builder.Build());
        }
        else
        {
            this.AndroidNotifications.NativeManager.Notify(this.notificationId.Value, this.builder.Build());
        }
    }

Intended Use Case
I'm trying to use your NuGet package in a project but the customer does not want a permanent notification.
Code of Conduct

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

The notification has to be persistent in order for the foreground service to remain live.  It is an Android requirement


Rename the font assets to match font repo

Basic PR to rename the font assets to use the final naming convention listed in the fonts repo.

Going to close this PR. V2 fonts will be going into template very soon which use the new naming convention rendering this PR unrequited.


StrawberryShake cannot consume JSON scalar

Is there an existing issue for this?

[X] I have searched the existing issues

Product
Strawberry Shake
Describe the bug
When a StrawberryShake query includes a JSON type it throws a runtime exception:
System.InvalidOperationException: The requested operation requires an element of type 'String', but the target element has type 'Object'.
   at System.Text.Json.ThrowHelper.ThrowJsonElementWrongTypeException(JsonTokenType expectedType, JsonTokenType actualType)
   at System.Text.Json.JsonDocument.GetString(Int32 index, JsonTokenType expectedType)

Steps to reproduce

Create a StrawberryShake client with a query that selects a JSON scalar type.

Relevant log output
No response
Additional Context?
No response
Version
13.5.1

I've followed the documentation in https://chillicream.com/docs/strawberryshake/v13/scalars/#any-or-json
and added an extension in schema.extensions.graphql and a custom ScalarSerializer and swapped around every combination of
serializationType and runtimeType but I can't make the error go away.
This feels like it should be something that should work out of the box so not sure if it is a bug or something that I am doing wrong?
extend scalar Any
  @serializationType(name: "global::System.Text.Json.JsonElement")
  @runtimeType(name: "global::System.String")

public class CustomJsonSerializer : ScalarSerializer<JsonElement, string>
{
    public CustomJsonSerializer(string typeName = BuiltInScalarNames.Any) : base(typeName)
    {
    }

    public override string Parse(JsonElement serializedValue) =>
        serializedValue.GetRawText();

    protected override JsonElement Format(string runtimeValue) =>
        JsonDocument.Parse(runtimeValue).RootElement;
}

Be careful with that code as JsonDocument.Parse(runtimeValue).RootElement; introduces a memory leak... JsonDocument is disposable as it rents memory form the ArrayPool and will only return it on dispose.

Alternatively you can parse with the JsonElement and specify that the memory shall not be pooled.

Have you registered the serializer with the client

It would appear that the Utf8JsonWriterHelper does not account for JsonElement and instead falls into the default case which calls .ToString() on the element itself.
https://github.com/ChilliCream/graphql-platform/blob/99d3adfc9b734cf0e2497f21919cf15a87ef3ff7/src/HotChocolate/AspNetCore/src/Transport.Abstractions/Serialization/Utf8JsonWriterHelper.cs#L175-L177

I would assume something similar to the existing implementation in JsonResultFormatter would solve this issue.  I tried to see if I could make a failure case but it's pretty deep in the code, would require changing the starwars schema to add a json or other scalar.
https://github.com/ChilliCream/graphql-platform/blob/99d3adfc9b734cf0e2497f21919cf15a87ef3ff7/src/HotChocolate/Core/src/Execution/Serialization/JsonResultFormatter.cs#L660-L667

@michaelstaib @david-driscoll  Eventually my workaround for this was to ditch the CustomJsonSerializer completely and in schema.extensions.graphql add:
extend scalar JSON
  @serializationType(name: "global::System.Text.Json.JsonElement")
  @runtimeType(name: "global::System.Text.Json.JsonDocument")

Then instead of a custom seriliser, register the built-in JsonSerilizer
services..AddSerializer(new JsonSerializer("JSON"))


CI Rust version mismatch?

These lint errors do not occur locally.
https://github.com/GiganticMinecraft/seichi-game-data-publisher/runs/7933964299?check_suite_focus=true

Should be fixed by 24591b7.


Not able to start datetimepicker

ISSUE
i am new to angularJS 2 and i am trying to use date time picker, after downloading it, when i try to run using npm start, i am getting error.
screen shot of the error

webpack-dev-server is not a recognized command
http://stackoverflow.com/questions/35810172/webpack-is-not-recognized-as-a-internal-or-external-command-operable-program-or


"Fling" scroll on rotated map eventually scrolls in unrotated direction.

Issue Type
[x] Bug
Description and/or steps/code to reproduce the problem
I first observed it in my app, but then realized, it is also there in the osmdroid demo app.
Rotate the map 90°, then fast scroll the map by swiping up, the map first follows the fingertip and scrolls up, but then scrolling goes ballistic and the map animation actually moves the map left.
If you rotate the map 180°, fling movements really "overbounce back", i.e. up becomes "first up, but eventually down".
Summary: After the follow finger, the actual animation moves the map in a direction, as if the map wasn't rotated below the finger.
Environment
Tested on OnePlus 6 (Android 9), but I hear it from my users on other devices.
If it's a bug, version(s) of android this affects:
9 in any case
Version of osmdroid the issue relates to:
6.1.0.

Hmm, did someone look into this? This is a bit annoying.

I've just tested the issue this morning on my Android Go, but couldn't reproduce it.

I've the same bug. Fling after rotation gesture or animateTo with orientation not working properly. Scroll without fling is ok.

@monsieurtanuki I see it on all devices I held so far. You really need to fling ballistic, as soon as the finger leaves the screen, weird directions happen.

I made a video of the effect. https://drive.google.com/file/d/1H2eKwU2EDF35Tah3zD9F_OtqQiTfubNZ/view?usp=drivesdk

@jlimek I can now fix it on one of my devices, just to make it bad on another. There seems to be a dependency on Android version. Can you second this?

Fixed by #1415


Wrap .card-actions on .card

Hi, I work with daisy (great project btw) and I noticed better badge wrap with this:
.card-actions {
   flex-wrap: wrap;
}

and for the card with badge example:

Colors

UI Design

Creativity

Thank you for your time!

Of course. I'll fix this tomorrow 👍

Please update to latest version 0.14.0
I also, added margin for card-actions items so space-x-2 is no longer needed.

Thanks !


Argument 1 passed to ZendDeveloperTools\Exception\SerializableException::__construct() must be an instance of Exception

The phenomenon of #10 (supposedly fixed since 2012) happens again. I'm running the current master branch, it started to appear some weeks ago.
Any time the application generates an exception, the toolbar blocks it and only shows
Fatal error: Uncaught TypeError: Argument 1 passed to ZendDeveloperTools\Exception\SerializableException::__construct() must be an instance of Exception, instance of Error given, called in /…/vendor/zendframework/zend-developer-tools/src/Collector/ExceptionCollector.php on line 45 and defined in /…/vendor/zendframework/zend-developer-tools/src/Exception/SerializableException.php:26 Stack trace:
#0 /…/vendor/zendframework/zend-developer-tools/src/Collector/ExceptionCollector.php(45): ZendDeveloperTools\Exception\SerializableException->__construct(Object(Error))
#1 /…/vendor/zendframework/zend-developer-tools/src/Profiler.php(210): ZendDeveloperTools\Collector\ExceptionCollector->collect(Object(Zend\Mvc\MvcEvent))
#2 /…/vendor/zendframework/zend-developer-tools/src/Listener/ProfilerListener.php(93): ZendDeveloperTools\Profiler->collect(Object(Zend\Mvc\MvcEvent))
#3 [ in /…/vendor/zendframework/zend-developer-tools/src/Exception/SerializableException.php on line 26

I've got the same problem, i suggest to modify the argument type in the constructor from
public function __construct(\Exception $exception)

to
public function __construct(\Throwable $exception)

there is no Throwable in php 5.6, maybe just remove the type hint ?

Could also drop PHP 5.x :trollface:
But yeah, should be just dropping the type-hint if we want to support 5.x

@Ocramius  I've create PR #217 for it.

Great, thanks! What are your release cycles? How long does it usually take to wrap this into a release?

@Ocramius may know about that, but if you need the patch now, you can do the following update.

update dependency to:

"zendframework/zend-developer-tools": "dev-patch-3"

update repository url in composer.json:

"repositories" : [ {
    "type" : "vcs",
 "url" : "git@github.com:samsonasik/ZendDeveloperTools.git"
}]

composer update

Thanks, the fix works.


EPEL release match

This pull request puts yum-epel in line with epel-release RPMs for EPEL 5/6/7, and pulls the gpgkey from https on EPEL 6/7 (as the mirrorlists also use https).
The two issues this solves is:

GPG public key being able to be replaced with another via a MITM - this is solved using https with SSL certificate verification for the GPG key
Unverified repodata allowing stale mirrors, giving bad mirrors/MITM the ability to give old packages with known security issues in, then attack the servers using exploits for those issues.

I understand this work was previously reverted due to complaints however:

The same would be the case for them installing epel-release
They can always turn off sslverify for the repos
They can always also specify the http url themselves
Semantic versioning should take care of people finding BC issues. If not (through lack of ~> x.y.z), it's their problem, not the people who want assurance on security.
The damage of having a MITM push a different gpg key and exploit the servers is far more serious than some complaints
The damage of having a bad proxy or MITM force stale packages with known exploits is ...

@someara sorry for invoking your name again, but do you know when this can be looked at?


Maximizer gone wild!

I think the recent changes to the maximizer code might have gone a bit overboard.  The script is now performing multiple, lengthy, maximizer passes before every adventure.  Here's the session log.
Arbos_20200619.txt

Much of the length is due to having to exclude or include individual items.
As for the repeated maximization...
So looking at a sample case, I am seeing 4.
first and second are simulations done by providePlusCombat(25) which is trying to give you +combat chance.
third is prefaced with [info] which is informing you that it is about to perform a maximize.
fourth is mafia informing you that a maximize occured.
All of this code is actually shared with autoscend. I will look into ways of trimming those messages.

It isn't the messages.  They are actually helpful as they give insight into what is happening and why.  Each one of the of the maximizer runs, and simulations, takes a lot of time to run through the options and it's happening for every adventure.
Two things not captured by the session logs are the amount of time taken for each maximizer pass and the number of permutations considered with each one.
The change in the autoscend code increased the, real world, time needed to complete even a single turn dramatically.

It isn't the messages. They are actually helpful as they give insight into what is happening and why. Each one of the of the maximizer runs, and simulations, takes a lot of time to run through the options and it's happening for every adventure.

Oh, I was assuming everyone puts a cap on max combinations allowed. It did not occur to me that some people allow maximizer to run unchecked. I will add temporarily switching the cap setting to a reasonable figure

ok. added limit on maximizer so it won't take so long.
Also, in retrospect I should not use providePlusCombat as it checks too many unnecessary in run things. Instead I should just make sure a couple of relevant buffs are on.

Fixed.  Thank you.  I did have an upper bound set by maximizerCombinationLimit but it was rather high. :-)  It isn't normally a problem but I hadn't run into any scripts that maximized every turn before now.


Revert "fix!: replace telescope mappings instead of merging (#106)"

This reverts commit 406d6199b231cfa4a5c51a122ff017fd93be24bb.

Thanks!


JBDS-4573 Fill in default folder when empty target input loses focus

so it doesn't just stay empty complaining about stuff

Codecov Report

Merging #894 into master will decrease coverage by 0.11%.
The diff coverage is 62.5%.

@@            Coverage Diff             @@
##           master     #894      +/-   ##
==========================================
- Coverage   97.71%   97.59%   -0.12%
==========================================
  Files          36       36
  Lines        3241     3414     +173
  Branches      274      494     +220
==========================================
+ Hits         3167     3332     +165
+ Misses         74       70       -4
- Partials        0       12      +12

Impacted Files
Coverage Δ

browser/services/data.js
98.48% <100%> (+0.07%)
:arrow_up:

browser/pages/location/controller.js
93.54% <40%> (-6.46%)
:arrow_down:

browser/main.js
98.41% <0%> (-1.59%)
:arrow_down:

browser/model/helpers/downloader.js
96.24% <0%> (-1.51%)
:arrow_down:

browser/pages/confirm/controller.js
94.92% <0%> (-1.47%)
:arrow_down:

browser/model/devstudio.js
96.8% <0%> (-0.97%)
:arrow_down:

browser/model/cdk.js
99.21% <0%> (-0.79%)
:arrow_down:

browser/model/jdk-install.js
96.61% <0%> (-0.42%)
:arrow_down:

browser/services/componentLoader.js
100% <0%> (ø)
:arrow_up:

... and 24 more

Continue to review full report at Codecov.

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


[FLINK-30133][hadoop] Changes log level and make log message more descriptive.

What is the purpose of the change
Lowers the log level to warning since Flink continues to work properly.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@flinkbot run azure

Error is unrelated to this change. I'm gonna go ahead and merge.


Problem with guanghechen/rollup-plugin-copy and transform

Hey there, first of all, thank you very much for maintaining the guanghechen/rollup-plugin-copy plugin, it's a tremendous help!
However, I started running into a problem occasionally: I use a transform function to parse a JSON file that is being copied and make some adjustments to when it is being copied. But for some reason, when the file is pretty large, and I make a change to it and save that, I don't seem to get the whole file, it's truncated at some point. And of course that means that parsing the JSON fails. Any idea why that might be happening? The function I use as transform is this:
export function convertJSONToPack(contents) {
    const jsonString = contents.toString();
    return (
        JSON.parse(jsonString)
            .map((entry) => cleanPackEntry(entry))
            .map((entry) => JSON.stringify(entry))
            .join("\n") + "\n"
    );
}

cleanPackEntry just performs some cleaning on the data, it's not relevant. It already fails on the JSON.parse. I also logged the contents to see what's going and and that shows that the contents are truncated.
This is the specification in my rollup.config.js:
                {
                    src: [`packs/*.json`],
                    dest: `${distDirectory}/packs`,
                    rename: (name) => `${name}.db`,
                    transform: convertJSONToPack,
                },

It doesn't happen when first building, only happens in watch mode when the file is changed.

Hi @ghost91-  Thanks for your feedback, I'll deal with this as soon as possible.
To make it easier to locate the problem, can you tell me how big your JSON file is?

It is this file: https://git.f3l.de/dungeonslayers/ds4/-/blob/master/packs/creatures.json, so 1.12 MB, apparently.

Hi @ghost91- , I think it works correctly now! So I made a new release<EMAIL_ADDRESS>can you upgrade it to confirm it works?
The code is a bit old, I refactored it and use chokidar to implement watch mode, now it should have better performance (only trigger copy action for a specific file when it changes).
I did a simple test with the big json file you provided, and it looks good in my machine.
Waiting for you confirmation.

Hm, with this change, watch mode doesn't seem to function properly anymore. The first time, a file is saved after having started watch mode, it gets copied over (though the process is quite long, maybe other stuff than actually copying happens, too, like building all of the ES module code..?). And after that, saving additional changes doesn't have any effect: Doesn't trigger anything being copied.

Do you mean that when file A is modified later, A's recopy operation will not be triggered?

Exactly. Or rather, it doesn’t even matter which file of the ones tracked by the copy plug-in it is. It does work when you modify a file for the first time, but after that, modifying additional files does not trigger recopys.

If it matters, I am doing this with yarn 3, using PNP. That may or may not have an effect on things…

Exactly. Or rather, it doesn’t even matter which file of the ones tracked by the copy plug-in it is. It does work when you modify a file for the first time, but after that, modifying additional files does not trigger recopys.

The first time will trigger the full copy operation, and then only the modified file will be re-copied. In my opinion, this is expected behavior, and I'm curious what scenarios would trigger a recopy of an unmodified file? Is there any point in re-copying an unmodified file?

If it matters, I am doing this with yarn 3, using PNP, and the rollup config is an es module (not commonjs). That may or may not have an effect on things…

Yes I noticed, but I don't think it matters, @guanghechen/rollup-plugin-copy provides modules in both esm and cjs formats.

The first time will trigger the full copy operation, and then only the modified file will be re-copied. In my opinion, this is expected behavior, and I'm curious what scenarios would trigger a recopy of an unmodified file? Is there any point in re-copying an unmodified file?

The point is that nothing is copied on subsequent changes, not even the modified file. If you want, I can create a video, showing the problem in more detail

Here is a video:
https://user-images.githubusercontent.com/236845/154328154-299b4e9f-2d42-4da0-8b29-b1596879e70f.mp4

Hi @ghost91- , thank you for the video, it's very clear, I think I get what you mean. This is actually because that message is only output when a full copy is made, and I will fix that. But in fact it triggers a copy now, can you add a verbose option to rollup.config.js first to verify that it is copied:
copy({
    verbose: true,
    targets: [
        { src: staticFiles, dest: distDirectory },
        {
            src: [`packs/*.json`],
            dest: `${distDirectory}/packs`,
            rename: (name) => `${name}.db`,
            transform: convertJSONToPack,
        },
        {
            src: [`packs/*.json.license`],
            dest: `${distDirectory}/packs`,
            rename: (name, extension) => `${name.replace(".json", ".db")}.${extension}`,
        },
    ],
})

@ghost91-  I will fix the message style , but should
I will fix the problem of message printing (copying details message should under the copied: line), but you need to always set verbose to true, because the current implementation does not always trigger the rebuild of rollup, but completes the copy in a separate task (chokidar is responsible for monitoring and processing).
In the current implementation, the modified files are accurately triggered to be re-copied (the full copy was always triggered in the past), which is also the key factor that I claim to have greatly improved the performance in this modification.

Do you think set the default value of verbose is better?

It's not about the message. While it is consistent with other stuff to have it (when you edit a source file, you also get that message, and it was actually the previous behavior), that is not the point. I can only repeat: The file is not being copied. That's why I opened the resulting file on the right. you can clearly see how the content of that file changes when rollup is first started. but when the source file is then changed, no more copies happen.

So, with verbose output, I see that something is happening. But the file simply isn't written. I added some logging statements to my transform function to check if it is even being invoked—it is not.

OK, I just played around a bit more with it. It seems to be related to the transform. Without a transform, everything seems to be working fine. But as soon as you something as simple as
transform: (contents) => contents.toString()

it just doesn't work anymore.
I tried that in https://github.com/guanghechen/node-scaffolds/tree/main/playground/rollup-plugin-copy and was able to confirm the problem there, too.

ACK, I will fix it.

@ghost91-  I think I fixed it, you can try the https://github.com/guanghechen/node-scaffolds/tree/main/playground/rollup-plugin-copy it now.
Would you mind if I put some of the data increatures.json under the playground?

Wait a minute, I can now reproduce your original problem, it should be caused by frequent modification of the file, I will try to solve it.

That JSON file is licensed under CC BY-NC-SA-4.0, so you can put it in your repo under the terms of that license. I can’t provide it under a more permissive license because it is based on CC BY-NC-SA-3.0 work (you would also need to attribute the original author of that, Crhistian Kennig, see http://www.dungeonslayers.net/).

That JSON file is licensed under CC BY-NC-SA-4.0, so you can put it in your repo under the terms of that license. I can’t provide it under a more permissive license because it is based on CC BY-NC-SA-3.0 work (you would also need to attribute the original author of that, Crhistian Kennig, see http://www.dungeonslayers.net/).

Ok, I got it, and I decided to give up on adding it.

Wait a minute, I can now reproduce your original problem, it should be caused by frequent modification of the file, I will try to solve it.

This problem discussed here https://stackoverflow.com/a/65044648, and I think is works now.
It is worth mentioning that you could feel that there is a delay in re-copy, but it will eventually be executed.
Can you try it in the latest main branch?

It seems to working correctly in version 1.9.0. Thanks for your help with this!


Update to 0.6.9

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 10 committers have signed the CLA.:white_check_mark: vitlibar:x: graebm:x: waahm7:x: DmitriyMusatkin:x: JonathanHenson:x: knightjoel:x: bretambrose:x: yasminetalby:x: TingDaoK:x: jmklixYou have signed the CLA already but the status is still pending? Let us recheck it.


Cross selling cms block frontend rendering

1. Why is this change necessary?
In the shopping world experiences you have the possibility to change elements within blocks. However the outer block wrapper is not being changed if you do so.
If you drag & drop a Cross-Selling CMS-Block into the shopping world and then change the element within the block, nothing is being rednered in the storefront due to the if-statement.
In the Cross-Selling CMS-Block there also the check see Cross Selling CMS-Element
{% if element.data.crossSellings.elements is defined %}
    {% for item in element.data.crossSellings.elements|filter(item => item.total > 0 and item.crossSelling.active == true) %}

2. What does this change do, exactly?
Removing the if-statement from the cross-selling block frontend-view to also get other elements rendered in the storefront if you change the element within that block
3. Describe each step to reproduce the issue or behaviour.
Go to the shopping experience drop a cross-selling block and then change the element within the block. Now visit the storefront and check if you see the changed element in the storefront
4. Please link to the relevant issues (if any).
Cross Selling CMS-Element
5. Checklist

[ ] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-22699
Please use this issue to track the state of your pull request.

Your PR has been merged, sorry it took so long. Thank you for your contribution 🎉 💙


bug: side menu background image syntax error after

Looks to be caused by a spurious  <=""
Type: bug
Platform: desktop browser
After exporting from creator.ionic.io firstly into a fresh project, I was trying to incorporate Meteor.
http://codepen.io/tomachi/pen/obwjYx
13inch:www tom$ meteor
[[[[[ ~/Dropbox/Sites/FoodPrices/www ]]]]]
=> Started proxy.
=> Started MongoDB.
[BABEL] Note: The code generator has deoptimised the styling of "/lib/ionic/js/angular/angular-animate.js" as it exceeds the max of "100KB".
[BABEL] Note: The code generator has deoptimised the styling of "/lib/ionic/js/angular/angular.js" as it exceeds the max of "100KB".
[BABEL] Note: The code generator has deoptimised the styling of "/lib/ionic/js/angular/angular.min.js" as it exceeds the max of "100KB".
[BABEL] Note: The code generator has deoptimised the styling of "/lib/ionic/js/angular-ui/angular-ui-router.js" as it exceeds the max of "100KB".
[BABEL] Note: The code generator has deoptimised the styling of "/lib/ionic/js/ionic-angular.js" as it exceeds the max of "100KB".
[BABEL] Note: The code generator has deoptimised the styling of "/lib/ionic/js/ionic-angular.min.js" as it exceeds the max of "100KB".
[BABEL] Note: The code generator has deoptimised the styling of "/lib/ionic/js/ionic.bundle.js" as it exceeds the max of "100KB".
[BABEL] Note: The code generator has deoptimised the styling of "/lib/ionic/js/ionic.bundle.min.js" as it exceeds the max of "100KB".
[BABEL] Note: The code generator has deoptimised the styling of "/lib/ionic/js/ionic.js" as it exceeds the max of "100KB".
=> Errors prevented startup:
While processing files with angular-templates (for target web.browser):
templates/settingsMenuUNUSED.html:15: Expected attribute name in tag
...) no-repeat center;" <="" ion-content="">...
^
index.html:1: Can't set DOCTYPE here.  (Meteor sets  for you)
=> Your application has errors. Waiting for file change.

Hello! Thanks for opening an issue with us! As this seems like more of a support question i will urge that you ask this question on our forum (forum.ionicframework.com). Thanks for using Ionic!


History for Custom check not showing instance name

On the Custom tab if you click history for a custom check the Instance column will be blank.  It expects an InstanceDisplayName column to be returned from the query.

Fix included in latest release.


added mTLS failure and workaround description (along with failure image)

this should complete the mTLS part

I don't see guestbook v2 mentioned here.  I'd assume the mTLS is between guestbook-v2 and analyzer.

Merging as this is a lot of changes.


feat(); Gnosis Decentralized Network; Add decentralized network deployments to the deployment.json

Why do we need the decentralized network part if it is not deployed?

I imagine the BE will try and query that endpoint UNKOWN too

If we want this to be a common pattern we prolly want to make UNKNOWN something the BE recognizes as no deployment, skip

If we want this to be a common pattern we prolly want to make UNKNOWN something the BE recognizes as no deployment, skip

My bad. I meant to have this PR on draft. Just had not deployed and pushed the ids yet, but they are updated now.


Fix a number of Codacy warnings

This pull request fixes 3 alerts when merging 1ce0de080b94d57c5defc30ef601eed54d2f4721 into 7f06f242b5721ff9b87f4a1473afd232c8f2f558 - view on LGTM.com
fixed alerts:

3 for Unused import

Comment posted by LGTM.com

This pull request fixes 3 alerts when merging e96cb7e524016eea7bf2b5553252d488a52df531 into 020193e3ee1a4a63ce89aaf236c9b37662eae92a - view on LGTM.com
fixed alerts:

3 for Unused import

Comment posted by LGTM.com


Update scheduler SPI to 1.0.2.RELEASE

This issue was create in response to the use of 1.1.0.BUILD-SNAPSHOT of scheduler in the SCDF-Server-K8's project.   After investigation,  SCDF-server-k8's should have used the 1.0.x. release.   The 1.1.was used to follow the version cadence of dataflow and later deemed not necessary.


Create Issue templates using GitHub Issue Forms

This PR creates Issue templates using the GitHub Issue Forms.
Fixes #9

@lohanidamodar I have made the requested changes

@lohanidamodar Sorry for the ping, but can this PR be reviewed? 😅

THANK YOU! All changes merged 🥳
Please reach out to eldadfux on our Discord server if you would like to claim your Appwrite swags! As a way of saying thank you, we would also love to invite you to join the Appwrite organization on GitHub. Please share your GitHub username with us on Discord.
You can accept the invite by visiting https://github.com/orgs/appwrite/invitation. By joining our team, you will officially be an Appwrite maintainer on GitHub.
You can change your membership visibility settings, so your new Appwrite team membership badge will show up on your personal GitHub profile.
Please feel free to look for more PRs you might be interested in helping with on our long list of Hacktoberfest friendly issues and help make Appwrite better :)


Make it installable by brew

Not sure, what is involved, never did it, but probably goreleaser we are using could do it

GoReleaser cannot generate valid formulas for Homebrew-core, only for Taps (Third-Party Repositories) https://docs.brew.sh/Taps.html. Taps involves using a Github repository to store and access formulas.
It may be convenient to use a separate repository for Homebrew formulas, for example umputun/homebrew-apps. Formulas for other utilities could also be added there. In this case, the installation would look like this:
brew tap umputun/apps
brew install umputun/apps/spot

Unfortunately, there is already a package called "spot" in Homebrew-core, so when using the unique name, you have to pass the formula name along with the path to brew install. If a unique name is used, only the name can be passed in the "install" command.
As a test, I added a "brews" section to goreleaser.yml and added a copy of the formula to Dockerfile.release. I ran "make release" locally and got a valid formula for Homebrew.
GoReleaser can push the formula directly to the formula repository, but in the Dockerfile.release of spot, it is run with the --skip-publish option. Unfortunately, I have not fully understood the release mechanism in spot yet. Therefore, I manually moved the obtained file to a separate repository https://github.com/olegmlsn/homebrew-apps.
You can try it like this:
brew tap olegmlsn/apps
brew install olegmlsn/apps/spot

The PR #81  is only for ease of viewing.
Is installation using homebrew taps suitable for you?
Are there any considerations to keep in mind?
Sorry for my English

thx. The release is not that complicated. All it does is build artifacts with gorleleaser inside the container and put it to .bin/artifacts of the host. As I release binary, I'm uploading them all manually to gihub. I don't recall what was the initial reason why this goreleaser thing runs in a container, maybe it is not even needed anymore. My goreleaser experience is very limited, so feel free to improve the flow.
Regarding your PR - I have tried it, and it produced spot.rb. Can you advise how I publish this tap? Quick googling mentions a separate git repo for this, but I don't care enough to make a separate repo and would like to keep it as simple as possible. If this is doable can you pls add a Makefile target "brew" publishing this tap?

I have added half-backed automation running release.yml pipeline and publishing things, including brew. Pls take a look and make it right if you like ;)

Unfortunately, brew tap repo clones the entire repository to the user's device. I think using a separate repository is not the worst solution. With GoReleaser and its commits during release, this looks like keeping it simple.
If you create a repository, I could make the remaining changes.
Is adding tap publication to the makefile still necessary?
In my opinion, everything looks fine in release.yml.

ok, I made the repo and restored the name to the original homebrew-apps as you suggested.

Is adding tap publication to the makefile still necessary?

nah, not needed to publish to brew, as the Makefie just build binaries, not publishes them. I think the current release target does it as is

It works great. Thank you!

@olegmlsn any idea why the release thing is so slow on GitHub actions? Locally it runs very fast, like a few seconds, but on github it takes 3+ minutes. This is not something that critical, but rather surprising

The reason may be in actions/setup-go@v4. It's just a guess. Changes in v4 are related only to caching, but it's worth checking. If possible, change it to v3 and check.


gcc{10,12}: fix builds on aarch64-darwin

Description of changes

GCC 10: add patch from the Darwin porting repo. Unfortunately, ?full_index=1 does not work with compare diffs.
GCC 12: update patch for GCC 12.4, which fixes the failing build (e.g., https://hydra.nixos.org/build/267015685).

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Having to compare against a branch is also not ideal, but comparing to a commit cherry-picks that commit. If the branch is updated, the patch will break. However, I think being updated is unlikely given 10.5 should be the last release in the 10.x series.

I'm getting a hash mismatch on the branch diff:
         specified: sha256-kVUHZKtYqkWIcqxHG7yAOR2B60w4KWLoxzaiFD/FWYk=
            got:    sha256-yI1u2ZSKHpa86ucTiiKweFyGNLMNDgmiFcmyyi34fYk=

I'm getting a hash mismatch on the branch diff:
         specified: sha256-kVUHZKtYqkWIcqxHG7yAOR2B60w4KWLoxzaiFD/FWYk=
            got:    sha256-yI1u2ZSKHpa86ucTiiKweFyGNLMNDgmiFcmyyi34fYk=

I fixed the patch link. It’s supposed to be with the branch not with the commit. As mentioned in https://github.com/NixOS/nixpkgs/pull/329205#issuecomment-2243479595, there’s a risk the branch could change. I think it’s unlikely, but the patch is too large (~1 MiB) to vendor, so I can’t think of anything better to do.


Added combat levels to the player indicator overlay

https://github.com/runelite/runelite/issues/6567

Is there a reason why this was never merged? (Other than the conflict)

Is there a reason why this was never merged? (Other than the conflict)

I think we were concerned about Offers additional information about other players, with the purpose of scouting PvP targets and are not sure how it applies to this.

Is there a reason why this was never merged? (Other than the conflict)

I think we were concerned about Offers additional information about other players, with the purpose of scouting PvP targets and are not sure how it applies to this.

Is it possible to have it disabled when in a PvP environment/scenario? I'm not familiar with the internals but if it's possible I'm happy to try.


Update datamodel to match deployed database

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.46%. Comparing base (12779c4) to head (358ce8d).

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #160   +/-   ##
=========================================
  Coverage     94.46%   94.46%
  Complexity      566      566
=========================================
  Files            92       92
  Lines          1861     1861
  Branches        100      100
=========================================
  Hits           1758     1758
  Misses           69       69
  Partials         34       34

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update Node version to 10.16.3

Related to https://github.com/jhipster/generator-jhipster/pull/10236

Please make sure the below checklist is followed for Pull Requests.

[x] Travis tests are green

[x] Tests are added where necessary

[x] Documentation is added/updated where necessary

[x] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

Although the build is going nuts currently going to merge some small PRs which are imho save to merge.


feat: generalize ReaderT instances

There are no behavioral changes here, only generalizing things like "ReaderT r m is an applicative when m is" to all the defined semimonad types instead of assuming Monad for everything.

bench!

I think the bang is on the other side. Can I do that?
!bench

!bench

!bench

@digama0 Did you find out what was triggering the stack overflow in debug mode?

No, in the end I ran out of time / patience on debugging and just did black box testing to find the part of the PR that doesn't hit the error. That's also why it scaled back from the original intent (it's not really generalizing the instances anymore, only adding implementations for the missing functions). If we could find the culprit we might be able to do this better later.

@digama0 Thanks for the explanation. I will try to take a look later at the StackOverflow issue.


GET /mnt/overlay/acs-commons/components/utilities/genericlist.html HTTP/1.1] libs.granite.ui.components.shell.propertiespage Unable to render properties page correctly org.apache.sling.api.scripting.ScriptEvaluationException:

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: 6.5.8_
[ ] ACS AEM Commons Version: 5.0.4
[ ] Reproducible on Latest? yes

Expected Behavior
Please describe briefly the expected behavior, i.e. when I do X, Y should happen.
Actual Behavior
Please describe briefly the actual behavior, i.e. when I do X, Z happens.
Steps to Reproduce
We have in-place upgrade of AEM from 6.4 to 6.5 and in one of the instance we are observing issue with generic list with more than 8 element doesn't load.
Links
Links to related assets, e.g. content packages containing test components
#2617

Hi, Is there any update on this? we are facing similar issue in Upgraded environment, cannot reproduce in local instances.

This issue has been fixed in the release 5.3.4 - 2022-08-22.
Fixed
https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/2617 - Fixed issue with NPE in Generic Lists


Jobs List API doesn't handle has_more parameter

jobs.List function doesn't use has_more parameter, so there will be incomplete results.

Jobs list now uses token-based pagination to fetch the complete list of results, so I'll close this out.


NoReverseMatch when using a record ID as a parameter

I'm trying to link to a Lead record using the ID of the SF Lead as a parameter.
urls.py
...
url(r'^leads/(?P<lead_id>\d+)/edit/$', views.edit_lead, name='edit_lead')
...

views.py
...
def edit_lead(request, lead_id):
    lead_obj = get_object_or_404(Lead, id=lead_id)
    form = LeadForm(instance=lead_obj)
    return render(request, 'edit_lead.html', {'form': form})
...

template
...
        {% for lead in paged_leads %}

                {% bootstrap_label "Name: "%} {{ lead.name }}
                {% bootstrap_label "Phone: " %} {{ lead.phone }}
                {% bootstrap_label "Email: " %} {{ lead.email }}
                - Edit

        {% endfor %}
...

models.py
...
class Lead(models.SalesforceModel):
    name = models.CharField(max_length=121, verbose_name='Full Name', sf_read_only=models.READ_ONLY)
    name = models.CharField(max_length=121, verbose_name='Full Name', sf_read_only=models.READ_ONLY)
    phone = models.CharField(max_length=40, blank=True)
    email = models.EmailField(blank=True, null=True)

class Meta(models.Model.Meta):
        db_table = 'Lead'
        verbose_name = 'Lead'
        verbose_name_plural = 'Leads'
        # keyPrefix = '00Q'
...

I get the following error:
django.core.urlresolvers.NoReverseMatch
django.core.urlresolvers.NoReverseMatch: Reverse for 'edit_lead' with arguments '('00Q1a0000034NePEAU',)' and keyword arguments '{}' not found. 1 pattern(s) tried: ['leads/(?P<lead_id>\\d+)/edit/$']

Any thoughts?  The ID renders correctly on the page if I just do {{ lead.id }}, but the URL never seems to work.  Thanks in advance.

This is a basic Django issue. You've used named groups in your URL pattern, so you need to declare the lead_id as a keyword argument by setting it to None in the view function signature.

My mistake.  Thanks for the help.


Background video playback does not work anymore on iOS 14

With iOS 14 changes now prevent background video playback from working. With the inline demo:

Detached / detached while locked modes do not work.
It is still possible to have background video playback, but the view has to be deattached before entering in background, apparently.

It is likely that this cannot be fixed at all. We will attempt to discuss this issue with Apple engineers to know if this is desired behavior or a regression, and if a fix is planned.
If no fix is planned or if this is intended behavior, we will have to either drop this feature entirely for all supported OSes (simple) or to update our code to disable it for iOS 14.

Reported to Apple as FB8722590.
A possible partial workaround is to use UIApplicationWillResignActiveNotification instead of UIApplicationDidEnterBackground. This has two negative side-effects, though:

Video layers are detached when multi-tasking, while this was previously not the case.
Playback is still paused when locking the device with the app active and the video layer attached.

I tried to hook earlier into the application lifecycle (the missing willEnterForeground) but could not solve the issue.

Seems to work with iOS 14.2 beta 2. We’ll wait until the final version, which should not be far away.

Still fixed in 14.2 GM. We can consider this issue has been resolved, though it will never work in iOS 14.0 and 14.1 of course.


2.0.0 m1 release

Ehm... what is this good for when it changes version to one in one commit and basically revert it the other?

once this is rebased and in master, one can find the point in time when (=the right commit from which) particular release/milestone was created. It also helps to keep full history


ligo-raven v1.15

Bump to v1.15, use gitlab release tarball url.

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Unsupported effects while spectating

Will need some time to test, but it seems Engine Off is fine but not Turbo? Could be opposite to what works for replays, very strange

Working:

Reactor level & type
Engine Off
Presumably ForceGas, NoBrakes, NoGrip, NoSteer

Not working:

Turbo
Turbo timer
Reactor timer

Unknown:

Slow-Mo


Stop highlighting at the end of the line

Editors like Sublime and Code do not highlight the entire line when doing a multi-line selection, instead they stop after the last character on that line.
I wanted to implement the same thing on Atom. The way I did it is very hacky and not production ready but it gets the job done. Right now, if you highlight 5 lines, the first and last lines are their own "tile" and all the remaining ones are grouped into a single rectangular tile. Since we want to have each line with a different width, I forced the tile size to 1 and removed the logic that makes it expand all the way.
I'd really like this feature to be in atom, let me know how I can refactor the code to make it shippable. Thanks
Different editors (top left: sublime, bottom left: code, top right: atom):

Before:

After:

+1

Loving this!

Thanks for contributing! It looks like this breaks several core specs though - can you take a look at those? This will also need some specs to be merged.

@vjeux Any chance you can come back and adjust specs? This would be awesome to have.
Furthermore it looks you're not selecting the new line characters. That's part of what makes the feature work / seem to flow.

I'm sorry but I'm not going to update this pull request. Feel free to create a new one.

^ Totally expected. Guy asked for help, he got a this breaks some things, fix it, and also add specs.

Shame. This is a very nice feature.


[HOLD for payment 2024-06-20] [$250] Web-Not found - LHN shows weird animation and becomes unresponsive after invoice deeplinking

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.78-2
Reproducible in staging?: Y
Reproducible in production?: N
Found when validating PR : https://github.com/Expensify/App/pull/41730
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Navigate to staging.new.expensify.com
Deep link to send invoice (e.g https://staging.new.expensify.com/create/invoice/start/1/7783051691252045
)
Click on the back icon of the not found RHP
Click on Settings

Expected Result:
RHP closes with the right animation and LHN becomes responsive
Actual Result:
RHP closes with a weird animation and LHN becomes stuck and unresponsive when opening settings
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/a2f3d1e9-7279-4857-9754-ea0b4e0d0254

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01adb4996302fbbbfc
Upwork Job ID:<PHONE_NUMBER>877054464
Last Price Increase: 2024-05-31
Automatic offers:

shubham1206agra | Contributor | 102545412

Issue OwnerCurrent Issue Owner: @johncschuster

@johncschuster I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

Production

Removed the deploy blocker as per this plan. Let's assign @tienifr to fix and @shubham1206agra for the c+ review, since they are the authors of the original PR 🙏

Sorry about the tag, @DylanDylann. We're assigning this one out to @tienifr and @shubham1206agra.
@tienifr / @shubham1206agra, can you comment on the issue so I can assign this to you?

Will be paying this out on the 20th if there are no regressions. @shubham1206agra, can you complete the BZ Checklist above before then? Thank you!

Got it. Thanks, @shubham1206agra!

@shubham1206agra bump on the above question!

@shubham1206agra Great! In that case, it sounds like we can close this one.


xfe fails to build

Installing 'xfe' failed for me, using a non-root Nix setup and the nixpgs-unstable channel.
The reason is that mesa-10.6.8 is no longer available as
ftp://ftp.freedesktop.org/pub/mesa/10.6.8/mesa-10.6.8.tar.xz
the correct URL is now
ftp://ftp.freedesktop.org/pub/mesa/older-versions/10.x/10.6.8/mesa-10.6.8.tar.xz.
Possibly this can be fixed in nixpkgs/pkgs/development/libraries/mesa/default.nix (I am too much of a Nix newbie to know for sure).

what channel verision are you using? It might be solved with a channel update

It is nixpkgs-unstable (mistyped in original post).

Fixed by #11942

I tried the #11942 fix (by a quick-and-dirty edit in nixpkgs/pkgs/development/libraries/mesa/default.nix) - it seems to work fine.
The problem is a stopper for other packages as well, for example `nix-env -i emacs' fails in the same way on nixpkgs-unstable.

Yep, unstable has many issues unfortunately since the channel currently isn't updating (#11865)

Closing as it's fixed


Using otlp, with prometheus and loki

As a user, thank you for creating otel-lgtm.
If using otlp in all components, it would be more easier to migrate from other components to lgtm, from lgtm to individual (prometheus, tempo, loki).
Can I contribute to this?

I don't understand - can you elaborate?

this is how we want otelcol to be:

replace prometheusremotewriteexporter to otlpexporter
replace lokiexporter to otlpexporter

exporters:
  otlp/traces:
    endpoint: http://localhost:4418
  otlp/metrics:
    endpoint: http://localhost:9090/api/v1/otlp/
  otlp/logs:
    endpoint: http://localhost:3100/otlp

service:
  pipelines:
    traces:
      receivers: [otlp]
      processors: [batch]
      exporters: [otlp/traces]
    metrics:
      receivers: [otlp,prometheus/collector]
      processors: [batch]
      exporters: [otlp/metrics]
    logs:
      receivers: [otlp]
      processors: [batch]
      exporters: [otlp/logs]

OK, now I get it :smile:
@fstab I think this is a good idea - wdyt?

YESSSS, please create a PR. I also wanted to upgrade to Tempo 3 and do this, but didn't find the time yet.


N+1 error for upsert_all with unique_by in Rails with postgres

Thanks for your great gem! I have just discovered it and so far it looks like a great alternative to bullet.
This one behaviour is slightly annoying though because my main motivation for using one of rails' upsert_alls is to reduce the number of queries. I think the main problematic part is the unique_by keyword, which triggers some internal lookups which are beyond your control:
       -- N+1 queries detected:
         SELECT a.attnum, a.attname
       FROM pg_attribute a
       WHERE a.attrelid = 119170
       AND a.attnum IN (3)

         SELECT a.attnum, a.attname
       FROM pg_attribute a
       WHERE a.attrelid = 119170
       AND a.attnum IN (3,2)

         SELECT a.attnum, a.attname
       FROM pg_attribute a
       WHERE a.attrelid = 119170
       AND a.attnum IN (2)

And if I run these queries by themselves I get back the attributes which I passed into the unique_by.
Example (haven't tested this precise code below, but it's more or less a simplified version of what I have):
# table
create_table :posts do |t|
  t.string :author_name
  t.string :title
  t.text :description
  t.index %i[author_name title], unique: true
end

# model
class Post < ActiveRecord::Base
  validates_uniqueness_of :title, scope: :author_name
end

# action
Post.upsert_all(
  [
    { author_name: "Geoff", title: "Foo", description: "bar" },
    { author_name: "Frank", title: "Baz", description: "hello world" },
    # etc...
  ],
  unique_by: %i[author_name title]
)

Would it be possible to exclude any queries to the pg_* tables for postgres users? Or maybe allow users to specify a whitelist for specific tables?
My current workaround is to find the specific line within active_record and add it to the allow_list - but this feels sub-optimal, as obviously this could very easily change:
Prosopite.allow_list = ["active_record/persistence.rb:243"]

Upon a bit more investigation, it appears that the queries which triggered it have the name SCHEMA - so could be a decent way of excluding these?
 # lib/prosopite.rb:176
      ActiveSupport::Notifications.subscribe 'sql.active_record' do |_, _, _, _, data|
        sql = data[:sql]
        p data.slice(:name, :sql)
        # outputs {:name=>"SCHEMA", :sql=>"SELECT a.attnum, a.attname\nFROM pg_attribute a\nWHERE a.attrelid = 119139\nAND a.attnum IN (2)\n"}
        # ...
      end

so maybe you could have
      ActiveSupport::Notifications.subscribe 'sql.active_record' do |_, _, _, _, data|
        sql, name = data[:sql], data[:name]

        if scan? && name != "SCHEMA" && sql.include?('SELECT') && data[:cached].nil?
          # ...
      end

or something to the same effect. Maybe I could open a PR for this?

Hello, thanks for reporting this! I am a bit busy these days but i will check it asap!


feat(coordinator): ability to define static peers for Grav connection

This PR allows the user to set ATMO_PEERS to "force" Grav connections to specific peers.
Discovery is also still enabled, and will create connections in addition to the static peers

Opened https://github.com/suborbital/grav/issues/61 to track moving static peer responsibility to Grav


add a test for filtering by profile status

:nut_and_bolt: Description
I wasn't clear on whether or not status filtering on compliance reporting profiles is supported or not, so i added a test. And it is!! 🎉
Figured we should commit the test, so we know for sure that it's supported
:+1: Definition of Done
test for filtering by status on compliance reporting profiles
Related Resources
https://github.com/chef/automate/issues/443

in conversations with @rickmarry, he stated this may be reflecting the overall report status and not the profile status. i'm going to look into this locally.

i've confirmed it is doing the profile status.
basically i did a scan job with 3 profiles. 2 failed, one skipped. overall status is obviously failed, so i tried to pull the compliance/profiles endpoint with a filter of status: skipped and it only returned the skipped profile

basically i did a scan job with 3 profiles. 2 failed, one skipped. overall status is obviously failed, so i tried to pull the compliance/profiles endpoint with a filter of status: skipped and it only returned the skipped profile

Is there a way to test this scenario in code? Probably not worth redoing the ruby integration test data for this but it seems worth testing if we can.

Is there a way to test this scenario in code?

i'm gonna parse through our data we send in and see if i can find a good example. if we run a skipped profile on a failed node and then try to filter for the skipped profile that should give us what we want


Night mode for Moments Page Title Links

Currently the theme works well with everything except for the title links in the heros for Moments. The links are too dark, but it appears changing '.MomentCapsuleSummary-title' handles the coloring well enough.
Current:

Suggested:

I can create a PR to do this, but @tsriram you might be able to perform the updated quicker. Thanks.

@Nickkaczmarek feel free to send a PR. Will update it in a day.

@Nickkaczmarek have just published a new version on Chrome / Firefox stores and it should be available soon. Thanks for your contribution :)


E2EE Message is not working for iOS users

For some reasons, E2EE message is not working on iOS (but works perfectly fine on mac, windows and android).
Code

Image from GKZ on repo's discord

func def cl.sendMessageWithChunks errors relatedMessageId is None?
CHRLINE.exceptions.LineServiceException: Code: 0, Message: invalid message: relatedMessageId

func def cl.sendMessageWithChunks errors relatedMessageId is None?
CHRLINE.exceptions.LineServiceException: Code: 0, Message: invalid message: relatedMessageId

Avoid make relatedMessageId as None, this problem will be resolved in the next version

func def cl.sendMessageWithChunks errors relatedMessageId is None?
CHRLINE.exceptions.LineServiceException: Code: 0, Message: invalid message: relatedMessageId

Avoid make relatedMessageId as None, this problem will be resolved in the next version

so relatedMessageId = None changed what?

I can't check if it works, v2.0 should get fixed

refs to https://github.com/DeachSword/CHRLINE/commit/3dda93b9b41eadb7a9a226db16a390f1d46ef31d


UID returns client access_token instead of user id?

Currently the uid attribute returns the access_token for the client request:
uid { access_token.client.id }

It seems like the uid attribute should return the Drip user's ID instead:
uid { raw_info["accounts"][0]['id'] }

This was causing a problem in my app because the client.id corresponds to the authorized application, not the user who is authorizing. My understanding of the OmniAuth Oauth schema is that the uid should be the user's unique ID in the authorizing application (e.g. Drip's user id). Is this also your understanding?
I made this change in my local gem, and it works as expected. I'd like to submit a PR, however I'm not sure how to write a test for this. Do you have any thoughts on that?

A PR would be welcome! This gem was originally authored by an external party, which is probably how this slipped through. You could look to the way the GitHub omniauth gem stubs out raw_info for guidance on how to test this. (Admittedly, test coverage as it stands right now is very limited).

Good deal... I've got that PR ready for you


Brackets { is out of sync with } when blocks of code get collapsed inside

As shown in images, when everything is fully expanded, works great.
If you collapse some of the code blocks it seems to lead to them being out of sync, ie should show purple, and blue. but has blue and green, so it don't register the outermost { and sees the other ones where the code is getting collapsed.
This is just a sample but this can be rather frustrating when you have a lot of nesting

As image shows, blocks do not have to be collapsed - opening and closing parens not matched in color:


[Question] where is the child process

Hi @alanz, as a usual way, there is always a child process to communicate with an executable which in this language server it should be haskell-ide-engine, but after walking throught the code, i can not find any part of it, so how does this language server actually interact with haskell-ide-engine? am i missing something here? thx in advance!

Hi. A good question.
At the moment, the main loop is invoked by the main thread in hie, via the "--lsp" flag, which comes out here: https://github.com/alanz/haskell-ide-engine/blob/lsp/app/MainHie.hs#L167
This in turn eventually calls the badly named "wait" in https://github.com/alanz/haskell-lsp/blob/master/src/Language/Haskell/LSP/Control.hs#L56
I have actually renamed that to "ioLoop" in my current (unpushed) branch, because it is confusing.
That said, I am still considering the control model, as it effectively needs to mediate between two async processes, the stdio transport for LSP and the dispatcher in hie. I am strongly considering a model something like an erlang gen_server, where everything gets fed into a single channel and processed sequentially against a common state, which can be used to tie up LSP requests with hie responses, and generate async notifications as required.

BTW, I see you have a gluon language server. The intent behind haskell-lsp is that it provides plumbing for language servers implemented in haskell, without the specifics. Hence it is a separate repo to hie.
I would love to see it getting use elsewhere, and any pull requests / design input from that.

@alanz thx for the explanation! now i know the things under the hood, i did not notice the JSON RPC sorry. 😅
yeah, maybe we can take this as a starting point to fix https://github.com/commercialhaskell/intero/issues/274 where i originally saw the link of this repo :)

I would love to see it getting use elsewhere, and any pull requests / design input from that.


Improve error logging and tracing

Story
As a developer, I want to be able to quickly find, identify, and trace any errors or downtime that happen in the dev, staging, and production environments.
We currently have grafana set up with prometheus and  loki for basic error logging and cluster monitoring statistics. Adding tracing to this as well as improving our error logging (perhaps with micrometer) would help in debugging any issues we experience.
Tasks

[ ] Talk to Dipayan about his experience with microlite and GCP. We may consider moving away from grafana and using GCP monitoring/tracing instead
[ ] Add HTTP request monitoring through ingress
[ ] Improve error logging across all components, paying particular attention to core and exporter

[ ] Having the project/submission UUID on all exporter logs would help a lot

[ ] Ensure we keep a history of logs backdated at least one month
[ ] ... More tasks to be added upon further research into what is possible ...

Relates to #323

tracked in other tickets: #742


After timeframe expires, best quality found is downloaded even if a better match to target is available

From http://flexget.com/ticket/3105
Consider a configuration such as
timeframe: 2 hours
target: 720p webdl
quality: 720p-1080p hdtv+

and the following scenario:
Flexget finds a suitable episode in two formats: 720p hdtv and 1080p hdtv, whether at the same time or within two hours of each other. Due to them not fulfilling the webdl requirement of the target quality it correctly doesn't download anything yet.
The timeframe then expires and there is still no 720p webrip available.
At this point, Flexget's current behaviour is to download the best quality it has seen, which in this case is the 1080p hdtv.
This may be what is wanted, but it also may not be. It's not what I want, and I would imagine that most users would agree that in this scenario the 720p hdtv would be more appropriate, since the resolution matches the target and the source, though not matching the target, is in the valid quality range. On the other hand the 1080p hdtv, which was downloaded, does not match the target resolution (though it is in the valid range) and the source also does not match the target (though again it is in the valid range).
I think the default behaviour should be to match as many of the target requirements as possible, while of course staying within the quality requirements.
If it seems not all users would want this new default behaviour, it could be configurable. It could either be a "choose closest match: yes or no" option, or it could be more sophisticated with the "target" option expanded. A user who wants "720p webrip" could somehow specify that the 720p part is important, and the webrip part not so important, or some other such combination.
liiight asked in a comment on the old ticket:

if you don't want 1080p to begin with? why even include it in the original requirement? if that is not a valid option on your part, do not include it.
i'm not saying i'm against adding a 'closest match' feature, that does sound interesting and doable, i'm just wondering.

The answer is that I do want 1080p if no 720p is available, but I prefer 720p. At present I have no way to tell Flexget that I prefer a "worse" quality, or rather I do since that's what I'm saying in the "target", but it isn't taken into account when timeframe expires.

Let's see.

You specified the target as the 'Preferred' so of course it's going to match exactly.
You specified a rant of 720p-1080p at hdtv or above. 1st it's going to pick the 'best' qualities available. If you wish that it doesn't. Then that is up to you to specify. If you specify a range well then it's going to give you the best of that range since there's not a single option.
All assigned values for qualities are given a numeric value. Those values are then totaled giving you a numeric weight to each episode, as long as the values are within the range assigned. That is then calculated on what episode is the 'best'. So there's no way currently to assign 720p as important over 1080p since their values are treated equally when added to the other values present.

You specified the target as the 'Preferred' so of course it's going to match exactly.

I don't understand what you mean by this.

You specified a rant of 720p-1080p at hdtv or above. 1st it's going to pick the 'best' qualities available. If you wish that it doesn't. Then that is up to you to specify. If you specify a range well then it's going to give you the best of that range since there's not a single option.

Well yes, that's the problem, that's why I opened this ticket. I specify as "target" 720p webdl, so it ought to be obvious that I prefer 720p webdl. But whether that means I prefer 720p hdtv over 1080p hdtv, if these end up being the only available options, is admittedly unclear. I want an option which allows me to make this clear no flexget.

All assigned values for qualities are given a numeric value. Those values are then totaled giving you a numeric weight to each episode, as long as the values are within the range assigned. That is then calculated on what episode is the 'best'. So there's no way currently to assign 720p as important over 1080p since their values are treated equally when added to the other values present.

So I want a way to configure these weights.

@liiight thoughts making a new option per quality list:
target:
  - 720p hdtv:
      priority: 2
  - 1080p hdtv:
      priority: 3
  - 1080p webdl:
       priority: 1

That sounds like a decent solution to me.

@crawln45 this is probably the best solution. Is it going to be added?

That suggestion makes even more complicated. It's already too confusing ... besides priority could be determined by order.

Quality option could be improved to take list of requirements in the order of preferece.

I would also be interested to be able to specify an ordered list of preferences (or even an unordered list). For example, I want to be able to express that I accept "720p h264" and "720p|1080p h265". I don't want to accept "1080p h264". I think, this is currently not possible.

Another option for me would be to be able to have something like that:
           normal:
             timeframe: 12 hours
            target: 1080p dd5.1|ac3|dts|dtshd
            quality:
              - "720p|1080p webrip|hdtv|webdl|bluray h265 !aac"
              - "720p webrip|hdtv|webdl|bluray h264 !aac"

Unfortunately, the quality setting of the serie plugin doesn't support a list (while the quality plugin does, this is a bit confusing).

Would the reorder_quality plugin solve this problem?
reorder_quality:
720p:
above: 1080p

Please reopen in feathub


Add OSGi manifest to build.

Thanks for your PR. Is that enough to be OSGI compatible?
Btw, the built failed due to my Continuous Delivery mechanism and is not related to your changes.

I merged it manually and it builds fine on Travis from my branch. In fact MANIFEST.MF has been enhanced automatically. Thanks for your contribution.
Your change will be available in "just being automatically released" version 2.0.1. It should be available in The Central Repository in several minutes.


Add more detail to the usage description

gtop is a great module, I really like the fidelity of the output as well! Just offering a small patch for the readme docs.
The usage description was good, but it didn't state how to start the application. To most veteran node users this would be self explanatory, but to others this may be difficult to work out.

@aksakalli, is there anything I can do to help this get reviewed and merged?

hi, It is also possible to quit by pressing q

@aksakalli great thanks, I've added that as well 👍


Add guide on using urql outside of React.

This PR is currently WiP, but I figured I'd get feedback early from folks. The purpose of this PR is to create a guide for using urql outside of React. Specifically, I'm focusing on highlighting how to use urql's client directly, since that's the easiest way for non-React consumers to get access to the core functionality.
Currently, I have two examples for using urql with NodeJS. I'd like to add a basic example with Vue as well before merging.
The biggest issue I see with using urql in non-React codebases is the peer dependency on React, and the way our internal references to react require it being installed even in a Node or Vue environment where it isn't used. I'd be curious to see if our tree-shaking works out in a Vue environment. Regardless, this makes me think of two things:

We should probably have a Caveats section in this doc that makes this plain. It's not a big deal if you're just using urql to write quick little Node scripts / CLIs for personal use, but the peerDep on react makes me question how useful this is.
As @kitten suggested in #253, there may be a time in the future where we consider splitting the client into a separate package to support non-React consumers who just want access to the client. But I don't think we're there yet 😅 We should continue to focus on making this the best it can be for React.

Do we want this to even include the HTTP server bits? This doesn't showcase a spec- compliant GraphQL server, so it may be a confusing example

I'll remove these. They're included purely because http is used in the CodeSandbox examples to write a response to the client (so that the user sees something), but I agree that they're just implementation details that are confusing in this context 👍

I've just been experimenting with urql.  We have an app that is angular 1 and react.  All new things are in react, but we have a ton of old angular components that we don't have time to rewrite.  Here is what I ended up using in my experimentation.  So far I just played with queries.
I export my urql client that I created and then pull that in to an angular factory.
import angular from 'angular'
import { pipe, subscribe } from 'wonka'
import { createRequest } from 'urql'
import { client } from '@/graphql'  // this is the client I created

angular.module('common').factory('GraphQL', GraphQL)

GraphQL.$inject = ['$q']
function GraphQL($q) {
  return { query }

  function query({ query, variables }) {
    const request = createRequest(query, variables)
    return $q(resolve =>
      pipe(
        client.executeQuery(request),
        subscribe(resolve)
      )
    ).then(({ data, error }) => (error ? $q.reject(error) : data))
  }
}

Somewhere else
return GraphQL.query({
      query,
      variables: { id }
    }).then(data => data.theThing)

If we ever decide to move forward with urql (we are currently playing with apollo) I'd be happy to share whatever we end up doing.
But, it sounds like you were asking for feedback, so I thought I'd pass this along.

Thanks for the example @dusty. I'm glad to see that folks already seem to be figuring a lot of what this guide will try to formalize. I think we're currently planning to keep the examples to Node and Vue, but if there's more interest I'd definitely be happy to have an Angular or Ember example in there as well; I'd probably just want to ensure we were showing examples in the latest versions of those frameworks.

@JoviDeCroock definitely. I never did see an invite come through, would you mind resending it?


Decouple the abi files

Right now, all the required abi are generated in the single file contracts/abi/SP1ICS07Tendermint.json. I often find myself adding new structs to the public api using apiPublicTypes function.
It should be possible to get rid of this function by breaking down the abi to different json files for each interface contract

issue moved to https://github.com/cosmos/solidity-ibc-eureka/issues/131


validation of ppa breaks ppa

Describe the Bug
#1052 introduced validation of ppa names
  # Validate the resource name
  if $name !~ /^ppa:([a-zA-Z0-9\-_]+)\/([a-zA-z0-9\-_]+)$/ {
    fail("Invalid PPA name: ${name}")
  }

But it does not seems to account for all valid ppa names
my ppa now fails
https://launchpad.net/~jason.grammenos.agility/+archive/ubuntu/php-static
Invalid PPA name: ppa:jason.grammenos.agility/php-static

Expected Behavior
valid ppa's should be acccepted
Steps to Reproduce
Steps to reproduce the behavior:
  ::apt::ppa { "ppa:jason.grammenos.agility/php-static":
    notify  => Exec['apt-update'],             # Resync package index files
    require => Package['add-apt-package'],     # Ensure system can add ppa's
  }

notice error in output
Environment

9.0.1
Ubuntu 20.04

Additional Context
Add any other context about the problem here.

Hey @jgrammen-agilitypr - wondering if this pr fixes the issue?

it does not appear to change the regex for ppa's, so no I do not think it does.

Thank you for the crazy fast fix.
correct me if I am wrong but it appears that the fix has a bug in it?
if $name !~ /^ppa:([a-zA-Z0-9\-_.]+)\/([a-zA-z0-9\-_\.]+)$/ {
should there not be a leading \ in front of the added . just like in the second half of the regex?
ex:
if $name !~ /^ppa:([a-zA-Z0-9\-_\.]+)\/([a-zA-z0-9\-_\.]+)$/ {

Not necessary to escape a dot in a character class.


Implement / test new AWS Toolkit feature for VSCode - ECSContainer Support

The SDE's for AWS Toolkit have implemented ECSContainer support for roles. We need to:

Vet the functionality of new feature in VSCode on ORBIT
Remove calls in juoyter-user startup for VSCode that installs aws credentials from cached values

REF: https://github.com/aws/aws-toolkit-vscode/pull/1865
REF: Justin Keyes (jmkeyes@) from AWS SDE team

Ongoing....the plugin is accessing the compute node role, not the role that the ServiceAccount (K8s) is attaching to the container.  A SIM was  created to address:
https://sim.amazon.com/issues/IDE-5647


GET users/me returns different ObjectId on each call

also on the /register route. See:
https://github.com/fastapi-users/fastapi-users/discussions/1142

I cannot reproduce this. Your setup (at least the parts shown in #1142) doesn't look suspicious, though. Do you pass the correct models to the functions generating the endpoints and all?


doc: use gcp-get-secret to retrieve the sensitive values at startup

- avoid storing plain text secrets in the container image
- update secrets without rebuilding the image

superseded by #131


Multi-threaded problems

https://github.com/openGemini/opengemini-client-python/blob/c96d4eb1f6f43460d88aef27ccae8b7681f8abe5/opengemini_client/client_impl.py#L89-L92
The increment and get_value operations are not simultaneous, and the behavior may not be as expected when concurrent.
Also, monotonically incrementing numbers and taking modulo operations is not a reasonable way to rotate.

closed by #58


For each override, have a unique and descriptive sets of summary and remarks that distinguish between the overrides

For example:

The same screenshot as above but with NeuroPixelsV1eBno055Data instead

Those screenshots above correspond to comments in the following files on the following lines:
BreakoutAnalogOuput.cs:
https://github.com/open-ephys/onix-bonsai-onix1/blob/7ac700fb8a696d4d9f8d97c9d1b85765b9951ca6/OpenEphys.Onix1/BreakoutAnalogOutput.cs#L35-L37
https://github.com/open-ephys/onix-bonsai-onix1/blob/7ac700fb8a696d4d9f8d97c9d1b85765b9951ca6/OpenEphys.Onix1/BreakoutAnalogOutput.cs#L95-L97
https://github.com/open-ephys/onix-bonsai-onix1/blob/7ac700fb8a696d4d9f8d97c9d1b85765b9951ca6/OpenEphys.Onix1/BreakoutAnalogOutput.cs#L116-L118
Neuropixelsv2eBno055Data
https://github.com/open-ephys/onix-bonsai-onix1/blob/7ac700fb8a696d4d9f8d97c9d1b85765b9951ca6/OpenEphys.Onix1/NeuropixelsV2eData.cs#L40-L42
Neuropixelsv1eBno055Data
https://github.com/open-ephys/onix-bonsai-onix1/blob/7ac700fb8a696d4d9f8d97c9d1b85765b9951ca6/OpenEphys.Onix1/NeuropixelsV1eBno055Data.cs#L35-L37