md: don't auto-link from headers

For example currently an H2 title in the dvc checkout cmd ref has the following md code:
## Automating `dvc checkout`

and looks like:

https://dvc.org/doc/command-reference/checkout#automating-dvc-checkout
"dvc checkout" should not be linked in this case, as it's part of a header. It should just be rendered as <code> per normal md rules. Similar to:

https://dvc.org/doc/user-guide/running-dvc-on-windows#enabling-paging-with-less

Notice that in both cases the right side nav/ToC widget renders the plain text including the backquotes. Not the same issue but should those internal links remove special characters?

@jorgeorpinel I think we want this to work for blog - check Heartbeats - we organize question with titles. Do you think this a big problem for docs? I think it's fine even if we have links in title? Please, feel free to reopen if it's critical.

I´m only seeing actual md links in community gem posts

Here I was referring to disabling auto linking `dvc command` strings.

Yup, you are right. Still, I think it's fine to auto-link if you use inline code even in titles. Why do you consider this as a problem?

Nah, it was a very minor change request. We can just avoid useing backquotes in docs titles


Fix unhandled configuration error

Both Signed API and Airnode feed validate the configuration before starting the service and throw an error in case there is an issue. This error is not handled as of now.

Closed by https://github.com/api3dao/signed-api/pull/217


Missing DNS validation recoed

I'm using terraform aws acm module - https://registry.terraform.io/modules/terraform-aws-modules/acm/aws/latest
the code is
`module "acm" {
source  = "terraform-aws-modules/acm/aws"
version = "~> 4.0"
domain_name  = "barak-kalai.com"
zone_id      = data.terraform_remote_state.zones.outputs.route53_zone_zone_id
validation_method = "DNS"
subject_alternative_names = [
"*.barak-kalai.com"
]
wait_for_validation = true
tags = {
Name = "barak-kalai_acm"
}
}`
plan works ok  but apply has issues
Terraform cloud create the certificate request ok
Terrafor create the validation certificate id=Z101159938VGOI878SQ13__a26f4216a9f84d4618c581f811c0a917.barak-kalai.com._CNAME but when it tries to validate i get missing *.barak-kalai.com DNS validation record: _a26f4216a9f84d4618c581f811c0a917.barak-kalai.com

I ran into a similar issue with the following:
module "acm-cert-two-zones" {
  source  = "terraform-aws-modules/acm/aws"
  version = "~> 4.0"

  domain_name = "api.domainone.com"
  zone_id     = data.aws_route53_zone.domain.zone_id  # zone_id for 'domainone.com'

  subject_alternative_names = concat(
    "api.domainone.com",
    "api.domaintwo.com",  # NOTE: I also own 'domaintwo.com', and have it in Route53...
  )

  wait_for_validation = true
}

This created a validation DNS record for _f0fc7cee4f586beaebf08a1080a87036.api.domaintwo.com.domainone.com, which wasn't going to work for validating the SAN api.domaintwo.com.
I'm not sure there's a good solution here short of:

Using data blocks to automagically figure out the route53 zones.  I use something like:data "aws_route53_zone" "domain" {
  name = "domainone.com"
}

Redesigning the zone_id parameters.

I'd sort of suggest a third option:

Document that Route53 validation only works for certs which cover domains in a single Route53 zone.

Does it make sense to document this limitation?  (Un-stale-ing this issue)

I can try to PR a documentation fix, since it seems there isn't an appetite for other fixes.


OAuth error handling fixes

Coverage remained the same at 100.0% when pulling 466c8db352dbe3b56dbb1344f41488759053fa4b on branch-0.6.7 into b9b8aca95540b32280a7abef8b85f15aee24f8fd on master.

Coverage remained the same at 100.0% when pulling 466c8db352dbe3b56dbb1344f41488759053fa4b on branch-0.6.7 into b9b8aca95540b32280a7abef8b85f15aee24f8fd on master.

Coverage remained the same at 100.0% when pulling 466c8db352dbe3b56dbb1344f41488759053fa4b on branch-0.6.7 into b9b8aca95540b32280a7abef8b85f15aee24f8fd on master.


Thread &'b mut F instead of F everywhere

I ran into a recursion limit bug when using this in Cargo because
there's a point where Foo<F> got turned into Foo<&mut F>. Now
everything would compile just fine as &mut F still implements FnMut,
but in the interest of avoiding #![recursion_limit] I figured I'd go
ahead and avoid the recursion in type parameters by just always using
&mut F everywhere.

I added 31ca0e70d7ccf500d1f923c7aebfa2f7a65e02da (that doesn't cancel out your change does it?) and released 0.0.2.

Ah yeah I think that works!
Still working through some more issues though, so there may be an 0.0.3 as well :)

False alarm, that bug was a missing #[serde(rename = "...")], so 0.0.2 looks like a likely "cargo's gonna use this" candidate!


Add select_for_update to prepare checkout

I want to merge this change because it prevents the issue where checkout object is deleted before it is updated.

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-8021 Save with update_fields did not affect any rows.

Moved to #10994


Create python-app.yml

Inits github action for continuous integration.
Next Steps: Add unit tests for pytest.

Github gives us 2000 free minutes of compute time to run our tests. After we use them up, we can buy more minutes or run on a local machine. I can probably get it running on a extra raspberry pi I have.

TODO: Write more unit tests using mocks for web requests or read disk methods.
Not in scope: Fixing flake8 errors. We'll do that in another PR.
As Testing Lead, Tianwei will be the primary contact for this feature. @ZTWHHH let me know if you have any questions about how this works.


RuntimeError of test_speedup.py

Hi! I ran the following command:
python test_speedup.py --model_type pruneLLM --base_model ../llama-7b-hf --ckpt prune_log/llama_prune/pytorch_model.bin
and got the following error:
Warning: module Embedding is treated as a zero-op.
Warning: module LlamaRotaryEmbedding is treated as a zero-op.
Warning: module LlamaMLP is treated as a zero-op.
Warning: module LlamaDecoderLayer is treated as a zero-op.
Warning: module LlamaModel is treated as a zero-op.
Warning: module LlamaForCausalLM is treated as a zero-op.
Traceback (most recent call last):
File "test_speedup.py", line 91, in
main(args)
File "test_speedup.py", line 69, in main
macs, params = get_model_complexity_info(model, (64,), as_strings=True,
File "/opt/conda/lib/python3.8/site-packages/ptflops/flops_counter.py", line 30, in get_model_complexity_info
flops_count, params_count = get_flops_pytorch(model, input_res,
File "/opt/conda/lib/python3.8/site-packages/ptflops/pytorch_engine.py", line 44, in get_flops_pytorch
_ = flops_model(batch)
File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1128, in _call_impl
result = forward_call(*input, **kwargs)
File "/dfs/data/LLM-Pruner/LLMPruner/models/hf_llama/modeling_llama.py", line 689, in forward
outputs = self.model(
File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/dfs/data/LLM-Pruner/LLMPruner/models/hf_llama/modeling_llama.py", line 532, in forward
inputs_embeds = self.embed_tokens(input_ids)
File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1110, in _call_impl
return forward_call(*input, **kwargs)
File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/sparse.py", line 158, in forward
return F.embedding(
File "/opt/conda/lib/python3.8/site-packages/torch/nn/functional.py", line 2156, in embedding
return torch.embedding(weight, input, padding_idx, scale_grad_by_freq, sparse)
RuntimeError: Expected tensor for argument #1 'indices' to have one of the following scalar types: Long, Int; but got torch.cuda.HalfTensor instead (while checking arguments for embedding)
Could you please help to have a look?

Hi. The test_speedup.py only supports the testing on GPU currently. Have you tested this script on GPU?

Hi. We update the test_speedup.py to support the testing on GPU.
Please let me know if the bug still exists with the current version.

Are there official FLOPs statistics on LLaMa-7B and its pruned version?

The FLOPs of the base model and the pruned model are:

here #MACs is equivalent to #FLOPs right? I just thought that they could be two different things.

The FLOPs of the base model and the pruned model are:

here #MACs is equivalent to #FLOPs right? I just thought that they could be two different things.

The results presented in the table is MACs rather than FLOPs and we didn't calculated the exact FLOPs of the model (roughly 2 times of MACs). To prevent any potential confusion, we chose not to use the term 'FLOPs' in our paper, and thus all the results are MACs and not equivalent to FLOPs.

Hi. The test_speedup.py only supports the testing on GPU currently. Have you tested this script on GPU?

Hi thanks for reply! Actually I was using GPU to run this code before.
Now I've tried the new version, when ran on CPU it worked fine. But if ran on GPU, the same error still existed.

Thanks for rapid reply! I've also made the same corrections following your code for the CPU implementations, and it works now. Thanks again!


How to eval the model?

Hi, do you have a plan to release evaluation code?
or can you explain how to eval the model?

I believe it is in the code base already. See https://github.com/phizaz/diffae/blob/d961b172df3064f7069d97d9992694352583999c/run_ffhq128_ddim.py#L10 and https://github.com/phizaz/diffae/blob/d961b172df3064f7069d97d9992694352583999c/run_ffhq128.py#L26. After running the aforementioned lines of code, you will get the FID numbers useful for comparing across generative models.


Use FYPO from Git/GitHub in Chado load

The Chado load scripts are still pointing at the subversion repo on SourceForge

The latest curation-tool-data-load-output logs suggest this isn't done ... I've stuck the next-load label on.

I've had trouble with the ontology loading step so we haven't had a new FYPO in a few days.  That's fixed now so the next load should be better.

The load failed due a typo in the load script.  The path to fypo-simple.obo has moved - loading again.


#6 add 2nd NA CA region northamerica-northeast2

see testing on the PR

incorporated in https://github.com/canada-ca/accelerators_accelerateurs-gcp/pull/26


Incorrect clipping for touched ListTile

Steps to reproduce

Add a ListView
return a ListTile Widget with a Text Widget as title property with onTap action in itemBuilder
Wrap ListView in Padding (top value eg. 30)
scroll list up until the first ListTile is cutted by 50%
tap and hold the ListTile

Expected results
Highlight/Focused color of ListTile is clipped by 50% ListTile height
Actual results
Highlight/focus box is complete visible (no clipping)
Code sample
Code sample
void main() {
  runApp(
    MaterialApp(
      home: Scaffold(
        appBar: AppBar(),
        body: Scaffold(
          body: Padding(
            padding: const EdgeInsets.only(top: 30),
            child: ListView.builder(
              itemBuilder: (_, __) {
                return ListTile(
                  onTap: () {},
                  title: const Text('TEST'),
                );
              },
            ),
          ),
        ),
      ),
    ),
  );
}

Screenshots or Video

Screenshots / Video demonstration
https://github.com/flutter/flutter/assets/80750776/07ae3631-aaa5-4aa3-9643-62c1c814c52b

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[!] Flutter (Channel stable, 3.16.2, on macOS 14.2 23C64 darwin-x64, locale de-DE)
    • Flutter version 3.16.2 on channel stable at /Users/mraguse/fvm/versions/3.16.2
    ! Warning: `dart` on your path resolves to /usr/local/Cellar/dart/3.2.2/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/mraguse/fvm/versions/3.16.2. Consider adding /Users/mraguse/fvm/versions/3.16.2/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 9e1c857886 (3 weeks ago), 2023-11-30 11:51:18 -0600
    • Engine revision cf7a9d0800
    • Dart version 3.2.2
    • DevTools version 2.28.3
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/mraguse/Library/Android/sdk
    • Platform android-34, build-tools 30.0.3
    • ANDROID_HOME = /Users/mraguse/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C65
    • CocoaPods version 1.14.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2023.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.3.1)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 71.3.6
    • Dart plugin version 223.8214.16

[✓] VS Code (version 1.84.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.74.0

[✓] Connected device (3 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 10 (API 29) (emulator)
    • macOS (desktop)                    • macos         • darwin-x64     • macOS 14.2 23C64 darwin-x64
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 120.0.6099.109
    ! Error: iPhone von Maurice has recently restarted. Xcode will continue when iPhone von Maurice is unlocked. (code -14)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Hi @mauriceraguseinit, I believe this is caused by how Ink works in the framework, it paints on the nearest Material ancestor. In this case, this would be your Scaffold.
Can you try wrapping your ListTile in a Material widget to see if you still experience this issue?
https://api.flutter.dev/flutter/material/InkResponse-class.html

Hi thanks for your response... Wrapping the ListTile or the whole ListView works here to clip the Ink.

Ideally, no, but this is how Ink currently behaves; and at its core, ListTile is an Ink widget wrapped in an InkWell. There are issues tracking improving this behaviour in https://github.com/flutter/flutter/issues/73315 & https://github.com/flutter/flutter/issues/86584 that you can follow for updates.
For now, you can work around using the Material either on the ListTile or the ListView.
Kindly follow up in the issues linked above for updates.
Closing from here.
Thank you


[Feature][Metrics] Increase granularity of metrics

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description

At present, Dolphin Scheduler only provides metrics with constant names and tags, which means we could not monitor the behavior of a specific workflow or task.
We shall also provide metrics with constant names but dynamic tags to increase granularity.

Use case

Some specific workflows / tasks on prod could be vital and may cause great loss to users if failed, we need metrics to monitor these VIP tasks / workflows.

Related issues
related: #9324
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

It seems micrometer will not automatically manage those dynamically-generated metrics for us.
First of all, we may use constant name and dynamic tags, e.g. workflow_id to generate metrics dynamically.
Secondly, I plan to use a class DynamicMetricsManager to provide a method to instantiate metrics during runtime. DynamicMetricsManager uses ConcurrentHashMap to store those dynamic metrics.
Thirdly, when workflow / task instance finished, DynamicMetricsManager will remove related metrics from ConcurrentHashMap and put them into Caffeine Cache with expireAfterAcess larger than Prometheus pull interval to manage the life cycle of those dynamic metrics.
WDYT? @ruanwenjun

It seems micrometer will not automatically manage those dynamically-generated metrics for us.
First of all, we may use constant name and dynamic tags, e.g. workflow_id to generate metrics dynamically.
Secondly, I plan to use a class DynamicMetricsManager to provide a method to instantiate metrics during runtime. DynamicMetricsManager uses ConcurrentHashMap to store those dynamic metrics.
Thirdly, when workflow / task instance finished, DynamicMetricsManager will remove related metrics from ConcurrentHashMap and put them into Caffeine Cache with expireAfterAcess larger than Prometheus pull interval to manage the life cycle of those dynamic metrics and avoid memory explosion.
WDYT? @ruanwenjun

@kezhenxu94 Any suggestions will be very appreciated. : )

Hi @EricGao888 , what kind of dynamic metrics do we have? You know, to fully use these metrics we need to display them and configure in Grafana, if we don't know these metrics in advance, how to configure these and use these metrics? Can you give some examples that we need dynamic metrics?

Hi @EricGao888 , what kind of dynamic metrics do we have? You know, to fully use these metrics we need to display them and configure in Grafana, if we don't know these metrics in advance, how to configure these and use these metrics? Can you give some examples that we need dynamic metrics?

@kezhenxu94 Thanks for the reply!
Considering the following scenario: some user A works for a big financial company, and A chooses to use DS to schedule ETL tasks. Some of these tasks are not that important, but some are crucial, which may strongly relate to their financial business. Therefore, A wants some metrics to monitor these specific VIP tasks to avoid breaching SLAs for their customers. For example, task running duration, retry times, failover times, etc.
To easily configure these dynamic metrics, we could name them as ds.dynamic.task.<task_id>.duration. Since A does not need to monitor dynamic metrics for every tasks, just the VIP ones, he could configure those metrics in grafana manually.
But indeed, there is a concern that how users could get the task_id / workflow_id.

Hi @EricGao888 , what kind of dynamic metrics do we have? You know, to fully use these metrics we need to display them and configure in Grafana, if we don't know these metrics in advance, how to configure these and use these metrics? Can you give some examples that we need dynamic metrics?

@kezhenxu94 Thanks for the reply!
Considering the following scenario: some user A works for a big financial company, and A chooses to use DS to schedule ETL tasks. Some of these tasks are not that important, but some are crucial, which may strongly relate to their financial business. Therefore, A wants some metrics to monitor these specific VIP tasks to avoid breaching SLAs for their customers. For example, task running duration, retry times, failover times, etc.
To easily configure these dynamic metrics, we could name them as ds.dynamic.task.<task_id>.duration. Since A does not need to monitor dynamic metrics for every tasks, just the VIP ones, he could configure those metrics in grafana manually.
But indeed, there is a concern that how users could get the task_id / workflow_id.

Hi, task id can be added as tags and the users can filter by tags when configuring Grafana. Also, is task ID constant and won't change between different executions? If they change it's impractical to configure these constantly changing IDs.

Hi @EricGao888 , what kind of dynamic metrics do we have? You know, to fully use these metrics we need to display them and configure in Grafana, if we don't know these metrics in advance, how to configure these and use these metrics? Can you give some examples that we need dynamic metrics?

@kezhenxu94 Thanks for the reply!
Considering the following scenario: some user A works for a big financial company, and A chooses to use DS to schedule ETL tasks. Some of these tasks are not that important, but some are crucial, which may strongly relate to their financial business. Therefore, A wants some metrics to monitor these specific VIP tasks to avoid breaching SLAs for their customers. For example, task running duration, retry times, failover times, etc.
To easily configure these dynamic metrics, we could name them as ds.dynamic.task.<task_id>.duration. Since A does not need to monitor dynamic metrics for every tasks, just the VIP ones, he could configure those metrics in grafana manually.
But indeed, there is a concern that how users could get the task_id / workflow_id.

Hi, task id can be added as tags and the users can filter by tags when configuring Grafana. Also, is task ID constant and won't change between different executions? If they change it's impractical to configure these constantly changing IDs.

Thanks for bringing up this point. In several other scheduling systems, task instance id will change but task id remains constant. But I need to double check how it works in DS and whether it is possible to track task instances of a specific task.

Hi @EricGao888 , what kind of dynamic metrics do we have? You know, to fully use these metrics we need to display them and configure in Grafana, if we don't know these metrics in advance, how to configure these and use these metrics? Can you give some examples that we need dynamic metrics?

@kezhenxu94 Thanks for the reply!
Considering the following scenario: some user A works for a big financial company, and A chooses to use DS to schedule ETL tasks. Some of these tasks are not that important, but some are crucial, which may strongly relate to their financial business. Therefore, A wants some metrics to monitor these specific VIP tasks to avoid breaching SLAs for their customers. For example, task running duration, retry times, failover times, etc.
To easily configure these dynamic metrics, we could name them as ds.dynamic.task.<task_id>.duration. Since A does not need to monitor dynamic metrics for every tasks, just the VIP ones, he could configure those metrics in grafana manually.
But indeed, there is a concern that how users could get the task_id / workflow_id.

Typo here, shouldn't use <task_id>s in metrics names but put them in tags as described in Description section and comment https://github.com/apache/dolphinscheduler/issues/10525#issuecomment-1168318015

Putting id as tag doesn't require dynamic metrics, no?

Things seem a little different in Micrometer to me. If it is possible to add tags during runtime, it would be much easier. However, I haven't figured out a way to add dynamic tags to metric instance during runtime. Please correct me if I miss something. Thanks! https://github.com/micrometer-metrics/micrometer/blob/main/micrometer-core/src/main/java/io/micrometer/core/instrument/Counter.java

https://github.com/micrometer-metrics/micrometer/blob/4de0d5f70dbd05dc1d66b48012399e90b22a6005/micrometer-core/src/main/java/io/micrometer/core/instrument/Counter.java#L122-L132
Not sure if there is a way to do the trick more gracefully.

It seems micrometer will not automatically manage those dynamically-generated metrics for us.
First of all, we may use constant name and dynamic tags, e.g. workflow_id to generate metrics dynamically.
Secondly, I plan to use a class DynamicMetricsManager to provide a method to instantiate metrics during runtime. DynamicMetricsManager uses ConcurrentHashMap to store those dynamic metrics.
Thirdly, when workflow / task instance finished, DynamicMetricsManager will remove related metrics from ConcurrentHashMap and put them into Caffeine Cache with expireAfterAcess larger than Prometheus pull interval to manage the life cycle of those dynamic metrics and avoid memory explosion.
WDYT? @ruanwenjun

I think there could be a better solution.
We give users ability to choose whether monitor a specific workflow or not on UI. DS will generate related metrics for a specific workflow only if users choose to. This is more reasonable because users definitely do not need to monitor every workflow / task in detail.

related: #13552


Feature Request: New ecs-python module to mirror lambda-python

"As a data developer and Infra Catalog user, I want to be able to deploy Python code to ECS as easily as I can using the python-lambda module."
Background:
The decision to use ECS vs Lambda is one that is often hard to make, and as limitations of one environment or the other are uncovered, there can be a lot of wasted effort involved in reversing this decision.
While we've fully automated the deployment of python lambda functions, it's much more difficult in comparison to create a docker image that runs an arbitrary python script. From an implementation perspective, however, there's no reason why ECS cannot support the same modular config approach as we have for python lambda functions.

FYI - @jacksandom ☝


model size

hello，i have aquestion about why you use vit-s and vit-tiny,and counterpart is resnet-50,these size is not equal.i know you have explained on openview,i want to know whether vit-base's matrix eigenvalue spectrum is like vit-tiny in your paper,just stretch to the right.

ok,i got it,but the number and the dimension of per head in Fig C.5 and Fig C.6 is from the same model like vit-s?i'm not sure the effect of lager heads and demension can resist the effect of bigger model like vit-base.
besides,i know the vit matrix eigenvalue is more aggregated than resnet from the specturm,so i guess the bigger model could follow this priciple.

You're right. As you might expect, Fig C.5 and C.6 report results for models with multiple head numbers/embed dims. All other hyperparameters are the same. I would like to leave a detailed investigation of large models for future work.

ok.it is just a little thing.i think this paper very good,i like it


Is there any way to edit YDocs directly instead of using YRoomDocument?

Hello Stefan, how are you?
Is there any way to edit YDocs directly instead of using YRoomDocument export?
The purpose of this is to be able to handle YDocs directly the way they are on the server side.

Hey, so this is where it's getting tricky. channels-yroom exists because ypy (the Python bindings to the Y eco system) do not support xml fragments and are currently not up to date with yrs APIs. I created my own higher-level API for syncing+awareness but exposing YDocs directly was not the goal of the project – that's more tricky and where ypy should come in.
I have a draft PR own ypy that upgrades to a newer yrs version.
I see two ways forward:

Get ypy upgraded and then use that API in the yroom worker. Definitely the long-term solution.
Expose some limited API in channels-yroom. For that I would need a better description of your use case. E.g. something like:

room_manager.add_xml_node_to_room(room_name, document_name, xml_string)
But this is clearly more of an ad-hoc band-aid solution for limited use cases.

Your solution is amazing! When I tried to use YPY directly I saw that some functions were not integrated yet, now I understand.
I will choose to help unlock the update.

@stefanw this is basically the other thing that we need to be able to do. I see that the PR against ypy is still open. What are your latest thoughts on this?

I haven't looked into this for some months now as there's no comments or guidance from the maintainers. I'm still learning Rust, so my approach in this PR was more "make it compile" and not "make it elegant", so I'm relying on some pointers if this is going into the right direction.

Cool, thanks. Let me see if I can poke the right people enough to get it moving haha.


OHDSI Tool - HADES (optional)

Verify Install
Introductory Tutorials
https://ohdsi.org/ohdsi2022-tutorial/
https://ohdsi.org/open-source-tutorials/
Practice with use cases
Discuss OHDSI tool among team with specific relevance to Integrating SCCRIP Data in OMOP format.
From https://ohdsi.org/software-tools/ below
HADES (previously the OHDSI METHODS LIBRARY) is a collection of open-source R packages that offer functions which can be used together to perform a complete observational study, starting from data in the CDM, and resulting in estimates and supporting statistics, figures, and tables. The packages interact directly with observational data in the CDM, and can be used simply to provide cross-platform compatibility to completely custom analyses, or can provide advanced standardized analytics for population characterization, population-level effect estimation, and patient-level prediction. HADES supports best practices for use of observational data and observational study design as learned from previous and ongoing research, such as transparency, reproducibility, as well as measuring the operating characteristics of methods in a particular context and subsequent empirical calibration of estimates produced by the methods.
HADES LINKS
Documentation: Book of OHDSI • HADES Web Site
Installation Information: Click Here
Source Code: GitHub

HADES is up and running.


pass

Hello, it doesn't appear there is any question or issue included in your issue description. If you have any questions or issues, please reopen this issue with additional details and I'll be happy to help!


Default theme SASS customization broken for text colors?

Short description of the problem:
When trying to override $bar-default-text, nothing happens.
What behavior are you expecting?
The text colors should change.
Steps to reproduce:

Add something like "$bar-default-text: white !default" to your ionic.app.scss file.
Watch nothing happen.
Cry.

I noticed that $bar-default-text isn't even included in the appropriate _xxx.scss files in the lib folder, so they're obviously being ignored completely.
Which Ionic Version? 1.x or 2.x
Cordova CLI: 6.1.0 (cordova-lib@undefined)
Gulp version:  CLI version 3.9.1
Gulp local:   Local version 3.9.1
Ionic Version: 1.2.4
Ionic CLI Version: 1.7.14
Ionic App Lib Version: 0.7.0
ios-deploy version: 1.8.5
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v4.1.2
Xcode version: Xcode 7.3 Build version 7D175

Hello, thanks for using Ionic. Is this still an issue? Thanks!

Hello all! As it seems it has been a while since there was any activity on this issue i will be closing it for now. Feel free to comment if you are still running into this issue. Thanks for using Ionic!


can I get face shape like oval, diamond, square, rectangle.

can I get face shape like oval, diamond, square, rectangle.

No.


Proper way to Parametrize conVar

Right now I am using a hack to parametrize unit vector for unitals in gamephys example:
Unit vector is the "basic concept" and the example has a "horizontal" unit vector and "vertical" unit vector.
iVect       = ucFromVC ivec metre
where ivec = cvR (dcc "unit_vect" (compoundPhrase' (cn "horizontal") (QM.unit_vect ^. term)) "unit vector" ) (QM.unit_vect ^. symbol)
is there a better way of doing this?

I don't think there is a better way to do this yet; we need to invent one, and then implement it.
This is not so straightforward.  We could 'do better' for 2D, but then that won't scale.  Plus it is hard to see 'unit_vector' as fundamental, as opposed to seeing it being made up of vector coupled with length=1.  Having said that, I have encountered situations where a simple change of perspective (i.e. such as making unit_vector fundamental) simplifies things a lot.  Long-winded comment to say 'needs design'.

duplicate of #91


Adaptive Card content is cleard on submit

Version
4.5.2
Describe the bug
Since version 4.5.0 the adaptive cards are behaving incorrectly. Version 4.5.1 and 4.5.2 have fixes for the issues #1680 and #1683 but the experience is still not at the same level as 4.4.2.
On 4.5.2 the form content is cleared on submit. Sometimes the data are correctly sent back to the bot, sometimes not.
Screenshots
See the difference between 4.4.2 and 4.52
Version 4.4.2

Version 4.5.2

Expected behavior
Form value must not be cleared on Adaptive Card
[bug]

HI @sachabruttin ,
Thanks for reporting.
I was aware of the form being cleared when the submit button was pressed, however, I did not realize that it was possible that the activity did not contain the form data when sent to the bot.
I'll investigate.

Hello @sachabruttin ,
I've been trying to reproduce the issue, and as expected the forms inside of the Adaptive Cards due clear on Submit, however, I could not reproduce the scenario in which the form data was not being sent to the bot.
The form being cleared is a current limitation of the way we render Web Chat within the Emulator, but the data should be correctly relayed to your bot upon pressing submit.
Do you happen to have steps for a reliable repro of the form data not being correctly transmitted to the bot?

Hello @tonyanziano
This not easy to reproduce because it occurs randomly. Please check these 2 videos.

I suspect the form data is not correctly submitted because the card is cleared before I click on the submit button.
On the next one, the dropdown is cleared just after I have selected a value...

As previously state, all these behaviors were not present on the 4.4.x release. Is there something that prevents you from rollback the rendering of the Adaptive Cards to the working one?

@sachabruttin Thank you for posting the recordings. From the looks of those gifs, it looks the root of the issue is still due to the forms being cleared and not the actual transmission of the form values to the bot.
What I can see from both your gifs is that both forms are cleared at the same time as a
POST 200 directline.postActivity
appears in the log panel. I think the addition of a

Notes to fixer:

re-render looks like it's caused by any state changes at all to the chat document associated with the livechat
potential fix could be to flatten out different parts of inner chat document structure into separate state slices so that a change within the chat document doesn't affect every component connected to the chat at the root level


Multiple URIs doesn't work

Version Reports:
Distro  version of host:
MacOS 10.13.5
Terraform Version Report
0.11.7
Libvirt version
4.5.0
terraform-provider-libvirt plugin version (git-hash)
77cf2f2574f2df43b92086135fe248b2b7e91b38

While aliasing different provider configurations, Terraform is still using default one.
Setup
provider "libvirt" {
  uri   = "qemu+ssh://root@x/system?socket=/var/run/libvirt/libvirt-sock"
  alias = "hpv1"
}

resource "libvirt_volume" "x" {
  provider = "libvirt.hpv1"
  name     = "xr"
  pool     = "vms"
  size     = 12123123
}

$ terraform plan
provider.libvirt.uri
  libvirt connection URI for operations. See https://libvirt.org/uri.html

  Enter a value:

Is the aliased one the only one you defined or there is one without alias?

Aliased one is the only one I've defined.

I can't reproduce it. This works fine for me:
provider "libvirt" {
  uri = "qemu:///system"
  alias = "foobar"
}

resource "libvirt_volume" "x" {
  provider = "libvirt.foobar"
  name     = "xr"
  pool     = "default"
  size     = 12123123
}

resource "libvirt_domain" "outofband" {
  name = "outofband"
  provider = "libvirt.foobar"
}

May be you can run it with TF_LOG=debug and attach the debug output and the full HCL file.

@dmacvicar your example works fine for me too.
I think it is a local issue from @kamsz
@kamsz  can you post your
TF_LOG=DEBUG terraform plan

? TIA
feel free to ask if you need any help

Sure, here's debug:
2018/08/21 10:35:42 [INFO] Terraform version: 0.11.7
2018/08/21 10:35:42 [INFO] Go runtime version: go1.10.1
2018/08/21 10:35:42 [INFO] CLI args: []string{"/usr/local/Cellar/terraform/0.11.7/bin/terraform", "plan"}
2018/08/21 10:35:42 [DEBUG] Attempting to open CLI config file: /Users/kamsz/.terraformrc
2018/08/21 10:35:42 [INFO] Terraform version: 0.11.7
2018/08/21 10:35:42 [INFO] Go runtime version: go1.10.1
2018/08/21 10:35:42 [INFO] CLI args: []string{"/usr/local/Cellar/terraform/0.11.7/bin/terraform", "plan"}
2018/08/21 10:35:42 [DEBUG] Attempting to open CLI config file: /Users/kamsz/.terraformrc
2018/08/21 10:35:42 [DEBUG] File doesn't exist, but doesn't need to. Ignoring.
2018/08/21 10:35:42 [DEBUG] File doesn't exist, but doesn't need to. Ignoring.
2018/08/21 10:35:42 [INFO] CLI command args: []string{"plan"}
2018/08/21 10:35:42 [INFO] CLI command args: []string{"plan"}
2018/08/21 10:35:42 [INFO] Building AWS region structure
2018/08/21 10:35:42 [INFO] Building AWS auth structure
2018/08/21 10:35:42 [INFO] Setting AWS metadata API timeout to 100ms
2018/08/21 10:35:42 [INFO] Building AWS region structure
2018/08/21 10:35:42 [INFO] Building AWS auth structure
2018/08/21 10:35:42 [INFO] Setting AWS metadata API timeout to 100ms
2018/08/21 10:35:42 [INFO] Ignoring AWS metadata API endpoint at default location as it doesn't return any instance-id
2018/08/21 10:35:42 [INFO] Ignoring AWS metadata API endpoint at default location as it doesn't return any instance-id
2018/08/21 10:35:42 [INFO] AWS Auth provider used: "SharedCredentialsProvider"
2018/08/21 10:35:42 [INFO] AWS Auth provider used: "SharedCredentialsProvider"
2018/08/21 10:35:42 [INFO] Initializing DeviceFarm SDK connection
2018/08/21 10:35:42 [INFO] Initializing DeviceFarm SDK connection
2018/08/21 10:35:42 [DEBUG] [aws-sdk-go] DEBUG: Request sts/GetCallerIdentity Details:
2018/08/21 10:35:43 [INFO] command: backend initialized: *s3.Backend
2018/08/21 10:35:43 [INFO] command: backend initialized: *s3.Backend
2018/08/21 10:35:43 [DEBUG] checking for provider in "."
2018/08/21 10:35:43 [DEBUG] checking for provider in "."
2018/08/21 10:35:43 [DEBUG] checking for provider in "/usr/local/Cellar/terraform/0.11.7/bin"
2018/08/21 10:35:43 [DEBUG] checking for provider in "/usr/local/Cellar/terraform/0.11.7/bin"
2018/08/21 10:35:43 [DEBUG] checking for provider in ".terraform/plugins/darwin_amd64"
2018/08/21 10:35:43 [DEBUG] checking for provider in ".terraform/plugins/darwin_amd64"
2018/08/21 10:35:43 [DEBUG] checking for provider in "/Users/kamsz/.terraform.d/plugins"
2018/08/21 10:35:43 [DEBUG] found provider "terraform-provider-kong_v1.4.0"
2018/08/21 10:35:43 [WARN] found legacy provider "terraform-provider-libvirt"
2018/08/21 10:35:43 [DEBUG] checking for provider in "/Users/kamsz/.terraform.d/plugins"
2018/08/21 10:35:43 [DEBUG] found provider "terraform-provider-kong_v1.4.0"
2018/08/21 10:35:43 [WARN] found legacy provider "terraform-provider-libvirt"
2018/08/21 10:35:43 [DEBUG] found valid plugin: "libvirt", "0.0.0", "/Users/kamsz/.terraform.d/plugins/terraform-provider-libvirt"
2018/08/21 10:35:43 [DEBUG] found valid plugin: "kong", "1.4.0", "/Users/kamsz/.terraform.d/plugins/terraform-provider-kong_v1.4.0"
2018/08/21 10:35:43 [DEBUG] found valid plugin: "libvirt", "0.0.0", "/Users/kamsz/.terraform.d/plugins/terraform-provider-libvirt"
2018/08/21 10:35:43 [DEBUG] found valid plugin: "kong", "1.4.0", "/Users/kamsz/.terraform.d/plugins/terraform-provider-kong_v1.4.0"
2018/08/21 10:35:43 [DEBUG] checking for provisioner in "."
2018/08/21 10:35:43 [DEBUG] checking for provisioner in "."
2018/08/21 10:35:43 [DEBUG] checking for provisioner in "/usr/local/Cellar/terraform/0.11.7/bin"
2018/08/21 10:35:43 [DEBUG] checking for provisioner in "/usr/local/Cellar/terraform/0.11.7/bin"
2018/08/21 10:35:43 [DEBUG] checking for provisioner in ".terraform/plugins/darwin_amd64"
2018/08/21 10:35:43 [DEBUG] checking for provisioner in ".terraform/plugins/darwin_amd64"
2018/08/21 10:35:43 [DEBUG] checking for provisioner in "/Users/kamsz/.terraform.d/plugins"
2018/08/21 10:35:43 [DEBUG] checking for provisioner in "/Users/kamsz/.terraform.d/plugins"
2018/08/21 10:35:43 [INFO] command: backend *s3.Backend is not enhanced, wrapping in local
2018/08/21 10:35:43 [INFO] command: backend *s3.Backend is not enhanced, wrapping in local
2018/08/21 10:35:43 [INFO] backend/local: starting Plan operation
2018/08/21 10:35:43 [INFO] backend/local: starting Plan operation
2018/08/21 10:35:43 [DEBUG] [aws-sdk-go] DEBUG: Request s3/ListObjects Details:
2018/08/21 10:35:44 [DEBUG] [aws-sdk-go]
2018/08/21 10:35:44 [INFO] terraform: building graph: GraphTypeInput
2018/08/21 10:35:44 [INFO] terraform: building graph: GraphTypeInput
2018/08/21 10:35:44 [DEBUG] Attaching resource state to "libvirt_volume.chef_server": &terraform.ResourceState{Type:"libvirt_volume", Dependencies:[]string{}, Primary:(*terraform.InstanceState)(0xc42056eeb0), Deposed:[]*terraform.InstanceState{}, Provider:"provider.libvirt", mu:sync.Mutex{state:0, sema:0x0}}
2018/08/21 10:35:44 [DEBUG] Attaching resource state to "libvirt_volume.chef_server": &terraform.ResourceState{Type:"libvirt_volume", Dependencies:[]string{}, Primary:(*terraform.InstanceState)(0xc42056eeb0), Deposed:[]*terraform.InstanceState{}, Provider:"provider.libvirt", mu:sync.Mutex{state:0, sema:0x0}}
2018/08/21 10:35:44 [DEBUG] Attaching resource state to "libvirt_domain.chef_server": &terraform.ResourceState{Type:"libvirt_domain", Dependencies:[]string{"libvirt_volume.chef_server"}, Primary:(*terraform.InstanceState)(0xc42056ee60), Deposed:[]*terraform.InstanceState{}, Provider:"provider.libvirt", mu:sync.Mutex{state:0, sema:0x0}}
2018/08/21 10:35:44 [TRACE] Graph after step *terraform.AttachStateTransformer:

libvirt_domain.chef_server - *terraform.NodeAbstractResource
libvirt_volume.chef_server - *terraform.NodeAbstractResource
2018/08/21 10:35:44 [TRACE] Graph after step *terraform.RootVariableTransformer:

libvirt_domain.chef_server - *terraform.NodeAbstractResource
libvirt_volume.chef_server - *terraform.NodeAbstractResource
2018/08/21 10:35:44 [TRACE] ProviderConfigTransformer: Starting for path: []
2018/08/21 10:35:44 [DEBUG] Attaching resource state to "libvirt_domain.chef_server": &terraform.ResourceState{Type:"libvirt_domain", Dependencies:[]string{"libvirt_volume.chef_server"}, Primary:(*terraform.InstanceState)(0xc42056ee60), Deposed:[]*terraform.InstanceState{}, Provider:"provider.libvirt", mu:sync.Mutex{state:0, sema:0x0}}
2018/08/21 10:35:44 [TRACE] Graph after step *terraform.AttachStateTransformer:

libvirt_domain.chef_server - *terraform.NodeAbstractResource
libvirt_volume.chef_server - *terraform.NodeAbstractResource
2018/08/21 10:35:44 [TRACE] Graph after step *terraform.RootVariableTransformer:

libvirt_domain.chef_server - *terraform.NodeAbstractResource
libvirt_volume.chef_server - *terraform.NodeAbstractResource
2018/08/21 10:35:44 [TRACE] ProviderConfigTransformer: Starting for path: []
2018/08/21 10:35:44 [DEBUG] adding missing provider: libvirt
2018/08/21 10:35:44 [DEBUG] adding missing provider: libvirt
2018/08/21 10:35:44 [DEBUG] resource libvirt_volume.chef_server using provider provider.libvirt.hpv1
2018/08/21 10:35:44 [DEBUG] resource libvirt_volume.chef_server using provider provider.libvirt.hpv1
2018/08/21 10:35:44 [DEBUG] resource libvirt_domain.chef_server using provider provider.libvirt
2018/08/21 10:35:44 [DEBUG] resource libvirt_domain.chef_server using provider provider.libvirt
2018/08/21 10:35:44 [DEBUG] ReferenceTransformer: "provider.libvirt" references: []
2018/08/21 10:35:44 [DEBUG] ReferenceTransformer: "libvirt_volume.chef_server" references: []
2018/08/21 10:35:44 [DEBUG] ReferenceTransformer: "provider.libvirt" references: []
2018/08/21 10:35:44 [DEBUG] ReferenceTransformer: "libvirt_volume.chef_server" references: []
2018/08/21 10:35:44 [DEBUG] ReferenceTransformer: "libvirt_domain.chef_server" references: [libvirt_volume.chef_server]
2018/08/21 10:35:44 [DEBUG] ReferenceTransformer: "libvirt_domain.chef_server" references: [libvirt_volume.chef_server]
2018-08-21T10:35:44.374+0200 [DEBUG] plugin: starting plugin: path=/Users/kamsz/.terraform.d/plugins/terraform-provider-libvirt args=[/Users/kamsz/.terraform.d/plugins/terraform-provider-libvirt]
2018-08-21T10:35:44.374+0200 [DEBUG] plugin: starting plugin: path=/Users/kamsz/.terraform.d/plugins/terraform-provider-libvirt args=[/Users/kamsz/.terraform.d/plugins/terraform-provider-libvirt]
2018-08-21T10:35:44.376+0200 [DEBUG] plugin: waiting for RPC address: path=/Users/kamsz/.terraform.d/plugins/terraform-provider-libvirt
2018-08-21T10:35:44.376+0200 [DEBUG] plugin: waiting for RPC address: path=/Users/kamsz/.terraform.d/plugins/terraform-provider-libvirt
2018-08-21T10:35:44.397+0200 [DEBUG] plugin.terraform-provider-libvirt: plugin address: timestamp=2018-08-21T10:35:44.397+0200 address=/var/folders/zl/k3s2fz8x2h763hf61tr_0k100000gn/T/plugin186978363 network=unix
2018-08-21T10:35:44.397+0200 [DEBUG] plugin.terraform-provider-libvirt: plugin address: timestamp=2018-08-21T10:35:44.397+0200 address=/var/folders/zl/k3s2fz8x2h763hf61tr_0k100000gn/T/plugin186978363 network=unix
2018/08/21 10:35:44 [DEBUG] command: asking for input: "provider.libvirt.uri"
2018/08/21 10:35:44 [DEBUG] command: asking for input: "provider.libvirt.uri"

And here's the actual code:
provider "libvirt" {
  uri   = "qemu+ssh://root@x/system?socket=/var/run/libvirt/libvirt-sock"
  alias = "hpv1"
}

terraform {
  backend "s3" {}
}

resource "libvirt_volume" "x" {
  provider = "libvirt.hpv1"
  name     = "x"
  pool     = "vms"
  size     =<PHONE_NUMBER>0
}

I've figured it out, it was an issue with old resource in state :) Thanks!

For anyone else that runs across this issue:  Anything in your .tf file that is a 'resource' needs to have the 'provider' line in it, or you will get this issue....at least that was my issue ;)


Messaging Extension All Command with HTML and WebView rendering

We have integrated following ME Action command in existing sample Messaging Extension Action Commands C#

Adaptive Card in Task Module
Task Module Webview
Task Module HTML Page

Following commands got added to Manifest as well:

Fixes #
Proposed Changes

Integrated adaptive card in Message extension task module

-Added Webview in Task Module on action command

Integrated HTML razor page as Task Module form

Integrated HTML razor view

Testing

Closing.  Teams samples are no longer in BF SDK Samples.


nvs: add documentation about the crc in nvs

fixed nvs part of #68699

@kartben, it seems I couldn't just accept your commit.

@kartben, it seems I couldn't just accept your commit.

You have to pull the PR to your local branch now and squash commits.


Deprecated selector in proto-repl\styles\proto-repl.less

In proto-repl\styles\proto-repl.less:
Starting from Atom v1.13.0, the contents of atom-text-editor elements are no longer encapsulated within a shadow DOM boundary. This means you should stop using :host and ::shadow pseudo-selectors, and prepend all your syntax selectors with syntax--. To prevent breakage with existing style sheets, Atom will automatically upgrade the following selectors:

:host atom-text-editor .highlight.block-execution .region, atom-text-editor atom-text-editor .highlight.block-execution .region, atom-text-editor::shadow atom-text-editor .highlight.block-execution .region => :host atom-text-editor .highlight.block-execution .region, atom-text-editor atom-text-editor .highlight.block-execution .region, atom-text-editor.editor atom-text-editor .highlight.block-execution .region

:host .highlights .block-execution .region, atom-text-editor .highlights .block-execution .region, atom-text-editor::shadow .highlights .block-execution .region => :host .highlights .block-execution .region, atom-text-editor .highlights .block-execution .region, atom-text-editor.editor .highlights .block-execution .region

Automatic translation of selectors will be removed in a few release cycles to minimize startup time. Please, make sure to upgrade the above selectors as soon as possible.

This is a duplicate of #221. I think most people who noticed this (including me) did so because of the deprecation-cop package. So it's a very minor issue, but disconcerting to see the "!" in the status bar :)

Well, it's a minor issue now.
It'll cease to be minor when this little bit at the end happens: "Automatic translation of selectors will be removed in a few release cycles to minimize startup time."

Thank you-- I was just about to report this issue. It's a bit disconcerting to see the exclamation mark. :P

Duplicate of #221 which is fixed


WIP: Implement theme system

Status
Type

:x:
Feature

Description
WIP, beacuse colors for darkmode are missing. Are they defined already?
Theme-System through CSS variables. The SCSS variables are references to the css variables now.
Themes are stored in the folder src/themes. The current theme is stored in Redux and can be changed through a Redux action.
Screenshots / GIFs (if appropriate):

Resolves
frontend/projects/1#card-51929616

Thank you for your long time ago contribution. Because we now decided to leave out tailwindcss from vocascan this PR can actually be continued. Here are just a few things I suggest to change:

1 translate the pr title and description to english

2 instead of an dark/light theming system we could maybe consider about multiple and also an extendable theme system like vscode have. My thoughts for the future are:

2.1 an key/value store for user settings on the backend where we store which theme a user prefers which can be overwritten per device on localstorage
2.2 . It would also be cool, if we could somehow select a custom css in the desktop app or mount custom themes into the frontend nginx docker container to select.
2.3 Maybe we could consider about dark/light implementation of each theme

Why have you closed this pr?

Oh scheiße

Hab den PR ausversehen gelöscht

Oh scheiße

Dont say that.


Add Team to the project

Adding the people who are working or contributing in this project and their details and achivements as a feature

@Priyaaa1 can you please assign this issue to me?


Thumbnail timeouts raise TimeoutError

Description

We recently changed the thumbnail service we're using and some of the business logic around it (#630). The previous thumbnail retrieval step did not have a timeout, whereas the new thumbnail retrieval step has a timeout of 5 seconds:
https://github.com/WordPress/openverse-api/blob/03362db8caed8014e31d3fe5274c9bb084086b85/api/catalog/api/views/media_views.py#L173
We should also add a case for the TimeoutError exception here:
https://github.com/WordPress/openverse-api/blob/03362db8caed8014e31d3fe5274c9bb084086b85/api/catalog/api/views/media_views.py#L183
I think it might also be best for us to increase this timeout just a bit (maybe 10s?) to give the thumbnails for larger images/museum images more time to load.
Additional context

Sentry issue: https://sentry.io/share/issue/3daf5a2129a449859200e2de0c458ed6/
Resolution

[ ] 🙋 I would be interested in resolving this bug.

It seems we may also want to add RemoteDisconnected exceptions as well

Perhaps we should have a broader exception clause in general? Here's a ConnectionResetError failure


Correct typos in man pages

Codecov Report

Merging #47 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #47   +/-   ##
=======================================
  Coverage   27.74%   27.74%
=======================================
  Files           6        6
  Lines         310      310
=======================================
  Hits           86       86
  Misses        224      224

Impacted Files
Coverage Δ

R/prepareHLAdb.R
27.74% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


[PE-D] Tabs not clickable

I don't see the point of having a tab in the GUI when it is not clickable.
I understand that this is a text-based application, but only being allowed to navigate across tabs using commands is unintuitive and unergonomic :))

Labels: type.FeatureFlaw severity.Low
original: CrownKira/ped#5

This is intended. Tab switching is done with 1 letter which makes it faster than clicking


Error while deploying to Vercel

Hello, I'm facing an error likely from the autoprefixer dependency or the caniuse-lite dependency on the build phase.
Here are the logs:
[19:56:46.707] Cloning github.com/gu-does-git/gudoesdev (Branch: master, Commit: 4499dc8)
[19:56:46.713] The cli flag --force was set. Skipping build cache download.
[19:56:51.317] Cloning completed: 4.609s
[19:56:52.318] Running "vercel build"
[19:56:52.823] Vercel CLI 28.4.17
[19:56:53.434] Running "install" command: `npm install`...
[19:57:00.837]
[19:57:00.837] ><EMAIL_ADDRESS>prepare
[19:57:00.837] > husky install
[19:57:00.837]
[19:57:00.901] husky - Git hooks installed
[19:57:00.916]
[19:57:00.917] added 548 packages, and audited 695 packages in 7s
[19:57:00.917]
[19:57:00.917] 226 packages are looking for funding
[19:57:00.917]   run `npm fund` for details
[19:57:00.918]
[19:57:00.918] found 0 vulnerabilities
[19:57:01.249]
[19:57:01.249] ><EMAIL_ADDRESS>build
[19:57:01.250] > astro build
[19:57:01.250]
[19:57:02.666] node:internal/modules/cjs/loader:985
[19:57:02.666]   const err = new Error(message);
[19:57:02.666]               ^
[19:57:02.666]
[19:57:02.666] Error: Cannot find module './features/mdn-css-unicode-bidi-isolate-override'
[19:57:02.666] Require stack:
[19:57:02.666] - /vercel/path0/node_modules/caniuse-lite/data/features.js
[19:57:02.666] - /vercel/path0/node_modules/caniuse-lite/dist/unpacker/features.js
[19:57:02.667] - /vercel/path0/node_modules/caniuse-lite/dist/unpacker/index.js
[19:57:02.667] - /vercel/path0/node_modules/autoprefixer/lib/autoprefixer.js
[19:57:02.667]     at Function.Module._resolveFilename (node:internal/modules/cjs/loader:985:15)
[19:57:02.667]     at Function.Module._load (node:internal/modules/cjs/loader:833:27)
[19:57:02.667]     at Module.require (node:internal/modules/cjs/loader:1057:19)
[19:57:02.667]     at require (node:internal/modules/cjs/helpers:103:18)
[19:57:02.667]     at Object.<anonymous> (/vercel/path0/node_modules/caniuse-lite/data/features.js:1:19495)
[19:57:02.667]     at Module._compile (node:internal/modules/cjs/loader:1155:14)
[19:57:02.667]     at Object.Module._extensions..js (node:internal/modules/cjs/loader:1209:10)
[19:57:02.667]     at Module.load (node:internal/modules/cjs/loader:1033:32)
[19:57:02.667]     at Function.Module._load (node:internal/modules/cjs/loader:868:12)
[19:57:02.667]     at Module.require (node:internal/modules/cjs/loader:1057:19) {
[19:57:02.668]   code: 'MODULE_NOT_FOUND',
[19:57:02.668]   requireStack: [
[19:57:02.668]     '/vercel/path0/node_modules/caniuse-lite/data/features.js',
[19:57:02.668]     '/vercel/path0/node_modules/caniuse-lite/dist/unpacker/features.js',
[19:57:02.668]     '/vercel/path0/node_modules/caniuse-lite/dist/unpacker/index.js',
[19:57:02.668]     '/vercel/path0/node_modules/autoprefixer/lib/autoprefixer.js'
[19:57:02.668]   ]
[19:57:02.668] }
[19:57:02.689] Error: Command "npm run build" exited with 1

Firstly, plz remove your node_modules folder by typing the below command in your project.
rm -rf node_modules

(You might need to delete node_modules folder in your repository too. It is unnecessary to store node_modules on your repo.) https://github.com/gu-does-git/gudoesdev/tree/master/node_modules
Then, install dependencies again with
npm i

As the final step, build the project and start the server
npm run build

npm start

The error might be gone after you do these steps. If not, let me know.

IT WAS THE NODE MODULES 😭😭😭😭
after removing the node_modules, it worked!
(weird thing is that in my .gitignore there's an entry to ignore node_modules)
Anyway, Thanks for the help!!!


UWP notifications not firing

Have implemented this into our xamarin app. Working great on Android and now I tried to get it to work in UWP
Registration works I get the proper WNS uri as token. When I send a WNS message after starting the app nothing happens on the windows device (laptop, phone, tablet). I noticed that when the app is either not started or in the background I received the message on the phone but not in the app. none of my breakpoints ever fire.
calling this in the App constructor in the UWP project
CrossPushNotification.Initialize();
any idea what could be missing?
thanks
Mike

just to add some more information.
In order to verify that push is actually working I commented out the initialization of the CrossPushNotification and added my own code to connect a push notification channel and with my own code it works. I receive push notifications inside the UWP app.

looks like there is an exception thrown inside the json serializer
jobject = JObject.FromObject(args.ToastNotification, serializer);
throws an exception 'The remote procedure call failed'

did a test by modifying the code to create a simpler object and it works.


Notebook 05g hyperparameters are not being used.

In the cell where the {DIR}/train.py is being written you have declared the arguments --lr (learning_rate) and --m (momentum) in the parser.
However, those variables are not being used in the optimizer (SGD).
I think the correct definition of the optimizer will be:
opt = tf.keras.optimizers.SGD(learning_rate = args.learning_rate, momentum = args.momentum)
Am I correct? Or how can the hyperparameters be connected with the training of the model??
Thank you for your work.

Great catch!  You are correct, the current values of the hyperparameters need to be passed to the optimizer definition in this example.  It turns out it was running without error because it was just using the default values for learning_rate and momentum.  This shows up in the results as the exact same value for loss regardless of the turning run.  I am making the correction and will push the update after I test it today.  Thank You!!
The examples in 05h and 05i don't have the mistake and take advantage of TensorBoard HPARAMS parallel coordinates view which would have likely helped me diagnose/notice this issue.


Update benchmark table in Readme

Closes #469

Adventures in Raspberry Pi benchmarking
I thought I was going mad. The numbers you quote here for the Raspberry Pi 4B are substantially worse than even the baseline that I measured in #462. Unfortunately, after doing a Manjaro OS update my Pi developed a boot issue, which I can't fix because I don't have a USB keyboard to hand. Thus, I couldn't actually re-run the benchmarks.
However, I had another MicroSD card lying around with an old Manjaro ARM image on it, so I used that instead, and was able to run benchmarks again. I got similar numbers to what you have here, e.g. 31.4 ms for QuickNet single-threaded. This is way worse than the 30.5 ms that I measured with the exact same hardware only a week or so ago in #462.
After quite a long time trying to work out what was going on, I physically handled the device, and lots of parts of it were very hot, including not just the CPU but also the USB/networking stack. And then I remembered that it's possible to disable some parts of the Pi to save power through a boot config file: /boot/config.txt. In fact I had previously done just this - I had disabled WiFi and Bluetooth on the device (I used a LAN cable for networking). But swapping to the other SD card meant this change was no longer applied.
After making the same change with the new MicroSD card, I restarted it, and recorded some highly variable benchmark runs on single threaded QuickNet, including one with an average latency of 30.7 ms. This is not quite as good as the 30.5 ms from #462, but is pretty close. However, the results were really sporadic. Most runs were around 31.4/5/6 ms, and that 30.7 was the only run (out of maybe 10) with an average below 31 ms. As a result, I'm not sure at all that disabling the WiFi/Bluetooth had an effect.
I have no idea what the conclusion should be drawn here. As far as I can remember, when recording the benchmarks for #462 the results were very consistent, and I haven't been able to reproduce them now. The hardware is clearly capable of faster runs than 31.4 ms, but I'm not sure what to do to make that happen. None of this makes much sense to me, but my best guess is that there's some nuance and/or weirdness relating to the Pi's power management that's at play.
I think the safest option is probably to go with the numbers you've recorded here - at the end of the day there's a < 3% difference between them. They also are definitely achievable with a 'stock' Pi without fiddling about with configuration.


Slowing down?

Since the last 2 months, the speed that the code runs on seemed to... slow down by about 4 times?
It's not a wifi or computer issue, as it's the same on other devices and other networks

code doesn't simply run 'slower' for no particular reason. It's more likely that the servers of deviantArt are having higher response times than usual, maybe due to DDoSing or infrastructure migration.


Is there a way to prevent thoughts from disappearing?

I.e. the intermediate thoughts appear in the most recent answer from the agent. However, when more messages are added, the intermediate thoughts disappear from the older answers (leaving just the final answer). I would like to be able leave the intermediate thoughts in place, to allow going back and inspecting them.

Hi, we don't have a simple built in way to do it, but you can retrieve the intermediate_steps object from the agent response and use that to re-draw the status in future streamlit reruns so it's persisted.
See a simple implementation here, particularly the highlighted portion.


Add support for the kakoune text editor

This is my first contribution to home-manager, so I’m not sure about some of the things I’ve done. In particular, I’m not sure about how to handle the following issue: The tabstop option, for instance, is set to 8 by default in kakoune and this value is assumed if not overriden in kakrc. Should I therefore:

Make a nix option of type int with default 8 and always emit it to kakrc?
Make a nix option of type int with default 8 and only emit it to kakrc if it differs from the default?
Make a nix option of type null or int with default null and only emit it to kakrc if it isn’t null?

Right now I’ve chosen the second option, was that correct?.

imo, #3 is best (null), with the default value mentioned in the
docstring. afaik, most options are "omitted" if null. and it's more future
proofed (e.g. if the underlying application changes its default). i feel
like subtle issues might come up if a user sets 8 explicitly, expecting
it to work, but then is confused when it doesn't (e.g. if there's a
global/system config that has a 7, which the user isn't aware of, and
which #2 may run into.)
(but I haven't contributed to home-manager yet, so take this with a grain
of salt).
On Mon, Mar 25, 2019, 08:34 Sebastian Zivota<EMAIL_ADDRESS>wrote:

This is my first contribution to home-manager, so I’m not sure about some
of the things I’ve done. In particular, I’m not sure about how to handle
the following issue: The tabstop option, for instance, is set to 8 by
default in kakoune and this value is assumed if not overriden in kakrc.
Should I therefore:

Make a nix option of type int with default 8 and always emit it to
kakrc?
Make a nix option of type int with default 8 and only emit it to
kakrc if it differs from the default?
Make a nix option of type null or int with default null and only
emit it to kakrc if it isn’t null?

Right now I’ve chosen the second option, was that correct?.
You can view, comment on, or merge this pull request online at:
https://github.com/rycee/home-manager/pull/637
Commit Summary

programs: Added module for the kakoune text editor
kakoune: Added a lot of options

File Changes

M modules/modules.nix
https://github.com/rycee/home-manager/pull/637/files#diff-0 (1)
A modules/programs/kakoune.nix
https://github.com/rycee/home-manager/pull/637/files#diff-1 (342)

Patch Links:

https://github.com/rycee/home-manager/pull/637.patch
https://github.com/rycee/home-manager/pull/637.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rycee/home-manager/pull/637, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACNoMQqnPfC8YwhJtTLkBGAQGCUfUjy4ks5vaOyagaJpZM4cHMHo
.

Both option 1 and 3 are OK with me. Home Manager already contain both these techniques.
I sort of am partial to 1 since it leads to nicer documentation and it is clearer for the user which value will be used in the end. But I think some others disagree with me on this. So I think as the module writer you can choose 🙂
(as long as it's not option 2)

I’ve now changed most of the options to be in accordance with variant 3 above. It didn’t seem sensible to make indentWithTabs and incrementalSearch have type nullOr bool, though. Is that acceptable?

How can I check the documentation that I’ve written? I can’t find a way to compile it.

Sorry for the delay! The easiest way to see the documentation is to use home-manager switch using your repo clone. Then man home-configuration.nix will contain your changes. You can also set
manual.html.enable = true;

in your Home Manager configuration and after home-manager switch you should be able to run
$ home-manager-help

to open the HTML-manual in a browser.

Thank you very much for the information. There were definitely some issues left with the documentation ;)

Thanks! Rebased to master in 7d68c46feb845c572ef335f824062f90fdebf655.
Note, I made a few minor changes and reformattings. Hopefully nothing broke 🙂


Find a better way to add to the changelog

Every time a PR gets merged, almost all other PRs end up requiring a rebase to fix up the changelog. This is annoying for everyone involved.

I definitely agree. If it may help as a source of inspiration, this is what we use in rkt: basically a script to collect all PR titles and summaries since last tag. It is up to the release pilot (the one currently owning the GPG key) to do the final cosmetic pass and push the artifacts. I think it could be even further automated via cargo-release.
/cc @sunng87

Thank you for mentioning. For now cargo-release cannot generate changelog because I still haven't found a standard way to collect changes. In my previous team, we use (feat), (fix) to prefix our commit message. But generating meaningful changelog is still difficult with that.
Changelog is an important part of software and maybe we need to write it formally just like our code.
Any suggestion is welcomed.

I think there should be a command-line option like --auto-changelog that you can opt into that will compile the changelog from commits that use the special syntax. The syntax should follow from that used by keepachangelog.com, which is Added, Changed, Fixed, Removed. Maybe something like:
Commit title:

Add AioCb::from_boxed_slice

Commit body:

The existing AioCb constructors work for simple programs where
everything is stored on the stack.  But in more complicated programs the.
borrow checker can't prove that a buffer will outlive the AioCb that
references it.  Fix this problem by introducting
AioCb::from_boxed_slice, which takes a reference-counted buffer.
Added AioCb::from_boxed_slice #582

This would then be appended to the end of the "Added" section of the changelog like:

Added AioCb::from_boxed_slice (#582)

So that last line follows the format of starting with one "Added", "Changed", "Fixed", or "Removed" (and various other conjugates of those verbs) following by a bunch of text. If there's a #XXX at the end, assume it's a valid PR and format it as a hyperlink as shown above with the proper URL.
I have another couple of projects on GitLab (gattii and serialport-rs) and I'd love if this could be designed from the beginning to support GitLab as well as GitHub (there they call PRs MRs) and I can help with testing that as well.

@Susurrus I'd be wary of mandating a single syntax, as I already seen multiple attempted proposals and all of them with limited success. Mostly because different projects have different requirements, and a one-size-fits-all quickly becomes too baroque.
To avoid stealing this thread I'm moving this cargo-release discussion over to https://github.com/sunng87/cargo-release/issues/31, so that nix maintainers can still discuss what they need here without too much noise.


Not using event.preventDefault on v12.2.5 and later

Version 12.2.5 of ng-file-upload is not using event.preventDefault anymore, which is causing chrome to redirect when user clicks on upload button.
A workaround for this now is using ng-click='$event.preventDefault()'.

Also event.stopPropagation() is not being used.
Line: https://github.com/danialfarid/ng-file-upload/blob/master/src/select.js#L160


Document SSL_OP_ENABLE_MIDDLEBOX_COMPAT

Documentation for SSL_OP_ENABLE_MIDDLEBOX_COMPAT got missed from an earlier PR.

Checklist

[x] documentation is added or updated
[ ] tests are added or updated

Darn. I spotted another bit of related documentation that is also missing. I will add that in a separate PR. I've pushed this one.


Permissions api

@DanielMSchmidt branch has been aligned with a new refactor to Device done on master, now running in CI.
I am writing documentation right now, hopefully we'll have everything merged and published tomorrow morning.

Super awesome, I will test it this afternoon 👍


[terra-table] Uplift Table components, create HTML Table

Summary
Uplift the terra-table component for accessibility and to handle selections, sections, and fixed header.
Create terra-html-table component to be a read-only pure semantic table.
Resolves #2132
Needs UX pass for icons and colors.
Thanks for contributing to Terra.
@cerner/terra
Deployment:
https://terra-core-deployed-pr-2615.herokuapp.com/components/terra-table/table/table

Is there a high-level summary of the packages being added for this change?

UX Input
Fusion Theme callouts are available on Zeplin.

Basic Table
Interactive Table

Could this have been split into multiple smaller PRs? 👀

Are there Test links for Table? I am not finding them

Functional review complete.  All issues that were identified have been resolved.


failed to save go file

local format_sync_grp = vim.api.nvim_create_augroup("GoImport", {})
vim.api.nvim_create_autocmd("BufWritePre", {
pattern = "*.go",
callback = function()
require('go.format').goimport()
end,
group = format_sync_grp,
})
when update to commit c61f9371cdaaec40cccf0783ff968bee83df5bda， could not save go file，write command is not working
remove  goimport config upon or rollback to last commit，format and import on write command works well.

Most likely you have other format tools hooked into BufWritePre
As currently goimport is async by default if you set it to use gopls. Autocommand from other setups may overwrite the buffer.

Most likely you have other format tools hooked into BufWritePre As currently goimport is async by default if you set it to use gopls. Autocommand from other setups may overwrite the buffer.

I have carefully reviewed my settings and indeed, such an issue existed. After making modifications, the problem has been resolved. Thank you very much.


Feature Request: Allow to create new "select" value if it doesn't exist

For example, we have here optionA, optionB, optionC. But we will allow on frontend side for user to send a request with extra value optionD, which we don't have in our list. For such cases, it makes sense to have a checkbox in field settings to enable autocreate of this value.

I think this will be confusing and difficult to restrict who can add or update options.
The select field works only with predetermined choices. If you want to allow arbitrary number of choices, I'd recommend creating a separate collection with its own API rules and link it with a relation field.


Android SDK License Agreement is a text box when it should be a checked box

Description
WHAT: Android SDK License Agreement is a text box when it should be a checked box
WHERE Catalog >> Mobile >> Services >> Mobile CI|CD
WHEN: trying to create it.
IMPACT: Usability

Environment
Operating system: iOS
OpenShift versions: 3.9
Steps to reproduce

Go to Catalog >> Mobile >> Services >>
Create a new Mobile CI|CD

Hi @camilamacedo86,
Thanks for your feedback. This repo is now deprecated and is replaced by Mobile Developer Console. Could you please give it a try and if you still experience the same problem, please open a new issue in that project.
Thanks.


Dockerfile

Would it be possible to create a Dockerfile so we can run the whole pipeline out of the box? It would be very practical to use this Dockerfile directly or use it as a dependency list to set up the environment needed in order to run this pipeline.

See https://github.com/microsoft/OmniParser/blob/76d6110a3545d180ec528b350f6a0cda2159a45a/Dockerfile (from https://github.com/microsoft/OmniParser/pull/52).


spotify-tui: add collection variant patch

Description of changes
Add patch adding the collection variant to rspotify used by spotify-tui.
https://github.com/Rigellute/spotify-tui/issues/960#issuecomment-1108813658
This fixes the issue of getting an error when playing liked songs. https://github.com/Rigellute/spotify-tui/issues/960

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

It fails to build for me:
In this branch:
nix-build -A spotify-tui .

gives:
error: failed to sync

Caused by:
  failed to load pkg lockfile

Caused by:
  failed to resolve patches for `https://github.com/rust-lang/crates.io-index`

Caused by:
  failed to load source for dependency `rspotify`

Caused by:
  Unable to update /build/source/rspotify-0.10.0

Caused by:
  failed to read `/build/source/rspotify-0.10.0/Cargo.toml`

Caused by:
  No such file or directory (os error 2)

Traceback (most recent call last):
  File "/nix/store/r3nwnppm7n0fbnynfrx2q0dzjz5dgfar-cargo-vendor-normalise/bin/.cargo-vendor-normalise-wrapped", line 42, in <module>
    main()
  File "/nix/store/r3nwnppm7n0fbnynfrx2q0dzjz5dgfar-cargo-vendor-normalise/bin/.cargo-vendor-normalise-wrapped", line 17, in main
    assert list(data.keys()) == ["source"]
AssertionError
error: builder for '/nix/store/a721mj4bc6jzdiw319api0y74fm069mk-spotify-tui-0.25.0-vendor.tar.gz.drv' failed with exit code 1;
       last 10 log lines:
       >   failed to read `/build/source/rspotify-0.10.0/Cargo.toml`
       >
       > Caused by:
       >   No such file or directory (os error 2)
       > Traceback (most recent call last):
       >   File "/nix/store/r3nwnppm7n0fbnynfrx2q0dzjz5dgfar-cargo-vendor-normalise/bin/.cargo-vendor-normalise-wrapped", line 42, in <module>
       >     main()
       >   File "/nix/store/r3nwnppm7n0fbnynfrx2q0dzjz5dgfar-cargo-vendor-normalise/bin/.cargo-vendor-normalise-wrapped", line 17, in main
       >     assert list(data.keys()) == ["source"]
       > AssertionError
       For full logs, run 'nix log /nix/store/a721mj4bc6jzdiw319api0y74fm069mk-spotify-tui-0.25.0-vendor.tar.gz.drv'.
error: 1 dependencies of derivation '/nix/store/3750ndflrd18gxkfrvwa024sxphza6a2-spotify-tui-0.25.0.drv' failed to build

Any idea why?

Oh I also had this error previously. I think because the rspotify-0.10.0 directory does not get linked correctly. But it is weird, that it builds on my machine. I'll clear my cached nix store builds and have another look...

Maybe you have a better idea how to patch rspotify and use it in the Cargo.lock. I haven't really done much with rust yet so I have limited experience.

Maybe you have a better idea how to patch rspotify and use it in the Cargo.lock. I haven't really done much with rust yet so I have limited experience.

I am also honestly not sure how to do this, I don't have too much rust/nix packaging experience. I tried to find similar packages in nixpkgs that do this, but I couldn't find any. Simply replacing a rust crate with a patched crate doesn't seem like an odd thing to do, so I'm not sure why I can't figure it out

Could you try again now?
Apparently a combination of cargoPatches and patches did the trick...

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

1 package built:

spotify-tui

Could you try again now? Apparently a combination of cargoPatches and patches did the trick...

works now using nixpkgs-review pr 170915. I guess cargoPatches is only for the lock file? https://github.com/NixOS/nixpkgs/blob/master/doc/languages-frameworks/rust.section.md#building-a-crate-with-an-absent-or-out-of-date-cargolock-file-building-a-crate-with-an-absent-or-out-of-date-cargolock-file
Anyways, I can't seem to reproduce the upstream issue of the inability to play liked song, they play fine for me. But I guess it depends on your spotify account?
If this patch fixes things for you then LGTM, it's better if upstream would move to a newer rspotify version though... But for now this will do.

For me, it only occurred when I previously played my liked songs on my mobile phone or via the web client, and never through spotify-tui.
And I agree that an upstream update would be best, but as far as I understood rspotify had a lot of breaking changes, so it would take a considerable effort.

Yeah I thougt about that as well, but apparently they are just joined together if I read the source  correctly. But hey if it works ¯\_(ツ)_/¯

If it works it works!

For me, it only occurred when I previously played my liked songs on my mobile phone or via the web client, and never through spotify-tui.

Ah yes, can confirm that this fixes the issue. I could reproduce it by playing something on my phone in liked songs and then opening spotify-tui

And I agree that an upstream update would be best, but as far as I understood rspotify had a lot of breaking changes, so it would take a considerable effort.

Yeah exactly, rspotify 0.11.0 apparently has a lot of breaking changes/api changes.
Anyways thanks for the patch!


Is this working on 62.0.2 (Quantum)?

I did exacly all steps on the README, plus disabled Test Pilot and Vivaldi-Fox but the style havent changed.
This is the layout of my browser (locale is pt-br):

Thanks in advance!

Can confirm it working on 63, so very likely works on 62 too. Try this tutorial.


[Federation] Add a note about DNS API scopes to the Federation tutorial

Not sure this is the right place for this, but I wanted to get the text out somewhere before I forgot about it.

This change is

cc @madhusudancs

Thanks for sending this update!
/assign @madhusudancs

Review status: 0 of 1 files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

docs/tutorials/federation/set-up-cluster-federation-kubefed.md, line 110 at r1 (raw file):

the `gcloud` command with the appropriate value in the `--scopes` field.
You cannot modify a GKE cluster directly to add this scope, but you can
create a new node pool for your cluster and delete the old one:

It seems a little odd to have this in the note and the command in a regular command block. I think we should just make this a regular paragraph.

docs/tutorials/federation/set-up-cluster-federation-kubefed.md, line 112 at r1 (raw file):

create a new node pool for your cluster and delete the old one:

```bash

Some docs use bash and some use shell. shell is more common than bash and it is at least consistent with other occurrences in this doc. Could you please change that?

docs/tutorials/federation/set-up-cluster-federation-kubefed.md, line 115 at r1 (raw file):

gcloud container node-pools create new-np \
    --cluster gke-cluster \
    --scopes "...,other_scope,https://www.googleapis.com/auth/ndev.clouddns.readwrite"

I would really prefer the examples to be copy/paste-able. One way to to do that here is:
gcloud container node-pools create new-np \
    --cluster gke-cluster \
    --scopes "$(gcloud container node-pools describe --cluster=gl --zone=us-west1-a default-pool --format='value[delimiter=","](config.oauthScopes)'),https://www.googleapis.com/auth/ndev.clouddns.readwrite"

OR
scopes="$(gcloud container node-pools describe --cluster=gl --zone=us-west1-a default-pool --format='value[delimiter=","](config.oauthScopes)')"
gcloud container node-pools create new-np \
    --cluster gke-cluster \
    --scopes "${scopes},https://www.googleapis.com/auth/ndev.clouddns.readwrite"

Comments from Reviewable

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

Comments from Reviewable

Review status: all files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

docs/tutorials/federation/set-up-cluster-federation-kubefed.md, line 115 at r1 (raw file):
Previously, madhusudancs (Madhusudan.C.S) wrote…
I would really prefer the examples to be copy/paste-able.  In a separate block, also give the command to delete the old node pool?

Comments from Reviewable

Review status: 0 of 1 files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

docs/tutorials/federation/set-up-cluster-federation-kubefed.md, line 110 at r1 (raw file):
Previously, madhusudancs (Madhusudan.C.S) wrote…
It seems a little odd to have this in the note and the command in a regular command block. I think we should just make this a regular paragraph.

Done.

docs/tutorials/federation/set-up-cluster-federation-kubefed.md, line 112 at r1 (raw file):
Previously, madhusudancs (Madhusudan.C.S) wrote…
Some docs use bash and some use shell. shell is more common than bash and it is at least consistent with other occurrences in this doc. Could you please change that?

Done.

docs/tutorials/federation/set-up-cluster-federation-kubefed.md, line 115 at r1 (raw file):
Previously, madhusudancs (Madhusudan.C.S) wrote…
Oh I forgot. In a separate block, also give the command to delete the old node pool?

Done and done.

Comments from Reviewable

Thanks for the comments! PTAL.

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

Comments from Reviewable

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

docs/tutorials/federation/set-up-cluster-federation-kubefed.md, line 103 at r1 (raw file):
Previously, perotinus (Jonathan MacMillan) wrote…
Done.

Done.

Comments from Reviewable


Include @angular/pwa in ng update

🚀 Feature request
Command (mark with an x)

- [ ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [X ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Description

Update @angular/pwa via ng update
Describe the solution you'd like

Hi. Since @angular/pwa is updated frequently alongside @angular/cli, it would be very useful to update both packages via ng update, which also increments consistency with another packages behavior. Thanks.
Describe alternatives you've considered

Alternative is to update @angular/cli and then run npm update for @angular/pwa

Strictly speaking @angular/pwa is not needed as a dependency. This is because the pwa package is a schematic and is only used once when generating adding pwa capabilities to your application. After that, this package is not used.

Interesting. I didn't know that. Thank you.


[theming] reduce webpack compile time during developing

Hi Guys, Is there any way to reduce webpack building .less file time?
I followed this article to create my own theme: https://gitter.im/Semantic-Org/Semantic-UI-React
However, every time I modified the .less file it would cause 3~5 minutes rebuild time.
Unfortunately I couldn't use the article's method like: include only component you used.
Because I use almost all of components.

Hi @DerekHung!
None of the LESS files are contained in this library. We only handle the React bindings that render SUI markup. The styles are in either the main SUI project, or related projects under Semantic-Org. If you are looking for only the LESS files, you may want to ask in the proper repository. https://github.com/Semantic-Org/Semantic-UI-LESS
or
https://github.com/Semantic-Org/Semantic-UI


replaced getUserName with getDisplayName for comments

This is to ensure that in notifications, those users who do not have a username,
are still displayed using their display name.
This is required because certain users are missing user names (caused by the usage of
the social sign-up options which does not create a username)

Thanks!


pydocstyle corections on extentions.py for #742

This is for #742
I changed the format of the docstrings for cookiecutter/extentions.py to followe pep257 style

Codecov Report

Merging #996 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #996   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          18     18
  Lines         780    780
=====================================
  Hits          780    780

Impacted Files
Coverage Δ

cookiecutter/extensions.py
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Hi @meahow! 👋
Thank you for your contribution. 🍪


Sort columns by dependency before groovy transformation

This PR aims to solve the  problem with transformation dependencies https://github.com/apache/incubator-pinot/issues/5351
We try to build the dependency graph between columns and sort them topologically before applying the transformation
Description
Add a description of your PR here.
A good description should include pointers to an issue or design document, etc.
Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes
If you have tagged this as either backward-incompat or release-notes,
you MUST add text here that you would like to see appear in release notes of the
next release.
If you have a series of commits adding or enabling a feature, then
add this section only in final commit that marks the feature completed.
Refer to earlier release notes to see examples of text
Documentation
If you have introduced a new feature or configuration, please add it to the documentation as well.
See https://docs.pinot.apache.org/developers/developers-and-contributors/update-document

Codecov Report

Merging #6241 into master will decrease coverage by 20.15%.
The diff coverage is 50.60%.

@@             Coverage Diff             @@
##           master    #6241       +/-   ##
===========================================
- Coverage   66.44%   46.29%   -20.16%
===========================================
  Files        1075     1240      +165
  Lines       54773    58948     +4175
  Branches     8168     8740      +572
===========================================
- Hits        36396    27292     -9104
- Misses      15700    29410    +13710
+ Partials     2677     2246      -431

Flag
Coverage Δ

integration
46.29% <50.60%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ot/broker/broker/AllowAllAccessControlFactory.java
100.00% <ø> (ø)

.../helix/BrokerUserDefinedMessageHandlerFactory.java
52.83% <0.00%> (-13.84%)
:arrow_down:

...org/apache/pinot/broker/queryquota/HitCounter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...che/pinot/broker/queryquota/MaxHitRateTracker.java
0.00% <0.00%> (ø)

...ache/pinot/broker/queryquota/QueryQuotaEntity.java
0.00% <0.00%> (-50.00%)
:arrow_down:

...ker/routing/instanceselector/InstanceSelector.java
100.00% <ø> (ø)

...ceselector/StrictReplicaGroupInstanceSelector.java
0.00% <0.00%> (ø)

...ava/org/apache/pinot/client/AbstractResultSet.java
26.66% <0.00%> (-30.48%)
:arrow_down:

.../main/java/org/apache/pinot/client/Connection.java
22.22% <0.00%> (-26.62%)
:arrow_down:

.../org/apache/pinot/client/ResultTableResultSet.java
24.00% <0.00%> (-10.29%)
:arrow_down:

... and 1228 more

Continue to review full report at Codecov.

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


Multithreading?

Is it possible to multithread the application?

Not quite sure what you mean as it is multithreaded. If you mean play on multiple accounts at the same time that is unfortunately not possible.


Mapping approaches for spatial object detection

Hi there,
I was wondering if there exists a solution tested with the spatial_bb node to remember the 3d position of annotated objects in long term memory. This is a follow up question to #450

Hi, as of now there is no such example in this repository, but in theory it should be pretty straightforward to create another node that subscribes to this information and keeps it in memory/in a file. I think a good reference could be this project.

Closing this issue, if you have further questions please feel free to reopen it.


PR is not updated with new version

We've updated to the new greenkeeper github integration. Now the PR is not updated with new package version, instead it is adding a comment where I have to manually create a new PR with the new version. It would be better if it could just update the existing PR. This new workflow is actually causing more work and adds the potential for more packages to become out of date because the updated version will be ignored if the original PR is merged.

Here is a good example https://github.com/greenkeeperio/greenkeeper/pull/393


GDPR Compliance

Fixes #12 and Fixes #4
This will finalize Cookies and make it GDPR Compliant.

I have fixed all of the issues you pointed out and hopefully fixed some of the config issues too.
Will need you to take another look.


zstd v1.4.5 support!?

Hello,
Wondering if there are any plans to add support for zstd v1.4.5 support to take advantage of the recent decompression speeds .. thanks!

Hi,
zstd v1.5.0 is now available. Just checking for an update.
@spiritloose, do you plan to maintain this repo?  Thanks!


Update UG and UI related issues.

Codecov Report

Merging #192 (2dbfc80) into master (87d4604) will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #192      +/-   ##
============================================
+ Coverage     55.72%   55.74%   +0.01%
  Complexity      699      699
============================================
  Files           138      138
  Lines          2988     2987       -1
  Branches        391      389       -2
============================================
  Hits           1665     1665
+ Misses         1133     1132       -1
  Partials        190      190

Impacted Files
Coverage Δ

.../main/java/seedu/address/ui/MemberDetailsCard.java
0.00% <0.00%> (ø)

...c/main/java/seedu/address/ui/MemberViewWindow.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Move to new version of Angular

2 versions behind @ 1.2

I've actually been keeping it at 1.2 because I had a business requirement to make it work in IE8, and 1.2 is the latest version that supports IE8. I figured I'd keep it there in case any others were also stuck supporting IE8. That's just the minimum required version, though--it works with 1.3 and 1.4 also.


Revert workarounds for JaCoCo issue with JDK 15

JaCoCo 0.8.6 is released.
@hazendaz ,
We should do this, right?

All good, I'd drop jdk 13/14 though and add 16.  While 13/14 are out there, bother are EOL now.

Thank you @hazendaz  for the merge and the follow up PR!


player can't accept tpa request from other server.

player can't accept tpa request from other server.
i tried reconnect or restart server but still same.
The settings of huskhomes and versions are all the same, and only the server.yml in the huskomes folder is different.
video: https://youtu.be/e7Rb807C6vY

Unable to reproduce. Likely a locale issue as the video demonstrates broken locales. Regenerate these.

I've just been setting things up on a small Velocity network and have hit a similar issue. If I have both servers set up identically with the latest version available for Paper on 1.21.1 (admittedly an alpha release!), then using the pluginmsg API cross-server TPA doesn't seem to work - the other player just receives nothing, though I can autocomplete the TPA command just fine with player names that are on other servers. If I swap to use Redis instead, it all works fine. No other changes made, just swapping to Redis and it seems totally happy.

Is the bungee plugin messaging channel turned off on your velocity proxy?

Aah interesting... it is, this is probably an issue because of ChatControl Red and VelocityControl in that case, apologies for misreporting. They promise in their setup that they handle it instead:
Set bungee-plugin-message-channel to false in your velocity.toml. Don't worry, if you have plugins utilizing this channel we will handle their data manually.


add gif to readme

I appreciate your efforts @thepmsquare ! But someone has already made a PR for that, so according to FCFS(First Come First Serve), I cannot merge this now.
Will be looking forward for more contributions from you. :+1:  :smile:


Column concatenation in .pdb file when residue index > 999

I was running RFdiffusion with dl_binder_design pipeline when suddenly I started randomly getting error about having non-unique residue indices. It seemed strange at first, as the output file from RFdiffusion was intact (chain A - binder, chain B - target, each residue index is unique). However, this is what happens when the residue index > 999:

ATOM  15333 2HE  MET B 999      29.180 -52.646 -15.485  1.00  0.00           H
ATOM  15334 3HE  MET B 999      27.618 -52.456 -14.654  1.00  0.00           H
ATOM  15335  N GLY B1000      26.291 -58.439 -16.471  1.00  0.00           N
ATOM  15336  CA  GLY B1000      26.746 -59.823 -16.438  1.00  0.00           C

At res.index >= 1000, columns 5 (chain) and 6 (res.index) are concatenated, so the input_check function inside interfaceAF2predict.py script is checking the wrong column at this point.
I guess this is linked to .pdb format itself. I implemented a small fix to get around this.

Thanks for notifying me about this issue and opening a PR. I have a slightly different implementation of this fix in mind and I will push that today.

I've pushed a fix to this. Thanks again!


Using PHPUnit expectations in @Then step?

There are examples in the Behat documentation that show how to use PHPUnit assertions in the @Then step. What if I don't expect some object property to be set, but instead, a method being called, like sendEmail()?
What I tried:
/**
 * @Then an offer to download the ebook must be sent via email
 */
public function anOfferToDownloadEbookMustBeSentViaEmail( $arg1 ) {

    $this->Lead->expects( TestCase::once() )->method( 'send' )->with( TestCase::equalTo( 'ebook' ) );

    $this->Lead->isEligibleForNextDigitalProduct();
}

But even though send is never called in my Lead::isEligibleForNextDigitalProduct method, the test passes.  Did I misunderstand something?

Well, I think you are missing the call checking the expectation on the mock object in your case (something you don't see when using mocks inside PHPUnit because PHPUnit does it for you already).
The exception would happen inside $mockObject->__phpunit_verify() and you don't call it (see the private TestCase::verifyMockObjects method to see the implementation in PHPUnit)

Thanks for the hint. Does it maybe make more sense to use something like Mailtrap - because all I do here is to check that a method was called, but the behaviour I really want to test via BDD/Behat is whether an email is sent.


Add ".asciidoc" as acceptable file suffix

I use the ".asciidoc" suffix for a lot of my files and so I suggest to make it an accepted suffix.
I could not find a test for legal (and illegal file suffixes), so no test for suffix ".asciidoc" added. If you think we could need such tests, I will try to come up with an appropriate test.

I'll look into this. Shouldn't be hard to support that.

I have added support for .asciidoc and .ad in the new release, v4.0. Please do try out the release.

Hi Saneef,
thanks for accepting my suggestion. I'm just using the plugin on a few projects, so far with no problems.
BR
Nik