Removing unrequired steps in Module 4 Lab_04L01 ADDS

Module: 04
Lab/Demo: 04L01 ADDS
Task: 01
Step: 2-6.9-10 and 11
Description of issue
There are some redundant steps in the lab and also linking to the DC then connecting to VM az140-21-p1-0 is putting and extra layer of virtualization that is not required.
There will be a need to renumber some of the step because of deleting some items
Repro steps:
1.delete items 2-6
2.delete items 9-10
3.change item 11 from
On the az140-21-p1-0 blade, select Connect, in the drop-down menu, select RDP, on the az140-21-p1-0 | Connect blade, in the IP address dropdown list, select the Private IP address entry, select Download RDP File, and then select Open.
to
In the Azure portal, search for and select Virtual machines and, from the Virtual machines blade, select az140-dc-vm11.
On the az140-21-p1-0 blade, select Connect, in the drop-down menu, select Bastion, on the Bastion tab of the az140-21-p1-0 | Connect blade, select Use Bastion.

Hi Paul,
would you mind clarifying the following?
Your suggested change 3 consists of the following sentences:
"In the Azure portal, search for and select Virtual machines and, from the Virtual machines blade, select az140-dc-vm11.
On the az140-21-p1-0 blade, select Connect, in the drop-down menu, select Bastion, on the Bastion tab of the az140-21-p1-0 | Connect blade, select Use Bastion."
It's not clear to me how exactly selecting the az140-dc-vm11 blade brings you to the az140-21-p1-0 blade?
rgds
Marcin

My apologises, that's me cutting and paste, the first sentence should reference az140-21-p1-0, the whole point us to stop using the az140-dc-vm11 as is just added another layer of connection into the mix.
Regards and Happy Holidays
Paul

From: polichtm @.>
Sent: Monday, December 27, 2021 12:49:09 PM
To: MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual-Desktop @.>
Cc: Foulger, Paul @.>; Author @.>
Subject: Re: [MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual -Desktop] Removing unrequired steps in Module 4 Lab_04L01 ADDS (Issue #125)
-- This email was sent to you by someone outside of QA. Be careful of attachments and links. Report suspicious emails to @.*** --
Hi Paul,
would you mind clarifying the following?
Your suggested change 3 consists of the following sentences:
"In the Azure portal, search for and select Virtual machines and, from the Virtual machines blade, select az140-dc-vm11.
On the az140-21-p1-0 blade, select Connect, in the drop-down menu, select Bastion, on the Bastion tab of the az140-21-p1-0 | Connect blade, select Use Bastion."
It's not clear to me how exactly selecting the az140-dc-vm11 blade brings you to the az140-21-p1-0 blade?
rgds
Marcin
—
Reply to this email directly, view it on GitHubhttps://url4.mailanyone.net/v1/?m=1n1pZU-0004az-6I&i=57e1b682&c=t6wJI-aNdMTBGb7AeScrth5EpMF2563NCEMoBva_entaxSk3A_hj-L3sTp_EZBN7d_DVSJ-tTop6_0b4WUXvi54ADt2aBjJF_i20MIRx7Pn3YdfrxuMHzGnpda-boukdmeOKhSGFkLVtp53eeUSttuKAxMg_LzjtDKsRvanztj1-BEpK6EHyNBISh8crzMI7t9vMoD4Fuo_VWatDoQT1QZijImy5TbK4nfRIvmTapW-NNWlX32ExDaTW7qvKt92qr4sFj27LY20dJI74_CitYMNa2LVpxvpeEQ7z6P91LHp53p0fzt-gZeby8IRJblVDoF7Ho2qOzIrwt6ud4EscIj0MWas4R2tUIEdx6kPI3THi0vs8TcpGsZ-U3Aotp-nS, or unsubscribehttps://url4.mailanyone.net/v1/?m=1n1pZU-0004az-6I&i=57e1b682&c=Wz-VrYoXMiy-QM51EvpaKwX5WqrL2BZhnJ9KLc64irGQTeZeCmiC2MFABsT3qtGTAJdy1esOiU9do6qolxbTek6F3HbRFJvHVao_7e2OPE-09L8m3oM3tmoci7NzMCaX_7ebDA13W-rF8ViSqNY1LsUQtZ6-sCnnd0SEiWCMKbX5gJpXdut2wSrjmU5vw0w5ny6Q5IZFtCr4yBVv5pp4td2Ul2PmZ3Hr6VGkbEOv-N35haTg0mSLqvRpuH7MY-BP8YHgU5zs5AndMyHHsyvKh7PwT5z02LCLLw1LhQXYEvJRFe6PAhnWsORqA3doNUiS.
Triage notifications on the go with GitHub Mobile for iOShttps://url4.mailanyone.net/v1/?m=1n1pZU-0004az-6I&i=57e1b682&c=RPXAIMKf2OL68yoABUYB4yNuyymvwBsnKq74jen6_OHqJv-wW7q9BmdGqugE1w9_vmKgoWwgKTn3l5JJz2R4zOcSPMPW8R_V76BCV843FNBx_RM5aZ1ZmH5srYbTUXqtOWY-zgbJ5gLUz9_ZzdjsCEEncKvmTw_V_xN4UfI0olxRcTVbZnwUTuWTpPTnHCdpz-Qpo1EfIBSkcmZthdB2WNE0rXRlIQDK9ofq9wM4fYX3PFkxNh3lXxaTG343_005qhrE_vcqZMDgyfnzhCFvx-dkmgVWY3eo0WLwk_CDTO657osk5PLbYbNw09Z2SLFK or Androidhttps://url4.mailanyone.net/v1/?m=1n1pZU-0004az-6I&i=57e1b682&c=rQNwmSgc8vQfczUltSyQX5ZyvZgd-fWMIdr7fa48D4XzZDGi8d-ay74jSPuWVigRbNvjJB1rO8Jng67k_iI7Ky4m_Xs80JDzyV9hvDlkSt7McD7VXdjNQiJj2G9Z_4d2WXYthZjyudg3mCSqnPCUkZqQsmrAwd7e6U6QvuS_wI7zO9pi5syHFOO8AIKB72PFqKOOi8W5LarL7QKQdL7EU1qKtOrK3_UxvtRQ7BA7SLwcCcO_yS3ojIOINv1VcEL2N4dzD_aGfWnW4-KUbxQ4g_LtaRUGJl7mtBXsYngi__IOt17AUuZbp5r1ycivsGLR1SBGE_Tr35Kajpw9DrmgznCUarysZP0l6ko38i-YFekkQybcHxhWqRsVGE6MCJzibnuUVQs0awOiQs8MUlwZLg.
You are receiving this because you authored the thread.Message ID: @.***>

Hi Paul,
updated
rgds
Marcin

Hi,
Having just checked the update it will not work, ex 1 task 1 item 4 should read.
"On the az140-21-p1-0 blade, select Connect, in the drop-down menu, select Bastion, on the Bastion tab of the az140-21-p1-0 | Connect blade, select Use Bastion."
Connecting with the RDP as you still have in the instructions will fail from the lab pc as it does not have a connection. This was used when you first connected to the DC, by taking out the DC you now use the Bastion instead of the RDP.
Regards
Paul

From: polichtm @.>
Sent: Thursday, December 30, 2021 12:52:11 PM
To: MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual-Desktop @.>
Cc: Foulger, Paul @.>; Author @.>
Subject: Re: [MicrosoftLearning/AZ-140-Configuring-and-Operating-Microsoft-Azure-Virtual -Desktop] Removing unrequired steps in Module 4 Lab_04L01 ADDS (Issue #125)
-- This email was sent to you by someone outside of QA. Be careful of attachments and links. Report suspicious emails to @.*** --
Hi Paul,
updated
rgds
Marcin
—
Reply to this email directly, view it on GitHubhttps://url6.mailanyone.net/v1/?m=1n2uun-0003hK-45&i=57e1b682&c=YeVO25r3IZE-FwqdpUT2iNhc2hBxAddtUavzlnV-YYUdM2VCRw90JoTBBtTGpmgfwzSCLN-uvjWYkLcMuyAws2Y6KYbzR0hzHJRWVqerPExyqL1KFk-U0MaZx4p-ucho_3FCsOQ0wy3ZhvqeNlFMRjsnRC1G7Ywws7jHb1ghfTzmMB0j7SRIcHr8iuoFQSAAvWV5af6GRqCzgwa4MUFrf4X74e-87Ms3oNEtQOtvp30wYOZ5fiNzoZ1qr1RHZXm1NjTmU8qOsUwGKvDh1lkcAZ3jxl40A_omlJHomLbgBPzRQykA5WYojxIl2jK8skN--eaS5rnbEzcqfWJSqLBIZbtkJXtxDSvhWd7ClVek0YNs_dTpSLpgCdM83cqtJuse, or unsubscribehttps://url6.mailanyone.net/v1/?m=1n2uun-0003hK-45&i=57e1b682&c=knQE4S0xIT90nqRczsRkZs5h7CnH8IKjGYcpyW5wJZRNPwn0OMuUCWnMFIAOqu2ugh4zrVyeY533zJad-qLMGX3JlECt-xXnzejQqY9hLQxiMrKgMaNKQwn7tB21CR054wJ4SbG9XxFdeNuPAOnJZjAXTHgDQL1PyS9ll6xM_QOu2PWdqbZQqo4fCgxJd_OvZESSIioJlj0shBV7K3TapAG4GsUwsMIHQBgWj6hhO-7Lrpm7ti9qOSgfj9VGxgoS17ZCDS46dYgviBYJeNxOIEOu7XoZeFiq1NNO0lWC1FxR285eDEUQNPcceID2MEDT.
Triage notifications on the go with GitHub Mobile for iOShttps://url6.mailanyone.net/v1/?m=1n2uun-0003hK-45&i=57e1b682&c=TzKxEsOF0w00mN_MOrw2K2h_-F_DoU0nA4kzWBql1G71MO_T4nkzwy70N6MtlaHK3NYFYFaRmdK2J3CN7WOFvbWW4X57vQDtq3BSgsg9j3B33-bFjyABUiksrJnrXjQjWXVQqqFXVN0WXaMoXptwU_-32XPi1b7hbi9N9PLg29gpLRB411bxOpsehqH0tW3JUGe9PtPzK7tPLxqAjHsFAkQRXT-YBY_kKd5H0YJZKzzc412jUAS1mJ10I5y3K6Di_SrnKtIAJu6yz4RAbm7A9mRkLsYDDjCF2BLjfyGSngaqtmTswh5TUt_BNzaqRMTX or Androidhttps://url6.mailanyone.net/v1/?m=1n2uun-0003hK-45&i=57e1b682&c=vuVLkbULoGTHrUgqhILFhlzUSH7G1ED8xFDifZpGqAnv_hvYMtL3XcL_OWwhuseRAAXJ2tctqxwL_g4HvmwBiSufWau2Scti56itv3LtOT1DFscBLJQd3E8Kmy0lmKYx1XTJdHqNhHhBUbbs2N8e_N3ZqUJbIjl7nXcC5jxEnTCteF6iReeQ3lmL7F7aO7qaVnVspKSMu6S15G6OyBc4Lg9bfaMRxIOBeYpYoPJAXCNj5zxw-B9CxXIgOs83d9zugFPLimk-vmGr0U8yptjdXAWzFjpZrPAG3syXiVhI-vdFd_WjINFPWxqckpkXNAPyfGmhhhnHn51qgaGUgPu88xmC8466tIwBz0eYjqX5BHgf9-FbtfnUnVO9GLPZoatzOrpiFZBu1jjjSBSy9hn1hQ.
You are receiving this because you authored the thread.Message ID: @.***>

Hi Paul
updated
rgds
Marcin


Lated commit bugs

The latest commit looks for a bundle.js and css that don't exist in the repository.
We are including this project as part of a larger page and are getting some incompatibility errors with Select2.js and slimscroll with the version we downloaded last week.
When it is included on the page, the editor, toolbar and preview disappear after a moment unless the container is given an absolute size in pixels. It didn't seem to establish any size for itself, which then triggered the overflow hidden, but if I disable that property it has overlapping content positioned atop it. Putting it in an outer div and specifying the size helped, but we can't do a responsive layout with it.
Great project, we are looking forward to using it and chose it because of your activity in updating the project, and the activity you show in responding to issues. Thanks you very much and look forward to your thoughts.

Please run the following to build the project first:
yarn install
yarn build

I don't recommend you upgrading at the moment. You can just stick to the old version. Because I am doing massively refactoring which might break things. Please give me a month or so to deliver a stable version.

Should we run the commit from November then? I am really excited about the update, but wondering if it will create breaking changes that we would be better off waiting to implement till then? I do have a choice to wait if I need to, but if we can do it now and drop in the new code later, that would be great.

November's code should be very stable.  No new features was added since then. The recent change was about adopting webpack. So you can wait until I release 3.0.0 later this year.


Fail while parsing an object that has an array of objects.

Parser returns a valid json but wrongly parsed.
{
	"metadata":
	{
		"numObjects":8,
		"numLabels":3,
		"labels":[0,1,2]
	},
	"data":
	[
		{
			"name":"Crawler",
			"label":2,
			"position":
			{
				"x":0.371,
				"y":-0.319,
				"z":1.166
			},
			"dimension":
			{
				"width":0.235,
				"height":0.169,
				"depth":0.204
			}
		},
		{
			"name":"Circuit_1",
			"label":1,
			"position":
			{
				"x":0.532,
				"y":0.034,
				"z":1.112
			},
			"dimension":
			{
				"width":0.047,
				"height":0.033,
				"depth":0.066
			}
		},
		{
			"name":"Circuit_2",
			"label":1,
			"position":
			{
				"x":0.297,
				"y":-0.067,
				"z":1.193
			},
			"dimension":
			{
				"width":0.105,
				"height":0.028,
				"depth":0.068
			}
		},
		{
			"name":"Circuit_3",
			"label":1,
			"position":
			{
				"x":0.16,
				"y":0.107,
				"z":1.189
			},
			"dimension":
			{
				"width":0.093,
				"height":0.042,
				"depth":0.09
			}
		},
		{
			"name":"Circuit_4",
			"label":1,
			"position":
			{
				"x":0.056,
				"y":-0.233,
				"z":1.273
			},
			"dimension":
			{
				"width":0.054,
				"height":0.023,
				"depth":0.061
			}
		},
		{
			"name":"Circuit_5",
			"label":1,
			"position":
			{
				"x":-0.108,
				"y":0.095,
				"z":1.264
			},
			"dimension":
			{
				"width":0.063,
				"height":0.029,
				"depth":0.079
			}
		},
		{
			"name":"Circuit_6",
			"label":1,
			"position":
			{
				"x":-0.017,
				"y":-0.073,
				"z":1.255
			},
			"dimension":
			{
				"width":0.70,
				"height":0.031,
				"depth":0.07
			}
		},
		{
			"name":"Battery",
			"label":0,
			"position":
			{
				"x":-0.11,
				"y":-0.320,
				"z":1.322
			},
			"dimension":
			{
				"width":0.035,
				"height":0.021,
				"depth":0.09
			}
		}
	]
}

The error is that it cannot parse negative real numbers. so inside:
        "position":
        {
            "x":0.371,
            "y":-0.319,
            "z":1.166
        }

It stops at "y" key and return an invalid number.

The error is that while parsing numbers '+' and '-' characters are not added to digitsstring. So while looking for dot character ('.') it stops at the beginning ('-' in this case)
const std::string digits("+-0123456789");
int c;
while(digits.find(mIn->peek()) != std::string::npos)
	c = mIn->get();

A possible solution is to add this charaters to digits string (and may be change it names to digitsAndSigns or something). Or create another string call signs and check in both. I tested it and it works.

Solved and added a test case.


RedHat-based systems' dependencies (CentOS/Fedora)

You missed to add the dependency: vim-commom
Without it the following error occurs:
xxd: command not found

Thanks, I totally forgot about that. I will add it to the readme.

This should be added for Debian/ Ubuntu as well.


Add TypeScript typings

Coverage remained the same at 100.0% when pulling 39a51529848a756c7ce731eb1995508f4c51c7ac on bcherny:master into 1c2238b256dd4579a28724ac4b99018ae42d6661 on timdp:master.

:white_check_mark: Build es6-promise-pool 20 completed (commit https://github.com/timdp/es6-promise-pool/commit/55ed66be87 by @bcherny)

Awesome, thanks! :+1:


entries can leak on peer death due to re-ordering of IPAM and DNS gossip

When a peer stops/dies shortly after it adds a DNS entry, the associated topology gossip may well be received by some other peer before the DNS gossip - due to it possibly travelling via different paths, and internal re-ordering. As a result that peer will remove all entries of the gone peer... and then add the entry it receives in the DNS gossip.
The same could also happen if a surviving peer receives the topology gossip which ends up removing the dead peer, and then some periodic DNS gossip from another peer that still contains the entries of the dead peer.

In thinking about how to prevent this, I just spotted the code that already does so :)


Press Enter on a button will trigger the click event twice

TuSimple/naive-ui version (版本)
2.20.3
Vue version (Vue 版本)
3.2.22
Browser and its version (浏览器及其版本)
Edge 80
System and its version (系统及其版本)
MacOS 10.15.7
Node version (Node 版本)
Reappearance link (重现链接)
https://codesandbox.io/s/nervous-morning-tpnof?file=/src/Demo.vue
Reappearance steps (重现步骤)
https://codesandbox.io/s/nervous-morning-tpnof?file=/src/Demo.vue.
Keyboard press enter will trigger the click event twice.
Expected results (期望的结果)
should trigger only once
Actual results (实际的结果)
redundant triggering
Remarks (补充说明)

I'm guessing both keyup and keydown are triggered. If I keep pressing down the enter key, the click event will execute all the time. I think it should execute only once for a long press.


dubbo_proxy: Add decoder for dubbo protocol

Signed-off-by: tianqian.zyf<EMAIL_ADDRESS>Description: Add decoder for dubbo protocol
Risk Level: low
Testing: unittests
Docs Changes: N/A
Release Notes: N/A
[Optional Fixes #Issue] #3998 #4200
[Optional Deprecated:]

I think per our extension policy this one only needs Lizan's sign-off


How to use operator LIKE in datagrid.yml ?

How to  use operator LIKE in datagrid.yml with %, eg link like “%word%”?
Native SQL looks
SELECT * from table WHERE link like '%word%';
I tried this in datagrid:
......
where:
and:
- referral.link LIKE '%%word%%'
.......
but it doesnt work!
Error: "The parameter "key" must be defined."
Do you have some solution?

Hi @VitaliyBelous,
Thank you for reporting the issue.
It seems that it is a bug in SystemAwareResolver.
I've added this issue in our backlog.
As a workaround you can create a listener for oro_datagrid.datagrid.build.before.your_grid_name event of your grid to add LIKE operator.

Meanwhile you could also use CONCAT(CHAR(37), 'word', CHAR(37)) as a workaround


Add helm chart

This is a first iteration to try to create a Helm chart for the service bus exporter. We can discuss it and I'll make the changes you need.
This is self hosted on Github pages, I reached out to the Kubeapps people and they did not reply. See the index file at: https://giggio.github.io/servicebus_exporter/index.yaml
A release was created to make it work: https://github.com/giggio/servicebus_exporter/releases/tag/servicebusexporter-0.0.1
I used the chart releaser from the Helm project to create the index and the release on Github: https://github.com/helm/chart-releaser
Please let me know what you think and we can move it from there.

Hi giggio,
Thank you for this contribution. I'll take some time to review it, because first I need to learn about creating helm charts a little bit :)

No problem, it is working from my fork. Whenever you have the time!

Hi giggio,
The chart looks good to me overall. I left some comments on the PR.
However I think it would make more sense if you contributed this chart to the official Helm repo at https://github.com/helm/charts They have several exporter charts already.
It will be difficult for me to take responsibility for maintaining this chart, because I don't use it in my deployments and I'm not an expert on Helm.


Add setup.cfg to message-parser

Added setup.cfg to define package metadata for the cachito build.
Signed-off-by: Don Penney<EMAIL_ADDRESS>

/assign @jzding

/lgtm

/ok-to-test

/retest

/retest

/retest

/retest

/test e2e-basic

/retest

/test e2e-basic

/cherry-pick main


评测结果的稳定性如何保证？

Describe the feature
大模型答案的生成和温度等参数有关？ 所以，opencompass的榜单模型，是保证每次生成的答案都是固定的吗？
Will you implement it?

[ ] I would like to implement this feature and create a PR!

Primarily, our decoding phase employs greedy search, and fixed generation parameters (like temperature and so on) for every model ensuring theoretical consistency in the responses generated. However, there are nuances to consider that can subtly alter outcomes. For instance, the presence or absence of inference accelerators such as VLLM or LMDeploy can introduce slight variations in the results due to their impact on the processing efficiency and potential optimizations they offer.

Primarily, our decoding phase employs greedy search, and fixed generation parameters (like temperature and so on) for every model ensuring theoretical consistency in the responses generated. However, there are nuances to consider that can subtly alter outcomes. For instance, the presence or absence of inference accelerators such as VLLM or LMDeploy can introduce slight variations in the results due to their impact on the processing efficiency and potential optimizations they offer.

我通过调用api服务测试，发现每次测试的结果分数都不一样，本地模型路径的方式加载测试是一样的，调用api测试是哪里设置不对吗，temperature=0，top_k=-1,top_p=1,可以查看下面issue，谢谢大佬
https://github.com/open-compass/opencompass/issues/1634


[Feature Request] Better rendering for the /lookup character command

After showing the bot recently, one of the points where people “get stuck” is reading the information with the /lookup character command, when asked however how they would like it.... they can't answer ... so I tried to interpret an acceptable solution for them.
Is it possible to improve the output markdown of the /lookup character command, using a better markdown template?
I'll put a couple of examples I've prepared to give you an idea of what would be needed.
BASIC MARKDOWN
___
>## XXX
>- **Size** Medium
>- **Race\Species** Human
>- **Alignment** neutral evil
>- **Exp** 900xp
>- **Spell Save DC** 16
>- **Spell Attack Mod** 16
>- **Pouch** 10gp
>___
>- **Armor Class** 16 (*breastplate*)
>- **Hit Points** 23
>- **Speed** 30 ft.
>___
>|STR|DEX|CON|INT|WIS|CHA|
>|:---:|:---:|:---:|:---:|:---:|:---:|
>|16 (+3)|15 (+2)|14 (+2)|14 (+2)|11 (+0)|14 (+2)|
>
>- **Saving Throws** Str +5, Dex +4, Wis +2
>- **Skills** Athletics +5, Deception +4, Intimidation +4
>- **Damage Resistances** Necrotic
>- **Damage Immunities**
>- **Senses** passive Perception 10
>- **Languages** Celestial, Infernal
>- **Proficiency Bonus** +2
>___
>### Equipment
> dbgfofig, dfhfhf, fhfhof

Result:

XXX

Size Medium
Race\Species Human
Alignment neutral evil
Exp 900xp
Spell Save DC 16
Spell Attack Mod 16
Pouch 10gp

Armor Class 16 (breastplate)
Hit Points 23
Speed 30 ft.

STR
DEX
CON
INT
WIS
CHA

16 (+3)
15 (+2)
14 (+2)
14 (+2)
11 (+0)
14 (+2)

Saving Throws Str +5, Dex +4, Wis +2
Skills Athletics +5, Deception +4, Intimidation +4
Damage Resistances Necrotic
Damage Immunities
Senses passive Perception 10
Languages Celestial, Infernal
Proficiency Bonus +2

Equipment
dbgfofig, dfhfhf, fhfhof

MORE COMPLEX MARKDOWN (maybe with additional options "showActios" and "showSpells" ?)
___
>## XXX
>- **Size** Medium
>- **Race\Species** Human
>- **Alignment** neutral evil
>- **Exp** 900xp
>- **Spell Save DC** 16
>- **Spell Attack Mod** 16
>- **Pouch** 10gp
>___
>- **Armor Class** 16 (*breastplate*)
>- **Hit Points** 2
>- **Speed** 30 ft.
>___
>|STR|DEX|CON|INT|WIS|CHA|
>|:---:|:---:|:---:|:---:|:---:|:---:|
>|16 (+3)|15 (+2)|14 (+2)|14 (+2)|11 (+0)|14 (+2)|
>
>- **Saving Throws** Str +5, Dex +4, Wis +2
>- **Skills** Athletics +5, Deception +4, Intimidation +4
>- **Damage Resistances** Necrotic
>- **Damage Immunities**
>- **Senses** passive Perception 10
>- **Languages** Clestial, Infernal
>- **Proficiency Bonus** +2
>___
>### Equipment
> dbgfofig, dfhfhf, fhfhof
>___
>### Traits
>***Brute.*** A melee weapon deals one extra die of its damage when the bandit lord hits with it (included in the attack).
>
>***Pack Tactics.*** The bandit lord has advantage on attack rolls against a creature if at least one of the bandit lord's allies is within 5 feet of the creature and the ally isn't incapacitated.
>
>### Actions
>***Longsword.*** *Melee Weapon Attack:*  +5 to hit, reach 5 ft., one target. *Hit:* 12 (2d8 + 3) slashing damage, or 14 (2d10 + 3) slashing damage if used with two hands.
>
>***Dagger.*** *Melee  or Ranged Weapon Attack:*  +5 to hit, reach 5 ft. or range 20/60 ft., one target. *Hit:* 5 (1d4 + 3) piercing damage, or 8 (2d4 + 3) piercing damage if used to make a melee attack.
>
>### Bonus Actions
>***Leadership (Recharges after a Short or Long Rest).*** For 1 minute, the bandit lord can utter a special command or warning whenever a nonhostile creature that it can see within 30 feet of it makes an attack roll or a saving throw. The creature can add a d4 to its roll provided it can hear and understand the bandit lord. A creature can benefit from only one Leadership die at a time. This effect ends if the bandit lord is incapacitated.
>
>### Reactions
>***Parry.*** The bandit lord adds 2 to its AC against one melee attack that would hit it. To do so, the bandit lord must see the attacker and be wielding a weapon.
>
>***Protect Me!.*** When a creature the bandit lord can see targets it with an attack, the bandit lord can sidestep behind an ally within 5 feet of it, moving to an unoccupied space within 5 feet of the ally. The chosen ally becomes the target of the attack instead.
>___
>### Spellcasting
>Cantrips (at will): *fire bolt*, *light*, *mage hand*, *prestidigitation*, *shocking grasp*
>1st level (4 slots): *detect magic*, *identify*, *mage armor*, *magic missile*
>2nd level (3 slots): *detect thoughts*, *mirror image*, *misty step*
>3rd level (3 slots): *counterspell*, *fly*, *lightning bolt*
>4th level (3 slots): *banishment*, *fire shield*, *stoneskin*
>5th level (3 slots): *cone of cold*, *scrying*, *wall of force*
>6th level (1 slot): *globe of invulnerability*
>7th level (1 slot): *teleport*
>8th level (1 slot): *mind blank*
>9th level (1 slot): *time stop*

Result:

XXX

Size Medium
Race\Species Human
Alignment neutral evil
Exp 900xp
Spell Save DC 16
Spell Attack Mod 16
Pouch 10gp

Armor Class 16 (breastplate)
Hit Points 2
Speed 30 ft.

STR
DEX
CON
INT
WIS
CHA

16 (+3)
15 (+2)
14 (+2)
14 (+2)
11 (+0)
14 (+2)

Saving Throws Str +5, Dex +4, Wis +2
Skills Athletics +5, Deception +4, Intimidation +4
Damage Resistances Necrotic
Damage Immunities
Senses passive Perception 10
Languages Clestial, Infernal
Proficiency Bonus +2

Equipment
dbgfofig, dfhfhf, fhfhof

Traits
Brute. A melee weapon deals one extra die of its damage when the bandit lord hits with it (included in the attack).
Pack Tactics. The bandit lord has advantage on attack rolls against a creature if at least one of the bandit lord's allies is within 5 feet of the creature and the ally isn't incapacitated.
Actions
Longsword. Melee Weapon Attack:  +5 to hit, reach 5 ft., one target. Hit: 12 (2d8 + 3) slashing damage, or 14 (2d10 + 3) slashing damage if used with two hands.
Dagger. Melee  or Ranged Weapon Attack:  +5 to hit, reach 5 ft. or range 20/60 ft., one target. Hit: 5 (1d4 + 3) piercing damage, or 8 (2d4 + 3) piercing damage if used to make a melee attack.
Bonus Actions
Leadership (Recharges after a Short or Long Rest). For 1 minute, the bandit lord can utter a special command or warning whenever a nonhostile creature that it can see within 30 feet of it makes an attack roll or a saving throw. The creature can add a d4 to its roll provided it can hear and understand the bandit lord. A creature can benefit from only one Leadership die at a time. This effect ends if the bandit lord is incapacitated.
Reactions
Parry. The bandit lord adds 2 to its AC against one melee attack that would hit it. To do so, the bandit lord must see the attacker and be wielding a weapon.
Protect Me!. When a creature the bandit lord can see targets it with an attack, the bandit lord can sidestep behind an ally within 5 feet of it, moving to an unoccupied space within 5 feet of the ally. The chosen ally becomes the target of the attack instead.

Spellcasting
Cantrips (at will): fire bolt, light, mage hand, prestidigitation, shocking grasp
1st level (4 slots): detect magic, identify, mage armor, magic missile
2nd level (3 slots): detect thoughts, mirror image, misty step
3rd level (3 slots): counterspell, fly, lightning bolt
4th level (3 slots): banishment, fire shield, stoneskin
5th level (3 slots): cone of cold, scrying, wall of force
6th level (1 slot): globe of invulnerability
7th level (1 slot): teleport
8th level (1 slot): mind blank
9th level (1 slot): time stop

@p4535992
Discord's markdown support is rather lacking. I'm using a Discord Embed to format /lookup's if you want to check that out as well. I could do format tables as an ascii code block, but it's not very aesthetic.


Double check the data being downloaded for Chandra pre-processed vs 'raw'

I just download secondary for 'raw' option and I don't necessarily think that is right.

I think chandra repro does need the primary folder as well


"May" is not a Strong Word for Software

Can you please add more specificity around what is implied by "may work with Node 0.12"? I'm running Node 0.12.1 and the latest version of Zombie after upgrading to avoid an issue described here:
https://github.com/assaf/zombie/issues/181
Which is (despite the status) unresolved for legacy 3.4.x versions. I am now running into an uncaught TypeError after having changed no code associated with my tests.

What confidence should I have in the fact that 4.x.x may work with Node 0.12?
Are there tests around the performance of this library in Node 0.12?
Is there going to be any development supporting future Node versions?

I've enjoyed using your library thus far and would like to continue doing so in Node.

The short answer is: Zombie 4.x is written and tested to work correctly on io.js.
The test suite passes under Node 0.12 (see https://travis-ci.org/assaf/zombie/builds/58037512), however, those tests are fairly simple, it may well refuse to work correctly on more complex test scenarios.
There are no performance tests.
I don't know what Node will do in the future, right now they're shipping a version that's far behind on the evolutionary path. There's some talk about them catching up and merging with io.js.

Updated README.


Adding a regular expression to validate VSTS account

A simple fix in case someone provides an account without protocol (account.visualstudio.com) or if the name includes http (for example myhttpaccount as account name)

Sorry for delay. I will review soon.


Change root_dir of pyls to CWD to prevent pyls from traversing HOME directory

The doc says the root_dir of pyls is "vim's starting directory" but actually it is home directory.
I tried to fix #118 and noticed that rope performs rename on files under home directory. This significantly harms performance.

Thanks. I'm confused about why home dir was chosen for any of these configs. ..


Feature request, no prompt when there are no choice points

When using the new REPL I get, it prompts 2 times:
$ node taupl
/* Tau Prolog new REPL */
?- X=a; X=b.
X = a ;
;
X = b.
;
false.
?-

Many Prolog systems detect that the solution X=b doesn't leave any further
choice points, and then omit the second prompt:
/* SWI-Prolog */
?- X=a; X=b.
X = a ;
X = b.

?-

Would it be possible to bring this feature to Tau Prolog?

Inspecting the new taupl.js I get:
Somehow the show seems to already detect that there are no choice
points, since Tau Proog writes a period X = b.. But next_answer is
possibly too greedy in that it always calls action. Not sure.

Ok, I made myself a new top-level:
Change 1: in taupl.js:
-     console.log(pl.format_answer(answer, session, {quoted: true}));
+     fs.writeSync(process.stdout.fd, pl.format_answer(answer, session, {quoted: true}));

        if(answer === false || pl.type.is_error(answer))
            callback();
+       else if (!(session.thread.points.length > 0))
+           callback();
        else
            action(callback);

Change 2: in core.js:
-				var delimiter = typeof thread === "undefined" || thread.points.length > 0 ? " ;" : ".";
+				var delimiter = typeof thread === "undefined" || thread.points.length > 0 ? " " : ".\n";

Seems to work now:
taupl.js.txt
core.js.txt

Done. Thanks for the improvement.


Add roundtrip tests for Axes3D archetype

https://github.com/rerun-io/rerun/pull/6510 added a new archetype, but no roundtrip tests were added for it

https://github.com/rerun-io/rerun/pull/6524#discussion_r1632997826


Add gender and gendered titles

This no longer seems applicable - only Honorifics and Titles as they used to exist really made sense as being gendered. Since both of these have been removed, it seems safe to close this.


Debugger expression evaluation only considers first line of multiline expressions for Flutter

If you evaluate a multi-line expression in Dart, it evaluates the whole multiline expression:

However if you do the same thing in a Flutter app, it only evaluates the first line:

Hi @DanTup
can you please provide your flutter doctor -v?
Thank you

[✓] Flutter (Channel master, v1.18.0-7.0.pre.47, on Mac OS X 10.15.4 19E287, locale en-GB)
    • Flutter version 1.18.0-7.0.pre.47 at /Users/danny/Dev/Google/flutter
    • Framework revision 240611b05e (4 days ago), 2020-04-23 11:54:02 -0400
    • Engine revision feb94f6c97
    • Dart version 2.9.0 (build 2.9.0-2.0.dev 64b8ded48b)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/danny/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 11.4.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.4.1, Build version 11E503a
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install:
        sudo gem install cocoapods

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.5)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 42.1.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] VS Code (version 1.44.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.9.1

[✓] Connected device (4 available)
    • iPhone 11 Pro Max • 42F08B32-ABC6-4D33-8D00-7C6D55FFC8AC • ios            • com.apple.CoreSimulator.SimRuntime.iOS-13-4 (simulator)
    • macOS             • macOS                                • darwin-x64     • Mac OS X 10.15.4 19E287
    • Web Server        • web-server                           • web-javascript • Flutter Tools
    • Chrome            • chrome                               • web-javascript • Google Chrome 81.0.4044.122

! Doctor found issues in 1 category.

I think this might related to how Flutter compiles expressions - seems like it might pass them to Dart, and Dart expects a single line for the expression:
https://github.com/dart-lang/sdk/blob/bd3d08a891791ac3b83a55bd6652ae0b995912e9/pkg/frontend_server/lib/frontend_server.dart#L1198
And Flutter just writes the expression as-is:
https://github.com/flutter/flutter/blob/b7e30cfc26040df3fac1467ecb4efb8acc5ae49c/packages/flutter_tools/lib/src/compile.dart#L763
I'm not super familiar with this, @aam might know if this is on the right track.

I think this might related to how Flutter compiles expressions - seems like it might pass them to Dart, and Dart expects a single line for the expression:

Yeah, I think you are right!

Hi there, anything planned to fix this?
Thx!


Can't build

Hello everyone, I tried to build cheat as Release x86 but all I get is this: (Also I have updated Visual Studio to latest version)
1>C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.36.32532\modules\std.ixx(147,1): fatal  error C1001: Internal compiler error.
1>(compiler file 'msc1.cpp', line 1589)
1> To work around this problem, try simplifying or changing the program near the locations listed above.
1>If possible please provide a repro here: https://developercommunity.visualstudio.com
1>Please choose the Technical Support command on the Visual C++
1> Help menu, or open the Technical Support help file for more information
1>INTERNAL COMPILER ERROR in 'C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.36.32532\bin\HostX86\x86\CL.exe'
1>    Please choose the Technical Support command on the Visual C++
1>    Help menu, or open the Technical Support help file for more information
1>Done building project "Osiris.vcxproj" -- FAILED.

also got this error out of nowhere on my ols paste

also got this error out of nowhere on my old paste

How you fixed that error?

you cant build osiris with visual studio 2022. delete vs 22 and download vs 2019 instead

you cant build osiris with visual studio 2022. delete vs 22 and download vs 2019 instead

Doesn't mean, I building Osiris with Visual Studio 2022 since january 2022 unit now...

You can fix it by changing C++ standard from 'latest' to 'C++20' or by turning off "Build ISO C++23 Standard Library Modules" under C/C++ >> Language in project properties.


Add tower.yml entry for audit log retention

Seqera Platform Enterprise ships with a default configuration which purges audit logs once they are older than 365 days.
This has been implemented to minimize performance degradation when retrieving data to populate the Audit Log GUI, but may prove to be too short a retention period to meet the data retention policies of specific implementing clients.
Fortunately, the value can be explicitly configured during setup. This issue has been created to implement the necessary changes:
Configuration Snippet
source
# Implement in tower.yml
tower:
  cron:
    audit-log:
      clean-up:
        time-offset: 365d

To Do

[x] Add snippet in tower.yml.tpl
[x] Add configurable value to tower.yml.tpl (mapped to time-offset)
[x] Add configurable value to terraform.tfvars:  tower_audit_retention_days
[x] Add mapping entry to data object containing the interpolated tower.yml.tpl


Eliminate dependencies on jquery, and underscore.

I don't think we should need all of jquery just for a couple of methods.
We could simplify this with document.querySelector if we don't care about IE <= 7.
I don't think underscore is used on the client, so I've removed that dependency too.
Does this look good?

Also, it would be nice if we could do away with underscore on the server. Could the template code be replaced with something as simple as this?
var injectHtml = '<script type="text/inject-data">' + data + '</script>';

I'm also okay with removing underscore from the server as well. Go for it and add it to here as well.
Once you did that, just add a comment to notify me.

@arunoda: done. I haven't done any testing however. Can you tell me a good way to run the tests?

We have some tests which covers most of the cases.
 Okay, all passing now.

Hey @gunn great shout on removing jQuery - I've had a local fork without jQuery for a while and kept meaning to do a PR!
Just curious - you mentioned about about using querySelector but haven't used it in your PR. Did you have issues with it? I only ask as my implementation is like this:
  var dom = document.querySelector('script[type="text/inject-data"]');
  // .innerText fallback for IE8
  var injectedDataString =
    ((dom && (dom.textContent || dom.innerText)) || '').trim();
  InjectData._data = InjectData._decode(injectedDataString) || {};

And I'm hoping I don't have a bug lurking about that I'm not aware of!
Thanks :)

@tomwasd no problems, just being cautious as I don't what browser versions we need to support. See http://caniuse.com/queryselector
I was going to point out someone had already had trouble with .trim() in IE at #3 but... :stuck_out_tongue:

Thanks @gunn
Ha I now use the lovely es5-shim package internally so I can be confident .trim() is present
Next step is removing jQuery from Kadira... :open_mouth:

@tomwasd Wow. A world without jQuery :)
@gunn I'll take the PR and do a release tomorrow.

Any news on this PR?

Would love to see this merged for a jquery-free world :)


Investigate multi-file object to send around check class

Investigate multi-file object to send around check class.
Use cases:

all files in a directory
a list of file paths
a list of netCDF4 Dataset objects (based on a set of files)

The "list of netCDF4 Dataset objects" example is used in our time-checks checkers:
https://github.com/cedadev/time-checks/blob/master/time_checks/multifile_time_checks.py

Hi @agstephens - starting to take look at this. I think we might be constrained by the way compliance-checker works regarding converting the dataset given as a string on the command line to an object passed to the check methods.
Currently it looks something like this:

Get ds_str and name of checkers to use from command line
Try to create an object using ds_str in various ways (from OPeNDAP URL, CDL, NetCDF file...)
Fall back to GenericFile if not successful with others
Look at available checker classes, select those whose supported_ds list contains the type of the dataset object

In the 3 checks in your above comment, we have the same input for all 3 (i.e. a list of filenames) but for (1), (2) we want a list of filenames and for (3) we want a list of Dataset objects. I don't see how this can work with the current code, because the choice of which object to create is made without considering the supported_ds of the checks.
To me it would more sense to:

Get checker classes first
For each type in supported_ds list, try and create an object from ds_str, continue if not successful (raise error if list is exhausted)

Then by having different supported_ds in checks (1)/(2) and (3) it should be possible to do what we want.
I think this would be a fairly big change to compliance checker itself though rather than a little patch. So would be good to get your thoughts on it.

@joesingo: Thanks for raising this issue. I agree that we are trying to do something new/special here.
I have always felt that the "try to guess which object we are working with" is not really the best way for the compliance checker to work. A better, and more flexible, approach would be to follow your suggestion above, something like:
for each checker class:
     find out supported types
     for each file/object to check:
           try parsing as each supported type until one is successful
           run checker on that object interpreted as that type

I think it is worth prototyping this.


Replatform Core to .NET standard

Replatform the library to .NET standard so I can build .NET core applications as well.

Completed as part of 6.0 release.


Embed primary Ultraviolet Window into Form controls?

Hello.
I'm looking for a way to embed a Ultraviolet window into a windows form control because I'm writing a small editor for my application. Is there any simple way to do it?
I've been trying to look into the source code but it looks like the corresponding class is missing (I'm refering to "TwistedLogik.Ultraviolet.OpenGL.Platform.OpenGLUltravioletWindow".)
I didn't actually expect this code to work but I guess this is the usual way you would do it with an actual control:
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;
using TwistedLogik.Ultraviolet;

namespace Test
{
    public partial class Editor : Form
    {
        public Editor()
        {
            InitializeComponent();

            var context = UltravioletContext.DemandCurrent();
            var window = context.GetPlatform().Windows.GetPrimary();
            splitContainerGame.Panel1.Controls.Add((System.Windows.Forms.Control)window);
        }
    }
}

Yes, Ultraviolet supports this directly. I am about to board an airplane, so I can't give you a detailed write-up at the moment, but please take a look at the UltravioletTool project in the master branch. It uses the TwistedLogik.Ultraviolet.WindowsForms library, which provides both an UltravioletForm and an UltravioletPanel class from which you can inherit.
If you're still having trouble, I'll do my best to help out tomorrow after I'm done traveling.

Were you able to get this working?

Oh, hello.
Thanks for your quick answer and sorry for the silence on my part. I've been able to take a look at the UltravioletTool project but I got busy with work and some health related issues.
From what I've observed it isn't possible to just add the window to an UltravioletApplication class, I would need to create a UltravioletContext for the actual form, which means I have to create a new class, correct? I'm asking this because of the way I tend to design my code. At the moment my GameController is just a subclass of the UltravioletApplication class, if I would use a second class which is very simular but is a subclass of another class, then I would need to create a static class which contains the code my current GameController contains because multiple inheritance is not an option in C#. Am I correct?

Yes, in a Windows Forms application, the UltravioletForm class serves as the Ultraviolet host (i..e, it replaces the UltravioletApplication class). There are other scenarios where this is the case as well: for example, if you're building your project for Android, you need to use UltravioletActivity. For this reason, shared library code generally shouldn't really make any references to any of the Ultraviolet host classes.

Hmmm, I see. So the way it's designed in UV is probably already the best way one could do it.
Many thanks for your time, I think I will be able to implement this change, it might take some time though, as I need to finish some other things first.

Sure thing. Let me know if you run into any issues.

Hello.
I can't get it working. I can't find a NuGet Package for "UltravioletTool". I could compile it myself but I'm stuck with Visual Studio 2013 because of the new license model used for newer versions, so I am pretty much stuck at this point.

Looking at it again, apparently I never bothered to make a NuGet package for the UltravioletTool project. I can get that fixed in the next couple of days when I have some time to look at it.

Sorry for taking so long to get back to this. I recently moved and started a new job, so the amount of time I've been able to invest in Ultraviolet has been very limited for the past several months.
I just pushed a new build to the NuGet servers. This build includes two new packages:

TwistedLogik.Ultraviolet.WindowsForms - the library which defines the Windows Forms controls
TwistedLogik.Ultraviolet.Game.WindowsForms - a simple skeleton project

Be aware that it may take a bit of time for these packages to appear in the NuGet index.
Please let me know if you still have issues.


is this still maintained ?

A lot of prs are piling up and I ended up having to create a frankenstein fork including all those I need but this does not really work as a long term solution so I am wondering what is the state of this project.

If it's a case of workload, I'm happy to volunteer some time to review pull requests if needed.

If it's a case of workload, I'm happy to volunteer some time to review pull requests if needed.

Hi,
Would you mind letting us know, what functionality you're missing?

Disclaimer and comment regarding if the project is dead, please note this are my personal opinions, other contributors might have their own view.

Our main interest is in a stable and safe implementation of GitHub PR Resource checks, although I admit, we're lacking a little bit in the speed of our reviews, a Pull Request is not guaranteed to be integrated, for example, we chose not to integrate GitHub Application authentication, because our own application found in telia-oss/sidecred issues a scoped token, as a separate component.
Otherwise, our iteration of this would probably mirror the description you made; nobody wants a frankenstein source either, if anyone wants to contribute, you could assist in refactoring the way we perform tests, as it's become a small nightmare to effectively isolate test-cases, which is one of the reasons we've slowed down integration of new features.
But the project isn't dead, we're just a bit distracted by our daily work, my organisation is moving towards GitHub Actions and some past contributors are busy as well, it's just nature of things, we do this on our own spare time, in a pro-bono fashion.
If someone wants to pitch in and incorporate some contributions, that person should probably help us make our tests more robust, there's a lot of large organisations that depend on this solution, I personally already screwed up a bit in our last release, so I was working on reducing the over-fetching but got stuck on dynamic GraphQL queries for filtering out PR's based on labels, I haven't solved that one since it would require me to template the queries, just like the GitHub CLI does.
I also spoke to DO's contributor of the fork to see if we can merge our efforts, he's left DO now and we haven't had the chance to catch up regarding that, but if some of you are serious about taking a more active role, I'd probably request help with the above, because that's where the majority of our time is spent, making every effort to not break stuff for people.

Hi,
Would you mind letting us know, what functionality you're missing?

Disclaimer and comment regarding if the project is dead, please note this are my personal opinions, other contributors might have their own view.

Our main interest is in a stable and safe implementation of GitHub PR Resource checks, although I admit, we're lacking a little bit in the speed of our reviews, a Pull Request is not guaranteed to be integrated, for example, we chose not to integrate GitHub Application authentication, because our own application found in telia-oss/sidecred issues a scoped token, as a separate component.
Otherwise, our iteration of this would probably mirror the description you made; nobody wants a frankenstein source either, if anyone wants to contribute, you could assist in refactoring the way we perform tests, as it's become a small nightmare to effectively isolate test-cases, which is one of the reasons we've slowed down integration of new features.
But the project isn't dead, we're just a bit distracted by our daily work, my organisation is moving towards GitHub Actions and some past contributors are busy as well, it's just nature of things, we do this on our own spare time, in a pro-bono fashion.
If someone wants to pitch in and incorporate some contributions, that person should probably help us make our tests more robust, there's a lot of large organisations that depend on this solution, I personally already screwed up a bit in our last release, so I was working on reducing the over-fetching but got stuck on dynamic GraphQL queries for filtering out PR's based on labels, I haven't solved that one since it would require me to template the queries, just like the GitHub CLI does.
I also spoke to DO's contributor of the fork to see if we can merge our efforts, he's left DO now and we haven't had the chance to catch up regarding that, but if some of you are serious about taking a more active role, I'd probably request help with the above, because that's where the majority of our time is spent, making every effort to not break stuff for people.

@rickardl You make some very good points around the test issues. I was similarly trying to shift more of the filtering/sorting work to the GraphQL backend but then testing things becomes pretty tricky since you're now entirely dependent on the behaviour of the backend. Trying to reimplement GitHub's API as some kind of mock doesn't seem like a great option either.
For example, I made a decent amount of progress refactoring checks using GitHub's search query, which means sorting and filtering can be moved almost entirely to the backend but I had to disable all the check tests to actually see if it worked. Now I'm not sure what the best way of testing this is.

@rickardl You make some very good points around the test issues. I was similarly trying to shift more of the filtering/sorting work to the GraphQL backend but then testing things becomes pretty tricky since you're now entirely dependent on the behaviour of the backend. Trying to reimplement GitHub's API as some kind of mock doesn't seem like a great option either.
For example, I made a decent amount of progress refactoring checks using GitHub's search query, which means sorting and filtering can be moved almost entirely to the backend but I had to disable all the check tests to actually see if it worked. Now I'm not sure what the best way of testing this is.

Hi @vixus0
That's exactly the kind of duality in our issues with this, there's no easy way to make the current test-bench accessible to developers, side from making a test-repo available (as we do), and having developers fork that repo, perform actions and later restore.
But that's not acceptable, although integration tests are a crucial last mile, so my train of thought it looking at integrating the changes and ideas from https://github.com/digitalocean/github-pr-resource, done by https://www.linkedin.com/in/solveproblemswithcode/ (not sure why his GitHub is not linkable, maybe he made the classic mistake of creating his GitHub under his employer), we talked about merging the efforts earlier (because DO's fork is now unmaintained).
I would be excited and interested in moving our integrations into more loosely coupled packages, improving and maybe using the existing client from the GitHub CLI, moving interactions and interfaces for concourse into its own packages and take a pointer from the way he integrated his test, as well as actually providing proper logs and debugging steps.
That way we can focus on the intersection between Concourse and GitHub, namely the interface of our check and out resources.

Hi @vixus0
That's exactly the kind of duality in our issues with this, there's no easy way to make the current test-bench accessible to developers, side from making a test-repo available (as we do), and having developers fork that repo, perform actions and later restore.
But that's not acceptable, although integration tests are a crucial last mile, so my train of thought it looking at integrating the changes and ideas from https://github.com/digitalocean/github-pr-resource, done by https://www.linkedin.com/in/solveproblemswithcode/ (not sure why his GitHub is not linkable, maybe he made the classic mistake of creating his GitHub under his employer), we talked about merging the efforts earlier (because DO's fork is now unmaintained).
I would be excited and interested in moving our integrations into more loosely coupled packages, improving and maybe using the existing client from the GitHub CLI, moving interactions and interfaces for concourse into its own packages and take a pointer from the way he integrated his test, as well as actually providing proper logs and debugging steps.
That way we can focus on the intersection between Concourse and GitHub, namely the interface of our check and out resources.

@rickardl Ok, sounds good - let's arrange something.

@rickardl Ok, sounds good - let's arrange something.

@rickardl I understand and I am not asking to merge everything but currently I cannot use this resource without #240, we have around 1200 prs with only a few open and without this fix the resource simply does not work as it takes so much time even concourse give up on it xD
Another nice addition is #227 which allows changing behavior depending on labels applied for more advanced workflows.

@rickardl I understand and I am not asking to merge everything but currently I cannot use this resource without #240, we have around 1200 prs with only a few open and without this fix the resource simply does not work as it takes so much time even concourse give up on it xD
Another nice addition is #227 which allows changing behavior depending on labels applied for more advanced workflows.

@rickardl could you clarify your earlier comment on the support model:

But the project isn't dead, we're just a bit distracted by our daily work, my organisation is moving towards GitHub Actions and some past contributors are busy as well, it's just nature of things, we do this on our own spare time, in a pro-bono fashion.

Before your org moved towards GitHub Actions, was this resource maintained as part of the contributors' daily work, or is the move away from Concourse a driving factor in adopting a pro bono support model for this resource?  Given how critical PR tracking is to folks who use Concourse, maybe we could find a way to hand this resource off to the Concourse developers for continuing development.

@rickardl could you clarify your earlier comment on the support model:
....
Before your org moved towards GitHub Actions, was this resource maintained as part of the contributors' daily work, or is the move away from Concourse a driving factor in adopting a pro bono support model for this resource?

Originally this was developed jointly in my current team, but time has passed, naturally some have left and our ecosystem has changed. The current the strategy has moved to using GitHub Actions as our main CI/CD tool, to reduce the overhead of onboarding developers to two systems and free up our time making these custom integrations or employing people to maintain Concourse.
This integration stills drives some "legacy" integrations internally, but it's no longer a critical path so has obviously been reduced in priorities as our focus is on other projects, which are available through other open-source repositories. This means that this work is completely done outside of working hours, even though the companies each contributor fully supports contributing to OSS projects, it still needs to make sense for the company.

Given how critical PR tracking is to folks who use Concourse, maybe we could find a way to hand this resource off to the Concourse developers for continuing development.

I won't speak on behalf of the Concourse developers, but I don't see a benefit it adding additional work on them, after all everyone is free in making contributions to this project, the issue at hand is "naive" way our tests have been implemented, see my earlier comment.
If I had confidence in our tests and we had a good way of handling integration testing against our repositories, it would be a must simpler task to accept contributions.

Thank you for clarifying!  Improving the tests to make it easier to accept contributions sounds like a good approach for unblocking some of the pending contributions here.  My suggestion to start a conversation with the Concourse team comes from longer-term concerns about development and maintenance.  Namely:  what happens to this repo when the remaining maintainers no longer work at Telia, and/or when all remaining usage of this resource is eliminated at the company?
While anyone is free to publish their own forks to pick up the torch, that only delays the problem: as we've seen with the original jtarchie repo and the digitalocean fork of this repo, eventually the teams maintaining those forks might move on as well.  If we could convince the Concourse team to provide a github PR resource just as they do the git resource, that would enable us all to continue to contribute (and fork as needed), and might provide more stability for development and maintenance.

@ctreatma Just a follow-up, we've been in contact with your open-source organisation, our response is similar to the above one and we're waiting for a response if Comcast is interested in discussing the future of this project.
Let us know if there's anything we can do together to support your continued use of this resource, as we explained the project is nowhere near dead, it's just that we want to make sure our releases don't break.

Hello, is there any update on the subject? I don't think any PR-s are getting merged at this point. And some seem to be pretty important, like the one where Github PR-s are getting forever skipped.

I would be happy to offer my help and refactor the unit tests using Ginkgo, for better clarity of those.
I understand that there are more concerns about integration tests implying actual GraphQL queries, but maybe this first work would be of help in having the code easier to maintain.
I think donating this project to the Concourse team would possibly help so that new contributors can be granted roles on the project in order to maintain it. Otherwise a fork will be necessary, as I've already done last summer with the keyvalue resource.

Are we asking this resource to do too much heavy lifting? Currently it is responsible for:

Filtering PRs.
Fetching PR metadata (a lighter version of this resource?).
Cloning the repo (https://github.com/concourse/git-resource).
Commenting on the PR (https://github.com/nderjung/concourse-github-pr-comment-resource).
Setting commit statuses (https://github.com/Pix4D/cogito).

To me, it would make sense to separate the above responsibilities into smaller resource types that could be more easily maintained. Some of them already exist.
For example, the Concourse team seems to be moving towards a pattern of provisioning a Concourse pipeline per PR using the new dynamic across step. See aoldershaw's fork and usage. A resource that focused exclusively on fetching a filtered list of PRs would be beneficial here.

Hey, sorry @vixus0 I had missed your answer back then.
About the across step what you say is correct, but the Concourse team at VMware had been reduced to one single employee, and that was before VMware got acquired by Broadcom..
The pace in Concourse new versions has drastically been slowed down. As I've seen so far, the investment Broadcom puts in Concourse can only cover basic maintenance work, and the roadmap to Concourse v10 has few chances to get major step forward anytime soon.
That's why we've forked this repo with friends from the Cloud Foundry community, so that the code can be maintained (bumping dependencies and Golang versions), and new maintainers possibly onboarded.
We've integrated some contributions so far and plan to continue with features that have been contributed as PRs here. We also expect the automated test suite to be developed at some point.
Don't hesitate to give it a try and share your feedback!


ES2015 Template String and Arrow Function Unexpected token

Using webpack and browserify are no errors, only Fusebox is wrong.

tsconfig.json
{
    "compilerOptions": {
        "module": "commonjs",
        "target": "es5",
        "noImplicitAny": false,
        "sourceMap": false,
        "preserveConstEnums": true,
        "newLine": "CRLF",
        "allowUnusedLabels": true,
        "allowUnreachableCode": true,
        "noEmitHelpers":true,
        "lib": [
            "dom",
            "scripthost",
            "es5",
            "es2016",
            "es2015.promise"
        ]
    },
    "include": [
        "src/**/*.ts"
    ]
}

gulpfile.js
gulp.task('test:appjs', function () {
    fsbx.FuseBox.init({
        homeDir: "./src/",
        sourceMaps: true,
        plugins: [
            fsbx.HTMLPlugin({ useDefault: true }),
            fsbx.TypeScriptHelpers()
        ],
        outFile: "./tmp/js/app.js"
    }).bundle(">appdev.ts");
});

How do you import that file? Typescript does not seem to compile it.

the error above means the file was not transpiled with TS, how do you import it in your code ?

Part of the code is imported using the require method, and the other part of the code is imported using the import of ES2015.

I found that the arrow function modified to normal anonymous functions and template string modified to normal string, it is correct.
It seems that this is not a TypeScript compiler problem, but fuse-box caused.

@mishengqiang FuseBox does not alter your code, unless you use aliases

Now I will have the arrow function test.ts as entry file, the result is still wrong.

Something is off with your typescript configs. Works perfectly for me

@nchanged Oh! My! God!
I modified the contents of the tsconfig.json file, and then restored, but the run task is correct, in fact I did not change tsconfig.json content.
This is amazing.

Maybe you did not restart?

I tried many times and found that if you remove tsconfig.json targe attribute will not be an error, otherwise there is still an error.

can we close it now?


iris.save always saves time and forecast_reference_time as 'hours since 1970-01-01 00:00:00'

I have below Scalar coordinates in NIMROD file
         forecast_reference_time: 2015-12-26 00:00:00           time: 2015-12-26 04:00:00
When I save it as GRIB file using iris.save(cube, "myfile.nc")
GRIB file has variable as below
`    double forecast_reference_time;
:units = "hours since 1970-01-01 00:00:00";
:standard_name = "forecast_reference_time";
:calendar = "standard";
double time;
  :units = "hours since 1970-01-01 00:00:00";
  :standard_name = "time";
  :calendar = "standard";`

and values as below..
forecast_reference_time =403080.0 time =403083.9999999963
Global attributes of GRIB File after save as below
  :field_code = 58S; // short   :nimrod_version = 2S; // short   :num_model_levels = 1S; // short   :source = "Best Medium-Range";   :Conventions = "CF-1.5";
How to convert time and forecast_reference_time as is in NIMROD file to GRIB file..

Hi @naveenreddy2811, sorry to have left you without a response for so long.  I’m going to close this on the assumption that you have long since moved on and forgotten this issue.  If that’s not the case, please feel free to ask questions on our Google Group, which is our dedicated place for getting help with how to use Iris.  The GitHub issues are really for requesting and planning changes in Iris, which is perhaps partly why your question was unfortunately overlooked.


Update gemspec to use later version of jekyll

Solves #24

Fixed in https://github.com/thelehhman/plainwhite-jekyll/commit/eb36cd6d6571023ee342657ee4ced037108a41bd


Publish Docker Image to ghcr.io

Describe what the PR does:
This PR introduces pushing Docker images to ghcr.io (as well as retaining the existing push to Docker Hub).
Does this fix a specific issue?
N/A
Checklist:

[ ] Confirm that one or more new tests are written for the new functionality.
[x] Run tests and ensure everything passes (with 100% test coverage).
[x] Update README.md with any new documentation.
[ ] Add yourself to AUTHORS.md.

@alexdelprete FYI ^^^.

it works! that was quick, thank you! :)


fromJust in ContractModel

Summary
Hi, there's a call to fromJust in the contract model that is causing an error I can't diagnose
  Standard Model Check: FAIL (0.12s)
    *** Failed! (after 3 tests):
    Exception:
      Maybe.fromJust: Nothing
      CallStack (from HasCallStack):
        error, called at libraries/base/Data/Maybe.hs:148:21 in base:Data.Maybe
        fromJust, called at src/Plutus/Contract/Test/ContractModel/Internal.hs:1500:50 in plutus-contract-<IP_ADDRESS>-JzCKyMWsUmI2lrOzmBFwW:Plutus.Contract.Test.ContractModel.Internal
    Actions
     [tok1 := SetupTokens]
    Use --quickcheck-replay=472094 to reproduce.

fromJust call is here https://github.com/input-output-hk/plutus-apps/blob/main/plutus-contract/src/Plutus/Contract/Test/ContractModel/Internal.hs#L1562
Steps to reproduce the behavior
Can't share code on this one.
Actual Result
fromJust
Expected Result
should not fromJust
Describe the approach you would take to fix this
No response
System info
NixOS

@MaximilianAlgehed

Hi,
without more detail about what code you're running I can't really help you. One possible reason for this is that shrinking maintains symbolic tokens that refer to actions that don't exist. However, without more details there isn't super much I can do. I will try to issue a patch that gives a more helpful error message however.

Thanks, I was able to see what I had done wrong. Thanks for the error message.


Fix python_experiments and jedi_memory telemetry events

Looking into telemetry I noticed that the python_experiment event triggers 700 times on avarage per user, and jedi_memory is being triggered 500 times per user. Perhaps we can optmize it to collect the events only when we need it. For python_experiments maybe it's not super necessary since we'll deprecate the old A/B test framework anyway

Yes! Thank you 😊

Closing in favour of https://github.com/microsoft/vscode-python/issues/15530.


Revert Mint component and doc

Hello,
I made the Mint component and I got a blocking issue since one week.
Please remove it and also documentation.
To revert:

https://github.com/home-assistant/home-assistant/pull/6132
https://github.com/home-assistant/home-assistant.github.io/pull/2102
I don't have to time to fix it in short delay, I hope fix it in few weeks...

Please make a PR

What's the issue? Maybe someone else can pick it up?

In the future, please just create a PR to remove it yourself instead of opening an issue.

@balloob Sorry and thanks to did it. I didn't have time yesterday to do it so, I just opened the issue to be sure to not release it and do it later.
Thanks again !

@lwis I got this issue 2 or 3 days after the home assistant daemon start:
Mar 11 12:55:04 androlapin hass[17138]:   File "/opt/homeassistant/custom_components/sensor/mint_finance.py", line 161, in update
Mar 11 12:55:04 androlapin hass[17138]:   File "/opt/homeassistant/deps/mintapi/api.py", line 231, in get_accounts
Mar 11 12:55:04 androlapin hass[17138]: 17-03-11 12:55:04 INFO (Thread-5) [custom_components.sensor.mint_finance] Mint get account failed. Retrying (Try 1/5)
Mar 11 12:55:04 androlapin hass[17138]: 17-03-11 12:55:04 INFO (Thread-5) [custom_components.sensor.mint_finance] Mint get account failed. Retrying (Try 2/5)
Mar 11 12:55:05 androlapin hass[17138]: 17-03-11 12:55:05 INFO (Thread-5) [custom_components.sensor.mint_finance] Mint get account failed. Retrying (Try 3/5)
Mar 11 12:55:05 androlapin hass[17138]: 17-03-11 12:55:05 INFO (Thread-5) [custom_components.sensor.mint_finance] Mint get account failed. Retrying (Try 4/5)
Mar 11 12:55:05 androlapin hass[17138]: 17-03-11 12:55:05 INFO (Thread-5) [custom_components.sensor.mint_finance] Mint get account failed. Retrying (Try 5/5)
Mar 11 12:55:05 androlapin hass[17138]: 17-03-11 12:55:05 ERROR (Thread-5) [custom_components.sensor.mint_finance] Could not parse account data: <error>1<description>Session has expired.</description><name></name><type></type></error>


missing python dependencies

I hit some trouble. I was doing all this in a python 2.7 virtualenv. Would be good to list what python version you expect.
I followed the pip install instructions and ended up with some broken imports:
ImportError: No module named requests
After installing requests manually, I get this:
ImportError: No module named pathlib
After manually installing pathlib, I get this:
Traceback (most recent call last):
File "/home/rhs/.virtualenvs/kubernaut/bin/kubernaut", line 11, in
load_entry_point('kubernaut==0.1.0', 'console_scripts', 'kubernaut')()
File "/home/rhs/.virtualenvs/kubernaut/lib/python2.7/site-packages/pkg_resources/init.py", line 564, in load_entry_point
return get_distribution(dist).load_entry_point(group, name)
File "/home/rhs/.virtualenvs/kubernaut/lib/python2.7/site-packages/pkg_resources/init.py", line 2652, in load_entry_point
return ep.load()
File "/home/rhs/.virtualenvs/kubernaut/lib/python2.7/site-packages/pkg_resources/init.py", line 2306, in load
return self.resolve()
File "/home/rhs/.virtualenvs/kubernaut/lib/python2.7/site-packages/pkg_resources/init.py", line 2312, in resolve
module = import(self.module_name, fromlist=['name'], level=0)
File "/home/rhs/.virtualenvs/kubernaut/lib/python2.7/site-packages/kubernaut/cli.py", line 15, in
config_root = Path.home() / ".config" / "kubernaut"
AttributeError: type object 'Path' has no attribute 'home'

Requires Python 3 and not Python 2. I'll make sure this is called out at some point in the future.


Add support for Metal arrays with MtlCellArray

This PR adds support for Metal arrays in CellArray.jl, introducing the type MtlCellArray similarly to other GPUArrays.jl compatible backends.

@omlins requested changes in tests have been implemented, requesting another review.

Thanks @GiackAloZ  and @albert-de-montserrat !

The CI fails for Windows and MacOS:
https://github.com/JuliaGPU/Metal.jl/issues/457
https://github.com/JuliaGPU/Metal.jl/issues/458
Merged nevertheless into main.


preserve all headers with X- (to support passthrough of tracing headers)

@kd7lxl I expect several projects leveraging konk will need to preserve X-Request-ID and X-B3-* headers for logging consistency and tracability.  Having it enabled as default will make it easier for users not familiar with the apiserver command-line flags.
I will deploy konk tomorrow, but don't expect any issue based on prior experience.  Will reach out to you on slack to discuss.

Closing since @kd7lxl stated the Konk CR permitted the arguments to be specified in spec.  Documentation doesn't provide an example or any description of available fields in Konk CRD.

Documentation doesn't provide an example or any description of available fields in Konk CRD.

Addressed in https://github.com/infobloxopen/konk/pull/223


Rerun failures and logic onTestFailure doesn't work for the tests with dependsOnMethods

Describe the bug
In case if we have the next structure:
@Test()
public void **test1**()
{
actions();
}

@Test(dependsOnMethods = "**test1**")
public void test2()
{
actions();
}

retry doesn't work properly.
As far as method test1 has dependencies we don't quit drivers (logic from CarinaListener.class)
if (automaticDriversCleanup && !hasDependencies(result)) {
                quitDrivers(Phase.BEFORE_METHOD, Phase.METHOD);
            }

So, we are trying perform rerun failure with already started driver.
Carina version 6.1.20.

PR: https://github.com/qaprosoft/carina/pull/733
Was added check if result == 2 (failure) then quit driver. So, we'll close driver anyway for failure.

fix should be in 6.1.21+
testing is available via 6.1.21-SNAPSHOT core build


Update to<EMAIL_ADDRESS>

Fixes: #15

only differences between what's reviewed are lint fixes: 2812c34daf5c2e9e8f51e298ad8a1429591d5f2d..5130e28fcdd4d7a0ae24780313b6144f21b40110


FanzyZone syncs Zones on multimonitor-Setup

Microsoft PowerToys version
0.60.0
Running as admin

[X] Yes

Area(s) with issue?
FancyZones, FancyZones Editor
Steps to reproduce
I'm having two identical secondary monitors. One in horizoontal and one in vertical orientation. When I set the Zones for one of these monitors the other one automaticaly copies these Zones.
These zones do not get copied onto my main-monitor which is a diffrent brand and modell.
PowerToysReport_2022-07-07-21-01-54.zip
✔ Expected Behavior
Set zones for each monitor individually, so effectively having multiple monitors with each one having a different zone-setup.
❌ Actual Behavior
Zones of two identical monitors get synced with the same zone-setup each.
Other Software
No response

This is also happening with my 3 identical monitors. All three are of the same model number, resolution, orientation, and scaling. The main monitor keeps its zone, but the two other monitors sync their zone regardless of which one is selected when you set the zone.
Alternatively, clicking through the monitors in the FancyZones Editor results in the main-display's zone being selected and overlaid on one or more of the non-primary monitors. Despite the main monitor's zone appearing and selected, it does not override the currently set zone on the non-primary monitor unless you click on the (incorrectly) selected zone in the editor.
Monitor 1:

Monitor 2 (Main Monitor):

Monitor 3:

Prior to installation, the custom layout in the screenshots was applied to monitor 1, the priority grid was applied to monitor 2, and the 6 way grid was applied to monitor 3. After installation, the custom layout was applied to both monitor 1 and 3, and the priority grid was applied to monitor 2.
Changing monitor 3 to use the 6 way grid results in monitor 1 and 3 both being updated to the 6 way grid. Monitor 2 is unaffected. Attempting to open the zone editor on monitor 3 results in it being opened on monitor 1 instead. Selecting monitor 3 in the zone editor results in the priority grid from monitor 2 being overlayed on monitor 3 AND that priority grid is selected in the editor, but if you close the editor that zone layout does not actually get applied to monitor 3.

Also having this issue. Three monitors.
I can only add a custom one to the left or the right. As soon as I change either one, the other one gets reverted and swapped. This is new behavior from latest update.

Same problem here.  3 4K Monitors and after the update it failed to return to normal.  Attempts to reassign layouts to monitors does not appear to be saving properly and it keeps reverting to the previous flawed or random layout selection after the update.
One of the layouts after the update altered a single zone to no longer fit on the monitor and overlap so there appears to be a problem with updating the zones during the update process so there may be errors in the zone saving preventing the layout from being properly saved.  Fixing that particular error did not solve the incorrect mapping of templates to monitors.  The mapping process won't even choose from the 3 selected custom templates and will use one of the normal templates with 2 of the custom templates so none of the monitor selections are saving.

/dup of https://github.com/microsoft/PowerToys/issues/19203
Working on a fix. Thank you.


Double entry in menu

Describe the bug
run predefined custom command entry appears twice in menu
To Reproduce
Steps to reproduce the behavior:

start lzd as container
press x on keyboard to open menu

Expected behaviour
one entry for c is enough
Screenshots

Desktop (please complete the following information):

OS: Linux (Deepin 15.11)
Lazydocker Version v0.7.4

Thanks for reporting this.


Notified in logs to inform you that the H6008s support colorwc

All three of my newly setup H6008s support colorwc.
[2/1/2022, 6:21:46 PM] [Govee] [Door Light] device supports 'colorwc' command, let me know on Github with model number [H6008].
[2/1/2022, 6:21:46 PM] [Govee] [Bed Light] device supports 'colorwc' command, let me know on Github with model number [H6008].
[2/1/2022, 6:21:47 PM] [Govee] [Desk Light] device supports 'colorwc' command, let me know on Github with model number [H6008].

Here I am letting you know :)

Thanks! This message shouldn't be shown anymore in the new version I have just released. Are you able to confirm that colour is still working with the new version?

Looks like color is not working. I tried setting my light to pure red(255,0,0) through my phone/Homebridge and it wasn't changing the color, just was an off white.

Actually after changing colors in Govee, setting the bulbs back to adaptive in homekit, and then changing the colors in homekit it seems to work fine now. Not too sure what was up. I noticed they change quite slowly too, takes around 5-6 seconds for an update to happen.

Can you turn on plugin debug logging, restart the plugin, then use the govee app to change the colour (block colour for the whole strip, nothing else enabled like gradient or segments etc)
and show me the logs that appear in homebridge?

[2/2/2022, 3:21:32 PM] [Govee] [Desk Light] [API] receiving update [{"source":"API","online":true,"powerState":"on","brightness":30,"color":{"r":0,"b":0,"g":255}}].
[2/2/2022, 3:21:47 PM] [Govee] [Desk Light] [API] receiving update [{"source":"API","online":true,"powerState":"on","brightness":30,"color":{"r":0,"b":0,"g":255}}].
[2/2/2022, 3:22:02 PM] [Govee] [Desk Light] [API] receiving update [{"source":"API","online":true,"powerState":"on","brightness":30,"color":{"r":0,"b":0,"g":255}}].
[2/2/2022, 3:22:02 PM] [Govee] [Desk Light] skipping adaptive lighting update as device is off or kelvin is same as before [3300K / 303M].
[2/2/2022, 3:22:17 PM] [Govee] [Desk Light] [API] receiving update [{"source":"API","online":true,"powerState":"on","brightness":30,"color":{"r":0,"b":0,"g":255}}].
[2/2/2022, 3:22:17 PM] [Govee] [Desk Light] current colour [rgb 0 255 0].
[2/2/2022, 3:22:17 PM] [Govee] [Desk Light] adaptive lighting disabled due to significant colour change.
[2/2/2022, 3:22:32 PM] [Govee] [Desk Light] [API] receiving update [{"source":"API","online":true,"powerState":"on","brightness":30,"color":{"r":0,"b":0,"g":255}}].
[2/2/2022, 3:22:47 PM] [Govee] [Desk Light] [API] receiving update [{"source":"API","online":true,"powerState":"on","brightness":30,"color":{"r":0,"b":0,"g":255}}].
[2/2/2022, 3:23:02 PM] [Govee] [Desk Light] [API] receiving update [{"source":"API","online":true,"powerState":"on","brightness":30,"color":{"r":0,"b":0,"g":255}}].

I've isolated the logs for one of my 3 lights in my ceiling. Though they're all synced on the same scheme on Govee's app.

I was hoping to see a
[AWS] receiving update...

log entry when you used the govee app to change the colour of the light. Could you try again? It could be the current AWS command being sent by the plugin isn't working properly and only changing when the API command is sent... this could explain why it's taking a few more seconds longer than you'd expect.
The [AWS] receiving update... will show the correct command needed for this model. The commands can vary between models so in some cases the plugin is using the wrong version.


Make linking tonlib-sys optional (for offline usage)

I'm aware my PR is very rough and this could be done finer, but I've managed to make applicable tests pass.

We split codebase for core&client parts, so core part doesn't depend on tonlib-sys anymore


central discovery, get channel scan happened on

Hi!
Are there any way (even nimble patching) to get channel (37, 38 or 39) I got BLE_GAP_EVENT_DISC with some advert packet?
Or another approach (worse, but still usable): are there any way limit scan procedure on some channel?
P.S. I am using HCI BLE adapter.

Found this issue https://github.com/apache/mynewt-nimble/issues/823
Made some fix, but validation needed


hircd tests sometimes freeze weirdly

https://github.com/Akuli/mantaray/runs/4907006229?check_suite_focus=true

Still happens https://github.com/Akuli/mantaray/runs/5124897653?check_suite_focus=true

Don't think it's just hircd https://github.com/Akuli/mantaray/runs/5139019491?check_suite_focus=true


Touchableopacity not clickable after integrating to existing iOS project

I have an ios project and I use offline bundle to integrate a react native app into my main project. I have a scorllview inside a view with a touchableopacity outside the scrollview. It works perfectly fine by itself. But after I integrate to my main app, the touchableopacity is not clickable. I tried putting the touchableopacity inside each slide it works. Please advice how to fix it. Thanks a lot!

@facebook-github-bot stack-overflow

I recommend creating a simple test case that reproduces your issue, preferably on RNPlay. Then post your question on Stack Overflow, where you'll likely get a solution quickly!

I figured it out myself. Please mark it as closed.


Cameras with multiple overlapping regions: Will it work?

I have a series of photos:
https://drive.google.com/drive/folders/1ZZgZUrFrnP47rx8bN5K6yvYnSC50a-9G?usp=drive_link
Which were take with an iPhone 13 Pro Max
I have used this dataset with Instant NGP from NVIDIA and with Gaussian Splatting to produce a good radiance field.
Do you think this dataset will work with the code in this repository.
My changes are recorded here
diff --git a/data/iphone.py b/data/iphone.py
index 05cf1d5..e34bcc8 100644
--- a/data/iphone.py
+++ b/data/iphone.py
@@ -17,7 +17,7 @@ from util import log,debug
 class Dataset(base.Dataset):

     def __init__(self,opt,split="train",subset=None):
-        self.raw_H,self.raw_W = 1080,1920
+        self.raw_H,self.raw_W = 3024,4032
         super().__init__(opt,split)
         self.root = opt.data.root or "data/iphone"
         self.path = "{}/{}".format(self.root,opt.data.scene)
@@ -62,7 +62,7 @@ class Dataset(base.Dataset):
         return image

     def get_camera(self,opt,idx):
-        self.focal = self.raw_W*4.2/(12.8/2.55)
+        self.focal = self.raw_W*1.6*35.0
         intr = torch.tensor([[self.focal,0,self.raw_W/2],
                              [0,self.focal,self.raw_H/2],
                              [0,0,1]]).float()
diff --git a/options/barf_iphone.yaml b/options/barf_iphone.yaml
index f344c7b..fbfcc38 100644
--- a/options/barf_iphone.yaml
+++ b/options/barf_iphone.yaml
@@ -1,6 +1,19 @@
-_parent_: options/barf_llff.yaml
+_parent_: options/nerf_iphone.yaml

-data:                                                       # data options
-    dataset: iphone                                         # dataset name
-    scene: IMG_0239                                         # scene name
-    image_size: [480,640]                                   # input image sizes [height,width]
+barf_c2f:                                                   # coarse-to-fine scheduling on positional encoding
+
+camera:                                                     # camera options
+    noise:                                                  # synthetic perturbations on the camera poses (Blender only)
+
+optim:                                                      # optimization options
+    lr_pose: 3.e-3                                          # learning rate of camera poses
+    lr_pose_end: 1.e-5                                      # terminal learning rate of camera poses (only used with sched_pose.type=ExponentialLR)
+    sched_pose:                                             # learning rate scheduling options
+        type: ExponentialLR                                 # scheduler (see PyTorch doc)
+        gamma:                                              # decay rate (can be empty if lr_pose_end were specified)
+    warmup_pose:                                            # linear warmup of the pose learning rate (N iterations)
+    test_photo: true                                        # test-time photometric optimization for evaluation
+    test_iter: 100                                          # number of iterations for test-time optimization
+
+visdom:                                                     # Visdom options
+    cam_depth: 0.2                                          # size of visualized cameras
diff --git a/requirements.yaml b/requirements.yaml
index 0baf8b0..2865db4 100644
--- a/requirements.yaml
+++ b/requirements.yaml
@@ -2,6 +2,7 @@ name: barf-env
 channels:
   - conda-forge
   - pytorch
+  - nvidia
 dependencies:
   - numpy
   - scipy
@@ -10,7 +11,8 @@ dependencies:
   - easydict
   - imageio
   - ipdb
-  - pytorch>=1.9.0
+  - pytorch
+  - pytorch-cuda=11.8
   - torchvision
   - tensorboard
   - visdom

and I removed "IMG_" from the file names.
I am training the model now.
Do you have an estimate of how long this might take on a RTX 3090.
What viewer can I use to make renders from the radiance field produced from this training run?
Example image below, EXIF information should be intact:

Sam

Options attached.
options.zip

After 2 hours it had not completed 10 iterations, what am I doing wrong?

@samhodge @samhodge-aiml I'm not super confident whether BARF would work well on your data, as the viewpoint coverage is not as dense as what we had been experimenting before. My estimate of the runtime on a 3090 would be 8-10 hours, but I don't have one to benchmark with so I cannot say for sure (also it has been quite a while since I developed this project). The training shouldn't get stuck at 10 iterations though -- could you share the training log?

that is the thing the GPU was loaded up (RTX 3090, 24 Gb, 98%+ GPU compute, ~20% GPU memory)
But nothing really being logged at all.
I will try running it again and see if I can get something to share with you.
There was no error, no Tensorboard logs to speak of, but a file in the output directory, so write permission was OK, I turned off visdom
Let me give you everything I have so far and we can get to the bottom of it.
Thanks a million for the response.

One hour later no progress, I will leave it running overnight and see if anything happens

It has been running for over 10 hours now and now progress, I am going to save the electricity.

This shouldn't happen. Could you help pinpoint which line it hangs at?

Could it be that the focal length for the camera is causing an unsolvable matrix?

related? https://github.com/chenhsuanlin/bundle-adjusting-NeRF/issues/76#issue-1716458063

Might try here tomorrow https://camp-nerf.github.io/

Yes, it is likely stuck in the loop as in #76. If you use batch size 1 the issue will likely go away -- I have not been able to figure out exactly where the bug was. CamP should be a quite decent improvement over BARF in joint camera optimization. I would definitely encourage you to try it out if they have the code released.

no code yet, batch size of one it is

Batch size of one didn't seem to work for me either.

Hi,
while I was working with this codebase I have faced similar issue (training stuck in endless loop). It has turned out that during sampling along the ray, there was exponential (kind of) grow in depth for last few samples with the last ones as big as few thousends (or even 10000 on one occasion). It caused gradients to explode during backward propagation and some of parameters became NaN's, hence calculeted rays got NaN values in them. I wasn't able to pinpoint specific error in implementation. Bare in mind that I was experimenting on heavily modified architecture so I encurage you to check for abnormal values, details in doc.
There are number of strategies to deal with this problem (assuming that gradient explosion is what causing it), the simplest is to clip abnormal samples, which is very fast workaround. This can affect the results, but erroneous samples make up a very small proportion of the total training data, so it shouldn't be too bad.

Thanks a million maybe tomorrow I can eek out a little time to see if I can make this into a PR
The information is very generous but I am not sure if my skills are ready right now to debug and patch the issue, but why die wondering right? I will see what I can do

@SwirtaB thanks for the feedback! I hadn't been able to deterministically reproduce this issue, and did not realize it had to do with the sampled coordinates. In this case, this line is likely the culprit, where the depth of the last sample is set to a very large number (1e10). @samhodge if you find that tweaking the code to lower it to e.g. 1e3 would help, please let me know and I'm happy to make a hotfix.

Yeah I can certainly write a smoothstep function to roll it off to a limit.
https://en.wikipedia.org/wiki/Smoothstep

Trying this
diff --git a/model/nerf.py b/model/nerf.py
index b0dcb2c..eefef60 100644
--- a/model/nerf.py
+++ b/model/nerf.py
@@ -393,7 +393,7 @@ class NeRF(torch.nn.Module):
         ray_length = ray.norm(dim=-1,keepdim=True) # [B,HW,1]
         # volume rendering: compute probability (using quadrature)
         depth_intv_samples = depth_samples[...,1:,0]-depth_samples[...,:-1,0] # [B,HW,N-1]
-        depth_intv_samples = torch.cat([depth_intv_samples,torch.empty_like(depth_intv_samples[...,:1]).fill_(1e10)],dim=2) # [B,HW,N]
+        depth_intv_samples = torch.cat([depth_intv_samples,torch.empty_like(depth_intv_samples[...,:1]).fill_(1e3)],dim=2) # [B,HW,N]
         dist_samples = depth_intv_samples*ray_length # [B,HW,N]
         sigma_delta = density_samples*dist_samples # [B,HW,N]
         alpha = 1-(-sigma_delta).exp_() # [B,HW,N]

I have another idea, that one didn't work:
https://numpy.org/doc/stable/reference/generated/numpy.heaviside.html

That didn't work.

Other things that do not work
diff --git a/data/iphone.py b/data/iphone.py
index 05cf1d5..e34bcc8 100644
--- a/data/iphone.py
+++ b/data/iphone.py
@@ -17,7 +17,7 @@ from util import log,debug
 class Dataset(base.Dataset):

     def __init__(self,opt,split="train",subset=None):
-        self.raw_H,self.raw_W = 1080,1920
+        self.raw_H,self.raw_W = 3024,4032
         super().__init__(opt,split)
         self.root = opt.data.root or "data/iphone"
         self.path = "{}/{}".format(self.root,opt.data.scene)
@@ -62,7 +62,7 @@ class Dataset(base.Dataset):
         return image

     def get_camera(self,opt,idx):
-        self.focal = self.raw_W*4.2/(12.8/2.55)
+        self.focal = self.raw_W*1.6*35.0
         intr = torch.tensor([[self.focal,0,self.raw_W/2],
                              [0,self.focal,self.raw_H/2],
                              [0,0,1]]).float()
diff --git a/model/nerf.py b/model/nerf.py
index b0dcb2c..9a02e77 100644
--- a/model/nerf.py
+++ b/model/nerf.py
@@ -391,9 +391,11 @@ class NeRF(torch.nn.Module):

     def composite(self,opt,ray,rgb_samples,density_samples,depth_samples):
         ray_length = ray.norm(dim=-1,keepdim=True) # [B,HW,1]
+        ray_length = numpy.clip(ray_length, 0, 1e3)
+
         # volume rendering: compute probability (using quadrature)
         depth_intv_samples = depth_samples[...,1:,0]-depth_samples[...,:-1,0] # [B,HW,N-1]
-        depth_intv_samples = torch.cat([depth_intv_samples,torch.empty_like(depth_intv_samples[...,:1]).fill_(1e10)],dim=2) # [B,HW,N]
+        depth_intv_samples = torch.cat([depth_intv_samples,torch.empty_like(depth_intv_samples[...,:1]).fill_(1e3)],dim=2) # [B,HW,N]
         dist_samples = depth_intv_samples*ray_length # [B,HW,N]
         sigma_delta = density_samples*dist_samples # [B,HW,N]
         alpha = 1-(-sigma_delta).exp_() # [B,HW,N]
diff --git a/options/barf_iphone.yaml b/options/barf_iphone.yaml
index f344c7b..d58794b 100644
--- a/options/barf_iphone.yaml
+++ b/options/barf_iphone.yaml
@@ -2,5 +2,7 @@ _parent_: options/barf_llff.yaml

 data:                                                       # data options
     dataset: iphone                                         # dataset name
-    scene: IMG_0239                                         # scene name
+    scene: bakerst                                         # scene name
     image_size: [480,640]                                   # input image sizes [height,width]
+max_iter: 10
+batch_size: 1
diff --git a/requirements.yaml b/requirements.yaml
index 0baf8b0..2865db4 100644
--- a/requirements.yaml
+++ b/requirements.yaml
@@ -2,6 +2,7 @@ name: barf-env
 channels:
   - conda-forge
   - pytorch
+  - nvidia
 dependencies:
   - numpy
   - scipy
@@ -10,7 +11,8 @@ dependencies:
   - easydict
   - imageio
   - ipdb
-  - pytorch>=1.9.0
+  - pytorch
+  - pytorch-cuda=11.8
   - torchvision
   - tensorboard
   - visdom

@chenhsuanlin no problem. I have gave your suggestion a try and it only delayed the problem for me, training have hang much later. Then I have cross checked your implementation of composit with NeRF article and their official implementation and I'm puzzled why you calculate T here. By my understanding whole equation 3 from article reduces to alpha composition. Also in their implementation they calculate it slightly different (original impl), so I gave it a try. I have commented T calculation and calculate prob as:
prob = (alpha * torch.cumprod(1.0 - alpha + 1e-10, dim=2))[..., None]
Unfortunately that didn't solve the problem, only delayed it again, so root cause is still unknown for me. That being said any workaround that ensures proper samples values (either by clipping or something else) works quite well. Maybe that is proper solution, since NeRF's are still neural networks and improper inputs could leads to all sorts of problems.


Update the doc and test case for core

Description
Improved and fixed the doc.
Two tests added:

test_store_eviction_policy
test_num_programs to test the function
https://github.com/triton-lang/triton/blob/main/python/triton/language/core.py#L1161

Currently, there is no test for this function. Although it is a simple function, no harm in adding a unit test.
Template
The core Triton is a small number of people, and we receive many PRs (thank
you!).  To help us review your code more quickly, if you are a new
contributor (less than 3 PRs merged) we ask that you complete the following
tasks and include the filled-out checklist in your PR description.
Complete the following tasks before sending your PR, and replace [ ] with
[x] to indicate you have done them.

[x] I am not making a trivial change, such as fixing a typo in a comment.

[x] I have written a PR description following these
rules.

[ ] I have used an LLM to copyedit my PR description and and code comments.

[x] I have run pre-commit run --from-ref origin/main --to-ref HEAD.

Select one of the following.

[x] I have added tests.

/test for lit tests
/unittest for C++ tests
/python/test for end-to-end tests

[ ] This PR does not need a test because FILL THIS IN.

Select one of the following.

[x] I have not added any lit tests.
[ ] The lit tests I have added are "minimal" -- they contain only the
instructions necessary to exercise the bug.  (Usually running Python code
and using the instructions it generates is not minimal.)

@jlebar can this get a merge? Thanks, feel free to lmk if there are things that need to be addressed 👍

I think I can't merge this until @Jokeren approves


Any new releases coming soon?

I am starting to see the double, and sometimes quadruple execution of tasks error more and more.  Hoping there is a release coming that addresses some of the open and closed issues?

When did you start seeing this issue? It has been happening to me for almost a week. All the tasks are being executed twice

I am starting to see the double, and sometimes quadruple execution of tasks error more and more. Hoping there is a release coming that addresses some of the open and closed issues?

When did you start seeing this issue? It has been happening to me for almost a week. All the tasks are being executed twice

I am starting to see the double, and sometimes quadruple execution of tasks error more and more. Hoping there is a release coming that addresses some of the open and closed issues?

https://github.com/node-cron/node-cron/issues/120

@TDola are you seeing issues mostly with 1 minute interval scheduled jobs? I will probably have 1 hour minimum schedule job use case on my end... I'm curious if I will also start to face issues if my app scales up in the near future while using node-cron.
I second this post! Any updates on new release? I think node-cron is an awesome library but seems to have some bugs here and there. I'm considering going with Bull.js but i'm still not 100% sure as it just seems like over-kill for my needs.

@jon-malave The bug mostly happens when the computer hardware is maxed out.  I mitigated the issue somewhat by setting the process priority in Windows to maximum.  I have no idea if it happens in other environments.
I am also not certain if it's truly this plugins fault.  This plugin is ultimately relying on setInterval(yourTask,1000) to repeatedly check to see if the time you provided for your task is now.  This interval get's busy and skips a few, lumping them all together in 1 bulk.
You can cover for some of the issues by debouncing your own task, and store your tasks as an array of objects and functions.  And set a isRunning style object property that is subsequently checked.  This way if your task is accidentally run twice while the first task is waiting on an async operation to complete, you can cancel it.  Also, if you have a lot of tasks running at the same time.  Collect them up into an array of runnable tasks then execute them, that way node-cron has enough time to call all your tasks before you start any blocking code.

Hey, I'm really sorry about my almost no time dedicated to node-cron, I've released the version 3.0.0 few days ago.


feat(ci): add linter, hound and lint files

add tslint with dedicated rules
add hound for code reviews
lint files 1 time to start clean

Dependent on #9 and #8
Also, changed these repo settings to require hound status to pass in order to merge.
Did this to both protected branches (master & next)

:tada: This PR is included in version 1.0.0-next.6 :tada:
The release is available on:

GitHub release
npm package (@next dist-tag)

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0-next.1 :tada:
The release is available on:

GitHub release
npm package (@next dist-tag)

Your semantic-release bot :package::rocket:

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket: