License requirement is taking too much page area

Licensing requirement is taking too much page area which causes a lot of unnecessary scroll to get to the required section.
There could probably be a separate page for license requirement for all the solutions

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

ID: ed0630c9-36da-97a6-2240-4d559611b887
Version Independent ID: f9b42858-3eb1-e720-9ba7-a812f38f9bfc
Content: Insider risk solution - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/insider-risk-solution-overview.md
Product: microsoft-365-enterprise
GitHub Login: @robmazz
Microsoft Alias: robmazz

@deeksha-microsoft - yes, agreed and worked on reformatting to list view instead of table. Should be live soon. Thanks!

@deeksha-microsoft - yes, agreed and worked on reformatting to list view instead of table. Should be live soon. Thanks!


Fix mypy settings to run select test files in strict mode

Changed all mypy invocation to not include paths and moved the paths
to the mypy section in setup.cfg.
This removes redudancy in directory specification and reduces the
potential that directories are changed in one invocation but not
others.
Ignore mypy errors in test directory.
Enable strict mypy config with in-file config comments in the
following files:

test/test_issue1447.py
test/test_issue_git_336.py
test/test_literal.py
test/test_parse_file_guess_format.py

Validation run: https://github.com/aucampia/rdflib/actions/runs/1599142819

seems it is not quite working on windows, looking into it.


Traceback when adding a new domain; target of repeat operator is not specified: /*.wikipedia.org/

https://metasmoke.erwaysoftware.com/domains/new crashed for me when attempting to add xiaomitrends.com
Not sure I'm able to copy/paste the diagnostics correctly, but here goes.
RegexpError in SpamDomainsController#create
target of repeat operator is not specified: /*.wikipedia.org/
Extracted source (around line #41):
39    after_create do
40      groups = Rails.cache.fetch 'domain_groups' do
41          DomainGroup.all.map { \|dg\| !dg.regex ? nil : [Regexp.new(dg.regex), dg.id] }.compact.to_h
42       end
43      groups.keys.each do \|r\|
44        DomainGroup.find(groups[r]).spam_domains << self if r.match? domain

Rails.root: /var/railsapps/metasmoke/releases/20210426205533
Application Trace | Framework Trace | Full Trace
app/models/spam_domain.rb:41:in `initialize'
app/models/spam_domain.rb:41:in `new'
app/models/spam_domain.rb:41:in `block (3 levels) in <class:SpamDomain>'
activerecord (<IP_ADDRESS>) lib/active_record/relation/delegation.rb:71:in `each'
activerecord (<IP_ADDRESS>) lib/active_record/relation/delegation.rb:71:in `each'
app/models/spam_domain.rb:41:in `map'
app/models/spam_domain.rb:41:in `block (2 levels) in <class:SpamDomain>'
activesupport (<IP_ADDRESS>) lib/active_support/cache.rb:323:in `block in fetch'
activesupport (<IP_ADDRESS>) lib/active_support/cache.rb:694:in `block in save_block_result_to_cache'
activesupport (<IP_ADDRESS>) lib/active_support/cache.rb:663:in `block in instrument'
activesupport (<IP_ADDRESS>) lib/active_support/notifications.rb:170:in `instrument'
activesupport (<IP_ADDRESS>) lib/active_support/cache.rb:663:in `instrument'
activesupport (<IP_ADDRESS>) lib/active_support/cache.rb:693:in `save_block_result_to_cache'
activesupport (<IP_ADDRESS>) lib/active_support/cache.rb:323:in `fetch'
app/models/spam_domain.rb:40:in `block in <class:SpamDomain>'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:426:in `instance_exec'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:426:in `block in make_lambda'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:236:in `block in halting_and_conditional'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:517:in `block in invoke_after'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:517:in `each'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:517:in `invoke_after'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:133:in `run_callbacks'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:816:in `_run_create_callbacks'
activerecord (<IP_ADDRESS>) lib/active_record/callbacks.rb:346:in `_create_record'
activerecord (<IP_ADDRESS>) lib/active_record/timestamp.rb:102:in `_create_record'
activerecord (<IP_ADDRESS>) lib/active_record/persistence.rb:705:in `create_or_update'
activerecord (<IP_ADDRESS>) lib/active_record/callbacks.rb:342:in `block in create_or_update'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:132:in `run_callbacks'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:816:in `_run_save_callbacks'
activerecord (<IP_ADDRESS>) lib/active_record/callbacks.rb:342:in `create_or_update'
activerecord (<IP_ADDRESS>) lib/active_record/persistence.rb:275:in `save'
activerecord (<IP_ADDRESS>) lib/active_record/validations.rb:46:in `save'
activerecord (<IP_ADDRESS>) lib/active_record/transactions.rb:310:in `block (2 levels) in save'
activerecord (<IP_ADDRESS>) lib/active_record/transactions.rb:387:in `block in with_transaction_returning_status'
activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract/database_statements.rb:267:in `block in transaction'
activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract/transaction.rb:239:in `block in within_new_transaction'
/home/undo/.rvm/rubies/ruby-2.5.7/lib/ruby/2.5.0/monitor.rb:235:in `mon_synchronize'
activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract/transaction.rb:236:in `within_new_transaction'
activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract/database_statements.rb:267:in `transaction'
activerecord (<IP_ADDRESS>) lib/active_record/transactions.rb:212:in `transaction'
activerecord (<IP_ADDRESS>) lib/active_record/transactions.rb:385:in `with_transaction_returning_status'
activerecord (<IP_ADDRESS>) lib/active_record/transactions.rb:310:in `block in save'
activerecord (<IP_ADDRESS>) lib/active_record/transactions.rb:325:in `rollback_active_record_state!'
activerecord (<IP_ADDRESS>) lib/active_record/transactions.rb:309:in `save'
activerecord (<IP_ADDRESS>) lib/active_record/suppressor.rb:44:in `save'
app/controllers/spam_domains_controller.rb:51:in `create'
actionpack (<IP_ADDRESS>) lib/action_controller/metal/basic_implicit_render.rb:6:in `send_action'
actionpack (<IP_ADDRESS>) lib/abstract_controller/base.rb:194:in `process_action'
actionpack (<IP_ADDRESS>) lib/action_controller/metal/rendering.rb:30:in `process_action'
actionpack (<IP_ADDRESS>) lib/abstract_controller/callbacks.rb:42:in `block in process_action'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:109:in `block in run_callbacks'
audited (4.9.0) lib/audited/sweeper.rb:14:in `around'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:118:in `block in run_callbacks'
audited (4.9.0) lib/audited/sweeper.rb:14:in `around'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:118:in `block in run_callbacks'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:136:in `run_callbacks'
actionpack (<IP_ADDRESS>) lib/abstract_controller/callbacks.rb:41:in `process_action'
actionpack (<IP_ADDRESS>) lib/action_controller/metal/rescue.rb:22:in `process_action'
actionpack (<IP_ADDRESS>) lib/action_controller/metal/instrumentation.rb:34:in `block in process_action'
activesupport (<IP_ADDRESS>) lib/active_support/notifications.rb:168:in `block in instrument'
activesupport (<IP_ADDRESS>) lib/active_support/notifications/instrumenter.rb:23:in `instrument'
activesupport (<IP_ADDRESS>) lib/active_support/notifications.rb:168:in `instrument'
actionpack (<IP_ADDRESS>) lib/action_controller/metal/instrumentation.rb:32:in `process_action'
actionpack (<IP_ADDRESS>) lib/action_controller/metal/params_wrapper.rb:256:in `process_action'
activerecord (<IP_ADDRESS>) lib/active_record/railties/controller_runtime.rb:24:in `process_action'
actionpack (<IP_ADDRESS>) lib/abstract_controller/base.rb:134:in `process'
actionview (<IP_ADDRESS>) lib/action_view/rendering.rb:32:in `process'
rack-mini-profiler (1.1.6) lib/mini_profiler/profiling_methods.rb:78:in `block in profile_method'
actionpack (<IP_ADDRESS>) lib/action_controller/metal.rb:191:in `dispatch'
actionpack (<IP_ADDRESS>) lib/action_controller/metal.rb:252:in `dispatch'
actionpack (<IP_ADDRESS>) lib/action_dispatch/routing/route_set.rb:52:in `dispatch'
actionpack (<IP_ADDRESS>) lib/action_dispatch/routing/route_set.rb:34:in `serve'
actionpack (<IP_ADDRESS>) lib/action_dispatch/journey/router.rb:52:in `block in serve'
actionpack (<IP_ADDRESS>) lib/action_dispatch/journey/router.rb:35:in `each'
actionpack (<IP_ADDRESS>) lib/action_dispatch/journey/router.rb:35:in `serve'
actionpack (<IP_ADDRESS>) lib/action_dispatch/routing/route_set.rb:840:in `call'
warden (1.2.9) lib/warden/manager.rb:36:in `block in call'
warden (1.2.9) lib/warden/manager.rb:34:in `catch'
warden (1.2.9) lib/warden/manager.rb:34:in `call'
rack (2.2.3) lib/rack/tempfile_reaper.rb:15:in `call'
rack (2.2.3) lib/rack/etag.rb:27:in `call'
rack (2.2.3) lib/rack/conditional_get.rb:40:in `call'
rack (2.2.3) lib/rack/head.rb:12:in `call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/http/content_security_policy.rb:18:in `call'
rack (2.2.3) lib/rack/session/abstract/id.rb:266:in `context'
rack (2.2.3) lib/rack/session/abstract/id.rb:260:in `call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/cookies.rb:670:in `call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/callbacks.rb:28:in `block in call'
activesupport (<IP_ADDRESS>) lib/active_support/callbacks.rb:98:in `run_callbacks'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/callbacks.rb:26:in `call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/debug_exceptions.rb:61:in `call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/show_exceptions.rb:33:in `call'
railties (<IP_ADDRESS>) lib/rails/rack/logger.rb:38:in `call_app'
railties (<IP_ADDRESS>) lib/rails/rack/logger.rb:26:in `block in call'
activesupport (<IP_ADDRESS>) lib/active_support/tagged_logging.rb:71:in `block in tagged'
activesupport (<IP_ADDRESS>) lib/active_support/tagged_logging.rb:28:in `tagged'
activesupport (<IP_ADDRESS>) lib/active_support/tagged_logging.rb:71:in `tagged'
railties (<IP_ADDRESS>) lib/rails/rack/logger.rb:26:in `call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/remote_ip.rb:81:in `call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/request_id.rb:27:in `call'
rack (2.2.3) lib/rack/method_override.rb:24:in `call'
rack (2.2.3) lib/rack/runtime.rb:22:in `call'
activesupport (<IP_ADDRESS>) lib/active_support/cache/strategy/local_cache_middleware.rb:29:in `call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/executor.rb:14:in `call'
actionpack (<IP_ADDRESS>) lib/action_dispatch/middleware/static.rb:127:in `call'
rack (2.2.3) lib/rack/sendfile.rb:110:in `call'
rack-cors (0.4.1) lib/rack/cors.rb:81:in `call'
rack-mini-profiler (1.1.6) lib/mini_profiler/profiler.rb:184:in `call'
railties (<IP_ADDRESS>) lib/rails/engine.rb:524:in `call'
/usr/lib/ruby/vendor_ruby/phusion_passenger/rack/thread_handler_extension.rb:107:in `process_request'
/usr/lib/ruby/vendor_ruby/phusion_passenger/request_handler/thread_handler.rb:157:in `accept_and_process_next_request'
/usr/lib/ruby/vendor_ruby/phusion_passenger/request_handler/thread_handler.rb:110:in `main_loop'
/usr/lib/ruby/vendor_ruby/phusion_passenger/request_handler.rb:415:in `block (3 levels) in start_threads'
/usr/lib/ruby/vendor_ruby/phusion_passenger/utils.rb:113:in `block in create_thread_and_abort_on_exception'

Request
Parameters:
{"utf8"=>"✓", "authenticity_token"=>"lotsofbase64==", "spam_domain"=>{"domain"=>"xiaomitrends.com"}}

https://metasmoke.erwaysoftware.com/data/sql/queries/269-domain-groups-search?search=wikipedia reveals two domain groups matching "wikipedia", the first of which is botched (item 24).

@Undo1  as a workaround, can you remove the problematic entry?

Obviously, this is due to the invalid regex *.wikipedia.org. Maybe we can add some validation when adding a new domain group.

Regression: Can't create domains at all again.
target of repeat operator is not specified: /*.wikipedia.org/

Extracted source (around line #41):

39   after_create do
40     groups = Rails.cache.fetch 'domain_groups' do
41       DomainGroup.all.map { \|dg\| !dg.regex ? nil : [Regexp.new(dg.regex), dg.id] }.compact.to_h <============
42     end
43     groups.keys.each do \|r\|
44       DomainGroup.find(groups[r]).spam_domains << self if r.match? domain

If you want a test case, the one I wanted to add was gamesncomps.com to link back to https://metasmoke.erwaysoftware.com/post/324790

(I guess the code is fine, but the incorrect regex needs to be removed before it can work.)

Oh NVM, I found the botched testing group #27 and removed it.

id
name
regex
created_at
updated_at

27
Testing Group
*.wikipedia.org
2021-09-09 15:13:54 UTC
2021-09-09 15:13:54 UTC

Good thing I had enough details in my testing notes before.


CRI, Issue 47577: Add client API for renaming folders

Issue
Tasks

[x] Server-side changes - 3 hr @RosalineP

[x] Implement
[x] code review @cnathe
[x] Merge

[ ] Java - 3 hr @RosalineP

[x] (Ramp up)
[x] Implement
[ ] code review @labkey-ians
[ ] Manual Testing @labkey-ians
[ ] Merge

[ ] R - 2 hr @cnathe

[ ] Implement
[ ] Manual Testing
[ ] Merge

[ ] Python - 2 hr @RosalineP

[x] Implement
[x] code review
[x] Manual Testing
[ ] Merge @labkey-alan

[ ] JS - 2 hr @RosalineP

[x] Implement
[ ] Manual Testing - 2 @labkey-nicka
[ ] Merge

[x] Manual test plan
[ ] Endpoint test against java api - @labkey-chrisj

[ ] Teamcity review and merge - 1hr

After modifying the controller to the correct value
Where the scenarios are as follows:

[x] New name only: Verify name and title become 'name1'
Name is correct, title is blank/not set.
[x] New title only: Verify only title changes to 'title2'
[x] New name and title: Verify both name and title update
[x] Required values missing: Verify error appears
[x] AddAlias explicitly true: Verify works, and name and title update
[x] AddAlias explicitly false: Verify works, and name and title update
[x] Empty name: Verify error appears
[x] Empty title: Verify error appears
[x] Empty name, populated title: Verify only title updates
[x] Empty title, populated name: Verify name and title update
Title is cleared
[x] Pre-existing name: Verify error appears

R API

[ ]  New name only: Verify name and title become 'name1'
[ ] Name is correct, title is blank/not set.
[ ]  New title only: Verify only title changes to 'title2'
[ ]  New name and title: Verify both name and title update
[ ]  Required values missing: Verify error appears
[ ]  AddAlias explicitly true: Verify works, and name and title update
[ ]  AddAlias explicitly false: Verify works, and name and title update
[ ]  Empty name: Verify error appears
[ ]  Empty title: Verify error appears
[ ]  Empty name, populated title: Verify only title updates
[ ]  Empty title, populated name: Verify name and title update
[ ] Title is cleared
[ ]  Pre-existing name: Verify error appears


add load_module and improve docs

this was the alternative I had in mind to https://github.com/myleott/monorepo/pull/3 but IMO https://github.com/myleott/monorepo/pull/3 (ie deprecation) is preferable as the added complexity vs simply setting PYTHONPATH isn't worth it.
This WIP PR adds load_module which works as follows:
import monorepo
# `load_module` function will search up from the current directory for a
# file named `.monorepo_root`, which is used to determine the root directory
# it then adds (if not already there) the root to sys.path, and then calls `importlib.import_module`
mymodule2 = monorepo.load_module("pkg1.pkg2.mymodule")
mymodule2.my_function()

It doesn't have any of the caveats of load_package (module duplication, module name clashes etc) mentioned in https://github.com/myleott/monorepo/pull/4
(see discussion in https://characterorg.slack.com/archives/C02T4TKCH0D/p1704849283733079?thread_ts=1704828327.016509&cid=C02T4TKCH0D)

tested; merging

actually I don't have write access so I can't merge

@myleott https://pypi.org/project/monorepo/ needs to be updated so pip reinstall monorepo will work


JavaScript sample missing in CONTRIBUTING.md

[ ] Issue Title is in form Algo/DS Name [Language].
[x] I searched or browsed the repo’s other issues to ensure this is not a duplicate.
[ ] This Algo/DS is worth adding to this repository.
[x] Assign this issue to me (I want to work on this).
[x] I will send my PR only after I'm assigned to this issue by any maintainer.

Thanks @Anshuman-Verma for reporting. Please proceed with this.

Thanks for assigning @singhpratyush, on it.


Remove duplicate GA4 tracking

What

Having the tracking on this link triggers a link click and a select_content as it expands an accordion on the same page. So we remove the link click tracking so only one event fires for the interaction. Example of the issue: https://www.gov.uk/government/consultations/primate-licensing-scheme on the 'See all updates' link.

Why

https://trello.com/c/jf1V3D02/770-placeholder-duplicate-content-history-events-firing-on-some-pages

Visual Changes

None.

@andysellick I think this line of code can be removed because I have a list of all the templates that use this See all updates link. So I don't think removing the code would affect anything we weren't aware of, since this tracking was added specifically for these templates and now it's being removed specifically for these same templates. Here is the list of see all updates links in government-frontend: https://gist.github.com/AshGDS/0f3bd5aa6c04cefed2839f4d9b6b7337#see-all-updates-link, all of them except the HMRC manual were making use of the code that I'm removing and they all had this bug that will be fixed by removal of this.
But interestingly, it looks like the move to using disable_ga4 instead of ga4_tracking: true is the reason for this PR was raised in the first place. As the previous work to remove the tracking involved removing ga4_tracking: true, but of course that wasn't replaced with a disable_ga4: true flag as we weren't aware of that coming into use at the time. https://github.com/alphagov/government-frontend/pull/2935
I can add disable_ga4 in government-frontend instead, but I guess that would mean we potentially leave in tracking code in this component that's never used, and as far as i'm aware this line of tracking was only used in those templates that have the duplicated event issue.
I believe only tracking See all updates is intentional, I can't remember specifically why (I vaguely recall something about being told that Pageview events can be used to track the user journey for those internal links) but I guess it can be brought to the team if tracking is needed on those links.

Thanks @andysellick 👍


Update Release 2.9 Series - Overview.md

Clarify yokozuna position - with community support now available from more than one organisation.

Moves develop-2.9 to RC3 candidate


The possible bug of the complex in-place matrix transposition function "cblas_cimatcopy"

The OpenBLAS project is wonderful, and thanks for the team's hardworking.
project version: r0.2.17.dev
**host:**Ubuntu 14.04 LTS
**target:**ArmV7
compile command: make CC=arm-linux-gnueabihf-gcc FC=arm-linux-gnueabihf-gfortran HOSTCC=gcc TARGET=ARMV7 CROSS=1 NO_LAPACKE=1 NO_SHARED=1 BINARY=32 USE_OPENMP=1
If the column and row of the  input matrix are not equal, the function "cblas_cimatcopy" got wrong result; but, the function "cblas_comatcopy" got the correct result.
I do not know it is a bug or not, thanks.

Can you get some code sample to compare against MKL?

@brada4 I have checked the corresponding MKL "mkl_cimatcopy", and it got the right result.
The MKL code sample:
`#include <stdio.h>
#include <stdlib.h>
#include <memory.h>
#include "mkl.h"
#include "mkl_trans.h"
#define M (3)
#define N (2)
int main(void)
{
int i, j;
int	lda = N,// A col
ldb = M;// B col
MKL_Complex8 alpha = { 1, 0 };
MKL_Complex8 in1[MN] = { 0 }, in2[MN] = { 0 }, out[M*N] = { 0 };
// Input complex matrix
printf("The input complex matrix:\n");
for (i = 0; i < M; i++)
{
	for (j = 0; j < N; j++)
	{
		in1[i * N + j].real = i + 1.0f;
		in1[i * N + j].imag = 2*j + 3.0f;
		printf("%4.1f+%4.1fI ", in1[i * N + j].real, in1[i * N + j].imag);
	}
	printf("\n");
}

memcpy(in2, in1, M*N*sizeof(MKL_Complex8));

// transposition using cblas_comatcopy
mkl_comatcopy('R', 'T',
	M, N, alpha, in2, lda, out, ldb);

printf("\nThe output complex matrix using mkl_comatcopy:\n");
for (i = 0; i < N; i++)
{
	for (j = 0; j < M; j++)
		printf("%4.1f+%4.1fI ", out[i * M + j].real, out[i * M + j].imag);
	printf("\n");
}
printf("\n");

// transposition using cblas_cimatcopy
mkl_cimatcopy('R', 'T',
	M, N, alpha, in1, lda, ldb);

printf("\nThe output complex matrix using mkl_cimatcopy:\n");
for (i = 0; i < N; i++)
{
	for (j = 0; j < M; j++)
		printf("%4.1f+%4.1fI ", in1[i * M + j].real, in1[i * M + j].imag);
	printf("\n");
}
system("pause");

}`

The OpenBLAS code sample:
`#define M (3)
#define N (2)
int main(void)
{
int i,j;
int	lda=N,// A的列,
ldb=M;// B的列,
float alpha = 1;
float in[MN2]={0}, in1[MN2]={0}, out[MN2]={0};
// Input complex matrix
printf("The input complex matrix:\n");
for (i=0; i<M; i++)
{
	for (j=0; j<N; j++)
	{
		in[i*2*N+2*j] = i+1.0f+3.0f*j;
		in[i*2*N+2*j+1] = 10+i*2+j;
		printf("%4.1f+%4.1fI ", in[i*2*N+2*j], in[i*2*N+2*j+1]);
	}
	printf("\n");
}
memcpy(in1, in, M*N*2*sizeof(float));

// transposition using cblas_comatcopy
cblas_comatcopy(CblasRowMajor, CblasTrans,
	M, N, &alpha, in1, lda, out, ldb);

printf("\nThe output complex matrix using cblas_comatcopy:\n");
for (i = 0; i < N; i++)
{
	for (j = 0; j < M; j++)
		printf("%4.1f+%4.1fI ", out[i * 2 * M + 2 * j], 	out[i * 2 * M + 2 * j + 1]);
	printf("\n");
}
printf("\n");

// transposition using cblas_cimatcopy
cblas_cimatcopy(CblasRowMajor, CblasTrans,
	M, N, &alpha, in, lda, ldb);

printf("\nThe output complex matrix using cblas_cimatcopy:\n");
for (i = 0; i < N; i++)
{
	for (j = 0; j < M; j++)
		printf("%4.1f+%4.1fI ", in[i * 2 * M + 2 * j],	in[i * 2 * M + 2 * j + 1]);
	printf("\n");
}

}`

The MKL fucntion mkl_comatcopy and mkl_cimatcopy, and OpenBLAS function cblas_comatcopy get correct results, but the cblas_cimatcopy fails.

@brada4 Er, I'll take time to check the algorithm and code, and poster the results.

it is actually 2x omatcopy... I wonder why it could fail

I can reproduce the results from above with cblas_cimatcopy. The kernel calls an out-of-place routine.
The first call copies the values to an array b then it copies the values back into a. Here is where I think its not working. Here is what I have done:
240    OMATCOPY_K_RT(*rows, *cols, alpha[0], alpha[1], a, *lda, b, *ldb );  241    //OMATCOPY_K_RN(*rows, *cols, (FLOAT) 1.0, (FLOAT) 0.0 , b, *ldb, a, *ldb );  242      OMATCOPY_K_RN(*cols, *rows, (FLOAT) 1.0, (FLOAT) 0.0 , b, *ldb, a, *ldb );  243    free(b);
I changed the order of the rows and columns in the call to OMATCOPY_K_RN.
Now it works. Probably there are other places which have to be changed accordingly.

I notice that @wernsaar lamented a lack of test cases when he set out to implement the XXmatcopy extensions in #51

Sadly the fix suggested above does not appear to be a complete or general solution. In particular when only a submatrix of the input is to be transposed (as is done in the test case that comes with MKL) the current implementation appears to affect unrelated matrix elements as well.

Actually I now think the problem is twofold - (1) the OMATCOPY_K_RN calls need to get their rows and cols parameters interchanged as suggested above (and for all four cases involving "Trans") and (2) the  "NEW_IMATCOPY" optimization that was added later to allow in-place operation when source and destination LDA are the same does not cope with operations on submatrices.

@grisuthedragon I think the NEW_IMATCOPY was your work - wonder if you could check if my assumption above makes sense ?

Sorry for my delay but I was away for the last two weaks. I took a look in your fix and run a few examples with it and it works for my. And you are right with the assumptions that the NEW_IMATCOPY only works if rows == cols and lda == ldb. It is still not the fastest  implementation as also mentioned by #1243 but I do not have time to port my more efficient implementation to a OpenBLAS style routine.

Thanks for the feedback. I will close this issue then as the immediate problem appears to be resolved. I hope to find time to explore the suggestions from #1243 in the near future.


Trie Implementation in C and Python

Feature = Trie in C and Python
Implementation of Trie in C & Python is not available in our repo. I would like to contribute them.
Assign this to me under SSOC'23.

Assigned!
@tarunvyshnav777 : C & Python


Ecs Task Definition example for laradoc

I want to run my container using AMZON ECS. It would be really great If you can help me providing a sample task_definition file.
I am using the workspace,nginx, php-fpm container for my app

@aniamin I was working on that to include it as part of LaraDock but didn't finished it. Would be awesome if someone can contribute with that.

I was asking for some help as I am getting it little bit difficult to
manage multiple containers through ECS
On Wed, Jun 29, 2016 at 12:07 AM, Mahmoud Zalt<EMAIL_ADDRESS>wrote:

@aniamin https://github.com/aniamin I was working on that to include it
as part of LaraDock but didn't finished it. Would be awesome if someone can
contribute with that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/LaraDock/laradock/issues/124#issuecomment-229133077,
or mute the thread
https://github.com/notifications/unsubscribe/AFoiAcRYuzhUrNxlP5Xq74YHMZvo555bks5qQWL3gaJpZM4JAW4B
.

Let's continue this conversation on this Issue #244

@aniamin
Any updates about you trying to deploy to ECS? I am currently working on deploying laradock containers to AWS elastic beanstalk but I don't know where or what to start. That would be great if you can share some knowledge.
Thanks!

I also would like to know how to deploy Laradock to ECS, any updates?


[Scheduled Maintenance] BRCRP down for Maintenance

Additional context
Reimplementing Vanillaworks Extended and maybe Adding Some Extra goodies
The server will be down until 9:00 PM UTC (3:00 PM CST) on the 28th. During this time, we are working on improving the system, including reimplementing Vanillaworks Extended and possibly introducing some new features.
Please note that there may be a temporary-degraded experience due to how our CAD system is set up, and it needs the FiveM server to be up for it to run to standards. Thank you for your patience!


Remove version: Rainmeter.Rainmeter version 4.5.12

This version is older than what has been set in max-versions-to-keep by the publisher.
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/rainmeter/rainmeter/releases/download/v<IP_ADDRESS>16/Rainmeter-4.5.12.exe
OK
(Automated message - build 917)

Hi @,
Do you approve of these changes?
(Automated response - build 888.)


terraform_1_0: 1.0.2 -> 1.0.3

Motivation for this change
https://github.com/hashicorp/terraform/releases/tag/v1.0.3
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 131014 run on x86_64-darwin 1

1 package marked as broken and skipped:

terraform-full

2 packages built:

infracost
terraform (terraform_1_0)


Texturing

This PR adds texture capabilities to the game. I've made some basic textures on GIMP and added just 4 basic textures to get things done.

Next step is load the atlas onto the AssetServer and update the shader to render the correct UV

Now that I'm reading my own PR, I think doesn't make sense to have multiple atlases for KindDescription, since it should be a single atlas always.
I'll change this...

A very primitive texturing is working:

There are some issues to be solved:
1- Fix texture rotation;
2- Add texture tiling;
3- Check if texture is working for different kinds;

In order to do the texture tiling, the following steps needs to be done, if I'm not mistaken:

[ ] Calculate TileTextureSize which is the TileSize in texture coordinates [0.0, 1.0]. This is just a matter of 1.0 / TileSize
[ ] Convert UV, which is [0, N] where N < TileCount, to tiled UV UV * TileTextureSize
[ ] Calculate BaseTextCoord which is the starting point of the tile in texture coordinates. This one can be calculated using TileOffset * TileTextureSize
[ ] Send TileTextureSize, TiledUV and BaseTextCoord to the shader. The first one can be a UniformBuffer, second and third may be a Vertex Data
[ ] Update fragment shader to calculate the final UV using uv = baseTextCoord + mod(tiledUV, tileTextureSize)

Texture tiling is working fine.

Next steps:

[ ] Fix texture rotation;
[ ] Do some cleanup and code organization;
[ ] Fix chunks sometimes not rendering (maybe something related to system ordering);
[ ] Add back commented tests generate_vertices and update_chunk

Texture rotation fixed

Found a bug were sometime the mesh::merge_faces ignores the voxel kind:

I wasn't able to reproduce:
Fix chunks sometimes not rendering (maybe something related to system ordering)
Skipping it for now

Ok I think everything is good now.


[webapi] Update suite.json and sub app manifest.json for app tools

Impacted TCs num(approved): New 0, Update 2, Delete 0
Unit test Platform: Crosswalk Project for Android 17.45.437.0
Unit test result summary: Pass 46, Fail 0, Blocked 67
https://crosswalk-project.org/jira/browse/XWALK-5760

Please don't merge, as the UT of reference and bdd tests are BLOCK.

Pls review https://github.com/crosswalk-project/crosswalk-test-suite/pull/3099


fix typo in readme.md

According to Apple's spec, the value of version is EC_v1.
(https://developer.apple.com/library/content/documentation/PassKit/Reference/PaymentTokenJSON/PaymentTokenJSON.html)

Thanks


Add more tests for Qt processors

References and relevant issues
Close #6847
Description
Add more tests for Qt processors (napari/_qt/_qapp_model/injection/_qprocessors.py)

Okay I've thought about this a bit more. I see I was completely mistaken about the dependency injection, if no viewer is provided we just do _provide_viewer. As for the tests themselves:

_add_plugin_dock_widget is pretty simple and essentially just uses add_dock_widget and has some minor logic on whether the viewer was given. For this, what @dalthviz had (with the mocks) perfectly a unit test. Testing further (with make_napari_viewer probably isn't needed, as _add_plugin_dock_widget itself should be tested. Sorry for my mistake.
_add_layer_to_viewer - the only thing worth testing here is maybe that the source is added correctly...?
_add_layer_data_tuples_to_viewer - I think the thing to test here is that if the viewer already has a layer with the same name, the new data updates the existing layer?
_add_layer_data_to_viewer - is it worth also testing that the the source and layer name are correctly added? (I'm on the fence)

@DragaDoncila I am confused about the _add_future_data, is it worth testing and how would we do it?
@dalthviz so sorry! I should have given it more thought when reviewing the first time round!

Thanks for the further review @lucyleeow ! Added some changes following your comment :+1: Regarding the _add_future_data, seems like there are some tests that could be already taking care of testing that function indirectly like the magicgui tests (I think those follow a similar approach as the one described by @DragaDoncila via Zulip 🤔 )? So tests like https://github.com/napari/napari/blob/b9dc8dca1e1afa7567586e7ffa7a36cc7cd9fb6f/napari/_tests/test_magicgui.py#L85-L116 Will give those a check to add more validations to the test here

That test does seem to test the future processor. I am interested to see that they added a function _assert_stuff but only used it once? I wonder if they originally wanted to add more tests? Or there is another reason, like putting the asserts in a function somehow make them behave differently?!

I am interested to see that they added a function _assert_stuff but only used it once? I wonder if they originally wanted to add more tests? Or there is another reason, like putting the asserts in a function somehow make them behave differently?!

I think maybe when doing the tests they started with a qtbot.waitUntil that used directly the function with the asserts. However, that way of doing things basically will constantly try to run the asserts until they pass or a timeout is reached. With the qtbot.waitSignal I think the asserts are only run once (after the inserted signal is emitted). So the tests maybe ended up that way to prevent calling things on the asserts more times than is needed, but since the asserts encapsulation with _assert_stuff was already there they just use it? 🤔

@DragaDoncila maybe 0.5.5 milestone?


Add support for GET URL params

Hi, thanks to create this very useful website.
I added support for GET URL params (Close #11).
Added

react-router-dom v4 and connected-react-router v5 (because v6 requires React v16.4)

Because react-redux-router is deprecated, I choose connected-react-router.

URL params change when users select languages, labels, pages, and orders.
Initial query and state is generated from URL params if URL params exist.

Questions

I'm not sure about where to put a function which modifies URL params. I put it in redux action for now. Is this correct?
URLSearchParams should be polyfilled? (Can I Use link)

hey @uu1t this is awesome!
sorry for the delayed reply.  I think i can get to this by the end of the week.  i'm excited to get your updates live!
thank you for contributing.

just did a quick review.  AWESOME work!  looking forward to testing this out

this is amazing!! :rocket: :rocket: :rocket: :rocket: :rocket: :rocket: :rocket:

deployed and working awesome!

@uu1t just want to say thanks again.
this really helps take the app to the next level.
further contributions are definitely welcome.
i really appreciate your help!


[Problem] Print is show blank page

Current Behavior
 When I click Print button it's show blank page but click download csv button it show data from table.

Your Environment

Tech
Version

Material-UI
^4.3.1

MUI-datatables
^3.7.1

React
16.8.6

browser
Chrome

etc

@nnt01
Could you create a codesandbox to show this issue?

@wdh2100 I use this project https://github.com/uilibrary/matx-react and add mui-datatables in new form


CollectionView should support appending items when reaching the end of current item list

https://github.com/eclipsesource/tabris-js/issues/271

Items can be appended to a CollectionView using the insert() method. The reload can be triggered from the itemchange event when the last item is resolved. This works well, but since this event is fired on the cell, it feels like the wrong hook. Moreover, items being resolved and items become visible are semantically different things.
We may think about an additional event like itemappear on CollectionView itself.

I also vote for a "scroll position" listener.

Here's a code example for Android: https://gist.github.com/ssinss/e06f12ef66c51252563e

Since we have all the low level infrastructure in place (eg item visible listeners) to create some more advanced JavaScript based abstraction on top of it i'll close the issue.


Fix/phone property

The phone number property value can, in fact, be null if not filled with a value in a Notion database.

What about the gh-actions annotations about stuff being deprecated?

See #1. We're going to ignore that until we have implemented new and better tests.


[Scenes] Decouple remove-all-scenes in step 1c in TC_S_2_6

Currently step 1b does:
./chip-tool scenesmanagement remove-all-scenes 0x0000 2 1 --commissioner-name beta
./chip-tool scenesmanagement remove-all-scenes 0x0000 3 1 --commissioner-name gamma
Should be split in 2 steps, 1b and 1c.

Mentioned here : https://github.com/project-chip/connectedhomeip/pull/32003#discussion_r1489648568


ISSUE12 - upgrade couch base version, add aliases to couch base query…

… builders and verify that they are used for the query method

Coverage remained the same at 100.0% when pulling 98b2b0b9189b765ea6246f5167326fad84f11243 on ISSUE-12 into e9ca56df1658dbccfd4af2ba666e69690799f2b5 on master.

Coverage remained the same at 100.0% when pulling 0c89b0085b78518c662bbc97a707887d9f40fd02 on ISSUE-12 into e9ca56df1658dbccfd4af2ba666e69690799f2b5 on master.

Coverage remained the same at 100.0% when pulling c89ef81e02d5887e360d6ae1473450d0cfcb505b on ISSUE-12 into e9ca56df1658dbccfd4af2ba666e69690799f2b5 on master.


Multi-cluster install instructions for Mac

What is missing?
The current install instructions don't appear to work as expected on MacOS. When running setup-kind-multicluster.sh --clusters 2 --kind-worker-nodes 4 we expect to see two clusters of 4 nodes each k8ssandra-0 and k8ssandra-1.
But running the above command and then kubectl ctx yields only one cluster:
kind-k8ssandra-0

This may be related to getopt on MacOS not being fully compatible with regular Linux style getopt. Running getopt -T as per the TODO note in setup-kind-multicluster.sh  yields:
getopt -T
 --

But installing gnu-getopt using brew install gnu-getopt doesn't seem to help, even when running export FLAGS_GETOPT_CMD="$(brew --prefix gnu-getopt)/bin/getopt" via .zshrc followed by exec $SHELL.
I am reluctant to run brew link --force gnu-getopt because that will break normal system functionality. We may need something in the script to find gnu-getopt and use it locally within the script only.
Why do we need it?
MacOS is one of the more common environments we expect to encounter.
┆Issue is synchronized with this Jira Task by Unito
┆Reviewer: 5fa4593062584c006b1e04d0
┆Fix Versions: k8ssandra-operator-1.0.0-alpha.1
┆Issue Number: K8SSAND-903
┆Priority: Medium

The problem was that there was no way to isolate the use of gnu-getopt to just this script. So all getopt calls across the whole system would be going to the GNU version. Now that may be fine, but as a user, I'm just not familiar enough with MacOS internals to know where else it is used or how MacOS default getopt differs to gnu-getopt.
This is a bit hand wavy but from a user perspective I would hesitate to switch out one of my system-standard CLI tools in order to run an install script. And adding the replacement tool to the PATH is also a bit much to ask for folks who might just be kicking the tires is also a big ask (IMO).

@adutra I believe you are on a Mac. I assume you installed gnu-getopt since you have been working on the project for a little while now and are using the script. Did it cause any problems for you?

Correct, I installed gnu-getopt with Homebrew (keg only, not symlinked), then followed the instructions and added the same line as @jsanda to my .zshrc file:
export PATH="/usr/local/opt/gnu-getopt/bin:$PATH"

I did this many years ago and never had problems since.
I do think it would be fair to consider gnu-getopt as a required binary for k8ssandra developers.

May I ask what we're getting out of gnu-opt that would justify requiring it as a dependency?
We can achieve the same behavior at low cost and without any dependency it seems.

May I ask what we're getting out of gnu-opt that would justify requiring it as a dependency?

With the changes in #144 we have ~65 lines of code vs ~10 with the current version in main. gnu-getopt is also used in the create-clientconfig.sh script so there will be a similar situation there.
I also want to point out that I tested the changes in #144 and the script changes work perfectly fine. If we decide to move forward with removing gnu-getopt, that's fine.
The bigger question/concern I have is what is the criteria we are establishing with respect to dependencies? I don't want to restrict ourselves in general though from using linux tools that might not be available out of the box in MacOS or in Windows.

The bigger question/concern I have is what is the criteria we are establishing with respect to dependencies?

That's a very good question which I guess is a matter of weighing the benefits of introducing the dependency vs the ease of installing it (and in our case, even knowing that you need to install it).
My replacement code for removing gnu-getopt could very well be compacted to almost the same number of lines if I remove the logic to check if a value was passed for each argument, which removes some of the benefits that we're expecting from gnu-getopt:
while test $# -gt 0; do
  case "$1" in
    --clusters) shift; export num_clusters=$1; shift;;
    --cluster-names) shift; export cluster_names=$1; shift;;
    --kind-node-version) shift; export kind_node_version=$1; shift;;
    --kind-worker-nodes) shift; export kind_worker_nodes=$1; shift;;
    -o|--overwrite) overwrite_clusters="yes"; shift;;
    -h|--help) help; exit;;
    --) shift; break;;
    *) break;;
  esac
done


Azure cost forecast

Created the initial Azure cost forecast and cone of confidence.
https://issues.redhat.com/browse/COST-737

Codecov Report

Merging #1750 (7033dae) into master (9d0005a) will decrease coverage by 0.07%.
The diff coverage is 54.54%.

@@            Coverage Diff             @@
##           master    #1750      +/-   ##
==========================================
- Coverage   77.41%   77.34%   -0.08%
==========================================
  Files         274      275       +1
  Lines        4419     4436      +17
  Branches      859      858       -1
==========================================
+ Hits         3421     3431      +10
- Misses        907      914       +7
  Partials       91       91

Impacted Files
Coverage Δ

src/api/forecasts/awsForecast.ts
66.66% <ø> (ø)

src/components/charts/costChart/costChart.tsx
7.93% <ø> (+0.18%)
:arrow_up:

...re/dashboard/awsDashboard/awsDashboardSelectors.ts
83.33% <ø> (ø)

...ashboard/azureDashboard/azureDashboardSelectors.ts
83.33% <ø> (ø)

.../dashboard/azureDashboard/azureDashboardActions.ts
76.00% <25.00%> (-24.00%)
:arrow_down:

src/api/forecasts/forecastUtils.ts
36.36% <33.33%> (-1.14%)
:arrow_down:

src/api/forecasts/azureForecast.ts
66.66% <66.66%> (ø)

src/api/forecasts/forecast.ts
100.00% <100.00%> (ø)

.../dashboard/azureDashboard/azureDashboardWidget.tsx
85.71% <100.00%> (+0.71%)
:arrow_up:

...tore/dashboard/awsDashboard/awsDashboardWidgets.ts
100.00% <100.00%> (ø)

... and 2 more

Continue to review full report at Codecov.

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

Thanks, Boaz and Doug!


Foot Trajectory

Hi @jmichalczyk
I am not able to get the entire foot trajectory. But, here in line 85 of loop.py foot0 and foot1 I am able to get the position of each step(contact made with ground). You can see that in the image below, it is just for a single step of 0.8s.

For a single step, I have a vector of 160 values for fx, fy, fz, ftheta, xcom, ycom, thetacom. But I have only 9 values for foot[0] and foot[1] in line 85 loop.py. Shouldn't this be 160 too? Am I doing something wrong here?

Solved with this commit


Support type conversion with multiple possible types

Currently our argument conversion logic allows specifying only one possible type for each argument. There are, however, use cases where declaring that, for example, either a list or a string is accepted and Robot should then try out different conversions. With type hints the natural syntax to use is  Union like arg: Union[list, str].
The basic logic with multiple possible types would be going through all types one-by-one, trying to convert using each type, and going to the next one if conversion fails. There are, however, plenty of design decisions to be made about different details:

Should we first check does the given argument match any of the possible types before doing conversion? If, for example, we have arg: Union[str, int] and the given argument is an integer, should we pass it as is (it matches one of the possible types) or should we test conversion with possible types one-by-one? In the latter case we'd convert the integer to a string because the first possible type is str. In this case checking does the argument match any of the types feels better.
On the other hand, if we have arg: Union[int, str] and call with a string 2, checking does the given argument match any of the type would mean that we don't do any conversion because one of the types is str. If we'd go through types one-by-one, we'd convert it to an integer because int is before str. In this case testing conversion one-by-one works better.
We could also make it so that we test conversions one-by-one if the given argument is a string and otherwise first check does the argument match any of the types. That feels a bit inconsistent and hard to explain, though. I got a feeling that going through types one-by-one works best. The key thing to remember is that str should be the last type to avoid converting all arguments to strings.

What to do if we don't recognize one of the possible types? I think we should test conversion with all recognized types that are before it but return the value as-is if no conversion succeeds. For example, arg: Union[int, MyThing] would convert string 2 to an integer but string xxx would be left as-is. This would be consistent with how we handle arg: MyThing.

How to support multiple types with the @keyword decorator? We could require using Union also with it, but then this syntax wouldn't be supported with Python 2 and needing to use typing may not always be desirable with Python 3 either. I think we could allow using a tuple to types like @keyword(types={'arg': (list, str)} as an alias for Union.

If we support a tuple of types with @keyword, should we also support it with type hints like arg: (list, str)? Type checkers probably won't like that but I see no harm us supporting it assuming it doesn't require extra work.

How to handle None as type? The typing module considers None an alias for NoneType in type hints and thus if there's and argument like arg: None, we actually get NoneType as the type because typing.get_type_hints() does conversion automatically. We probably want to do that same also with types specified using @keyword(types: {'arg': None}). When giving types using @keyword as a list, None currently means that the matched argument doesn't have type info and that probably cannot be changed.

After the above commits this is now mostly done. Missing tasks:

Decide how to support @keyword.
Documentation.

@pekkaklarck Everything can be converted to a string, should we add a special case where if string is in the valid types, run that conversion last?

There are similar other types such as boolean.

We possibly could change this so that if the given argument is not a string, we only convert it if it isn't already one of the accepted types. Anyway, changes need a new issue and cannot be done before RF 4.1.


Object type spec bug fix

If the flow based type doesn't match the type on the objTypeSpecFldInfo and we need a monomorphic type check, indicate a type mismatch for that operand and avoid fast path.

This change is

@pleath please take a look. Let me know if you think that existing callsites of opnd->SetTypeMismatch should be augmented to cover this case instead.

@dotnet-bot test Windows x64_debug please

What does Lower generate for this type check if "type mismatch" is set?

@pleath , lowerer generates fast path with inline cache lookup and a helper call if the inline cache misses.

Why the lower can still generates fast path even if type mismatch?

In reply to: 267412547 [](ancestors = 267412547)

@ThomsonTan, hmm, it does sound unnecessary. @pleath what do you think?

:shipit:

the underlying issue was fixed by #3341


Oversized Sia logo in OS X's status menu.

I blame Apple, the OS should be able to resize as necessary. I'm pretty sure that's what both Linux and Windows does.
heh. We should fix it.

The icon shouldn't even exist in the status menu unless there are clickable items or other display info in it. Are there actions the status menu currently provides? If there are none we should remove it.

I think it was supposed to have the buttons that allowed you to minimize the window or something.
I'm not sure how it got there, you'd have to talk to Ming.
@Mingling94 ideas?


docs: add debug draco page

count soft constraints violations for example charts

Added a comment. Would Altair work better than mpl in this case?

I think so, yes.
Regarding "Provide different sorting options (by sum, sum of non-zero, alphabetic, etc)", what would make more sense: an ipywidget-based interactively steerable approach, or simply outputting the result of the different options below each other?
The former approach would be nice, but would be sensible only if the docs are opened with Binder or served locally, the latter would be beneficial to display results statically via JupyterBook.

What do you think about plotting weights & violation counts this way? Are there other sorting combinations that might be interesting?

Regarding "Provide different sorting options (by sum, sum of non-zero, alphabetic, etc)", what do you think, what would make more sense: an ipywidget-based interactively steerable approach, or simply outputting the result of the different options below each other?

I was thinking a widget. But I'm not sure how that would render in the book. Need to try and decide.

What do you think about plotting weights & violation counts this way? Are there other sorting combinations that might be interesting?

Beautiful. Two requests. First, make the upper chart have a smaller height. Second, can you use white for exactly 0? It would make it easier to interpret the chart.
Other than that, I think we can merge this.

What do you think about plotting weights & violation counts this way? Are there other sorting combinations that might be interesting?

Beautiful. Two requests. First, make the upper chart have a smaller height. Second, can you use white for exactly 0? It would make it easier to interpret the chart.
Other than that, I think we can merge this.

Implemented in a63de80

Regarding "Provide different sorting options (by sum, sum of non-zero, alphabetic, etc)", what do you think, what would make more sense: an ipywidget-based interactively steerable approach, or simply outputting the result of the different options below each other?

I was thinking a widget. But I'm not sure how that would render in the book. Need to try and decide.

I looked into it. While it works nicely when serving the notebook locally, the statically-built Jupyter Book site does not support interactivity by default, because we are attempting to call Python code. The Jupyter Book docs mention that for this use-case, remote kernels are necessary to interact with Python via the web site.

I looked into it. While it works nicely when serving the notebook locally, the statically-built Jupyter Book site does not support interactivity by default, because we are attempting to call Python code. The Jupyter Book docs mention that for this use-case remote kernels are necessary to interact with Python via the website.

I figured this would be an issue. Would we still have a default, though? I would be okay with having only a static default on the website and a dynamic version if someone runs the kernel.

I looked into it. While it works nicely when serving the notebook locally, the statically-built Jupyter Book site does not support interactivity by default, because we are attempting to call Python code. The Jupyter Book docs mention that for this use-case remote kernels are necessary to interact with Python via the website.

I figured this would be an issue. Would we still have a default, though? I would be okay with having only a static default on the website and a dynamic version if someone runs the kernel.

I did not push the code for the widget, the notebook merged to main only has the static VConcatChart for now. Maybe as we progress with the project and extend the docs we can dedicate a PR to add more interactivity to the examples?

Yep, I saw that. We can always do follow up PRs even for smallest additions. Feel free to send one for the widget.


Fix eslint script and clean up resulting warnings

npm run lint:js was broken. Fixed it and ran it on the code again.

Codecov Report

Merging #2449 (7cdecee) into develop (6fadc76) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2449   +/-   ##
========================================
  Coverage    39.40%   39.40%
========================================
  Files           20       20
  Lines         1609     1609
========================================
  Hits           634      634
  Misses         975      975

Impacted Files
Coverage Δ

js/server.js
91.11% <ø> (ø)

Continue to review full report at Codecov.

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


ObjectDoesNotExist should raise a ValueError

This ensures that it shows up as connected to a particular field with
a nice red error-box rather than as a stacktrace
Acceptance Criteria
This is a completely shoot-from-the-hip style PR; no new tests were created, I did not run tests, etc. I did briefly check, for my personal use-case, this indeed makes it so that an object that cannot be founds shows up with a nice red exclamation-mark in the import, rather than a big traceback.
I understand that this is in some way only half the work but my thinking is that something is better than nothing, and that others are better positioned to do the other half of the work. If nothing else, it make the raised issue more easily understandable.

Coverage decreased (-0.06%) to 98.33% when pulling 8a82ec275f6684de41013e4b1b43c913da9744df on vanschelven:object-does-not-exist-should-raise-value-error into eb27821a899a651fd11fe92a5b88955eed35ae0c on django-import-export:main.

Thanks for submitting this, but I am closing as per comment


Migrate to Node.js native test runner

Remove esm-tape-runner and tap-monkey to use the Node.js native runner.
https://nodejs.org/api/test.html
i18n has been migrated few days ago (so do not hesitate to check the scripts and tests).

Hello, I take this issue.

Thanks for the contribution!


Props is not being passed forward when using composition

Bug report
Describe the bug
I'm trying to pass a prop from one component to another using composition, but apparently the prop is not being passed.
To Reproduce
Heading 1 should render with $brand-1 color. The same with Heading 2 with $brand-2 color
https://codesandbox.io/s/flora-forked-c2l50?file=/src/App.tsx
In the Text component, I added a console.log to line 20. This log is not being triggered, btw.
System information

macOS
Browser Chrome
Version of Stitches: 0.1.9

More details
If I remove the comments of these lines, it works, but it would be better to use the power of composition instead https://codesandbox.io/s/flora-forked-c2l50?file=/src/components/Heading.tsx:471-564

I think this is because the as prop in <Heading> is replacing the <Text> component, so <Text> is never rendered. Similar to #448

I've tried another way to reproduce the issue -> https://codesandbox.io/s/blue-platform-66snm?file=/src/App.js

Thanks for this. It certainly is an issue. We'll address this once we're done with performance work and typescript. Probably ~4 weeks from now.

I believe this is an example of the issue outlined in #488, my team also encountered this issue. We had to create internal documentation highlighting this "gotcha", and typically avoid passing a component to styled.
In case it clarifies the issue, our internal documentation is:

Issues with using styled(Component)

There are a couple of issues with styling elements this way.

Passing a React component to styled will only work if the component passed down accepts a className prop. If it doesn't, the styles will not be applied. Sandbox.
The as prop can cause unexpected behaviour when using styled(Component). This prop overrides the rendered DOM node, but as a side-effect, it also loses reference to the original Component which was styled. This sandbox illustrates the issue. There is also an open issue about it, which the Modulz team have acknowledged.

Fixed. Will be out in next release

AH MULEKE!

@peduarte 0.2.0?

@franklinjavier yeahm, it'll be 0.2.0 - release prob next week.

Hey @peduarte
I created another sandbox pointing to 0.2.0, but didn't work. Yeah i know, i'm a little anxious 🥺
Maybe, it will help you to test https://codesandbox.io/s/stitches-ci-cra-forked-g1jp3?file=/package.json:188-282
Would you mind sharing your demo?

Another point, it seems that it is swallowing the props

@franklinjavier that would be because there is no 0.2.0, it hasn't been released yet :)

I know, I'm referring about this
  "dependencies": {
    "@stitches/react": "https://pkg.csb.dev/modulz/stitches/commit/b11739b0/@stitches/react",

@franklinjavier Hmm. I used the sandbox you created here
Here's the forked sandbox with it working: https://codesandbox.io/s/hungry-brook-qk9uc?file=/package.json

@franklinjavier looking at your latest sandbox though, I can see the issue still remains. Thanks for that, it was a bit of a mix up of sandboxes 😅

@franklinjavier Ok, after some more digging, it turns out that even your last sandbox is working as expected

This is the sandbox you provided https://codesandbox.io/s/stitches-ci-cra-forked-g1jp3?file=/package.json:188-282
This is the forked sandbox working as expected https://codesandbox.io/s/stitches-ci-cra-forked-4v695?file=/src/App.jsx

In your sandbox, you're not mapping the color prop to the CSS color property. So it'll never get applied.

Am I missing something?

He's spreading props on L27, no?

Pedro, I'm extending Text component and this component is the responsible for getting the color props and put inside css prop. Makes sense?

@franklinjavier Hmm, I'm not following. Can you provide a minimal, reproducible example please?
I know there're a few examples, but they have too many abstractions and files and that can be distracting.
Without it, all I can tell is that the sandbox and screenshot from my previous comment seem to make sense and behave as expected.

Im not convinced it's a bug, though. There are multiple layers of composition. Some happening in Stitches via the styled function, some happening via as prop, and some via React Component wrappers.
It feels to me that it's more of an architectural issue. The moment you wrap StyledHeading in Heading, you're out of Stitches composition logic. So you need to manually map the color prop to the CSS.
Im not sure what we can do about this, but once our top priority issues are out of the way we can dedicate some time to explore this further

@peduarte is correct. We wouldn't be able to "fix" this, and if we decided to revisit how it is architected, we would need to think thru how we would want to change the way things work.


TzParser: test gmTime pointer

When gmtime fails, a NULL pointer is returned. This
will cause a crash.
Signed-off-by: Christophe Chapuis<EMAIL_ADDRESS>

@yogishSan Please share your review status.

Seems this was now merged. I guess you can close the PR?

Merged upstream.


Get Unrecognized directory type when: gcloud beta app gen-config --custom

Trying to follow the readme to generate default configs:
$ gcloud beta app gen-config --custom
ERROR: (gcloud.beta.app.gen-config) Unrecognized directory type: [/Users/admin/test]

The directory is empty and writable

You need to at least have an app.yaml with runtime: python specified. Gen
config custom reads the existing config and generates a custom runtime from
it.
On Wed, Oct 5, 2016, 6:44 PM Gary Wu<EMAIL_ADDRESS>wrote:

Trying to follow the readme to generate default configs:
$ gcloud beta app gen-config --custom
ERROR: (gcloud.beta.app.gen-config) Unrecognized directory type: [/Users/admin/test]
The directory is empty and writable
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/python-runtime/issues/31, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPUc_ncbzdLJaJPG3Vwy3RMX0wu-AUXks5qxFJqgaJpZM4KPdp0
.

the requirement of app.yaml is not mentioned in any of the places that document this command. And it does not work:
$ cat app.yaml
runtime: python
vm: true
entrypoint: gunicorn -b :$PORT main:app

runtime_config:
  # You can also specify 2 for Python 2.7
  python_version: 2
$ gcloud beta app gen-config --custom
ERROR: (gcloud.beta.app.gen-config) Unrecognized directory type: [/Users/admin/test]

but having an empty py file in a directory (no app.yaml or other files) works, i.e.:
touch empty.py
gcloud beta app gen-config --custom
This looks like a Python app.  If so, please enter the command to run
the app in production (enter nothing if it's not a python app): :

Gotcha. I'll update the README. Is there any reason you're using gen-config instead of writing the dockerfile yourself?


MathJax render newline error

test code
<!DOCTYPE html><script<EMAIL_ADDRESS>
  # Error test

  $$
  A \rightarrow \beta R \\
  B \rightarrow \alpha R | \epsilon
  $$

rendering effect

which rendering effect show be right?

I think this is a bug of MathJax because I test such code by mathjax.
test code using MathJax
<!DOCTYPE html>
<head>
<script src="https://cdn.bootcss.com/mathjax/3.0.5/es5/tex-mml-chtml.js"></script>
</head>

<body>
<div>
  $$
  A \rightarrow \beta R \\
  B \rightarrow \alpha R | \epsilon
  $$
  </div>
  </body>

rendering effect using MathJax

Now I have an ugly method to avoid this bug by using two LaTex paragraphs.
<!DOCTYPE html><script<EMAIL_ADDRESS>
  # Error test

  $$
  A \rightarrow \beta R
  $$
  $$
  B \rightarrow \alpha R | \epsilon
  $$

But I think it's not an elegant method. So, Is there any better method to avoid the render question? thanks.

I found a better way to render multiline equation.
<!DOCTYPE html><script<EMAIL_ADDRESS>
  # Error test

  $$
  \begin{align}
  & A \rightarrow \beta R \nonumber \\
  & B \rightarrow \alpha R | \epsilon \nonumber
  \end{align}
  $$

& was used to align two formulas, \nonumber was used to avoid the formula number at the end of the line.
rendering effect

The $$ delimiter is used to typeset a single equation, not multiple equations.
The rendering example you have shown for the first example is correct and expected behaviour. In other words, the LaTeX code $$ f \\ g $$ renders the text "f" and the text "g" on the same line. This is expected behaviour. The original LaTeX (such as the one available with TeXLive, pdflatex, etc., not MathJax LaTeX) behaves the same way too.
To display multiple equations on separate lines, align, gather, or some such environment should be used, like you have shown in your second comment. See section 3.2 (Single Equations) and section 3.5 (Multiple Equations) for more details on LaTeX syntax for mathematical typesetting.

Thank you.

You are very welcome. Thank you for using TeXMe. If you have any more questions about it, please do not hesitate to create a new issue for this project on GitHub and ask your questions.


"C'tor/d'tor is not implicitly called" error with Unreal Engine (VS 2017)

Hi,
Unfortunately akrzemi1::optional doesn't compile in Unreal Engine projects. It gives the following errors:
Error	C4582	'akrzemi1::storage_t<T>::value_': constructor is not implicitly called	MyTestProject	optional.hpp	270
Error	C4583	'akrzemi1::storage_t<T>::value_': destructor is not implicitly called	MyTestProject	optional.hpp	275

The lines in question are:
constexpr storage_t( trivial_init_t ) noexcept : dummy_() {};

and
  ~storage_t(){}

Outside of Unreal Engine projects, it of course works flawlessly. Unfortunately, Unreal Engine has its own build system (UBT), and it's very peculiar on how it uses Visual Studio. It's basically a VS 2017 project and uses the VS 2017 compiler, but in C++14 mode, with the compiler flags set by some Unreal Engine C# build scripts (yes, it's horrible...).
This issue arose in ThePhD/sol2 too: https://github.com/ThePhD/sol2/issues/235. I think @ThePhD took the implementation of his optional from you, and modified it. He fixed it in this and this commit, with quite some extensive changes (which are beyond my C++ skills).
Now I would be interested in: Why does this occur (apparently) only when building akrzemi1::optional in Unreal Engine, which uses the VS 2017 compiler, so it should presumably work? What makes it fail? Could it be some peculiar compiler flag? Or is it a bug in akrzemi1::optional that @ThePhD fixed in his copy?

One thing that I remember from looking at Visual Studio last time is that it was not able to handle constexpr member functions. For this reason I gave up trying to implement std::optional on Visual Studio. Maybe this has changed. Maybe Visual Studion still does not correctly implement constexpr unions?

Thank you very much both for your replies!
I think I wasn't 100% clear enough what's going on, apologies. So:

It works fine with the VS 2017 compiler in C++17 mode. But obviously it's not necessary there because it has std:: <optional>.
It works fine with the VS 2017 compiler in C++14 mode. This is the default mode and some people are restricted to this. There is no std:: <optional> available in this mode so akrzemi1::optional is a great substitute.
It does not work and produces above compiler errors with the VS 2017 compiler in an Unreal Engine solution, which uses the UnrealBuiltTool. They set the compiler to C++14 mode, so std:: <optional> is not available and a substitute is needed. It's not supported to change the mode to C++17 and also changing compiler flags (or even finding out what they're set to) is horrible thanks to the horrendeous UnrealEngine build system.

So I was hoping basically that the compiler error might help you guys to guess what could be going on.

If you look at the following links,
https://answers.unrealengine.com/questions/607946/anonymous-union-with-none-trivial-type.html
https://social.msdn.microsoft.com/Forums/en-US/c33e7d32-6e90-42ce-8ceb-2664f2cfa036/compiler-errors-c4582-and-c4583-constructordestructor-is-not-implicitly-called?forum=vcgeneral
The problem seems to be quite common. The former link suggests changing the configuration in UE4 build to not treat  C4582 and C4583 as errors. I am not sure if it is doing the right thing though.
Changing the implementation from union storage to std::aligned_storage would solve your warning, but it would compromise the constexpr guarantee. And I cannot do this. Maybe you can afford using boost::optional, which provides std::aligned_storage implementation?

Oh wow! Particularly the first link is awesome! I googled quite a lot about this but I never found that first link. That explains it really well. So it's a particular MSVC compiler warning that is only enabled under /Wall (which I think MSVC users normally don't use since it's a really crazy warning level, much higher than gcc's -wall), and Unreal Engine chose to enable exactly this warning and even further, treat it as error.
That certainly explains everything. Nice, thank you very much!
That basically also means that any users that uses your optional and uses MSVC, is affected by this warning - they just never see it because nobody enables this warning or treats it as errors. You probably have many MSVC users, and nobody seemed to have complained yet, so this warning can probably be ignored.
Thank you for your ideas regarding solutions - I will think about which one is best for my use case :-) None of them is great, but the Unreal Engine is horrible anyway from a C++ point of view, so it's a matter of finding the least horrible one.

At least in my case,
I can tell that this error was gone when I included my header files BEFORE Unreal Engine's.
I suspect something might be wrong-handled in Unreal Engine headers,
but I'm not sure what exactly occurred in them.


Fix jumpy months

Fixes #536

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending.

Comments from Reviewable


Error on example

Hi,
When I execute this example from documents:
winsay.speak("Good News', 'You've won the internet!", function () {
console.log("text to speech complete");
});
I receive this error:
C:\test2\node_modules\winsay\winsay.js:25
throw error;
^
Error: Object reference not set to an instance of an object.
at Error (native)
at :1:55
at speakSync (C:\test2\node_modules\winsay\winsay.js:22:5)
at process. (C:\test2\node_modules\winsay\speakAsync.js:4:5)
at emitTwo (events.js:87:13)
at process.emit (events.js:172:7)
at handleMessage (internal/child_process.js:695:10)
at Pipe.channel.onread (internal/child_process.js:440:11)
This text is printed after you encounter the wild Pikachu.

Hi @LeandroFavero, on what OS are you running into this? Windows 7, 8 or 10?

Speak in synchronous mode, this error happens only in Async


Update README to mention Makefile variables

Also mention bootc-image-builder target.

This will need to be updated for all recipe readmes.

@MichaelClifford @sallyom @lmilbaum @Gregory-Pereira PTAL


fix 10216: prefab needs to be available to 2+ samples

Overview
This change moves the FingerChaser prefab from Demos - Utilities to the Common folder in the Examples package to make it available to the Demos - Input sample as well.
Changes

Fixes: #10216

/azp run

@davidkline-ms is this going into 2.7.3?

@davidkline-ms is this going into 2.7.3? And if so, can it target the release branch?

Thanks for the catch! I missed that step...

/azp run

/azp run


There is no log output when rocketmq-streams running.

There is no rocketmq-streams log output right now, when a APP running base on rocketmq-streams. It is inconvenient for troubleshooting.
We can implement a inner log, which should not conflict with user's local log dependencies. A client log in rocketmq:
org.apache.rocketmq.client.log.ClientLogger
which solve this problem based on InnerLoggerFactory .
There are other log dependency, such as log4j, in rocketmq-streams, we also need remove it and only use one log dependency.

Hello, I would like to contribute in this issue. can i have a try?

Hello, I would like to contribute in this issue. can i have a try?
of course, welcome!


Bug: unaccessible menu buttons due to notifications

Describe the bug
By updating the profile information and clicking on 'Update profile' button, a notification appears in the left right corner of the screen.
But due to its padding (4rem), the header buttons are not accessible anymore as the notification uses z-index: 100.
An example with the colored padding:

Solution
It would be better to use top: 4rem to set the position of the notification.
The z-indexvalue should be updated too: it's currently 100 but 10 for the dropdown menu of the header. By still having the notification visible, the menu is hidden behind.
Steps to reproduce

go to https://insights.opensauced.pizza/user/settings
update an information
click on update profile button
try to click on the icon avatar in the header

Affected services
insights.opensauced.pizza
Platforms
Desktop
Browsers
Chrome
Environment
Production, Development
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

I just checked it fastly, as for the padding maybe it can be removed and the top property be set to 4rem.
As for the menu behind it, changing its z-index to 10 is a great idea,
Here is the related code:
https://github.com/open-sauced/insights/blob/beta/components/atoms/Toast/toast.tsx#L16
For open sauced team, I would like to fix this, what would you suggest as a solution?

.take

i will work on this issue during this week

I think this is intentional, and you have to close the notification before doing anything else!
if it's a bug I have two thoughts:

should the user be able to open the profile? Not sure about the best design for this one.
The notification could be on the bottom as well.

I would like to have some input from the open sauced team.
Here is how it looks on mobile:

Curious to hear your thoughts here @OgDev-01

Hey @a0m0rajab Thanks for picking this up. We should move the toast a little down below the nav on mobile screens 👍
Around here

Users should be able to open their profile even then toast it visible.

@OgDev-01 Thank you for the information. I just started working on it but got stuck. Here is my update:
Removing the padding and adding the top position to the toast solves the issue. Yet, I got stuck on another issue: when the user is in the middle of the page. Having the previous design is better unless we have a way in tailwind to check if the page is on top or middle (not sure how)
Here are the screenshots:
Good behavior:

The bad behavior:

Another solution we can have is to keep the same position but change the Z-index to let the user use the profile menu.
The code:
at https://github.com/open-sauced/insights/blob/beta/components/atoms/Toast/toast.tsx#L16
Removed:
p-6 lg:p-16 sm:top-auto

Added:
top-24 md:top-14

Plus, the Ipad Mini's behavior is a bit odd:

Might need to open an issue for it.

Just did a Google search to find other examples:
Here is the HP guide: https://design-system.hpe.design/templates/toast-notifications
They do hide the links behind it.

Removing the padding and adding the top position to the toast solves the issue. Yet, I got stuck on another issue: when the user is in the middle of the page.

Toasts are obstructive in nature and I don't think we should be concerned about any element that is blocked except the nav on this.
This work's, but you will have to include px-6 to move it out of the edge 👍

Thank you for the information!
Just to make sure doing this will have a space like the second picture I shared (the timezone one) is that acceptable?

I am assuming the like in the last comment is a yes.
Did not get a notification about it, though.
I did change the code. Here are the results

With the nav menu:

Large screens:

The toast can still be in the upper place in the mobile and hide the nav behind it.
Side note: px is not the best for a position. I went with top to not hide the elements behind it.


[5.3] Flash messages are now displayed on login page when REFERER is not sent

This is a fix for a bug I found while writing test for a project. If the client doesn't send a REFERER header when posting data on a login form, the client is not redirected properly and the flash messages (containing the errors) are lost in the process.
The bug occur in the following circumstances:

You have a route redirecting from / to /login
The client post his (wrong) credentials on the /login page without the  REFERER header
Laravel return / when executing redirect()->back() because the referer is missing and the session cannot be accessed (session is always null)
The client tries to access / but is redirected to /login, losing the flash messages (flash messages are flushed once the request is completed)
The client access /login with no error messages displayed

Here's relevant code snippets mentioned in the flow above:

https://github.com/laravel/framework/blob/5.3/src/Illuminate/Foundation/Auth/AuthenticatesUsers.php#L118
https://github.com/laravel/framework/blob/5.3/src/Illuminate/Routing/Redirector.php#L55
https://github.com/laravel/framework/blob/5.3/src/Illuminate/Routing/UrlGenerator.php#L136

The bug affect 5.2 and 5.3 and exists since this commit: https://github.com/laravel/framework/commit/4326dc07d8d96161fdc18d0d5859928c395819ad
The fix I propose is relatively simple. It gives UrlGenerator access to the session, so it is able to access $session->previousUrl() when the referer is missing. The session is set from Redirector when setSession($session) is called.

I don't understand why this is necessary because the URL generator already receives a session resolver in RoutingServiceProvider.

I don't know the Laravel internals enough to tell why, but the setSessionResolver() method is never called. If I place a break point here https://github.com/laravel/framework/blob/5.3/src/Illuminate/Routing/UrlGenerator.php#L739 and execute the flow in my browser using the xdebug plugin, this line is never called.

I was running on v5.2.43 and I just updated to v5.2.45. The issue is still there.

Well, I'm sorry. The issue was on my side. A co-worker found the issue and fixed it.


[Site]: deployment-role website 404

Current State
Description of the current state/issue:

open [set-up-multiple-ca-certificates](https://apisix.apache.org/docs/apisix/certificate/#set-up-multiple-ca-certificates]
click Deployment Modes
Link to the page:

https://apisix.apache.org/docs/apisix/architecture-design/deployment-role
https://apisix.apache.org/docs/apisix/certificate/architecture-design/deployment-role.md/
Screenshots (if relevant):

Desired State
Description of the change you are proposing:
Would you like to fix this issue?
None

The document corresponding to this address no longer exists, and change to deployment-modes.md. I think we should change the URL.
We should also move this issue to APISIX...


Fix typos

base58: fix typo
bitcoin: fix typo
chacha20_poly1305: fix typo
hashes: fix typo
units: fix typo

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 82.824%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
20422

Relevant Lines:
24657

💛  - Coveralls

BTW. https://github.com/crate-ci/typos

Thanks!


Any LICENSE for this?

Was this boilerplate released to the public domain? :)
Looking through this, it seems like exactly the boilerplate I've been looking for, to at least get my hands dirty and explore Rust more in-depth. My only question would be what the LICENSE for this repo is? Is it intended to be in the public domain?

Thanks @jkcdarunday :upside_down_face:


feat(core): Added way to determine FileType by stream via magic bytes

Added StreamFileTypeProvider
Added tests

ISSUE: #15

@kryptobi As more ways of validation / look-up are implemented, we should focus on a streamlined project- and test structure, I think. Let's discuss this next year.
btw Visual Studio for Mac is not that great ;)

Visual Studio for mac is crap.


categories

on clicking categories no categories are visible...kindly assign it to me..i wana add some categories menu bar

Hmm that points to a bug in your DB connection. You should see 3 shoe categories in that tab. Please re-check your database connectivity and try again.

Just checked with the latest master. Everything is working as expected. Please check your database connection and try again.
Closing the issue.


More flexibility in configuring cxxopts

Is your feature request related to a problem? Please describe.
The cxxopts submodule is a required dependency for building umpire, but appears to only be necessary for building umpire's tests, examples and tools.
Describe the solution you'd like
Ideally,

Umpire could be configured to not require cxxopts when its dependant targets (e.g. tools, examples, tests) are disabled.
One could pass in a path to an external cxxopts in a similar manner to how an external blt is passed in.

Alternatives ideas
Since cxxopts is relatively small, perhaps it could be built it.
Additional context
I'm working on a project that does not allow submodules and having a cxxopts submodule appears to currently be mandatory.

Hey @kennyweiss - thanks for opening this issue. I can see the problem, and will implement the solution you are proposing.

We decided to just bundle cxxopts - thanks for the suggestion @kennyweiss.
Closed by #257

Thanks @davidbeckingsale !


Highlight doesn't work properly with korean text

Describe your issue here.
Steps to reproduce
Great highlighter, works perfectly, thank you for your work. Recently found that it works properly with every site except with korean sites. For instance, text selected on 'http://www.yonhapnews.co.kr/' gives some random results rather then select identical strings.
I am using it programmatically, here is sample:
	var marker = new Mark(document.querySelector('body'));
	var options = {
		'separateWordSearch': false,
		'iframes': true,
		'acrossElements': true,
	};
	marker.mark(txt, options);

Where 'txt' is text selected on the page.
Environment

FF 59
mark.js version 8.11.1

H @ashemetov,
Thanks for submitting.
Please read the contribution guidelines and provide the required information named there, especially the fiddle.

Issue is gone when I set 'acrossElements' to 'false'.


searching for more DEVELOPPERS

 Issue by Erkan-Yilmaz
Friday Jan 20, 2017 at 19:23 GMT
Originally opened as https://github.com/Erkan-Yilmaz/Gridcoin-tasks/issues/8

e.g. interns, from Google's Summer of Code
flyers at universities or other acedemic places
see also comments in Gridcoin roadmap

mentioned in hangout 1: 1:14:55-1:17:47

 Comment by grctest
Saturday Jan 21, 2017 at 20:49 GMT

I believe that Rob plans to recruit a couple developers soon-ish.

 Comment by MagixInTheAir
Monday Aug 28, 2017 at 20:23 GMT

Any guides on how to start ? List of requirements ? Ressources new developpers should be aware of ?
Thanks
I found : https://github.com/Erkan-Yilmaz/Gridcoin-tasks/issues/87

 Comment by TheCharlatan
Friday Sep 01, 2017 at 20:06 GMT

Hi @MagixInTheAir , join us on the slack channel: teamgridcoin.slack.com for dev talk in the #development channel.


hps: switch SPI flash to DDR mode

FYI @kgugala this works nicely on HPS, and gives a reduction in resource usage. But I confirmed this only reduces model evaluation time by ~0.35%.

FYI @kgugala this works nicely on HPS, and gives a reduction in resource usage. But I confirmed this only reduces model evaluation time by ~0.35%.

is this with picolibc and LTO?

Yes, these are the numbers I have in my spreadsheet running on HPS proto2 board, evaluating HPS model.
commit 10400f2f7fe0eafc32ed3c1f42d2f46aa553fb96 (last week)
= 193473536 cycles
commit 740150c96dc8e511f1254750bb4adf212ae3f414 (current main)
= 176910336 cycles
this PR
= 176288768 cycles
It suggests picolibc and LTO give a good speed improvement, as you reported, but effectively no change from doubling the SPI flash clock. It doesn't seem right.

maybe to biggest bottleneck is not in flash now?

@danc86, @kgugala: I'd like to merge https://github.com/litex-hub/litespi/pull/60 that simplifies DDR mode but need feedback (I was only able to test on Arty), could you do a test with it? Thanks.


Install MS MARCO Passage V1 Parquet regressions

Performed Faiss to Parquet conversion, setting up regressions from Parquet encoding of dense vectors.

Removed this statement from docs because it isn't true anymore...

Note that here we are explicitly using Lucene's NoMergePolicy merge policy, which suppresses any merging of index segments.
This is because merging index segments is a costly operation and not worthwhile given our query set.

LGTM!


Add --preemptible flag while creating instance

Separate command is added to create instance with --preemptible flag which cost lesser
but it will be shutdown automatically within 24 hrs.

Preemptible instances can be taken away in 10 minutes. There is no guarantee you'll keep the instance for 24 hours.


repart-image: remove --map-root-user flag from unshare

Description of changes
Fix for https://github.com/NixOS/nixpkgs/issues/311978. Remove --map-root-user flag from unshare invocation. Does not seem to break anything, and this lets us build system images for other platforms using binfmt and the binary cache
Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux (binfmt)
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

We could also pass the config into repart-image.nix and still include the flag when the build and host platforms are the same, but i thought I would try this first, since it isn't clear to me why the --map-root-user flag is required. Everything seems to be working fine without it

@ofborg test appliance-repart-image

Can't use dashes in system.image.version. Test is passing now

Can't use dashes in system.image.version. Test is passing now

This is actually a bug in nixos/version... I'm working on a fix.

Looks like we don't need unshare at all anymore. We should remove all of util-linux then.

@nikstur: removed it

@nikstur or @WilliButz: mind taking another look?

@nikstur or @WilliButz: mind taking another look?

If you remove the change to the image name (that was fixed), I'm happy to merge.

@nikstur: reverted the image name

@ofborg test appliance-repart-image


constants larger than 2^53 - 1 break

Because of the limitations in JavaScript any constants larger than 2^53 - 1 break (see Number.MAX_SAFE_INTEGER).
This wasn't a problem until TEALScript stopped manually tracking constant values and just relied on TS node types. For numbers, TEALScript needs to manually track constant values and save them as bigints to avoid this problem

Closed by #453


problem on adaptation pocketsphinx's acoustic model for jasper

i run jasper on raspberry pi 3 model b with jessie.
i used pocketsphinx for stt.
jasper works but recognition is very poor so i tried this url "https://cmusphinx.github.io/wiki/tutorialadapt/"
for acoustic adaptation.
i did some early steps but i couldn't finish all steps because of some errors.
jasper is a little diferent from thistutorial.
please if anyone had did all adaptation steps help me.
thank a lot.

in the adaptation step of this url :   https://cmusphinx.github.io/wiki/tutorialadapt/    i have this problem:
pi@raspberrypi:~/adaptation-test $ ./map_adapt -moddeffn hub4wsj_sc_8k/mdef.txt  -ts2cbfn .ptm. -meanfn hub4wsj_sc_8k/means  -varfn hub4wsj_sc_8k/variances -mixwfn hub4wsj_sc_8k/mixture_weights -tmatfn hub4wsj_sc_8k/transition_matrices -accumdir . -mapmeanfn hub4wsj_sc_8k-adapt/means -mapvarfn hub4wsj_sc_8k-adapt/variances -mapmixwfn hub4wsj_sc_8k-adapt/mixture_weights  -maptmatfn hub4wsj_sc_8k-adapt/transition_matrices
ERROR: "cmd_ln.c", line 602: Unknown argument name '-moddeffn'
ERROR: "cmd_ln.c", line 713: Failed to parse arguments list
ERROR: "cmd_ln.c", line 762: Failed to parse arguments list, forced exit
what should i do?

Better ask such question in cmusphinx forum


Feedback on /userland

First off, I love it ❤ 💯 🎉 🎈

I think it's so good, the format almost doesn't do it justice. I would suggest making the links on the front page into buttons or panels, something that really draws my attention to entice me to click them. Something like the following:
Can you put counts on the packages listed on the other page? I recognize they'd be out of date from the moment of inception, but how about some indicator of how popular/active?

Can you put counts on the packages listed on the other page? I recognize they'd be out of date from the moment of inception, but how about some indicator of how popular/active?

Maybe instead of showing numbers it could be simplified with just a range of 5... like for the stars it would just show from ⭐  to ⭐⭐⭐⭐⭐  stars maybe same with ⬇  downloads. Then the absolute number matters less and it can be updated once a week/month.

The wall of contributors avatars is pretty neat. But for me the avatrs in userland/package_authors don't show up. The url seems fine, but they show as having a1x1 size. Maybe a security issue that doesn't load them properly?

The link + description is a bit hard to parse. Like it's hard to see where the description starts. Maybe use the list like http://electron.atom.io/community or a different link/text color?

Opened #617, which addresses some of this stuff.

The url seems fine, but they show as having a1x1 size. Maybe a security issue that doesn't load them properly?

Which browser are you using, @simurai? I'm not able to reproduce in Chrome, Safari, or Firefox on macOS.

Chrome. Yep, other browsers are fine. Also Chrome in incognito mode is fine. Probably just some local glitch. Interestingly I also don't see your avatar on https://www.npmjs.com/~zeke.

Interestingly I also don't see your avatar on https://www.npmjs.com/~zeke.

I don't either.  @zeke is secretly a 👻

Hmm.
Can you see the image when you visit the raw URL?
https://s.gravatar.com/avatar/8f344b1c4bdcfc28bd848e97e94c3523?size=496&default=retro
Can you see others avatars on npm?
https://www.npmjs.com/~electron
https://s.gravatar.com/avatar/341d73a1c4caa8fec88ed24181562113?size=496&default=retro

Maybe you two are using an ad blocker that blocks gravatar domains?

Ad block is completely disabled and those image URL's are 503'ing for me.

Error 503 Backend fetch failed

Probably an issue on Gravatar's side of things

Maybe you two are using an ad blocker that blocks gravatar domains?

Ouch.. yes.. that's it in my case. Ok, whitelisted Gravatar.

I wanted to be super sure so I ran curl
C:\Users\Samuel>curl https://s.gravatar.com/avatar/8f344b1c4bdcfc28bd848e97e94c3523?size=496&default=retro
<!DOCTYPE html>

    <title>503 Backend fetch failed</title>

Error 503 Backend fetch failed

Backend fetch failed

Guru Meditation:

XID:<PHONE_NUMBER>

Varnish cache server

99% sure that isn't my fault 😆

@MarshallOfSound do you get something different if you wrap that URL in quotes?

Hopefully we don't have to set up a gravatar proxy..

Closing this out. We have followup issues for stuff mentioned in this issue, e.g. #622


tweak implementation of resourcecache

builds on https://github.com/openshift/library-go/pull/1235 to improve code-reuse

/lgtm


[hotfix][docs] Correct reference link for DB2 docs

Correct reference link for DB2 docs

Thanks @Jiabao-Sun for this contribution. I left a litter comments.

On line 163 of the file
and "latest-offset". Please see Startup Reading Positionsection
A space is required here
besides, LGTM

Thanks @rookiegao for the detailed review.
Fixed.


Expose properties so that an app can determine if AOT is active

This commit introduces an internal optimization which exposes new
properties to the application runtime, so that it can determine if
AOT is enabled. An easy way is for an application to define a bean
annotated with @ConfigurationProperties:
@ConfigurationProperties("micronaut.aot")
public class AotConfiguration {
    private boolean enabled;
    private String runtime;
    private List<String> optimizations;
    ...
}

The diagnostics that we expose are intentionally limited.
For example, we don't want to expose the whole configuration,
as it would contain entries which are sensitive to the build
environment (absolute paths for example) which would defeat
caching.

This is great! 🚀
Looking forward to it!


removed deprecated and nodejs 8 images from e2e tests and README

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind failing-test
What does does this PR do / why we need it:
Removes deprecated and nodejs 8 images from e2e tests
Which issue(s) this PR fixes:
Fixes https://github.com/openshift/odo/issues/2641
How to test changes / Special notes to the reviewer:
CI shouldn't fail

/approve

/retest

error: some steps failed:
  * could not run steps: step [release:latest] failed: failed to create or restart release pod: unable to create pod: timed out waiting for the condition

/retest

/hold
Readme needs an update too @mik-dass - give me a second

/retest

/lgtm

/retest


Garnet installation not working

tobi@tyche:~/Downloads$ adb install Settings_GARNETEEAGlobal_from_OS<IP_ADDRESS>.UNREUXM.apk
Performing Streamed Install
adb: failed to install Settings_GARNETEEAGlobal_from_OS<IP_ADDRESS>.UNREUXM.apk: Failure [INSTALL_FAILED_INVALID_APK: Package com.android.settings are not updateable.]
My device has <IP_ADDRESS>.UNREUXM

I guess the ROM version <IP_ADDRESS> is too new to downgrade.

my rom is 1.0.11 same error


Ardusub parameter

Hi,
I would like to know if there have been any modifications to the ArduSub parameters?

Hi, I have discovered that the Ardusub parameter is automatically modified by the extension available at this GitHub repository


typos

Typos in /includes

Deploy preview for calico ready!
Built with commit 49dc8e135a4b98cdd44f8629afb9b46f0ff91e42
https://deploy-preview-3250--calico.netlify.com


Document declare V2 from CLI

We should create a section in cairo 1 support for this command
When we have this we can release 0.9.3

If nobody takes it until next Wednesday, I can work on this issue.