ninja rules not conceived from custom toolchain

Describe the bug
ninja rules are not being conceived from from custom toolchain, likely as xmake not recognising such. Subsequent wondering whether xmake is even utilising those binaries on its own.
Expected behavior
ninja rules being conceived, and utilised by xmake, from custom toolchain
Error output
there is no error output but just a lack of ninja rules
Related Environment
Please provide compiling and running environment information:

xmake version: v2.3.6+202008021551 dev
os: linux/x86_64

Additional context

with this custom toolchain

toolchain ("my_llvm")
  set_kind  ("standalone")
  set_toolset ("cc", "clang-10")
  set_toolset ("cxx", "clang++-10")
  set_toolset ("as", "clang-10")
  set_toolset ("ar", "llvm-ar-10")
  set_toolset ("ld", "ld.lld-10")
  set_toolset ("mm", "clang-10")
  set_toolset ("mxx", "clang++-10")
  set_toolset ("sh", "ld.lld-10")
  set_toolset ("strip" , "llvm-strip-10")

  add_defines ("my_llvm")
toolchain_end()

ninja rules for ld & sh & strip not being conceived.
N.B. ld.lld-10 is the linker binary (actually it symlinks to /usr/lib/llvm-10/bin/ld.lld) provided from LLD (https://lld.llvm.org/design.html | https://lld.llvm.org/index.html)
As clang is only a compiler and not a linker and on its own leverages only the standard systemlinker lld needs to be be specified instead.

changing in the custom toolchain

- set_toolset ("ld", "ld.lld-10")
+ set_toolset ("ld", "clang++-10")

then a ninja rule for ld is being conceived, similar when changing sh in the toolchain. Still leaves strip rule not being conceived.

Somehow it seems that xmake does not recognise/acknowledge as valid the binaries:

llvm-strip-10
ld.lld-10

and discards them from conceiving ninja rules. Since this happens it raises the question whether xmake in its processing also discards them or whether the issue is limited to conceiving the ninja rules.

ninja rules for ld & sh & strip not being conceived.
N.B. ld.lld-10 is the linker binary (actually it symlinks to /usr/lib/llvm-10/bin/ld.lld) provided from LLD (https://lld.llvm.org/design.html | https://lld.llvm.org/index.html)
As clang is only a compiler and not a linker and on its own leverages only the standard systemlinker lld needs to be be specified instead.

I know this, but for xmake, we will try to use clang++ as ld. Although clang++ is just a compiler front end, it will also call the real ld internally to link the program. The advantage of using ld is that we don’t need additional configuration. e.g. libc++, crt.o and other additional compiler library links and search paths will be handled automatically by clang++, which is more reliable, less error-prone, and convenient for us and your configuration process.
If you use lld directly, you may need add -lc++ -L/clang/lib and etc .. manually.
Therefore, for xmake, we use g++ and clang++ as ld first. If it is only a c program, you can also use gcc/clang directly. Usually we don't directly pass ld/lld to ld config. If you want to modify the internal ld program, you can set -fuse-ld=lld to clang++

then a ninja rule for ld is being conceived, similar when changing sh in the toolchain. Still leaves strip rule not being conceived.

The default release rule add_rules("mode.release") is configured with set_strip("all"), it will automatically use clang -s to strip the program, it has the same effect as using strip independently, so usually we will not call the strip command extra.
Only the mode.releasedbg rule will additionally call strip because it will also generate additional symbol files.
However, rules and custom scripts only support direct compilation and run mode xmake build, and plugins generated by projects (xmake project -k ninja/cmake/make ..) are not supported.
Usually, we would recommend that users run the xmake command directly to compile the program instead of generating third-party project files. This is not our main feature.

Thank for the valuable feedback. I would thus adjust the custom toolchain to
toolchain ("my_llvm")
  set_kind  ("standalone")
  set_toolset ("cc", "clang-10")
  set_toolset ("cxx", "clang++-10")
  set_toolset ("as", "clang-10")
  set_toolset ("ar", "llvm-ar-10")
  set_toolset ("ld", "clang++-10")
  set_toolset ("mm", "clang-10")
  set_toolset ("mxx", "clang++-10")
  set_toolset ("sh", "clang++-10")
  set_toolset ("strip" , "llvm-strip-10")

  add_defines ("my_llvm")

  on_load(function (toolchain)
    toolchain:add ("ldflags", "-fuse-ld=lld, -stdlib=libc++")
    toolchain:add ("shflags", "-fuse-ld=lld, -stdlib=libc++")

toolchain_end()

Would that be the way to go?


Fix bug with DKIM and DMARC

🗣 Description
Adjusted the Rego tests for EXO 2.3 and 2.4 so that the DNS answers are expected as a list, rather than a single string.
💭 Motivation and context
Closes #214.
🧪 Testing
Tested on both G5 and E5 tenants. Manually inspected results for correctness as well as compared results with v0.2.0.

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

Compared the Rego and the output with the main branch's.  DNS answers are now expected as a list and it works as intended. Approved.


Add BDD to project

Create an MVP for BDD tests using either of these two:

pytest-bdd
behave

IMHO this would make for a nice addition to the unit tests, because (stating the obvious):

automating the test workflow (beyond unit tests)
easier ensure we don't accidentally break feature X by creating feature Y
Gherkin language gives a good framework to define scenarios/what we want to do in workable pieces
Gherkin language is pretty straight-forwad so it's easier to communicate scenarios with non-tech team members (I'm sure they could easily write them as well with minimal learning)

...

depends on #26


package or namespace load failed for ‘UpSetR’

Description of the bug
I keep getting the following error using nextflow version 22.10.4 and chipseq v2.0.0
Does anyone has any idea what's going on ?
Thanks.
Command error:
Error: package or namespace load failed for ‘UpSetR’ in loadNamespace(i, c(lib.loc, .libPaths()), versionCheck = vI[[i]]):
namespace ‘scales’ 1.1.1 is being loaded, but >= 1.2.0 is required
Execution halted
Command used and terminal output
nextflow run nf-core/chipseq --input sample.csv --outdir test --fasta Arabidopsis_thaliana.TAIR10.dna.toplevel56.fa.gz --gff Arabidopsis_thaliana.TAIR10.56.gff.gz --macs_gsize 119667750 --read_length 150 --save_macs_pileup TRUE -profile ifb_core

Relevant files
No response
System information
No response

Could you provide the whole error message or the .nextflow.log. Also, are you using Conda to run the pipeline?

I'm not using conda. Here is the whole error message:

Error executing process > 'NFCORE_CHIPSEQ:CHIPSEQ:MACS2_CONSENSUS (H3K9)'
Caused by:
Process NFCORE_CHIPSEQ:CHIPSEQ:MACS2_CONSENSUS (H3K9) terminated with an error exit status (1)
Command executed:
sort -T '.' -k1,1 -k2,2n CONTR1_H3K9_peaks.broadPeak CONTR2_H3K9_peaks.broadPeak CONTR3_H3K9_peaks.broadPeak frim7_1_H3K9_peaks.broadPeak frim7_2_H3K9_peaks.broadPeak frim7_3_H3K9_peaks.broadPeak pom_1_H3K9_peaks.broadPeak pom_2_H3K9_peaks.broadPeak pom_3_H3K9_peaks.broadPeak
| mergeBed -c 2,3,4,5,6,7,8,9 -o collapse,collapse,collapse,collapse,collapse,collapse,collapse,collapse > H3K9.consensus_peaks.txt
macs2_merged_expand.py
H3K9.consensus_peaks.txt
CONTR1_H3K9,CONTR2_H3K9,CONTR3_H3K9,frim7_1_H3K9,frim7_2_H3K9,frim7_3_H3K9,pom_1_H3K9,pom_2_H3K9,pom_3_H3K9
H3K9.consensus_peaks.boolean.txt
--min_replicates 1 \
awk -v FS='	' -v OFS='	' 'FNR > 1 { print $1, $2, $3, $4, "0", "+" }' H3K9.consensus_peaks.boolean.txt > H3K9.consensus_peaks.bed
echo -e "GeneID	Chr	Start	End	Strand" > H3K9.consensus_peaks.saf
awk -v FS='	' -v OFS='	' 'FNR > 1 { print $4, $1, $2, $3,  "+" }' H3K9.consensus_peaks.boolean.txt >> H3K9.consensus_peaks.saf
plot_peak_intersect.r -i H3K9.consensus_peaks.boolean.intersect.txt -o H3K9.consensus_peaks.boolean.intersect.plot.pdf
echo "H3K9.consensus_peaks.bed	H3K9/H3K9.consensus_peaks.bed" > H3K9.consensus_peaks.antibody.txt
cat <<-END_VERSIONS > versions.yml
"NFCORE_CHIPSEQ:CHIPSEQ:MACS2_CONSENSUS":
python: $(python --version | sed 's/Python //g')
r-base: $(echo $(R --version 2>&1) | sed 's/^.R version //; s/ .$//')
END_VERSIONS
Command exit status:
1
Command output:
(empty)
Command error:
Error: package or namespace load failed for ‘UpSetR’ in loadNamespace(i, c(lib.loc, .libPaths()), versionCheck = vI[[i]]):
namespace ‘scales’ 1.1.1 is being loaded, but >= 1.2.0 is required
Execution halted
Work dir:
/shared/ifbstor1/projects/project_nbouche/frim7_ChIP_CONTR/work/d0/c9bf0fd655e9a11bbae24e711026ab
Tip: view the complete command output by changing to the process work dir and entering the command cat .command.out

I think that it could be that you have a local version of the R package installed in your environment which overwrites the one in the image (which actually meets the requirements since is 1.14). If this is the case, maybe it will be worth to remove the local version of the UpSetR package.

I closed the issue now as it had any activity, if you need any further help feel free to reopen it.


"Missing structure fields" error in a compiling code

rust-analyzer version: 7ae247f (just pulled a few mins ago)
Error line: https://github.com/osa1/tiny/blob/f69c85234a959f5c3a0a894c30d94b0146786956/libtiny_tui/src/lib.rs#L120
Screenshot:

This code builds fine so there shouldn't be any errors. I think the problem is in name resolution. A few lines above the error line I have this
use tui::TUIRet::*;

and the struct that rust-analyzer complains about is coming from that line (Input). There's another struct with same name: term_input::Input, but clearly that's not what we're referencing here.

Perhaps rust-analyzer doesn't handle overriding existing names after use statements. There's an Input in scope in the error above, but it's overridden by the use tui::TUIRet::*; line.

Local imports (use) are indeed not supported.


[add-resource] New Plugin Image Comparer

I would like to add my new plugin to the Grav Repository.
Here are the project details: https://github.com/Karmalakas/grav-plugin-image-comparer

Added!


Suggested documentation improvements following our integration

We recently incorporated codecov.io into DotNetAnalyzers/StyleCopAnalyzers. Based on this experience, I recommend the following changes to the documentation.
Combine test and code coverage stages
The steps to do this are elusive, but relatively straightforward when you see them.

Add the -returntargetcode argument to OpenCover.Console
Change the after_test section to test_script

:thought_balloon: It might make sense to make a note that the code coverage gathering process may be run as a separate build step for users who are unable to or do not wish to run their complete automated testing through OpenCover. Users will want to integrate codecov.io either in their test_script section or in their after_test section, but not both.
Include information for XUnit users
Users working with XUnit will need to modify their OpenCover call as follows:

The -target argument becomes the following:

-target:"%xunit20%\xunit.console.x86.exe"

The -targetargs argument becomes the following (you may want to replace our target assembly path with the one in your original sample; I wasn't familiar enough with the escape sequences for spaces to do it myself):

-targetargs:"C:\projects\stylecopanalyzers\StyleCop.Analyzers\StyleCop.Analyzers.Test\bin\Debug\StyleCop.Analyzers.Test.dll -noshadow -appveyor"

:bulb: I don't use NUnit, but it would be good to eventually include information for those users as well.
Include sample reports
If you don't want to add C# code directly to this repository, you can instead link to other GitHub projects which are currently using codecov.io. I wouldn't object to including a link to ours. :smile:

Hey @sharwell thank you so much for this contribution. I've updated our readme 2ade5041e115d6383779a3ecaf397406c8c68f39. Cheers!


Enumerated types

Add support for enumerations, i.e. types whose value can be one of a closed set of symbolic constants

One possible implementation is to do basically what C does: the atoms are just a nice placeholder over a numeric value. Keep a table of (string -> number), and carry around a pair of (table, string) for each value of the enumeration.


Nothing happen when trying to interact with underlaying view controller when LPWebInterstitial banner is presented

Expected Behavior
Interact with underlaying view controller when LPWebInterstitial banner is presented.
Actual Behavior
Nothing happen when trying to interact with underlaying view controller when LPWebInterstitial banner is presented. The only way to unfreeze to dismiss banner using close button.
Steps to Reproduce the Problem

Present LPWebInterstitialViewController not in full screen but as a banner.
Try to interact with underlaying view controller.

Specifications

Version: 4.1.0. This bug appear in version 4.0.0. Previously was ok.
Platform: iOS 15.5.

https://user-images.githubusercontent.com/28777985/195603151-0a4cd23c-530d-4f5f-bbb0-eb10e36644cd.mp4

Hi @vlad-shostak
Would it be possible to test using this branch?

Hi @nzagorchev
I tried early to change modalPresentationStyle but it didn't help. And now I tried again and the bug still reproduce.

The changes in the branch mimic the code used before 4.0.0.
Chronologically:

the modalPresentationStyle was changed together with other updates
I applied this fix #496  - the bug was no longer reproducing on our test applications, however, it seems it solved it only partially
I was able to reproduce it again yesterday with another sample app, hence the changes in the branch linked above - it presentOverVisible and uses overCurrentContext.

Using the above branch I no longer reproduce the issue on the sample apps we have.
Let me know if it works with the linked branch, otherwise, we will need a sample application that reproduces the issue.

FYI Branch is merged into master


accessibility: allow me to jump quickly to various debugger treeviews

As a keyboard only user, I'd like to quickly be able to jump to the variables, or watch area of the debugger.

I press f5 to start debugging. A breakpoint is hit, I am placed on the line. If I want to investigate the variables, breakpoints, or call stack, I have to control+shift+d, then tab several times. Then, I press f11 to step in, or f5 to continue, ...
Now, my cursor is thrown into the editor, and I have to repeat those multiple tabs to get back to the variables/call stack/watching.

I propose a keyboard shortcut to quickly ump to the variables area, and/or making it possible to move between the debugger areas with less keystrokes if possible (By putting buttons into toolbars I can arrow along).

@derekriemer Thanks for the feedback. I propose introducing some new actions in the command pallete like

Debug: focus variables
Debug: focus watch expressions
Debug: focus callstack
Debug: focus breakpoints

These actions would have no keybinding by default, but a user could assign any keybinding to those actions.
Does this make sense? I am open for suggestions.

Yes, that would be great. Also, is there a "focus last focused editor" action? sometimes, if I am in variables I want to focus the last editor, but I forget what group it's in, so ctrl+num is a guess.

@derekriemer yes there is "Focus Active Editor Group"

@joaomoreno is there currently some way for me to get an instance of a registered viewlet view?
You can see from this feature reuqest that I would need to get my views in an action in order to focus them.
Currenly the only thing I found is this which seems very internal
https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/browser/parts/views/viewsViewlet.ts#L550

oh, wow, that's useful

@isidorn Yeah that's the problem with the registry... But yes, you should either call this.getViews() and look for the one in there or this.getView(id), both from the DebugViewlet.

@joaomoreno got it. Would you mind if I change the focus of the composite to take the optional ID of the view when focusing? This way I could make everything flow through nicely
https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/common/composite.ts#L49

@isidorn can you delete comment of the localize point?
In transifex, this key has not the "Debug". This comment may be confusing.

@EbXpJ6bp done via 5f2bd928eb

@ichiriac I'm sorry I didn't make it clear enough...
Focus Variables, Focus CallStack, Focus Breakpoints are also same situation.


Is the project abandoned?

There are not any updates about the project for a long time. And there are lots of unanswered issues. Is the project abandoned? I'm not sure that I could get an answer though. What does the community think?

looks dead to me. I am going for:
https://material.io/develop/ios/components/tabs/


Adjust styling to fit new text in image card / link to code schools

Description of changes
Card "Code School Scholarship" swapped position with card "Conference Scholarships". Height and width of the individual cards got adjusted to hold the new content provided for the renamed card "Code School Scholarship". Renamed card "Scholarships" also links to the corresponding page.
Issue Resolved

Fixes #715

True. Was thinking the same when I was getting familiar with the project. The ImageCard component itself is already quite generic, so it makes sense to implement the style the properties as props. Just decided to take this approach, since it was already defined in a static manner in the corresponding css files.

Just a heads up that once #hacktoberfest finishes, we will be combing PRs and finishing them. Let us know if you have a problem with taking the work you've done and completing it ourselves. Your work is still graciously appreciated, and we know that this is all a volunteer effort. Please understand that it is simply a complement - we want your work so bad, that we're going to put it in ASAP!

I'm happy to pick this up and submit an updated pull request - don't want to step on anyone's toes though so let me know.

@tal87 and @kylemh 's points are totally understandable and valid. Currently don't have enough time to adjust my solution to make it more generic though, so feel free to pick it up @ianroberts131!

@kylemh I'm wondering how yo solve the text overflow in the scholarships section. Should I reduce the text size or increase the box? In any case I think we should apply the same to all the boxes of the section.
What do you think?

Looks like it's been a while since this was looked at - what needs to be changed on this?


Added nullability and converted tests

Note: comparison methods removed from header as they are already present on NSObject


fix clippy style warning introduced by #5916

Signed-off-by: Yilin Chen<EMAIL_ADDRESS>
What have you changed?
The original code checks if an Option is not None and unwrap it later. It leads to a clippy warning. This PR resolves it.
What is the type of the changes?

Engineering (engineering change which doesn't change any feature or fix any issue)

How is the PR tested?

[x] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No

@qrr1995 Please investigate why the lint check was skipped in #5916. Thanks!

/merge

/test


http2?

Is there any way to use http2 with serve?

In programatic way, replace http with http2 in https://github.com/zeit/serve/blob/master/README.md#api

Thanks, worked! :)

would be nice to have same as argument


How do I know when a particular message has been delivered to user or not within the conversation ?

I am using facebook messenger and here is my use case:
I want to take feedback from user only after a conversation has end, and the conversation contains many multimedia content like video, images , and carousels.
Code:
`
bot.startConversation(message, (error: any, convo: any) => {
            let attachment = FacebookComponent.createTemplate(attachmentType.template, templateType.generic, ads);
            convo.say(Common.formatString("contextualads.start_message"));
            convo.say({ attachment: attachment });
            convo.next();
            this.commonDialog.takeFeedback(bot, message);
        });
    });

`
But the messages are delivering randomly.
Please help.

I am not sure if I got your problem but it seem to me that you need to pass require_delivery: true, when initializing controller.
var controller = Botkit.facebookbot({
        access_token: process.env.access_token,
        verify_token: process.env.verify_token,
        require_delivery: true,
})

Just look it up in docs, hope it helps.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
21 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
19 seconds
deprecated-base-image-check

✅ Succeeded
20 seconds
sanity-inspect-image

✅ Succeeded
17 seconds
clair-scan

✅ Succeeded
56 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
11 seconds
sanity-optional-label-check

✅ Succeeded
11 seconds
sanity-label-check

✅ Succeeded
5 seconds
show-summary

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
21 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
20 seconds
sanity-inspect-image

✅ Succeeded
19 seconds
deprecated-base-image-check

✅ Succeeded
17 seconds
clair-scan

✅ Succeeded
56 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
11 seconds
sanity-optional-label-check

✅ Succeeded
11 seconds
sanity-label-check

✅ Succeeded
5 seconds
show-summary


How to force BERT DNN featurisation in AutoML

[Enter feedback here]
I have GPU compute and enabled deep learning but only see statistical featurisation in Models Tab ?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8796f28c-1050-d80d-c1c7-ebb0e9fc665f
Version Independent ID: b1f690ff-0f93-402a-2a98-15d3fb6aa0d4
Content: Featurization in AutoML experiments - Azure Machine Learning
Content Source: articles/machine-learning/how-to-configure-auto-features.md
Service: machine-learning
Sub-service: core
GitHub Login: @nibaccam
Microsoft Alias: nibaccam

@ashishanand7 Are you referring to the Azure ML portal(ml.azure.com) models tab? This tab lists all models registered with your workspace.

I am referring to Automated ML function. I am doing text classification in Hindi language about which docs mention AutoML uses Multilingual BERT for featurisation when Deep Learning Checkbox is enabled and we use a GPU compute to train. However when the AutoML run is completed, I am not able to find even one model trained using BERT for featurisation, it's all tf-idf primarily.
I also tried using the SDK with enable_dnn=True instead of portal UI but the results seem to be the same.
My workspace is in SouthEastAsia , is it related to region ?

@ashishanand7 With respect to the application of BERT with AutoML for any other language other than english and german it uses multilingual BERT model.
Multilingual BERT support in AutoML was released with AzureML Python SDK 1.7.0
A sample code snippet:
featurization_config = FeaturizationConfig(dataset_language='jpn')

automl_settings = {
"experiment_timeout_minutes": 20,
"primary_metric": 'accuracy',
"iterations":1,#decide number of iterations you expect
"max_concurrent_iterations": 4,
"max_cores_per_iteration": -1,
"enable_dnn": True,
"enable_early_stopping": True,
"validation_size": 0.3,
"verbosity": logging.INFO,
"enable_voting_ensemble": False,
"enable_stack_ensemble": False,
"featurization": featurization_config,
}

You can also provide feedback related to your current run to our team from the Azure ML portal so our team can advise you further.

@ashishanand7 We will now proceed to close this thread. If there are further questions regarding this matter, please tag @RohitMungi-MSFT in your reply or you can send your feedback from the ML portal as shown in the previous comment. Thanks!!


Debugging breakpoints are not persisted

Debugging breakpoints are not persisted in between debugging sessions - this is particularly an issue when the node.js app crashes and needs to be re-started; all the breakpoints are lost.

Didn't realize this issue already exists at #41


update kyaml to v0.12 and cli-utils to latest

This PR updates kpt to kyaml v0.12.0 and includes an important change referenced by https://github.com/GoogleContainerTools/kpt/issues/2478 and https://github.com/kubernetes-sigs/kustomize/issues/4024.
The following annotations are being migrated:

config.k8s.io/id -> internal.config.kubernetes.io/id
config.kubernetes.io/path -> internal.config.kubernetes.io/path
config.kubernetes.io/index -> internal.config.kubernetes.io/index

For some time, we will support both annotations by adding both legacy and the new annotation when reading files. When writing files, the orchestrator consumes both and reconciles correctly by comparing the resource before and after running a function to infer which annotation has changed (if they differ). See the kyaml PR that implemented this change here: https://github.com/kubernetes-sigs/kustomize/pull/4190.
After some grace period, support for the legacy annotations will be dropped.

Adding e2e tests revealed an issue; will investigate and fix.

@mortent Do you want to take another look given you requested changes earlier in https://github.com/GoogleContainerTools/kpt/pull/2523#pullrequestreview-779279647?


S3 extensions does not retry on 503 slow down responses

Affected Version
0.19
Description
While ingesting a large amount of data using the parallel index method, subtasks and sometimes even the parent task will fail because of an exception due to 503 slow down responses. This is likely because (1) we have lots of files and (2) we were querying the data with Athena at the same time. Either way, the expectation is that Druid would retry these types of failures and expose configurations for tuning the backoff and retry policies.
The current workaround for us is several fold:

Using fewer larger files, but this has disadvantages for our ETL pipeline
Limit the use of other high volume jobs reading from the bucket, but this directly impacts other jobs than can properly handle failures
Build retry logic into our Druid loading process, but this logic seems better suited for Druid to handle.

This particular type of error is particularly painful since there is no configurable retry logic for top level tasks  (https://github.com/apache/druid/issues/5428).

Hi @aewhite, Druid already has a retry logic in reading from S3, but this error is missing there. I guess we could backoff and retry on this error too.

I believe this is fixed in 0.22.0 by #10761, which added retries for throttling exceptions. If anyone sees this on 0.22.0 or newer feel free to reopen.


Updated line 5

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

1 commit


Value match

Spefication
This validator cannot do fixing. It's purpose is to check whether all datafield's subfields match the regex.
Name
datafield-value-match
Configuration
Configuration is passed in as an object which as field tags as keys (The tag is a regex pattern). The values are objects with subfield codes as keys and a regex patterns as value which are used to check if the subfield value matches the pattern.
Example
{
  "013": { "b": "af" },
  "STA":  { "a": "CIRC-CREATED" }
}

Obsolete. See #17


fixing footer

Fixes #22
Added a small margin to the footer.

@tsl143 actually on tushar's laptop logo was looking fine but this links were not loooking down in footer so thats why i changed for the links only :stuck_out_tongue:

It looks uneven from bottom

We need to bottom align logo and links

updated look


mergeSchema - fragments can be included on subschema even if they can never satisfy it

If interface is added to a type at mergeSchema level, the resolve of this type can include fragment that satisfies mergedSchema types, but not subschema types.
Example (second query)

This stale issue is likely solved by ExpandAbstractTypes transform.


feat(spec): remove list/map header from type meta spec

What does this PR do?
This PR removes list/map header from type meta spec.
Such header may be computed ahead sometimes. But it may need to compute based the data at runtime. If we write header into type meta, we may still need to compute and write header when serialization. This will introduce extra space cost.
Instead, we can write all such header at runtime, and when creating/generating serializer, we can compute part of such header ahead, and let remaing parts of header computed at serialization.
When deserialization, the deserializer can generate different deserializer based on read header, which will be more efficient.
Related issues
#1413
Does this PR introduce any user-facing change?

[ ] Does this PR introduce any public API change?
[ ] Does this PR introduce any binary protocol compatibility change?

Benchmark

cc @theweipeng @PragmaTwice @LiangliangSui


chore(cxx_indexer): remove redundant working_directory field

(This ensure that the working directory in the CompilationUnit is updated and used exclusively and will eliminate the warning message from cc_indexer_test of a single source file, rather than bundled compilation).


How to modify the log_level in cni config in configmap when using operator to install calico

{
      "name": "k8s-pod-network",
      "cniVersion": "0.3.1",
      "plugins": [
        {
          "type": "calico",
          "datastore_type": "kubernetes",
          "mtu": 0,
          "nodename_file_optional": false,
          "log_level": "Info",  this value
          "log_file_path": "/var/log/calico/cni/cni.log",
          "ipam": { "type": "calico-ipam", "assign_ipv4" : "true", "assign_ipv6" : "false"},
          "container_settings": {
              "allow_ip_forwarding": false
          },
          "policy": {
              "type": "k8s"
          },
          "kubernetes": {
              "k8s_api_root":"https://master.lyk8s.com:6443",
              "kubeconfig": "__KUBECONFIG_FILEPATH__"
          }
        },
        {
          "type": "bandwidth",
          "capabilities": {"bandwidth": true}
        }
      ]
    }

Current Behavior
Modify directly, and restore automatically
No modification method found from the document

Operating System and version:
Link to your project (optional):

Client Version:    v3.23.0
Git commit:        29de1d3f8
Cluster Version:   v3.23.0
Cluster Type:      typha,kdd,k8s,operator,bgp,kubeadm

There is currently no supported way to do this. It would probably be a good feature to add. I'm guessing we could do that by adding a field to the Installation resource under spec.CalicoNetwork and then use that when constructing the cni config.
If you just need to set it temporarily, like for debugging you could use the unsupported annotation and put that on the ConfigMap cni-config in the calico-system namespace and then you could change the config and the operator won't reset it. Just make sure you remove the annotation when you're done so the operator can update it if needed.

There is currently no supported way to do this. It would probably be a good feature to add. I'm guessing we could do that by adding a field to the Installation resource under spec.CalicoNetwork and then use that when constructing the cni config.
If you just need to set it temporarily, like for debugging you could use the unsupported annotation and put that on the ConfigMap cni-config in the calico-system namespace and then you could change the config and the operator won't reset it. Just make sure you remove the annotation when you're done so the operator can update it if needed.

I mainly use it for debugging, thank you very much.

And is it possible to change cni-config by tigera-operator?
For example, I want to change "host-local" ipam parameter to "calico-ipam"?
How can I do it?

And is it possible to change cni-config by tigera-operator? For example, I want to change "host-local" ipam parameter to "calico-ipam"? How can I do it?

I guess not, because tigera-operator only supports a few configurations

@ivanovpavel1983 for something like IPAM you can configure it through the Operator Installation resource. Though be aware the operator does not handle migration of IPAM data or pod restarts that would be needed with a change to that type of configuration.

The feature to configure calico cni logging properties (log_level, ... ) from operator is enabled now.
you can find the new feature implementation in this commit if interested.
The feature is currently available in master and will be in the v1.30.0 release.


Heterogeneous TreeDataGrid nodes

I want to use the TreeDataGrid in order to show a hierarchy with heterogeneous elements. All samples I have seen use nodes of the same type (e.g., Person).
For simplicity and because I cannot share confidential data about my use case, let's say my data structure is similar to a Team -> Players structure. The Team has a name and a city, while a Player has a name and age. (In my particular use case, there will be only one level of depth, like in this toy example)
I have implemented this using a fictitious TeamPlayerNode that mixes both classes Team and Player - this is a screenshot from the resulting UI, and it is roughly what I want to achieve:

My question:  Is there a cleaner way to implement this? I am concerned with the "cleanliness" of my code going forward with this hybrid TeamPlayerNode.

I have the same problem, TreeDataGrid doesn't seem to be able to do it currently


zsdl: fixup macos rpath

Fixes #252 but I'm not sure if there is a better way

This hardcodes the path to the dev env path. So it doesn't work.


When transition between routes page content is moving down to the page

When transition to the next page, the content of the page appears from bottom.
<RouteTransition
    pathname={this.props.location.pathname}
    atEnter={{ opacity: 0 }}
    atLeave={{ opacity: 0 }}
    atActive={{ opacity: 1 }}
>
    { this.props.children }
</RouteTransition>

This usually happens when the RouteTransition is not absolutely positioned.


drop coreapi dependency

(venv) sivan@titan:~/pylabber/pylabber$ ./manage.py generateschema
Traceback (most recent call last):
  File "./manage.py", line 15, in <module>
    execute_from_command_line(sys.argv)
  File "/home/sivan/pylabber/venv/lib/python3.6/site-packages/django/core/management/__init__.py", line 381, in execute_from_command_line
    utility.execute()
  File "/home/sivan/pylabber/venv/lib/python3.6/site-packages/django/core/management/__init__.py", line 375, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/sivan/pylabber/venv/lib/python3.6/site-packages/django/core/management/base.py", line 323, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/home/sivan/pylabber/venv/lib/python3.6/site-packages/django/core/management/base.py", line 364, in execute
    output = self.handle(*args, **options)
  File "/home/sivan/pylabber/venv/lib/python3.6/site-packages/rest_framework/management/commands/generateschema.py", line 20, in handle
    assert coreapi is not None, 'coreapi must be installed.'
AssertionError: coreapi must be installed.

The message indicates that in order to generate a schema using django's CLI coreapi need to be installed, because it is not installed, and therefore there is no dependency to drop.


Add missing targets in Compiler options

Fixes #1085
Added ES2018, ES2019 and ES2020
ScriptTarget enum in TypeScript repo: https://github.com/microsoft/TypeScript/blob/c15928ee38b34eab0a371bd50a15b3a3f04a3edc/src/compiler/types.ts#L4803-L4815

Yep - perfect, great call on switching to multi-line too. Thanks!


re.error: missing ), unterminated subpattern at position 21

Traceback (most recent call last):
File "structure_bgl.py", line 66, in
eventmap = match(BGL)
File "structure_bgl.py", line 41, in match
if re.match(r''+item,log_event) and re.match(r''+item,log_event).span()[1] == len(log_event):
File "/home/lepton00/opt/miniconda/lib/python3.7/re.py", line 173, in match
return _compile(pattern, flags).match(string)
File "/home/lepton00/opt/miniconda/lib/python3.7/re.py", line 286, in _compile
p = sre_compile.compile(pattern, flags)
File "/home/lepton00/opt/miniconda/lib/python3.7/sre_compile.py", line 764, in compile
p = sre_parse.parse(p, flags)
File "/home/lepton00/opt/miniconda/lib/python3.7/sre_parse.py", line 930, in parse
p = _parse_sub(source, pattern, flags & SRE_FLAG_VERBOSE, 0)
File "/home/lepton00/opt/miniconda/lib/python3.7/sre_parse.py", line 426, in _parse_sub
not nested and not items))
File "/home/lepton00/opt/miniconda/lib/python3.7/sre_parse.py", line 819, in _parse
source.tell() - start)
re.error: missing ), unterminated subpattern at position 21
Hi does anyone have this problem? Some answers I found on Stack Overflow suggest adding r before the regex string which the owner has already done it.
Edit: I used Drain in logpai as the parser.

*Hi i have the same problem !


Maybe a better way to fix decoding issues. Fix #51.

Use cchardet to detect the encoding before decoding.
Output from test.py:
[  1/250] 0007games.com passed in normalized mode.
[  2/250] 0007games.com passed in default mode.
[  3/250] 0031fashion.com passed in normalized mode.
[  4/250] 0031fashion.com passed in default mode.
[  5/250] 123vitamine.com passed in normalized mode.
[  6/250] 123vitamine.com passed in default mode.
[  7/250] 2x4.ru passed in normalized mode.
[  8/250] 2x4.ru passed in default mode.
[  9/250] 365calendars.com passed in normalized mode.
[ 10/250] 365calendars.com passed in default mode.
[ 11/250] 9v.lt passed in normalized mode.
[ 12/250] 9v.lt passed in default mode.
[ 13/250] about.museum passed in normalized mode.
[ 14/250] about.museum passed in default mode.
[ 15/250] abouttubes.com passed in normalized mode.
[ 16/250] abouttubes.com passed in default mode.
[ 17/250] actu.org.au passed in normalized mode.
[ 18/250] actu.org.au passed in default mode.
[ 19/250] alibaba.jp passed in normalized mode.
[ 20/250] alibaba.jp passed in default mode.
[ 21/250] alliancefrançaise.nu passed in normalized mode.
[ 22/250] alliancefrançaise.nu passed in default mode.
[ 23/250] anink.com passed in normalized mode.
[ 24/250] anink.com passed in default mode.
[ 25/250] anonne.ws passed in normalized mode.
[ 26/250] anonne.ws passed in default mode.
[ 27/250] anonnews.org passed in normalized mode.
[ 28/250] anonnews.org passed in default mode.
[ 29/250] aol.com passed in normalized mode.
[ 30/250] aol.com passed in default mode.
[ 31/250] aridns.net.au passed in normalized mode.
[ 32/250] aridns.net.au passed in default mode.
[ 33/250] arkeysolutions.com passed in normalized mode.
[ 34/250] arkeysolutions.com passed in default mode.
[ 35/250] asiahotel.co.th passed in normalized mode.
[ 36/250] asiahotel.co.th passed in default mode.
[ 37/250] atheme.org passed in normalized mode.
[ 38/250] atheme.org passed in default mode.
[ 39/250] australia.gov.au passed in normalized mode.
[ 40/250] australia.gov.au passed in default mode.
[ 41/250] b.ro passed in normalized mode.
[ 42/250] b.ro passed in default mode.
[ 43/250] baligems.co.uk passed in normalized mode.
[ 44/250] baligems.co.uk passed in default mode.
[ 45/250] bäckerei.de passed in normalized mode.
[ 46/250] bäckerei.de passed in default mode.
[ 47/250] bidtheatre.com passed in normalized mode.
[ 48/250] bidtheatre.com passed in default mode.
[ 49/250] blackburn.ac.uk passed in normalized mode.
[ 50/250] blackburn.ac.uk passed in default mode.
[ 51/250] bristol.ac.uk passed in normalized mode.
[ 52/250] bristol.ac.uk passed in default mode.
[ 53/250] bts.co.th passed in normalized mode.
[ 54/250] bts.co.th passed in default mode.
[ 55/250] byme.at passed in normalized mode.
[ 56/250] byme.at passed in default mode.
[ 57/250] communigal.net passed in normalized mode.
[ 58/250] communigal.net passed in default mode.
[ 59/250] cryto.net passed in normalized mode.
[ 60/250] cryto.net passed in default mode.
[ 61/250] daemonrage.net passed in normalized mode.
[ 62/250] daemonrage.net passed in default mode.
[ 63/250] davicom.com.tw passed in normalized mode.
[ 64/250] davicom.com.tw passed in default mode.
[ 65/250] defunctkernel.me passed in normalized mode.
[ 66/250] defunctkernel.me passed in default mode.
[ 67/250] direct.gov.uk passed in normalized mode.
[ 68/250] direct.gov.uk passed in default mode.
[ 69/250] dns4pro.com passed in normalized mode.
[ 70/250] dns4pro.com passed in default mode.
[ 71/250] donuts.co passed in normalized mode.
[ 72/250] donuts.co passed in default mode.
[ 73/250] drpciv.biz passed in normalized mode.
[ 74/250] drpciv.biz passed in default mode.
[ 75/250] edis.at passed in normalized mode.
[ 76/250] edis.at passed in default mode.
[ 77/250] engine.com passed in normalized mode.
[ 78/250] engine.com passed in default mode.
[ 79/250] evalsed.info passed in normalized mode.
[ 80/250] evalsed.info passed in default mode.
[ 81/250] example.com passed in normalized mode.
[ 82/250] example.com passed in default mode.
[ 83/250] expopack.com.mx passed in normalized mode.
[ 84/250] expopack.com.mx passed in default mode.
[ 85/250] f63.net passed in normalized mode.
[ 86/250] f63.net passed in default mode.
[ 87/250] formule1fo.com passed in normalized mode.
[ 88/250] formule1fo.com passed in default mode.
[ 89/250] foxiepa.ws passed in normalized mode.
[ 90/250] foxiepa.ws passed in default mode.
[ 91/250] geko.dk passed in normalized mode.
[ 92/250] geko.dk passed in default mode.
[ 93/250] get.moe passed in normalized mode.
[ 94/250] get.moe passed in default mode.
[ 95/250] globallatedeals.com passed in normalized mode.
[ 96/250] globallatedeals.com passed in default mode.
[ 97/250] globaltravelgroup.com passed in normalized mode.
[ 98/250] globaltravelgroup.com passed in default mode.
[ 99/250] google.cn passed in normalized mode.
[100/250] google.cn passed in default mode.
[101/250] google.co.jp passed in normalized mode.
[102/250] google.co.jp passed in default mode.
[103/250] google.co.th passed in normalized mode.
[104/250] google.co.th passed in default mode.
[105/250] google.co.uk passed in normalized mode.
[106/250] google.co.uk passed in default mode.
[107/250] google.com passed in normalized mode.
[108/250] google.com passed in default mode.
[109/250] google.com.tw passed in normalized mode.
[110/250] google.com.tw passed in default mode.
[111/250] google.it passed in normalized mode.
[112/250] google.it passed in default mode.
[113/250] hl3.eu passed in normalized mode.
[114/250] hl3.eu passed in default mode.
[115/250] hopjb.eu passed in normalized mode.
[116/250] hopjb.eu passed in default mode.
[117/250] huskeh.net passed in normalized mode.
[118/250] huskeh.net passed in default mode.
[119/250] hyves.nl passed in normalized mode.
[120/250] hyves.nl passed in default mode.
[121/250] imperial.ac.uk passed in normalized mode.
[122/250] imperial.ac.uk passed in default mode.
[123/250] ireland.ie passed in normalized mode.
[124/250] ireland.ie passed in default mode.
[125/250] ismtgoxdeadyet.com passed in normalized mode.
[126/250] ismtgoxdeadyet.com passed in default mode.
[127/250] jizzbo.com passed in normalized mode.
[128/250] jizzbo.com passed in default mode.
[129/250] keybase.io passed in normalized mode.
[130/250] keybase.io passed in default mode.
[131/250] linux.conf.au passed in normalized mode.
[132/250] linux.conf.au passed in default mode.
[133/250] lowendbox.com passed in normalized mode.
[134/250] lowendbox.com passed in default mode.
[135/250] lowendshare.com passed in normalized mode.
[136/250] lowendshare.com passed in default mode.
[137/250] luka-netconsult.com passed in normalized mode.
[138/250] luka-netconsult.com passed in default mode.
[139/250] microsoft.com passed in normalized mode.
[140/250] microsoft.com passed in default mode.
[141/250] mu.oz.au passed in normalized mode.
[142/250] mu.oz.au passed in default mode.
[143/250] nepasituation.com passed in normalized mode.
[144/250] nepasituation.com passed in default mode.
[145/250] nic.buzz passed in normalized mode.
[146/250] nic.buzz passed in default mode.
[147/250] nic.ir passed in normalized mode.
[148/250] nic.ir passed in default mode.
[149/250] nic.ps passed in normalized mode.
[150/250] nic.ps passed in default mode.
[151/250] nic.pw passed in normalized mode.
[152/250] nic.pw passed in default mode.
[153/250] nic.ru passed in normalized mode.
[154/250] nic.ru passed in default mode.
[155/250] nominet.org.uk passed in normalized mode.
[156/250] nominet.org.uk passed in default mode.
[157/250] nsa.gov passed in normalized mode.
[158/250] nsa.gov passed in default mode.
[159/250] nttpc.co.jp passed in normalized mode.
[160/250] nttpc.co.jp passed in default mode.
[161/250] nytimes.com passed in normalized mode.
[162/250] nytimes.com passed in default mode.
[163/250] oli.id.au passed in normalized mode.
[164/250] oli.id.au passed in default mode.
[165/250] ovh.fr passed in normalized mode.
[166/250] ovh.fr passed in default mode.
[167/250] pcmups.com.tw passed in normalized mode.
[168/250] pcmups.com.tw passed in default mode.
[169/250] pixelmania.asia passed in normalized mode.
[170/250] pixelmania.asia passed in default mode.
[171/250] porn.com.tw passed in normalized mode.
[172/250] porn.com.tw passed in default mode.
[173/250] prq.se passed in normalized mode.
[174/250] prq.se passed in default mode.
[175/250] quadranet.com passed in normalized mode.
[176/250] quadranet.com passed in default mode.
[177/250] realtek.com.tw passed in normalized mode.
[178/250] realtek.com.tw passed in default mode.
[179/250] redd.it passed in normalized mode.
[180/250] redd.it passed in default mode.
[181/250] ricoh.co.th passed in normalized mode.
[182/250] ricoh.co.th passed in default mode.
[183/250] rs.co.th passed in normalized mode.
[184/250] rs.co.th passed in default mode.
[185/250] servequake.com passed in normalized mode.
[186/250] servequake.com passed in default mode.
[187/250] siamparagon.co.th passed in normalized mode.
[188/250] siamparagon.co.th passed in default mode.
[189/250] simpardaz.com passed in normalized mode.
[190/250] simpardaz.com passed in default mode.
[191/250] sina.com.cn passed in normalized mode.
[192/250] sina.com.cn passed in default mode.
[193/250] singularity.fr passed in normalized mode.
[194/250] singularity.fr passed in default mode.
[195/250] starbucks.co.th passed in normalized mode.
[196/250] starbucks.co.th passed in default mode.
[197/250] swisscom.ch passed in normalized mode.
[198/250] swisscom.ch passed in default mode.
[199/250] sydney.edu.au passed in normalized mode.
[200/250] sydney.edu.au passed in default mode.
[201/250] tattitude.co.uk passed in normalized mode.
[202/250] tattitude.co.uk passed in default mode.
[203/250] test.de passed in normalized mode.
[204/250] test.de passed in default mode.
[205/250] textfiles.com passed in normalized mode.
[206/250] textfiles.com passed in default mode.
[207/250] theregister.com passed in normalized mode.
[208/250] theregister.com passed in default mode.
[209/250] tip.it passed in normalized mode.
[210/250] tip.it passed in default mode.
[211/250] toyota.co.th passed in normalized mode.
[212/250] toyota.co.th passed in default mode.
[213/250] twitter.com passed in normalized mode.
[214/250] twitter.com passed in default mode.
[215/250] ufpa.br passed in normalized mode.
[216/250] ufpa.br passed in default mode.
[217/250] unwire.hk passed in normalized mode.
[218/250] unwire.hk passed in default mode.
[219/250] urlte.am passed in normalized mode.
[220/250] urlte.am passed in default mode.
[221/250] via.com.tw passed in normalized mode.
[222/250] via.com.tw passed in default mode.
[223/250] vulnweb.com passed in normalized mode.
[224/250] vulnweb.com passed in default mode.
[225/250] wa.us passed in normalized mode.
[226/250] wa.us passed in default mode.
[227/250] warwick.ac.uk passed in normalized mode.
[228/250] warwick.ac.uk passed in default mode.
[229/250] whirlpool.net.au passed in normalized mode.
[230/250] whirlpool.net.au passed in default mode.
[231/250] whois.com passed in normalized mode.
[232/250] whois.com passed in default mode.
[233/250] whois.us passed in normalized mode.
[234/250] whois.us passed in default mode.
[235/250] whoiser.ir passed in normalized mode.
[236/250] whoiser.ir passed in default mode.
[237/250] winamp.com passed in normalized mode.
[238/250] winamp.com passed in default mode.
[239/250] wosoccer.com passed in normalized mode.
[240/250] wosoccer.com passed in default mode.
[241/250] x.it passed in normalized mode.
[242/250] x.it passed in default mode.
[243/250] xboxmoments.com passed in normalized mode.
[244/250] xboxmoments.com passed in default mode.
[245/250] yahoo.com.tw passed in normalized mode.
[246/250] yahoo.com.tw passed in default mode.
[247/250] yahoo.it passed in normalized mode.
[248/250] yahoo.it passed in default mode.
[249/250] zem.org.uk passed in normalized mode.
[250/250] zem.org.uk passed in default mode.
Timing in default mode: 15ms avg, 0ms min, 52ms max
Timing in normalized mode: 16ms avg, 0ms min, 60ms max
All tests passed!

I'm slightly wary about including an external dependency, given Python's somewhat erratic package management system. Would there be any way to replicate this functionality without using cchardet as a dependency?
Additionally, could you have a look at #97, and see whether your solution works for all usecases on all supported Python versions? I've created a canonical thread there with all the requirements and a list of issues.

Thanks for your attention and comment. I wll be back after some progress.

Any progress on this? Why is this closed?


Error when using 'Preview in OpenSCAD'

vscode settings:
{
    "scad-lsp.fmtExePath": "D:\\clang-format.exe",
    "scad-lsp.defaultParam": true,
    "scad-lsp.launchPath": "C:\\Users\\username\\scoop\\apps\\openscad\\2021.01\\openscad.exe",
    "scad-lsp.fmtStyle": "Microsoft",
}

When using 'Preview in OpenSCAD':
terminate called after throwing an instance of 'boost::filesystem::filesystem_error'
  what():  boost::filesystem::read_symlink: ��֧�ָ�����: "C:"

I wonder if it's scoop's problem and normal installed openscad to "C:\\Program Files\\OpenSCAD\\openscad.exe" and changed launchpath, still same error.
using vscode 1.73.0 on windows 10 19045
clang-format version 15.0.3

I don't have a Windows computer around right now, i will check it on weekends.

You can try to install to 'C:\Program Files\Openscad\openscad.exe', and delete the 'lanchpath'

You can try to install to 'C:\Program Files\Openscad\openscad.exe', and delete the 'lanchpath'

Still got same error. I'm using OpenSCAD 2021.01
Then I found OpenSCAD has Development Snapshots with 2022.11.07 and it works. So it's totally OpenSCAD's problem I guess.

OK，I‘ll close this issue, feel free to commit new one if have any problem.


⚠ DesiTorrents has degraded performance

In 3ad17ce, DesiTorrents ($DST) experienced degraded performance:

HTTP code: 200
Response time: 3348 ms

Resolved: DesiTorrents performance has improved in 2e1bc0e.


Fix evaluation status which never was set to Queued.

Description
Evaluations are initially Queued and only move to Started when the worker actually starts on the task.
Related Issues
Not linked yet.

@mmabrouk -- added refactoring based on comments.

QA tests

Adding a delay at the beginning of the evaluation.py and checking that the frontend shows Queued.
This could be automated in test_variant_evaluators_router.py with a lenghty evaluation and some polling with delays to fetch the evaluation status.


Cerebro doesn't load a host after inserting a failing host.

I inserted a host that didn't exist. After that cerebro doesn't load any host. I have to close the tab in my browser and reopen it again.
How to reproduce it:

Insert a host that doesn't exist
Try any other host that do exist.

Thank you

this should now be solved :)

Thank you!


4.x add deprecation warning to response.parse()

In Version 5 Http.get(url).parse throws ArgumentError "wrong number of arguments (given 0, expected 1)"
https://github.com/httprb/http/blob/53b7b91ac3012dbdfb43fa65947fd8d9c2311c7c/lib/http/response.rb#L159 introduced with #538.
I suggest adding a deprecation warning in Version 4.

That's a good idea.

https://github.com/httprb/http/commit/fa542332fd625a26480bb8677be16c66660bd507

Actually.... I am not sure I understand why this API was changed at all? This change makes the #parse method much less usable. If I know what the content type is, I do not need the HTTP gem to parse it for me - I can JSON.parse it myself.... I was assuming that the intention of the #parse method is to do "best effort" to parse it.
In addition, the response object already knows what the mime type is, so why not use it to parse?
The below code feels wrong:
require 'http'

url = "https://jsonplaceholder.typicode.com/posts/1"
response = HTTP.get url

# Why do we have to do this...
parsed_response = response.parse(response.mime_type)

# ...instead of this?
# parsed_response = response.parse

p parsed_response

To me, the beauty of the HTTP gem is that it "just works".
Can't the #parse method be changed to have its type argument optional?
def parse(type = nil)
  type ||= mime_type
  MimeType[type].decode to_s
end

What am I missing?

@DannyBen
Even if you pass Accept: application/json header, server might respond with plain text, or html, etc. So implicit mime type inferring from the response might lead into a confusing experience IMO:
HTTP.headers(:accept => "application/json").get("https://httpbin.org/html").parse
# HTTP::Error: Unknown MIME type: text/html

Let's say you define HTML decoder for that:
module HTTP
  module MimeType
    class HTML < Adapter
      def decode(str)
        ::Nokogiri::HTML str
      end
    end

    register_adapter "text/html", HTML
    register_alias   "text/html", :html
  end
end

And let's say you expect result to be a JSON, but it comes back as HTML (error or such):
payload = HTTP.headers(:accept => "application/json").get("https://httpbin.org/html").parse

Now you have a parsed response. But it's far from what you expect. So, to be honest I consider parse as a simple helper over JSON.parse(response.to_s), thus I think response.parse(:json) (which can be enhanced to pick up the most efficient adapter) is a better choice.

It's easy enough to revert this API change locally:
module HTTP
  class Response
    module ImplicitParse
      def parse(as = mime_type)
        super(as)
      end
    end

    prepend ImplicitParse
  end
end

You can also overload HTTP::Response#parse completely.

How about if we let parse() be automatic again, but require an allow list to be provided.
So only if the mimetype matches the expected one, automatic parsing takes place.
Http.get(url).parse(:json, :xml)

if response is json or xml it's parsed
if response type anything else it throws an error

IMHO on-topic would have been there: #538

IMHO on-topic would have been there: #538

A closed 2 years old topic? I doubt any reply there would have received any attention, but one thing is certain - discussing whether or not this is on topic, is by itself off topic... :)
Bottom line, your call of course how to handle this. If there is any weight to my opinion, I say either make it parse by mime type (and fail gracefully if it can't), or remove the parse method altogether. Any other route is awkward at best, and bad form at worst.

If you really liked parse() with no argument and want it back, I'd suggest opening a PR that reverts #538 and we can discuss it there.

I've reverted the deprecation warning in 4.x.x branch, further discussion of #parse future should go in #670


Feature/route permission api edit

PR #452 might replace this.

Closing this issue since #452 is ready for merging.


Add pytest release marker

What does this PR do?
Annotate a test with @pytest.mark.release and it only gets run with pytest integration-tests --release.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

Can you add the flag  --release ( it's when we run the CI on main, like where we push on dockerhub + azure &co)


Changed os2-atomic-proxy to be able to launch inside of both a vpc and classic. Changed STG to default to launching in a vpc, prod still launches in classic. Also cleaned up some cruft.

:+1:


Using content blocks anywhere

This is kind of a branch off of https://github.com/rustygeldmacher/jekyll-contentblocks/issues/11 But my issue is that I'd love to use a contentblock in my loop of posts on the homepage.
in my post I'd have something like:
{% contentfor otherstuff %}
* Some content
{% endcontentfor %}

and In my index.html something like:
  {% for post in site.posts %}
    {% contentblock otherstuff %}
  {% endfor %}

But I noticed this doesn't work and using  {% contentblock otherstuff %} only works on the post.html layout

Hi @thegregthomp -- I'm not sure this is possible, because in your example the post hasn't been rendered yet so there's no way for the loop to know what the content is for otherstuff.


请问出现这种问题要怎么解决？

Error response from daemon: could not find a network matching network mode ctfd_containers: network ctfd_containers not found
service "ctfd" is not running

docker-compose down一次再up


Upgraded beaker to 1.4.2-1-ge55c059

Updated beaker notebook version
With 1.4 the url doesn't have the v in the url anymore

Thank you for the contribution. It was merged directly as a2baad5fb58019939e8a116ada85333256ae591a to keep commit history cleaner. Your contribution is still credited to you.


Fix build with OCaml 4.06.0 (and -safe-string)

OCaml 4.06.0 uses -safe-string by default and so we have to distinguish between immutable string and mutable bytes.
This PR fixes the build for 4.06.0 and switches on -safe-string for all OCaml versions.

Cool, great PR. David, many thanks for fixing this, really appreciate it! 💯


object literal error

function id(s: number) { return s }
let y = { id }

yields error in type checker.

Solution for now is to outlaw shorthand property assignment.

Fixed by https://github.com/Microsoft/pxt/commit/cda535b4dd9d370eaa43388e2e36ecb52715893a


feat(babel): support babel 6.x.x

upgrade modules
upgrade .babelrc
fix tests
fix import / export
fix build script

:+1:

:+1:

You can use this git branch (https://github.com/neoziro/isparta/tree/babel-6-support-with-lib) as npm package if you want to use isparta with babel 6.
@iamstarkov I changed it because I think it's important to test the latest stable, I think you or @douglasduteil can modify the .travis.yml after merging this PR.

Please merge this

bump

@neoziro just a little correction:
npm install neoziro/isparta#babel-6-support-with-lib

That worked pretty well for me.

Alright let's go

woohoo \o/


Provide a JSON Schema

It would be great if you could provide an official JSON Schema for the engine json file.

would be good to have a spec defined for all Data Types indeed.
that way, after one generates the report, he can actually validate if his report is respecting the spec or not.
any update on this would be highly appreciated.
cc @ale7714 , @codeclimate-hermes

Creating a JSON format without a schema is a big issue these days.


Skyline: UI re-organization of BuildLibraryDlg to support integration with Carafe

This is a new design for the BuildLibraryDlg with more flexible panel layout that is easier to design using the WizardPages class, and which creates a new final page for Carafe that allows a user to specify transfer learning data sources.
I believe it still passes all of the existing tests and may be a useful starting point for Jason later as he integrates Carafe.

This work seems ready. It now provides comparable UI and passes all tests, and it would be necessary for adding either or both AlphaPeptDeep or/and Carafe which we know we want to do.


Error report after initially executing init.lua on starting nvim

Describe the bug
Error report after initially executing init.lua on starting nvim
To Reproduce

I was following along with this YouTube video: https://youtu.be/stqUbv-5u2s?si=M1PuANMGG9FPxtaW
I created a nvim directory in ~/.config
I copied the text in GitHub: init.lua
I pasted the contents in a new textfile called init.lua in the nvim directory
I saved and exited
I exited shell session
I started a new shell session
I ran: nvim
nvim executed init.lua
Plugins started installing
I got an error that some 'translation' file couldn't install
I got a shell error output, please refer pastebin link: https://pastebin.com/Hp6Ks7Ym
Any assistance would be appreciated

Desktop

Tablet: Samsung Galaxy Tab S9 Ultra (SM-X910)
OS: OneUI (v. 5.1.1: Android 13)
Terminal: Termux (v. 0.118.0)

Neovim Version
NVIM v0.9.4
Build type: Release
LuaJIT 2.1.0-beta3

The log you posted says:
[mason-lspconfig.nvim] failed to install lua_ls. Installation logs are available in :Mason and :MasonLog

it would be helpful to provide also the output of :MasonLog
Anyway, the issue is that lua_ls is not supported on Android.
This seems to be a duplicate of: https://github.com/nvim-lua/kickstart.nvim/issues/214
Also reported in mason repo: https://github.com/williamboman/mason.nvim/issues/306
According to the comments in the above tickets, you could either try to install lua_ls on android manually, or if you don't need it, you can disable it by commenting out or deleting it from the mason config, here:
https://github.com/nvim-lua/kickstart.nvim/blob/4d0dc8d4b1bd6b94e59f7773158149bb1b0ee5be/init.lua#L573
You can use --[[ --]] to comment out a block of code.

Thank you so much, that worked. The error output has gone quiet.

I closed this case as issue, resolved.


Update glob-parent dependency

Update glob-parent dependency
glob-parent before 6.0.1  is vulnerable to Regular Expression Denial of Service (ReDoS).
Versions (please complete the following information):

Chokidar version 3.5.3

Additional context
This issue gets caught by github actions package audit. chokidar currently uses glob-parent 5xx.
More here: https://snyk.io/vuln/npm:glob-parent

Why was this issue closed as soon as it was opened? We're having the same issue and npm audit fix is not able to find a fix for it

Because I saw this issue that directly references the problem: https://github.com/paulmillr/chokidar/issues/1191. So it seems like the problem does not actually exist and should be an issue with the build tool @ovideozn

I can reopen if you think that would be useful.


options[:version] is not set in Entity

From the docs:

The option keys :version and :collection are always defined.

I've tried exposing :version with a block and with a method, both are coming up "null". I've dumped the :env keys and it's not there.
Searching through this repository, I don't see anywhere that this would be set by default.
Am I just looking over that four leaf clover or is the above correct?

Does your API have a version? It comes from here in Grape.


Play video file from CDN does not work

Hi,
Is it possible to play video (previously decoded with ffmpeg) hosted in CDN with http endpoint?
Thanks,
Hanan

Yes, but you need to add a CORS header for the .ts file on the CDN, if you want to embed the video from another domain.
I.e. for all .ts files on the CDN, specify:
Access-Control-Allow-Origin: *
and JSMpeg will be able to load the video from the CDN.

Fixed.
Thanks !


feat: Add Intercom messenger to UI

To do:

[ ] Add VITE_INTERCOM_APP_ID  on vercel and fleek

Summary
Closes: #240
Refer to the documentation here
https://developers.intercom.com/installing-intercom/web/installation/
How to test

Go to any page on UI and you should see a widget in right side bottom, try reloading page and navigating to different pages and it should maintain its state

VITE_INTERCOM_APP_ID is added on Fleek, we dont use Vercel for SX

Looks good, can you just add more bottom margin on the right sidebar of proposal page so it's possible to see everything without overlap here:
http://localhost:8080/#/s:uniswapgovernance.eth/proposal/0xe7274e00eb2a084cdc3b7510a8b40aa303ac2d7944e9706ad090c974c76e71bf

It should be at least 88px. And this space should not be on small screen.

Will create a separate PR to fix overlaps


Upload Notifications API Demo

Description
The Notifications API allows web pages to control the display of system notifications to the end user.
Fixes #140
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
locally
Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@JustE3saR Could you please update this branch once with the latest code changes?


update readme file for generate devise model usage

env
rails version: <IP_ADDRESS>
ruby version: 2.7.4
I tried using the readme file init, but when I do rails generated devise MODEL.
I get this problem

so this PR is to update the README and help others not to have the same problem

oh... I got mixed up


add the ability to decrypt TLS private keys

added the ability to accept a passphrase and decrypt a TLS private key, by adding a 'Passphrase' to the Options struct and the functionality required to decrypt TLS keys.
This is needed to address https://github.com/docker/docker/issues/30935. A few rounds of reviews were done in https://github.com/docker/docker/pull/31364 on the code in this PR as part of stubbing out the overall implementation.
Signed-off-by: Arash Deshmeh<EMAIL_ADDRESS>

@cyli, @tiborvass

Thanks for working on this @adshmh!


CRC checking seems to have an issue

I am getting a significant number of CRC mismatches while testing a ds18b20 with an stm32f103.
I've tested with 4+ different ds18b20's, and also confirmed they work with an Arduino nano (not using this library).
I disabled the CRC checking, and dumped the scratchpad memory, and it appears that the data coming from the ds18b20 is correct, but the CRC check is failing. Reading the scratchpad multiple times always returns the exact same data.
Here is some sample data I captured of temp data read from the sensor.
Changes in scratchpad index 6 seems to cause a CRC mismatch. I'm still reading through the datasheets to understand how the CRC function works, it might be an easy fix.

Raw Temp Value
CRC match?
Scratchpad Data (Decimal)

355
true
[99, 1, 75, 70, 127, 255, 13, 16, 21]

355
false
[99, 1, 75, 70, 127, 255, 14, 16, 3]

353
true
[97, 1, 75, 70, 127, 255, 15, 16, 2]

353
false
[97, 1, 75, 70, 127, 255, 16, 16, 181]

351
true
[95, 1, 75, 70, 127, 255, 1, 16, 155]

351
false
[95, 1, 75, 70, 127, 255, 2, 16, 141]

The device id for these samples is [40, 255, 21, 32, 11, 0, 0, 87] (decimal)

I tested a bunch between an Arduino and my STM32F103. It actually looks like the first byte of the scratchpad is being occasionally read wrong by the STM32. The least significant bit is always 1 (it's always an odd number). For all of the rows where CRC match is false, if you set the LSB to 0, the CRC matches. I'm still not sure what is causing this though.

With the exact same wiring on the same device, when I use an Arduino library to interface with the ds18b20 it works fine, but when using this library, the LSB of scratchpad[0] is always 1 (causing the CRC check to fail ~50% of the time). So I'm fairly confident I've ruled out hardware issues, but this also isn't an issue with the CRC algorithm so I'll update the issue name.


Add files via upload

Added
"Universidad Colegio Mayor de Cundinamarca"
unicolmayor.edu.co

@JackSmith25 Pull request merged. Thank you!


Deployment

Deployment
User Story
I want the project cards, navigation and contacts to be structured on a a webpage
so that I can easily find and read the information.
Acceptance Criteria
GIVEN a portfolio using bootstrap
WHEN I visit the website using the URL
THEN I can access my website from any browser.

deployment succesful


Prevent zero pixel ratio in renderer

... so that on browsers where page is zoomed out (zoom level less than 100%), the canvas dimension is not zero on page refresh.
When the width and height of the canvas are zero on initialization, the loaded scene is black.

I signed it!


Does v2.0.1 provide Android 12 support.

According to latest updates "Starting in November 2022, app updates must target API level 31 or above and adjust for behavioral changes in Android 12"
When targeting to Android 12, getting build error:
Apps targeting Android 12 and higher are required to specify an explicit value for android:exported when the corresponding component has an intent filter defined. See https://developer.android.com/guide/topics/manifest/activity-element#exported for details.
These are the activity inside the manifest without exported tag
•   com.adobe.marketing.mobile.FullscreenMessageActivity"
•   com.adobe.marketing.mobile.LocalNotificationHandler"
react-native-acpcore- v2.0.1
https://developer.android.com/google/play/requirements/target-sdk#:~:text=Starting in August 2022%2C new,behavioral changes in Android 12

Hi,
Thanks for the report.  We are evaluating this.
Best Regards,
Calise

@e-pradeep Thanks for your patience.  We are planning to fix this in the next release.  Will keep you updated when it is ready.

@cacheung  Can you confirm when will be the next release date??

Hi @e-pradeep We plan to release at the beginning of August.

@e-pradeep We just released react-native-acpcore- v2.0.2, the andrdoid:exported attributes are updated in the extension's AndroidManifest file.  Thanks again for your report.

@cacheung I'm experiencing an issue, maybe related to this topic, sorry if it is unrelated. (If it ends up being unrelated I will open a new issue).
Working on upgrading react native to 0.68 from 0.67 and one of the upgrades is gradle from 4.x.x -> 7.x.x
Cannot build currently, seeing this error (cleaned for readability)
> Task :app:processDebugMainManifest FAILED
Execution optimizations have been disabled for task ':app:processDebugMainManifest' to ensure correctness due to the following reasons:

- Gradle detected a problem with the following location<EMAIL_ADDRESS>
Reason:
Task ':app:processDebugMainManifest' uses this output of task ':@adobe_react-native-acpanalytics:processDebugManifest' without declaring an explicit or implicit dependency.

This can lead to incorrect results being produced, depending on what order the tasks are executed.
Please refer to https://docs.gradle.org/7.3.3/userguide/validation_problems.html#implicit_dependency for more details about this problem.

Hi @archcorsair,  this is a different issue.
Can you open an issue in https://github.com/adobe/react-native-acpanalytics with the details,  we will take a look.
Thanks,
Calise

@archcorsair I looked into the issue you reported.  People saw this issue after upgrading to Grade 7.  It doesn't seem like the root cause is from react-native acpcore or acpanalytics.
This is a discussion I saw from Stackoverflow: link
Please let us know if you have any other question.
I am going to close this issue which is originally reported for AndroidManifest 'android:exported' attribute.
You can open a new issue for further discussion.


[docs]:  translate kitex-retry doc based on changes

zh doc changes: https://github.com/cloudwego/cloudwego.github.io/pull/310/files
Notes:

Do not change the original layout, code style or content sequence.
Software Translator is a tool, not god, do not use the result as the final result.
Translation principle: faithfulness, expressiveness and elegance.
Submit your PR here in this repo.

I am willing to work for this :)

@KevinChens  thanks


🍒 Cherry pick PR #13525 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/13525 to staging 🍒

🚀 Cherry-picked to staging by @OSBotify in version: 1.2.38-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @chiragsalian in version: 1.2.38-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


netmap: Support protocol changes related to NetworkInfo

Is it possible to update to the latest protobuf to the latest version as well, as soon as we recompile everything there?
https://github.com/protocolbuffers/protobuf/releases/tag/v3.18.0

Is it possible to update to the latest protobuf to the latest version as well, as soon as we recompile everything there? https://github.com/protocolbuffers/protobuf/releases/tag/v3.18.0

I'd merge https://github.com/nspcc-dev/neofs-api-go/pull/344 before.

Is it possible to update to the latest protobuf to the latest version as well, as soon as we recompile everything there? https://github.com/protocolbuffers/protobuf/releases/tag/v3.18.0

I'd merge #344 before.

Sorry, I meant to leave this comment in #344, not this PR.


Allow the use of callback in events

Closes none
Subject

To be able to use callback render must be called after dispatch
https://github.com/dompdf/dompdf/wiki/Usage#callbacks

Thanks @kl3sk

Your welcome.
Because I have a lot of troubles to use it. I can add a PR for the doc if you want.

Would appreciate that :)


(fix) Revert SocialNetwork: add pipeline for redis cluster - fix Docker build

@gavinlichn please take a look into your latest change, it breaks Docker build for social network

Need to build the thrift-microservice-social-deps image first.  Added funcion "get_shards_pool" to the redis++ API and included to the deps image.
Please refer to socialNetwork/docker/thrift-microservice-deps/cpp/Dockerfile

@gavinlichn - thanks for explanation!
Looks like public images were not updated for a long time:

@cdelimitrou - can we have them updated? Currently social network is not building without new deps image mentioned by @gavinlichn.

@cdelimitrou - we can close this PR or merge it and re-apply @gavinlichn changes once deps image is updated.

Hi, @cdelimitrou can you help to share the plan please?
Wil we update the  yg397/thrift-microservice-social-deps  deps image, and merge the patch again?
Or prefer to re-work the patch?


Readme File Update

The Creative Commons license of the design requires that you give appropriate credit to the author. Therefore, you must do it in the README of your project.

This task is assigned to Omachi


Add protocol buffer Stream Decoder

Allows users to ingest protocol buffers in realtime streams.
Pending -

Descriptor file is needed to parse the input. Mechanism needed to make this file available on all servers.
Support for schema registry

Codecov Report

Merging #8972 (96bb773) into master (e8f9d88) will decrease coverage by 2.30%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #8972      +/-   ##
============================================
- Coverage     68.56%   66.26%   -2.31%
- Complexity     4640     4695      +55
============================================
  Files          1741     1361     -380
  Lines         91475    68700   -22775
  Branches      13674    10746    -2928
============================================
- Hits          62724    45525   -17199
+ Misses        24363    19896    -4467
+ Partials       4388     3279    -1109

Flag
Coverage Δ

integration1
?

unittests1
66.26% <ø> (+0.03%)
:arrow_up:

unittests2
?

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

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

@xiangfu0 @navina can you please review this? We need to merge this before 0.11 release


Fixes Android not displaying non-scheduled notifications and Android notification size when title, subtitle or body not set

Fixes:

Android notifications not showing up when options.at not set: https://github.com/EddyVerbruggen/nativescript-local-notifications/issues/109
Android notifications always have the same size, even if no title, subtitle and body are set: https://github.com/EddyVerbruggen/nativescript-local-notifications/issues/110

@EddyVerbruggen Do not merge this yet, I need to verify if I also need to change NotificationRestoreReceiver#scheduleNotification.

Excellent work, thanks a lot!