How to select different db execution "keys *"

redisReply * _reply;
_reply = (redisReply*)redisCommand(_redisCxt, "select 1");
_reply = (redisReply*)redisCommand(_redisCxt, "keys *");
if (_reply->type == REDIS_REPLY_ARRAY){
for (int i = 0; i < _reply->elements; ++i){
_reply->element[i]->str;
}
}
question：
The keys I got are all in the database db0.
I need to select different db to get all keys at the same time, what do I need to do. thanks!

Using select 1 is the correct way to use tell Redis that the client wants to use logical db 1.
Either the Redis instance has been setup as a cluster node (select is not supported in cluster mode) or there are other errors.
One suggestion in to make sure the first redisCommand(select 1) results in a non-null reply, and is of type REDIS_REPLY_STATUS and str is OK.

As @bjosv says, you should check to make sure the SELECT command is not returning an error.
#include <hiredis/hiredis.h>
#include <string.h>
#include <assert.h>

#include <hiredis/hiredis.h>
#include <string.h>
#include <assert.h>

int main(void) {
    redisContext *_redisCtx;
    redisReply *_reply;

    _redisCtx = (redisContext*)redisConnect("localhost", 6379);
    assert(_redisCtx != NULL && _redisCtx->err == 0);

    _reply = (redisReply*)redisCommand(_redisCtx, "select 1");
    assert(_reply != NULL && _reply->type == REDIS_REPLY_STATUS &&
           _reply->len == 2 && !strcmp(_reply->str, "OK"));
    freeReplyObject(_reply);

    _reply = (redisReply*)redisCommand(_redisCtx, "keys *");
    assert(_reply != NULL && _reply->type == REDIS_REPLY_ARRAY);

    for (size_t i = 0; i < _reply->elements; i++) {
        assert(_reply->element[i]->type == REDIS_REPLY_STRING);
        printf("KEY[%zu]: %s\n", i, _reply->element[i]->str);
    }

    freeReplyObject(_reply);
    redisFree(_redisCtx);
}

Cluster mode being enabled is the most likely issue (as it disables database numbers) but you can also verify this via redis-cli.
I'm going to close this issue since it's unlikely to be a bug in hiredis but I'm happy to help you figure it out. 😄

Using select 1 is the correct way to tell Redis that the client wants to use logical db 1. Either the Redis instance has been setup as a cluster node (select is not supported in cluster mode) or there are other errors. One suggestion in to make sure the first redisCommand(select 1) results in a non-null reply, and is of type REDIS_REPLY_STATUS and str is OK.

Your reply solved my problem. Thank you!

Your reply solved my problem. Thanks.
std::vectorstd::string _vectDb;
_reply = (redisReply*)redisCommand(_redisCxt, "select %s", _vectDb[x]);
I found reply-->len==43,
_reply = (redisReply*)redisCommand(_redisCxt, "select %s", _vectDb[x].c_str());
I found reply-->len==2,
oh, I forget about c_str().
Thanks again！

As @bjosv says, you should check to make sure the SELECT command is not returning an error.
#include <hiredis/hiredis.h>
#include <string.h>
#include <assert.h>

#include <hiredis/hiredis.h>
#include <string.h>
#include <assert.h>

int main(void) {
    redisContext *_redisCtx;
    redisReply *_reply;

    _redisCtx = (redisContext*)redisConnect("localhost", 6379);
    assert(_redisCtx != NULL && _redisCtx->err == 0);

    _reply = (redisReply*)redisCommand(_redisCtx, "select 1");
    assert(_reply != NULL && _reply->type == REDIS_REPLY_STATUS &&
           _reply->len == 2 && !strcmp(_reply->str, "OK"));
    freeReplyObject(_reply);

    _reply = (redisReply*)redisCommand(_redisCtx, "keys *");
    assert(_reply != NULL && _reply->type == REDIS_REPLY_ARRAY);

    for (size_t i = 0; i < _reply->elements; i++) {
        assert(_reply->element[i]->type == REDIS_REPLY_STRING);
        printf("KEY[%zu]: %s\n", i, _reply->element[i]->str);
    }

    freeReplyObject(_reply);
    redisFree(_redisCtx);
}

Cluster mode being enabled is the most likely issue (as it disables database numbers) but you can also verify this via redis-cli.
I'm going to close this issue since it's unlikely to be a bug in hiredis but I'm happy to help you figure it out. 😄

Through the example you gave, I checked my code and found that reply->len==43, when "select %s", a vector parameter value forgot to convert to c_str(). Thanks you!

As @bjosv says, you should check to make sure the SELECT command is not returning an error.
#include <hiredis/hiredis.h>
#include <string.h>
#include <assert.h>

#include <hiredis/hiredis.h>
#include <string.h>
#include <assert.h>

int main(void) {
    redisContext *_redisCtx;
    redisReply *_reply;

    _redisCtx = (redisContext*)redisConnect("localhost", 6379);
    assert(_redisCtx != NULL && _redisCtx->err == 0);

    _reply = (redisReply*)redisCommand(_redisCtx, "select 1");
    assert(_reply != NULL && _reply->type == REDIS_REPLY_STATUS &&
           _reply->len == 2 && !strcmp(_reply->str, "OK"));
    freeReplyObject(_reply);

    _reply = (redisReply*)redisCommand(_redisCtx, "keys *");
    assert(_reply != NULL && _reply->type == REDIS_REPLY_ARRAY);

    for (size_t i = 0; i < _reply->elements; i++) {
        assert(_reply->element[i]->type == REDIS_REPLY_STRING);
        printf("KEY[%zu]: %s\n", i, _reply->element[i]->str);
    }

    freeReplyObject(_reply);
    redisFree(_redisCtx);
}

Cluster mode being enabled is the most likely issue (as it disables database numbers) but you can also verify this via redis-cli.
I'm going to close this issue since it's unlikely to be a bug in hiredis but I'm happy to help you figure it out. 😄

Through the example you gave, I checked my code and found that reply->len==43, when "select %s", a vector parameter value forgot to convert to c_str(). Thanks!

As @bjosv says, you should check to make sure the SELECT command is not returning an error.
#include <hiredis/hiredis.h>
#include <string.h>
#include <assert.h>

#include <hiredis/hiredis.h>
#include <string.h>
#include <assert.h>

int main(void) {
    redisContext *_redisCtx;
    redisReply *_reply;

    _redisCtx = (redisContext*)redisConnect("localhost", 6379);
    assert(_redisCtx != NULL && _redisCtx->err == 0);

    _reply = (redisReply*)redisCommand(_redisCtx, "select 1");
    assert(_reply != NULL && _reply->type == REDIS_REPLY_STATUS &&
           _reply->len == 2 && !strcmp(_reply->str, "OK"));
    freeReplyObject(_reply);

    _reply = (redisReply*)redisCommand(_redisCtx, "keys *");
    assert(_reply != NULL && _reply->type == REDIS_REPLY_ARRAY);

    for (size_t i = 0; i < _reply->elements; i++) {
        assert(_reply->element[i]->type == REDIS_REPLY_STRING);
        printf("KEY[%zu]: %s\n", i, _reply->element[i]->str);
    }

    freeReplyObject(_reply);
    redisFree(_redisCtx);
}

Cluster mode being enabled is the most likely issue (as it disables database numbers) but you can also verify this via redis-cli.
I'm going to close this issue since it's unlikely to be a bug in hiredis but I'm happy to help you figure it out. 😄

Through the example you gave, I checked my code and found that reply->len==43, when "select %s", a vector parameter value forgot to convert to c_str(). Thank you!


Fix freeze workspace when delete comments

This PR resolves one part of https://github.com/LLK/scratch-blocks/issues/1762, where clicking to delete or minimize a comment on touch would freeze the workspace.
We were stopping event propagation but not clearing the touch identifier, resulting in a gesture sequence that never terminated.

In another PR I'm reviewing (https://github.com/LLK/scratch-blocks/pull/1963), I noticed that we are using gestures. Before submitting, I would want to make sure that for these cases we're either not starting one, or, if we are, that we actually cancel it. iOS' behavior here is different than everywhere else, but hopefully I'll get to the bottom of that as part of that review I mentioned.

Closing this because we should be using the opt_noCaptureIdentifier option in bindEventWithChecks instead.


Feature : Splitter not present in Misc Circuits

Is your feature request related to a problem? Please describe

Describe the solution you'd like
The 'splitter' is mentioned in the docs of circuitverse but if you checkk out the 'misc' in the 'circuit elements' it is not present over there. It is better to remove any extra unnecessary documentation that might confuse user's when they need particular circuits.
Describe alternatives you've considered
No response
Additional context
No response
Are you working on this?
Yes

@tanmoysrt can you please review this ?

@Simer13 please assign this issue to me.

cc @Arnabdaz @Prerna-0202

@yashmangal112 Sorry but I cannot assign as I am also a contributor here like you.

Scroll up in the misc tab and you'll find the splitter

closing as splitter is already there in the misc section


Catch and log errors parsing CLA.json

The concierge should still post, and the error will be logged to stdout in the event the CLA request fails or does not parse.

@ggetz, thanks for the pull request! Maintainers, we have a signed CLA from @ggetz, so you can review this at any time.
I noticed that CHANGES.md has not been updated. If this change updates the public API in any way, fixes a bug, or makes any non-trivial update, please add a bullet point to CHANGES.md and comment on this pull request so we know it was updated. For more info, see the Pull Request Guidelines.

:sparkles: :sparkles: :sparkles:I'm the Cesium Concierge and I'm here to test things. :sparkles: :sparkles: :sparkles:

If I'm reading this right, yesterday's error with this change would end up not asking for the CLA, is that correct?  We definitely don't want that.  Perhaps we can make the PR comment indicate that concierge had a problem reading the CLA?

If I'm reading this right, yesterday's error with this change would end up not asking for the CLA, is that correct? We definitely don't want that. Perhaps we can make the PR comment indicate that concierge had a problem reading the CLA?

Correct. I'll make the  comment indicate it instead.

Deployed.


Uncaught TypeError -v0.9.2

When I ran grunt command serve,  I got some errors.
The first is Uncaught TypeError: angular.module(...).decorator is not a function.
I'm using angular 1.4.7

I get this error in 1.3.x but I can't reproduce in 1.4.x

I google it  and found something
https://github.com/angular/batarang/issues/275

If you disable Batarang or run it in another browser does it work? I can perhaps decorate the service in a config block.

It works for me when I disable Batarang.
Thanks!


Change labs -> results in bluebuttonjs.com docs

Where is the repo for bluebuttonjs.com located? I think I'm blind.
This:

It's a little bit of github black magic. You check out the gh-pages branch of the same repo and it magically turns into a different codebase / controls bluebuttonjs.com


Add support for digitalocean 'spaces' in filestore backend

The problem faced currently?
How can we solve it?

If you want this feature to be implemented, give it a thumbs up reaction, so that we can determine which features are important to you.
👍

Hey @s-kris, digitalocean spaces is an S3 compatible object store. So technically space cloud should work with spaces as space cloud already supports S3.

Oh yeah! It's s3-compatible.
Can you add that sc-filestore works with s3-compatible services in the docuemntation?

Hey @s-kris, would love if you can contribute to the docs! 😃  This issue can be a good starting point to get comfortable with the project structure of the docs since it requires minor changes.
These are the two files you would need to change -

File storage page - https://github.com/spaceuptech/space-docs/blob/master/content/storage/filestore/_index.md
Features page - https://github.com/spaceuptech/space-docs/blob/master/content/introduction/features.md

@HeyItsJs not documentation please 😬


Locale not working proper in Android 11 and Android 12

Hey @yahya-ebizz
Could you please share more details on what is wrong with Android 11 and 12?
I've tested the sample app using the latest versions, but I haven't come across any issues.

I spotted it is not working on Android 13 ether. It is translating half of the app, also changing language 2 or more times when app is active is especially critical..

the library works great on Android 10, but when tested on Android 12 I found that words do translate to the right language but layout don't change direction ltr to rtl and vise versa

@zeroDivider @ahmed-elkhadragy Trying to figure out what is the cause of your issue with new android versions

Do you restart your activity/app once the new locale is applied?
Do you use the navigation component/fragments for your screens?

@zeroDivider @ahmed-elkhadragy Trying to figure out what is the cause of your issue with new android versions

Do you restart your activity/app once the new locale is applied?
Do you use the navigation component/fragments for your screens?

Hello, yes I used receeate() to restart the activity and it indded changes language but it's all about layout direction and I don't use navigation component nor fragment my app is single screen single activity app..
I turned around the issue right now until it's figured out by making the locale change in different activity and reopening the original activity and it works fine but my original plan was to change locale from the same screen/activity

reason:
IME_AUTOFILL_DEFAULT_SUPPORTED_LOCALES_IS_EMPTY
Change ID: 169273070
Default State: Enabled for apps that target Android 12 (API level 31) or higher.
For apps targeting Android 12 (API level 31) and higher, getSupportedLocales() now returns an empty locale list when it's not set, instead of the default system locale.

My option
fun Configuration.getFirstLocale(): Locale =
if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.S) {
getSystemLocale()
} else {
val locales = ConfigurationCompat.getLocales(this)
if (locales.isEmpty) localeENGLISH
else (locales[0] ?: localeENGLISH)
}
fun getSystemLocale(): Locale = getSystemLocales().get(0) ?: Locale.getDefault()
fun getSystemLocales(): LocaleListCompat =
ConfigurationCompat.getLocales(Resources.getSystem().configuration)


Lab10

Fixes #641
Review of colleague's PR #636
Changes proposed in this PR:
Implement lab10 functionality + write tests

Create puppyHandler which is essentially a REST wrapper around our Storer interface
Which exposes the functionality we built over the past few labs over a REST API
Expand flag for user to specify storer type as well as other config settings like port number and file to load data from
Learn how to use net/http/httptest package for testing REST APIs

LGTM!
well done on completing 10 labs


It can't work with BBC

I try it with BBC, for example :https://www.bbc.com/news/world-europe-61204543
it can't fetch Authors ,and can't fetch the full text.
something went wrong.

Take a look at this document that I wrote on using newspaper3k.  It has details on extracting content from BBC.  Let me know if my extraction guidelines work for you.

Take a look at this document that I wrote on using newspaper3k. It has details on extracting content from BBC. Let me know if my extraction guidelines work for you.

Yes,it works. You pick the elements from script segment, in this way, you can also pick the 'article content  text' in 'script nonce' seg. It seems like there are no easy and clear way to work with BBC.
Thank you.
it can be closed.


fix passing null tracer

fix for #41

Pull Request Test Coverage Report for Build 149

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 44.549%

Totals

Change from base Build 147:
0.1%

Covered Lines:
380

Relevant Lines:
853

💛  - Coveralls


http://<IP_ADDRESS>:8080 is shown for real phone

When I use my smart phone as AP and let my laptop connect to it, the link will be http://<IP_ADDRESS>:8080 and it cannot show anything.
The simplest solution I found is change the URL to router's ip address, for example, http://<IP_ADDRESS>:8080, and then it will work. Can you make the address show in the logcat?

Let me check the issue.
Thanks
Amit Shekhar


KAFKA-4507: NodeApiVersions clean-ups

@cmccabe, please take a look and merge to your branch if you're happy. One thing that I didn't do, but I think would be good to do is to flesh out the usable version calculator tests. They seem to be quite bare at the moment.

Thanks, looks good.  I forget why I thought I needed to cast to int when printing a short.  It sounds like it works as-is, which is good.


Java (OpenJDK) using commonmark-java

Since this pull request, Java (OpenJDK) has included a copy of commonmark-java:

https://github.com/openjdk/jdk/pull/16388

When they copied the source files, they made a few adjustments too. There's a list here:
// This module is primarily an import from a recent tagged version of
//     https://github.com/commonmark/commonmark-java
//
// The following parts are imported:
//
// * commonmark/src/main/java
// * commonmark/src/main/resources
// * commonmark-ext-gfm-tables/src/main/java
// * commonmark-ext-gfm-tables/src/main/resources
//
// For source and resource files, the following transformations are applied:
//
// * legal headers are added
// * package and import statements are updated
// * characters outside the ASCII range are converted to Unicode escapes
// * @SuppressWarnings("fallthrough") is added to getSetextHeadingLevel
// * the value for ENTITY_PATH is updated with the modified package
// * the file `entities.properties` is renamed to `entities.txt`

Let's make it easier for them next time by adopting some of the changes.

Hi @jonathan-gibbons! I saw that OpenJDK recently included commonmark-java for Markdown parsing in Javadoc:

https://github.com/openjdk/jdk/pull/16388

Just wanted to recommend upgrading to version 0.24.0, with these changes:

https://github.com/commonmark/commonmark-java/pull/339

This was released in 0.23.0 and is a fix for source positions being incorrect with some lazy continuation lines

https://github.com/commonmark/commonmark-java/pull/348

This should simplify the source position mapping (this code in MarkdownTransformer.java can be removed)

https://github.com/commonmark/commonmark-java/pull/343

This should make it easier to update the vendored copy


Editor does not auto-scroll to keep the cursor visible

VSCode Version: Code 1.16.0 (787b31c0474e6165390b5a5989c9619e3e16f953, 2017-09-06T16:19:37.192Z)
OS Version: Linux x64 4.4.0-93-generic
Extensions:

Extension
Author (truncated)
Version

Bookmarks
ale
0.16.0

python
don
0.7.0

Go
luk
0.6.65

cpptools
ms-
0.12.3

perforce
sle
2.1.1

I believe this is a recent regression:
Steps to Reproduce:

open any source file with matching brackets far away from each other so one of them is outside the visible lines
jump to the matching bracket (editor.action.jumpToBracket)
cursor moves to the matching bracket correctly, but it's not outside the visible lines (editor does not auto-scroll to keep cursor visible as expected)

Reproduces without extensions: Yes/No

https://github.com/Microsoft/vscode/issues/32597


Remove outdated documentation in LifeCycle events

The documentation previously included an outdated claim that users interested in automatic focus should call register_for_focus() in response to LifeCycle::WidgetAdded. This is no longer true since the addition of LifeCycle::BuildFocusChain, so the relevant section was deleted. Additionally, this commit fixes a minor spelling error in the documentation of the latter.
Fixes #2318 .

Sorry about the delay on this pull request. Its been an unexpectedly busy few days.


j.tools.threebot.init_my_threebot broken

j.tools.threebot.init_my_threebot is supposed to be the tool used to register its 3bot identity into the phonebook. It seems this tool is broken on development right now (247286cd7e661f3c403b7ce26b5f08ba156aaa99)
We need this back in order for users/farms to be able to register their identity on the grid before doing any other kind of interaction.
how to reproduce:
j.tools.threebot.init_my_threebot()
your threebot name: zaibon.3bot
Thu 05 13:47:40 /BASECLASSES/Attr.py -  68 - __getattr__                        : EXCEPTION:
    could not find attribute:namespace in gedisclient_explorer (error was:'GedisClient' object has no attribute 'namespace')
--TRACEBACK------------------
<stdin> in <module>
    1
/sandbox/lib/jumpscale/Jumpscale/tools/threebot/ThreebotToolsFactory.py in init_my_threebot
    43    r = j.tools.threebot.explorer.threebot_record_get(name=name, die=False)
/sandbox/lib/jumpscale/Jumpscale/tools/threebot/ThreebotExplorer.py in threebot_record_get
    36    r = self.actors.phonebook.get(tid=tid, name=name, die=die)
/sandbox/lib/jumpscale/Jumpscale/tools/threebot/ThreebotExplorer.py in actors
    21    return self._client.actors_default
/sandbox/lib/jumpscale/Jumpscale/clients/threebot/ThreebotClient.py in actors_default
    45    return self.actors_get("default")
/sandbox/lib/jumpscale/Jumpscale/clients/threebot/ThreebotClient.py in actors_get
    30    g = j.clients.gedis.get(name=self.name, host=self.host, port=self.port, namespace=namespace)
/sandbox/lib/jumpscale/Jumpscale/core/BASECLASSES/JSConfigsBCDB.py in get
    129    if not getattr(jsconfig, key) == val:
/sandbox/lib/jumpscale/Jumpscale/core/BASECLASSES/Attr.py in __getattr__
    68    raise j.exceptions.NotFound(msg)
-----------------------------

This is blocking for the release of grid 2.0.1.beta2, specifically:

https://github.com/threefoldtech/home/issues/429
https://github.com/threefoldtech/home/issues/430
https://github.com/threefoldtech/jumpscaleX_threebot/issues/128

Fixed the threebot client here https://github.com/threefoldtech/jumpscaleX_core/commit/298f332dec30710adbfdb8bb64859ea9f529fcf6 and fixed the explorer commands execution here https://github.com/threefoldtech/jumpscaleX_core/commit/f7fa4978aabff88315b7f81c2e07b785bf37319f

Tested with your fixes @serboctor. I don't have the same bugs anymore but another one. This is the error server side:
Thu 05 15:48:07 rializerJSXObject.py -  73 - loads                              : EXCEPTION:
    AssertionError()
--TRACEBACK------------------
/sandbox/lib/jumpscale/Jumpscale/servers/gedis/handlers.py in _handle_request
    398    result = cmd_method(*params_list, user_session=user_session, **params_dict)
/sandbox/lib/jumpscale/Jumpscale/core/BASECLASSES/Decorators.py in wrapper_action
    60    res = func(self, **kwargs)
/sandbox/code/github/threefoldtech/jumpscaleX_threebot/ThreeBotPackages/tfgrid/phonebook/actors/phonebook.py in get
    182    users = self.phonebook_model.find(name=name)
/sandbox/lib/jumpscale/Jumpscale/data/bcdb/BCDBModel.py in find
    632    res.append(self.get(id))
/sandbox/lib/jumpscale/Jumpscale/data/bcdb/BCDBDecorator.py in wrapper_queue_method
    60    res = func(*args, **kwargs)
/sandbox/lib/jumpscale/Jumpscale/data/bcdb/BCDBModel.py in get
    498    obj = self.bcdb._unserialize(obj_id, data, return_as_capnp=return_as_capnp, schema=self.schema)
/sandbox/lib/jumpscale/Jumpscale/data/bcdb/BCDB.py in _unserialize
    712    obj = j.data.serializers.jsxdata.loads(bdata, bcdb=self, schema=schema)
/sandbox/lib/jumpscale/Jumpscale/data/serializers/SerializerJSXObject.py in loads
    73    assert schema_old.url == schema.url

Seems like schema version bug is back.

After some more fixes, I tested it yesterday against a running explorer, and it worked fine.


Integration Test - Release - 2022/01/27 09:05:09

Auto create for integration test. AnotherAuthorUpdatedArticleFlow

@prmerger-dev-1 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Docs Build status updates of commit a743f32:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch release-82b5c7d605df4c2786da54605d09a737 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Missing file

CMake Error: File /scratch/gdsjaar/seacas-parallel/TPL/netcdf/netcdf-c/nc_test4/run_par_test2.sh.in does not exist.
CMake Error at CMakeLists.txt:1011 (configure_file):
  configure_file Problem configuring file

CMake Error at CMakeLists.txt:1016 (FILE):
  FILE COPY cannot find
  "/scratch/gdsjaar/seacas-parallel/TPL/netcdf/netcdf-c/build/tmp/run_par_test2.sh".

This is with current master.

I will fix...

Note that this results in a build failure when parallel is enabled.

@gsjaardema can you send me the cmake command you are using to build?

You have to undo commit commit 3823307b9e461acacc931b5c3376a6ea143ad393
Author: Ward Fisher<EMAIL_ADDRESS>Date:   Tue Mar 26 15:38:22 2019 -0600

    Disable run_par_test2.sh.in template processing code temporarily.

Then, my cmake command is:
cmake ..   Unless there is a file @edhartnett wants to add in, I'd say that this isn't so much an issue, as the issue was resolved by commenting out the lines in the CMakeLists.txt file.  If there isn't a file to be added back in, we can probably remove the commented out lines altogether;  I'll take care of that while prepping the 4.7.1 release in the next day, unless I hear differently.

OK, I have confirmed that there is no run_par_test2.sh file to include or run. The parallel testing is all done in run_par_test.sh, and that's working fine.
Also, I don't see run_par_test2.sh in any of the CMakeLists.txt files, except the commented out lines in the main CMakeLists.txt file, which can actually just be removed.
Seemingly run_par_test2.sh was an artifact of development activities and we can just remove all mention of it.


Per-codon simulated data tests

Request that add some @amandamok's simulated ribosome profiling data into RiboViz as another vignette. This should provide ideal testing that reads are correctly aligned and of codon-level and gene-level statistics.
Suggest:

To test alignment/counts/ribogrid are functioning correctly, generate a dataset with only 3 genes of varying length and coverage. Then we can write automated regression tests to check that the reads are correctly aligned. (Software engineering regression not statistical regression). Ideal upload would include counts-per-a-site used to generate the data and in the format we will use for regression, simulated fastq, and transcriptome / genome sequences and gff.
To test that statistics are behaving correctly on per-codon parameters, a larger dataset such as half a yeast chromosome (YALnnnx genes) or indeed the entire yeast genome. Then we can find out how well the codon-level and gene-level statistics are behaving based on ground truth. It will take more thought to figure out automated tests for this.

We should discuss exactly what is needed. Request this goes in new branch codon-sim-data-112.

As agreed on 18 Dec meeting, @amandamok to questions about data on this ticket.
We need data that RiboViz is able to parse.
@amandamok can you please update progress on this ticket? Thanks.

Here are some sample data of footprint sequences generated from 3 s. cerevisiae transcripts:
https://github.com/amandamok/footprint-bias/tree/master/sample_data
File descriptions:

Asite_rules.txt : flat table with column name (frame of 5' nucleotide of footprint sequence) and row names (footprint length). Table cells refer to number of nucleotides from 5' end of footprint to A site codon. For example, a 27 nt footprint in frame 1 has 14 nucleotides on the 5' end of the sequence, 3 nucleotides of A site codon, and 10 nucleotides on the 3' end of the sequence.

riboViz_transcripts_18cds15.fa : fasta file of transcript sequences; introns removed; includes 18 nucleotides upstream of start codon and 15 nucleotides downstream of stop codon

riboViz_simAsite.fa.gz : zipped fasta file of footprint sequences

riboViz_simASite_footprints.sam.gz : zipped sam file of footprint alignments

riboViz_simASite_countsPerPosition : flat table with footprint counts per codon position (includes header row)

column 1: transcript (string)
column 2: position of A site codon (integer; 1-based)
column 3: footprint count (integer)

riboViz_simASite_countsTable.txt : flat table with footprint counts per codon position per frame and footprint length (includes header row)

column 1: transcript (string)
column 2: position of A site codon (integer; 1-based)
column 3: A site codon identity (string; 3 of {A,C,T,G})
column 4: P site codon identity (string; 3 of {A,C,T,G})
column 5: E site codon identity (string; 3 of {A,C,T,G})
column 6: number of nucleotides between 5' end and A site (integer; between 14 and 18)
column 7: number of nucleotides between A site and 3' end (integer; between 9 and 11)
column 8: 2 nucleotides at 5' end of footprint (string; 2 of {A,C,T,G})
column 9: 2 nucleotides at 3' end of footprint (string; 2 of {A,C,T,G})
column 10: footprint count (integer)

When release 2.0.beta has been built (#99) and @mikej888 has done Build 2.0.beta DataSync and GitHub data resources #137, then @amandamok and/or @FlicAnderson could run RiboViz on @amandamok's data and create a GitHub repository with the output data from the run, for use as another regression test.

The current version is in the nextflow-167 branch, https://github.com/riboviz/riboviz/tree/nextflow-167 (this will be merged into the develop branch in due course)

I've contacted @amandamok to set up a time to look at setting this up.  I'm hoping to get things properly moving on this in the next month, depending on other tasks.
Now we'll be aiming to run successfully using Nextflow method, on the develop branch (since things have moved on a little since previous discussion here.

New batch of simulated data! in the appropriate format for for the Riboviz pipeline.
https://github.com/amandamok/simRiboSeq/tree/master/simulation_runs/riboviz
Summary of data:

same transcriptome and gff3 file as in RiboViz vignette input directory (yeast_YAL_CDS_w_250utrs)
100,000 footprints generated from 5 genes (YAL003W, YAL005C, YAL012W, YAL035W, YAL038W)
features contributing to footprint counts: A site codon identity, 5' and 3' end nucleotides
footprints concatenated to CTGTAGGCACCATCAAT adaptor sequence, output as fastq

Files:

simRiboviz.fq.gz: zipped footprints in fastq format
simRiboviz_footprints.bam.gz: zipped bam file of footprint alignments
simRiboviz_footprints_counts.txt: flat text table with footprint counts per transcript, codon, frame, and length

Hi Amanda, I have forked this and will take a detailed look including writing a config.yaml (example-datasets#31).
A couple of observations:

the .bam alignment file is made with bowtie, but riboviz uses hisat2 (and not yet bowtie). This means the output is likely to have small differences, but I don't know if that's important. I can test that.
simRiboviz_footprints_counts.txt is ordered first by position, then by gene. It's more intuitive for me to have it ordered first by gene, then by position. Then counts at adjacent positions in the same gene would be next to each other in the table. I can try and pr to your repo, unless you'd like to change that?

Hi Amanda, I have forked this and will take a detailed look including writing a config.yaml (example-datasets#31).
A couple of observations:

the .bam alignment file is made with bowtie, but riboviz uses hisat2 (and not yet bowtie). This means the output is likely to have small differences, but I don't know if that's important. I can test that.
simRiboviz_footprints_counts.txt is ordered first by position, then by gene. It's more intuitive for me to have it ordered first by gene, then by position. Then counts at adjacent positions in the same gene would be next to each other in the table. I can try and pr to your repo, unless you'd like to change that?

Updated:

simRiboviz_footprints.sam.gz: zipped sam file of footprints aligned with hisat2
simRiboviz_footprints_counts.txt now ordered by transcript -> codon position -> footprint length -> frame of 5' end

Updated:

simRiboviz_footprints.sam.gz: zipped sam file of footprints aligned with hisat2
simRiboviz_footprints_counts.txt now ordered by transcript -> codon position -> footprint length -> frame of 5' end

We can run riboviz on this simulated data, as of closing pull request example-datasets#34.
We can now build on this issue, per-codon simulated data tests, to test if the R code accurately recovers per-codon differences in read density.
This issue intersects with #213, per-codon analysis is one of the key functions to test.

We can run riboviz on this simulated data, as of closing pull request example-datasets#34.
We can now build on this issue, per-codon simulated data tests, to test if the R code accurately recovers per-codon differences in read density.
This issue intersects with #213, per-codon analysis is one of the key functions to test.


Not working in Brave and Edge

This extension is not working in brave and edge as I am submitting the code but it is not updating my github repository and the extension.

Can't use it in Brave lol


Add correct dates upon Patient registration and Immunization tracking

IMPORTANT: Where possible all PRs must be linked to a Github issue
Fixes #195
Type
Choose one: (Bug fix | Feature | Documentation | Testing | Code health | Release | Other)
Checklist

[ ] I have written Unit tests for any new feature(s) and edge cases for bug fixes
[ ] I have updated the  CHANGELOG.md file for any notable changes to the codebase
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the project's style guide
[x] I have built and run the fhircore app to verify my change fixes the issue and/or does not break the app

I think we can close this, it's going to be cherry picked not merged, is that correct?


Add test coverage for creating a venv with an active venv

Okay at least this is intentional when --python is provided per https://github.com/astral-sh/uv/pull/3728

I'm still not sure if we should be reading interpreters from the active environment. I can't find a place where we discussed that.


add minimumRps parameter on Cache instance demonstration (README)

Tried to use the lib, but for some reason the cb was not working propertly.
Reading the code, I noticed that the minimumRps parameter existed and that it was a source of my problem.
So, I'm suggesting to add this parameter just like that:
const cache = new Cache({
  redis, // required to inject a redis client
  options: { // everything is optional from here
    name: 'someService', // name of the service the cache will handle, useful for RTA
    circuitBreaker: { // circuit breaker config for the service
      // percentage of failing producer requests to trigger the circuit breaker
      threshold: 0.2,
      // time window to be considered by the threshold, in milliseconds
      duration: 10000,
      // attempt to half open the circuit after this time in milliseconds
      halfOpenAfter: 20000,
      // don't open the circuit if there's less than this amount of RPS.
      // This avoids opening the circuit, during failures in low load periods.
      minimumRps: 5,
    },
  },
});

(instructions copy-pasted from classes/policies)

closed by #24


cassandra22 - cqlsh error AttributeError: 'list' object has no attribute 'has_more_pages'

after a fresh install of cassandra22, any select query, whether from an existing system table or a user defined table, results in the the above error, debug output below:
➜  ~  brew install cassandra22
==> Installing cassandra22 from homebrew/versions
==> Downloading https://www.apache.org/dyn/closer.cgi?path=/cassandra/2.2.4/apache-cassandra-2.2.4-bin.tar.gz
==> Best Mirror http://mirrors.sonic.net/apache/cassandra/2.2.4/apache-cassandra-2.2.4-bin.tar.gz
######################################################################## 100.0%
==> Downloading https://pypi.python.org/packages/source/s/setuptools/setuptools-12.0.5.tar.gz
######################################################################## 100.0%
==> python -c import setuptools... --no-user-cfg install --prefix=/usr/local/Cellar/cassandra22/2.2.4/libexec/vendor --single-version-externally-managed --reco
==> Downloading https://pypi.python.org/packages/source/t/thrift/thrift-0.9.2.tar.gz
######################################################################## 100.0%
==> python -c import setuptools... --no-user-cfg install --prefix=/usr/local/Cellar/cassandra22/2.2.4/libexec/vendor --single-version-externally-managed --reco
==> Downloading https://pypi.python.org/packages/source/f/futures/futures-2.2.0.tar.gz
######################################################################## 100.0%
==> python -c import setuptools... --no-user-cfg install --prefix=/usr/local/Cellar/cassandra22/2.2.4/libexec/vendor --single-version-externally-managed --reco
==> Downloading https://pypi.python.org/packages/source/s/six/six-1.9.0.tar.gz
######################################################################## 100.0%
==> python -c import setuptools... --no-user-cfg install --prefix=/usr/local/Cellar/cassandra22/2.2.4/libexec/vendor --single-version-externally-managed --reco
==> Downloading https://pypi.python.org/packages/source/c/cql/cql-1.4.0.tar.gz
######################################################################## 100.0%
==> python -c import setuptools... --no-user-cfg install --prefix=/usr/local/Cellar/cassandra22/2.2.4/libexec/vendor --single-version-externally-managed --reco
==> Downloading https://pypi.python.org/packages/source/c/cassandra-driver/cassandra-driver-2.6.0.tar.gz
######################################################################## 100.0%
==> python -c import setuptools... --no-user-cfg install --prefix=/usr/local/Cellar/cassandra22/2.2.4/libexec/vendor --single-version-externally-managed --reco
==> Caveats
To load homebrew/versions/cassandra22:
  launchctl load ~/Library/LaunchAgents/homebrew.mxcl.cassandra22.plist
==> Summary
🍺  /usr/local/Cellar/cassandra22/2.2.4: 4,707 files, 103.6M, built in 16 seconds
➜  ~  launchctl load ~/Library/LaunchAgents/homebrew.mxcl.cassandra22.plist
➜  ~  cqlsh --debug
Using CQL driver: <module 'cassandra' from '/usr/local/Cellar/cassandra22/2.2.4/libexec/vendor/lib/python2.7/site-packages/cassandra/__init__.pyc'>
Using connect timeout: 5 seconds
Connected to Test Cluster at <IP_ADDRESS>:9042.
[cqlsh 5.0.1 | Cassandra 2.2.4 | CQL spec 3.3.1 | Native protocol v4]
Use HELP for help.
cqlsh> desc keyspaces;

system_traces  system_auth  system  system_distributed

cqlsh> use system_auth;
cqlsh:system_auth> desc tables;

resource_role_permissons_index  role_permissions  role_members  roles

cqlsh:system_auth> select * from roles;

Traceback (most recent call last):
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1110, in onecmd
    self.handle_statement(st, statementtext)
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1147, in handle_statement
    return custom_handler(parsed)
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1179, in do_select
    self.perform_statement(statement)
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1184, in perform_statement
    success, future = self.perform_simple_statement(stmt)
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1235, in perform_simple_statement
    self.print_result(result, self.parse_for_table_meta(statement.query_string))
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1251, in print_result
    if result.has_more_pages and self.tty:
AttributeError: 'list' object has no attribute 'has_more_pages'
cqlsh> create keyspace test with replication = { 'class': 'SimpleStrategy', 'replication_factor': 1 };
cqlsh> use test;
cqlsh:test> create table foobar ( a uuid PRIMARY KEY, b text);
cqlsh:test> desc foobar;

CREATE TABLE test.foobar (
    a uuid PRIMARY KEY,
    b text
) WITH bloom_filter_fp_chance = 0.01
    AND caching = '{"keys":"ALL", "rows_per_partition":"NONE"}'
    AND comment = ''
    AND compaction = {'class': 'org.apache.cassandra.db.compaction.SizeTieredCompactionStrategy'}
    AND compression = {'sstable_compression': 'org.apache.cassandra.io.compress.LZ4Compressor'}
    AND dclocal_read_repair_chance = 0.1
    AND default_time_to_live = 0
    AND gc_grace_seconds = 864000
    AND max_index_interval = 2048
    AND memtable_flush_period_in_ms = 0
    AND min_index_interval = 128
    AND read_repair_chance = 0.0
    AND speculative_retry = '99.0PERCENTILE';

cqlsh:test> select * from foobar;

Traceback (most recent call last):
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1110, in onecmd
    self.handle_statement(st, statementtext)
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1147, in handle_statement
    return custom_handler(parsed)
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1179, in do_select
    self.perform_statement(statement)
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1184, in perform_statement
    success, future = self.perform_simple_statement(stmt)
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1235, in perform_simple_statement
    self.print_result(result, self.parse_for_table_meta(statement.query_string))
  File "/usr/local/Cellar/cassandra22/2.2.4/libexec/bin/cqlsh.py", line 1251, in print_result
    if result.has_more_pages and self.tty:
AttributeError: 'list' object has no attribute 'has_more_pages'
cqlsh:system_auth> exit

@ckhall Can you see if you can reproduce this on a non-Homebrew Cassandra?

@mikemcquaid working on a binary install of 2.2.4 as we speak...i'll drop the details when I finish.

@mikemcquaid works fine with binary install of cassandra 2.2.4.
➜  ~  mkdir ~/opt/packages && cd ~/opt/packages
➜  ~  cd ~/opt/packages
➜  ~/opt/packages  curl -O http://apache.arvixe.com/cassandra/2.2.4/apache-cassandra-2.2.4-bin.tar.gz
➜  ~/opt/packages  tar -xvzf apache-cassandra-2.2.4-bin.tar.gz
➜  ~/opt/packages  ln -s ~/opt/packages/apache-cassandra-2.2.4 ~/opt/cassandra
➜  ~/opt/packages  mkdir -p ~/opt/cassandra/data/data && mkdir -p ~/opt/cassandra/data/commitlog && mkdir -p ~/opt/cassandra/data/saved_caches && mkdir -p ~/opt/cassandra/logs
➜  ~/opt/packages  cd
➜  ~  ~/opt/cassandra/bin/cassandra -f

(another terminal)
➜  ~  ~/opt/cassandra/bin/cqlsh
Connected to Test Cluster at <IP_ADDRESS>:9042.
[cqlsh 5.0.1 | Cassandra 2.2.4 | CQL spec 3.3.1 | Native protocol v4]
Use HELP for help.
cqlsh> desc keyspaces;

system_traces  system_auth  system  system_distributed

cqlsh> use system_auth
   ... ;
cqlsh:system_auth> desc tables;

resource_role_permissons_index  role_permissions  role_members  roles

cqlsh:system_auth> select * from roles;

 role      | can_login | is_superuser | member_of | salted_hash
-----------+-----------+--------------+-----------+--------------------------------------------------------------
 cassandra |      True |         True |      null | $2a$10$CtlTydOgUUpnqMvcH6Bqgu/GgEUCz25KtA7rhAuhXUTNttdPmX6NC

(1 rows)
cqlsh:system_auth>

@mikemcquaid ok, so tracked this down to the 2.6.0 cassandra-driver.  it looks like the 2.2.4 binary comes with 3.0.0 driver.  I edited the formula to use 3.0.0, reinstalled cassandra22 and it works correctly now.

@ckhall thanks for the efforts, been banging my head against the wall for a few hours on this. Since others might have the same question: how do we get this updated formula?
I've run brew update, but brew edit still reflects the previous formula, and I have to manually apply this patch to get it working. Am I missing something obvious as to how to get the fixed formula?


add duckOthers option on iOS

I have added duckOthers option for iOS

also updated the index.d.ts params here for setActive method. which was wrong

@zmxv hey, I was wondering if there is anything I can do to get this PR merged? Tested it, and it worked as expected.

Any change this can be reviewed and merged?

Heads up - we patched this into our repo, this causes a crash on Android.


Users need the title of the pre-payment page to reflect the content

Assigning this to Sanjay and Lizzie to advice on the service design and content copy.

Suggest the page title is reworded:

Your permit details

This would also work as a title for the summary page for free HMO permits. I don't think the "You're about the pay" line should change. It's preparing them for the next page (payment). The page title change should solve the issue. We could possibly add:

"Check your permit details and costs below."

This isn't the right place to add the temporary permit, or mention it. The focus of this page is what you've paid for, referring to a temp permit here would be distracting. I'm presuming the start date in the table is asap, not 5 working days' time.

hey @EUzkuraityte  just to let you know I edited my comment since you thumbs upped it, just in case you want to retract the thumbs up! :-D

@lizziebruce :D I'm going to keep the thumbs up and yes, usually there is a hologram printed on the parking permit

@EUzkuraityte :-D Thanks and oh that's a shame. Also on 2. I've put the "as soon as poss" copy in 497 (referenced above).

See also PR #607

Update. I don't think this page should change title any more. It already has an alternative title for free permits: "Confirm your permit details."
I've added the line of copy "Check details and costs:" though.
After:


Intellisense docs not being packaged into NuGet

I'm not really sure why, but for some reason the Intellisense documentation in this repo isn't making it into the final NuGet package.
E.g. https://github.com/dasiths/NEasyAuthMiddleware/blob/master/NEasyAuthMiddleware/Core/EasyAuthOptions.cs#L9-L12

Looking around briefly, I wonder if something like this is necessary in the csproj?
<PropertyGroup>
    <GenerateDocumentationFile>true</GenerateDocumentationFile>
  </PropertyGroup>


Issue race condition test_deploy_no_code

Description
Fix race condition in test 'test_deploy_no_code'
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[x] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

I'm not sure this is the right fix. It fixes the test (previously I always saw at least one failure when running the test 30 times in a row, now I don't anymore), but maybe it is the code that needs a fix.

Processing this pull request

Pull request rejected by merge tool. Failed to parse change entry file: Change entry /tmp/tmp4jxbkr2v/changelogs/unreleased/fix-race-condition-version-resource.yml does not match pydantic schema: 1 validation error for ChangeEntry
issue-repo
Can't set issue-repo without setting issue-nr (type=value_error)

Processing this pull request

Merged into branches master in 2bb5979213b5690cb079b41cd8f29a6bcbd34b01

Processing #6163.

Processing #6161.


AutoCreateTable - does not set Xml Datatype for Properties Column

Using the AutoCreateTable option, I was assuming that the Properties column Datatype would be set to XML, so that I can do SQL Queries using XQuery as shown on at the bottom of the readme

Looking into this it seems the Properties column has a C# string type set
https://github.com/serilog/serilog-sinks-mssqlserver/blob/dev/src/Serilog.Sinks.MSSqlServer/Sinks/MSSqlServer/MSSqlServerSinkTraits.cs#L307
And that the SqlGetType method that is a Switch/Case checks for the C# String type and sets the SQL database column to NVARCHAR

Same request, different reason: #130
My objection (see that thread) is that logging should always optimize for write-performance first and foremost. Though I'm still tempted to agree it warrants a config option: not everybody is running the next eBay. Maybe it'se enough to default to nvarchar.
FWIW, you can do XQuery against non-XML data if you cast the nvarchar to XML (either in the SELECT or FROM clause). It's actually pretty fast. I have a system at work that has been accumulating data since 2002, we do this all the time, around a million rows, XML data around 40K or 50K/row.

Closing as duplicate, but I tagged the other issue as a feature request for an optional setting.


0.3.0 failed on lapack-test

System spec: Intel SKL / CentOS 7 / gcc-7
make PREFIX=/tmp/scratch/OpenBLAS/install.V54Ovz8vgp/root/usr/local lapack-test -j36
(cd ./lapack-netlib/TESTING && rm -f x* *.out)
make -j 36 -j 1 -C ./lapack-netlib/TESTING xeigtstc  xeigtstd  xeigtsts  xeigtstz  xlintstc  xlintstd  xlintstds  xlintstrfd  xlintstrfz  xlintsts  xlintstz  xlintstzc xlintstrfs xlintstrfc
make[1]: warning: -jN forced in submake: disabling jobserver mode.
make[1]: Entering directory '/tmp/scratch/OpenBLAS/lapack-netlib/TESTING'
make[1]: Leaving directory '/tmp/scratch/OpenBLAS/lapack-netlib/TESTING'
make[1]: *** No rule to make target 'xeigtstc'.  Stop.
make: *** [Makefile:297: lapack-test] Error 2

First build openblas library by running make
Then change to that directory TESTING and run make -sj

This appears to be an overlooked change from the update to 3.8.0 (or perhaps even earlier), these tests
have been split into separate directories, EIG for xeigtst and  LIN  for xlintst.

Also got issues with make blas-test where (cd ./lapack-net/BLAS && cat *.out) reports *.out doesn't exist.


no "ollama" option in the setting (Win 11)

I have installed Ollama, however it does not appear in the option setting.
I am using Windows 11.

@jen-art-dev it should be working now, master was broken.  If you pull from main and reinstall it should work.  Safest way to reinstall is the following from the backend directory:
pip uninstall openui
pip install .

Let me know if that doesn't work.

@jen-art-dev it should be working now, master was broken. If you pull from main and reinstall it should work. Safest way to reinstall is the following from the backend directory:
pip uninstall openui
pip install .

Let me know if that doesn't work.

hi @vanpelt, thank you but it doesn't work.

It works now, I set the env in PowerShell:
$env:OLLAMA_HOST="http://<IP_ADDRESS>:11434"
Thanks a lot @vanpelt for this great project


Statement on support for newer versions of SharePoint on-premises?

There is a question on this page, "Does Azure AD B2C work with SharePoint on-premises 2016 or earlier?"
Can we please have an updated statement on whether B2C is supported for Subscription Edition/etc?

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

ID: fc5e148d-b7cf-92e7-4cc7-a65d20d037b6
Version Independent ID: 2ba6e31b-5899-f35a-0f08-d280110ded13
Content: Frequently asked questions (FAQ) for Azure Active Directory B2C
Content Source: articles/active-directory-b2c/faq.yml
Service: active-directory
Sub-service: b2c
GitHub Login: @garrodonnell
Microsoft Alias: godonnell

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

@garrodonnell
can you look into this case?

Any updates?

We've added this issue to our documentation backlog work item for internal tracking and prioritization. We'll update this issue as documentation updates are published. Thanks again for your feedback. #please-close


Investigate using Crossbeam lock-free queue + threads

There seems to be a fair amount of locking going on in the threadpool we're using. This, combined with how short the actual tasks to run are mean using a thread pool doesn't actually seem to make anything faster.
Let's investigate using the lock-free queue in Crossbeam along with some sort of custom thread worker. It's probably sufficient to only run a single thread worker as well.
http://aturon.github.io/crossbeam-doc/crossbeam/sync/struct.MsQueue.html

Performance looks promising so far.
test test_benchmark_statsdclient_count_async_buffered_udp ... bench:         502 ns/iter (+/- 160)
test test_benchmark_statsdclient_count_buffered_udp       ... bench:         391 ns/iter (+/- 131)
test test_benchmark_statsdclient_count_cb_buffered_udp    ... bench:         344 ns/iter (+/- 254)
test test_benchmark_statsdclient_count_nop                ... bench:         165 ns/iter (+/- 3)
test test_benchmark_statsdclient_count_udp                ... bench:       2,960 ns/iter (+/- 151)

The "cb" benchmark is the one that uses crossbeam.


Select tags options overflow from bootstrap modal-body and gets out of parent div

I have a modal that works correctly.
Inside a modal I have a form with select tag generated from database with long list of names. when I click select the list gets out of parent div and even out of browser. How do I keep select inside of modal-body?

		    	<form action="Action.cfm" method="POST" class="form-group">

				        <button type="button" class="close" data-dismiss="modal">×</button>

Machine ID =

			      			<cfoutput>
					      		<input type="hidden" name="actionID" value="">
								<input type="hidden" name="tMID" value="">
					        	<select class="form-control input-lg">
					        		<optgroup>
					        			<cfloop query="operator">
											<option value="#EmployeeID#" <cfif session.tOperator IS EmployeeID>selected</cfif>>#ename#</option>
										</cfloop>
					        		</optgroup>
					        	</select>
				        	</cfoutput>

			        	<button type="submit" class="btn btn-default" data-dismiss="modal">Change Screen</button>

		    	</form>

```

As far as I know, there is no way to contain native browser elements to their parent like that. We have no control over it on our end. If you absolutely need that, I'd recommend building your own menu.

Got it, thanks. My work around was HTML5 datalist


cask-repair: the requested upstream branch does not exists

After all steps listed here, cask-repair is not able to create pull request.
Script command:
cask-repair --pull origin --push MilusW yourkit-java-profiler
Script output:
Submitting…
error: the requested upstream branch 'MilusW' does not exist
hint:
hint: If you are planning on basing your work on an upstream
hint: branch that already exists at the remote, you may need to
hint: run "git fetch" to retrieve it.
hint:
hint: If you are planning to push out a new local branch that
hint: will track its remote counterpart, you may want to use
hint: "git push -u" to set the upstream config as you push.
Error creating pull request: Unprocessable Entity (HTTP 422)
Invalid value for "head"
Error creating pull request: Unprocessable Entity (HTTP 422)
Invalid value for "head"

git remote -v:
MilusW  git@github.com:MilusW/homebrew-cask.git (fetch)
MilusW  git@github.com:MilusW/homebrew-cask.git (push)
origin  https://github.com/caskroom/homebrew-cask (fetch)
origin  https://github.com/caskroom/homebrew-cask (push)

brew info cask-repair hub | grep -E '(cask-repair:|hub:)':
vitorgalvao/tiny-scripts/cask-repair: stable 0.24.1
hub: stable 2.2.3 (bottled), HEAD

I have done everything from #20, but it still doesn't work.

The branch exists. Delete it and try again.

I have done it several times - manually and with cask-repair --push MilusW --delete-branches. Still the same output.
cask-repair is able to successfully push new branch to my repository, commit changes, but it is unable to create pull request.

Edit your local copy of cask-repair and make the line 393 read git push -u "${remote_push}" "${cask_branch}" --quiet instead of git push "${remote_push}" "${cask_branch}" --quiet.
If that doesn’t work, I’ll have to abandon diagnosing it, as I cannot reproduce. Like in #20, I’ve spent a lot of time on this already, and it was all a waste. What I cannot reproduce I cannot fix unless I am give the cause (which was the case with that issue).

Unfortunately, it doesn't work. Thank you anyway

Thank you for reporting it. If you can get closer to the reason it is happening, I’ll take a look at it, but I’ve pretty much exhausted all the blind suggestions in #20. This is particularly hard to fix since hub seems to be the main culprit.

Sorry if I'm just adding noise by reopening.  I had successfully used cask-repair previously, but today was having the same issues trying to update a different formula.
I think I narrowed down the issue to upstream handling in git branch, line 394. I exited the script after the failure and did some inspection. I was able to set origin as an upstream without a problem, and master was automatically used as the tracking branch. However, specifying my jbylsma remote would not work, returning
[/usr/local/Library/Taps/caskroom/homebrew-cask/Casks]
jbylsma$ git branch --set-upstream-to=jbylsma
error: the requested upstream branch 'jbylsma' does not exist
hint:
hint: If you are planning on basing your work on an upstream
hint: branch that already exists at the remote, you may need to
hint: run "git fetch" to retrieve it.
hint:
hint: If you are planning to push out a new local branch that
hint: will track its remote counterpart, you may want to use
hint: "git push -u" to set the upstream config as you push.

I reran the git branch using dtrace and noticed that there was a failed call to  the remote's HEAD:
47746/0x392b1:  lstat64(".git/refs/remotes/jbylsma/HEAD\0", 0x7FFF548E6C70, 0x1000)              = -1 Err#2

origin still had it's HEAD, which is probably why I could switch to it. I'm not sure when I lost or if I ever had HEAD in jbylsma, but maybe that's the root hub issue.
Anyways, if the lack of HEAD is the culprit, the easiest fix is to explicitly set to the upstream to remote/branch. I was able to successfully use cask-repair with the upstream update, and I think the explicit upstream branch shouldn't cause any problems. I'll open a pull request with the change for review.

I have the same issue. Seems to it was not fixed yet.


Towards personalized SSG

Overview

What is it?

[ ] Feature / enhancement
[ ] Bug
[X] Docs / tests / types / typos

Description
Hi folks,
I am just discovering Qwik, coming from Next.js with a solid experience regarding static rendering.
My pet peeve is the belief that static rendering is best for content that is the same for all visitors. I think this doesn't give static rendering justice, because SSG is also perfectly viable for content that is the same for groups of visitors.
So, my proposal:

first I've changed the doc in this PR to be clearer about the fact that SSG is not limited to content that is the same for all visitors. More on that below.
then, I think it would be interesting to open up the discussion and document more advanced patterns, like redirecting to static content depending on the request. Perhaps this has been discussed already, or considered very easy, then it would be interesting to include the relevant links directly in the documentation.

This is in my opinion a relatively important matter when trying to compare Qwik with say Next.js on use cases where Next.js shines, namely content app (e-commerce websites, media websites, basically any app where you have a strong marketing service behind with contradictory requirements between rich content, personalization and performances).
I'd be eager to get Qwik developers stance on this idea!
Use cases and why
The difference between "same for everybody" and "same for groups of people" seems subtle, but actually opens up so many use cases for static rendering:

A/B tests: some users see version A, some version B. But A and B are known in advance: let's statically render that!
i18n: some users see language FR, some se language EN, DE...
marketing segmentation: "top-notch VIP", "people who won't buy without a coupon", "new customers"
dark/light theme
multi-tenancy: the company logo in the layout changes depending on the organization you belong to.

And more broadly whatever personalization pattern you may encounter in the wild.
For previous art, I've opened similar issues with explanations:

for Persus, a Next.js inspired Rust framework: https://github.com/framesurge/perseus/issues/295
for Next.js: https://github.com/vercel/next.js/pull/36018

And I've written articles on the topic:

"A New Pattern For The Jamstack: Segmented Rendering" for Smashing Magazine https://www.smashingmagazine.com/2022/07/new-pattern-jamstack-segmented-rendering/
Same idea presented at React Advanced London: https://portal.gitnation.org/contents/treat-your-users-right-with-segmented-rendering
The math behind SSR/SSG (a bit old and hard to read but lay down some ideas): https://tinyurl.com/ssr-theory

That's for the rough idea. If you think a bit about this idea a bit more, you should quickly understand that the key to this personnalisation is to redirect the user towards the right version of the page.
In Next.js, this can be done natively using middlewares. I deplore the fact that Next.js only allows static rendering based on a route parameter, and not say cookies, headers, or URL search params. Hopefully this limitation can be bypassed by using URL rewrites. I've written another aticle showing this pattern: "Render anything statically with Next.js and the Megaparam" https://blog.vulcanjs.org/render-anything-statically-with-next-js-and-the-megaparam-4039e66ffde"
I think Qwik has a similar approach so could benefit from this pattern too but not sure about the specific implementation.
Lastly, you might think: "ok but this is actually SSR coupled with HTTP caching, not SSG". Yes but no, doing static personalization at "framework level" is more flexible. HTTP cache is not evenly supported, namely it's hard to cache variations depending on a subtile change like the precise value of one cookie (Vary header and such).
Bonus: you may encounter another kind of personalization, that is altering the page at the Edge (Eleventy-style). This is great too but often limited to slight adaptations, because you have to alter already rendered HTML so you can't rely on the rendering framework at this stage.

FWIW, I've been doing cached SSR in my react apps since 2015 or so, make a key based on hash of build, request, user, mobile browser etc, and use that to check a cache, falling back to SSR. Works great.
For me though, SSG is for allowing deploying without any server component. Is that not the case?

FWIW, I've been doing cached SSR in my react apps since 2015 or so, make a key based on hash of build, request, user, mobile browser etc, and use that to check a cache, falling back to SSR. Works great.

I wish this was more common! Modern frameworks do not really let you do that easily, you have to twist Next.js a bit to achieve that and I am still trying to learn how it's done in Qwik/Vite.

For me though, SSG is for allowing deploying without any server component. Is that not the case?

I think a distinction should be made between having no rendering server and having no server at all. Cached SSR is about avoiding to useless re-renders but it doesn't tell much about when you render, either build-time or request-time, so let's suppose you render at first request and cache the thing.
The point of SSG is indeed to allow deployment without a rendering server (read a full-fledged Node.js server that can run JS), so you have to render everything at build-time.
But that doesn't mean you cannot have a server. Actually you need only one feature to support personalized static rendering: URL rewriting, in order to point a given request to the right prerendered path (which acts as your cache key). Nginx, Route 53, Edge functions/middlewares are smart enough to support that. You can assume that this feature is not hard to implement if the developer really wants it and more broadly that nothing prevents you from rendering personalized content at build-time that is different for many users.

Thanks a lot for this effort. I am building an online ordering and invoicing SPA using Qwik and it is totally SSG served by nginx, a Jamstack. After each page load I fetch the need data from Pocketbase. I loose some of the benefits of Qwik like server actions, but I found that jamstack is better me.


When trying to use "Beacons.detectIBeacons()" getting error undefined is not an object (evaluating 'beaconsAndroid.addParser')

I checked the link also -
react-native link
rnpm-install info Android module react-native-beacons-android is already linked
Any other thing i can check.

Why did you already closed the issue? 😄
However I'm feeling it must be related to #14, did you already check it?
What version of React-Native are you using?

I just tested it, if you're on RN >= 0.30 you must link the project using react-native link and it will link react-native-android-beacons correctly.
I'm closing the issue but feel free to comment here 👍

Thanks for the reply, the issue got solved.
The RN version is react-native: 0.31.0.
I am facing another problem in detecting the beacons, the error is - "No Bluetooth adapter.  beaconService cannot scan." Any help is greatly appreciated.

No worries.
That error sounds weird, are you using an emulator/virtual device?

Yes I am using Android emulator .
Is it something related to this. Because when I am using my phone android 4.3 then on real device it's giving me error that batchedBridge is not bundled.

C:\Users\jcbar\Desktop\affibox\reactnative\RuralKing\node_modules\react-native\Libraries\JavaScript…:61 Cannot read property 'addParser' of undefined

Have same error

Please follow this steps and let me know the results 👍

I just added
import com.mmazzarolo.beaconsandroid.BeaconsAndroidPackage;

and
 new BeaconsAndroidPackage()

to my MainApplication.java and it solved issue. Reinstalling or using react-native link did not help.

@olegdizus On which version of React-Native?
It worked for me on both RN0.30 and RN0.31

@mmazzarolo react-native 0.29.0

Ah, that version seems to have some problem with linking...

At RN 0.31.0 linking works pretty well. Thanks for help!

No worries, have fun 👍


Support for torch.Tensor data types

The learners should support torch.Tensor datatypes, see: #109

It works for modAL==0.3.6

it does not work with newer release modAL 0.4.1 ^^
going to try 0.3.6 :)


Correction for raw affiliation CEREA, École des Ponts and EDF R&D, Île-de-France, France

Correction needed for raw affiliation CEREA, École des Ponts and EDF R&D, Île-de-France, France
raw_affiliation_name: CEREA, École des Ponts and EDF R&D, Île-de-France, France
new_rors: 02nwvxz07;03a84yw54;03wb8xz10
previous_rors: 02nwvxz07;03a84yw54
works_examples: W4385329453;W4382491292;W4322006236
contact: 076239f12438ce2bb9c04698d653d920:e8c95459e040109780a5847a79c8e1 @ recherche.gouv.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Speaker sponsor Saeloun

Sponsoring: Vipul AM

URL: https://www.saeloun.com/

Can you ask them to add RubyConf TH to that page: https://www.saeloun.com/conferences.html ?

Asked.


Fancy zones stopped working & then taskbar icon clicking not working

Environment
Microsoft Windows [Version 10.0.18363.778] (1909)
PowerToys 0.17.0
FancyZones
Steps to reproduce
Had set Fancy zones with a 3 wide by 2 down grid (so 6 panes) with space around zones set to 10 on 1 of my 3 monitors (all external to the laptop I use).
Was working fine initially when I dragged 6 apps into each pane on my non-primary 4K monitor
Later I moved one of the windows to another monitor but when I dragged it back (with shift pressed) the screen didn't change to having the grid on it and it didn't snap into place.
I went into the settings from the taskbar icon but clicking on "Edit Zones" did nothing
After I quit the PowerToys settings window, right clicking on its taskbar icon did nothing
After I killed the PowerToys.exe process (after taking a memory dump which is 106MB compressed) and restarted it, everything started working fine again
Expected behavior
Window snaps into grid position after showing grid outline on the monitor I had set up
Actual behavior
Window didn't snap into grid position and didn't show grid lines on any monitor
Screenshots

Hi @guyrleech
thank you for reporting the issue, the problem is tracked in https://github.com/microsoft/PowerToys/issues/2563
Please check this comment https://github.com/microsoft/PowerToys/issues/2563#issuecomment-622462804
I'm closing this issue as duplicate.
Thank you.


Steps: new component

Designs
https://www.figma.com/file/YsTklBecdddy9RZ3HXddIseD/?node-id=11787%3A47920
Steps

A component that can have 2+ Step  components
Can be oriented vertically or horizontally

Step

2 sizes (large, medium)
Consists of a marker, title, optional description
Has states for incomplete, current, completed
Optional description is only shown when step is in the "current" state
The number in the step marker for current/incomplete can be visible or not visible
Can be clickable (and have a hover/active state) so you can navigate to previously completed steps

Done. https://design-system.pluralsight.com/components/steps


Fix displaying multilang strings in docs

See here: https://riverbench.github.io/datasets/muziekweb/dev/#general-information

There is something wrong with the item name of the institute
The two lang variants of the name should have some information about the language (e.g., italicized (en) or something).

I've fixed the issue in the ci-worker (let's wait and see how it works), it now appends the language tag in italics at the end of the language string. On mouse hover, the full name of the locale is displayed – this works even with funky stuff like country codes, language variants, different scripts, and so on. All courtesy of ICU4J.

Done!


Resource existence check should handle response 403

Found 2 scenarios where AEM is responding with 403 when:

user existence check couldn't find an authorizable ID and hence it ends up checking the path without the node, e.g. /home/users/c/
replication/flush agent deletion attempting to delete a node that doesn't exist (maybe already deleted) also results in 403 instead of 404, ditto with existence check

For the above cases, 403 should be treated as a node not found. Ideally it should be a 404, but that's not what AEM is responding with.

Done. Included in 1.0.13 .


tests: Enable checkstyle for tests

Summary of the changes / Why this improves CrateDB

Fix checkstyle warnings for tests.
Add checkstyle changes to .git-blame-ignore-revs
Add instruction for devs to use the .git-blame-ignore-revs file.
Enable checkstyle checks for tests/testFixtures
Add suppressions of certain checks for test files and enable checkstyleTest and checkstyleTestFixtures to run with github actions on PRs.

Checklist

[ ] Added an entry in CHANGES.txt for user facing changes
[ ] Updated documentation & sql_features table for user facing changes
[ ] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

unrelated test failure:
REPRODUCE WITH: ./gradlew :server:test --tests "org.elasticsearch.indices.recovery.IndexRecoveryIT.testRerouteRecovery" -Dtests.seed=56E2BFC22C031774 -Dtests.locale=ff-Adlm-GH -Dtests.timezone=Europe/Tiraspol

Another unrelated failure:
REPRODUCE WITH: ./gradlew :server:test --tests "io.crate.integrationtests.PostgresJobsLogsITest.testBatchOperationStatsTableFailure" -Dtests.seed=E4DF4AD709178245 -Dtests.locale=mk-Cyrl-MK -Dtests.timezone=EAT

@mfussenegger Do you think we should backport it to 4.8?

No strong opinion but small tendency towards yes to avoid conflicts when backporting other fixes.

retest this please


Added filters and search to chooser modal view, which needed to be in…

… the manage-view body because of it being a slideable modal. Made style tweaks accordingly. Also added a allowedInChooser option to filters, which, if explicitly set to false, doesn't display that filter in choosers (for filters like trashed and published, which shouldn't be used when joining)
Ref #500
@boutell

Looks good, one question: does it really make sense to forbid joining with unpublished things in all cases? For one thing, you could use it to set relationships up in advance of publication.

Merging in the meantime but we should hash that out.

Yeah, I'm not sure what we ultimately want to do in those particular cases, though trash would almost certainly make sense to hide. It's configuration now though, so we can adjust this depending on our decision.


log response

Jira link (if applicable)
https://tools.hmcts.net/jira/browse/EM-5846
Change description
log ccd  callback response dto to check failure in demo

no longer required as ticket resolved


Migrate spring-webflux-security-xsuaa-usage to use spring-security

spring-webflux-security-xsuaa-usage is still using deprecated spring-xsuaa dependency, it should be migrated to use spring-security instead

Hi,
If I understand it right, spring-security doesn't have reactive support yet. We would still need to use spring-xsuaa for reactive use cases. Could you please confirm?
Thanks,
Sathish

Hi,
the reactive support is not yet available in the spring-security module so the only way at the moment is to use spring-xsuaa and migrate at later stage to spring-security once it's extend with the reactive support.
BR,
Liga

Done in #1439.


ORC-1717: Add geometry type

What changes were proposed in this pull request?
Add a new geometry type to Apache ORC.
Why are the changes needed?
Apache Iceberg is adding geospatial support: https://docs.google.com/document/d/1iVFbrRNEzZl8tDcZC81GFt01QJkLJsI9E2NBOt21IRI. It would be good if Apache ORC can support geometry type natively.
How was this patch tested?
N/A

Thank you, @wgtmac . Is it for Apache ORC 1.1.0? If this is final, I'm +1.

Also, cc @williamhyun

@dongjoon-hyun This is not finalized yet. I will update it once ready.

Thank you! If it's finalized, please send a head-up email to dev@orc once more.

Thank you for updating. Is this ready, @wgtmac ?

Yes, I believe so. And I will start PoC implementations later.

Got it. When do you want to release orc-format v1.1?

Now to help the PoC?
After finishing PoC?

I think we can wait until both Java and C++ PoC implementations are finished.

+1 for the decision. Thank you for the confirmation.


[Android] German Translation. Added missing lines.

German translation. Added all missing lines. It's not a perfect translation, so if any German native speaker is reading this, please help us :)

Thank you so much!


Fix wrong REMB detection with Chrome 60

Description
Fixes #938
Thanks to @kekkokk for the heads up. Turned out to be that Chrome is sending REMB packets within compound audio RR. And they can sometimes have a SSRC different from video's. [1]
[] It needs and includes Unit Tests
Changes in Client or Server public APIs
Not needed.
[] It includes documentation for these changes in /doc.
[1] https://bugs.chromium.org/p/webrtc/issues/detail?id=7860

WHOA!
thanks! At this purpose,  may I ask you:
I'm trying to make edge works, and with bowserStack it connect to Licode and do his things but he ignores REMBS, so the bw is always low and the video stuck at certain point. Is there any chance that Licode is sending him REMBS in a "wrong" or "not standard" way?
I'm talking about a standard scenario, not simulcast.
Because I think we can make edge work with very low effort and we'll be the first in webrtc sfu scenario I think. tried other "competitors" last week and no one worked.
thanks for the fix anyways!


type setAtom with never

to make ts users happier

https://codesandbox.io/s/loving-kilby-mzyqm?file=/src/App.tsx
Hmmm, it's not working as expected.

https://codesandbox.io/s/loving-kilby-mzyqm
Working now. cc @gsimone


Accept cbor and str as input type in submit_tx

Making submit_tx backward compatible.

Codecov Report

Merging #231 (00f2a70) into main (93fd889) will decrease coverage by 0.12%.
The diff coverage is 28.57%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #231      +/-   ##
==========================================
- Coverage   85.56%   85.45%   -0.12%
==========================================
  Files          26       26
  Lines        2952     2957       +5
  Branches      707      709       +2
==========================================
+ Hits         2526     2527       +1
- Misses        314      318       +4
  Partials      112      112

Impacted Files
Coverage Δ

pycardano/backend/base.py
93.58% <28.57%> (-5.05%)
:arrow_down:


Add libnl-route-3 to libnl-3

Please add the following dependency to the rosdep database.
Package name:

Add libnl-route-3-200 to libnl-3
Add libnl-route-3-dev to libnl-3-dev

Package Upstream Source:
https://www.infradead.org/~tgr/libnl/
Purpose of using this:
Measurement of network traffic.
These packages are part of libnl-3 or libnl-3-dev except Debian and Ubuntu.
Links to Distribution Packages

Debian: https://packages.debian.org/

https://packages.debian.org/bullseye/libnl-route-3-200
https://packages.debian.org/bullseye/libnl-route-3-dev

Ubuntu: https://packages.ubuntu.com/

https://packages.ubuntu.com/jammy/libnl-route-3-200
https://packages.ubuntu.com/jammy/libnl-route-3-dev

Fedora: https://packages.fedoraproject.org/

Included in libnl-3, libnl-3-dev

Arch: https://www.archlinux.org/packages/

Included in libnl-3

Gentoo: https://packages.gentoo.org/

Included in libnl-3

macOS: https://formulae.brew.sh/

not available

Alpine: https://pkgs.alpinelinux.org/packages

not available

NixOS/nixpkgs: https://search.nixos.org/packages

Included in libnl-3

openSUSE: https://software.opensuse.org/package/

Included in libnl-3

Oh sorry, I forgot debian package entries.
I will add tomorrow.

@nuclearsandwich
Fixed. Would you check this?

Thanks for iterating. It will need a second review before being merged.


bugdown: Support dashes for bulleted lists.

We should allow the markdown syntax for bulleted lists, using dashes in column one, in addition to keeping support for the current bugdown syntax of asterisks.  The only thing we probably need to consider when making this change is how it will affect people who want to post diffs or patches to Zulip.

I'd be game for trying this out if we disable it for the zephyr mirroring realms.  Can do a test rerender on a large base of messages to measure impact.

To detail how this should work a bit more, I think we should make:
* first bullet
* second bullet

and
- first bullet
- second bullet

work, but not mixed things like:
- minus
* times

or
+ first item
+ second item


[AutoPR] hanaonazure/resource-manager

Created to accumulate context: hanaonazure/resource-manager

(message created by the CI based on PR content)

@AutorestCI rebuild https://github.com/Azure/azure-rest-api-specs/blob/master/specification/hanaonazure/resource-manager/readme.md

Working on generating https://github.com/azure/azure-rest-api-specs/blob/master/specification/hanaonazure/resource-manager/readme.md for you!!!

Build SDK finished and cleaned


Implement a new UX for filling tax forms on Open Collective

Part of [Project] Internalize the tax forms process #7216
The current flow

Filling in a tax form

Expense payee submits an expense and gets a warning telling him an email to fill in a tax form

User clicks on the link in the email

The user fills in the tax form fields, which adapt themselves based on some custom inputs

The user can preview the document, then submits it by clicking "I agree"

User gets a confirmation by email with the PDF attached. The expense is "unlocked" and will be paid by the host.

Goals of the new flow

Implement the same form logic we have on Dropbox forms in our own interface
Preview/generate the PDFs (W9, W8 or W8-BEN) directly in the flow
Adapt the existing flow to direct users to this new form in a friendly way
Ensure we respect https://github.com/opencollective/opencollective/issues/7220 (use the same language as the existing form, collect the signature explicitly, ask for 2FA).

https://www.loom.com/share/b5ceb31431324211b3d44be1dab70a0d?sid=4513009a-e943-44b5-928e-65d669ec4b30
@Betree

@Memo-Es Adding the form in the settings as proposed in this design makes sense. Adding the ability to update your tax information is a nice touch (that was previously not possible with Dropbox Forms), but it creates two additional challenges that we'll need to address:

Tax forms have an expiry date: once generated, they're valid for 3 years. According to https://github.com/opencollective/opencollective/issues/6649, we don't want to keep storing the info after this delay. So if an account is still submitting expenses with tax information that's about to expire (i.e., the tax info is 2 years old), we should have an email notification + a warning on the form asking them to re-confirm their info.
We don't want people to update their tax information for already paid expenses. That means we shouldn't override past tax information and make sure that expenses are tied to a specific entry in LegalDocuments (the table where we store tax information).

As a simplification, I would keep the updating part out of the scope of the initial implementation. We can eventually add a message like If your tax information needs to be updated, please contact support.

Some constraints we'd like to enforce in the form, based on this year's feedback:

Names should be collected using 3 separate fields: First/Middle/Last Name
For "Status" use a select to try and enforce types (LLC, C-Corp, etc). "Other" should be an option and will pop up a string field allowing custom values.
US Addresses are expected to be collected as:

Address Line 1
Address Line 2
City
State
Zip
Country

We should collect a "Main email address", to clarify who's responsible for replying if there's any enquire about the provided tax info.

API requirements (to be updated)

Add a requiredLegalDocuments field on Account (to know if the account needs to complete a tax form).

@Memo-Es The Figma link in your comment does not work anymore, I was not able to find the design on Figma (both Lab and Production ready).
I remember you showing the signature as part of a modal. I'm not sure why we had that in a separate modal; for now I'm going with the same solution as Dropbox Forms, which is to show it directly in the form:

@Betree it was in the production-ready file before yesterday:

I integrated the sign field in the form as you suggested:

I also tagged you in a comment.
Do let me know if there is any design input still :)

@Betree
https://www.loom.com/share/4c78ff9109ad420bafaa89f4baf5ac53?sid=ef8a7cc9-5bb3-4003-81d0-66d253b36f22

After discussing with @znarf, we've decided to remove the "Current Tax Info" button that lets users download a copy of their own PDF tax form from the dashboard. Here are the reasons:

Payees already receive a copy of their PDF tax form by email
We haven't got any requests for this feature; it just came out as a "nice to have" in the design
Even though the download endpoint is protected with 2FA, not everyone has 2FA enabled, and it dangerously increases the risk of data theft (this form contains addresses, SSN, tax IDs, ...etc)

In summary, high risk VS unclear benefit.
We still want to allow the download of PDFs for host admins, but we can force them to have 2FA enforced - which will greatly limit the risk.


Consider how to treat discrepancies in success and version fields of signer responses

Currently, signer responses received by the combiner are filtered for successes in CombineAction. Consider whether we want to handle discrepancies in the success and version fields of responses differently

Sprint planning:

ticket to investigate how to deal with discrepancies
but might need to restructure files
Alec - slightly unintuitive behaviour in the code (hard to reason about logic)

Sprint planning:

Alec - nice to have

This is also related to why we can't return quota status information from the combiner on pnpSign responses


feature: ats9360 driver

Changes proposed in this pull request:

generalises ATS.py so it can be used with other alazar cards
adds specific driver for ATS9360

TODO:

[ ] make use of the acquisition controller (currently commented out)
[ ] check it hasnt broken it for the ATS9870
[ ] make the aquire into a parameter so it can be used in a loop (?)
[ ] remove debugging print statements and generally tidy up
[ ] pep8 stuff (broken laptop -> lab computer and notepad++ editor)
[ ] check some of the functions which I couldnt get to work (get_board_info and find_boards)
[ ] call_dll error handling needs improving

@giulioungaretti @alexcjohnson

@nataliejpg shall i close this ?

@giulioungaretti yes please :)


Markers don't have size

Handle size for marker. Currently (because of the current very limited typing), we return size as size, but for marker, we have to return radius (1/2 size).
Use this issue to also provide better typing for marker.

Fixed with #43


Bitfinex asset sell fail

Using the command line: zenbot sell bitfinex.BTC-USD --pct=10
The Bitfinex website shows: "Invalid order price precision (5 sig figures limit)"

I suppose it is sell.js problem to send values with figures over the limit. .format('0.00000000')

I'm not skilled enough to jump into the java, I tried a few changes as suggested, but with no good result.
Running the command line "zenbot sell bitfinex.BTC-USD --pct=10" a few times gives one of the following results:

"Submitting exchange limit sell order for 0.04467526 BTC." immediately followed by "Exchange limit sell order of 0.04467526 BTC has been canceled." resulting in no order execution.

"Submitting exchange limit sell order for 0.04467526 BTC." that ends in a successful order execution.

"Invalid order price precision (5 sig figures limit)" resulting in no order being placed.

These results seem to be random.

It's Bitfinex side error. It is not common. Their API should truncate price with precision >5. More info: Bitfinex API

This still is a problem with Bitfinex. Current solution has all products/pairs with precision of 0.0001 in product.json, but this makes the bot 'useless' for pairs like IOT/ETH and that act in the range of 0.0020389. The page referenced by @nedievas tells that 0.0020389 has a precision of 5 significant digits (20389) and is valid to place as sell price. A work around is setting the IOT/ETH pair at 0.0000001, but all pairs then need to be checked and changed.

Closing this on our side for now.


Save viewstate per unique device identifier

This would allow a VM to be shared on multiple devices and have their own view state.

Implemented in v4.0.2 as part of UTMRegistry.


Compile with Java 16 source & target

Rationale
Allow Java 16 syntax
Related Pull Requests

https://github.com/LabKey/server/pull/153

The change from 14 to 16 seems straightforward enough, but it looks like something else is required to make this go.

@labkey-tchad @labkey-susanh Yes, TeamCity needs to be configured to compile with JDK 16 on develop

Updated TeamCity to use Java 16 and merged from develop to trigger tests to run

Build is failing with:
Execution failed for task ':server:embedded:bootJar'.
     > Error while evaluating property 'mainClassName' of task ':server:embedded:bootJar'
        > Unsupported class file major version 60

Other branches are not showing this error.

That means that something older than Java 16 is trying to read a Java 16-compiled class. Class version 60 is what the Java 16 compiler target produces.
https://en.wikipedia.org/wiki/Java_class_file

Which build/repo/branch shows the failure? TeamCity link?

The failure is from this branch. It was the first build on the agent, so there isn't anything lingering from a previous build.
https://teamcity.labkey.org/buildConfiguration/bt2/1621223

Caused by: java.lang.IllegalArgumentException: Unsupported class file major version 60
       at org.springframework.asm.ClassReader.<init>(ClassReader.java:196)
       at org.springframework.asm.ClassReader.<init>(ClassReader.java:177)
       at org.springframework.asm.ClassReader.<init>(ClassReader.java:163)
       at org.springframework.asm.ClassReader.<init>(ClassReader.java:284)
       at org.springframework.boot.loader.tools.MainClassFinder.createClassDescriptor(MainClassFinder.java:25
       at org.springframework.boot.loader.tools.MainClassFinder.doWithMainClasses(MainClassFinder.java:136)
       at org.springframework.boot.loader.tools.MainClassFinder.findSingleMainClass(MainClassFinder.java:109)
       at org.springframework.boot.gradle.plugin.MainClassConvention.findMainClass(MainClassConvention.java:9
       at org.springframework.boot.gradle.plugin.MainClassConvention.resolveMainClass(MainClassConvention.jav
       at org.springframework.boot.gradle.plugin.MainClassConvention.call(MainClassConvention.java:60)

Looks like SpringBoot is attempting to read some class (via ASM) and failing. Our SpringBoot version is way out-of-date: 2.2.x was end-of-lifed in July. 2.5.x is the first SB version to claim Java 16 support. 2.6.1 is the latest.

I think we'll need to upgrade SpringBoot before we can require JDK 16. But I'm not the CTO 😄

@labkey-tchad test build is now failing. IAJC seems unhappy with the newer Java version.

FYI: I've been testing a newer version of AspectJ (1.9.8.RC3) over here: https://github.com/LabKey/testAutomation/pull/880
However, it's hard to imagine how running with AspectJ 1.9.7 would cause these failures. I've built the tests with JDK 16 and 17 many times.

Syntax error, annotations are only available if source level is 1.5 or greater (!!)
It looks like somebody is very confused about the source level

Looks like we do need the AspectJ upgrade.

Okay, that looks much better. We do have a strange DRT failure ciTestsSqlserver2019 and ciTestsPostgres13 (what are those?) both failing with "Could not find matching constructor for: java.io.File(String, File)". That's a private constructor. Anybody?

https://teamcity.labkey.org/viewLog.html?buildId=1621836&buildTypeId=bt3

@labkey-tchad @labkey-susanh this looks like a problem in how TeamCity and Gradle are interacting with each other. I found this in the test logs in TeamCity:
Caused by: groovy.lang.GroovyRuntimeException: Could not find matching constructor for: java.io.File(String, File)
at TestDescriptorWrapper.attacheBigFailTestMessage(/opt/teamcity-agent/plugins/gradle-runner/scripts/init.gradle:759)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:78)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at TestDescriptorWrapper.result(/opt/teamcity-agent/plugins/gradle-runner/scripts/init.gradle:679)
at TestDescriptorWrapper$result$2.call(Unknown Source)
https://teamcity.labkey.org/buildConfiguration/bt3/1621836?showLog=1621836_5876_849.5853.5861&logFilter=debug

Looks like a bug in TeamCity's gradle plugin: https://github.com/JetBrains/teamcity-gradle/issues/6
I'll see what I can do about updating that plugin.

@labkey-tchad any luck getting TeamCity's gradle plugin updated? Or let me know if I should merge these PRs before the update.

The fix is in the next version of TeamCity. We're scheduled to upgrade on the 12th.


Defaults not set for exporter when using options.defaultSpanExporterFactory(options)

Hi Splunk team,
In my application I am getting a handle on the defaultSpanExporter using options.defaultSpanExporterFactory(options) API and then using the exporter as one of the exporters in a custom spanProcessorFactory function
const { defaultSpanExporterFactory } = require('@splunk/otel/lib/options');
const exporterInstance = new defaultSpanExporterFactory(options);

I ran into an issue where I wasn't specifying the endpoint in the options and as a result the endpoint was set as undefined.
I am wondering whether it would be better to set the default options for the exporter within the defaultSpanExporterFactory itself instead of the _setDefaultOptions function so exporter specific options would be normalized even if defaultSpanExporterFactory is directly used?
Or do you recommend supplying all options if we directly try to use the defaultSpanExporterFactory?

Or do you recommend supplying all options if we directly try to use the defaultSpanExporterFactory?

Short answer: yes.
Long answer:
The idea was that we'd process options, load it with default for missing values, parse/validate any fields that require it and pass on the "processed" options to the factories so factories could rely on options always being correct. As a result, the default factories will always rely on fields being set in options object they receive.
Custom factories are free to either use options or completely ignore it and use custom or even hard-coded values.

Thanks. Closing it for now. Will re-open if I come across something similar.


[skip email] updating soundcard package

Updates Harp.SoundCard bonsai package to 0.2.0
After merging, run dynamic-foraging-task/bonsai/setup.cmd to install the new package

[ ] 446 @XX-Yin @ZhixiaoSu
[ ] 447 @XX-Yin @ZhixiaoSu
[ ] 428 @hagikent
[ ] Ephys1 @KanghoonJ
[ ] Ephys3 @XX-Yin

This is a hot fix because the SoundCard package was manually updated in the 447 computers through the bonsai IDE, which leads to inconsistencies in the Bonsai.config file. We are updating the Bonsai.config file to make sure all rigs are consistent.

@hagikent Thank you for the super helpful information! This manual installation seems to be compatible with main behavior workflow when @XX-Yin and I tested yesterday. (Sound player normally and attenuation changed tone level.) Will adding the package manually affect the daily refresh at 6 am?

affect the daily refresh at 6 am?

No, but the manually added package will be pushed back to 0.1.0 next 6am.
With a minor (yellow) alert from RoboAlex.

What is strange is that the update this morning didn't reset the repo. The bonsai/bonsai.config file had untracked changes after the 6am update.  Completely unclear to me why this happened. I manually reset the 447 computers, but it was a mysterious process. I recompiled bonsai, and did a git pull origin main in the command line, which somehow worked. I'll close this PR for now, but in the future, when installing new bonsai packages we need to ensure there are no untracked changes.

Hmm, that's mysterious... never happened to me at 446/428, where I did this manual package adding >50times.

Actually, now that I look. This warning is in the logs for 446. Maybe because no one has run mice they haven't been noticing

I updated all the 446 computers except 7CD because IT is doing something remotely

Yes, Ian from IT is working on 7C, trying to solve the Soundcard compatibility issue.


[Documentation Update]: LeetCode Problem 0190 - Reverse Bits

Is there an existing issue for this?

[X] I have searched the existing issues

Issue Description
Reverse Bits
Suggested Change
Add solution
Rationale
No response
Urgency
High
Acknowledgements

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository
[X] I have read and followed the Contribution Guidelines
[X] I have followed the code style guidelines of this project
[X] I have checked for any existing open issues that my pull request may address
[X] I have ensured that my changes do not break any existing functionality
[X] Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently
[X] I have read the resources for guidance listed below
[X] I have not used AI-generated content (e.g., ChatGPT, other AI tools)
[X] I have not used content from existing sites (e.g., text, images)
[X] I have followed all security rules and only shared trusted resources

@Ajay-Dhangar please assign it to me


Checkbox label overflows modal

TuSimple/naive-ui version (版本)
2.19.7
Vue version (Vue 版本)
3.2.20
Browser and its version (浏览器及其版本)
Firefox(93.0)
System and its version (系统及其版本)
Windows(10)
Node version (Node 版本)
Reappearance link (重现链接)
https://codesandbox.io/s/charming-andras-d80kd?file=/src/Demo.vue
Reappearance steps (重现步骤)
See modal
Expected results (期望的结果)
Label should use ellipsis
Actual results (实际的结果)
Label overflows modal
Remarks (补充说明)

e it in the source of naive-ui, how could I possibly handle this situation? I want the modal to keep the maximum width of 600px

`xxx

e it in the source of naive-ui, how could I possibly handle this situation? I want the modal to keep the maximum width of 600px

<n-checkbox>xxx<n-checkbox>

I've already tried something similar and it didn't work. This also doesn't work
See https://codesandbox.io/s/boring-kilby-3dv7r?file=/src/Demo.vue