Attach policy to specified users allowing assumption of the Terraform state read-write role

🗣 Description
This pull request attaches a policy to specified users allowing assumption of the Terraform state read-write role.  It also adds:

Instructions for operators to help them initially deploy such an environment and grant specific users the ability to redeploy it.
Instructions for users to help them to set up locally and then redeploy an environment.

See also cisagov/cool-assessment-provisioner-iam#30.
💭 Motivation and context
It is better to attach the policy in Terraform instead of requiring operations to do so manually in the AWS console.
🧪 Testing
All automated tests pass.  I deployed these changes to env6 in our COOL staging environment and verified that they function as expected.
✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated to reflect the changes in this PR.
[x] All new and existing tests pass.

After further reflection, I think that the two markdown files that you added don't belong in this repo.  The info from those files should be passed to our Ops team and users so that it can be stored wherever they keep their various runbooks.

After further reflection, I think that the two markdown files that you added don't belong in this repo. The info from those files should be passed to our Ops team and users so that it can be stored wherever they keep their various runbooks.

Those commits have been removed.


Added Bitcoin to ISK endpoint

Added Bitcoin to ISK endpoint with data from Myntkaup.is.
Checklist

[X] Write tests
[X] Write documentation

Will this be merged anytime soon?


How to use Confirm widget

I do not understand how to return the selected value when i use a confirmWidget.
Thank you for your support

OK, solved. Just disable the button (if the new widget is a button)


Add more details about modern feature being used

Distinguishing the different signature now relies on the EventTarget constructor to check the instance. While WeakMap and Element.closest() can be polyfilled in IE 11, HTML elements being instance of the EventTarget constructor cannot, so this is the biggest blocker for compatibility.

I initially wanted to use this fork (I already have polyfills loaded in IE for WeakMap and Element.closest, and I could easily make babel transpile this package code in my build) to benefit from the bugfixes (in particular https://github.com/bfred-it/delegate-it/commit/c6bb88c2aa8097b25f22993a237cf09c96bcbfb8).
But things did not went well because of this usage of EventTarget, which cannot be polyfilled. So I think it is good to mention it.

Ouch, thanks for the mention. I'm not very happy about supporting IE11 but it it means a few more characters we can do that.
Do you know what instanceof EventTarget can be replaced with?

well, probably reverting #4 to keep detecting the presence of the addEventListener function instead (IE 11 implements the EventTarget methods on DOM nodes, but does not expose it as its own constructor as that's more recent in the spec).

Note that if #4 was reverted, the support for IE 11 (and older Edge) could be changed to "supported if you load Weakmap and closest polyfills and transpile the code to ES5). And the ES5 part may indeed be done by shipping a separate transpile file in the npm package if you want to make it even simpler.
For project target corporate environments, IE 11 is unfortunately far from being dead (my stats are making it the most used desktop browser, with Chrome being very close behind). And the original delegate package seems unmaintained so I would be very happy if I could use the debugged version (having to load polyfills explicitly is fine for me, as I already does it, with some browser detection to skip them in modern browsers).

I think that the support text can stay as is, also because I can't guarantee that IE compatibility will stay, I don't have automated tests for that.
However you're welcome to send a PR to undo #4 to get the blame for it 😊


JF Edits

Introductory copy update.

@gadgetb0y probably meant to send this to another branch?

Doh! Apologies.

No worries! The interface is confusing.


Unable to resolve correctly for node_moduels soft links on Windows

Describe the bug
→ Failed to load plugin 'mdx' declared in 'BaseConfig': Cannot find module 'D:\a\eslint-mdx\eslint-mdx\node_modules\eslint-plugin-mdx\lib\index.js'. Please verify that the package.json has a valid "main" entry

Reproduction
https://github.com/mdx-js/eslint-mdx/actions/runs/7378053284/job/20072722711?pr=508
https://github.com/mdx-js/eslint-mdx/tree/build/yarn_v4
System Info
Windows on CI
local:
  System:
    OS: macOS 14.2.1
    CPU: (10) arm64 Apple M1 Max
    Memory: 12.77 GB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.10.0 - ~/Library/Caches/fnm_multishells/89034_1703864877615/bin/node
    Yarn: 4.0.2 - ~/Library/Caches/fnm_multishells/89034_1703864877615/bin/yarn
    npm: 10.2.3 - ~/Library/Caches/fnm_multishells/89034_1703864877615/bin/npm
    bun: 1.0.20 - /opt/homebrew/bin/bun
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 120.0.6099.129
    Safari: 17.2.1
  npmPackages:
    @vitest/coverage-istanbul: ^1.1.1 => 1.1.1
    vitest: ^1.1.1 => 1.1.1

Used Package Manager
yarn
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

I don't understand what is the problem here. From what I can see the error happens in @eslint/eslintrc/lib/shared/relative-module-resolver.js, why is this a Vitest error?

From what I can see it uses require:
https://github.com/coderaiser/eslintrc/blob/99adbac4fec25a50e466946833677351eb8794b9/lib/shared/relative-module-resolver.js#L23
Vitest never supported and will never support intercepting require.

Well, the error only occurs on Windows.

why is this a Vitest error?

I'm migrating from jest, there is no such issue when using jest.


⚠ Mobitopup has degraded performance

In 90d31b7, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3112 ms

Resolved: Mobitopup performance has improved in 4d7536c.


[Tooltip] Fix useMemo dependency

deepmerge supports null and undefined source,
no need to default to an empty object.
This is a follow-up to #19851.
That being said, if PopperProps is defined inline in user land, it would bust the cache all the time, so not sure how useful useMemo is. Unless we expect users to also memoise PopperProps 🤷♂

[x] I have followed (at least) the PR section of the contributing guide.

I doubt we can write a test for this, unfortunately.
https://github.com/mui-org/material-ui/blob/b9dec2780203e37148d87788f727c668d9019581/packages/material-ui/src/Tooltip/Tooltip.test.js#L509

I doubt we can write a test for this, unfortunately.

Strictly speaking, I could with mocks but I’m not sure it’s worth the effort? Actually I’m not sure how mocking is done in this environment, but it’s relatively easy with Jest 🙂

Thanks for the patch


ethtool: don't report -1 for speed in ethtoolLink()

the same way we don't in the legacy part of the interface.
Closes: #7382.
Signed-off-by: Christian Brauner<EMAIL_ADDRESS>

Got some test failures here, although not sure if related to the PR.

On Mon, May 18, 2020 at 11:18:32PM -0700, TomP wrote:

Got some test failures here, although not sure if related to the PR.

We can retrigger but that seems unlikely.
(
Btw, we really should somehow always dump the log when a container
fails to start, i.e.
EROR[05-19|05:47:38] Failed starting container                project=default name=baz action=start created=2020-05-19T05:47:36+0000 ephemeral=false used=1970-01-01T00:00:00+0000 stateful=false
Error: Failed to run: /lxc-ci/build/tmp.elUCHnBcua/go/bin/lxd forkstart baz /lxc-ci/build/tmp.elUCHnBcua/go/src/github.com/lxc/lxd/test/tmp.aEx/JHZ/containers /lxc-ci/build/tmp.elUCHnBcua/go/src/github.com/lxc/lxd/test/tmp.aEx/JHZ/logs/baz/lxc.conf:
Try lxc info --show-log local:baz for more info
should immediately result in:
lxc info --show-log local:baz
That make it easier to debug without combing through the tar.gz file
that Jenkins produces for the build.
)
being called

jenkins: test this please

@brauner yes that would be useful to see the log if lxc start fails.


Is that normal?

The command to trash starred has never been in there, unless am mistaken.
Also sorry for the late reply, I am very busy.


Fix prerequisites doc page for content is partly occluded by navigator

Type of change

Bug fix

Description

The change is intended to fix #4745 that the prerequisites page's content is partly occluded by navigator. As analyzed in the discussion of the issue, embedded div elements cause the issue. Therefore the change is going to remove all <div> elements
Additional details

Related issues
#4745

@Mergifyio backport release-2.5

https://github.com/Mergifyio backport release-2.5


Allow specifying type definition files from .luaurc

Implements #418
Right now, parsing the config checks the file existence, but this won't always work since the frontend file resolver might not work like the standard CLIFileResolver.

Thanks for the contribution! Some questions...

Can more than one type definition file be loaded?
Are type definition files loaded relative to the luaurc file?
Can we have a unit test that ensures the decorations are in scope?

There's a few caveats on the mechanical part of the change besides the above, most notably that Config code should not perform file system access, and Luau should never use anything from std::filesystem as this causes compat issues on earlier versions on macOS. We'd need to have that addressed -- however, the other issue is that we actually have not stabilized the type definition syntax yet, it only exists as an implementation detail.
As such I think we'd need an RFC that specifies the type definitions fully before being able to merge any externally visible support for type definitions, whether it's via command line or luaurc. cc @AmaranthineCodices for additional comments/concerns if any, as it's been a while since we talked about type defns internally.

Are type definition files loaded relative to the luaurc file?
This would require a more substantial implementation, the file path isn't currently carried across the config.

most notably that Config code should not perform file system access, and Luau should never use anything from std::filesystem as this causes compat issues on earlier versions on macOS.
Will find another way tomorrow.

the other issue is that we actually have not stabilized the type definition syntax yet, it only exists as an implementation detail.
We don't have to support the type definition syntax for now. Only allowing type exports should be fine.

I'm unsure how we should tell the user about missing global types files. Either tell them in the config or do it while loading the module types. The former isn't viable and I'm not sure where the error should be for the latter. Do the maintainers have an opinion on this?

Anymore questions or concerns on this? @asajeffrey @zeux

I think I'm going to rename this feature to just typePaths instead of globalTypePaths. This is because globalTypePaths is misleading since the module will use the type paths of the nearest config.

I think I will close this for now. Maybe I will open it again in the future when it gets more attention.


Regressions in 0.3.0

Hello,
I'm using this great crate. Thank you so much :)
I found some regressions in version 0.3.0.
Changing the name property of AseSpriteSlice doesn't work.
I think the query for update_aseprite_sprite_slice should be as follows:
mut nodes: Query<
(Entity, &mut Sprite, &AseSpriteSlice),
Or<(Without<FullyLoaded>, Changed<AseSpriteSlice>)>,
>,

The same goes for update_aseprite_ui_slice.
Related: #8
It's a different issue, but I had to add the bevy_winit and bevy_window features to run the example.

Thx for the issue. This is a core function, I consider to have very high prio!
I will investigate and push an update later today.

I implemented your change and added the functionality to the examples (UI & sprites).
You can now cargo update to version 0.3.1!
(also made the dev features just a dev-dependency. The examples run by default again)

@Lommix
I have confirmed that it works correctly.
Impressively quick work, thank you!


A union function for features

Requesting for a union function for features.

Are you talking about geometries? You may want to try turf.js or JSTS. Also check the official OpenLayers examples to see how to use them on features.

Closing due to lack of activity.


shaders wont work

Describe your bug here.
i tried adding a glitch shader to a song by putting the script.lua in the song data folder it doesnt work when i open the song
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
yes i added a week and tried adding shaders (obviously)

I tried this myself and it worked fine so there's need for more info

i think it needs to be on the lua background script

or if it still doesnt work try this compiled version someone made and put it on the lua background script

instead of just "game", do camGame for the game, and do camHUD for the hud

doesnt work

increase the value. maybe it's there and you can't see it

nah it just doesnt work

i increased it to 0.500 and still no

i think it needs to be on the lua background script

i dont have a bg yet, im porting whitty to psych engine and idk how to add the animated ballistic bg
thats what song im adding the shader to

are you using 0.5.1 or a build from source

0.5.1 source code

you can't use shaders then

No the build with shaders

so it's not 0.5.1, then you have to be doing something wrong because it works jsut fine

Only works in the latest builds.


link documentation from README or elsewhere

There is a very nice ReadtheDocs documentation page availble for this package, but as far as I can tell, it is not linked anywhere in the README file or the tutorials. It should be linked so that it is discoverable by users.

Thank you for pointing that out! I’ve made the following updates:
ReadTheDocs Link in README: I have ensured that the link to the ReadTheDocs documentation page is prominently included in the README file, under the documentation section, making it easily accessible for users visiting the GitHub repository.

Thanks @rodoulak


WAF installation problems

when i try to install waf it stays like this :

when i try to install waf.sh manually it says :
" error: Need HAProxy version 1.8 or later X "

Can you write messages with full description.... I think you know my next question.... Which version haproxy do you use?

Haproxy version :
HA-Proxy version 2.0.11 2019/12/11 - https://haproxy.org/
( ive installed this one via web interface using ansible )
also the installation of HA have been successfully done
but waf wont install

Haproxy version :
HA-Proxy version 2.0.11 2019/12/11 - https://haproxy.org/
( ive installed this one via web interface using ansible )
also the installation of KeepAlived have been successfully done
but waf wont install :/

Go to the server where you try install to the /tmp and run:
./waf.sh VERSION=2.0.11 HAPROXY_PATH=/etc/haproxy/

You're the GOAT bro !

but why from the overview page i cant see WAF is activated !!!

It's because of you run installation manually and there aren't needs records in DB. You have to create property records like this:
sqlite> insert into waf values('server_id','0');

its so weirdo !
[root@mastserv app]# sqlite3
SQLite version 3.7.17 2013-05-20 00:56:22
Enter ".help" for instructions
Enter SQL statements terminated with a ";"
sqlite> insert into waf values('mastserv','0');
Error: no such table: waf
sqlite>

after executing your command, Metrics master and Metrics workers sometimes in Red sometimes in Green !!!
and no WAF apperance in overview as you can see in the picture :(

You have to always look logs before something write, I have no idea what there happens. Also show results after your actions it's a good idea :)

Thanks for your comprehension,
this is the metrics-log :

i dont know how to get the WAF errors or why he didnt show up in the overview page.

First of all show me what exactly you added in the db

Sorry for the late,
this is what ive addded :
sqlite> insert into waf values('masterServer','0');

Sorry for the late,
$ cd /var/www/haproxy-wi/app
then : sqlite3 haproxy-wi.db
then :
sqlite> insert into waf values('masterServer','0');

I sad the server's id, not name... It's must be 1and 2 for example

Done <3


Remove string_to_bytes(), generate_signature()

Looks like they were something used with korben.

Codecov Report

Merging #349 into develop will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop    #349      +/-   ##
==========================================
- Coverage    95.42%   95.4%   -0.02%
==========================================
  Files           67      67
  Lines         2493    2483      -10
  Branches       207     206       -1
==========================================
- Hits          2379    2369      -10
  Misses          84      84
  Partials        30      30

Impacted Files
Coverage Δ

datahub/core/utils.py
75% <ø> (-5%)
:arrow_down:

Continue to review full report at Codecov.

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


auto complete of recipients broken?

Expected behavior
When i try to enter a recipients address k9 mail stops completion one character after the @ sign.
Example: I wanna sent an email to<EMAIL_ADDRESS>I would expect to be able to type this email address.
Actual behavior
Auto completion seems to catch ht@m. After that i cannot type any longer.
Steps to reproduce

Enter in the to field<EMAIL_ADDRESS>auto completion jumps in at hr@m
no further input possible

Environment
K-9 Mail version: 5.403 / 5.503
Android version: 8.1
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

Uh i should mention this error only occurs on email addresses which are not in the carddav address book. auto completion for existing entries work fine.

Duplicate of #2829


Why are the action and observation always out of sync? [Question]

Question ❓
when i try the bacic_example.ipynp, i found that actions do not take effect immediately, but rather after two steps. Why is that?
Additional context
Add any other context about the question here.
Checklist

[x] I have read the documentation (required)
[x] I have checked that there is no similar issue in the repo (required)

:pencil: Please, don't forget to include more labels besides question if it is necessary.

Hi @jingjijiji !
This is due to the functioning of the EnergyPlus API which is what Sinergym uses as middleware. It has a delay from the time the action is sent until the change occurs in the building, usually two steps as you mentioned. At the moment, I have no more information about it.


[Feature] don't track outbound type dns

Verification Steps

[X] I have read the documentation and confirmed that this feature is not implemented
[X] I have searched for the feature request I want to propose in the Issue Tracker and did not find it
[X] I am a non-Chinese user.

Description
When using tproxy with outbound type DNS please don't put it in the connections list. It flooded the connections list and stay there for a while. XD dashboard was lagging a lot, only yacd can handle it.
Thanks.
proxies:
  - name: 'dns-hijack'
    type: dns

rules:
  - DST-PORT,53,dns-hijack

How about this?
diff --git a/constant/metadata.go b/constant/metadata.go
index 7b73c21a..e0aafdb9 100644
--- a/constant/metadata.go
+++ b/constant/metadata.go
@@ -157,6 +157,8 @@ type Metadata struct {
 	RawDstAddr net.Addr `json:"-"`
 	// Only domain rule
 	SniffHost string `json:"sniffHost"`
+
+	ProxyType AdapterType
 }

 func (m *Metadata) RemoteAddress() string {
diff --git a/tunnel/statistic/manager.go b/tunnel/statistic/manager.go
index 0b309299..3a33ba8f 100644
--- a/tunnel/statistic/manager.go
+++ b/tunnel/statistic/manager.go
@@ -5,6 +5,7 @@ import (
 	"time"

 	"github.com/metacubex/mihomo/common/atomic"
+	C "github.com/metacubex/mihomo/constant"

 	"github.com/puzpuzpuz/xsync/v3"
 	"github.com/shirou/gopsutil/v3/process"
@@ -82,7 +83,9 @@ func (m *Manager) Memory() uint64 {
 func (m *Manager) Snapshot() *Snapshot {
 	var connections []*TrackerInfo
 	m.Range(func(c Tracker) bool {
-		connections = append(connections, c.Info())
+		if c.Info().Metadata.ProxyType != C.Dns {
+			connections = append(connections, c.Info())
+		}
 		return true
 	})
 	return &Snapshot{
diff --git a/tunnel/tunnel.go b/tunnel/tunnel.go
index c379846b..9591ef8a 100644
--- a/tunnel/tunnel.go
+++ b/tunnel/tunnel.go
@@ -409,6 +409,7 @@ func handleUDPConn(packet C.PacketAdapter) {
 			}
 			logMetadata(metadata, rule, rawPc)

+			metadata.ProxyType = proxy.Type()
 			pc := statistic.NewUDPTracker(rawPc, statistic.DefaultManager, metadata, rule, 0, 0, true)

 			if rawPc.Chains().Last() == "REJECT-DROP" {
@@ -547,6 +548,7 @@ func handleTCPConn(connCtx C.ConnContext) {
 	}
 	logMetadata(metadata, rule, remoteConn)

+	metadata.ProxyType = proxy.Type()
 	remoteConn = statistic.NewTCPTracker(remoteConn, statistic.DefaultManager, metadata, rule, 0, int64(peekLen), true)
 	defer func(remoteConn C.Conn) {
 		_ = remoteConn.Close()


Question: Can arrow be used to load a parquet file from an Azure blob store?

I'm researching the functionality of opening a parquet file stored in an Azure blob store from a Jupyter notebook using a Python 3 kernel.
Specifically, I do not want a PySpark kernel.  I can already do that using the following code:
salesData =<EMAIL_ADDRESS>The question is, can arrow be used to somehow connect to a blob store from pure python code:
import pandas as pd
<EMAIL_ADDRESS>Results in:
ArrowIOError                              Traceback (most recent call last) ... ... ... ArrowIOError: Failed to open local file<EMAIL_ADDRESS>, error: No such file or directory

When you want to use the storefact example of @xhochy. You don't need the buffered reader because the store.open method return an IOInterface can be used by the pyarrow read_table method.
from storefact import get_store
import pyarrow as pa
import pyarrow.parquet as pq

params = {
    'account_name': 'test',
    'account_key': 'XXXsome_azure_account_keyXXX',
    'container': 'my-azure-container',
   'create_if_missing': False
}
store = get_store('azure', **params)

reader = store.open(key)
table = pq.read_table(reader, columns=columns).to_pandas()

Can we read a list of parquet files under subdirectory of a container in Azure blob storage? For example, under container "test_container", there are a list of files under subdirectory of "test.parquet/department=HR/date=2020-01-01/"?
Besides that, I had tested the performance on reading a parquet file of 298MB on Azure blob storage from my local machine, I am selecting 3 columns to return. It took about 2 minutes to return the result. Can this be improve to less than one minute?
from storefact import get_store
import pyarrow as pa
import pyarrow.parquet as pq
params = {
'account_name': 'test',
'account_key': 'XXXsome_azure_account_keyXXX',
'container': 'my-azure-container',
'create_if_missing': False
}
store = get_store('azure', **params)
reader = store.open(key)
table = pq.read_table(reader, columns=columns).to_pandas()
print(table.head(10))


Add streamlined CardCallout component

Checklist before requesting a review

[x] Working on iOS
[x] Working on Android
[x] Integration tests added
[x] Visual regressions screenshots are up to date
[ ] Design validated

@kimak Could you update the Android screenshot because I think with the Typo fix you merged, it will impact the Card descriptions

@kimak Could you update the Android screenshot because I think with the Typo fix you merged, it will impact the Card descriptions

done

@judastheo I don't see an updated iOS screenshot? Is it possible to regenerate please?


EDA: We want to be sure, that we have clean, fixed data to not have bad performance in the models based on the input data.

Continuing to what @Kestener said. I would also like to do a little EDA myself. Any suggestions on how we separate this task ? Or more than one person does it, but with a certain personal deadline so that we don't lose time ?


Integration tests are broken

test -z "$SECRET_PASSWORD" || (cd $SRCDIR/integration; . ./gce.sh hosts; ./setup.sh)
Activated service account credentials for<EMAIL_ADDRESS>export SSH=ssh
export HOSTS="host1-679-0.us-central1-a.scope-integration-tests host2-679-0.us-central1-a.scope-integration-tests host3-679-0.us-central1-a.scope-integration-tests "
export WEAVE_DOCKER_ARGS="--add-host=host1-679-0.us-central1-a.scope-integration-tests:<IP_ADDRESS> --add-host=host2-679-0.us-central1-a.scope-integration-tests:<IP_ADDRESS> --add-host=host3-679-0.us-central1-a.scope-integration-tests:<IP_ADDRESS> "
Copying scope images and scripts to hosts

FATA[0032] Post http://<IP_ADDRESS>:2375/v1.18/images/load: dial tcp <IP_ADDRESS>:2375: i/o timeout. Are you trying to connect to a TLS-enabled daemon without TLS? test -z "$SECRET_PASSWORD" || (cd $SRCDIR/integration; . ./gce.sh hosts; ./setup.sh) returned exit code 1

Fixed by ad1ffa87d35d37d32ea6a046e4eef625275e3040


feat(cli): support deleting the tag in "gas tag" command

Pull Request Test Coverage Report for Build 882245412

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

Totals

Change from base Build 882154546:
0.0%

Covered Lines:
4300

Relevant Lines:
5267

💛  - Coveralls


SOLR wiht basic auth

Is there a way to enter the user and password for the solr connection?

any luck?

Finally was resolved by a colleague, let me ask him.

i solved it : "admin:password@https://localhostToSolar"
but go new issue
requests.exceptions.InvalidSchema: No connection adapters were found for 'admin@pass@http://localhost:8983/solr/new_core44/admin/luke?show=schema&wt=json'
any idea about this?


Link to sequenza in README.md is wrong

It should be https://bitbucket.org/sequenzatools/sequenza (no underscore).

You can find the sequenza here.
https://cran.r-project.org/web/packages/sequenza/vignettes/sequenza.html


Match atomic number in amino acid MCSS

Heavy Degree alone was insufficient to prevent CB from becoming HA

Good catch!

Travis failures have been fixed in https://github.com/choderalab/perses/pull/236

#236 fixes the test failures due to conda build changes.
@juliebehr : Please git fetch origin ; git merge origin/master or rebase and see how your tests do.


Single [indent] not working properly in RichTextLabel

Godot version
v4.0.beta.custom_build [61021c08f]
System information
Kubuntu 22.04
Issue description
A single [indent] (or push_indent) is handled incorrectly (indented only after \n) if there is another open Item before it. However, if [indent] is the first one, or there are multiple [indent], then the indentation is correct.
In particular, this bug can be seen in Editor Help:

Steps to reproduce

BBCode
Result
Comment

[indent]123
456

Correct.

[u][indent]123
456

The bug.

[indent][u]123
456

Correct.

[u][indent][indent]123
456

If there is more than one [indent], then it is correct.

[u][indent]
123
456

000[u][indent]123
456

Is this also a bug? What should be in this case?

[u][indent][u]123
456

Minimal reproduction project
No response

Can you reproduce this in 3.5 (which uses an entirely different RichTextLabel implementation)?

Can you reproduce this in 3.5 (which uses an entirely different RichTextLabel implementation)?

In 3.5, this bug is not reproduced, regardless of the order of [u] and [indent]. The penultimate case (adding a line break before [indent]) works the same as in 4.0-dev.

The bug is still reproducible in d2699dc7a, but in Editor Help it is no longer visible. Probably something has changed recently.


How to create Sagemaker studio project?

P.S. Don't attach files. Please, prefer add code snippets directly in the message body.
I am trying to create sagemaker project following below steps:
create domain (using the example)
create user (using the example)
create jupyter app
create project
Code for creating jupyter app:
`class SagemakerStudioJupyterAppConstruct(Construct):
def __init__(self, scope: Construct,
             construct_id: str, *,
             app_name: str,
             app_type: str,
             domain_id: str,
             user_profile_name: str,
             depends_on=None, **kwargs) -> None:
    super().__init__(scope, construct_id)

    sagemaker_jupyter_app = sg.CfnApp(self, construct_id,
                                      app_name=app_name,
                                      app_type=app_type,
                                      domain_id=domain_id,
                                      user_profile_name=user_profile_name
                                      )
    sagemaker_jupyter_app.add_depends_on(depends_on_user_creation)`

Code for creating project:
`class SagemakerStudioProjectConstruct(Construct):
def __init__(self, scope: Construct,
             construct_id: str, *,
             project_name: str,
             project_description: str,
             product_id: str,
             depends_on=None,
             **kwargs) -> None:
    super().__init__(scope, construct_id)

    sagemaker_studio_project = sg.CfnProject(self, construct_id,
                                             project_name=project_name,
                                             service_catalog_provisioning_details={
                                                 "ProductId": "prod-7tjedn5dz4jrw"
                                             },
                                             project_description=project_description
                                             )`

Domain, user, jupyter app all gets created successfully. The problem comes in with project.
Below is the error :
Resource handler returned message: "Product prod-7tjedn5dz4jrw does not exist or access was denied (Service: SageMaker, Status Code: 400, Request ID: 768116aa-e77b-4691-a972-38b83093fdc4)" (RequestToken: 45ca2a0c-3f03-e3e0-f29d-d9443ff4dfc1, HandlerErrorCode: GeneralServiceException)
I am running this code from an ec2 instance that has SagemakerFullAccess
I also tried attaching SagemakerFullAccess execution role with project...but got the same error.

Based on the error, it seems that ProductId from the service catalog can not be reached in this case your EC2 instance needs permission to AWS Service Catalog

Issue resolved :
https://stackoverflow.com/a/71982829/10214532


Archi 222 update k8s client

Issue
https://gravitee.atlassian.net/browse/archi-222
Description
update kubernetes client to version 3.0.0
Additional context

Just a bump of version, merging without review

:tada: This PR is included in version 4.4.0-alpha.2 :tada:
The release is available on:

4.4.0-alpha.2
GitHub release

Your semantic-release bot :package::rocket:


 There are many examples in integrations directory.

This issue is actually already resolved in commit https://github.com/go-gitea/gitea/commit/fe94032f74298569512f6ece2fbe4f7ab9a24e46 and this PR just fixes older migrations that will be overridden by migration v26 anyway


TAPI IDE model builder for a set of Kotlin DSL scripts

There exists a TAPI model builder to fetch a single Kotlin DSL script model. IDEs need to query sets of scripts efficiently for various features, e.g. indexing them for find usages, refactorings etc... One TAPI request per script means paying the cost of configuration time for each script, which makes it way too slow.
Expected Behavior
Gradle should provide a TAPI builder that, given a set of Kotlin DSL script files, returns the script dependencies for all of then in a single request.
Current Behavior
IDEs have to collect script models with one TAPI request per script, that run sequentially because a daemon can only handle a single TAPI request at a time, paying the cost of configuration times many times.
Context
Discussions with the IntelliJ & Kotlin teams about UX/performance problems on large builds / builds with slow configuration time.

This was delivered in 6.0 via #10776


Tab Completion Not working

Hey,
love the plugin.
Keep up the great work. :)
I found that i have a minor issue.
Tab Completion for Keywords is not working for me.
I have set keywords for my js files but when i press tab when writing
it is not completing to angularjs tried this with a variety of keywords.
:Dash searchterm ang

Is still the case for me any ideas ?
Just pulled latest from master


inconsistent error reporting by plugin and detekt

I have
potential-bugs:
  active: true
  severity: error
  ImplicitDefaultLocale:
    active: true

in my detekt.yml and when I run detekt it reports problem in the code as error

but in the editor (Android Studio Hedgehog | 2023.1.1 Patch 1) the problem is shown as a warning :-(

The detekt plugin (2.3.0) is configured to use the config, but it looks like it overrides severity with "treat detekt findings as errors" (which is not checked by my because I don't want ALL findings from the config to be reported as error)
Is this expected behavior or is this a bug?

I think the "problem" is in the way "treat as error" handled (https://github.com/detekt/detekt-intellij-plugin/blob/e808894ee69bc746446b353fb138a84caf6b9281/src/main/kotlin/io/gitlab/arturbosch/detekt/idea/DetektAnnotator.kt#L60C32-L60C54)
I think instead of
val severity = if (settings.treatAsErrors) HighlightSeverity.ERROR else HighlightSeverity.WARNING

it should be something like
val severity = if (settings.treatAsErrors) HighlightSeverity.ERROR else convertSeverity(finding.severity)

Or, at least, do this for the case when user specified a custom detekt config. Of course if it is ok to make such "backward incompatible" change.
Would you be open for such PR?

Closed with 0df1e1a


Add Spell Effect: Dragon Wings that grants Spell Effect: Dragon Claws

Combine Dragon Wings flying speed RE;
Add Spell Effect: Dragon Wings (2 Focus Points) that grants Spell Effect: Dragon Claws.

This could probably be done with the single effect and a toggle on the draconic and wymblessed class features.

I'm not sure that a toggle is necessary since Dragon Wings are tied to these sorcerer bloodlines (and Dragon Disciple, that is supposed to be associated with a dragon too).
I also missed the heightening.
This would likely mean that Dragon Claws RE would have to be copied to Dragon Wings, unless I'm missing a RE that can affect the effect duration (both this and granted).

The toggle would be to switch between one and two focus points

If we delete effects we have to migrate the links. Those effects definitely need to go, but if we're replacing them then we should use the same ID, and if not we should keep the old effect and add them to the list of effects to migrate away: #4144

I kept the same ID, just renamed the files. Thankfully, the number of items stays the same in this case. Although I didn't verify the ordering of references in the description of imported spells.
It would be great if it was possible to compress it in one (ItemAlter the duration based on item.level), but I expect that would require modifications to the RE to change the property only on item creation.

We could use an Item Alteration and a toggle on the spell itself to make it cost 2 focus points and append the Dragon Claws effect

I'm not sure how ItemAlteration would work for Focus Point cost (additional RollOption toggle?). Can he information from the spell (number of FP used) be passed into the Spell Effect somehow? If not, this would grow the number of Spell Effects to 4: 1FP-1minute/2FP-1minute/1FP-10minutes/2FP-10minutes.
I can see how the Effect reference can be added to the chat card depending on the spell level. And what about looking at the spell card in a compendium, at least one of the Effects should be visible there, right?

What I mean is 3 REs on the spell itself: a toggleable Roll Option, and two Item Alterations: one to alter the focus point cost and one to add the dragon claws spell effect to the description, in which case there wouldn't be a need for a Dragon Wing effect that grants Dragon Claws. Unfortunately, since heightened Dragon Wings make the claws last 10 minutes, you don't end up with any less effects at the end of the day, so maybe just disregard this.

Also for consideration (not necessarily in this PR, or ever) - putting the actual applicable effect into the ItemAlteration added text like this

Here I also removed the effect references in the base description, which, I think, looks neater in the chat, but means they disappear in the spellcasting panel and in the compendium.

Sorry for the conflicts, been trying to get that big PR in for some time now. Should be easy to resolve though.

Please confirm that I did it the intended way.
Also, I'm not sure that focus-point-cost ItemAlteration works as expected: The system checks the number of FP correctly, but it subtracts 1 even if the checkbox is set.

This is the only Note (well, ItemAlteration description) that I can think of adding.
The Dragon Claw effect in the Spell Effect is toggled by that toggle in the spellcasting option. Otherwise there would have to be 4 Spell Effect items to cover all cases.


Updated Dribbble.php

I was working on the Social Auth Dribbble module for Drupal which allows users to register and login to their Drupal site with their Dribbble account. The module allows websites to request any scopes, so any tasks requiring authentication with Dribbble services can be performed.
However, I noticed that when I try to add additional scopes via the form, an error is produced and I am not able to authenticate. Therefore I made some changes to the Dribbble.php so that the additional scopes can be requested.

I'll test this and make sure it's not got any breaking changes before merging.


Pull-to-refresh does not work programmatically

This is a bug.

Framework7 Version: february 2016
Platform and Target: chrome on linux.

When adding the pull-to-refresh programmatically, neither the arrow down nor the spinner shows.
Actually, you can reproduce the same by just pasting in the pull-to-refresh-html in developer modus in chrome instead of doing it by code. The sample below doesn't format properly, but I hope you understand.
html:

js:
var myApp = new Framework7();

var mainView = myApp.addView('.view-main', {
    dynamicNavbar: true
});

var win = require('window');
win.$$ = Dom7;
win.app = myApp;

var region = document.getElementById('home');
var cont =    '

Climbing gyms

';
win.$$('#home').html(cont);

The html I am inserting dynamically in the home element is this:

Climbing gyms

Maybe you can format your code first, and others can see clearly

Yellow Submarine

Beatles

The html structure is the important in F7!!!! If there was any wrong with the code ,plz checked the structure of the html from the official demo

I have formatted it properly now.
I left out the listening on the refresh event in this sample for convenience.

you can't create it dynamically! the various listeners wont get added!

Well, that is major problem. Shouldn't a framework like this support adding html at runtime ?
Is there any workarounds for this ?

it's actually quite tricky to do. maybe explain why you want to do it?

I have a recursive list of items. Inside an items there could be more items. I do not know in advance how deep it will be be. I depends on there data received from the server.

Just checked the docs and i suggest you try
myApp.initPullToRefresh("#pullToRefreshGyms");

Success !
Thank you.

I had the same issue. The official doc example mentions the sample code as
// Add 'refresh' listener on it
ptrContent.on('ptr:refresh', function (e) {
It should be
// Add 'refresh' listener on it
ptrContent.on('refresh', function (e) {

@vishnoor Is right, it should be 'refresh' instead of 'ptr:refresh'. The docs should be changed to reflect this. Thanks vishnoor.


timer: fix operation of periodic timers

Summary
The POSIX timers and timerfd are based on wdog. The wd_start() API adds 1 tick to a delay parameter so periodic timers have accumulative error. Subtract 1 tick from delay before calling wd_start() from wdentry callback to remove accumulative error.
Remove pt_last member from struct posix_timer_s because it is not used anywhere.
Impact
POSIX timers and timerfd users
Testing
WIP

Subtracting one is not always the right solution.  It only works in your case, again because your specific test case is running synchronously with the system time.  But that might not always be the case.  I do not recommend using this solution.
I provided you with a better, adaptive solution in #5421 that will work in ALL cases.  this case, basically one:

given the last time and desired period, it gets the desired time of the next timer expiration:

clock_t next_time= last_time + period

At the time of the timer expiration, it calculates the error between the desired time and the actual time:

clock_t error = next_time - clock_systime_ticks();

Then it uses that error to correct the time for the next tick:

clock_t tick = period + error
This will give you perfect timing regardless of the clock phase.
You cannot defeat quantization errors by adding or subtracting one.  That just doesn't work in a robust way.

I do not fully follow you point. In POSIX timer use case wd_start is called from wd_timer that is called from nxsched_process_timer that is called from systick interrupt, so IMO all phases are synchronized and in this case we do not need extra tick.

The alternative way is to set a flag at the start in wd_timerand clear it at the end, then in wd_start check the flag and if it is set, then do not perform ++delay.

@pkarashchenko @patacongo 's suggestion is the right direction to avoid the cumulative error in a long run. You can't avoid the cumulative error with the relative timeout, because the code from the timer interrupt happen to the next wd_start need time to finish the execution. You must use the absolute time like above to compensate this gap, otherwise the cumulative error will become bigger and bigger.

Maybe, you can read vTaskDelay and vTaskDelayUntil from FreeRTOS to get the ideal:
https://www.freertos.org/a00127.html
https://www.freertos.org/vtaskdelayuntil.html

@xiaoxiang781216 I'm familiar with FreeRTOS delay interface and delays are not a part of this PR. I think that periodic POSIX timer use case that I'm trying to fix is more similar to https://freertos.org/RTOS-software-timer.html than to https://www.freertos.org/vtaskdelayuntil.html
But in a proposed example
  clock_t last_time = current_time;
  for (; ; )
    {
      clock_t next_time = last_time + x;
      clock_t current_time = clock_systime_ticks();
      int error= next_time - current_time;
      last_time = next_time;
      delay(x + error);
   }

if the task is preempted (for example for 1-2 system ticks) right before a delay call, then app periodic logic and compensation calculation will be screwed-up. I've been working for a years with mission critical systems and learned that lesson well.
The only reliable way is to have a timer that posts a periodic event and task that is waiting for that event. Now the question is should it be a MCU HW timer or OS SW timer based on internal OS clock. @xiaoxiang781216 @patacongo please undesrtand me write, the question is not about can I or can not I code a periodic processing in a write way, but about how timers are operating. I could easily implement board_timerhook() and post an event to my task each third system tick and get the reliable operation. What I'm trying to say is that if there is not blockers for periodic POSIX timer to expire in time then it should expire in time (for example if I have 10ms system tick and configure POSIX timer to have periodic expiration with 30ms period then timer should expire each 30ms and not each 40ms. I'm not configuring 35ms or 37ms, but 3 multiples of system tick). If you trying to convince me that we should have crappy timers, then I'm very disappointed.
How the POSIX timers are implemented, that is another question. Currently they are relying on wdog that has it's implemented is a way as it is designed, but if we need to change that to get reliable and well expected operation then I'm fine with that. I feel that we mixed in a discussion timers and wdog, so there are multiple threads in this discussion that I want to separate.
I'm porting an application that is running reliably with RT Linux that have same configuration for a system clock and talking honestly I do not see any reason (strong argument) why it should not run the same with NuttX.
Let's focus discussion on POSIX timers only and leave the wdog alone. We can have a separate discussion why we need to add one extra tick if wd_start is called from wdog expiration handler that is actually called from the timer ISR (exactly at the beginning of a phase). I agree with @patacongo that in general wd_start can be called at any time and we do not know are we at the beginning or in the middle or an the end of the phase, so +1 100% needed in general case, but if we are building a reoccurring event with wdog and wd_start is called from wd_timer (eventually from the system tick ISR) then adding one tick is a bit strange and IMO we just introduce the extra delay in the place where it is really not needed.

@xiaoxiang781216 I'm familiar with FreeRTOS delay interface and delays are not a part of this PR. I think that periodic POSIX timer use case that I'm trying to fix is more similar to https://freertos.org/RTOS-software-timer.html than to https://www.freertos.org/vtaskdelayuntil.html But in a proposed example
  clock_t last_time = current_time;
  for (; ; )
    {
      clock_t next_time = last_time + x;
      clock_t current_time = clock_systime_ticks();
      int error= next_time - current_time;
      last_time = next_time;
      delay(x + error);

Here should be error not x + error.

}

if the task is preempted (for example for 1-2 system ticks) right before a `delay` call, then app periodic logic and compensation calculation will be screwed-up. I've been working for a years with mission critical systems and learned that lesson well.

No, in this case you will delay with few ticks(even skip the sleep in case of error <= 0) and then catch up the sched jitter. So you get one tick error at most in each callback for a long run(assume you don't get error < 0).

The only reliable way is to have a timer that posts a periodic event and task that is waiting for that event. Now the question is should it be a MCU HW timer or OS SW timer based on internal OS clock.

It is no difference(of course, I assume that both implementation is right and accuracy) since the internal OS clock is also built on top of one HW timer.

@xiaoxiang781216 @patacongo please undesrtand me write, the question is not about can I or can not I code a periodic processing in a write way, but about how timers are operating. I could easily implement board_timerhook() and post an event to my task each third system tick and get the reliable operation. What I'm trying to say is that if there is not blockers for periodic POSIX timer to expire in time then it should expire in time (for example if I have 10ms system tick and configure POSIX timer to have periodic expiration with 30ms period then timer should expire each 30ms and not each 40ms. I'm not configuring 35ms or 37ms, but 3 multiples of system tick).

If you config your system with 10ms tick and set a 30ms expiration, you will get it fire between 30ms-40ms(any values between them is possible). Even with your patch(#5421), you will get it fire between 20ms-30ms, no difference from the accuracy view. If you set a 30ms period POSIX timer, and always get the callback at 30+10, 60+10, 90+10... 30*n+10, it's a good implementation.

If you trying to convince me that we should have crappy timers, then I'm very disappointed.

If you want to achieve the more accuracy, the only choice is to reduce the tick length. For example, if you set tick to 1ms, you get the sequence as 30+1, 60+1, 90+1...30*n+1.

How the POSIX timers are implemented, that is another question. Currently they are relying on wdog that has it's implemented is a way as it is designed, but if we need to change that to get reliable and well expected operation then I'm fine with that. I feel that we mixed in a discussion timers and wdog, so there are multiple threads in this discussion that I want to separate.
I'm porting an application that is running reliably with RT Linux that have same configuration for a system clock and talking honestly I do not see any reason (strong argument) why it should not run the same with NuttX.
Let's focus discussion on POSIX timers only and leave the wdog alone. We can have a separate discussion why we need to add one extra tick if wd_start is called from wdog expiration handler that is actually called from the timer ISR (exactly at the beginning of a phase). I agree with @patacongo that in general wd_start can be called at any time and we do not know are we at the beginning or in the middle or an the end of the phase, so +1 100% needed in general case, but if we are building a reoccurring event with wdog and wd_start is called from wd_timer (eventually from the system tick ISR) then adding one tick is a bit strange and IMO we just introduce the extra delay in the place where it is really not needed.

@xiaoxiang781216 without my patch I'm getting 40ms always (not between 30ms-40ms) and with my patch I'm getting 30ms always (not between 20ms-30ms).

If you can describe an situation even theoretical when I can have less than 30ms with my patch, then please describe it.

@xiaoxiang781216 without my patch I'm getting 40ms always (not between 30ms-40ms) and with my patch I'm getting 30ms always (not between 20ms-30ms).

It's just because you start the POSIX timer at the same time of schedule timer interrupt(30*n in above example).

If you can describe an situation even theoretical when I can have less than 30ms with my patch, then please describe it.

You can try to start your POSIX timer in a GPIO interrupt handler, toggle the GPIO by hand, and measure the result. In most case, you will see what I describe, because GPIO can happen randomly in one tick period.

It's just because you start the POSIX timer at the same time of schedule timer interrupt(10*n in above example).
It does not matter when I'm starting a timer. This change is not about timer start, but about timer reschedule that is done in SYS Tick ISR handler, so it is always in sync with timer interrupt.
You can try to start your POSIX timer in a GPIO interrupt handler, toggle the GPIO by hand, and measure the result. In most case, you will see what I describe here, because GPIO can happen randomly in one tick period(0ms-9.9999ms).
I will try, but I can ensure you that it will be the same. But I will make a test with real HW and send you a picture from the logic analyzer.

BTW my programs look like:
int periodic_main(int argc, char *argv[])
{
  int ret;
  timer_t timerid;
  struct itimerspec time_value;
  struct sigevent ev;
  struct siginfo value;
  sigset_t set;

  ret = timer_create(CLOCK_REALTIME, &ev, &timerid);
  if (ret < 0)
    {
      syslog(LOG_ALERT, "ERROR: timer_create() failed: %d\n", errno);
      return EXIT_FAILURE;
    }

  /* Ignore the default signal action */

  signal(SIGALRM, SIG_IGN);

  /* Set up a signal to wake-up periodically */

  time_value.it_value.tv_sec     = 0;
  time_value.it_value.tv_nsec    = 30000000;
  time_value.it_interval.tv_sec  = 0;
  time_value.it_interval.tv_nsec = 30000000;

  ret = timer_settime(timerid, 0, &time_value, NULL);
  if (ret < 0)
    {
      syslog(LOG_ALERT, "ERROR: timer_settime() failed: %d\n", errno);
      return EXIT_FAILURE;
    }

  for (; ; )
    {
      /* Wait for a signal */

      sigemptyset(&set);
      sigaddset(&set, SIGALRM);
      ret = sigwaitinfo(&set, &value);
      if (ret < 0)
        {

        }
      else
        {
          /* Perform periodic action here */
        }
    }

  return 0;
}

If you are familiar with Linux kernel, two set of API exist:
timer: http://www.makelinux.net/ldd3/chp-7-sect-4.shtml
hrtimer: https://www.kernel.org/doc/html/latest/timers/hrtimers.html
NuttX's wdog is same as Linux's timer from the concept.
If you want to achieve the accuracy as the hardware, the solution I suggest is:

Implement the high resolution timer like Linux kernel
Implement wdog on top of hrtimer(then you will understand what I said in this thread).
Implement POSIX timer and timer fd on top of hrtimer

This is exactly how Linux kernel switch from the tick resolution to the high resolution.

If you are familiar with Linux kernel, two set of API exist: timer: http://www.makelinux.net/ldd3/chp-7-sect-4.shtml hrtimer: https://www.kernel.org/doc/html/latest/timers/hrtimers.html NuttX's wdog is same as Linux's timer from the concept. If you want to achieve the accuracy as the hardware, the solution I suggest is:
1. Implement the high resolution timer like Linux kernel

2. Implement wdog on top of hrtimer(then you will understand what I said in this thread).

3. Implement POSIX timer and timer fd on top of hrtimer

This is exactly how Linux kernel switch from the tick resolution to the high resolution. Before this huge work is done, the fast fix is set CONFIG_USEC_PER_TICK to the accuracy you want. Of course, you need to enable tickless(CONFIG_SCHED_TICKLESS) to reduce the schedule timer interrupt loading.

No matter what accuracy do I want I'm still having +1 tick with current code. So if I configure CONFIG_USEC_PER_TICK to 1000, then I'm having 11ms on a 10ms timer and that is 10% overhit (this is huge) no matter if I use CONFIG_SCHED_TICKLESS or not.
Here is a picture when I start timer from the button ISR:

Here is toggle GPIO as a periodic action.
I do not understand why proposed changes that improve accuracy almost as 0 cost are pushed back so hard.

I'm looking for arguments like: "This change leads to such bug (inconsistent behavior) under next circumstances".

I'm not pushing back; I am thinking about what @patacongo wrote (also at here) about the API contract being to wait at least the given time and not knowing whether the (in that case delay, but in other cases non-delay usage) is starting in phase with the tick timer or not. I also see what you're saying about the timing being off by (in this case) 10%, which also seems wrong. I'm not sure what to suggest here. All I know if what I've always done: since software execution is sequential in nature and (with compiled code + multiple threads + CPU caches + other factors) it is not possible to predict what its timing will be, so my go-to solution is a hardware timer when I care about timing and software timers when all I want is to ensure that some time passes without caring too much about accuracy. I appreciate that this is not a good solution for everyone.

I'm not pushing back; I am thinking about what @patacongo wrote (also at here) about the API contract being to wait at least the given time and not knowing whether the (in that case delay, but in other cases non-delay usage) is starting in phase with the tick timer or not. I also see what you're saying about the timing being off by (in this case) 10%, which also seems wrong. I'm not sure what to suggest here. All I know if what I've always done: since software execution is sequential in nature and (with compiled code + multiple threads + CPU caches + other factors) it is not possible to predict what its timing will be, so my go-to solution is a hardware timer when I care about timing and software timers when all I want is to ensure that some time passes without caring too much about accuracy. I appreciate that this is not a good solution for everyone.

I really believe that this change does not contradict with anything that is written in https://cwiki.apache.org/confluence/display/NUTTX/Short+Time+Delays

One thing that I especially dislike about this solution is that introduces a very ugly couple, external logic that uses the wdogs would have special internal knowledge of the implementation of the wdog.  That is the worst form of coupling and terrible modular thinking.

One thing that I especially dislike about this solution is that introduces a very ugly couple, external logic that uses the wdogs would have special internal knowledge of the implementation of the wdog.  That is the worst form of coupling and terrible modular thinking.

I will do an alternative PR with other solution in wdog

Do not substract 1 from the delays. It is very bad engineering for one module to make assumptions about the internal design of another module. Please don't do this.

I think there should be a thread on the dev@ mailing list to design the right solution. Once a design is agreed, then let's come back to PRs.

Closing in favor of https://github.com/apache/incubator-nuttx/pull/5432


Added new term venom allergy MONDO:0600005

fixes #158

@matentzn I refreshed the chebi import per the instructions we discussed today (I added them here: https://mondo.readthedocs.io/en/latest/editors-guide/import-terms-for-logical-axioms/#alternative-instructions-not-tested)
but I don't see the new chebi term in Mondo.
There wasn't a chebi_terms.txt file, so I created one, I'm not sure if that was the right thing to do?

@matentzn nevermind the comment above - I got it to work. I need to try this again to ensure I properly document how I did this though.
@kallia-p CHEBI venom is now available on this branch to use in the logical definition. You should add an equivalence axiom that is:
'allergic disease'
and ('realized in response to' some venom)


fix be cannot compile on MacOS (#26155)

build on MacOS meet error: reference to 'detail' is ambiguous. Because there is a detail namespace under std
Proposed changes
cherry-pick #26155 to branch 2.0, otherwise we compile failed with following errors:
/Users/zhuxiaoli/Codes/doris-2.0/be/src/vec/common/memcpy_small.h:68:5: error: reference to 'detail' is ambiguous
    detail::memcpy_small_allow_read_write_overflow15_impl(reinterpret_cast<char*>(dst),
    ^
/Users/zhuxiaoli/Codes/doris-2.0/be/src/vec/common/memcpy_small.h:49:11: note: candidate found by name lookup is 'detail'
namespace detail {
          ^
/Users/zhuxiaoli/Codes/doris-2.0/be/../thirdparty/installed/include/boost/multiprecision/cpp_int/limits.hpp:15:11: note: candidate found by name lookup is 'std::detail'
namespace detail {

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


lv_list: check actual object type instead of relying on signal handler

Fixes #1010.

Looks perfect, thank you!

Upps, I've just realized the conditions in the while loops were wrong. See: https://github.com/littlevgl/lvgl/commit/6a3c7d2b95ede01f646d2402eaf61a3d25337b3a
It should iterate until finding a btn/label/img so skip the other types.

@kisvegabor My bad. Sorry about that!

No problem :+1:


Updated GET request headers

'Pragma' is deprecated, so removed it
'Platform' key doesn't exist, it's -> 'Sec-Ch-Ua-Platform'
Added 'Sec-Ch-Ua-Mobile' to indicate we don't want the mobile version of the site

Thanks 🙏


Mention PaymentRequest is a W3C standard

Developers have asked if this was a standard or a Chrome API.

@agektmr - a bunch more changed in this, would you please re-review and LGTM.


:scroll:  set up your Kanban board

Git hub repo: https://github.com/SimonGideon/Catalog-of-my-things
Team Members :person_bald: 3

@sebalp100 Team member 1
@SimonGideon Team member 2
@enoqueJonas Team member 3

:warning: Note: The tasks allocated to each member is according to the project requirement then we broke down the task into small subsection. For the main cards as the Project requirements please follow the provided links below:

@sebalp100 Team member 1 #2 Card task
@SimonGideon Team member 2 #3 Card task
@enoqueJonas Team member 3 #4 Card task

Hi team 😊,
STATUS: APPROVED! 🥇 💯

Your project is complete!  There is nothing else to say other than... it's time to start it :shipit: 🚀
Congratulations! 🎉
Highlights
✅ The tasks managed with the team and own time on the project properly
✅ Project has met all the technical aspects you needed for this project. Keep rocking!
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

@omarsalem7 :slightly_smiling_face: Thanks for the review :clinking_glasses:


On Linux, use /usr/share/zoneinfo instead of bundled database

It is unfortunately necessary for the timezone information to
occasionally be updated urgently -- for example the government of Russia
passed a law in July 2014 causing timezone changes in October 2014.
These changes are still not represented in the HHVM bundled database,
see https://phabricator.wikimedia.org/T74556 .
The distros have long dealt with this by patching PHP to get the timezone
database from the system, instead of using the bundled binary. The patch
from php5_5.5.9+dfsg-1ubuntu4.5 happens to apply cleanly to HHVM's copy
of timelib, and compiles correctly if one replaces php_scandir() with
scandir(). In PHP on Linux, php_scandir is an alias for scandir.
The patch was originally by Joe Orton of Red Hat:
http://thread.gmane.org/gmane.comp.php.devel/47609
It has since been maintained by Red Hat and regularly copied into
Debian and Ubuntu. The Ubuntu package carries this copyright notice:
patch: use_embedded_timezonedb.patch
contributor: Joe Orton<EMAIL_ADDRESS>copyright © 2008 Red Hat, Inc.
may be used/modified/redistributed under the terms of PHP itself
The CMakeLists.txt change is my own.
This change is submitted on behalf of a third party, see above for
details.

what about adding a cmake parameter ?  -DUSE_SYSTEM_ZONEINFO or someting?

We'd still prefer not to fork. If you could persuade upstream to take a patch adding support behind a flag, we'd happily add support for it to cmake.

I re-read some old internals threads - sounds like 'it crashes on some formats' was untested hypothesis from one of the maintainers, but the formats are designed to be compatible, and when tested it actually worked fine with both formats. That said, that discussion was in 2008.
re-opening for testing/discussion, though we would still prefer not to fork.
cc @sgolemon

yes, it'd be nice if we could get PHP to accept it directly too.

So, how long will we wait for php to accept it before forking it?

heh, whoops; we plan to merge this. I thought we already had (searched open issue titles for 'tz' and 'timezone' ) - will hopefully get to this soon.

@fredemmott : will you merge it to the LTS releases too ?

Sorry, no; this is a fairly substantial change to an important component.

This still applies cleanly, however:

/home/fred/hhvm/third-party/timelib/parse_tz.c: In function ‘find_zone_info’:
/home/fred/hhvm/third-party/timelib/parse_tz.c:489:12: error: invalid storage class for function ‘index_filter’
 static int index_filter(const struct dirent *ent)
            ^
/home/fred/hhvm/third-party/timelib/parse_tz.c:499:12: error: invalid storage class for function ‘sysdbcmp’
 static int sysdbcmp(const void *first, const void *second)
            ^
/home/fred/hhvm/third-party/timelib/parse_tz.c:508:13: error: invalid storage class for function ‘create_zone_index’
 static void create_zone_index(timelib_tzdb *db)
             ^
/home/fred/hhvm/third-party/timelib/parse_tz.c:592:13: error: invalid storage class for function ‘fake_data_segment’
 static void fake_data_segment(timelib_tzdb *sysdb,
             ^
/home/fred/hhvm/third-party/timelib/parse_tz.c:636:12: error: invalid storage class for function ‘is_valid_tzfile’
 static int is_valid_tzfile(const struct stat *st)
            ^
/home/fred/hhvm/third-party/timelib/parse_tz.c:643:14: error: invalid storage class for function ‘map_tzfile’
 static char *map_tzfile(const char *timezone, size_t *length)
              ^
/home/fred/hhvm/third-party/timelib/parse_tz.c:680:12: error: invalid storage class for function ‘inmem_seek_to_tz_position’
 static int inmem_seek_to_tz_position(const unsigned char **tzf, char *timezone, const timelib_tzdb *tzdb)
            ^
/home/fred/hhvm/third-party/timelib/parse_tz.c:719:12: error: invalid storage class for function ‘seek_to_tz_position’
 static int seek_to_tz_position(const unsigned char **tzf, char *timezone,
            ^
/home/fred/hhvm/third-party/timelib/parse_tz.c:855:16: error: invalid storage class for function ‘fetch_timezone_offset’
 static ttinfo* fetch_timezone_offset(timelib_tzinfo *tz, timelib_sll ts, timelib_sll *transition_time)
                ^
/home/fred/hhvm/third-party/timelib/parse_tz.c:899:16: error: invalid storage class for function ‘fetch_leaptime_offset’
 static tlinfo* fetch_leaptime_offset(timelib_tzinfo *tz, timelib_sll ts)
                ^
/home/fred/hhvm/third-party/timelib/parse_tz.c:977:1: error: expected declaration or statement at end of input
 }
 ^

This is on ubuntu 14.04
We would like to merge this ,but with a high risk of revert depending on reported compatibility issues.

2ad7a554f5bd9446a966de24c9abfc77bc43a62d


Register for Editor tool access

Link to register: https://edit.zoningatlas.org/signup
Training video on getting started in Editor: https://youtu.be/g-4cy0wRah4?feature=shared

@zoesreden This is probably the fastest step everyone can take. It's low-hanging fruit but incredibly important! Once you've registered for the Editor tool, please email me or Zoe and let us know which email address you used to join so we can assign you to the Vermont team inside the content management system.

@thebestyoshibird Sounds good!

@tuckers15 Let me know when you get the invite to join and you register! Then I can add you to the team and this task, too, can be marked as complete!

@thebestyoshibird I have registered and am on the team.

Everyone is signed up.


Add phone number validations for European countries

ISSUE
There is no constructor for zio.schema.validation.Validation which can validate a European country phone number.
NOTE: This is 50 tickets in one. Please signup for one or more of the tickets by checking them below.

[ ] Albania
[ ] Andorra
[ ] Armenia
[ ] Austria
[ ] Azerbaijan
[ ] Belarus
[ ] Belgium
[ ] Bosnia and Herzegovina
[ ] Bulgaria
[ ] Croatia
[ ] Cyprus
[ ] Czechia
[ ] Denmark
[ ] Estonia
[ ] Finland
[ ] France
[ ] Georgia
[ ] Germany
[ ] Greece
[ ] Hungary
[ ] Iceland
[ ] Ireland
[ ] Italy
[ ] Kazakhstan
[ ] Kosovo
[ ] Latvia
[ ] Liechtenstein
[ ] Lithuania
[ ] Luxembourg
[ ] Malta
[ ] Moldova
[ ] Monaco
[ ] Montenegro
[ ] Netherlands
[ ] North Macedonia
[ ] Norway
[ ] Poland
[ ] Portugal
[ ] Romania
[ ] Russia
[ ] San Marino
[ ] Serbia
[ ] Slovakia
[ ] Slovenia
[ ] Spain
[ ] Sweden
[ ] Switzerland
[ ] Turkey
[ ] Ukraine
[ ] Vatican City (Holy See)

SOLUTION
Using other constructors which already exist in zio.schema.validation.Validation (which include regex validation, string length checking, and numeric range checking), introduce a new constructor that satisfies the requirements of this issue.
object Validation {
  ...
  val phoneNumber<2-Letter CountryCode>: Validation[String] = ???
}

IMPACT
Increasing the number of constructors for Validation will make it far easier to use the validation package to
WHERE TO LOOK

Schema validation

PROBLEMATIC ISSUES
Issue #248 must be completed first, or at least, this issue must be based off of #244.

I'll try to get that one started by implementing Switzerland, but once I got the hang of it, I can continue with other countries too.

I'll try to get into this as well and will start with Germany.

@marioaae should we pair up, maybe? If you want to, just ping me on discord (I posted in the zio-schema channel, so you should easily be able to find me)

I will try Serbia.

I will try Portugal.

I will do Hungary.

I can't update the initial task, but I went through the list: all of the validations (and many more) are done, so this issue can be closed, I suppose.
(Only thing I noticed, in the comment the republic of North Macedonia is referred to as the republic of Macedonia, which isn't the official name any more.)


OpenSDS GUI endpoint should be displayed

After the installation job done, there is no guidance to show the endpoint of OpenSDS GUI process, and it would be confused for users how to connect to it. Can we print out GUI endpoint if the dashboard installation work is done?

I think we can add the guidance of 8088 port in the document.


fix: bug caused by inter-dependency miss-assigned map

Description
This PR is related to #137 , and further information is available in the issue.
Changes
The following is the summary of changes done in this PR:

update the map key from depName into its jobName
prioritised the already found job as the dependency to minimise access to database
remove unused key in the dependency
update the unit test

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 9 (77.78%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.008%) to 72.407%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

job/dependency_resolver.go
7
9
77.78%

Totals

Change from base Build<PHONE_NUMBER>:
0.008%

Covered Lines:
5117

Relevant Lines:
7067

💛  - Coveralls

Let's address all the concerns around the dependency resolution as a whole as part of  https://github.com/odpf/optimus/issues/176


In batch cell verification, take commitment for each cell

This PR implements the changes described in the spec PR:

https://github.com/ethereum/consensus-specs/pull/3834

I'm going to merge this so I can start on the next thing. Please review and I'll be happy to make changes if necessary.


Update dx-graphics-hlsl-function-syntax.md

return type doesn't seem to be a factor when trying to disambiguate a function

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

Good, because varying only by return type is just invalid.  However, I'd argue that varying only by function name isn't actually an overload so it's a weird thing to say.

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

Thanks, all!


Is there an iOS beta？

Nope, and as this repo state, we focus on Android dev here.
Please ask your questions regarding other stuff directly on Discord ☺


Stateful server doesn't issue new tickets on a failed resumption

A TLSv1.3 server operating statefully (e.g. SSL_OP_NO_TICKET is set, or max_early_data > 0) does not issue new tickets in the event that the resumption fails. It should (by default) issue two new tickets.

The problem appears to be here:
https://github.com/openssl/openssl/blob/a9cf71a3716f8f624b711faa0d5ea391bb26d9f6/ssl/statem/extensions_srvr.c#L1012-L1037
ticket_expected should be set regardless of the outcome of looking up the session.


🚨 test(VSecM): 902 add test for url.go

Closes: 902

This is a pull request template. Please provide the following information to
help us review your pull request.
Please remove placeholder text (including this text)and replace it with your
own content.

Title of the Pull Request
Description
Please provide a summary of your changes here. Include the purpose of the change
and any relevant issues this PR addresses.
Changes
List the major changes you have made in bullet points:

Change 1
Change 2
Change 3

Test Policy Compliance

[x] I have added or updated unit tests for my changes.
[ ] I have included integration tests where applicable.
[x] All new and existing tests pass successfully.

Code Quality

[x] I have followed the coding standards for this project.
[x] I have performed a self-review of my code.
[x] My code is well-commented, particularly in areas that may be difficult
to understand.

Documentation

[ ] I have made corresponding changes to the documentation (if applicable).
[ ] I have updated any relevant READMEs or wiki pages.

Additional Comments
Include any additional comments or context about the PR here.
Checklist
Before you submit this PR, please make sure:

[x] You have read the contributing guidelines and
especially the test policy.
[x] You have thoroughly tested your changes.
[x] You have followed all the contributing guidelines for this project.
[x] You understand and agree that your contributions will be publicly available
under the project's license.

By submitting this pull request, you confirm that my contribution is made under
the terms of the project's license and that you have the authority to grant
these rights.

Thank you for your contribution to VMware Secrets Manager
🐢⚡!

Thanks for your contributions; merged it in 🙏 .


Upload coverage reports to sonarqube as well

Codecov Report

Merging #48 (784b61b) into dev (0aaaf8f) will increase coverage by 0.03%.
The diff coverage is n/a.

:exclamation: Current head 784b61b differs from pull request most recent head 4b65ba1.  Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


wrong decode

var opus = require('node-opus');

// var channels = 1;
var rate = 24000;
var encoder = new opus.OpusEncoder( rate );

var encoded = new Uini8Array([56,255,168,198,27,96,104,141,249,196,107,126,250,251,91,53,81,141,38,101,62,36,181,66,0,171,100,114,9,111,252,155,64,247,247,30,203,1,198,137,50,168,63,71,7,71,178,156,88,104,94,40,103,2,216,187,
68,244,191,92,105,73,102,172,170,38,52,219,132,194,112,197,173,88,161,182,144,64,55,14,92,114,133,104,167,224,78,167,121,229,245,138,218,10,125,190,54,8,210,190,197,169,84,249,245,156,140,108,89,91,147,95
,12,144,85,41,48,179,194,8,190,213,75,147,18,132,130,91,203,24,1,88,24,215,148]);

var frame_size = rate/100;
var decoded = encoder.decode( encoded, frame_size );

console.log(decoded.length == 11520); // must be equal but not, why?
console.log(decoded);

// must be [0,192,207,188,0,128,111,188,0,128,0,188,0,0,48... but not, why?

Must be equal based on what? Where did you get that reference data?
Opus is a lossy format and there's various things, such as sample rates that would change the output.

I saw it in zello-channel-api in https://github.com/zelloptt/zello-channel-api/blob/master/sdks/js/dist/zcc.decoder.js
t.prototype.decodeFloat32=function(e){return new Float32Array(this._decode(e,4,_opus_decode_float))}

I added console.log to debug it
this.decodeFloat23 = function(e) {
    console.log(e); // encoded
    const x = this._decode(e,4,_opus_decode_float);
    console.log(x); // decoded
    return new Float32Array(x);
}

I know sampleRate and channels and try use your code to do this task but unsuccessfully

Based on the comments in that other issue, you're after raw opus frames. The OpusEncoder gives you opus audio within ogg container, so this is not what you want.

Yes, I need pcm data from row opus frames. In my example var encoded is such data.

const encoder = new opus.OpusEncoder(sampleRate, channels);
const decoded = encoder.decode(packet);

that is all, sorry for time


An illegal reflective access operation has occurred

This issue is imported from pivotal
Since the java 11 upgrade I see this error in the logs. Is it related?

feb 23 09:51:42 t07.ams.surfconext.nl systemd[1]: Started myconext.
feb 23 09:51:50 t07.ams.surfconext.nl java[78858]: WARNING: An illegal reflective access operation has occurred
feb 23 09:51:50 t07.ams.surfconext.nl java[78858]: WARNING: Illegal reflective access by org.springframework.security.saml.saml2.encrypt.KeyEncryptionMethod (jar:file:/opt/myconext/myconext-server-6.0.0.jar!/BOOT-INF/lib/spring-security-saml2-core-2.0.0.M31.jar!/) to field java.lang.Enum.name
feb 23 09:51:50 t07.ams.surfconext.nl java[78858]: WARNING: Please consider reporting this to the maintainers of org.springframework.security.saml.saml2.encrypt.KeyEncryptionMethod
feb 23 09:51:50 t07.ams.surfconext.nl java[78858]: WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
feb 23 09:51:50 t07.ams.surfconext.nl java[78858]: WARNING: All illegal access operations will be denied in a future release

@phavekes Yes it is a direct result op the upgrade to JDK11. It can not be suppressed, but it also does not break anything. (Okke Harsta - Feb 23, 2022)


Update network-security.md

According to last conversation with PG, the Firewall rules limit has been increased from 127 to 1K, please check and update accprdingly.

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

@msmbaldwin @ktoliver is there any update for this? I recently got to know that the limit for Vnet rules increased from 127 to 200 and the limit for IPV4 rules increased from 127 to 1000

#sign-off

Thanks, all.


Add Aspire Component / DI Extensions

This PR:

Adds DI extensions to register LlamaParseClient with DI.
Fixes reading InMemory files in Aspire sample.

@colombod ready for review when you get a chance. Given that this is standard DI, I don't think it needs Aspire branding. Also, instead of having a separate project for this, maybe it's worth just adding another class to the LlamaParse project with these extensions. Thoughts?


CycloneDX 1.5 is not considered a valid spec

CycloneDX 1.5 is not considered a valid spec. Needs to be added as a supported spec.

Do we have a timeline when CycloneDx 1.5 will be supported ?

It should be supported currently. I forgot to update this issue. You facing any issue?

Supported in version 0.0.30. closing.

It should be supported currently. I forgot to update this issue. You facing any issue?

Thanks , I just verified in 0.0.30 its working

@deepharness happy to see you are using the tool. Do let us know if you would like any particular features or improvements.