Pass documentation context its only bundle during initialization

Bug/issue #, if applicable: rdar://136208312
Summary
This is the fourth of many changes to redefine how documentation contexts are created.
This adds a new DocumentationContext initializer that takes a DocumentationBundle as a parameter. This change means that the context doesn't need a DocumentationContextDataProvider to provide it a bundle. It only needs a simplified (different) data provider to read the data for the bundle's files.
It also updates ConvertService and ConvertAction to create the context of documentation-to-build using this initializer. In doing so, they can't use DocumentationConverter anymore because it expects to register content with the context itself using its DocumentationWorkspace (which conforms to DocumentationContextDataProvider).
Changing DocumentationConverter to work this way would require source breaking changes—I know because I tried—so instead I looked into extracting its functionality into pieces that ConvertService and ConvertAction both could use. This investigation revealed that they share very little code. The following image shows the lines of DocumentationConverter.convert(outputConsumer:) highlighted depending on if it's

workspace registration code, no longer needed or wanted. (red)
only used by the convert action (blue)
only used by the convert service (yellow)

As you can see, less than 10% of the code is shared between the convert action and the convert device. This realization led me to create distinct implementations for each, making the convert service implementation short enough that it could be inlined into ConvertService.convert(request:messageIdentifier:).
Lastly, now that ConvertAction no longer used DocumentationConverter I moved some properties around so that tests could inspect what used to be converter configuration on the action instead.
Dependencies
None.
Testing
Nothing in particular this isn't a user-facing change.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] ~Added~ Updated tests
[x] Ran the ./bin/test script and it succeeded
~[ ] Updated documentation if necessary~ A future PR in this series will update the contributor documentation when a few more changes are in place.

@swift-ci please test

@swift-ci please test


:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject

群晖Python升级后就这样了。

我也是这个提示，什么情况？

可能是gevent库和Python版本不兼容， 试试重新安装subfinder。

重新装了也没用

升级gevent试试, pip install --upgrade gevent.

依然收到 “:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject” 提示，但是字幕能够下载了

pip3 install -U --force-reinstall --no-binary :all: gevent

upgrade gevent 的时候报这个错：
ERROR: Could not find a version that satisfies the requirement geven
ERROR: No matching distribution found for geven

补充一下，搞定了：
Package    Version Latest  Type

gevent     1.5.0   20.12.0 wheel
setuptools 19.6    51.1.0  wheel
先更新setuptools，在更新gevent，就可以运行了。
不能运行的兄弟，可以pip全部更新一下试试。

macos python3.8遇到相同问题，重新安装greenlet 0.4.14版本解决：
pip install -U --force-reinstall greenlet==0.4.14

macos python3.8遇到相同问题，重新安装greenlet 0.4.14版本解决：
pip install -U --force-reinstall greenlet==0.4.14

上面提到的问题都是由于 subfinder 的依赖中指定的 gevent 版本过低, 将 gevent 强制升级到最新版本就好了.
pip install -U upgrade-strategy eager gevent

> pip install --help
...
  -U, --upgrade               Upgrade all specified packages to the newest available version. The handling of dependencies depends on the upgrade-strategy used.
  --upgrade-strategy <upgrade_strategy>
                              Determines how dependency upgrading should be handled [default: only-if-needed]. "eager" - dependencies are upgraded regardless of whether the currently installed version satisfies
                              the requirements of the upgraded package(s). "only-if-needed" -  are upgraded only when they do not satisfy the requirements of the upgraded package(s).
...

我已经在最新版本(2.0.2)的 subfinder 中修复了这个问题.

上面提到的问题都是由于 subfinder 的依赖中指定的 gevent 版本过低, 将 gevent 强制升级到最新版本就好了.
pip install -U upgrade-strategy eager gevent

> pip install --help
...
  -U, --upgrade               Upgrade all specified packages to the newest available version. The handling of dependencies depends on the upgrade-strategy used.
  --upgrade-strategy <upgrade_strategy>
                              Determines how dependency upgrading should be handled [default: only-if-needed]. "eager" - dependencies are upgraded regardless of whether the currently installed version satisfies
                              the requirements of the upgraded package(s). "only-if-needed" -  are upgraded only when they do not satisfy the requirements of the upgraded package(s).
...

我已经在最新版本(2.0.2)的 subfinder 中修复了这个问题.

不好意思。我不知道为啥，我在群晖上面装了好多次（无论是pip 安装，还是下载安装包安装），版本还是停留在2.0.1， issue依然存在。
xxx@xxx-Synology:/volume1/homes/xxx/Apps/Github/subfinder-2.0.2$ sudo pip install subfinder
Collecting subfinder
  Using cached subfinder-2.1.0-py3-none-any.whl (236 kB)
Requirement already satisfied: rarfile in<EMAIL_ADDRESS>(from subfinder) (4.0)
Requirement already satisfied: gevent<22.0.0,>=21.1.2 in<EMAIL_ADDRESS>(from subfinder) (21.1.2)
Requirement already satisfied: beautifulsoup4<5.0.0,>=4.9.3 in<EMAIL_ADDRESS>(from subfinder) (4.9.3)
Requirement already satisfied: requests in<EMAIL_ADDRESS>(from subfinder) (2.25.1)
Requirement already satisfied: six in<EMAIL_ADDRESS>(from subfinder) (1.15.0)
Requirement already satisfied: lxml in<EMAIL_ADDRESS>(from subfinder) (4.6.2)
Requirement already satisfied: soupsieve>1.2 in<EMAIL_ADDRESS>(from beautifulsoup4<5.0.0,>=4.9.3->subfinder) (2.2)
Requirement already satisfied: setuptools in<EMAIL_ADDRESS>(from gevent<22.0.0,>=21.1.2->subfinder) (54.1.0)
Requirement already satisfied: zope.event in<EMAIL_ADDRESS>(from gevent<22.0.0,>=21.1.2->subfinder) (4.5.0)
Requirement already satisfied: zope.interface in<EMAIL_ADDRESS>(from gevent<22.0.0,>=21.1.2->subfinder) (5.2.0)
Requirement already satisfied: greenlet<2.0,>=0.4.17 in<EMAIL_ADDRESS>(from gevent<22.0.0,>=21.1.2->subfinder) (1.0.0)
Requirement already satisfied: idna<3,>=2.5 in<EMAIL_ADDRESS>(from requests->subfinder) (2.10)
Requirement already satisfied: chardet<5,>=3.0.2 in<EMAIL_ADDRESS>(from requests->subfinder) (4.0.0)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in<EMAIL_ADDRESS>(from requests->subfinder) (1.26.3)
Requirement already satisfied: certifi>=2017.4.17 in<EMAIL_ADDRESS>(from requests->subfinder) (2020.12.5)
Installing collected packages: subfinder
Successfully installed subfinder-2.1.0

xxx@xxx-Synology:/volume1/homes/xxx/Apps/Github/subfinder-2.0.2$ subfinder --version
<frozen importlib._bootstrap>:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
<frozen importlib._bootstrap>:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
<frozen importlib._bootstrap>:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
<frozen importlib._bootstrap>:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
<frozen importlib._bootstrap>:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
<frozen importlib._bootstrap>:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
<frozen importlib._bootstrap>:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
<frozen importlib._bootstrap>:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
<frozen importlib._bootstrap>:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
<frozen importlib._bootstrap>:219: RuntimeWarning: greenlet.greenlet size changed, may indicate binary incompatibility. Expected 144 from C header, got 152 from PyObject
subfinder 2.0.1

@zhanglianbo35
pip install -U subfinder

试过的了。还是一样

可能你的 Python 环境已经被你搞乱了, 不知道具体情况我也无法给出解决方法.
可以试试创建一个新的虚拟环境, 然后在虚拟环境中安装 subfinder.


Reverse token bar stats

Most RPG's are useing HP as Full healt like 12/12 that works perfectly with FoundryVTT and the displayed stat bar under a token.
Star Wars goes the other way. We track wounds and strain. If my Char is at full HP, I have 0 wounds. Which totally makes sense. the stat bar under and over the token is empty - which is ok. But the colors are going into the false direction. 1 wound is red and full wounds are green. I think you understand my "problem" and the thing I want to suggest if that could be changed.

This one's been on the to-do list since the beginning, but I've not got round to it yet. 😅

I have made changes to the Dev branch no now support this, should be in the next update.


Ensure "parse" fuzz target covers all options

The fuzz target parse.rs (intended to find crashes/panics) was missing some newer options.
Change to use Options::from_bits_truncate to generate arbitrary combinations of all options, current and future.
Not sure if the field order in the struct makes much difference, but I figured it's better to have the fixed length field first, so it doesn't move around and change unintentionally.
Tried it out, and it found #973 in under a minute (on v0.12.2 before the fix was merged).

Forgot to mention this does preclude using the -only_ascii=1 flag with libfuzzer (all option combinations were possible when they were bools but now they're not). Shouldn't make much difference, but I haven't played with it that much.

@kdarkhan in the README example you had put -only_ascii=1. Have you found that to be helpful?
If we really wanted, it could be supported while keeping this option-agnostic via something like [bool; Options::all().bits().count_ones()].

I haven't really tested it much with fuzzing this project. I put the option only as an example of fixing the build and not as the suggested option. Feel free to remove it if it is misleading now.


feat(compiler-sfc): support more TS syntaxes

closes #7161
upgrade @babel/parser to support new TS syntaxes.

TS 4.9: satisfies operator: obj satisfies SomeType
TS 4.7: Instantiation Expressions: const makeHammerBox = makeBox<Hammer>;

Sadly closed
https://github.com/vuejs/core/commit/53e3533b75adc32f979adde34bde3b581e48084b


Review Docker stack

Depends on https://github.com/nlpsandbox/phi-deidentifier/pull/47
Depends on #56

We made great progress in the past couple of weeks on improving the security and user-friendliness of the docker-compose.yml. We have identified a docker-compose.yml content suitable for the launch of the NLP Sandbox.


Change router params to a generic/any type instead of Dictionary

Expected behavior, to push params of any type, like:
this.$router.push({
name: 'piloto',
params: {
id: item.id.toString(),
isThisTrue: (this.opN === 1),
isThisFalse: (this.opN === 2)
}
})
Currently this code on typescript would produce the following error:
No overload matches this call
https://github.com/vuejs/vue-router/blob/39d46b254bd8130464626539e7b73b4cc760241b/types/router.d.ts#L140

Duplicate of https://github.com/vuejs/vue-router/issues/2895


win 10 2004 WHEA_UNCORRECTABLE_ERROR bluescreen

windows bluescreens when im trying to load airhv.sys, minidump file:
`
Microsoft (R) Windows Debugger Version 10.0.21349.1004 AMD64
Copyright (c) Microsoft Corporation. All rights reserved.
Loading Dump File [C:\Windows\Minidump\061421-10859-01.dmp]
Mini Kernel Dump File: Only registers and stack trace are available
************* Path validation summary **************
Response                         Time (ms)     Location
Deferred                                       srv*
Symbol search path is: srv*
Executable search path is:
Windows 10 Kernel Version 19041 MP (12 procs) Free x64
Product: WinNt, suite: TerminalServer SingleUserTS Personal
Edition build lab: 19041.1.amd64fre.vb_release.191206-1406
Machine Name:
Kernel base = 0xfffff8020ce00000 PsLoadedModuleList = 0xfffff8020da2a230
Debug session time: Mon Jun 14 22:24:21.834 2021 (UTC + 3:00)
System Uptime: 0 days 0:05:36.650
Loading Kernel Symbols
...............................................................
................................................................
................................................................
.................................
Loading User Symbols
Loading unloaded module list
..........................
For analysis of this file, run !analyze -v
nt!KeBugCheckEx:
fffff8020d1f6c20 48894c2408      mov     qword ptr [rsp+8],rcx ss:0018:ffff870029320910=0000000000000124
`

and stack

i got rid of previos bluescreen in airhv.sys by updating to last commit but now i have bluescreen in airhvctrl.sys, stack:

also found pretty same issue in gbhv repository https://github.com/Gbps/gbhv/issues/21


The About->Partners page is incomplete

The About->Partners page (https://edirepository.org/about/partners-and-affiliations) is incomplete.

This page has been temporarily removed from the main menu "About" section and the corresponding link in the shared footer. It will return once the page is finished. See PASTAplus/web-x@88a2f3be6965fe1f377b40642b3995d77358028b.


Figure out Firefox config

see if I can Drop in a config file to make FireFox exactly how I like it with no fuss

reminder to set up password manager first I think

changing description to be libreWolf instead of firefox

sticking with firefox at the moment. Don't need that much security. Firefox is close enough removed for now from google that I am happy. closing


Support branching for flutter repo

To support Flutter to be a branch based build, test and release process, both cocoon backend and frontend need to be updated.
Backend will support branch property in commit model, and also support branch matching to pre-defined regular expressions.
Following backend changes

[x] basic support (https://github.com/flutter/cocoon/pull/658)
[x] list commits based on branches, and update indexes for checklists to use the the branch (https://github.com/flutter/cocoon/pull/668)
[x] Use both branch and SHA for commit Key (https://github.com/flutter/cocoon/pull/684)
[x] create /api/branches to return available branches (https://github.com/flutter/cocoon/pull/682)
[x] calculate tree status based on branches (https://github.com/flutter/cocoon/pull/679)
[ ] calculate cirrus task status based on branches (https://github.com/flutter/cocoon/pull/688)
[ ] update reserve-task API to pick tasks regardless of branch
[ ] push build status to github for all branches
[ ] push gold status to github for all branches (requires support from Gold for branching)
[ ] refresh chromebot status for all branches

Following frontend changes

[X] query branches from backend
[ ] update build state to track branches
[ ] pass build state branches to cocoon server for fetching commit statuses
[ ] add dropdown to build dashboard to interacting with the branches on build state

https://github.com/flutter/cocoon/pull/658 focuses on the backend part.

Linking here https://github.com/flutter/flutter/issues/51884

[ ] list commits based on branches

is done via https://github.com/flutter/cocoon/pull/668

@CaseyHillers I have finished parts of the backend. do you mind implementing the frontend part? please let me know if you need anything or find any problems.

@keyonghan this still requires a call that can return the current branches to the frontend.
Such as calling /api/public/get-branches would return a json version of List<String> like ["master", "stable", "beta", "dev", "v0.0.1"] etc so the frontend can show these branches.

I've added mention of Gold and branching above, I'll also open and link an individual issue around branching support for goldens.

I've updated the frontend changes with the specific steps left to get branching supported.

Marking this as solved.


Use optimised version of our logo (badge.png)

Use optimised version of badge image.
From 8kb to 2kb.
Related: https://github.com/auth0/lock/pull/386

@ramasilveyra merged, thx!


program to merge two sorted linked list

Please accept my merge request

Only provide solutions of the given problems.


New version: JetBrains.PyCharm.Community.EAP version 222.3345.40 [FP-O]

Result: Installation Successful

Manifest
Add/Remove Programs

Name
PyCharm Community Edition (EAP)
PyCharm Community Edition 222.3345.40

Version
222.3345.40
222.3345.40

Publisher
JetBrains s.r.o.
JetBrains s.r.o.

ProductCode
PyCharm Community Edition 222.3345.3
PyCharm Community Edition 222.3345.40

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 2737

/azp run


"Convert to using super parameters" correction in the analysis server defaults to null with required List arguments

When I have a constructor like this:
  _MenuBarMenuRenderWidget({
    Key? key,
    required List<Widget> children,
  }) : super(key: key, children: children);

The analysis server suggests converting to using super parameters (correctly), but when I apply the fix, I get:
  _MenuBarMenuRenderWidget({
    super.key,
    required super.children = null,
  });

Which is incorrect, since "Required named parameters can't have a default value.", and because a default for a non-nullable List shouldn't be null.
Dart version: Dart SDK version: 2.18.0-30.0.dev (dev) (Mon Apr 18 01:06:35 2022 -0700) on "linux_x64"

What does the constructor in the superclass of _MenuBarMenuRenderWidget look like?

it is:
  _MenuBarMenuRenderWidget({
    required super.children,
  });

and it's superclass is:
  MultiChildRenderObjectWidget({ super.key, this.children = const <Widget>[] });

Here's a reproducible standalone example:
class Key {}

class Widget {
  Widget({this.key});

  final Key? key;
}

class Base extends Widget {
  Base({ super.key, this.children = const <Widget>[] });

  List<Widget> children;
}

class Child extends Base {
  Child({Key? key, required List<Widget> children}) : super(key: key, children: children);
}

When I convert Child, it gives me:
class Child extends Base {
  Child({super.key, required super.children = null});
}

Which is incorrect.

Thanks, I can reproduce the bug now.


Separate chains, network and connectors

Currently we need to initialize a connector with an specific network, and then pass this connector, together with a list of chains in order to create the SorobanReactProvider.
This is not only confusing, but also causes the problem that when the user might have its connector (Freighter) connected to another network, he/she will be forced to sign a message in the wrong network.
We should pass a list of allowed chains when creating the SorobanReactProvider, and when creating the provider, initialize it if the network given by the connector is allowed by our app, otherwise should break with an error.
This will also allow the creation of multichain (testnet & mainnet) dapps

Ready


"AttributeError: pydantic_model" when response_model is union

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

        # Also for simplicity we just don't relay all dunder names
        if '__origin__' in self.__dict__ and not _is_dunder(attr):
            return getattr(self.__origin__, attr)
>       raise AttributeError(attr)
E       AttributeError: __pydantic_model__

../../../../../.pyenv/versions/3.9.9/lib/python3.9/typing.py:710: AttributeError

Operating System
macOS
Operating System Details
No response
FastAPI Version
0.81.0
Python Version
3.9.9
Additional Context
It also does not work in FastAPI 0.7* and the pydantic version (1.9 vs 1.0) has no influence, either.

Here is a test with pure pydantic which shows the behaviour.
ValidationError seems not to support a union of models for the model parameter.
def test_error_from_union():
    class One(BaseModel):
        val: int

    class Two(BaseModel):
        val: int

    with pytest.raises(ValidationError) as err_one:
        One()

    with pytest.raises(ValidationError) as err_two:
        Two(val="abc")

    model = typing.Union[One, Two]

    error = ValidationError([err_one, err_two], model=model)
    assert error.errors()

It looks like a bug.
Validation is just fine, but if any validation errors occured, this line (137) in routing.py will be executed:
if errors:
        raise ValidationError(errors, field.type_)

and at that time, field.type_ is exactly typing.Union[One, Two]
This is then passed to Pydantic's error_wrappers.py:
def errors(self) -> List['ErrorDict']:
        if self._error_cache is None:
            try:
                config = self.model.__config__  # type: ignore
            except AttributeError:
                config = self.model.__pydantic_model__.__config__  # type: ignore
            self._error_cache = list(flatten_errors(self.raw_errors, config))
        return self._error_cache

and we have AttributeError because typing.Union don't have __config__ or __pydantic_model__ attributes.

So it's more of Pydantic issue you mean to say?

Hard to say, but I would say it's FastAPI's problem.
I'm still new to FastAPI, so I don't understand it well.
Here's why I think it's a problem with FastAPI
In this case:
class One(BaseModel):
    value: int

class Two(BaseModel):
    val: int

@app.get("/", response_model=typing.Union[One, Two])
def home():
    return {"valuee": 1}  # wrong

response is properly validated (proper errors are generated):

but field.type_ doesn't have __config__ (nor __pydantic_model__) attribute and this is the cause of the problem (Pydantic is expecting it)

on the screenshot you can see that the type_.__config__ attribute is missing.
On the other hand, in this case (I would say it's similar, because response_model is an object from typing, but the difference is that it takes single argument):
class One(BaseModel):
    value: int

class Two(BaseModel):
    val: int

@app.get("/", response_model=typing.List[One])
def home():
    return {"valuee": 1}  # also wrong, but response_model is typing.List

response is also properly validated (proper errors), but field.type_ contains __config__ attribute:

and there's no exception from Pydantic - it's working well.
These fields are created by create_cloned_field so I would be looking there for an answer.
Another difference that I noticed that feels like potential clue is that in case of typing.List, type_ is a class variable, but in a case of typing.Union, type_ is an attribute of an instance (if the debugger is not tricking me).
TLDR:
If response_model=typing.List[SomeModel], the ModelField in serialize_response has attribute type_.__config__, but if response_model=typing.Union[SomeModel1, SomeModel2], it hasn't.
In case of errors, field.type_ is passed to pydantic.ValidationError which expects field.type_ to have __config__ attribute.
It hasn't, so it raises AttributeError and client receives response with status code 500.

Is this a duplicate of https://github.com/tiangolo/fastapi/issues/3772 ?

@JarroVGIT thanks dir bringing it up. Yes, it is.


Does not work on Motorola Nexus6

I am seeing this FATAL exception after boot saying:
java.lang.RuntimeException: Unable to instantiate receiver com.example.android.directboot.BootBroadcastReceiver: java.lang.ClassNotFoundException: Didn't find class "com.example.android.directboot.BootBroadcastReceiver" on path: DexPathList

Was this ever resolved? I am having the same issue on Nexus 5X. @DirajHS please let me know if you ever found the solution to this.

I am closing this issue/PR, as it has been migrated to the new repo linked above in the comments. Thank you!


Debrief not loading

Once the final round has finished and the debrief starts, this is all that displays on all devices:

Fixed by commit 195a2e7e2c2ed9f925c298f71729bf36557062b1.


feat: Support focus to checkbox, radio, button-toggle and slide-toggle

Closes #1131
R: @jelbourn

LGTM

@tinayuangao can you rebase this PR?


Cloudflare Plugin

Is your feature request related to a problem? Please describe.
Implement a Cloudflare plugin for the OpenCost Plugins ecosystem
Additional context
Cloudflare billing data available here: https://developers.cloudflare.com/api/operations/user-billing-history-(-deprecated)-billing-history-details
Example PR with an implemented Plugin: https://github.com/opencost/opencost-plugins/pull/41
This issue qualifies for our bounty program, read more details here: https://www.opencost.io/blog/introducing-opencost-plugins

Hi Robin,
First off, I assigned you this ticket so nobody duplicates this work. hope that is OK.
Second of all, I think your idea for interpolated costs is fine - if you can adjust the description or something to append - INTERPOLATED or something to that effect so that people know what you are doing.


Bug: Github create flow

Edge case that we hit today: We create a user without an email when frontend sends us data. Then, we go fetch that user's info from Github. In cases where that user returns an email address and it's already in our system, we get a unique constraint error on database.

@asood123 is this still unsolved?


Fix gcc 7 fallthrough warnings

When compiling with gcc 7 the current fallthrough comments
are not accepted. Moving the comments outside of the block
solves the problem.
JerryScript-DCO-1.0-Signed-off-by: Peter Gal<EMAIL_ADDRESS>

The problem with this PR is that the vera checker expects the comment at its original place...

Do we need to update vera as well? Btw why gcc changed its policy? Is it a bug in gcc?

@zherczeg AFAIK they simply introduced a stricter checker for the fallthrough things. And yeas we nee do update vera also. Another idea would be to disable the gcc fallthrough check.

I've been digging around and it seems it's really a gcc bug

@zherczeg, @LaszloLango  I've update the PR to disable the check in gcc for now.

Still LGTM


Fixing error when plan is updated and mutateOnUpdate deletes ClusterServicePlanRef value

This PR is a

[X] Bug Fix

What this PR does / why we need it:
When plan is updated for a service instance, there is a clean up of ClusterServicePlanRef for the new serviceInstance object at function mutateOnUpdate:
https://github.com/kubernetes-sigs/service-catalog/blob/b5a5c4bdb6f4bcf13171476113467feb274de1f7/pkg/webhook/servicecatalog/serviceinstance/mutation/handler.go#L137
Having this ClusterServicePlanRef nil, will make syncLabels function fail.
Which issue(s) this PR fixes

Fixes #
Merge Checklist:

[ ] New feature

[ ] Tests
[ ] Documentation

[ ] SVCat CLI flag
[ ] Server Flag for config

[ ] Chart changes
[ ] removing a flag by marking deprecated and hiding to avoid
breaking the chart release and existing clients who provide a
flag that will get an error when they try to update

/assign @piotrmiskiewicz

/check-cla

/ok-to-test

/retest

/lgtm

Hi, @piotrmiskiewicz @mszostok @jberkhahn , Is it ready to merge? could you please take a look?

/approve


Feature/major refactor

This PR merges in the beta version of gFTL 2.0.   It is backward compatible,  but users that wish to try the 2.0 interfaces can link against GFTL::gftl-v2.  Or alternatively, users that wish to insulate themselves from the eventual release where v2 becomes the default library, can update now to link against GFTL::gftl-v1.

CI tests are apparently failing due to an issue on the CI end with OS X.   Tests pass in a local OSX environment.


Updating a class from BQ to PQ throws the error: invalid hnsw config: more than a single compression methods enabled

When creating a class with BQ enabled:
collection = client.collections.create(
            name="TestClass",
            vector_index_config=wvc.Configure.VectorIndex.hnsw(quantizer=wvc.Configure.VectorIndex.Quantizer.bq()),
 )

if we go ahead and update the class to set a new compression method, for example PQ:
collection.config.update(
     vectorizer_config=wvc.Reconfigure.VectorIndex.hnsw(quantizer=wvc.Reconfigure.VectoriIndex.Quantizer.pq()),
)

Then the update method will fail with:
Error updating class 'TestClass': Collection configuration may not have been updated.! Unexpected status code: 422, with response body: {'error': [{'message': 'rpc error: code = Internal desc = updating schema: TYPE_UPDATE_CLASS: bad request :parse class update: parse vector index config: parse vector index config: invalid hnsw config: more than a single compression methods enabled'}]}.

It seems that when updating the vectorizer_config to enable a different compression method, the previous one doesn't get its flag set to false, leaving it to Weaviate to try to set two compression methods at the same time.

Moving this issue to the server: https://github.com/weaviate/weaviate/issues/5311


library compile error - rollup -c -w

Forked repo, cloned and installed dependencies. Running yarn dev / build produces this error.
What is this refering to? I'm intending to submit a PR, however error is preventing this.

Hi @trueChazza
Sorry for the slow reply. I've just run an update on some of the dependencies thanks to #400 - perhaps give it another go now if you're still interested in submitting a PR.
Cheers

Thankyou @rowanwins for the great library. I will give it another go and submit PR

Hi @trueChazza , just pushed up a commit with everything now lined up so now would be a great time for a PR :)


Speed-up the validate phase

Hi,
I'm training your model using a different target space.
I'm noticing a very long computation time in the validate-typing-dev-# phase (15 minutes per epoch when the target space has 63 labeled vectors, 48 minutes when the types are approximately 200).
I know that in this phase the pyFLANN library is used an I think that is the Nearest Neighbor algorithm that slows down this phase.
Since the original dataset presents more than 8k types in the target space I think that you speed up this phase (or maybe I'm missing something in the code, but I didn't touch anything in the validation code) so I am here to ask if there is some method to speed up the training phase.
Thanks

Hi,
In my case, with more than 8k labels, the training time of one epoch was 14 minutes, and the validation time was 7 minutes approximately.
These times are from runs using one P40 GPU and at least 64 GB of RAM memory in the server. If you are not using GPUs to train the model, or you can't fit pyFLANN in memory, things will be much slower.


[WIP] Fix proposal engine (minor things)

I was going to do this in the geometry PR but my git-fu is not as strong as I would like. This should be a quick PR to shore up some issues identified in the course of #92 .

Deleted PolymerTopologyProposal as suggested in #99

Added the new_chemical_state_key and old_chemical_state_key to the TopologyProposal and adjusted SmallMoleculeSetProposalEngine accordingly.

Merging this so that other fixes to the ProposalEngines can be PR'd.


[Bug]: 404 Page no found on link in Styles Specifications

Relevant scope
Frontend specs
Description
In https://realworld-docs.netlify.app/specifications/frontend/styles/ , a link pointing to "templates & info required to get up and running" is not found.

@geromegrignon @ngterry this error related for all Markdown links containing .md extensions. I've a solution that should fix this bug. https://github.com/gothinkster/realworld/pull/1580


pluggable ruletypes

No longer assume JVM rules (java_library) are the only supported rule types.

This is a big step towards Issue #8.


[WIP] Create a Caching Artefact Retriever with error handling

The format routing constraint didn't handle errors coming back from the
content API. This introduced potential issues around content API timeouts
which wouldn't be cached and therefore could trigger a cascading error in
Frontend.
Here we introduce a new class which is responsible for memoising the
response from the content API, and catches any errors. Future requests
will return early if an error was encountered, saving us a long wait.
The error-caching behaviour makes this new class usable in our controllers
too, where we can rethrow or interrogate the exception and handle it
accordingly.

Should we add some sort of integration tests? There are some for checking timeouts in page_rendering_test.rb. We could have similar ones for each refactored format (sort of like Rspec shared examples), since the page_rendering_test.rb will disappear once we're done refactoring... ?

@davidbasalla :eyes:?

With this commit we can now remove require 'application_controller' from https://github.com/alphagov/frontend/blob/master/test/unit/artefact_retriever_test.rb#L2


mysql version admin user login problem

when deploying umami with mysql we can not login using default admin/umami user password in the mysql migration this line is missing https://github.com/mikecao/umami/blob/1141b6b7b062ad82bde4a2d367c0833fa2ac0401/db/postgresql/migrations/01_init/migration.sql#L132
@mikecao

Fixed in v1.33.2


🐛 [BUG]: Astro plugin fails with Nx 19.1.2

Is there an existing issue for this?

[X] I have searched the existing issues

What package does the issue affects?
@nxtensions/astro
Describe the bug
Enabling this plugin with Nx 19.1.2 causes Nx to fail. This breaks Nx completely and every command returns a Failed to process project graph. error. This affects versions 19.1.2 up to latest(19.2.0).
Opening the Nx Graph in VS Code shows:
 The "@nxtensions/astro" plugin threw an error while creating dependencies:
    TypeError: targetProjectLocator.findProjectWithImport is not a function
      at .../node_modules/@nxtensions/astro/src/project-graph/create-dependencies.js:55:55
      at TypeScriptImportLocator.fromNode (.../node_modules/nx/src/plugins/js/project-graph/build-dependencies/typescript-import-locator.js:41:17)
      at .../node_modules/nx/src/plugins/js/project-graph/build-dependencies/typescript-import-locator.js:80:53
      at visitNodes (.../node_modules/typescript/lib/typescript.js:28321:24)
      at forEachChildInSourceFile (.../node_modules/typescript/lib/typescript.js:28964:18)
      at Object.forEachChild (.../node_modules/typescript/lib/typescript.js:28406:37)
      at TypeScriptImportLocator.fromNode (.../node_modules/nx/src/plugins/js/project-graph/build-dependencies/typescript-import-locator.js:80:18)
      at collectDependencies (.../node_modules/@nxtensions/astro/src/project-graph/create-dependencies.js:52:23)
      at collectDependencies (.../node_modules/@nxtensions/astro/src/project-graph/create-dependencies.js:71:15)
      at Object.createDependencies (.../node_modules/@nxtensions/astro/src/project-graph/create-dependencies.js:30:19)

Removing the plugin from the plugins option in the nx.json file fix it.
"plugins": [
    "@nxtensions/astro",
]

Expected behavior
Nx should be able to build the Astro project
Steps to reproduce

Create a new Nx workspace using the @nxtensions/astro plugin

npx create-nx-workspace@latest --preset=@nxtensions/astro

Link to minimal reproducible example
No response
Environment
Failed to process project graph. Run "nx reset" to fix this. Please report the issue if you keep seeing it.

Running
> nx --version

returns
Nx Version:
- Local: v19.1.2
- Global: v19.2.0

findProjectWithImport was renamed to findProjectFromImport in https://github.com/nrwl/nx/pull/23307/files#diff-6120d1e1d8228cc6caee07dae7c2c4fa47b74095d60f51797cc2c5b47dc6f1a9R76 😮💨

Would it be possible to Review #646 so this one can be closed ?


Implement dart builder

As discussed in #1

looks good and works mostly as expected.
It should be mentioned in the README that all assets have to be placed inside of the lib folder to make regenerating the assets.dart possible.
This should be adjusted for the fonts as well in the example.
From what I read in the documentation it should not be a problem to have assets inside of the lib folder.

Updated as requested.

thank you very much for your hard work.
I will release this version as the version 0.2 to pub.dev

Cool, thanks. I just realized readme contains references to my fork, please change these before publishing.


Feature request: ALLUPPERCASE

I want to use cppo for improving my ISO/IEC 13211-1:1995 (a.k.a "ISO-Prolog") compliant Prolog processor.
Specifically, I want to reduce the use of string constants in code blocks like the following one:

type plOpSpecifier = (* pre-fix *) FX|FY | (* in-fix *) XFX|XFY|YFX | (* post-fix *) XF|YF;;

let plOpSpecifier_of_string_opt = function
  |  "fx" -> Some  FX
  |  "fy" -> Some  FY
  | "xfx" -> Some XFX
  | "xfy" -> Some XFY
  | "yfx" -> Some YFX
  | "xf"  -> Some XF
  | "yf"  -> Some YF
  | _____ -> None;;

let plAtom_of_plOpSpecifier = function
  |  FX -> `Atom "fx"
  |  FY -> `Atom "fy"
  | XFX -> `Atom "xfx"
  | XFY -> `Atom "xfy"
  | YFX -> `Atom "yfx"
  | XF  -> `Atom "xf"
  | YF  -> `Atom "yf";;

So far, I have come up with the following:

#define X(x)  STRINGIFY(x) -> Some(CAPITALIZE(x))
  let plOpSpecifier_of_string_opt = function
    | X(fx) | X(fy) | X(xfx) | X(xfy) | X(yfx) | X(xf) | X(yf)
    | _ -> None;;
#undef X

#define X(x)  CAPITALIZE(x) -> `Atom(STRINGIFY(x))
  let plAtom_of_plOpSpecifier = function
    | X(fx) | X(fy) | X(xfx) | X(xfy) | X(yfx) | X(xf) | X(yf);;
#undef X

Alas, CAPITALIZE only touches the first character, so I get the following not-yet-perfect output:

$ cat sample.txt | cppo | ocamlformat --enable-outside-detected-project - --impl
let plOpSpecifier_of_string_opt = function
  | "fx" -> Some Fx
  | "fy" -> Some Fy
  | "xfx" -> Some Xfx
  | "xfy" -> Some Xfy
  | "yfx" -> Some Yfx
  | "xf" -> Some Xf
  | "yf" -> Some Yf
  | _ -> None

let plAtom_of_plOpSpecifier = function
  | Fx -> `Atom "fx"
  | Fy -> `Atom "fy"
  | Xfx -> `Atom "xfx"
  | Xfy -> `Atom "xfy"
  | Yfx -> `Atom "yfx"
  | Xf -> `Atom "xf"
  | Yf -> `Atom "yf"

I need all characters in uppercase. Having a ALLUPPERCASE builtin function in cppo would do it.
I wanted to start the discussion on this before filing a PR:) Comments?

Although such tricks certainly make it harder to make mistakes when writing code, I would suggest that a PPX is probably a better mechanism here than hacking in cppo. In particular, code refactoring tools and formaters are broken by preprocessors like cppo.


Bug#1 - Post request to /encoder results in an internal server error

OS: Windows 10
Steps to reproduce:
Make a post request to /encoder using the following body:
{
"serialNumber": 2
}
Expected result:
The body should be populated in the database and the server should return a successful response.
Actual result:
Return of an internal server error.
Visual proof:

The payload for a post request should be something like this
{   "displayName": "Camera 1B",   "serialNumber": "BtmC8ckj" }
But even then that should return you a 400 instead of a 500

@FGRCL with our current DB structure the device entity has all the details, the encoder/decoder has only a serial number that references to the device(displayName is an attribute of the device)

Bug fixed in https://github.com/bean-pod/switchboard/commit/86b34ab3cf29fc77f6884af2b219cddb51efecbf


Implement arbitrage using multi-hop routing

Form a path from Penumbra -> Penumbra, and find out whether the optimal price is <1
If so, execute, and burn the profit

Should happen after #1724

Closed by #2539


python3.12 AttributeError: 'zlib._ZlibDecompressor' object has no attribute 'unconsumed_tail'

Issue in python3.12
Python version
❯ python3.12 --version
Python 3.12.0b4

Error
            uncompress = self._decompressor.decompress(buf, size)
>           if self._decompressor.unconsumed_tail != b"":
E           AttributeError: 'zlib._ZlibDecompressor' object has no attribute 'unconsumed_tail'

pgzip/pgzip.py:721: AttributeError

FWIW, I got the original mgzip code working on Python 3.12 with this patch, in case its helpful for others running into this issue.
When I say "working" I mean my test cases that de/compress data are passing. There may be corner cases I haven't run into yet.
try:
    from zlib import _ZlibDecompressor
except ImportError:
    # Python <= 3.11
    pass
else:
    # Python >= 3.12
    #
    # This version has changed from internally using
    # ``zlib.decompressobj`` to using the new
    # ``zlib._ZlibDecompressor`` class, which has almost, but not
    # quite, the same API. We:
    # - implement an API compatible version of _ZlibDecompressor;
    # - patch __init__ to use this instead of the original.

    # pylint:disable=protected-access

    # We are not allowed to use _ZlibDecompressor as a base class,
    # so we have to proxy to it. This slows us down some.
    class _MgzipDecompressor:
        unconsumed_tail = b''

        def __init__(self, *args, **kwargs):
            self.__backing = _ZlibDecompressor(*args, **kwargs)

        def __getattr__(self, name):
            return getattr(self.__backing, name)

    _orig_init = mgzip.multiProcGzip._MulitGzipReader.__init__
    def _mgzip_init_(self, *args, **kwargs):
        _orig_init(self, *args, **kwargs)
        assert self._decomp_factory is _ZlibDecompressor
        self._decomp_factory = _MgzipDecompressor
        self._decompressor = self._decomp_factory(**self._decomp_args)
    mgzip.multiProcGzip._MulitGzipReader.__init__ = _mgzip_init_

    # This method has been inlined in 3.12. It is only called from the end
    # of read()
    if not hasattr(mgzip.multiProcGzip._MulitGzipReader, '_add_read_data'):
        import zlib
        def _add_read_data(self, data):
            self._crc = zlib.crc32(data, self._crc)
            self._stream_size += len(data)
        mgzip.multiProcGzip._MulitGzipReader._add_read_data = _add_read_data

def _mgzip_read_exact(self, n):
    # Taken from Python 3.10. Their comment:
    #
    # Read exactly *n* bytes from `self._fp`
    # This method is required because self._fp may be unbuffered,
    # i.e. return short reads.
    #
    # mgzip relies on this internally right now, and so doesn't
    # work with 3.11 or 3.12. We patch it in.
    # pylint:disable=protected-access
    data = self._fp.read(n)
    while len(data) < n:
        b = self._fp.read(n - len(data))
        if not b:
            raise EOFError("Compressed file ended before the "
                           "end-of-stream marker was reached")
        data += b
    return data

if not hasattr(
    # Note typo.
    # pylint:disable-next=protected-access
    mgzip.multiProcGzip._MulitGzipReader,
    '_read_exact'
):
    # pylint:disable-next=protected-access
    mgzip.multiProcGzip._MulitGzipReader._read_exact = _mgzip_read_exact

See #37

@jamadden Can you please fix this problem for pgzip.


Fix reference search index

addresses #458
This PR fixes an issue where methods with the same name were overwriting each other in the search index due to ignoring class names. This PR does the following:

Adds method class names into relevant reference search indices
Adds parentheses to methods in the search index for a more consistent appearance across the site
Adds aliases to reference items that have a class or parentheses so they still rank high in searches without those elements included
Adds a set of unit tests that ensures the above

Will request review after CI build finishes

Also, I was hoping to add @meezwhite as a reviewer but the repository settings don't allow review requests from people without a maintainer role.
I requested a review from three maintainers just as a way of flagging. One reviewer is likely enough. Thanks!

Thanks @stalgiag!

Looks great! Thanks @stalgiag!


Error Message once in a while

Bug/Issue Description
As I review my cards an error message pops up as soon as I rate an card (happens about every 10th or so card)
To Reproduce
don't know any other occasion than just reviewing..
Screenshots/Visuals
Log pasted
System/Debug Info
Anki 2.1.54 (b6a7760c) Python 3.9.7 Qt 5.14.1 PyQt 5.14.1
Platform: Mac 12.6.1
Flags: frz=True ao=True sv=3
Add-ons, last update check: 2022-12-01 10:22:33

===Add-ons (active)===
(add-on provided name [Add-on folder, installed at, version, is config changed])
AMBOSS add-on ['0amboss_addon', 2022-06-27T18:14, 'None', mod]
AnKing Note Types Easy Customization ['952691989', 2022-11-22T19:37, 'None', '']
Anki Simulator ['817108664', 2021-08-07T21:24, 'None', '']
Anki Zoom ['538879081', 2020-04-04T02:20, 'v1.1.3', '']
AnkiWebView Inspector ['31746032', 2022-04-08T21:44, 'None', '']
Asset Manager ['656021484', 2021-08-29T23:07, 'None', '']
BetterSearch ['1052724801', 2022-01-25T23:25, 'None', '']
Closet For Anki ['272311064', 2022-08-12T10:27, 'None', '']
Cloze Overlapper for Anki 21 ['109531687', 2021-01-12T04:12, 'None', '']
Copy notes ['1566928056', 2021-12-24T20:56, 'None', '']
Cross Profile Search and Import ['310394744', 2019-05-28T18:15, 'None', '']
Custom Background Image and Gear Icon ['1210908941', 2022-11-28T19:08, 'None', mod]
Customize Keyboard Shortcuts ['24411424', 2022-09-14T11:12, 'None', '']
Deadline2 ['723639202', 2021-05-02T06:45, 'None', mod]
Frozen Fields ['516643804', 2020-03-01T05:12, 'None', '']
Image Occlusion Enhanced ['1374772155', 2022-04-09T09:15, 'None', '']
Leaderboard ['41708974', 2022-05-12T11:46, 'None', mod]
Learning Step and Review Interval Retention ['1949865265', 2022-08-28T22:21, 'None', '']
Leech Toolkit ['368380974', 2022-10-13T00:27, 'None', '']
Mini Format Pack ['295889520', 2018-07-24T19:45, 'None', '']
More Decks Stats and Time Left ['1556734708', 2022-10-09T14:05, 'None', mod]
Postpone cards review ['1152543397', 2021-02-13T17:47, 'None', '']
Refocus Card when Reviewing 21 ['1642550423', 2018-09-19T16:14, 'None', '']
Remaining time for Anki 21 ['1508357010', 2020-12-13T15:14, '20.12.13i151', mod]
Removes Empty Note Types for 21 ['530953383', 2019-06-12T16:11, 'None', '']
Reset Card Scheduling ['300884351', 2018-09-18T18:17, 'None', '']
Resize images in editor ['1103084694', 2021-07-19T08:35, 'None', mod]
Review Heatmap ['review_heatmap', 2020-04-30T15:23, 'None', '']
Set Card Ease ['1387847178', 2022-11-17T19:41, 'None', mod]
Special Fields ['1102281552', 2022-11-22T02:49, 'None', mod]
Speed Focus Mode auto-alert auto-reveal auto-fail ['1046608507', 2019-06-02T11:11, 'None', '']
Straight Reward ['957961234', 2022-03-31T21:37, 'None', '']
Symbols As You Type ['2040501954', 2022-06-08T06:09, 'None', '']
Transfer scheduling data from one card to another ['94685914', 2020-09-18T11:23, 'None', '']
True Retention by Card Maturity ['923360400', 2017-08-29T21:00, 'None', '']
Ze Add Note Id ['886709118', 2021-03-02T21:06, 'None', '']
ZzLegacy Spelling Police ['1410276506', 2020-03-03T03:13, 'None', mod]
autoLapseNewInterval  automatically adjust the new interval after a lapse ['372281481', 2022-09-28T09:26, 'None', mod]

===IDs of active AnkiWeb add-ons===
1046608507<PHONE_NUMBER> 109531687<PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER><PHONE_NUMBER> 24411424 272311064 295889520 300884351 310394744 31746032 368380974 372281481 41708974 516643804 530953383 538879081 656021484 723639202 817108664 886709118 923360400 94685914 952691989 957961234

===Add-ons (inactive)===
(add-on provided name [Add-on folder, installed at, version, is config changed])
Advanced Review Bottom Bar ['1136455830', 2022-09-29T15:31, 'None', mod]
Auto Ease Factor ['1672712021', 2021-03-21T19:11, 'None', mod]
ImageResizer ['1214357311', 2022-07-15T15:05, 'None', '']
Legacy ReMemorize ['323586997', 2020-11-20T06:27, 'None', mod]
Speed Focus Mode ['speed_focus_mode', 2019-06-29T00:35, 'None', '']
True Retention ['613684242', 2017-11-19T21:43, 'None', '']

Additional Context
Error Log:
Fehler
Es ist ein Fehler aufgetreten.

Debuginformationen:
Anki 2.1.54 (b6a7760c) Python 3.9.7 Qt 5.14.1 PyQt 5.14.1
Platform: Mac 12.6.1
Flags: frz=True ao=True sv=3
Add-ons, last update check: 2022-12-01 10:22:33
Möglicherweise beteiligte Erweiterungen: Leech Toolkit
Caught exception:
Traceback (most recent call last):
File "aqt.taskman", line 122, in _on_closures_pending
File "aqt.taskman", line 71, in
File "aqt.taskman", line 90, in wrapped_done
File "aqt.operations", line 125, in wrapped_done
File "aqt.reviewer", line 437, in after_answer
File "aqt.reviewer", line 450, in _after_answering
File "aqt.hooks_gen", line 3345, in call
File "/Users/Winkel/Library/Application Support/Anki2/addons21/368380974/src/reviewer.py", line 229, in on_answer_v3
self.on_answer(context, card, ease)
File "/Users/Winkel/Library/Application Support/Anki2/addons21/368380974/src/reviewer.py", line 368, in on_answer
self.handle_card_updates(card, handle_card_answer)
File "/Users/Winkel/Library/Application Support/Anki2/addons21/368380974/src/reviewer.py", line 287, in handle_card_updates
changes = self.reviewer.mw.col.merge_undo_entries(last_step)
File "anki.collection", line 1004, in merge_undo_entries
File "anki._backend.generated", line 1756, in merge_undo_entries
File "anki._backend", line 146, in _run_command
anki.errors.InvalidInput: Ungültige Eingabe: target undo op not found
:)

As I review my cards an error message pops up as soon as I rate an card (happens about every 10th or so card)

Ope, sorry you're running into this!
I created a quick patch that might fix the issue here:
https://github.com/iamjustkoi/LeechToolkit/releases/download/v1.1.1-a1/LeechToolkit.ankiaddon
Defs let me know how that goes! Will continue to try and look into it more on my end, though, too.

Thank you so much for the reply & quick fix!
I tried it but unfortunately the issue still persists :)
But as far as I can tell it happens far less (like every 40th-50th card)
Apart from that the error message changed a tiny bit:
I highlighted the differences in bold text as far as I spotted them.
I also noticed that the (most recent) error message ist always the same:
[…]
Debuginformationen:
Anki 2.1.54 (b6a7760c) Python 3.9.7 Qt 5.14.1 PyQt 5.14.1
Platform: Mac 12.6.1
Flags: frz=True ao=True sv=3
Add-ons, last update check: 2022-12-02 10:38:19
Möglicherweise beteiligte Erweiterungen: Leech Toolkit
Caught exception:
Traceback (most recent call last):
File "aqt.taskman", line 122, in _on_closures_pending
File "aqt.taskman", line 71, in
File "aqt.taskman", line 90, in wrapped_done
File "aqt.operations", line 125, in wrapped_done
File "aqt.reviewer", line 437, in after_answer
File "aqt.reviewer", line 450, in _after_answering
File "aqt.hooks_gen", line 3345, in call
File "/Users/Winkel/Library/Application Support/Anki2/addons21/368380974/src/reviewer.py", line 229, in on_answer_v3
self.on_answer(context, card, ease)
File "/Users/Winkel/Library/Application Support/Anki2/addons21/368380974/src/reviewer.py", line 379, in on_answer
self.handle_card_updates(card, handle_card_answer)
File "/Users/Winkel/Library/Application Support/Anki2/addons21/368380974/src/reviewer.py", line 295, in handle_card_updates
changes = self.reviewer.mw.col.merge_undo_entries(last_step)
File "anki.collection", line 1004, in merge_undo_entries
File "anki._backend.generated", line 1756, in merge_undo_entries
File "anki._backend", line 146, in _run_command
anki.errors.InvalidInput: Ungültige Eingabe: target undo op not found
Another thing I noticed:
When I select a leech to Un-leech it, a small window ('conf') pops up (see attachment)

So far amazing addon with impressive depths! :o

Hmmm I tried multiple times but a .txt. file is not being created :/
I wonder why nobody else mentioned anything like that yet ':D

Hmmm I tried multiple times but a .txt. file is not being created :/

Oh, actually, it not working is a pretty good clue, too! Made another edit, this time it should output a text file, otherwise I can't think of much else I can do on my end, unfortunately.

I wonder why nobody else mentioned anything like that yet ':D

Yeah, may end of being OS-specific or something more unique to your install. Hopefully nothing too bad, though.
(https://drive.google.com/file/d/103Bio5kbizH7znIh1BiqF10uTHtHUgeK/view?usp=sharing)
If that doesn't end up making anything though, you might also want to try going to wherever you have Anki's launcher and launching it with the console version, instead (probs a bat, or sh, or something else with console in its name). It might give a bit more of a clue for what's happening when you run the Un-Leech/Leech action in the browser.
Oh, also, just double checking if the first issue was solved or not, just in case? Will probs move this to a new thread for better tracking.

KeyError: 'conf'

Nicely done, thought it was something like this! Thanks for sticking with it!
I'll see what I can do about building a fix for it as soon as I can.

I will know tomorrow as I do my reviews and see if the error pops up again :)
But tbh if I get an error message every 40th to 50th card it's not the end of the world ';D

Haha, sounds good, will still try to fix up as much as I can on my end! 👍

KeyError: 'conf'

Also gonna move this one over to a new thread (#6).

I will know tomorrow as I do my reviews and see if the error pops up again :)

I reviewed around 700 cards and not a single error msg popped up! Guess it's fixed then, good job! 👍

I reviewed around 700 cards and not a single error msg popped up! Guess it's fixed then, good job! 👍

Actually I'm pretty sure this is causing issue #7 now, to be honest!
Gonna keep the thread open until I can push the fix to the main branch/schedule for release, for the time being.
Appreciate it, either way! 😄

Scheduled for the next release!
Should be fully resolved in this most recent hot-fix: https://github.com/iamjustkoi/LeechToolkit/releases/download/v1.1.1-a5/LeechToolkit.ankiaddon

Thanks again for reporting in!


refactor(material/schematics): do not print hammer migration message when updating to v13

The Hammer gesture migration runs only for v9 and v10 but the global
post migration messages are printed regardless of whether the update
runs for v9 or v10. This commit fixes that by providing the target
version to the globalPostMigration functions for migrations.

@wagnermaciel Done


Tweak English locale;

Closes #nothing.
Changes proposed in this pull request:

fix a typo me => be
spell out 3 in letters
0 is plural in English

Pull request checklist:

[ ] branch is rebased on master *
[ ] proper commit messages *
[ ] proper coding style *
[ ] code is properly tested
[ ] tests are passing
[ ] document API changes (optional)
[ ] document migration notes (optional)
[ ] reviewer assigned (@marienfressinaud)
[x] i haven't done anything on this list because i'm terrible at open-source (and this only touches a locale, so it's pretty safe)

@marienfressinaud Can you have a look, please? Thanks!
* Additional information in the documentation.

and this only touches a locale, so it's pretty safe

almost ;) Can you amend your commit so the message matches the required format? :p I'll have a look at the CI later :/

Hmm, do you consider this to be a fix (because of the typo) or an improvement (because of the explicit spelling of 3)? :neutral_face:

It is more likely an improvement (there is no strong rules for that but I use fix for bugs)

Merged in #194


gitlab.com/gitlab-org/cli has changed the asset file names

package name
gitlab.com/gitlab-org/cli
aqua version
Please use the latest version.
aqua version 2.36.1 (423bf97060599e911f9a5a2c5622cf886673dd65)

Environment

OS (Windows, Linux, macOS, etc): macOS
CPU Architecture (amd64, arm64, etc): arm64

Overview
https://gitlab.com/gitlab-org/cli/-/merge_requests/1701
v1.47.0 is the first release with the updated asset file names: https://gitlab.com/gitlab-org/cli/-/releases/v1.47.0
How to reproduce
Install<EMAIL_ADDRESS>Debug Output
Please set the environment variable AQUA_LOG_LEVEL=debug and run aqua and tell us the debug output.
ERRO[0000] install the package                           aqua_version=2.36.1 download_url="https://gitlab.com/gitlab-org/cli/-/releases/v1.47.0/downloads/glab_1.47.0_macOS_arm64.tar.gz" env=darwin/arm64 error="download a package: status code >= 400" http_status_code=404 package_name=gitlab.com/gitlab-org/cli package_version=v1.47.0 program=aqua registry=standard

Expected Behaviour
GitLab CLI installs successfully
Actual Behaviour
GitLab CLI fails to install because the expected asset doesn't exist.
Important Factoids
References

Thank you for reporting the issue.
I'll take a look.

https://github.com/aquaproj/aqua-registry/pull/27469

Fixed.
v4.228.1 is out 🎉
https://github.com/aquaproj/aqua-registry/releases/tag/v4.228.1


Update versioning in nodeos

#7468 added better version reporting to cleos.

Add similar capability to nodeos.
Add a --full-version option to nodeos that reports the full version with the hash similar to cleos version full
Add a new server_full_version_string to get info RPC to report this new full version.
Modify existing server_version_string of get_info RPC to report similar as cleos version client

Resolved by #8062 (or #8072).


PowerShell v6.0.4 hangs on second (or third) run after restart (no prompt appears, just blinking cursor)

Steps to reproduce
Execute pwsh twice.
/home/local/luser$ pwsh
PowerShell v6.0.4
Copyright (c) Microsoft Corporation. All rights reserved.

https://aka.ms/pscore6-docs
Type 'help' to get help.

PS /home/local/luser> exit
/home/local/luser$ pwsh
PowerShell v6.0.4
Copyright (c) Microsoft Corporation. All rights reserved.

https://aka.ms/pscore6-docs
Type 'help' to get help.

<blinking cursor here>

Expected behavior
The prompt should appear again?
Actual behavior
Environment data
pwsh.trace.txt

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      6.0.4
PSEdition                      Core
GitCommitId                    v6.0.4
OS                             Linux 4.15.0-33-generic #36~16.04.1-Ubuntu SMP Wed Aug 15 17:21:05 UTC 2018
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

What terminal do you use?

MATE Terminal 1.16.3

You could try TERM=xterm pwsh.

Issue still persists.
On 7 Sep 2018, at 10.43, Ilya <notifications<EMAIL_ADDRESS>wrote:
TERM=xterm pwsh

remove $profile
remove powershell,
then reinstall ？


Better tslint.json Search Behavior

The current algorithm searches for the tslint.json file in parent directories of the current file being linted. It would be much more predictable if the search was instead from the current working directory.

I suggest we try and match whatever TypeScript (with tsconfig.json) and typescript-formatter (it searches for multiple files) do for consistency's sake.

I believe typescript-formatter uses the cwd as well.
As an example, let's say you ran this:
cd A/B
tslint ../C/someFile.ts  ../D/someFile.ts

and say there's a tslint.json file in A, B, and C. Which one would you expect to get used?

@JKillian - these are the rules that I had in mind:
https://github.com/vvakame/typescript-formatter#read-settings-rules

@myitcv Didn't see that, thanks. That sounds to me that it works more similarly to how things are happening now - search locations are based off of the location of the individual files, not the cwd when the command is run. Wonder how TS does things?

See here, specifically first couple of paragraphs

@myitcv Thanks, the TS algorithm is very similar to how I thought TSLint would do things. The advantage with the current algorithm is that theoretically you could have different configurations for different files, but I'm guessing people don't really use that "feature".

I'm guessing people don't really use that "feature".

For me it's actually the other way around.
Since a tslint.json can extend another one, it is very convenient to loosen or tighten the rules for certain directories.
And now that you can specify "exclude" in tslint.json to ignore files by matching patterns, this might be used like the .tslintignore file from another feature request.

By now, this is out of scope for the repository. See #4534. 😞

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


fix: updated remaining _id to id in CheckoutAction

Replaced remaining action._id with action.id in CheckoutActions component.

🚀 Preview deployed
Built with commit 75c77094765a4c2562ae34a2a14a7a1e84ed523d
https://deploy-preview-316--stoic-hodgkin-c0179e.netlify.com


Better message fields

This PR improves the way of obtaining message field information and values:

MessageFieldException is introduced, which improves diagnostics for error cases.

@armiol, PTAL.


Added testcase to test dump while running workload - v3

This test triggers crash while running cpu and memory workload ebizzy.
Signed-off-by: Pavithra<EMAIL_ADDRESS>

logs_sles_rhel.txt

@Pavithra1602 Please rebase your code

did rebase of PR

Raised https://github.com/open-power/op-test/pull/716


Update linktap to 1.0.1

Increased node version.
Updated dependencies

Please create PRs for new releaaes eitehr from a clean environment or by using www.iobroker.dev website. This PR contains too many commits (see above) so automatid tests fail...

Too many changes in this commit. Stop analysis.

replaced by #4144


Navigation bar requirements

The website should include the following bootstrap components:
- A Navigation bar

- A navigation menu at the top. Feel free to use the navbar code provided in Bootstrap's documentation, or create your own navbar by applying the correct Bootstrap classes to your HTML.

  - Include links that are applicable to your portfolio.

   - Links should navigate to the appropriate sections

Navbas done with mobile responsiveness.
Test the links before closing this

navigation not smooth. would like to customise it before submission


After a object/class method call, remove the object/class from the stack

Fixes #12081.

Those one liners (well, two) can be the worst. The hardest part of this for me will be finding the workarounds; I'll run some tests tomorrow.

Maybe it's not needed there but it's similar after https://github.com/vim/vim/blob/78378c0ea0853a8f2ea274cf2b9e6aa1e2c2dd40/src/vim9instr.c#L1927-L1930

Maybe it's not needed there but it's similar after
https://github.com/vim/vim/blob/78378c0ea0853a8f2ea274cf2b9e6aa1e2c2dd40/src/vim9instr.c#L1927-L1930

The generate_UCALL() function is used to generate instructions for calling
a function that is not yet defined.  As the function is not yet defined, the number of
arguments and their types are not yet known and the argument type checks cannot
be performed here.  So the change is not needed here.


I get the TypeError 'NoneType' object is not subscriptable

After a few "Evaluated order" messages, the script aborts with the following error message:
Fatal error
Traceback (most recent call last):
File "C:/Users/qlikadm/IdeaProjects/optimus-py/optimus-py-master/optimuspy.py", line 167, in main
optimus_result = OptimusResult(cube_name, permutation_results)
File "C:\Users\qlikadm\IdeaProjects\optimus-py\optimus-py-master\results.py", line 127, in init
self.best_result = self.determine_best_result()
File "C:\Users\qlikadm\IdeaProjects\optimus-py\optimus-py-master\results.py", line 227, in determine_best_result
process_speed_range = [result.median_process_time(result.process_name) for result in self.permutation_results]
File "C:\Users\qlikadm\IdeaProjects\optimus-py\optimus-py-master\results.py", line 227, in
process_speed_range = [result.median_process_time(result.process_name) for result in self.permutation_results]
File "C:\Users\qlikadm\IdeaProjects\optimus-py\optimus-py-master\results.py", line 73, in median_process_time
median = statistics.median(self.process_times_by_process[process_name])
TypeError: 'NoneType' object is not subscriptable

Hi @il66ezov,
sorry for the late reply.
Can you please provide the command to execute the script with all parameters?
This will help me to replicate the error

Hi,
I have the same error. The command is:
python optimuspy.py -i="Serv1" -v="Optimus" -e="10" -f="True"
I've tried to add  print (process_name) to median_process_time: it returns None
What is the meaning of --process parameter in optimuspy.py? Should it be left blank?

Hi,
we recently added a new feature to OptimusPy. Now the process runtime is also considered. In the resulting plot the size of the points in the scatterplot represents the runtime of the process.
If the process runtime is not relevant for you, you can provide a dummy process without code in it.

Hi @AlexanderDvoynev,
if fixed the issue that occurs when no process_name is provided and I updated the readme.
Thanks for re-reporting the issue. I appreciate it!
Please re-open if the issue persists after upgrading to the latest master branch.


ColorPicker: fix OKHSL circle in HSV mode

Fixes #68286.
OKHSL and HSV are different things, they should be stored separately in different fields.

In addition to fixing the issue, I think it should be allowed to use other color wheels in okhsl mode, not just okhsl circle. If possible I would like to update this PR, or I will open a new PR after this merges.

Please do it in a separate pr!

#99515 should make it easier to add new shapes.

Actually in my test, after this pr, we just need to remove the get_shape_override and _get_actual_shape in okhsl mode to let it work in other color wheels. IMO the shape override is unnecessary and this why I want to also do it in this pr.
Of course, Refactoring and cleaning are great and necessary.

Thanks! Congratulations on your first merged contribution! 🎉


Add code comments for all public structures

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."


Use git fetch depth 0 in all GH workflows

Honestly I'm just copy-pasting this from the upstream repo 😄
I don't think that this has ever become a problem in this repo, but hey, it's an almost free change.


feat: add support for request and transaction tagging to Spanner

Adds support for request and transaction tagging. Request tags are set on SpannerCommand and SpannerBatchCommand. Transaction tags are set on SpannerTransactions and are automatically included for all requests that are executed on the transaction, including the commit request. Commit requests cannot include a separate request tag.
Ephemeral transactions and read-only transactions do not support transaction tags. Commands that are executed using an ephemeral or read-only transaction can include a request tag.
Example usage:
var requestTag = "request-tag-1";
var transactionTag = "transaction-tag-1";
SpannerConnection connection = ...;
SpannerTransaction transaction = connection.BeginTransaction();
transaction.TransactionTag = transactionTag;

var command = connection.CreateSelectCommand("SELECT * FROM FOO");
command.Transaction = transaction;
command.RequestTag = requestTag;
using (var reader = command.ExecuteReader())
{
  // do something with the data
}
transaction.Commit();

cc @skuruppu

We'll look on Monday

I think the @amanda-tarafa's idea of separating the wrappers for each kind of option is neat and may help prevent users from setting options based on the context. I think the C++ team had done a similar thing for timestamp bounds which has a similar issue.

Thanks @olavloite I'll take a look tomorrow.

I'm on vacation tomorrow, but I'll look again on Monday - again, I'm assuming that as this depends on proto changes that haven't been published yet, it's not super urgent.

@amanda-tarafa @jskeet Thanks both for the valuable input. And yes, Monday is certainly no problem.

Yep, this is way cleaner now. Thanks.
A couple more tests I would like to add:

Mostly for documentation purposes, tests that check when the tags are set to null, it's really set to string.Empty. This is somewhat of an "unexpected" behaviour due only to how the underlying protos work so it'll be nice that we have tests for this so we don't inadvertently modify it in the future. Or, implement the separate CommandOptions and TransactionOptions classes that could easily hide this behaviour from client code.
A unit test for RunWithRetryableTransaction in which the user provided lambda sets the transaction tag before executing any command. Make the code retry at least once so that we can check that the lambda can always set the transaction tag, i.e. that we are not passing the old transaction instance with the _hasExecutedCommands set to true because it was used on the previous run.

Good point, those would certainly be useful. I've added tests for both.

I've updated this PR to use the same approach as #5657, i.e. only create RequestOptions when building the request, and use strings to hold the tags in SpannerCommand, SpannerBatchCommand and SpannerTransaction.

Proto options now merged in https://github.com/googleapis/google-cloud-dotnet/pull/6285

Proto options now merged in #6285

The PR has been merged with the latest changes from master and the manually generated code has been removed from the PR.
Any last thoughts / questions / comments on this PR from anyone?

Still LGTM :)


⚠ Bofrost.lu has degraded performance

In abb6eba, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1767 ms

Resolved: Bofrost.lu performance has improved in 2886dc9 after 6 minutes.


Support for indexing

Indexing support in pagefind would be useful in this integration.
Thanks for developing this integration, it makes things easier.

Wiat a minute, I just came to a realization after opening the issue. It's all about html tags right? There's nothing about the integration. It's already supported.


Fix cli tests and default option of formatOption

The FlacocoMain tests weren't executing the assertions after System.exit, so there were actually some issues with the file testing that are now fixed.
To overcome this we now have these assertions being executed outside the actual test unit itself. It is not very beautiful, but for testing after System.exit it seems like the best solution.
I also improved the default handling of formatOption, following the suggestion in https://github.com/remkop/picocli/issues/1386

I also improved the default handling of formatOption

Nice!


Feature: ZDD and BDD conversion

Closes #187

Codecov Report

Merging #219 (9fb9c9b) into main (517789c) will decrease coverage by 0.849%.
The diff coverage is 30.303%.

@@              Coverage Diff              @@
##              main      #219       +/-   ##
=============================================
- Coverage   97.001%   96.152%   -0.849%
=============================================
  Files           53        54        +1
  Lines         2601      2625       +24
=============================================
+ Hits          2523      2524        +1
- Misses          78       101       +23

Impacted Files
Coverage Δ

src/adiar/bdd/bdd.cpp
76.471% <0.000%> (-3.121%)
:arrow_down:

src/adiar/bdd/bdd.h
100.000% <ø> (ø)

src/adiar/internal/convert.h
0.000% <0.000%> (ø)

src/adiar/internal/decision_diagram.h
100.000% <ø> (ø)

src/adiar/zdd/complement.cpp
60.000% <0.000%> (-40.000%)
:arrow_down:

src/adiar/zdd/zdd.h
71.429% <0.000%> (-28.571%)
:arrow_down:

src/adiar/internal/intercut.h
100.000% <100.000%> (ø)

src/adiar/zdd/change.cpp
100.000% <100.000%> (ø)

src/adiar/zdd/expand.cpp
100.000% <100.000%> (ø)

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 517789c...9fb9c9b. Read the comment docs.

Codecov Report

Merging #219 (325913b) into main (517789c) will decrease coverage by 53.419%.
The diff coverage is 100.000%.

:exclamation: Current head 325913b differs from pull request most recent head 4df3520. Consider uploading reports for the commit 4df3520 to get more accurate results

@@              Coverage Diff               @@
##              main      #219        +/-   ##
==============================================
- Coverage   97.001%   43.582%   -53.419%
==============================================
  Files           53        36        -17
  Lines         2601      1675       -926
==============================================
- Hits          2523       730      -1793
- Misses          78       945       +867

Impacted Files
Coverage Δ

src/adiar/bdd/bdd.cpp
8.163% <ø> (-71.429%)
:arrow_down:

src/adiar/internal/decision_diagram.h
53.846% <ø> (-46.154%)
:arrow_down:

src/adiar/bdd/bdd.h
71.429% <100.000%> (-28.571%)
:arrow_down:

src/adiar/convert.cpp
100.000% <100.000%> (ø)

src/adiar/internal/convert.h
100.000% <100.000%> (ø)

src/adiar/internal/intercut.h
100.000% <100.000%> (ø)

src/adiar/zdd/change.cpp
100.000% <100.000%> (ø)

src/adiar/zdd/complement.cpp
100.000% <100.000%> (ø)

src/adiar/zdd/expand.cpp
100.000% <100.000%> (ø)

src/adiar/zdd/zdd.cpp
8.475% <100.000%> (-53.594%)
:arrow_down:

... and 44 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #219 (a172221) into main (517789c) will increase coverage by 0.060%.
The diff coverage is 100.000%.

:exclamation: Current head a172221 differs from pull request most recent head 4df3520. Consider uploading reports for the commit 4df3520 to get more accurate results

@@              Coverage Diff              @@
##              main      #219       +/-   ##
=============================================
+ Coverage   97.001%   97.061%   +0.060%
=============================================
  Files           53        55        +2
  Lines         2601      2654       +53
=============================================
+ Hits          2523      2576       +53
  Misses          78        78

Impacted Files
Coverage Δ

src/adiar/bdd/bdd.cpp
79.592% <ø> (ø)

src/adiar/internal/decision_diagram.h
100.000% <ø> (ø)

src/adiar/bdd/bdd.h
100.000% <100.000%> (ø)

src/adiar/convert.cpp
100.000% <100.000%> (ø)

src/adiar/internal/convert.h
100.000% <100.000%> (ø)

src/adiar/internal/intercut.h
100.000% <100.000%> (ø)

src/adiar/zdd/change.cpp
100.000% <100.000%> (ø)

src/adiar/zdd/complement.cpp
100.000% <100.000%> (ø)

src/adiar/zdd/expand.cpp
100.000% <100.000%> (ø)

src/adiar/zdd/zdd.cpp
62.712% <100.000%> (+0.643%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


index.d.ts does not include exported member L

While importing L from module using
import {L} from 'vue2-leaflet';
an error is being thrown:
Module '"vue2-leaflet"' has no exported member 'L'.
And we CANNOT use import from leaflet directly because some functions fail checks because leaflet sometimes uses instanceof and it is a huge app size increase.
Versions

Leaflet: v1.3.4
Vue: v2.5.21
Vue2Leaflet: v1.2.3

install @types/leaflet, I am unsure why it's not in your dependency tree maybe it needs to be in a different slot in the package.json for this project.

@drafu @miyujach The reason why I did not address this is because my knowledge of TS is minimal and the reason why @HIMISOCOOL is instead answering is because he did create the TS support.
@HIMISOCOOL Could you provide a pr fix for this? or some instructions to proceed ? I would be happy to do a release afterwards.
In the meantime you can see how 'double leaflet' errors can be fixed here #281 (until I manage to release a 2.0 with leaflet as peerDependency )

Thank for temporary double leaflet fix. That was exactly one of the problems
Still looking for your fix as installing leaflet separately doubles the final app size.

@drafu does it still double leaflet in the final build? It should not because webpack should prune everything beside the leaflet instance in the root of your node_module

@lordfuoco
It was just yesterday with latest versions of everything I could possibly get

@drafu Sorry I meant after applying the alias fix does it still double the size?

@lordfuoco
Will check and reply the results today or tomorrow

@types/leaflet is already installed because I genuinely use TypeScript for the whole app.
It is just index.d.ts that lacks it. Manually rewriting that single file to add exported member fully fixes the problem and the app worlks flawlessly further on
It is just the problem that author should fix

Hey dude, can you show me what you changed? I use this project in a solution that's in UAT right now and don't see this issue

OH @lordfuoco is this library meant to re-export the L namespace?? I just import L from leaflet and the components from here.

@HIMISOCOOL yes to try to help and mitigate the multiple leaflet references

@lordfuoco OH you added that just after I made the types, I have a few minor changes in a local branch at work, I'll add the export on Monday. The need to install @types/leaflet is still odd to me but for now you could add it to the readme 'requires npm install @types/leaflet if using typescript'

@HIMISOCOOL I just copied to match block from src/main.js:29 to types/index.d.ts:707

@drafu thanks man. @lordfuoco in the future if you are gonna add new exports definitely add them to both files like above and ideally we'd make some types for the new component too

Hey @drafu I made a pr #296 can you give it a check?

@drafu @HIMISOCOOL I just published a new beta version containing #269
https://www.npmjs.com/package/vue2-leaflet/v/1.2.3-beta.9
Can you please test it ?

@lordfuoco the issue still persists when I clone master so its probably going to be happening on the beta version.
Ill look at it now and get back to you.


Force version 3.29.0 of org.eclipse.core.runtime to address CVE vulnerabilities

Description
Address CVE vulnerability in plugin/build.gradle: org.eclipse.core.runtime-3.26.100.jar by forcing version 3.29.0
Related Issues
Resolves #[Issue number to be closed when this PR is merged]

Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] API changes companion pull request created.
[ ] Commits are signed per the DCO using --signoff.
[ ] Public documentation issue/PR created.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Cool, looks like Mend Security Check is Successful. While you are working on CVE, let's look into this too: https://github.com/opensearch-project/ml-commons/issues/1865


terminate called without an active exception

./src/zcash-miner -G -stratum="stratum+tcp://us-east1.zcash.miningpoolhub.com:20570" -user=test -password=x
[OPENCL]:Found suitable OpenCL device [Tonga] with<PHONE_NUMBER> bytes of GPU memory
[OPENCL]:Using platform: AMD Accelerated Parallel Processing
[OPENCL]:Using device: Tonga(OpenCL 2.0 AMD-APP (1800.11))
terminate called without an active exception
Aborted (core dumped)
Test mode works OK, aside from #48
built with gcc 6.1.1, will try with 4.8.4
edit: same result with gcc 4.8.4

Same problem here (builds on Ubuntu 14.04 w/AMDGPU-PRO fresh install)

Same problem here as well (Ubuntu 16.04 & AMDGPU-PRO 16.30)

I'm having the same issue.
Linux deluzion.net 4.7.9-200.fc24.x86_64 #1 SMP Thu Oct 20 14:26:16 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
glibc-2.23.1-10.fc24.x86_64
opencl-headers-1.2-9.fc24
gcc version 6.2.1 20160916 (Red Hat 6.2.1-2) (GCC)

Does flypool work for you guys? Just want to know because, it would give us hints for debugging the issue.

Interesting.  For eth mining, pools accept usernames without a rig fine.
Nevertheless, I supplied a rig name and can connect to mph, but get "Checking solution against target... too large" errors.
I tried flypool, and get the same "too large" errors.

as per @omaralvarez, error is due to lack of worker name, which causes stratum auth failure.


Remove unnecessary network attribute

Remove unnecessary network attribute from functions.json.
Signed-off-by: Richard Gee<EMAIL_ADDRESS>

Are any present in store.json?

Are any present in store.json?

Are there?

That's a question for you. I didn't think we were using functions.json yet, or are we?
store*.json does appear to contain some entries to func_functions, can you please cover that as part of this PR?


Added cpplint support. Fixes #46.

Hi, first of all very nice tool :+1:
I want to use it for c++ code and thats why the PR.
Tests are added.
The output of cpplint:
main.cpp:9:  At least two spaces is best between code and comments  [whitespace/comments] [2]
Done processing main.cpp
Total errors found: 1

... is mapped to git-lint output ...
Linting file: main.cpp
line 9: 2: [whitespace/comments]:  At least two spaces is best between code and comments

The tests fail for c++ and h++ but not for the other file-endings.
Seems to me there is a problem with the "+" in the file name?
======================================================================
ERROR: test_linter_cpplint_with_c++ (test.e2etest.test_e2e.TestHgE2E)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/sk-/git-lint/test/e2etest/test_e2e.py", line 147, in test_linter
    self.assert_linter_works(linter_name, extension)
  File "/home/travis/build/sk-/git-lint/test/e2etest/test_e2e.py", line 122, in assert_linter_works
    response, output = self.lint()
  File "/home/travis/build/sk-/git-lint/test/e2etest/test_e2e.py", line 31, in lint
    response = gitlint.main([], stdout=out, stderr=out)
  File "/home/travis/build/sk-/git-lint/gitlint/__init__.py", line 234, in main
    filename, modified_lines, gitlint_config)
  File "/home/travis/build/sk-/git-lint/gitlint/linters.py", line 187, in lint
    linter_output = linter(filename, lines)
  File "/home/travis/build/sk-/git-lint/gitlint/linters.py", line 109, in lint_command
    for data in filtered_lines:
  File "/home/travis/build/sk-/git-lint/gitlint/utils.py", line 30, in filter_lines
    pattern = re.compile(filter_regex)
  File "/home/travis/virtualenv/python2.7.13/lib/python2.7/re.py", line 194, in compile
    return _compile(pattern, flags)
  File "/home/travis/virtualenv/python2.7.13/lib/python2.7/re.py", line 251, in _compile
    raise error, v # invalid expression
error: multiple repeat

Thanks Benjamin for the CL. To fix the isue you are having you would just need to escape the filename in https://github.com/sk-/git-lint/blob/master/gitlint/linters.py#L103-105 with re.escape (https://docs.python.org/2/library/re.html#re.escape)

Awesome, thanks for the hint. Will commit a fix.

Done

Changes Unknown when pulling ac6b60da47ee59b5614cf04ce882ad4143909f8d on benjaminfuchs:master into ** on sk-:master**.


Async cluster shutdown conflicts

Fixes the merge conflicts

Started a new branch here too since I am not sure if I rebased correctly:
https://github.com/dask/distributed/pull/7283/files


Add "Bugs" tab

It would strictly link to the bug label, which is the most common label name. The "Pinned issues" section would be hidden like it's hidden in the PR tab.
If no issues are found under the label, the tab wouldn't be added.
Useful or annoying? I feel that the current "Issues" tab hides bugs that need to be fixed by merging them with feature requests and all kinds of other noise.

Maybe not limit it to issues only, but also to PRs: https://github.com/sindresorhus/refined-github/labels/bug

why not be like the is merged?

but also to PRs

I don't think it would be very useful. While PRs may have that label, they aren't bugs, they're bugfixes 😅

Maybe they're a failing test case to showcase the bug ;)

Feedback wanted in #2750, there are a couple of points to discuss

I don't think it would be very useful. While PRs may have that label, they aren't bugs, they're bugfixes 😅

Same way as not every issue is a bug, not every PR is a bugfix.
Probably there should be another BugFix tab ?


⚠ Samsung has degraded performance

In 334cc61, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2304 ms

Resolved: Samsung performance has improved in 1c4055a.


Can't use subexpression in partial parameter

So I'm trying to do something like this:
{{> head title=(_ "Sign Up") }}
where _ is a helper I've registered, but I'm getting this error:
HandlebarsDotNet.HandlebarsCompilerException: Token ')' could not be converted to an expression
This works in the JavaScript library:
Handlebars.registerHelper('_', function(s) {
	return new Handlebars.SafeString(s);
});
Handlebars.registerPartial('head', 'title is "{{title}}"');
var template = Handlebars.compile('{{> head title=(_ "Sign Up") }}');
alert(template());

The current implementation of the hash parameters parser is very simple... it doesn't support anything but literals in the value position. We need to remove the parsing logic from the WordParser and move it to the core lexer, then the other tokenizers like the SubExpression parser will be able to pick up the () and interpret the contents.
I may get to this in the next few weeks. PR always welcome in the meantime!

I'm interested in helping with this.
The tokenizing should be simple to update - instead of treating param=value as a token, it can be broken into three tokens with = as it's own token.
The converters are more difficult, I wonder if parameters and subexpressions need to be handled in the same converter? It's difficult because hash parameters can exist in subexpressions and vice versa (eg. {{>partial item1=(helper item1="val1") item2="val2"}}).
@rexm how were you thinking about handling this?

The extent of my thinking so far is in the comment above - anything deeper is welcome. Could you explain the one you started?

Converters/Accumulators
I had found I needed to convert to hash parameters before and after the SubExpressionConverter. The HashParametersConverter would skip hash parameters if the next token was a StartSubExpressionToken. I've been thinking more about this and I don't think it's ideal as it won't support deeply nested combinations of sub-expressions and parameters. I'm currently thinking it would be worth handling parameters with an interim expression for the parameter assignment excluding the value (param=), that the SubExpressionConverter isn't going to throw on, and then run an accumulator after the SubExpressionConverter to pick up the value.
Binders
With the binders I've looked some more into it and returning a BlockExpression for creating the parameter dictionary seems like a good way to go as it'll return the result of the last expression, so something like this will do what we want it to do:
var x = new HashParameterDictionary();
x.Add(key, value);
...
x;

Visitors
The other main change I was looking at was a refactor of the Visitors, starting with the base HandlebarsExpressionVisitor. The System.Linq.Expressions.ExpressionVisitor methods follow a pattern of calling Visit() on all child Expressions and creating a new expression if they differ, ensuring any changes will be propagated. Applying this pattern to the HandlebarsExpressionVisitor means fewer overrides need to be written in the sub-classes.

I took another look at the Binder and creating the equivalent of new HashParameterDictionary { { "key", value }, ... } is trivial.

So to make sure I am following - we need to introduce a more sophisticated notion of a hash parameter where the right side of each parameter (the value) is an expression that could either be a const or a subexpression, and hash parameter converter should assume the right side has already been converted/processed, but it is not sufficient to simply run the subexpression converter/accumulator before hash parameters converter runs?

I've opened a PR (#262) with my approach to it - I've got it working using the two converters:

The first one converts a WordExpressionToken + AssignmentToken into a HashParameterAssignmentExpression before the SubExpressionConverter (and HelperConverter) runs.
The second one accumulates (HashParameterAssignmentExpression + Expression)+ into a HashParametersExpression. It needs to Visit SubExpressionExpressions & HelperExpressions in order to update handle nested parameters.

The converter changes can be seen here: https://github.com/rexm/Handlebars.Net/pull/262/commits/3f972b057725dd8ddd8a77f018b541ffc3ba288e

#262 merged


chore(main): release 1.18.0

:robot: I have created a release beep boop
1.18.0 (2024-04-11)
Features

Add a simple zh_TW Traditional Chinese locale (cec6539)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/viarotel-org/escrcpy/releases/tag/v1.18.0 :sunflower:


Dimensions versus ImageDimensions, what's their function? Can they be merged?

Vulkano has had two types for representing the dimensions of an image, Dimensions and ImageDimensions. Originally there was just one, but the split between these types was added already in 2016 in #260.
Neither of the types is documented, so it's not clear what their function is, nor which is to be used for which purpose. The result is a lot of duplication and an IMO very confusing situation. For example, ImageAccess::dimensions() and UnsafeImage::dimensions() both give you an ImageDimensions, while ImageViewAccess::dimensions() and ImmutableImage::dimensions() return Dimensions. Dimensions has a function to convert to ImageDimensions, but there is no function to perform the reverse operation. Either that simply never got added, or the conversion is lossy and Dimensions contains more information.
I would find it a lot clearer if there was just one type, or if the purpose of each one were documented. Does anyone who has worked with Vulkano longer have any idea what's going on? If there is a clear distinction, can they be renamed to make that clearer? If nobody knows, I'm going to try changing all instances of ImageDimensions to Dimensions in a future PR.

@Rua I agree there seem to be some inconsistency in implementation or maybe naming.  vulkano::image::Dimensions is capable of describing wider range of options. This might be a reason why they were split. So, maybe we can't just merge them together. Also if I not mistaken they used not only in function returns but in some function arguments too.
I don't mind about refactoring, but just a few brief notes to take into consideration.

I had another look at the situation and these are my observations:

Vulkan's vkImage type fits ImageDimensions perfectly in the data it needs.
Vulkan's vkImageView fits ViewType, which has the same enum variants as Dimensions but without the numbers.

A Dimensions could therefore be understood as a combination of ImageDimensions and ViewType, so it would only be needed in contexts that work with image views. The cubemap_compatible flag on ImageDimensions is an oddity, however. This is really a property of the image object, and has nothing to do with dimensions.
All in all I propose the following changes:

Dimensions is renamed to ImageViewDimensions.
ViewType is renamed ImageViewType just for clarity's sake.
The cubemap_compatible member is removed from ImageDimensions::Dim2d.
UnsafeImage takes a cube compatibility flag upon creation, either as a simple bool or, perhaps more future-proof, as a struct. UnsafeImageView checks for this flag when the user requests a cube map.

If #1499 is merged, then the point is moot because it removes Dimensions altogether.


Create metrics for the exemplar

We need to have metrics to measure how well the managing system is performing.
My idea so far is to measure the time the AUV takes to complete the mission

@JulianePa suggestions:

We could try to find some metric that measures how indecisive a managing system is. Maybe measure how often it switches between configurations and divide that by the time it needs to complete the mission. Because i guess it's not good to always switch between configurations if it's not necessary.
We could measure how long it takes the managing system to react to a fault of the managed system. E.g. the time from when a thruster broke to when the managing system adapted the managed system to only use 5 thrusters.
We could also detect whether it adapts to a fault at all/whether it goes to a higher altitude if possible. This is probably related to the time it takes to complete the mission.
We could measure the distance it covers until it completes the mission. Probably also related to the time, but it would show whether the robot moves around unnecessarily much.

Other options:

Number of unique configurations used. The more configurations used could be both positive and negative however. Many configurations used can show high adaptability, but also indecisiveness.
To add to the time needed to react to a fault in the managed system: the time needed to reason about adaptation and apply.
Maybe not related to the simulation, but it might be interesting to show number of unique adaptation options that are available. This would show high adaptability.
One that is definitely needed is to show how much faster/more accurate the UUV is with adaptation vs without.

I would say we need a utility function. This utility function should be based on the requirements of the system. This paper has a nice example of deriving one: http://www.es.mdh.se/pdf_publications/4695.pdf

From the meeting:
Important metrics:

Time for searching pipeline
Time for inspecting the pipeline (minimize it)
Sum both times
Distance of the pipe detected and inspected (if we set a time limit) (maximize it)
Relationship between them


fix(glide.lock,glide.yaml): revert to old version of pkg/aboutme

this commit reverts part of
https://github.com/deis/minio/pull/36/files#diff-395f41b2a27b70ce44399c9
1c82158c2R14.
On master, the following error is observed in production when minio
pods launch:
{"log":"Fatal error  pods \"deis-minio-2i9kv\" not
found\n","stream":"stdout","time":"2015-12-29T23:58:53.330822679Z"}`

cc/ @sgoings @slack

Still in the process of manual testing. Please do not merge until done.

@sgoings @slack I verified that this patch fixes. PTAL, I believe this is ready for merge.
ENG000656:minio aaronschlesinger$ kd describe pod deis-minio-tvqu1
Name:				deis-minio-tvqu1
Namespace:			deis
Image(s):			quay.io/arschles/minio:v2-alpha
Node:				gke-aaron-e39d4573-node-hecx/<IP_ADDRESS>
Start Time:			Tue, 29 Dec 2015 16:24:06 -0800
Labels:				app=deis-minio
Status:				Running
Reason:
Message:
IP:				<IP_ADDRESS>
Replication Controllers:	deis-minio (1/1 replicas created)
Containers:
  deis-minio:
    Container ID:	docker://94b8207c6507ec5e3c653d4dc9ed46105b555b14a76d9da27ecce23babdc69d0
    Image:		quay.io/arschles/minio:v2-alpha
    Image ID:		docker://413a7a3ecf6b2b49768f30092076879987a56565394e8a1ec1bd14eec464f5bd
    State:		Running
      Started:		Tue, 29 Dec 2015 16:24:24 -0800
    Ready:		True
    Restart Count:	0
    Environment Variables:
Conditions:
  Type		Status
  Ready 	True
Volumes:
  minio-admin:
    Type:	Secret (a secret that should populate this volume)
    SecretName:	minio-admin
  minio-user:
    Type:	Secret (a secret that should populate this volume)
    SecretName:	minio-user
  default-token-c4ia8:
    Type:	Secret (a secret that should populate this volume)
    SecretName:	default-token-c4ia8
Events:
  FirstSeen	LastSeen	Count	From					SubobjectPath				Reason		Message
  ─────────	────────	─────	────					─────────────				──────		───────
  30s		30s		1	{scheduler }									Scheduled	Successfully assigned deis-minio-tvqu1 to gke-aaron-e39d4573-node-hecx
  29s		29s		1	{kubelet gke-aaron-e39d4573-node-hecx}	implicitly required container POD	Pulled		Container image "gcr.io/google_containers/pause:0.8.0" already present on machine
  29s		29s		1	{kubelet gke-aaron-e39d4573-node-hecx}	implicitly required container POD	Created		Created with docker id 3ca1123267e3
  29s		29s		1	{kubelet gke-aaron-e39d4573-node-hecx}	implicitly required container POD	Started		Started with docker id 3ca1123267e3
  29s		29s		1	{kubelet gke-aaron-e39d4573-node-hecx}	spec.containers{deis-minio}		Pulling		Pulling image "quay.io/arschles/minio:v2-alpha"
  12s		12s		1	{kubelet gke-aaron-e39d4573-node-hecx}	spec.containers{deis-minio}		Pulled		Successfully pulled image "quay.io/arschles/minio:v2-alpha"
  12s		12s		1	{kubelet gke-aaron-e39d4573-node-hecx}	spec.containers{deis-minio}		Created		Created with docker id 94b8207c6507
  12s		12s		1	{kubelet gke-aaron-e39d4573-node-hecx}	spec.containers{deis-minio}		Started		Started with docker id 94b8207c6507

ENG000656:minio aaronschlesinger$ kd logs -f deis-minio-tvqu1
minio server /home/minio/

AccessKey: 8TZRY2JRWMPT6UMXR6I5  SecretKey: gbstrOvotMMcg2sMfGUhA5a6Et/EI5ALtIHsobYk

To configure Minio Client.

	$ wget https://dl.minio.io/client/mc/release/linux-amd64/mc
	$ chmod 755 mc
	$ ./mc config host add http://localhost:9000 8TZRY2JRWMPT6UMXR6I5 gbstrOvotMMcg2sMfGUhA5a6Et/EI5ALtIHsobYk
	$ ./mc mb localhost/photobucket
	$ ./mc cp ~/Photos... localhost/photobucket

Starting minio server:
Listening on http://<IP_ADDRESS>:9000
Listening on http://<IP_ADDRESS>:9000

Tested full deploy of example-go against your minio image. :+1:

@technosophos FYI on this. we'll need to investigate what's going wrong with FromEnv() inside minio


How to turn off auto scroll?

When I operate a FormBuilderImagePicker in SingleChildScrollView, the picker will scroll to the top, how do I turn off this function?

I found this code.
state.requestFocus();
void requestFocus() {     FocusScope.of(context).requestFocus(effectiveFocusNode);     Scrollable.ensureVisible(context);   }
This will make scroll automatically when you delete the image in the ScorllView.
Hopefully a parameter can be added to control this.


Fixing default server value

fix of problem described in this comment https://github.com/UffizziCloud/uffizzi_cli/pull/22#discussion_r849473515