[BUG]: API: Example password for creating new users is too short

How are you running AnythingLLM?
Docker (local)
What happened?

the example password "hunter2" is too short. All passwords are required to have 8 signs. Use "hunter21" instead. ;)
Are there known steps to reproduce?
No response

While true...

You should never use the example for a password in the documentation
This setting is actually configurable by the admin via the ENV settings
https://github.com/Mintplex-Labs/anything-llm/blob/master/server/models/user.js#L237


Change user id column type to uuid incl all createdBy / modifiedBy foreign key colums

Better now than later
ALTER TABLE app_user alter id type uuid USING id::uuid

https://stackoverflow.com/a/51052624/4292075
reaplce uuid for system user with 00000000-0000-0000-0000-000000000001

todo: add "ON DELETE SET NULL" to FK Contraints see https://codeburst.io/edit-existing-foreign-key-constraints-in-postgresql-with-the-alter-table-command-c1de89256ff7

todo: add "ON DELETE SET NULL" to FK Contraints see https://codeburst.io/edit-existing-foreign-key-constraints-in-postgresql-with-the-alter-table-command-c1de89256ff7


please assist me

hi
I am trying to validate my token  wizard contract here: https://etherscan.io/verifyContract
Having some issues is there a guide somewhere i can use please?
also can tokens be bought directly from the contract address here:
0xe33a11ae66f8a17f7bbfb3b7d7b1d60a18970379
again is there a guide and if they are bought directly will the bonus amounts apply or must it go through the wizard

Verification guide:
https://github.com/poanetwork/token-wizard/wiki/Token-Wizard-Verifying-Contracts
Contribution guide:
https://github.com/poanetwork/token-wizard/wiki/Contribution-to-crowdsale
FAQ section:
https://github.com/poanetwork/token-wizard/wiki/FAQ
If anything is unclear use Token wizard gitter to ask your questions


subnet_ids expect string but is list

I get the following error when creating a web-service module.
Error configuring: 1 error(s) occurred:

* variable subnet_ids in module browser_client should be type string, got list

Go the configuration from pingdummy. Unsure if this is a Stack issue or pingdummy but I thought I would report it regardless. Changed my config to join by , which seemed to fix the issue.
subnet_ids       = "${join(",",module.stack.external_subnets)}"

This is the expected way of doing it I believe, there hasn't been work done on porting to terraform 0.7+ to support passing lists as variables.

@achille-roussel We need to use terraform lower then v0.7 ? I saw this https://github.com/segmentio/stack/blob/master/circle.yml#L4
I am getting a same problem as @esbenp , fixed with join


Enhance gorm to specify the table to search.

Enhance gorm to specify the table to search.
Enable query syntax as following syntax
type DataObj struct {
	ID            int    `json:"id" gorm:"primary_key"`
	Data      string `json:"string"`
}
dataObj := DataObj{}
rep.db.Debug().SetTable(&dataObj).Where("id = ?", 64).Scan(&dataObj)

--- Result SQL
SELECT * FROM "data_objs"  WHERE (id = 64)

Make sure these boxes checked before submitting your pull request.

[] Do only one thing
[] No API-breaking changes
[] New code/logic commented & tested

For significant changes like big bug fixes, new features, please open an issue to make an agreement on an implementation design/plan first before starting it.
What did this pull request do?

Have you tried .Model()?


Add flag to print configuration

It would be useful to be able to print current configuration without uncommenting and recompiling the application here . Perhaps there could be a --print-config option that does this on demand.

Should be easy to add, will try to do it tomorrow.


matlab: support cell mode separators

This patch adds some support for Matlab comments which are actually cell mode separators.
In Matlab you can write a script in "cell mode" where each cell (delimited by a single line comment which starts with %%) can be executed separately when using the Matlab editor.
In the syntax file, I highlighted such comments as Todo (to get the background highlighted). In the ftplugin, I redefined the nroff section movement commands [[ ]] ][ [] to move between cells. The code is pretty much copied from Learn Vimscript the Hard Way.

I've emailed the maintainers of these two files.

Maurizio expressed via email a desire to transfer maintainership of the MATLAB syntax file. I don't know the protocol here, but I've added a commit to do that.

Thanks, I'll include the patch.  For further changes you can just email me the new version.

It looks like syntax/matlab.vim made it in, but ftplugin/matlab.vim has not been updated.

Alex Burka wrote:

It looks like syntax/matlab.vim made it in, but ftplugin/matlab.vim has not been updated.

It has a different maintainer.  I haven't heard from him.
--
The History of every major Galactic Civilization tends to pass through
three distinct and recognizable phases, those of Survival, Inquiry and
Sophistication, otherwise known as the How, Why and Where phases.
For instance, the first phase is characterized by the question 'How can
we eat?' the second by the question 'Why do we eat?' and the third by
the question 'Where shall we have lunch?'
-- Douglas Adams, "The Hitchhiker's Guide to the Galaxy"
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


fix rendering error if list inside tt_container

There is a bug if mode == 'vertical' and tt_tab includes list (ul + li).
The content lis and the tab lis are mixed. Some content will duplicate and some navigation lis are unusable.
This is fixed if you use selector ">li" instead of "li".

Thanks for your contributon :)


adding readme for samples

Identity README : Fix API changes in example
KV-secret : move samples section into sample/README.md

Fixes #5935

The repo's root README.md also needs a s/October/November/g


Change default CONFIG to none

Default to using provided CXX (either via environment variable or make).
Some of the ifeq ($(CONFIG), ..) checks might also be worth changing to check $(CXX) instead to, e.g. use a clang sanitizer if one is selected without config-clang.

This changes default behavior since now CXX is detected and on Linux it gets defaulted to g++, and current Makefile is enforcing clang by default.
Suggest removing -Wno-c++11-narrowing from Makefile since that will make ABCMKARGS behave same on gcc and clang config paths. (best checking building abc by removing abc directory before build)

This changes default behavior since now CXX is detected and on Linux it gets defaulted to g++, and current Makefile is enforcing clang by default.

To be honest it was pretty weird that clang would be picked by default even on Linux (I strongly prefer clang but it's still weird).

This changes default behavior since now CXX is detected and on Linux it gets defaulted to g++, and current Makefile is enforcing clang by default.

There's nowhere that actually says what the default compiler is, just that running make config-* was necessary for a specific compiler so I don't think that's a problem.  And as someone who prefers clang I would always run make config-clang on a fresh install, having assumed until making this PR that gcc would be the default.


ubuntu apache2 header rewrites note

in the deploying .net core apps with apache section it mentions setting up forwarded headers and configuring them in the example .conf file:
<VirtualHost *:*>
    RequestHeader set "X-Forwarded-Proto" expr=%{REQUEST_SCHEME}
</VirtualHost>

Using ubuntu - might be good to add a note in here that this might not necessarily work out of the box unless you enable the apache headers module with
sudo a2enmod headers
Also, you may need to disable the default apache domain with sudo a2dissite 000-default.conf  and enable the example domain file with sudo a2ensite helloapp.conf
I couldn't use the existing document to get my example up and running without doing these two additional things on ubuntu and was a real head scratcher.

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

ID: cfd9a1f0-c1b6-4443-0700-4c5beafeed03
Version Independent ID: c22ca7a0-d1d4-200d-fc36-a9798d350c6e
Content: Host ASP.NET Core on Linux with Apache
Content Source: aspnetcore/host-and-deploy/linux-apache.md
Product: aspnet-core
Technology: aspnetcore-hostdeploy
GitHub Login: @Rick-Anderson
Microsoft Alias: shboyer

Fix and through the three distros with Apache reverse proxy for proper instruction that we are now covering in issue #28060


[Utils] Add log timestamp type

This patch adds types fo log timestamp type.
NNTRAINER_LOG_TIMESTAMP_SEC: year, mon, day, hour, min, sec NNTRAINER_LOG_TIMESTAMP_MS: milliseconds from log start
Signed-off-by: Jiho Chu<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2076. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @jihochu, nntrainer/nntrainer_logger.h does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md


Cross compiling using cmake-tools

This issue is being closed because it appears to have been abandoned. If you require further assistance, please let us know and we can reopen it.


How to uninstall?

I'd like to uninstall in win11 but there is no uninstaller.
While there are registry changes (like it is said in the description) it should be not enough to just delete the folder in Programme files.
Best regards

Hi. Since more people were asking I'll add an option in the installer to
remove.
wt., 21 cze 2022, 06:11 użytkownik Maz111111 @.***>
napisał:

I'd like to uninstall in win11 but there is no uninstaller.
While there are registry changes (like it is said in the description) it
should be not enough to just delete the folder in Programme files.
Best regards
—
Reply to this email directly, view it on GitHub
https://github.com/lyckantropen/moonlight_hdr_launcher/issues/17, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AASJWLVMINZCVU7R4VCLJU3VQE6GRANCNFSM5ZK4PHXQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Release 1.2.0 adds an uninstall option to the installer (d8612eb). Apologies for taking a longer time.


CDPD-4781 Increase HBase max blocking store files

Increase hbase.hstore.blockingStoreFiles configuration value to 50 for HBase RegionServers.

Nice! Looks about right.
Have you tested this out by hand? I can walk you through that if not. Cloudbreak doesn't have automation in place to validate blueprint changes, so we can do them a big favor by making sure it works as expected.

I haven't done testing apart from verifying CM uses hbase_hstore_blockingStoreFiles for hbase.hstore.blockingStoreFiles.

Testing done:
Added custom cluster template and created Data Hub Clusters based on this template using GBN 1511602.
On the created cluster I checked configuration for hbase.hstore.blockingStoreFiles and it is 50.

@keyki can you or someone on the team please help us get this merged? Peter has done his due-diligence to test this and show it works. Thanks in advance!


Add metrics exemplars to explore metrics page

Describe the issue or suggestion
There are docs that explore the metrics page. A new feature in 8.1 is metrics exemplars: https://learn.microsoft.com/en-us/dotnet/aspire/fundamentals/dashboard/explore#metrics-page

Yes, OTEL - exemplars: https://opentelemetry.io/docs/specs/otel/metrics/data-model/#exemplars


[Question] http request against authorized api:s and  server pre-rendering

Hi folks
Have a question in regards to the prerendering functionality. It seems that when prerendering is run it doesnt recognize for example an active forms session through Identity (where the method is utilizing the Authorize attribute). It will render on the server-side, resulting in a status 401, then rendered again at the client-side (this time successfully).
The question is two-fold. 1: Is there any way of making the prerendering functionality take identity authorization into consideration and if not, is there preferred way (other than buildning control mechanism yourself) which can be used to only run certain ng2 calls / methods if rendered on client-side?

The intended way to pass through whatever auth tokens you need into your server-side JavaScript is via the aspnet-prerender-data parameter (see the docs under the heading Passing data from .NET code into JavaScript code here).
We have considered trying to do some kind of automatic transparent pass-through of cookie data, so that any calls to Node's http get/post APIs automatically have attached whatever cookies originally came into the ASP.NET application. But from a security point of view that's very dangerous - we'd have to reimplement the browser's same-origin policy logic completely and perfectly, plus all the CORS logic that runs in browsers, otherwise you might unexpectedly leak auth credentials to third party API endpoints. We're not realistically going to do that, plus even if we did, it would only account for cookie-based auth, and not any other kind of authentication that's common in SPAs (such as just holding tokens in memory and including them in fetch calls).
So to answer your questions directly:

1: Is there any way of making the prerendering functionality take identity authorization into consideration

Yes, get whatever auth tokens you want in your .NET code, and then pass them through into your boot-server.ts logic via the aspnet-prerender-data API. See the docs linked above for exact details. Then your Node-side code can include it in any API calls you make.

2: is there preferred way (other than buildning control mechanism yourself) which can be used to only run certain ng2 calls / methods if rendered on client-side?

Yes, you can have logic like this:
if (typeof window === 'undefined') {
    // code that runs only on the server
} else {
    // code that runs only on the client
}

If you have any proposals for how we could make this better then please let us know!

You can also import isBrowser or isServer from Universal to test whether it's browser / server as well. Just throwing that out there!

@MarkPieszak Cool, I didn't know about those. Thanks for the info!

Thanks guys! Suspected something like isBrowser, isServer, just couldnt find it. If i have any useful input / suggestions ill post it :).

We have considered trying to do some kind of automatic transparent pass-through of cookie data, so that any calls to Node's http get/post APIs automatically have attached whatever cookies originally came into the ASP.NET application.

We went through this same thought process for our own APIs and ended up implementing an isomorphic HTTP request library that uses cookies that were passed in from the C# process when on the server side, but we have to be careful to only pass through cookies for internal requests. As @SteveSandersonMS said this would be either very difficult or very dangerous to implement in a generic way. :smile:


Update architecture-overview.md

Many updates  - see word doc for Pictures

Hmmm – I see the old page:
https://documentation-accp.cfapps.io/best-practices/architecture/introduction/architecture-overview
From: Adam Dupaski<EMAIL_ADDRESS>Sent: vrijdag 8 maart 2019 11:49
To: mendix/docs<EMAIL_ADDRESS>Cc: Andreas Lennevi<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [mendix/docs] Update architecture-overview.md (#1533)
Merged #1533https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmendix%2Fdocs%2Fpull%2F1533&data=02|01||1a776477618b432b0c5708d6a3b3a5f8|b4e3c78d8e3b46d8bc565540da23ba4d|0|1|636876389323125059&sdata=tYN1gLy7bE4mjKYSkHe%2FM0OWr%2F%2Fj29rpRyIRJZdnCXM%3D&reserved=0 into development.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmendix%2Fdocs%2Fpull%2F1533%23event-2189780430&data=02|01||1a776477618b432b0c5708d6a3b3a5f8|b4e3c78d8e3b46d8bc565540da23ba4d|0|1|636876389323125059&sdata=Op1UMRsXeKw1Uvpk%2BGB2YJN1CKN4K9kGkSa0Mt6jNd0%3D&reserved=0, or mute the threadhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAq4c_dPSdPlTMxeXlU_ArAwfPrTnvaQbks5vUkAQgaJpZM4bk-0s&data=02|01||1a776477618b432b0c5708d6a3b3a5f8|b4e3c78d8e3b46d8bc565540da23ba4d|0|1|636876389323135068&sdata=G154M98DunBX5837TwtHxziu17%2BW5X01EdNvXx96vz4%3D&reserved=0.

It takes 10min to build, then clear your cache.
From: AndreasLennevi<EMAIL_ADDRESS>Sent: March 8, 2019 11:52 AM
To: mendix/docs<EMAIL_ADDRESS>Cc: Adam Dupaski<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [mendix/docs] Update architecture-overview.md (#1533)
Hmmm – I see the old page:
https://documentation-accp.cfapps.io/best-practices/architecture/introduction/architecture-overview
From: Adam Dupaski<EMAIL_ADDRESS>Sent: vrijdag 8 maart 2019 11:49
To: mendix/docs<EMAIL_ADDRESS>Cc: Andreas Lennevi<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [mendix/docs] Update architecture-overview.md (#1533)
Merged #1533https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmendix%2Fdocs%2Fpull%2F1533&data=02|01||1a776477618b432b0c5708d6a3b3a5f8|b4e3c78d8e3b46d8bc565540da23ba4d|0|1|636876389323125059&sdata=tYN1gLy7bE4mjKYSkHe%2FM0OWr%2F%2Fj29rpRyIRJZdnCXM%3D&reserved=0 into development.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmendix%2Fdocs%2Fpull%2F1533%23event-2189780430&data=02|01||1a776477618b432b0c5708d6a3b3a5f8|b4e3c78d8e3b46d8bc565540da23ba4d|0|1|636876389323125059&sdata=Op1UMRsXeKw1Uvpk%2BGB2YJN1CKN4K9kGkSa0Mt6jNd0%3D&reserved=0, or mute the threadhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAq4c_dPSdPlTMxeXlU_ArAwfPrTnvaQbks5vUkAQgaJpZM4bk-0s&data=02|01||1a776477618b432b0c5708d6a3b3a5f8|b4e3c78d8e3b46d8bc565540da23ba4d|0|1|636876389323135068&sdata=G154M98DunBX5837TwtHxziu17%2BW5X01EdNvXx96vz4%3D&reserved=0.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmendix%2Fdocs%2Fpull%2F1533%23issuecomment-470887892&data=02|01||da1f9e0569364c7bff3508d6a3b40a52|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|636876391004271308&sdata=uIlPvp7oNdKxL4vgDuvlp5xqz55Ev1bqnjL1aMfUjG0%3D&reserved=0, or mute the threadhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FATZPMToeLuWrEyJdgjQVdCKyd_qIkI7eks5vUkC5gaJpZM4bk-0s&data=02|01||da1f9e0569364c7bff3508d6a3b40a52|b4e3c78d8e3b46d8bc565540da23ba4d|0|0|636876391004271308&sdata=cCGA3vwKElMpdJoXDxyid7LxDMPrYILIuezQBv67wZw%3D&reserved=0.


soc: imxrt1064: Minor fixes with CONFIG_PM build

This PR fixes 2 issues when building with CONFIG_PM on the the i.MX RT1064 SoC.

@danieldegrasse is the low-power mode tested on the mimxrt1064 evk if it works properly?

@pdgendt I have not tested with that setup, no. As you can see, the code will disable the video PLL when entering low power mode, but I have not verified this using a display. The code should simply reenable the PLL with the same configuration when exiting low power mode. Are you seeing different behavior?
My testing was done by running the tests/subsys/pm/power_mgmt_soc test. Regression testing was also run against this board using twister (see the initial PR: https://github.com/zephyrproject-rtos/zephyr/pull/41172#pullrequestreview-897129154)

@pdgendt What is your test setup? I assume you have a display connected to the EVK, but what sample are you running and flashing?

@pdgendt What is your test setup? I assume you have a display connected to the EVK, but what sample are you running and flashing?

@danieldegrasse it’s a custom board with custom display connected to eLCDIF

@pdgendt Thank you for the clarification- I think this may be enough for me to test with, but an example would also be helpful if you are willing to provide one. If the issue appears without any changes being made to the display, it is likely an issue with the low power mode code implementation. I'll try a minimal application with the display driver enabled like you described and see what occurs on the EVK.

Codecov Report

Merging #47639 (3d2b04c) into main (b161554) will decrease coverage by 0.01%.
The diff coverage is 30.37%.

:exclamation: Current head 3d2b04c differs from pull request most recent head 07a548e. Consider uploading reports for the commit 07a548e to get more accurate results

@@            Coverage Diff             @@
##             main   #47639      +/-   ##
==========================================
- Coverage   50.14%   50.12%   -0.02%
==========================================
  Files         662      662
  Lines       82606    82639      +33
  Branches    19430    19434       +4
==========================================
+ Hits        41419    41420       +1
- Misses      34315    34349      +34
+ Partials     6872     6870       -2

Impacted Files
Coverage Δ

drivers/flash/flash_shell.c
17.94% <ø> (ø)

include/zephyr/debug/coredump.h
20.00% <ø> (ø)

modules/openthread/platform/radio.c
62.72% <ø> (ø)

subsys/bluetooth/host/conn.c
0.00% <ø> (ø)

subsys/bluetooth/host/hci_core.c
18.09% <ø> (ø)

subsys/bluetooth/host/id.c
7.84% <0.00%> (+0.13%)
:arrow_up:

.../debug/coredump/coredump_backend_flash_partition.c
42.70% <0.00%> (-12.70%)
:arrow_down:

subsys/net/lib/lwm2m/lwm2m_observation.c
3.18% <0.00%> (ø)

subsys/pm/device.c
60.52% <60.00%> (-1.83%)
:arrow_down:

subsys/pm/device_runtime.c
71.11% <75.00%> (ø)

... and 4 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 87692cd...07a548e. Read the comment docs.

@danieldegrasse Thanks for getting back with the results.
I can't really give a sample where the issue is reproducible, I have enabled all drivers for my custom board and tried to put it into the suspend state. This is where it goes wrong, so it can be any peripheral.
I'll get back to you once I figure it out.

@danieldegrasse I've made a minimal application, where I can go into suspend after a delay but it fails to wake up.
Board defconfig
CONFIG_SOC_MIMXRT1064=y
CONFIG_SOC_SERIES_IMX_RT=y
CONFIG_CONSOLE=y
CONFIG_UART_CONSOLE=y
CONFIG_SERIAL=y
CONFIG_GPIO=y
CONFIG_MPU=y
CONFIG_ARM_MPU=y
CONFIG_PINCTRL=y
CONFIG_FPU=y
CONFIG_FP_HARDABI=y
CONFIG_XIP=y

prj.conf
CONFIG_PM=y
CONFIG_PM_DEVICE=y
CONFIG_IDLE_STACK_SIZE=4096

main.c
#include <stdio.h>
#include <zephyr/zephyr.h>
#include <zephyr/device.h>
#include <zephyr/pm/policy.h>

LOG_MODULE_REGISTER(main);

static int32_t sleep_time_ms = 20000;

/* This has to be done before anything tries to sleep, which means
 * before the threading system starts up between PRE_KERNEL_2 and
 * POST_KERNEL.  Do it at the start of PRE_KERNEL_2.
 */
static int disable_suspend(const struct device *dev)
{
	ARG_UNUSED(dev);

	if (!pm_policy_state_lock_is_active(PM_STATE_SUSPEND_TO_IDLE, PM_ALL_SUBSTATES)) {
		pm_policy_state_lock_get(PM_STATE_SUSPEND_TO_IDLE, PM_ALL_SUBSTATES);
	}

	return 0;
}
static int enable_suspend(const struct device *dev)
{
	ARG_UNUSED(dev);

	if (pm_policy_state_lock_is_active(PM_STATE_SUSPEND_TO_IDLE, PM_ALL_SUBSTATES)) {
		pm_policy_state_lock_put(PM_STATE_SUSPEND_TO_IDLE, PM_ALL_SUBSTATES);
	}

	return 0;
}
SYS_INIT(disable_suspend, PRE_KERNEL_2, 0);

void main(void)
{
	while (true) {
		k_msleep(sleep_time_ms);
		printk("suspending\n");
		enable_suspend(NULL);

		k_msleep(sleep_time_ms);
		printk("wakeup\n");
		disable_suspend(NULL);
	}
}

Also worth noting is that I have a reduced DCD header compared to the imx rt1064 evk where all EMC and SEMC entries are removed.
It's hard to figure out, since the debugger can't be used, any suggestions on how to find the issue?
Thanks!


Mobile app menu - close on mask tap

Closing the mobile app menu requires selection of an option. Better to close on mask tap
While looking at this - can the mask be made lighter to match that shown behind dialogs? I find it somewhat dislocating currently and think a more consistent lighter mask would help

https://github.com/exhi/hoist-react/pull/1215


The acceleration of ort-stable-diffusion is not significant compared to pytorch-stable-diffusion

I have made a stable-diffusion test and compared the speed between ort and pytorch version, hope to get results similar to the example.
But the acceleration between ort-stable-diffusion and pytorch-stable-diffusion is not significant like the example.
This is my result:

my command:
ort: accelerate launch --config_file=accelerate_config.yaml --mixed_precision=fp16 train_text_to_image.py --ort --pretrained_model_name_or_path=CompVis/stable-diffusion-v1-4   --dataset_name=lambdalabs/pokemon-blip-captions --use_ema --resolution=512 --center_crop --random_flip --train_batch_size=1 --gradient_accumulation_steps=4 --gradient_checkpointing   --max_train_steps=5000 --learning_rate=1e-05 --max_grad_norm=1 --lr_scheduler=constant --lr_warmup_steps=0 --output_dir=sd-pokemon-model
pytorch:
accelerate launch --config_file=accelerate_config.yaml --mixed_precision=fp16 train_text_to_image.py --pretrained_model_name_or_path=CompVis/stable-diffusion-v1-4   --dataset_name=lambdalabs/pokemon-blip-captions --use_ema --resolution=512 --center_crop --random_flip --train_batch_size=1 --gradient_accumulation_steps=4 --gradient_checkpointing   --max_train_steps=5000 --learning_rate=1e-05 --max_grad_norm=1 --lr_scheduler=constant --lr_warmup_steps=0 --output_dir=sd-pokemon-model
Who can help me please!

Hi @zhengcaizheng, the compute target you reference is different than the V100s that were used for the reported results. Additionally, 8 V100s GPUs were used to train the model in distributed fashion. These differences explain the lack of speedup you see. Please let me know if there is anything else we can help you with.


EXC_BAD_ACCESS swift 4

I have this problem, any idea ?

I solved, I forgot to add the class into view

thanks


Support for Inline price data

Feature request

Extension name: firestore-stripe-subscriptions

Is your feature request related to a problem? Please describe.
Stripe supports Checkout Sessions with inline/custom prices: https://stripe.com/docs/payments/checkout/set-up-a-subscription#creating-prices-inline
Can we get support for that in this extension?
Describe the solution you'd like
Describe alternatives you've considered
Additional context

Digging into the next release (2020-08-20) shows that support for multiple line items is there. Does that mean that I can simply pass a single line item with price_data instead of price to use inline/custom prices?

@moesalih Yes, this would work via passing price_data in line_items, however in most cases you should not send price data from the client side, as that information can be manipulated by the sender (e.g. customer opening dev tools and modifying your price data).
You should only use this approach from the client-side if the price is meant to be modifiable by the customer, e.g. they can decide how much they want to pay via an input field etc. In all other cases you should define the price data securely in Stripe beforehand.


Added Github Action for publish

coverage: 94.128%. remained the same when pulling e1604e9029fea9bcf9d6caa0ac3b3d6ffc22e32c on cody-publish-github-action into 10b8d3587ea0614a2556b8db432962a7a4e79d09 on main.


Post new videos

Is your feature request related to a problem? Please describe.
There currently isn't the ability to post new videos/content.
Describe the solution you'd like
An API call we can use to post new videos/content.

Same question :)

Closing as this is a duplicate of #57.


use the last version of decompress

Due to the release of git.3.0.0 which includes the new version of decompress, it could be interesting to upgrade albatross to the last vesion of decompress (>= 1.0.0).

thanks. just one question: is the compression done by decompress < 1.0.0 compatible with >= 1.2.0?

thanks. just one question: is the compression done by decompress < 1.0.0 compatible with >= 1.2.0?

thanks. just one question: is the compression done by decompress < 1.0.0 compatible with >= 1.2.0?

Yes, the format does not change, it's only implementation and speed improvement (in details, we still respect RFC1951 + zlib header).

thanks. just one question: is the compression done by decompress < 1.0.0 compatible with >= 1.2.0?

Yes, the format does not change, it's only implementation and speed improvement (in details, we still respect RFC1951 + zlib header).

thanks, pushed as 38f5085 with the tab character in src/dune converted to spaces.

thanks, pushed as 38f5085 with the tab character in src/dune converted to spaces.


Add Deadlocked (phonk) by Cerulity

Song Name
Deadlocked (phonk)
Artist Name
Cerulity
Source
Youtube
Youtube Link
QsoL69j5pzCR62mx
Direct File Link
No response
Song ID
19
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 11549880

reject
invalid YouTube link


Consider whether we should do something about deafness

As pointed out in https://github.com/NCATSTranslator/Feedback/issues/286#issuecomment-1579007519, "some people believe that deafness is neither a disability nor a burden; maybe avoid the perception we are taking a position on that". A lot of terms have deafness as a synonym. Is there some middle ground we can take -- not blocking the term, but providing some context for its use.

This is really a question for the Blocklist; I'm moving this issue to https://github.com/NCATSTranslator/Blocklists/issues/17 and closing it here.


Account Login and Tokens

Make it so that model runs only happen in the context of a particular user, using their tokens to run the model and associate it with their account.

[X] Provide users with tokens
[X] Make token for logged in user available within Stormchaser
[x] Send token as part of authorization header with requests from stormchaser
[x] Store token in SessionStorage so that users don't have to log in again after reloading the page (should we use LocalStorage instead?)
[x] Define model area that user is running and send that as part of request for regions
[ ] Only return regions if user is a member of a group/organization that has access to run the model area

I'm moving some of this up to the current milestone because I'm working on it now.

Still need to change how we set and select the current model area the user is working in. We might want to select a per-user default model area, then have a Django permission that specifies if they're allowed to change their model area, at which point, they see an option in the interface. This is mostly a pretty advanced feature though since in our current deployment plans, installs won't have more than one model area. But we still need to do something to stop hardcoding the organization/model area IDs into stormchaser - it needs to come from the API or a user definition in the application in some way.

Implemented Model Area selector for those with access to multiple areas.

Implemented Model Area selector for those with access to multiple areas.


Update Windows build requirements

Changed the required compiler to VS 2013 as node zmq requires the v120
build tools.
Removed required zmq install.

Visual Studio 2015 Community doesn't seem to provide the required build tools for node-gyp required by zmq.node.
Tested on windows 10 with fresh install of atom.

Thanks for this!


Curious whether MicroWebSrv supports https?

I’ve been wanting to handle web requests using https on a PyCom WiPy 3.0 for awhile now without success.  Maybe this is asking too much of the hardware?
Thanks in advance,
Tim

Hello,
I released a fully new version (v2.0) of my web server here :
github.com/jczic/MicroWebSrv2.
Open source MIT, fully asynchronous, more robust, more fast and more
efficient!
It is delivered with a good documentation.
Thank you for your support and feedback. ☺
Jean-Christophe Bos
Le mar. 29 janv. 2019 à 04:15, Timothy M. Shead<EMAIL_ADDRESS>a écrit :

Thanks for all the help, I’ll check it out!
Cheers,
Tim
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jczic/MicroWebSrv/issues/48#issuecomment-458391543,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAegLMT8U51Bpl20RTX4yWoF8KX8xZ7Iks5vH7y7gaJpZM4aUMkp
.

--
Sincèrement,

Thanks for the followup on this … I will check the new version out!
Cheers,
Tim


Apps: displayRange min and max should be numbers

See https://github.com/VEuPathDB/microbiomeComputations/issues/17
Update the api here.

Closed by https://github.com/VEuPathDB/web-eda/issues/1107
Nothing to QA. Moving to Complete


Improved API response validation

Issue #, if available:
#285
Description of changes:
Adds additional validation logic and unit tests to make sure the SDK gracefully (non-disruptively) handles invalid responses from the sampling APIs.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hello @willarmiros @bhautikpip
I have problem with this PR after update to 3.0.1 of sdk.
In logs I see many errors Error: AWS X-Ray SamplingTargets API returned invalid response
After debugging I see that response from daemon looks like that:
{
  LastRuleModification: 0,
  SamplingTargetDocuments: [
    {
      FixedRate: 0.05,
      Interval: 10,
      ReservoirQuota: 0,
      ReservoirQuotaTTL:<PHONE_NUMBER>,
      RuleName: 'Default'
    }
  ],
  UnprocessedStatistics: []
}

It looks ok right? So why in code is condition to throw if LastRuleModification is 0 or unset?
Thanks for help.

@rysi3k that was a miss on my end, it should treat that response as valid and is only creating an error because the value 0 is falsey. I'll get a fix out.


Fix: Replace basic field with empty value in TerserUtil

I came across this bug when doing some resource merging logic.
Before the fix, TerserUtil would fail silently and log the error at debug level.

It looks like some of the tests in hapi_fhir_jpaserver_mdm are a bit flaky. It does not seem to be related to this change.

It looks like some of the tests in hapi_fhir_jpaserver_mdm are a bit flaky. It does not seem to be related to this change.

Yup the MDM tests are undergoing a minor reckoning right now. Unrelated to this change.

@nigtrifork I have added credit for your fix here. Can you review this PR and ensure my description of the fix is roughly accurate? Thanks for the contribution!
https://github.com/hapifhir/hapi-fhir/pull/4865


feat(sdk-metrics): bootstrap metric streams

Which problem is this PR solving?
This is a prototype of the implementation described in https://docs.google.com/document/d/1EluDhj5UanNATS3hR8xx30sv55Se7uYjsGYslvNWHtc/edit#.
Bootstrap metric streams:

[x] instruments delegate measurements to WritableMetricStorage
[x] SyncMetricStorage (a WritableMetricStorage) stashes delta accumulations in DeltaMetricStorage
[x] DeltaProcessor flushes accumulations when the data is being collected so it won't persist in memory
[x] SyncMetricStorage builds metric data with TemporalMetricStorage for collector's preferred aggregation temporality
[x] TemporalMetricProcessor stashes unreported accumulations for each collector
[x] TemporalMetricProcessor keeps a reported history for each collector for temporality merging.

Related: https://github.com/open-telemetry/opentelemetry-js/issues/2574
Depends on: https://github.com/open-telemetry/opentelemetry-js/pull/2634
Type of change

[x] New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[x] Basic interface testing.
[x] Multiple collectors initiating collections should not have side effects on each other.
[x] Integrated testing with MeterProvider/Meter interface.

Checklist:

[x] Followed the style guidelines of this project
[x] Unit tests have been added
[x] Documentation has been updated

to those @open-telemetry/javascript-approvers who didn't see, this is now ready for review.


Support configuring the interface (i.e. host) the server will listen to

I am trying to integrate livereload with an Express application running on Replit

When deployed there, Replit assigns you a hostname
It proxies all incoming connections to your service, so long as it listened to <IP_ADDRESS>

To make this all work I then need the Express application to listen on <IP_ADDRESS> (not a big deal, it's the default)
I also need to tell livereload to listen on a a different interface, e.g. <IP_ADDRESS>

But this, unfortunately is currently not supported

Is there appetite for adding an additional config, host, for livereload to use while creating the listening server? Let me know, I have already pushed some changes into my branch, and it will not take me much to create a PR off of it.
Thanks,
M.

@iamFIREcracker Yea I'm not opposed to this.  Let's use -b for the CLI option, and --bind for the long option, and bindHost as the argument?
Do you want to make a PR?


Change error messages

Hi,
I'm still clueless to change error message when using text-io, do I need to call withParseErrorMessagesProvider for every input reader that have been made?
Thanks.

Yes, because this method is intended to customize the error message of a specific input reader. There is currently no API method to set a global custom ErrorMessageProvider.
Here is an example of changing the error messages of an age input reader:
int age = textIO.newIntInputReader()
        .withDefaultValue(0)
        .withValueChecker((val, itemName) -> {
            if(val < 0) return List.of("How is this possible? Do you come from the future?");
            else if(val > 120) return List.of("Oh, come on! Nobody is that old.");
            else return null;
        })
        .withParseErrorMessagesProvider((sVal, itemName) -> List.of(
                "What do you mean by " + sVal + "?.

I just want to have custom error message and custom validation, and that code is exactly what I needed.
But days after tweaking it I get into more better solution, this code set the withMinLength to zero to be able to use custom validation and will loop and clean the console when there is an error:
String name;
textIO.getTextTerminal().setBookmark("name");
while (true) {
    name= textIO.newStringInputReader().withMinLength(0).read("Enter your name: ");
    if (name.length() < 3) {
        textIO.getTextTerminal().println("Name is not valid!");
        textIO.newStringInputReader().withMinLength(0).read("Press enter to continue.");
        textIO.getTextTerminal().resetToBookmark("name");
        continue;
    }
    break;
}

Thanks for the response :)


surface: command not found

Hi, qzed!
First, thank you for your hard work to provide us this convenient command line tool!
I want to try it so I downloaded it and build from source on my surfacebook2. However, I'm not familiar with cargo and rust, I followed the instructions in the README file, and get this output:
...
Finished dev [unoptimized + debuginfo] target(s) in 16.19s
     Running `target/debug/surface`

surface 0.2.1
Maximilian Luz<EMAIL_ADDRESS>Control various aspects of Microsoft Surface devices

USAGE:
    surface [FLAGS] <SUBCOMMAND>

FLAGS:
    -h, --help       Prints help information
    -q, --quiet      Keep output quiet
    -V, --version    Prints version information

SUBCOMMANDS:
    dgpu           Control or query the dGPU power state on the Surface Book 2
    help           Prints this message or the help of the given subcommand(s)
    latch          Control the latch/dtx-system on the Surface Book 2
    performance    Control or query the current performance-mode
    status         Query the current system status

But when I tried to type some commands to see whether it works, I got the "surface: command not found" information. I'm not very clear about how to set it up. Would you please help me with it? Thanks~

This is because the program cannot be found via the PATH variable, e.g. because it has not been installed yet.
When building from source you've got multiple options. If you're on debian, you can run cargo deb (provided you have the cargo-deb extension installed which can be done via cargo install cargo-deb) which will give you a .deb package in target/debian, which you can then install with dpkg. If you're on Arch, you can use the PKGBUILD from this repo.
On any distribution, running cargo install can install the program under your local user, i.e. to ~/.cargo/bin if not specified otherwise. In this case you have to add that path to your PATH variable. The way to do that depends on the shell you use.
For testing/experimenting you can also run the executable directly from the project directory running cargo run -- <args>. Note that the -- is required to separate arguments to cargo from arguments to the surface executable and the --release is optional. This seems to be what you've done already. This does nothing more than making sure the executable is up to date and then run it. The executable itself can be found under target/<target>/surface, where <target> is either debug or release.
If you want to create a new package, you can also have a look at pkg/arch/PKGBUILD which contains all the required commands to build and install the program and completion-files for zsh/bash/fish in the respective functions (note that the paths for the completion files may be different for your distribution).

Thank you very much!
I forgot to tell you that my linux distribution is ubuntu, but I followed your instructions on any distribution and run cargo install --path [path to linux-surface-control] and successfully installed it! It's really cool!
And these two websites told me about what the argument '--path' mean:
https://www.systutorials.com/docs/linux/man/1-cargo-install/
https://stackoverflow.com/questions/25233487/what-is-the-intended-planned-way-of-configuring-installing-software-that-uses-ru
Then I add PATH variable to my shell, follow the answers here:
https://stackoverflow.com/questions/11530090/adding-a-new-entry-to-the-path-variable-in-zsh
Actually I'm pretty new to computer science field, and I've spent some time to figure out what cargo is. Is it something for rusty just like the cmake to c++? And finally, does create a new package means to develop a sub command based on this project?
Thanks~

Since you're on Ubuntu I'd actually recommend installing the .deb file (unless you have some reason not to). I should probably update the readme and mention Ubuntu there.

I've spent some time to figure out what cargo is. Is it something for rusty just like the cmake to c++?

You could compare it that way, yes. Although I'd say there's a bit of a difference where cargo is the standard tool provided by the rust dev team itself, whereas cmake is one of many build tools provided by others. It's more like pip for python.

does create a new package means to develop a sub command based on this project?

No, what I meant by that was that, if you were using e.g. fedora (or any other distribution I don't provide a ready-to-install package for), you could create a package for it which you can then install via the package-manager of that distribution.

Yes, I actually tried .deb releases at first, but maybe there's some problem with my Ubuntu. Every time I double click a .deb file to install it, the "Ubuntu software" (something like app store) will open and the Install button won't work, and thus I have to download the source file and try to build and install it.
And thank you for your explanations, I can now fully understand your answers.
P.S. I wonder if I could ask something not very related to this issue... That is, how to systematically understand what the build tools & the operating system are doing, and finally can do some interesting things like writing these applications just like you are doing? I find myself learning all these things in fragmentation and usually don't understand the relations between them, e.g. answers on stackoverflow.com.
I know this might be a really really large topic... I'll be really grateful if you'd like to provide a general path.

Can you try and install it via sudo dpkg -i <path-to-.deb> if this doesn't work it should give a bit more information why.
I can try, the best place to discuss that would probably be gitter.

Yes, the output shows that some dependency problems occurred.
dpkg: dependency problems prevent configuration of surface:
 surface depends on libc6 (>= 2.28); however:
  Version of libc6:amd64 on system is 2.23-0ubuntu10.
 surface depends on libgcc1 (>= 1:8.3.0); however:
  Version of libgcc1:amd64 on system is 1:6.0.1-0ubuntu1.

dpkg: error processing package surface (--install):
 dependency problems - leaving unconfigured

I then tried sudo apt-get install libc6, and it prompts:
You might want to run 'apt-get -f install' to correct these:
The following packages have unmet dependencies:
 libc6-dbg : Depends: libc6 (= 2.23-0ubuntu10) but 2.23-0ubuntu11 is to be installed
 libc6-dev : Depends: libc6 (= 2.23-0ubuntu10) but 2.23-0ubuntu11 is to be installed
 libc6-i386 : Depends: libc6 (= 2.23-0ubuntu10) but 2.23-0ubuntu11 is to be installed
 surface : Depends: libc6 (>= 2.28) but 2.23-0ubuntu11 is to be installed
           Depends: libgcc1 (>= 1:8.3.0) but 1:6.0.1-0ubuntu1 is to be installed

I have no idea about whether it is safe to run apt-get -f install , will it affect other applications which depend on old version packages?

Ah, I see. I don't think -f is going to work here, in any case I wouldn't recommend it. The dependencies on libc6 and libgcc1 are added implicitly by cargo-deb, and I believe are determined by the current versions installed on my build container (debian-buster). These libraries are pretty much guaranteed to be backward-compatible, so I'll probably try to set up an older debian version, which should fix this.

Ah, because my email didn't show the whole comments, I've just read the Edit: part just now. I'll try it tomorrow. Thanks!

@Hustwireless Okay, I've rebuilt the package on a Debian 8 container and replaced it on the latest release. Requirements are now libgcc1 >= 1:4.9.2 and libc6 >= 2.19, so it should work on your system now, if you want to give it a try.

Yes, I tried cargo clean and it works in zsh. And this showed that if I didn't learn things systematically, I might end up guessing it and would not know the true reasons that lay behind.
Besides, would it be safe to dpkg the new release without remove the previous one? Will the command deal with it automatically?

I tried, it works automatically. And the new release works very well. It's very nice of you to be so patient, thank you very much!

Guessing is usually a good start, the rest comes with experience and curiosity, so always keep asking questions!

would it be safe to dpkg the new release without remove the previous one?

Yes (as you've figured out), the standard package management tools take the usual cases (e.g. package-upgrade, re-install) into account.
Awesome that it's working now!


Docker steps

I was trying to build the apollo on Mac and I already have Docker installed.
Do I still need to run install_docker.sh script.
I tried without the install script and hitting error on unable to add user. Any tips to circumvent this issue?

No need to run install_docker.sh. Mac support is tracked at #47

You should be able to use docker/scripts/dev_start.sh and docker/scripts/dev_into.sh as a temp workaround. release_start.sh doesn't quite work with MAC for some reason. @wanglei828 for SA.

I had succeed buliding apollo in MBP(2015,OS: 10.2.5),but I can't visit localhost:8887, it says"ERR_CONNECTION_REFUSED".the fellowing is my step:
1. I had install docker deamon in my MBP, so I jump "bash docker/scripts/install_docker.sh"  step, when  I execute "bash docker/scripts/dev_start.sh",there are some error:
-  find: /dev/fd/scripts: No such file or directory
- find: /dev/fd/scripts: No such file or directory
- \e[33m[WARNING] Failed to find device with pattern "ram*" ...
- find: /dev/fd/scripts: No such file or directory
- find: /dev/fd/scripts: No such file or directory
- \e[33m[WARNING] Failed to find device with pattern "loop*" ...
- docker: Error response from daemon: Bad response from Docker engine.
- See 'docker run --help'.
- Error response from daemon: Bad response from Docker engine

 I changed docker source and try serval times,"bash docker/scripts/dev_start.sh" executes succeed with following message:

- 505679240dc6: Pull complete
- Digest: sha256:09aeb7850578a877fdb855bfe7319c857289faa1f68df3eede1a548a09783d09
- Status: Downloaded newer image for apolloauto/apollo:dev-20170707_1129
- Error response from daemon: No such container: apollo_dev
- Error response from daemon: No such container: apollo_dev

I try to pull the image by hand, it shows message:-docker pull apolloauto/apollo:dev-20170707_1129

- dev-20170707_1129: Pulling from apolloauto/apollo
- Digest: sha256:09aeb7850578a877fdb855bfe7319c857289faa1f68df3eede1a548a09783d09
- Status: Image is up to date for apolloauto/apollo:dev-20170707_1129

then I fellow the step,execute "./dev_into.sh ",the terminal show nothing.
then I execute "bash apollo.sh build",It execute success:

- WARNING: Running Bazel server needs to be killed, because the startup options are different.
- INFO: Found 190 targets...
- Slow read: a 194-byte read from /apollo/modules/perception/conf/adapter.conf took 7200ms.
- INFO: Elapsed time: 713.802s, Critical Path: 206.74s
- ============================
- [ OK ] Build passed!
- [INFO] Took 1816.160 seconds
- ============================

then I exectue "bash scripts/hmi.sh",the terminal show message:
-    Start roscore...
- HMI ros node service running at localhost:8887
- HMI running at http://localhost:8887

But I still can't visit "http://localhost:8887", Can you give me some advice?

Regards.

Yes. I met the same issue "http://localhost:8887 can't be visited" after build it successfully.
I followed the instructions in the page step by step.
Any suggestion for that?

@zx08uestc @yipinp can you check if there is any error in the log in data/log/ ?

In data/log/ . I found the files below:
hmi.out
hmi_ros_node_service.out
ros_node_service.INFO
ros_node_service.in_dev_docker.zhangxin.log.INFO.20170709-083132.8429
roscore.out

the message in each files is:
zhangxindeMacBook-Pro:log zhangxin$ cat hmi.out
Traceback (most recent call last):
  File "modules/hmi/web/hmi_main.py", line 20, in <module>
    import gflags
ImportError: No module named gflags

zhangxindeMacBook-Pro:log zhangxin$ cat hmi_ros_node_service.out
nohup: /Users/zhangxin/apollo/apollo/modules/hmi/ros_node/ros_node_service: No such file or directory

zhangxindeMacBook-Pro:log zhangxin$ cat ros_node_service.INFO
Log file created at: 2017/07/09 08:31:32
Running on machine: in_dev_docker
Log line format: [IWEF]mmdd hh:mm:ss.uuuuuu threadid file:line] msg
I0709 08:31:32.895217  8446 ros_node_service.cc:65] [INFO] Server listening on <IP_ADDRESS>:8897

zhangxindeMacBook-Pro:log zhangxin$ cat ros_node_service.in_dev_docker.zhangxin.log.INFO.20170709-083132.8429
Log file created at: 2017/07/09 08:31:32
Running on machine: in_dev_docker
Log line format: [IWEF]mmdd hh:mm:ss.uuuuuu threadid file:line] msg
I0709 08:31:32.895217  8446 ros_node_service.cc:65] [INFO] Server listening on <IP_ADDRESS>:8897

zhangxindeMacBook-Pro:log zhangxin$ cat roscore.out
nohup: roscore: No such file or directory

This day morning, I execute "bash scripts/hmi.sh" again, it needs super permission, when I input the password: it shows the below message:
zhangxindeMacBook-Pro:apollo zhangxin$ bash scripts/hmi.sh
/Users/zhangxin/apollo/apollo/scripts/apollo_base.sh: line 22:<PHONE_NUMBER>N: value too great for base (error token is "1499648011N")
/Users/zhangxin/apollo/apollo/scripts/apollo_base.sh: line 22:<PHONE_NUMBER>N: value too great for base (error token is "1499648011N")
Password:
mknod: illegal option -- -
usage: mknod [-F format] name [b | c] major minor
       mknod [-F format] name [b | c] major unit subunit
       mknod name [b | c] number
       mknod name w
Start roscore...
HMI ros node service running at localhost:8887
HMI running at http://localhost:8887

@startcode  @ycool

@zx08uestc
This problem: ImportError: No module named gflags can be addressed by installing the python-gflags package
sudo pip install python-gflags
You can try run this command bash scripts/hmi.sh inside docker. The bash script does not work well on Mac.

I have some problems in building the apollo. After I bash apollo.sh build,  it always shows that Running Bazel server needs to be killed, because the startup options are different. And some errors following. In some forum, some people said that please make sure you have nuked your local cache and it no longer refers to boringssl-master-with-bazel, a version that will cause the issue. But I find that it is very difficult to nuke local cache. Does anyone knows how to nuke cache?

Sorry that the question is hung on for a long time. I will close it after this reply. Please open another issue if it's not solved.
Generally you could try the following commands to refresh bazel:
bazel clean
rm -fr ~/.cache/bazel


Introduce configuration-based switch to enable/disable identifier quoting [DATAJDBC-548]

Mark Paluch opened DATAJDBC-548 and commented
Based on the feedback from Dave Cramer, who runs the pgjdbc driver, we should consider a possibility to disable identifier quoting

Reference URL: https://twitter.com/dave_cramer/status/1262396128156422150
Issue Links:

DATAJDBC-553 Force quote default makes using multiple databases very hard
("is duplicated by")

1 votes, 2 watchers

What is a solution 3 years later? I saw blog entry showing mappingContext.setForceQuote(false) - but shouldn't similar config be applied on column level to quote it when it is needed and not quote by default? SpringBoot with Postgres + HSQLDB is popular mix and it would be nice it is supported out of the box by Spring Data JDBC.

I have the opposite issue, where configuration to set forceQuote to true would also help: A pre-existing MariaDB database with a column group that I want to use with R2DBC, but fails on inserts, because GROUP is "special" syntax and causes a grammar error.
In my mind to solve #604, the dialect should be able to specificy a set of reserved identifiers that have to be quoted to prevent malformed sql being generated.
The forceQuote property is actually a bit of a misnomer -- what it does is set SqlIdentifier#quoted which in turn acts more like enableIdentifierProcessing when SqlIdentifier#toSql is called. Which I'd argue is a bug.
The dialect, specifically it's IdentifierProcessing should always get a chance to influence quoting and not depend on generic settings in the relational (meta-)model (i.e. BasicRelationalPersistentProperty#forceQuote)
What might solve this is:

Mark setForceQuote and getForceQuote as deprecated because it's easy to misuse because it's not "forcing quoting" but disabling identifier processing.
Add setPreferQuote and getPreferQuote instead, use those, rename the field
Add a new method to IdentifierProcessingdefault boolean shouldQuote(String identifier, boolean perferQuoted) {
    return preferQuoted;
}

Add a new IdentifierProcessing subclass that considers a set of reserved words (or Predicate) that will always return true from shouldQuote and preferQuoted otherwise.
Replace the condition of the ternary operator in DefaultSqlIdentifier#toSql and DerivedSqlIdentifier#toSQL with a call to shouldQuote giving the string and quoted.

This way, the current observable behavior doesn't change and dialects can be changed over time to specify their reserved syntax.

@kodstark While I strongly recommend against using different databases in test and production (use Testcontainers instead) Spring Data JDBC supports running against the two databases just fine, as long as table and column names are the same, which they aren't if you use lower case names in postgres and upper case names in HsqlDB.

@Xiphoseer

The forceQuote property is actually a bit of a misnomer -- what it does is set SqlIdentifier#quoted which in turn acts more like enableIdentifierProcessing when SqlIdentifier#toSql is called. Which I'd argue is a bug.

This is very intentional behaviour. This guarantees that we use the default conversion used by the database in use (upper case for most). Without that users of certain databases would have to quote each and every identifier in their database by default, which most people most certainly wouldn't like.
We decided explicitly against any handling of quoting based on keywords, because this is brittle and expensive in terms of maintenance, since we'd would have to monitor all supported databases for changes in keywords, and quoting identifiers resolves that issue.


Get rid of separate subjects definition

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

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

Files with Coverage Reduction
New Missed Lines
%

isso.go
1
95.52%

Totals

Change from base Build<PHONE_NUMBER>:
0.3%

Covered Lines:
402

Relevant Lines:
460

💛  - Coveralls


New version: tibbo.ioninja version 5.5.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

/AzurePipelines run

1/23/2024 9:35:14 PM

****************************************************************************************
Installation Verification
****************************************************************************************
Status: Started
==============================================================================
Task         : Installation Validation
Description  : Invoke an Azure Function
Version      : 1.220.0
Author       : Microsoft Corporation
Help URL     : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/azure-function
==============================================================================
2024-01-23T21:35:15.8047535Z POST ***?code=***&
Request body: {
  "operationId": "WinGetSvc-Validation-359-135301-20240123-1",
  "BuildId": "534695",
  "PlanUrl": "https://dev.azure.com/ms/",
  "HubName": "build",
  "ProjectId": "ed6a5dfa-6e7f-413b-842c-8305dd9e89e6",
  "PlanId": "822dbe47-b8a3-4851-853e-53705abb3de3",
  "JobId": "68997d86-8ece-5ae8-3ac5-ccdbfa458731",
  "TimelineId": "822dbe47-b8a3-4851-853e-53705abb3de3",
  "TaskInstanceId": "b79910ce-dca5-5e27-ad7f-bd285eecb5a2",
  "AuthToken": ***
}
				Response Code: OK
				Response:

1/23/2024 11:59:13 PM

****************************************************************************************
Installation Verification
****************************************************************************************

-----------------------------------------------------
Result: Failed
-----------------------------------------------------
Status: Completed

Hmm. How do I fix this error? The only information I get from the log is Result: Failed.
Tried installing the manifest locally with winget install -m 5.5.0 and it works.
Where do I go from here? Thanks!

Hi @vovkos,
This error indicates an issue with our pipelines - that the Validation process did not complete, but got far enough that I can perform a manual validation, which I'll perform soon. It doesn't give any feedback about the package.
The domain host might have an IP block preventing download. But that would usually show up under a different error. Timeout usually happens around the 3 hour point, so this could be a timeout, but seems to have timed out early.


[WIP] Experimental color log

Make sure you have checked all steps below.
Jira

[ ] My PR addresses the following Airflow Jira issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-XXX
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a Jira issue.
In case you are proposing a fundamental code change, you need to create an Airflow Improvement Proposal (AIP).
In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

Description

[ ] Here are some details about my PR, including screenshots of any UI changes:

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference Jira issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters (not including Jira issue reference)
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

All the public functions and the classes in the PR contain docstrings that explain what it does
If you implement backwards incompatible changes, please leave a note in the Updating.md so we can assign it to a appropriate release

Code Quality

[ ] Passes flake8

I am still thinking about pretty printing argument if it is a dictionary. The log will be easier to read but also longer...

I think it's better to keep one-line per log entry. There are some really nice tools in IDEs to look at search  logs - for example ideolog: https://plugins.jetbrains.com/plugin/9746-ideolog. Also day before yesterday release of IntelliJ introduced capability of handling log files of arbitrary size easily - a lot of those tools rely on one log/lin.
"
IntelliJ IDEA works with large files smoothly
You can now easily open files larger than 25MB in IntelliJ IDEA and search through them without tedious waiting. This is a very nice little improvement for those of you who work with the log files."

Regarding the log file, I think the colors should be only for console handler, do you agree?

By default yes. I think you should be able to choose (but I guess you can using airflow configuration)

@mik-laj any thoughts before I open PR to apache? I suppose there will be further discussion 😳

Yes. I have alternative simpler implementation

Today I will do a review.


Cover ListExtensions with unit tests

I've submitted a pr for BinarySearch.
I'm almost done with ToDataTable(), but it looks like this won't work correctly on lists of C# built-in types like string, int, etc.  String throws an exception, int gives a datable with the correct # of rows, but no values in those rows.
So I'm wondering if ToDataTable() should have a check to make sure we aren't trying to make a datable out of a list of built-in system types?
There's a few ways to do it:

we could do myName.GetType().Namespace and check if it has system in it.
we could also do something like this: http://stackoverflow.com/a/15578098


Ensure XML output is still operational

Considering the use case such that the output of this command be piped into a tool like XMLStarlet (http://xmlstar.sourceforge.net/), we must ensure the Razor template work hasn't impeded the ability to output XML (perhaps by default, or -X switch)?  I don't think it was working when I last checked.
XMLStarlet is awesome BTW, inline CLI XSLT transforms and the like :-)

Ok to close this one off now? 😆


[chainname] when developing with custom chain names, if changed, can cause error

For developers that are using localnets when creating new chains, e.g. https://docs.cosmoskit.com/provider/chain-provider#adding-localnet-and-testnets upon changing chain name and refreshing, there is an unrecoverable error.
Steps to reproduce:

run the project, by doing yarn dev
change the chainname
refresh
see that the app will error out

example: https://github.com/pyramation/rollkit-frontend
we add a chain info and asset list:
https://github.com/pyramation/rollkit-frontend/blob/main/config/chain.ts
https://github.com/pyramation/rollkit-frontend/blob/main/config/assetlist.ts

The error will occur because the app is written in a way that it will remember the last selected chainName and store it in localStorage, so when you change the chainName and refresh, it will still use the old one to connect and couldn't find the chain info, therefore cause an error

To get around this, I think there're two solutions:

go to file components/wallet.tsx, disable the line below, refresh the page and reselect the previous chain from the dropdown:

  useEffect(() => {
    // Comment this out:
    // setChainName?.(window.localStorage.getItem('selected-chain') || 'osmosis');
  }, [setChainName]);

after changing the chainName in the file, also go to localStorage and update the chainName there too:

is there a way to check if it's in the chains/assets that were loaded from chain-registry or JSON of chain-registry/types?

Yes, actually there is a way to check that, we can replace the above code with this:
  useEffect(() => {
    const selectedChain = window.localStorage.getItem('selected-chain');
    const isChainValid = chainRecords.some((chain) => chain.name === selectedChain);

    setChainName?.(selectedChain && isChainValid ? selectedChain : defaultChainName);
  }, [chainRecords, setChainName]);


MediatorManager - MM

Description
The MediatorManager decides about which MediatorInstanceManager to be addressed for provisioning of a mediator process for an individual device. It is regarding device type and e.g. load balancing criteria. Future versions might support re-allocation of mediator processes based on planned operational activities like replacement of a mediator virtual machine or unplanned events like failure of a mediator virtual machine.
Dependencies

[ ] #15

I am volunteering for ApplicationOwner.


Redesign color schemes page

Summary of the Pull Request
This PR performs a large overall polish of the color schemes page:

Ensures keyboard navigation is holistically improved (i.e. fully accessible, no lost focus, etc...)
Adds tooltips and automation properties to all controls
Redesigns the page according to @mdtauk's approved design (link). Note, there are some minor modifications to the design that were approved by @cinnamon-msft.
Automatically reflow's the color buttons when they do not fit in horizontal mode

References
#8997 - Based on the work from @Chips1234
#6800 - Settings UI Epic
Closes #8765 - Polish Color Schemes page
Closes #8899 - Automation Properties
Closes #8768 - Keyboard Navigation
Detailed Description of the Pull Request / Additional comments

Redesign

Two data templates were introduced to make color representation consistent and straightforward
ContentControl is used to hold a reference to the ColorTableEntry or Color and represent it properly using the aforementioned data templates.
The design is mainly a StackPanel holding two grids: color table & functional colors.
The color table is populated via code. After much thought, this seems to be the easiest way to correctly bind 16 controls that are very similar.
The functional colors are populated via XAML manually.
We need a grid to separate the text and the buttons. This allows for scenarios like "selection background is the longest string" to force the buttons and text to be aligned.

Reflow

The VisualStateManager generally defines states, but we would have to navigate between states manually (generally through a size changed event handler).
I saved us a bit of code and complexity by just hooking up a SizeChanged handler that checks if the color tables fit when they are horizontal. If they don't fit, we switch the orientation to vertical.

Keyboard Navigation

(a lesson from SettingContainer) ContentControl can be focused as opposed to the content inside of it. We don't want that, so we set IsTabStop to false on it. That basically fixes all of our keyboard navigation issues in this new design.

Automation Properties and ToolTips

As in my previous PRs, I can't seem to figure out how to bind to a control's automation property to its own tooltip. So I had to do this in the code and add a few x:Name around.
The accessibility isn't perfect, but it's a lot better than we have now and your context is clear to the user. One particular issue is that we can't seem to access the inner UIElements of a DataTemplate for ContentControl::ContentTemplate. However, we can access the ContentControl itself. Setting the ContentControl's tooltips results in expected behavior for sighted users. However, setting the automation properties there (as opposed to the content itself), results in the screen reader interpreting the name as a "grouping" rather than the content itself. For example, the "black" button is read as "black grouping button". This isn't ideal, but again, it's better than nothing and it does establish a context for the user.

Validation Steps Performed

Manually tested...

tab navigation
accessibility insights
NVDA
changing color schemes updates color table

Demos

Note, there are some minor modifications to the design that were approved by @cinnamon-msft.

just for awareness, can you explain what they were and where they were approved?

@carlos-zamora
I am a bit confused on this PR because #8997 isn't ready to merge.

There are some ideas not implemented (column title, improved tooltip).
You have copied a bug into your PR that isn't fixed completely in #8997.

Tweaks to @mdtauk's design include moving the color buttons closer to the labels on the left and also making the "Color table" and "Functional colors" text subtitles. Also, we moved the functional colors closer to the color table to keep it more compact.

@carlos-zamora
I am a bit confused on this PR because #8997 isn't ready to merge.

There are some ideas not implemented (column title, improved tooltip).
You have copied a bug into your PR that isn't fixed completely in #8997.
(See my reviews on the other PR.)

Sorry for the confusion. I gave #8997 a deadline of 2/15 here because it's blocking closing a few bugs for the Settings UI. As we're preparing for release, I'd like to make sure the keyboard navigation and accessibility story are complete, and the redesign kinda encompasses that.
Regarding some other feedback from #8997...
From this comment:

I think you should add a small header above the two columns to indicate that the first color column are the non-bright version and the second color column are the bright version of the color.

Personally, I think adding the extra small headers would make it feel more cluttered. The format of the color table is already standard across terminal color schemes. That said, to make it more inclusive, maybe a link to the docs or a small description would be nice? @cinnamon-msft Thoughts?
From this comment:

I have looked at the tooltip text and want to bring up one point: As example the tooltip for the red non-bright color element says "red". But isn't this confusing if I choose an other color because I create a special color sheme? (Example: color=purple, tooltip say "red")
This could be a special problem for people with eye problems like people with a red-green weakness.

First off, I'm trying to minimize localization changes so if we go with this, I'll want it to be a follow up. Aside from that, this really is the same kind of issue as your last comment. "Red" actually being some hex code that doesn't look red at all is pretty standard across terminals. In fact, when you use something like PowerShell and get the "Red" color, you just get the color table's "red", not the actual red color. Again, in an effort to make it more inclusive, I'd say adding a link to the docs or a small description might work. @cinnamon-msft Thoughts?
Alternatively, we could extract the hex value from the color picker and display that as the tooltip. But I don't see much value to it, and I think this would be better as a follow-up if we decide to go with it.

@carlos-zamora
Have you looked onnmy code review on #8997? Because you have the same bug on filling the colors in _CurrentNonBrightColorTabel and _CurrentBrightColorTabel (line 399 - 408 in ColorShemes.cpp).

From this comment:

I think you should add a small header above the two columns to indicate that the first color column are the non-bright version and the second color column are the bright version of the color.

Personally, I think adding the extra small headers would make it feel more cluttered. The format of the color table is already standard across terminal color schemes. That said, to make it more inclusive, maybe a link to the docs or a small description would be nice? @cinnamon-msft Thoughts?

I know we want to eventually add docs links to the right side of all of the pages, so maybe we hold off until then? I agree that adding headers over each color may be too cluttered.

From this comment:

I have looked at the tooltip text and want to bring up one point: As example the tooltip for the red non-bright color element says "red". But isn't this confusing if I choose an other color because I create a special color sheme? (Example: color=purple, tooltip say "red")
This could be a special problem for people with eye problems like people with a red-green weakness.

First off, I'm trying to minimize localization changes so if we go with this, I'll want it to be a follow up. Aside from that, this really is the same kind of issue as your last comment. "Red" actually being some hex code that doesn't look red at all is pretty standard across terminals. In fact, when you use something like PowerShell and get the "Red" color, you just get the color table's "red", not the actual red color. Again, in an effort to make it more inclusive, I'd say adding a link to the docs or a small description might work. @cinnamon-msft Thoughts?

Same as my comment above. The colors in the tooltips align directly with how shells interpret their colors, for example Ubuntu uses "blue" and "green" as its colors in the prompt. This means it'll display whatever is set to "blue" and "green" even though they may not actually be blue and green, which is totally fine.

Same as my comment above. The colors in the tooltips align directly with how shells interpret their colors, for example Ubuntu uses "blue" and "green" as its colors in the prompt. This means it'll display whatever is set to "blue" and "green" even though they may not actually be blue and green, which is totally fine.

Sorta yes, sorta no. Ubuntu uses "four" and "two" as the colors for its prompt. The more correct expression for this is, like... "ANSI Color 0 (Usually Black)" "ANSI Color 7 (Usually White)"

But we accept as a shortcut the "usually" part, and boil it down to Red. Bright red. Black. "Bright black", whatever that is.

Same as my comment above. The colors in the tooltips align directly with how shells interpret their colors, for example Ubuntu uses "blue" and "green" as its colors in the prompt. This means it'll display whatever is set to "blue" and "green" even though they may not actually be blue and green, which is totally fine.

Sorta yes, sorta no. Ubuntu uses "four" and "two" as the colors for its prompt. The more correct expression for this is, like... "ANSI Color 0 (Usually Black)" "ANSI Color 7 (Usually White)"

We could use "Show  as ..." so for light red the tooltipp says "Show light red as ...". This makes clear what the tooltip means.
But I am okay when you let it as is. If the users are confused they will fill issues and the you can think on a solution again.


JSDoc grammar highlights @return {@link ...} as type

With the following code, TS claims that {@link TreeItem} is incorrect
export interface TreeDataProvider<T> {
/**
 * @return {@link TreeItem} representation of the element
*/
 getTreeItem(element: T): TreeItem;
}

adding a character between return and {@link TreeItem} resolves it.

I believe that the root cause is that we are highlighting {@link ...} as if was a type. When looking for types, we probably need a look ahead for the { to make sure it is not followed by @


OpenAI and AsyncOpenAI accept proxy now Fixes #1529

Added a new parameter called proxies to OpenAI and AsyncOpenAI

[ ] I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested
Add support to pass proxy to AsyncOpenAI and OpenAI
Additional context & links
Fixes #1529

Thanks for the PR, passing proxies is already supported - https://github.com/openai/openai-python/issues/1529#issuecomment-2212506242


problem

i made a bot with this repo and certainly the bot doesnot moves or play faster it is very slow can you tell me how to fix this and make bot faster

2021-06-22T09:09:37.432463+00:00 heroku[worker.1]: Error R14 (Memory quota exceeded)
2021-06-22T09:09:57.903602+00:00 heroku[worker.1]: Process running mem=533M(104.1%)
2021-06-22T09:09:57.934687+00:00 heroku[worker.1]: Error R14 (Memory quota exceeded)
2021-06-22T09:10:19.690224+00:00 heroku[worker.1]: Process running mem=533M(104.1%)
2021-06-22T09:10:19.692565+00:00 heroku[worker.1]: Error R14 (Memory quota exceeded)
2021-06-22T09:10:25.000000+00:00 app[api]: Build started by user<EMAIL_ADDRESS>2021-06-22T09:10:39.391755+00:00 heroku[worker.1]: Process running mem=533M(104.1%)
2021-06-22T09:10:39.415960+00:00 heroku[worker.1]: Error R14 (Memory quota exceeded)
2021-06-22T09:11:01.964424+00:00 heroku[worker.1]: Process running mem=533M(104.1%)
2021-06-22T09:11:02.024604+00:00 heroku[worker.1]: Error R14 (Memory quota exceeded)
2021-06-22T09:11:06.195524+00:00 heroku[worker.1]: Restarting
2021-06-22T09:11:06.199517+00:00 heroku[worker.1]: State changed from up to starting
2021-06-22T09:11:05.926596+00:00 app[api]: Release v9 created by user<EMAIL_ADDRESS>2021-06-22T09:11:05.926596+00:00 app[api]: Deploy 06c2c36b by user<EMAIL_ADDRESS>2021-06-22T09:11:07.547335+00:00 heroku[worker.1]: Stopping all processes with SIGTERM
2021-06-22T09:11:08.410395+00:00 heroku[worker.1]: Process exited with status 143

You’ve reached the maximum memory quota of heroku. Reducing hash should fix this.


Display reactions for issue's comments

before
after

Thanks for the PR, really nice feature. Anyhow, there's a simpler way of solving this issue.
The comments are already fetched using the Timeline API, and simply adding the header "Accept: application/vnd.github.squirrel-girl-preview" to that requests is enough to get the reactions. So you don't need to fetch all comments again, and it'll work fine with pagination (your approach wont work for issues with >100 comments).
Unfortunately, I tried this and it doesn't work due to a bug in the API (the body_html param, which we use to show the comment's text, is not returned), so we can't implement it just yet. I've reported it to GitHub and hope they can fix it soon so we can have this feature in the next release.
You can try to fix this PR and I will merge it once GitHub fixes the bug.

The comments are already fetched using the Timeline API, and simply adding the header "Accept: application/vnd.github.squirrel-girl-preview" to that requests is enough to get the reactions.

Unfortunately, timeline request accepts another header "Accept: application/vnd.github.mockingbird-preview". I've got an error when tried "Accept: application/vnd.github.squirrel-girl-preview". I tried to use reaction request, but I got too many information - list of reactions including user info wich needed to be agregate (List<Reaction> to ReactionSummary). Finally I choosed this way to get all reactions by one request.
Founded one trouble while developing. Timeline refreshes for three or more different issues when user clicks on only one issue to view details. For example click on issue 300 fires 299, 300, 301 issue details updating. IMHO that's a real reason of slow refreshing repository's issue info.

So you don't need to fetch all comments again, and it'll work fine with pagination (your approach wont work for issues with >100 comments).

So pagination service not used properly:(
Collection<TimelineEvent> timelineEvents = paginationService.getAll();
Yep, there is a perfomance problem with big amount of comments. GraphQL API v4 may partially resolve a problem, but don't know how to use it yet.

Unfortunately, timeline request accepts another header "Accept: application/vnd.github.mockingbird-preview". I've got an error when tried "Accept: application/vnd.github.squirrel-girl-preview" even with separated getTimeline1() method.

You can add various headers like this:
@Headers({
    "Accept: application/vnd.github.squirrel-girl-preview",
    "Accept: application/vnd.github.mockingbird-preview"
})

Actually, I've been doing some tests just now, and setting this headers works! 🎉
@Headers({
    "Accept: application/vnd.github.v3.full+json",
    "Accept: application/vnd.github.squirrel-girl-preview",
    "Accept: application/vnd.github.mockingbird-preview"
})

You can try using this to solve the issue.

Founded one trouble while developing. Timeline refreshes for three or more different issues when user clicks on only one issue to view details. For example click on issue 300 fires 299, 300, 301 issue details updating. IMHO that's a real reason of slow refreshing repository's issue info.

This is due to the issue view being a ViewPager (you can scroll left or right to change issues), so Android loads more issues for smooth scrolling.

Thanks a lot for your advice!

You're welcome.
Ping me when you're ready for merge and I review it again 😉

I'm ready to merge. Reactions for a cimmit's comment added. I'm not found separate wiev for a pull request's comment reaction. Issue #280 seems ready to be closed.

OK, I'll rewrite/refactor my changes into another branch/PR with 3 commits.

There is 4th change - refresh issue optimization. Should it be placed in separate commit/PR or not?

Refactored pull is here #307


Q# (python.vs code) new console >python file

from <...>
Red underline error prompt.
It's not appropriate Error

/needsMoreInfo

A small mistake.

Please file the issue against the quantum extension.


Installation. Not enough space, EXIT 95 reported when space seems to be available.

I am on unraid Version: 6.12.4 trying to install the docker for virtual-dsm.
Here is my config:

Pulling image: vdsm/virtual-dsm:latest
IMAGE ID [2005165774]: Pulling from vdsm/virtual-dsm.
IMAGE ID [a3c0f33fc620]: Already exists.
IMAGE ID [681cfb1b8911]: Pulling fs layer. Downloading 100% of 39 MB. Verifying Checksum. Download complete. Extracting. Pull complete.
IMAGE ID [506315990503]: Pulling fs layer. Downloading 100% of 11 KB. Verifying Checksum. Download complete. Extracting. Pull complete.
IMAGE ID [64fda018ef8f]: Pulling fs layer. Downloading 100% of 4 MB. Verifying Checksum. Download complete. Extracting. Pull complete.
IMAGE ID [f7dbad1400df]: Pulling fs layer. Downloading 100% of 10 KB. Verifying Checksum. Download complete. Extracting. Pull complete.
Status: Downloaded newer image for vdsm/virtual-dsm:latest

TOTAL DATA PULLED: 43 MB

Command execution
docker run
  -d
  --name='virtual-dsm'
  --net='bridge'
  -e TZ="Europe/Berlin"
  -e HOST_OS="Unraid"
  -e HOST_HOSTNAME="Tower"
  -e HOST_CONTAINERNAME="virtual-dsm"
  -e 'DISK_SIZE'='16G'
  -e 'RAM_SIZE'='512M'
  -e 'CPU_CORES'='1'
  -e 'DHCP'='N'
  -l net.unraid.docker.managed=dockerman
  -l net.unraid.docker.webui='http://[IP]:[PORT:5000]'
  -l net.unraid.docker.icon='https://raw.githubusercontent.com/MountainGod2/unraid-templates/main/virtual-dsm/logo.png'
  -p '5005:5000/tcp'
  -p '5006:5001/tcp'
  -v '/mnt/user/appdata/dsm/':'/storage':'rw'
  --device='/dev/kvm'
  --device='/dev/vhost-net'
  --stop-timeout 60
  --cap-add NET_ADMIN
  --device-cgroup-rule='c *:* rwm' 'vdsm/virtual-dsm:latest'
fcd77ca2ee2652d7807411fbb55f0461d8b18f7ee597ecc7e319f78eeda0754e

The command finished successfully!

The logs of the container report:
text  error  warn  system  array  login

❯ ERROR: Not enough free space for installation.
❯ Starting Virtual DSM for Docker v4.27...

** Press ANY KEY to close this window **

Unless it is checking the USB key, all drives have multiple hundreds of GB as free space.
I allocated the SSD for the virtual disk, also tried the array, also tried to remove the value altogether, always the same error.
I would not know how to continue or check here, please advise, thanks.

May I ask how many memory do you have in your system?

Thanks working well with new version 4.28.
I am using:
Intel® Core™ i3-9100 CPU @ 3.60GHz
32GB DDR4 ECC


Cleanup_tests

Codecov Report
Attention: Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.

Project coverage is 88.55%. Comparing base (9ac6422) to head (0a11557).

Files
Patch %
Lines

pkg/cli/argonap.go
80.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #31      +/-   ##
==========================================
+ Coverage   87.97%   88.55%   +0.57%
==========================================
  Files           8        8
  Lines         291      297       +6
==========================================
+ Hits          256      263       +7
+ Misses         35       34       -1

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


Add entry for Leaflet.EdgeBuffer plugin

Please rebase on top of the current gh-pages-master, or resubmit to gh-pages.


[ BUG ] Select element is hidden behind Dialog

Describe the bug
Rendering for web, the <Select> options are not visible when the component is inside a popup <Dialog>. The options are hidden behind the dialog (see screenshot).
It seems like a zIndex issue but I tried modifying the code from dialog.tsx to z-10 instead of z-50, still doesn't work. So if anyone has a quick workaround, it's very much appreciated.
To Reproduce
Change the starter-base/app/index.ts by the code below (Additional context) and run npm run web.
Expected behavior
The Select should be visible even if used inside a Dialog element.
Screenshots
The <Select> in the image is opened: looking closely, see the last options behind the black overlay (at the bottom of the dialog).

Platform (please complete the following information):

Type: Browser, web
OS: OSX
Browser: Chrome

Additional context
Reproduce the bug by changing starter-base/app/index.tsx code from the repo:
import * as React from 'react';
import { View } from 'react-native';
import { Button } from '~/components/ui/button';
import { Text } from '~/components/ui/text';
import { Dialog, DialogContent, DialogHeader, DialogTitle, DialogTrigger } from '~/components/ui/dialog';
import { Select, SelectContent, SelectGroup, SelectItem, SelectLabel, SelectTrigger, SelectValue } from '~/components/ui/select';

export default function Screen() {
  return (
    <View className='flex-1 justify-center items-center gap-5 p-6 bg-secondary/30'>
      <Dialog>
        <DialogTrigger asChild>
          <Button variant="secondary" size="sm" className="flex-row">
            <Text>See issue</Text>
          </Button>
        </DialogTrigger>
        <DialogContent className="sm:max-w-[625px]">
          <DialogHeader>
            <DialogTitle>My dialog</DialogTitle>
          </DialogHeader>
          <View className="aspect-video">
            <Select defaultValue={{ value: "apple", label: "Apple" }}>
              <SelectTrigger className="w-[250px]">
                <SelectValue
                  className="text-foreground text-sm native:text-lg"
                  placeholder="Select a fruit"
                />
              </SelectTrigger>
              <SelectContent className="w-[250px]">
                <SelectGroup>
                  <SelectLabel>Fruits</SelectLabel>
                  <SelectItem label="Apple" value="apple">
                    Apple
                  </SelectItem>
                  <SelectItem label="Banana" value="banana">
                    Banana
                  </SelectItem>
                  <SelectItem label="Blueberry" value="blueberry">
                    Blueberry
                  </SelectItem>
                  <SelectItem label="Grapes" value="grapes">
                    Grapes
                  </SelectItem>
                  <SelectItem label="Pineapple" value="pineapple">
                    Pineapple
                  </SelectItem>
                </SelectGroup>
              </SelectContent>
            </Select>
          </View>
        </DialogContent>
      </Dialog>
    </View>
  );
}

@DonBraulio The latest version of the dialog does not use z-index because of this reason. Please try to remove the z-index class and if that doesn't work, try with the latest version of the Dialog from rnr.
If both of those still fail, please provide a minimal reproduction repo

Yes, I had a similar issue with the Alert Dialog and just removing the z-index fixed it :)


ImportExportAnchorManager doesn't change the current state to DataRequest when start Downloading

ImportExportAnchorManager.cs doesn't change the current state to DataRequest when start Downloading the Anchor from the room.
roomManager.DownloadAnchor(currentRoom, currentRoom.GetAnchorName(0))
doesn't return true even the download process is alrealdy undergoing
This bug causes the IEAM keeps call startanchorprocess() in the update loop and potentially breaks the downloading and loading the WorldAnchor.

Probably belongs in the main MixedRealityToolkit repository.

@leo92613 I've updated the sharing service.  Please check to see if this is still an issue.


citrus-selenium module

Selenium webdriver testing module for citrus. With this module one can write easily selenium integration tests using the citrus framework.

Had a first look at your pull request - nice work here! Thanks a lot! It takes me some time though to review all the changes. Just wanted to inform you that I am working on it.
You can follow the progress on this new branch: https://github.com/christophd/citrus/tree/selenium

Thanks a lot. Actually that was the initial draft. We had improved it more and put some more actions such as upload files from classpath.
We will send them also later. We are using this in our testing environments, and developing it actively.  If you want, I can also deliver some documentation on how to use it. If you have problems, please don’t hesitate to contact me.
Thanks,
Tamer Erdogan
From: Christoph Deppisch<EMAIL_ADDRESS>Sent: Wednesday, March 30, 2016 11:47
To: christophd/citrus<EMAIL_ADDRESS>Cc: vdsrd<EMAIL_ADDRESS>Subject: Re: [christophd/citrus] citrus-selenium module (#68)
Had a first look at your pull request - nice work here! Thanks a lot! It takes me some time though to review all the changes. Just wanted to inform you that I am working on it.
You can follow the progress on this new branch: https://github.com/christophd/citrus/tree/selenium
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com/christophd/citrus/pull/68#issuecomment-203351166


DBZ-7695 add _id field to tombstone rewrite events, for mongodb delete events

https://issues.redhat.com/browse/DBZ-7695

@ismailsimsek Thanks for the PR. Could you please add a config option that will enable this behaviour so it is not used by default?

@ismailsimsek Thanks for the PR. Could you please add a config option that will enable this behaviour so it is not used by default?

@jpechane which config key would you suggest? added 4 option below
transforms=unwrap,...
transforms.unwrap.type=io.debezium.connector.mongodb.transforms.ExtractNewDocumentState
transforms.unwrap.drop.tombstones=false
transforms.unwrap.delete.handling.mode=drop
-- options ------------
transforms.unwrap.mongodb.delete.tombstone.handling.mode.rewrite-with-id
transforms.unwrap.mongodb.delete.tombstone.handling.rewrite-with-id
-- using `includes`
transforms.unwrap.mongodb.delete.tombstone.handling.mode.rewrite-icludes-id
transforms.unwrap.mongodb.delete.tombstone.handling.rewrite-icludes-id
------------------------

@jpechane if you have time.. its ready for a review.

@Naros  @jpechane its ready for review.

@ismailsimsek Applied, thanks a lot! Could you please open a separate PR with docs updated?


<EMAIL_ADDRESS>is broken

The same as https://github.com/unpkg/unpkg.com/issues/115
<EMAIL_ADDRESS>was working yesterday (see #115), but not today:

Cannot find module "dist/sweetalert2.all.js" in package<EMAIL_ADDRESS>
~Since v7.24.2 is the latest, https://unpkg.com/sweetalert2 is broken as well.~ Not anymore, v7.24.3 has been released.
Sorry, but this is ridiculous. I'm reporting this and moving my projects to jsdelivr.
You guys are currently unreliable :/ Hopefully, you can do better in the future.

Same bug, @limonte. Closing this as a duplicate. Please stop opening new issues for the same issue.


TA to TA communication results in segfault

from one TA to another TA results in segfault during TEE_InvokeTACommand
I know that the APIs for TEE_OpenSession, TEE_InvokeTACommand have been added since "TEE Internal Core API v1.2".
Just wanted to know if there any plan to support the TA to TA communication?

There was a bug related to TA2TA communication and TEE_TYPE_MEMREF_XXXX, but it did not end up to segfault. The fix has been committed into tee-engine-repository.
Would you be able to provide reproducing steps?
Unfortunately currently OpenTEE allows ta2ta communication when connection is established and therefore it is possible only within "TA_InvokeCommandEntryPoint"-function. It needs a bit investigating whether it is feasible to add other entrypoint-functions.
As a trivia then "4.9 Internal Client API"-chapter has been in since TEE Internal Core API v1.0 (I guess from the first spec).
A bug fix tee-engine
https://github.com/Open-TEE/tee-engine/commit/e8f42a4851ae74937f74a1310f8a7c8443adcb91
A basic smoke test for TA2TA communication
https://github.com/Open-TEE/TAs/commit/0c497fd24224ff3e5d13acaa27c7516bfe4f7e5b
https://github.com/Open-TEE/tests/commit/646b329675f110fdb10edb0d755817dc0eebbbe2

@TanelDettenborn  ta2ta_conn_test_app_ctrl.h file is not committed ? getting build error.

@TanelDettenborn
Thanks for the fix.
The usecase was that one TA was opening session and invoking commands in its own OpenSession call.
With the latest code, I see the following observation.
msg gets dropped with "tee_manager: ../../emulator/manager/logic_thread.c:invoke_cmd:921  Invalid sender or senders status". Since the TA's status is proc_initialized.
Please consider the following patch which fixes the issue.
From a84de0298d7f8653a3c95ab5e5509d533c389a14 Mon Sep 17 00:00:00 2001
From: Harish Jenny K N<EMAIL_ADDRESS>Date: Mon, 7 Feb 2022 13:55:04 +0530
Subject: [PATCH] logic_thread: TA to TA communication in OpenSession of a TA

Allowed TA invoke command from another TA during opensession
and closesession.

Signed-off-by: Harish Jenny K N<EMAIL_ADDRESS>Change-Id: I7f6695980a0d41cb9dbba635c8f0138fe5df5e0b
---
 manager/logic_thread.c | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/manager/logic_thread.c b/manager/logic_thread.c
index 0b47b24..dd63c7f 100644
--- a/manager/logic_thread.c
+++ b/manager/logic_thread.c
@@ -918,8 +918,11 @@ static void invoke_cmd(struct manager_msg *man_msg)

 	/* Function is only valid for proc FDs */
 	if (man_msg->proc->p_type == proc_t_session || man_msg->proc->status != proc_active) {
-		OT_LOG(LOG_ERR, "Invalid sender or senders status");
-		goto discard_msg;
+		/* Allow TA invoke command from another TA during opensession */
+		if (!(man_msg->proc->p_type == proc_t_TA && man_msg->proc->status == proc_initialized)) {
+			OT_LOG(LOG_ERR, "Invalid sender or senders status");
+			goto discard_msg;
+		}
 	}

 	/* REsponse to invoke to command can be received only from TA */
@@ -1535,8 +1538,11 @@ static void close_session(struct manager_msg *man_msg)

 	/* Function is only valid for proc FDs */
 	if (man_msg->proc->p_type == proc_t_session || man_msg->proc->status != proc_active) {
-		OT_LOG(LOG_ERR, "Invalid sender or senders status");
-		goto ignore_msg;
+		/* Allow TA closesession during proc_initialized */
+		if (!(man_msg->proc->p_type == proc_t_TA && man_msg->proc->status == proc_initialized)) {
+			OT_LOG(LOG_ERR, "Invalid sender or senders status");
+			goto ignore_msg;
+		}
 	}

 	session = get_sesLink_by_ID(man_msg->proc, close_msg->msg_hdr.sess_id);
--
2.17.1

@hkandiga
It has been a while and therefore I need to sit down and think through changes, because what I can recall rational behind ta2ta communication limitation was a reliability. As a retrospect the previous design choice has not been optimal. It might limit too much usability.
Above patch might be enough, but I will think through and try to get the patch in as soon as possible!