stephane-eyskens.apim.apimversioned.apimversioned@4 leads to errors

Hello,
I have a task definition as below:
- task: stephane-eyskens.apim.apimversioned.apimversioned@4
  displayName: 'API Management - Create/Update API '
  inputs:
    ConnectedServiceNameARM: 'Azure DEV/INT Subscription'
    ResourceGroupName: 'rg-org-iot-we-dev-outbound-shared'
    ApiPortalName: 'apim-org-iot-we-dev-outbound-shared'
    OpenAPISpec: v3
    UseProductCreatedByPreviousTask: false
    product1: manage
    swaggerlocation: 'https://web-org-iot-we-dev-manage-api-v1.azurewebsites.net/api-docs/swagger.json'
    targetapi: manage
    scheme: Header
    VersionHeader: 'X-Api-Version'
    pathapi: manage
    MicrosoftApiManagementAPIVersion: '2017-03-01'

The task always leads to the error :
2020-03-15T09:55:46.9744207Z ##[section]Starting: API Management - Create/Update API
2020-03-15T09:55:46.9857600Z ==============================================================================
2020-03-15T09:55:46.9858029Z Task         : API Management - Create or Update Versioned API
2020-03-15T09:55:46.9858389Z Description  : API Management - Create or Update Versioned API
2020-03-15T09:55:46.9858672Z Version      : 4.1.1
2020-03-15T09:55:46.9859062Z Author       : Stephane Eyskens
2020-03-15T09:55:46.9859351Z Help         : API Management - Create/Update API
2020-03-15T09:55:46.9859677Z ==============================================================================
2020-03-15T09:55:47.8252137Z Preparing API publishing in v3 format json using Azure API 2017-03-01
2020-03-15T09:55:47.9986123Z
2020-03-15T09:55:48.0029700Z checking whether https://management.azure.com/subscriptions/467d0b3b-6a6a-4c6f-b4ed-03dda940ll3r/resourceGroups/rg-org-iot-we-dev-outbound-shared/providers/Microsoft.ApiManagement/service/apim-org-iot-we-dev-outbound-shared/apis/managev1?api-version=2017-03-01 exists
2020-03-15T09:55:48.7980180Z ##[error]You cannot call a method on a null-valued expression.
2020-03-15T09:55:48.8367847Z ##[section]Finishing: API Management - Create/Update API

Can you please help on this? I even tried the version 3 task, that led to similar errors.
Previous to this I was exposing using the task 'stephane-eyskens.apim.apim.apim@3' and that works fine and the APIs are published on the API management - however all attempts to publish a versioned API leads to errors.

Hi,
Did you purposely specify api version 2017-03-01? As of V4, the api should be 2019-01-01 (default value) so try to update this.
Best regards

Thanks for the quick response. I just tried it with API version 2019-01-01, it gives the same error as before. I even tried V3, it leads to the same error.
Best regards

Thanks for the quick response. I just tried it with API version 2019-01-01, it gives the same error as before. I even tried V3, it leads to the same error.
Best regards

I had the same problem, but changing API version fixed it for me... Could you post new logs with changed API version.

Also, having this with V3 means that you have an issue with your config. I might as well suspect a lack of permission. V3 hasn't be changed for ages and is pretty stable.

Hi @iamjyotirmoy
I had same problem when my server certificate was invalid


[FR] Add keypoint evaluation

Similar to evaluate_detections() and evaluate_classifications(), it would be useful to add a built-in method to evaluate keypoint labels. A good place to start is likely replicating the COCO method for keypoint evaluation defined here: https://cocodataset.org/#keypoints-eval

https://github.com/voxel51/fiftyone/pull/2928 closes this issue


Package upload APIs should be included in swagger docs

Description
There are many API endpoints for uploading packages of various types.  However, these endpoints are not described in the API swagger data, so they are not visible in the swagger UI.
Example: PUT https://gitea.example.com/api/packages/{owner}/debian/pool/{distribution}/{component}/upload is the endpoint for uploading a .deb package to an APT repository.  It is described here on docs.gitea.com, but not here in the swagger docs.
Other examples include conda, generic, go.
Are these API endpoints treated differently because they exist under /api/packages/ rather than /api/v1/packages/?  If so, can they be aliased?
Gitea Version
1.21.10, 1.23.0+dev-172-geb24d973b0
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots

Git Version
No response
Operating System
No response
How are you running Gitea?
container image, and try.gitea.io
Database
None

But the protocol is this package's own standard. Should we list them?

It's not mentioned in swagger because what lunny says. But all packages which use curl in the examples may benefit from swagger because the package type does not have a buildin upload.

Yeah, you're both right.  I'm thinking of the ones that use curl.  The package types which have official and well-defined clients, like docker, don't need explicit documentation as much.

I think this is a proposal not a bug


chore: add snapshots for text widgets

/ok-to-test<EMAIL_ADDRESS>
Summary by CodeRabbit

Tests

Added new test cases for Anvil's Heading Widget, covering Canvas, Preview, and Deploy modes.
Introduced test cases for Anvil's Paragraph Widget in Canvas Mode, Preview Mode, and Deploy Mode.

[!CAUTION]
🔴 🔴 🔴 Some tests have failed.
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/9696369218
Commit: 52012811b03abe7efc586fb5ecbd68f7b39953a7
Cypress dashboard.
Tags: @tag.Anvil
The following are new failures, please fix them before merging the PR:
cypress/e2e/Regression/ClientSide/Anvil/Widgets/AnvilCurrencyInputWidgetSnapshot_spec.ts
cypress/e2e/Regression/ClientSide/Anvil/Widgets/AnvilHeadingWidgetSnapshot_spec.ts
cypress/e2e/Regression/ClientSide/Anvil/Widgets/AnvilInputWidgetSnapshot_spec.ts
cypress/e2e/Regression/ClientSide/Anvil/Widgets/AnvilParagraphWidgetSnapshot_spec.ts
cypress/e2e/Regression/ClientSide/Anvil/Widgets/AnvilPhoneInputWidgetSnapshot_spec.ts
List of identified flaky tests.

Merged here.


[Flaky CI] [Serial]SRIOV VirtualMachineInstance with sriov plugin interface - multiple tests

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
/triage build-watcher
What happened:
These 3 tests failed:

[Serial]SRIOV VirtualMachineInstance with sriov plugin interface [test_id:3956]should connect to another machine with sriov interface over IPv4

tests/vmi_multus_test.go:869
Expected success, but got an error:
    <*errors.errorString | 0xc000ace0d0>: {
        s: "Failed to ping VMI testvmidht8nwl8wwxn6n5mwbwc6ngcqskhrblsrw2g9vctptclk2wz, error: Failed to run [ping <IP_ADDRESS> -c 1 -w 5] at VMI testvmidht8nwl8wwxn6n5mwbwc6ngcqskhrblsrw2g9vctptclk2wz, error: ping <IP_ADDRESS> -c 1 -w 5 failed",
    }
    Failed to ping VMI testvmidht8nwl8wwxn6n5mwbwc6ngcqskhrblsrw2g9vctptclk2wz, error: Failed to run [ping <IP_ADDRESS> -c 1 -w 5] at VMI testvmidht8nwl8wwxn6n5mwbwc6ngcqskhrblsrw2g9vctptclk2wz, error: ping <IP_ADDRESS> -c 1 -w 5 failed
tests/vmi_multus_test.go:865

[Serial]SRIOV VirtualMachineInstance with sriov plugin interface [test_id:3957]should connect to another machine with sriov interface over IPv6

tests/vmi_multus_test.go:873
Expected success, but got an error:
    <*errors.errorString | 0xc0025a5030>: {
        s: "Failed to ping VMI testvmi9lzn66wssfp92nhj95pcnkwmcv25ct6k6fwqpg8f8grq7h26, error: Failed to run [ping -6 fc00::2 -c 1 -w 5] at VMI testvmi9lzn66wssfp92nhj95pcnkwmcv25ct6k6fwqpg8f8grq7h26, error: ping -6 fc00::2 -c 1 -w 5 failed",
    }
    Failed to ping VMI testvmi9lzn66wssfp92nhj95pcnkwmcv25ct6k6fwqpg8f8grq7h26, error: Failed to run [ping -6 fc00::2 -c 1 -w 5] at VMI testvmi9lzn66wssfp92nhj95pcnkwmcv25ct6k6fwqpg8f8grq7h26, error: ping -6 fc00::2 -c 1 -w 5 failed
tests/vmi_multus_test.go:865

[Serial]SRIOV VirtualMachineInstance with sriov plugin interface [test_id:3985]should create a virtual machine with sriov interface with custom MAC address

tests/vmi_multus_test.go:789
MAC "de:ad:00:00:be:ef" was not found in the VMI testvmiskmw6972j7pzpkq8vzp5h4tms8tmlncd6fdkqmsqts2fwtxn within the given timeout
Unexpected error:
    <expect.TimeoutError>:<PHONE_NUMBER>0
    expect: timer expired after 15 seconds
occurred
tests/vmi_multus_test.go:1065

https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4415/pull-kubevirt-e2e-kind-1.17-sriov/1321724193020055552
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/4415/pull-kubevirt-e2e-kind-1.17-sriov/1321579487917248512

Thanks, duplicates of https://github.com/kubevirt/kubevirt/issues/4432
and https://github.com/kubevirt/kubevirt/issues/3840


Failed to parse json, Unexpected token '{' at 10:36

I'm having this problem while installing npm. I cloned the project, cd into pwa/template. Then run the npm install command. And it is throwing this error:
npm WARN Failed to parse json
npm WARN Unexpected token '{' at 10:36
npm WARN     "build": "node build/build.js"{{#unit}},
npm WARN                                    ^
npm WARN File: /var/www/html/pwa/template/package.json
npm WARN template No description
npm WARN template No repository field.
npm WARN template No README data
npm WARN template No license field.

Hi, try to run
npm i

in the root folder, without navigating onto template folder.


133

Finally fix #132 & #133

PR-Codex overview
This PR focuses on improving the display of questions and answers by adding line breaks and links.
Detailed summary

Added parseLinks function to handle links in Question.tsx and QuestionView.tsx
Added break-words class to wrap long text in Question.tsx and QuestionView.tsx
Changed className of Question.tsx components to improve text color and add chevron icon
Replaced currentQuestion.text and currentQuestion.answer with dangerouslySetInnerHTML to enable link parsing in QuestionView.tsx

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Codecov Report
Patch and project coverage have no change.

Comparison is base (47e26a9) 71.03% compared to head (021b291) 71.03%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff            @@
##             master     #137   +/-   ##
=========================================
  Coverage     71.03%   71.03%
  Complexity      172      172
=========================================
  Files            79       79
  Lines          1260     1260
  Branches        115      115
=========================================
  Hits            895      895
  Misses          259      259
  Partials        106      106

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add Minzu University of China

Official website URL: https://www.muc.edu.cn/
Email domain<EMAIL_ADDRESS>Official website URL of long-term IT related course: https://www.muc.edu.cn/info/1037/7170.htm

@smy1999 I have to close this request without merging, because you requested to add a non-existent domain to this repository: ;; connection timed out; no servers could be reached Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


Minor clarification on handling of fragmented packets

Created in response to discussion on this issue: https://github.com/Azure/DASH/issues/61

I think https://github.com/Azure/DASH/pull/63/ and https://github.com/Azure/DASH/pull/64 are both trying to address the same issue, somewhat differently.

Everything in PR #63 looks reasonable to me, but I do not see how #63 addresses the issue that non-first fragments must be dropped if there is no matching temporal flow state that matches it.

It seems like PR64 is a superset. Perhaps the two authors @jafingerhut and @mhanif can agree on a single unified PR? Thanks, it's nice to have a surplus of enthusiasm!

If I understood correctly, in the meeting today Gerald suggested I create a PR for Microsoft engineers to look at and think about when they are doing their careful review of IP fragment handling, and likely this PR will be closed without being merged, as they will make their own changes.  It is fine with me either way.

Hi Andy, I merged and closed PR 63, would you mind resyncing and submit 64 again so I can view the changes?
Thank you very much, Kristina

@KrisNey-MSFT I have merged the latest changes in the main branch with the changes proposed in this PR.


Tapping screen on an iPad2 slows down the simulation

While investigating https://github.com/phetsims/plinko-probability/issues/26, we found that tapping plinko-probability slows down the animation. On the second tab, playing the simulation in continuous mode on iPad2, and tapping the blue background slows down the rendering from 45fps to 20fps.
Here is the link to the dev version.
http://www.colorado.edu/physics/phet/dev/html/plinko-probability/1.0.0-dev.5/plinko-probability_en.html
This is mostly apparent in simulations that have a running animation. Using Rutherford scattering as an example, then one can slow down the animation by simply tapping the simulation repeatedly (say 5 times a second) anywhere on the screen.
Tapping the play button repetitively in plinko is an expected user interaction.
The browser found the issue using safari but we can reproduce it using chrome ( 51.0.2704 ) as well.

I have tried to reproduce on a MondoPad (a giant touchscreen, running windows 7) using chrome 51.0.2704 but to no avail. The MondoPad runs plinko at 60fps no matter how much tapping we do (we tried with 30 fingers of tapping). For reference, tapping down with 5 fingers repetitively bring the iPad to 2fps.

I just tried to observe this issue in rutherford-scattering and plinko-probability, but I could not reproduce in iOS 9.3.1 with an iPad2.  I tried tapping repetitively with 1-5 fingers.

6/30/16 dev meeting:  Suggested to try built version with all query parameter options off. Also try with and without other tabs open in the browser.

I just tested this plinko-probability 'Lab' screen in requirejs.  The frame rate hovers around 26 fps.  When I repeatedly tap the blue background with 5 fingers, the fps drops to 16.

It would also be worth looking into inputs. Pressing a button on any simulation on the iPad causes the fps to drop. If a button is pressed repeatedly then the fps can drop to below 10. This is true for most simulations, but most simulations don't involve much button pressing by design. For Plinko being able to press a button repeatedly is necessary for the simulation to work properly.

I'd recommend checking what is hit tested, and how expensive it is to hit test. You can run popupDebug and look for the "hits" text in the output to see what is hit tested. Remember that hit-testing Path is much more expensive than hit-testing the optimized versions (Circle,Rectangle/etc.)

Since (1) IPad2 is no longer a supported platform, (2) Plinko-probability has been published., it is safe to tag this issue as stale and close it.


Smarter (impzlength-like) choice of impulse response evaluation length

Is your feature request related to a problem? Please describe.
I would like to calculate the energy in impulse responses for various filters using signal.dimpulse. However, the number of time points is not very smart (actually, it is 100 all the time). For high aspect ratio filters, this means that the best part of the impulse response is not taken into account.
https://github.com/scipy/scipy/blob/adc4f4f7bab120ccfab9383aba272954a0a12fb0/scipy/signal/ltisys.py#L3439-L3441
Describe the solution you'd like
MATLAB has the impzlength function to make an educated guess on the effective length of an FIR/IIR filter.
Describe alternatives you've considered
I have seen Sammy Pfeiffer start implementing this function in python, but it seems to be unfinished.
I think it would be good to have this function included and maintained in scipy, and used by dimpulse if n==None.

This is on my to-do list for scipy for IIR cases but already implemented in harold which is working quite OK for almost all cases. You can find the implementation below
https://github.com/ilayn/harold/blob/master/harold/_time_domain.py#L280-L454
FIR part is another story that needs me sparing some more extra time.


[Draft - WIP] Alcail/doc review for consistency

Purpose

Pass through all docs (still Work In Progress)

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

Why deleting generate_aad_token_manually.md file?


Capture Multiple Screens(Monitors)

Would use WindowsForms, look for the WPF way later

All Screens are now Captured


Amalgamation for 4.0.0

@Naios
Congratulations on release 4.0.0, the changes look really useful. Get's a feeling of javascript's Promise and async/await into C++!
Anyway, can you please provide an amalgamation for release 4.0.0 and link it in the documentation?

Thanks, I generated a new amalgamation for 4.0.0: https://gist.github.com/Naios/25d731aa4707d35a9bcec507f3cb9038
I'm going to link it into the documentation.

Just as note: The amalgamation header only contains the headers that require the C+ standard library and function2 which is included as part of the header. The include/continuable/external/ headers which extend (boost) asio and GTest are not included.

can we get one for 4.1.0 ?
I would also prefer it if the headers used relative includes so it's use was not dependent on a -I compiler flag, or env var, eg.:
in continuable_base.hpp:
#include "continuable-primitives.hpp"
#include "continuable-result.hpp"

If you like, you can create an amalgamation that fits your needs:
I think it uses this tool: https://github.com/edlund/amalgamate
The shell script is located in: tools/amalgamate/amalgamate.sh
Simply running this script should be enough to create the header, you can tweak it to fit your needs afterwards.

i have tried doing this, but i can't get it to work. it wowuld be great if someone could post an amalgamation of the most recent release on here.


softmax in GraphormerAttentionHead

softmax = torch.softmax(a, dim=-1)
The softmax function is not exact here since we should compute the attention in each graph. However, such direct computation causes the attention between nodes from different graphs in a batch.
Maybe combining batch_mask with -10^6 replacing 0 is a solution.

Thanks for the issue!
Yeah, that sounds like a good solution!

fixed in https://github.com/leffff/graphormer-pyg/commit/f56d6487f05ffc105a0f977b303ce596338a23fc
You're welcome to test!!!

Hi，
in the forward method of  class GraphormerAttentionHead(nn.Module) , I think there are two changes that should be made to ensure that the sum of each row vector after applying softmax function equals 1:

Change batch_mask_neg_inf[ptr[i]:ptr[i + 1], ptr[i]:ptr[i + 1]] = 1 to batch_mask_neg_inf[ptr[i]:ptr[i + 1], ptr[i]:ptr[i + 1]] = 0.

Change a = (a + b + c) * batch_mask_neg_inf to a = (a + b + c) + batch_mask_neg_inf.

Would you agree with these proposed changes?


[Feature request] Redirect if the location is not found (notFoundRedirect)

It would be cool to have an option to redirect to a specific page (for example the initial page) when the requested page/location is not found.
At the moment the only option is to show a 'Page not found' page.

@devj3ns yes, would be nice. Will you be implementing this?


fine tune model

Is Vllm compatible for inference of a custom fine tune model??

Hi @deepanshu3061999, yes. As long as the model architecture is supported (supported list), you can run your local model with vLLM by just providing the path to the model directory as the model argument.

i have a peft model , i am merging and uploading the weights of that model with base model and the loading it, want to make this time faster
Reload model in FP16 and merge it with LoRA weights
base_model = AutoModelForCausalLM.from_pretrained(
model_name,
low_cpu_mem_usage=True,
return_dict=True,
torch_dtype=torch.float16,
device_map=device_map,
)
model = PeftModel.from_pretrained(base_model,"/content/gdrive/MyDrive/28nov2023")
model = model.merge_and_unload()
Reload tokenizer to save it
tokenizer = AutoTokenizer.from_pretrained(model_name, trust_remote_code=True)
tokenizer.pad_token = tokenizer.eos_token
tokenizer.padding_side = "right"
can we use vllm instead of this ?

Hello~I have the same question. How did you solve this problem? Thanks so much!


Added support for datetimes on Bounds streams

Add support for datetimes on linked Bounds streams by converting bokeh timestamp to datetime64 like other streams. Addresses https://github.com/ioam/holoviews/issues/1674

Tests are passing, going to merge this.


not showing auto complete when use in bootstrap modal

HI,
When i bind plugin with input box which was in bootstrap modal box .then it will not show autocomplete suggestion.

I confirm this issue with the latest Bootstrap 3.3.7 and jQuery 3.1.0. Any ideas for a fix?

i checked when i increase the z index of auto complete div then its showing
.don't know proper solution
On Aug 4, 2016 8:43 PM, "juliusvinton"<EMAIL_ADDRESS>wrote:

I confirm this issue with the latest Bootstrap 3.3.7 and jQuery 3.1.0. Any
ideas for a fix?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ubilabs/geocomplete/issues/280#issuecomment-237584291,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AS1QEg5j3AAEU4zPkvowpUZ3cwEhcyeEks5qcgGRgaJpZM4JWHW5
.

Can you write an example? I have increased z-index of text_field with autocomplete but it still doesn't show.

not to increase input zindex . increase the z index of div which was auto
generated by auto completed.
On Sep 23, 2016 6:28 PM, "Jirka1111"<EMAIL_ADDRESS>wrote:

Can you write an example? I have increased z-index of text_field with
autocomplete but it still doesn't show.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ubilabs/geocomplete/issues/280#issuecomment-249185134,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AS1QEhly3nzuEWDrO3L9FOo78biF6KuVks5qs8zhgaJpZM4JWHW5
.

I've solved it this way. Maybe something to consider for the next update.
.pac-container {
	z-index: 9999;
}

Sorry guys, still don't get it:

Write a review

        <%= form_for @review, url: reviews_path, method: "post" do |f| %>

                          <%= f.text_field :place, id: "cities", class: "form-control", type: "text" %>
                          Place

Which one has to have big z-index?

Oh my god, I couldn't send it normally so sorry for the redirect.

Now i get it. Thanks you both! It was a .pac-container:<PHONE_NUMBER> !important
I always use !important in case of Bootstrap...

I think this solution has to be mentioned in the description of that plugin...
Explanation (sum up): Geocomplete works even in Bootstrap modal, but it is not visible. You have to set z-index of autogenerated div to something like<PHONE_NUMBER>9 with !important.
Solution (by juliusvinton):
.pac-container {
z-index:<PHONE_NUMBER> !important;
}
.pac-container is autogenerated class of geocomplete plugin. You don't have it in your code, but it is visible in code inspector in Chrome.


[WIP][Do not merge][video] CSN networks

Initial implementation of the Channel Separated ConvNets as described in https://arxiv.org/abs/1904.02811, with belonging code and models.
Please do not merge this PR before 1 and 2 are not merged into their respective masters.
[1] Slightly changes the video model API - this has been tested with and without the changes there and it works, but for the sake of consistency, let's wait until they all look the same.
[2] There is no efficient implementation of the 3D depthwise convolution, and until there is, the FLOPs saved by using depth-wise channel-separated convolutions is meaningless given the inefficiencies in the kernel.

Discussion point:
Which pre-trained model do we want to provide with them kinetics as everywhere else, or a more suitable one such as Sports1M or URU?

@bjuncek

Which pre-trained model do we want to provide with them kinetics as everywhere else, or a more suitable one such as Sports1M or URU?

Let's keep using Kinetics-400, as that's what the other models use already, and it is also referenced in the CSN paper.

I have a question. Why do you only provide pre-training models for ir-csn-152. what about ir-csn-50.


Fix export

Pylance is crying that AbstractRepository and ObjectIdField are not exported.
adding them to all fixed the problem.
also, couldn't test it on my system as there's a subdependency missing (setuptools). added it

Thanks!


added vehicle hashes for two missing trailers

I noticed that two vehicle hashes were missing.
One of them is of the docktrailer and the other of the freighttrailer.

Nice catch!


explicit stroke and fill settings for drag icon

The white drag icon is almost invisible on bright images, so we draw a
stroke around it. Settings are made via CropperDrawSettings.

Thank you!


The default dtype of np.complex64 for the DensityMatrixSimulator is not precise enough for relatively small circuits.

Description of the issue
How to reproduce the issue
Use the example circuit:
example-circuit.json
import cirq

with open("example-circuit.json", "r") as f:
    circuit = cirq.read_json(f)

simulator = cirq.DensityMatrixSimulator(dtype=np.complex64, noise=None, seed=None, split_untangled_states=True)
obs = cirq.PauliString(cirq.X(cirq.LineQubit(28)), cirq.X(cirq.LineQubit(29)))
simulator.simulate_expectation_values(program=circuit, observables=obs)

Now, if we change the dtype to np.complex128, the error goes away.
Suggested solution: set the default value of dtype to np.complex128 or np.complex256

1018     raise ValueError(f'Density matrix does not have trace 1. Instead, it has trace {trace}.')
1019 if not np.all(np.linalg.eigvalsh(density_matrix) > -atol):
-> 1020     raise ValueError('The density matrix is not positive semidefinite.')
ValueError: The density matrix is not positive semidefinite.

Cirq version
1.3.0

csynkque meeting - this works as intended, but the error message can be improved to suggest using a higher precision complex128 type.  Another consideration is to expose a tolerance atol argument which would allow higher round-offs for complex64 data type.


merge coverprofiles instead of overwriting them

This change will save coverprofiles in a separate folder and merge them together at the end. Currently the coverprofile gets overwriten by each go test run.

fixes #30


Tutorial with React?

I've been trying to use this with Soluto's oidc server mock but for some reason it's very difficult to make the Getting Started app work with it (constantly getting errors). If I want to make the user log in with credentials, how do I make them work together? Any practical demo would be greatly appreciated.

Maybe you could add here a minimum example of your solution, which is not working. Then we could point on things, where you need to fix something. Indeed I'm not sure if this board is for such support. Otherwise you could ask chatgpt. There we get a lot of helpful ideas.


Electron 9: increase in binary size

Something to check when we update Electron is increase binary sizes. So far I can see some increase comparing stable to insiders:

Not sure if there is a build issue in Electron that could explain this or simply new code that accounts for this.
Something I noticed while checking randomly: libGLESv2.dylib is now 10 MB when it was 4 MB before.

There is now a 3MB increase in Electron Framework executable and 6MB increase in libGLESv2.dylib, there is no issue with stripping here, the framework increase is newly added code. As for the gl library, it is the same 10MB size in chrome app as well. I will check with upstream about what caused this size increase but there is nothing actionable at the moment.

This is a known size increase in chromium and they are investigating possible solutions.


Dockerfile correction

Thanks for the work you're doing in the repo, this has been extremely helpful!
One note here: https://github.com/evarga/docker-images/blob/master/ubuntu-js/Dockerfile#L12
Within that line, you're actually blowing out the sources.list so will likely see some broken dependency errors down the road.
RUN echo "deb http://archive.ubuntu.com/ubuntu precise main universe" > /etc/apt/sources.list

should be
RUN echo "deb http://archive.ubuntu.com/ubuntu precise main universe" >> /etc/apt/sources.list

Oh sorry, my mistake I see that you're appending multiple sources within that line.


Possible to run on 8 x 24GB 3090?

This model looks amazing, thank you! We have a machine with 8 x 3090 (192GB total), I tried to run the examples, but I get:
building GPT2 model ...

RuntimeError: CUDA out of memory. Tried to allocate 76.00 MiB (GPU 3; 23.70 GiB total capacity; 22.48 GiB already allocated; 70.56 MiB free; 22.78 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

For someone who is not an expert with pytorch etc., perhaps you have a suggestion?
We would try to make a conversation partner for language learning (add TTS, translation, NLP etc.) for our project: https://dev.languagereactor.com/
Regards, David :)

Shouldn't it be 100B x sizeof(double) or x sizeof(float)?

Weights are bfloat16, which is 16 bits, so you need at least 200GB to load those, plus some extra for inference.

Maybe a silly question: would it help to put a 9th card (9 x 24GB) in the machine? I have one extra.


Dark steel armor client-side crash

Well, not a full client crash. This tends to kick people out to the main menu.
http://pastebin.com/nv2gsrCB

Actually, this is a dupe of #2743


Application ID and information of how to use it

Align the terms used on the application confirmation page and the application PDF. Instead of just including the application ID in parenthesis in the downable PDF, make it clear that the number is application ID. It would also be great to add information about if they can use their application ID to track status. Please see attached two images.

What priority is this?
Low
Associated civic entity
Seattle
User Group
Resident feedback
Additional Context
This feedback is from a user testing session with a city staff who has not used CiviForm.
Done when:
Applicant ID terms align on the confirmation page and PDF download.

This feels like a small work item the intern can pick up. Adding myself as the "assignee".

@Peggy-Liao 's first submission!! Wahhhoooo!

This appears to be fixed by turning on Universal Name question

Closing this issue per Matt's comment.

Closing this issue per Matt's comment.


ERROR in GoogleSignInComponent is not an NgModule

Is this still maintained?
Broken with Angular 4. You need to update the dependencies:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

Hi! I will put an eye on it in a few days. You are welcome to send me a PR in the meantime.

+1 :-)
Can't really help, I'm a newbie :-(

any update on a potential fix?

any updates on this?

I am not sure whether I have that peer dependencies error or not(but I guess I have it too), but the point is you need to declare it yourself then it will work. I am using Angular 4.3.3 and don't have any problem

Hi! I will look into it this weekend, bare with me!

AOT build is still broken as well, any news/plans on that matter :) ?

Maybe someone else with enough experience can fix this and send a PR? :-)


Fix the Apache Brooklyn source release build

Ignore tests relying on binary files in the source distribution.

👍  LGTM, merging


questions about pretrained vgg16 backbone for training faster rcnn?

I am using vgg16-faster RCNN to train cityscapes. I find that the model trained with mmdetection vgg16 backbone is much lower than the model trained with other faster RCNN implementations (I use https://github.com/jwyang/faster-rcnn.pytorch)  (map 47 for faster-rcnn. pytorch and map 34 for mmdetection). My config file is as follows:
model settings
model = dict(
type='FasterRCNN',
# pretrained='open-mmlab://vgg16_caffe',
backbone=dict(
type='VGG',
depth=16,
num_stages=5,
out_indices=(4,),
frozen_stages=2,
style='pytorch'),
rpn_head=dict(
type='RPNHead',
in_channels=512,
feat_channels=512,
anchor_scales=[8, 16, 32],
anchor_ratios=[0.5, 1.0, 2.0],
anchor_strides=[16],
target_means=[.0, .0, .0, .0],
target_stds=[1.0, 1.0, 1.0, 1.0],
loss_cls=dict(
type='CrossEntropyLoss', use_sigmoid=False, loss_weight=1.0),
loss_bbox=dict(type='SmoothL1Loss', beta=1.0 / 9.0, loss_weight=1.0)),
bbox_roi_extractor=dict(
type='SingleRoIExtractor',
roi_layer=dict(type='RoIAlign', out_size=7, sample_num=2),
out_channels=512,
featmap_strides=[16]),
bbox_head=dict(
type='SharedFCBBoxHead',
num_fcs=2,
in_channels=512,
fc_out_channels=4096,
roi_feat_size=7,
num_classes=9,
target_means=[0., 0., 0., 0.],
target_stds=[0.1, 0.1, 0.2, 0.2],
reg_class_agnostic=False,
loss_cls=dict(
type='CrossEntropyLoss', use_sigmoid=False, loss_weight=1.0),
loss_bbox=dict(type='SmoothL1Loss', beta=1.0, loss_weight=1.0)),)
model training and testing settings
train_cfg = dict(
rpn=dict(
assigner=dict(
type='MaxIoUAssigner',
pos_iou_thr=0.7,
neg_iou_thr=0.3,
min_pos_iou=0.3,
ignore_iof_thr=-1),
sampler=dict(
type='RandomSampler',
num=256,
pos_fraction=0.5,
neg_pos_ub=-1,
add_gt_as_proposals=False),
allowed_border=0,
pos_weight=-1,
debug=False),
rpn_proposal=dict(
nms_across_levels=False,
nms_pre=12000,
nms_post=2000,
max_num=2000,
nms_thr=0.7,
min_bbox_size=0),
rcnn=dict(
assigner=dict(
type='MaxIoUAssigner',
pos_iou_thr=0.5,
neg_iou_thr=0.5,
min_pos_iou=0.5,
ignore_iof_thr=-1),
sampler=dict(
type='RandomSampler',
num=256,
pos_fraction=0.25,
neg_pos_ub=-1,
add_gt_as_proposals=True),
pos_weight=-1,
debug=False))
test_cfg = dict(
rpn=dict(
nms_across_levels=False,
nms_pre=6000,
nms_post=300,
max_num=300,
nms_thr=0.7,
min_bbox_size=0),
rcnn=dict(
score_thr=0.05, nms=dict(type='nms', iou_thr=0.5), max_per_img=300))
dataset settings
dataset_type = 'CityscapesDataset'
data_root = '../data/cityscapes/'
img_norm_cfg = dict(mean=[102.9801, 115.9465, 122.7717], std=[1, 1, 1], to_rgb=False)
train_pipeline = [
dict(type='LoadImageFromFile'),
dict(type='LoadAnnotations', with_bbox=True),
dict(type='Resize', img_scale=(1200, 600), keep_ratio=True),
dict(type='RandomFlip', flip_ratio=0.5),
dict(type='Normalize', **img_norm_cfg),
dict(type='Pad', size_divisor=32),
dict(type='DefaultFormatBundle'),
dict(type='Collect', keys=['img', 'gt_bboxes', 'gt_labels']),
]
test_pipeline = [
dict(type='LoadImageFromFile'),
dict(
type='MultiScaleFlipAug',
img_scale=(1200, 600),
flip=False,
transforms=[
dict(type='Resize', keep_ratio=True),
dict(type='Normalize', **img_norm_cfg),
dict(type='RandomFlip'),
dict(type='Pad', size_divisor=32),
dict(type='ImageToTensor', keys=['img']),
dict(type='Collect', keys=['img']),
])
]
data = dict(
imgs_per_gpu=2,
workers_per_gpu=2,
train=dict(
type=dataset_type,
ann_file=data_root + 'cityscapes_train.json',
img_prefix=data_root,
pipeline=train_pipeline),
val=dict(
type=dataset_type,
ann_file=data_root + 'cityscapes_val.json',
img_prefix=data_root,
pipeline=test_pipeline),
test=dict(
type=dataset_type,
ann_file=data_root + 'cityscapes_val.json',
img_prefix=data_root,
pipeline=test_pipeline))
optimizer
optimizer = dict(type='SGD', lr=5e-3, momentum=0.9, weight_decay=0.0005)
optimizer_config = dict(grad_clip=dict(max_norm=35, norm_type=2))
learning policy
lr_config = dict(
policy='step',
warmup='linear',
warmup_iters=500,
warmup_ratio=1.0 / 3,
step=[8, 11])
checkpoint_config = dict(interval=1)
yapf:disable
log_config = dict(
interval=50,
hooks=[
dict(type='TextLoggerHook'),
dict(type='TensorboardLoggerHook')
])
yapf:enable
runtime settings
total_epochs = 12
dist_params = dict(backend='nccl')
log_level = 'INFO'
work_dir = './work_dirs/city_base'
load_from = "../data/vgg16_caffe.pth"
resume_from = None
workflow = [('train', 1)]
the checkpoint load from is converted from vgg16_caffe pretrained models, so the mean and std is different from mmdetection.
Can you give me some helps?

faster-rcnn.pytorch result:
faster rcnn.pytorch  检测结果
AP for person = 0.4058
AP for rider = 0.4995
AP for car = 0.5373
AP for truck = 0.3605
AP for bus = 0.6009
AP for train = 0.5106
AP for motorcycle = 0.4145
AP for bicycle = 0.4270
Mean AP = 0.4695
mmdetection result:
+------------+------+------+--------+-----------+-------+
| class      | gts  | dets | recall | precision | ap    |
+------------+------+------+--------+-----------+-------+
| person     | 3419 | 1415 | 0.219  | 0.530     | 0.205 |
| rider      | 556  | 364  | 0.326  | 0.497     | 0.300 |
| car        | 4667 | 2551 | 0.367  | 0.671     | 0.360 |
| truck      | 93   | 217  | 0.441  | 0.189     | 0.355 |
| bus        | 98   | 198  | 0.622  | 0.308     | 0.550 |
| train      | 23   | 101  | 0.565  | 0.129     | 0.483 |
| motorcycle | 149  | 173  | 0.336  | 0.289     | 0.289 |
| bicycle    | 1175 | 855  | 0.317  | 0.436     | 0.271 |
+------------+------+------+--------+-----------+-------+
| mAP        |      |      |        |           | 0.352 |
+------------+------+------+--------+-----------+-------+

Are the model architecture and training settings the same?

After my modification, the architecture of the model is basically the same, except that my mmdetection implementation does not use dropout in the Shared fc， but faster rcnn.pytorch used. I visualized the results of some images and found that many of the small objects were not detected.

 For mmdetection, I tried 1x and 2x with random hflip and got approximate results, 0.34+ map.

I modified the anchor scales to [2,4,8,16,32] instead of [8,16,32], and I got map of 50.6. I noticed that resnet50-c4 also set 5 anchors in the same way. Is there a deviation in my understanding of the anchor of mmdetection? For Faster RCNN, we should choose three scales and three ratios, is right?

I don't know whether the definition of anchor in mmdetection is exactly the same as py-faster-rcnn, but after I changed the anchor scales to [4,8,16], I got mAP 47.9.This result looks similar to faster rcnn.pytorch. Faster rcnn.pytorch used the same configuration as py-faster-rcnn, with ANCHOR_SCALES = [8,16,32].  I have reading some code about anchor generator and don't have find something different.  So I don't know why there's this performance gap.  Can you give me some tips?

You can also try the same lr schedule as in faster-rcnn.pytorch, which corresponds to 18 epochs in total and a decrease in the 16th epoch.
For the anchor settings, we just follow the typical convention and also the official release of detectron. On  COCO dataset, C4 models adopts 5 scales and FPN models adopts 3 scales for each level.
I have not read the code of faster-rcnn.pytorch  so that I cannot conclude whether the anchors are the same. You may look into the implementation details of it.

Thank you for your answer. After looking into the implementation of mmdetection and faster-rcnn.pytorch, I think the problem may be on the set of min_pos_iou. when the setting of anchors does not accord with object distribution, min_pos_iou = 0.5 may cause a lot of performance degradation, especially in the absence of FPN structure. I have set min_pos_iou to  0 and  map from 0.34 to  0.41 in the case of 1 x.  I will continue to try 2 x and report on the issue.

Hi,bro!~
I want to train faster rcnn with pretrained vgg16 backbone, could you provide your code for reference?
thx!

Hi,bro!~
I want to train faster rcnn with pretrained vgg16 backbone, could you provide your code for reference?
thx!


[ie/zaiko] add thumbnails from event pages

IMPORTANT: PRs without the template will be CLOSED
Description of your pull request and other information

Hi.
This PR improves #7254.
This commit could not be finished without help of @c-basalt.
Different thumbnails?
The newly added thumbnail, which is from the event page, may be the same as the one of "poster_url", but not always. I participated in a live event that has two different thumbnails before. A thumbnail was shown first and contained something like "there's a special guest coming". The second thumbnail, a few days later, replaced the first one and contained that special guest's name and photo. Fans were pleasantly surprised.
The first thumbnail I saw looks like this:

Local test
Unfortunately, the testing URL "ZAIKO STREAMING TEST" does not seem to work anymore, so I've tested the code with my own ticket only. I don't post the url because posting a real URL might give Zaiko a chance to find the user who shared their ticket.
{
    "id": "...",
    //...
    "thumbnails": [
        {
            "url": "https://d38fgd7fmrcuct.cloudfront.net/pf_1/1_3ybh***",
            "id": "1_3ybh***"
        },
        {
            "url": "https://media.zaiko.io/pf_1/1_3ybi***",
            "id": "1_3ybi***"
        }
    ],

> Get-FileHash *1_3yb*

Algorithm       Hash         Path
---------       ----         ----
SHA256          9B7***       ***
SHA256          C60***       ***

Template

Before submitting a pull request make sure you have:

[x] At least skimmed through contributing guidelines including yt-dlp coding conventions
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Fix or improvement to an extractor (Make sure to add/update tests)
[ ] New extractor (Piracy websites will not be accepted)
[ ] Core bug fix/improvement
[ ] New feature (It is strongly recommended to open an issue first)

Copilot Summary

🤖 Generated by Copilot at 7ad8088
Summary
:sparkles::hammer::mag:

Added support for multiple thumbnails in zaiko.py. Extracted thumbnail URLs from different sources and assigned them IDs based on their filenames.

We'll scrape the web for thumbnails, me hearties, yo ho ho
We'll use the url_basename to name them as we go
We'll fetch them from the JSON and Open Graph, you see
And store them in the zaiko extractor, one, two, three

Walkthrough

Import url_basename function from utils module to generate thumbnail IDs from URLs (link)
Modify _real_extract function to collect two possible thumbnail URLs from player_meta and Open Graph tags (link)
Replace thumbnail field with thumbnails field in info_dict, using url_basename to create a list of dictionaries with id and url keys for each thumbnail (link)
Update test case to reflect the change from thumbnail to thumbnails field, and provide two expected thumbnail URLs (link)

The comment in fee7e257199974f22e810c89a9679aa3db30ae58 about "thumbnails" is wrong:

The field "thumbnails" is ignored, see "sanitize_got_info_dict()".

The real reason to add "thumbnail" back is that the field "thumbnail" will be generated from "thumbnails" in "process_video_result()", so the test will fail if no "thumbnail" specified.

Correct, but no worries about the message, commits will be squashed when merged


[5.1] Yet more minor fixes for property wrappers

Fix two more small issues with property wrappers:

A crash involving the mutating-ness of accessors in the presence of willSet/didSet: rdar://problem/51669930
A SILGen assertion when property wrappers are initialized with tuple values: rdar://problem/50711880

@swift-ci please test


extracted_sql

How-to get sql files ? 1,568 entries
Can you get all 1568 files in a ZIP ?
"Sorry, we had to truncate this directory to 1,000 files. 1,568 entries were omitted from "

Clone the repo locally and you will get all files.


BigNumber: switch from base-10 to base-2 encoding

We currently use a base-10 encoding for big number words. Concretely: we use the range 0..(10^18)-1 for storing one word. We should instead use the full 64 bit. This will have multiple advantages:

allows us to store larger numbers with the same amount of bits
should be significantly faster because we can replace the "mod 10^..." by a simple bit shift
enables us to implement faster gcd: #23

https://github.com/loda-lang/loda-cpp/blob/main/src/include/big_number.hpp
https://github.com/loda-lang/loda-cpp/blob/main/src/big_number.cpp

done


Can't select area in Eskom Loadshedding

Version of the custom_component

Configuration

Add your logs here.

This error originated from a custom integration.

Logger: aiohttp.server
Source: custom_components/eskom_loadshedding/config_flow.py:64
Integration: Eskom Loadshedding Interface (documentation, issues)
First occurred: 7:05:13 AM (14 occurrences)
Last logged: 6:21:00 PM

Error handling request
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_protocol.py", line 435, in _handle_request
    resp = await request_handler(request)
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_app.py", line 504, in _handle
    resp = await handler(request)
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_middlewares.py", line 117, in impl
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 60, in security_filter_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 94, in forwarded_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 81, in ban_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 236, in auth_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 136, in handle
    result = await result
  File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 180, in post
    return await super().post(request, flow_id)
  File "/usr/src/homeassistant/homeassistant/components/http/data_validator.py", line 73, in wrapper
    result = await method(view, request, data, *args, **kwargs)
  File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 110, in post
    result = await self._flow_mgr.async_configure(flow_id, data)
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 286, in async_configure
    result = await self._async_handle_step(
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 374, in _async_handle_step
    result: FlowResult = await getattr(flow, method)(user_input)
  File "/config/custom_components/eskom_loadshedding/config_flow.py", line 64, in async_step_area_search
    self.area_list = areas["areas"]
KeyError: 'areas'

## Describe the bug
A clear and concise description of what the bug is.

## Debug log

<!-- To enable debug logs check this https://www.home-assistant.io/components/logger/ -->

After insert API key get error when select area

Add your logs here.

Hi, unfortunately I can't replicate this on my side. I tested with a few HA versions and I am able to select areas without any issues.
A bit of googling shows that error popping up on a number of integrations with no real solutions. Which HA version are you on and can you try to update? Looks like it might be a SSL problem.
I'm tracking similar problems under #31 so let's shift the conversation there. Closing this one.


Add info about yourself

                            Hello
                            My name is Julia. I am 24 years old. I am studying in the third year of KPI. I like to read, swim.


[TS] incorrect type inference when using curried function with initial state

let fn = produce((draft: number|string) => {}, Math.random())
fn('undefined')

In the example above, the recipe function signature should take precedence over the initial state's type.

I had quite some issues in the past if TS needs to infer the type from something you pass to it (it usually works for direct functions if not to complex, but not beyond that). I think in general you'll be more safe by writing
let fn = produce<number|string>(draft => {}, Math.random())

@mweststrate I've figured it out already. Just not pushed yet.

:tada: This issue has been resolved in version 1.9.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Feature request: AutoMigration support for postgres composite types

Describe the feature
Allow for an AutoMigrateType function, that can take a struct and translate it to a composite type.
Convert the following:
type AnimalGroups struct {
	Cats  int
	Dogs  int
	Birds int
}
db.AutoMigrateType(AnimalGroups{})

to postgres sql:
create type animalgroups as
(
    cats       integer,
    dogs       integer,
    birds      integer
);

And subsequently, convert following:
type AnimalGroups struct {
      Cats   int
      Dogs   int
      Birds  int
      Total  int  // <-- added property
}
db.AutoMigrateType(AnimalGroups{})

to postgres sql:
alter type animalgroups add attribute total integer

Motivation
This simplifies management of types since composite types can not be replaced but instead must be altered.
Which must be managed by the developer by manually writing alter statements.
considerations:
- Derive Scanner / Value implementation based on the type
- Support for dropping attributes
- struct tag for default values / checks
- attribute order should be preserver in SQL types

@saeidee is there a PR for this ?

This feature request is not considered to be implemented, we need more developer requests to implement this feature, or please feel free to submit a feature PR.


What's changed in new version 2.0 ?

I use 1.3.3 and i want to know about changes in new version.

New Features
Infinity Loop
Center item
Smart Speed
Stage Padding
Item Margin
Ability to make almost all options responsive
Various Widths
Callback Events
RTL
YouTube/Vimeo support (fetching thumbnails as well)
Anchors navigation
Merged Items
and more...
http://smashingboxes.github.io/OwlCarousel2/docs/started-welcome.html


Add default command to base Dockerfile template

This specifies in the generated Dockerfile that the jupyter server should be the default application when the instance starts.  I tested this with https://github.com/jakevdp/PythonDataScienceHandbook and verified that the default app was bash before and jupyter after.
Full disclosure: I don't know fully understand the buildpack interrelationships and whether this change works for all the buildpacks.

Nice! Does anyone have opinions on using exec vs shell style CMD?
There is also the legacy docker image, not sure if we should also edit that to have a default command or not. On the one hand it is legacy (don't edit it) on the other hand having different behaviour might confuse users. I'd be +0 to also modify the legacy docker file.

This is great! I think it should be the exec style variant (so CMD ["jupyter", "notebook", etc]) than the current one because sh doesn't do a great job of passing signals through to things it starts sometimes, causing TERM and KILL signals to take a while to pass through.
+1 to not modifying the legacy dockerfile, it should probably remain frozen and we should remove support for it at some point.

And congratulations on your first PR, @AaronWatters!

Second friendly bump :-)
What @yuvipanda is talking about by "exec style" is documented here: https://docs.docker.com/engine/reference/builder/#cmd

ok made the exec-style change and resolved conflicts, LMK if that looks OK @yuvipanda

ping - I think this is ready for merge, but I won't do it since I've now touched the PR. cc @betatim @yuvipanda who have commented on this PR earlier

Congrats @AaronWatters, your first PR got merged!


Added 1.10 example for nginx

Tested and verified NGINX package v1.10.3 for DC/OS 1.10 beta1 version.

https://jira.mesosphere.com/browse/DCOS_COMMUNITY-136

I think now we can close issue #106 .


Crash when Characters die from fall damage during server prediction

What you were trying to do
Move a deer with high movement speed off a cliff
What actually happened
Server crashes with ConcurrentModificationException, similar to (but distinct from) #2468 since it's a different collection and the event is dispatch from the update method of a component system rather than a BT Node.
How to reproduce
Kill a (non-player) character with fall damage (via CharacterMoveInputEvent).
Log details and game version
"main@1" prio=10 tid=0x1 nid=NA runnable
  java.lang.Thread.State: RUNNABLE
      at org.terasology.logic.characters.ServerCharacterPredictionSystem.onDestroy(ServerCharacterPredictionSystem.java:97)
      at org.terasology.logic.characters.ServerCharacterPredictionSystemMethodAccess.invoke(Unknown Source:-1)
      at org.terasology.entitySystem.event.internal.EventSystemImpl$ByteCodeEventHandlerInfo.invoke(EventSystemImpl.java:506)
      at org.terasology.entitySystem.event.internal.EventSystemImpl.sendStandardEvent(EventSystemImpl.java:269)
      at org.terasology.entitySystem.event.internal.EventSystemImpl.send(EventSystemImpl.java:260)
      at org.terasology.entitySystem.entity.internal.PojoEntityManager.destroy(PojoEntityManager.java:538)
      at org.terasology.entitySystem.entity.internal.BaseEntityRef.destroy(BaseEntityRef.java:138)
      at org.terasology.logic.health.EntityDestructionAuthoritySystem.onDestroy(EntityDestructionAuthoritySystem.java:32)
      at org.terasology.logic.health.EntityDestructionAuthoritySystemMethodAccess.invoke(Unknown Source:-1)
      at org.terasology.entitySystem.event.internal.EventSystemImpl$ByteCodeEventHandlerInfo.invoke(EventSystemImpl.java:506)
      at org.terasology.entitySystem.event.internal.EventSystemImpl.sendStandardEvent(EventSystemImpl.java:269)
      at org.terasology.entitySystem.event.internal.EventSystemImpl.send(EventSystemImpl.java:260)
      at org.terasology.entitySystem.entity.internal.BaseEntityRef.send(BaseEntityRef.java:145)
      at org.terasology.logic.health.HealthAuthoritySystem.doDamage(HealthAuthoritySystem.java:185)
      at org.terasology.logic.health.HealthAuthoritySystem.checkDamage(HealthAuthoritySystem.java:200)
      at org.terasology.logic.health.HealthAuthoritySystem.onLand(HealthAuthoritySystem.java:241)
      at org.terasology.logic.health.HealthAuthoritySystemMethodAccess.invoke(Unknown Source:-1)
      at org.terasology.entitySystem.event.internal.EventSystemImpl$ByteCodeEventHandlerInfo.invoke(EventSystemImpl.java:506)
      at org.terasology.entitySystem.event.internal.EventSystemImpl.sendStandardEvent(EventSystemImpl.java:269)
      at org.terasology.entitySystem.event.internal.EventSystemImpl.send(EventSystemImpl.java:260)
      at org.terasology.entitySystem.entity.internal.BaseEntityRef.send(BaseEntityRef.java:145)
      at org.terasology.logic.characters.KinematicCharacterMover.walk(KinematicCharacterMover.java:647)
      at org.terasology.logic.characters.KinematicCharacterMover.updatePosition(KinematicCharacterMover.java:559)
      at org.terasology.logic.characters.KinematicCharacterMover.step(KinematicCharacterMover.java:103)
      at org.terasology.logic.characters.ServerCharacterPredictionSystem.stepState(ServerCharacterPredictionSystem.java:168)
      at org.terasology.logic.characters.ServerCharacterPredictionSystem.onPlayerInput(ServerCharacterPredictionSystem.java:126)
      at org.terasology.logic.characters.ServerCharacterPredictionSystemMethodAccess.invoke(Unknown Source:-1)
      at org.terasology.entitySystem.event.internal.EventSystemImpl$ByteCodeEventHandlerInfo.invoke(EventSystemImpl.java:506)
      at org.terasology.entitySystem.event.internal.EventSystemImpl.sendStandardEvent(EventSystemImpl.java:269)
      at org.terasology.entitySystem.event.internal.EventSystemImpl.send(EventSystemImpl.java:260)
      at org.terasology.entitySystem.entity.internal.BaseEntityRef.send(BaseEntityRef.java:145)
      at org.terasology.flexiblemovement.FlexibleMovementSystem.update(FlexibleMovementSystem.java:37)
      at org.terasology.engine.modes.StateIngame.update(StateIngame.java:176)
      at org.terasology.engine.TerasologyEngine.mainLoop(TerasologyEngine.java:413)
      at org.terasology.engine.TerasologyEngine.run(TerasologyEngine.java:368)
      - locked <0x1906> (a org.terasology.engine.TerasologyEngine)
      at org.terasology.engine.Terasology.main(Terasology.java:152)

I looked into this issue and from what I can tell the "ConcurrentModificationException" is being thrown because the deer and the player are in the same hash map called "characterStates", which is in the "ServerCharacterPredictionSystem" class. One thread is using the hash map for the player while the other is deleting the deer.
Possible fixes:

Put a synchronized block in the update method in the "ServerCharacterPredictionSystem" class.
Not Recommended, but confirmed to be a working solution through my testing.

Change the "characterStates" hash map to a concurrentMap which is thread safe.
Also tested to be a confirmed fix through my testing. Not sure about the performance difference, but it shouldn't be much.
I'm still trying to write a test to see the performance difference.

@Cervator mentioned that we could perhaps wait a tick to delete the deer. I'm not sure about this because I think the class iterates through the hash map 60 times every second. It might also cause issues with some other code relying on the player being somewhere.
That's my 2 cents, but let me know if I am mistaken.

Out of all 3 choices, I would say that number 2 would be the best option, but I need guidance to know for sure. Any help is appreciated.

So I ran this in debug mode and successfully caught the exception with a break point. For anybody curious for an easy way to replicate: I spotted a tall tree near spawn, flew up there, spawnPrefab deer, then started chopping leaf blocks below the deer till it fell off. Poor deer.
@Elyahu41 apologies if I'm about to write up the exact thing you already did, unsure 😅 I'm also not 100% sure this is correct since I rarely get to actually delve deeper in the code these days.
As often is the case with ConcurrentModificationException it is actually not a threading issue at all.
If you look at ServerCharacterPredictionSystem.update then we iterate through characterStates in a loop which is handled by the main game thread. For each entity you replay their last input. In the case of the deer that then caused its death, meaning as part of that input event handling the main thread ends up in onDestroy also inside ServerCharacterPredictionSystem where the destroy method removes an entry from characterStates concurrently with the loop going through the same Map - Collections don't like that.
Synchronizing or using a concurrent map gets around the issue, but at a cost (synchronization is generally considered expensive, plus opens the risk of deadlocks). There are other options as this is a pretty common issue. See https://www.baeldung.com/java-concurrentmodificationexception for some more on the topic and several solutions.
I like the 3.2 option, which is what I had in mind by waiting to remove entities (by "next tick" I just meant literally next game frame, not a full second or anything). A quick conceptual way to do that would be "For each entity removed during the processing of the for loop put it into a different Collection, then continue iterating, and after the loop finishes go remove all those entities". Another typical fix is 3.1 which also uses a different collection, but instead of one aiming for thread safety (expensive) it just iterates and removes in a more safe fashion - but that assumes the removal happens directly inside that loop, which it doesn't (it calls out to event handling which comes in through a side door indirectly and later hits the active collection)
So! That brings me to two thoughts:

Should server prediction even predict removals in the first place? I really don't know, but I remember from other games that lag prediction at least sometimes did not actually cover removal of things. If you did actually lag then a creature would just kinda keep sliding in some direction till the server caught up and broadcast the death. I guess this might be done for :reasons: perhaps this exact problem? A predicted death could "leak" to a player what might happen next - loot drops? It actually wasn't its final form and it just grows stronger? You could conceivably lag yourself on purpose to see what prediction would reveal.

In onDestroy rather than remove the entity should it be added to a class level collection, that at the end of the loop in update we then go ahead and all remove at the same time? Trouble here is stuff like: is that method ever executed not from the event stream that comes out of the update method there? Or is there an impact later in the loop if a dead deer isn't removed? 🤔

My concern with option 1 (seeing past a removal) is probably just me being silly, I dunno. It might be worth trying to fix it via option 2 while leaving in place a TODO considering whether or not removal should truly be part of prediction. We could try to discuss that elsewhere but the prediction system has been untouched for years so I'm not sure anybody would be able to participate 🤷


[czmq] code clean up

Add parameter AUTO_CLEAN to vcpkg_copy_tools and delete redundant function vcpkg_clean_executables_in_bin

[x] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Tesed feature tool in the following triplet:

x86-windows
x64-windows
x64-windows-static

Thanks :)


remove an unnecessary cast

This fixes a ClassCastException I'm running into on my 4.1.x based deployment. This patch is no longer needed after the admin backend changes on master, but I don't feel it's worth the work to back port that patch set to 4.1.x.

Very good.


⚠ Bofrost.de has degraded performance

In 85662fb, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 915 ms

Resolved: Bofrost.de performance has improved in 9cc2c75.


Convenience Methods

Motivation and Context
Why is this change required? What problem does it solve?:
Added numerous convenience methods for general usage, major changes to the checkout column data reader API to allow support of arbitrary column layouts.
Description
Describe your changes in detail:

Added log method to checkout instances
added diff method to Repository class
added CLI diff command

API Changes to Checkout __getitem__() and get() methods.
Checkout object can be thought of as a "dataset" ("dset") mapping a
view of samples across columns.
>>> dset = repo.checkout(branch='master')
>>>
# Get an column contained in the checkout.
>>> dset['foo']
ColumnDataReader
>>>
# Get a specific sample from ``'foo'`` (returns a single array)
>>> dset['foo', '1']
np.array([1])
>>>
# Get multiple samples from ``'foo'`` (returns a list of arrays, in order
# of input keys)
>>> dset[['foo', '1'], ['foo', '2'],  ['foo', '324']]
[np.array([1]), np.ndarray([2]), np.ndarray([324])]
>>>
# Get sample from multiple columns, column/data returned is ordered
# in same manner as input of func.
>>> dset[['foo', '1'], ['bar', '1'],  ['baz', '1']]
[np.array([1]), np.ndarray([1, 1]), np.ndarray([1, 1, 1])]
>>>
# Get multiple samples from multiple columns\
>>> keys = [(col, str(samp)) for samp in range(2) for col in ['foo', 'bar']]
>>> keys
[('foo', '0'), ('bar', '0'), ('foo', '1'), ('bar', '1')]
>>> dset[keys]
[np.array([1]), np.array([1, 1]), np.array([2]), np.array([2, 2])]

Arbitrary column layouts are supported by simply adding additional members
to the keys for each piece of data. For example, getting data from a column
with a nested layout:
>> dset['nested_col', 'sample_1', 'subsample_0']
np.array([1, 0])
>>>
# a sample accessor object can be retrieved at will...
>>> dset['nested_col', 'sample_1']
<class 'FlatSubsampleReader'>(column_name='nested_col', sample_name='sample_1')
>>>
# to get all subsamples in a nested sample use the Ellipsis operator
>>> dset['nested_col', 'sample_1', ...]
{'subsample_0': np.array([1, 0]),
 'subsample_1': np.array([1, 1]),
 ...
 'subsample_n': np.array([1, 255])}

Retrieval of data from different column types can be mixed and combined
as desired. For example, retrieving data from both flat and nested columns
simultaneously:
>>> dset[('nested_col', 'sample_1', '0'), ('foo', '0')]
[np.array([1, 0]), np.array([0])]
>>> dset[('nested_col', 'sample_1', ...), ('foo', '0')]
[{'subsample_0': np.array([1, 0]), 'subsample_1': np.array([1, 1])},
 np.array([0])]
>>> dset[('foo', '0'), ('nested_col', 'sample_1')]
[np.array([0]),
 <class 'FlatSubsampleReader'>(column_name='nested_col', sample_name='sample_1')]

If a column or data key does not exist, then this method will raise a KeyError.
As an alternative, missing keys can be gracefully handeled by calling :meth:get()
instead. This method does not (by default) raise an error if a key is missing.
Instead, a (configurable) default value is simply inserted in it's place.
>>> dset['foo', 'DOES_NOT_EXIST']
-------------------------------------------------------------------
KeyError                           Traceback (most recent call last)
<ipython-input-40-731e6ea62fb8> in <module>
----> 1 res = co['foo', 'DOES_NOT_EXIST']
KeyError: 'DOES_NOT_EXIST'

Screenshots (if appropriate):
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

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

Is this PR ready for review, or a work in progress?

[ ] Ready for review
[x] Work in progress

How Has This Been Tested?
Put an x in the boxes that apply:

[ ] Current tests cover modifications made
[x] New tests have been added to the test suite
[x] Modifications were made to existing tests to support these changes
[ ] Tests may be needed, but they are not included when the PR was proposed
[ ] I don't know. Help!

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have signed (or will sign when prompted) the tensorwork CLA.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

@hhsecond please review.


ClamAv errors are not handled properly

ClamAv error responses are not taken into account. They cause a notice (which is often promoted to an exception) in the constructor of ScannedFile.
How to reproduce: try to scan a file that the clamav server cannot read (permission denied).

Thanks for reporting this. It's now fixed in https://github.com/sineflow/clamav/tree/v1.1.0


Link updates: Matrix Routing and get key

Link updates: How to get AzMaps key and remove Preview from Post Route Matrix urls
#sign-off

@outinyman : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Verified the updates.
@outinyman Thanks for your contribution.


US-55: Implement realistic terrain generation

Description
The terrain should be generated so the world looks like a realistic map with naturally formed islands
Tasks

[ ]

As we spoke about in person it would be nice if you refactored the MapGenerator interface so we can create maps with different dimensions not just squares.


Fix reaction appears for deleted message

Details

Fix reaction appears for deleted message
Fixed Issues

$ https://github.com/Expensify/App/issues/23325
PROPOSAL: https://github.com/Expensify/App/issues/23325#issuecomment-1644893174
Tests

Let User A Create a new room where User B is a member.
Using User B click on the room avatar on top.
Verify that user B can't change the room name
From userA copy url to the rename page and paste to the userB browser.
Verify that user B can't change the room name

[x] Verify that no errors appear in the JS console

Offline tests
Bug is not related to offline mode itself. No extra steps needed.
QA Steps

Let User A Create a new room where User B is a member.
Using User B click on the room avatar on top.
Verify that user B can't change the room name
From userA copy url to the rename page and paste to the userB browser.
Verify that user B can't change the room name

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots

Web
https://github.com/Expensify/App/assets/16502320/5a603df3-52e1-49b9-9688-5ccc61665254

Mobile Web - Chrome
https://github.com/Expensify/App/assets/16502320/1035d4e9-2c4c-4974-82a5-5fe7ad3d1603

 Mobile Web - Safari
https://github.com/Expensify/App/assets/16502320/cecea6ac-4c1c-4336-a716-1dfbf6f2bbc4

Desktop
https://github.com/Expensify/App/assets/16502320/ab5fca98-e802-468a-b1c7-ae2693cbe7bf

IOS
https://github.com/Expensify/App/assets/16502320/f54f1231-a389-49c3-8af4-b4b9661208c4

Android
https://github.com/Expensify/App/assets/16502320/e956cff9-f75c-4069-abf2-5a3fba5c67ec

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/MonilBhavsar in version: 1.3.48-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.3.48-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/MonilBhavsar in version: 1.3.49-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Beamanator in version: 1.3.49-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


new Common Crawl year indices

The cdx_toolkit code gets Common Crawl index dates by parsing the index name, CC-MAIN-2020-50
The new (old) indices don't have a week number:
"id": "CC-MAIN-2012",
"id": "CC-MAIN-2009-2010",
"id": "CC-MAIN-2008-2009",

which leads to the bug that 2012 is ignored, and I think 2009 and 2008 are treated as if they have a week number of 20.

@wumpus
Are you using some default week number for 2012 as well? I tried this and it returns nothing:
cdx = cdx_toolkit.CDXFetcher(source='cc')
url = "www.cnn.com/*"
objs = cdx.iter(url, from_ts='201207', to='201304', filter=['status:200', 'mime:text/html'])

However, objs = cdx.iter(url, from_ts='201201', to='201204', filter=['status:200', 'mime:text/html']) can return some results. I wonder does this command search the whole "CC-MAIN-2012" index? i.e. is there any part of the index that's not searched if I run this command for every two months (e.g.<PHONE_NUMBER>02,<PHONE_NUMBER>04)?

I don't think there are any crawl results after July 2012.
BTW the bug I mention above was solved a while ago.


Feature/add non default languages

@kwin  go ahead and review :)

@kwin  I did a major refactor, it now takes away the limitation of having the same node names and it also made everything more clear between which resources come from the resource provider and which are normal message entries (and no more labels ;))
A lot of the issues logged were also fixed while refactoring. You are welcome to test it out, I will make a release afterwards (unless you want a release on the current main branch)

Please go ahead. Only this minor thing appeared during our testing.

@kwin  perfect, released! thanks for the thorough review :)


i23: implement optical edge detection algorithm in dodal device

Existing optical edge detection algorithm is in an adPython areadetector plugin (it is a relatively simple script that calls into OpenCV). adPython is becoming unsupported at diamond long term.
Following discussion in MX office, pull this algorithm into the dodal layer to avoid introducing adPython onto i23.
Note: refers to pin tip centring not loop centring.

There have been historic issues in the CPU usage of the plugin. We should be mindful of this. We may be able to get away with not doing it for every frame but just doing it once for the current frame


Seems like it isn't possible to use websocket and api clients at same time

I'm looking to use the websocket for getting ticker into and the api to make orders but the api gives:  Signature for this request is not valid when using websocket.  Is this a limitation or can you give a simple example of how to get this two pieces to work in harmony?

I am using the web socket for klines, and user data socket, and place buy and sells with the api.   Perhaps you can share a code sample to help us see what you are doing?

I was able to get around the issue using binance websocket api directly.  I
feel the problem with the python lib was that it passed in client with
credentials to both the websocket connection and was being used for all
requests.  The websocket would work but then all request had an error about
my signature.  I be already made the code changes so can't reproduce at
moment for easy traceback or share my code. By the way, python-binance is
awesome.
Best Regards
On Feb 16, 2018 9:35 PM, "Chingis Khan"<EMAIL_ADDRESS>wrote:

I am using the web socket for klines, and user data socket, and place buy
and sells with the api. Perhaps you can share a code sample to help us see
what you are doing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sammchardy/python-binance/issues/182#issuecomment-366413272,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOdPmbw2qlbZPOP9HQcDmZHcFqNabq0pks5tVkjngaJpZM4SHpDw
.

@Chingis-Khan I was wondering how you use your websocket kline data? I'm currently using the API for kline data and I get back the proper intervls as I cal for, but I'm trying to convert over to websocket and I'm getting second by second no matter what interval time I pass to it. Any suggestions on how to catch the proper intervals? I'm getting connection problems that I think are from using the API too often in issue #195

Web sockets feed data to you when they are ready, in theory should be once a seconds from binance but it's not 100% reliable becasue of the internet being not 100% reliable among other things.
So I have it feed to a queue and poll the queue when I am ready.
        self.bm = BinanceSocketManager(self.binance_client)
        self.socket_queue = queue.Queue()
        self.kline_conn_key = self.bm.start_kline_socket(self.pair_code, self.kline_socket_error_check, KLINE_INTERVAL_1MINUTE)
        self.bm.start()

    def kline_socket_error_check(self, msg):
        if msg['e'] == 'error':
            logger .info('Error on ticker socket: %s ' % (self.pair_code)
           # handle errror
        else:
            self.socket_queue.put(msg, block=False)

Hi, I have an issue related to this topic:
I couldn't figure out how can I get prices from websocket and put an order.
My code sample goes like this:
`class Klass(object):
def __init__(self, bm):
    self.bm = bm
    self.bm.start()

def depth(self):
    depth_key = self.bm.start_depth_socket('NEOBTC', self.process_message_lean, depth=WEBSOCKET_DEPTH_10)

    order_book = {
        'buys': [item[:2] for item in msg['bids']],
        'sells': [item[:2] for item in msg['asks']]
    }

    table_lines = []
    for item, item2 in zip(order_book["buys"], order_book["sells"]):
        item.append(item2[0])
        item.append(item2[1])
        table_lines.append(item)

    return(table_lines)`

then putting buy order:
`DX = Klass(BinanceSocketManager(Client))
buy_price=DX.depth()[0][1]
binance_client.order_limit_buy(timeInForce=TIME_IN_FORCE_GTC,
symbol='NEOBTC',
quantity=2,
price=buy_price
)`


Show default values in the option editor

The default value for the period in the chuck norris kamelet is 10000
https://github.com/apache/camel-kamelets/blob/main/chuck-norris-source.kamelet.yaml#L24
It would be nice to see this value somewhere in the options editor (maybe in description, tooltip, or as a text below the input field.)

A text below textfiled makes UI noisy, so I have added to the Help tooltip


Create Release v0.28.0

Description
Create Release v0.28.0
Auto-generated summary

🤖[deprecated] Generated by Copilot at a1ef43b
Summary
:green_circle::yellow_circle::red_circle:

Added support for new channel 0.28 and deprecated channel 0.27 in versions.yaml. This file defines the available and outdated versions of radius for clients.

radius evolves
New channel joins, old ones fade
Autumn of versions

Walkthrough

Add new channel 0.28 to the supported list and move channel 0.27 to the deprecated list in versions.yaml (link)

Should the release notes be added as part of this PR?

@AaronCrawfis, I read the release documentation and thought it was a 2 step process. I'll review and update it.


⚠ Samsung has degraded performance

In 7ab7651, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2525 ms

Resolved: Samsung performance has improved in 9e8eae4.


Installation fails

I'm following instructions found here https://imperialcollegelondon.github.io/PyProBE/user_guide/installation.html
All seems to install just fine but when I try to load the module with
import pyprobe
I get:
[WARNING] citations.read_citations(76): Citations could not be read because the 'pybtex' library is not installed. Install 'pybamm[cite]' to enable citation reading.
Also, my I suggest that all dependencies are kept in pyproject.toml only. The requirements.txt seems redundant to me?

Thanks @jakobhaervig, that user warning is from PyBaMM (a dependency). It causes no runtime issues but I will try to supress it.

Regarding requirements.txt, this is auto-generated from the pyproject.toml using pip-tools which ensures that the dependencies are compatible. This way they are not independent of each other. Having it in the repo removes the  pip-compile step in the installation.

Step 3 and the requirements.txt is redundant. You may have to specify versions in pyproject.toml directly. You can basically leave it out (step 3) and simply do pip install ., which will read dependencies from pyproject.toml. Only having dependencies listed one place is cleaner in my opinion.


Question about EXACT MATCH ACC

Hello, your work has made amazing progress on the spider data set, LGESQL + ELECTRA achieves 75.1 on the dev set, my question is whether this result is beam acc or sql acc?

It is sql acc (exact set match accuracy without values, but we use a small widely-used trick which returns the first prediction in the beam without execution error). Not Beam Acc (as long as one prediction in the beam is correct compared to the ground truth, we consider it correct). Acctually, the beam acc can be larger than 80% w.r.t exact set match without values.

Thank you very much😁


adding allOf to PAP schema and adding PAP:CLEAR

This PR adds PAP:CLEAR to the examples, schema, and adoc files. It also adds an AllOf clause to the PAP JSON schema. Per conversation with Chris, there may be some differences in the PAP schema in contrast to the CUI schema for a couple reasons. The CUI spec explicitly restricts some items, such as definition_type, while the PAP spec does not. However, both schemas include required elements in the AllOf clause e.g. properties, required, etc.

@clenk @ghackford-mitre,

The CUI spec explicitly restricts some items, such as definition_type, while the PAP spec does not.

Those properties are deprecated, and shouldn't be used with extension definitions.  If the PAP spec doesn't explicitly say the same thing as the CUI one, we should update the PAP spec.

Also I have never seen "$definitions".  If should be just "definitions".
Also, drop the Word doc file - we won't be keeping that up to date

Regarding "definitions" - My IDE gives me a big warning when using "definitions" saying that value is deprecated. Checking up on it, I found this: https://json-schema.org/draft/2019-09/release-notes#semi-incompatible-changes. Should I still use regular "definitions" over "$defs" for consistency with the other schema?

I guess we will go with $defs :-)


Encode & Decode anything as root

Issue
Currently, we are only able of encoding and decoding records as root, but we cannot encode and decode primitive values, arrays, or maps.
Proposal
Be able of encoding and decoding (and generating the associated schema) for any type of value:

record
union (not only with records)
enums
primitives (boolean, int, long, float/double, enum, string, bytes, null)
fixed

Released in v2.0.0


Make the fire first aid kit orange

About the PR

Title.
Why / Balance

CM randomly picked between them but that could be confusing for clarity. Orange fits more with what it treats (fire) and better matches stuff like the health analyzer.
Technical details

Media

Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

no cl just sprit

Unrando'd and just changed it to the orange variant


Can't change focus inside app running under Wayland

Software Versions

Python: v3.8.3 (default, May 17 2020, 18:15:42)
OS: Manjaro Linux
Kivy: v2.0.0rc3, git-20c14b2, 20200615
Kivy installation method: pip install

Describe the bug
Focus can't be changed from one application UI element to another.
Expected behavior
User can focus on any UI element.
To Reproduce
Run kivy app under Wayland.
Code and Logs and screenshots
w_test.py
from kivy.app import App

class w_testApp(App):
    title = "Wayland test"

    def __init__(self):
        super().__init__()

if __name__ == "__main__":
    w_testApp().run()

w_test.kv
GridLayout:
    cols: 1
    TextInput:
        id: one
        size_hint_y: None
        multiline: False
        height: 32

    TextInput:
        id: two
        size_hint_y: None
        multiline: False
        height: 32
        focus: True

Example App:
Only TextInput id:two will be active.
You can't change focus on TextInput id:one
Additional context
I developing my application on Manjaro Linux.
I used KDE plasma that run under Xorg and everything was fine.
But I lost ability to change focus between elements in my app when switch my system on plasma wayland mode.

Problem was on wayland side.
Fixed in new version wayalnd(1.18.0)/kwayland(5.73.0)


Feature request: add ability to manually check for updates

Related with this #12817, it will be cool if there will be ability to have full control over app update process. I mean disable automatic updates but have ability to check for updates manually.
Right now I need to go in "User Settings", enable automatic update, restart VSCode, update VSCode, go to "User Settings" again and disable automatic updates. Over complicated, isn't it.
And we can see from many other apps, many of them have such ability to disable automatic updates, but preserve ability to check for updates manually (or install them in other time, if user wants this).

We made this choice because we always want our users as much on the latest as possible.


BeamNGpy should allow to control existing vehicles

After joining a running BeamNG instance, BeamNGpy should be able to query which vehicles are there, and taking control over them.

Implemented in v1.4


Add option return user details on login

I'm finished few projects where token from login view is useless, because we use session auth for SPA and oAuth2 for mobile applications(https://django-oauth-toolkit.readthedocs.io/en/latest/).
I'm though this PR can be useful for people who want make more elegant auth/get_user_data module in their SPA app. Without this update they will must get user data in 2 steps: login, get_user.

Coverage increased (+0.04%) to 96.97% when pulling ac4fdfcb8487ae3c626d6d7e3a4625fccce99a13 on Akay7:AddOptionReturnUserDetailsOnLogin into 8c58782e30a80cacb78bd671bc798d7751ccbf1f on Tivix:master.

Coverage increased (+0.04%) to 96.967% when pulling f48d19b3ba2b81904b1722163aa0d7e3c05de45c on Akay7:AddOptionReturnUserDetailsOnLogin into 8c58782e30a80cacb78bd671bc798d7751ccbf1f on Tivix:master.

Coverage increased (+0.07%) to 97.001% when pulling a08155a4810d262b0e0c9a4f8d03438b32a847ae on Akay7:AddOptionReturnUserDetailsOnLogin into 8c58782e30a80cacb78bd671bc798d7751ccbf1f on Tivix:master.

Hi,
as a user of rest-auth, thanks for the contribution!
This repo is not maintained anymore, so the development moved to dj-rest-auth. (reference: https://github.com/Tivix/django-rest-auth/issues/568)
It may be best, if you move this PR there. (and upgrade to using dj_rest_auth)
new repo link: https://github.com/jazzband/dj-rest-auth (I'm not the upkeeper of that repo, it just makes sense for me to help you merge your PR)
Many Thanks,
Barney


GOV endpoints

It would be ideal if endpoints could be overloaded for alternate Azure environments like Gov.  I believe just the TLD changes to .us (instead of .com), but need to validate that.  I'll try to work on a PR, but wanted to mention her as well.

It's not just a change from .com to .us.  You need to change the endpoints for both AAD and ARM.  The below file has a mapping of these endpoints for each of the different Azure clouds.
https://github.com/projectkudu/ARMClient/blob/master/ARMClient.Authentication/Constants.cs
I'd recommend exposing a new environment property under credentials section in the yaml which maps to Public, China, Germany, and USGov.  Internally within the exporter, the code can look up the appropriate endpoint.

Do Microsoft have a library for this?

I don't know if there is an official library for this but in Azure PowerShell, there is a cmdlet called Get-AzureRmEnvironment which returns back the environment settings for each of the clouds.  Looking at the PowerShell code, it maps back to this file -- which is probably more official than the link I posted above.
https://github.com/Azure/azure-powershell/blob/8bd4c84a1b51b86ffc2d8977d5dcc438df480afa/src/Common/Commands.Common.Authentication.Abstractions/AzureEnvironmentConstants.cs

That's CSharp which won't help us much. Is there a java one somewhere?

I checked the Azure SDK for Java but I didn't see anything.  I would check with the folks here.
https://github.com/Azure/azure-libraries-for-java


dep init fails

Hi guys,
I don't seem to be able to run dep init
(precise)babube@localhost:~/go$ pwd
/home/babube/go
(precise)babube@localhost:~/go$ echo $GOPATH
/home/babube/go
(precise)babube@localhost:~/go$ dep init -v
determineProjectRoot: /home/babube/go not in any $GOPATH

This is really weird, what version of dep are you running?

Meet the same problem

I am a go newbie so I am unsure how to get the version of the installed packages in go, I just installed dep today with go get

start with https://golang.org/doc/code.html
you are using the wrong path, the GOPATH specifies the root of the workspace, your code is not supposed to be there but in a subdirectory under src

thanks it is working, I misunderstood the instructions to use dep

For more details, see #313


Bonsai Link at bottom of article is irrelevant?

At the bottom of the article there is a link to Bonsai, but I don't see how it applies.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 95f29e0e-bc41-bcd5-6c59-d661a60b189a
Version Independent ID: 5fc05205-d317-51e8-08fd-f61caa0cf007
Content: Using a service principal to authenticate - Azure Quantum
Content Source: articles/optimization-authenticate-service-principal.md
Service: azure-quantum
Sub-service: core
GitHub Login: @anraman
Microsoft Alias: anraman

@lwoodyiii I am afraid I´m missing the link. What paragraph is it in?


Identify real-time API connections to external data we may wish to use

Real-time connections to Wikidata, DbPedia, LOC, FAST, and Digital Collections were employed. Separate issues will be created for other work phases/cycles..


Add List and List with Debug features

This corresponds to PNut's CTRL+L and CTRL+I shortcut keys.

Propeller Tool's Debug Mode plays a role in modifying this to be one feature.
Also, the Info window displays much of the information shown in the list.  Consider enhancing the Info display to provide the rest of the details and to allow saving the list file to storage.


.txt input error NVivo

I seem to be having problems with the online converter - using a .vtt file downloaded from the Teams chat, when I import the converted .txt file into NVivo 12 and use the options "One transcript row for each tab-delimited line, File includes header row," I get an error in the Data Preview of  "Line 2 : Each line must contain the same number of tab characters."
I've tried with different .vtt files and get the same error, also checked the files themselves and tried to adjust Line two but no success.  I can't anonymise the files to attach unfortunately due to the transcript content but hoping you can help!

Please would you edit your files manually to provide something anonymous to test with. So long as I have something that has the same behaviour I can work on it.

This seems to be due to the change in Teams format that we have detected separately. Now fixed.


qs.parse is parsing the string with + properly

<EMAIL_ADDRESS>is converting the email's + to a space.
{email: "abc<EMAIL_ADDRESS>externaluserid: "454545454", token: "dtc5uwvmkC", order: "165615"}
It should keep + in email instead of converting this to a space.

+ is a space in a query string. You need to properly encode the value.
https://stackoverflow.com/questions/2678551/when-to-encode-space-to-plus-or-20

@sindresorhus  Not working

This is the behavior of the browser API too:
> let x = new<EMAIL_ADDRESS>x.get('email')
< "abc<EMAIL_ADDRESS>
And this is how it should have looked like if it had been properly encoded:
> let x = new URLSearchParams({email<EMAIL_ADDRESS>x.toString()
< "email=abc%2B123%40gmail.com"

In such cases, I think using Base64 is much safer than passing it as a raw string.