ICU (Unicode) support?

Is there any particular reason you ship a binary without Unicode (ICU) support?
You ship with the full-text search extension, which makes sense - but how is full-text search any use without Unicode support? Literally all the text on the internet is in Unicode format.
I can understand shipping SQLite without Unicode support for things like internal databases in apps, or the system registry in your computer or phone - but (I would assume) a primary reason people are interested in SQLite in Deno, is to build microservices, or smaller/simpler web apps, and I can't imagine what those services or apps would even be doing that doesn't require Unicode support?
Maybe this is a cultural divide of some sort? But I swear, I have researched this topic everywhere, and the only explanation I can find is, the ICU extension is "big" and makes some things "slow" - and I can understand this position from SQLite itself, being designed and optimized for use as an embedded database.
But shipping this as a Deno package is very different from the SQLite default use-case, I think? Are we more concerned about being "small" and "fast", or being actually useful?
I understand it's possible to build the extension from source and load it myself, but I'm just a humble web developer, and Unicode support seems very much like something basically everyone would need?
I can't understand how people are building anything with SQLite without Unicode?
Or are they all going out on their own to solve this problem first? am I just dumb or lazy? 😅

To the best of my understanding, SQLite should at least support storing Unicode UTF-8 strings?
So I attempted to run a simple query SELECT upper('abc æøå') just to see what happens, and:

I also tried running PRAGMA encoding='UTF-8' first, same result.
I then swapped to npm:libsql, which runs the query, but of course return ABC æøå, since there's no ICU support there either.
I found this article, which sums up 5 ways to handle Unicode, all of them slow or impractical in various ways. So is custom collations and bad performance what people put up with in JS? Or does everyone build their own binaries from source?

I was able to reproduce your issue with a straight sql string - db.prepare(select * from table where somerow = '🦄');
However, if it is parameterized, there is no problem. db.prepare("select * from table where somerow = ?").run("🦄");
So this means it is practically not a big deal.
sqlite's CLI does allow unicode literals, though, so I suppose it should be supported.

Can confirm, it works with parameter binding. 👍

sqlite's CLI does allow unicode literals, though, so I suppose it should be supported.

I think most clients allow it? I guess queries containing unicode literals might be rare - I just did it for testing. 🙂
I guess I've kind of mixed two different issues into one though.
Really the main thing that baffles me is why no client ships with something as basic and essential as Unicode support?
I know ICU is "big" and makes things "slower". I also understand there are some issues with OS kernel differences, leading to practical subtleties around DB file portability. Yet, the internet runs on Unicode - everything right down to something as trivial as JSON requires Unicode.
So I just spent another half day on this, and finally came up with a tolerable solution - this relies on a lot of fragile, user-supported tooling though.
Do you think it would make sense to bundle some loadable extensions like nalgeon/text with the package? (I read that the official icu extension is "big", but this one is just ~64KB.)
Alternatively, would it make sense to integrate sqlpkg somehow?
Perhaps providing a function like e.g. getExtension("nalgeon/text") which would download and install sqlpkg locally in the project, run sqlpkg init and sqlpkg install nalgeon/text, and then return a Promise<string> with the resolved path, e.g. .sqlpkg/nalgeon/text/text.so.
Alternatively some maybe some way to literally import from e.g. @sqlpkg/nalgeon/text to get an installed package and path? Obviously that's way outside the scope of this project, I'm just airing the idea in case you find this approach preferable to including a loadable extension.
I'm just wondering how we can save the next person from a day or two of adventuring through a long list of half solutions and bad ideas before discovering how to do something as basic as just searching and sorting by Unicode strings. 😅


[Bug] fix flink-start didn't load plugins/${pluginName}/lib/*

Purpose of this pull request
[Bug] fix flink-start didn't load plugins/${pluginName}/lib/*
issue :
https://github.com/apache/incubator-seatunnel/issues/2551
Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[x] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

close https://github.com/apache/incubator-seatunnel/issues/2551

it will error for file which use  Paths.endswith("jar")

@TyrantLucifer Please review it again

LGTM


Add extract for T2w FLAIR MRIs

Add extract function for T2w FLAIR MRIs.

It looks like these modification are already included in PR #403


Add support for float_to_int_unchecked

Resolves #3629
Keeping it as draft till I add more tests.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Please don't forget to update docs/src/rust-feature-support/intrinsics.md.

Please don't forget to update docs/src/rust-feature-support/intrinsics.md.

Thanks for the reminder. Added.

This PR could use quite a bit of cleanup, especially around hard-coded values and unit tests. In particular, it's probably better to store the hard-coded values using their byte representation as opposed to their decimal one.

I'm really sorry, I think you might've misunderstood my comment about the as. I think we should be using as in the kani-compiler code since this is getting rustc's values as the source of truth.
My comment about the as is about Kani's implementation of casting float to integer using as. I would expect the same bounds to be used for both, but it sounds like we don't.

I'm really sorry, I think you might've misunderstood my comment about the as. I think we should be using as in the kani-compiler code since this is getting rustc's values as the source of truth.

Not at all. I was planning to make this change before your comment. The reason is that casting a decimal value into a floating-point value can have surprising behavior. For example, this program:
    println!("{}", u32::MAX);
    let f1: f32 = u32::MAX as f32;
    let f2: f32 = u32::MAX as f32 + 1.0;
    let f3: f32 = (u32::MAX as u128 + 1) as f32;

    println!("{:.32}", f1);
    println!("{:.32}", f2);
    println!("{:.32}", f3);

prints:
4294967295
4294967296.00000000000000000000000000000000
4294967296.00000000000000000000000000000000
4294967296.00000000000000000000000000000000

So casting the decimal value u32::MAX<PHONE_NUMBER>) to a float results in a different number. This is because the cast operation rounds to the nearest number that can be represented as an f32. In other words, when assigning an f32 a decimal value, the actual value it stores may end up being different (could be smaller or larger). This calculator, which @tautschnig pointed out to me helped me determine the correct bounds.
One way to guarantee a specific value is to use the byte representation of the floating-point number. This is the reason I switched to it.

I made a few final touches:

Moved most of the code to a new float_utils module to avoid polluting intrinsics.rs (Thanks @celinval for the suggestion)
I added the code I used to determine the bounds as a unit test. If run with --nocapture, it'll print the constants.
I added a small optimization that skips the comparison if the bound is +/- infinity.

@tautschnig @celinval Let me know if you want to take another look. Otherwise, I'll go ahead and merge.


Bishop's Soldier image shows twice

This is a bluebones issue following the BFM fix earlier today.

This is because when we parse the sitemap of magiccards.info we pick up both the normal link to Ixalan AND the link to "new" sets at the top for Ixalan. Now we allow more than one image for non-dfcs (to support BFM) it is an issue. Shipping fix to bluebones …


Introduce SEV Offline KBC

Adapted from offline_sev_kbc by @Jakob-Naucke and built on EFI Secret Kernel Module by @dubek. This is a big piece of the answer to #16. Works fine retrieving an injected secret from inside an SEV guest.

I've updated the README to address some of the things that are mentioned here. I can add some additional links once the efi secret module is merged.

@fitzthum We'll be able to merge after you rebase that PR, thanks a lot.


UBsan findings in OpenSSL 1.1.d

I'm testing OpenSSL 1.1.d with UBsan. There are about a dozen failures like shown below.
../test/recipes/15-test_ec.t ....................... Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/5 subtests
../test/recipes/15-test_ecdsa.t .................... Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/1 subtests
../test/recipes/15-test_ecparam.t .................. Dubious, test returned 2 (wstat 512, 0x200)
Failed 2/134 subtests

Closer inspection:
$ openssl-1.1.1d$ PERL5LIB=util/perl TOP=$PWD perl test/recipes/15-test_ec.t
1..5
ok 1 - require '/home/jwalton/openssl-1.1.1d/test/recipes/tconversion.pl';
    # Subtest: /home/jwalton/openssl-1.1.1d/test/ectest
    1..11
crypto/ec/ecp_nistp521.c:144:15: runtime error: load of misaligned address 0x7ffd19dc40a7 for type 'limb', which requires 8 byte alignment
0x7ffd19dc40a7: note: pointer points here
 c2 31 7e 7e f9  9b 42 6a 85 c1 b3 48 33  de a8 ff a2 27 c1 1d fe  28 59 e7 ef 77 5e 4b a1  ba 3d 4d
             ^
...

It is easy enough to duplicate - just add -fsanitize=undefined to CFLAGS and LDFLAGS.
Also see OpenSSL 1.0.2r and UBsan findings and friends.

As far as I know, they go away with -DPEDANTIC

Thanks @kroeckx.
No joy. The UB is still present with -D.
The UB breaks al least 14 other programs and libraries during testing. These programs and libraries depend upon OpenSSL.

That's one that's enabled when you pass enable-ec_nistp_64_gcc_128

Yes, enable-ec_nistp_64_gcc_128 is in effect. I use it whenever I'm on a little-endian machine and __SIZEOF_INT128__ is defined. If I recall correctly, DH is 2x to 4x faster.
I believe Andy or Dr. Henson recommended it way back when.

Yes, you do want to enable it. But clearly the code has some
undefined behaviour in it that works properly for some platforms,
and not for others.


Reduce dependencies size

Initial checklist

[X] I read the support docs
[X] I read the contributing guide
[X] I agree to follow the code of conduct
[X] I searched issues and couldn’t find anything (or linked relevant results below)

Problem
I am working on a project using unified, remark and rehype and trying to improve deploying times I realized that the size of this dependency is huge because of parse5.
A let you part of the installation sizes.

Solution
Don't use parse5; try to replace it with another solution or use the unified system to parse.
Alternatives
A custom parser.

This project doesn't use parse5. Please check what this project does

The question doesn’t clearly describe a problem to me, but I think you are looking for rehype-dom.

Sorry about that, I was looking for hast-util-from-html
Thank you for the quick answer.


[Improvement] Add typescript types declaration (index.d.ts)

Is your feature request related to a problem? Please describe
When we use your library with Vue/Nuxt application with Typescript, we have an error message something like that: Type declaration is missing
Describe the solution you'd like
The minimal solution is to add index.d.ts of the root lib folder, like this:
// index.d.ts
declare module 'vue-advanced-chat'

Sounds good, don't hesitate to make a pull request if you want this to be fixed asap

I created a Nuxt project with Typescript, but I couldn't see any Type declaration is missing error.
It seems to work fine on my side without index.d.ts file. Could you let me know how I can reproduce?
Any help to improve Nuxt & Typescript compatibility is much appreciated :)

Have you set the strict mode of Typescript ?

Yes, I have "strict": true in my tsconfig.json

Resolved in # 91 and pending deployment to npm

Released in v.0.5.7

@LouisMazel The travis build is failing since the implementation of typescript:
https://travis-ci.com/github/antoine92190/vue-advanced-chat/builds/209822282
><EMAIL_ADDRESS>gen:types /home/travis/build/antoine92190/vue-advanced-chat
> tsc --declaration
dist/vue-advanced-chat.js:1:10 - error TS9005: Declaration emit for this file requires using private name '(Anonymous function)'. An explicit type annotation may unblock declaration emit.

I update typescript version without success. Do you have any clue?

Actually, the build fails because of this specific line:
https://github.com/antoine92190/vue-advanced-chat/blob/master/src/ChatWindow/RoomsList.vue#L108
No idea why, for some reason travis don't like format-links syntax. Will investigate.

Fixed in https://github.com/antoine92190/vue-advanced-chat/commit/24982cc4265428e1af641ae5b0ceb89ad51c8d76


[_]:(feature) Update photo by ID method

Update a photo by ID, currently only previews update is allowed

Codecov Report

Merging #123 (ffd75ca) into master (fcdc0f6) will decrease coverage by 0.42%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master     #123      +/-   ##
==========================================
- Coverage   76.42%   76.00%   -0.43%
==========================================
  Files          39       39
  Lines         844      850       +6
  Branches      106      106
==========================================
+ Hits          645      646       +1
- Misses        187      192       +5
  Partials       12       12

Impacted Files
Coverage Δ

src/photos/types.ts
100.00% <ø> (ø)

src/photos/photos/index.ts
21.53% <16.66%> (-0.50%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remove Excessive Unit Test Logging

Type of change

Test update

Description

Removed excessive logging in unit tests by changing logging from debug to info in multiple test files

Just curious, why decrease the log level, unit tests don't dump their logs unless a test fails, why wouldn't we want the tests in debug mode on failure?


add convenience methods

This PR provides a couple convenience methods on the Store class:

keys gets the list of all keys in the store
values gets the list of all the values in the store
entries gets a list of key-value tuples in the store
length gets the number of entries in the store

Superseded by #7


Tokens for deployer and committer users

Another couple of tokens that might be useful for commands would be the current deployer and committer. Not wanting to make too many tokens and make it messy, but perhaps {{ deployer_email }} {{ deployer_name }} {{ committer_email }} and {{ committer_name }}.
Example use case: we're using Rollbar, and we can notify Rollbar of a deploy being made by posting a webhook, and you can optionally provide the user who made the deploy.

👍 Makes sense, I am away until next week so don't have a chance to do it but I will look early next week

If a deployment is triggered via a webhook it is possible for the {{ deployer_name }} and {{ deployer_email }} to not have values.
What would you expect to happen in that case, blank values? The webhook can specify a name, so I guess {{ deployer_name }} would be the value specified via the hook otherwise webhook but not sure about email

Hmm, that's a good point. Well I suppose either it should fall back to blank values, or we could instead replace with a token that represents a "deploy source" (or something worded better) - and then either that is "webhook" or the username?


following code is not working in my system :( why ?

Your HTML5 doctype is not working ? What's the real problem ?

I have got the solution by the way thanks ☺
I didn't include jQuery library top of angular library...


Automating the versioning of docker images

Update the current CI/CD pipeline to automate the building and deployment of the docker image to both the team's DockerHub and GHCR. This image can then be used in the IDP template repository for setting up the environment
This may need to be done for both the main library and the 'lite' version that has yet to be implemented

Likely not necessary given the recent discussions on containerization
If it does become necessary one day, I suggest uploading to the team's GHCR (requires an admin's key) rather than Docker Hub in order to have all project assets on a single platform


Release 1.2 of the project

Includes some tweaks that should go in prior to the release.

Note that this release declares a dependency on an unstable version of Smack: 4.5.0-alpha4-SNAPSHOT
As a result, the behavior of this version might change over time (if Smack releases newer variants of the same SNAPSHOT).
We're pressing on with this release anyway, as we'd like to benefit from changes that it brings. That said, this version should probably not be used by end-users, and should be regarded as an interim-release only.


Wrong orientations

I can make a work around for this, but just raising this for visibility.
It seems that the Y and Z axis are swapped.

https://github.com/Looking-Glass/JoyconLib/issues/8#issuecomment-374910169


Made insetPadding configurable for Date Picker Dialog

This PR adds following properties to the DatePickerDialog:

insetPadding

Fixes https://github.com/flutter/flutter/issues/155650
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] I followed the [breaking change policy] and added [Data Driven Fixes] where supported.
[x] All existing and new tests are passing.

favor

CC @MitchellGoodwin

I think before merging we need a clear use case for this. Because this is going to introduce a different way to get the date value along with what we had before, it feels off. It may be possible that the problem this is trying to fix could be solved by something else.

Yeah, may be instead of adding the callbacks to the dialog, we can expose a new widget that composes of the Calendar and Input date picker, and have the callbacks in it. Then, use the widget to build DatePickerDialog without exposing the callbacks.

I would split out the insetPadding change from the onStart/EndDateChanged change. It's totally independent, right? I think the insetPadding change looks like it's probably easy to get landed on its own.

For now, I removed the callbacks from the PR.

Ah I missed the Google testing failure. @sarbagyastha can you push a merge commit? I see no failures internally.


Try using the expected enum assist (help-based) inserts serde::__private::Some

"Try using the expected enum" inserts  serde::__private::Some where Some is expected.
See also issue #8512.
Given this code (in ndarray), serde is an optional dependency:
struct Foo;

impl Foo {
    pub fn f(&self) -> Option<i32> {
        if 3 + 3 == 6 {
            1
        } else {
            None
        }
    }
}

There's a diagnostic:
[rustc E0308] [E] mismatched types
expected enum `std::option::Option<i32>`
   found type `{integer}`
——————————————————————————————————————————————————————————
[rustc E0308] [H] try using a variant of the expected enum

The full rustc error is this:
error[E0308]: mismatched types
    --> src/impl_methods.rs:2582:13
     |
2580 |     pub fn f(&self) -> Option<i32> {
     |                        ----------- expected `Option<i32>` because of return type
2581 |         if 3 + 3 == 6 {
2582 |             1
     |             ^
     |             |
     |             expected enum `Option`, found integer
     |             help: try using a variant of the expected enum: `Some(1)`
     |
     = note: expected enum `Option<i32>`
                found type `{integer}`

If I use this assist I get the following result, and it is wrong because I expect Some instead.
struct Foo;

impl Foo {
    pub fn f(&self) -> Option<i32> {
        if 3 + 3 == 6 {
            serde::__private::Some(1)
        } else {
            None
        }
    }
}

rust-analyzer 2021-04-05
Editor: Vim + Coc

This is a fix provided by rustc and needs to be fixed there.

ok. I saw this in json output from cargo build --message-format=json:
"suggested_replacement":"Some(1)","suggestion_applicability":"MaybeIncorrect"
there is no mention of serde in the json output, is there another place to look?

I was unable to reproduce with the provided example until I added use serde; to the top, which then led to "suggested_replacement":"serde::__private::Some(1)","suggestion_applicability":"MaybeIncorrect".

seems to be rustc issue


also publish triton executable

Description
Publish executable from the build script

Codecov Report
Patch coverage: 71.75% and project coverage change: +1.53 :tada:

Comparison is base (bb5073f) 72.08% compared to head (1cc1b17) 73.62%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #2617      +/-   ##
============================================
+ Coverage     72.08%   73.62%   +1.53%
- Complexity     5126     6949    +1823
============================================
  Files           473      684     +211
  Lines         21970    30333    +8363
  Branches       2351     3138     +787
============================================
+ Hits          15838    22333    +6495
- Misses         4925     6477    +1552
- Partials       1207     1523     +316

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...djl/modality/cv/transform/RandomFlipLeftRight.java
25.00% <0.00%> (-25.00%)
:arrow_down:

...djl/modality/cv/transform/RandomFlipTopBottom.java
25.00% <0.00%> (-25.00%)
:arrow_down:

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

.../modality/cv/translator/ImageFeatureExtractor.java
0.00% <0.00%> (ø)

.../ai/djl/modality/cv/translator/YoloTranslator.java
27.77% <0.00%> (+18.95%)
:arrow_up:

...ain/java/ai/djl/modality/cv/util/NDImageUtils.java
67.10% <0.00%> (+7.89%)
:arrow_up:

api/src/main/java/ai/djl/modality/nlp/Decoder.java
63.63% <ø> (ø)

... and 227 more

... and 354 files with indirect coverage changes

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


Add converter (HF <-> Nanotron) for Mamba

Include bug fix in Mamba modeling
Can now convert HF Mamba model to Nanotron in every parallelism (DP/PP/TP)
Can now convert Nanotron model in every parallelism (DP/PP/TP) to HF model
Mamba generation in every parallelism (DP/PP/TP) match the version with no parallelism

Ready for first review. Just need to fix the PP bug because of inference_param which is not a TensorPointer

should be done now

going to split this PR into multiple PR as it has a lot changes


[WIP]: Load PTIB

I added a load script for PTIB built off of the stp enrol branch.
Decided to go with machine readable column names compared to previous cycles:
year                                   - Year
credential                          - Credential (Program) (Program)
cip                                     - CIP
age_group                         - Age Group
immigration_status           - Immigration Status
sum_of_graduates             - Sum of Graduates
sum_of_enrolments           - Sum of Enrolments
sum_of_total_enrolments  - Sum of Total Enrolments
Name of table to be loaded into decimal is PTIB_Credentials based on the table name from PSSM_2019-2020_Private Training Institutions.R (T_Private_Institutions_Credentials_Imported_2021-03).

I updated to readxl, added dbreadtable statement to the testing code at the bottom, and switched to the new data. I had to add some cleaning code for the cip column with the new data.


Add labels related to kamelets trust level

We need to add a label for this purpose.
I'd like to discuss about:

Label name (it should be in metadata)
Level names. For example: Basic, Medium, High

What do you think? This would be useful even from a catalog perspective.
cc @nicolaferraro @lburgazzoli @valdar

Would that be a sort of "maturity level indicator" of the kamelet?

Yes, something like that.

What if we have like in the other camel projects

firstVersion: 1.0, 1.1, 1.2 and so on
supportLevel: Stable, Preview, Experimental

Then its the same for an kamelet as it would be for a component, extension, etc.


[DSW] Non mandatory enhancement - add warning message when run dumpwallet or dumpprivkey

Add a warning message to display on the rpc console the first time the user runs dumpwallet or dumpprivkey.  Successive runs of the command will not trigger the warning message.

@LotCooler

Compilation is giving errors above in dev1 server.

That's weird.  Is that a memory issue?  It had no issues in jenkins
https://jenkins.4444.tools/job/DSW/job/DSW-develop-AddDumpWalletWarning/

Oh, it's errorring in a test modoule.  I'm sure jenkins does a configure with  --disable-tests

Yes develop branch is also giving error now. It seems the last QT related commit to develop has a bug in test section. We can fix it.

Added another commit to this PR. The commit only contains comments that were meant to be in there originally but seemed to have been removed. The comments warn about code duplication and changes in one location should be replicated in the other duplicate location.

@LotCooler thank you for your help, your changes were already included on the main develop branch :)


Error in running graph_alexnet on 1.2 OpenCL

Hello ARM,
I am trying to evaulate ACL library.
I have
Device Version                                  OpenCL 1.2
Device OpenCL C Version                         OpenCL C 1.2
Device Type                                     GPU
Device Profile                                  EMBEDDED_PROFILE
To run CL samples, I have changed
concat_str += " -cl-std=CL1.2 "; as device doesn't have non_uniform_workgroup_support or OpenCL2.0.
The ./cl_convolution, ./cl_events, ./cl_sgemm and other samples work welll.
However on running ./graph_alexnet and other samples at runtime I am getting below error:
BuildGroup_40:524:44: error: automatic variable qualified with an address space
__constant VEC_DATA_TYPE(DATA_TYPE, 4) type_min4 = (VEC_DATA_TYPE(DATA_TYPE, 4))(MINVAL);
^
terminate called after throwing an instance of 'cl::Error'
what():  clCreateKernel
Please let me know if there is a way to enable support for devices with OpenCL1.2 profile supported.
Regards,

Hello ARM,
I have the same issue as Anthony. However, even if I added NULL to clEnqueueNDRangeKernel, graph_alexnet still throws the error, please see below:
BuildGroup_31:671:44: error: automatic variable qualified with an address space
__constant VEC_DATA_TYPE(DATA_TYPE, 4) type_min4 = (VEC_DATA_TYPE(DATA_TYPE, 4))(MINVAL);
^
terminate called after throwing an instance of 'cl::Error'
what(): clCreateKernel
I added to OpenCL.cpp. line 332.
Do you have any advices for me?
Best Regards,
Khoi


Seems to be an issue opening any PHP service after BLT VM completes

Issue Type

Bug Report / Support Request

Your Environment

Vagrant 2.2.1
VirtualBox 6.0.14r133895
ansible 2.9.2
  config file = None
  configured module search path = ['/Users/erekj/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/Cellar/ansible/2.9.2/libexec/lib/python3.7/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 3.7.5 (default, Nov  1 2019, 02:16:23) [Clang 11.0.0 (clang-1<IP_ADDRESS>)]

Your OS

macOS (10.15.1 catalina)

Full console output

Output from guest apache config test:
$>apachectl configtest
AH00112: Warning: DocumentRoot [/usr/share/php/xhprof_html] does not exist
AH00558: apache2: Could not reliably determine the server's fully qualified domain name, using <IP_ADDRESS>. Set the 'ServerName' directive globally to suppress this message
Syntax OK

Solr and MailHog services are available. Seems isolated to PHP services
>tail -f on apache error log when trying to access Adminer:
[Tue Dec 10 20:55:48.864470 2019] [proxy:error] [pid 14936:tid<PHONE_NUMBER>08736] (111)Connection refused: AH00957: FCGI: attempt to connect to <IP_ADDRESS>:9000 (*) failed
[Tue Dec 10 20:55:48.864677 2019] [proxy_fcgi:error] [pid 14936:tid<PHONE_NUMBER>08736] [client <IP_ADDRESS>:59899] AH01079: failed to make connection to backend: <IP_ADDRESS>, referer: http://dashboard.local.projblt.com/

Summary

I've also created an issue in the Acquia BLT issue queue here in the event it is more related to the BLT usage of DrupalVM: https://github.com/acquia/blt/issues/3948
There seems to be an issue opening any PHP service after BLT VM completes. So, Adminer, PimpMyLog, and the site are not running.
I’ve re-installed VirtualBox, completely destroyed the VM (vagrant destroy) then, followed through with normal composer install, BLT VM. Same thing I've been doing for the last year every so often on our project. My coworker has almost the exact same Mac / Setup and is able to run through the BLT VM, blt setup, through to the BLT sync, no issues. This issue began for me about a month ago.

This appears to be a dupe of https://github.com/geerlingguy/drupal-vm/issues/1996
It would probably be better to centralize discussion there.


Add number suffixes

[ ] I've included my change in CHANGELOG.md

Codecov Report
Patch coverage: 87.25% and project coverage change: +0.39% :tada:

Comparison is base (13fa23b) 82.93% compared to head (86b87d2) 83.33%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #481      +/-   ##
==========================================
+ Coverage   82.93%   83.33%   +0.39%
==========================================
  Files          74       75       +1
  Lines        9257     9689     +432
==========================================
+ Hits         7677     8074     +397
- Misses       1580     1615      +35

Files Changed
Coverage Δ

src/error.rs
33.54% <0.00%> (-9.73%)
:arrow_down:

src/de/mod.rs
73.76% <52.63%> (-1.15%)
:arrow_down:

src/ser/mod.rs
71.38% <54.92%> (-0.19%)
:arrow_down:

src/parse.rs
81.18% <86.27%> (+5.87%)
:arrow_up:

src/ser/tests.rs
85.97% <100.00%> (-0.34%)
:arrow_down:

tests/370_float_parsing.rs
66.07% <100.00%> (-32.15%)
:arrow_down:

tests/481_number_underscores_suffixes.rs
100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Manually checking the coverage shows that 100% of the patch seems to be covered

?r @torkleyy and @cart


feat: add new action type for Flow API call

Pull Request Test Coverage Report for Build 421

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 61.153%

Totals

Change from base Build 417:
0.04%

Covered Lines:
470

Relevant Lines:
683

💛  - Coveralls

I like “REMOVE_WORKFLOW”
From: Paul Kim<EMAIL_ADDRESS>Sent: Wednesday, December 11, 2019 9:47 AM
To: microsoft/ConversationLearner-Models<EMAIL_ADDRESS>Cc: Lars Liden<EMAIL_ADDRESS>Review requested<EMAIL_ADDRESS>Subject: Re: [microsoft/ConversationLearner-Models] feat: add new action type for Flow API call (#229)
@msft-paulkim commented on this pull request.

In src/Action.tshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FConversationLearner-Models%2Fpull%2F229%23discussion_r356742285&data=02|01|Lars.Liden%40microsoft.com|76fba46587624478ab9c08d77e62167c|72f988bf86f141af91ab2d7cd011db47|1|0|637116832058847178&sdata=fQ217sw%2FlJEykhDphqEuNMlDCze1EMlLDj4eO%2B37V3Q%3D&reserved=0:

@@ -13,6 +13,7 @@ export enum ActionTypes {

SET_ENTITY = 'SET_ENTITY',
DISPATCH = 'DISPATCH',
CHANGE_MODEL = 'CHANGE_MODEL',

FLOW = 'FLOW',

How about "EXTERNAL_WORKFLOW" or "REMOTE_WORKFLOW"?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FConversationLearner-Models%2Fpull%2F229%3Femail_source%3Dnotifications%26email_token%3DABRZ3ZRLAURLL444AOWXEHLQYERQDA5CNFSM4JZGWFJ2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCO3HPFQ%23discussion_r356742285&data=02|01|Lars.Liden%40microsoft.com|76fba46587624478ab9c08d77e62167c|72f988bf86f141af91ab2d7cd011db47|1|0|637116832058847178&sdata=3IbJv4yywAf6NDNpX646oISjEnSGRKae0iCK2D5Uitg%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABRZ3ZW3HGIAEORILRAN7Y3QYERQDANCNFSM4JZGWFJQ&data=02|01|Lars.Liden%40microsoft.com|76fba46587624478ab9c08d77e62167c|72f988bf86f141af91ab2d7cd011db47|1|0|637116832058857176&sdata=byfU7Gb%2BlPq2JquwshEgIWMim5IdTK0ct%2F16fRnTOhk%3D&reserved=0.

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Node worker fail to read batch result

HKube micro-service
@hkube/worker-data-adapter
Describe the bug
Node worker fail to read batch result
"failed to extract data input: items is not iterable (cannot read property undefined)"
execute
{
    "name": "jnk1111",
    "nodes": [
        {
            "nodeName": "node0",
            "algorithmName": "node-new-rand",
            "input": [
                3,
                5
            ]
        },
        {
            "nodeName": "node1",
            "algorithmName": "node-new-rand",
            "input": [
                "#[491,337,337,474,353,779,754,487,844,379,604]",
                "@node0"
            ]
        },
        {
            "nodeName": "node6",
            "algorithmName": "eval-alg",
            "input": [
                12,
                17,
                "@node1"
            ]
        }
    ],
    "priority": 5,
    "experimentName": "main",
    "options": {
        "ttl": 3600,
        "batchTolerance": 80,
        "progressVerbosityLevel": "info"
    },
    "jobId": "main:jnk1111:ddv4laf5",
    "startTime":<PHONE_NUMBER>396,
    "lastRunResult": {
        "timestamp": "2020-07-20T13:38:31.169Z",
        "status": "failed",
        "timeTook": 43.046
    },
    "types": [
        "stored"
    ]
}

"trace": [
{
"column": 35,
"file"<EMAIL_ADDRESS>"line": 151,
"method": null
},

@tamir321 the fix is in the nodejs wrapper. need to build the algorithm again

tested on {
"systemVersion": "v1.3.94",
"fullSystemVersion": "v1.3.94-1596093890817",


Feature/admin pages

Adminn' log in.
Admin page for users.
Template for articles

Honestly, this PR is such a mess that I suggest to close it all together and just start from the scratch, it is clear that the scope of the work that you decided to do is too big for you.
CC @mmrshk

Honestly, this PR is such a mess that I suggest to close it all together and just start from the scratch, it is clear that the scope of the work that you decided to do is too big for you. CC @mmrshk

:-) I know that professional may count my atempt to create a sustainable code as something rediculus imitation of real site's admin control tool. Nevertheless I'm not going to leave it. It is a good sandbox for understandin the processes I meet in the future work. And here is put a lot of efforts.
So I would appreciate any possible assistance on this interesting way.
Thank you for your attention to this matter.
Sincerely yours, StasFly

I think refactoring is enough. Clean up your views, the helper methods u defined are not intuitive to understand.
I suggest to search for tutorial how to do admin pages.
CC @mmrshk

Nevertheless I'm not going to leave it.

Why do you think that it's up to you to decide?

I think refactoring is enough.

With how slow things are going, I can bet that this PR is not going to be merged next week, unless someone else writes the changes for him.

Just as expected, you guys could have listen to me and let someone else to it, yet you decided that my opinion is not important and now we still have this PR hanging and I almost certain that it is not going to be merged the next week either. 👍


Notebook run action not visible on Windows but visible on Mac

Trying to figure if this is native notebook bug or a jupyter extension bug. I'm currently trying capture the end to end experience for opening a notebook file in VS Code.

Open VS Code with an empty data & extensions directory
Open a notebook file
Install the recommended extension (Jupyter)
Re-open file
Trust notebook
:bug: Run a cell, run action is not visible

However on macOS, the run actions DOES show up, so there is some weird behavior here:

Python     2.7.15
Edition	Windows 10 Pro
Version	20H2
Installed on	3/24/2021
OS build	19042.928
Experience	Windows Feature Experience Pack 120.2212.551.0
Version: 1.57.0-insider (user setup)
Commit: 1082913dd012e21cc5f35a06f15c480ca447f67f
Date: 2021-05-27T06:55:24.593Z
Electron: 12.0.9
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042
cc @claudiaregio

I can reproduce the exact behavior that Miguel describes if I uninstall all Python interpreters, uninstall the Python extension, and delete all kernels from my jupyter/kernels directory.
@misolori You mentioned you have the default system Python 2.7, I assume that is on Mac? Do you have Python installed on your Windows machine at all?
If you don't have Python installed on your Windows machine (and no kernels), I think what might be happening here is that if you have no Python interpreters or kernels, the Jupyter extension does not register any notebook controller, and under the kernel push UX, the run button only appears when a kernel becomes available. If the Jupyter extension can't create and return any controllers, the run button stays hidden. On macOS Python 2.7 is available by default so we are able to find and set up a controller for that. We have plans to work around this by returning a dummy Python 3 kernel which would address this bug: https://github.com/microsoft/vscode-jupyter/issues/5861

@misolori You mentioned you have the default system Python 2.7, I assume that is on Mac? Do you have Python installed on your Windows machine at all?

I misspoke, I had previously installed Python 2.7 so I did have that installed. I also tried the repro steps on a Windows Sandbox environment and still saw the same issue. However, in today's insiders I am no longer seeing this issue so not sure if the Jupyter extension pushed an update that fixed this. Here are my current versions:
Jupyter: v2021.6.885080120
Version: 1.57.0-insider (user setup)
Commit: 541a5d4fa70ab3095f23b28b88914e1f6a8f65c4
Date: 2021-05-28T09:50:57.063Z
Electron: 12.0.9
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19041

Closing as I can no longer repro and it appears to be fixed. Appreciate everyone's help here!


[FEATURE]-Webex Call Authentication Prompt

Is your feature request related to a problem? Please describe.
In Cisco Room Os, when joining a Webex call with no host, there is a prompt that asks if you are the host or not.  Currently, there is no mechanism to move on from that screen.
Describe the solution you'd like
A notification when the call is prompting you for a host pin and the ability to join as either a host or guest w/ pin if necessary
Additional context
Event registration string: xFeedback register Status/Conference/Call/AuthenticationRequest\n
page 236:
https://www.cisco.com/c/dam/en/us/td/docs/telepresence/endpoint/roomos-1011/api-reference-guide-roomos-1011.pdf

@ngenovese11 This stil an issue, should we move it to the EPI?

It's been addressed in the API


How to use HTTPS proxy with aiohttp?

I noticed the code in ProxyConnector：
assert proxy.startswith('http://'), ("Only http proxy supported", proxy)
Is there any way to use HTTPS proxy with aiohttp?
If not, is this feature under consideration?
Any help will be appreciated.

Try this: http://aiohttp.readthedocs.org/en/stable/client.html#proxy-support

Currently there is no way to use HTTPS proxies.

Thanks anyway :)

Could someone share some light on why HTTPS proxies are unsupported? I might be able to help out with a PR, but I want to understand whatever issue underlay this decision.

Perhaps the main problem is that proxies are covered by mocked tests only.
aiohttp test suite just doesn't detect proxy regressions.
I see the only option: test via real proxy.
Tests should be skipped if proxy is not configured but travis should be configured to use proxies at least.
Aslo testing page should have instructions for proxy server setup.
If you want work on it -- it would be awesome!

We can write simple proxy, so we can use it for tests and as an example

@fafhrd91 it is the best solution but who will do it?
I could be a volunteer if KeepSafe will pay for the task. Right now I'm overwhelmed by other aiohhtp issues and they are more important than proxy support, isn't it?

I think this task is low priority in general, also KeepSafe does not use any proxies. I am not with KeepSafe anymore, so I can not comment on payment.
@danielnelson is responsible for KeepSafe backend

I think #1366 may not fix this, now that I read over it again. #1366 fixes use of CONNECT tunnelling of an https server through an http proxy (recently broken).

#1366 did not work for me. I get a ProxyConnectionError(1, 'Can not connect to localhost:8081 [[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:749)]') error. I'm assuming that I need to point to the cert somehow. In requests I set a REQUESTS_CA_BUNDLE environment variable.

try verify_ssl=False

@fafhrd91 thanks for the suggestion. I need to use a ProxyConnector object which doesn't expose access to the verify_ssl attribute of the BaseConnector. I tried setting the _verify_ssl attribute of ProxyConnector to False, but am still seeing the same issue. Will dig in a little more.

ProxyConnector is deprecated and removed in master

Thanks for your response @fafhrd91 . Trying again with latest zeep and aiohttp code. Here's my relevant aiohttp code:
        if async:
            loop = asyncio.get_event_loop()
            auth = aiohttp.BasicAuth(username, password)
            if proxy:
                conn = aiohttp.TCPConnector(verify_ssl=False)
                session = aiohttp.ClientSession(loop=loop, connector=conn)
                t = ProxyAsyncTransport(loop, session=session,
                                   proxy=url, auth=auth)
            else:
                session = aiohttp.ClientSession(loop=loop, auth=auth)
                t = AsyncTransport(loop, cache=None, session=session)

The ProxyAsyncTransport class looks like this:
class ProxyAsyncTransport(AsyncTransport):
    def __init__(self, loop, proxy, session=None, auth=None):
        super().__init__(loop, session=session)
        self.auth = auth
        self.proxy = proxy

    async def post(self, address, message, headers):
        self.logger.debug("HTTP Post to %s:\n%s", address, message)
        with aiohttp.Timeout(self.operation_timeout):
            if self.auth:
                response = await self.session.post(
                    address, data=message, headers=headers,
                    proxy=self.proxy, proxy_auth=self.auth)
            else:
                response = await self.session.post(
                    address, data=message, headers=headers,
                    proxy=self.proxy)
            self.logger.debug(
                "HTTP Response from %s (status: %d):\n%s",
                address, response.status, await response.read())
            return response

    async def get(self, address, params, headers):
        with aiohttp.Timeout(self.operation_timeout):
            if self.auth:
                response = await self.session.get(
                    address, params=params, headers=headers,
                    proxy=self.proxy, proxy_auth=self.auth)
            else:
                response = await self.session.get(
                    address, params=params, headers=headers,
                    proxy=self.proxy)
            from pretend import stub
            return await stub(
                content=await response.read(),
                status_code=response.status,
                headers=response.headers)

Am running into this Error:
 Only http proxies are supported
If I comment out the line throwing that error in aiohhtp/client_regrep.py, I get:
[SSL: UNKNOWN_PROTOCOL] unknown protocol (_ssl.c:749)]

Looks like I have it working. I need to change my proxy url to be http:// instead of https://. Also, for authentication, I needed to pass the auth parameter to ClientSession.
My working code looks like this:
        if async:
            loop = asyncio.get_event_loop()
            auth = aiohttp.BasicAuth(username, password)
            if proxy:
                url = url.replace('https://', 'http://')
                conn = aiohttp.TCPConnector(verify_ssl=False)
                #ctx = ssl.create_default_context(cafile=config['PROXY_CA_BUNDLE'])
                #conn = aiohttp.TCPConnector(ssl_context=ctx)
                session = aiohttp.ClientSession(loop=loop, connector=conn, auth=auth)
                t = ProxyAsyncTransport(loop, session=session,
                                   proxy=url)
            else:
                session = aiohttp.ClientSession(loop=loop, auth=auth)
                t = AsyncTransport(loop, cache=None, session=session)

@mattharrison If I'm understanding correctly, your example code doesn't address this use case. This issue is about proxying over an HTTPS proxy connection. I.e. the proxy URL is itself an https - the connection between the client and the proxy is itself TLS/SSL-encrypted. Squid is an example of a proxy server with this feature. You code changes the proxy URL so it becomes proxy-over-http.
All that said, IMO, proxying over HTTPS is ill-advised in almost all cases, and actually decreases security. When people want to use it, it is almost always a result of misunderstanding (not being aware of how CONNECT works to maintain security when proxying https over http proxy). Unfortunately, misunderstanding or not, there are business environments where proxy-over-https is required by policy.


tolerance in meters or degrees

documenation showing tolerance in meters in example as degrees. i think meters or correct its working. and isLocationOnEdge returns 1 on true nothing on false

Hi @prasadrajuv,
Apologies for the late response.
Please can you try validating response from isLocationOnEdge  using empty()  function ?
Regards,
Alexander


jshint some js file /usr/bin/env: node: No such file or directory

i use npm install jshint and Add -g to install globally on your system.
but when i use jshint  detect some js file, it show error:
/usr/bin/env: node: No such file or directory
Xubuntu13.04

Why was this closed? The standard Ubuntu repos don't require nodejs-legacy, so this is broken out of the box.


[opentelemetry-collector] StatefulSets do not delete PVCs when scaled or deleted

When creating PVCs through the OTelCol StatefulSet the PVCs are not removed on scale-down or when the StatefulSet is deleted.
This behaviour could be changed by adding .spec.persistentVolumeClaimRetentionPolicy.
More information here: https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/#persistentvolumeclaim-retention

#1057 will allow setting the values:
statefulset:
  persistentVolumeClaimRetentionPolicy:
    enabled: true
    whenDeleted: Delete
    whenScaled: Delete

which resolves this issue.


Remove redundant assignments

This PR basically intends to remove redundant variables and assignments.
This would remove the following warning:
$ bundle exec rspec
...
lib/falcon/configuration.rb:98: warning: assigned but unused variable - parent
...

It seems like @environments[name] used to be assigned directly Build::Environment.new(parent, name: name, &block) according to this change.
However, the process including the assignment of @environments[name] has been moved into merge method.
After all of this, the assignments of local vars(environments, parent)  in add method are not necessary.
Thank you.

cf. Falcon::Configuration.merge

Thanks for this I will review it tonight or tomorrow.

Thanks, merged :)


[Package Request]: EaseUS Todo PCTrans Free

Package Requested

[ ] I would like someone else to build the manifest.
[X] I would like help so I can submit the manifest.
[X] I have performed a search and couldn't find this package.
[X] I have checked that the installer for this package is not contained in a .zip file
[ ] I think there is a new version available and I have provided the URL.

Please provide the following information
Publisher: EaseUS
Publisher Url: https://easeus.com/pc-transfer-software
Package Name: EaseUS.TodoPCTrans
Description: Automatically transfer everything including files, apps, settings, accounts, passwords, and more to a new computer, hard drive or Windows OS while keeping your data safe.
Package Version: 13.2
Installer URL: From https://www.easeus.com/download-offline.html:
https://download3.easeus.com/free/pctrans.exe
By the way, EaselJS.DataRecovery and EaseUS.PartitionMaster should include Free in the package name like this one and EaseUS.TodoBackup

I have tested the manifest, but I am not sure it was necessary to modify the file "EaseUS.EaseUSTodoPCTrans.installer.yaml" to include the following strings:
Scope: machine
InstallModes:
- interactive
- silent
- silentWithProgress
InstallerSwitches:
  Silent: /VERYSILENT
  SilentWithProgress: /SILENT

I am still newbie

By the way, EaselJS.DataRecovery and EaseUS.PartitionMaster should include Free in the package name like this one and EaseUS.TodoBackup

@Trenly sorry for the messy commits, do you think it's necessary to rename any of those packages?

By the way, EaselJS.DataRecovery and EaseUS.PartitionMaster should include Free in the package name like this one and EaseUS.TodoBackup

@Trenly sorry for the messy commits, do you think it's necessary to rename any of those packages?
I think I have managed somehow to rename the folder lol

Possibly yes or possibly no. They already are in the repository and its based on the display name of the installed application, not necessarily the fact that the application. I'll double check them and update if needed


Another fix for #97, stringify true/false

Coverage remained the same at 77.632% when pulling 8337eeffb5ba6b450346f9cb905669359dc3904c on icetrain:master into 8ab5fb8d0d9a7544c7e260f6560cbf19a4b7d224 on biemond:master.


WIP: ANC Subsequent visits - Treatment Review activity

#247
#248
#249
#250
#251
#252
#253

Referred to PMTCT:
:

Not referred to PMTCT:

@nedSaf please review


Use new version identifier SystemZ instead of S390X.

The 64-bit architecture is called System z since 2000.

Auto-merge toggled on


LaTeX

URL: https://github.com/egeerardyn/awesome-LaTeX
I have read your guidelines, so I can understand if you want to wait a few weeks before merging :smile:
However, I would love to get any feedback (and hopefully some exposure to attract new contributors).

Looks good and thanks for reading the contribution guidelines :)

Added.

Nice to see a LaTeX awesome repo :-) Forking it now.


Something odd with sites object in unit test for forward bidir search

In test Long_site_and_repeated_snp_on_edge_of_site at commit 2a41ce162e87b7d69c89eb079343705551be5803
We have
prg = gacatagacacacagt5gtcgcctcgtcggctttgagt6gtcgctgctccacacagagact5ggtgctagac7c8a7ccagctgctccacacagaga
and
 //read aligns across sites 5 and 7, allele 1 in both cases
   query="tagacacacagtgtcgcctcgtcggctttgagtggtgctagacccca";

First the unit test does a bidir backward search and all the unti test assertions pass
`    EXPECT_EQ(true,first_del);
EXPECT_EQ(1,sa_intervals.size());
EXPECT_EQ(no_occ,1);
EXPECT_EQ(sites.front().front().first, 7);
EXPECT_EQ(sites.front().front().second.front(), 1);
EXPECT_EQ(sites.front().front().second.size(), 1);
EXPECT_EQ(sites.front().back().first, 5);
EXPECT_EQ(sites.front().back().second.front(), 1);
EXPECT_EQ(sites.front().back().second.size(), 1);`

If we print the sites variable we see this
p sites
$1 = std::list = {[0] = std::vector of length 2, capacity 100 = {{first = 7, second= std::vector of length 1, capacity 1 = {1}}, {first = 5, second = std::vector of length 1, capacity 1 = {1}}}}

Then we clear variables:
    sa_intervals.clear();     sa_intervals_rev.clear();     sites.clear();
and search with the bidir forwards search. Now as soon as we finish the forward search, we print sites and see this:
p sites $3 = std::list = {[0] = std::vector of length 4, capacity 4 = {{first = 5, second =std::vector of length 3, capacity 3 = {1, 2, 1}}, {first = 5, second = std::vector of length 4, capacity 4 = {1, 2, 1, 2}}, {first = 7, second = std::vector of length 4, capacity 4 = {1, 2, 1, 2}}, {first = 7, second = std::vector of length 5, capacity 5 = {1, 2, 1, 2, 1}}}}
That doesnt look right to me at all, and when we get to the assertions we have this code
    EXPECT_EQ(sites.front().front().first, 5);     EXPECT_EQ(sites.front().front().second.front(), 1);     EXPECT_EQ(sites.front().front().second.size(), 1);     EXPECT_EQ(sites.front().back().first, 7);     EXPECT_EQ(sites.front().back().second.front(), 1);     EXPECT_EQ(sites.front().back().second.size(), 1);
and this output
`./test/unittest_bidir_search_bwd_fwd.cpp:586: Failure
Value of: 1
Expected: sites.front().front().second.size()
Which is: 3
./test/unittest_bidir_search_bwd_fwd.cpp:589: Failure
Value of: 1
Expected: sites.front().back().second.size()
Which is: 5
`

So looks like in the forward search we are not adding to sites object properly.

In commit 318a43b5656560fa65fdcc538a7ea5f7ea8f318b I've added a similar check in an earlier unit test (2SNPs) and that also fails:
[ RUN      ] BackwardSearchTest.TwoSNPs
./test/unittest_bidir_search_bwd_fwd.cpp:229: Failure
Value of: 1
Expected: sites.front().back().second.size()
Which is: 3
[  FAILED  ] BackwardSearchTest.TwoSNPs (476 ms)

I notice we clear empty_allele differently in fwd and bwd

No longer relevant.


Replace Glide by Dep

Also resolves #474.
Signed-off-by: drasko<EMAIL_ADDRESS>

Drasko - we discussed this extensively about a month ago and we all agreed at the Core and TSC meetings that there was no reason to move to dep with vgo still coming.  We discussed again today at the DevOps meeting and the feeling was still the same - no immediate need for this and didn't seem to fit our strategy with regard to Glide replacement at this time.  If you have new evidence or rationale to cause this to resurface, please come to the Core WG to reopen the issue.

@jpwhitemn this one also resolves https://github.com/edgexfoundry/edgex-go/issues/474 which makes current master currently broken and unusable.
Additionally, I have one engineer reporting me that he had problems with Glide on 32-bit ARM, but I do not have much details on this. I'll ask him to run from my branch with Dep and report the results back.


JUnit fixes done before #2927

This is everything in #2958 except the fix for #2927.
See https://github.com/scala-js/scala-js/issues/2927#issuecomment-302926675 for details.

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3977/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/5267/
Test PASSed.


Unable to HR content inside NavigationView

[Body]
	View body()
		=> new NavigationView {
			new AboutModifyingState()
		}.Background(Colors.Black);

I'm using Reloadify. When I update the content of AboutModifyingState, nothing updates though the changes are sent. If I make that page the app root, then it works.

Which platform?


Update tenant-allow-block-list.md

Update docs to show block only UI and cmdlets.

@JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks


Question: Representation of device in HA

I have the TV listed as a simple Media Player in HA.
The front page only shows the On/Off button and going into more details gives me the volume controls and FF/REW (video style).
Is there a more complete UI that you can recomment that exposes more remote-like capabilities to HA UI?
I had a look at "TV Remote Card" ( https://github.com/marrobHD/tv-card ) - but have not managed to get it working in Lovelace yet.

I've always used the UI provided by home assistant.
Although the plugin you've linked looks great the component just implements a small subset of functions:

Next track, previous track
Turn on, off
Play / Pause / Stop
Volume control

Other features you'd find on your remote (up, down, numbers ... ) are not implemented in the component.
The media player component would support some additional things (developer doc) but not enough to build a full remote.
Especially the above mentioned buttons are not supported.
The sony api lib exposes these features and it could be used to implement them in some way.
If you find a way to integrate these in the component I'd be happy to accept a PR but implementing these is currently not on my agenda.


test snapit

/snapit

/snapit

@robin-drexler snapit is running into E404 Not Found errors when trying to publish the remote-dom/core, @remote-dom/preact, and @remote-dom/signals packages to NPM. Do you think it might be because we're missing the NPM_TOKEN environment variable?
https://github.com/Shopify/remote-dom/blob/d22c88e07bcacefaa9c93d54141aa4324be4401d/.github/workflows/snapit.yml#L22-L23

@robin-drexler snapit is running into E404 Not Found errors when trying to publish the remote-dom/core, @remote-dom/preact, and @remote-dom/signals packages to NPM. Do you think it might be because we're missing the NPM_TOKEN environment variable?
https://github.com/Shopify/remote-dom/blob/d22c88e07bcacefaa9c93d54141aa4324be4401d/.github/workflows/snapit.yml#L22-L23

yeah it doesn't work (yet)
We had to merge the action to actually try it.
I'm not sure yet why it's not working, but noticed the 404.
We do feed it a token: https://github.com/Shopify/remote-dom/blob/main/.github/workflows/snapit.yml#L23, but maybe it's not the correct one.
It's the same one we use in other places, but as NODE_AUTH_TOKEN.
github actions are unfortunately quite difficult to test.

/snapit


Possible syntax error when using Prefix Declaration for SPARQL Queries

I downloaded the nightly Brick version but got the error shown below when I tried to use it in python.
from brickschema import Graph

g = Graph(load_brick=True, load_brick_nightly=True)
g.load_file(bldg_brick_file)
<Graph identifier=Nd22db2bd15974015ab4bbd8c477fa056 (<class 'brickschema.graph.Graph'>)>
print(f'Before: {len(g)} triples')
Before: 75986 triples
g.expand(profile='owlrl+shacl+vbis+shacl')
*** pyshacl.errors.ConstraintLoadError: sh:declare must have exactly one sh:prefix predicate.
For reference, see https://www.w3.org/TR/shacl/#sparql-prefixes

I tracked the issue to the following block in the Brick.ttl file. It seems to be a syntax issue when using sh:declare.
<https://brickschema.org/schema/1.4-rc1/Brick> a owl:Ontology ;
    rdfs:label "Brick" ;
    dcterms:creator ( [ a sdo:Person ;
                sdo:email<EMAIL_ADDRESS>;
                sdo:name "Gabe Fierro" ] [ a sdo:Person ;
                sdo:email<EMAIL_ADDRESS>;
                sdo:name "Jason Koh" ] ) ;
    dcterms:issued "2016-11-16" ;
    dcterms:license <https://github.com/BrickSchema/brick/blob/master/LICENSE> ;
    dcterms:modified "2024-03-28" ;
    dcterms:publisher [ a sdo:Consortium ;
            sdo:legalName "Brick Consortium, Inc" ;
            sdo:sameAs <https://brickschema.org/consortium/> ] ;
    rdfs:isDefinedBy <https://brickschema.org/schema/1.4-rc1/Brick> ;
    rdfs:seeAlso <https://brickschema.org> ;
    owl:imports <http://data.ashrae.org/bacnet/2020>,
        <http://qudt.org/2.1/schema/shacl/overlay/qudt>,
        <http://qudt.org/2.1/schema/shacl/qudt>,
        <http://qudt.org/2.1/vocab/dimensionvector>,
        <http://qudt.org/2.1/vocab/prefix>,
        <http://qudt.org/2.1/vocab/quantitykind>,
        <http://qudt.org/2.1/vocab/sou>,
        <http://qudt.org/2.1/vocab/unit>,
        <https://brickschema.org/schema/Brick/ref>,
        <https://w3id.org/rec/recimports> ;
    owl:versionInfo "1.4-rc1.0-rc1" ;
    sh:declare [ sh:namespace "http://www.w3.org/1999/02/22-rdf-syntax-ns#"^^xsd:anyURI ;
            sh:prefix "rdf" ],
        [ sh:namespace "https://w3id.org/rec#"^^xsd:anyURI ;
            sh:prefix "rec" ],
        [ sh:namespace "https://brickschema.org/schema/Brick/ref#"^^xsd:anyURI ;
            sh:prefix "ref" ],
        [ sh:namespace "http://www.w3.org/2002/07/owl#"^^xsd:anyURI ;
            sh:prefix "owl" ],
        [ sh:namespace "http://data.ashrae.org/standard223#"^^xsd:anyURI ;
            sh:prefix "s223" ],
        [ sh:namespace "http://www.w3.org/2000/01/rdf-schema#"^^xsd:anyURI ;
            sh:prefix "rdfs" ],
        [ sh:namespace "https://brickschema.org/schema/Brick#"^^xsd:anyURI ;
            sh:prefix "brick" ],
        [ sh:namespace "http://www.w3.org/ns/shacl#"^^xsd:anyURI ;
            sh:prefix "sh" ] .

I changed the syntax based the website listed in the error to the following and it seems to have resolved the issue. Not sure where it is in the source code but found one reference where the syntax may be fixed.

<https://brickschema.org/schema/1.4-rc1/Brick> a owl:Ontology ;
    rdfs:label "Brick" ;
    dcterms:creator ( [ a sdo:Person ;
                sdo:email<EMAIL_ADDRESS>;
                sdo:name "Gabe Fierro" ] [ a sdo:Person ;
                sdo:email<EMAIL_ADDRESS>;
                sdo:name "Jason Koh" ] ) ;
    dcterms:issued "2016-11-16" ;
    dcterms:license <https://github.com/BrickSchema/brick/blob/master/LICENSE> ;
    dcterms:modified "2024-03-28" ;
    dcterms:publisher [ a sdo:Consortium ;
            sdo:legalName "Brick Consortium, Inc" ;
            sdo:sameAs <https://brickschema.org/consortium/> ] ;
    rdfs:isDefinedBy <https://brickschema.org/schema/1.4-rc1/Brick> ;
    rdfs:seeAlso <https://brickschema.org> ;
    owl:imports <http://data.ashrae.org/bacnet/2020>,
        <http://qudt.org/2.1/schema/shacl/overlay/qudt>,
        <http://qudt.org/2.1/schema/shacl/qudt>,
        <http://qudt.org/2.1/vocab/dimensionvector>,
        <http://qudt.org/2.1/vocab/prefix>,
        <http://qudt.org/2.1/vocab/quantitykind>,
        <http://qudt.org/2.1/vocab/sou>,
        <http://qudt.org/2.1/vocab/unit>,
        <https://brickschema.org/schema/Brick/ref>,
        <https://w3id.org/rec/recimports> ;
    owl:versionInfo "1.4-rc1.0-rc1" ;
    sh:declare [
            sh:namespace "http://www.w3.org/1999/02/22-rdf-syntax-ns#"^^xsd:anyURI ;
            sh:prefix "rdf" ;
    ] ;
    sh:declare [
            sh:namespace "https://w3id.org/rec#"^^xsd:anyURI ;
            sh:prefix "rec" ;
    ] ;
    sh:declare [
            sh:namespace "https://brickschema.org/schema/Brick/ref#"^^xsd:anyURI ;
            sh:prefix "ref" ;
    ] ;
    sh:declare [
            sh:namespace "http://www.w3.org/2002/07/owl#"^^xsd:anyURI ;
            sh:prefix "owl" ;
    ] ;
    sh:declare [
            sh:namespace "http://data.ashrae.org/standard223#"^^xsd:anyURI ;
            sh:prefix "s223" ;
    ] ;
    sh:declare [
            sh:namespace "http://www.w3.org/2000/01/rdf-schema#"^^xsd:anyURI ;
            sh:prefix "rdfs" ;
    ] ;
    sh:declare [
            sh:namespace "https://brickschema.org/schema/Brick#"^^xsd:anyURI ;
            sh:prefix "brick" ;
    ] ;
    sh:declare [
            sh:namespace "http://www.w3.org/ns/shacl#"^^xsd:anyURI ;
    ] .

Here is one example where the syntax made need to be fixed.
https://github.com/BrickSchema/Brick/blob/3af06c05f8f8725db9d270722f138afeb7d42690/support/ref-schema.ttl#L123

@gtfierro Thanks for the quick response. I continued to investigate the issue, and you are correct; it is not a syntax issue. The error does not appear when I only run shacl inference but it still persists if I use owlrl+shacl even though I don't have any sh:declare blocks in my building graph and also don't have a Brick copy in it.
However, it seems that I don't need owlrl+shacl. I was thinking it was a multistep inference process where for example the owlrl part would do the graph simplification and inference and the part shacl would do the validation. But if I understand correctly using shacl is doing both.

Yep! SHACL does both as of Brick 1.3. I believe you can still turn on "simplification" in the brickschema package and that will get done as a post-SHACL processing step

Closing for now ; let me know if you have any further questions!


Inappropriate module name for "pvlib.iam.schlick_diffuse"

Describe the bug
I am one of the major developers of the "Fresnel Equations" for Diffuse radiation on Inclined photovoltaic Surfaces (FEDIS) which is referred to, in pvlib, as pvlib.iam.schlick_diffuse. First of all, thank you for including FEDIS in pvlib. However, as we discussed in the PVSC meeting, it is an apparent mistake to use the name "Schlick diffuse" in pvlib. Schlick model is only a simple approximation of the Fresnel equations for direct radiation. It is much complicated to give the precise iam solution for diffuse radiation due to the integration in space. FEDIS derived an analytical solution of the iam model for diffuse radiation, with limited and clearly stated approximation, using the Schlick equation for direct radiation. The solution given by FEDIS is completely different with Schlick equation, however, and was not provided in Schlick paper.  It is thus very misleading and essentially wrong to use the current name to describe FEDIS model.
Therefore, I strongly suggest that the module name should be changed to pvlib.iam.fedis or pvlib.iam.fedis_diffuse.
Thank you for considering my comments.
Yu Xie
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
I strongly suggest that the module name should be changed to pvlib.iam.fedis or pvlib.iam.fedis_diffuse.
Screenshots
If applicable, add screenshots to help explain your problem.
Versions:

pvlib.__version__: 0.10.1
pandas.__version__:
python:

Additional context
Add any other context about the problem here.

Hello @xieyupku, thanks for opening this issue.  If there is a mistake in naming that function, the mistake is mainly mine.
The motivation for the name "schlick_diffuse" is that it is the diffuse IAM model that follows from the Schlick direct IAM model.  It was intended to reflect the theoretical basis of the model, not its publication origin.  It is beneficial for pvlib.iam.schlick and pvlib.iam.schlick_diffuse to have similar names so that the mathematical relationship is clear.  Hypothetically, if someone developed a diffuse model based on the ASHRAE direct model, I think it would be reasonable to call it "ashrae_diffuse", even if the diffuse model was not directly associated with ASHRAE itself.  Another example already in pvlib is faiman_rad, which is an extended form of the faiman model.  "faiman_rad" was not developed by Faiman, but we still include the name to emphasize the conceptual connection with the original model.  So to me, the name "schlick_diffuse" makes sense and I am not convinced it would be better to change it to anything else.
Additionally, I don't think the name "fedis_diffuse" would be correct.  When we were implementing the code, we came to the conclusion that, in our view, the refractive index weighting coefficient $w$ in the FEDIS model is not theoretically justifiable.  Because of that, we did not want pvlib to include any model with the $w$ term.  However, we decided that the analytical integration of the Schlick equation was suitable to include on its own, so we implemented only that part while leaving out the $w$ term.  Since that function does not have $w$, it does not implement the FEDIS model, so it would be incorrect to call it "fedis_diffuse".
However, I see the point that some people might interpret the name as suggesting that the analytical integration was given by Schlick.  To try to make it clear where the model came from and give credit to the FEDIS authors, of course we were sure to cite the FEDIS paper in the function's documentation (link) and include comments in the code indicating the origin of each equation (link).  So readers should understand that the source of the analytical integration is the FEDIS paper, but any ideas to improve the documentation so that this is clearer are very welcome.  Maybe we should expand the description in the documentation of "schlick_diffuse" to be more explicit about where the model came from?

I agree with @kandersolar on this issue.

Maybe we should expand the description in the documentation of "schlick_diffuse" to be more explicit about where the model came from?

We can add a :math: expression for the Schlick approximation for direct IAM, and describe that Yu et al. provide the integration of that expression to obtain an IAM model for isotropic sky diffuse irradiance.

@kandersolar @cwhanse  Thank you so much for specifying more details and your concern on choosing the module name. I agree that it is very important to let users know the correct origin and theoretical basis of the model. For this module, pvlib.iam.schlick_diffuse, the theoretical basis is the Fresnel equations, however, not the Schlick equation. The Schlick equation itself does not have any physical meaning. Instead, it is only a mathematical approximation of the Fresnel equations. Because of that, I feel it would be much more appropriate if the module name is Fresnel_diffuse, which is actually very similar as the real name, the "Fresnel equations" for diffuse radiation over on inclined PV Surfaces (FEDIS).
I understand you want to follow deeper to the origin of the equations. However, this is usually not the case in the scientific field. For example, the famous Newton's law of universal gravitation was derived from the Kepler's law. But it is incorrect to refer it to something related to Kepler's law due to the further derivation and investigation. For FEDIS, we only used Schlick equation as a simple tool and a starting point. The theoretical basis of FEDIS is the Fresnel equations. The final form of the equations and coefficients actually have no much relation with the Schlick equation. Because of that, we believe it is incorrect to use the current name. Please note that this is not just my opinion. All the developers and sponsor of FEDIS are very surprised and unhappy to see this inappropriate name of the module when used by pvlib.
Also, the FEDIS equations were not approved by Schlick. I am not sure if we might have legal issues to use the name, Schlick diffuse.
As for the refractive index issue, FEDIS is more flexible to use different refractive index. A hard coded refractive index in the equations in pvlib can be considered as a special case of FEDIS. So we believe it is still appropriate to use FEDIS to represent the module.
Those are our opinions as the module developers. Thank you for your consideration!

@xieyupku Eq. 4 in your paper is an integration of Schlick's approximation of the Fresnel equation. It is not an integration of the Fresnel equation. Hence the name is not fresnel_diffuse.
I think too much is being read into the use of the name schlick.

@cwhanse @kandersolar Thanks for joining in the discussion. I understand it is too much discussion for a small module included in pvlib, but it took the developers much longer time to build the model. For the benefit of all the developer, I am just expressing the opinion on behalf of them. Of course, it is totally up to you to make the final decision.
To be short, I think the current name is fine if we numerically solve the integration of Schlick model by just using Eq.(A4 and A5). If you look at the derivation of the analytical solution in the Appendix, Eq.(4) has no much relation with Schlick equation. The solution was not given by Schlick, nor approved by Schlick. The current name is not favorable by the developers, the sponsor, and Schlick. I think it would be the best that we simply replace the name as so many people do not like it.

If you look at the derivation of the analytical solution in the Appendix, Eq.(4) has no much relation with Schlick equation.

Eq. 4 in the paper is Eq. A10. Eq. A10 is the result of evaluating the integral in Eq. A8a, which is the integration of Schlick's formula (stated in Eq. A7). So Eq. 4 is the integration of Schlick, unless I am missing something major in your paper.

@cwhanse @kandersolar As you can see in the paper, 99% of the efforts in developing FEDIS is to analytically solve the integration. Schlick equation is a very small part to start with. As I mentioned, you use Kepler's law to derive Newton's law of universal gravitation, the solution is not Kepler's law anymore. However, I leave the final decision to the pvlib team.

The model in question does not use the Fresnel equations, so we cannot name it "fresnel_diffuse".  Neither can we name it "fedis_diffuse", since it is not the FEDIS model.  The model is precisely the diffuse form of the Schlick equation, so "schlick_diffuse" is the correct name for it.  The function name reflects what the model is, not where it came from.  I think we will not be convinced that the function name should be changed.
I have submitted a proposed change to the function's documentation in #1812 to clarify the origin of the integration and the function's relationship with the FEDIS model.  A preview of the updated documentation is available here: https://pvlib-python--1812.org.readthedocs.build/en/1812/reference/generated/pvlib.iam.schlick_diffuse.html
Comments on this proposed change are welcome.


feat(android): bubble action

Branches off full-screen-intent p/r. Should be merged after that's released.
Interesting though, it looks like a version bump to androidx.core:*:1.5.0-rc01 is needed. For the bubble to appear (https://developer.android.com/guide/topics/ui/bubbles#when_bubbles_appear), the following needs to be met:

The notification uses MessagingStyle, and has a Person added.
The notification is from a call to Service.startForeground, has a category of CATEGORY_CALL, and has a Person added.
The app is in the foreground when the notification is sent.

To meet the requirement for has a Person added , as far as I can tell, it requires this API  NotificationCompact.Builder.addPerson(person) which isn't there on the current stable version. There is NotificationCompact.Builder.addPerson(uri) but it's deprecated.
API design decisions:

To call this property a 'bubbleAction' to stick with the current theme that all actions (fullScreenAction, pressAction) have a launchActivity and a mainComponent.

hi did you solved this problem : https://github.com/invertase/notifee/issues/153#issuecomment-784474979

@mikehardy Can we have this opened please?
Would be a good addition if possible to fully integrate.

No promises but I can give it a look sure. In my experience Helena's work was great so I actually have high hopes


Request: Enhanced Evasion Idea

Can u add support for complete memory integration. For example, can you get the payload to load and run completely in memory and not right any thing to disk, and start up using the registry.
As this would greatly help to evade detection, by creating less of a footprint and have less for the AV to scan additionally, lots of AV don’t scan memory or registry.
Thanks

I am very busy nowadays, but if you write this i will happily merge it :)


Change email/password information

Hi Alexx & contributors.
I started automation in my house using my personal email and now I noticed that I have more that 100 devices, most part of them Sonoff. I already created lot of automation in homeassistant. I was wondering to change my email from xxxxx@gmail,com to<EMAIL_ADDRESS>into ewelink app (change email address option) and after that, update sonoff integration in homeassistant setup. I did´t find where I can update email and password information without interfering in existing devices, entities and automation.
Would be intresting to have something like LocalTuya integration where we find into "configure", an option "Reconfigure Cloud API Account" to update email/password/other info.
Do you have a procedure to change email/password without affect defined entrances?
PS: My system is running latest sw releases.
Kind Regards,
Leonardo

Would be intresting to have something like LocalTuya integration where we find into "configure", an option "Reconfigure Cloud API Account" to update email/password/other info.

I had a similar situation and fixed it this way: Keep the old integration in Home Assistant and add another Sonoff integration with the new credentials. It'll show a "Re-authentication successful" dialog without duplicating the integration or the devices. My automations also still work. Do a backup and see if it works for you.

I had a similar situation and fixed it this way: Keep the old integration in Home Assistant and add another Sonoff integration with the new credentials. It'll show a "Re-authentication successful" dialog without duplicating the integration or the devices. My automations also still work. Do a backup and see if it works for you.

/config/.storage/core.config_entries

Hello, I'm in the same process as I will sell my house. so to transfer account I have

created a new ewelink account for the buyer.
add the devices in this new account
all devices have the same IDs as before but a different name
My questions are:
1- should I first change the account/password in ? : /config/.storage/core.config_entries

2- then I will have double devices in HA?

You won't have double devices because deviceid will be same

You won't have double devices because deviceid will be same

Thank you very much Alex :-)
About SONOFFLAN, should I change th password and username only in configuration .yaml?
sonoff:
username: xxxx
password: xxxx
or I just reinstall SONOFFLAN ?
thank you :-)

Better to change user/pass in the /config/.storage/core.config_entries. Because all devices will be deleted if you reinstall integration


Gender should be nullable?

Dear OSDM Team,
For most sale request gender is not required from "offer perspective".
In osdm specification gender doesn't have "nullable" enabled.
Is it valid or not?
Kamil

Hello,
Yes, I know, but we have a problem with C# language.
Swagger generator generate this "ENUM" as NOT NULLABLE, so if we create object by default constructor, always put "MALE" enum value as default.
I see that sometimes in swagger specification you decorate also by "nullable" attribute.
I changed generated code and set this property as nullable.

I can imagine the issue - I personally use Java (Spring) generator and there, the generator developers found out late that they interpret required and nullable incorrectly.
We will have to revise the whole spec if we use required and nullable property at each attribute in the next versions (we are now about to release 3.3.0 so this can be accommodated.)
I would use X as default if a gender is not known and must be filled in. That is value we introduced for Sweden for diverse sleeping compartments.

Do you have other attributes where this issue plagues you?
It would be best, if you could join our weekly discussion on Friday 09:00 to 11:00 so that we can support you best,

TODO: define Gender nullable


How do I check whether hotspot is on or off?

A method is needed to check whether hotspot is enabled or disabled, something like MyOreoWifiManager.isHotspotOn();

I have added a pull request that implements this #13, though the method name is different.


Default to Composer-based site building instead of Drush Make

Background:

#506 and PR #694
http://docs.drupalvm.com/en/latest/deployment/composer/

Drupal VM currently supports building codebases using Composer or Drush make files. We were going to default to building with drupal-project and/or a composer.json file instead of the example drush.make.yml file, but some performance testing showed abysmal regressions in terms of the initial site build—and even fully cached site builds using Composer were 2x or more slower than Drush make file builds.
I'd like to move to using Composer as the default, as that's kind of where the puck is going... but before that can happen, we need to continue poking and prodding at Composer and all the default setup to make sure the Composer install is not a horrendously slow user experience, especially the first time (uncached).
@oxyc did some awesome investigative work and found that there are a few things we can do out of the gates to speed things up, like:

Configure vagrant-cachier correctly to cache everything from Composer outside the VM so it persists across builds.
Include prestissimo to speed up composer commands by parallelizing downloads
Use --prefer-dist

But I'm hoping there are some other optimizations we can make to get the composer installation time down quite a bit. I know @thom8 suggested https://github.com/composer/composer/pull/4961 might help by using shallow Git clones, but we definitely need hard data (benchmarks) to back up theories.

Also worth checking how much slowdown "minimum-stability": "dev" causes. Ref

did you turned off x-debug from the CLI?

I'm curious about build times in comparrison to the drupal-console and drush make methods, and the possibility of optionally switching xdebug on/off during this process...

xdebug is by default disabled so the tests from earlier ran without it. I added a few tests in PR #700 to get some basic comparison readings with different configs.

Test
802
803
804
805

Makefile
9:12
9:16
9:11
9:03

Composer

11:59
11:56

Composer + XDebug
13:19
13:11
14:17
13:11

Composer min stable

11:35
11:44
13:20

Composer shallow clones
12:36
12:00
12:09
11:47

Before when we used working copy in the makefile the build time was 10-11min. Now it's pretty consistently ~9min.
Another note, I ran composer install -vvv. Downloading all those packagist json files took:
[217.6MB/400.82s] Analyzed 3319 packages to resolve dependencies
[217.6MB/400.82s] Analyzed 51116 rules to resolve dependencies

as soon as it was in my cache:
[215.2MB/24.05s] Analyzed 3275 packages to resolve dependencies
[215.2MB/24.05s] Analyzed 49891 rules to resolve dependencies

Perfect timing! https://github.com/hirak/prestissimo/pull/101
With empty cache it's now down from 400s to:
[303.1MB/165.89s] Analyzed 5377 packages to resolve dependencies
[303.1MB/165.89s] Analyzed 94262 rules to resolve dependencies

With parellel download of packages.json files I think we can actually go forward with defaulting to composer based installs. 10min 18sec on travis vs 9min 23sec for makefiles.

Opened up a WIP PR https://github.com/geerlingguy/drupal-vm/pull/701

Well... Maybe we helped push them over the edge there :)
Good thing I didn't push a tag yet!

did you turned off x-debug from the CLI?

I don't enable it unless I explicitly need it—I'm an old-fashioned guy who normally prints/logs debug statements and I only resort to Xdebug and its ilk if needed :)

From my home in STL, on my laptop, using time vagrant up each time:

Test type
Time

Makefile default config
00:06:01

Composer (#701), first run
TODO

Composer (#701), second run
TODO

Composer (#701), cachier
TODO

@oxyc @geerlingguy you might also look at what is being done in https://github.com/platformsh/platformsh-example-drupal/pull/20 for some goodness.
Specifically: drush/policy.drush.inc

@heddn - So far I don't want to go so far as to make people using a drush/make-based workflow feel like it's a painful experience, so a policy file discouraging/blocking the use of pm commands would be a bit extreme... but at some point, we'll be dropping Drupal 7 support entirely, and at that time I think it's fine to do this.

Let's do this thing! I'm writing up a blog post with some of the learnings from here—thanks in HUGE part to @oxyc's experimentation. I think that's like 92 beers or so at this point. Sooner or later I'll just have to buy a keg. 🍻

@heddn that looks to be based on drupal-composer/drupal-project which we actually default to now :) https://github.com/drupal-composer/drupal-project

Blog post posted: http://www.jeffgeerling.com/blog/2016/speeding-composer-based-drupal-installation
Please let me know if there are errors or things missing there! Let's poke and prod the community to get things even better! :)

Great post Jeff.  One item related to drupal-packagist infrastructure.  drupal.org is now publishing this information for contrib modules.  Once the drupal core split is available drupal.org infrastructure can be used for the composer repository.

@joestewart - Great point! Do you think it will be set up for caching via Fastly out of the gate, or will it take some time to get that part of the drupal.org-based packagist configured and running?

packages.drupal.org/8 is already backed by Fastly. I don't plan to change the infrastructure of https://packagist.drupal-composer.org/. I think its not worth the effort. I hope packages.drupal.org becomes stable in a few weeks.

For reference here's my output of running:
rm -rf foo
rm composer.lock
composer clear-cache
composer create-project drupal-composer/drupal-project:8.x-dev foo --stability dev --no-progress --profile --prefer-dist -vvv

https://gist.github.com/oxyc/e9a89257fd05368992b0b722a71a0604
Some notes:

0-40s is essentially because we use create-project (auch). It seems Composer doesn't load plugins until after the project has been created and therefore the first repository packages.json files are downloaded synchronously. I'm guessing this is a Composer limitation but I opened a question over at https://github.com/hirak/prestissimo/issues/103. If this is fixed upstream somehow/somewhere it would be a nice performance improvement for us.
42-233s is downloading all the dependency information. This was the step that used to be 400sec. This will improved further once https://packages.drupal.org is up and running.
233-236s dependency resolution
236-256s download packages. Last time I checked this was way faster, I think it's just temporary network issues on my side.
256-277s extracting zip files (I have a 1.1ghz dual-core CPU so this will be faster for others).
277-281s parsing suggestions etc, generating autoload
281-297s drupal scaffold which downloads index.php/.htaccess etc (this could technically be faster, it's synchronous).

Looks like getting packages.drupal.org up and running is going to be the biggest short-term win!

Since we were mentioned... :smile:
The disabling of pm-* commands isn't a Platform.sh thing. It's a change that was made upstream to https://github.com/drupal-composer/drupal-project about 3 months ago. We're inheriting that project and following it as closely as we can, just tweaking the result a bit to make Drupal play nicer with our read-only file system.
I also just switched our installer to use packages.drupal.org today, at Mixologic's recommendation. Hopefully that will go live today or early tomorrow.

I also just switched our installer to use packages.drupal.org today, at Mixologic's recommendation. Hopefully that will go live today or early tomorrow.

Nice - I'd love to hear how it goes? Any issue on drupal.org I could follow for progress/updates?

I also just switched our installer to use packages.drupal.org today, at Mixologic's recommendation. Hopefully that will go live today or early tomorrow.

Cool I didn't understand it was up and running!
That particular step decreased by 62sec by switching over to https://packages.drupal.org/8 on my laptop.

@oxyc - Can you submit a PR for it (the change to packages.drupal.org), and I'll merge once we get word upstream that it will be semi-officially or officially supported?

Note that you'll need to change the "conflict" line to "replace" when switching to the official repos.

Prestissimo 0.3.1 now hooks into create-project and downloads all packages.json files in async. https://github.com/hirak/prestissimo/issues/103#issuecomment-225470009

@oxyc - Nice! We're using https://github.com/geerlingguy/drupal-vm/blob/73444343861b48b7ee030b687d27e4e8d92f4591/default.config.yml#L238, so we're already covered in terms of new installs and updates, right?

Yup, not sure if the version is updated for existing boxes though.


createTx checks

Hello,
I'm working on a forked version of your wallet, it'll add external inputs to the transactions. I saw that WalletService.prototype.createTx does not check if all the inputs are owned by the wallet, but there is a TODO about this:

todo: check inputs are ours and have enough value

Do you intend to add this check? If yes, will you add a way to skip the check?
Thank you

The inputs check is done in other parts of the code, so it is not possible create txs with heterogeneous inputs using createTx. I create the txs on the client, with problems to transactions data into the wallet


not rendering images

I copy your sample code into my app.
slider display but not showing images.(uri images not loading but local images loads)

@MostafaEbrahimi Can you try the provided example and tell me if it works properly for you?

Closing as no further feedback was provided.
See #135 for more information about a similar issue.


[vx.dev] implements #154

[vx.dev] This PR implements #154, created by vx.dev.

[vx.dev]: This code snippet creates a newsletter subscription form with a title, description, input for the email address, a checkbox for terms and conditions, and a submit button with an icon. The form is styled with Tailwind CSS classes to be centered, with a maximum width, and a shadow for a card-like appearance.


[mysql] Offer MySqlParellelSource DataStream API

1. Decouple debezium config Flink config in MySqlParallelSource
2. Support Table that hasn't primary key
3. Vlidate the primary key defined on DDL is same with the primary key in MySQL table
4. Refactor StatefulTaskContext.
5. Align the old Source metrics and add new metrics like Fetch Delay/Emit Delay/TPS(tableRows)/Network bandwith/SnapshotRunning (P0)

Implemented in #495


Fbopt matchduva

It looks like the only useful change is in a_model_classif.y

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (845dcd3) 92.88% compared to head (7f82bc9) 92.83%.
Report is 1 commits behind head on fbopt.

Files
Patch %
Lines

domainlab/models/model_wrapper_matchdg4vae.py
50.00%
2 Missing :warning:

domainlab/models/a_model_classif.py
66.66%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##            fbopt     #641      +/-   ##
==========================================
- Coverage   92.88%   92.83%   -0.05%
==========================================
  Files         138      138
  Lines        5410     5416       +6
==========================================
+ Hits         5025     5028       +3
- Misses        385      388       +3

Flag
Coverage Δ

unittests
92.83% <57.14%> (-0.05%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


FP_PDN_AUTO_ADJUST should guarantee that it snaps to the manufacturing grid

Prompt
The FP_PDN_AUTO_ADJUST flag, as implemented here currently sets the pitch and offset of the power grid by dividing the core width and height by the hard-coded values 4 and 8.
While this is sufficient for currently-supported PDKs, it is not future-proof.
Proposal
The method in question should get information about the manufacturing grid and use it to snap the power grid to the manufacturing grid.
More discussion on this topic, and implementation ideas, can be found in the comments of #1044.

FP_PDN_AUTO_ADJUST will be removed in the future


Wrong constant evaluation of floating-point comparison and logical-not

#include <stdio.h>
int fpeq = 0.1 == 0.2;
int fpnot = !0.1;
int main(void) {
  printf("%d,%d\n", fpeq, fpnot); // expected 0,0; got 1,1
}

Fixed with e6a28692f4f52a7b76bdfd9a05ec50d79c31f547 and 900c7e3d120f38bc54b0770c14d031f7891e1c6f


Bold errors

Addresses issue #3405.

No that's total nonsense.  It looks fine on my Windows AND Linux boxes, but non-bold looks too dull on Windows.

@adidahiya Seriously, you can't change the colours on Windows 7 console.  You need to output them differently from the program.  How about being able to configure the colours?


BUG console, can't insert string with a ; inside

using a document database with OrientDB 1.3.0
orientdb> create class a
Class created successfully. Total classes in database now: 11
orientdb> create property a.title string
Property created successfully with id=1
orientdb> insert into a set title="poil;pipo";
Inserted record 'a#12:0{title:null} v0' in 0,004000 sec(s).
!Unrecognized command: 'pipo"'
orientdb> insert into a set title="poil;pipo";
Error: com.orientechnologies.orient.core.exception.OStorageException: Error on executing command: OCommandSQL [text=insert into a set title="poil]
Error: java.lang.ArrayIndexOutOfBoundsException: 6!Unrecognized command: 'pipo"'
orientdb>
Do I have missed something ?

insert into a set title="poil;pipo"; works fine in SNAPSHOT-2.0.


Stop using <input type="button" ...

Issue Summary
It looks like we do use <input type="button" ... in the style guide (L508, L519, L530), which we probably should stop doing given that it says on that very same same page not to do so.
Steps to Reproduce

Search te code for `<input type="button"

Refs: https://github.com/wagtail/wagtail/pull/6369#pullrequestreview-489947548

Closed via https://github.com/wagtail/wagtail/commit/3ab18aa901ddf6b7152e6d0a547cf7c2a7d054e3 in #6388


make debezium'name effective

solve issue #47  issue #51  issue #52  issue #53

CI hangs on your change


pipeline return error status in pipeline load

Proposed changes

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.14 seconds
stream load tsv:          452 seconds loaded<PHONE_NUMBER>9 Bytes, about 157 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          57 seconds loaded<PHONE_NUMBER> Bytes, about 18 MB/s
stream load parquet:          29 seconds loaded 861443392 Bytes, about 28 MB/s
insert into select:          67.5 seconds inserted 10000000 Rows, about 148K ops/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230630145102_clickbench_pr_170623.html


Pretrained Weights

Would it be possible for you to share pretrained weights for resnet GN? Transferring directly from the pytorch BN weights gives very bad accuracy.

I don't see why would you expect a different model (with BN to work).
Pretrained weights in tensorflow was mentioned in https://github.com/ppwwyyxx/GroupNorm-reproduce/tree/master/ImageNet-ResNet-TensorFlow

The instructions are now updated to include the command to evaluate the above model on ImageNet.

Why would weights of a different model (with BN) work?

I wasn't expecting them to, but they're the only weights I could find. Especially for smaller models like ResNet18.
Do you happen to have pytorch weights? I can convert the TF weights if necessary, but I was trying to follow your Pytorch code.

There are also caffe2 weights at https://github.com/facebookresearch/Detectron/tree/master/projects/GN#pretrained-models-with-gn, but I don't have pytorch weights at the moment.

A pytorch model was uploaded at https://github.com/ppwwyyxx/GroupNorm-reproduce/releases/tag/v0.1


Remove Typescript 2.7.2 dependency

Hello,
regardless which typescript version is configured in the node project, typedoc currently use a hardcoded version 2.7.2 that is be installed right into the module folder.
You can see it in the first line:
Using TypeScript 2.7.2 from \node_modules\typedoc\node_modules\typescript\lib
As result it breaks for typescript feature higher than 2.7.2:
for example Error: /node_modules/@types/bluebird/index.d.ts(318)
'>' expected.
Can you remove the hard version dependency and use the version that is provided by the project?

This has already been completed. Just need to cut a new release.