Building for android "Could not resolve all dependencies for configuration ':app:_debugApk'"

Description
I recently started working on an existing React Native app implemented only in ios, and started the task of trying to build and develop it on Android as well. I followed all the instructions on the React Native docs, but keep getting this error and can't figure out why:
A problem occurred configuring project ':app'.
> Could not resolve all dependencies for configuration ':app:_debugApk'.
   > Configuration with name 'default' not found.

Based on research, I'm assuming that it has to be something with my settings.gradle file, but I can't figure out what. This is what that file looks like:
include ':react-native-firebase-analytics'
project(':react-native-firebase-analytics').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-firebase-analytics/android')
include ':tipsi-stripe'
project(':tipsi-stripe').projectDir = new File(rootProject.projectDir, '../node_modules/tipsi-stripe/android')
include ':react-native-sentry'
project(':react-native-sentry').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-sentry/android')
include ':react-native-webview-bridge'
project(':react-native-webview-bridge').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-webview-bridge/android')
include ':react-native-vector-icons'
project(':react-native-vector-icons').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-vector-icons/android')
include ':react-native-device-info'
project(':react-native-device-info').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-device-info/android')
include ':react-native-config'
project(':react-native-config').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-config/android')

include ':ReactAndroid'

project(':ReactAndroid').projectDir = new File(
    rootProject.projectDir, '../node_modules/react-native/ReactAndroid')

include ':app'

Does anyone know why this is happening and how to solve it? Thanks!
Reproduction Steps and Sample Code
react-native run-android
Solution
Why won't this build?
Additional Information

React Native version: 0.42.0
Platform: both
Development Operating System: MacOS

try cd android && ./gradlew assembleDebug --info will show the build progress detail.
hope it help u.
refer:  React-native debug apk

This issue looks like a question that would be best asked on StackOverflow.
StackOverflow is amazing for Q&A: it has a reputation system, voting, the ability to mark a question as answered. Because of the reputation system it is likely the community will see and answer your question there. This also helps us use the GitHub bug tracker for bugs only.
Will close this as this is really a question that should be asked on StackOverflow.

I tried yarn install again and it works !

@doraeminemon   thanks  for it works!!!

"react-native upgrade" solved my problem.

@LaPoPvLe Thank you so much

check your package.json file how many packages is adding into "dependencies" : {}, seem lost new dependencies you're just add. Because of some reason like syntax install or settings 👍

check your package.json file how many packages is adding into "dependencies" : {}, seem lost new dependencies you're just add. Because of some reason like syntax install or settings 👍


pattern.matcher returns false

pattern.matcher(parameterName).matches() line returns false when "][]Class][<>" is passed as parameterName. I am not sure if this is a code issue or testing issue. Please provide some guidance on how to test classloader manipulation issue.

Hi,
That parameter name should not be a problem since it cannot dereference the class property. In order to access the class property you need to use:

dot notation: xxxx.class
bracket notation: xxxx['class'] or xxxx.["class"]

Thats why the regular expression checks for the single or double quotes
Cheers,
A

Hi,
test/#$".class  as well as test&(){]['class'] also returns false. It would be great if you can provide me with couple of values that would return true.
Thanks,
Sohil

The regexp looks like:

We are trying to protect access to the class.classloader property, so values that return true are:
xxx.class.classloader
xxx['class'].classloader
xxx['class']['classloader']
...

Note: xxx is the name of one of the action public properties.
This regular expression is the one used by Struts2 paramterInterceptor to protect from classloader manipulation

we are using struts 1 in our application. I am using debug mode in eclipse and changing the value of the parameter as you suggested in above comment but still no luck.
Currently I am using below regex:
(..|^|.|[('|"))(c|C)lass(.|('|")]|[).,^dojo..,^struts..,^session..,^request..,^application..,^servlet(Request|Response)..,^parameters..,^action:.,^method:.
I have also tried using below regex but same issue:
(..|^|[('|"))(c|C)lass(.|('|")]|[).
Can you guide which is the correct regex?

private static final String DEFAULT_BLACKLIST_PATTERN = "(.\.|^|.|\[('|"))(c|C)lass(\.|('|")]|\[).*";


Update Content “2024-09-07-premier-jour”

Automatically generated by Decap CMS

👷 Deploy Preview for legendary-empanada-86b18b processing.

Name
Link

🔨 Latest commit
14d06f752cfc96d430cb7d2d9183fa38a709df02

🔍 Latest deploy log
https://app.netlify.com/sites/legendary-empanada-86b18b/deploys/66ddc0891fb588000853ab09


YCSB 0.7.0 with cassandra2-cql binding, Cassandra deployed using binaries

Added support for YCSB 0.7.0 with cassandra2-cql binding #872 #865
Instead of compiling Cassandra, using precompiled binaries from Apache
Modify /etc/hosts to include an entry with internal ip address of virtual machine.

Thank for the contribution!  Once you have the CLA signed we can review the code and complete the pull  request.  The CLA basically says the code you are contributing is yours to give.
@yuyantingzero - Looks like the move to 0.7.0 happened!  Can you handle this code review?  Also once it is reviewed and ready kick off the integration tests with "test this please" as the sole comment.

I signed it!

Since this PR updates both cassandra and ycsb. I will run a few tests after the code review, which might take some time.
cassandra-ycsb
hbase-ycsb
bigtable-ycsb
aerospike-ycsb
cloud-bigtable-ycsb
mongodb-ycsb
redis-ycsb

FYI - The cut-off date for next release is EOD Thursday 3/31.

I've fixed schema generation.

Looks good in general. I will kick off a few side-by-side runs.

@yuyantingzero any progress?

@kamsz runs finished. Looks good.
Could you sync up your branch? Also how about the vm host file issue? Have you made any progress?

@kamsz , any updates on this pull request, ready to check in :)?
YCSB version in PerfkitBenchmarker is 0.3.0, which is a bit outdated. If you are getting too busy, I can take over this PR and bump YCSB version to 0.9.0 by end of this month, if you do not mind.

@yuyantingzero go ahead, I don't mind :)

@yuyantingzero hey I see ycsb did get updated to 0.9.0. Is this ok to close?

@tedsta seems good to me.


duplicate lang item issue with no_std build

Problem
When building
https://github.com/encointer/encointer-worker/commit/7252e20264e63c7dce11470922d7c2965378842e
I get an error:
error: duplicate lang item in crate `sgx_tstd` (which `palette_support` depends on): `f32_runtime`.
  |
  = note: first defined in crate `std` (which `pallet_encointer_ceremonies` depends on).

error: duplicate lang item in crate `sgx_tstd` (which `palette_support` depends on): `f64_runtime`.
  |
  = note: first defined in crate `std` (which `pallet_encointer_ceremonies` depends on).

error: duplicate lang item in crate `sgx_tstd` (which `palette_support` depends on): `panic_impl`.
  |
  = note: first defined in crate `std` (which `pallet_encointer_ceremonies` depends on).

error: duplicate lang item in crate `sgx_tstd` (which `palette_support` depends on): `begin_panic`.
  |
  = note: first defined in crate `std` (which `pallet_encointer_ceremonies` depends on).

error: duplicate lang item in crate `sgx_tstd` (which `palette_support` depends on): `oom`.
  |
  = note: first defined in crate `std` (which `pallet_encointer_ceremonies` depends on).

error: aborting due to 5 previous errors

The strange thing is that it is caused by the last commit, introducing the dependency on
https://github.com/encointer/pallet-encointer-ceremonies
but that crate has no dependencies (and no new features) that were not present in the previous commit.
Steps
(you may need to install Intel SGX SDK first) or binary install
git clone https://github.com/encointer/encointer-worker
cd encointer worker
git checkout 7252e20264e63c7dce11470922d7c2965378842e
cd enclave
make

You should see the error above
Whereas the following should work
git checkout 5220a7560dde3cdd893aa913c73c561e0969e5f3
make

Notes
cargo 1.41.0-nightly (8280633db 2019-11-11)
cargo nono check:
cargo nono check warns about exactly the same crates in both commits
webpki: FAILURE
log: FAILURE
serde_json: FAILURE
sgx-externalities: FAILURE
env_logger: FAILURE
rustls: FAILURE
num-bigint: FAILURE
webpki-roots: FAILURE
substratee-stf: FAILURE
chrono: FAILURE
base64: FAILURE
yasna: FAILURE
serde: FAILURE
bit-vec: FAILURE
rust-base58: FAILURE

whereas none of them are dependencies of pallet-encointer-ceremonies.
However, serde is a dev-dependency, but I've already checked if I'm running into #5730 or one of the similar issues. I just removed all dev-dependencies in pallet-encointer-ceremonies.  I forgot to put #![no_std] in the new crate.

 I forgot to put #![no_std] in the new crate.


Make sure all layers from demo.opengeo.org/geoserver show up in mapproxy

There seems to be a problem with some SRIDs that are not properly proxied via mapproxy.

Fixed in upstream djmp.


fix block filter

remove blocked classes from addnew dropdown
lookup blocked classes in root and theme config

Hi Shea,
Can you please tag a release with this?
Cheers, Flo

Yep! https://github.com/sheadawson/silverstripe-blocks/tree/1.0.13

Thanks mate!


add contiv to networking.md

addresses #325

gentle ping @johndmulhausen and other maintainers...

timed out...
$ ping again

Ping @thockin
This simple 2-liner doc pr is awaiting tech review for more than a month


statediff: rename to consensuswarn

CC @odeke-em

Also the repository name has been changed to consensuswarn, thank you @elias-orijtech!


Using register_properties as the name of the method for register_with causes Godot to crash if the definition is later removed

The book suggests using a fn register_properties to register properties on classes.. With code like:
#[derive(NativeClass)]
#[inherit(gdnative::api::Node)]
#[register_with(Self::register_properties)]
pub struct MyNode {
    // ...
}

#[gdnative::methods]
impl MyNode {
    fn register_properties(builder: &ClassBuilder<MyNode>) {
        // ...
    }
}

If the definition of register_properties is removed, but the #[register_with(Self::register_properties)] annotation is left in place:
#[derive(NativeClass)]
#[inherit(gdnative::api::Node)]
#[register_with(Self::register_properties)]
pub struct MyNode {
    // ...
}

#[gdnative::methods]
impl MyNode {}

Neither the compiler nor linters raise a warning, because - as far as I can tell, this might be wrong - Self::register_properties now refers to the automatically derived register_properties from derive_native_class. The automatically derived register_properties calls the argument to register_with at its end, so in this case it goes into an infinite loop, overflows the stack, and crashes. This makes Godot crash immediately upon opening any project using this code.
Ironically, using the name register_properties for the function triggers (if enabled) the Clippy restriction same_name_method, but removing the definition - while leaving #[register_with(Self::register_properties)] in place - causes the warnings to go away.

Ideally, using register_with(Self::register_properties) should cause a compile error if register properties isn't actually defined by the user. Since that might be tricky, I would suggest at least renaming the function - either the one generated by godot-rust or just in the book examples - to something where it's unlikely the user will reuse the same name as godot-rust, triggering this issue.

Hello @slavfox , thank you for bringing this up! I was able to reproduce this.
@Bromeon I'm not as familiar with the proc macro creating the "register_properties" method?
I think the best option will be to update the documentation to use a different name for the time being. I assume that the "register_properties" is being defined separately. Another option could be to mangle the "register_properties" name that is being internally created.

Basically, the proc macro implements NativeClass for the user-defined type:
pub trait NativeClass: 'static {
    type Base: GodotObject;
    type UserData: UserData;

    fn init(_owner: TRef<'_, Self::Base, Shared>) -> Self;
    fn register_properties(_builder: &ClassBuilder<Self>);
    fn new_instance() -> Instance<Self, Unique>;
    fn emplace(self) -> Instance<Self, Unique>;
}

We can unfortunately not rename register_properties() here because it's currently part of the public API (see docs).
I do see a use case for MyRustClass::new_instance() and my_rust_class.emplace(), but I'm not sure if the init() and register_properties() methods should be directly called by the user, or if they should be #[doc(hidden)].
Going forward, I see two options:

Rename register_properties() and also init() (but keep deprecated compat methods until 0.11) to something unlikely to collide with the user, e.g. with a nativeclass_ prefix.
Change docs everywhere. This doesn't really fix the issue, as users might still name their method register_properties -- it's a pretty straightforward name for registering properties, and it's already being used in a lot of code.

I implemented (1) in #888, let me know what you think.

Since this doesn't really cause an issue unless the user makes a mistake, perhaps a warning in the documentation cautioning about reserved method names would also be sufficient?

Yes, for now that's definitely an option -- but it's still nice if there are no "magic traps" like the one you described. At least not ones that are very easy to trigger accidentally.
This reminds me of a very similar issue I had with https://github.com/godotengine/godot-cpp/issues/348, where I spent quite a bit of time debugging it.


Kabanero dashboard missing collection URLs

Describe the bug
As reported over Slack, the Kabanero dashboard lists a single .yaml URL to the collection hub. This is fine for Appsody, but Codewind and Che use the .json equivalent. As a result, posting that URL into the tools won't work.
Expected behavior
The user is given a collection hub URL that they can directly use in any of our tools.
Actual behavior
The only url shown is Appsody-specific.
Screenshots
Attached, note the single link.

Additional context
Short term the only solution is to list both URLs. We should also have an architecture discussion about whether we can just use the root URL in all cases or align on a single URL.

@kaczyns Do you know if the JSON URL for kabanero-index.json is included in the CRD? I only see the kabanero-index.yaml URL when running kubectl describe kabaneros
  Collections:
    Repositories:
      Activate Default Collections:  true
      Name:                          incubator
      URL:                           https://github.com/kabanero-io/collections/releases/download/v0.1.2/kabanero-index.yaml

Looks like the landing page can just replace .yaml with .json at the end of the URL to show both options on the landing page, but I wonder if we should have both URL's in the CRD since the landing page shouldn't really be altering the URL like that.

@alohr51 The user provides the URL in the Kind: Kabanero object.  We could provide a second place in the CRD for the JSON equivalent, but the user would need to type that in separately.  That seems like a burden for the user, who is probably just going to cut/paste the URL.  I could be convinced otherwise though as I don't really understand that part of the workflow that well (ie what does Champ do after he's finished curating the collections and now wants to publish them to his cluster)

@kaczyns to reduce the burden on the user to enter more information maybe codewind and che could be altered to accept either the .json or the .yaml.
I am not sure how much work that would be or whether it would just be easier to provide both the yaml and json URL's by bothering the user to enter both URL's in the Kabanero object.
@stephenkinder I would like to see if you have an opinion ^

@deboer-tim @alohr51 I think Codewind should support either format.

Codewind uses the same json format as Che and CodeReady Workspaces, so this is not something we can resolve in Codewind alone. We had a discussion with the Kabanero architects a couple months ago that we shouldn't create a new format for this if we didn't need to, and I don't know if there was a compelling reason why Appsody did. Either way, changing Appsody or working with Che & OpenShift to add a second format is something that isn't going to happen overnight and should probably be discussed on another architectural call.
In the meantime, what we currently have on the dashboard is wrong and does not match how Kabanero actually works. This needs to be treated like the defect it is, or we'll get more confused users and complaints. If we're going to start an architectural discussion I'd strongly suggest we start a new issue and fix this in the meantime.

@deboer-tim perhaps we can convince @seabaylea to alternatively support .json collections.  Might be fewer communities to impact.
In the meantime, @ianpartridge, we can solve this problem by 1) having the collection build, build both yaml and json formats; and 2) update the website to point at both.
We need to support customers that are using Appsody CLI standalone and those that are using Codewind.

The collection build already generates both formats, so I think the website just needs to point at both?

Correct, both are there. I was able to connect to the collection hub above from Codewind and create projects by replacing .yaml in the URL with .json.

Currently, the JSON URL isn't provided in the install of Kabanero (see https://github.com/kabanero-io/kabanero-landing/issues/5#issuecomment-534228246)  so the landing page doesn't have a sure-fire way to get the url.
We can either use the yaml URl and replace .yaml with .json (links look identical) and show both url's that way - this is assuming both URL's don't change in the future.
Or have the installer provide the JSON url during install and use the API to grab it like we do for the yaml URL already.

The landing page obtains the collection hub url from the "kabanero" resource. If there are additional urls that need to be exposed, they should be handled like the collection repository (via kabanero cr). Unless, the plan is to switch to one format for all.

I think it is pretty icky to have to tell the operator both index urls.  For 0.2.0 would it be all that awful to add two links to the website, one with a yaml extension for Appsody and one with a json extension for Codewind, and then work with the communities to simplify?  This seems to be the cheapest option until we can get to a better solution.

As long as we validate that the Codewind URL is ok before showing it, that should be ok. I would prefer the validation to be done on the operator level and the URL to be exposed in the CR so that users can figure that out without going to the UI (which will help with tests/automation). The operator can always default the Codewind URL if one is not explicitly passed in.

@jgawor I agree we should validate the Kabanero JSON is valid and works with Codewind.  For 0.2.0, I think manual validation is good enough for the short term until we have a real solution.
The collection schema would require both the YAML and JSON files to be present and I don't think it is that horrible for the website to assume it is a valid collection for the short term.

Going back several comments, there's another long-term solution that might be more palatable to everyone: standardizing on the filenames (or at least have a default) and supporting folder URLs. If each of the tools allowed you to enter a folder URL and automatically append index.[some-extension] when necessary, we could show one URL and users wouldn't need to know the difference.

Here's what I got so far, does anyone have any comments going forward?

@alohr51 Looks mah-va-luss.  @deboer-tim how do you want to attack the more significant problem?  It seems like for simplicity, we need both Codewind and Appsody to support the same string -- or teach Codewind and Appsody not to care the format -- .json and .yaml -- Kubernetes has formatting tools for both so that it wouldn't be that strange.

@stephenkinder we added the .json url (which will be the solution for this issue) I would suggest opening a new issue to talk about a long term solution.

@alohr51 @deboer-tim I created this issue in Appsody:  https://github.com/appsody/stacks/issues/394 and we can close this issue when Andrew's code merges.


bump to version 0.0.5

Codecov Report

:exclamation: No coverage uploaded for pull request base (dev@6f1d6e6). Click here to learn what that means.
The diff coverage is n/a.

@@          Coverage Diff           @@
##             dev      #21   +/-   ##
======================================
  Coverage       ?   80.00%
======================================
  Files          ?        6
  Lines          ?      270
  Branches       ?        0
======================================
  Hits           ?      216
  Misses         ?       54
  Partials       ?        0

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 6f1d6e6...cff4fa8. Read the comment docs.


Fix date.format option for digital-clock

While this does seem to work it wasn't really broken before for me. Was there a specific configuration the previous version didn't work with which this fixes?

Setting date.format to something other than shortDate, i.e longDate or isoDate won't do anything without this patch.


Raft/Node: stop ticker when node is shutdown

to avoid resource leak
ping @abronan @aaronlehmann
Signed-off-by: runshenzhu<EMAIL_ADDRESS>

LGTM

LGTM


fix($injector): add workaround for fat-arrow stringification in Chrome v50

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix.
What is the current behavior? (You can also link to an open issue here)
Chrome v50 throws when trying to annotate fat-arrow functions.
See #14487.
What is the new behavior (if this is a feature change)?
Fat-arrow functions get annotated correctly.
Does this PR introduce a breaking change?
No
Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#commit-message-format
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:
Everything works correctly on v52 without the fix. Not sure about v51.
I couldn't write a test, because the problem appears only when creating a fat-arrow function directly (not through eval()).
Closes #14487

BTW, a similar fix might help with #14240.

Add a note it's fixed in v52 and LGTM


[Typing] Mypy typing part 2

NOTE: There are many fields that are lazy initialized and assume these are accessed only after lazy initialization is done. I fixed them by using the solution suggested in this approach; https://stackoverflow.com/questions/60925137/using-mypy-with-with-lazy-initialization-of-instance-attributes
Handles some parts of https://github.com/vllm-project/vllm/issues/3680
mypy vllm/engine/*.py --follow-imports=skip --config-file pyproject.toml
mypy vllm/worker/*.py --follow-imports=skip --config-file pyproject.toml
mypy vllm/spec_decode/*.py --follow-imports=skip --config-file pyproject.toml
mypy vllm/model_executor/*.py --follow-imports=skip --config-file pyproject.toml

Remaining:
# mypy vllm/lora/*.py --follow-imports=skip --config-file pyproject.toml

And some other nested directories (such as model_executors/layer)

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

cc @simon-mo @zhuohan123 this PR is ready to review

@simon-mo I think we can merge it (I believe spec decoding failure is from the master)


fix(host-deployer): add retry on check is vgactived

这个 PR 实现什么功能/修复什么问题:
fix(host-deployer): add retry on check is vgactived

是否需要 backport 到之前的 release 分支:
release/3.6
/cc @zexi

Codecov Report

Merging #9790 (eb79cc7) into master (376dfa4) will increase coverage by 5.97%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master    #9790       +/-   ##
===========================================
+ Coverage    5.98%   11.96%    +5.97%
===========================================
  Files         848      231      -617
  Lines      131760    25756   -106004
===========================================
- Hits         7888     3082     -4806
+ Misses     123171    22363   -100808
+ Partials      701      311      -390

Flag
Coverage Δ

aFlag
11.96% <0.00%> (+5.97%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/hostman/diskutils/nbd/driver.go
0.74% <0.00%> (-0.01%)
:arrow_down:

pkg/hostman/diskutils/nbd/lvmutils.go
4.70% <0.00%> (-0.24%)
:arrow_down:

pkg/monitor/models/alertrecord.go

pkg/cloudcommon/db/list.go

pkg/compute/models/host_recycle.go

pkg/compute/models/groupjoints.go

pkg/multicloud/ucloud/client.go

pkg/util/seclib2/certfile.go

pkg/compute/models/guests.go

pkg/keystone/models/implied_role.go

... and 609 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 376dfa4...eb79cc7. Read the comment docs.

Codecov Report

Merging #9790 (eb79cc7) into master (376dfa4) will increase coverage by 5.97%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master    #9790       +/-   ##
===========================================
+ Coverage    5.98%   11.96%    +5.97%
===========================================
  Files         848      231      -617
  Lines      131760    25756   -106004
===========================================
- Hits         7888     3082     -4806
+ Misses     123171    22363   -100808
+ Partials      701      311      -390

Flag
Coverage Δ

aFlag
11.96% <0.00%> (+5.97%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/hostman/diskutils/nbd/driver.go
0.74% <0.00%> (-0.01%)
:arrow_down:

pkg/hostman/diskutils/nbd/lvmutils.go
4.70% <0.00%> (-0.24%)
:arrow_down:

pkg/monitor/models/alertrecord.go

pkg/cloudcommon/db/list.go

pkg/compute/models/host_recycle.go

pkg/compute/models/groupjoints.go

pkg/multicloud/ucloud/client.go

pkg/util/seclib2/certfile.go

pkg/compute/models/guests.go

pkg/keystone/models/implied_role.go

... and 609 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 376dfa4...eb79cc7. Read the comment docs.


Maintenance update

:tada: This PR is included in version 4.2.2 :tada:
The release is available on:

v4.2.2
GitHub release

Your semantic-release bot :package::rocket:


Refine error tips when rendering template

What type of PR is this?

/kind feature
What this PR does / why we need it:
Refine error tips when rendering template and log the error details.
Which issue(s) this PR fixes:
Fixes #
Special notes for reviewers:
Please check the following list before waiting reviewers:

[ ] Already committed the CRD files to the Helm Chart if you created some new CRDs
[ ] Already added the permission for the new API
[ ] Already added the RBAC markers for the new controllers

Does this PR introduce a user-facing change??

None

/cc @kubesphere/sig-devops

/approve


Implementation directory suggestion is wrong

The implementation section suggestions moving X to directory X/.meta which I don't think would fix root directory issues much...

PR inc


jsx-one-expression-per-line with CRLF on Windows

The rule always triggers, even when I have an expression per line, as the rule expects.

Then I realized that I am on Windows and the line break is done with a \r\n(CRLF). What I did was to change it to what Linux and OSX do and make the line breack as \n (LF). The plugin then works as expected.
Is it supposed to work only with that type of line break?

I wouldn't expect anyone on any platform, including Windows, to use anything but normal LF, since CRLF breaks so many things.

Well, Git expects you, by default (at least on Windows), to use CRLF. If you don't do it, it automatically changes it. Yes, I know you can change that and add a configuration. But why? VSCode does the same. He uses the Windows CRLF. Unless you want to change that in all your files in your project, I think the rule should detect this.

Yes, you should be changing that in all files in all projects, and also using git config to ensure that you convert everything to LF on clone/commit/etc.
(I also think the rule should detect it, however)

Yes, I am not defending that what I am doing is the right or wrong thing to do. Just saying that are other possibilities.
Will add tests ASAP and merge them.

I have the same problem on Windows yet my EOL is set to LF.

Yes, you should be changing that in all files in all projects

I'm running into this myself, so I thought I would chime in. The default and recommended setting when installing Git on Windows is core.autocrlf = true which coverts line endings to CRLF on Windows and commits them as LF.
Unless I'm missing something, in that configuration, you are indeed expected to have CRLF in your files when editing on Windows.

@dominic-p the option you want to select is the second one, imo.

@ljharb, that's understandable. I'm just pointing out that since it's the default and the Git recommendation, we're not talking about an edge case here. It might be worth supporting.
Thanks, btw, for all the work you do on this config. It's super helpful.

A PR that handles line endings regardless of LF or CLRF throughout the plugin would be appreciated.

I can take a stab at it, but I've never worked with eslint code before. Any pointers on where to start?

I'd start by writing the test cases :-)

@luissmg is there any chance this is fixed in the latest version of the plugin?

Closing, but will be happy to reopen if there's still something actionable.


KEDA should check whether scale target is already managed by another ScaledObject

Proposal
When a new ScaledObject is reconciled by Operator, it should be checked whether the scale target referenced by the new ScaledObject is being already managed by another ScaledObject, if so an error should be raised.
Use-Case
No response
Anything else?
No response

Nice idea!
What are you thinking about? Maybe an admission controller?

As a start error while creating the new ScaledObject is enough imho.

BTW, this is duplicated I think https://github.com/kedacore/keda/issues/3087

I think you can implement this using GateKeeper policy if you would like to

We prefer to manage it internally better than using a 3rd party software. Something like checking it during the HPA creation or an admission hook

I actually want to contribute but I think I prefer to contribute in areas that I am more familiar with 😄

makes sense totally, don't worry :)

I have been thinking about this, and I guess that an admission controller is the best option, because the person who deploys the SO could not be the person with access to KEDA logs. In companies with SRE teams managing the clusters, developers maybe don't have access to KEDA logs because it's in other namespace and they don't have access to check the logs. With an admission hook, we can give the feedback to those users during the deployment process.
IMO this component should be optional, WDYT? Maybe we can just add another component inside keda repo, or maybe in another split repo we can extend in the future with new features for this hook.
If we agree with the admission hook, I'm open to do a PoC to check it

+1
Admissions controller inside this repo is the best approach imho. We can then extend this with another checks (HPA targeting this).

Given we already have a discussion on https://github.com/kedacore/keda/issues/3087 and this is a duplicate, can we merge the conversation please?

As per the issues on https://github.com/kedacore/keda/issues?q=is%3Aopen+label%3Aprevention+sort%3Aupdated-desc, I believe that blocking creation is the only user-friendly way as error logs will not be consumed indeed.
The resource should be prevented from being created.
@JorTurFer I see you started working on this; would you mind making sure that the approach is extensible so that the other scenarios can be easily added in the future please?
As per https://github.com/kedacore/keda/issues/3087 we discussed to make it configurable and make it an opt-in. However, I think we can make it an opt-out instead as this would reduce the support cases given the nature of using multiple SOs and how of an anti-pattern that is.

The resource should be prevented from being created.

Exactly, the webhooks' server logs the error, but the k8s api blocks the resource creation with a clear message like:
the workload 'XXX' of type 'YYY/ZZZ' is already managed by the ScaledObject 'foo' or the workload 'XXX' of type 'YYY/ZZZ' is already managed by the hpa 'foo'

@JorTurFer I see you started working on this; would you mind making sure that the approach is extensible so that the other scenarios can be easily added in the future please?

I thought in this and that's why I split the admission hook in other (the 3rd) deployment instead of including it in the operator. We will have the webhooks' server and we can include all new validating (or mutating) webhooks in the future, just adding the new code and registering them in the cluster, but without any "major change" as now.

As per https://github.com/kedacore/keda/issues/3087 we discussed to make it configurable and make it an opt-in. However, I think we can make it an opt-out instead as this would reduce the support cases given the nature of using multiple SOs and how of an anti-pattern that is.

Using kubectl, I have to install the webhook because we use those manifest for e2e tests (and obviously, I'm covering it with e2e test), but in this case, I don't block the resource if the webhook isn't alive, only if it is ready and blocks the validation (which I think is correct, because there is an error indeed). My idea for helm chart is to make it configurable, deployed as default in the same way as the manifest, but supporting the restrictive mode (the webhook MUST validate the resource and k8s blocks it if the webhook isn't available) and also not installing the webhook (for multi-tenant scenarios where you can only have just 1 webhook for all the tenants due to k8s webhook api is a single endpoint registration at cluster scope)

Sounds good to me, thanks. For Helm, I would make it on by default though since it has it's own cycle and they can just turn it off when need be.

I have question regarding the implementation.
I looked at this feature on gatekeeper and I see that in gatekeeper you need to cache all of the hpas and scaledobject in the cluster and then you can filter only by namespace to check if there is HPA or scaledobject that points to your scaleTarget (cause both are namespaced object).
In real production environment the amount of hpas and scaledobjects can get up to thousands of objects.
How do you solve the slowliness in your implementation??

Would you mind moving this to the PR to keep the conversation close to that please?

Just to be sure; are we effectively checking this today or was it closed by accident? (I think it's fine but just checking)

It was closed automatically because the feature is fully implemented.
Feature + Docs + Helm chart changes are merged (helm changes merge closed this)

hi,
Is there a reason why it's not allowed to have 2 or more ScaledObjects that control one Deployment/etc?
We are using keda to scale our database servers and since one Scaled object can go either UP or DOWN, we have two, because we scale up on one condition and down on another. this is not optimal in terms of resource utilization ofc, but we cannot afford to kill user connects.
and after 2.10 it doesn't work anymore

sure, gladly
so we have database servers that (let's say) can handle 10 users each.  we have a ScaledObject that is triggered by Prometheus metric and if the average number of connections across servers is higher than the threshold - keda scales the servers up. if there is no connection left - another scaledObject scales them down.

Yeah, they are having the same number of target replicas. I forgot to
mention one thing -
The downscaling object has upscale disabled and vice versa.
On Wed, Jul 19, 2023, 7:36 PM Jorge Turrado Ferrero <
@.***> wrote:

I'm quite sure that both ScaledObjects requires the same replicas or you
should see the workload flapping between the required replicas for
ScaledObject A and the required replicas for ScaledObject B.
Could you share them? Maybe I'm missing something 🤔
—
Reply to this email directly, view it on GitHub
https://github.com/kedacore/keda/issues/3755#issuecomment-1642410409,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALNTT466VYL5F3HJJIF77H3XRAEKXANCNFSM6AAAAAARHCFPVM
.
You are receiving this because you commented.Message ID:
@.***>

You're right, I was missing something xD
But in any case, it won't work on kubernetes >= 1.26

In fact, Kubernetes (1.26) has added a check in HPA Controller that disables the autoscaling if the HPA detects multiple deployments (based on labels).

I don't think that we should remove the validation if the upstream doesn't roll back that feature

But if you specify a threshold and minReplicas you should be good.
If for example your threshold is 10 and you minReplicas is 4 then if the threshold is not met then it will scaleDown to minimum 4 replicas but if the threshold is met than it will upload more pods.
You can use same scaledObject for both upward and downward

The problem is that there is a chance that there is a connection on the 5th
server and scaling down to 4 will disconnect the user.
Sure, you shouldn't remove the check, especially cause k8s no longer
supports it. I'll work around the Prometheus query, maybe something can be
done there.
Thanks for your prompt reply :)
On Wed, Jul 19, 2023, 8:47 PM yuval weber @.***> wrote:

But if you specify a threshold and minReplicas you should be good.
If for example your threshold is 10 and you minReplicas is 4 then if the
threshold is not met then it will scaleDown to minimum 4 replicas but if
the threshold is met than it will upload more pods.
You can use same scaledObject for both upward and downward
—
Reply to this email directly, view it on GitHub
https://github.com/kedacore/keda/issues/3755#issuecomment-1642502728,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALNTT47VNHTMKUENRBPFLSDXRAMTBANCNFSM6AAAAAARHCFPVM
.
You are receiving this because you commented.Message ID:
@.***>

If the connections of the users are not super long then you can add a preStop hook to the pod which will maybe wait for termination or will terminate the connection gracefully.
If you want you can provide a bit more details regarding your use-case and we will try to help

Even if HPA would support it, I don't think having multiple autoscaling definitions tell 1 app to scale is a good idea.

I know but there are some things I would do differently.
For example it would be nice if you could configure different scaling policies for different metrics.
Cause maybe to cpu I want to bring 2 more pods at a time but for memory I want to bring 4 pods at a time

I know but there are some things I would do differently. For example it would be nice if you could configure different scaling policies for different metrics.
Cause maybe to cpu I want to bring 2 more pods at a time but for memory I want to bring 4 pods at a time

I think that here is where KEDA could try to extend the HPA for bringing that capabilities somehow and where KEDA can add value. I mean, KEDA is who is exposing the metrics to the HPA Controller and who manages the HPA, nothing blocks that KEDA modifies the HPA based on some conditions and modify the exposed values.
I mean, IMO that's a good idea and it's something that we can try to address in KEDA side, more than going against the upstream with multiple HPAs. In fact, didn't we created an issue for adding dynamic scaling rules @tomkerkhove ?

Yes - https://github.com/kedacore/keda/issues/2614


Mac OS X Process blocked

I experienced the weird issue that the process was kept alive when reading ps data. In a conclusion the library freezed.
-- Mac OS X 10.11.2
I could refactor the ProcessesUtils.java like this to get it running:
private static String readData(Process process) {
		StringBuilder commandOutput = new StringBuilder();
		BufferedReader processOutput = new BufferedReader(new InputStreamReader(process.getInputStream()));

		try {
			String line;
			while ((line = processOutput.readLine()) != null) {
				if (!line.isEmpty()) {
					commandOutput.append(line).append(CRLF);
				}
			}
		} catch (IOException ex) {
			Logger.getLogger(ProcessesUtils.class.getName()).log(Level.SEVERE, null, ex);
		} finally {
			try {
				if (processOutput != null) {
					processOutput.close();
				}
			} catch (IOException ioe) {
				Logger.getLogger(ProcessesUtils.class.getName()).log(Level.SEVERE, null, ioe);
			}
		}
		try {
			process.waitFor();
		}catch (InterruptedException ex){
			Logger.getLogger(ProcessesUtils.class.getName()).log(Level.SEVERE, null, ex);
		}
		return commandOutput.toString();
	}

Ok, thank you for your help. I do not have much time these days but I will certainly take it a look next week.
Since I do not have a Mac to test I will compare your proposed code in order to figure out what happened.
Thank you again!

Hi, is this still happening in the last version? (1.4)

List<ProcessInfo> processesList = JProcesses.getProcessList();
This code blocks in OS X.
Version 1.4

Hi,
Could you tell me what is the output in your system is you perform a:
ps -e -o pid,ruser,vsize,rss,%cpu,start,cputime,nice,ucomm

Could you also use this snapshot version of jprocesses and tell me what is the output before hanging?
https://oss.sonatype.org/content/repositories/snapshots/org/jprocesses/jProcesses/1.5-SNAPSHOT/jProcesses-1.5-20160517.173920-1.jar
Thank you

serge@mbp ~ $ ps -e -o pid,ruser,vsize,rss,%cpu,start,cputime,nice,ucomm
  PID RUSER                VSZ    RSS  %CPU STARTED      TIME NI UCOMM
    1 root             2538172  15996   0.0 Mon06AM   7:14.25  0 launchd
   39 root             2556660  23924   0.0 Mon06AM   1:45.49  0 UserEventAgent
   40 root             2520136   2316   0.0 Mon06AM   1:16.74  0 syslogd
   42 root             2555744  17812   0.0 Mon06AM   0:08.21  0 kextd
   43 root             2544932  10540   0.0 Mon06AM   2:00.52  0 fseventsd
   47 _appleevents     2544108   8144   0.0 Mon06AM   0:00.29  0 appleeventsd
   48 root             2546756   8184   0.0 Mon06AM   0:02.02  0 configd
   49 root             2546880   3812   0.0 Mon06AM   0:05.81  0 powerd
   54 root             2547680  13352   0.0 Mon06AM   0:04.84  0 airportd
   55 root             2519568  12324   0.0 Mon06AM   0:00.72 10 warmd
   57 root             2716920  68524   0.0 Mon06AM   5:19.77  0 mds
   61 _iconservices    2517712   5428   0.0 Mon06AM   0:00.05  0 iconservicesd
   62 root             2517712   5188   0.0 Mon06AM   0:00.02  0 iconservicesagen
   63 root             2541620   3756   0.0 Mon06AM   0:00.48  0 diskarbitrationd
   65 root             2554760  23308   0.0 Mon06AM   0:23.10  0 coreduetd
   66 root             2517628   8196   0.0 Mon06AM   0:00.03  0 wdhelper
   68 root             2584956  14812   0.0 Mon06AM   0:15.22  0 opendirectoryd
   69 root             2518060   6060   0.0 Mon06AM   0:05.36  0 wirelessproxd
   71 root             2547288  15896   0.0 Mon06AM   0:05.29  0 apsd
   72 root             2582400  95124   0.0 Mon06AM   1:52.18  0 launchservicesd
   74 root             2545936   7824   0.0 Mon06AM   0:13.48  0 securityd

With this new .jar such code:
    public static void main(String[] args) {
        System.out.println(1);
        List<ProcessInfo> processesList = JProcesses.getProcessList();
        System.out.println(2);
    }

output this:
1
May 17, 2016 10:01:56 PM org.jutils.jprocesses.util.ProcessesUtils readData
INFO: Ready to read command output...
May 17, 2016 10:01:56 PM org.jutils.jprocesses.util.ProcessesUtils readData
INFO: Command output successfully read.

Hi,
These are good news because the readData method (which blocked onepiers) seems to work.
I have created another snapshot jar with more logs in order to find the real blocking cause.
https://oss.sonatype.org/content/repositories/snapshots/org/jprocesses/jProcesses/1.5-SNAPSHOT/jProcesses-1.5-20160518.185707-2.jar
Could you also test this command: ??
ps -e -o pid,command

Thank you

This issue is open since 18/05 and I have no feedback.
I guess this problem is resolved already (maybe with the ProcessBuilder fix) so I close it.


[WB-7274] - Support raw ip addresses for launch

https://wandb.atlassian.net/browse/WB-7274
Description

Adds logging lines
adds support for raw ip addresses like local

Testing
How was this PR tested?
Release Notes
Below, please enter user-facing release notes as one or more bullet points.
If your change is not user-visible, write NO RELEASE NOTES instead, with no bullet points.
------------- BEGIN RELEASE NOTES ------------------
NO RELEASE NOTES
------------- END RELEASE NOTES --------------------

I'll tone the logging down a bit. This was extensive to minimize testing for a user who was trying it out.


Example fails to run

When running flutter packages get the following error is returned:
Because every version of flutter from sdk depends on meta 1.1.5 and every version of file_picker depends on meta ^1.1.6, flutter from sdk is incompatible with file_picker.
So, because basicapp depends on both file_picker any and flutter any from sdk, version solving failed.
pub get failed (1)

Any workaround?

Fixed. Meta needs to be at least at 1.1.5.


Search results 0

The script has worked fine until yesterday..
Now the script returns 0 results..
How to debug the script? Maybe Google has detected the scraping

or maybe google has changed the data structure?

Have tried the script from 3-4 different IP adresses and all returns 0
$positions = $page->getPositions();

same thing here.
No result

Ok, let me check very soon

Ok, google replaced  with  in its DOM. It should be ok for now. Please let me know it some errors still happen, I dont have much time to test now because of summertime.
Thanks

it's seems ok for me
Thanks

In my tests there are local listings in the natural listings now.
I tried to rewrite const NATURAL_QUERY to exclude li[@id = 'lclbox'] but I failed.
My test query: "cafe berlin"
-> natural listings on page 1 (15 are 5 too much):
1: http://www.berlin.de/cafes/
2: http://www.berlin.de/cafes/stadtteile/
3: http://www.berlin.de/bars/adressen/bar/cafe-commune-0d1ab041fbae2cc0a7f4503ae944708a.html
4: http://www.schwarzescafe-berlin.de/
5: https://plus.google.com/110132661049009362940/about?hl=de&socfid=web:lu:unknown:localgroupedresult&socpid=1
6: http://www.cafeeinstein.com/
7: http://www.einsteinudl.com/
8: http://www.literaturhaus-berlin.de/
9: http://www.oliv-cafe.de/
10: http://www.cafe-butter.de/
11: http://www.cremeguides.com/de/berlin-kategorien/food-in-berlin/cafes-kuchen-kaffee-in-berlin/
12: http://www.top10berlin.de/de/cat/essen-267/cafes-fuer-kaffeeliebhaber-567
13: http://www.yelp.de/c/berlin/cafe
14: http://www.visitberlin.de/de/erleben/essen-trinken/cafes
15: http://www.cafe-k.com/

Is there is already a solution?


500 Server internal error when using japanese voice "Microsoft Server Speech Text to Speech Voice (ja-JP, Haruka)"

I can see the voice in the GetVoices call:

But when I try to do tts using the voice "Microsoft Server Speech Text to Speech Voice (ja-JP, Haruka)", it just return a 500 error. and the backend server didn't print any useful log.

other languages/voices is fine.

Thank you very much for your issues. It seems that you have deeply used and solved some problems.
Regarding the 500 error that occurs with Japanese TTS, I modified the C# code to print out the error messages.

This shows an 80004005 error, which is a COM call error. It seems that during the Japanese TTS process, the engine calls other DLLs. Therefore, I used the Process Monitor tool on my machine to analyze the differences in DLL calls between Japanese and English TTS processes. By analyzing the logs, I found several suspicious DLLs.

I placed these suspicious DLLs into the Wine environment, and then the following error was reported:

This proves that the missing DLLs caused the issue, but I still haven't found a way to solve the "Bad EXE format" error. I will look into this further when I have time. Unfortunately, Wine itself is a very unstable environment, and the problems encountered may not be fully resolved.
I have attached two logs for you to download and view.
en-request.CSV
jp-request.CSV

Fortunately, I tested a version of the NaturalLanguage6.dll file. Copying it to the system32 folder resolves the issue. I have already uploaded the latest Docker image to Docker Hub, version v2, and the code has been updated accordingly.
You can pull it using:
docker pull xeden3/msspeechserver:v2


Dynamic icon components

Is it possible to dynamically use icons?
I'm using the vitesse template and would previously use something like <Icon :icon="myIconVariable" /> but now it seems like that no longer exists so I was wondering if it is still possible to do something like that.
I tried something like this without much luck
<template>
  <component :is="myIconVariable.replace(':', '-')" />
</template>

No, this plugin is designed to be used statically. Can you combine it with vite-plugin-purge-icons to have both SSR icons and runtime dynamic icons.

No, this plugin is designed to be used statically. Can you combine it with vite-plugin-purge-icons to have both SSR icons and runtime dynamic icons.

Oh okay, gotcha. I'll give that a shot, thanks!

Oh okay, gotcha. I'll give that a shot, thanks!

Oh okay, gotcha. I'll give that a shot, thanks!

Have you make it work with vite-plugin-purge-icons?

Due to the mechanism of how unplugin-vue-components works, there is no way to infer a component from a dynamic string. (unplugin-icons only resolved icons at build time)
The workarounds for this is:
<IconA v-if="cond1"/>
<IconB v-else-if="cond2"/>
<IconC v-else-if="cond3"/>
<IconD v-else />

Or

<template>

    <component :is="i.icon"/>

</template>

With autoImport one can achieve even more elegant solution. Use the same IconResolver you used in unplugin components:
  AutoImport({
      imports: ["vue"],
      resolvers: [
        IconsResolver({
          componentPrefix: "icon",
          enabledCollections: ["carbon", "mdi"],
        }),
      ],
      dts: "src/auto-imports.d.ts",
    }),

Then you don't need to even import and the above will look like this:

<template>

    <component :is="i.icon"/>

</template>

AutoImport

你这么写没用吧

Alternatively this component method can also be used to get the icons dynamically rendered using string. Is it advisable to use it? @antfu

@husayt it works!

Along with autoImport one can achieve even more elegant solution. Use the same IconResolver you used in unplugin components:
  AutoImport({
      imports: ["vue"],
      resolvers: [
        IconsResolver({
          componentPrefix: "icon",
          enabledCollections: ["carbon", "mdi"],
        }),
      ],
      dts: "src/auto-imports.d.ts",
    }),

Then you don't need to even import and the above will look like this:

<template>

    <component :is="i.icon"/>

</template>

Uncaught (in promise) ReferenceError: IconMdiAlarm is not defined

I don't think you need unplugin-icons for this at all. Here is what I did:
<template>
  <svg viewBox="0 0 24 24" class="icon" :class="{inline}" v-html="getIcon(`${style}-${name}`)"/>
</template>

And you only have to install the icon package (in my case it's @iconify-json/ic). Other icon packs may require different name formats, but the general idea is the same.

Just found that defineAsyncComponent also won't work. Had to use @iconify/vue and @iconify/utils to render icons dynamically. Hope there would be a way eventually.

https://github.com/iconify/iconify/tree/master/packages/vue

[...]
Uncaught (in promise) ReferenceError: IconMdiAlarm is not defined

Doing this on multiple sites at the moment without issues. May want to make sure you have explicitly enabled the collection via enabledCollections. Can't recall if the auto-import as non-component requires that or not, but I do have that in my config, and no issues.

Due to the mechanism of how unplugin-vue-components works, there is no way to infer a component from a dynamic string. (unplugin-icons only resolved icons at build time)
The workarounds for this is:
<IconA v-if="cond1"/>
<IconB v-else-if="cond2"/>
<IconC v-else-if="cond3"/>
<IconD v-else />

Or

<template>

    <component :is="i.icon"/>

</template>

How do I use it in vue jsx?

unocss Icons preset is welcome.

我项目中涉及到大量icon，如果每一个icon都要一个vue文件，再导入那就太恶心了

与 autoImport 一起可以实现更优雅的解决方案。使用与拔出组件相同的 IconResolver：
  AutoImport({
      imports: ["vue"],
      resolvers: [
        IconsResolver({
          componentPrefix: "icon",
          enabledCollections: ["carbon", "mdi"],
        }),
      ],
      dts: "src/auto-imports.d.ts",
    }),

然后你甚至不需要导入，上面的内容将如下所示：

<template>

    <component :is="i.icon"/>

</template>

谢谢，按照你的方法至少我不用再创建一个个vue文件了


Handle denominator equal to 0 while finding the row values for the pivot

While computing the row values (for the pivot selection) an error could be raised if the row value (the denominator in the division) was equal to 0.
The division should not happen because the pivot value should be greater than 0.
To fix the problem i catch the exception that occurs when the denominator is equal to 0, and append to the list of possible pivots the value 0 as a placeholder (Note : the value could be replaced with anything between ] -inf, 0]. The scope is only to make the lambda test to get the row_index fail).

Thanks !


Improve error messages for failed conversions

I was really excited when I found this project, because I work on a project that involves parsing data structures defined in an ASN.1 UPER file.
When I tried to run asn1rs -t rust ./src ./SAE_J2735_ASN.asn, I got the following backtrace with no useful information about what exactly caused the failure, at least not that I could discern.
Failed to convert SAE_J2735_ASN.asn, reason: Model(UnexpectedToken(stack backtrace:
   0: <asn1rs::model::Model<asn1rs::model::Asn>>::try_from::h36a0d20e67a03670 (0x56555557ce6e)
   1: asn1rs::converter::convert_to_rust::h8eb6f104679bc63e (0x56555556a584)
   2: asn1rs::main::he3fecb671a1b975a (0x56555559a7bd)
   3: std::rt::lang_start::{{closure}}::hc542518d32103aaa (0x565555569913)
   4: std::rt::lang_start_internal::{{closure}}::h8ad4264c6b68797c (0x565555628183)
             at src/libstd/rt.rs:49
      std::panicking::try::do_call::h7a0381557c6c2cee
             at src/libstd/panicking.rs:297
   5: __rust_maybe_catch_panic (0x56555562aafa)
             at src/libpanic_unwind/lib.rs:92
   6: std::panicking::try::h72cb0fef6e9c0ab1 (0x565555628cd6)
             at src/libstd/panicking.rs:276
      std::panic::catch_unwind::hbff071ae76e6f224
             at src/libstd/panic.rs:388
      std::rt::lang_start_internal::he0d8d06abc6f912f
             at src/libstd/rt.rs:48
   7: main (0x56555559aca2)
   8: __libc_start_main (0x7efbff50409b)
   9: _start (0x5655555611aa)
  10: <unknown> (0x0), Text("-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_-_")))

In the current version the location (line and character) as well as what character was expected is displayed :)


File search is not working and rg process eats up CPU

Issue Type: Bug
(cmd + P) to search a file is not Working. It just keeps on loading. rg process eats up all CPU usage always above 100% +. My vs code is newly installed and doesn't have any extensions in it.
VS Code version: Code 1.44.1 (a9f8623ec050e5f0b44cc8ce8204a1455884749f, 2020-04-11T01:47:00.296Z)
OS version: Darwin x64 19.4.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
8, 5, 4

Memory (System)
16.00GB (2.42GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

For me Cmd + P has changed. It now shows the definition of the code the caret is at.
vscode V1.44.1
System:

I changed nothing on my Mac and suddenly many keybinding have started misbehaving in vscode v1.44.1

Cmd + b for view explorer display toggle
Cmp + p for file search doesn't work
Option + up/down key used to move line vertically

I have upgraded Mac but the issue still exists

Apparently Cmd + P file search works but on a new file (Cmd + N) active tab.

@frazer-robert found a solution and it fixed my keybindings
https://github.com/microsoft/vscode/issues/94666
Look for @RMacfarlane 's solution

Most likely https://github.com/microsoft/vscode/issues/94928, do you have a folder in your workspace which comes from a filesystem provider extension? Should be fixed in Insiders.

Global text search is working but file search is not working. This process named rg makes my system very slow.

If this still happens in the new 1.44.2 release, could you check whether any tips here help? https://github.com/microsoft/vscode/wiki/Search-Issues#slow-search-rg-running-for-a-long-time-or-consuming-lots-of-cpumemory

Did all those changes. But still it's very slow.

It looks like there are a ton of files in your workspace for it to go through. It seems like we are just bound by the rate that we can read them all. You might try using the search.exclude or files.exclude setting to narrow it down.


Add info about GoDaddy to the domain auth docs page

GoDaddy has a specific domain authentication rule when adding CNAME records that seems to always trip people up. They automatically add domains at the end of records that are input, so customers that put in our SG domain auth records have them fail due to this problem.
Under step 5 here, add in this note:

GoDaddy automatically adds your domain to the end of any record you input.  Because SendGrid CNAME records are formatted as sendgrid.domain.com, GoDaddy interprets this as sendgrid.domain.com.domain.com - this fails CNAME validation.  To correct this, input the subdomain of the "host" value from SendGrid into GoDaddy - instead of sendgrid.domain.com, enter sendgrid.

This saved my life...thank you for pointing it out...


Microsoft search != modern SharePoint search

MIcrosoft search defaults to a disabled state in office 365 tenants today and requires admins to flip it on under services.  Without this enabled, SharePoint home uses the modern personalized graph based SharePoint search but not the new capabilities shown as a part of Microsoft Search.
The modern SharePoint search on sphome benefit from enabling Microsoft Search.
Once an admin turns on Microsoft search then the bookmarks, faqs, and locations from the Microsoft search admin center light up on SharePoint home.

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

ID: fb71f13f-b678-3cb0-58e3-4e798201aa7a
Version Independent ID: fe5470d4-8703-2ce3-7cf6-b53416f13199
Content: Get started with Microsoft Search in SharePoint Online
Content Source: MicrosoftSearch/get-started-search-in-sharepoint-online.md
Service: mssearch
GitHub Login: @tlarsen
Microsoft Alias: tlarsen

In Targeted Release, Microsoft Search is on by default for the apps that currently support Microsoft Search, including SharePoint.

Thank you for your feedback! We are in the process of rolling out Microsoft Search in Bing to all supported tenants at this time and it will be turned on by default. Previous for the preview version, you did need to explicitly turn this feature on.
Cheers,
-Wendy


action_erase_event erases event for both stick directions.

Godot version: 3.2.3
OS/device including version: Windows 10
Issue description: If you try to use Input.action_erase_event() for key rebinding purposes, the events for both stick directions will be deleted. (ex: left and right movement).
This happens because InputMap::_find_event() calls InputEventJoypadMotion::action_match() which returns true, even if not in the same direction:
bool match = (axis == jm->axis); // Matches even if not in the same direction, but returns a "not pressed" event.
Steps to reproduce: Code example:
		var event := InputEventJoypadMotion.new()
		event.axis = 0
		event.axis_value = -1.0
		# Snippit from my rebind logic:
		for action in InputMap.get_actions():
			if (action.find("ui_") == -1): # don't rebind default UI controls
				Input.action_release(action) # Make sure all actions are released when we unbind things so the character doesn't get stuck moving with an unbound key
				InputMap.action_erase_event(action, event)

Minimal reproduction project:
test_key_unbind.zip

I think I may have a fix:
void InputMap::action_erase_event(const StringName &p_action, const Ref<InputEvent> &p_event) {

	ERR_FAIL_COND_MSG(!input_map.has(p_action), "Request for nonexistent InputMap action '" + String(p_action) + "'.");

	bool pressed = false;
	List<Ref<InputEvent> >::Element *E = _find_event(input_map[p_action], p_event, &pressed);
	if (E && pressed)
		input_map[p_action].inputs.erase(E);
}

Just need to check the "pressed" status before erasing?

Also tweaked action_add_event for the same issue:
void InputMap::action_add_event(const StringName &p_action, const Ref<InputEvent> &p_event) {

	ERR_FAIL_COND_MSG(p_event.is_null(), "It's not a reference to a valid InputEvent object.");
	ERR_FAIL_COND_MSG(!input_map.has(p_action), "Request for nonexistent InputMap action '" + String(p_action) + "'.");
	bool pressed = false;
	if (_find_event(input_map[p_action], p_event, &pressed) && pressed)
		return; //already gots

	input_map[p_action].inputs.push_back(p_event);
}

But I'm noticing some strange behavior, now.  I think "pressed" is unreliable due to the deadzone checks, so sometimes things behave inconsistently in my tests.  I'm trying to avoid large changes, but it may require something more significant to fix properly than just checking pressed.

Finally ended up doing a less hacky, but more invasive approach and added a "match_direction" parameter to all the functions.  I'll try to do a pull request soon.


micro: port ops BATCH_TO_SPACE_ND and SPACE_TO_BATCH_ND from lite

@tensorflow/micro
Disclaimer: This is my first contribution to a bigger open source project, so please let me know if I'm doing something wrong or if I forget something - I highly appreciate your feedback.
In my project I use causal convolution, implemented with the Keras Layer Conv1D. After the conversion to the TFLite model, the convolution will be performed by the Conv2D op. This requires the ops BATCH_TO_SPACE_ND and SPACE_TO_BATCH_ND and since causal convolution has a huge field of use, I think that it makes sense to add these two ops to tflite micro as well. Since these ops are inverses of each other and (at least sometimes) used together, I thought it is appropriate to create only one issue to add both ops.
I already got a model running on my mcu using these ops and now I would like to share this with the other tensorflow users. I am not sure yet about all the required steps and which Pull Requests I need to make but I am sure that I will figure this out and I will link the pull requests to this issue and document it properly.
Steps

PR 1: refactor flatbuffer_conversions parsing function #45696
PR 2: refactor reference implementation from lite/kernels/internal/reference/reference_ops.h into its own header without making any changes. #45699
PR 3: copy the reference kernel from lite to micro and adjust it to micro by removing optimized ops. Add the File to the build. #45704
PR 4: Add tests for the kernels to be run with bazel test //tensorflow/lite/micro/kernels:all
Potential PR 5: Add to AllOpsResolver, make use easier. Can be discussed later.

Hi Stephan,
We are getting some quite high-priority requests from internal teams to port basic versions of these operators. In order to speed things along, I have created a TFLM float and int8 implementation for batch_to_space along with relevant tests in this PR. I hope this can be a good starting point for fully featured batch_to_space and for space_to_batch, and help serve as an example for porting TFLM tests.

Hi Stephan,
We are getting some quite high-priority requests from internal teams to port basic versions of these operators. In order to speed things along, I have created a TFLM float and int8 implementation for batch_to_space along with relevant tests in this PR. I hope this can be a good starting point for fully featured batch_to_space and for space_to_batch, and help serve as an example for porting TFLM tests.

Hi Stephan,
We are getting some quite high-priority requests from internal teams to port basic versions of these operators. In order to speed things along, I have created a TFLM float and int8 implementation for batch_to_space along with relevant tests in this PR. I hope this can be a good starting point for fully featured batch_to_space and for space_to_batch, and help serve as an example for porting TFLM tests.

Hi Nat,
Thanks for your PR! Looks really good to me. I think I could continue working on this next week, this week I won't have any time I guess. If it's so time critical, feel free to do it, if not, I will do it next week :)

Hi Stephan,
We are getting some quite high-priority requests from internal teams to port basic versions of these operators. In order to speed things along, I have created a TFLM float and int8 implementation for batch_to_space along with relevant tests in this PR. I hope this can be a good starting point for fully featured batch_to_space and for space_to_batch, and help serve as an example for porting TFLM tests.

Hi Nat,
Thanks for your PR! Looks really good to me. I think I could continue working on this next week, this week I won't have any time I guess. If it's so time critical, feel free to do it, if not, I will do it next week :)

Sounds good. I think with the time pressure, I'll upload a similar version for space_to_batch_nd so that our internal teams can go ahead with their work. Hopefully these versions can serve as a starting point, and if you have additional features or especially additional tests we can work together to land those.

Sounds good. I think with the time pressure, I'll upload a similar version for space_to_batch_nd so that our internal teams can go ahead with their work. Hopefully these versions can serve as a starting point, and if you have additional features or especially additional tests we can work together to land those.


Create issues' page.

The issues' page can be accessed in the nav bar item "issues"after the project is chosen or after the project is clicked.
The issues are separed in columns where their information is displayed in cards. The user can:

create an issue and give all the information in the model/Issue.java;
save an issue created in the database, in issues table.
can edit the issue by 2 clicks on it;
move the card to other columns with drag-and-drop.
The columns are the group in which the issue is assigned. Can be what the user is doing now, what have been done, etc.
The priority is shown in the issue's card.

The task is divided in back-end and front-end. The commits of the back-end CRUD operations is in issue #29 and the front-end is here.


0.7 compatibility updates (WIP)

This is a work and progress and I haven't been able to build stuff yet. I'm also not sure if I have the syntax and format right for the ffi javascript files.
I thought I'd put it up as a PR as soon as possible so no work was duplicated.

update javascript imports to use external ffi code
https://github.com/purescript/purescript/wiki/0.7-Migration-Guide#user-content-no-more-inline-ffi-code

update Array syntax
https://github.com/purescript/purescript/wiki/0.7-Migration-Guide#no-more-array-sugar

I have added Purescript 0.7 support in my fork and opened an issue report here: #31 (see details there). It seems we have already duplicated some work but it should be easy to merge/compare our two approaches.

Closed because #34 has it.


Fix #11 - Many host-side tests are failing + other changes

All test passing now, except test/facets_packs.cpp, which I expect to fix soon.

This still leaves one failing host-side code test: run-test-facets_pack-header-only. But we can look into that separately.


using initContainer to conduct image pull and import to PVC instead of CDI

Is your feature request related to a problem? Please describe:
We are using kubevirt and cdi to create VMs, but recently we hit some schedule issues which caused by the virt-launcher temporary pod as described at https://github.com/kubevirt/kubevirt/issues/6531 and  https://github.com/kubevirt/kubevirt/issues/8487.
As @awels and @rmohr discussed at this link: https://github.com/kubevirt/kubevirt/issues/6531 , just starts the virt-launcher pod normally, and waiting the image to be pulled and imported to the PV is a potential solution, but this solution requires the CSI driver to support the PV can be access by multiple pods on the same node in RWO mode. The CSI driver can meet the requirement in our product environments, but my concern is that in this situation although the cdi importer pod only consume a few mount of resources, in an extreme situations, it is possible that the importer pod can't be running due to no resources left on the PV node.
I am wondering if it's possible to add an initContainer into the virt-launcher pod, then the initContainer conduct image pull and import to PVC instead of CDI importer pod ?
Could you help to provide some comments?  Correct me pls if I am wrong, thanks a lot.
Describe the solution you'd like:
Add an initContainer to the virt-launcher pod, then using the initContainer to conduct image pull and import to PVC instead of CDI importer pod.
Describe alternatives you've considered:
Additional context:

I think using init containers in this way would introduce a deep coupling between kubevirt and cdi that we've consciously been trying to avoid.  CDI doesn't know anything about VMs.  And how would virt-controller know how to construct the init container?
As mentioned in the referenced issues, we hope to remove/minimize the use of the "temp pod" by leveraging volume populators internally by the datavolume controller or externally by users creating PVCs with dataSourceRefs directly.


packaging setup

added basic packaging setup (pep-517 compliant)
added draft readme file (edit this as you wish)
@erensahin @cemkaragozlu @kori73

thanks!


How to run this

Hello,
I am new to this type of development. Though I am really interested in your project I have everything cloned and I think everything is installed for OpenFL and HaxelFlixel. I was just wondering how I run the project. Just a single command I assume thanks for your help! BTW saw the video and love it!

Hello, I am new to this type of development. Though I am really interested in your project I have everything cloned and I think everything is installed for OpenFL and HaxelFlixel. I was just wondering how I run the project. Just a single command I assume thanks for your help! BTW saw the video and love it!

Hi! I'll be including build instructions in the README.md soon. I haven't gotten around to it yet because the method of creating the timers for the PresentationState is made with a slightly modified version of the FlxTimer class in HaxeFlixel
Might I ask what operating system you're running? I might just be able to provide some one-stop instructions here if you're running a Linux distribution or macOS.
Also, I'm glad you liked the video! It's a very indev build of it and it uses OBS for chroma keying, so it may change eventually in the future.


Grid hAlign doesn't appear to have an effect

Cell stuff works but so far can't get Grid's flexCells, hAlign, align to have any effect when set.
This is in Chrome on a windows 8.1 box.

flexCells only have an effect when the cell elements inside the grid have different heights.
As to the hAlign: you will only be able to horizontally center cells inside the grid if they all have widths.
E.g. this modified snippet taken from the example folder will horizontally center the containing cells:
        <Grid hAlign='center'>
          <Cell size='4/12'>

              horizontally centered.

          </Cell>
          <Cell size='4/12'>

              horizontally centered.

          </Cell>
        </Grid>

I did try this with a single cell size='6/12' with no effect

I'm having a hard time debugging this, since the code in the example folder is working as intended. Besides that we use flexr in production and haven't had any issues so far.
I would to help out though, do you have any code that I can review?
You could also try manually centering with css to rule out that you don't have any interferring styles. The containing element should have.
justify-content: center;
display: flex;

and then the centered item inside needs to have a width smaller than the container.

I'll close this for now. If there's any new in this case, I'll consider reopening the issue.

@chriskjaer Supplying a size for every cell does indeed fix the alignment issue of hAlign not applying, but for some reason anytime i give a size to every cell (5 cells at 1/5 size) the fifth cell wraps to the next row.

Ignore me, obviously if having 1/5 * 5 sized cells would be a full width container and hAlign wouldn't be necessary.


Investigate using Arquillian Cube to run Docker domain integration tests

We could tidy up some of the Docker domain testing by using Arquillian Cube. It's still in Alpha but seemed to work pretty well when I had a play around with it back in February.
Potentially it may also solve #175.

I've got an Arquillian Cube POC committed here https://github.com/jamesnetherton/wildfly-camel/commit/95c271ce88895644a7c6f40ff3f64cea0eb3adc6.
It works, but it's held back by the jboss/wildfly docker image inability to handle a graceful shutdown. Hence the tests finish running but the build hangs because it cannot shut down the Docker containers properly. I have proposed a PR to fix this https://github.com/jboss-dockerfiles/wildfly/pull/16.
The thought that came to mind when building this was - should we introduce yet another framework to manage Docker? Probably not since we already have our own API and the Docker Maven plugin.....

Going to close this as there's not much value in adding another Docker framework into the project.
My POC will remain on my fork should we ever need a baseline reference for Arquillian Cube.

Time for another look at this. It'll help with #1359, #1329 & #1327.

Related to https://github.com/tdiesler/openshift-playground/issues/30


Fix small NPE on Android

Fixes a small NPE on Android, which happens because Security.getProperty call returns null there.

Thank you for this!
For some reason the CI didn't run so no way to test the regression, will try get that going.

Seems to be concurrency related error. Not sure what's causing it.
I didn't test locally, because some tests fail on my machine.
No problem if this cannot be merged. Just thought someone may want to run it on Android in future.

There may be something wrong with the CI scripts not sure why it stopped running automatically or why it is failing all the open PRs. Will need to make time and come through this...


Previous S2 Shadow Plugin draft

Before you submit this PR: make sure to put all operations-related information in a wiki-note, a PR should be about code and is publicly accessible
What does the code in this PR do / what does it improve?
This plug in could compute the previous s2 shadow variable, which could be used to reduce the isolated S1 and S2 events.
Can you briefly describe how it works?
we define the shadow parameter as [previous s2 area/ time difference].
To calculate this, we loop over the previous 20 events (about 3 seconds given the event rate now)
and take the maximum as the shadow value.
Can you give a minimal working example (or illustrate with a figure)?
here is an example of how to use the variable
https://github.com/chnlkx/XENONnT/blob/main/Iso_S1_new.ipynb
Please include the following if applicable:

[ ] Update the docstring(s)
[ ] Update the documentation
[ ] Tests to check the (new) code is working as desired.
[ ] Does it solve one of the open issues on github?

Notes on testing

Until the automated tests pass, please mark the PR as a draft.
On the XENONnT fork we test with database access, on private forks there is no database access for security considerations.

All italic comments can be removed from this template.

Duplicate of https://github.com/XENONnT/straxen/pull/664, not sure how this can happen.


Font rendering issue; characters appear at wrong size, position

Environment
Windows build number: 10.0.18363.0
Windows Terminal version (if applicable): 0.8.10091.0

Font: JetBrains Mono

Steps to reproduce
Sometimes font doesn't render properly: some characters are too tall or too short. Sometimes it's numbers, sometimes it's capital case letters. Regular conhost window or ConEmu don't have this problem.
Expected behavior
Font characters render properly.
ConEmu example. Everything seems to render properly.

Actual behavior
Some characters are too tall or too short. You can see it in capital case letters.

I have the same issue for Japanese characters.
I created a gist to reproduce the problem https://gist.github.com/hnakamur/2a123ce7bdc0d7e5070218c258f990a8
I also added some screenshots there.
You can run the following command in a Ubuntu (Windows Subsystem for Linux) shell.
curl -sS https://gist.githubusercontent.com/hnakamur/2a123ce7bdc0d7e5070218c258f990a8/raw/b3cd604b6f8b0781cd51758afa3e1dd264e7c6d7/swap-myth.txt

I know @miniksa was mucking with the baselines in #5345, maybe this is related?

Looks like it was fixed. I do use a 4K display with 150% scaling. It might be that #5345 fixed my issue.

@tuscen that's excellent to hear. I'd usually love to have an actual commit to link this to for future people that run into this issue, but we've changed so much about our rendering stack since 0.8 that it'd be hard to pin this down exactly.
Thanks for following up though!

Sorry for late reporting, but my issue was also resolved since version: 0.10.781.0.
Thanks for a great product!


Fixed ship workflow YAML

#696  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @CatChen and the rest of your teammates on  Graphite


[Feature request] Provide profile config parameter for hugetlbfs mountpoint

Distribution: Gentoo
Distribution version:
The output of "lxc info" or if that fails:

Kernel version: 5.6.15-x86_64
LXC version: 4.0.2-r2
LXD version: 4.0.1
Storage backend in use: lvm

Gentoo doesn't populate folder /dev/hugepages for hugetlbfs, instead user
generates any mountpoint and loads it through fstab, for example, /mnt/hugepages.
As devtmpfs is going to be populated on boot, there would be need to find
out if there is udev rules to populate it, or any other userspace tool that
can populate it for you. It will ease quite a lot if it could be just passed in
the profile as a string, and let lxd consume it.

[ x ] Container log (lxc info NAME --show-log)

$ lxc info --show-log vm-test
Name: vm-test
Location: none
Remote: unix://
Architecture: x86_64
Created: 2020/06/27 19:41 UTC
Status: Stopped
Type: virtual-machine
Profiles: vm-1
Log:
qemu-system-x86_64: can't open backing store /dev/hugepages/ for guest RAM: Is a directory

[ ] Container configuration (lxc config show NAME --expanded)

$ lxc config show vm-test --expanded
architecture: x86_64
config:
boot.autostart: "0"
image.architecture: amd64
image.description: ubuntu 18.04 LTS amd64 (release) (20200626)
image.label: release
image.os: ubuntu
image.release: bionic
image.serial: "20200626"
image.type: disk1.img
image.version: "18.04"
limits.cpu: "4"
limits.memory: 16GB
limits.memory.hugepages: "true"
volatile.base_image: 238450c866b2a735cb6e134b25094897b58565123d8f5d6725f3273dcc468b3a
volatile.eth0.hwaddr: 00:16:3e:1a:7c:64
volatile.eth1.hwaddr: 00:16:3e:29:8f:c3
volatile.eth2.hwaddr: 00:16:3e:54:4b:40
volatile.last_state.power: STOPPED
volatile.vm.uuid: e109c7a6-01d0-4775-bd52-aa532013d8ee
devices:
eth0:
name: eth0
nictype: bridged
parent: br0
type: nic
eth1:
name: eth1
nictype: bridged
parent: virbr1
type: nic
eth2:
name: eth2
nictype: bridged
parent: virbr2
type: nic
root:
path: /
pool: LXDPool
size: 50GB
type: disk
ephemeral: false
profiles:

vm-1
stateful: false
description: ""

Added logic to locate a hugetlbfs instance. This will work so long as you either only have a single hugetlbfs instance or have multiple ones with one of those being at /dev/hugepages.

Excellent, works out of the box! Thanks Stéphane!


Set max protocol version for dtls renegotiation tests to dtls1.2

There's no functional change today but it reflects that renegotiation is not supported in dtls 1.3. The changes follows the same pattern applied to the tls renegotiation tests.
Checklist

[x] tests are added or updated

This pull request is ready to merge

Merged to the master branch. Thank you for your contribution.


How do I change the port?

What if I have a server running on port 8000? How do I change to another port?

If you server is already running and you start the Printer Application, it will start on port 8001. Only if yout server starts after the Printer Application, it will not be able to start or has to take an alternative port.
At least some Printer Applications can be started with an alternative port:
ps-printer-app -o server-port=10000 server

What we need is some way to set the default port of the Printer Application by a configuration file.


Documentation for the self host deployment

Hey team - Love the project, and have been trying it out the last week.
I can not seem to find any of the documentation regarding how to try  the self host option. Is anyone able to point me in the correct direct?
Cheers.

Hi @josh-at-straker,
The self-hosted UI is only available on our commercial plans. If you are interested, would love to setup a call and discuss! Feel free to reach out on Slack or let me know how to contact you.


Voice mail / recording feature to desktop app

Feature request:
It would be very nice to have the voice mail recording feature straight in the app in all platforms like Windows, Mac, Linux and iOS. It is possible now only in the Adroid app. All format should be also supported on all platforms eg. sound should be mp3 and videos should be mp4 etc. Or the play support for the different file types should be incluced in all platforms.

Thanks @kaksikanaa,
Just read your note again, and realized I had misunderstood. mp3 audio and mp4 video files can be uploaded to the desktop app, and played directly within the app. But you're looking for a voice record directly in the app.
Would you like to contribute this in the feature idea forum so it can be discussed, upvoted and considered for a ticket accepting pull requests?
Please include a link back to this GitHub Issue. If you're interested in implementing, please say so and we'll prioritize the review.
You get 10 votes in the feature idea forum, and each one influences the future of the project.

I'll close the issue here, but you're welcome to contributing the idea to the feature idea forum as noted above. https://mattermost.uservoice.com/forums/306457-general/
Thanks @kaksikanaa again for the feedback!


Test: pinned tabs

Refs: https://github.com/microsoft/vscode/issues/12622

[x] anyOS @jrieken
[ ] anyOS @joaomoreno

Complexity: 4
Authors: @bpasero
Create Issue

Pinned tabs work when tabs are enabled through the following interactions:

from the context menu of a tab ("Pin" / "Unpin")
via keybinding
via global action from the command palette targeting the active editor

Verify

tabs can be pinned and unpinned in each editor group
pinned tabs always appear before unpinned tabs

opening a new tab always opens it after the last pinned tab

pinned tabs remain visible even when many tabs are opened and scrollbars appear

non-pinned tabs scroll "under" pinned tabs
pinned tabs eventually scroll as normal tabs once you make the size of an editor group small enough (120px is the limit)
the active tab is always fully revealed in the presence of pinned tabs

pinned tabs show with a small fixed size

showing only the icon if icons are enabled
showing the first letter of the file name otherwise

pinned tabs can only be closed explicitly (e.g. via CtrlCmd+W) and are exempted from mass close actions (like "Close Others")

enabling workbench.editor.limit.enabled will never close a pinned tab when the limit is reached

pinned tabs work together with any of the tab related settings

workbench.editor.tabSizing set to shrink and fit
workbench.editor.tabCloseButton set to left, right or off (note that pinned tabs never show any close button)
workbench.editor.highlightModifiedTabs enabled to show a dirty indication for dirty pinned tabs
workbench.editor.openPositioning set to left, right, first or last does never mix pinned editors with non-pinned

pinned tabs preserve their state

saving an untitled editor that is pinned shows the result as pinned too
switching an editor for displaying a resource (markdown preview vs markdown text) should preserve pinned state
the command to reopen a closed tab restores a pinned tab at the right index and preserves pinned state

verify drag and drop

moving pinned tabs preserves pinned state if dropped onto other pinned tabs
moving pinned tabs onto a non-pinned tab unpins it
moving unpinned tabs onto pinned tab pins it
this works in the same group and across editor groups or windows

pin an editor that we do not restore after restart (e.g. a SCM diff editor) and verify that after restart the number of pinned vs unpinned editors is not broken
disable tabs when you have pinned tabs and ensure that they drop their pinned status (e.g. now mass close commands work on them)

I noticed that the pinned tabs (without file icons enabled) do show the first letter of the file name - but it is always capitalized.  I think it would help to distinguish pinned tabs if the first letter of the filename follows the case of the actual file.

Damnnn @jrieken leave some issues for the rest of us 🔥

hehe, waiting for dentist...

Cool stuff! 👏


contrib,test: bump cri-tools

Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>
- What I did
bump cri-tools and adapt our code to it
- How I did it
vim
- How to verify it
CI
- Description for the changelog

/test all

/test all

@mrunalp this is ready and picks up latest changes in crictl and new tests for critest

LGTM


How do you import this in your project ?

I am trying my best to import this in my project but because there is no exported member the import will not work

Having problems here too...

@marteiro  To use this in an ionic app:
Install the plugin.
Than inside your app after your imports but before the export class ....  just do a:
declare const Tesseract: any;
Just access the methods with Tesseract.recognise()

Thanks for the reply,
I've installed the plugin, but I'm getting "library not found for -lTesseractOCRiOS" when compiling from XCode. Works fine form command line... trying to figure it out.


Namespaces are removed in 1.18.2

Hi,
We recently upgraded from 1.18.1 to 1.18.2 and namespaces in xml are removed during parse.
Document doc = Jsoup.parse(new ByteArrayInputStream("<?xml version=\"1.0\" encoding=\"UTF-8\"?><test1><test xmlns:p1=\"test-namespace\"> </test></test1>".getBytes("UTF-8")),
        "UTF-8", "", Parser.xmlParser());
    doc.outputSettings().charset("UTF-8");
    doc.outputSettings().escapeMode(Entities.EscapeMode.xhtml);
    System.out.println(doc.toString());

The result was
<?xml version="1.0" encoding="UTF-8"?><test1><test xmlns:p1="test-namespace"> </test></test1>

in 1.18.1
But with 1.18.2:
<?xml version="1.0" encoding="UTF-8"?><test1><test> </test></test1>

Is there any configuration to keep the namespaces?

Thanks for reporting, and sorry about the issue! This is the same as #2235, hitting the digit case.


Sign in with Apple and Google One Tap

I am looking to implement Sing in with Apple and Google one Tap in react native - native implementation, not web based implementation. I am wondering if Hasura auth would work with that.
Here are the docs:
Apple: https://developer.apple.com/documentation/sign_in_with_apple/sign_in_with_apple_rest_api
Google:  https://developers.google.com/identity/one-tap/android/idtoken-auth
We plan on using React Native libraries which implements client-side portion in our app.

We need this... https://firebase.google.com/docs/auth/flutter/custom-auth
The ability to use 3rd party authentication and generate an Nhost token using Admin SDK.


Implement "create voting rationale" dialog

As a User, I what to type /vote, so the discord bot sends me a message containing an URL for the new voting creation.
Similar to https://github.com/comrade-coop/apocryph-dao-bot/issues/16, but returning a different URL

Todo

in DiscordProxyHostedService - add /vote bot command support; Discord should return the URL from appsettings.json
Dao:CreateVoteProposalAddressUrlTemplate

Note: as this is a single command, there is no need to dive into perper flow;


copy some tools into the venv bin when the wheel is installed

@superlopuh which friends exactly? i.e., other than -opt and -translate...

mlir-cpu-runner would be great

Thank you very much for adding this!

np. if you realize you need another one lmk (or just send a PR since now you know "how the sausage is made")

(will merge after everything passes...)


Extract variables names to placeholders in .po and .pot files

Example:
t`Hello ${ username}

For now this will be extracted to
msgid "Hello ${ 0 }"

This behaves so because we are missing variable names information while resolving translations at the runtime. (details described here).
But what if we will transform our translations object (json with translations) in form of Hello ${ username } to Hello ${ 0 } in the runtime. This operation will be not so hard to perform (traverse tree and clone). And with this solution we will have the best of 2 worlds:

The translator will see the variables names (provides more context for the translator).
Translations will be working without transpile because we will transform our translations obj to the format that works now.
Babel plugin has information about variable names also.

@MrOrz @alxpy

As for me, it sounds good!
Additionally we can use my old PR (https://github.com/c-3po-org/babel-plugin-c-3po/pull/54)

:+1: LGTM!

Already done by @alxpy. Greate job. This feature is available in plugin and lib in 0.4.0 version.


Attempt to merge in upstream before running Cirrus tests

Description
Before this change, Cirrus ran tests on feature branches as is, ignoring upstream changes since the branch was forked off of master (unless the author manually merged in master). This leads to situations where pre-submit tests could pass but post-submit would fail:
D     Merge feature branch Bar into master
| \
|  C  Create implementation of Foo called Bar, with unit tests
|  |
B  |  Upstream change to API of interface Foo
| /
A     Create interface Foo

In the above example, with the current setup, opening a PR with commit C branching off of commit A will pass fine on CI, even though upstream the interface of Foo changed and once merging the branch to master, CI will fail post-submit.
This change attempts to merge upstream (on Cirrus the remote is named origin, however) into the current branch if it is pre-submit for a PR. If there is a merge conflict, the script logs out the merge conflict, aborts the merge, and continues to run the suite. In this situation, the author will be blocked from landing their PR until resolving the merge conflict anyway, after which CI will run again.
Here is a test case with an intentional merge conflict introduced that shows that the test suite still continues: https://cirrus-ci.com/task/5383806748196864?command=setup#L344.
Examples

PR that passed pre-submit but failed post-submit.
https://github.com/flutter/flutter/pull/56694
https://github.com/flutter/flutter/pull/59196

Related Issues
fixes https://github.com/flutter/flutter/issues/45626
Tests
This will improve existing tests by ensuring they are run against the latest upstream.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I signed the [CLA].
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read [Handling breaking changes].

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

(PR triage): Are there still plans for this PR?

(PR triage): Are there still plans for this PR?

Yes, I have it on my TODO list.

Closing since most CI has migrated to LUCI


Process images

Fixes #287

Upgrade media-manager to 0.1.0
Implement image processors
Allow users to get ORIGINAL, PREVIEW, THUMBNAIL variants

Doc: https://g0v.hackmd.io/@mrorz/cofacts-media-manager#Extended-goal-supporting-signed-URLs-and-preview-URLs
PREVIEW: fixed-width downsized image for website detail page, webp
THUMBNAIL: fixed-height downsized image for chatbot & website list page, jpg
ORIGINAL: Only for logged-in website users

All media urls are signed URL with 1-day duration
Add integration test with GCS & media-manager

Coverage decreased (-0.006%) to 87.538% when pulling 1d7e79ed6f781e423daf056dd64caf7be14cd53a on process-image into f5e6992fbe65dda7214eeaeaf05f025f2d440b8e on master.