Authenticate endpoints return HTML messages on fail

Expected Behavior
When checking if Node.js application is still authenticated with openBalena server using isLoggedIn, I expect it to return a boolean with true / false, even if current token is expired.
Actual Behavior
When true, everything goes well. If never tried to login (no ~/.balena/token file), it returns false. But if token is expired, it returns a 404 error trying to get whoami and I expected it to be false at that time.
BalenaRequestError: Request error: <!DOCTYPE html>

<meta charset="utf-8">
<title>Error</title>

Cannot GET /whoami

    at Object.<anonymous> (~/node_modules/balena-request/build/request.js:190:27)
    at Generator.next (<anonymous>)
    at fulfilled (~/node_modules/tslib/tslib.js:111:62)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:97:5) {
  body: '<!DOCTYPE html>\n' +
    '\n' +
    '\n' +
    '<meta charset="utf-8">\n' +
    '<title>Error</title>\n' +
    '\n' +
    '\n' +
    '
Cannot GET /whoami
\n' +
    '\n' +
    '\n',
  statusCode: 404,
  requestOptions: {
    url: '/whoami',
    baseUrl: 'https://api.<balena-server>/',
    refreshToken: false,
    timeout: 59000
  }
}

Steps to Reproduce the Problem

Run a openBalena server (Can't test it with balenaCloud atm)
Authenticate with server using sdk.auth.login
Wait till token expires (I don't really know the expiry time)
Check if SDK is logged in using sdk.auth.isLoggedIn

Specifications

SDK version: 15.24.0
Browser:
Node version: 14.3.0
Npm version: 6.14.5
OS: macOS 11.1

References

Internal discussion thread:
See: https://www.flowdock.com/app/rulemotion/p-opensource/threads/qnGgwmvppSbFItlU5_U9Rk5JtPn

Hi @bartversluijs ,
Thanks for reporting this.
After some digging I figured out that this happens because balena-request it trying to auto-refresh the token when your token is not fresh any more
See: https://github.com/balena-io-modules/balena-request/blob/1e1f0269772680da4ec6c02f58defcbcf34f4468/lib/request.js#L80-L89
and it does so by using the /whoami endpoint which at the moment is not part of open balena
See: https://github.com/balena-io-modules/balena-request/blob/1e1f0269772680da4ec6c02f58defcbcf34f4468/lib/request.js#L389
I will have to start an internal discussion about this before we can fix the issue.
In the meantime as a way to unblock you I would like to suggest you to use the models.apiKey.create() method o generate an api key which isn't expiring and use that to authenticate with the SDK using the auth.loginWithToken() method. When using an api key you should no longer face this 404 error.

After discussing this internally we ended up that open-balena should offer the /user/v1/refresh-token endpoint.
Once that's done we can fix this by just switching balena-request to use the new endpoint.
Depends-on: https://github.com/balena-io/open-balena-api/issues/575

The /user/v1/refresh-token endpoints are now available in open-balena-api v0.181.0 and balena-request has already been changed to use the new endpoitns in v11.3.0, which the SDK has picked up in v15.30.0
See: https://github.com/balena-io-modules/balena-request/blob/master/CHANGELOG.md#v1130
See: https://github.com/balena-io/balena-sdk/blob/master/CHANGELOG.md#15300---2021-03-12


Added panel regarding ammount of HPA

added panel regarding HPA current/max replicas
Relates to #3911

Added 😄 @zroubalik

Would you mind posting a screenshot please?

Would you mind posting a screenshot please?

Do we want to see just over time or also current? We might want to add a gauge?
https://grafana.com/docs/grafana/v9.0/visualizations/#gauge

Do we want to see just over time or also current? We might want to add a gauge? https://grafana.com/docs/grafana/v9.0/visualizations/#gauge

Do you want to add a new one or to replace the current one with gauge?

It's just an idea but I would add both

hey @zroubalik can you comment regarding this change?

 also tried to added this as a gauge and I want your opinions about the look (I liked the time based better because I found the option using the graph to show the current vs max when it let you shows what the max value is).
But please also comment on this and if you think it's helpful I'll upload it so we could deliver the grafana dashboard to users 😄  @tomkerkhove

I think both are perfect - Left shows current status easily, right you can see it over time if you want more details.
What are your thoughts @zroubalik @JorTurFer ?

So I'll add the second panel today to the PR and merge ok?
@tomkerkhove @zroubalik

So I'll add the second panel today to the PR and merge ok? @tomkerkhove @zroubalik

yeah :)

I can't seem to be authorize to merge to this branch so whenever you can please merge 😄

I can't seem to be authorize to merge to this branch so whenever you can please merge 😄

That is by design :)
Thank you!

@yuvalweber Do you have an example screenshot of the full dashboard that we can add to our release blog post?

yeah of course I'll do it today @tomkerkhove

@yuvalweber Did you manage to create one? If not, no problem.

Sorry for late response.
This is what I have @tomkerkhove


ListView has been removed from React Native

Can you please update the library? as many of my readers are reporting the bug in the library at https://aboutreact.com.
I think the problem is related to the Listview being deprecated in the React Native 0.56 and your library has used the ListView. Can you please shift it to the FlatList?
It is showing the following error:
E | ReactNativeJS ▶ ListView has been removed from React Native. See https://fb.me/nolistview for more information or use deprecated-react-native-listview.
E | ReactNativeJS ▶ Module AppRegistry is not a registered callable module (calling runApplication)

I wrote a hotfix PR for this issue, and am using it successfully in my application. #70

Can you please update the library? as many of my readers are reporting the bug in the library at https://aboutreact.com.
I think the problem is related to the Listview being deprecated in the React Native 0.56 and your library has used the ListView. Can you please shift it to the FlatList?
It is showing the following error:
E | ReactNativeJS ▶ ListView has been removed from React Native. See https://fb.me/nolistview for more information or use deprecated-react-native-listview.
E | ReactNativeJS ▶ Module AppRegistry is not a registered callable module (calling runApplication)

To solve this issue, go to this repository and replace all of the contents in the lib folder with the current repository's lib folder. All the examples will work with no issue.

Thank you @nadernt. Are you the maintainer of that library?

Hi Snehal, no I don't. It is a good idea to update this repository because the mentioned repository looks a fork of react-native-timeline-listview.

@nadernt Ok. Thank you for the reference to the new library. It is working fine.


Fix SQL Bindings README

This PR fixes the links shown in the SQL Bindings extension when installed:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.007%) to 42.045%

Files with Coverage Reduction
New Missed Lines
%

extensions/notebook/src/common/ports.ts
4
74.14%

Totals

Change from base Build<PHONE_NUMBER>:
-0.007%

Covered Lines:
28013

Relevant Lines:
62324

💛  - Coveralls


Does lmerge have a CI cutoff value?

Hello again, I have a question regarding lmerge's methodology and would appreciate your insight. I have been reading through the files and wanted to make sure that my understanding was correct. In lmerge.py, the merge function is called by r_cluster. If the given break point has a larger minimum starting position than the current maximum ending position of the last SV, then merge is called on all SVs previously found to have overlapping position intervals in the same chromosome (lines 525-536).
I understand this, but I am concerned-- does lmerge account for calls that are wildly imprecise (say a call with a start position with confidence intervals plus or minus 100 bp)? I have seen some similar calls in Lumpy VCFs when just considering the 95% confidence interval. To combat this problem, my team was using the 95% CI and made a imprecise cutoff value at +/- 10 bp. Given that lmerge is utilizing the entire CI, I would think that there is some way of accounting for this.
I will try to figure out the specifics of the merge function next, as I am trying to internalize the criteria used by lmerge. I am curious as to whether SV type is considered (as my team suggested it ought be) and if not, why? I also am going to try and find the section of code where probability curves are used and try to understand that. I just figured that I would ask questions as I go along so that I do not forget them.
Thanks again for all of the help.

On Wed, Jun 27, 2018 at 3:20 PM egoetz<EMAIL_ADDRESS>wrote:

Hello again, I have a question regarding lmerge's methodology and would
appreciate your insight. I have been reading through the files and wanted
to make sure that my understanding was correct. In lmerge.py, the merge
function is called by r_cluster. If the given break point has a larger
minimum starting position than the current maximum ending position of the
last SV, then merge is called on all SVs previously found to have
overlapping position intervals in the same chromosome (lines 525-536).
I understand this, but I am concerned-- does lmerge account for calls that
are wildly imprecise (say a call with a start position with confidence
intervals plus or minus 100 bp)? I have seen some similar calls in Lumpy
VCFs when just considering the 95% confidence interval. To combat this
problem, my team was using the 95% CI and made a imprecise cutoff value at
+/- 10 bp. Given that lmerge is utilizing the entire CI, I would think that
there is some way of accounting for this.

I am not sure I would call an SV with +- 100 bp wildly imprecises and I
don't think it is a problem per se. Large CIs are fairly common with SVs
that are supported only by paired-end reads, and in cases were an SV is
flanked by repeative regions you will not get split-reads. If you are doing
an analysis that needs single-base precision I would expect that even 10bp
could be too wide.
Each posistion in the CI has a probablity associated with it, and use the
product of all CIs to get the merged CI. If there is a mix of paired-end
(wide CI) and split-read (narrow CI) in a given cluster, then the product
with be narrow.

I will try to figure out the specifics of the merge function next, as I am
trying to internalize the criteria used by lmerge. I am curious as to
whether SV type is considered (as my team suggested it ought be) and if
not, why?

Type is considered as so far as we do not merge evidence for a DEL with
evidence for a DUP. Are you referring to a different type of consideration?

I also am going to try and find the section of code where probability
curves are used and try to understand that. I just figured that I would ask
questions as I go along so that I do not forget them.

The process used by lmgere is very similar to the process used by LUMPY,
so you may want to check out that paper as a companion to the code.

Thanks again for all of the help.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/hall-lab/svtools/issues/262, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAlDUdGWd8-FHHY15j9FQDTXd4jL0d6vks5uA_cigaJpZM4U6Z5w
.


breaking: update debug-plugin to use install method

close #91

Codecov Report

Merging #134 into master will decrease coverage by 0.4%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #134      +/-   ##
==========================================
- Coverage   96.01%   95.61%   -0.41%
==========================================
  Files          27       27
  Lines         226      228       +2
==========================================
+ Hits          217      218       +1
- Misses          9       10       +1

Impacted Files
Coverage Δ

src/plugins/debug-plugin.js
90.9% <100%> (+0.9%)
:arrow_up:

src/util/debug-message.js
50% <0%> (-50%)
: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 748a12d...2a07592. Read the comment docs.


Minor updates on tests

Codecov Report

Merging #8 into main will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main        #8   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            9         9
  Lines           80        80
=========================================
  Hits            80        80

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 3184a05...a402d3a. Read the comment docs.


Add open graph template

It can be used e.g. for sending songs:
https://developers.facebook.com/docs/messenger-platform/send-messages/template/open-graph

Great! What do you think if I just add you to the list of the project contributors so you don't have to use the fork and can just commit directly and close PRs on your own? You seem to be hands on with the codebase now, unlike me.


Updating plots.py

fixing up plots.py, added common.py for testing purposes (can remove later if unnecessary)

pos has been removed

I've fixed the first and third issues, but for 2) how should I notate large negative values? The graphs graph the mean and standard mean error and do not actually show individual negative outliers. Is there a standard practice for this?

You can leave the individual outliers alone. From what I see component >=3 are entirely blank, so we certainly need to make it better. Merge this PR first, and we can open a new one to solve this.


Move api handlers to separate file

Cleans up server.go so it pretty much just has the setup stuff.

+1 (this needs rebase unluckly)

rebased

I wonder if it makes more sense to divide the handlers in specific files, like we've done with the client.

I've thought about it... at least something like containers.go, images.go, exec.go

I'm +1 for "domain" split, this file is way to huge ;-)

@cpuguy83 any update on this? I think it's welcome but I agree with others it needs to be split like the client is.

By "domain" to me is different than by action like the cli is today.
I guess if we just clarify which way to go I can get this going.

Splitting by domain sounds like a good start, but I'm not really the "coding guy" here, so just my 0.02 ;-)

@cpuguy83 +1 on domain, rebase please :)

Updated

cool. I like it much much better :fireworks: LGTM.

Nice! LGTM (fwiw)

it needs a rebase. Let's try to merge this as soon as janky is happy.

Rebased again

@cpuguy83 I tentatively moved this to "ready to merge", Following the above discussion, I think that's correct here? Or do you want another final LGTM check before merging?

LGTM


[TI-2735] Add DesignerPencil16, DesignerPencil24, productManager16 and productManager24 icons

TI-2735
Description
Add DesignerPencil16, DesignerPencil24, productManager16 and productManager24 icons
Figma: https://www.figma.com/design/TqaGgbpjGSUDf7qq153Isq/Iconography?node-id=3126-3&t=y7w4QkGKRfhXrdiU-0
How to test

Temploy

Screenshots

description
image

product manager icons

designer pencil icons

Development checks

[ ] Add changeset according to guidelines (if needed)
[ ] Read CONTRIBUTING.md and Component API principles
[ ] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[ ] Annotate all props in component with documentation
[ ] Create examples for component
[ ] Ensure that deployed demo has expected results and good examples
[ ] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[ ] Self reviewed
[ ] Covered with tests (visual tests included)

Breaking change

[ ] codemod is created and showcased in the changeset
[ ] test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers


Add enum values to the GNSS message, for readability

The fix type field in the GNSS message is taken from the UBX protocol. This currently requires users to refer to the UBX protocol to understand this field. This request adds enum values to the message, which will allow for easier and cleaner programming. The message itself is unchanged, so this will not break compatibility. The enum fields were taken from UBX read.

Thanks!

Thanks!
I am waiting for travis, which looks like it timed out. I just restarted it, so hopefully we can merge this soon.
FYI, Travis needs to pass before any PR can be merged.   You should have gotten an email saying that Travis failed.  All you have to do is jump into the Travis console to see what happened, and in the case of something like this, just click "restart Build" at the top of the log.


Atom restart button issue

Looks to me there is a colour issue for the text on the restart button. It displays on hover, but it's invisible until then. Otherwise, just wanted to say thanks, I love the UI and theme, very pro and love the UX focus!

Fixed. Thanks for reporting @onetrev! It will be released with 0.6.0 soon.
Spread the word if you like the theme! It will help me make it better.

Awesome, thank you! And I am definitely trying to spread the word as it very much deserves to be spread.

Thank you 🙏🏼


Feat/otel metrics

Summary
Support sending OTEL metrics using KONG
This pr grabs Prometheus metrics and translate to OTLP format.
The metrics are protobuf-serialized and sent to the endpoint
configured in the OpenTelemetry plugin's config.metrics_endpoint field.
Checklist

[ ] The Pull Request has tests
[ ] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[ ] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference
N/A
Discussion thread: https://github.com/Kong/kong/discussions/13620

 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.

Hello @SuzyWangIBMer and thank you so much for your contribution.
As discussed privately, there are ongoing efforts to support native OTel metrics in Kong in a way that is highly optimized, to reduce performance impact to the minimum. Because of that, I would propose closing this pull request, which represents a valid workaround you may want to continue using until metrics are officially supported by the OpenTelemetry plugin.
Thanks again for sharing this approach, let me know in case of any doubts.


Refactor - More readable

Looking good. But I think Pagination.js is no longer being used :(
We should keep it though


Icon for sync history.

Add an icon for sync history according to the theme of our tool.

completed on commit no: 01940f4


[Feature Request]: Align NAT Gateway Public IP implementation with e.g. Bastion & Azure Firewall

Description
The implementation of the NAT Gateway Public IP differs a lot from the other resources which introduces quite some inconsistency.
This is partly rooted in the fact that the NAT Gateway can have multiple Public IPs & alternative multiple Public IP Prefixes.
We should anyways align the approach and, for example, allow the user to either specify an array of Public IPs, or provide the configuration to create multiple Public IPs (- the same for prefixes).

ref: Azure Bastion and Azure Firewall


Update create-external-load-balancer.md

describe Health check node port response
add info about new response header  in 1.28
https://github.com/kubernetes/kubernetes/pull/118999
This was discussed on SIG Network on 6th July 2023

/milestone clear


Implement object pooling to avoid memory leaks

Taking a look to the Unity Profiler, I noticed some memory leaks when using these outline effects.
I implemented object pooling, using the same approach Unity did in its UI
https://bitbucket.org/Unity-Technologies/ui
Hope it helps ;)

It seems you replaced ModifiedShadow.ApplyShadow with Shadow.ApplyShadowZeroAlloc.
It's correct if we use Unity >= 5.4, because Unity fixed the bug of capacity calculation:
https://bitbucket.org/Unity-Technologies/ui/diff/UnityEngine.UI/UI/Core/VertexModifiers/Shadow.cs?diff2=4bccdca73cf7&at=5.4
However, for Unity < 5.4, ModifiedShadow.ApplyShadow is better.
So we should implement a workaround using #if UNITY_5_1 || UNITY_5_2 || UNITY_5_3.
I will be glad if you will do this workaround, but it's ok for me to implement it after merging this PR.

@n-yoda I did it! Please take a look ;)
(sorry for indents, I misconfigured Mono Develop ._.)

@ProGM Thank you!

After merging, I found UNITY_5_4_OR_NEWER directive. So I rewrote the workaround for versions in a simpler manner.

👍


Multiple backends refactoring - 1.0rc

Todo!

[ ] Update docs with new settings
[ ] Update docs on guide to writing new backends
[ ] Added a small guide to migrating to 1.0

Coverage increased (+0.37%) to 98.31% when pulling dc1d679b6b12e2a26fafa6783bed0e54108336ce on 1.0-rc into 2b2ceafe47da1689ea08a35b66e3cfdfe72121b3 on master.


Avatar/details from Github username

We should ask for a user's github username. We can easily get their details from this endpoint:
https://api.github.com/users/GarySiu
Which gives us a JSON response like this:
{
"login": "GarySiu",
"id": 13659533,
"avatar_url": "https://avatars.githubusercontent.com/u/13659533?v=3",
"gravatar_id": "",
"url": "https://api.github.com/users/GarySiu",
"html_url": "https://github.com/GarySiu",
"followers_url": "https://api.github.com/users/GarySiu/followers",
"following_url": "https://api.github.com/users/GarySiu/following{/other_user}",
"gists_url": "https://api.github.com/users/GarySiu/gists{/gist_id}",
"starred_url": "https://api.github.com/users/GarySiu/starred{/owner}{/repo}",
"subscriptions_url": "https://api.github.com/users/GarySiu/subscriptions",
"organizations_url": "https://api.github.com/users/GarySiu/orgs",
"repos_url": "https://api.github.com/users/GarySiu/repos",
"events_url": "https://api.github.com/users/GarySiu/events{/privacy}",
"received_events_url": "https://api.github.com/users/GarySiu/received_events",
"type": "User",
"site_admin": false,
"name": "Gary Siu",
"company": null,
"blog": null,
"location": null,
"email": null,
"hireable": null,
"bio": null,
"public_repos": 44,
"public_gists": 0,
"followers": 6,
"following": 7,
"created_at": "2015-08-05T11:58:20Z",
"updated_at": "2016-07-03T11:09:48Z"
}

This doesn't require oauth or any kind of authentication. Other potentially interesting fields from the response include their public email, github url, hireable status and company.

I agree, it also should match your FCC username which is also helpful.


display token beneficiary with payout action

stored as payAddress in database

fixed by https://github.com/nance-eth/nance-interface/commit/1344e1142b521bf5b3b77fc1c35f77816b56b287


Recommended way to check for empty PayloadSegment property

Describe your question
What is the correct, recommended way to check for empty PayloadSegment property?
I've seen
if(applicationMessage.PayloadSegment == EmptyBuffer.ArraySegment)
{
	return null;
}

if (applicationMessage.PayloadSegment.Array == null)
{
	return null;
}

(https://github.com/dotnet/MQTTnet/blob/v<IP_ADDRESS>2/Source/MQTTnet/MqttApplicationMessageExtensions.cs#L20-L28)
I've also seen
if (e.ApplicationMessage.PayloadSegment.Count > 0)

(https://github.com/dotnet/MQTTnet/blob/v<IP_ADDRESS>2/Source/MQTTnet.TestApp/ClientTest.cs#L37)
Shouldn't it be sufficient for practical purposes to only use the latter?
n.b.: The Count property is backed by a field, at least in latest NET runtimes. No performance penalties and no need to check the underlying array for null.
Which project is your question related to?

Client
ManagedClient

Using "EmptyBuffer.ArraySegment" is somewhat internal of this library. So I recommend to not use it.
Using "e.ApplicationMessage.PayloadSegment.Count " is fine because since the array segment is a struct you will always get a proper value (0).


Fix android screen suspension logic

It turns out the logic wasn't even right for android screen suspension as it was missing a negation. Until now, the KeepScreenOn flag was set when AllowScreenSuspension.Value = true, which should actually be the other way around.

Weird, because I did actually test this on an android device..

I tested both master and this PR on one of my devices to be sure I didn't regress behavior

I recall testing this manually too but might have not paid enough attention.


Add --max-tokens parameter

I'm approving this knowing full well there will be more dimensionality added later. Tokens for one model are 10% the cost as others -- so they're not apples to apples. Also, a user may want unbounded tokens for local models. I can also imagine a per-change or per-codemod limit.

Why are we doing this now? Can we defer until we know more about the problem we're solving?

@gilday the goal is to give callers the ability to enable safeguards to prevent overuse of LLM tokens.


Error in GitHub Deploy Tool : Sandbox Support

I tried to use the link given to use Deploy to sandbox. It throws the below error message. Screenshot below.
classes/SObjectDataLoaderTest.cls(521,87):Method does not exist or incorrect signature: [SObjectDataLoader.SerializeConfig].addToUserFieldBlackList(Map>)

This issues belongs in another repo https://github.com/afawcett/apex-sobjectdataloader/issues/19, please track it over there.


client certificate expiration seconds must greater or qual to 3600

In PR: https://github.com/open-cluster-management-io/registration/pull/312, a option --client-cert-expiration-seconds was exposed so that a user can specify a expiration seconds for the client cert. And we use the minimum value of expiration time in CSR - 10min to check if the expiration time is validated.
However, for the clientCertificateController in registration, 600 is too short:

clientCertificateController to check if cert is expired every 5min.
kubeclient resync the client cert every 5min.
clientCertificateController will refresh the cert if the cert has less than a random percentage range from 20% to 25% of its life remaining.

so the minimum expiration second would be (5 * 60)/(1/5) = 1500s for clientCertificateController. otherwise, the kube client will use a expired client cert to send request to the remote cluster.
Considering to add some margin. 1h(3600s) could be a good choice.

hrm, seems there is some issue with operator e2e...I am looking at it.

@youhangwang would you rebase the PR, there was a bug in the code that is fixed in https://github.com/open-cluster-management-io/ocm/pull/150

@qiujian16 branch was rebased on the main branch


Update arc evaluation for linux steps

Add steps for setting MSFT_ARC_TEST=true environment variable for evaluating ARC machines on Linux Azure VMs.

@johnmarco
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


[Bug] systemPrompts not effective

Describe the bug
systemPrompts not effective, even if my enableInjectSystemPrompts is true
To Reproduce
Steps to reproduce the behavior:

F12 open console
Create new chat
Send message
Check network

Expected behavior
messages should have system prompt
Screenshots
messages not have system prompt, only user's message
Deployment

[ ] Docker
[ ] Vercel
[x] Server

Additional Logs
maybe is this commit's bug, https://github.com/Yidadaa/ChatGPT-Next-Web/commit/d2d615c84a6a30879488528c4537e5a3ecaebca5, line 416  const systemPrompt = createMessage use systemPrompt but not inject

Changing the system prompt in this way has taken effect, but I'm not sure if this is the correct change. I don't know TypeScript, it's just the correction plan that copilot-chat taught me.

latest commits : 1096fe55e12ee32aa3f575fcda593d2d3b701905
its correct

here is another showcase of knowledge  :

here is another showcase of knowledge :

Maybe there is something wrong with my expression. The problem I pointed out is that the system prompt information is not injected into the messages. You can open the console network to view the body.
What I expected was：
messages = [{
"role": "system",
"content": "You are an AI"
},
{
"role": "user",
"content": "hello"
}
]

But what is actually sent is
messages = [
{
"role": "user",
"content": "hello"
}
]

if you want to use a custom injection system, you can create your own masks and untick this feature for disabled which is enabled by default :

Can you try creat new chat and not use mask, enble Inject System Prompts, then check your sent body
excepted bosy shoule like:
messages = [
  {
    "role": "system",
    "content": "You are ChatGPT, a large language model trained by OpenAI.
Carefully heed the user's instructions.
Respond using Markdown.
Knowledge cutoff: {{knowledgeCutoff}}
Current model: {{model}}
Current time: {{time}}"
  },
  {
    "role": "user",
    "content": "hello"
  }
]

I'm not sure if the system prompt is sent when I use mask, but your test is incorrect, in this case you should be asking who are you or other instead of asking it with something close to the prompt word, Even if you don't provide any system prompt, your current question may still get the expected answer.

lmao how is it incorrect ? you can see in console its legit changing date for a specific model such as newer models

not console.log, you log it, but network is not send, and i will give you photo

If you check the submitted code, you can clearly see that systemPrompt is not used, instead systemPrompts (more one s) is used, of course I'm not sure if there is any other way to use it, I don't know TypeScrpit

its used but not put in that (clearly hidden)

here is another showcase

if you ask to ai like that it will be hallucination

not, your sent body lost system prompt, so why you init system prompt, and add Knowledge cutoff, current model, current time, if this just console log but not send?

its send but it's not showing in that history network because clearly hidden

this is correct? enableInjectSystemPrompts Does this option make sense? and in terms of network requests, the actual request body sent does not have a system prompt, and is not hidden as you may think.

to make sure that you still not believe
you can put a logger in server-side then see how it goes in the wilds

Assume that the system prompt built into our code is something else such as github copilot chat. Do you think this system prompt will take effect when it has not been sent?
ok, i will change the system prompt built into our code, and give you photo

how about this ? I am interpreting without github copilot or anything hahaha

i can answer any question even if my global system prompt is github copilot chat's prompt

my mead is your global system prompt not take effect, not mask or other

its does take effect like what I said if you still not believe
try put a logger in server-side mode then see how it goes in the wilds

ok, you are right, I don't want to argue with you anymore, you just did the test according to your idea, or asked me to do the test. Even if I did it, you didn't interpret it. I hope that when this issue is fixed, this issue will be linked.

also how it work in this code if you are not understand that typescript

the code is only changing date of time for a new models otherwise it keep like old/default by @Yidadaa

if you check your files change, you can find you set systemPrompts (have s) is [], and set systemPrompt (not have s) message;
but last code, when enableInjectSystemPrompts is true, the systemPrompts (have s) set message

Changing the system prompt in this way has taken effect, but I'm not sure if this is the correct change. I don't know TypeScript, it's just the correction plan that copilot-chat taught me.

Why not take a look at other people’s code tips?

wait hold on

check tbh a response its still same
https://github.com/Yidadaa/ChatGPT-Next-Web/pull/3157

check tbh a response its still same

I'm not sure if this is the correct change. I don't know TypeScript

@lvguanjun its correct
from your code/commits it just stored a system inject system prompts
latest models :

additional note
its still same even without inject system prompt "nothing about bug effective or not"


TcpDuplexConnection supports multiple subscribes

All implementations of DuplexConnection should support repeated connecting to input/output.
This is doing it for the TcpDuplexConnection.

replaced with https://github.com/rsocket/rsocket-cpp/pull/458


eb cli pip install not working

Description
Starting from today when installing eb cli using pip install --upgrade pip awsebcli on the python:3 container it fails with:
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error

  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [68 lines of output]
      /tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/config/setupcfg.py:293: _DeprecatedConfig: Deprecated config in `setup.cfg`
      !!

              ********************************************************************************
              The license_file parameter is deprecated, use license_files instead.

              By 2023-Oct-30, you need to update your project and remove deprecated calls
              or your builds will no longer be supported.

              See https://setuptools.pypa.io/en/latest/userguide/declarative_config.html for details.
              ********************************************************************************

      !!
        parsed = self.parsers.get(option_name, lambda x: x)(value)
      running egg_info
      writing lib3/PyYAML.egg-info/PKG-INFO
      writing dependency_links to lib3/PyYAML.egg-info/dependency_links.txt
      writing top-level names to lib3/PyYAML.egg-info/top_level.txt
      Traceback (most recent call last):
        File "/usr/local/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/usr/local/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/usr/local/lib/python3.11/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 118, in get_requires_for_build_wheel
          return hook(config_settings)
                 ^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 341, in get_requires_for_build_wheel
          return self._get_build_requires(config_settings, requirements=['wheel'])
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 323, in _get_build_requires
          self.run_setup()
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/build_meta.py", line 338, in run_setup
          exec(code, locals())
        File "<string>", line 271, in <module>
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/__init__.py", line 107, in setup
          return distutils.core.setup(**attrs)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/_distutils/core.py", line 185, in setup
          return run_commands(dist)
                 ^^^^^^^^^^^^^^^^^^
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/_distutils/core.py", line 201, in run_commands
          dist.run_commands()
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/_distutils/dist.py", line 969, in run_commands
          self.run_command(cmd)
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/dist.py", line 1234, in run_command
          super().run_command(command)
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
          cmd_obj.run()
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/command/egg_info.py", line 314, in run
          self.find_sources()
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/command/egg_info.py", line 322, in find_sources
          mm.run()
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/command/egg_info.py", line 551, in run
          self.add_defaults()
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/command/egg_info.py", line 589, in add_defaults
          sdist.add_defaults(self)
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/command/sdist.py", line 104, in add_defaults
          super().add_defaults()
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/_distutils/command/sdist.py", line 251, in add_defaults
          self._add_defaults_ext()
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/_distutils/command/sdist.py", line 336, in _add_defaults_ext
          self.filelist.extend(build_ext.get_source_files())
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "<string>", line 201, in get_source_files
        File "/tmp/pip-build-env-bw9t4kgw/overlay/lib/python3.11/site-packages/setuptools/_distutils/cmd.py", line 107, in __getattr__
          raise AttributeError(attr)
      AttributeError: cython_sources
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

Steps to reproduce
docker run -it --rm python:3 bash
pip install --upgrade pip awsebcli

Additional environment details

OS: Linux (docker container)

Found a workaround for this, use pip install awsebcli --no-build-isolation

@Mickeypeng Hi! This issue is still preventing us from deploying.
Should we wait for a new release for this to be fixed?

@Mickeypeng we are experiencing the same issue as @marciondg.
This issue is blocking our deployments.

Found a great workaround
echo "Cython<3" > cython_constraint.txt
PIP_CONSTRAINT=cython_constraint.txt pip install awsebcli
Looks like the problem is with Cython 3.0.0

Hi @Mickeypeng, using the latest release we still have the same problem


update extensions documentation for Rancher 2.9 updates

Summary
Fixes #11361

Occurred changes and/or fixed issues

Create page to cover the Rancher 2.9 updates
Create support matrixes page to aggregate all of the support matrixes available in the documentation in one easy-to-access page
Update Concepts page to be more in line with how a proper route in an Extension should look like
Create Known issues page to cover glob package dependency problem with incorrect version
Add shell version support matrix
Other small updates

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

@richard-cox Thanks for the review 🙏 I've updated the PR with your recommendations

@richard-cox updated with known issues page

@richard-cox maybe I've reverted some changes done with a force-push. I think now everything is all good


Dynamic Calibration tool intrinsics

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
Linux (Ubuntu 16)

Kernel Version (Linux Only)
(4.14.13)

Platform
PC

SDK Version
{ legacy / 2.. }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
Robot

Issue Description
Hello All,
I am using the D400 series dynamic calibration tool to just read and write calibration to the camera. The calibration is done by a custom-developed tool (and I am not using the dynamic calibration tool for calibration, just using its read and write only).
My custom calibration tool outputs intrinsics and with the distortion model Brown Conrady model.
If I need to write intrinsics (using the dynamic calibration tool) to RGB, ir1, or ir2 (of a d415 camera), the distortion parameters need to be of the Brown Conrady model right? (and not the inverse brown Conrady model for the RGB sensor right?).

This question has been responded to at the Intel RealSense Help Center forum.  Thank you.
https://support.intelrealsense.com/hc/en-us/community/posts/4406673626259


Remove --incompatible_depset_is_not_iterable

Remove this flag from the .bazelrc.
Bazel no longer supports this flag, see https://github.com/bazelbuild/bazel/issues/5816
Attention Googlers: This repo has its Source of Truth in Piper. After sending a PR, you can follow http://g3doc/third_party/bazel_rules/rules_typescript/README.google.md#merging-changes to get your change merged.
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature (please, look at the "Scope of the project" section in the README.md file)
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes
[x] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Using Bazel built from https://github.com/bazelbuild/bazel/commit/8f075d238fa099950b470da2541548d932b2b535, bazel-head info breaks. Culprit is Bazel no longer supports --incompatible_depset_is_not_iterable, added here: https://github.com/bazelbuild/rules_typescript/blob/62bbdc8b3b4f9a8222b135648e8e425e23cad056/.bazelrc#L15
Issue Number: N/A
Did not open any issue, because the bug template asks to open issues with rules_nodejs, but the bug is specific to this repo.
What is the new behavior?
bazel-head info works.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Ping, this still shows up on Bazel CI: https://buildkite.com/bazel/bazel-at-head-plus-downstream/builds/1294#d82635ff-91b5-4e76-8149-3cebea573ab9

merged as e9b2131da038c7e65dde33c04ee7c5888063827c


[Unitary Hack] Save Python qsharp-widgets RE output to PNG

Submission for Unitary Hack issue: fixes #1063
This change adds the ability to save the resource estimates diagram locally as a PNG. For this I added a new dependency called html2canvas that takes a DOM and renders it to an image. This currently works only on IPython qsharp-widgets, could be extended to the VSCode version.
Additionally we take into account the color theme within VSCode (which is provided in the stylesheet qsharp-ux.css) and this is passed in as the background color for rendering. Tested on (dark/light) Modern VSCode themes.
https://github.com/microsoft/qsharp/assets/3744263/898de6d9-fba4-4fea-b729-a3bb698dc51c
~Current issues:~

~This button is placeholder, nicer version: https://github.com/microsoft/vscode-notebook-renderers/pull/118~
~Saved imaged is slightly truncated at the bottom~

@microsoft-github-policy-service agree

We are very conservative in this project on taking new dependencies. I see you added one called 'html2canvas'. Looking at that project I can see it has over 900 open issues, hasn't had a commit for 2 years, and the README states
"The script is still in a very experimental state, so I don't recommend using it in a production environment nor start building applications with it yet, as there will be still major changes made".
Based on the above, I'm hesitant to take a change which adds this dependency.

Would it be ok to add a different dependency? This one for example, is more active and has less outstanding issues.
Also the way I have been doing development is to first run the watcher using npm run dev, make some changes, after which I switch to a jupyter notebook (also in VSCode) and must reload the kernel to get the newest version built by the watcher, then re-run the cell. Is there any way to attach a debugger or to view print logs?

We use SVG a lot, as well as dynamic DOM nodes (via Preact) for our widgets, so issues such as https://github.com/bubkoo/html-to-image/issues/425, https://github.com/bubkoo/html-to-image/issues/463, and https://github.com/bubkoo/html-to-image/issues/392 could come into play (I've been optimizing some SVG sizes via def/use lately, so likely to be even more of an issue going forward).
Sadly again, the project seems unmaintained. The last release was 18 months ago, and all issues since just have an automated response and no follow up (including this one 🤣 https://github.com/bubkoo/html-to-image/issues/455)

Also the way I have been doing development is to first run the watcher using npm run dev, make some changes, after which I switch to a jupyter notebook (also in VSCode) and must reload the kernel to get the newest version built by the watcher, then re-run the cell. Is there any way to attach a debugger or to view print logs?

@minestarks @ivanbasov Do you have any pointers on a faster way to work on qsharp-widgets?

Fixed prior issues, ready for review.

@minestarks Not sure if I can finish up in time for UH submission, so to speed thing up I have proposed commit 6ef794fa396d0faae8b117056a1de59e8cde3f53. This would be saving to PNG, and a simpler API for the UI side that takes in the React RefObject and optionally a filename and does the rest.

We should be good on the hackathon deadline. This is so close, but it doesn't work in VS Code for me. Probably because VS Code webview doesn't support window.open? Feel free to take a stab at implementing proper VS Code extension support (possibly using showSaveDialog ?) but that may be tricky and not required for this PR.
In the meantime please disable it for VS Code (consider taking this commit to pass an allowSaveImage flag ) .
Once VS Code is good to go and the other comments are resolved I'll merge this. We're OK on the deadline.

Is it possible for you to merge those changes on your end?

@minestarks I was thinking that based on the below setting, it should have been okay for you to commit directly to this branch. Not sure what the convention is.

In any case this merge was possible on my end. Tested it out by building VSIX package. The button does not show up at all, is this expected?

@minestarks I was thinking that based on the below setting, it should have been okay for you to commit directly to this branch. Not sure what the convention is.

@nirajvenkat sorry, it just didn't occur to me that I could push to your branch. Glad you were able to pull the changes regardless

I am investigating an issue with this branch where the UI is flickering when trying to interact with the points on the graph. Specifically the tooltip that appears when you hover over points keeps flickering. I have a hunch on what it may be but will continue to investigate tomorrow.


refactor: remove environment injection

fixes #253

pred mergnutim doresit https://keboola.slack.com/archives/C03JBGZDG/p1523553815000221 !


google sign problem

i have problem with sign google
back in <IP_ADDRESS>
The redirect URI in the request, http://<IP_ADDRESS>:3000/auth/google_oauth2/callback, does not match the ones authorized for the OAuth client. To update the authorized redirect URIs
But application is configured good
host my dns

@touristcoin Please, follow the instructions on the https://developers.google.com/identity/sign-in/web/sign-in
Google Sign in is not broken. Please, use your credentials.


Starting Dev Server for Cypress Hangs

My team has been working on migrating to using this plugin as part of our Nx14 upgrade, and one issue we've encountered is when we attempt to use the serve target to start serverless offline, the signal never makes it that the executor is successful, so Cypress never starts.
my-app/project.json:
{
  targets: {
    "serve": {
      "executor": "@ns3/nx-serverless:sls",
      "options": {
        "command": "offline"
      }
    }
  }
}

my-app-e2e/project.json:
{
  targets: {
    "e2e": {
      "executor": "@nrwl/cypress:cypress",
      "options": {
        "cypressConfig": "apps/my-app-e2e/cypress.json",
        "devServerTarget": my-app:serve",
        "tsConfig": "apps/my-app-e2e/tsconfig.json"
      }
    }
  }
}

It seems like the call to execa.command in executor.ts is being awaited, but because the command is serverless offline the process doesn't exit, and thus execa doesn't return a value, causing the whole executor to stall, and never returning a ready state.

Hi, yeah, I have never tested it for. You may be right. I feel like there would be a lot of work to make it work as expected - taking into the account format output locally and on CI - which is important for nx-cloud.
await works great for all "normal" targets like "package" or "deploy", but with "offline" you are right, it doesn't end. Perhaps we need a separate executor for "continuous tasks"... I am not happy about it :/ I like the simplicity of the current solution. I personally always test deployed instance so "offline" mode was always an afterthought, but I do recognise the need.
If you have an idea for elegant solution to that issue I always welcome a PR ;)

As a workaround you can probably just wrap my-app:serve in some simple executor that executed original serve without awaiting result and yielding right away.

Just as a follow-on for anyone who comes across this thread, another def on my team found a workaround for this issue.  We added the following target that we use as the devServerTarget for our e2e project:
    "cypress": {
      "executor": "nx:run-commands",
      "options": {
        "command": "npx nx run <serverless-project>:serve",
        "readyWhen": "[HTTP] server ready"
      }
    },


Change twitter to X on home page

Description
With the recent rebranding of Twitter to X, we need to update all references to Twitter on our homepage. This includes changing the name, icon, and link associated with Twitter to reflect the new branding.
Tasks

Update Name:

Change all instances of "Twitter" to "X" on the homepage.
Ensure consistency in all mentions, including tooltips and descriptive texts.

Update Icon:

Replace the Twitter bird icon with the new X logo.
Ensure the new icon is appropriately sized and matches the design language of the homepage.

Update Link:

Update the hyperlink currently pointing to https://twitter.com/ to https://x.com/.
Verify that the link directs users to the correct X homepage.

Hey @ceilican I would like to work on this issue
can you please assign this to me ??

@ceilican can you please assign this issue to me


how to run docx2rash ?

Following https://github.com/essepuntato/rash/tree/master/tools/docx2rash I get:
ERROR: Resource not found "from-docx.xsl"
The file is present in the jar (in xslt/) but cannot be located?

Hi @VladimirAlexiev
Thanks for your message, I'll try to check what does not work. Just tried on my local machine (an iMac) and I have a similar problem (I've got a "URI is not hierarchical" message). Funnily enough, it works correctly on another Mac I have and on Linux (Ubuntu 14).
Can I ask you what is the configuration from where you have run the tool (computer, operating system, etc.)?

Windows 7, Cygwin, java version "1.8.0_144"
Java(TM) SE Runtime Environment (build 1.8.0_144-b01)
Java HotSpot(TM) 64-Bit Server VM (build 25.144-b01, mixed mode)

@essepuntato i have the same exact issue. I tried with the command
D:\Download\rash\tools\docx2rash\bin> java -jar docx2rash.jar -i f.docx -o testout/
But the output was ERROR: Resource not found "from-docx.xsl"
Tested on Windows 10 with java version "1.8.0_201"
Java(TM) SE Runtime Environment (build 1.8.0_201-b09)
Java HotSpot(TM) 64-Bit Server VM (build 25.201-b09, mixed mode)


Tooltip not working on site

Description
Tooltip hover threw an error.
CodeSandbox/Snack link
No response
Steps to reproduce

Go to Tooltip Docs
Find the  component Hover on me! .
Hover the component Hover on me! .
Select any placement
See error

gluestack-ui Version
0
Platform

[ ] Android
[ ] Expo
[ ] iOS
[ ] Next

Other Platform
Site
Additional Information

Hey @N0N4T0  , Thanks for reporting the issue. We will have a look.

Hi , thank you for reporting .
We are working on improving the api so this might change all together .
As of now i am closing this issue .


[OTHER] unable to compile julec

What would you like to share?
When I run this command:
go build -o julec -v src/cmd/julec/main.go
rather than building julec I get this error:

src/cmd/julec/main.go:16:2: no required module provides package github.com/julelang/jule: go.mod file not found in current directory or any parent directory; see 'go help modules'
src/cmd/julec/main.go:17:2: no required module provides package github.com/julelang/jule/build: go.mod file not found in current directory or any parent directory; see 'go help modules'
src/cmd/julec/main.go:18:2: no required module provides package github.com/julelang/jule/cmd/julec/gen: go.mod file not found in current directory or any parent directory; see 'go help modules'
src/cmd/julec/main.go:19:2: no required module provides package github.com/julelang/jule/lex: go.mod file not found in current directory or any parent directory; see 'go help modules'
src/cmd/julec/main.go:20:2: no required module provides package github.com/julelang/jule/parser: go.mod file not found in current directory or any parent directory; see 'go help modules'

Additional information
go version: go1.20.1 linux/amd64

Hi,
With the development of the project, this command became obsolete over time. Please go to the src directory and execute the command go build -o julec -v cmd/julec/main.go in your terminal to get the expected result.
I would like to point out that this wrong command will be removed from the relevant places and will be fixed.
Thanks for your contribution.


can not find the YYTextView on [UIResponder currentFirstResponder]?

the [UIResponder currentFirstResponder] returns the YYTextView's superview.

What is currentFirstResponder? I can't find this API in UIResponder's header file.

sorry, it's code from internet
`static __weak id currentFirstResponder;
@implementation UIResponder (FirstResponder)

(id)currentFirstResponder {
currentFirstResponder = nil;
[[UIApplication sharedApplication] sendAction:@selector(findFirstResponder:) to:nil from:nil forEvent:nil];
return currentFirstResponder;
}

(void)findFirstResponder:(id)sender {
currentFirstResponder = self;
}
@end`

In UIKit, there’s no public way to detect the first responder.
Apple has a workaround in ResearchKit and you may try it:  https://github.com/ResearchKit/ResearchKit/blob/1.0.0/ResearchKit/Common/UIApplication%2BResearchKit.h
But this method was deleted in master branch, maybe it's not good.


Appstudio update test-component-pac-bijs

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-bijs-on-pull-request has successfully validated your commit.


Pin kombu to 4.3.0

Celery can install kombu>=4.2.0
https://github.com/celery/celery/blob/v4.1.1/requirements/default.txt#L3
But kombu>4.3.0 requires redis>=3.2
https://github.com/celery/kombu/blob/v4.5.0/requirements/extras/redis.txt#L1

You can reproduce this running celery in a fresh installation

This will blow up in production probably. kombu==4.3.x requires a version of redis that it's not compatible with it.
We should pin kombu==4.2.1 which is the version that we have currently installed in production.

In .org production we are using redis 3.2.7

4.2.1 and 4.3.0 require the same version, actually a version of redis >= 2.10.5
https://github.com/celery/kombu/blob/v4.2.1/requirements/extras/redis.txt
https://github.com/celery/kombu/blob/v4.3.0/requirements/extras/redis.txt


No longer works in Firefox Nightly

The IPFS Companion stopped working one–three weeks ago in Firefox Nightly.
It can’t connect to IPFS Desktop anymore. IPFS is listening on the expected ports, however. IPFS Companion still works in Chromium.
I’ve tried reinstalling IPFS Companion and  have tested with older versions all the way back to 2.4.0.
Confirmed issue on Firefox Nightly on macOS 10.14 and Windows 10.1809.

OK, the admin interface is returning 403 Forbidden.
The difference between Chrome and Firefox is that Firefox appends a Origin: moz-extension://{extension-installation-id} header. If I remove that header in Firefox, everything starts working again.

Thank you for reporting!
Interesting. This is basically Firefox version of https://github.com/ipfs-shipyard/ipfs-companion/issues/615 (which we fixed in https://github.com/ipfs-shipyard/ipfs-companion/pull/616).
In short, Chrome was adding Origin: null to fetch made from webextension background pages and acted as sort of false-positive trigger for CORS validation producing 403 Forbidden.
Seems that Firefox Nightly started setting Origin: moz-extension://{extension-installation-id} (which is better than null) but that produces the same error.  Will probably should apply similar fix here.

Heads up: I’d expect this behavior to change in Firefox. The internal UUID they send is unique and persistent per installation. It would allow for user tracking.

Correction: it’s been like this for a year according to bugz#1405971. I’m not sure why this issue only started affecting IPFS now.

@da2x  shipped a fix in v<IP_ADDRESS>30 (Beta), give it a try! :)

Works for me.


Azure AD Connectivity Requirements for AADC

Microsoft is recommending that AADC server be treated as Tier 0 server and yet this article links to Office 365 URLs and IP address ranges which are end user (Tier 2) requirements for connectivity to the whole of Office 365 functionality. It also refers to Azure data center IP ranges for Pass-through Authentication with Single Sign On (SSO) and Password Hash Sync with Single Sign On (SSO). This  download page mentions it is "Deprecating" so what is the new source of truth?
The article does separately list endpoint connectivity requirements for AADC Health Agent. Can we have the same succinct list for AADC Sync service as well so that this Tier 0 server can be harden with the care it deserves?
Additionally, the article omits the endpoint connectivity requirements for Password Writeback. It is not clear which of these links cover the connectivity requirements. Believe *.servicebus.windows.net endpoint connectivity is needed but that is not covered in Office 365 URLs and IP address ranges.

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

ID: 41c38d39-e36b-dd70-9c8c-1fc28a1dba5e
Version Independent ID: 4a9c6f6b-7627-2d61-402a-a2c405812a61
Content: Hybrid Identity required ports and protocols - Azure
Content Source: articles/active-directory/hybrid/reference-connect-ports.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@billmath Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

@NileshGhodekar
Thank for submitting this. I have created a work item for it and will send this to engineering so that they can review it and provide me with updated details. I am not sure what the timeline will be, so I am going to close this for now. But once I have an update I will drop it in here and let you know.
Thank you!
Bill
#please-close


Proton external stream to support Debezium CDC message in Avro schema

Since Proton 1.5.2, Kafka schema registry is supported. Then we should start improving the Debezium integration.
Currently if you configure Debezium to generate JSON messages without Avro or Protobuf, Proton can read them and process them. Please check the example at https://github.com/timeplus-io/proton/tree/develop/examples/cdc
However, if Avro format is turned on with a confluent (compatible) schema registry, not all fields can be read.
What is working
op is a string field and always exists in all messages. If you create the external steam, only with op field, it's working
CREATE EXTERNAL STREAM customers_avro(op string)
SETTINGS 	type='kafka',
          brokers='redpanda:9092',
          topic='dbserver1.inventory.customers',
          data_format='Avro',
          kafka_schema_registry_url='http://redpanda:8081';

Run select * from customers_avro where _tp_time>earliest_ts() you will get
┌─op─┐
│ r  │
│ r  │
│ r  │
│ r  │
└────┘

Avro Schema
The full Avro schema for the value is
{
  "type": "record",
  "name": "Envelope",
  "namespace": "dbserver1.inventory.customers",
  "fields": [
    {
      "name": "before",
      "type": [
        "null",
        {
          "type": "record",
          "name": "Value",
          "fields": [
            { "name": "id", "type": "int" },
            { "name": "first_name", "type": "string" },
            { "name": "last_name", "type": "string" },
            { "name": "email", "type": "string" }
          ],
          "connect.name": "dbserver1.inventory.customers.Value"
        }
      ],
      "default": null
    },
    { "name": "after", "type": ["null", "Value"], "default": null },
    {
      "name": "source",
      "type": {
        "type": "record",
        "name": "Source",
        "namespace": "io.debezium.connector.mysql",
        "fields": [
          { "name": "version", "type": "string" },
          { "name": "connector", "type": "string" },
          { "name": "name", "type": "string" },
          { "name": "ts_ms", "type": "long" },
          {
            "name": "snapshot",
            "type": [
              {
                "type": "string",
                "connect.version": 1,
                "connect.parameters": {
                  "allowed": "true,last,false,incremental"
                },
                "connect.default": "false",
                "connect.name": "io.debezium.data.Enum"
              },
              "null"
            ],
            "default": "false"
          },
          { "name": "db", "type": "string" },
          { "name": "sequence", "type": ["null", "string"], "default": null },
          { "name": "table", "type": ["null", "string"], "default": null },
          { "name": "server_id", "type": "long" },
          { "name": "gtid", "type": ["null", "string"], "default": null },
          { "name": "file", "type": "string" },
          { "name": "pos", "type": "long" },
          { "name": "row", "type": "int" },
          { "name": "thread", "type": ["null", "long"], "default": null },
          { "name": "query", "type": ["null", "string"], "default": null }
        ],
        "connect.name": "io.debezium.connector.mysql.Source"
      }
    },
    { "name": "op", "type": "string" },
    { "name": "ts_ms", "type": ["null", "long"], "default": null },
    {
      "name": "transaction",
      "type": [
        "null",
        {
          "type": "record",
          "name": "block",
          "namespace": "event",
          "fields": [
            { "name": "id", "type": "string" },
            { "name": "total_order", "type": "long" },
            { "name": "data_collection_order", "type": "long" }
          ],
          "connect.version": 1,
          "connect.name": "event.block"
        }
      ],
      "default": null
    }
  ],
  "connect.version": 1,
  "connect.name": "dbserver1.inventory.customers.Envelope"
}

Sample value in the customers topic
{
    "after": {
        "dbserver1.inventory.customers.Value": {
            "email"<EMAIL_ADDRESS>            "first_name": "Anne",
            "id": 1004,
            "last_name": "Kretchmar"
        }
    },
    "before": null,
    "op": "r",
    "source": {
        "connector": "mysql",
        "db": "inventory",
        "file": "mysql-bin.000003",
        "gtid": null,
        "name": "dbserver1",
        "pos": 157,
        "query": null,
        "row": 0,
        "sequence": null,
        "server_id": 0,
        "snapshot": {
            "string": "last_in_data_collection"
        },
        "table": {
            "string": "customers"
        },
        "thread": null,
        "ts_ms":<PHONE_NUMBER>000,
        "version": "2.5.0.Final"
    },
    "transaction": null,
    "ts_ms": {
        "long":<PHONE_NUMBER>915
    }
}

What is not working
Let's try to get the ts_ms field

{ "name": "ts_ms", "type": ["null", "long"], "default": null },

Sample message is

"ts_ms": {
"long":<PHONE_NUMBER>915
}

SQL
CREATE EXTERNAL STREAM customers_avro2(op string, ts_ms tuple(long int64))
SETTINGS type='kafka',
brokers='redpanda:9092',
topic='dbserver1.inventory.customers',
data_format='Avro',
kafka_schema_registry_url='http://redpanda:8081';

Nothing shown in the SQL client
tail /var/log/proton-server/proton-server.log

2024.03.17 00:34:44.346018 [ 349 ] {f0d7f343-0a39-4af7-9b28-445fb4b4aafe}  External-dbserver1.inventory.customers: Failed to parse message at 3: Type tuple(long int64) is not compatible with Avro union:
[
"null",
"long"
]: column ts_ms

Maybe there is a better way to map the Avro union.
The after field is "type": ["null","Value" ], while the Value is defined in before
        {
          "type": "record",
          "name": "Value",
          "fields": [
            {
              "name": "id",
              "type": "int"
            },
            {
              "name": "first_name",
              "type": "string"
            },
            {
              "name": "last_name",
              "type": "string"
            },
            {
              "name": "email",
              "type": "string"
            }
          ],
          "connect.name": "dbserver1.inventory.customers.Value"
        }

Doing a bit more research. The annoying part of Avro is that union object can be wrapped. For example "type": ["null", "long"], this could be just "ts_ms":null or "ts_ms":<PHONE_NUMBER>915  . But the default encoder will generate "ts_ms": {"long":<PHONE_NUMBER>915} The issue has been reported 10 years ago. Some people created custom encoder/decoder but this remains the default behavior
https://issues.apache.org/jira/browse/AVRO-3210
https://issues.apache.org/jira/browse/AVRO-1582
The other thing we at Proton team should look into is how to retrieve the after field with data_format=Avro
    "after": {
        "dbserver1.inventory.customers.Value": {
            "email"<EMAIL_ADDRESS>            "first_name": "Anne",
            "id": 1004,
            "last_name": "Kretchmar"
        }
    },

I found a workaround to simply the Avro message, so that Proton can read all columns without worrying about nullable or complex data structure, by following the doc at https://debezium.io/documentation/reference/stable/transformations/event-flattening.html
The key change is add transforms=unwarp while creating the Debezium job
"transforms": "unwrap",
"transforms.unwrap.type":"io.debezium.transforms.ExtractNewRecordState",
"transforms.unwrap.drop.tombstones":"false",
"transforms.unwrap.delete.handling.mode":"rewrite",
"transforms.unwrap.add.fields":"table,lsn",

Then the schema will turn to
{
  "type": "record",
  "name": "Value",
  "namespace": "dbserver1.inventory.customers",
  "fields":
    [
      { "name": "id", "type": "int" },
      { "name": "first_name", "type": "string" },
      { "name": "last_name", "type": "string" },
      { "name": "email", "type": "string" },
      { "name": "__deleted", "type": ["null", "string"], "default": null },
      { "name": "__table", "type": ["null", "string"], "default": null },
    ],
  "connect.name": "dbserver1.inventory.customers.Value",
}

then sample message will be
{
  "__deleted": {
      "string": "false"
  },
  "__table": {
      "string": "customers"
  },
  "email"<EMAIL_ADDRESS>  "first_name": "Anne",
  "id": 1004,
  "last_name": "Kretchmar"
}

So you can just create an external stream via
CREATE EXTERNAL STREAM customers_avro(
  id int,
  first_name string,
  last_name string,
  email string
)
SETTINGS type='kafka',
          brokers='redpanda:9092',
          topic='dbserver1.inventory.customers',
          data_format='Avro',
          kafka_schema_registry_url='http://redpanda:8081';

Then query it via select * from customers_avro where _tp_time>earliest_ts();
Get result
┌───id─┬─first_name─┬─last_name─┬─email─────────────────┐
│ 1001 │ Sally      │ Thomas    │<EMAIL_ADDRESS>│
│ 1002 │ George     │ Bailey    │<EMAIL_ADDRESS>  │
│ 1003 │ Edward     │ Walker    │<EMAIL_ADDRESS>       │
│ 1004 │ Anne       │ Kretchmar │<EMAIL_ADDRESS>  │
└──────┴────────────┴───────────┴───────────────────────┘

Need to port the enhancements from CH to our repo:

Our code: https://github.com/timeplus-io/proton/blob/develop/src/Processors/Formats/Impl/AvroRowInputFormat.cpp#L399
CH's code: https://github.com/ClickHouse/ClickHouse/blob/master/src/Processors/Formats/Impl/AvroRowInputFormat.cpp#L554

Just read this issue carefully, so for this part:

Let's try to get the ts_ms field
{ "name": "ts_ms", "type": ["null", "long"], "default": null },
Sample message is
"ts_ms": {
"long":<PHONE_NUMBER>915
}
SQL
CREATE EXTERNAL STREAM customers_avro2(op string, ts_ms tuple(long int64))
SETTINGS type='kafka',
brokers='redpanda:9092',
topic='dbserver1.inventory.customers',
data_format='Avro',
kafka_schema_registry_url='http://redpanda:8081';

This is wrong. the type for ts_ms should be nullable(int64), it's not a tuple. You should check the schema itself to determine the types of the fields, not the value itself.

Hi Gimi, I guess you refer to the 1st comment.
Check the 2nd comment

The annoying part of Avro is that union object can be wrapped. For example "type": ["null", "long"], this could be just "ts_ms":null or "ts_ms":<PHONE_NUMBER>915  . But the default encoder will generate "ts_ms": {"long":<PHONE_NUMBER>915}

If we don't apply io.debezium.transforms.ExtractNewRecordState transformation, just reading the raw JSON to external stream, the ts_ms is a complex type, i.e. tuple, not a nullable(long)

Hi Gimi, I guess you refer to the 1st comment.
Check the 2nd comment

The annoying part of Avro is that union object can be wrapped. For example "type": ["null", "long"], this could be just "ts_ms":null or "ts_ms":<PHONE_NUMBER>915  . But the default encoder will generate "ts_ms": {"long":<PHONE_NUMBER>915}

If we don't apply io.debezium.transforms.ExtractNewRecordState transformation, just reading the raw JSON to external stream, the ts_ms is a complex type, i.e. tuple, not a nullable(long)

No, that's wrong. That's why I said "You should check the schema itself to determine the types of the fields, not the value itself.". It generates the message "ts_ms": {"long":<PHONE_NUMBER>915} is totally fine, but it does not change the type of the ts_ms field. Please read the schema:
 { "name": "ts_ms", "type": ["null", "long"], "default": null }

ts_ms has the type of optional long, it's not a record, you should not use tuple.

I see where you came from.
sure, to analyze the data,, ts_ms should be an optional long, nullable(long)
the JSON payload we got is "ts_ms": {"long":<PHONE_NUMBER>915}. In my comment, if we want to parse the JSON directly, one solution is to treat the ts_ms as tuple, then get the nested long. This is just like raw string, as a temp solution to access certain data.
In the end, we do need to get a long type or even datetime64 type of ts_ms


Nice one!

Really great implementation of bloomfilter in lua!
It will be shared in our next RedisWeekly!

Thank you! This is a great pleasure!


Triggers panel: unable to edit panel in Grafana 6.0

Describe the bug
Cannot edit triggers panel after upgrading Grafana to 6.0.0.
Screenshots
Browser shows blank screen:

Browser's console shows the following:

Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

6.0.0
4.0.5
3.10.0

Yes, it's due to changes in panel architecture in grafana. I will add a fix in the next plugin release.

Fixed by adding "dataFormats": [] into plugin.json.
Related to https://github.com/grafana/grafana/issues/15217

I am working with an older version of Zabbix having the same edit panel issues for the older version of this plugin. Where in plugin.json do you add "dataFormats":[] to fix this?

@JonZolot see the 18babc8e5f00f21e711c1a7f414ebc1923c2371f


Set due dates for 2019-2020

Check the calendars to create the deadlines for the projects.

Finished for Semester 1.  Leaving this issue until Semester 2 is done.

Finished for Semester 2.


Very small contribution on pip installing the requirements.

I was following the steps, then running failed because haikunator was not installed so I realised there was this step missing.
Thanks for this example! It's really helpful :) Hope it had more visibility, it took me a while to run into it.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Hi @raquel-ucl , thanks for the contribution! Could you update your line to requirements.txt? (missing extension).
Thanks!!

Actually, this is easy enough, I won't bother you with that, I will merge and edit master right away. Thanks!


Chabot-widget not working

Getting error on rasa server side

Hi @Prskriti,
This chatbot widget is setup to use the rest API, not socket.io so you need to comment out the socket.io section in credentials.yml that looks something like this:
#socketio:
#  user_message_evt: <event name for user message>
#  bot_message_evt: <event name for but messages>
#  session_persistence: <true/false>

Give that a try

Getting Failed to load resource: the server responded with a status of 404 (Not Found) on browser

Hi @jwheat
I have already commented socket.io section then also I am facing same issue.

Getting connection refused error on browser side.

Hey @Prskriti, are you using the chatbot widget that I have designed or something else because in my widget I don't any url configured that calls socket.io.🙂

Getting connection refused error on browser side.

To resolve the issue related to webchat integration, check this video:
https://youtu.be/J1n3Y8SVxVM

Now I am getting "I am facing some issues, please try again later!!!" as a response each time when asked anything.This is what I am getting on browser side.

hey @Prskriti, while you are running the rasa server make sure you run using the below command:

rasa run -m models --enable-api --cors "*" --debug
you are facing the above issue due to cors.

thanks for ur help my bot is working the way as expected. :)

Closing the issue since it has been fixed ;)

Closing the issue since it has been fixed ;)


Add hidden extraFields to CountTokenResponse

The Vertex AI SDK adds another field to the count tokens response, but we don't
want to expose vertex details to the users of this SDK. The Vertex AI SDK uses
src/ imports for now.

Parse fields beyond 'totalTokens' into a separate private Map field.
Add top level methods to read the field from a CountTokensResponse, and to
construct an instance with extra fields. The latter is likely to be useful for
testing.
Add a vertex_hooks.dart library. Now that there is more than one single API
we expose specifically for the Vertex SDK, add a separate library to collect
them.

@cynthiajoan - Can you try this and verify it will work for you?
I'd recommend including a public extension like:
extension CountTokensResponseFields {
  int? get totalBillableCharacters => countTokenResponseFields(this)['totalBillableCharacters'] as int?;
}

@cynthiajoan - Can you try this and verify it will work for you?
I'd recommend including a public extension like:
extension CountTokensResponseFields {
  int? get totalBillableCharacters => countTokenResponseFields(this)['totalBillableCharacters'] as int?;
}

It works nicely, thanks Nate!


Add support for FlameGraph output

New, optional command line parameter: -f
If enabled, writes folded call stacks to output file, instead
of statistics
Use with the FlameGraph tool: https://github.com/brendangregg/FlameGraph

The command-line mode of plop.collector is not the only one (I rarely use it myself), so the part that actually writes to a file should probably be moved to a method of Collector (instead of callers like main() reaching into stack_counts).

Maybe have the collector strictly just collect stack samples, and a different object would filter and format samples, with —format only configuring the latter object?
That means the collector would always collect a list of frames though (in case the formatter needs that), the default collector would be the one folding those into a counter.

Good idea. What do you think of the changes?

I just fixed the last commit, in case you're wondering. After a semi-failed git reset the diff was referring to a hidden commit and thus rather confusing.

@andreasf the ArgumentParser conversion should probably live in a separate PR (well played on -m being a switch, I would have missed that completely and tried to set up an exclusion group of some sort).
Could use a list literal for Collector.stacks, and maybe in a future PR look into preallocating it (after profiling) to avoid a possible list resizing in the middle of stack collection.
BTW if you initialise ArgumentParser with formatter_class=ArgumentDefaultsHelpFormatter it'll automatically insert the default values after the help text: https://docs.python.org/3/library/argparse.html#formatter-class

You're right on all points. I thought about doing a separate PR for the argparse changes, but since the last commit is from 8 months ago, I was too lazy to split it up. People over processes ;-)

Wow, this looks great. For future reference I'd have a slight preference for separate PRs for the PEP8 changes and the move to argparse, but it's not worth splitting it up.
The one thing I might change is actually the -m flag: it's intended to be like python's -m flag and take an argument. But in any case this is a big improvement over what was there before and going to go ahead and merge it; don't worry about changing -m unless you feel like it.

Thanks!


not able to sign in

why i am not be able to sign in from instagram desktop messenger.. what is cokkie sessionid issue..!!

what is authorised the thread.. n how to fix that... is there anyother way
On Thu, Apr 19, 2018 at 8:54 PM, Shrenika Solanki<EMAIL_ADDRESS>wrote:

but now how to fix this problem
On Thu, Apr 19, 2018 at 7:34 PM, enesemree<EMAIL_ADDRESS>wrote:

me too
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ifedapoolarewaju/igdm/issues/151#issuecomment-382748688,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AkvcuSFYeWU3h0VfHqJh197_RNuOTmNJks5tqJl8gaJpZM4TbyrO
.

this issue is fixed for me now, can you please try again. Closing this


Update Copyright Headers?

Currently most copyright headers use the following format:
Copyright YEAR Google Inc. All rights reserved.
The latest generated code uses a slightly different format:
Copyright YEAR, Google LLC All rights reserved.
Notice the comma after the year, and the change to "Google LLC" from "Google Inc.". Is this a change that we should make globally on all existing copyright headers?
/cc @geigerj

Yes. Additionally, "All rights reserved" should be removed. This is the canonical license header:
Copyright 2017 Google LLC

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

    https://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.

Should we make this change to the generated files as well? The newly generated files still state “All rights reserved.”, and have a comma after year.

Sure, if you're doing it mechanically, updating autogen headers should do no harm. We actually just updated the generator a few minutes ago (https://github.com/googleapis/toolkit/pull/1748), so everything should be in sync on refresh.

Comma, no comma, or doesn't matter?

I'm the wrong person to ask whether it matters 😛
But the canonical text has no comma, so I'd follow that.

Should this change be retroactively applied to files from past years? For example, should files from before 2017 be updated to "Google LLC", even though the name of the company was "Google Inc." at the time they were authored?
It would seem logical to me that only new files should get the new license header.

Or, should old files be updated to use the current year?

Should this change be retroactively applied to files from past years? For example, should files from before 2017 be updated to "Google LLC", even though the name of the company was "Google Inc." at the time they were authored?

Yes. However, the year should not be changed.
The correct copyright is:
Copyright XXXX Google LLC

Here is the gist of what is going on here:

Google changed its legal name (actually, its corporate structure, which ultimately affects the name) to "Google LLC" on September 30, 2017.  We retain a d/b/a as "Google, Inc." but things should be changed. This is roughly analogous to what would happen with property ownership when a person gets married and changes his or her surname; previous legal documents with the old name are still valid, but effort is also taken to go back and update them.

"All Rights Reserved" was never, ever correct. Licensing things under the Apache 2.0 license (or almost any other valid license) constitutes a grant of rights (in Apache's case, to use, distribute, etc.).

I am fairly certain the comma after the year does not actually matter, but the reference implementation of the Apache license and our internal licensing guidelines both exclude it.

@geigerj @lukesneeringer ^ Can either of you answer these questions?

I beat you by five seconds. ;-)

I was 11 seconds too late! Thanks @lukesneeringer

Should we make this change to the generated files as well? The newly generated files still state “All rights reserved.”, and have a comma after year.

If it is not extra trouble, please do. If it is a pain, then do not bother, and our next auto-gen refresh will deal with it.

I submitted #1883 that updated all files, including the generated files. PTAL.


Using No Proxy

How do we set up no proxy directly in http yac?
I use proxy settings for vscode but in it too, we cannot set no proxy ... can I set no proxy directly for httpyac?
something like "thunder-client.excludeProxyHostList": "localhost"

Unfortunately, I am just discovering that this is very difficult to do. About an own hook in a .httpyac.js file it would work. But I better add an explicit excludeList or metaData.

.httpyac.js

module.exports = {
configureHooks: (api) => {
  api.hooks.onRequest.addHook('delete_proxy', function (_request,context) {
   delete context.config.proxy;
  });
}
}

With next release a setting proxyExcludeList is added to ignore proxy setting. Alternativly you could ignore proxy setting with metadata # @no-proxy.


Listing Block Variation: Event Listing

Is your feature request related to a problem? Please describe.
Plone Classic has a dedicated event listing view for the Collection content type. Plone 6 should have something similar.
Describe the solution you'd like
There are two ways of solving this:
a) create a dedicated listing block variant just for events (that show the full start and end date)
b) amend the "default" variant to show event details (basically create sub-templates for each content type that can be extended and overriden)
Describe alternatives you've considered
We implemented both approaches in our client projects. Though, we believe that Plone 6 should ship with an out-of-the-box solution for this.

b) amend the "default" variant to show event details (basically create sub-templates for each content type that can be extended and overriden)

This is on my wishlist for a long time... I think this needs to be integrated as a form of "item view", together with the Teaser block work. Basically, we need to be able to have a bunch of visual representations of objects, which we can reuse in various contexts: listings, teaser/grid, etc.

@reebalazs implemented a configurable "item view" for us that could be re-used. I wanted to get some feedback first on which way to go first though...

hey there, I am new to opensource and I would like to work on this issue. please guide me on what needs to be done.


how to have a better score

Hey,
I tried your repo and it is perfect for what I need to do, but the problem is that I can't seem to improuve it so that there is no more abnormal normal and no more normal abnormal.
can you help me improuve it ? of maybe some guidance please ? thank you !

@michaelhoarau

Hi,
What your are asking is not possible. There will always be false positives and false negatives. In many cases, a classification algorithm cannot be perfect as there will also be observations that cannot be distinguished (event by a human for that matter). The only thing you can do, is to integrate a machine learning results into a human-in-the-loop process. There, you can use confidence level of a given results: when the confidence level is lower than a threshold that you decide depending on the criticality of your business, you send it to someone to check it. Instead of checking 100% of the results, you manually check the few % that the ML system cannot decide on.

hi I understand, do you know how I can improuve it please ? right now I at 64 abnormal normal and 69 normal abnormal.

As I mentioned, you can't improve the results by just improving the "algorithm". You need to change the approach, or collect more data. If you don't collect more data, it's not possible to improve the outcome. You have to compromise between recall and precision in this specific case. I recommend you read about machine learning metrics, identify which one is best for your business case and focus on this one. I also you recommend you research human-in-the-loop approach as I mentioned in my previous message.

I see, I will be trying to implement a human in the loop
thank you very much


outcome name is too long - show tooltip

Positions - if outcome name is too long - show tooltip - design need to create rules for the tooltip, max width etc
Need design for tooltip for outcome name, .... whenever there is content too long for space.

Figma:
https://www.figma.com/file/bZh7tOiLm6Ls9Hl4RL6qMC5X/Augur-Redesign?node-id=7181%3A15
Rules:

Displays at the centre point of the truncated name
Min width 248px
Max width 320px
Responsive height

Tooltip isn't working for me.

Tooltip still isn't working for me. Tried on portfolio > open orders and on the trading page

works now - closing

Still not working for me


Multiple deprecations in php 8.1

Hello, first of all, thanks for the great tool, I have been using it for quite a long time.
But it seems that the library needs to be updated in order to work with PHP 8.1
I get these errors:
Deprecated: Return type of HumbugBox3140\KevinGH\RequirementChecker\RequirementCollection::getIterator() should either be compatible with IteratorAggregate::getIterator(): Traversable, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///usr/local/bin/deptrac/.box/src/RequirementCollection.php on line 14

Deprecated: Return type of HumbugBox3140\KevinGH\RequirementChecker\RequirementCollection::count() should either be compatible with Countable::count(): int, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in phar:///usr/local/bin/deptrac/.box/src/RequirementCollection.php on line 19

Could you please fix that?

Thanks for the compliment and the bug report. I will have a look at it when I have time, but I can't promise we can fix those right away as they seem to come from a library we are using which might not be fully deprecation free.
Being deprecation free is not a top priority for me. This is a great article explaining why: https://wouterj.nl/2021/11/deprecations-are-not-errors


After you switch decks, your player ID becomes -1 instead of 1.
(lldb) po game.player.id
-1

Guess we can close this one with your PR ?


Bump up binfmt image to latest

Bump up binfmt image to latest
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


fix unit-of-measure for humidity transport

original unit (as shown in the device website) is g/d, not mg/d
Correct unit of measure after change:

Thanks!


Fix building on raspberry pi

Description
Fix building package on raspberry pi
Type of change

[x] Bug fix

Checklist:

[ ] I've properly formatted my code according to the guidelines

[ ] By running Black + Flake8
[ ] By running pre-commit hooks

[ ] This PR addresses an already submitted issue for TensorFlow Addons
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] This PR contains modifications to C++ custom-ops

How Has This Been Tested?
Build of package in raspbian buster container did work after applying the suggested changes

@googlebot I signed it!

Any chance anyone could do  aarch64 build for the pi? I tried but failed even after doing a symlink to _pywrap_tensorflow_internal

@StuartIanNaylor could you be so kind to have a look at pull request #2420, A fix for the Raspberry Pi 64-bit OS and Jetson Nanon?

sudo nano configure.py Change 2 lines to the following:-
def is_raspi_arm():
return (os.uname()[4] == "armv7l") or (os.uname()[4] == "aarch64")
if is_macos() or is_linux():

write("build --copt=-mavx") <-not known in gcc aarch64
    write("build --cxxopt=-std=c++14")
    write("build --host_cxxopt=-std=c++14")

sudo ln -s /home/pi/google-kws/venv/lib/python3.7/site-packages/tensorflow/python/_pywrap_tensorflow_internal.so /usr/lib/lib_pywrap_tensorflow_internal.so
Works for Aarch64 just do the normal install after


Phpseclib 2

@foreachlt you get phpseclib ~2.0  the 2.0-dev version, what if we specify the 2.0.0? what are your thoughts about it?