Build failure: python3.pkgs.spacy

Steps To Reproduce
Steps to reproduce the behavior:
nix build 'nixpkgs#python3.pkgs.spacy'
Build log
Sourcing python-remove-tests-dir-hook
Sourcing python-catch-conflicts-hook.sh
Sourcing python-remove-bin-bytecode-hook.sh
Sourcing pypa-build-hook
Using pypaBuildPhase
Sourcing python-runtime-deps-check-hook
Using pythonRuntimeDepsCheckHook
Sourcing pypa-install-hook
Using pypaInstallPhase
Sourcing python-imports-check-hook.sh
Using pythonImportsCheckPhase
Sourcing python-namespaces-hook
Sourcing python-catch-conflicts-hook.sh
@nix { "action": "setPhase", "phase": "unpackPhase" }
Running phase: unpackPhase
unpacking source archive /nix/store/d9q39zwc63mjq9wxvdkhh2ixh3zgvpzs-spacy-3.7.2.tar.gz
source root is spacy-3.7.2
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file spacy-3.7.2/setup.cfg
@nix { "action": "setPhase", "phase": "patchPhase" }
Running phase: patchPhase
substituteStream(): WARNING: '--replace' is deprecated, use --replace-{fail,warn,quiet}. (file 'setup.cfg')
substituteStream(): WARNING: pattern thinc\>=8.1.8\,\<8.2.0 doesn't match anything in file 'setup.cfg'
@nix { "action": "setPhase", "phase": "updateAutotoolsGnuConfigScriptsPhase" }
Running phase: updateAutotoolsGnuConfigScriptsPhase
@nix { "action": "setPhase", "phase": "configurePhase" }
Running phase: configurePhase
no configure script, doing nothing
@nix { "action": "setPhase", "phase": "buildPhase" }
Running phase: buildPhase
Executing pypaBuildPhase
Creating a wheel...
* Getting build dependencies for wheel...
Copied /build/spacy-3.7.2/setup.cfg -> /build/spacy-3.7.2/spacy/tests/package
Copied /build/spacy-3.7.2/pyproject.toml -> /build/spacy-3.7.2/spacy/tests/package
Cythonizing sources
Compiling spacy/training/alignment_array.pyx because it changed.
Compiling spacy/training/example.pyx because it changed.
Compiling spacy/parts_of_speech.pyx because it changed.
Compiling spacy/strings.pyx because it changed.
Compiling spacy/lexeme.pyx because it changed.
Compiling spacy/vocab.pyx because it changed.
Compiling spacy/attrs.pyx because it changed.
Compiling spacy/kb/candidate.pyx because it changed.
Compiling spacy/kb/kb.pyx because it changed.
Compiling spacy/kb/kb_in_memory.pyx because it changed.
Compiling spacy/ml/parser_model.pyx because it changed.
Compiling spacy/morphology.pyx because it changed.
Compiling spacy/pipeline/dep_parser.pyx because it changed.
Compiling spacy/pipeline/_edit_tree_internals/edit_trees.pyx because it changed.
Compiling spacy/pipeline/morphologizer.pyx because it changed.
Compiling spacy/pipeline/multitask.pyx because it changed.
Compiling spacy/pipeline/ner.pyx because it changed.
Compiling spacy/pipeline/pipe.pyx because it changed.
Compiling spacy/pipeline/trainable_pipe.pyx because it changed.
Compiling spacy/pipeline/sentencizer.pyx because it changed.
Compiling spacy/pipeline/senter.pyx because it changed.
Compiling spacy/pipeline/tagger.pyx because it changed.
Compiling spacy/pipeline/transition_parser.pyx because it changed.
Compiling spacy/pipeline/_parser_internals/arc_eager.pyx because it changed.
Compiling spacy/pipeline/_parser_internals/ner.pyx because it changed.
Compiling spacy/pipeline/_parser_internals/nonproj.pyx because it changed.
Compiling spacy/pipeline/_parser_internals/_state.pyx because it changed.
Compiling spacy/pipeline/_parser_internals/stateclass.pyx because it changed.
Compiling spacy/pipeline/_parser_internals/transition_system.pyx because it changed.
Compiling spacy/pipeline/_parser_internals/_beam_utils.pyx because it changed.
Compiling spacy/tokenizer.pyx because it changed.
Compiling spacy/training/align.pyx because it changed.
Compiling spacy/training/gold_io.pyx because it changed.
Compiling spacy/tokens/doc.pyx because it changed.
Compiling spacy/tokens/span.pyx because it changed.
Compiling spacy/tokens/token.pyx because it changed.
Compiling spacy/tokens/span_group.pyx because it changed.
Compiling spacy/tokens/graph.pyx because it changed.
Compiling spacy/tokens/morphanalysis.pyx because it changed.
Compiling spacy/tokens/_retokenize.pyx because it changed.
Compiling spacy/matcher/matcher.pyx because it changed.
Compiling spacy/matcher/phrasematcher.pyx because it changed.
Compiling spacy/matcher/dependencymatcher.pyx because it changed.
Compiling spacy/symbols.pyx because it changed.
Compiling spacy/vectors.pyx because it changed.
[ 1/45] Cythonizing spacy/attrs.pyx
[ 2/45] Cythonizing spacy/kb/candidate.pyx
[ 3/45] Cythonizing spacy/kb/kb.pyx
[ 4/45] Cythonizing spacy/kb/kb_in_memory.pyx
[ 5/45] Cythonizing spacy/lexeme.pyx
[ 6/45] Cythonizing spacy/matcher/dependencymatcher.pyx
[ 7/45] Cythonizing spacy/matcher/matcher.pyx
[ 8/45] Cythonizing spacy/matcher/phrasematcher.pyx

Error compiling Cython file:
------------------------------------------------------------
...
        self._validate = validate

        self.mem = Pool()
        self.c_map = <MapStruct*>self.mem.alloc(1, sizeof(MapStruct))
        self._terminal_hash =<PHONE_NUMBER>22620965
        map_init(self.mem, self.c_map, 8)
               ^
------------------------------------------------------------

spacy/matcher/phrasematcher.pyx:48:16: Call with wrong number of arguments (expected 2, got 3)

Error compiling Cython file:
------------------------------------------------------------
...
                    warnings.warn(Warnings.W021)
                    break
                result = <MapStruct*>map_get(current_node, token)
                if not result:
                    internal_node = <MapStruct*>self.mem.alloc(1, sizeof(MapStruct))
                    map_init(self.mem, internal_node, 8)
                           ^
------------------------------------------------------------

spacy/matcher/phrasematcher.pyx:224:28: Call with wrong number of arguments (expected 2, got 3)

Error compiling Cython file:
------------------------------------------------------------
...
                    break
                result = <MapStruct*>map_get(current_node, token)
                if not result:
                    internal_node = <MapStruct*>self.mem.alloc(1, sizeof(MapStruct))
                    map_init(self.mem, internal_node, 8)
                    map_set(self.mem, current_node, token, internal_node)
                          ^
------------------------------------------------------------

spacy/matcher/phrasematcher.pyx:225:27: Call with wrong number of arguments (expected 3, got 4)

Error compiling Cython file:
------------------------------------------------------------
...
                    result = internal_node
                current_node = <MapStruct*>result
            result = <MapStruct*>map_get(current_node, self._terminal_hash)
            if not result:
                internal_node = <MapStruct*>self.mem.alloc(1, sizeof(MapStruct))
                map_init(self.mem, internal_node, 8)
                       ^
------------------------------------------------------------

spacy/matcher/phrasematcher.pyx:231:24: Call with wrong number of arguments (expected 2, got 3)

Error compiling Cython file:
------------------------------------------------------------
...
                current_node = <MapStruct*>result
            result = <MapStruct*>map_get(current_node, self._terminal_hash)
            if not result:
                internal_node = <MapStruct*>self.mem.alloc(1, sizeof(MapStruct))
                map_init(self.mem, internal_node, 8)
                map_set(self.mem, current_node, self._terminal_hash, internal_node)
                      ^
------------------------------------------------------------

spacy/matcher/phrasematcher.pyx:232:23: Call with wrong number of arguments (expected 3, got 4)

Error compiling Cython file:
------------------------------------------------------------
...
            if not result:
                internal_node = <MapStruct*>self.mem.alloc(1, sizeof(MapStruct))
                map_init(self.mem, internal_node, 8)
                map_set(self.mem, current_node, self._terminal_hash, internal_node)
                result = internal_node
            map_set(self.mem, <MapStruct*>result, self.vocab.strings[key], NULL)
                  ^
------------------------------------------------------------

spacy/matcher/phrasematcher.pyx:234:19: Call with wrong number of arguments (expected 3, got 4)
Traceback (most recent call last):
  File "/nix/store/naw86kmy92335k30fv11cfyydxbicva8-python3.11-pyproject-hooks-1.0.0/lib/python3.11/site-packages/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
    main()
  File "/nix/store/naw86kmy92335k30fv11cfyydxbicva8-python3.11-pyproject-hooks-1.0.0/lib/python3.11/site-packages/pyproject_hooks/_in_process/_in_process.py", line 335, in main
    json_out['return_val'] = hook(**hook_input['kwargs'])
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/naw86kmy92335k30fv11cfyydxbicva8-python3.11-pyproject-hooks-1.0.0/lib/python3.11/site-packages/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
    return hook(config_settings)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/dyl7f7v3cfi7j9d4z42pn2ss0bgw0r33-python3.11-setuptools-69.0.2/lib/python3.11/site-packages/setuptools/build_meta.py", line 325, in get_requires_for_build_wheel
    return self._get_build_requires(config_settings, requirements=['wheel'])
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/nix/store/dyl7f7v3cfi7j9d4z42pn2ss0bgw0r33-python3.11-setuptools-69.0.2/lib/python3.11/site-packages/setuptools/build_meta.py", line 295, in _get_build_requires
    self.run_setup()
  File "/nix/store/dyl7f7v3cfi7j9d4z42pn2ss0bgw0r33-python3.11-setuptools-69.0.2/lib/python3.11/site-packages/setuptools/build_meta.py", line 311, in run_setup
    exec(code, locals())
  File "<string>", line 221, in <module>
  File "<string>", line 208, in setup_package
  File "/nix/store/swcsynzqd5d82yzmykc94wc1gsz4z6bv-python3.11-cython-0.29.36/lib/python3.11/site-packages/Cython/Build/Dependencies.py", line 1115, in cythonize
    cythonize_one(*args)
  File "/nix/store/swcsynzqd5d82yzmykc94wc1gsz4z6bv-python3.11-cython-0.29.36/lib/python3.11/site-packages/Cython/Build/Dependencies.py", line 1238, in cythonize_one
    raise CompileError(None, pyx_file)
Cython.Compiler.Errors.CompileError: spacy/matcher/phrasematcher.pyx

ERROR Backend subprocess exited when trying to invoke get_requires_for_build_wheel

Additional context
The error is caused by dependency preshed which is  at the unsupported version 4.0.0,  but version 4.0.0 of that package has been yanked, see #284758 . With previous version 3.0.9 it builds flawlessly.
Notify maintainers
@GaetanLepage
@mweinelt
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.7.1, NixOS, 24.05 (Uakari), 24.05.20240125.ae5c332`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.19.3`
 - channels(root): `"nixos"`
 - channels(azazel): `"home-manager, old"`
 - nixpkgs: `/nix/store/b9c11c60n2svicqc6zhk5wdngzym8daq-9s5qs4hni9fj88x79iw6im7amv7ghb76-source`

Add a :+1: reaction to issues you find important.

I don't understand why this tries to compile when the package lists a dependency for preshed like "preshed>=3.0.2,<3.1.0", shouldn't the build stop earlier because the requirements with preshed 4.0.0 aren't met?

The dependencies are checked on the resulting wheel, because that provides the metadata in a unified way.
EIther way: spacy builds again after reverting preshed.


Timed Assignment based on Fetch

I was just wondering if there is any scope currently for adding an additional function to the Timed assignments where students may have 24 hours perhaps after fetching an assignment to complete it.
This would allow for staggered start times within a certain window. We have this use case where students have access to assignments for a week but then only have 24 hours to submit it.

hi @milandeepbassi,
Thanks for your request - we're in the middle of a fairly major upgrade at the moment, we're working through a release of version "0.2.*" for the grader-service, grader-convert and grader-labextension packages. Our efforts will be going into bugfix releases for the near term.
We've talked a bit about a feature that would enable a sort of automatic upload of assignments after the deadline expires, but have not thought much about a timer started by the students as you are suggesting - it's kind of a nested deadline if I understand you correctly.


Feature request: table view editor

I would like a table component for plugin development. My use case is a PostgreSQL client plugin for Lapce, but other may be interested in making CSV, Excel, or any tabular data reader/editor plugins. I can even think of build-in SSH tunneling for database connections.
I would begin by creating a table component in lapce-ui. Like the current editor, we can scroll horizontally and vertically. By selecting a cell, we can edit the content or apply actions. It could be place in the main part as a tab or in bottom panel.
My inspiration would be Postico2 table from when I was on MacOS, now on Sway Java database clients like Dbeaver are frustrating.
The table would be something like that:

And bottom panel:

Before coding for this feature, I wanted to discuss it here.

Here's some random design questions to maybe think about:
How to handle large tables
Some tables, like massive csv files or just large requests to a database, can be one or both of (1) painfully large in memory; (2) huge in the number of widgets stored and attempted to layout with, which if done naively could slow down things.
So, it might be beneficial to make so the table has some methods for knowing when it should provide more information. This could be done by sending a Command to the parent widget, telling it to update the table it holds. (Since in normal Lapce UI code, you'd usually have some parent widget which contains the table and handles what should be in the table)
Though, I'm not sure of a good general way to do that. Especially once filtering and sorting is done.
A nice producer method isn't essential (especially for an initial implementation, since most tables will easily be stored in memory, but I thought I'd bring it up since it would be useful in the future.
It would still be good to try only drawing what is within the view (to avoid doing the logic for painting a 1000 table entries that are outside the view).
Are they editable in the table?
Allowing users to double-click to edit the entries in the table would be nice, but not necessarily essential (with a sidebar like in the first image, you can simplify that). It also isn't trivial to implement.
Are the table entries arbitrary widgets
Allowing arbitrary widgets can complicate the code, but also allows it to express more.
What List does is be generic over an element type and have the element types provide limited drawing functions via a custom trait ListPaint (which also makes it easier to provide parameters specific to the list), without the full widget system.
(However, for plugins, they'd have some standard type for entries, instead of being more generic since plugins can't really nicely paint things.)
Good test case
A good test-case for the implementation would be to make keymaps (the keybinding table/menu in the settings; the code in there is rough) use the table component.
Plugin UI
I'll note that currently we don't have any decided upon way of adding UI via plugins, though it is planned. So even if this is finished, it may be a bit before we decide on the right way of allowing plugins to create custom UI panels and editors (and then more time to actually implement it).

How to handle large tables

Generally, the SQL clients solve this problem with dynamic pagination of the steamed data on a lived query (or bulk of lines for CSV), this should be a problem for the plugin part like sorting or filtering the table. The table should handle resizing the column and calling for new data till the end message. The hardest part look like the communication between the plugin and the table with windowed data, even more if the message need to pass by the parent widget.

Are they editable in the table?

I were going for an inline editable main table and non-editable bottom table. But a sidebar form widget is better because it can be reuse and reduce the size of code only for a table.

Are the table entries arbitrary widgets

Textual or numeric data should the enough for most used cases, so I wanted to included a generic cell widget in Lapce with optional actions and an icon (like opening an link in browser, an image in image viewer or a foreign key in bottom panel).

Good test case
A good test-case for the implementation would be to make keymaps (the keybinding table/menu in the settings; the code in there is rough) use the table component.

Thanks, I didn't think of that first step, maybe I can begin with refactoring the keymaps to extract an improved generic table widget with a sidebar form widget for edition.

Plugin UI
I'll note that currently we don't have any decided upon way of adding UI via plugins, though it is planned. So even if this is finished, it may be a bit before we decide on the right way of allowing plugins to create custom UI panels and editors (and then more time to actually implement it).

Is it okay to make a table widget included in Lapce when most options would be used for making plugin ?
The advantages would be opening a wide range of plugins using the same UI component, easier to develop and with the same designs and ecosystem felling. The disadvantages are obvious between not being the objective of a code editor, having more code to maintain used for plugin only and the RPC communication of table windowed data.
So, I have two courses of action, begin a refactoring of the keymaps  to make a proof of concept and give an idea of the code complexity or I wait for plugin UI.


Update Helm to address CVE-2023-25165

Relates:
https://issues.redhat.com/browse/ACM-3721

I'll investigate the compile failure

This may be fixed by #73 instead.

Closing in favor of https://github.com/open-cluster-management-io/governance-policy-addon-controller/pull/73.

i was waiting for test passed


Removes typescript-esbuild

typescript-esbuild was used in earlier versions of astro-compress, but you don't need it now. It's bundled internally and only used in development as a devDependency. Using it as a dependency in package.json would only create overhead and install all of these dependencies:
"dependencies": {
	"@types/node": "20.12.2",
	"commander": "12.0.0",
	"deepmerge-ts": "5.1.0",
	"esbuild": "0.20.2",
	"esbuild-plugin-copy": "2.1.1",
	"fast-glob": "3.3.2",
	"typescript": "5.4.3"
},
"devDependencies": {
	"ts-node": "10.9.2"
},
"peerDependencies": {
	"@playform/document": "0.0.2"
},

Hi @NikolaRHristov
We really don't know why that dependency is there and even less why it is in dependencies and not in devDependencies. Sounds like a guy or some approved PR who let that happen.
Thank you


Add support for '24 hour days'

BigCalendar considers allDay = endDate - startDate > 1
Drawbacks:
24hours events aren't considered (e.g. 1st 00:00 to 2nd 00:00). It would have to be at least 24hours + 1 min
Any events that spans 2+ days are considered (e.g. 1st 11:59PM to 2nd 00:01AM is a 2-day allDay event, that only lasts 2 min)
Whereas gcalendar considers allDay >= 24 hours, this is what I'm proposing to support.
I can think of 2 ways to implement it:

native support with a '24hourDay' calendarProp
Improving allDayAccessor function (currently although you can define what an allDay is, it won't be considered when calculating the cells span value)

I'll present a PR with some progress, looking forward to your comments.

@jquense may you take a look at this. It was already left behind. The refactor is a bugfix, the behaviour is a feature.


Add a StyleGuideline for Params block format

At the moment there is a lot of inconsistencies with the params block formats.
E.g.

Option 1:

param (
  [string] $Name
)

Option 2:

param (
  [string]
  $Name
)

Option 1 looks to be the most common and is also a seems the most logical as declaring variables is usually done with:
[string] $Name

Param decorations should always be on their own lines.
@TysonJHayes, @IainBrighton, @TravisEz13?

Hello all,
As a contributor, I'd prefer Option 2 because it's arguably more readable in the following cases:

There are other attributes (e.g., [Parameter()], [ValidateNotNullOrEmpty()] etc.).
There are a lot of parameters with different types.
There are parameters with long type names (e.g., [System.Security.AccessControl.FileSystemSecurity]).

The main advantage of Option 2 is that parameter names always start at the very same position making the code easier to follow.
Please see the examples below:

Option 1. The $Path parameter name starts at column 18 while the $Acl parameter name starts at column 34.

    param
    (
        [Parameter(Mandatory = $true)]
        [ValidateScript({Test-Path -Path $_})]
        [String] $Path,

        [Parameter(Mandatory = $true)]
        [ValidateNotNullOrEmpty()]
        [System.Security.AccessControl.FileSystemSecurity] $Acl
    )

vs.

Option 2. Both the $Path and the $Acl parameter names start at column 9.

    param
    (
        [Parameter(Mandatory = $true)]
        [ValidateScript({Test-Path -Path $_})]
        [String]
        $Path,

        [Parameter(Mandatory = $true)]
        [ValidateNotNullOrEmpty()]
        [System.Security.AccessControl.FileSystemSecurity]
        $Acl
    )

Personally I vote for Option 1, it's the most common, and most logical.

I vote for Option 1 too as it's closer to what I tend to use. This is typical of what I have settled on:
param
(
    [Parameter(Mandatory)] [ValidateScript({Test-Path -Path $_})]
    [String] $Path,

    [Parameter(Mandatory)] [ValidateNotNullOrEmpty()]
    [System.Security.AccessControl.FileSystemSecurity] $Acl
)

I don't bother with Mandatory = $true as DSC only runs on v4+

To be honest, this detail of style guideline feels to rigid to me. I mean, heck,  I don't do either of these options,  instead putting a parameter per line which, for me, is the easiest to read:
param (     [Parameter(Mandatory=$true)][String][ValidateSet("Root","Issue")]$Role,        [Parameter(Mandatory=$true)][String]$CAName,     [Parameter(Mandatory=$false)][String]$DomainDN,     [Parameter(Mandatory=$false)][String]$FQDN = "pki.contoso.com"     )
For me it is the fastest way to see "ok I have 4 parameters, 2 are mandatory" as well as making it a little more obvious and clear where that trailing coma goes ;).  My point isn't that my method "is best", rather that the param block is inherently flexible and I don't see why we should be enforcing a specific format for this part, especially for such a small part of the codebase.  I think should stay more general like "when to indent" and "back ticks are evil, just stop that".

I'm happy with either way, as long as it is consistent. I normally code using option 1, but for DSC I used option 2 because that is what xDSCResourceDesigner give me, so I stuck with that as I thought that was the guideline.
And as a side not, is there a guideline when to use or not to use default parameters.

My preference is option 1 as well, but agree with @johlju that either way is fine as long as it's consistent across all resources

Is the whitespace required between [Type] $Name in Option 1?

On a side note - in the guideline for this we should have example with not only type, but also other attributes (as pointed by @SNikalaichyk )

@KarolKaczmarek , I agree, there does need to be guidance on the attributes.  @johlju raises a good point though -if xDSCResourceDesigner outputs parameters with Option 2 formatting then either xDSCResourceDesigner should be changed to output option 1 format or we go with Option 2. Seems like a bad idea to have a style guideline that goes against xDSCResourceDesigner?

@TravisEz13 - care to weigh in on this one?
Perhaps in the absence of a clear decision on which format is best we go with the "Make it match the format of any existing resources in the module?"

I voted with @KarolKaczmarek ' agreement  "that either way is fine as long as it's consistent across all resources"

It seems most people suggested option 1, we just need to file an issue and fix xDSCResourceDesigner to generate proper formatting.

Ok, cool. Thanks for that. I'll get the changes under way (including raising an issue on xDscREsourceDesigner).

Fixed with https://github.com/PowerShell/DscResources/pull/126
Thanks!


Headers from a redirected request are not displayed in the response

I have an endpoint which searches for AWS s3 bucket objects and, if present, returns a redirect to a pre-signed url for the object. My problem is that the response headers of such redirect are not displayed in the response container of RapiDoc. This is particularly annoying, because the Content-Disposition header is ignored. RapiDoc correctly inrefers the response body and prompts a download button, but the filename, which is set in the header, is being ignored.
The headers displayed:

The headers of the redirect:

There are restriction to access response headers using scripts running on the browser. You may see them in Browsers -Developer Tools but can access them through scripts unless you allow them specifically on your server through access-control-expose-headers
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Access-Control-Expose-Headers

my bad, thank you very much :)


Feature/openhelpd 100 faq review changes languages

This PR fixes EPIC-OPENHELPD-15-PR and implements most of Rowan's feedback.
The parts of Rowan's feedback not implement are:

using a HSTORE field  for FAQ translations instead of the parent-child relationship.
using smartmin's built in CSV import

The reasons are:

For each FAQ we need to store three things: the translated question, the translated answer and the language code. Using an HSTORE would mean having to use the same parent question for each translation and that is not ideal.
Smartmin's CSV import expects the CSV file to be one row per item and for the columns to match the field names of the model. WCL's CSV file puts all the translations on the same row of the CSV file as the parent FAQ. I am deferring to Mike's knowledge of the problem space here and he said this is a good format they have shown to work well in practice.

For ease of reviewing this PR I have based it against develop rather than the ancient EPIC-OPENHELPD-15-PR branch. However once it is reviewed and everyone is happy I will rebase it onto of EPIC-OPENHELPD-15-PR and that is where it will be merged to as that is what the upstream PR is based on.

@rudigiesler @smn @KaitCrawford @miltontony Ready for review.

I'm about as happy as I can be with this PR so :+1: from me. The size of it (and how old it is) makes reviewing more challenging than it should be and so I wouldn't be surprised if more, smaller PRs, fixes will come out of this one getting merged in.


Is there an example of rendering the form with initial values

There are some issues which seem to indicate that there is a demo which exemplifies how I can use autoform with initial values. Can someone please provide a link here, can't find it

Found it in the react-autoform-bootstarp-ui example. Need to pass another prop- initialValues


Update jnifuse dynamic lib

Update jnifuse library files

Codecov Report

:exclamation: No coverage uploaded for pull request base (branch-2.5@787b898). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             branch-2.5   #13060   +/-   ##
=============================================
  Coverage              ?   16.81%
  Complexity            ?     2543
=============================================
  Files                 ?     1343
  Lines                 ?    77494
  Branches              ?     9449
=============================================
  Hits                  ?    13032
  Misses                ?    63303
  Partials              ?     1159

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 787b898...be4d5f4. Read the comment docs.

alluxio-bot, merge this please


Infinite rendering loop

Output from ember bootstrap:info (available since 1.0.0-alpha.9):
Npm packages:
ember-bootstrap: ^1.0.0-alpha.9 -> 1.0.0-alpha.9
ember-cli: 2.12.1 -> 2.12.1
bootstrap: ^3.3.7 -> 3.3.7
bootstrap-sass: n/a
ember-cli-sass: n/a
ember-cli-less: n/a
Bower packages:
bootstrap: n/a
bootstrap-sass: n/a
ember-bootstrap configuration:
bootstrapVersion: 3
importBootstrapFont: true
importBootstrapCSS: true

Tried to use this library and stumbled upon following problem:

Basically main application template is rendered infinite number of times under certain conditions. I tried to strip down the sample app to bare minimum. Curious thing is that there is nothing really except this template: https://github.com/MichalBryxi/cryptopals.com/blob/ember-2.12-LTS/app/application/template.hbs

Tested with 2.12 (future LTS).
The problem seems to be gone with 2.13.
I'm absolutely not sure whether the problem might be in this addon or Ember itself. Trying to poke the "lower instance" with this.

We've seen this before and found it to be due to Ember and glimmer error handling when there is an error on the page. I don't have the reference handy since I'm on my phone. In this case, I would guess you don't have a defined whatever route. Create the route or reference an existing route and I'd bet it will behave.

Thanks @srvance, yes I can confirm that whatever route does not exist. It's mentioned in the source code & attached screenshot. It's very specific setup. Not sure how many cases can cause this error. My feeling is that this is Ember/Glimmer level problem, right? It seems odd that addon would cause yielding parent template into DOM multiple times.
Just want to verify before reporting to Ember/Glimmer.

The addon gets rendered multiple times because it's correct. The render restart happens at the point of the error.
Yes, this is an Ember/glimmer problem that a core contributor characterized as "halting problem hard." There's another e-b issue that has pointers to those issues IIRC.


X509_digest error on updating to 1.1

Bug Report
I am getting this new error after having moved to the new 1.1 version
Since it doesn't reference any of my code, I am assuming this is just broken dependencies.
rishav@rishav-sbook:/mnt/c/Users/risharan/Documents/GitHub/digglu$ crystal run src/digglu.cr
Showing last frame. Use --error-trace for full trace.

In lib/openssl_ext/src/openssl_ext/lib_crypto.cr:374:3

 374 | fun x509_digest = X509_digest(x509 : X509, type : EVP_MD, md : UInt8*, len : UInt32*) : Int32
       ^----------
Error: fun redefinition with different signature (was `fun x509_digest = X509_digest(x509 : X509, evp_md : EVP_MD, hash : ::Pointer(UInt8), len : ::Pointer(Int32)) : Int32` at /usr/share/crystal/src/openssl/lib_crypto.cr:171:3)

nevermind.  I forgot to update the shards. 😿


Add check to verify auth token

This pull request adds a requests import and a small check in the base class' init function to verify the token's authencity.

I don't see the need to validate the token in client initialization, as an invalid token should raise when making a request anyway. Additionally, using requests will block the event loop, even though this is meant to be an asynchronous API wrapper.


hung mkcloud does not collect supportconfigs

e.g. https://ci.suse.de/job/openstack-mkcloud/12920/
mkcloud should have its own timeout handler which ensures that supportconfigs are always collected if the build takes too long.

jenkins mkcloud does have its own timeout of 4h, but there is an additional timeout configured in jenkins that watches stalls in output... and it would be hard to replicate that.
Maybe we could setup an alarm timer for things that could timeout and use a trap ALRM to handle it.

is this still an issue? if so, a pointer to a more recent failure would be nice.

I think I saw it a few weeks ago but I can't remember the failure.  Why would #740 help?  Isn't that only a UI change?

It is explained in the commit message. Jenkins kills runs when it sees no newline for 1h, which means no supportconfig is collected in that case.

Oh cool! Sorry I missed that detail.  Yeah maybe that will fix it.

We can reopen if we see this happen again.


Avatar imagen when sharing author profiles

Feature Request
Is your feature request related to a problem? Please describe.
Right now, when sharing the page of an author profile (/authors/author-name) the avatar image is not shown as the sharing image, but rather the standard one defined in the sharing_image field in our params.tomlfile.
Describe the solution you'd like
Make it so the avatar image is shown instead when sharing that link.

Relevant part of the code that needs to be modified: https://github.com/wowchemy/wowchemy-hugo-modules/blob/49515d9e144b446a7011fc16724b2f78ba6cd333/wowchemy/layouts/partials/site_head.html#L178
There are already some lines which deal with this:
{{ if (and (eq .Kind "taxonomy") $avatar_image) }}
    {{ $og_image = ($avatar_image.Fill "270x270 Center").Permalink }}{{/* Match image proc in About widget. */}}

Thanks for spotting this.
As you noticed, we already had this feature implemented and it was working fine in previous Hugo versions.
It turns out that Hugo team made a breaking change in recent Hugo versions to rename the page .Kind to term instead of taxonomy so the statement was no longer triggered.


BigQuery exporter: multiple SLO but one row

I've 3 SLOs (one per continent Asia/Europe/America), who share the same config
Only these 4 values change

metadata.name
metadata.labels['location']
spec.service_level_indicator.filter_good
spec.service_level_indicator.filter_valid

apiVersion: sre.google.com/v2
kind: ServiceLevelObjective
metadata:
  name: entrypoint-ingestion-availability-asia
  labels:
    slo_name: availability
    service_name: entrypoint
    feature_name: ingestion
    scope: internal
    source: load-balancer
    location: asia-east2
    provider: gcp
spec:
  goal: 0.999
  description: xxx
  backend: cloud_monitoring/sre
  error_budget_policy: default
  exporters:
  - bigquery/sre

  method: good_bad_ratio
  service_level_indicator:
    filter_good: >
      project = "XXX"
      metric.type = "loadbalancing.googleapis.com/https/request_count"
      resource.type = "https_lb_rule"
      resource.label.backend_target_name = "entrypoint-default"
      metric.label.proxy_continent = "Asia"
      metric.label.response_code_class >= 200
      metric.label.response_code_class <= 400

    filter_valid: >
      project = "XXX"
      metric.type = "loadbalancing.googleapis.com/https/request_count"
      resource.type = "https_lb_rule"
      resource.label.backend_target_name = "entrypoint-default"
      metric.label.proxy_continent = "Asia"

When I run slo-generator compute -c config -f slo/ --export only one row is written into BigQuery, but if I run each file independently, all the rows is correctly write into BigQuery

I'd like to look at your structure of the slo/ folder: do you have subfolders in there which could explain this ?

So you correctly load and compute SLO report for those 3 SLOs, but it exports only one of them ? Could you please paste the run logs here ?

I've run slo-generator with the same configuration as before, and get a weird behavior
When I look at the dataset from the console via the preview tab I see all the record, but when I run SELECT * <dataset>.<table> I got only one record, I guess I've an issue with BigQuery not slo-generator, I'll open a case
But here the log if you want https://gist.github.com/Eraac/864f55ae9bdd0d7dd22ae3f75ca3aee5

@ocervell I've found the issue and bigquery is out of cause
slo-generator generate the same rows_ids for my case, which means BigQuery while de-deduplicate them
https://github.com/google/slo-generator/blob/5f1f1f114d65fdd5a1aebcd4668607e5b617cd3b/slo_generator/exporters/bigquery.py#L72
https://github.com/google/slo-generator/blob/5f1f1f114d65fdd5a1aebcd4668607e5b617cd3b/slo_generator/exporters/bigquery.py#L92

Ah, I see the row_ids is not unique. We can set it to None to automatically generate an id, I will make a PR to fix it.

Will be fixed by https://github.com/google/slo-generator/pull/200


Correct missing fontawesome-webfont in Firefox (under Archlinux)

Correction is just hardcoded fontawesome-webfont urls in bokeh.css or icons don't appear on Firefox.
So not so nice but at least it works.

\o/


Is Aeron working if I use aeron::ipc commnuncation for two processes

Dear all,
I am trying to setup IPC communication for two processes on my M1 MacBook,
First I break this example of two agents(https://aeroncookbook.com/aeron-cookbook/ipc/) into two individual applications. Unfortunately, they are unable to exchange with each other. Am I understanding correctly about IPC?
Sender
`package org.example;
import io.aeron.Aeron;
import io.aeron.Publication;
import io.aeron.driver.MediaDriver;
import io.aeron.driver.ThreadingMode;
import org.agrona.concurrent.AgentRunner;
import org.agrona.concurrent.BusySpinIdleStrategy;
import org.agrona.concurrent.IdleStrategy;
import org.agrona.concurrent.ShutdownSignalBarrier;
public class Main {
public static void main(String[] args) {
final String channel = "aeron:ipc";
final int stream = 10;
final int sendCount = 1_000_000;
    final IdleStrategy idleStrategySend = new BusySpinIdleStrategy();
    final ShutdownSignalBarrier barrier = new ShutdownSignalBarrier();

    //construct Media Driver, cleaning up media driver folder on start/stop
    final MediaDriver.Context mediaDriverCtx = new MediaDriver.Context()
            .dirDeleteOnStart(true)
            .threadingMode(ThreadingMode.SHARED)
            .sharedIdleStrategy(new BusySpinIdleStrategy())
            .dirDeleteOnShutdown(true);

    final MediaDriver mediaDriver = MediaDriver.launchEmbedded(mediaDriverCtx);

    //construct Aeron, pointing at the media driver's folder
    final Aeron.Context aeronCtx = new Aeron.Context()
            .aeronDirectoryName(mediaDriver.aeronDirectoryName());
    final Aeron aeron = Aeron.connect(aeronCtx);

    //construct the subs and pubs
    final Publication publication = aeron.addPublication(channel, stream);

    //construct the agents
    final SendAgent sendAgent = new SendAgent(publication, sendCount);

    //construct agent runners
    final AgentRunner sendAgentRunner = new AgentRunner(idleStrategySend,
            Throwable::printStackTrace, null, sendAgent);

    System.out.println("Starting SendingAgent...");

    //start the runners
    AgentRunner.startOnThread(sendAgentRunner);

    //wait for the final item to be received before closing
    barrier.await();

    //close the resources
    sendAgentRunner.close();
    aeron.close();
    mediaDriver.close();
}

}
`
Receiver
`package org.example;
import io.aeron.Aeron;
import io.aeron.Subscription;
import io.aeron.driver.MediaDriver;
import io.aeron.driver.ThreadingMode;
import org.agrona.concurrent.AgentRunner;
import org.agrona.concurrent.BusySpinIdleStrategy;
import org.agrona.concurrent.IdleStrategy;
import org.agrona.concurrent.ShutdownSignalBarrier;
public class Main {
public static void main(String[] args) {
final String channel = "aeron:ipc";
final int stream = 10;
final int sendCount = 1_000_000;
    final IdleStrategy idleStrategySend = new BusySpinIdleStrategy();
    final IdleStrategy idleStrategyReceive = new BusySpinIdleStrategy();
    final ShutdownSignalBarrier barrier = new ShutdownSignalBarrier();

    //construct Media Driver, cleaning up media driver folder on start/stop
    final MediaDriver.Context mediaDriverCtx = new MediaDriver.Context()
            .dirDeleteOnStart(true)
            .threadingMode(ThreadingMode.SHARED)
            .sharedIdleStrategy(new BusySpinIdleStrategy())
            .dirDeleteOnShutdown(true);
    final MediaDriver mediaDriver = MediaDriver.launchEmbedded(mediaDriverCtx);

    //construct Aeron, pointing at the media driver's folder
    final Aeron.Context aeronCtx = new Aeron.Context()
            .aeronDirectoryName(mediaDriver.aeronDirectoryName());
    final Aeron aeron = Aeron.connect(aeronCtx);

    //construct the subs and pubs
    final Subscription subscription = aeron.addSubscription(channel, stream);

    //construct the agents
    final ReceiveAgent receiveAgent = new ReceiveAgent(subscription, barrier,
            sendCount);

    //construct agent runners
    final AgentRunner receiveAgentRunner = new AgentRunner(idleStrategyReceive,
            Throwable::printStackTrace, null, receiveAgent);

    System.out.println("Starting Receive...");

    //start the runners
    AgentRunner.startOnThread(receiveAgentRunner);

    //wait for the final item to be received before closing
    barrier.await();

    //close the resources
    receiveAgentRunner.close();
    aeron.close();
    mediaDriver.close();
}

}
`

IPC requires clients that communicate to use the same media driver. Either launch the media driver in only one of the processes, or separately in a 3rd process, and have all the clients use a common aeron direction to find the media then they can communicate.

Thank you very much @mjpt777.


[TFS + VSMac] Version Control Plugin <IP_ADDRESS> error

The latest Update <IP_ADDRESS> shows dependency error while installing. See the attached error image

@Abhijit-Revamp Thanks for reporting this error, however this is not the correct avenue to do that. Please report it over at Developer Community.


Migrate to Swift 5

This PR upgrades to Swift 5 and sets recommended Xcode 10.2 settings.
It would also be nice to get a new release. The reason why I started this PR was that it failed to build on Xcode 10.2 with Carthage since 2.0.1 has SWIFT_VERSION=3.0.
This error will probably hit quite a few users of this library using carthage like this:
github "madebybowtie/FlagKit" "~> 2.0"

Tests seem to fail because of the size of the generated image and the fixture differs. Not really sure how to proceed here. The generated images seem smaller so I guess it have been optimised in the latest SDK. Maybe updating the fixture images is enough?

Thanks, I will have a look at this right away!

@blommegard cool! I've updated the fixtures so the tests should pass now.

Forgot about Apple TV using @1x assets. Updated them as well and now the tests are passing.

Thanks!
I will merge this, do some additional migrations and then try to get a new release out.


Disassembly and Decompiler does not share equates

Describe the bug
On the same code example I used in #304, I found that if I define some value in the disassembler, the decompiler will not use the equate value
To Reproduce
                             void __cdecl16near _display_control(uint flags)
                               assume DS = 0xec25
             void              <VOID>         <RETURN>
             uint              Stack[0x2]:2   flags
                             _display_control
       e000:a672 55              PUSH       BP
       e000:a673 8b ec           MOV        BP,SP
       e000:a675 8b 5e 04        MOV        BX,word ptr [BP + flags]
       e000:a678 b4 00           MOV        AH,DISPLAY_CONTROL
       e000:a67a cd 12           INT        INT_DISP
       e000:a67c 5d              POP        BP
       e000:a67d c3              RET

I'm replacing some value by #define value, like on the INT INT_DISP but the decompiler still show swi(0x12):
void __cdecl16near _display_control(uint flags)

{
  code *pcVar1;

  pcVar1 = (code *)swi(0x12);
  (*pcVar1)();
  return;
}

Expected behavior
Equate should be shared between both view
Screenshots
See above
Attachments
N/A
Environment (please complete the following information):

OS: Debian linux
Java Version: 11.0
Ghidra Version: 9.0

Additional context
N/A

This is a known limitation.  While we had previously made an effort to support this capability, constants within the decompiler tend to loose track of their instruction origination which prevents applying an address-based equate reference within the decompiler.  In some situations an enum datatype can be used instead where a constant is associated with an integer variable or return type.

Mapping of equates into the decompiler is broken for 16-bit x86 programs. This seems to be a side-effect of the segmented address space.  Working on a patch.

Fix in master


chore: revert vue-router temp fix

Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Reminded by @Atinux in https://github.com/nuxt/nuxt.js/pull/5488#issuecomment-480824914 , the temporary fix can be reverted since vue-router has been released.
I'll do some testing before we merge it
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Codecov Report

Merging #5490 into dev will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #5490      +/-   ##
==========================================
+ Coverage   96.09%   96.13%   +0.03%
==========================================
  Files          74       74
  Lines        2562     2562
  Branches      651      651
==========================================
+ Hits         2462     2463       +1
+ Misses         84       83       -1
  Partials       16       16

Impacted Files
Coverage Δ

packages/vue-renderer/src/renderer.js
95.6% <0%> (+0.48%)
:arrow_up:

Continue to review full report at Codecov.

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

Codecov Report

Merging #5490 into dev will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev    #5490      +/-   ##
==========================================
+ Coverage   96.09%   96.13%   +0.03%
==========================================
  Files          74       74
  Lines        2562     2562
  Branches      651      651
==========================================
+ Hits         2462     2463       +1
+ Misses         84       83       -1
  Partials       16       16

Impacted Files
Coverage Δ

packages/vue-renderer/src/renderer.js
95.6% <0%> (+0.48%)
:arrow_up:

Continue to review full report at Codecov.

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

For reference, the actual fix is done in vuejs/vue-router#2506 (by @clarkdo)

Reverting this as the issue is still happening without this fix.


[Doxygen docs] Fix API documentation to match libbpf doxygen docs

Describe the bug

The API documentation in ebpf-for-windows/include/bpf/bpf.h comments are different from the libbpf doc https://github.com/libbpf/libbpf/blob/master/src/bpf.h#L191.
Also, there are incorrect exception keywords.

This issue is created to investigate and match libbpf doxygen documentation.
Details in https://github.com/microsoft/ebpf-for-windows/pull/3652
OS information
Windows 10 and above.
Steps taken to reproduce bug
Check
https://microsoft.github.io/ebpf-for-windows/bpf_2bpf_8h.html,
https://microsoft.github.io/ebpf-for-windows/libbpf_8h.html
Expected behavior
No exception keyword should be present.
Actual outcome
Exception keyword is present in each API.
Docs do not match to libbpf.
Additional details
No response

I am waiting for code reviewer approval.


Add JavaScript Question on String Concatenation

Summary of changes
This PR adds a question to the JavaScript quiz on string concatenation.
Checklist

[x] I have read and followed the contribution guidelines.
[x] I have read through the Code of Conduct and agree to abide by the rules.
[x] This PR is for one of the available issues and is not a PR for an issue already assigned to someone else.
[x] My PR title has a short descriptive name so the maintainers can get an idea of what the PR is about.
[x] I have provided a summary of my changes.

Changes have been implemented.

Thank you :D


use node kibana version attribute in download url

kibana download_url must use node kibana version attribute

This change leads to bug when using wrapper cookbook. download_url will be wrong.
It is really bad idea to calculate attributes in such way, isn't?

@jsirex it should not be and i think if we modify the version attribute in a role or wrapper cookbook we would not need to set  download_url

if wrapper cookbook set a different download location, then its up to the user whether the user wants to use version attribute or not.

There is branch version_test for you. Check it.
I've overrode version in wrapper cookbook (simple-kibana-test) to 5.5.5.
Expected calculated resource to be version now: 5.5.5, but you'll get:
rspec spec/version_spec.rb

simple-kibana-test::default
[2015-06-08T14:06:32+03:00] WARN: version now: 4.0.2
  expect calculated version right

If you need some explanations:
You code literally means:
default['kibana']['version'] = '4.0.2'
default['kibana']['download_url'] = "https://download.elasticsearch.org/kibana/kibana/kibana-#{node['kibana']['version']}-linux-x64.tar.gz"
default['kibana']['version'] = '5.5.5'

So it will be impossible to override url in wrapper cookbook.
But, looks like if you set node attributes explicitly via node object/environment/role with higher precedence it will work for you. May be these attributes loads right before cookbooks attributes evaluations. Not sure

@jsirex i think precedence for node/environment/role is the expected flow to override default. i have not seen any issue with that. but it may be just me, i try to ignore updating default attribute in wrapper cookbook especially when other attributes are evaluated by them.
but i could be wrong, it is really comes down to how the precedence is followed in different scenarios.

Similar issue faced long time ago in elasticsearch cookbook: https://github.com/elastic/cookbook-elasticsearch/issues/178
So, I close it.


Adds Mocha, Chai and Enzyme and sanity check.

Adds a foundation for testing, updated the npm "test" script and adds a sanity check.

Nice work! Sanity check is working on my end.  Merging and closing.


UI: dont allow, or handle errors from, docker when the API is not quite started yet

Problem
The docker daemon can take a while to start on reboots, causing unintuitive errors in the UI
Tasks

check if the daemon is up, and if not, disallow actions or give some basic feedback

Fixed


Support for an "idling" state after a set duration

Description of changes
Other VSCode discord rich presence plugins can detect and show an "idling" status, which neocord (and presence.nvim) lacked. Friends joke that I am on neovim for 20 hours, but that's only because I leave my machine turned on constantly and don't close my nvim sessions.
This PR allows options to tweak the duration before neocord reports an idling status (idle_timeout), and to allow whether to show this idling status on discord (modifiable with option idling_text) or completely remove any activity once idling (show_idle).
This works by attaching handlers to the CursorMoved and CursorMovedI events. This does not pick up changes within the terminal, though (and I am a heavy user of toggleterm.nvim), so I halted the internal idling timer on TermEnter.
This PR also indirectly fixes the global_timer option to work properly within workspaces so that the timer will reset for each new buffer entered when global_timer = false. For some reason, the behavior of neocord was the exact same regardless of the option setting unless I'm on my own changes.
Relevant Issues
This was a WIP feature from presence.nvim before that project became lowkey stale: https://github.com/andweeb/presence.nvim/projects/1#card-60538677, https://github.com/andweeb/presence.nvim/issues/40. I based my implementation off from the half-baked code, though it's my first time working deeply with Lua, let alone nvim plugins 😅.

How is this going? Is it ready for review?

Apologies! I am pretty busy with stuff atm, so I will get back to this once I have the time (probably not until this summer). Feel free to build off what I've started, though!

Sorry for never getting back to this lol, frankly my interests waned in implementing this fully (and with discord rich presence in general), so I won't be finishing this anytime soon.
If someone does get around doing this, that'd be awesome! I think cord.nvim has an "idling" implementation, but in my experience still doesn't work properly 100% of the time. Anyone, feel free to pick up on where I left off, but I'll be closing this PR for now.


added my name to guestbook.md

Hello,
Great course! I would like to add my name to the guestbook. Cheers.

Thank you!


Eliminate usages of Options utility.

This PR is a follow-up to https://github.com/SpineEventEngine/base/pull/333.
Though Options utility is removed from base, core-java still uses it.
This PR eliminates said usages in favor of using Option subclasses, namely EntityOption.

@alexander-yevsyukov, after https://github.com/SpineEventEngine/base/pull/333 is merged, the build should pass. Meanwhile, let's discuss the naming of the EntityOption.


Trace Information on Statements Page in Admin UIo

Background: https://airtable.com/tblD3oZPLJgGhCmch/viw1DKmbKhg2MIECH/recDDX3AlQ7rNqPja
Description: In addition to statements in the Admin UI, customers will be able to view a trace of each statement in the Statements page.
Team: John Kendall, Piyush Singh, Anne Birzin, Andrei Matei, Raphael 'kena' Poss

Dup of https://github.com/cockroachdb/docs/issues/6253


Update the Policy Generator for dev builds

/cherry-pick release-2.11

/test test-unit

@mprahl  The kind e2e test failed duo to a recent CRD update bug.  We have fixed in upstream and I just merged it to the midstream repo.
Could you rebase the PR to see if it help address the kind e2e test failure.
Also could you update the Docker.prow as well? The Docker.prow is used by CI/CD for building ACM DS images.
https://github.com/stolostron/multicloud-operators-subscription/blob/653c77378e533a389e05509b2ee08523750e6e74/build/Dockerfile.prow#L2

/approve
/lgtm


[#889] Minor debug in get_raytracing_from_pts()

Main changes:

Minor debug

Issues:
Fixes, in devel, issue#889

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

In the file tofu/data/_class8_reverse_ray_tracing.py:

Line 429:45: E262 inline comment should start with '# '


[SCB-753]NPE when query not exists service

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Format the pull request title like [SCB-XXX] Fixes bug in ApproximateQuantiles, where you replace SCB-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage increased (+0.0006%) to 87.06% when pulling 36fae2b03cd2e35c007cda2121f4dbd728cd46b5 on liubao68:test-011 into 6c4c1c1e9adc8e99fcc57cc4165576dc9655c4d9 on apache:master.


Temperature icon is sometimes cut off

Depending on the values being displayed the temperature icon is being cut off in the bar. Here's the display when everything is correct:

And here's when the icon is cutoff:

Here's my style and config files:
config
style.css

Have you tried to replace the temp icon with plain text to see if it is cut off as well ?

Seems a dup of #363


Can implement a simple CURD

Leave the serialization of parameters to the upper level
Signed-off-by: D<EMAIL_ADDRESS>Co-authored-by: D<EMAIL_ADDRESS>

Thank you for the submission. We're trying to keep the official samples to a manageable set of blockchain core concepts and aligned with the tutorials in the documentation, therefore we are not looking to expand the set for various permutations of CRUD patterns beyond the asset-transfer samples. The main point of this PR is valid, but I would suggest to mention the option of passing JSON instead of inputs in the existing asset-transfer-basic sample and associated doc tutorial with a small code snippet, rather than creating yet another chaincode asset in fabric-samples that needs to be maintained going forward. I'll close for now.


Fixed PHPC-357: The exception for "invalid namespace" does not list the broken name

The test case is copied over from HHVM

Rebased in https://github.com/10gen-labs/mongo-php-driver-prototype/commit/dead06a80bdc9905838f8cc0ce56407bafe45528, revisions in https://github.com/10gen-labs/mongo-php-driver-prototype/commit/e61b67809eedbc1893ed173677269d7440d49a9c, and merged in https://github.com/10gen-labs/mongo-php-driver-prototype/commit/2436e158c9a7e8d6a0bf2eba5edfb06ff381123e.


feat(lib): handle callbackId in contextRequest

What?
Modify the requestContext action to return a Promise in place of returning nothing.
Why?
JIRA: EXT-1920
The corresponding pull request in Storyfront is a prerequisite: https://github.com/storyblok/storyfront/pull/4368
This PR has the same purpose (and approach) as this one but is now for the getContext action (not selectAsset)
Usage implementation:

Result:

How to test? (optional)
This function can be tested using our Sandbox application as described in or [CONTRIBUTING.md →]

I read it briefly, and it looks good!
Any doubt or question?

Hi Demetrius, Let's pause this task.
Johannes is bringing a big change around this: https://github.com/storyblok/field-plugin/pull/260
We need more work on the Storyfront side to enable this change on the SDK side, but I'm very happy if we get this. Sorry that this PR probably is going to be closed (because it conflicts with #260), but I hope it was a good practice for you. My PR #253 also will be overridden by this, but we're moving very fast at this point, and it was a bit expected.

closing this PR in favor of #260


Add/Edit screen fields are not mis aligned

The screen has fields scrambled where they dont properly align. So is true with the tooltip icons.

Looks like there was a type and .form-horizontal somehow lost the 'f', thus the weird UI. Fixing.


Skipping rates for various calculations - k8s 1.8.0

I am running heapster 1.4.3 on a k8s 1.8.0 cluster (I've tried 1.4.0 too). InfluxDB 1.3.3. No matter what I set --metric-resolution to, I see this, always slightly larger than the value set for --metric-resolution. Here, it is using the default of 60s:
I1012 15:02:05.040687       1 rate_calculator.go:53] Skipping rates for namespace:kube-system/pod:kube-dns-7797cb8758-wjs94 - different create time new:2017-10-12 15:01:11 +0000 UTC  old:2017-10-12 15:00:10 +0000 UTC

I see this for most resource types, and as a result, metrics are sporadically missing when I query via kubectl top {pods,nodes}.
Example:
# kubectl top pods
NAME                       CPU(cores)   MEMORY(bytes)
foo-ab2d3s          17m          7Mi
busybox-56db8bd9d7-2s96p   0m           1Mi
busybox-56db8bd9d7-jm2j9   0m           1Mi
busybox-56db8bd9d7-lwzzn   0m           0Mi
busybox-56db8bd9d7-lkv8l   0m           0Mi
busybox-56db8bd9d7-cqlgq   0m           0Mi
busybox-56db8bd9d7-xpvz9   0m           1Mi
foo-98b9se          58m          8Mi
foo-kipb9a          37m          8Mi
...
# kubectl top pods
NAME                       CPU(cores)   MEMORY(bytes)
busybox-56db8bd9d7-cqlgq   0m           0Mi
busybox-56db8bd9d7-2s96p   0m           1Mi
busybox-56db8bd9d7-jm2j9   0m           1Mi
busybox-56db8bd9d7-lwzzn   0m           0Mi
foo-98b9se          59m          8Mi
foo-kipb9a          42m          8Mi

And here are some of the associated log messages:
1012 15:08:05.039841       1 rate_calculator.go:53] Skipping rates for namespace:default/pod:busybox-56db8bd9d7-lkv8l - different create time new:2017-10-12 15:08:05 +0000 UTC  old:2017-10-12 15:05:22 +0000 UTC
I1012 15:08:05.039867       1 rate_calculator.go:53] Skipping rates for namespace:kube-system/pod:kube-flannel-ds-8ntzq/container:kube-flannel - different create time new:2017-10-12 15:08:05 +0000 UTC  old:2017-10-12 15:05:22 +0000 UTC
I1012 15:08:05.039878       1 rate_calculator.go:53] Skipping rates for namespace:default/foo-ab2d3s - different create time new:2017-10-12 15:08:05 +0000 UTC  old:2017-10-12 15:04:22 +0000 UTC

Please can you advise? I've just cranked things up to --v=9 and am capturing the log, so if you need any of that information, I can provide it.

I've tried running heapster standalone, and it seems considerably worse:
I1012 17:58:05.129966       1 rate_calculator.go:53] Skipping rates for namespace:default/pod:busybox-56db8bd9d7-lwzzn/container:busybox - different create time new:2017-10-12 17:57:53.918666406 +0000 UTC  old:2017-10-12 17:49:33.262530678 +0000 UTC

There is an ~eight minute difference there. Can some light be shed on what is going on?

hmmm...
First things first, double check that the times are identical between all your nodes and masters (NTP, etc).  Once you've confirmed that, does anything jump out at you at log-level 9?  Look for messages about creating placeholder entries and similar.

Yeah, ntp/time sync across all nodes is good (was the first thing I checked), as for logs, I'll check when I'm working on this again (context switching is a great thing), probably within the next day or so.


please What is the difference between the fmt function and the add_snippet function

and which one should I use to write my snippet. It looks like fmt is easier to use, does he have all the features of the latter?  tankyou

Oh, they are entirely different: add_snippet(ft, snippets, opts) adds a snippet to luasnip, fmt creates a table of nodes via interpolation (in a way).
I won't go into any detail here, I'd only reiterate what is much better explained in DOC.md (search for both fmt and add_snippets).
BUT! I'd recommend some other resource to get started with luasnip, there are some nice videos by tjdevries on youtube for example, the README contains some pointers

(nvm, readme doesn't have any resources, but look for "Resources" in this comment)

thankyou

@L3MON4D3 Please, put this explanation to the docs under API.md file. Thanks.

I don't think we need a new file for those, maybe a Overview section in DOC.md would fit this better


resolve EC:1.14.13.- mappings

tocotrienol omega-hydroxylase activity ProperSubClassOf EC:1.14.13.-		0.08
4-chlorobenzaldehyde oxidase activity ProperSubClassOf EC:1.14.13.-		0.08
carbon disulfide oxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
phenanthrene 9,10-monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
3,5-xylenol methylhydroxylase activity ProperSubClassOf EC:1.14.13.-		0.08
2-octaprenyl-3-methyl-6-methoxy-1,4-benzoquinol hydroxylase activity ProperSubClassOf EC:1.14.13.-		0.08
chlorophenol 4-monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
2-hydroxy-phenylacetate hydroxylase activity ProperSubClassOf EC:1.14.13.-		0.08
thalianol hydroxylase activity ProperSubClassOf EC:1.14.13.-		0.08
dimethyl sulfide monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
1-hydroxy-2-oxolimonene 1,2-monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
2-methylnaphthalene hydroxylase activity ProperSubClassOf EC:1.14.13.-		0.08
phenanthrene 3,4-monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
phenanthrene 1,2-monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
1-methylnaphthalene hydroxylase activity ProperSubClassOf EC:1.14.13.-		0.08
branched-chain dodecylbenzene sulfonate monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
4-hydroxypyridine-3-hydroxylase activity ProperSubClassOf EC:1.14.13.-		0.08
3-methyl-2-oxo-1,2-dihydroquinoline 6-monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
oxidoreductase activity, acting on paired donors, with incorporation or reduction of molecular oxygen, NAD(P)H as one donor, and incorporation of one atom of oxygen ProperSuperClassOf EC:1.14.13.-		0.08
3-HSA hydroxylase activity EquivalentTo EC:1.14.13.-	(most probable)	0.8
bisphenol A hydroxylase A activity ProperSubClassOf EC:1.14.13.-		0.08
styrene monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
2-oxo-delta3-4,5,5-trimethylcyclopentenylacetyl-CoA 1,2-monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
isobutylamine N-hydroxylase activity ProperSubClassOf EC:1.14.13.-		0.08
limonene monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
xylene monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
tetrahydrofuran hydroxylase activity ProperSubClassOf EC:1.14.13.-		0.08
toluene-4-sulfonate monooxygenase activity ProperSubClassOf EC:1.14.13.-		0.08
phenylacetate hydroxylase activity ProperSubClassOf EC:1.14.13.-		0.08

3-HSA hydroxylase activity EquivalentTo EC:1.14.13.- (most probable) 0.8

This is incorrect; this partial EC corresponds to GO:0016709 oxidoreductase activity, acting on paired donors, with incorporation or reduction of molecular oxygen, NAD(P)H as one donor, and incorporation of one atom of oxygen
I am removing all partial ECs, this leads to incorrect mappings, and see if others need to be obsoleted.
Annotations are here: https://github.com/geneontology/go-annotation/issues/5221

-limonene monooxygenase activity :

[ ] remove partial EC

[ ] remove PMID:820855 reference, links to the wrong paper ("Long term changes in augmentation, potentiation, and depression of transmitter release as a function of repeated synaptic activity at the frog neuromuscular junction")

[ ] 1-hydroxy-2-oxolimonene 1,2-monooxygenase activity

phenylacetate hydroxylase activity

[ ] add EC:<IP_ADDRESS>
[ ] add RHEA:53392
[ ] fix def and term label to match RHEA and EC
[ ] change parent to 'oxidoreductase activity, acting on paired donors, with incorporation or reduction of molecular oxygen, reduced flavin or flavoprotein as one donor, and incorporation of one atom of oxygen'

2-hydroxy-phenylacetate hydroxylase activity

[ ] removed partial EC; cannot find an EC or a RHEA for this reaction.
[ ] added the reference cited by EAWAG-BBD , PMID: 8919815 - very weak evidence.

dimethyl sulfide monooxygenase activity

[ ] Add EC:<IP_ADDRESS>
[ ] Add RHEA:31355
[ ] Add parent monooxygenase activity

dimethylsulfoxide oxygenase activity (not in the list because the partial EC is on the def xref, not on the xref):

phenanthrene 3,4-monooxygenase activity
phenanthrene 1,2-monooxygenase activity
phenanthrene 9,10-monooxygenase activity

Citing PMID:1599252 (via EAWAG-BBD )
No evidence that this specific activity exists; also phenanthrene seems to be a grouping class for many compounds, see https://www.ebi.ac.uk/chebi/searchId.do?chebiId=CHEBI:25961
Removed the partial EC; added the PMID.

2-octaprenyl-3-methyl-6-methoxy-1,4-benzoquinol hydroxylase activity
[x] Removed partial EC

GO:0052871 alpha-tocopherol omega-hydroxylase activity >>
GO:0052870 tocopherol omega-hydroxylase activity
GO:0052872 tocotrienol omega-hydroxylase activity
Email MetaCyc

I am struggling with this entry
(RXN-11003), that has a reaction:
α-tocopherol   reduced [NADPH-hemoprotein reductase]   dioxygen → 13'-hydroxy-α-tocopherol   oxidized [NADPH-hemoprotein reductase]   H2O
Note that this reaction equation differs from the official Enzyme Commission reaction equation for this EC entry, which can be found here.
The official EC reaction uses leukotriene B4. How are these two substrates related?

Reply from Ron Capsi

if you go to the actual MetaCyc page for this EC number,
http://metacyc.ai.sri.com/META/NEW-IMAGE?type=EC-NUMBER&object=EC-<IP_ADDRESS>
(you can get there by scrolling down in the page you referred, the table under the reaction diagram has a row for the EC number that is clickable),
you will see that the reaction with leukotriene B4 is listed as the official reaction, but there are 4 additional reactions (including the one you pointed out) that are listed as "unofficial reactions".
The justification can be found in the summary for the enzyme, CYP4F2 (http://metacyc.ai.sri.com/gene?orgid=META&id=HS02675-MONOMER), where it is explained that the enzyme catalyzes all of these reactions.
Even though  the structures of leukotriene B4 and α-tocopherol are quite different, the enzyme catalyzes a pretty similar reaction - hydroxylation of the last carbon at the linear tail of the molecule. It is typical for P450s to catalyze similar reactions on a large number of different substrates.

Others should be handled by https://github.com/geneontology/go-ontology/issues/28108


Publish "uncharted" yaml along side charts

In addition to charts, we should publish the same source files as plain
yaml. Overlays are still applied (although chart specific overlays have
moved into the ./overlays/helm directory) and images are still resolved
to digests.
This will make it easier for non-helm based install via tools like kapp
or kubectl.
Published uncharts are at:

gs://projectriff/charts/uncharted/${version}/${chart}.yaml

Snapshots are at:

gs://projectriff/charts/uncharted/snapshots/${slug}/${charts}.yaml

Refs #86

Also not seeing the CRDs in uncharted\istio.yaml

Also not seeing the CRDs in uncharted\istio.yaml

this is by design, so never mind


[Eventhub] Way to check the ownership of partition of a consumer

Is your feature request related to a problem? Please describe.
It seems to have a loadbalancing bug when we have the setup :
2 workers 4 partitions, launch 2 additional workers then delete these 2 workers, the 2 remaining workers no longer consume messages. I'd like to relaunch and do a healtch check => if the worker doesn't consume any more messages, or has no more partitions, relaunch the pod. To do this, I'd need a way of knowing how many partitions the worker has.
Describe the solution you'd like
A way to have a healthcheck for consumer of event hub.

@xirzec If I understand correctly, this is a feature request and requires https://github.com/Azure/azure-sdk-for-js/issues/16406.
@gabrielSoudry Thanks for sharing this, I will work on this item next. The general scenario I am thinking to address your request is to use the checkpoint store client to list ownership where each ownership links a particular partition to a consumer using a unique identifier for the consumer but as it stands, that consumer unique identifier is not exported and https://github.com/Azure/azure-sdk-for-js/issues/16406 tracks exporting it. Please let me know if that works for you.

Yes perfect thanks you !

Hi @gabrielSoudry<EMAIL_ADDRESS>has been released with support for setting a client identifier in the client constructor options. You can use that identifier to lookup the partition association with the checkpoint store. Please let me know if you have any further questions.


[preprocessinator_extractor] Invalid byte sequence in UTF-8

Hi,
One of our header files(from SDK) has �(replacement character U+FFFD)  in the body which is causing problems in preprocessinator_extractor.rb.
The error which I observe is:
ArgumentError: invalid byte sequence in UTF-8 /var/lib/gems/2.5.0/gems/ceedling-0.31.0/lib/ceedling/preprocessinator_extractor.rb:46:in =~'`
To fix this issue I added small change in this file:
line.encode!('UTF-8', 'binary', invalid: :replace, undef: :replace, replace: '')
Which solves my problem.
Ceedling version:
    Ceedling:: 0.31.0
       CMock:: 2.5.3
       Unity:: 2.5.2
  CException:: 1.3.3

Merged to master.
Thank you @mvandervoord


When I have a book, and I have a last name like Passerin d'Entrèves then the author is not rendered

Updating to include a space like Passerin d' Entrèves renders the author in the book.

Thanks. I was struggling with solving this.

I'm new to this so take it all with a grain of salt, but this is what I'm seeing - corrections welcome!
For a new user like me: I see the missing name in the list of sources in the Source Hub task, and also in the New Source task you get to by clicking on a source in the Source Hub list.
Both of those source descriptions with the missing author are being pulled from the cached field of the corresponding row of the Sources table. The cached field gets set by a call to set_cached when the source gets saved (by clicking Save in the New Source task for example): https://github.com/SpeciesFileGroup/taxonworks/blob/190d33aba7659b368eaeabf438f047be58aaffee/app/models/source.rb#L224
set_cached for a book is defined in models/source/bibtex.rb - it calls get_cached, which calls cached_string, which calls render_with_style.
render_with_style calls to_citeproc, which calls ::TaxonWorks::Vendor::BibtexRuby.namecase_bibtex_entry since the default parameter normalize_names is true.
https://github.com/SpeciesFileGroup/taxonworks/blob/190d33aba7659b368eaeabf438f047be58aaffee/app/models/source/bibtex.rb#L802-L804
namecase_bibtex_entry calls parse_names from the bibtex-ruby gem, and that's where the author name "Passerin d'Entrèves, Alessandro" fails to parse. At that point the author of the BibTeX::Entry object that was passed into parse is set to the empty string; that empty author gets passed back up the call chain and cached gets set without an author.

There's already an issue in the bibtex-ruby repository for parsing this kind of name: https://github.com/inukshuk/bibtex-ruby/issues/149
You can test in irb:
irb(main):001> require 'bibtex'
=> true
irb(main):002> book = BibTeX::Entry.new
=> #<BibTeX::Entry >
irb(main):003> book.author = "Passerin d'Entrèves, Alessandro"
=> "Passerin d'Entrèves, Alessandro"
irb(main):004> book.parse_names
=> #<BibTeX::Entry author = >
irb(main):005> book.author = "Passerin Entrèves, Alessandro" # Remove the "d'"
=> "Passerin Entrèves, Alessandro"
irb(main):006> book.parse_names
=> #<BibTeX::Entry author = Passerin Entrèves, Alessandro> # That name was parsed
irb(main):008> book.author = "Passerin d'Entrèves En, Alessandro" # A third capitalized family name works too
=> "Passerin d'Entrèves En, Alessandro"
irb(main):009> book.parse_names
=> #<BibTeX::Entry author = Passerin d'Entrèves En, Alessandro>

I tried the namae package as inukshuk suggested and had the same issue, "Passerin d'Entrèves, Alessandro" fails to parse.

the only way it works now if you put the name like 'Passerin d' Entrèves, Alessandro' with an extra space...

Yes, as @kleintom notes this falls all the way back to namae and the parser.  Ideally we work with their code-bases to add the fixes, instead of hacking a work-around here.

Yes, as @kleintom notes this falls all the way back to namae and the parser.  Ideally we work with their code-bases to add the fixes, instead of hacking a work-around here.


[QUESTION/HELP] How to use k3d-managed registry as a pull-through-cache registry for docker hub

Question / Where do you need Help?
The documentation states that registries.yaml can be used to define redirects like normally in k3s. the --registry-config param is used here. I found some tutorials online how to define your own pull-through-cache registries mainly using the Environment Variable REGISTRY_PROXY_REMOTEURL and similar to the docker image registry:2 or similar.
However, the documentation recommends to use the k3s-managed registries, where the --registry-use param is used. Unfortunately the documentation of k3d registry create does not have any extra parameters which would indicate how to enable pull-through-cache.
How then to use k3d-managed registries as a pull-through-cache? I.e. how to set variables like REGISTRY_PROXY_REMOTEURL.
Scope of your Question
k3d documentation

Here is an example k3d-config.yaml I use. Feel free to tag me if you need any help to get this to work
k3d cluster create --config k3d-config.yaml
apiVersion: k3d.io/v1alpha5
kind: Simple
metadata:
  name: yolooooo
servers: 1
agents: 3
registries:
#   create:
#     name: docker-io # name of the registry container
#     proxy:
#       remoteURL: https://registry-1.docker.io # proxy DockerHub
#       username: <DOCKER_USERNAME>
#       password: <DOCKER_PASSWORD>
#     volumes:
#       - /tmp/reg:/var/lib/registry # persist data locally in /tmp/reg
#   config: | # tell K3s to use this registry when pulling from DockerHub
#     mirrors:
#       "docker.io":
#         endpoint:
#           - http://docker-io:5000
  config: |
    mirrors:
      "docker.io":
        endpoint:
          - http://<IP_ADDRESS>:5000
      "quay.io":
        endpoint:
          - http://<IP_ADDRESS>:5001
      "ghcr.io":
        endpoint:
          - http://<IP_ADDRESS>:5002
options:
  k3d:
    disableLoadbalancer: true
  k3s:
    extraArgs:
      - arg: --disable=cloud-controller
        nodeFilters:
          - server:*
      - arg: --disable=helm-controller
        nodeFilters:
          - server:*
      - arg: --disable=traefik
        nodeFilters:
          - server:*
      - arg: --disable=servicelb
        nodeFilters:
          - server:*

Thank you for sharing your k3d-config.yaml
I am using k3d to create clusters when I need them for testing and developing purposes. Hence I like the registry to be separate from the cluster. Unfortunately k3d registry create does not have a --config option. Is there an alternative?

Now I understood, that the volume directive /tmp/reg:/var/lib/registry is probably already enough to preserve registry cache across cluster starts.

I now tried to setup the cluster+registry at once using the following configuration
apiVersion: k3d.io/v1alpha5
kind: Simple
# using default cluster name
kubeAPI:  # same as `--api-port k3d-jolin:6445` (where the name would resolve to <IP_ADDRESS>)
    host: jolin.localhost
    hostIP: "<IP_ADDRESS>"
    hostPort: "6445"
registries:
    create:
        name: registry.localhost
        hostPort: "5000"
        proxy:
            # proxy DockerHub
            remoteURL: https://registry-1.docker.io
        volumes:
            # persist data locally in /tmp/reg
            - /tmp/reg:/var/lib/registry
    # tell K3s to use this registry when pulling from DockerHub
    config: |
        mirrors:
          "docker.io":
            endpoint:
              - http://registry.localhost:5000
options:
    kubeconfig:
        updateDefaultKubeconfig: false
        switchCurrentContext: false
image: rancher/k3s:v1.29.1-k3s1
ports:
    # same as `--port '80:80@loadbalancer'`
    - port: 80:80
      nodeFilters:
        - loadbalancer

Unfortunately I couldn't find a way how to push into this registry using docker. With the externally k3d-managed registry it worked. I tried to set <IP_ADDRESS> registry.localhost in my hostfile and tried to push using docker buildx --tag registry.localhost:5000/mydocker:latest --push .. Or with also using localhost:5000/mydocker:latest. Nothing worked.
I get an ERROR: unsupported.
Also when trying the official registry test, it just cannot connect, retrying and retrying and retrying.
docker pull nginx:latest
docker tag nginx:latest registry.localhost:5000/nginx:latest
docker push registry.localhost:5000/nginx:latest

The push refers to repository [registry.localhost:5000/nginx]
fd31601f0be4: Retrying in 10 seconds
93b4c8c4ac05: Retrying in 10 seconds
b7df9f234b50: Retrying in 10 seconds
ab75a0b61bd1: Retrying in 10 seconds
c1b1bf2f95dc: Retrying in 10 seconds
4d99aab1eed4: Waiting
a483da8ab3e9: Waiting

@venkatamutyala can you help?

This seems to be a docker problem - a registry with pull-through-cache does not support pushing to it so it seems
https://stackoverflow.com/questions/33610215/unable-to-push-image-to-a-docker-registry-configured-as-proxy-cache

@venkatamutyala a new problem appeared: the username and password are not correctly linked
here the most important logs
Unable to retrieve some image pull secrets (container-registry); attempting to pull the image may not succeed.

pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed

It would be great if the documentation gets updated so that the k3d-config.yaml description as well as the k3s registry create command include these new proxy arguments

Hey @schlichtanders in case it helps, here is how i create the registry using docker compose. I use a separate VM for this since all our developers leverage it
version: '3'
services:
  registry-docker:
    image: registry:2
    volumes:
      - ./data:/var/lib/registry
    ports:
      - "5000:5000"
    environment:
      REGISTRY_PROXY_REMOTEURL: https://registry-1.docker.io
    restart: always
  registry-quay:
    image: registry:2
    volumes:
      - ./data:/var/lib/registry
    ports:
      - "5001:5000"
    environment:
      REGISTRY_PROXY_REMOTEURL: https://quay.io
    restart: always
  registry-ghcr:
    image: registry:2
    volumes:
      - ./data:/var/lib/registry
    ports:
      - "5002:5000"
    environment:
      REGISTRY_PROXY_REMOTEURL: https://ghcr.io
    restart: always
  registry-gcr:
    image: registry:2
    volumes:
      - ./data:/var/lib/registry
    ports:
      - "5003:5000"
    environment:
      REGISTRY_PROXY_REMOTEURL: https://gcr.io
    restart: always
  registry-aws:
    image: registry:2
    volumes:
      - ./data:/var/lib/registry
    ports:
      - "5004:5000"
    environment:
      REGISTRY_PROXY_REMOTEURL: https://public.ecr.aws
    restart: always
  registry-mcr:
    image: registry:2
    volumes:
      - ./data:/var/lib/registry
    ports:
      - "5005:5000"
    environment:
      REGISTRY_PROXY_REMOTEURL: https://mcr.microsoft.com
    restart: always
  registry-gitlab:
    image: registry:2
    volumes:
      - ./data:/var/lib/registry
    ports:
      - "5006:5000"
    environment:
      REGISTRY_PROXY_REMOTEURL: https://registry.gitlab.com
    restart: always

#https://distribution.github.io/distribution/recipes/mirror/
#https://distribution.github.io/distribution/recipes/mirror/#configure-the-cache
#https://distribution.github.io/distribution/about/deploying/#customize-the-storage-location


How to separate project for Cargo.lock file

Hello,
The Cargo.lock file is treated differently for library versus binary targets by Cargo.
With the project structure as in the example-service you have, the library is not logically separated from the server. In my use-case, the library is public and I should not commit Cargo.lock with it; while the server is a daemon that should have Cargo.lock committed with it for reproducibility. This is explained in detail here:  Why do binaries have Cargo.lock in version control, but not libraries?.
If you could elaborate on this point I would appreciate it.

Hi! I think the Cargo.lock is only used by binaries. Any project only depending on the [lib] won't use the Cargo.lock, but it will be used by the [[bin]]s.

@tikue Hello there. You are right! thanks for the insight.


dune: 1.11.1 -> 1.11.2

Motivation for this change
Changelog: https://github.com/ocaml/dune/releases/tag/1.11.2
Things done

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

[X] NixOS
[X] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @

@GrahamcOfBorg build flow fstar trv eff stog


Brackets or Brackets Beautify added a space before every closing quote in my file.

Prerequisites

[ ] Can you reproduce the problem with Debug -> Reload Without Extensions?
[ ] Did you perform a cursory search to see if your bug or enhancement is already reported?
[ ] Did you read the Troubleshooting guide?

For more information on how to write a good bug report read here
For more information on how to contribute read here
Description
[Description of the bug or feature]
Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

Expected behavior: [What you expected to happen]
Actual behavior: [What actually happened]
Versions
Please include the OS and what version of the OS you're running.
Please include the version of Brackets. You can find it under Help -> About Brackets (Windows and Linux) or Brackets -> About Brackets (macOS)

Not sure whether it was Brackets or Brackets Beautify. Probably was Beautify because it happened after saving (which beautified it). Really liked this extension and Brackets until now...just spent over an hour eliminating all those spaces, because most of them cause errors (class names etc.)
If there's not a decent explanation and a fix for this, I will never use this program again.

Hi @seejayjames,
there's an issue on the brackets-beautify repo that has a similar issue with certain quotes, which seemed to be caused by invalid HTML.
You could try pinpointing the issue by removing lines until brackets-beautify works correct to find the issue or report it to brackets-beautify for more debugging. Brackets itself doesn't do any code modifying on save by default.

Ah ok, it did in fact do it after I entered a line incorrectly. Good to know about the known issue on the repo, I had a feeling it was Beautify and not Brackets itself. Thanks for the info!


Chinese keyboard layout broken?

Since upgrading to Sierra, input source switching has been broken. Not sure what the problem is. Seems like some other languages work, but not Chinese. I attached a 10 second clip. Key presses show up on the Keyboard Viewer but is not handled otherwise.
https://youtu.be/dkkM9Ion20Q

Is it Karabiner-Elements issue?


Wrong label: "We are hosting X collectives"

Describe the bug
On https://opencollective.com/stumptownsyndicate on the bottom of the page there is this string: "We are hosting 5 collectives". But something is wrong, there is no collective below.

This is happening because the member table has 5 rows where this collective has the role=HOST
Those collectives are status=FALSE and therefore are not showing up.
We shouldn't show that a host is hosting a collective until that collective is Active - but this pulling the information as expected.

Instead of status=FALSE you mean isActive=false? Or where are you exposing status?

my mistake, it's isActive=FALSE

Deployed.


Incorrect / incomplete rebuild for harfbuzz 10

Solution to issue cannot be found in the documentation.

[x] I checked the documentation.

Issue
This was green and merged but failed on main
https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=1104754&view=results
Something about gio not being available. Not sure if it's worth a restart or why it happened on main but not on that PR?
In any case it is making it so I can't upgrade some libraries because of this one's dependency on older harfbuzz.
Installed packages
N/A

Environment info
N/A

Yeah, it looks like there were some package updates in between the PR test build and the merged build that caused the latter to fail — we now need to explicitly list a few more dependencies to pull in some development-only files. This should be dealt with in #82.

#82 is merged now; hopefully new packages should actually get built and make it through the system this time.

Looks like this is indeed now fixed. The final build was all green and the bot has proposed harfbuzz 10 migrations for some packages that depend on pango.


ENH: Adding sensible row and column names to biplot_scores in CCA and…

… RDA
Addresses #1322

Thanks for the fixes! I noticed that the unit tests are calling assert_ordination_results_equal with ignore_biplot_scores_labels=True, so I don't think the labels are actually being compared to the expected labels. I can't remember why we added that option to assert_ordination_results_equal at the sprint last year. Can it just be removed so that biplot score labels are always compared? You might need to search the codebase for ignore_biplot_scores_labels to see where else it's being used, and if we can drop it.
Does this also fix item 2 in #1322?
Can you add a changelog mention? Up to you whether this is tagged as a bug fix or feature enhancement.
Thanks again!

Right on.
Yeah.  Turns out that only the canonical axes are reported for this.  So
this also fixes part 2.
On Fri, Jun 3, 2016 at 3:13 PM, Jai Ram Rideout<EMAIL_ADDRESS>wrote:

Thanks for the fixes! I noticed that the unit tests are calling
assert_ordination_results_equal with ignore_biplot_scores_labels=True, so
I don't think the labels are actually being compared to the expected
labels. I can't remember why we added that option to
assert_ordination_results_equal at the sprint last year. Can it just be
removed so that biplot score labels are always compared? You might need to
search the codebase for ignore_biplot_scores_labels to see where else
it's being used, and if we can drop it.
Does this also fix item 2 in #1322
https://github.com/biocore/scikit-bio/issues/1322?
Can you add a changelog mention? Up to you whether this is tagged as a bug
fix or feature enhancement.
Thanks again!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/biocore/scikit-bio/pull/1368#issuecomment-223707377,
or mute the thread
https://github.com/notifications/unsubscribe/AD_a3eF1i7RVHrYPyFvVAiCDE2mlslYaks5qIKcjgaJpZM4It2Ka
.

Awesome! Let me know when you're ready for me to take another pass through.

Should be good to go!

Thanks @mortonjt!


Detect Aliases

NoInconsistentAliases.detectAliases
config : List Rule
config =
    [ NoInconsistentAliases.config []
        |> NoInconsistentAliases.detectAliases
        |> NoInconsistentAliases.rule
    ]

Success (Solo Alias)
-- src/Page/Home.elm
module Page.Home exposing (view)
import Html.Attributes as Attr
view = ""

Success (Matching Aliases)
-- src/Page/About.elm
module Page.About exposing (view)
import Html.Attributes as Attr
view = ""

-- src/Page/Home.elm
module Page.Home exposing (view)
import Html.Attributes as Attr
view = ""

Failure (Two Inconsistent Aliases)
Both of these modules will be reported as we don't know which alias you would prefer.
-- src/Page/About.elm
module Page.About exposing (view)
import Html.Attributes as A
view = ""

-- src/Page/Home.elm
module Page.Home exposing (view)
import Html.Attributes as Attr
view = ""

Inconsistent alias A for module Html.Attributes.
I found the following aliases for Html.Attributes across your project:AAttrYou should pick one alias to be consistent everywhere. If you configure this rule with your chosen alias then I can attempt to fix this everywhere.

Failure (One Inconsistent Alias)
Only the alias in Page.Contact will be reported as it's clear that you prefer Attr. We don't offer a fix here, but if you configure the rule with your preferred alias then we will.
-- src/Page/About.elm
module Page.About exposing (view)
import Html.Attributes as Attr
view = ""

-- src/Page/Contact.elm
module Page.Contact exposing (view)
import Html.Attributes as A
view = ""

-- src/Page/Home.elm
module Page.Home exposing (view)
import Html.Attributes as Attr
view = ""

Incorrect alias A for module Html.Attributes.
This import does not use your preferred alias Attr for Html.Attributes.You should update the alias to be consistent with the rest of the project. Remember to change all references to the alias in this module too.

Todo

[x] Report when an import is aliased differently in two modules
[x] Report only the least popular inconsistent aliases
[x] Fall back on preferred module alias if one is configured (integration)
[x] Do not report when an import is only aliased in one module (happy path)
[x] Collision detection (see #3)

No report when an import with a preferred alias in the way
Report when a preferred alias is an option, but there's no import in the way

[x] Documentation

Fixes #5

I want to merge #31 first which will likely introduce a lot of conflicts.


Merge beta theming work into main

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: # (issue, if applicable)
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Type of change:
Please delete options that are not relevant.

[x] New capability
[x] Revision of an existing capability

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

Summary by Sourcery
Merge beta theming work into main by introducing a new 'flat' button variant, enhancing theming support with CSS custom properties, and updating documentation and tests to reflect these changes. Deprecate old attributes in favor of a more flexible variant system.
New Features:

Introduce a new 'flat' variant for the auro-button component, allowing for a minimalistic button style without borders or background colors.

Enhancements:

Refactor button styles to use CSS custom properties for better theming support, including new tokens for button colors and states.
Deprecate 'secondary' and 'tertiary' attributes in favor of a unified 'variant' attribute for button styling.
Add support for 'onDark' attribute to improve button appearance on dark backgrounds.

Documentation:

Update README and documentation files to reflect new button variants and theming capabilities, including examples for the 'onDark' attribute.

Tests:

Update test cases to verify the new 'flat' variant and the use of the 'onDark' attribute, ensuring backward compatibility with deprecated attributes.

Chores:

Remove outdated TODO comments related to Sass and CSS variable processing, as these have been addressed with the new theming approach.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: jason-capsule42:white_check_mark: jordanjones243:x: semantic-release-botYou have signed the CLA already but the status is still pending? Let us recheck it.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Connect-SPOService Global Administrator 401 Unauthorized

Running the command:
Connect-SPOService -Url $siteURL -Credential $userCred
using an account that has global administrator privileges. Any thoughts on what could be the reason?

Hello @iyadzikra
How are you? Hope you are doing good.
Here we are limited to documentation issues and changes, if you need support, I recommend opening a support ticket directly from your tenant.
Thank you

@get-itips Thank you very much for the contribution and sharing this explanation. @iyadzikra Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. We proceed here to close it.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Lightning distance

Hello and Season's Greetings. In advance of the new year I've acquired a lightning detector and made some mods to my web site which borrows from yours (acknowledged). Thanks for your "lightning bug" fix. However, I am seeing
Average Strike Distance | $day.lightning_avg_distance.avg
instead of a value reported for the average distance (expected value zero atm). I haven't yet had time to look into it but am wondering if you have a variable declared in weewx.conf or what I'm missing here.

So I am using the driver mapped parameters for a WeatherFlow Tempest Station which might be different from your variable map used by your weather station. I would check your station documentation to see what lightning data is sent from your station. It's likely that the variables one station uses are different from another.