Risk management and RoSH summary pages pull data from OASys

Changes in this PR

Updates to our OASys import code, copied from CAS1
Change to our RoSH level page - showing a warning if risk data could not be read from OASys
Changes to our risk management plan page - getting questions and pre-filled answers from OASys, or falling back to JSON-driven questions, rather than using a static set of questions
A versioning pattern, used on our risk management plan page

Screenshots of UI changes
Before
RoSH level page

Risk management page

After
RoSH level page

Risk management page

Risk management page errors

Risk management page CYA

Release checklist
Release process documentation.
Pre-merge checklist

[ ] Are any changes required to dependent services for this change to work?
(eg. CAS API)
[ ] Have they been released to production already?

Post-merge checklist

[ ] Manually approve
release to test
[ ] Manually approve
release to preprod
[ ] Manually approve
release to prod

@tahb Couple of things I want to flag:

When we can't communicate with OASys, we fall back to using the questions in oasysStubs.json, as per CAS1. I've not delved into whether those are definitely the right questions for CAS3

Given that getOasysSections() is necessarily called both when the page is first displayed and when the user clicks save, and given that data is effectively co-owned (imported from OASys, but editable by the user), it's quite hard to reason whether we're doing the right thing when it comes to displaying the "last updated" date. I think my change in my "Update dates persisted by getOasysSections()" is an improvement, but I think there are still some failure cases

I've also extracted this ticket - https://trello.com/c/GKSfwwdF/1430-fix-question-sorting-on-risk-management-plan-apply-screen

@bryangaledxw Selina has suggested that the 4 new sections of the RMP aren't useful for our users. How much work would it be to put an allow list around the 4 questions we care about so that we forget/ignore the remaining 4 for either manual and automatic input?

@tahb I don't think huge, if you're happy for me to spend more time on this. On Selina's point about sorting, I pulled out a ticket for that, as I didn't feel I had time to address that - https://trello.com/c/GKSfwwdF/1430-fix-question-sorting-on-risk-management-plan-apply-screen

@bryangaledxw Agree on sorting, they should still be able to succeed in spite of that.
I've talked the idea of removing the 4 new questions through with Selina some more and I think we have to do default back to the original 4 questions.
The original 4 were on the old CPP referral forms and have been tested. The new questions are untested and may cause problems.
Referrals are still disabled in prod so you have the option of merging this in as is and adding a follow up PR. That may also give us an easier way to revert and add the questions all back in should we learn that HPTs really need them after all.

@tahb Question filtering is in if you want to take another look. Relevant commit is "Filter risk management questions"

@tahb Yeah, I think ideal world this filtering would be API side. Checking the RM numbers seemed safer than checking the labels, and in any case, if things change, the E2Es will fail so we'll know about it


List all Auditors for the Organization documents invalid query parameter

The API documentation for List all Auditors for the Organization describes a organization_guid filter. This filter is invalid as the organization_guid is included in URI instead.

Will be addressed once https://github.com/cloudfoundry/cloud_controller_ng/commit/ebbae0ab4bad7d1ce8fec3402812c45e352d4210 makes it into a cf-release.
You can find docs that include this change at
http://s3.amazonaws.com/cc-api-docs/104228398/index.html


Failed test diffs are not coloured when multiple test suites are run in parallel on Node 6

Do you want to request a feature or report a bug?
Bug
What is the current behavior?
Repro here: https://github.com/gsteacy/jest-bug-repro. Running npm test -- --no-cache is sufficient.
When Jest is running multiple test suites in parallel it does not show colours in the diffs for failed tests. Running the test again with npm test -- --no-cache -i so that they run serially will show the colours. Also, running the same tests on Node 8 works as expected.

What is the expected behavior?
Diff is coloured

Please provide your exact Jest configuration
Defaults
Run npx envinfo --preset jest in your project directory and paste the
results here
  System:
    OS: Windows 8.1
    CPU: x64 Intel(R) Core(TM) i7 CPU         920  @ 2.67GHz
  Binaries:
    Node: 6.13.1
    Yarn: 1.3.2
    npm: 3.10.10
  npmPackages:
    jest:
      wanted: ^22.4.2
      installed: 22.4.2

I'm facing the same problem running under these conditions:
  System:
    OS: macOS High Sierra 10.13.2
    CPU: 2,3 GHz Intel Core i5
  Binaries:
    Node: 8.9.4
    Yarn: 1.3.2
    npm: 5.6.0
  npmPackages:
    jest:
      wanted: ^22.4.2
      installed: 22.4.2

The highlighted errors were working when I was using jest 22.1.4

The highlighted errors were working when I was using jest 22.1.4

Not for me. It fails right back to 22.0.0. 21.2.1 shows the colours as expected. 22.0.0 is the first release with the source code context shown in the diff so that might be related (see #5087).

Rather than speculating further I did a git bisect and it seems the offending commit is eb70265


[BUG] LogLevel not mute all log output

webpack.config.js
module.exports = {
  stats: 'errors-only',

  devServer: {
    hot: true,
    clientLogLevel: 'none'
  },
}

When I edit my site and click save, the log messages ([HMR] and [WDS]) do not get to my console. But the first time the page loads, or if I press F5 (refresh the page), then I see this line in the console: [HMR] Waiting for update signal from WDS.... It should not appear, cuz I turned off the log level.
however, if I directly edit the file node_modules/webpack/hot/log.js and write there var logLevel = "none"; then the log level works as it should, and there is nothing in the console.
The bug is that when you first load the page, HMR has time to write one line exactly

UPD
i inistall some npm packages, and my webpack  package will repare, and loging is back(

Not related to webpack, it is bug on webpack-dev-server side and will be fixed in v4


fix(rag): knowledge base searche error

What problem does this PR solve?

Fix BeartypeCallHintParamViolation Error for Knowledge Base Searches

Traceback (most recent call last):
  File "ragflow/api/apps/chunk_app.py", line 286, in retrieval_test
    ranks = retr.retrieval(question, embd_mdl, kb.tenant_id, kb_ids, page, size,
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "ragflow/rag/nlp/search.py", line 304, in retrieval
    sres = self.search(req, [index_name(tid) for tid in tenant_ids], kb_ids, embd_mdl, highlight)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<@beartype(rag.nlp.search.Dealer.search) at 0x3375ea2a0>", line 72, in search
  File "ragflow/rag/nlp/search.py", line 88, in search
    matchText, keywords = self.qryr.question(qst, min_match=0.3)
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<@beartype(rag.nlp.query.FulltextQueryer.question) at 0x3375e9760>", line 31, in question
  File "ragflow/rag/nlp/query.py", line 180, in question
    return MatchTextExpr(
           ^^^^^^^^^^^^^^
  File "<@beartype(rag.utils.doc_store_conn.MatchTextExpr.__init__) at 0x152abdb20>", line 28, in __init__
beartype.roar.BeartypeCallHintParamViolation: Method rag.utils.doc_store_conn.MatchTextExpr.__init__() parameter fields=['title_tks^10', 'title_sm_tks^5', 'important_kwd^30', 'important_tks^20', 'content_ltks^2'...'] violates type hint <class 'str'>, as list ['title_tks^10', 'title_sm_tks^5', 'important_kwd^30', 'important_tks^20', 'content_ltks^2'...'] not instance of str.

In line 181 of query.py, self.query_fields is of type list, but MatchTextExpr needs to initialize arguments of type str
￼

￼
Looking for places where the "fields" field is used, I found that it is always being used as a list, so the "fields" type can be changed to list[str].
￼

Alright, now everything is working properly.
￼

Type of change

[X] Bug Fix (non-breaking change which fixes an issue)

@jiyegege I've fixed the issue by changing MatchTextExpr.fields' to list[str]`.
Thanks for your fix anyway!

how can I use this fix immediately in my local computer please? should I rebuild docker container first?

 > build
148.3 > umi build
148.3
150.2 info  - [你知道吗？] 如果你有 MPA（多页应用）需求，可尝试新出的 mpa 配置项，详见 https://umijs.org/docs/guides/mpa
150.2 info  - Umi v4.3.34
151.7 info  - Preparing...
153.3 info  - [icons] generate icons local:google, local:github
155.0
155.0 Rebuilding...
156.6
156.6 Done in 1731ms.
156.6 info  - [plugin: ./node_modules/@umijs/plugins/dist/tailwindcss] tailwindcss service exited
158.9 ℹ Compiling Webpack
194.9 [BABEL] Note: The code generator has deoptimised the styling of /ragflow/web/src/assets/svg/login-avatars.svg as it exceeds the max of 500KB.
541.7
541.7 <--- Last few GCs --->
541.7
541.7 [198:0x59f6000]   389807 ms: Mark-Compact 966.2 (997.5) -> 965.4 (996.8) MB, 1568.38 / 0.00 ms  (average mu = 0.108, current mu = 0.039) allocation failure; scavenge might not succeed
541.7 [198:0x59f6000]   391999 ms: Mark-Compact (reduce) 975.3 (998.8) -> 974.2 (999.0) MB, 1483.38 / 0.01 ms  (+ 446.0 ms in 91 steps since start of marking, biggest step 22.8 ms, walltime since start of marking 1968 ms) (average mu = 0.115, current mu = 0.120
541.7
541.7 <--- JS stacktrace --->
541.7
541.7 FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
541.7 ----- Native stack trace -----
541.7
541.7  1: 0xb8ced1 node::OOMErrorHandler(char const*, v8::OOMDetails const&) [umi]
541.7  2: 0xf06460 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, v8::OOMDetails const&) [umi]
541.7  3: 0xf06747 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, v8::OOMDetails const&) [umi]
541.8  4: 0x11182e5  [umi]
541.8  5: 0x1118874 v8::internal::Heap::RecomputeLimits(v8::internal::GarbageCollector) [umi]
541.8  6: 0x112f764 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::internal::GarbageCollectionReason, char const*) [umi]
541.8  7: 0x112ff7c v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [umi]
541.8  8: 0x1106281 v8::internal::HeapAllocator::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [umi]
541.8  9: 0x1107415 v8::internal::HeapAllocator::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [umi]
541.8 10: 0x10e4a66 v8::internal::Factory::NewFillerObject(int, v8::internal::AllocationAlignment, v8::internal::AllocationType, v8::internal::AllocationOrigin) [umi]
541.8 11: 0x1540896 v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [umi]
541.8 12: 0x1979ef6  [umi]
542.2 Aborted (core dumped)

Why does the front-end need to rebuild webpack? Can't we use the existing webpack?

docker build process here...
=> [builder 4/6] RUN --mount=type=cache,id=ragflow_builder_npm,target=/root/.npm,sharing=locked cd web && npm install --force && npm run build
 > build
148.3 > umi build
148.3
150.2 info  - [你知道吗？] 如果你有 MPA（多页应用）需求，可尝试新出的 mpa 配置项，详见 https://umijs.org/docs/guides/mpa
150.2 info  - Umi v4.3.34
151.7 info  - Preparing...
153.3 info  - [icons] generate icons local:google, local:github
155.0
155.0 Rebuilding...
156.6
156.6 Done in 1731ms.
156.6 info  - [plugin: ./node_modules/@umijs/plugins/dist/tailwindcss] tailwindcss service exited
158.9 ℹ Compiling Webpack
194.9 [BABEL] Note: The code generator has deoptimised the styling of /ragflow/web/src/assets/svg/login-avatars.svg as it exceeds the max of 500KB.
541.7
541.7 <--- Last few GCs --->
541.7
541.7 [198:0x59f6000]   389807 ms: Mark-Compact 966.2 (997.5) -> 965.4 (996.8) MB, 1568.38 / 0.00 ms  (average mu = 0.108, current mu = 0.039) allocation failure; scavenge might not succeed
541.7 [198:0x59f6000]   391999 ms: Mark-Compact (reduce) 975.3 (998.8) -> 974.2 (999.0) MB, 1483.38 / 0.01 ms  (+ 446.0 ms in 91 steps since start of marking, biggest step 22.8 ms, walltime since start of marking 1968 ms) (average mu = 0.115, current mu = 0.120
541.7
541.7 <--- JS stacktrace --->
541.7
541.7 FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory
541.7 ----- Native stack trace -----
541.7
541.7  1: 0xb8ced1 node::OOMErrorHandler(char const*, v8::OOMDetails const&) [umi]
541.7  2: 0xf06460 v8::Utils::ReportOOMFailure(v8::internal::Isolate*, char const*, v8::OOMDetails const&) [umi]
541.7  3: 0xf06747 v8::internal::V8::FatalProcessOutOfMemory(v8::internal::Isolate*, char const*, v8::OOMDetails const&) [umi]
541.8  4: 0x11182e5  [umi]
541.8  5: 0x1118874 v8::internal::Heap::RecomputeLimits(v8::internal::GarbageCollector) [umi]
541.8  6: 0x112f764 v8::internal::Heap::PerformGarbageCollection(v8::internal::GarbageCollector, v8::internal::GarbageCollectionReason, char const*) [umi]
541.8  7: 0x112ff7c v8::internal::Heap::CollectGarbage(v8::internal::AllocationSpace, v8::internal::GarbageCollectionReason, v8::GCCallbackFlags) [umi]
541.8  8: 0x1106281 v8::internal::HeapAllocator::AllocateRawWithLightRetrySlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [umi]
541.8  9: 0x1107415 v8::internal::HeapAllocator::AllocateRawWithRetryOrFailSlowPath(int, v8::internal::AllocationType, v8::internal::AllocationOrigin, v8::internal::AllocationAlignment) [umi]
541.8 10: 0x10e4a66 v8::internal::Factory::NewFillerObject(int, v8::internal::AllocationAlignment, v8::internal::AllocationType, v8::internal::AllocationOrigin) [umi]
541.8 11: 0x1540896 v8::internal::Runtime_AllocateInYoungGeneration(int, unsigned long*, v8::internal::Isolate*) [umi]
541.8 12: 0x1979ef6  [umi]
542.2 Aborted (core dumped)

this issue goes off after export NODE_OPTIONS="--max-old-space-size=8192 so the RUN command in Dockerfile.slim becomes RUN --mount=type=cache,id=ragflow_builder_npm,target=/root/.npm,sharing=locked cd web && export NODE_OPTIONS="--max-old-space-size=8192" && npm install --force && npm run build


Attempted to convert Variant to a managed array of unmarshallable element type (ArrayMesh)

Godot version
3.4.2.stable.mono.official
System information
Manjaro Linux 5.10, GLES3, NVIDIA RTX 3070 (495.44)
Issue description
When trying to add a Surface to an ArrayMesh in C# I get the following error:

Attempted to convert Variant to a managed array of unmarshallable element type

Steps to reproduce
The following code (namely, the line that starts with "mesh.AddSurfaceFromArrays") causes the error message to appear:
private void BuildMesh() {
        var arrays = new Godot.Collections.Array();
        arrays.Resize((int)Mesh.ArrayType.Max);

        Vector3[] verts = new Vector3[4];
        Vector2[] uvs = new Vector2[4];
        Vector3[] normals = new Vector3[4];
        int[] indices = new int[6];

        Vector3 normal = new Vector3(0,1,0);

        verts[0] = new Vector3(0,0,0);
        uvs[0] = new Vector2(0,0);
        normals[0] = normal;
        verts[1] = new Vector3(this.SizeX,0,0);
        uvs[1] = new Vector2(this.CellsX,0);
        normals[1] = normal;
        verts[2] = new Vector3(this.SizeX,0,this.SizeY);
        uvs[2] = new Vector2(this.CellsX,this.CellsY);
        normals[2] = normal;
        verts[3] = new Vector3(0,0,this.SizeY);
        uvs[3] = new Vector2(0,this.CellsY);
        normals[3] = normal;

        indices[0] = 0; indices[1] = 1; indices[2] = 2;
        indices[3] = 2; indices[4] = 3; indices[5] = 0;

        arrays[(int)Mesh.ArrayType.Vertex] = verts;
        arrays[(int)Mesh.ArrayType.TexUv] = uvs;
        arrays[(int)Mesh.ArrayType.Normal] = normal;
        arrays[(int)Mesh.ArrayType.Index] = indices;

        ArrayMesh mesh = new ArrayMesh();
        mesh.AddSurfaceFromArrays(Mesh.PrimitiveType.Triangles, arrays);

        this.meshInstance = new MeshInstance();
        this.meshInstance.Mesh = mesh;

        AddChild(this.meshInstance);
}

Minimal reproduction project
No response

arrays[(int)Mesh.ArrayType.Normal] = normal;

Should be = normals;. With this typo fixed I'm getting no errors.
But even with that typo I'm getting different errors (complaining about passing not a Vector3[] in):

E 0:00:01.037   _surface_set_data: Condition "p_arrays[ai].get_type() != Variant::POOL_VECTOR3_ARRAY" is true. Returned: ERR_INVALID_PARAMETER
<C++ Source>  servers/visual_server.cpp:491 @ _surface_set_data()
 :0 @ void Godot.NativeCalls.godot_icall_4_153(IntPtr , IntPtr , Int32 , IntPtr , IntPtr , UInt32 )()
ArrayMesh.cs:265 @ void Godot.ArrayMesh.AddSurfaceFromArrays(Godot.Mesh+PrimitiveType , Godot.Collections.Array , Godot.Collections.Array , UInt32 )()
ArrayMeshGeneration.cs:52 @ void ArrayMeshGeneration.BuildMesh()()
ArrayMeshGeneration.cs:15 @ void ArrayMeshGeneration._Ready()()
E 0:00:01.040   mesh_add_surface_from_arrays: Invalid array format for surface.
<C++ Error>   Condition "err" is true.
<C++ Source>  servers/visual_server.cpp:1285 @ mesh_add_surface_from_arrays()
 :0 @ void Godot.NativeCalls.godot_icall_4_153(IntPtr , IntPtr , Int32 , IntPtr , IntPtr , UInt32 )()
ArrayMesh.cs:265 @ void Godot.ArrayMesh.AddSurfaceFromArrays(Godot.Mesh+PrimitiveType , Godot.Collections.Array , Godot.Collections.Array , UInt32 )()
ArrayMeshGeneration.cs:52 @ void ArrayMeshGeneration.BuildMesh()()
ArrayMeshGeneration.cs:15 @ void ArrayMeshGeneration._Ready()()

@kleonc

Should be = normals;. With this typo fixed I'm getting no errors.

Haven't noticed that, thank you!

But even with that typo I'm getting different errors (complaining about passing not a Vector3[] in):

I am unable reproduce this.

I don't know if this has been said already but these mesh generation/creation functions could really use a rework (or some wrappers) to have proper type checking.


Proposed fix for #755 "setCurrent is not a function"

Prior to this change, if the immediate: true option was supplied for useMachine() and the service had a transient transition in its initial state, the onTransition() handler’s reference to setCurrent was undefined because useState() had not yet been executed to populate it, resulting in a "setCurrent is not a function" runtime error on service.start().
I propose resolving this by splitting up the service creation logic such that the onTransition() handler is not added until after useState() has been executed and setCurrent is defined.
See: #755

Could you add a test for this?

As astutely observed and suggested by @Andarist, the better solution was just to re-order the service.start() and useState() statement blocks, rather than split up the service creation logic. Revised the code to implement that approach.
Added a couple unit tests for the immediate option, including the scenario described in the original issue.

Looks good to me! @Andarist ?

👌 perfect, thanks!


Packet cache sweep issue

There appears to be an issue with packet cache sweeping in the erldns_storage_json module. Specifically I have noticed items in the packet cache that seem to be more than 2 weeks old whereas they should have a much shorter lifetime (1 minute or less).
At this point I'm not sure why items are not being swept properly - more investigation is necessary. Specifically I plan on executing the MatchSpec against a running instance packet_cache ets table to see what keys are returned during the sweep.

Fixed by 1d59eeb65f56799e1954d63a563b4566f9b738d8


empty placeholder issue.

Not showing editing lines when in edit mode with empty placeholder.

Before

After

Any update for this PR?

The PR fixes the issue as well https://github.com/Mehmetyaz/dashboard/issues/17

@Mehmetyaz give some response/feedback please it would show that contributing to the project makes sense and it's not dead. I have plenty of requests that I could do myself and create pull requests on review.

@GRyabkin Waiting since December still there is no response. I'm using it for production application. Hope he will reply soon.

Hi @GRyabkin , @rohanjariwala03 . Sorry for late. In the past weeks, I couldn't take care of it due to the birth of my babies. I will make the project more lively as soon as possible.

@Mehmetyaz congratulations. Thank you for the merge.


CircularReferenceError with rails 'to_json' method

Hi,
I added an enum to my (working) class, and now i get CircularReferenceError when i try to convert my instance into JSON.
By default class is serialized with the enum, which is serialized with the owner... which is serialized with the enum :-)
I think the generator (or the base class) should override as_json to except the owner attribute.
I did this into by base enum class:
def as_json(options = {})
{:sym => to_s, :name => name}
end

I looked into this a little but was getting a stack level too deep error, and not a CircularReferenceError. Perhaps it's the same thing? Either way, it should be an easy fix by just filtering out the owner from the json response as you suggested. I should have a patched version of the gem ready in the next day or so.

Which version of the gem are you currently using?

Just pushed v 1.3.1 which fixes this issue. Thanks for reporting it.


Vim keybindings

It would be nice to use ctrl+h/j/k/l to navigate the form, or at least the command-tree.

It's difficult for this to work in the form because ctrl+k for example deletes the text between the cursor and the end of the line. It could work for the command tree through.

That's an easter egg I was not aware of. Implementing this specific to the command-tree would be great, since that's the most applicable use-case anyway. I can happily tab/shift+tab my way through the form.

Just for clarity: it's not an easter egg, it's one of the documented bindings for the Textual Input widget (being well known to macOS users, amongst others).

I see, pardon my ignorance of Textual. That part of why I regard Trogon so highly; I can have a TUI without necessarily understanding the guts that power it (Textual). Relative to my little brain and the Trogon project itself, this was a happy little surprise.

To have Vim-style key bindings, I think you need to make the entire UI modal, which requires the ability to enable/disable key bindings dynamically, which can further be useful in many other scenarios, e.g., enabling/disabling tab-specific hotkeys when switching tabs.


Members Portal is Down

Good Morning,
We are currently aware of a widespread system outage on the Members Portal: https://portal.coastguard.com.au/
We are working to identify the root cause and resolve this issue as soon as possible.
Please note, there is no need to raise further support tickets regarding this issue.
You can check the service status of all AVCGA IT Services via this link: https://status.coastguard.com.au/
Further communication will be sent once service is restored or you can view service restoration via the link above.
Thanks

Resolved: Members Portal is back up in 2249274.


ENH: Stop using siemens2rads from old nipype workflows

A new interface is proposed, to be used in phase-difference based
workflows.
@AKSoo identified a potential issue with siemens2rads
(https://github.com/poldracklab/sdcflows/pull/30#discussion_r346640162),
although the re-centering did not seem to make a big difference.
This implementation of the rescaling uses percentiles to robustly
calculate the range of the input image (as recommended in the original
paper about FSL PRELUDE), and makes sure the output data type is
adequate (float32).

Okay, now I remember why we were not using the 0-6.28 range: it needs recentering (which is not the case for the 2 phases case).
I'll add a recentering node to this PR.

Still doing something odd: https://102-189292208-gh.circle-artifacts.com/0/tmp/tests/sdcflows/sub-HCP101006/fmap/sub-HCP101006_phasediff.svg

Trimming wrapped phase images at 2nd and 98th percentiles seems odd, since the values outside that range are not really outliers in a wrapped image. They are the connecting values necessary for smooth unwrapping.
In the original PRELUDE paper, the percentile trimming is done on the magnitude image if no mask is specified by the user.

Trimming wrapped phase images at 2nd and 98th percentiles seems odd, since the values outside that range are not really outliers in a wrapped image. They are the connecting values necessary for smooth unwrapping.

The intent of this was to deal correctly with phase images cast to float datatypes. Since it doesn't seem to solve any problem, and the particular change goes beyond the scope of this PR, I think you are right I should remove it 👍(done in the latest commit).


python310Packages.aiohomekit: 1.2.5 -> 1.2.6

Description of changes
https://github.com/Jc2k/aiohomekit/releases/tag/1.2.6

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

3 packages built:

home-assistant-component-tests.homekit_controller
python310Packages.aiohomekit
python39Packages.aiohomekit


Marging on chrome Android when not connected

Describe the bug
Shouldn't be left/right marging when not connected on this page https://opencollective.com/opencollective/admin/host-virtual-cards ?
To Reproduce
Steps to reproduce the behavior:

Logout
Go to https://opencollective.com/opencollective/admin/host-virtual-cards

Expected behavior
Left / right margin
Screenshots

Chrome / Android in portrait

Additional context
Add any other context about the problem here.

Triage Template (core team only)
This template is for members of the team to triage for prioritisation. For more guidance see https://www.loom.com/share/369ab467fbc64dec848085d38ff57ca0:
P1 high frequency, high impact
P2 low frequency, high impact
P3 high frequency, low impact
P4 low frequency, low impact
Examples of high impact - a problem affects users during an essential process (expenses and payments > onboarding and registration > contributing) with no workaround.
High frequency - >10% of users affected (measured as a proportion of total potential users for this case).

cc @SudharakaP

@scenaristeur : Thanks for reporting this issue. We'll fix this soon. 👍🏽


The programmer had performed a minor refactoring operation, …

… inexplicably causing all database tables to drop

not funny didn't laugh


Change --border-radius-circle to --border-radius-full

Percentage-based border-radius creates undesirable ellipse on non-square content. Instead, use pixel value and use same wording full like tailwind does, but increast to 99999px over their 9999px.

CSS recently gained support for an infinity value so calc(infinity * 1px) should work in recent browsers, but I think it's still too early to make use of that.

@silverwind please fix the merge conflicts. :tea:

@silverwind please resolve the conflicts.


move download mirror from  Kerbalstuff

@plague006  this should be more or less the same as #2287

Testing failed due being reliant on #2287 being merged. Normally recommend individual PRs for changes, exception being when they depend on each other.

so the problem was I put the link there? whoops :)
thanks @techman83

@Sigma88 more that it's using the meta data in the repo that is no longer correct for the dependency and failing it. Where as the build script injects all the metadata contained inside the PR for when the mods depend on each other :)
I don't know that you can rebase a PR made with the GitHub interface, I primarily use git on the cli.

I'm going to start over, I'm still getting a hang of this.
sorry for the problems

No issue at all @Sigma88 - appreciate the effort :-)


failing to install a postgresfixture with setuptools 38

We've started hitting this since setuptools 38 was released:
Collecting postgresfixture==0.4.0 (from -r requirements_test.txt (line 7))
  Downloading postgresfixture-0.4.0.tar.gz
    Complete output from command python setup.py egg_info:
    error in postgresfixture setup command: 'tests_require' must be a string or list of strings containing valid project/version requirement specifiers; Unordered types are not allowed

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-f03jemry/postgresfixture/

ERROR: could not install deps [-rrequirements.txt, -rrequirements_test.txt]; v = InvocationError('/run/code/.tox/py35/bin/pip install -rrequirements.txt -rrequirements_test.txt (see /run/code/.tox/py35/log/py35-1.log)', 1)

So far it's just for that postgresfixture package and was curious if there was a workaround to this or do we need to get in touch with their upstream to fix setup.py?

That's because for some reason this package's setup.py is using sets where list/tuples are normally expected (for install_requires, test_requires, even packages), and this is not allowed anymore starting with 38.0.0.

@benoit-pierre perfect, thank you. I'll take this up with upstream.


Generated translatable URLs

It would be nice to be able to generate translated urls. Thanks to the reroute hook (in hooks.ts) we can have translated routes for different languages, but I don't see a way to generate these translated addresses with route.
Describe the solution you'd like
The third parameter of this function could be a setting where I would choose the locale for the translation and then it could theoretically work, right
Contact
generates Contact
but Kontakt
generates Kontakt
Of course it would be nice if I didn't have to pass a locale to each route call, the question is how to implement this, everyone can manage locales differently.
Additional context
https://kit.svelte.dev/docs/hooks#universal-hooks

Thank you for the question, let's see how to tackle this.
First off, you need to store all raw information somewhere /contact, Contact, /kontakt, Kontakt, en, cs, and it needs to be linked somehow. (also maybe a language is "neutral" or "fallback"?).
& You need some info in src/hooks.js.
I see that in your example, the locale is not displayed in the url, but in a lot of cases, it's displayed. (So probably the two use cases should be handle)
With all this, I'm not sure it should be core of vite-plugin-kit-routes, but things are possible :)
1/ Today in vite-plugin-kit-routes

Site |

I would even suggest that you do a component on your side, handling the reactivity of $page.params.lang
2/ Extend vite-plugin-kit-routes
I started to play with pseudo code to give an idea
import { route, type KIT_ROUTES } from './ROUTES.js'

// Your supported languages _(we could imagine also translation of the label here... I don't know)_
type langs = { en: string; cs: string }

// With this type, you are sure that you will not miss any translation!
// When you add a page, it will complain that this new page is not translated. Kinda nice :)
const translatedRoutes: Record<keyof KIT_ROUTES['PAGES'], langs> = {
  '/main': {
    en: '/main',
    cs: '/main-in-cs',
  },
}

type RouteArgs = Parameters<typeof route>

export const routei18n = (lang: keyof langs, args: RouteArgs) => {
  return translatedRoutes[route(...args)][lang]
}

I didn't test it at all, don't ship this to prod 😅
It's just to give ideas, let me know if you manage something nice.
 It's ok on a small website, but I'm not sure it's the best pratice. Should be good in a lot of cases!
3/ Proper i18n full end to end solution ?
inlang seems a very good solution for that. Disclaimer: I never used it! My Apps are ONLY in French 😅
And I'm pretty sure it's possible to couple it with vite-plugin-kit-routes 🤞
Here is a nice article about it: https://inlang.com/m/dxnzrydw/library-inlang-paraglideJsAdapterSvelteKit shared by @LorisSigrist (hope we can maybe link to each other one day)

Thank you for answer!
I fixed the URL to include the languages - that was more of a bug (I can imagine that for the default language there won't be a language in the URL).
I will investigate later.

@pepa-linha should we close the issue?


Update favicon

You'll also need to update manifest.json to remove references to the logo PNG files.


model: add Global field to IndexInfo

What problem does this PR solve?
pingcap/tidb#18402
What is changed and how it works?
Add Global field to IndexInfo.
Check List

Test in tidb.

Code changes

Has exported variable/fields change

Side effects

None.

Related changes

None.

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


GH Actions: use Azure Artifacts binary cache

This will allow sharing a binary cache between this repo, the mixxx code repo, and local development without needing to use https://downloads.mixxx.org

I didn't look into this yet. Is it possible to version it? E. g. Have buildenvs for 2.3 and 2.4 in parallel?

I didn't look into this yet. Is it possible to version it? E. g. Have buildenvs for 2.3 and 2.4 in parallel?

I don't think that would be desirable. That would require rebuilding all packages for different Mixxx versions, which shouldn't be necessary.

I don't think that would be desirable. That would require rebuilding all packages for different Mixxx versions, which shouldn't be necessary.

This won't work until a secret credential is added in GitHub Actions.

This won't work until a secret credential is added in GitHub Actions.

I don't think that would be desirable. That would require rebuilding all packages for different Mixxx versions, which shouldn't be necessary.

So then we shouldn't merge this. If we use Qt6 for 2.5, we won't be able to build 2.4 anymore.

I don't think that would be desirable. That would require rebuilding all packages for different Mixxx versions, which shouldn't be necessary.

So then we shouldn't merge this. If we use Qt6 for 2.5, we won't be able to build 2.4 anymore.

Huh? What makes you think that?

Huh? What makes you think that?

I think you're making some incorrect assumptions.

I think you're making some incorrect assumptions.

What I meant: say we need different versions of the same package for 2.3 and 2.4 (or 2.4 and 2.5), will that be possible?

What I meant: say we need different versions of the same package for 2.3 and 2.4 (or 2.4 and 2.5), will that be possible?

I don't see why not as long as we don't need over 2 GiB of binaries total.

I don't see why not as long as we don't need over 2 GiB of binaries total.

If we do end up needing more than 2 GiB for multiple versions, we could easily hack around that by having another developer sign up for a free Azure Artifacts account.

If we do end up needing more than 2 GiB for multiple versions, we could easily hack around that by having another developer sign up for a free Azure Artifacts account.

This is now waiting for @daschuer to add the AZURE_ARTIFACTS_ACCESS_TOKEN secret.

This is now waiting for @daschuer to add the AZURE_ARTIFACTS_ACCESS_TOKEN secret.

@Be-ing The org secret has already been added.

@Be-ing The org secret has already been added.

Oh right, this PR is from my personal fork.

Oh right, this PR is from my personal fork.


Microsoft.VisualStudio.2022.Community version 17.6.0

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @edwardskrod,
This package installed normally but errored when launched:

Activation context generation failed for "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\devenv.exe". Dependent Assembly Automation,processorArchitecture="*",type="win32",version="<IP_ADDRESS>" could not be found. Please use sxstrace.exe for detailed diagnosis.

Is this the correct application to validate?

/azp run

Hi @edwardskrod,
This package installed normally but errored when launched:

Activation context generation failed for "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\devenv.exe". Dependent Assembly Automation,processorArchitecture="*",type="win32",version="<IP_ADDRESS>" could not be found. Please use sxstrace.exe for detailed diagnosis.

Is this the correct application to validate?

Yes.  This looks correct to me.

Hi @edwardskrod,
This package installed normally but errored when launched:

Activation context generation failed for "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\Common7\IDE\devenv.exe". Dependent Assembly Automation,processorArchitecture="*",type="win32",version="<IP_ADDRESS>" could not be found. Please use sxstrace.exe for detailed diagnosis.

Is this the correct application to validate?

I don't think so.  The application should be installed to a different location on disk - more than likely the default for the Community sku:
"C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\devenv.com"
Is the test installing the app to the Enterprise directory?  If so, then yes, that would be the correct location.
I tested the InstallerUri in the manifest:  https://download.visualstudio.microsoft.com/download/pr/12bf87fa-ff4f-4b5d-9fb2-2b1c1b59eb10/019e9c92dc19bfc056bc294ee40f4b87f597d4bded4bb3395cadb0c2ec32db09/vs_Community.exe
I'm able to successfully download, install, and run the Community edition of VS without error.
Any other ideas?  We need to be able to install this version of the Community edition via WinGet for a Build keynote.

I ran winget install --manifest locally.
The app downloads and installs Visual Studio as expected to C:\Program Files\Microsoft Visual Studio\2022\Community\
The VS process is at "C:\Program Files\Microsoft Visual Studio\2022\Community\Common7\IDE\devenv.exe"
It opens locally for me fine.


Email notification when group process is completed.

Users would like to get an email when groups are done processing, or if files have failed.

sure it can be done. Will do. It can be optional too.

I separate this to two different issues.

I added group process completed.


Distribute solana-bankrun-linux-arm64-gnu

When using Linux VMs on Apple M1/M2 for solana development, arm64-linux-gnu builds are required.
Right now, I am using the following workaround:
$ git clone https://github.com/kevinheavey/solana-bankrun
$ cd solana-bankrun
$ cargo build --release
$ cp target/release/libsolana_bankrun.so PATH_TO_NODE_MODULES/solana-bankrun/dist/solana-bankrun.linux-arm64-gnu.node

It would be nice having this work out of the box, without requiring manual compilation.

One problem here is I successfully cargo-culted the build.yml so I'm not sure which host and target I need to put in the workflow file to produce a linux-arm64-gnu build https://github.com/kevinheavey/solana-bankrun/blob/main/.github/workflows/build.yml

Ah, arm64 linux runners appear to in private beta for github ci :( https://github.com/orgs/community/discussions/19197


Execution cache file is per script folder leading to error when running scripts in parallel

I'm using dotnet script for some scripts which I run in parallel. Both scripts are in the same folder, but unless I turn off caching, this leads to a clash on the common cache file script.dll:

The process cannot access the file 'c:\somewhere\scripts\execution-cache\script.dll' because it is being used by another process.

The first script to get there works fine, but the 2nd fails because it tries to use the cache at the same time.
~It works if I use --no-cache but I would rather not.~

Oddly, it doesn't even work when I use --no-cache:

dotnet script --no-cache .\somewhere\configure-backend.csx
The process cannot access the file 'C:\Users\jenkins\AppData\Local\Temp\dotnet-script\C\somewhere\build-scripts\execution-cache\script.dll' because it is being used by another process.

Hi @mungojam
The reason it probably won't work even with the --no-cache option is that we always run the script from a compiled script (script.dll).
That being said, the cache should definitely handle executing two scripts in parallell from the same folder. Looking into it👍

I've just spotted https://github.com/filipw/dotnet-script/issues/507 which is related.
I'm planning on submitting a PR for this one first though, using the filename in the temporary folder path if that makes sense?

Is it possible to implement an abstraction layer of file system just like what scriptcs did?

A workaround is to set DOTNET_SCRIPT_CACHE_LOCATION to a separate directory for each of the scripts you are running in parallel.

Aha! Neat idea, thanks. I can then put the cache within the build workspace
too so it's easier to find and archive as needed for build diagnostics
On Mon, 12 Apr 2021, 13:28 Michael Ganss, @.***> wrote:

A workaround is to set DOTNET_SCRIPT_CACHE_LOCATION to a separate
directory for each of the scripts you are running in parallel.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/filipw/dotnet-script/issues/540#issuecomment-817768767,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAYCFS3V2BU6GFQBLDIPPPLTILRORANCNFSM4NPTG3FQ
.

Fixed by #596


Create wlodekm.is-cool.dev.json

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] Your website is not hosting on the following due to SSL issues: Vercel, Netlify

Description

My personal website (that i host myself)
Link to Website

uuuh this i guess?

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

recheck


Follow googlecartographer/cartographer#677.

RFC=0001

@wally-the-cartographer merge

Merge requested by authorized user wohe. Merge queue now has a length of 1.


Package vast-bridge as PyPI package

:notebook_with_decorative_cover: Description

Rename vast-bridge to pyvast-threatbus as per discussion in Slack
Package as PyPI package
Add a CHANGELOG to the app to track it as an individual project

:memo: Checklist

[x] All user-facing changes have changelog entries.
[x] The changes are reflected on docs.tenzir.com/vast, if necessary.
[x] The PR description contains instructions for the reviewer, if necessary.

:dart: Review Instructions
File-by-file or commit by commit, up to taste.
Test the module as follows:

navigate to apps/vast
run make dist
setup a virtual environment, somewhere on your computer, via virtualenv venv
activate it source venv/bin/activate
install the freshly built module: pip install /path/to/threatbus/apps/vast/dist/pyvast-threatbus-2020.11.26.tar.gz
run it via venv/bin/pyvast-threatbus -c apps/vast/config.yaml

@mavam I updated the PR instructions for how to test a Python module. Please let me know if that works on your machine.

Thanks, the instructions are great. Unfortunately I'm running into an error when trying to install.
python3 -m venv venv
source venv/bin/activate
pip install dist/pyvast-threatbus-2020.11.26.tar.gz
Processing ./dist/pyvast-threatbus-2020.11.26.tar.gz
Processing /Users/mavam/Library/Caches/pip/wheels/95/a4/59/10cd5378d52f92cdb45025f040e4686e10ae5217961c25fd66/black-20.8b1-py3-none-any.whl
Collecting coloredlogs>=14.0
  Downloading coloredlogs-14.0-py2.py3-none-any.whl (43 kB)
     |████████████████████████████████| 43 kB 5.1 MB/s
Collecting confuse
  Downloading confuse-1.3.0-py2.py3-none-any.whl (64 kB)
     |████████████████████████████████| 64 kB 3.7 MB/s
Collecting pyzmq>=19
  Downloading pyzmq-19.0.2-cp38-cp38-macosx_10_9_x86_64.whl (806 kB)
     |████████████████████████████████| 806 kB 9.0 MB/s
Collecting pyvast>=2020.10.29
  Using cached pyvast-2020.10.29-py3-none-any.whl (5.1 kB)
ERROR: Could not find a version that satisfies the requirement threatbus>=2020.11.26 (from pyvast-threatbus==2020.11.26) (from versions: 0.3.0, 0.3.1, 0.3.2, 2020.1.31, 2020.2.27, 2020.4.29, 2020.5.28, 2020.6.25, 2020.7.28, 2020.9.30, 2020.10.29)
ERROR: No matching distribution found for threatbus>=2020.11.26 (from pyvast-threatbus==2020.11.26)
WARNING: You are using pip version 20.2.1; however, version 20.2.4 is available.
You should consider upgrading via the '/Users/mavam/code/tenzir/threatbus/apps/vast/venv/bin/python3 -m pip install --upgrade pip' command.

Thereafter I tried pip install threatbus, which ran through, but the above error still persists.

Yeah, you need to install Threat Bus from this repo.The latest version of the threatbus package on PyPI is 2020.10.29. You can do that the same way you install this app or you use development mode, e.g., python setup.py develop in the top-level folder.

Here's the latest error message when calling venv/bin/pyvast-threatbus -h:
Traceback (most recent call last):
  File "venv/bin/pyvast-threatbus", line 33, in <module>
    sys.exit(load_entry_point('pyvast-threatbus==2020.11.26', 'console_scripts', 'pyvast-threatbus')())
  File "venv/bin/pyvast-threatbus", line 25, in importlib_load_entry_point
    return next(matches).load()
  File<EMAIL_ADDRESS>line 77, in load
    module = import_module(match.group('module'))
  File<EMAIL_ADDRESS>line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 973, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'pyvast_threatbus'

@mavam please re-test this.

cd apps/vast

make dist

python3 -m venv venv

source venv/bin/activate

pip install dist/pyvast-threatbus-2020.11.26.tar.gz
ERROR: Could not find a version that satisfies the requirement threatbus>=2020.11.26 (from pyvast-threatbus==2020.11.26) (from versions: 0.3.0, 0.3.1, 0.3.2, 2020.1.31, 2020.2.27, 2020.4.29, 2020.5.28, 2020.6.25, 2020.7.28, 2020.9.30, 2020.10.29)
ERROR: No matching distribution found for threatbus>=2020.11.26 (from pyvast-threatbus==2020.11.26)

pip install threatbus
Same error.

pushd ../../ (top-level repo dir)

make dist
make[1]: [clean] Error 1 (ignored)
rm -f -r build
python plugins/apps/threatbus_misp/setup.py sdist bdist_wheel
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
   or: setup.py --help [cmd1 cmd2 ...]
   or: setup.py --help-commands
   or: setup.py cmd --help

error: invalid command 'bdist_wheel'
make: *** [dist] Error 1

Summary: it looks like a certain threatbus version is needed. When trying to get a compatible version from the top-level repo via make dist, I am running into the above error.

@mavam Before attempting to build you must install build-dependencies. I.e., for the bdist_wheel target, you need the wheel package.

@mavam Before attempting to build you must install build-dependencies. I.e., for the bdist_wheel target, you need the wheel package.

Shouldn't dependencies be installed automatically? (At least the review instructions didn't include a special dependency setup for the venv.)

It's not in the review instructions because I assumed that the person reviewing this would have built at least some python package in the past already or at least know that wheel needs to be present for building wheel packges. It cannot be installed automagically, that is an ongoing discussion, see here why. We do list it in our setup_requires section.
So in order to move forward: have you installed the build dependency and tested this package?

So in order to move forward: have you installed the build dependency and tested this package?

This is what I'm trying; I'm not getting very far. My list of steps should tell you exactly where I'm at. Sorry, I'm a Python packaging noob. If you let me know what extra commands or packages I need, I'll execute them. I did pip install wheel and the warning is no longer here. No the output is only littered with no-such-file-or-directory warnings (errors?)
I'm still trying to get threatbus installed locally, via make dist. This is the output:
python setup.py sdist bdist_wheel
running sdist
running egg_info
creating threatbus.egg-info
writing threatbus.egg-info/PKG-INFO
writing dependency_links to threatbus.egg-info/dependency_links.txt
writing entry points to threatbus.egg-info/entry_points.txt
writing requirements to threatbus.egg-info/requires.txt
writing top-level names to threatbus.egg-info/top_level.txt
writing manifest file 'threatbus.egg-info/SOURCES.txt'
reading manifest file 'threatbus.egg-info/SOURCES.txt'
writing manifest file 'threatbus.egg-info/SOURCES.txt'
running check
creating threatbus-2020.11.26
...
make clean
find . -type d -name "*egg-info" -exec rm -f -r {} \;
find: ./apps/vast/pyvast_threatbus.egg-info: No such file or directory
find: ./apps/vast/build/lib/pyvast_threatbus.egg-info: No such file or directory
make[1]: [clean] Error 1 (ignored)
find . -type d -name "__pycache__" -exec rm -f -r {} \;
find: ./apps/vast/venv/lib/python3.8/site-packages/wheel/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/wheel/cli/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/wheel/vendored/packaging/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/wheel/vendored/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/setuptools/_vendor/packaging/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/setuptools/_vendor/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/setuptools/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/setuptools/command/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/setuptools/extern/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/pkg_resources/_vendor/packaging/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/pkg_resources/_vendor/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/pkg_resources/__pycache__: No such file or directory
find: ./apps/vast/venv/lib/python3.8/site-packages/pkg_resources/extern/__pycache__: No such file or directory
make[1]: [clean] Error 1 (ignored)
rm -f -r build

In general, can we fail hard if requirements are not met? Then it would be obvious what's going on.
Anyhow, I got some files to work with:
ls dist
threatbus-2020.11.26-py3-none-any.whl  threatbus-misp-2020.10.29.tar.gz      threatbus_cif3-2020.10.29-py3-none-any.whl      threatbus_zeek-2020.10.29-py3-none-any.whl
threatbus-2020.11.26.tar.gz            threatbus-rabbitmq-2020.10.29.tar.gz  threatbus_inmem-2020.10.29-py3-none-any.whl     threatbus_zmq_app-2020.10.29-py3-none-any.whl
threatbus-cif3-2020.10.29.tar.gz       threatbus-zeek-2020.10.29.tar.gz      threatbus_misp-2020.10.29-py3-none-any.whl
threatbus-inmem-2020.10.29.tar.gz      threatbus-zmq-app-2020.10.29.tar.gz   threatbus_rabbitmq-2020.10.29-py3-none-any.whl

I then did a pip install dist/threatbus-2020.11.26.tar.gz, which seemed to work. Thereafter, pip install dist/pyvast-threatbus-2020.11.26.tar.gz also worked.
Then I was wondering how the config should look like for testing. I don't have a MISP currently up and running. I have only the zmq-app activated under apps in the YAML file. When I start threatbus -c config.yaml, I get this output only:
threatbus -c config.yaml
2020-11-16 14:30:44 INFO     [threatbus] Starting plugins...

Is there anything missing? It seems that the bridge cannot find Threat Bus:
venv/bin/pyvast-threatbus -c config.yaml
2020-11-16 14:34:09 INFO     PyVAST: VAST client configured to use endpoint localhost:42000
2020-11-16 14:34:09 DEBUG    Calling Threat Bus management endpoint localhost:13370
2020-11-16 14:34:09 INFO     Subscribing to topic 'threatbus/intel'...
2020-11-16 14:34:14 ERROR    Subscription failed

That test completes this story. Both packages could be built, the build artifacts could be installed. Hence, they both form valid packages and I'll merge this PR :tada:
Explanation of the log messages:

make dist has successfully produced the Python package artifacts
The no such file or directory logs you see come from the clean target (Makefile), i.e., when find does not find all the patterns we want to clean.
we do fail hard if requirements are not met, except for cleaning
see config.yaml.example for an example config

From the two application logs:

There are no threatbus plugins installed, otherwise they would pop up in the log. Make sure you install them, e.g., via pip install dist/some-plugin.tgz. Your config won't do anything if you don't install the plugins.
pyvast-threatbus ideed cannot find Threat Bus. That is because the zmq-app plugin is not installed (see 1.)

Thanks for the explanations. This makes sense now. I have a suggestion regarding UX: how about printing a warning if we have a config key but no corresponding plugin installed?

Sure, we can do that. Can you make a story for it, please?


User encounters browser error when opening test books in firefox or safari

student testers report Safari browser error of "Safari cannot open the page because the address is invalid" when attempting to open test books in Safari from  iPad4G- iOS-6.0.1.
I was able to reproduce this error in Safari and Firefox on my 64 bit Win 7 Pro desktop browsers as well as our iPad air.
Daniel & Ric have suggested this may be a network firewall issue and is now being investigated by our in house tech support team. updates to follow...

Thanks @duanwalker
Given that we now know that this is due to firewall issues (invalid SSL HTTPS certificates), should we close this issue?
Dan

I would suggest that we wait for Duan to update the Testing document with the proper approach for ensuring testers don't run into this problem, then he can close it.

@duanwalker Thanks for updating the doc.  Closing.


haveOpenCL return False

Expected behaviour
Should return True if the the prebuilt binaries are compiled with OpenCL support. I've seen many examples where it just works out of the box. So my assumption is that the binaries are built with OpenCL support. So why might it not be working for me?
Also it works from C++ for OpenCV installed from source. So my computer has OpenCV runtime installed:

[ INFO:0] Initialize OpenCL runtime...

Steps to reproduce

example code: cv2.ocl.haveOpenCL()
operating system: MacOS High Sierra 10.13
architecture (e.g. x86):x86
opencv-python version: 3.4.2

Based on CMake output in the build logs macOS builds have OpenCL support. Could you post the output of cv2.getBuildInformation() here? Unfortunately I do not own macOS devices but I'll try to do my best to debug the issue.

You're right.
Turns out I've been using the cv2 from anaconda. There I get:

Use Cuda:                    NO
Use OpenCL:                  NO

When using cv2 from opencv-python I get:

OpenCL:                        YES (no extra features)
Include path:                NO
Link libraries:              -framework OpenCL

Sorry for opening such a silly issue.

You're right.
Turns out I've been using the cv2 from anaconda. There I get:

Use Cuda:                    NO
Use OpenCL:                  NO

When using cv2 from opencv-python I get:

OpenCL:                        YES (no extra features)
Include path:                NO
Link libraries:              -framework OpenCL

Sorry for opening such a silly issue.

No problem :)


Update jax.numpy.lcm function to match with np.lcm in case of overflow

Close #12728
The details are in the issue linked.
The change produces result matching with np and passing tests locally.
CLA signed.

Hi. Thanks for the review. Added test for non-small intergers with dtype int8 to detect the difference between np and jnp in overflow. Locally fails with the original and passes with the change. 🙂

Could you please squash your changes into a single commit? See https://jax.readthedocs.io/en/latest/contributing.html#single-change-commits-and-pull-requests. Thanks!


Update index.md to include authenticator config value

Added authenticator value to the documentation.
Test Plan

[ ] acceptance tests

[ ] …

References

Hey @Zesky665. Thanks for the suggestion.
This change cannot be merged, though. The reason is, that the docs (markdown files) are generated. You have to change the input (examples/provider/provider.tf) and then regenerate the docs. Without such a change it will be lost with the next docs generation. To generate them, you would have to:

update the example
setup dev env (make dev-setup)
generate the docs (make pre-push).

I am just writing this as an explanation for the future. I will introduce such change today with one of my other PRs.
We do not have proper contribution guidelines yet, but this is something we will introduce in the next couple of weeks. Docs generation will be one of the topics described there.

@sfc-gh-asawicki Thank you for the heads up.
Please let me know when the contribution guidelines are introduced, I would very much like to work on improving the documentation.

Superseded by #2515

Hey @Zesky665 . We have refreshed the contribution guidelines: https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/CONTRIBUTING.md. Please let me know if you think anything is missing.


104 update the release action

@soumendrak Can you please look at the FOSSA Compliance report and fix the 9 discovered issues?


Override equality of ResourceItem

Closes #209

Now outputs
/open-dotnet/test/DSE.Open.Localization.Generators.Tests.Functional/Resources/Strings.restext(6,1): error DSERG0006: Duplicate key 'Test' found.

as it should have.


feat: #2072 - removed dialog from food preferences radio buttons

Impacted file:

attribute_button.dart

What

The dialog each time a food preference was changed seemed intrusive
Now it's removed

Fixes bug(s)

Closes: #2072

thanks @monsieurtanuki @AshAman999 👍


[Feature Request] Message Announcing System

Is your feature request related to a problem? Please describe.
Telling everyone is too difficult and exhausting, therefore an announcing system is essential.
Describe the solution you'd like
Adding a message box with messages to display.

./my.html
./policy.html
./help.html

The following is not necessary for announcements.


Detect if replicas have been defaulted in rolling-update and use old replica count

Fixes #9645
@bprashanth @bgrant0607

cc @jlowdermilk

Thanks. I'll try to take a look as soon as I can, but not assigning to myself since I'm at Dockercon today.
@smarterclayton might also want to take a look.
Vaguely related to #7111, which (among other things) discusses decoupling unmarshaling from conversion.

I'll try although I'm at Red Hat Summit :)

On Jun 23, 2015, at 1:21 PM, Brian Grant<EMAIL_ADDRESS>wrote:
Thanks. I'll try to take a look as soon as I can, but not assigning to myself since I'm at Dockercon today.
@smarterclayton might also want to take a look.
Vaguely related to #7111, which (among other things) discusses decoupling unmarshaling from conversion.
—
Reply to this email directly or view it on GitHub.

LGTM, but @smarterclayton and/or @bgrant0607 should take a look.

@k8s-bot ok to test

LGTM

ok to merge

@mikedanese want to kick the jenkins bot to test this?

@k8s-bot ok to test

@roberthbailey k8s-bot says LGTM

Thanks. gce-e2e just went red so I'll merge once it is happy again.


Authorizer does not go to proper depth when authenticator is specified.

I was having trouble with the DataAdapterMixin not sending data in the headers.  When I dug down a bit, I discovered that in the authorizer's (I discovered it in the DeviseAuthorizer) authorize() function ended up with data that looked like this:
data = {authentication: "authenticator:devise", user: Object}
When it tried to then find the user information, it didn't go deep enough to properly populate the data.
I am unsure whether my input data is incorrect, but it seems like the function should look for my user information.

This has been fixed in master but it not yet released. If you face the issue, you can apply the following workaround until next release:
import Devise from 'ember-simple-auth/authorizers/devise';

export default Devise.extend({
  // FIXME: Waiting for ember-simple-auth > 1.1.0
  authorize(data, block) {
    this._super(data['user'], block);
  }
});

Yes, this was merged in but did not get closed.

Yup 😁 Will there be an intermediate release like v1.1.1 before v1.2 for bugfixes?
It's a good think the workaround is there, without it it would be unusable for us ^^


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target/pull/837 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target.git

Docs Build status updates of commit 165985b:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-05-24-13-46-16 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


problem with discounts, after setting discount price in appstore the price is still without discount

Steps to reproduce

choose subscription with discount
try to buy subscription
the price is without discount

Expected results
price should be with discount
Actual results
regular price when want to pay
Code sample
Code sample
[Paste your code here]

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[Paste your output here]

Are you using in_app_purchase package? If so, please try to provide all information indicated in Flutter's first-party package issue template so we have enough information to verify the issue. Thanks!

The version of the package
Target platforms
A minimal sample code to reproduce the issue

hi, thanks for the quick response, so I'm using the 3.2.0 version of the library, target platforms: Android (works fine), iOS (a problem when I set discount in the store). I use Future buyNonConsumable from library, and after my small investigation i found problem in
await _skPaymentQueueWrapper.addPayment(SKPaymentWrapper(
    productIdentifier: purchaseParam.productDetails.id,
    quantity:
        purchaseParam is AppStorePurchaseParam ? purchaseParam.quantity : 1,
    applicationUsername: purchaseParam.applicationUserName,
    simulatesAskToBuyInSandbox: purchaseParam is AppStorePurchaseParam &&
        purchaseParam.simulatesAskToBuyInSandbox,
    paymentDiscount: purchaseParam is AppStorePurchaseParam
        ? purchaseParam.discount
        : null));

it's 113 line from in_app_purchase_storekit_platform.dart file

when I want to crate SKPaymentWrapper in_app_purchase_storekit_platform.dart (113 line) paymentDiscount always is null, but purchaseParam -> productDetails -> skProduct -> discounts is not null

What is the instance type of your purchaseParam in the implementation code? Is it AppStorePurchaseParam? Can you share a sample code for this? Please check https://github.com/flutter/flutter/issues/145898#issuecomment-2026903867 to see if some info there is useful.

yeah you are right, I didn't implement a in_app_purchase_store_kit, now my code looks like:
final discounts = (product as AppStoreProductDetails).skProduct.discounts;
  final purchaseParam = AppStorePurchaseParam(
    productDetails: product,
    applicationUserName: userId,
    discount: discounts,
  );
  await _iap.buyNonConsumable(purchaseParam: PurchaseParam(productDetails: product, applicationUserName: userId));

but still doesn't work I can't add discount to AppStorePurchaseParam

but still doesn't work I can't add discount to AppStorePurchaseParam

Do you mean the price has not been updated with the discount you provided to AppStorePurchaseParam? Can you please provide a complete sample code and the version of in_app_purchase_storekit you are using? You may try with plugin example and modify it but please point out which line of code you modified since the sample code is quite long. Thanks!

I set a 50% discount price for 1 year for 1 year subscription in App Store Connect, this is working ok because when I download data from app store I see discount in AppStoreProductDetails -> SKProduct model. But when I choose this subscription and go forward to payment I see price without discount. I use 0.3.18+3 in_app_purchase_storekit version. I can't mapped SKProduct.discount to AppStorePurchaseParam.discounts I can't find any information how I should implement this in my app.

I can't mapped SKProduct.discount to AppStorePurchaseParam.discounts I can't find any information how I should implement this in my app.

I am not sure about the mapping you are mentioned to but if you want to get discount you passed to AppStorePurchaseParam, you can try getting it on SKPaymentQueueDelegateWrapper as below:

Check ExamplePaymentQueueDelegate at https://github.com/flutter/packages/blob/79f8b0b6655132ac8d11a50243afe85867a09405/packages/in_app_purchase/in_app_purchase/example/lib/main.dart#L499
On shouldContinueTransaction method, get discount from transaction: transaction.payment.paymentDiscount


Removed TestID from all tests

TestIDs are not used anywhere in LISAv2.
Instead, we continue to use TestName as unique test identifier.

What about $testId in BVT-CAPTURE.VHD.ps1?

@juhlee-microsoft BVT-CAPTURE.VHD.ps1 is unused and not referenced anywhere in LISAv2. We have alternate script - CAPTURE-VHD-BEFORE-TEST.ps1 to do this job.
PR#138 addresses this issue.
https://github.com/LIS/LISAv2/pull/138

@iamshital  I think you should remove BVT-CAPTURE.VHD.ps1 file here not a separate PR.

@iamshital  close PR #138

@juhlee-microsoft , I've tested it again with complete BVT test category. [Microsoft-Test-Execution-Pipeline/238]
Complete LISAv2 exucution is really not necessary to verify this PR.


Add option to warn when max_width is exceeded

I'd like for cargo fmt -- --check to be able to produce a warning if the max_width is exceeded (e.g. in code inside macros that is not formatted). This option should be opt-in.
This would be useful in rust-lang/rust, since the tidy script currently detect this everywhere, but rustfmt does not.

cc @nrc @topecongiro

@scampi is that a CLI option?
I've tried cargo fmt --all -- --unstable-features --error-on-unformatted --check but it does not seem to error on exceeded max_width inside macros.

Ah, it is a parameter that can be added to the rustfmt.toml. Thanks, that fixes it :)

@gnzlbg no pb, however, it's strange that you didn't get an error on the invalid command line option. Maybe worth opening a separate issue for that ?


Ingestion refactor

Do not merge.  PR is only here for an initial review

this branch is merged with new-metric-aggregation and is obsolete


SBOX_FATAL_MEMORY_EXCEEDED

Describe the bug
浏览器页面崩溃
Link to the blitz that caused the error
https://stackblitz.com/edit/nuxt-starter-kun5yb?file=app.vue
Steps to reproduce

刷新浏览器
等待资源加载

Expected behavior

完成依赖安装
资源加载完成并显示页面

Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform
Browser name  = Chrome
Full version  = <IP_ADDRESS>
Major version = 108
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 101514556,
  "usedJSHeapSize": 72724704,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = 6ebe18b1e1db1833797e3e4eb6ff802c375ee61b

Additional context
No response

I also get the same issue with: https://stackblitz.com/edit/node-angular-transloco
Just open the link and npm install works fine. But the npm start crashes the tab

I use:
OS: Windows 10
CPU: i7-1065G7
RAM: 16GB
Chrome: 108.0.5359.99

Same here. After many years using StackBlitz, now I'm getting this error.
Don't know what else to do to fix it. 😪

Update
It looks like after a while of using it, memory gets bloated.
I refreshed it, and now it's working (until it happens again).

WebContainer does use quite a bit of memory. We're having plans to improve this in the future. But apparently Chrome crashes with that error if Chrome itself runs out of memory. The only thing that might help at this point is if you would disable some/all extensions and make sure you don't open multiple StackBlitz projects at the same time.
Here's some additional information https://www.chromestory.com/2020/06/sbox-fatal-memory-exceeded/

WebContainer does use quite a bit of memory. We're having plans to improve this in the future. But apparently Chrome crashes with that error if Chrome itself runs out of memory. The only thing that might help at this point is if you would disable some/all extensions and make sure you don't open multiple StackBlitz projects at the same time.
Here's some additional information https://www.chromestory.com/2020/06/sbox-fatal-memory-exceeded/

Thank you for your time and for your so helpful response. Wishing you health, peace, and success.

Going to close this issue as I think there's not much we can do on our end.


Implement airtable

Codecov Report

Merging #113 into master will increase coverage by 0.74%.
The diff coverage is 87.5%.

@@            Coverage Diff            @@
##           master    #113      +/-   ##
=========================================
+ Coverage   90.16%   90.9%   +0.74%
=========================================
  Files          11      12       +1
  Lines          61      77      +16
  Branches        3       5       +2
=========================================
+ Hits           55      70      +15
- Misses          6       7       +1

Impacted Files
Coverage Δ

src/app.js
100% <100%> (+9.52%)
:arrow_up:

src/utils/airtableReturns.js
57.14% <57.14%> (ø)

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 10e11b1...e8adf84. Read the comment docs.


list of suggestions is too long

Take a look at this one: https://github.com/yegor256/qulice/issues/1242#issuecomment-2019426668
The list of suggestions is too long. Because of this, programmers simply ignore the entire message. Would be great to limit the list of suggestions to just three, to increase the probably of improvement.

@yegor256 thanks for the report. Got it. I like the idea of limiting the number of "things to improve"

@yegor256 now it should generate maximum 3 suggestions per issue/puzzle


Update Copy-DbaSqlServerAgent.ps1

Typo in code. Suspect is the cause of error "Exception calling "Query" with "1" argument(s): "Exception calling "ExecuteWithResults" with "1" argument(s): "Execute with results failed for Database 'master'. """

Type of Change

[x] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Pester test is included

Purpose

Approach

Commands to test

Screenshots

Learning

Hi,
we got the error below and I noticed the typo. I've not tested this change though so perhaps it's unrelated.
Cheers,
Ally
VERBOSE: Performing the operation "Copying Agent Properties" on target "#####".
VERBOSE: [Copy-DbaSqlServerAgent][11:14:35] Copying SQL Agent Properties
SourceServer      : #####
DestinationServer : #####
Type              : Agent Properites
Status            : Failed
Notes             : Exception calling "Query" with "1" argument(s): "Exception calling "ExecuteWithResults" with "1" argument(s): "Execute with results failed for Database 'master'. ""
DateTime          : 2017-08-23 11:14:35.555
WARNING: [Copy-DbaSqlServerAgent][11:14:35] Issue copying agent properties | Exception calling "Query" with "1" argument(s): "Exception calling "ExecuteWithResults" with "1" argument(s): "Execute with resu
lts failed for Database 'master'. ""

Thanks for the PR @allyreilly but just noticed this spelling error is fixed in a previous PR that just got merged.
I'm going to go ahead and close this PR.


PDF just one chapter, with correct chap/sect numbering

I want to make a PDF of just one chapter from a book, when that chapter is not necessarily chapter 1, and yet have the chapter/section numbering be correct (so if it's chapter 2, the first section would be 2.1, etc.)
I can think of a couple ways this could work:

Some way to define/override the value for :sectnum (probably an attribute)
Insert dummy sections before the first level-1 section but somehow suppress them from printing

Is there some way to do option 2 that I'm missing?

I wouldn't want to encourage you to do option 2 as that is a very non-semantic hack. Option 1 is a much better pursuit.
This need actually comes up in all outputs, not just PDF, so we likely want to solve it in core. I'm thinking something like sectnumoffset so that it begins counting at the specified number instead of 1. This would be a pretty easy change to make, I think.
In the meantime, the best way to do this is by writing a Treeprocessor extension. With a Treeprocessor, you can walk all the block-level nodes and modify them. In this case, you would just descend the section notes and assign a new section number.
require 'asciidoctor/extensions'

include ::Asciidoctor

Extensions.register do
  # A treeprocessor that increments each level-1 section number by the value of
  # the `sectnumoffset` attribute. The numbers of all subsections will be
  # incremented automatically since those values are calculated dynamically.
  treeprocessor do
    process do |document|
      if (sectnumoffset = (document.attr 'sectnumoffset', 0).to_i) > 0
        ((document.find_by context: :section) || []).each do |sect|
          next unless sect.level == 1
          sect.number += sectnumoffset
        end
      end
      nil
    end
  end
end

I've added this extension to the extensions lab.
Here's the issue in core to track. https://github.com/asciidoctor/asciidoctor/issues/1113

This is pretty similar to #188 : https://github.com/asciidoctor/asciidoctor-pdf/issues/188

I finally got around to figuring out how to install and run this treeprocessor.  Thanks!
It doesn't seem to work on appendixes, though. If I try to use :sectnumoffset: on an appendix, it says "can't convert FixedNum to String" (because appendixes use letters instead of numbers for their chapter number).

@stallio Uh oh. Could you file an issue in the extensions lab and cite and example document to test. This Treeprocessor now lives there.
https://github.com/asciidoctor/asciidoctor-extensions-lab
I'll provide another technique for rendering a single chapter that gets all the numbering right. Load the document using the API, delete all the other chapters from the AST (or copy the chapter you want to another AST), then render the result. Perhaps we can add that to the extensions-lab for demonstration purposes. You can see something similar in the multipage HTML converter. See https://github.com/asciidoctor/asciidoctor-extensions-lab/blob/master/lib/multipage-html5-converter.rb

...granted, that trick is somewhat ugly, but it certainly gets the job done :)

The sectnumoffset extension works for me when outputting HTML, but gives me this error when outputting PDF:
/usr/lib/ruby/gems/2.6.0/gems/rouge-3.15.0/lib/rouge.rb:55:in `block in <top (required)>': asciidoctor: FAILED: /documents/book/excerpt-concurrent-state.adoc: Failed to load AsciiDoc document - uninitialized constant Rouge::Lexers (NameError)
        from /usr/lib/ruby/gems/2.6.0/gems/rouge-3.15.0/lib/rouge.rb:55:in `each'
        from /usr/lib/ruby/gems/2.6.0/gems/rouge-3.15.0/lib/rouge.rb:55:in `<top (required)>'
        from /usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:130:in `require'
        from /usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:130:in `rescue in require'
        from /usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:34:in `require'
        from /usr/lib/ruby/gems/2.6.0/gems/polyglot-0.3.5/lib/polyglot.rb:65:in `require'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/rouge_ext.rb:2:in `<top (required)>'
        from /usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        from /usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        from /usr/lib/ruby/gems/2.6.0/gems/polyglot-0.3.5/lib/polyglot.rb:65:in `require'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/helpers.rb:27:in `require_library'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/syntax_highlighter/rouge.rb:78:in `load_library'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/syntax_highlighter/rouge.rb:74:in `library_available?'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/syntax_highlighter/rouge.rb:12:in `highlight?'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/substitutors.rb:1288:in `commit_subs'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:918:in `next_block'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:379:in `next_section'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:364:in `next_section'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:364:in `next_section'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:364:in `next_section'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:97:in `parse'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/document.rb:549:in `parse'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/load.rb:83:in `load'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/convert.rb:78:in `convert'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/convert.rb:183:in `block in convert_file'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/convert.rb:183:in `open'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/convert.rb:183:in `convert_file'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/cli/invoker.rb:128:in `block in invoke!'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/cli/invoker.rb:111:in `each'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/cli/invoker.rb:111:in `invoke!'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/bin/asciidoctor:15:in `<top (required)>'
        from /usr/bin/asciidoctor:23:in `load'
        from /usr/bin/asciidoctor:23:in `<main>'
/usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require': cannot load such file -- rouge (LoadError)
        from /usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        from /usr/lib/ruby/gems/2.6.0/gems/polyglot-0.3.5/lib/polyglot.rb:65:in `require'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/rouge_ext.rb:2:in `<top (required)>'
        from /usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        from /usr/lib/ruby/2.6.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        from /usr/lib/ruby/gems/2.6.0/gems/polyglot-0.3.5/lib/polyglot.rb:65:in `require'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/helpers.rb:27:in `require_library'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/syntax_highlighter/rouge.rb:78:in `load_library'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/syntax_highlighter/rouge.rb:74:in `library_available?'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/syntax_highlighter/rouge.rb:12:in `highlight?'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/substitutors.rb:1288:in `commit_subs'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:918:in `next_block'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:379:in `next_section'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:364:in `next_section'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:364:in `next_section'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:364:in `next_section'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/parser.rb:97:in `parse'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/document.rb:549:in `parse'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/load.rb:83:in `load'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/convert.rb:78:in `convert'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/convert.rb:183:in `block in convert_file'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/convert.rb:183:in `open'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/convert.rb:183:in `convert_file'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/cli/invoker.rb:128:in `block in invoke!'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/cli/invoker.rb:111:in `each'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/lib/asciidoctor/cli/invoker.rb:111:in `invoke!'
        from /usr/lib/ruby/gems/2.6.0/gems/asciidoctor-2.0.10/bin/asciidoctor:15:in `<top (required)>'
        from /usr/bin/asciidoctor:23:in `load'
        from /usr/bin/asciidoctor:23:in `<main>'

I use docker-asciidoctor and am running asciidoctor-pdf with additional flags -r ./lib/sectnumoffset-treeprocessor.rb -a sectnums -a sectnumoffset=9 --trace.
If I remove those extra flags the PDF renders fine, so somehow this extension is interfering somehow, maybe some ordering issue?

@arosien The issue you are seeing has nothing to do with Asciidoctor PDF. It's a problem with the version of Rouge you are using and a conflict with internal Asciidoctor behavior. I'll follow-up in the issue you posted to the extension lab. See https://github.com/asciidoctor/asciidoctor-extensions-lab/issues/122

I want to make a PDF of just one chapter from a book, when that chapter is not necessarily chapter 1, and yet have the chapter/section numbering be correct (so if it's chapter 2, the first section would be 2.1, etc.)

This is also possible using the following command:
asciidoctor-pdf -a doctype=book -a chapter-number=1@ -a sectnums ch02.adoc

At this point, I've shared everything I can offer to accomplish this goal. There's nothing else actionable for this converter, so I'm marking it as resolved.


Workspace parameter not opening workspace on Windows

Using the workspace parameter in an Obsidian URI to load a workspace works for me on Mac, but if I try the same URL on Windows, against the same vault, and with exactly the same workspace name set up, it does not work. When the URI is opened on Windows, the Obsidian app is opened, or is switched to the foreground, but no loading of the workspace takes place. When doing this on Mac, the workspace loads immediately .

Other URI I have tried work on both platforms, and plugin is enabled on both.
Restarting the Obsidian app made no difference.
Remove and readded the plugin, but to no avail.

e.g. this URI works for me on Mac but not on Windows against the same vault and configuration.
obsidian://open?vault=demo&workspace=1-Main

Windows: 10 (21H1, build 19043.1165)
Obsidian: 0.12.15
Advanced Obsidian URI Plugin: 1.11.1

I doubt that this URI works on your Mac. It's obsidian://advanced-uri and not obsidian://open

Thanks for the prompt response. I will recheck it sounds highly plausible as an explanation 😖
I was going through some notes for setting up a few things that worked on my Mac, then followed them again on my PC. Entirely possible I've misread something and got differing URLs, so I will check and no doubt close the issue when I find this is indeed a user error on my part.

Confirmed as user error. Closing.


yabai: could not initialize event_loop! abort..

Hi, I have installed yabai + skhd on my mac, and it works perfectly on one user (lukasz.mach).
However, when I switched to another user (maho) - I cannot start yabai because it says:
yabai: could not initialize event_loop! abort..

I tried to remove sockets from /tmp/, removed logs from /usr/local/var/log/yabai/ , I think it's not permissions to socket/logs problem.
I'm so far running yabai from terminal, but running from service (brew services start yabai) brings no difference.
'lukasz.mach' is the user I used to brew install yabai, if it matters

Fixed on master.


fixing blue theme

made everything darker and fixed: ping menu, ping color, emoji picker as well

looks like this now


Driver: introduce new -libc option for Windows

On Windows, there are multiple variants of the C runtime that must be
explicitly specified and consistently used from the runtime to the
application.  The new -libc option allows us to control the linking
phase by correctly embedding the requested library to be linked.  It is
made into a required parameter on Windows and will add in the
appropriate flags for the imported C headers as well.  This ensures that
the C library is not incorrectly linked.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @jrose-apple

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 3e36f23a89325231d91fa378f60005ff07094834

Build failed
Swift Test OS X Platform
Git Sha - 3e36f23a89325231d91fa378f60005ff07094834


Docker Container Exiting Immediately

When running the given ./docker_run.bash command I get a string with a bunch of characters (which changes every time I run it) and it appears to exit.
When I check to see if any docker containers are currently running (with docker ps) nothing shows to be running but when running docker ps --filter "status=exited" it shows the containers have already exited.
Is there a log file for errors to look to see what the issue is?

Yes, you can use docker to figure out why the container exited immediately. First, find the container with the docker ps -a command:
$ docker ps -a
CONTAINER ID        IMAGE               COMMAND                  CREATED             STATUS                      PORTS               NAMES
8d061c172934        inventory-hunter    "python /src/run.py …"   30 seconds ago      Exited (1) 29 seconds ago                       sad_maxwell

Then, pass the container name or id to to the docker logs command:
$ docker logs 8d061c172934
E2020-11-25 12:36:12,920 caught exception
Traceback (most recent call last):
  ...

You can also use a UI such as Portainer to view logs and manage containers.
Please let me know if you have any other questions.

That worked perfectly, thank you!


Support for more 3rd party RSpec DSL/matchers

[ ] https://github.com/thoughtbot/shoulda-matchers
[ ] https://github.com/rspec/rspec-mocks
[ ] https://github.com/rspec/rspec-rails
[ ] https://github.com/wspurgin/rspec-sidekiq
[ ] https://github.com/bblimke/webmock
[ ] https://github.com/brooklynDev/airborne
[ ] https://rubydoc.info/github/teamcapybara/capybara/master/Capybara/Node/Matchers

Other?

I wondered about some sort of FactoryBot support - eg trying to jump to the definition for create(:blog_post) would jump me to the factory :blog_post do.... declaration.
Or, with regular Rails fixtures, whether blog_posts(:introduction) might be able to jump to fixtures/blog_posts.yml .
(Both sound quite hard to me, not sure what's possible).

@jdelStrother thanks for bringing this up! As the topic of FactoryBot is slightly different from matcher utils, I created a separate issue, and will try to answer you there: https://github.com/lekemula/solargraph-rspec/issues/4

Most 3rd party libraries support (soon to be released) is added here: https://github.com/lekemula/solargraph-rspec/commit/07f1394a08e58e045895392f03f55023ddfb724a


Formal review process by adding review's template

Feature Description
In a heavily constrained development process, a feature to standardize review would be an amazing must have.
The idea is to offer the option, like the existing pull request template, to configure template for reviewers.
E.g. file : .gitea/default_review_message/REVIEW_TEMPLATE.md
The result of such template could be a combination of texts and checkboxes with timestamps and reviewer name.
In addition, the content of those reviews from the pull request could be put in a variable.
Then, to be used in merge message, the same way the pull request description can be copied in merge message.
E.g. ${ListOfReviews}
With this feature, review's content can be standardized and committed to git with the content/code reviewed.
That way, we can demonstrate that the reviewer committed on a given list of tasks to perform at review stage.
Screenshots
No response

If I understand your request correctly, isn't that effectively a concrete use of #22787?
If yes, let's close this issue in favor of the other.
Feel free to reopen if that is not the case.

I saw this feature request but does not seem to be the same for my point of view?
Imagine a repo for a project on which we have to provide the proof that a list of review tasks has been performed in the frame of a pull request.
Then we would ensure, by providing a template, that the text box for approval/rejection (submit a review) is used as a mean to let he reviewer commit on these tasks. This text would then be used in the frame of the merge commit message to store the proof.
Maybe this textbox (submit a review) is considered as "response" according to #22787 ? in that case yes it is a duplicate

I don't see why the "review textbox" shouldn't get the saved replies…
So yes, I guess this issue is then indeed a duplicate.


Add field to loader: last loaded model

It's strange to me that the loaded scene is discarded entirely if another game object is not given as its parent. This PR allows consumers to get the reference to the model after loading is complete.

I changed LastLoadedModel to LastLoadedScene. I think it better represents the GameObject.
LGTM 👍


mark tests that are failing at the moment

@baldersheim please review
@lesters we need to go through and fix/rewrite these tests, they have not been properly run for a while.

👍


Add info which process consuming memory to Apify.getMemoryInfo()

Function can return info about process only if we set some parameter to it. Because get info about all processes can be demanding task.

Is this still valid?

I don't think so, the function is not even public API anymore, no?

getMemoryInfo is public and part of crawlee (also in crawlee metapackage):
https://crawlee.dev/api/utils/function/getMemoryInfo

Ah, ok. Thought we would remove it and make it only a private helper of the AutoscaledPool. Maybe other time then 😄

I guess I will still close this, its more than 4 years old, so apparently it wasnt needed or requested by anybody :]


Crawler error

Hello there, I really like the idea of this cli-tool. However, i am getting this error when attempting to use it:
Running until canceled, check info.log for details...
Traceback (most recent call last):
  File "/usr/local/bin/wg-gesucht-crawler-cli", line 11, in <module>
    sys.exit(cli())
  File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/wg_gesucht/cli.py", line 84, in cli
    wg_gesucht.search()
  File "/usr/local/lib/python3.6/dist-packages/wg_gesucht/crawler.py", line 383, in search
    self.email_apartment(ad_url, template_text)
  File "/usr/local/lib/python3.6/dist-packages/wg_gesucht/crawler.py", line 316, in email_apartment
    ad_info = self.get_info_from_ad(url)
  File "/usr/local/lib/python3.6/dist-packages/wg_gesucht/crawler.py", line 267, in get_info_from_ad
    online_status = ad_submitter.find('span')
AttributeError: 'NoneType' object has no attribute 'find'
Stopped running!

I am guessing this happens because the site's html has changed

Dirty fix
Edit the crawler.py in /usr/local/lib/python3.6/dist-packages/wg_gesucht/crawler.py
On line 266 replace text-capitalise with panel-body
On line 267 replace ad_submitter.find('span') to ad_submitter.find('div', {'class': 'col-md-6','class':'text-right'})
On line 320 replace btn-orange with wgg_orange

I'm getting the same error, just with a different tag that's not found
"/Users/danijel/anaconda3/lib/python3.7/site-packages/wg_gesucht/crawler.py", line 240, in process_filter_results post_date_link = result.find("td", {"class": "ang_spalte_datum"}).find("a")
@Pipazoul how did you proceed in finding out which class it is, that was replaced and what the new tag is?

I've retested it, it still works maybe just try to replace the file crawler.py in your pip library path with this one
https://github.com/Pipazoul/wg-gesucht-crawler-cli/blob/master/wg_gesucht/crawler.py
To find the new classes i've searched the nearest class available in the html off a wg-gesucht ad
The crawler searches in the
panel panel-rhs-default rhs_contact_information hidden-sm class to get the posted date
And gets the url to send the message from the orange button btn btn-block btn-md wgg_orange

@moar55 Yeh they unfortunately change their site a lot, I've updated it recently, but it looks like you still have the older version, if you update with pip install --upgrade wg-gesucht-crawler-cli it should work
@sechsneun Do you have any Gesucht/Request filters saved on your profile? If you do the script will try search them and ang_spalte_datum tags won't be on the page (will be ges_spalte_datum).
try running the script with wg-gesucht-crawler-cli --filter-names="Name you gave the filter you saved"

@grantwilliams Understandable. Thank you for the tool neverthless :)


Make sources for sprite_download.png available, and/or add .tar.bz2

In my project, I don't offer .zip downloads, but rather .tar.gz and .tar.bz2. So I wanted to modify sprite_download.png accordingly. But its content is rather low-res and "fuzzy" (due to aliasing artifacts), and I don't know which font was used...
It would be nice if there was a way to do this cleanly, e.g. by having a high-res version of the folder symbol, without text, and info about the font, to compose them. Having a Photoshop/Gimp/whatever version of this with layers would also work...
Any chance you could make something like this available? Alternatively, just having a version with .tar.gz and .tar.bz2 instead of .tar.gz and .zip would help me. (Though hypothetically, other people might wish to use other formats, too).
Anyway, thank you for this nice them!

@fingolfin would having something in the releases section for a new downloadable copy that included high-res versions of the images be a good middle ground?
To get an idea of how these extra assets work, take a look at some of the downloads of the Atom Repo: https://github.com/atom/atom/releases/tag/v0.188.0. There's just extra items that you can attach to releases, and i think your high-res image would fit here!

Or I'll just replace these with SVGs tonight :grin:
Sorry to be a slackerrrr

This is no longer relevant in the new version of the layout.