add routes/custom-host to operator rbac

Description
Fixes https://github.com/grafana-operator/grafana-operator/issues/646
Relevant issues/tickets
https://github.com/grafana-operator/grafana-operator/issues/646
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist

[ ] This change requires a documentation update
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added a test case that will be used to verify my changes
[ ] Verified independently on a cluster by reviewer

Duplicate https://github.com/grafana-operator/grafana-operator/pull/644


List repositories of the logged in User

Calls GitLab projects v4 api on each visit to the page
Renders the projects with pagination

@sreecodeslayer, could you also some tests for this PR?


Update java-dependency-guide contributor-doc reflecting gcpbomupgrader script

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @damccorm

fixed the cross link to the script, merging for now


proposal: add test file using "testing" package

Hi,
I'm working on #417 now.
At the moments(just now), there is a problem that completion candidates after alias-ed struct child methods candidates are not displayed at all. So need change any internal function (other than package_bin.go)
I know gocode have _testing test. But it's mock testing. If change any func behavior for backporting, It's difficult to test whether it causes bugs.
 So it is necessary to test run the actual completion candidate.
So, I want to add the function level testing that using testing stdlib package in the future. But yes, I also know gocode now "maintenance mode". This proposal not good, not sense.
Also, If adding test file, it needs to introduce the CI test that I proposed earlier. but it was rejected.
However, you said that you guarantee the operation of gocode against stable Go1 version. I also decided to correspond to the tip as much as possible. (Yeah, I'm not an owner of gocode. without consent. But I'm glad if that helped you as a result)
I think that it will be nice if it helps new Go runtime feature backporting etc. And, If you're not planned to support Go 2.0, I will write a new completion engine. Maybe also it will be helpful.
Anyway, I {want to | will write} a function level testing. The subject of this issue is whether I can pull request it.
If not like, I'll maintenance it in the forked my repository (only test file). it's no problem because this test is almost for me (but it might be helpful for other contributors).
What do you think about it?

Add everything you need. All that matters is that feature is working. When it's working we'll see what and how we can merge.

@nsf Thanks.
I'll only include the mock test in that PR because it is the priority to fully works for type alias. But I'll add some test in the future.
Is CI ok too? If so, as for CI, I don't have commit privilege of the gocode repository, so need to your setup it. I will mention to you when sent PR that adding configuration(like .travis.yml or circle.yml) file.
Well... the CI test is not useful without commit privileges(authority of managing the CI) like can't sent rebuild request, can't notification settings. so maybe I'll only adapt to my forked repository.

I don't think gocode needs CI. CI is useful for large scale projects with a fast update rate and frequent deployments. Gocode isn't one of them.

@nsf Ah, I thought not CI for testing. actually, that's useful for test coverage.
i.e, if use codecov.io, I(we) can easily see the coverage.
https://codecov.io/gh/zchee/appleopensource/src/bdba210c3dbaee1b140c5d10680a2f64fe472a28/appleopensource.go
With chrome extension:

So, I want to CI test. but it's can see on zchee/gocode, so maybe CI setting are I'll only adapt to my forked repository.

You can get coverage via go tool cover as well. I do agree with you that it'd be nice to convert tests to go test format instead of using external scripts.

@nsf OK, got it and agreed :)
Anyway, I'll continue to try write test and support type alias.
I close this issue when I pull request the adding first test file.


curl import added

Fixes issue #93
Hey @liyasthomas  I have tried to add functionality to import curl commands. Please let me know if any changes are required.

Hey @nityanandagohain,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 71e88880-d095-11e9-97e2-99a13ad28605

Hey @nityanandagohain,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 71cc3d20-d175-11e9-abee-f3892bc79bc5

Hey @nityanandagohain,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 08872740-d1e2-11e9-96c5-8bf35d42f9db

@NBTX merge after your approval

@nityanandagohain @hosseinnedaee
Ran into a bug in cURL parser! After importing a cURL, PATH is appended to URL which is okay. But tje existing PATH field stays same. We manually need to empty PATH field before sending next request.
To reproduce:

Generate a cURL code (⚡ icon under URL field) for the default test request
Copy thc cURL and import the same using Import cURL button
See URL is appended with PATH as well as the existing PATH stays as same as before
Fix:
Either replace the existing PATH field with the one from the imported cURL (which is the best UX)
Or
Append PATH to URL, but empty the existing PATH field.

I wonder why anybody haven't reported this yet. And need to look at error handler on index.vue regarding importCURL as well since from 2nd import cURL request, function lands me on error handler shooting error toast.
https://github.com/liyasthomas/postwoman/blob/master/assets/js/curlparser.js


Connections should stop consuming if a unacceptable message threshold is reached

By default unacceptable messages (e.g. messages that failed to be read from message infrastructure)
are acknowledged and drop.
A configuration setting is needed to override this behaviour and allow a threshold to be set.
When the threshold is reached, messages should stop being read by the Connection consumers.

Release v3.0.129


3D tiles do not load as expected during Simulate play mode.

The 3D tiles load properly when in Editor and when in Play-In-Editor mode, but break during Simulate. It seems that Cesium doesn't know what camera frustrum to use, and the tiles are only showing far away from the user. The expected behavior would be that Cesium falls back to the Editor viewport frustrum during this play mode.

We added support for multiple simultaneous views in v1.5.0, which I just confirmed fixes this issue.


exporting imodel to obj

I am trying to use the test app export-obj but the imodel keeps failing to be loaded due to a profile issue
I have tried to open directly a local .imodel file published from OpenPlant or a .idgndb created by Navigator getting a different error on each case
using .imodel
PS D:_Siemens\DevProjects\imodeljs\test-apps> node export-obj/lib/ExportObj.js --input "m3321a010.imodel" --output "OPM_Example.obj"
Error   |DgnCore| Error BE_SQLITE_ERROR_InvalidProfileVersion opening [m3321a010.imodel]
Error   |imodeljs-backend.IModelDb| Invalid Profile Version: Could not open standalone iModel {"pathname":"m3321a010.imodel","ExceptionType":"IModelError","ActivityId":"86d1117a-2ef0-46f3-9e09-99be22d1b286","SessionId":"43592aa1-8462-455e-b82c-fc87142a72e6","ApplicationId":"","ApplicationVersion":""}
Could not open standalone iModel
Invalid Profile Version: Could not open standalone iModel
at Function.openStandalone (D:_Siemens\DevProjects\imodeljs\core\backend\lib\BriefcaseManager.js:792:19)
at Function.openStandalone (D:_Siemens\DevProjects\imodeljs\core\backend\lib\IModelDb.js:221:68)
at Function.openSnapshot (D:_Siemens\DevProjects\imodeljs\core\backend\lib\IModelDb.js:231:31)
at doExport (D:_Siemens\DevProjects\imodeljs\test-apps\export-obj\lib\ExportObj.js:18:48)
at Object. (D:_Siemens\DevProjects\imodeljs\test-apps\export-obj\lib\ExportObj.js:101:5)
at Module._compile (internal/modules/cjs/loader.js:689:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
at Module.load (internal/modules/cjs/loader.js:599:32)
at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
at Function.Module._load (internal/modules/cjs/loader.js:530:3)
using idgndb
PS D:_Siemens\DevProjects\imodeljs\test-apps> node export-obj/lib/ExportObj.js --input "m3321a010.idgndb" --output "OPM_Example.obj"
Error   |BeSQLite| Cannot open file: The file's ECDb profile (<IP_ADDRESS>) is too old. Expected version: <IP_ADDRESS>.
Error   |DgnCore| Error BE_SQLITE_ERROR_ProfileTooOld opening [m3321a010.idgndb]
Error   |imodeljs-backend.IModelDb| Profile Too Old: Could not open standalone iModel {"pathname":"m3321a010.idgndb","ExceptionType":"IModelError","ActivityId":"e5ce12bc-b324-4d65-919b-1bda223b2f1b","SessionId":"cc789673-5b75-4ce6-9ec3-224e696b21a1","ApplicationId":"","ApplicationVersion":""}
Could not open standalone iModel
Profile Too Old: Could not open standalone iModel
at Function.openStandalone (D:_Siemens\DevProjects\imodeljs\core\backend\lib\BriefcaseManager.js:792:19)
at Function.openStandalone (D:_Siemens\DevProjects\imodeljs\core\backend\lib\IModelDb.js:221:68)
at Function.openSnapshot (D:_Siemens\DevProjects\imodeljs\core\backend\lib\IModelDb.js:231:31)
at doExport (D:_Siemens\DevProjects\imodeljs\test-apps\export-obj\lib\ExportObj.js:18:48)
at Object. (D:_Siemens\DevProjects\imodeljs\test-apps\export-obj\lib\ExportObj.js:101:5)
at Module._compile (internal/modules/cjs/loader.js:689:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
at Module.load (internal/modules/cjs/loader.js:599:32)
at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
at Function.Module._load (internal/modules/cjs/loader.js:530:3)
I am not sure if I am missing anything.

The second error is happening because you are trying to open a 1.6 iModel which is not compatible with iModel.js (iModel.js only supports 2.0 iModels, ECDb profile 4 and greater).  The latest versions of Navigator CONNECT Edition (10.x) support 2.0 iModels.  This newer version of Navigator converts the older iModels in place so it can view them but does not support saving the exported file.  The suggested workflow is to publish directly to the 2.0 format from the original source.
The first error is less clear, some more information would help.  Can you provide the versions of OpenPlant and iModel.js you are using.

A .imodel file is basically a zip archive containing a .bim or .ibim file. export-obj wants the uncompressed file - it won't decompress it for you.

If you have a recent .imodel package file, you can extract the .bim/.ibim using these two steps:

Upload it to the hub from the developer registration dashboard
Run the snapshot utility to pull out the contents:

If that doesn't work, perhaps you are using an old version of OpenPlant.  Either get a newer version, or you can use a recent BentleyView which is a free download.

A .imodel file is basically a zip archive containing a .bim or .ibim file. export-obj wants the uncompressed file - it won't decompress it for you.

A .imodel file is basically a zip archive containing a .bim or .ibim file. export-obj wants the uncompressed file - it won't decompress it for you.

Thanks, it worked when I used i.bim file


FR: Add inverse format rule to format-tags-in-yaml, to normalize tags to be prefixed with a '#'

Is Your Feature Request Related to a Problem? Please Describe.
Starting from v1.4.4, Obsidian now supports '#'-prefixed tags in the YAML frontmatter for the tags:  property, provided that the entire tag is in quotes. These tags aren't visually distinct in the UI from non-prefixed tags, and there are uses for tags prefixed with a '#', which leads to people worrying over which format to use.
Describe the Solution You'd Like
It would be great to have a rule that automatically adds the '#' to YAML frontmatter tags and wraps them in the quotes of choice.
Before
---
tags:
  - one
  - #two
  - "#three"
  - 'nested/four/five'
---

After
---
tags:
  - "#one"
  - "#two"
  - "#three"
  - "#nested/four/five"
---

Describe Alternatives You've Considered
N/A
Additional Context
The images below show YAML fronmatter tag formats and their representation in Obsidian's Live Preview UI.

I plan to close this as a duplicate of #845 .


Updated to renamed staking-deposit-cli, fixed typo

Motivation
I wanted to keep lodestar docs up-to-date and clean.
Description
Ethereum updated the name of the cli to staking-deposit-cli, so that the naming was consistent.
Also, changed "latter" to "later" to be grammatically correct (sorry to nit-pick!)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

The CLA Assistant check link isn't working, this is what I see


feat(explorer): mark pending batches in red after 5 minutes

Description
Marks pending batches in red after 5 minutes
Testing

Start a local devnet
Run the explorer
Kill operators so that batches don't reach quorum
Send proofs
Verify that after 5min it goes red.

Type of change

[x] New feature

Checklist

[ ] “Hotfix” to testnet, everything else to staging
[ ] Linked to Github Issue
[ ] This change depends on code or research by an external entity

[ ] Acknowledgements were updated to give credit

[ ] Unit tests added
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

[ ] This change is an Optimization

[ ] Benchmarks added/run

[ ] Has a known issue

Link to the open issue addressing it

[ ] If your PR changes the Operator compatibility (Ex: Upgrade prover versions)

[ ] This PR adds compatibility for operator for both versions and do not change batcher/docs/examples
[ ] This PR updates batcher and docs/examples to the newer version. This requires the operator are already updated to be compatible

Nice, I updated your comment with a testing section 👍

Thanks! Forgot about it, just promoted the draft. I'll update myself tomorrow for the other one.


Response Header.get() implementaion is wrong

The Response Headers implementation is wrong. The Headers.get function converts arrays to strings. This neither conforms to Map nor the URLSearchParams class definitions. This is a problem when you try to parse the set-cookie headers
/**
  * Return combined header value given name
  *
  * @param   String  name  Header name
  * @return  Mixed
  */
	get(name) {
		name = `${name}`;
		validateName(name);
		const key = find(this[MAP], name);
		if (key === undefined) {
			return null;
		}

		return this[MAP][key].join(', '); //this line is wrong
	}

Expected behavior, if applicable
Map implementation
const map = new Map();
map.set('set-cookie', ['cookie1','cookie2'])
map.get('set-cookie');
// ['cookie1','cookie2']

The Headers class looks like a re-implementaion of URLSearchParams class, which is supported natively. You could save yourself a lot of code, and the implementation could match the standard.
const searchParams = new URLSearchParams();
searchParams.append('set-cookie', 'cookie1');
searchParams.append('set-cookie', 'cookie2');
searchParams.get('set-cookie');      //"cookie1"
searchParams.getAll('set-cookie');  //["cookie1", "cookie2"]

As you can see searchParams.get('set-cookie') doesn't join the values but Headers.getAll doesn't exist in your implementation.
Your Environment

software
version

node-fetch
2.6.0

node
12.18.2

npm
6.14.5

Operating System
Win10

From MDN
https://developer.mozilla.org/de/docs/Web/API/Headers
Obsolete methods
Headers.getAll()
Used to return an array of all the values of a header within a Headers object with a given name; this method has now been deleted from the spec, and Headers.get() now returns all values of a given name instead of just the first one

Did you forgot to read the part of where it says: "Obsolete"?
Used to return an array of all the values of a header within a Headers object with a given name; this method has now been deleted from the spec, and Headers.get() now returns all values of a given name instead of just the first one.

https://github.com/node-fetch/node-fetch/issues/251#issuecomment-428143940

But the implode is still wrong... And it also says Headers.get() now returns all values of a given name

To get a name name from a header list list, run these steps:

If list does not contain name, then return null.
Return the values of all headers in list whose name is a byte-case-insensitive match for name, separated from each other by 0x2C 0x20, in order.

https://fetch.spec.whatwg.org/#terminology-headers
0x2C === comma
0x20 === space
Browser behavior: https://jsfiddle.net/zL04o17w/

This might be relevant for ya: https://github.com/whatwg/fetch/issues/973


Relationship info supercedes queried object info

It looks like when you add a relationship(ex: domian -> ip) the information returned when querying the IP returns information for the first relationship in the table(and vice versa).

Also, I couldn't figure out how to remove a relationship, am I just missing the obvious?

<side_thought>The Relationship table would probably be a good place to have a link to pivot to that objects page actually, since I just accidentally tried to click, it to no avail.</side_thought>

Hey, sorry for the late response, been swamped. There isn't a way to remove a relationship just yet, but, I think with our new proposed database structure, we'll have this sorted out. I agree that in the relationships table should have a link to the actual indicators page that has a relationship with.

No worries, I know how it goes. Thanks for looking into it.

Seems like this is still a thing It actually, IMO throws off the true relationships. Upon linkage, the data about each entity becomes the relationship data, and loses the data about the entity itself. I.E: I add <IP_ADDRESS> and  then add  google.com, I link them (obv) but once I do that, the data for the original entity merges into the relational identity, and gives up it's original properties.


Unable to install pcfdev plugin

Windows Version: Windows 7 Enterprise SP1
CLI version: cf.exe version 6.29.0+ff886fa.2017-07-24
PCF Dev: pcfdev-v0.26.0+PCF1.10.0-windows.exe
I am attempting to install PCF DEV as shown below.  I have seen other issues with the same behavior which pointed to a possible local firewall issue.  All of those issues were closed without a definitive resolution.  I am fairly confident that I do not have a firewall issue.  I started netmon and tried setting some proxy configurations and I never see any traffic being attempted on the port listed in the error message (which changes with every attempt).
PS C:\pcf> cf install-plugin .\pcfdev-v0.26.0+PCF1.10.0-windows.exe
Attention: Plugins are binaries written by potentially untrusted authors.
Install and use plugins at your own risk.
Do you want to install the plugin .\pcfdev-v0.26.0+PCF1.10.0-windows.exe? [yN]: y
y
dial tcp <IP_ADDRESS>:49299: socket: An invalid argument was supplied.
exit status 1
File is not a valid cf CLI plugin binary.
FAILED
Any help is appreciated.

This looks like a duplicate of #248. This is not an issue with PCF Dev. Please follow the issue upstream.

There is no resolution listed for any upstream issue.  As I mentioned, I am pretty confident that there is NOT a firewall issue.  There is no documentation, that I could find, that pointed to required ports being opened.  If this is all local to the machine, then that shouldn't be an issue anyway.  Is there possibly some other required prerequisite that is needed that isn't listed in the documentation?

Again, this issue affects all cf CLI plugins, not just PCF Dev. The cf CLI needs to be able to bind to ports on the local loopback device in order to install and run plugins. This is not working on your machine, for an unknown reason.
The upstream issue is currently in-flight: https://www.pivotaltracker.com/n/projects/892938/stories/149303367
This means that engineers are likely actively looking into what system configuration could cause this problem on Windows 7.


Problem with multi-line template literals

Hi, great project. Noticed an obscure bug, though:
Template literals starting with whitespace and containing a newline will sometimes confuse the obfuscator. For example:
function hi() {
    return ` Hello
        There`;
}

This gets obfuscated into:
function hi() {
    return;
    '\x20Hello\x0a\x09There';
}

Note that the return got separated from the literal, so the obfuscated function returns nothing.
The same thing happens if the backtick is immediately followed by a newline, e.g.:
function hi() {
    return `
        Hello There
    `;
}

So, i think i will temporary fix it from my side by correcting wrong AST tree

any ETA on our side? I've pinged on esprima already.

this week

https://github.com/javascript-obfuscator/javascript-obfuscator/pull/192

Fixed in<EMAIL_ADDRESS>

With following source code still wrong obfuscated code:
function foo () {
	return`foo
bar` + 1;
}

yep, nothing changed it seems );

0.14.1 fixed template literals without additional expressions, like example in the first message

Fixed. Expect release in a hour or so.

Should be fixed in<EMAIL_ADDRESS>

it's not working with following syntax
sql_query = select * from blahblah where myids in (${ids.map(d=>'${d}').join(',')})                                 ${projectWhere}                                 ;

Hi, please format your code, looks like formatting is wrong.

Actual code
sql_query = `select * from sub JOIN ps   ON
sub.subsystem_id=ps.subsystem_id
                                JOIN project ON project.id = ps.project_id
                                where sub.subsystem_uid in (${ids.map(d=>`'${d}'`).join(',')})
                                ${otherWhere}
                                `;

Created issue: https://github.com/javascript-obfuscator/javascript-obfuscator/issues/288


plumb through HomeDir as optional build configuration

This is already supported in the passwd package, we just don't plumb it through to ImageConfiguration yet.
Turns out some software (jenkins.......................................) parses /etc/passwd instead of respecting common env var overrides ($HOME).
ref: https://github.com/chainguard-images/images/pull/2763


Callback does not work when only one method is defined

I'm starting working on: https://github.com/kstateome/django-cas/issues/28 and for that, I only defined one callback:
CAS_RESPONSE_CALLBACKS = (
    'webapp.cas.callbackresponse'
)

In that case, I get a 500 error and the callback method is never called.
After a successful CAS authentication, from: https://github.com/kstateome/django-cas/blob/develop/cas/backends.py#L70, the cas_response_callbacks method is called: https://github.com/kstateome/django-cas/blob/develop/cas/utils.py#L6.
With only one callback method defined (like above), the callbacks property is strangely an array of all characters: ['w', 'e', 'b', 'a', 'p', 'p', '.', 'c', 'a', 's', '.', 'c', 'a', 'l', 'l', 'b', 'a', 'c', 'k', 'r', 'e', 's', 'p', 'o', 'n', 's', 'e'] and thus, the callback is never properly called.
With two callback methods defined or more, it works.

It expects a tuple, you provided ('webapp.cas.callbackresponse') (no ',' after the string), which is identical to 'webapp.cas.callbackresponse'. It unpacks the string, thus you get ['w', 'e', 'b', 'a', 'p', 'p', '.', 'c', 'a', 's', '.', 'c', 'a', 'l', 'l', 'b', 'a', 'c', 'k', 'r', 'e', 's', 'p', 'o', 'n', 's', 'e'].
Try again with
CAS_RESPONSE_CALLBACKS = (
    'webapp.cas.callbackresponse',
)

You're right, it works. Sorry for the wrong issue.


multicast socket addresses are not reported properly

final DatagramChannel channel = DatagramChannel.open(StandardProtocolFamily.INET);
channel.bind(new InetSocketAddress(port));
channel.join(InetAddress.getByName("<IP_ADDRESS>"), networkInterface);

Socket address will be reported as <IP_ADDRESS>/port in the above scenario.

expected behaviour


Sign release artefacts using cosign

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area build
What this PR does / why we need it:
Sign the release artefacts using cosign and also generate the SBOM using syft for now
the secret key is already set in circleci
/assign @leogr @Issif
cc @danpopnyc
Which issue(s) this PR fixes:
Fixes #300
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cpanato
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [cpanato]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@leogr I've run in a sample repo the commands and that worked :)
let's try, what we can do is when we cut the release, I will be watching to check and if fail can act right away
@Issif

I think I will work on the new release this month, I would to manage source and tags in payload first, this is important because of plugins and talon, but this is a big work as we have many outputs

@cpanato I created the PR for dealing with source and tags in the events, feel free to rebase this PR on it after

@cpanato I created the PR for dealing with source and tags in the events, feel free to rebase this PR on it after

There are no conflicts. I guess we can approve this PR. @Issif is that ok for you?

i can rebase just to make sure, one sec

@leogr @Issif rebased

LGTM label has been added.  Git tree hash: 6fb279971631c5d5763edabc1feb874be85e92ee

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: cpanato, Issif
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Issif,cpanato]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Issif when you cut let me know in case we need to tweak anything :)


Text with fixed-width columns does not align properly in slack

[X] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[X] I've read and agree to the Code of Conduct.
[X] I've searched for any related issues and avoided creating a duplicate issue.
Description

The API does not correctly send text with formatted width.

Reproducible in:

[https://github.com/brent-riva/python-slack-api-tutorial ] This is reproducible in the sample project.
python-slackclient version: Latest.
Python Version: Both latest 2 & 3 versions.
OS Version: Windows 10 & Mac

This issue is also in my mac's bot program as well, and my team wrote that in 2.7.
Steps to reproduce:

Use sample program, or use your own : ).
Send slack formatted strings with fixed-width columns
(e.g. response += "\n{:12}{:25}{}".format(row[0], row[1], row[2])
Print the response in the python repl as well.

Expected result:
I expected the message in slack to stand in straight, organized columns
Actual result:
The spacing is close but definitely not fixed-width. As a developer I would love to have output where columns are aligned.
Attachments:
http://imgur.com/a/enH1j
Is a picture of the two outputs side by side.

Blame Slack for this one. To get what you want, you need to mark the text as pre-formatted by escaping it with triple-backticks https://get.slack.help/hc/en-us/articles/202288908-Format-your-messages#code-blocks

I'm having the same problem outputting a simple text table with tabs using Javascript. Testing the output alone prints the desired table, but sending the output to a slack bot messes the tabs, even with triple backticks surrounding the text table.

@araizaga-yael you can experiment with message formatting options using the Slack Message Builder: https://api.slack.com/docs/messages/builder
for multi-line code blocks, you'll need 3 backticks.

Hey, thanks for the quick response @Roach. Didn't expect it from a closed issue.
After messing around a bit more with the API and trying to send the table as a file, I think it's a difference with how slack uses the "\t" character.
In output and files the tab best tries to fit the text in columns like so: https://imgur.com/4Q65rNU
When posting it to slack, this is lost and it would seem as \t is just interpreted as a set amount of blank spaces
https://imgur.com/71Wr79f
I tried to use multi-line blocks, but the issue persists. I can even get it in git's markdown successfully:
CABR		5		1		88
CAFR		34		11		5
CAKE		0		0		0
CALA		2404		8		79

(copying this exact text to Slack will mess up the tabs, even if it's inside 3 backticks)
Is there a workaround/fix?

Hey,
There is a workaround that I use. I’m posting from mobile email, but lookup
fixed column width string formatting. That solved my issue, combined with
the three ticks for preformatted text. IIRC, it looks like:
“{:5}{:15}{:5}”.format(var1,var2,var3)
Hope that helps!
On Tue, Apr 17, 2018 at 9:52 PM araizaga-yael<EMAIL_ADDRESS>wrote:

Hey, thanks for the quick response @Roach https://github.com/Roach.
Didn't expect it from a closed issue.
After messing around a bit more with the API and trying to send the table
as a file, I think it's a difference with how slack uses the "\t" character.
In output and files the tab best tries to fit the text in columns like so:
https://imgur.com/4Q65rNU
When posting it to slack, this is lost and it would seem as \t is just
interpreted as a set amount of blank spaces
https://imgur.com/71Wr79f
I tried to use multi-line blocks, but the issue persists. I can even get
it in git's markdown successfully:
CABR		5		1		88
CAFR		34		11		5
CAKE		0		0		0
CALA		2404		8		79
(copying this exact text to Slack will mess up the tabs, even if it's
inside 3 backticks)
Is there a workaround/fix?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/slackapi/python-slackclient/issues/164#issuecomment-382219241,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADBKWBDtVY2pTrl5mvxWWFFzHgmybnguks5tppxGgaJpZM4Lv2b6
.

Thanks for the help @brent-riva and @Roach. I ended up doing something similar to string formatting in Javascript, using this function to pad all strings with spaces:
function pad(pad, str) {
    if (typeof str === 'undefined')
        return pad;
    return (str + pad).substring(0, pad.length);
}

Then just surrounded the output with triple backticks and it worked like a charm. Hopefully tables can be implemented in the future.

@araizaga-yael https://trello.com/c/OiblqzOg/40-new-message-attachment-display-options

Awesome little fix with the code block!
I can't get it to print from a pandas dataframe though. I am cleaning up using tabulate. Code blocks has worked for triple speech marks but can't seem to figure out how to wrap it around my dataframe out put any help on this? cheers!
This doesn't work:
response = """```         %s         ```         """ % (tabulate(PSDATA, headers='keys', tablefmt='psql', numalign='right'))
Where as this does:
`"""Hello!\n         Please use the following commands:\n         whereis *item*\n         prep         --- Prints prep items from wunderlist\n         stock *category* --- Prints current stock\n               - WineBTG, cocktail, spirits, wine\n
    """`

response = """%s """ % (tabulate(PSDATA, headers='keys', tablefmt='psql', numalign='right'))

This worked for me

Awesome little fix with the code block!
I can't get it to print from a pandas dataframe though. I am cleaning up using tabulate. Code blocks has worked for triple speech marks but can't seem to figure out how to wrap it around my dataframe out put any help on this? cheers!
This doesn't work: response = """``` %s ``` """ % (tabulate(PSDATA, headers='keys', tablefmt='psql', numalign='right'))
Where as this does: """Hello!\n Please use the following commands:\n whereis item\n prep --- Prints prep items from wunderlist\n stock category --- Prints current stock\n - WineBTG, cocktail, spirits, wine\n`
    """`

This doesn't work for me when sending a message via slack webhook or slack chat.postMessage. The table with fixed width is getting distorted by slack formatting.
@petelah : Could you please elaborate on the solution that worked for you.


Add different wave forms

[ ] triangle
[x] square
[ ] saw tooth

Didn't manage to create triangle and sawtooth. Maybe some other time.


Add test case for simplified Chinese text width

fixes #723

@chrfalch I added a bug fix where we were using the wrong string length

It's in 0.1.149


chore: de-mathlib Data/List/Count.lean

Moves Mathlib/Data/List/Count.lean to Std. I believe there are no notable changes from mathlib, aside from dropping various lemmas about prod.
Depends on #98 #96 #94 #97. EDIT: all merged :)

Cleaning this up again; the list lemma files have quite a few conflicts because many of the list lemmas were added by someone at some point.

I've made https://github.com/leanprover-community/mathlib4/pull/6575 so we can check where these suspicious simp lemmas are used.

RE: questionable simp lemmas -- each lemma and where it is used in mathlib (for my own reference)
List.count_eq_length
 - List.count_replicate_self
List.count_eq_length
 - Std.Data.List.Count._auxLemma.15
List.count_eq_zero
 - List.count_bagInter
List.count_eq_zero
 - List.sum_map_count_dedup_filter_eq_countp
List.count_eq_zero
 - List.count_replicate
 - Std.Data.List.Count._auxLemma.14
List.count_pos
 - List.count_bagInter
List.count_pos
 - Nat.Prime.factorization_pos_of_dvd
List.count_pos
 - Std.Data.List.Count._auxLemma.12
List.count_pos
 - List.dedup_eq_cons
List.count_pos
 - List.count_eq_one_of_mem
List.count_pos
 - List.subperm_append_diff_self_of_count_le
 - List.one_le_count_iff_mem
List.count_pos
 - List.count_eq_zero_of_not_mem
List.count_pos
 - Mathlib.Data.List.Pairwise._auxLemma.38
 - List.perm_iff_count
List.count_pos
 - List.not_mem_of_count_eq_zero
List.countp_eq_length
 - List.count_eq_length
List.countp_eq_length
 - Std.Data.List.Count._auxLemma.7
List.countp_eq_zero
 - Std.Data.List.Count._auxLemma.6
List.countp_filter
 - Multiset.countp_eq_countp_filter_add
 - List.count_filter
List.one_le_count_iff_mem
 - Std.Data.List.Count._auxLemma.13
List.one_le_count_iff_mem
 - List.pairwise_of_forall_mem_list

Outstanding research work! Sadly, I don't have time to look at these carefully right now but this is absolutely great!

I think this is good for review. The mathlib PR is helpful to see the changes from the mathlib version.

By the way, I have a tool that lists all lemmas used by other declarations, and can flag lemmas used by simp. If you need to do this research again and I haven't published that tool, please ping me. :-)


[Bug & Proposal] doesn't work in FlyoutPage & Init and call directly in C#

When put it inside a FlyoutPage, the CameraBarcodeReaderView doesn't show.
We need to put it in a separate page and navigate to it,
or call it directly in C# (which is way better) like in ZXing.Net.Mobile:
var scanner = new ZXing.Mobile.MobileBarcodeScanner();
var result = await scanner.Scan();
if (result != null)
Console.WriteLine("Scanned Barcode: " + result.Text);
This FlyoutPage scenario might relates to #7, which still exists in latest version (Android 10+; iOS not tested).
No workaround found yet.
Also, could you please put the FlipCamera and ToggleTorch as a standard overlay, so as to make the functionalities easier to be called.
Thanks a lot.
Best Regards

Below is a demo project illustrating the issue:
maapp.zip

This is basically #7
Created a PR for this but It has not been approved yet.

@nebula2 's PR was brought in, the latest release has the fixes. Thanks!


Set up data-sources-v2-dev and data-sources-v2 Digital Ocean Apps

Now that we're ready to start doing more flexible testing, it's time to start setting up these applications.
Currently, data-sources-v2 exists in Digital Ocean, but its build has been failing. I think I know why, but to make sure I'm right, I'll need to get data-sources-v2-dev started first and be able to implement steps that data-sources-v2 will emulate.
I can probably do a lot of the requisite steps on my own, but things that I probably can't do on my own is:

[x] Create the initial version of data-sources-v2-dev and connect that to the proper Github Repository and Branch (I tried, and GitHub said that I need to be the organization owner). @josh-chamberlain I think this means you'll need to step in here.

@maxachis data-sources-v2-dev created. I used the dev database envar and didn't add a secret key yet since I'm not sure whether we need a separate API key for dev stuff.

@josh-chamberlain Excellent. Working on setting it up now. I am wondering if we want to give the app admin-level permissions to the dev database -- it's dev, so it's not the most risky thing, but we probably want to develop a precise role that is more specific, if we don't have that yet (and I think we do).

@josh-chamberlain It's up!
https://data-sources-v2.pdap.dev/

@maxachis woohoo! that's great.
re: admin-level permissions, sounds good—I think we should discuss it. We do plan to give people described as "admins" the ability to edit requests here: #256
we don't have too many other admin actions planned yet. we would probably use retool or something, with dedicated endpoints, for internal tools (or develop internal tools as their own little app)
so, now's a good time to get more precise.

Now that everything here seems to be properly set up, I think we can comfortably close this issue.


BUG: make sure the palette is empty when IsReadAsScalarPlusPalette is false

This is a proposal to force the palette to be empty when the palette is expended. It seems to be the most expected behavior, also uniform across the two classes. It should avoid also problem when a instance is reused, and a palette was already read.
Change-Id: Ie12910c8ebdd70f47afff12fdfacca08134a8856

Is it the right place for these commits, or shall I push to gerrit ?

It seems that the test errors are unrelated. all these tests are passing on my machine

move to gerrit


unit test for master.build

SSIA

(2023/05/12 edited)
Add first unit test to master.build() .
This is obviously not good way to test because it tests logging output. However, I'd like to add this test at first because

better than no test
by test code only, no production code changes

After adding this test, we can improve production code to easier to test.
related: https://github.com/m3dev/kannon/pull/24#issuecomment-1511329401

Sounds fragile though... as a first step

[bonus]
by adding new case for #24, it detects a bug. Needs to be fixed as
 if self.max_child_jobs is not None and len(running_task_ids) >= self.max_child_jobs:
     logger.info(f"Reach max_child_jobs, waiting to run task {self._gen_task_info(task)} on child job...")
+    task_queue.append(task)  # re-enqueue task to check later
     continue

I think the task execution order is not important because kannon don't ensure the order. So, I don't think the log order test is good.
I think the following test is better

all tasks are completed after kannon.build is finished
when a task run, all requires tasks should be completed.

What do you think about this? @yokomotod @maronuu

@kitagry Yes, I totaly agree that testing logs is not ideal way.
However,

all tasks are completed after kannon.build is finished
when a task run, all requires tasks should be completed.

is not enough I think. In addition,  we also want to assert that

TaskOnBullet had run in parallel as much as possible
with #24 , parallel number of  TaskOnBullet are limited correctly

One my idea is that recording events of task launching/completing and assert events.... but It look like the kind of a duplication of logging...
So I think we can start from testing logging output, then we can improve/disuss the test method with keeping test perspective. If it can be agreed, I will merge this PR and then create a issue for that.

One my idea is that recording events of task launching/completing and assert events.... but It look like the kind of a duplication of logging...

I have come to the same conclusion as yours.
Of course it's not an ideal one, but I think

we can improve/disuss the test method with keeping test perspective.

too.

@maronuu @kitagry I moved this test to integration_test (and improved PR description), so please review again!


NIFI-2585: Add attributes to track where a flow file came from when r…

For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically master)?

[X] Is your initial contribution a single, squashed commit?

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

I'm uncertain how to add a unit test for this as I'm not sure why TestStandardSiteToSiteProtocol is entirely commented out.

The failing appveyor CI tests are deal with ListFile and TailFile issues unrelated to this PR.

Reviewing...

@randerzander I tested this out and I think we have to move the addition of the attributes to inside the loop where the flow files are being created. If we do it after the loop, then session.transfer has already been called on each flow file and we can no longer update it, it was causing an error when I tested it out.
I made a couple of updates and have a working version in the latest commit here:
https://github.com/bbende/nifi/commits/NIFI-2585
The changes I made were the following:

Move the attributes into the while loop in AbstractFlowFileServerProtocol
Added the same attributes in StandardRemoteGroupPort for pull-based site-to-site
Used "s2s." prefix for the attribute names (I know I suggest "remote." in the JIRA, but I started thinking it wasn't descriptive enough).

I tested this out with two NiFi instances on my laptop doing push and pull based s2s and it appears to be working correctly.
If you agree with the changes you could put my commit into your branch and update this PR, or if you want to close this I can submit my branch with both our commits as a new PR.

@bbende Thanks for the review and for the fixes! I applied your changes to my fork and I'm able to receive FlowFiles over S2S.
However, when sending from minifi-cpp, I'm getting s2s.address with a value of "null:-1", and I'm not getting any s2s.host attribute.
From the nifi logs, it's using SocketFlowFileServerProtocol - would the SocketFlowFileServerProtocol or SocketRemoteSiteListener need an update to add these properties as well?

Thanks for pointing that out, let me give it a try with minif-cpp and see if I can reproduce. I was hoping that by putting the attributes in AbstractFlowFileServerProtocol it would automatically work since SocketFlowFileServerProtocol extends that, but let me give it a try. I'll let you know what I find out.

@randerzander I haven't been able to reproduce this yet. Running a simple GetFile -> RPG on minifi-cpp to a regular NiFi with InputPort -> LogAttribute, I am seeing the two attributes come through correctly.
I'm trying to look at the rest of the s2s code to see how a Peer instance could be created and not get the correct host or port. Would you be able to add the following code to the end of the Peer constructor?
https://github.com/apache/nifi/blob/master/nifi-commons/nifi-site-to-site-client/src/main/java/org/apache/nifi/remote/Peer.java#L51
if (this.host == null || this.port == -1) {
            throw new IllegalStateException("Unable to get host or port from peerUrl" + peerUrl);
        }

My theory is that it somehow the parsing of the URI is considered successful, but it didn't actually find  a host or port, since the Javadocs of uri.getPort() say it can return -1. Let me know if you can get that stacktrace from that added code to see what the peerUrl is when this happens. Thanks.

This may be an artifact of Docker generating hostnames that don't parse well.
Here's what I see in my testing:
monitor.dev_1      | 2016-12-09 19:41:34,127 INFO [Site-to-Site Worker Thread-6] o.a.nifi.remote.SocketRemoteSiteListener Received connection from techops_web-service.dev_1.techops_dev/<IP_ADDRESS>, User DN: null
monitor.dev_1      | 2016-12-09 19:41:34,127 ERROR [Site-to-Site Worker Thread-6] o.a.nifi.remote.SocketRemoteSiteListener Handshake failed when communicating with nifi://techops_web-service.dev_1.techops_dev:42626; closing connection. Reason for failure: java.lang.IllegalStateException: Unable to get host or port from peerUrlnifi://techops_web-service.dev_1.techops_dev:42626

@randerzander I wrote a simple Java main method with the following:
URI uri = new URI("nifi://techops_web-service.dev_1.techops_dev:42626");
System.out.println("Host: " + uri.getHost());
System.out.println("Port: " + uri.getPort());

Sure enough this prints the host as null and the port as -1. From what I can tell the "_" causes the URI to not parse properly. Since this appears to be based on the RFC for URIs, I don't think this should hold up merging this in.
I made an additional commit on my branch here https://github.com/bbende/nifi/commits/NIFI-2585 which in the case where host is null or port is -1, it will use the string "unknown" so that it looks a little nicer in the attributes.
Thoughts?

@bbende, thanks for spending time looking into this.
Makes sense to me. Since you did all the work, I'll close this PR and you can open a new one with your branch.


Coverage is buggy

It seems that the coverage of Pinocchio is not well updated at each master branch pushes.
@nim65s Can you fix this issue?

Are you talking about coveralls.io integration, or gitlab-ci ?

coveralls.io. But of course, we can synchronize gitlab with it, if you prefer.

Ok, I'll look at it

Solved by #530!


Getting APIError. Is this app dead now?

I haven't used it for a while but wanted some variety in my music so I launched it and get this API message:

Is this app dead now?

Thanks! It was a couple-week period where it wasn't working so I uninstalled it. I just reinstalled it from the app store and it's working just fine now.


#57: cage recursion

Closes #57

PR-Codex overview
The focus of this PR is to address recursion issues in the cage object by implementing a mechanism to detect and handle deep recursion.
Detailed summary

Added a blog post discussing recursion in cage
Updated the EOcage::lambda method to detect and handle recursion
Introduced PhTracedEnclosure to track and handle recursion in cage dataization

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@levBagryansky reading...

@maxonfjvipon Maybe there are problems with rho in this snippet?
# correct-cage-recursion.
[] > correct-cage-recursion
  cage > cge
    if.
      i.as-bool
      TRUE
      seq
        *
          i.write
            TRUE
          $.cge
  memory > i
    FALSE
  cge > @

@maxonfjvipon I think we should either release it as how it worked before, or delay its release until we can fix it in a future version.


Re-training SSD-Mobilenet

Hello, I have a problem about Re-training SSD-Mobilenet in fruit example..
I trained my model and I created ssd-mobilenet.onnx file. you look below.. But I take this error. I checked my folder path. ıt seen correct. Please Can you help me ?

Hi @AhmetEnesYalcinkaya , it is finding your model, but failing to parse it correctly (with the tensors.count(input_name) error)
Which version of JetPack are you running?
Can you try using the jetson-inference container to see if that works for you?   https://github.com/dusty-nv/jetson-inference/blob/master/docs/aux-docker.md

Thank you for solve. It worked fine..
I  wonder it "How I can change screen size (I want to do fullscreen) ?" and "How I can apply seen just box. don't conf etc.?( I used --overlay but it didnt worked")
Thank you again.

"How I can apply seen just box. don't conf etc.?( I used --overlay but it didnt worked")

You want to launch detectnet program with the --overlay=box option, for example:
$ detectnet --overlay=box /dev/video0

"How I can change screen size (I want to do fullscreen) ?"

You need to make a couple edits to jetson-inference/examples/detectnet/detectnet.cpp:

Add an #include "glDisplay.h" near the top
Add these lines below line 158:

if( output->IsType<glDisplay>() )
     ((glDisplay*)output)->SetMaximized(true);

cd jetson-inference/build and re-run make and sudo make install

It worked fine. (detectnet --overlay=box /dev/video0)

But I am wondering it why didnt it work in my-detection.py?

For example : net = jetson.inference.detectNet(argv=['--model=models/ssd-mobilenet.onnx', '--labels=models/labels.txt', '--input-blob=input_0', '--output-cvg=scores', '--output-bbox=boxes', '--overlay=box,label'], threshold=0.5)

Can I do fullsize in my_detection.py ?

But I am wondering it why didnt it work in my-detection.py?

For example : net = jetson.inference.detectNet(argv=['--model=models/ssd-mobilenet.onnx', '--labels=models/labels.txt', '--input-blob=input_0', '--output-cvg=scores', '--output-bbox=boxes', '--overlay=box,label'], threshold=0.5)

That's because the overlay flags are an (optional) argument to detectNet.Detect() as opposed to the constructor.
You can add this to your net.Detect() call in my-detection.py:
detections = net.Detect(img, overlay='box')

Can I do fullsize in my_detection.py ?

I think you would need to edit the glDisplay C++ implementation in jetson-utils.   Comment out this code block from your jetson-inference/utils/display/glDisplay.cpp:649 :
              /*if( !mResizedToFeed || ((GetWidth() < width || GetHeight() < height) && (width < mScreenWidth && height < mScreenHeight)) )
		{
			SetSize(width, height);
			mResizedToFeed = true;
		}*/

Then re-compile by running make and sudo make install again.

if( output->IsType<glDisplay>() )
     ((glDisplay*)output)->SetMaximized(true);

((glDisplay*)output)   give errorr ! you can check it this line ? @dusty-nv

What was the error?  Did you also #include "glDisplay.h"?

From: Ahmet Enes Yalçınkaya<EMAIL_ADDRESS>Sent: Tuesday, December 1, 2020 6:20:10 AM
To: dusty-nv/jetson-inference<EMAIL_ADDRESS>Cc: Dustin Franklin<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dusty-nv/jetson-inference] Re-training SSD-Mobilenet (#801)
if( output->IsType() )
((glDisplay*)output)->SetMaximized(true);
((glDisplay*)output) give errorr ! you can check it this line ? @dusty-nvhttps://github.com/dusty-nv
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dusty-nv/jetson-inference/issues/801#issuecomment-736448825, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADVEGK76VGK5KSRXEX2IE5DSSTGOVANCNFSM4TYN2FJQ.


Port 1514 is open on harbor appliance

The docker-compose files for harbor and notary have this port open for syslog. However, this service should be bound only to localhost, and not to any address (<IP_ADDRESS>).
Docker versions:
# docker version
Client:
 Version:      1.12.6
 API version:  1.24
 Go version:   go1.6.4
 Git commit:   78d1802
 Built:        Wed Jan 11 00:23:16 2017
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.6
 API version:  1.24
 Go version:   go1.6.4
 Git commit:   78d1802
 Built:        Wed Jan 11 00:23:16 2017
 OS/Arch:      linux/amd64
# docker-compose version
docker-compose version 1.11.1, build 7c5d5e4
docker-py version: 2.0.2
CPython version: 2.7.13
OpenSSL version: OpenSSL 1.0.1t  3 May 2016

@wy65701436 please see if we can bind it to <IP_ADDRESS> instead of <IP_ADDRESS>

fixed in #1938


hasOther has different behavior in dropdown question than dropdown cell in matrixdynamic

Hello,
if a dropdown question has the hasOther option activated when submitted the value in the result Json is as:

questionname=Other
questionname-comment=text in other

But a dropdown cell in a matrixdynamic question the result it is as:

questionnamecell=text in other

Is this an issue?
Regards,

@madceb We set the property storeOthersAsComment to false (true by default) for radiogroup/checkbox/dropdown questions. Do you have a problem with that?
Thank you,
Andrew

Hello @andrewtelnov
I prefer to have it as in the first option that it is the default behavior:

questionname=Other
questionname-comment=text in other

but in the cellType, with storeOthersAsComment to true or false, it still get the same with no second property with the comment.
Regards

We have implemented support for survey.storeOthersAsComment property in dynamic/dropdown matrices. It will be available from v1.8.4.
Thank you,
Andrew


Kaptain's Log is mucking about with DMagic's progress parser

File           : GameData/DMagicUtilities/ProgressParser/License.txt
Installing Mod : KaptainsLog 0.1.0
Owning Mod     : ProgressParser
CKAN Version   : v1.22.3+64309722a446

ProgressParser is needed fuctionality, but that should all be based on a depends relationship, not a direct inclusion in the release package. It's dying on the license file, but I presume that it would also die on the other files included in the release (which is everything, since he packaged it all together).
Should probably ask @linuxgurugamer to re-release without Progress Parser, and add it in as a dependency instead.

Being fixed now, sorry

No need to rerelease, I'm fixing the netkan.


flannel: Why does flannel call ipam.ExecAdd() to invoke delegated plugin

In the delegateAdd() of flannel plugin (https://github.com/appc/cni/blob/master/plugins/meta/flannel/flannel.go#L158), I see ipam.ExecAdd() is called to invoke the delegated plugin (e.g., bridge), it looks a little strange to me that a IPAM method is called to invoke a main plugin, Why not directly call invoke.ExecPluginWithResult() to invoke the main plugin?

@squaremo can you jump in here? Github told me you wrote the line in question :-)

Ahhh yes, I think I know this one -- the Exec* procedures originally lived in the ipam package. I factored them out so they could be used by libcni, leaving shims behind for convenience -- and apparently neglected to update this use. I'll correct it.


return list of impact areas

double check that all dimensions can be listed

@codymccoy we've been tracking the ID of a given impact area only. Does a list of the IDs of the impact areas meet your needs, or shall I refactor to return a List
We've not made use of the ImpactArea object. It seems thus far it has not been necessary, though this may be the use case requiring that we do use the ImpactArea object.

This is what I am currently doing:

And then i grab our impact areas and match to the id's:

I am not sure how to get the id's with an AlternativeResults object:

If we go with what i am currently doing, then we just won't display a row if the impact area doesn't exist anymore. If you had an internal list of the string names of the impact areas, then this wouldn't depend on the impact areas still existing in fda. It sounds like that might be tough to do. I am fine with the way it is currently working. I could maybe add a popup message telling the user that an impact area doesn't exist anymore that we have results for.

If we go with what i am currently doing, then we just won't display a row if the impact area doesn't exist anymore. If you had an internal list of the string names of the impact areas, then this wouldn't depend on the impact areas still existing in fda. It sounds like that might be tough to do. I am fine with the way it is currently working. I could maybe add a popup message telling the user that an impact area doesn't exist anymore that we have results for.

@codymccoy  What would it take to delete an impact area? We're not modifying the shapefile at all with the UI, right? So the user would have to delete the impact area set and reimport a new one without the given impact area. It seems if that is the case - then the user is undertaking effort to remove the impact area, and so I think it would then be natural to not display results for the deleted impact area.

You are correct. If the user deletes the impact area set, I believe we pop up a message that lists the scenarios that rely on those impact areas, and we delete those also if the user wants to continue with the delete.


step-by-step guide on how to use HWI and Bitcoin Core

This page shows in details through screenshots and description on how to use a hardware wallet with Bitcoin Core through the help of HWI. I hope this one-pager helps people to see how it works, takes their fear and increments the use of hardware wallets with Bitcoin Core.

Thanks Andrew @achow101 for your valuable feedback on PR #413.
I addressed nearly all of your issues and followed your suggestions.
Your suggestions have been added via the 3 commits 067e0c06f473d4fdd0519597b308877d0c4637d0 , 58379f6652531fb4b66c7be4997f72f7ef2a5c85 and 30b1bb4bb600df9fc79631410813c0162bd229cc.
Please have a look at the improved version. Let me know if there is anything else that needs to be done.
Thanks again and cheers!

Hi Andrew,
All done.
The commits are squashed.
Screenshots have nice descriptive names now.
Please have a look at this revised version. Thanks!

ACK 7de0237c5590efde7364209c30fc3dbb77544786

Great! The page is live at
https://hwi.readthedocs.io/en/latest/examples/walkthrough/walkthrough.html


chore(main): release 2.136.0

:robot: I have created a release beep boop
2.136.0 (2024-08-13)
Features

Add automatic tagging for github integration (#4028) (7920e8e)
Add tags for GitHub integration FE (#4035) (3c46a31)
Support Aptible deployments (#4340) (3b47ae0)
Use environment feature state instead of fetching feature states (#4188) (b1d49a6)

Bug Fixes

ensure that usage notification logic is independent of other organisations notifications (#4480) (6660af5)
Remove warning about non-unique health namespace (#4479) (6ef7a74)

Infrastructure (Flagsmith SaaS Only)

reduce task retention days to 7 (#4484) (4349149)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v2.136.0
:sunflower:


Parameter Data Type does not print.

The param data type or return type is not present when either of them is an annotated hibernate entity.
A simple POJO with getters and setters shows up fine. However something like this does not.
@Entity
public class BaseBills implements java.io.Serializable {

	private static final long serialVersionUID =<PHONE_NUMBER>319195892L;

	@Id
	private BaseBillsId ID;

	@Column(name = "PO_ID")
	private Integer PO_ID = 0;

	public BaseBillsId getID() {
		return ID;
	}

	@JsonProperty("ID")
	public void setID(BaseBillsId iD) {
		ID = iD;
	}

	public Integer getPO_ID() {
		return PO_ID;
	}

	@JsonProperty("PO_ID")
	public void setPO_ID(Integer pO_ID) {
		PO_ID = pO_ID;
	}

}

@pprabhu3430 what does your request mapping look like on the controller

Just a standard controller. Here is the full code of the rest controller method.
@RequestMapping(value = "/create", method = RequestMethod.POST)
@ApiOperation(value = "Create a Bill", notes = "Create a bill with this API. ", position = 1)
public @ResponseBody Result create(@RequestBody BaseBills bill) {
	billsDAO.create(bill, bill.getID());
    return new Result(true, "Successfully created bill with the following bill_id - " + bill.getID().getBILL_ID());
}

@pprabhu3430 could you share you repo perhaps? Its not clear why it would not render any of the properties.
Or you could try 2.0.1 snapshot and turn log level to debug for package springfox.documentation.schema

I'm closing this issue for now. Im unable to reproduce this issue. Feel free to reopen if it doesn't work in 2.0.1


Update Hash Provider pbkdf2 version to 0.1.5

Update Hash Provider pbkdf2 version to 0.1.5.
Related to,

https://github.com/wso2/product-is/issues/20708

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/10036189895

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/10036189895
Status: success


create a steal-socket.io

That does the nasty config for you.

This could also stub when running in Node so you don't have to do the typeof io check.

+1
Sent from my iPhone

On Sep 24, 2015, at 1:14 PM, Matthew Phillips<EMAIL_ADDRESS>wrote:
This could also stub when running in Node so you don't have to do the typeof io check.
—
Reply to this email directly or view it on GitHub.

This is done.  https://github.com/stealjs/steal-socket.io


QR Factorization

monomials(4) returns [array([1.]), array([1., 0.]), array([1., 0., 0.]), array([1., 0., 0., 0.])] as an output. How are we supposed to QR factorize this since it's not a complete matrix.

Recall that each column represents a polynomial. You have written code to compute inner products (and norms) of polynomials, and the notebook constructed projections of polynomials using those primitives. Your factor Q will also be a matrix of polynomials (represented as a list of arrays, each array representing a polynomial).


Hub connection doesn't work when having a proxy with authentication

@ilaborie Thanks for reporting - this is an issue around proxies with Docker-machine.

any luck fixing this bug ?

@ahalem We're still having problem with this issue. It's not an isolated issue with Kitematic as we use docker-machine under the hood.
Are you running into this problem? If so, would it be possible to give us logs to troubleshoot further?

Is Kitematic and Docker Machine connecting to IP or domain name  when connecting to the Docker Hub, would you please provide me the value for this IP or domain name ? I might be able to consult some network Engineer here, I got similar issue with Open VPN before.
———————————————————————
Amr Abdelhalem
Principal Architect - Application Tier Engineering
ADP, LLC – Roseland, NJ | Office:<PHONE_NUMBER>  - Application Tier 24/7 Support Hotline: 1-855-848-3895
“Creativity is just connecting things. When you ask creative people, they feel a little guilty because they didn't really do it, they just saw something”  - Steve Jobs
From: Michael Chiang <notifications<EMAIL_ADDRESS>Reply-To: kitematic/kitematic <reply<EMAIL_ADDRESS>Date: Friday, June 12, 2015 at 4:24 PM
To: kitematic/kitematic <kitematic<EMAIL_ADDRESS>Cc: "Amr (CORP) Abdelhalem" <Amr.Abdelhalem<EMAIL_ADDRESS>Subject: Re: [kitematic] Hub connection doesn't work when having a proxy with authentication (#562)
@ahalemhttps://github.com/ahalem We're still having problem with this issue. It's not an isolated issue with Kitematic as we use docker-machine under the hood.
Are you running into this problem? If so, would it be possible to give us logs to troubleshoot further?
—
Reply to this email directly or view it on GitHubhttps://github.com/kitematic/kitematic/issues/562#issuecomment-111604571.

This message and any attachments are intended only for the use of the addressee and may contain information that is privileged and confidential. If the reader of the message is not the intended recipient or an authorized representative of the intended recipient, you are hereby notified that any dissemination of this communication is strictly prohibited. If you have received this communication in error, notify the sender immediately by return email and delete the message and any attachments from your system.

#1031 Merging with

If anyone still has problems with this on Mac or Windows 10+, please contact me at<EMAIL_ADDRESS>We are working on something to help address this issue.


Type tests: complete and fine-tune

This PR completes the testing of types with several functions:

events:

alert
prompt
confirm
beforeunload

helpers:

evaluate
to
into
waitfor
accept
dismiss
getConfig
setConfig

proximity selectors:

toLeftOf
toRightOf
above
below
near

Additionally this PR refines the approach to types returned by selectors: e.g. now button() returns a ButtonWrapper instead of a generic ElementWrapper.
This allows to better specify which members are allowed on each returned wrapper.

See also https://github.com/getgauge/taiko/issues/1465#issuecomment-691895805


initial TheKey configuration & logout support

Thanks for the head start on this, Daniel.

gladly, feel free to direct questions regarding The Key OAuth library my way, I'm the one that wrote it, so I should be able to help with it :)


Avatar status item text and tooltip support

SUMMARY:

Add Avatar status item text and tooltip support (this is needed because we required Avatar+Tooltip support in AvatarGroup but it was not working because of use of cloneElement in AvatarGroup)
Improve Avatar status item placement CSS
Fix undesired Avatar status item displacement in AvatarGroup due to border

GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
ENG-40399
CHANGE TYPE:

[x] Bugfix Pull Request
[x] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:
Pull the PR branch and run yarn and yarn storybook. Verify the Avatar Group stories behave as expected.
Avatar status item text

Avatar tooltip

Avatar status item displacement in AvatarGroup - before and after the fix

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@993d98a). Click here to learn what that means.
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #560   +/-   ##
=======================================
  Coverage        ?   84.82%
=======================================
  Files           ?      791
  Lines           ?    16845
  Branches        ?     5919
=======================================
  Hits            ?    14289
  Misses          ?     2490
  Partials        ?       66

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


OIDC-Issuer flag ignored (Github Actions)

Description
Hi folks,
I had a strange behaviour while trying to sign a blob via GitHub Actions with keyless-signing. The command seems to ignore the --oidc-issuer flag which was set to the GitHub-OIDC-Endpoint and uses the default Sigstore-Issuer instead. I then did the verification-login to sigstore manually. The verification for the received token then failed with the message, that the token-issuer (sigstore) was not the expected one (github).
Cosign-Version: 1.4.1
$ COSIGN_EXPERIMENTAL="1" cosign sign-blob --oidc-issuer=https://token.actions.githubusercontent.com --output-signature provenance.json.sig provenance.json

Using payload from: provenance.json
Generating ephemeral keys...
Retrieving signed certificate...
Non-interactive mode detected, using device flow.
Enter the verification code SNMG-GQNG in your browser at: https://oauth2.sigstore.dev/auth/device?user_code=SNMG-GQNG
Code will be valid for 300 seconds
Token received!
Error: signing provenance.json: getting key from Fulcio: retrieving cert: oidc: id token issued by a different provider, expected "https://token.actions.githubusercontent.com" got "https://oauth2.sigstore.dev/auth"
main.go:46: error during command execution: signing provenance.json: getting key from Fulcio: retrieving cert: oidc: id token issued by a different provider, expected "https://token.actions.githubusercontent.com" got "https://oauth2.sigstore.dev/auth"

When someone needs more information for this issue, please reach out to me!
Thank you all!!

If you're running this in the context of a GitHub Action workflow run, you shouldn't have to specify --oidc-issuer to leverage the GitHub OIDC Token that is available for the scope of the particular run.
Did you set the correct permission on your workflow definition to make sure the ID token would be made available?
https://docs.github.com/en/actions/deployment/security-hardening-your-deployments/configuring-openid-connect-in-amazon-web-services#adding-permissions-settings

okay, so maybe I can omit the --oidc-issuer flag, as it runs within GitHub. However, it seems to be ignored even its explicitly set.
The environment is the following:

GitHub-Action with correct permissions (id-token) calls a workflow via nektos/act and sets the GITHUB_TOKEN as secret.
The "inner" workflow runs an action with the cosign command, specified above.

I think the Github-Context is lost in this scenario, but the outer GITHUB_TOKEN is reused in the act-workflow.

do we have to manually click on the link generated to enter the verification code, for each workflow run?
if we don't, we're getting this error https://github.com/kyverno/kyverno/runs/4729805789?check_suite_focus=true#step:10:16

For anyone who runs into this issue, use the --oidc-provider flag to set which provider you want to use.


Example for sending without listening?

I have a remote service which is listening for UDP packets at a specific host/port.
Is there an example of how I would, from react-native, bind a socket to send to that host/port? The examples all seem based on listening (server) but I want to send without any listening (client).

@GeoffreyPlitt the example shows how to send: https://github.com/tradle/react-native-udp/blob/master/examples/rctsockets/index.ios.js#L76. I don't remember whether I ever tried it without listening. Does it not work for you?

Gotcha, will try that!


mdTableEdit as textarea

Hello,
we need a way to edit a textarea in a table. The mdTableEdit only provides an input field.
I could implement it, but i have to know the api (own component or a property for mdTableEdit)
And the mdTableEdit is missing in the docs since v0.4.0. Is there a reason for it or have you forgotten it?

Hi @kpilard.
Yes. I didn't mentioned because it's not finished yet. I will improve the md-table component with the table-edit. J would love to see an improvement over this element, but I would recommend you to way until the next version.
Thank you.

Yes i will wait for the next version


Timeout keeps coming out. (JAVA SDK)

The API key is also correct, but why does Time out keep appearing?
System.out.println("Event1111");
		Amplitude amplitude = Amplitude.getInstance();
		amplitude.init("API KEY");
		amplitude.setLogMode(AmplitudeLog.LogMode.DEBUG);
		amplitude.setFlushTimeout(2000L); // 2 seconds

		AmplitudeCallbacks eventCallbacks =
				new AmplitudeCallbacks() {
					@Override
					public void onLogEventServerResponse(Event event, int status, String message) {
						// Event: Event processed.
						// status: response code, like 200, 400, etc.
						// message: success or error message.
						System.out.println(event.toString());
						System.out.println(status);
						System.out.println(message);
					}
				};
		amplitude.logEvent(new Event("btn_clicked_test", `"test_user_id"),eventCallbacks);

Log
END: Events count 1
RESPONSE: {
"code": 408,
"status": "TIMEOUT"
}
RETRY: Events count 1
RESPONSE: {
"code": 408,
"status": "TIMEOUT"
}
RETRY: Events count 1
RESPONSE: {
"code": 408,
"status": "TIMEOUT"
}
RETRY: Events count 1
RESPONSE: {
"code": 408,
"status": "TIMEOUT"
}
RETRY: Events count 1
RESPONSE: {
"code": 408,
"status": "TIMEOUT"
}
RETRY: Events count 1
RESPONSE: {
"code": 408,
"status": "TIMEOUT"
}
RETRY: Events count 1
RESPONSE: {
"code": 408,
"status"

Hi @Mercy811.
Is there a way to check if amplitude API calls are working on my computer?
If I just call the api in the browser, it will look like the following.
{"code":400,"error":"Missing request body"}

Hi @hyha9, you are receiving 400 because you don't send a body with your request. Please

follow the example here
change the payload to what you want to send
to see if you could make connections with Amplitude HTTP V2 API.

Are there other possible problems besides network problems?
I cannot think of any as you are getting 408 error code which means that the server would like to shut down this unused connection. So highly possible a network problem.

I got this error.
sun.security.validator.ValidatorException: PKIX path building failed: sun.security.provider.certpath.SunCertPathBuilderException: unable to find valid certification path to requested target
When I call the API directly, I get the following error: When it's https, an error pops up, but when it's called with http, it works fine.

@hyha9 Could you provide an example project that can reproduce this error consistently? The code snippet you provided works without any error so unfortunately I cannot reproduce it.


Crypto login

AES encryption working for user change password, and user change username.
Need to encrypt AES symmetric key, but this is a good starting point.

I will look at this tomorrow if you remind me. Too late right now. Burning
a little N64 and crashing.
On Feb 13, 2018 12:20 AM, "Mike Mulchrone"<EMAIL_ADDRESS>wrote:

AES encryption working for user change password, and user change username.
Need to encrypt AES symmetric key, but this is a good starting point.
You can view, comment on, or merge this pull request online at:
https://github.com/mtmulch/mulchronedigital/pull/59
Commit Summary

make sure keys are encoded in base64 format
AES symmetric encryption working on request bodys for changing
username and user password.

File Changes

M client/app/modules/dashboard/user-dashboard/components/
user-dashboard-change-password/user-dashboard-
change-password.component.ts
https://github.com/mtmulch/mulchronedigital/pull/59/files#diff-0
(45)
M client/app/modules/dashboard/user-dashboard/components/
user-dashboard-change-username/user-dashboard-
change-username.component.ts
https://github.com/mtmulch/mulchronedigital/pull/59/files#diff-1
(40)
M client/app/shared/services/user-dashboard.service.ts
https://github.com/mtmulch/mulchronedigital/pull/59/files#diff-2 (9)
M server/models/user.ts
https://github.com/mtmulch/mulchronedigital/pull/59/files#diff-3 (4)
M server/routes/classes/BaseRouter.ts
https://github.com/mtmulch/mulchronedigital/pull/59/files#diff-4
(19)
M server/routes/users/UserDashboard.ts
https://github.com/mtmulch/mulchronedigital/pull/59/files#diff-5
(12)
M shared/Encryption.ts
https://github.com/mtmulch/mulchronedigital/pull/59/files#diff-6
(45)
M shared/UniqueIdentifer.ts
https://github.com/mtmulch/mulchronedigital/pull/59/files#diff-7
(22)

Patch Links:

https://github.com/mtmulch/mulchronedigital/pull/59.patch
https://github.com/mtmulch/mulchronedigital/pull/59.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mtmulch/mulchronedigital/pull/59, or mute the thread
https://github.com/notifications/unsubscribe-auth/ATe95Eu58-rZK13z494ZuTqGSEmhKynIks5tUSmrgaJpZM4SDRV5
.


Add extension-dapp section and extend util-crypto for signature verification

There are some tiny repetitions between usage and cookbook, but I feel what's in usage is of value to users that already have vaguely an idea of how to use such interfaces, whereas the cookbook provides more steps.

I'm discovering this options?: Partial<SignerOptions> param from signAndSend, it's retrieved automatically if we pass the address from a keyPair, but we should consider as best practice to specify the signer is that right?

I added the signer explicit everywhere. So when an address is specified, it will -

try and use the signer specified in the options
if not available, use the global api signer (the code we removed)


Port to Redux

I intend to port this example to Redux.
I'm not doing that to promote my library, but because reusable store utilities (like pagination) are way easier to implement as pure functions. What I have right now in PaginatedStoreUtils is a mess, and will be much more sensible with Redux approach.
However, I understand that not everybody will be happy with this change, as Redux is not exactly Flux. I think it solves the same problems better than Flux, but that's my opinion and some may not share it.
Therefore I ask what you think:

Should I rename the repository to reflect that it's a Redux example now?
Should I keep the old vanilla Flux code in a branch?

To clarify again:

I intend to keep it minimal, just like the current repo, and handle the same cases: pagination, “load more”, spinners.
The “content stores” / “list stores” distinction will be even clearer with reusable reducers.
This is not an artificial effort to port for the sake of porting. The code will be clearer after porting to Redux, and the idioms currently used by the code will be more apparent and natural.

Thoughts?

Imho rename and kee pthe flux in an old branch and keep a link in the readme.... Between redux and react-router 1.0 there is going to be A LOT of changes. Also, thanks for all the work you've been doing with reflux lately!

like @felipeleusin has said it, rename it and keep the old version. maybe as branch or maybe even only as link to current revision.

agree with @felipeleusin. V. interested in seeing this port.

It might be better to make a new repository redux-react-router-example

Awesome, can't wait for this!

The port will be part of the official Redux repo.
PR: https://github.com/gaearon/redux/pull/460

The Redux port is here: https://github.com/rackt/redux/tree/master/examples/real-world


How to decide which output file to use when using i18n plugin?

I use i18n plugin to generate multi-language output files,
e.g.
en_US.output.js
zh_CN.output.js
What's the best practice to decide whick file should be include?
writing judgement inside my backend template
or
writing another webpack.config.js file?

That's backend logic. The Accept-Language header can be used for this purpose.

Thanks any way.
Our back-end guy use cookie to set language.
I use JavaScript to get the cookie value.


All tab appears when library is ungrouped

Steps to reproduce

Set your library to ungrouped, only having the default category may exhibit this issue as well
Open the library or exit a manga
The category tab shows for a moment then disappears, or stays there

Expected behavior
The All category tab should always remain hidden
Actual behavior
The category tab shows for a moment then disappears, or stays there
Crash logs
No response
Tachiyomi version
Preview R405
Android version
Android 9
Device
Amazon Fire HD Plus 2021
Other details
This issue seems to not be exhibited in the latest Tachiyomi Preview r3783 with the Default Category
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.7.0.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

This is a main Tachi bug, you can reproduce it like this

Have multiple categories
Disable Library -> Filter -> DIsplay -> show category tabs
Issue reproduced


Weird output results (poa_global)

Why is poa_global greater than ghi?
When I run the official case (https://pvlib-python.readthedocs.io/en/stable/gallery/adr-pvarray/plot_simulate_system.html#sphx-glr-gallery-adr-pvarray- plot-simulate-system-pay), I cannot understand the results.
Is there any physics or maths behind this that can be explained?

Best regard!!!!!!!!

I think it is a misunderstanding of the terms. You can check out the mini-definitions we have on the Variables and Symbols page.
GHI is the global horizontal irradiance, i.e., the irradiance that gets to an horizontal surface (more often the terrain than a PV collector).
https://pvpmc.sandia.gov/modeling-guide/1-weather-design-inputs/irradiance-insolation/global-horizontal-irradiance/
On the other hand, POA_global is all the irradiance that gets to a collector. You would usually tilt a collector to minimize the angle of incidence of the beam component. See https://pvpmc.sandia.gov/modeling-guide/1-weather-design-inputs/plane-of-array-poa-irradiance/ and subsequent sections.
From the equations point of view:

$GHI = DHI + DNI cos(\theta_Z)$, where $\theta_Z$ is the solar zenith.
$POA_{global} = POA_{direct} + POA_{sky-and-ground-diffuse}$ where the direct component will be often the most important. $POA_{direct} = DNI cos(AOI)$. A designer will usually try to make AOI, the angle of incidence, as small as possible (face the surface to the Sun essentially) so it gets more irradiance ===> usually, $AOI < \theta_Z$ for the Sun above horizon.

The point of $DHI$ and $DNI$, and/or $GHI$ is that they can be measured in a normalized way everywhere, and POA components depend heavily on the orientation of the collector surface. So translating the former values to POA components has more maths involved (transposition, DNI estimation, ...; depends on input data) than measuring the available irradiance resource (DHI, DNI, GHI).
I suggest you some references:

https://pvpmc.sandia.gov/modeling-guide/
https://www.pveducation.org/

Feel free to close if cleared up😃


unnecessary-constructor prompts to remove necessary constructor

Bug Report

TSLint version: 6.0.0-beta1
TypeScript version: 3.6.4
Running TSLint via: gulp-tslint

TypeScript code being linted
class A {
        private _x: number;
        protected constructor() {
            this._x = 43;
        }
    }

    class B extends A {
       // unnecessary-constructor  Remove unnecessary empty constructor.
        public constructor() {
            super();
        }
    }

with tslint.json configuration:
"unnecessary-constructor": {
            "options": {
                "check-super-calls": true
            }
        }

Actual behavior
unnecessary-constructor  Remove unnecessary empty constructor. error reported
Expected behavior
rule should take into account visibility of base constructor

Indeed. As a followup to #4511 this should not be reported.
Accepting PRs for.. the next ~10 days! Per #4534. 😄

Whoops, didn't found that issue. Anyway, probably I need to find some time and finally move to eslint :smile:

💀 It's time! 💀
TSLint is deprecated and no longer accepting pull requests other than security fixes. See #4534. ☠
We recommend you instead use typescript-eslint to lint your TypeScript code with ESLint. ✅
👋 It was a pleasure open sourcing with you!

🤖 Beep boop! 👉  TSLint is deprecated 👈  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


[Question][CI/CD] How can we stop auto collection of CI/CD metrics?

Question
Hi, what is the correct config to use if we do not want to automatically collect deployment/builds/etc data? Referencing https://devlake.apache.org/docs/DataModels/DevLakeDomainLayerSchema#data-models, I have tried not choosing CI/CD for my data scope config...but deployment data is still being collected. Using v1beta5 with azure devops GO connection
FYI: i asked this also in slack. see https://devlake-io.slack.com/archives/C03APJ20VM4/p1714025708040129
Screenshots

@thiDucTran Hi, by design, the previously collected data of selected entities won't be deleted but simply skipping those related subtasks,  please check if the collectApiBuilds subtask is showing in the pipeline plan JSON for the plugin

i dont see collectApiBuilds
    [
      {
        "plugin": "azuredevops_go",
        "subtasks": [
          "collectAccounts",
          "collectApiPullRequests",
          "convertRepo",
          "extractAccounts",
          "convertAccounts",
          "extractApiPullRequests",
          "collectApiPullRequestCommits",
          "convertApiBuilds",
          "convertApiPullRequests",
          "convertPrLabels",
          "extractApiPullRequestCommits",
          "convertApiPullRequestsCommits",
          "convertApiTimelineRecords"
        ],

Weird, why there is a convertApiBuilds in the subtasks list.
It looks like a bug indeed, would you like to file it in a separate issue and we will look into it?

@thiDucTran Already effective in v1.0.0-beta6

issue is not resolved for me in  v1.0.0-beta6 ? CI/CD metrics still gets collected... I even deleted the project...purge scope's data...make sure CI/CD is not in my scope config...colleted data..and I still see ci/cd metrics in Azure DevOps dashboard

@thiDucTran What is the value of your environment variable ENABLE_SUBTASKS_BY_DEFAULT?

@thiDucTran That is weird, I don't see any related subtasks in the list. Can you check the raw tables and see if the data gets purged?

i edited my previous comment....seems data is not purged

To be investigated.

Hi, would I need to create another issue for the table cicd_deployment_commits ? I have delete the project and data scope..but data of of deleted project is still there in cicd_deployment_commits


getChatHistory error only for Telegram official chat 777000

I am using getChatHistory to get messages but getting the below error only for the Telegram Official chat with chat id = 777000
     {'code': 400, 'message': 'Invalid value of parameter from_message_id specified'}

the from_message_id that I pass in is larger than the max message id in the chat by 1, for example if the max message id in a chat is 12345 I would pass in the value 12346.
This has been working for every other chat, and it has been working before for the Telegram official chat before until today.
If I call getChatHistory() with the max message id, it works fine.
Additional note that might be relevant to the issue is that, I am also using an offset of -99 in order to get messages newer than from_message_id

The error message can't be wrong. You send an invalid request with an invalid from_message_id parameter. See https://core.telegram.org/tdlib/getting-started#getting-chat-messages for the correct way to get chat messages and ensure that you pass a message identifier received previously from TDLib to getChatHistory as from_message_id.

I was under the assumption that we can use any int53 number as the from_message_id.
But it sounds like if we use a from_message_id that doesn't belong to a message in this chat, there's no guarantee of validness?

But it sounds like if we use a from_message_id that doesn't belong to a message in this chat, there's no guarantee of validness?

If there could be a message with the provided from_message_id in the chat, then it will work correctly.

Thank you


Reverse Linked List in group of K

Is your feature request related to a problem? Please describe.
Yes, my feature request is related to a problem. I would like to write a sorting algorithm for sorting a linked list in group of k using Cpp.
Original Linked List-

Reverse Linked List (k=3)-

I Will provide the solution for Reverse linked list in group of k problem with proper comments and documentation.
I would like to work on this issue.
Could you please assign this issue to me under SSoC ' 23.

@pearlrathour Could you implement this in all four languages?

Yes, I will provide code in all 4 languages.

Assigned!
@pearlrathour : C, C++, Python and Java