Twitter login broken

Problem
After log in with Twitter, it redirects to wrong url. It should be healthsites.io but it's redirect to healthsites only. I have done some inspections:

In Twitter Apps
I thought in the twitter apps, it sets the redirect url. I have even created new app, but it still doesn't give the right redirect URL
In nginx configuration
Nothing is strange. All is set to healthsites.io. Nothing is mentioning healthsites only
In django code
I have checked my python-social-auth document, and there is nothing that mention set redirect url

This is working in my local deploy...

Following the tutorial from python-social-auth here: http://python-social-auth.readthedocs.org/en/latest/backends/twitter.html.
I set the call back URL to http://staging.healthsites.io/complete/twitter. But it will give you http://healthsites/complete/twitter (without the .io). It will give you an error, of course.
I also tried to hack the python-social-auth like in https://github.com/omab/python-social-auth/issues/713 (because, once I got 401 also). But, it doesn't give a better result.

What was the old configuration? This was working. Can we see the old code?

http://stackoverflow.com/questions/32460547/python-social-auth-redirect-to-wrong-url

On marks old site we got this though I don't know if it is relevant.
SSLHandshakeError at /accounts/twitter/login/
[Errno 1] _ssl.c:504: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed
Request Method:	GET
Request URL:	http://www.konektaz.info/accounts/twitter/login/
Django Version:	1.4.2
Exception Type:	SSLHandshakeError
Exception Value:
[Errno 1] _ssl.c:504: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed
Exception Location:	/home/webdeploy/sites/konekta.info/env/local/lib/python2.7/site-packages/httplib2/__init__.py in connect, line 1037
Python Executable:	/home/webdeploy/sites/konekta.info/env/bin/python
Python Version:	2.7.3
Python Path:
['/home/webdeploy/sites/konekta.info/repo',
 '/home/webdeploy/sites/konekta.info/env/bin',
 '/home/webdeploy/sites/konekta.info/env/local/lib/python2.7/site-packages/distribute-0.6.24-py2.7.egg',
 '/home/webdeploy/sites/konekta.info/env/local/lib/python2.7/site-packages/pip-1.1-py2.7.egg',
 '/home/webdeploy/sites/konekta.info/env/src/django-mptt',
 '/home/webdeploy/sites/konekta.info/env/lib/python2.7',
 '/home/webdeploy/sites/konekta.info/env/lib/python2.7/plat-linux2',
 '/home/webdeploy/sites/konekta.info/env/lib/python2.7/lib-tk',
 '/home/webdeploy/sites/konekta.info/env/lib/python2.7/lib-old',
 '/home/webdeploy/sites/konekta.info/env/lib/python2.7/lib-dynload',
 '/usr/lib/python2.7',
 '/usr/lib/python2.7/plat-linux2',
 '/usr/lib/python2.7/lib-tk',
 '/home/webdeploy/sites/konekta.info/env/local/lib/python2.7/site-packages',
 '/home/webdeploy/sites/konekta.info/env/local/lib/python2.7/site-packages/PIL',
 '/home/webdeploy/sites/konekta.info/repo']

I would like to compare the server set ups for konektaz.info and healthsites.io
The issue we're currently experiencing on konektaz concerns a failed certificate whereas the healthsites.io issue described above is different. Oauth is directing back to 'healthsites' not 'healthsites.io'
Therefore, my suggestion is to compare the server set ups of the two and look for the difference.

I just installed latest version on my local VM. Twitter auth is working. I presume there is some issue with prod configuration.

Please read the log messages from gitter - as I have explained there I believe this is caused by the reverse proxy in front of the site.
One option (sledgehammer fix) would be to deploy on a server without a reverse proxy (which means the server would only be able to run a single site easily). But I believe we can fix the reverse proxy issue - I just need to have a bit of time to look into it.

@timlinux the reverse proxy you are referring is not the part of the project. Therefore we're not able to replicated it. The develop version at http://healthsites.<IP_ADDRESS>.xip.io is also behind the reverse proxy which looks like this:
server {
  listen 80;
  server_name cittadelcapo.com;
    location / {
      access_log off;
      proxy_pass http://localhost:49361;
      proxy_set_header X-Real-IP $remote_addr;
      proxy_set_header Host $host;
      proxy_set_header X-Forwarded-For $proxy_add_x_forwarded_for;
    }
}

Can you share your configuration so we can test it?
Apart from this nginx config everything else is running as prod_docker on http://healthsites.<IP_ADDRESS>.xip.io therefore I'd say this ticket is invalid.

@markherringer should we move to done?


Encodings requiring padding are not provided

I'm not sure whether you consider this problem, or just something to document, but I realised that while RFC 8771 section 3.1-<IP_ADDRESS> explains that encodings may include padding to allow more combinations of characters to match the size of IP addresses.
The current implementation only provides encodings that require no padding.
This limits the number of encodings provided, and in some cases reduces the available confusion level from the maximum possible.
Related: my Rust implementation has the same issue: https://gitlab.com/andybalaam/rust-i-dunno/-/issues/1

I'll consider it a problem and see if I can look into doing that at some point. I remember reading that now when I initially read the RFC but I missed it when doing my implementation. The spec does say "MUST" and I don't think it'll be too bad since combinations where the end is padded can be added by modifying lines 76 - 77 (and I don't think lines 81 - 82 need to change since if I remember right, that condition would only apply to areas before the end of the final address bitstring and I'm reading the RFC to say the padding bits go after all of the address bits).

Yeah, I also read it as padding goes after the meaningful bits.  I take it as somewhat optional, since the I-DUNNOs produced by the existing code are perfectly valid, just not selected from the full available range.
I do note that the number of possble encodings will increase vastly when this is implemented.


Replacement for 2N2007E MOSFET

Hi. I am unable to find stock availability of this component, 2N2007E mosfet from Farnell (Part #1894721) as well as from DigiKey. Can you suggest any replacement for this component?.
TIA,
Prakash.

Hi Prakash,
Thanks for noticing, indeed NXP stopt producing it but there are plenty of others. Please use this one instead: Farnell 2317616
I will update the documentation.
Best regards,
Tijn


improve easing type for autocomplete

currently the type for easing "linear" | "ease-in" | "ease-out" | "ease-in-out" | string doesn't let typescript suggest the possible values. By using ({} & string) typescript shows the values while also alllowing any string as indended originally
before

after

This is cool, but I don’t understand it — can you point me to some kind of reference on how/why this works @sebastianwd?

@justin-schroeder hello, this is a small hack I got from Matt Pocock's youtube channel, I don't remember the exact video but it's also mentioned in the "Loose Autocomplete" section of this one https://youtu.be/h5ymIiux6NU?t=273


How to fix ohmyzsh startup error

Here is the error when i open my console and oh-my-zsh as default shell
zsh: done       echo |
zsh: abort

Can you post your .zshrc file?

I see this while using termux.

Without more information we can't do anything. Does this happen when running zsh manually as well or only when first opening the console? What are the contents of your .zshrc file?

Hi, I am also experiencing the same thing. I experienced this in the termux application, first I installed zsh, this is fine. However, when I install oh-my-zsh, and then I change the default shell to zsh. And oh-my-zsh displays a message like:
zsh: done echo | zsh: abort

And then, I quit zsh, then I re-entered zsh, and zsh displays the message:
# zsh
zsh: done echo |
zsh: suspended (signal)
[6] + done echo | abort

Please help.

How did you set zsh as the default shell?

I only press "Y" when oh-my-zsh asks a question.

In my tests the install script can't actually set zsh as the default shell, I can't really reproduce the error.


Prometheus data from tests should be retained

I've sent https://github.com/kubernetes/perf-tests/pull/518 to add experimental snapshot support (GCP only). It's hacky but should work as a temporary solution for now.
Long term, we should investigate other options and rewrite this to something better, e.g. exporting Prometheus data from tests to some db or maybe there is already existing tool that we could use that addresses our requirements.

/lifecycle-frozen

/remove-lifecycle stale

/remove-lifecycle stale

/remove-lifecycle stale
/lifecycle frozen

I believe we can close this one.
While the original idea was to use Cortex, we ended up implementing a "snapshot-base" solution that seems to be working nicely.
Putting it in other words, as of now the Prometheus test data is retained for all continuous tests. The goal of this issue has been achieved.
/close


Ensure Case-Sensitive Columns are Preserved in SQL Check Extractors

Purpose:
In certain databases (cough cough Snowflake cough cough), column names have enforced cases, meaning that something like the SQLColumnCheckOperator may not emit column lineage if the column name in the column mapping isn't the right case.
Proposed implementation
By moving the [build_column_check_facets](https://github.com/OpenLineage/OpenLineage/blob/main/integration/airflow/openlineage/airflow/utils.py#L374) function into the SqlColumnCheckExtractor's [get_input_facets](https://github.com/OpenLineage/OpenLineage/blob/main/integration/airflow/openlineage/airflow/extractors/sql_check_extractors.py#L80) method, the SnowflakeExtractor's [is_uppercase_names](https://github.com/OpenLineage/OpenLineage/blob/main/integration/airflow/openlineage/airflow/extractors/snowflake_extractor.py#L22) property can be used to ensure correct column name capitalization. And this can use the property across all extractors to ensure this problem never, ever, ever happens again.

Also I can make this change.

@denimalpaca makes sense 👍


Access and test environment

@VRagulina  I can move this into In Progress. I'd like to start doing some UAT this week now that the test environment is accessible. The description needs filling out.

FYI @VRagulina @aanand-gsa, I got a head start on this as a first pass. Still more to do, but here is where and how I'm tracking items. It's not test case-based, but a start nonetheless. Thanks!


how  to identify the monomer and HOR sequence

centroFlye assumes that the set of monomers and an HOR are given. However, we have been recently developing tools for automatic derivation of monomers and HORs. Please write me an email<EMAIL_ADDRESS>if you wish to discuss that.


Add files via upload

@VksnJokmanovic Please provide us with a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.
You can provide us with any of the following:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

Apologies, but I would like to know whether the domain was successfully accepted or not.

@philipto
I made another error as well, sorry for the inconvenience. Yes, udg.edu.me is the intended domain, I misunderstood how the directories work and I thank you for correcting me.

@VksnJokmanovic No problem, it's easy to fix. All the additional checks were because of my attempts to find any proof of official recognition of udg.me. Thank you for your patience while waiting for a review and processing your request. Pull request merged.


[Volunteer #2892] Synchronize sorting of plugins.

Synchronize sorting of plugins. #2892
test case:

update plugin sort in admin, the order of ShenyuWebHandler’s plugins have changed.
After the plug-in sequence changes. After restart, the sequence is the same.
When the plugins of ShenyuWebhandler changes, the ongoing requests will not be affected.

testing procedure：

admin use mysql and run admin.
Run gateway ShenyuBootstrapApplication.
Run test demo of ShenyuTestHttpApplication.
now send a request to gateway.
curl --location --request POST 'http://localhost:9195/http/order/save?a=1' \ --header 'Content-Type: application/json' \ --header 'Cookie: JSESSIONID=302C524F06306C42AE4DB7A7D9716BF4' \ --data-raw '{    "id":666,    "name": "hello world" }'
debug ShenyuWebHander handler method, we find the sign plugin is before jwt plugin.
now we change jwt plugin order from 20 to 35.
now repeat step 4. we can find the sign plugin is after jwt plugin.
now we test case 2. stop and restart gateway ShenyuBootstrapApplication.
now repeat step 4. we can find the sign plugin is after jwt plugin.This indicates that the plug-in sequence has a persistent effect.
For concurrent modification of plug-in order, we use copy on write.

Event notification mechanism
problem：
the module of shenyu-plugin not dependent shenyu-web, so when happen sort plugin event,we can not operate directly shenyuWebHandler。
terms of settlement:
I used spring event publish function.

Codecov Report

Merging #2904 (632662a) into master (d4d52fd) will decrease coverage by 0.47%.
The diff coverage is 13.04%.

@@             Coverage Diff              @@
##             master    #2904      +/-   ##
============================================
- Coverage     60.00%   59.52%   -0.48%
+ Complexity     4462     4428      -34
============================================
  Files           692      693       +1
  Lines         19978    20017      +39
  Branches       1782     1787       +5
============================================
- Hits          11988    11916      -72
- Misses         6983     7091     +108
- Partials       1007     1010       +3

Impacted Files
Coverage Δ

.../java/org/apache/shenyu/common/dto/PluginData.java
0.00% <0.00%> (ø)

...ache/shenyu/plugin/base/cache/SortPluginEvent.java
0.00% <0.00%> (ø)

...enyu/sync/data/http/refresh/PluginDataRefresh.java
100.00% <ø> (ø)

...rg/apache/shenyu/web/handler/ShenyuWebHandler.java
26.66% <0.00%> (-6.67%)
:arrow_down:

.../plugin/base/cache/CommonPluginDataSubscriber.java
69.79% <21.42%> (-8.26%)
:arrow_down:

...g/apache/shenyu/admin/transfer/PluginTransfer.java
100.00% <100.00%> (ø)

...pringboot/starter/gateway/ShenyuConfiguration.java
88.88% <100.00%> (ø)

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
68.75% <0.00%> (-31.25%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
70.00% <0.00%> (-13.34%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
42.85% <0.00%> (-12.70%)
:arrow_down:

... and 33 more

Continue to review full report at Codecov.

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

pls refactor pr tilte

pls refactor pr tilte

I got. done


Nested Parquet read failure

Issue found whilst testing https://github.com/jorgecarleitao/arrow2/pull/1007.  Testing the Parquet generated by the following Python script on that branch fails on reading the Parquet file.
Python code to generate test Parquet:
import numpy as np
import pandas as pd
import pyarrow as pa
import pyarrow.parquet as pq

d1 = {
    "id": pd.Series(['t1', 't2']),
    "prices": pd.Series([
        [
            {
                "currency": "GBP",
                "value": 3.14,
                'meta': [
                    {'loc': 'LON'}
                ]
            }
        ],
        [
            {
                "currency": "USD",
                "value": 4.14,
                'meta': [
                    {'loc': 'NYC'}
                ]
            }
        ]
    ]),
    "bids": pd.Series([
        [
            {
                "currency": "GBP",
                "value": 3.04,
                'meta': [
                    {'loc': 'LON'}
                ]
            }
        ]
    ], dtype='object')
}

df = pd.DataFrame(d1)

list_type = pa.list_(
    pa.struct([
        ('currency', pa.string()),
        ('value', pa.float64()),
        ('meta', pa.list_(
            pa.struct([
                ('loc', pa.string())
            ])
        ))
    ]))

schema = pa.schema([
    ('id', pa.string()),
    ('prices', list_type),
    ('bids', list_type)
])

table = pa.Table.from_pandas(df, schema=schema)
filename = '/tmp/two_level_nested.parquet'
pq.write_table(table, filename)

# Read the Parquet to check
expected_table = pq.read_table(filename).to_pandas()
print(expected_table.to_string())

Rust code (against https://github.com/jorgecarleitao/arrow2/pull/1007)
use arrow2::{
    array::*,
    chunk::Chunk as AChunk,
    io::parquet::write::{FileWriter, RowGroupIterator, WriteOptions},
};
use arrow2::{datatypes::Schema, io::parquet::read::FileReader};
use parquet2::encoding::Encoding;
use std::{fs::File, sync::Arc};

type Chunk = AChunk<Arc<dyn Array>>;
type Result<T> = arrow2::error::Result<T>;

pub fn main() -> Result<()> {
    let filename_to_read = "/tmp/two_level_nested.parquet";
    let filename_to_verify = "/tmp/two_level_nested_verify.parquet";

    // Read the Parquet created by PyArrow
    let (schema, chunks) = read_parquet(filename_to_read);

    // Write a new Parquet file from what what we just read
    write_parquet(filename_to_verify, schema, chunks)?;

    // Read what we just wrote
    let (_schema, _chunks) = read_parquet(filename_to_verify);

    // Compare the original Parquet file and the one we wrote
    compare(filename_to_read, filename_to_verify)?;

    Ok(())
}

fn compare(filename_to_read: &str, filename_to_verify: &str) -> Result<()> {
    let (_, expected_chunks) = read_parquet(filename_to_read);
    let expected_chunks = expected_chunks
        .into_iter()
        .map(|res| res.unwrap())
        .collect::<Vec<_>>();

    println!("Expected chunks: {:#?}", expected_chunks);

    let (_, written_chunks) = read_parquet(filename_to_verify);
    let written_chunks = written_chunks
        .into_iter()
        .map(|res| res.unwrap())
        .collect::<Vec<_>>();

    println!("Chunks written: {:#?}", written_chunks);

    assert_eq!(expected_chunks, written_chunks);

    Ok(())
}

fn read_parquet(filename: &str) -> (Schema, Vec<Result<Chunk>>) {
    println!("Reading {}", filename);
    let reader = File::open(filename).unwrap();
    let reader = FileReader::try_new(reader, None, None, None, None).unwrap();
    let schema: Schema = reader.schema().clone();
    // println!("schema: {:#?}", schema);
    let mut chunks = vec![];
    for chunk_result in reader {
        chunks.push(chunk_result);
    }
    (schema, chunks)
}

fn write_parquet(filename: &str, schema: Schema, chunks: Vec<Result<Chunk>>) -> Result<()> {
    println!("Writing {}", filename);
    let options = WriteOptions {
        write_statistics: true,
        version: parquet2::write::Version::V2,
        compression: parquet2::compression::CompressionOptions::Snappy,
    };

    let iter = chunks.into_iter();
    let encodings = schema.fields.iter().map(|_| Encoding::Plain).collect();
    let row_groups = RowGroupIterator::try_new(iter, &schema, options, encodings)?;

    let file = std::fs::File::create(filename)?;
    let mut writer = FileWriter::try_new(file, schema, options)?;
    writer.start()?;
    for group in row_groups {
        writer.write(group?)?;
    }
    writer.end(None)?;
    Ok(())
}

Results in
Reading /tmp/two_level_nested.parquet
thread 'main' panicked at 'called `Option::unwrap()` on a `None` value', /home/ahmed/.cargo/git/checkouts/arrow2-8a2ad61d97265680/b6a516c/src/io/parquet/read/deserialize/mod.rs:272:44

@jorgecarleitao what's the required at this point in the example that's needed?
    let encodings = schema.fields.iter().map(|_| Encoding::Plain).collect();
    let row_groups = RowGroupIterator::try_new(iter, &schema, options, vec![encodings])?;

Using the above, and master after the merge, I still get the assertion failure during the write:
thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `3`,
 right: `1`', /home/ahmed/.cargo/git/checkouts/arrow2-8a2ad61d97265680/7cc874f/src/io/parquet/write/pages.rs:210:5

I fielded #1018, which is what we need here - we need to transverse the fields, get the leaf columns, and create encodings with it.
I ran the example, but the writing is still not there for such nesting - I am looking into it

@jorgecarleitao thanks a lot for the explanation.


Add golang library to main repository

What type of PR is this?
/kind feature

Any specific area of the project related to this PR?

What this PR does / why we need it:
This adds a generic golang library to interact with kilt definitions. This will be used by all runtimes.

kilt-lib: adding the golang library

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign admiral0
You can assign the PR to them by writing /assign @admiral0 in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

7f23694 Publish kilt golang library

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: fntlnz
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [fntlnz]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 0c11bd14f2a2151b49b6a9b2d820a24cbb794f3e


⚠ BroadcasTheNet has degraded performance

In eb880b1, BroadcasTheNet ($BTN) experienced degraded performance:

HTTP code: 502
Response time: 29084 ms

Resolved: BroadcasTheNet performance has improved in 598047e after .


MarkerManager functions should gracefully return if scene loading failed

Even though this was a side effect of commeting out scene loading in the tizen plugin, it does make sense for marker
manager to not crash (uninitialzed members and nullptrs) when a scene loading fails.

@blair1618 can you give this a look. Thanks

Updated with requested changes.


VSM improvements and fixes

use EVSM (exponential VSM), helps a lot with light bleeding
don't apply shadow biases with VSM
improve VSM user settings: added more controls, expose to java
minor code cleanups:

rename some structures
remove unused parameters
remove unused uniforms

I marked the PR as draft because I think I need to rework the VsmShadowOptions so the variance/light-bleed parameters are per-light.

I marked the PR as draft because I think I need to rework the VsmShadowOptions so the variance/light-bleed parameters are per-light.

It's actually pretty involved to move those settings per-light, so if we do it, it'll be in a separate PR.

Can we even move those settings per light? We would have to store more data in the froxels (another case were deferred rendering shines :(). We would also want to keep a global set of options to apply by default to all the lights.

Can we even move those settings per light? We would have to store more data in the froxels (another case were deferred rendering shines :(). We would also want to keep a global set of options to apply by default to all the lights.

I don't think so, it's just a couple of uniforms for the the light-bleed and min variance. I'm not suggesting to be able to chose PCF/VSM per light.
We can't  move mipmaping/anisotropy per-light either because we only have one sampler for all the shadow maps.

I don't think so, it's just a couple of uniforms for the the light-bleed and min variance. I'm not suggesting to be able to chose PCF/VSM per light.

It's two uniforms per light though, so they have to go in the froxels. I know we're not talking about choosing PCF/VSM per light.

I don't think so, it's just a couple of uniforms for the the light-bleed and min variance. I'm not suggesting to be able to chose PCF/VSM per light.

It's two uniforms per light though, so they have to go in the froxels. I know we're not talking about choosing PCF/VSM per light.

It's not needed per-froxel, these values are not different per froxel. But in fact we don't even need them per-light, because we only support a limited number of shadows -- so we just need these 2 uniforms per shadowmap, and we already have a UBO per spotlight shadow, so there is a precedent for doing this.

Yes, sorry I meant "light record" not "froxel". But yeah if it's only for the shadow maps that's easier.

I added a CL that fixes the "receivers are also casters" with VSM in a more robust way.


Support VK_MVK_ios_surface and VK_MVK_macos_surface in vulkan.h

MoltenVK extensions VK_MVK_ios_surface and VK_MVK_macos_surface are now available in the standard Khronos vulkan.h header file.
This PR modifies atw_vulkan.c to use these extensions, and their correct extension numbers, directly from vulkan.h.

Can I get some feedback on why this PR is causing issues with the AppVeyor and Travis builds and tests, please?
The issues do not seem to be related to the changes in this PR.


[zh] Resync pod-failure-policy.md

Sync with the en upstream PR #37242.
Changes are here:
https://github.com/kubernetes/website/pull/37242/files#diff-070b1cbe33e1868588a20382c4af19b8266e9a0c38be652bb5eb27750b619fef

/lgtm

/approve


Carats point in different/inconsistent directions for links and comments

When collapsed, the carat for links points up and the carat for comments points to the right. Shouldn't we make these consistent? Thx!

See: https://openshift.io/openshiftio/openshiftio/plan/detail/1040


Decode transaction from string, fix transaction serialization bug

Hey @Tougee could you please check the linter again? Looks like you fixed the issues in the test files but not in the main source set. You can run ./gradlew build to execute all the checks locally. Also, it applies standard IntelliJ IDE formatting so you can just reformat it in the IDE and that should be enough.

Thank you @Tougee. The changes are now available in v0.4.2


Incomplete normalization

Hi guys,
Let's say that I wanna normalise that string:
"Protein Recommendations for Bodybuilders: In This Case, More May Indeed Be Better.

Without extra Cucco setup (normalizations) I received:
"Protein Recommendations Bodybuilders Case".

With extra Cucco setup:
normalizations = [
	'remove_extra_whitespaces',
	'remove_accent_marks',
	'remove_stop_words',
	('replace_hyphens', {'replacement': ''}),
	('replace_punctuation', {'replacement': ''}),
	('replace_symbols', {'replacement': ''}),
]

I received:
"Protein Recommendations Bodybuilders Case Better"

My question is: where is the rest part of the string?

Hi @Katharsis,
First of all thank you very much for taking time on reporting a possible bug. I really appreciate it. This is the way to improve cucco.
I've been checking the behavior you comment and in this case the output is the expected one. In the first execution, the one with default normalizations, normalizations are applied this way:

Protein Recommendations for Bodybuilders In This Case More May Indeed Be Better
(punctuation replaced).
Protein Recommendations for Bodybuilders In This Case More May Indeed Be Better (extra white spaces removed).
Protein Recommendations for Bodybuilders In This Case More May Indeed Be Better (symbols removed).
Protein Recommendations Bodybuilders Case (stop words removed)

Here you can see the execution:
$ cucco normalize 'Protein Recommendations for Bodybuilders: In This Case, More May Indeed Be Better.'
Protein Recommendations Bodybuilders Case

The normalization you propose would look like this:

Protein Recommendations for Bodybuilders: In This Case, More May Indeed Be Better. (whitespaces removed)
Protein Recommendations for Bodybuilders: In This Case, More May Indeed Be Better. (accent marks removed)
Protein Recommendations Bodybuilders: Case, Better. (stop words removed)
Protein Recommendations Bodybuilders: Case, Better. (hyphens replaced)
Protein Recommendations Bodybuilders Case Better (punctuation replaced)
Protein Recommendations Bodybuilders Case Better (symbols replaced)

$ cat config.yaml
normalizations:
  - remove_extra_whitespaces
  - remove_accent_marks
  - remove_stop_words
  - replace_hyphens
  - replace_punctuation
  - replace_symbols
$ cucco -c config.yaml normalize 'Protein Recommendations for Bodybuilders: In This Case, More May Indeed Be Better.'
Protein Recommendations Bodybuilders Case Better

Note that in the last case I'm omitting the values for replacement as the value you set is actually the default value.
So, I'm closing this issue as I don't think is a real bug. If I didn't understand you or if you think the behavior should be different, please, feel free to comment and reopen it.
Happy normalization ;)


Odd "dark" areas of statusline, unable to remove/control

Thanks for this excellent plugin @glepnir!
I've finally recreated my old vimscript-based statusline over to lua via galaxyline..
I have one final odd thing; for some reason there's this dark/black background area to the left of the "right" section (looks to be hex #1c272d).
Here's a screenshot to illustrate the area at the bottom right:

Here's a link to my current statusline setup: https://github.com/megalithic/dotfiles/blob/main/nvim/.config/nvim/lua/mega/statusline.lua
If this is not the right place to ask about this, please direct me to a better spot!
Thanks again!

Ugh, ok so it's the space taken up by the diagnostics.. will just add a condition for those.. thanks!


Add greadle tasks for Angular

Try to run:

build
dev
lint
tests

Added github actions for angular, right now i don't need set up gradle for angular project.


[#156224014] Don't disable the uaa.require_https flag.

What
The default is true upstream but we've set it to false in [1].
If require_https is disabled the Current-User cookie won't have the secure flag which came up in our latest pentest.
There is no reason to disable this anymore. Whatever caused issues previously it doesn't seem to be present anymore.
[1] https://github.com/alphagov/paas-cf/commit/9a4dda67ead70bd39d811cc373d8fd10dbb9df44
How to review

Deploy your CF from this branch

Make sure the tests pass. Log in to UAA on the web.

Who can review
Not me.

This doesn't actually fix the secure flag for the Current-User cookie. It does fix it for the X-Uaa-Csrf cookie though.

I've updated the commit message to reference the correct cookie.


+= note on enabling BOOT_UART on Raspi4

The documentation says that this is to enable debug output on that UART. Not 100% sure what that means. Does that mean videocore output?
I never enabled this on my pi4 explicitly. So either it was enabled by factory settings, or it might not be needed in my case 🤔

I'm also unsure, unfortunately. I also get some garbled output after MiniPush started and before it pushes the binary. Maybe this is the debug output... the question is why it's not working without this.
Could you please check vcgencmd bootloader_config on your pi4? Maybe it is enabled on yours? For mine, I had to set BOOT_UART=1 to make the chainloader work.

Okay. Will check.
I currently don't have a ready-to-go Linux image available, so it might take some days until I get to it.

I just undid the change to test it and it works (without the garbled output). Strange, as I did not change other things. Before, my kernel8.img was overwritten (and zero sized eventually) every time I would run the MiniPush and it yielded a Protocol Error.

The Pi4 behaves strange in different ways for me too. The Pi3 seems much more reliable.
For example, I was unable to fix the frequency of the MiniUart. It would always dynamically scale it during runtime, no matter what many config.txt I had tried to not do that.
Typical first instance bugs of a new HW generation I presume =/


Add GUI and tidy up

A lot of tidying up as well as added a dedicated GUI with sliders and event listeners for changes and actions.
I'll help @skal65535 with the rest of his work after this. Exciting times!

That is awesome! Thanks
I'm a bit discouraged now to work on the C++ version when seeing the JavaScript version advancing so swiftly!

very nice! The UI is very clean and much more polished than my cheap attempt at CSS hacking ;)
Few things i noticed:

we lost the 'r' and 'o' key to generate new random parameters and reset, respectively. Although we it's still available through the UI, having it as key stroke was useful
the hash value in the URL seems to be lagging one behind
changing the hash in the menu doesn't trigger a corresponding new set of parameters to be drawn
i would be nice to have a little color dot or icon to remind the actual color as visual help (when trying to play with pairs of particles for instance, it's easier to find the corresponding pair of params on the UI if you have a visual color help imho)

For the particles, what's missing is a per-particle radius (instead of the global cutOff param) which produced interesting pattern, as well as the main loop fix that updates the velocity separately from position. I can have a look at that later, leaving the UI to Dan's expertise...


How to get my backend address?

Firstly，Thanks for you share!
And now I try to use it but have some problem.
How to get my backend address?
try to search in internet

Hi YIRU69,
Do you have PHP running on your Server or PC?
When you host local, do the following:
$ cd ~/ob2ki-directory // go to the directory where you saved the files and index.php is located and open the terminal.
$ php -S localhost:8000 // run this command! now your local backendadress is http://localhost:8000.
Don't forget to install the gd lib.
Greetings,
Simeon

yeah, I complete install PHP and Apache24, and also install the gd lib
And now I don't konw how to do next. 0.0

I haven't ~/ob2ki-directory , Can I create file folder?
How can  I create "index.php"

Thanks to help me a lot!

Just download the files from the repo.
In the root folder is the index.php file.
Put all the files from the repo in one folder, and than start the command.
I hope this helps.
Thanks for your patience.
Greetings!

Look it many error...

Warning: Undefined array key "HTTP_ORIGIN" in D:\PHP\Obsidian2Kindle\index.php on line 11

Deprecated: Optional parameter $verb declared before required parameter $ip is implicitly treated as a required parameter in D:\PHP\Obsidian2Kindle\MOBIClass\Http.php on line 16

Warning: Undefined array key "text" in D:\PHP\Obsidian2Kindle\index.php on line 28

Deprecated: str_replace(): Passing null to parameter #3 ($subject) of type array|string is deprecated in D:\PHP\Obsidian2Kindle\parsedown.php on line 43

Warning: Undefined array key "Bilder" in D:\PHP\Obsidian2Kindle\index.php on line 36

Warning: Undefined array key "toc" in D:\PHP\Obsidian2Kindle\index.php on line 66

Warning: Undefined array key "title" in D:\PHP\Obsidian2Kindle\index.php on line 77

Warning: Undefined array key "author" in D:\PHP\Obsidian2Kindle\index.php on line 78

Warning: Undefined array key "author" in D:\PHP\Obsidian2Kindle\index.php on line 79

Warning: Undefined array key "title" in D:\PHP\Obsidian2Kindle\index.php on line 87

Deprecated: imagettftext(): Passing null to parameter #8 ($text) of type string is deprecated in D:\PHP\Obsidian2Kindle\index.php on line 87

Warning: Undefined array key "author" in D:\PHP\Obsidian2Kindle\index.php on line 88

Deprecated: imagettftext(): Passing null to parameter #8 ($text) of type string is deprecated in D:\PHP\Obsidian2Kindle\index.php on line 88

Warning: Undefined array key "toc" in D:\PHP\Obsidian2Kindle\index.php on line 93

Deprecated: strlen(): Passing null to parameter #1 ($string) of type string is deprecated in D:\PHP\Obsidian2Kindle\MOBIClass\MOBIFile.php on line 34

Warning: Undefined array key "title" in D:\PHP\Obsidian2Kindle\index.php on line 132

Fatal error: Uncaught Error: Call to undefined function mb_str_split() in D:\PHP\Obsidian2Kindle\MOBIClass\RecordFactory.php:41 Stack trace: #0 D:\PHP\Obsidian2Kindle\MOBIClass\MOBI.php(162): RecordFactory->createRecords('<html lang='de'...') #1 D:\PHP\Obsidian2Kindle\MOBIClass\MOBI.php(182): MOBI->preparePRC() #2 D:\PHP\Obsidian2Kindle\index.php(132): MOBI->save('.mobi') #3 {main} thrown in D:\PHP\Obsidian2Kindle\MOBIClass\RecordFactory.php on line 41

and try to run plugin

I have a little software experience , maybe I should give up...
And I should to say
Thank you very much! Nice to you reply


Enable post conditions in the declarative pipeline

Feature Request
Enable Post conditions:

https://www.jenkins.io/doc/book/pipeline/syntax/#post-conditions

For instance:
pipeline {
  agent any
  stages {
    stage ('Hello') {
      steps {
        echo "hello World"
      }
    }
  }
  post {
        always {
            sh 'echo "always"'
        }
        cleanup {
            sh 'echo "cleanup"'
        }
    }
}

Produces:

You mean that you want to capture a "Post Condition" wrapping span, correct?


Proposal

Avoid using abbreviations or function names like MSE and RandomForestRegressor in the proposal. Otherwise, you should define them.

Other than that, everything else looks fine and you have done a great job in this assignment in all of the tasks.

Hi @hsmohammed,
Thank you very much for your feedback and your kind words! We will improve the README.md file accordingly.

Hey Guys @huan-ds @charlessuresh ,
Here are my thoughts to improve the README.md file. Please record your thoughts and suggestions in this thread as well.

Under Section Usage -> Step2, better to say Run the file to see the figures and analysis generated from the EDA process.
Add the full name of EDA, MSE etc at the place where they appear for the first time in README.
Better to stand out some machine learning model names by either making them bold or and tick marks around them i.e. Ridge, RandomForestRegressor  etc.

@chuangw46 I've changed  READ.me file.  Deleted some content only related to Milestone 1.  But I have a small format issue, please help me check why there is no newline here.


Spec wal size

This PR specs wal size to be less than the 1G default

Work in this PR landed as part of another PR.


Fix VideostreamGDNative audio buffer handling

When working on gdnative video decoder (https://github.com/KidRigger/godot-videodecoder), I spot this audio buffer issue. Fix the audio buffer start when there are previous remains.

CC @kidrigger

Thanks for continuing this @starryalley
Good catch.

Thanks!


onclick window.location links are not being picked up

This is a valid link, but the crawler ignores this link. This should be fixed.

This requires a JavaScript runtime.
https://github.com/stevenvachon/broken-link-checker/projects/2#card-2200507

Closing for now as this is not currently on track to be implemented.


[SPARK-17905] [SQL] [TEST] Added test cases for InMemoryRelation

What changes were proposed in this pull request?
This pull request adds test cases for the following cases:

keep all data types with null or without null
access CachedBatch disabling whole stage codegen
access only some columns in CachedBatch

This PR is a part of https://github.com/apache/spark/pull/15219. Here are motivations to add these tests. When https://github.com/apache/spark/pull/15219 is enabled, the first two cases are handled by specialized (generated) code. The third one is a pitfall.
In general, even for now, it would be helpful to increase test coverage.
How was this patch tested?
added test suites itself

Test build #66884 has started for PR 15462 at commit b5e9866.

Test build #66884 has finished for PR 15462 at commit b5e9866.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66884/
Test FAILed.

Merged build finished. Test FAILed.

Jenkins, retest this please

Test build #66889 has started for PR 15462 at commit b5e9866.

Test build #66889 has finished for PR 15462 at commit b5e9866.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66889/
Test PASSed.

retest this please

Test build #69025 has started for PR 15462 at commit b5e9866.

Test build #69025 has finished for PR 15462 at commit b5e9866.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69025/
Test PASSed.

Merged build finished. Test PASSed.

Test build #69138 has started for PR 15462 at commit 42eade8.

Test build #69138 has finished for PR 15462 at commit 42eade8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/69138/
Test PASSed.

@andrewor14 thank you for your review. Could you please see it again?

LGTM, merging into master 2.1 thanks.

@kiszk is there a JIRA associated specifically with adding tests for InMemoryRelation?

Thank you for pointing out an JIRA issue. I made a correction to a JIRA entry.

Done. Corrected the JIRA. Thanks!


key6 unmangling

Discussion of key6 is banned in discord and the subreddit so I'm posting it here for the community
laverdet/pcrypt.

@laverdet heaps of thanks for sharing your work and letting us know! was actively looking for a decryption way for educational puposes and deeper understanding of the inner workings of the native code <3 Cheers!

https://github.com/rastapasta/pokemon-go-mitm/blob/master/example.unkown6.coffee

@laverdet do you maybe know where the Signature.session_hash is coming from?

No idea how that is being generated, no.

It only changes on app restart but the source is unknown.

Any way to fix this?
/pokemon-go-mitm-node/node_modules/pcrypt/pcrypt.js:88
return Buffer.from(outputBuffer).slice(0, output8.length - output8[output8.length - 1]);
^
TypeError: this is not a typed array.
at Function.from (native)

@Lennix Buffer.from is a node >= 6.0.0 interface, see laverdet/pcrypt#1 for workaround


Feature request: Restart openvpn when it fails and exits

What's the feature?

Restart openvpn from within the container if it disconnects or exits

Why do you need this feature?

That may allow to keep linked containers connected as the container would not fully restart if openvpn crashes. This should be in a beta testing image tag for a while as it goes against Docker principles of having the Docker daemon manage the restart lifecycle of the container.

Extra information?

This should be tested, especially for people when disconnection problems with linked containers such as @frepke (without OPENVPN_ROOT=on) or @gurabli
The Docker tag will be :restart

Refers to #140

Quentin,
For me the problem isn't a disconnection problem with linked containers due to a problem with the vpn-connection.
The only thing is when I restart the pia container (or recreate when a new version is available), I've to restart the connected containers also. For some reason: network_mode: "service:pia" isn't enough to use the service if the pia container is running again.

I can confirm this too. All the connected containers need rebuilding too.
As reported already, going back to a previous version of qbittorrent solved the connection problem over vpn (reported by others too on qbittorrent github).

Hi Quentin,
many thanks for your work on the issue and the whole project!
Will it then also be possible to force an ovpn restart inside the container - i.e. not just in case of an disconnect or exit?
I'm looking for an option to force restart the ovpn connection through an attached container (network-mode). Unfortunately, I don't have any idea how this could be implemented within the pia-docker.

So my idea now (see #91) would be to have a web ui to restart openvpn, change parameters etc.
As this might take some time undertaking, I think a nice first step would be to have an http server built-in the container with routes triggering actions. For example you could from another connected container
wget -qO- http://localhost:8000/openvpn/actions/restart

or from a browser do the same assuming you published the port etc.
I will try to do that this weekend.

Hi @chrmei can you please give more details why you need this feature? Please comment back on #147 . I implemented the changes locally, but I would need to know why you need to restart openvpn? Because for now it simply restarts the openvpn process but doesn't change its configuration for example. Thanks!

Reddit post on restarting the vpn container without restarting the linked containers let's see what the experts say :wink:


Add text element for statement ID F##/I##

Fixes #75
Adds a text element to the statement group displaying the numerical ID prepended by an F or I to indicate a formal or informal statement respectively.

I'm wondering though, about how the styling is done. Is it all done so manually, setting each individual text attribute? Would be good if we can have a mechanism where changing the source isn't required. Does CSS work with SVGs?

I definitely agree, Ole already suggested this earlier, so it's already listed as issue #23 . Luckily, it does seem like SVG and CSS can be used together, see this tutorial


alsa-scarlett-gui: Fix

Attemt to fix the desktop integration for alsa-scarlett-gui as it would not show up (at least in Gnome) in any menu.
Description of changes
Things done
For the fix I added a patch file that changes Exec = PREFIX/bin/alsa-scarlett-gui in the resulting .desktop file to the two
lines:
Exec = alsa-scarlett-gui %F
TryExec = alsa-scarlett-gui

In  pkgs/applications/audio/alsa-scarlett-gui/default.nix I added following line:
patches = [ ./fix-deskop-integration ];
Goal
This way you must only make sure the executable file is in your PATH environment variable.

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

What is the %F used for? I would prefer patching the build-output in postInstall via substitueInPlace or sed because it's more update-proof than a patch we have to revisit again.
Other than that good-catch after cleanup/reboot I could reproduce the issue :)

At first %F was copied from a neovim example desktop entry. After doing some digging it allows the entry to accept a list of files when executed.
As alsa-scarlett-gui does not accept any files as argument the %F should be safe to remove.


added BFS algorithm

this is new code
Thank you for your contribution. Please provide the details requested below.
ISSUE NUMBER
#104
Please provide a link to the issue this pull request addresses.
#104
SHORT DESCRIPTION
added Bfs algorithm
TESTING
Just have to run the code

There are some conflicts , please resolve them


Refactor uniqFuncSequence to prevent mutation of input array #1

Summary
In this PR, I refactored uniqFuncSequence to avoid mutation of the input array. Instead of manipulating the original array, the function now creates a copy using Array.prototype.slice and performs operations on it.
Motivation
Mutation of input can lead to unexpected side effects, especially when the function is used in different contexts or when the input is reused later in the code. This change is a step towards enhancing the predictability and purity of functions across our codebase.
Discussion
While this PR focuses on uniqFuncSequence, I think it's worth discussing the potential benefits of avoiding mutation throughout the library. This approach could improve readability, predictability, and better adhere to functional programming principles.

if you look carefully, uniqFuncSequence does not actually mutate the input array.  there are no mutation events, since sliceOut returns a shallow copy of two portions of the array and ties them together into a new one.
I don't believe there is mutation anywhere else in the repo, but feel free to check 👍


Have CI coverage require other tests passing

There's no needs clause on coverage in PR CI, which means if the tests are definitely going to fail then they will fail twice: once for the normal test run and then again when run under coverage. Either coverage should be the way to run those tests so they aren't run twice or coverage should probably not run if the test suite failed.

hey , I would love to work on this issue, thanks


Presentation of links is too generic

Showing links as just "link" makes it unclear where the url is pointing to, which makes for poor UI in some situations, see screenshot.
Alternative solutions:

github.com
github.com/pg-irc/pa... (all urls truncated to the same maximum length with ellipsis, this is my preferred solution)
https://github.com/pg-irc/pathways-frontend/issues/new

duplicate: #526


WIP: Add hostname prefix validation to ha deploy

If this ever gets merged, https://github.com/rancher/rancher/pull/31783 should go first. A lot of the formatting changes here are a side effect of running black on the code.


Add a service definition for the ColectionTypeExtension

Depends on https://github.com/Limenius/Liform/pull/22
Add a service for the CollectionTypeExtension to automatically set the serialisation strategy for CollectionTypes.
See https://github.com/Limenius/Liform/pull/22 for more information 😄

I'm going to close this for now and reopen when I get some more time to think about it. I have made some substantial changes which are applicable to my use cases as I'm time restricted on this project - I believe my changes will help all users of this repo but need to think it out a little more for generic cases.


0.4.x - Missing https://github.com/mausch/SolrNet/blob/0.4.x/Documentation/Core-admin.md

Hello,
I miss the SolrCoreAdmin class on 0.4.4001 version and also the documentation on the following link:
https://github.com/mausch/SolrNet/blob/0.4.x/Documentation/Core-admin.md
Regards,
David

If you compare https://github.com/mausch/SolrNet/blob/0.4.x/SolrNet/Startup.cs and https://github.com/mausch/SolrNet/blob/master/SolrNet/Startup.cs you'll see that the latter includes ISolrCoreAdmin and the former does not.
If you want ISolrCoreAdmin use https://www.nuget.org/packages/SolrNet/0.5.0-alpha1 (or a build of the master branch).

Thanks so much Mauricio!!!


Wsldev3

Why are these changes needed?

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

Closing this. If there is a reason for this PR please reopen it with updated description.


Local settings: Copyable studio paths

Changes

it is possible to select and copy studio path of  executable in local settings

Task linked: OP-2061 Copyable paths in local settings


fix: ensure sse is spec compliant

Motivation and Context

The previous implementation of SSE worked using a query param (wg_sse), which isn't spec compliant. This parses the value of the MIME type in Accept header, which is per the spec
TODO

[ ] Tests or benchmark included
[ ] Documentation is changed or added on https://app.gitbook.com/
[ ] PR title must follow conventional-commit-standard

Please don't forget to document the new approach in the docs and remove the old one. We can remove it after a grace period.

Encountered https://github.com/99designs/gqlgen/issues/3328 while verifying the PR, which is why it took so long to close this


Add link to GOBL

Motivation
This new link is special because it offers a brand new way for developers to build electronic business documents and specifically invoices. The project and sub-projects are all open source, and we're very keen to get more feedback.
Affiliation

[x] I am the author of the article or project
[ ] I am working for/with the company which is publishing the article or project
[ ] I'm just a rando who stumbled upon this via social networks

Self checks

[x] I have read the Code of Conduct
[x] I applied all rules from the Contributing guide
[x] I have checked there is no other Issues or Pull Requests covering the same topic to open

Thanks @samlown for the contribution!


Support "IS [NOT] UNKNOWN" syntax

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Similar to the issues regarding "IS NOT TRUE/FALSE" (#2265) and "IS TRUE/FALSE" (#3159), we should also support the "IS [NOT] UNKNOWN" boolean operator as well. Here is the Postgres documentation about how it works: https://www.postgresql.org/docs/current/functions-comparison.html
Describe the solution you'd like
This should be possible now that https://github.com/sqlparser-rs/sqlparser-rs/pull/583 has been merged. I believe the implementation should be similar to the one in #3189
Describe alternatives you've considered
None.
Additional context
None.

@sarahyurick are you currently working on this (or interested in it) (was wondering whether I can pick it up as my first contribution)?
Also quick question (to both you and @andygrove), I was briefly looking into the implementation of IS UNKNOWN and noticed that Arrow doesn't have a one-to-one mapping of unknown type. Does that mean IS UNKNOWN is going to check whether the value is null or not (like is null)?

@sarahyurick are you currently working on this (or interested in it) (was wondering whether I can pick it up as my first contribution)?
Sure, go for it! Feel free to tag me when you open the PR, too.

Also quick question (to both you and @andygrove), I was briefly looking into the implementation of IS UNKNOWN and noticed that Arrow doesn't have a one-to-one mapping of unknown type. Does that mean IS UNKNOWN is going to check whether the value is null or not (like is null)?
From what I understand, IS UNKNOWN should only work on boolean and null types (as opposed to IS NULL which works on all types). This would be consistent with the Postgres documentation.

In my current PR #3189, IS TRUE/FALSE only works on boolean and null types as well. My guess is that you can do pretty much the exact same implementation, and then in common/src/scalar.rs make is_unknown and/or is_not_unknown that are like the existing is_null function, except they only allow 2 types and error for all other types.

Thanks a lot for your answer @sarahyurick! Really happy to work on it (I'll probably wait for #3189 to be merged first to send the PR, so we can reduce the number of conflicts).

Sounds good to me - I'll also be opening a PR for #2265 after #3189 merges, so hopefully with both of our contributions we can get all of the boolean operations working!


add code analyzers

code sanitize like https://github.com/trailofbits/llvm-sanitizer-tutorial
cppcheck
...

MSAN requires to Build libc++ and libc++abi with MemorySanitizer as in https://github.com/google/sanitizers/wiki/MemorySanitizerBootstrappingClang#build-libc-and-libcabi-with-memorysanitizer
Is it good idea to create separate dockerfile for sanitizers like MSAN?
Also i plan to use CXCMake_Sanitizers from CXCMake https://github.com/blockspacer/CXCMake/blob/master/cmake/core/sanitize/CXCMake_Sanitizers.cmake


Support highlight？

When I output the entire markdown content as a pdf, there is no block highlighting and it shows the incomplete markdown, with a portion on the right side being cut
from markdown_pdf import MarkdownPdf
from markdown_pdf import Section

def markdown_pdf_write(file_fullname, file_contents):
    pdf = MarkdownPdf(toc_level=2)
    pdf.add_section(Section(file_contents, toc=False))

    pdf.save(file_fullname)

At the moment lib doesn't support syntax highlight. See this topic.


Parse a bcs serialized RawTransaction and return important fields

When a RawTransaction is being signed on the ledger nano device, we want to display relevant transaction fields on the display so that the user can confirm them before signing.
Source of RawTransaction struct : https://github.com/diem/diem/blob/0bf8a9702c847bb9741b3b4747ed4e0ca5f1a827/types/src/transaction/mod.rs#L67
Fields that we want to extract:

sequence_number: u64
payload: TransactionPayload

If payload is of type ScriptFunction(ScriptFunction), parse

module: ModuleId,
function: Identifier,
ty_args: Vec,
args: Vec<Vec>,

Example serialized (hex-encoded) RawTransaction in bcs is:
000000000000000000000000000000016d0500000000000003000000000000000000000000000000010e5061796d656e74536372697074731a706565725f746f5f706565725f776974685f6d657461646174610107000000000000000000000000000000010358555303585553000410000000000000000000000000000000dd0801000000000000000100010048010000000000009b0300000000000003585553287412000000000004

The code for constructing this RawTransaction object and serializing it is here : https://github.com/ankushagarwal/libra/blob/test-txn-serialize/api/src/tests/transactions_test.rs#L1701
We need to take this serialized RawTransaction in our app and write a function which can extract the fields mentioned above
Step 1
Write a function in rust-app/src/utils.rs which takes as an input the raw_txn as a byte array (&[u8]) and parses returns the u64 sequence number.
Sample fn signature : fn seq_num(raw_txn: &[u8]) -> Result<u64, u8>

@ankushagarwal : to confirm on the input parameter, raw_txn. It is a byte array([u8]), rather than string(https://github.com/ankushagarwal/libra/blob/test-txn-serialize/api/src/tests/transactions_test.rs#L1701) or the RawTransaction object.
The byte array is after we apply bcs::to_bytes() on the RawTransaction object, so using the above example(transactions_test.rs#L1701), the array is following. Thank you for confirming!

[0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 109, 5, 0, 0, 0, 0, 0, 0, 3, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 14, 80, 97, 121, 109, 101, 110, 116, 83, 99, 114, 105, 112, 116, 115, 26, 112, 101, 101, 114, 95, 116, 111, 95, 112, 101, 101, 114, 95, 119, 105, 116, 104, 95, 109, 101, 116, 97, 100, 97, 116, 97, 1, 7, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 3, 88, 85, 83, 3, 88, 85, 83, 0, 4, 16, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 221, 8, 1, 0, 0, 0, 0, 0, 0, 0, 1, 0, 1, 0, 72, 1, 0, 0, 0, 0, 0, 0, 155, 3, 0, 0, 0, 0, 0, 0, 3, 88, 85, 83, 40, 116, 18, 0, 0, 0, 0, 0, 4]

Yeah it will be a byte array. The string is a hex-encoding of the same byte array.


🪲 PSPreworkout.md generation is adding script file name in last segment

Expected Behavior

Each cmdlet documented in this file should only have the cmdlet or function name as a sub heading and then the function synopsis below that.
Current Behavior

The segment for Update-AllTheThings is appending that function's file name and looks like:

Update all the things! Update-AllTheThings.ps1

Context (Environment)

Operating System and version as reported by $PSVersionTable.OS: Linux and Windows 11
PowerShell versions as reported by $PSVersionTable.PSEdition: 7.4.5

This was happening because the build script runs Get-Help -Detailed $_ and this function was present in the module and as an installed script. (I had previously run Install-Script Update-AllTheThings.) I uninstalled the script and this extra content is no longer appended.


REQUIRE_PARTS settings doesn't work as the example in the documentation

The documentation on https://dateparser.readthedocs.io/en/latest/settings.html#handling-incomplete-dates shows :
>>> parse('2012', settings={'REQUIRE_PARTS': ['month']})
None

I tested it on my computer and that make :
>>> from dateparser import parse
>>> parse('2012', settings={'REQUIRE_PARTS': ['month']})
datetime.datetime(2020, 1, 2, 0, 0)

My computer is MacOs Catalina 10.15.7 python version is Python 3.7.10 and the version of dateparser is dateparser==0.7.6.

Hi @tiretfa, thank you for opening the issue.
This was fixed in dateparser=1.0.0 (here: https://github.com/scrapinghub/dateparser/pull/786). This happens because it's wrongly recognizing the 2012 as a date without spaces.
As you know may guess, the old version is not supported anymore and we can't fix it, so I will close this issue.
Thanks :)


contents of public directory

Hi,
How do you create the html files in public folder such as public/content/contributing/index.html

For people who might be interested, files in public folder are generated by Hugo. It comes from the tutorial : Hosting Hugo website in Github Pages.
In a nutshell, the public is the generated html pages displayed when you go to the documentation of this theme.


Make the invite code input readonly so it can be manually copied from when the copy button fails.

So the last fix should have made the copy invite code button actually work, but from the Chromium bug and stackoverflow links below, this seems to be a support issue with OS X.
SO: http://stackoverflow.com/questions/36196248/execcommandcopy-fails-on-chrome-on-os-x
Chromium: https://bugs.chromium.org/p/chromium/issues/detail?id=552975
Thus, to mitigate the issue and make the invite code accessible to users on OS X in the meantime, I've made the input box itself readonly so the text can be manually copy and pasted. We can add support to notify users of this limitation at some later time if we like, but this seemed like a simple stopgap until then.

This change is

:+1:


Feature excel add table validator

Closes #452

Added a new button that checks whether the excel table is a valid arc/isa table or not.
When there is a problem, a message will be displayed for each occured problem, pointint at the column that is causing the problem for further investigation.

How would this handle a single column "Comment"?

How would this handle a single column "Comment"?

@Freymaurer For some reason I cannot reply to your comment directly.
When it isn't breaking a building block there should not be a problem, unless the arc table says there is one.

How would this handle a single column "Comment"?

@Freymaurer For some reason I cannot reply to your comment directly.
When it isn't breaking a building block there should not be a problem, unless the arc table says there is one.

Sorry it is difficult to review on phone. So you are using building block based arctrl parsing for this?

Sorry it is difficult to review on phone. So you are using building block based arctrl parsing for this?

@Freymaurer I thought it is the most important thing to be honest...

Sorry it is difficult to review on phone. So you are using building block based arctrl parsing for this?

@Freymaurer I thought it is the most important thing to be honest...

Yeah dont worry! If you are using arctrl for this it is totally fine.

In the building block tab the selected building block and those next to it can be validated, checking whether it can be part of an Arc / ISA table. Only the selected building block and those next to it can be check because it is not possible to group them correctly, when a header is false.

In the building block tab the selected building block and those next to it can be validated, checking whether it can be part of an Arc / ISA table. Only the selected building block and those next to it can be check because it is not possible to group them correctly, when a header is false.

This should not be a separate button, but be part of the "validate/auto fill" function in the quick access button bar

By clicking on the Validate / Update Ontology Term button, currently, the table will be validated because the rest of the functionality of the button is not implemented yet.
You will be shown at least two messages, indicating which building block and in a more detailed message, which column is causing problems.

In the building block tab the selected building block and those next to it can be validated, checking whether it can be part of an Arc / ISA table. Only the selected building block and those next to it can be check because it is not possible to group them correctly, when a header is false.

This should not be a separate button, but be part of the "validate/auto fill" function in the quick access button bar

@Freymaurer Fixed

@HLWeil Is it better than with two separate warnings?

IMO yes, nice work!


Need more test, coverage -> 100%

Done. https://github.com/abersheeran/rpc.py/commit/06324d8d595f66c19915a7ae157a2a79839630b0


Remove the field data plugin examples

The ones in this repo are stale and not working. Point folks to the proper landing places.

Fixde by #87


over 100% CPU, even after quitting (Code Helper)

Issue Type: Performance Issue
Badly this is random, and while for the same project(s) open it could be working fine, another start VSCode suddenly uses over 100% CPU even before I start using it.
Tho one thing that remains the same is, when that issue happen and I close VSCode, 4 or more Code Helper processes keep running forever at +100% in the background and I have to kill them.
I know some other issues have been opened like this one, but the only one still open was asking to report the bug using the Help Report Issue so here it is.
Also here are the processes that keep running after I close VSCode when my issue hapen:
❯ ps aux | grep -i 'Visual Studio Code.app'
huafu             2222  93.9  2.1  5133144 176060   ??  R     7:38AM   3:10.22 /Applications/Visual Studio Code.app/Contents/Frameworks/Code Helper.app/Contents/MacOS/Code Helper /Applications/Visual Studio Code.app/Contents/Resources/app/out/bootstrap --type=extensionHost
huafu             1776  92.8  0.8  5189252  66200   ??  R     7:09AM  24:33.56 /Applications/Visual Studio Code.app/Contents/Frameworks/Code Helper.app/Contents/MacOS/Code Helper /Applications/Visual Studio Code.app/Contents/Resources/app/out/bootstrap --type=extensionHost
huafu             1819  92.4  0.7  5132796  59688   ??  R     7:09AM  24:09.17 /Applications/Visual Studio Code.app/Contents/Frameworks/Code Helper.app/Contents/MacOS/Code Helper /Applications/Visual Studio Code.app/Contents/Resources/app/out/bootstrap --type=extensionHost
huafu             1780   0.0  0.3  5011088  23992   ??  S     7:09AM   0:00.29 /Applications/Visual Studio Code.app/Contents/Frameworks/Code Helper.app/Contents/MacOS/Code Helper /Applications/Visual Studio Code.app/Contents/Resources/app/out/bootstrap --type=searchService
huafu             1775   0.0  0.1  4307360   8320   ??  S     7:09AM   0:00.04 /Applications/Visual Studio Code.app/Contents/Frameworks/Electron Framework.framework/Resources/crashpad_handler --no-rate-limit --no-upload-gzip --database=/var/folders/dr/dyg3_v313zjfpq82wd_4b0lr0000gn/T/VSCode Crashes --metrics-dir=/var/folders/dr/dyg3_v313zjfpq82wd_4b0lr0000gn/T/VSCode Crashes --url=https://rink.hockeyapp.net/api/2/apps/21a48a66799e47fea4f52c0ff81e803d/crashes/upload --handshake-fd=60

VS Code version: Code 1.25.1 (1dfc5e557209371715f655691b1235b6b26a06be, 2018-07-11T15:33:29.235Z)
OS version: Darwin x64 17.7.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4288U CPU @ 2.60GHz (4 x 2600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledvideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
6, 5, 5

Memory (System)
8.00GB (0.04GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
   22	   115	  2325	code main
    4	   139	  2326	   gpu-process
    0	   451	  2327	   window (get-typescript-config.ts — ts-jest)
    0	    33	  2349	     terminal
    0	     0	  2350	       /usr/local/bin/fish -l
   82	   172	  2373	     extensionHost
    0	     0	  2377	       (git)
    0	     0	  2378	       (git)
    0	     0	  2380	       (which)
    0	     0	  2381	       (which)
    0	    33	  2382	       electron_node electronForkStart.js tsserver.js
    0	    49	  2374	     watcherService
    0	    98	  2370	   shared-process
   11	    98	  2379	   window (Issue Reporter)
    7	     0	  2383	   /bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=

Workspace Info
|  Window (get-typescript-config.ts — ts-jest)
|    Folder (ts-jest): 232 files
|      File types: ts(112) json(75) js(13) gitignore(10) babelrc(4) snap(4)
|                  md(3) tsx(2) yml(2) DS_Store(1)
|      Conf files: package.json(30) tsconfig.json(30) tslint.json(1)
|                  launch.json(1) settings.json(1)
|      Launch Configs: node;

Extensions (58)

Extension
Author (truncated)
Version

rainbow-brackets
2gu
0.0.6

EditorConfig
Edi
0.12.4

json-to-ts
Mar
1.5.5

vscode-jest
Ort
2.8.0

vscode-docker
Pet
0.0.27

ejs-language-support
Qas
0.0.1

html-css-class-completion
Zig
1.17.1

Bookmarks
ale
9.0.0

vscode-color
ans
0.4.5

markdown-preview-github-styles
bie
0.1.2

vscode-yarn-script
car
0.0.1

gitignore
cod
0.5.0

snippetier
daf
0.1.0

vue-peek
dar
1.0.2

vscode-quick-select
dba
0.2.5

vscode-dash
dee
1.10.0

Vue2
duc
0.1.3

gitlens
eam
8.5.2

ts-tslint
eg2
0.0.7

LogFileHighlighter
emi
2.4.0

vsc-material-theme
Equ
2.3.0

prettier-vscode
esb
1.5.0

gitlab-workflow
fat
0.6.0

code-runner
for
0.9.3

shell-format
fox
1.1.3

vscode-yarn
gam
1.6.0

icon-fonts
idl
2.1.0

path-autocomplete
ion
1.12.0

plantuml
jeb
2.8.3

theme-karyfoundation-themes
kar
18.0.1

vscode-gist
ken
1.2.1

Lua
key
0.0.9

vscode-gutter-preview
kis
0.12.2

vscode-commitizen
Kni
0.5.0

vscode-github
Kni
0.30.0

javascript-test-runner
leg
0.2.0

vue
liu
0.1.5

dotenv
mik
1.0.1

prettify-json
moh
0.0.3

debugger-for-chrome
msj
4.8.0

vscode-gitignore-generator
pio
1.0.1

material-icon-theme
PKi
3.5.2

typescript-mono-repo-import-helper
q
0.0.6

seti-icons
qin
0.1.3

vscode-coverage-gutters
rya
2.0.1

partial-diff
ryu
1.2.0

code-settings-sync
Sha
3.0.0

language-stylus
sys
1.9.2

json-schema-validator
tbe
0.1.0

ayu
tea
0.14.0

vscode-gitmoji-snippets
thi
3.2.0

stylus-supremacy
thi
2.12.0

json2yaml
tux
0.2.0

gitflow
vec
1.2.1

stylint
vtf
0.0.2

vscode-todo-highlight
way
1.0.3

markdown-all-in-one
yzh
1.6.0

tslint-snippets
zuo
1.1.0

(4 theme extensions excluded)

...now it's stuck all the time when I open one project. I can type code, but most of my extensions are not loaded, and when I go to the SCM pane it shows "There are no active source control provider" 😧

I would like to add my note, my vscode stops typescript intellesense and takes 100% CPU, unless I disable ALL extensions, I tried to enable just one to test: TODO Highlight, and it went crazy again.

yes, I don't what the odds are, but I happened to disable the one extension that was causing the trouble! This one and scss-scan caused the trouble, my guess is they do some sort of "file scanning" and it takes forever (node_modules is excluded but it doesn't seem to honer that)

I have the same problem on my side and Git is not working as it should be...

same here, and I disabled all extensions, still > 90%, and always > 90%.

Same issue running Python. I had 99.8%.

Reading your description it seems that the issue originates from an extension and not VS Code itself. Thereby I will close this issue and kindly ask you to report this issue against the extension repository.
Most extensions are on GitHub as well so it should be easy to file it against the extension. If you open an extension on the market place, click on the Support link to find out more.

Thanks again for your feedback!


Original work hardware

Check before submitting issues

[X] Make sure to pull the latest code, as some issues and bugs have been fixed.
[X] Due to frequent dependency updates, please ensure you have followed the steps in our Wiki
[X] I have read the FAQ section AND searched for similar issues and did not find a similar problem or solution
[X] Third-party plugin issues - e.g., llama.cpp, text-generation-webui, LlamaChat, we recommend checking the corresponding project for solutions
[X] Model validity check - Be sure to check the model's SHA256.md. If the model is incorrect, we cannot guarantee its performance

Type of Issue
Other issues
Base Model
None
Operating System
Linux
Describe your issue in detail
Thank you for your astonishing work, I want to use the concepts suggested in the paper for adapting LlaMa for new language.
For assessing the computing costs, it will be really helpful if you could share the the hardware use for training (how much GPU? What kind? How many days of training?)
Thank you!
Dependencies (must be provided for code-related issues)
No response
Execution logs or screenshots
No response

We are using NVIDIA A40 for most of our experiments, taking up to 48 GPUs, depending on the model sizes.
Unfortunately, it is really hard to tell an exact training time for each model, as we are optimizing our training strategies from time to time.
A rough estimate on our latest Chinese LLaMA-33B is that iterating over 20G raw text data roughly takes 10-14 days, depending on various training factors, such as training stability, inter-machine communication speed, etc.
Hope these information helps.

Thank you for your reply!
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
Virus-free.www.avg.com
http://www.avg.com/email-signature?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
בתאריך יום ה׳, 15 ביוני 2023 ב-9:21 מאת Yiming Cui <
@.***>:

We are using NVIDIA A40 for most of our experiments, taking up to 48 GPUs,
depending on the model sizes.
Unfortunately, it is really hard to tell an exact training time for each
model, as we are optimizing our training strategies from time to time.
A rough estimate on our latest Chinese LLaMA-33B is that iterating over
20G raw text data roughly takes 10-14 days, depending on various training
factors, such as training stability, inter-machine communication speed, etc.
Hope these information helps.
—
Reply to this email directly, view it on GitHub
https://github.com/ymcui/Chinese-LLaMA-Alpaca/issues/599#issuecomment-1592436346,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AO42QBIJ2GPLZUR3O4DZHJ3XLKSX3ANCNFSM6AAAAAAZHKMUEA
.
You are receiving this because you authored the thread.Message ID:
@.***>


I want to be able to pause / unpause a container

Fixed in https://github.com/lcallarec/gnome-docker-manager/pull/23


Clarify title when importing a table

The flow to import a table definition in Superset is confusing, user may
think they are creating a table or what not. This makes the flow a bit
more clear.

Codecov Report

Merging #5454 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #5454   +/-   ##
=======================================
  Coverage   59.11%   59.11%
=======================================
  Files         372      372
  Lines       23756    23756
  Branches     2758     2758
=======================================
  Hits        14044    14044
  Misses       9697     9697
  Partials       15       15

Impacted Files
Coverage Δ

superset/connectors/sqla/views.py
70.47% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Not Receiving incoming calls in android

Hello @sajanIocod
Could you please provide more details about the issue you're facing? Specifically, what exactly is happening, and any relevant context would be helpful for us to understand the situation better.
Thank you!


mult op questionable performance

Try to improve mult op by using example here: http://wikiti.brandonw.net/index.php?title=Z80_Routines:Math:Multiplication
originally here: https://github.com/joeldipops/gbz80-pseudoOps/pull/8
'Now constant time, and uses less regs'
eg.
;;;
; mult r16, r8
; Multiples an 8-bit number (in LOW(\1)) with H, result in HL
; @param \2 A temporary register. Make SURE it's not one of H, L, HIGH(\1), or LOW(\1).
; @return HL The result
; @return HIGH(\1) 0
; @return \2 0
; @return ZF Set
; @destroy CF
;;;
mult: macro
ld L, 0
ld HIGH(\2), L
ld \2, 7
; Optimized first iteration
sla H
jr NC, .loop\@
ld L, LOW(\2)

.loop@
add HL, HL
jr NC, .noAdd@
add HL, DE
.noAdd
dec \2
jr NZ, .loop@
endm

Replaced previous mult macro with code similar to the above. Will multiple A with the first term, result in HL.  Should work with any value eg
mult B
mult [BC]
mult [$1234]
mult $32


Update expressroute-howto-linkvnet-arm.md

Corrected the format for note

Hi @shchauh, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

Author approved in prior PR.


Sample does not work

Got Error with no userid
client.js:8 azure:ACS:error 2023-01-27T12:25:34.490Z CallClient1:CallAgent1 op:StartCall failed, message=Invalid CommunicationUser identifier specified, code=422 in 0s
client.js:79
   CallingCommunicationError: Invalid CommunicationUser identifier specified
at validateIdentifier (sdk.bundle.js:8:63104)
at sdk.bundle.js:139:1885743
at Array.forEach (<anonymous>)
at CallAgentImpl.assertCallUserIds (sdk.bundle.js:139:1885562)
at CallAgentImpl.startCall (sdk.bundle.js:139:1879954)
at p.value (sdk.bundle.js:8:60497)
at ./client.js.startCallButton.onclick (client.js:75:1)

./client.js.startCallButton.onclick @ client.js:79

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

ID: 1f6c0b6e-a3b1-ed87-f673-4842029e498f
Version Independent ID: 10cb7160-76d2-3f5c-3a6d-b9bbddc75d0e
Content: Quickstart - Add video calling to your app (JavaScript) - An Azure Communication Services quickstart
Content Source: articles/communication-services/quickstarts/voice-video-calling/get-started-with-video-calling.md
Service: azure-communication-services
Sub-service: calling
GitHub Login: @tophpalmer
Microsoft Alias: chpalm

@devwheel
Thanks for your feedback! We will investigate and update as appropriate.

@devwheel
Error code 422 means, the communication user identifier of the callee (identifier of the user you are calling to) is invalid.
For example- if you are using public cloud, then the identifier should start with 8:acs:
Could you please share the first 10 characters of the communication identifier?

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Replace all instances of Guava cache with Caffeine

Caffeine is the replacement for the Guava cache api and has several improvements including efficiency with its admission policy and performance. Currently the Accumulo code base has a mixture where some places use Guava and some places use Caffeine so it is not consistent. Switching to Caffeine is nearly a drop in replacement and this commit replaces all instances of Guava with Caffeine for consistency and also adds a checkstyle rule to prevent using Guava in the future.
From the Guava docs regarding Caffeine:

Prefer Caffeine over Guava's caching API
The successor to Guava's caching API is Caffeine. Its API is designed to make it a nearly drop-in replacement -- though it requires Java 8 APIs, is not available for Android or GWT/j2cl, and may have different (usually better) behavior when multiple threads attempt concurrent mutations. Its equivalent to CacheBuilder is its Caffeine class. Caffeine offers better performance, more features (including asynchronous loading), and fewer bugs.

Kicked off full IT build

Full IT build passed successfully


Multiply glyph alpha with surface alpha

Fixes #6239.

[x] Run ImageTests

@a-gagnon do you need this backported to 4.2.x?

ImageTests showed up two views of the same iModel where text that is supposed to be semi-transparent now renders as such when drawn as raster glyphs.

@a-gagnon do you need this backported to 4.2.x?

No, I think it's fine if we pick it up in the next minor release.
Thanks!


Asking for D2 Notation of session 3 in your paper

Hello, I raise this issue to asking for some notation of your paper.
In your paper, you note that "From D3 to D0, the set of strides of their four stages for each model are {1,2,4,8}, {1,2,4,4}, {1,2,2,2} and {1,1,1,1}, respectively"
So I understood D notations such as:
D3: [1,2,4,8]
D2: [1,2,4,4]
D1: [1,2,2,2]
D0: [1,1,1,1]
But you note that "PointPillar model D2", and I understood PointPillar model has [1,2,2,2] stride(following your noation, it is D1). So is it notation miss or I misunderstood the pointPillar?
thanks

Thanks for your interest.
Before further discussion, may I ask where you find the  PointPillars of strides [1, 2, 2, 2] ?

I found it in PointPillar Paper  and recheck in code in OpenPcdet pointpillar.yaml
Like bellow:
"BACKBONE_2D:
NAME: BaseBEVBackbone
LAYER_NUMS: [3, 5, 5]
LAYER_STRIDES: [2, 2, 2]
NUM_FILTERS: [64, 128, 256]
UPSAMPLE_STRIDES: [1, 2, 4]
NUM_UPSAMPLE_FILTERS: [128, 128, 128]
"
and mmdetection3D configuration too("hv_pointpillars_secfpn_kittti.py")
"model = dict(
type='VoxelNet',
voxel_layer=dict(
max_num_points=32,  # max_points_per_voxel
point_cloud_range=[0, -39.68, -3, 69.12, 39.68, 1],
voxel_size=voxel_size,
max_voxels=(16000, 40000)  # (training, testing) max_voxels
),
voxel_encoder=dict(
type='PillarFeatureNet',
in_channels=4,
feat_channels=[64],
with_distance=False,
voxel_size=voxel_size,
point_cloud_range=[0, -39.68, -3, 69.12, 39.68, 1]),
middle_encoder=dict(
type='PointPillarsScatter', in_channels=64, output_shape=[496, 432]),
backbone=dict(
type='SECOND',
in_channels=64,
layer_nums=[3, 5, 5],
layer_strides=[2, 2, 2],
out_channels=[64, 128, 256]),
"

The name layer_strides is a little bit misleading. I think layer_strides = [2, 2, 2] means downsampling feature maps 3 times with stride 2. So in terms of feature map size, the real network stride is [2, 4, 8].
And in Waymo Open Dataset, the PointPillars in MMDetection3D use layer_strides=[1, 2, 2], where only downsample 2 times. So in terms of feature map size, the real network stride is [1, 2, 4]. We split the last stage into two stages, so in our notations, the stride of D1 is [1, 2, 4, 4].
I hope I made it clear. Let me know if you have further questions.

We may have the different view of "stride" term.
Thanks for nice anwser!


Add LeafSpy Integration

Description:
Adds new integration for Leaf Spy application to send data directly to Home Assistant
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#9386
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] Untested files have been added to .coveragerc.

@OttoWinter Is there a reason this PR is not getting merged. I have been waiting for 3 releases now to have this in so I don't have to keep rebuilding my hass.io image.

@jesserockz are you planning to finish here?

Hi @MartinHjelmare
Yes. I did make some changes locally but got too busy after that and also away on holiday so have not gotten around to finishing it off.

We're trying to decrease our open PR buffer. Please close this PR and you can reopen it yourself when you're ready to finish. Then we can continue without loosing thread history. Thanks!


Is it acceptable to only support vec int instruction

If the customer want to disable the F/D（floating point) extension, but still want the V(vector) extension. So, in the MISA, f,d bit will be cleaned and bit v will be set. In this case, is this allowed to only support INT instructions in vector extension?

Yes, see #550


not working on android - undefined is not an object

I followed the example in the readme (tried both manual installation and automatic) and no matter what I do, when I press the share button Share.open always fails with:
undefined is not an object (evaluating '_reactNativeShare2.default.open')
building on Ubuntu 14.04 64-bit and running on Android 5.0.1

Works perfectly for me wth React Native 0.29.2.

If anyone has any idea what the problem is I will be happy to fix it myself but I don't have a clue where to start looking for the problem

Since react native 0.29, you have to apply the adjustments to MainApplication.java, not MainActivity.java. See https://github.com/facebook/react-native/releases/tag/v0.29.0.
The installation via rnpm link doesn't do that at the moment.

@levito so what should MainApplication.java look like for this to work ?

Yes, rnpm link doesn't work. In React Native >= 0.29 you need:
Android

npm install react-native-share --save
Open up android/app/src/main/java/[...]/MainApplication.java

Add import cl.json.RNSharePackage; to the imports at the top of the file
Add new RNSharePackage() to the list returned by the getPackages() method

Append the following lines to android/settings.gradle:
include ':react-native-share'
project(':react-native-share').projectDir = new File(rootProject.projectDir, 	'../node_modules/react-native-share/android')

Insert the following lines inside the dependencies block in android/app/build.gradle:
 ```
 compile project(':react-native-share')
 ```

@EstebanFuentealba it works with these instructions, thanks :+1:


Rogue mesh for OpenXR spatial mesh observer

Describe the bug
We have some rogue mesh appearing. It looks like chunks of mesh that are separate from the rest of the main spatial observer mesh. It appears separate because when we enable/disable or switch to occlusion/wireframe, it isn't affected and remains there. Sometime the mesh is the no-shader pink and sometimes it is just invisible. The main way we can recognise it is by the hand ray colliding with it. The object the hand ray collides with is called SpatialMesh - xxxxxxxxx.
They have appeared as small as 10cm to 3-4m large. I believe the only way to remove them is by resetting the holograms in the OS. We have tried CoreServices.SpatialAwarenessSystem.ClearObservations() and CoreServices.SpatialAwarenessSystem.Reset(), neither of those have worked.
The first image attached shows the hand ray colliding with the mesh when we have the spatial awareness system disabled (CoreServices.SpatialAwarenessSystem.Disable()). The second image is when it's visible with the no-shader pink, it is mixed in with a hologram.
It seems these appear more often when doing an app version upgrade. E.g BIMHolovew-v2.2.2 -> BIMHoloview-v2.2.3.
To reproduce
Steps to reproduce the behavior:

Have an OpenXR Unity build with these MRTK Spatial Awareness settings and OpenXR Plugin settings:

Build and deploy to HoloLens 2.
We haven't narrowed down the exact steps to reproduce this issue. It appears to be random at the moment.

Expected behavior
They should not appear or they should change based on CoreServices.SpatialAwarenessSystem.
Screenshots

Your setup (please complete the following information)

Unity Version 2020.3.13f1
MRTK Version v2.7.2
OpenXR Plugin Version v1.2.8
Mixed Reality OpenXR Plugin v1.1.2

Target platform (please complete the following information)

HoloLens 2

Additional context
We never experienced this with the Windows Mixed Reality Plugin or earlier versions of Unity/MRTK.

I work with Shayne and have found another case where instead of pink or invisible mesh showing, actual mesh is showing as the rogue artifact. The first image is with the Spatial Observer enabled.  The second is with it disabled and there are a few rogue mesh artifacts left.

@peterneil-bimholoview, is this something you're able to reproduce? @ShayneRC mentioned that the issue with the pink mesh is random.

Unfortunately I am not what conditions cause this to happen.  Essentially we enable the spatial observer, then (after user action) we disable the spatial observer and sometime after that the rouge mesh sometime appear.  If it is straight after the disable then we will generally see mesh (bottom case), but if it occurs later in the session then the mesh will be 'no shader pink' or colorless.

This is something I recall addressing for the legacy spatial mesh observer. i am trying to repro now

while i get the project loaded, i thought i would jot down a few notes..
this feels as if there is a mesh bake in flight and the callback is not honoring the flag that indicates whether or not the observer should be running. the 'no shader pink' condition is a bit puzzling, but is something i will definitely look for cases where there may be confusion regarding which material to apply

looking at the code, if my hunch is correct, this should impact all spatial mesh observers shipped in MRTK

We only have seen this with the OpenXR plugin, and we did a lot of testing with the WMR Plugin before switching over.

Just seen this with WMR Plugin.

@peterneil-bimholoview, was this still a random event or is it happening consistently now?

@polar-kev still intermittent.  This was the first time I have seen in with WMR, but with OpenXR it occurs in about 1;4 application sessions.  However, it may relate to the Observer Shape being User Aligned Cube rather than the plugin type.  Our OpenXR version is running User Aligned Cube, and we just switched our WMR version from Axis Aligned Cube to User Aligned Cube and saw the issue. But I need to do some more testing on this to confirm.

@peterneil-bimholoview Just to clarify is this the same as #10343 or these two issues are different?

@MaxWang-MS No different issue.  The Rogue mesh issue is where you mesh turns up (generally Pink) when the observer is off.  Issue #10343 is when the observation extents becomes offset due to world locking.

@polar-kev Just confirming tests from today.  The issue appears in both User Aligned Cube & Axis Aligned Cube.

@peterneil-bimholoview, was this still a random event or is it happening consistently now?

I have made the observation that it always occurs to me when I change my Level of Detail to custom at runtime, plus the density (Triangle per Cubic Meter) and then restart the observer. Changing only to custom does nothing. Only by additionally changing the density. (With WMR).
The oberver is reseted, cleared and restarted after the changes at runtime by the following:
CoreServices.SpatialAwarenessSystem.Reset();
CoreServices.SpatialAwarenessSystem.ClearObservations();
CoreServices.SpatialAwarenessSystem.ResumeObservers();
Maybe this will help to find the bug...

@Gescheld I am good to hear someone else is seeing this issue - not just us!!!
We don't change the density but we do set the DisplayOption when we Enable the Observer.  I wonder if this is issue only appears if you Enable/Disable plus set another property on the observer?

@peterneil-bimholoview,
I had only tried from default visible and LoD coarse to not visible and swappable LoD with changeable Triangle per Cubic Meter.
And as it is with programming, when things don't work, you don't know why. And when they do work, you don't know why either :D
I changed my package.appxmanifest because of another task:

@peterneil-bimholoview,
I had only tried from default visible and LoD coarse to not visible and swappable LoD with changeable Triangle per Cubic Meter.
While continuing to work, I changed my package.appxmanifest due to another task. And I noticed that in my script, where I change the observer, I refer to an unassigned text element (but without error message and the program ran anyway). Since then, the pink mesh has not yet reappeared.
But as it is with programming, when something doesn't work, you don't know why. And when it works, you don't know why either :D
This is part of my package.appxmanifest:
<Package
	xmlns:mp="http://schemas.microsoft.com/appx/2014/phone/manifest"
	xmlns:uap="http://schemas.microsoft.com/appx/manifest/uap/windows10"
	xmlns:uap2="http://schemas.microsoft.com/appx/manifest/uap/windows10/2"
	xmlns:uap3="http://schemas.microsoft.com/appx/manifest/uap/windows10/3"
	xmlns:uap4="http://schemas.microsoft.com/appx/manifest/uap/windows10/4"
	xmlns:iot="http://schemas.microsoft.com/appx/manifest/iot/windows10"
	xmlns:mobile="http://schemas.microsoft.com/appx/manifest/mobile/windows10"
	xmlns:rescap="http://schemas.microsoft.com/appx/manifest/foundation/windows10/restrictedcapabilities" (NEW)
	IgnorableNamespaces="uap uap2 uap3 uap4 mp mobile iot rescap" (rescap is NEW)
	xmlns="http://schemas.microsoft.com/appx/manifest/foundation/windows10">

and
<Capabilities>
	<Capability Name="internetClient" />
	<Capability Name="internetClientServer" />
	<Capability Name="privateNetworkClientServer" />
	<uap2:Capability Name="spatialPerception" />
	<rescap:Capability Name="perceptionSensorsExperimental" /> (NEW)
	<DeviceCapability Name="backgroundSpatialPerception"/>  (NEW)
	<DeviceCapability Name="webcam" />
	<DeviceCapability Name="microphone" />
	<DeviceCapability Name="gazeinput" />
</Capabilities>

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visithttps://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


error - sudo: command not found

Describe the bug
Starting today (January 20, 2023) am unable to get this action to run.  Throws sudo: command not found errors.
Version

[ ] I have checked releases, and the bug exists in the latest patch version of v1 or v2.
[X] v2
[ ] v1

Runners

[X] GitHub Hosted
[ ] Self Hosted

Operating systems
Ubuntu
PHP versions
8.1
To Reproduce
  phpunit:
    name: "PHPUnit Unit & Feature Tests w/ Code Coverage Enabled"
    runs-on: ubuntu-latest
    container:
      image: kirschbaumdevelopment/laravel-test-runner:8.1
    services:
      mysql:
        image: mysql:5.7
        env:
          MYSQL_ROOT_PASSWORD: password
          MYSQL_DATABASE: phpunit
        ports:
          - 33306:3306
        options: --health-cmd="mysqladmin ping" --health-interval=10s --health-timeout=5s --health-retries=3
    steps:
      - uses: actions/checkout@v1
        with:
          fetch-depth: 1
      - name: Install composer dependencies
        run: |
          composer install --no-scripts
      - name: Prepare Laravel Application
        run: |
          mkdir -p .coverage-report/html
          php artisan passport:keys
      - name: Setup PHP with PCOV
        uses: shivammathur/setup-php@v2
        with:
          php-version: '8.1'
          ini-values: pcov.directory=app
          coverage: pcov

Expected behavior

Screenshots/Logs

@shawnhooper Fixed in https://github.com/shivammathur/setup-php/commit/dd0115673c4a47a832dff844189234f0e9b3966e. It wIll be in the next release.

Oh, I had same problem here. Nice to hear that already exists a fix. I'll be waiting the release

Thanks for the fix. 😊 @shivammathur When are you planning the release?

Thanks for the fix! Next release will be soon?

Released the fix in 2.24.0