Cannot load from scholar

Have you checked that your issue isn't already filed?

[X] I read through FAQ and searched through the past issues, none of which addressed my issue.
[X] Yes, I have checked that this issue isn't already filed.

Bug description
I moved to the latest version of al-folio. Yesterday, everything was oke. But today, the numbers of schoolar citations for all my .bib are N/A.
How to reproduce the bug

bundle exec jekyll serve
Look at my website: https://dohaison.github.io/publications/

Error messages and logs
Error fetching citation count for ufrVoPGSRksC: OpenURI::HTTPError - 429 Too Many Requests
Error fetching citation count for 9yKSN-GCB0IC: OpenURI::HTTPError - 429 Too Many Requests
Error fetching citation count for qjMakFHDy7sC: OpenURI::HTTPError - 429 Too Many Requests
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
Error fetching citation count for u-x6o8ySG0sC: OpenURI::HTTPError - 429 Too Many Requests
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
Error fetching citation count for : OpenURI::HTTPError - 429 Too Many Requests
Error fetching citation count for Tyk-4Ss8FVUC: OpenURI::HTTPError - 429 Too Many Requests
Error fetching citation count for WF5omc3nYNoC: OpenURI::HTTPError - 429 Too Many Requests
Error fetching citation count for YsMSGLbcyi4C: OpenURI::HTTPError - 429 Too Many Requests
Error fetching citation count for W7OEmFMy1HYC: OpenURI::HTTPError - 429 Too Many Requests
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
Error fetching citation count for UeHWp8X0CEIC: OpenURI::HTTPError - 429 Too Many Requests
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
Error fetching citation count for 2osOgNQ5qMEC: OpenURI::HTTPError - 429 Too Many Requests
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
Error fetching citation count for IjCSPb-OGe4C: OpenURI::HTTPError - 429 Too Many Requests
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
Error fetching citation count for d1gkVwhDpl0C: OpenURI::HTTPError - 429 Too Many Requests
           Warning: Empty `slug` generated for ''.
           Warning: Empty `slug` generated for ''.
Error fetching citation count for u5HHmVD_uO8C: OpenURI::HTTPError - 429 Too Many Requests
[NbConvertApp] Converting notebook /tmp/jekyll-jupyter-notebook20241005-1679-idtfgf.ipynb to html
/home/haison98/.local/lib/python3.10/site-packages/nbconvert/filters/highlight.py:71: UserWarning: IPython3 lexer unavailable, falling back on Python 3
  return _pygments_highlight(
                    done in 57.976 seconds.
 Auto-regeneration: enabled for '/mnt/c/Users/SON/Desktop/al-folio'
LiveReload address: http://<IP_ADDRESS>:35729
    Server address: http://<IP_ADDRESS>:4000
  Server running... press ctrl-c to stop.
        LiveReload: Browser connected

What operating system are you using?
Linux
Where are you seeing the problem on?
Running locally without Docker, Deployed site
More info
No response

And I wonder that those numbers of scholar citation is live update like almetrics website or static generated when I deploying?

For Google Scholar and INSPIRE they are statically generated during build.
The problem in your case is that every time you build your site it scraps the information from Google Scholar for every paper you added this information. Since you added for a lot of them, it tried to fetch this for all of them, but Google found weird a lot of connection requests from the same IP and blocked it for some time.
One thing that might work is to increase the delay between requests, so it doesn't do a lot of requests in a small time frame and avoid being blocked by Google. Increase it here:
https://github.com/alshedivat/al-folio/blob/421bdf860bb24a052480b99109384361ea7dc73b/_plugins/google-scholar-citations.rb#L32

For Google Scholar and INSPIRE they are statically generated during build.
The problem in your case is that every time you build your site it scraps the information from Google Scholar for every paper you added this information. Since you added for a lot of them, it tried to fetch this for all of them, but Google found weird a lot of connection requests from the same IP and blocked it for some time.
One thing that might work is to increase the delay between requests, so it doesn't do a lot of requests in a small time frame and avoid being blocked by Google. Increase it here:
https://github.com/alshedivat/al-folio/blob/421bdf860bb24a052480b99109384361ea7dc73b/_plugins/google-scholar-citations.rb#L32

Thx for your reply,
I increased the delay from 1.5->3s to 3->5s. However, sometime, the request still be blocked and the same error.
Moreover, more than 3s to fetch a request is too slow. Especially in case, I have a lot of publications in the future (it's more than 10, rn).

One solution that could work them would be to store the results from these fetches and simply reuse them, but them the citation count wouldn't be updated.
Since afaik there's no official Google product to provide these citations count, and we can't simply fetch this information from your profile because they might not all be loaded, there's not much we can do.
Maybe the best solution for you would be to enable this only for some papers, or disabling it entirely.

One solution that could work them would be to store the results from these fetches and simply reuse them, but them the citation count wouldn't be updated.
Since afaik there's no official Google product to provide these citations count, and we can't simply fetch this information from your profile because they might not all be loaded, there's not much we can do.
Maybe the best solution for you would be to enable this only for some papers, or disabling it entirely.

I see. I think right now it's still oke for me, waiting a little bit until fetch all of citation numbers is worthy. Hopefully in the future, we will have official method to get the citation number from scholar.
Thx for all, I'll close this issue.


Dragging the minimap slider sometimes doesn't scroll the editor

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.74.0
OS Version: Windows 11

Steps to Reproduce:

Start dragging the minimap slider from the left edge.
Drag the mouse to the right of the minimap while scrolling.
Notice how the slider on the minimap stops scrolling the editor.

https://user-images.githubusercontent.com/101095063/206411531-0f7820d4-6411-455a-b988-5c7d9d4baf53.mp4

@bevis-hobbs Do you see any errors in developer tools? (Help > Toggle Developer Tools)

@alexdima No, I don't see any errors in the developer tools.
I think it's caused by a really annoying feature of windows that is standard across all scrollbars. This explains the feature: https://ux.stackexchange.com/questions/55623/why-do-scrollbars-revert-to-original-scroll-distance-when-mouse-is-dragged-sidew
It seams strange that in VSCode the minimap slider snaps back even when the mouse is still on the minimap slider. I think you should either increase the buffer zone referred to in the link, to make sure that the minimap slider never snaps back when the mouse is still on the slider while dragging, or completely remove the snap back feature which is how it is on macOS: https://superuser.com/questions/1635460/win10-annoying-back-snapping-of-the-scroll-slider


brave browser icon

this should be on browser category

Looks great. Could you follow the template in https://github.com/edent/SuperTinyIcons/blob/master/CONTRIBUTING.md


Add btpd runit service

Btpd is a really cool BitTorrent client. It can be installed from the AUR (btpd-git).

Hi @ChausseBenjamin, thanks for contributing.
Could you please add support for logging the output of btcli via svlogd?
Basically, just copy over and modify the files conf and log/run from any of the service directories in this repo.
If btcli produces no output worth logging (not a user of it myself), I can merge the PR as is - just let me know.

btpd now logs it's stuff correctly. As of btcli, it doesn't produce any logs. You can view btcli is a command to remotely control the daemon.


Disable schema migration for non-dev-mode

Requests authenticated with master key can temporarily override
this restriction.
connects #93

ping @cheungpat travis fails

@rickmak fixed


[enhancement]: Refactor HotKeys Implementation

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
No response
What should this feature add?
Hotkeys registered all over the app and they often conflict, or do two things for one hotkey
The solution is to use the react-hotkeys-hook’s scoping features to change the hotkey scope depending on where you are in the app, and enable hotkeys based on this.
Discussion notes:
We also want to centralize the key combos & associated actions and make them user-editable.
This would be implemented in stages:

Store the hotkey data in memory (e.g. just a constant)
Implement editing
a. Persist the hotkeys to localStorage
b. Allow editing of the hotkeys
Implement backend permanent storage (?)

Are hotkeys server state or client state? I think server state, so they need to be stored in the backend, but this is open to dissenting opinions
Needs backend storage mechanism - JSON or db?
Needs API routes

Upstream issue: https://github.com/JohannesKlauss/react-hotkeys-hook/issues/1035 → this is not a factor if proper scoped implementation is in place

Alternatives
No response
Aditional Content
No response

Can you assign this to me, please?

The react-hotkeys-hook library has a number of issues. It's not clear if it is capable enough for our needs. It looks like a theoretical v5 will improve things.


InteractionSourceDetected  doesn't work correctly on build (HeadSet, 2017.3)

I get input and controller childs via InteractionSourceDetected and InteractionSourceUpdated. In Editor its always works fine, but in build InteractionSourceDetected not work in first start app. After restart app work fine. (work correctly In 2017.2.0p1-MRTP4)
Unity 2017.3.0f3
HeadSet

I'm unable to repro this in recent Unity versions. Please reopen if it's still an issue.


fix: possible panic if refresh token has a null session_id

What kind of change does this PR introduce?

Prior to the auth.sessions table being created, some refresh tokens can contain a null session_id. In those cases, attempting to use those refresh tokens to obtain a new session will result in a panic.
This PR creates a new session for those refresh tokens that do not have a session_id to prevent panics from happening.

What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

1 of 14 (7.14%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 57.118%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/models/user.go
1
14
7.14%

Totals

Change from base Build<PHONE_NUMBER>1:
-0.04%

Covered Lines:
9565

Relevant Lines:
16746

💛  - Coveralls

we don't need to do the same check here because it's using forUpdate = true - the intention in this block is to retry accessing the refresh_tokens or sessions table after some time because the FOR UPDATE lock is being held by another transaction so it's acceptable for the session to not be found here

👍


⚠ Bofrost.de has degraded performance

In 0bb1ace, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1239 ms

Resolved: Bofrost.de performance has improved in 88cad2a after 6 minutes.


add status checker for README and metadata files

Description
Runs checks for malformed or missing README and metadata files. We'll need to iterate on this since I'm not sure it's perfect.
What To Review
All files please.
How to Test
Might need to merge it in and then throw malformed READMEs and metadata at it.

Merging it in, and will try testing it in different scenarios


Fix cloud networks access in API

I noticed that the cloud networks subcollection actions were not configured correctly, so I added some failing tests and corrected/refactored
@miq-bot add-label api, bug
@miq-bot assign @abellotti

👍  🎵


Upgrade mantle container image to latest HEAD in main

Automated changes by create-pull-request GitHub action

Closing, this pull only the github action updates. Will rerun the actions to bring in the falco driver loader version update.


fix/Sanitise the auth URI to remove invalid / characters

This is a temporary patch to overcome an issue with some OS, like Fedora, where the URI returned has '/' characters after the ':' making the URI invalid for decoding.

Coverage increased (+0.02%) to 83.252% when pulling 4733ce5474dfbd5663a85bbe8a9f769e862f8b93 on bochaco:temp-patch-to-handle-invalid-uri into a6f98916384056434bc7cbae5b0361a3093c3425 on maidsafe:master.


Update CHANGELOG.md

Codecov Report

Merging #19 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #19   +/-   ##
=======================================
  Coverage   71.26%   71.26%
=======================================
  Files          13       13
  Lines         985      985
=======================================
  Hits          702      702
  Misses        283      283

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 34c7959...a89310d. Read the comment docs.


Update mapping for expected events

It has to work with #360 and 0.9 deploy

You probably updating it in #384 @kasperski95, am I right?

duplicate of #357


Upgrade typings to flow 0.53.1

I simply ran the flow-upgrade command, and fixed the ReactElement types. I didn't change anything else, perhaps you'd like me to replace the import  * as React (which is what flow-upgrade does) by more granular imports?

Oops, sorry I didn't see this one and merged #486 first...
Can you check if that PR is suitable?

The PR was correct, but didn't take into account the React.Node type, which I believe is more suited for the return value of the children function.

Sounds fair. Thanks!

Wait, no, React.Children.only returns a ReactElement: https://github.com/facebook/react/blob/cd52f7ed80e23b569634d34e42f8a6ae5948444b/src/isomorphic/children/onlyChild.js#L28

Yep, you're right, my bad, didn't consider that React.Node is a React.ChildrenArray, so a children function with my definition could return [<div />, <div />], which is not compatible.
Sorry for the trouble!

No worries, thanks for the PR still =)

The problem with the ReactElement typing is that it doesn't allow null, falsy or string values, which are still valid return values of render(). For example, in my code I have:
<Motion style={{ opacity: spring(someCondition ? 0 : 1) }}>
  {({ opacity }) => opacity !== 0 && (
    <SomeComponent />
  )}
</Motion>

Which fails the type check because it can return false.

@dapetcu21 I created the PR (#497) to fix this, could you test your use cases against my fork (the branch is flow/onlyChildren)?

Cool! I'll do that, but I can't right now. I'll check it out on Wednesday when I get back to work

@dapetcu21 any update on that?

Oh. Sorry. I forgot about this. I'll add a reminder for tomorrow.

Yup. I can confirm #497 solves my issue. Sorry for taking this long. Things came up and I forgot about this issue.


Deprecate/remove captureQuery

/cc @MaxBittker @benvinegar

My understanding from a discussion with Cramer was that the
server/docs don't even know what query captures are anymore, so it's safe
to remove as long as it doesn't break existing code. If you think we
should keep the existing functionality for now, I'm fine with that.
On Monday, November 28, 2016, Ben Vinegar<EMAIL_ADDRESS>wrote:

@benvinegar commented on this pull request.
In lib/client.js
https://github.com/getsentry/raven-node/pull/239#pullrequestreview-10446956
:

 return this.captureException.apply(this, arguments);

},

captureQuery: function captureQuery() {
utils.consoleAlert('captureQuery has been deprecated and will be removed in v2.0');

will be removed
Doesn't this straight-up remove it? Why not keep the original
implementation w/ this warning and remove in 2.0?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/getsentry/raven-node/pull/239#pullrequestreview-10446956,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYQYKL2ppGgtXMcLbLzleGII0JocVBuks5rC3NHgaJpZM4K-Yb-
.


Improvement for k8s.io/docs/concepts/overview/kubernetes-api/

Is it possible to include one/more example for which utilities(e.g.: kubectl or kubeadm or something else) can take "--runtime-config" parameter?

Hi @shahamit2 , have you looked at the below documentation:
https://kubernetes.io/docs/tasks/administer-cluster/cluster-management/#advanced-topics
Do you think we should include this in the kubernetes-api page?

/language en

/priority backlog


[rust] Support if component of case statements

Split from #831
Checklist

[ ] I have added tests
[ ] I have updated the docs and cheatsheet
[ ] I have not broken the cheatsheet

Seems reasonable. I presume if clauses in Python list comprehensions are treated similarly?

Seems reasonable. I presume if clauses in Python list comprehensions are treated similarly?

Hmm I don't believe that they are.  Does make me wonder if it's what we want; feels a bit small for "if state", as it's not a full statement.  Ie "state" should be a superset of "if state", and I don't think "state" wants to get trapped this far down
Otoh I do sometimes find myself wanting to target the if in python comprehensions
cc/ @AndreasArvidsson wdyt

I guess it's similar to trying to target a ternary in other languages. I do agree that if statement isn't the best description for any of those, but it probably would be useful to be able to target them.

Discussed in meet-up:

[ ] We'd like to switch this to condition
[ ] For "change" it doesn't include the if keyword
[ ] For "chuck" it does include the if keyword

@purpleP fyi we are now using "condition" instead of "if state" here


Register New Device Type and Cluster

I have a device and its type is not listed in Matter Device Library doc.
How can i add (or define )new device type and cluster funtion  ? should i register it with csa ?
Thanks.

@DonP2748 You can add a custom cluster and its attributes, commands. You can check the RainMaker cluster for the APIs here as a reference
Note that, other commissioners won't be able to interact/understand this cluster as this is not a standard cluster. So, you need to have your own controller/commissioner that understands it.

@DonP2748 What is your device type/usecase? Is it like a standard device which is not a part of the specification yet or it's a custom or proprietory feature?

@DonP2748 You can add a custom cluster and its attributes, commands. You can check the RainMaker cluster for the APIs here as a reference

Hello @dhrishi , thanks for your quick reply. Does it  mean that my project must integrate with rainmaker ? Does espressif matter or chip have the same api ?

@DonP2748 What is your device type/usecase? Is it like a standard device which is not a part of the specification yet or it's a custom or proprietory feature?

yeah, my device type is not common in the world, only available in my country.

Does it mean that my project must integrate with rainmaker ? Does espressif matter or chip have the same api ?

No. It was just for reference. That will help you with the APIs you can use to create custom data model (clusters, attributes, commands etc.)

One way to deal with this is to make a composite endpoint with children. The root of this could be as simple as an OnOff cluster and then one of the children is your custom cluster. Doing it this way allows Alexa/Google to still turn the device OnOff. And then your app can have greater control via the custom cluster. Or maybe use a media endpoint for the root cluster. If you use custom only Alexa/Google will have zero control.

That's great idea, i will follow it. Thank you @jonsmirl.


Documentation Error

Hi Folks...
On the page
http://karma-runner.github.io/0.13/intro/installation.html
you have an FAQ link with the href of
http://karma-runner.github.io/0.13/intro/04-faq.md#which-version-of-nodejs-does-karma-run-with-
which is broken.
I believe this should be
http://karma-runner.github.io/0.13/intro/faq.html

thanks want to make a PR? :)

Hi Mark...
Ummm... I'll give it a try. Never done that for a public repo or for documentation. ;-)

Hi Mark...
Gave it
On Tue, Nov 3, 2015 at 4:35 PM, Mark Trostler<EMAIL_ADDRESS>wrote:

thanks want to make a PR? :)
—
Reply to this email directly or view it on GitHub
https://github.com/karma-runner/karma/issues/1682#issuecomment-153534069
.

Hi Mark...
Sorry for the interrupted message before. Anyway, I gave it a shot. Hope
this help.
Cordially,
DHz
On Tue, Nov 3, 2015 at 4:35 PM, Mark Trostler<EMAIL_ADDRESS>wrote:

thanks want to make a PR? :)
—
Reply to this email directly or view it on GitHub
https://github.com/karma-runner/karma/issues/1682#issuecomment-153534069
.

I do not see your PR?
thanks!
Mark
On Tue, Nov 3, 2015 at 6:52 PM, David Holzgang<EMAIL_ADDRESS>wrote:

Hi Mark...
Sorry for the interrupted message before. Anyway, I gave it a shot. Hope
this help.
Cordially,
DHz
On Tue, Nov 3, 2015 at 4:35 PM, Mark Trostler<EMAIL_ADDRESS>wrote:

thanks want to make a PR? :)
—
Reply to this email directly or view it on GitHub
<
https://github.com/karma-runner/karma/issues/1682#issuecomment-153534069>
.

—
Reply to this email directly or view it on GitHub
https://github.com/karma-runner/karma/issues/1682#issuecomment-153555049
.

Closing, as this appears to be fixed by https://github.com/karma-runner/karma-runner.github.com/commit/c2760258284cb195f60dd8e961ecbc065b9a88ce#diff-dc3f769f4b7820f0f31d4c5c9065b392


SampledProperty.prototype.getValue returns invalid Cartesian3

Using a HOLD backwardExtrapolationType in an empty sampledProperty causes getValue to return an invalid Cartesian3.
var sampledPosition = new Cesium.SampledPositionProperty()
sampledPosition.backwardExtrapolationType = Cesium.ExtrapolationType.HOLD
var result = sampledPosition.getValue(Cesium.JulianDate.now())

I would expect the result to be undefined.
Instead, the result is a Cartesian3 with {x: undefined, y: undefined, z: undefined}
When using the default backwardExtrapolationType (NONE), the result is undefined, as expected.

Thanks, nice catch.  I just opened #2605 to address this.  It was a problem with all sampled properties, not just position, thankfully it was an easy fix.


Investigate performance implications of using postgres HStore in DGIdb

From JIRA CI-9.

To be considered as part of #110


'Phantom' symbols after some commands (exit, clear) in the PowerShell running on MacOS

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce

MacOS Ventura v. 13.5
Open any shell (zsh, bash, fish)
Install the latest stable version via brew (brew install --cask powershell)
Run pwsh script
Try to print exit, or clear or l symbols.

Expected behavior
Nothing more is visible after the printed symbols.

Actual behavior
After the command text, there will be 'phantom' symbols like

PS /Users/georgiizagladin> clear1;33R
PS /Users/georgiizagladin> exit3;32R
PS /Users/georgiizagladin> l7;35Re7;41Ra7;47Rr7;53R

However, these symbols are not blocking the execution of the command (you may finish the command, for example, print 'clear', press 'enter' key, and the log will be cleared)

'1;33R' and '3;32R' looks like the coordinates of the pointer in the terminal window.

Error details
Nothing is printed out.

Environment data
PS /Users/georgiizagladin> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.3.6
PSEdition                      Core
GitCommitId                    7.3.6
OS                             Darwin 22.6.0 Darwin Kernel Version 22.6.0: Wed Jul  5 22:21:53 PDT…
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

Hello @StevenBucher98, yes, I found that file it contained all the commands history. I cleared everything from this file, and the error disappeared. Now there are no more additional symbols at the end of a command. Do you know why it works?

I am not sure how you got the phantom symbols to begin with, since they were in the history file, history predictor automatically tries to suggest a command based on what you have executed perviously. If you'd like to disable this behavior you can run Set-PSReadLineOption -PredictionSource None.

Thank you for your explanation.


200% CPU usage and constantly lagging behind tip by ~80 blocks

I am using the cardano-graphql Docker setup. The issue is that since I upgraded it to 5.0.0, the sync is no longer working properly.
The initial sync took ages, and now the situation is that it reports as fully synced but in reality it's always lagging ~80 blocks behind the tip, and the cardano-db-sync process is using 200% CPU (2 out of 4 cores) and ~9 GB RAM. Note that the remaining two cores  are mostly idle.
The system is on a server well beyond the minimum spec listed here. It's a r5a.xlarge AWS server, with Ubuntu 20, 4 CPU cores, 32 GB RAM and 160 GB harddisk.
The logs for cardano-db-sync-extended seem to show normal operation (but if you compare the block numbers to a block explorer you'll see it's lagging behind):
cardano-db-sync-extended_1  | [db-sync-node:Info:16454] [2021-08-17 05:22:45.20 UTC] insertShelleyBlock(Mary): epoch 284, slot 37609429, block 6122619, hash 215ed4b1f0c6f21de81dabbbde04f1cee5cd22e79020e05d9e54426b6fab7013
cardano-db-sync-extended_1  | [db-sync-node:Info:16454] [2021-08-17 05:23:20.97 UTC] insertShelleyBlock(Mary): continuing epoch 284 (slot 284680/432000)
cardano-db-sync-extended_1  | [db-sync-node:Info:16454] [2021-08-17 05:23:20.97 UTC] insertShelleyBlock(Mary): epoch 284, slot 37609480, block 6122620, hash 132d4ee2a795f06d327ac2a4ae69fb513189f2d5820b55873bd2982f6a49d5bc
cardano-db-sync-extended_1  | [db-sync-node:Info:16454] [2021-08-17 05:23:20.97 UTC] Offline pool metadata fetch: 0 results, 28 fetch errors
cardano-db-sync-extended_1  | [db-sync-node:Info:16454] [2021-08-17 05:24:03.76 UTC] insertShelleyBlock(Mary): epoch 284, slot 37609498, block 6122621, hash 1e595380f8f550373cc3d0de2d20608adc431da68bcae284c80584891c38024e
cardano-db-sync-extended_1  | [db-sync-node:Info:16454] [2021-08-17 05:24:39.46 UTC] insertShelleyBlock(Mary): epoch 284, slot 37609514, block 6122622, hash 1cacfd1015b217c975fb043874e4fce4627eb42bf389535be30a33ff3e17e2c5
cardano-db-sync-extended_1  | [db-sync-node:Info:16454] [2021-08-17 05:25:23.05 UTC] insertShelleyBlock(Mary): epoch 284, slot 37609558, block 6122623, hash ec329c076cd2b7242f31fae7bcc5c67370ce0f3cd9091c29c3809a35794fa3f3
cardano-db-sync-extended_1  | [db-sync-node:Info:16454] [2021-08-17 05:26:06.12 UTC] insertShelleyBlock(Mary): epoch 284, slot 37609581, block 6122624, hash 07fd6200ceab95424f6750d8921f9b3199a51ceb25e324893e75e2606380fea3
cardano-db-sync-extended_1  | [db-sync-node:Info:16454] [2021-08-17 05:26:45.19 UTC] insertShelleyBlock(Mary): epoch 284, slot 37609620, block 6122625, hash c9d2b6bfd5dedd12273a76f46378507ef571093c638fc3658c23988834abf8b1

What can I do to diagnose and fix this?

Is that an electro-magnetic disk or an SSD?

SSD... (via EBS)

Do you have IO throughtput numbers for your storage?

If I understand the AWS EBS docs corrwctls, this volume would give me 128 MiB/s.
That's indeed a bit slow. But why would this have become an issue only after the upgrade?
Also, I tried something else now: upgrading to an instance with 8 cores instead of 4. To my surprise that solved the problem, but it's not really logical to me why that would be the case. (plus, it's very expensive and I'd prefer not to keep it this way...)
Disk speed would have sounded much more logical but the number of CPU cores shouldn't affect the issue that much then, right? I'm confused.

4.0.0 stored much, much less data in the database than 10.0.x . 4 CPU cores is sufficient.
There is simply no reason why 8 cores would be faster. I suspect that its simply a different machine with faster IO.

Hm. Thank you, I'll upgrade to a larger volume then (which comes with higher iops as well) and see.

Hi @CherryDT, are you using gp2 or gp3? Maybe you can try gp3 with higher IOPS and you don't have to increase the volume size.

@CherryDT If this is sorted out now, please close this ticket.

It seems the 8 core thing worked only temporarily/randomly, the issue came back.
Anyway, I have switched over to a gp3 volume with 6000 iops and 500mb/s throughput. But, the problem still exists, although it's "a bit" better - it's now lagging only 10-15 blocks behind, but still doesn't fully catch up even after hours. CPU usage is now 90-100% by cardano-db-sync, and 2x 40-50% by two postgres processes...

It seems the 8 core thing worked only temporarily/randomly, the issue came back.

8 cores is more than double what is actually required, When you first moved machines I suspect you had better IO and that is what improved.
This storage is direct connected to the machine and not via a LAN right?

I just checked, it shows up as NVMe device directly.

@rhyslbw ?

@CherryDT Based on my testing for running ADATools.io, I've found AWS largely unusable / extremely expensive. Sometimes queries timeout or never complete. I couldn't read anywhere what instance type you're using, but keep in mind that AWS loves to use crediting systems. I.e, you can have "burstable" instances but then they drop of dramatically once you've used the credits. While it might say that you're using high CPU resources, it can be deceptive as it can also mean how much is stuck in the queue for the CPU to process.
Based on my tests, the 1st thing you need to prioritise is your disk IOPS and 6000 is actually really small for DB usage of this nature. As a reference, DigitalOcean and Vultr can consistently hit above 100,000 SSD IOPS. I've found Vultrs baremetal severs to get your best value for money, and since you have the whole box you have consistent resources. Here's an article that shows the IOPS difference between AWS and DO: https://dzone.com/articles/iops-benchmarking-disk-io-aws-vs-digitalocean
The 2nd thing to keep an eye on is your CPU frequency. Since a lot of it prefers single thread resources, a higher frequency CPU can have better outcomes then multi-cores. That's another reason I prefer DigitalOcean and Vultr and they both offer High Frequency CPU options.

Thank you, I'll try DO or Vultr... Maybe the iops and/or speed should be part of the minimum system requirements in the docs then...?

Thank you, I'll try DO or Vultr... Maybe the iops and/or speed should be part of the minimum system requirements in the docs then...?

Unfortunately there's no direct formula to decide what is a minimum IOPS as it all depends on how complex your query is. E.g, doing a simple transaction lookup would be a lot quicker than doing a sum of all transaction outputs, which would mean someone might only need 5000 IOPS while someone else would need 30,000 IOPS.
IOPS is just another aspect of system design, however due to the diversity of Cardano it makes it more obvious.

Hm but here the issue is that the DB doesn't even get fully synced of it, so the baseline of "no query at all yet" should probably have some number on it... (which I appear to be not reaching)

I wouldn't know what a minimum IOPS would be as I've always gone for systems that are above 65,000 IOPS. It could run on less, but I wouldn't know.

I have now same issue.
Node is synced, only db-sync is actually ~40 minutes behind (not stucked, still inserting blocks, but slowly).
v10
bare metal, nvme, 48 cores, 50% cpu usage.
isnt this related to reward calculations as we are in middle of epoch?

Node is synced, only db-sync is actually ~40 minutes behind (not stucked, still inserting blocks, but slowly).

So db-sync is 40 minutes behind the node for 40 minutes? How much behind?

@erikd it was behind in time of writing 40 minutes behind node (avg 30 minutes) maybe total lagging for 4 hours. in some time 60 minutes behind node, later then 20 minutes behind node, avg maybe 30 minutes. now is it okay.

@erikd btw is db-sync v10 compatible with cardano-node 1.29.0?

@cardanians Sorry, not sure.

We are also encountering this issue - dbsync seems to be struggling to insert blocks at the same speed as the node is receiving them.
Are there any debugging logs we can turn on to discover the source of latency?

PG is co-located on the same physical machine as the node and dbsync.
CPU usage < 30%
RAM usage < 50%
disk IO queue is very small, no spikes...

@stefanofranz Do you have PostgreSQL running on the same machine as db-sync as per the system requirements?

@stefanofranz Machine specs?

@stefanofranz Machine specs?

Ryzen 5900, 64gb ram and 1tb 970 EVO plus SSD x2 in raid 1

Raid 1 is mirroring. I would be expect that to be slower. Would you be able to try this without mirroring?

yes - I can turn it off, but it's very unlikely to be the problem - I've benchmarked PG and it's recording between 50-100K IOPS with latencies in < 1ms on average. The disk queue is effectively empty all the time.
It just seems that dbsync itself is running slow, is there a way I can print out the time it's taking to do PG ops? The next step will be to install monitoring in PG itself.

@stefanofranz Machine specs?

Ryzen 5900, 64gb ram and 1tb 970 EVO plus SSD x2 in raid 1

Is the pg.conf from this git setup or is this your own Postgres setup config? I haven't changed or seen db-sync configs at all so I'm not sure on what the defaults are, but max_connections = 20 seems small to me for any database.

@stefanofranz Machine specs?

Ryzen 5900, 64gb ram and 1tb 970 EVO plus SSD x2 in raid 1

Is the pg.conf from this git setup or is this your own Postgres setup config? I haven't changed or seen db-sync configs at all so I'm not sure on what the defaults are, but max_connections = 20 seems small to me for any database.

it came from: https://pgtune.leopard.in.ua/#/ which has always worked very well - I set connections to 20 as it is ONLY used for dbsync

@stefanofranz Machine specs?

Ryzen 5900, 64gb ram and 1tb 970 EVO plus SSD x2 in raid 1

Is the pg.conf from this git setup or is this your own Postgres setup config? I haven't changed or seen db-sync configs at all so I'm not sure on what the defaults are, but max_connections = 20 seems small to me for any database.

it came from: https://pgtune.leopard.in.ua/#/ which has always worked very well - I set connections to 20 as it is ONLY used for dbsync

When you say, only used for db-sync, do you have no applications connections to Postgres?
I can understand db-sync falling behind if it's fighting for a connection to Postgres while applications are hogging the connections. To me this is the most likely scenario in what's happening, especially if it changes throughout the day based on application peak loads from users.
If it's just sitting there and all it's doing is inserting a block every 20 seconds and it's somehow falling behind, that's a weird one.

On Vultr it works OK now, it seems.
I still think there should be a big warning about this issue at the place where the minimum specs are described, stating that the latest version needs huge I/O resources and any regular configuration on AWS won't suffice even if CPU/RAM resources are well beyond the minimum, recommending a good configuration like Vultr bare metal as you did here. I imagine I'm not the only one who has their whole infrastructure set up on AWS (given it's a pretty popular service used my many large companies too) and had it working well there before until this update...

To be honest I'm still surprised this is even such an issue, because I also have a Bitcoin node with ElectrumX on top which has many times larger a blockchain

The bitcoin blockchain may be bigger in terms of bytes, but is probably not bigger in terms of number of blocks. BTC has a block every approx. 10 minutes, while Cardano has one every 20 seconds.
I suspect the system requirements of ElectrumX are significantly less than db-sync and PostgreSQL.

I can give you access to the troubled install on the AWS server if that helps... You can find me in the member list of the Cardano Discord with the same username.

To be honest I'm still surprised this is even such an issue, because I also have a Bitcoin node with ElectrumX on top which has many times larger a blockchain

If the IOPS is the issues as it's appearing to be, then I'm not surprised and this more of an understanding on how IOPS has an impact on database performance in general and also how AWS and  EBS works.
For a general overview for those that aren't aware, GP2 on AWS increases IOPS as you increase storage or you need to provision enough IOPS yourself to get to the same level as a direct access nvme drive.
Depending on the storage type, AWS also uses quota systems which might explain why it sometimes drops out. E.g, some EBS volumes give you credits to "burst" above the baseline for as long as you have credits. If Cardano has a large amount of transactions to process, it could start eating into the credits and if it runs out, it would then start to lag.
Here's how AWS provisions their IOPS: https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/ebs-volume-types.html

It looks like there hasn't been any activity on this issue for a some time. We're closing it for now. If you're still experiencing this problem or have any updates, please reopen or create a new issue. Your feedback is valuable, thank you!


reopen

Greetings,
Is it possible to reopen startify after I close a file?

never mind. I found it.
:startify

@iomari close this


Basic Support for OpenTelemetry Metrics and Token Usage Metrics in OpenAI V1

Hello all, this is my first PR for issue#251
This PR introduces fundamental components for OpenTelemetry Metrics support, and it add counters for token usage data of openai.resources.chat.completions  in OpenAI V1.
My goal with this PR is to present my approach to addressing the issue at hand and to seek your feedback on the implementation. Your insights and suggestions will be extremely valuable for refining this solution. Moving forward, based on your feedback, I plan to enhance this implementation further and extend support to other instrumentors.
Looking for your reviews and constructive criticism to help improve this integration.

 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.

Nice work @Humbertzhang! Let's connect it to some observability platform to see if it works?

Sharing some work that's being done now with the OpenTelemetry community that we should align with - https://github.com/traceloop/semantic-conventions/pull/2

Hi @nirga!
I setup a demo environment using otel-collector and prometheus, and then setup a openai + traceloop demo for generate metrics.
You can find those files at: https://github.com/Humbertzhang/demo_otel_prometheus
And I think it reports metrics as expect!

Sharing some work that's being done now with the OpenTelemetry community that we should align with - traceloop/semantic-conventions#2

ok @nirga , I will look into this pr  align with it !

Update to Align with https://github.com/traceloop/semantic-conventions/pull/2
Hello @nirga, I have updated my code to align with the changes proposed of openai.chat_completions.tokens metric in https://github.com/traceloop/semantic-conventions/pull/2.
You can see the results under normal conditions in the attached image. I have appropriately added attributes such as llm.response.model and llm.usage.token_type .

You can observe the results under exceptional conditions in below images.
During this run, the network was initially stable but became unreachable for OpenAI requests after a few minutes, resulting in an APIConnectionError.

However, I encountered challenges in retrieving the server.address. I attempted to acquire it in the same manner as the Span(Like https://github.com/traceloop/openllmetry/blob/main/packages/opentelemetry-instrumentation-openai/opentelemetry/instrumentation/openai/shared/init.py#L40), but this it did not get the base_url in practical scenarios(). Do you have any alternative suggestions or insights on this matter?

Looks good @Humbertzhang! Before I review it, can you:

rebase
fix lint issues
add tests (this actually helped me better review PRs in the past 😅)

Reg. your question - can you give an example of when it didn't work for you?

Hi @nirga !
I have added all the suggested metrics of openai and their corresponding tests in this pr:

llm.openai.chat_completions.tokens
llm.openai.chat_completions.choices
llm.openai.chat_completions.duration
llm.openai.embeddings.tokens
llm.openai.embeddings.vector_size
llm.openai.embeddings.duration
llm.openai.image_generations.duration

and I have manually tested all metrics in prometheus.
Additionally, I've completed the rebase and addressed all linting issues.
Regarding server.address, I add a function called _get_openai_base_url, and I always get a "" response by calling it.
This function is intended to mirror the method used by spans to retrieve the OpenAI URL.
I'm uncertain if there's a more effective approach to obtain the OpenAI URL—any suggestions would be appreciated.
For the metric Metric: openai.embeddings.vector_size, I'm contemplating whether a counter is the most suitable instrument for recording it. Maybe Gauge is better for recording it? What do you think about it?
I look forward to your review and any feedback you may have!

Hi @nirga , I have updated tests for metrics using VCR.

Thanks @Humbertzhang !
Hey @nirga , can we get this merged? I was planning to create a PR to enable watsonx as well, and it will depend on this PR, thanks!

@gyliu513 yeah probably today / tomorrow. I need to see why the tests are failing, and I want to move this to a common semantic conventions.

@Humbertzhang looks like there's regression in some tests? 🤔

Hi @nirga, there's a bug that causing the failure of streaming tests, and i have fixed it, u can try to run tests again!

Looks like it's still failing @Humbertzhang. Does it work for you locally?

The previous streaming releated failure is fixed, but there are still some other failure that i am trying to fix. I add metrics tests in traceloop-sdk/tests/metrics , and it breaks other tests in traceloop-sdk/tests/. I'm looking at what causes them to interact with each other (which wasn't a few days ago)

Thank you for your insights and for integrating the tests under OpenAI, which indeed makes more sense.
On the topic of separating metrics accumulation from trace building, my initial intent was to minimize the need for altering existing functions like _with_tracer_wrapper and chat_wrapper, aiming to preserve the current codebase integrity. This approach was chosen to avoid significant disruptions. However, I acknowledge the duplication issue you've pointed out, where we're handling similar attributes in both metrics and traces.
While I had concerns about the complexity and potential bugs that merging these functionalities might introduce, I see the merit in reconsidering this strategy to reduce redundancy. I'm open to further discussions to refine our approach, balancing minimal code impact with efficiency.

Let's do that then @Humbertzhang. I think it will result in cleaner code

Hi @nirga, I've integrated the metrics recording with the previous processes, ran tests locally, and conducted manual testing with Prometheus. Looking forward to your feedback!

Resolved poetry.lock issue affecting tests. Please re-run tests for it.

Nice work @Humbertzhang! It's a significant milestone for OpenLLMetry :)

@Humbertzhang thanks for the work here, could you resolve the conflicts so we can merge? :)

@Humbertzhang thanks for the work here, could you resolve the conflicts so we can merge? :)

Hi @paolorechia, I resolved conflicts in my local env, but when I run tests of traceloop-sdk and openai, I got
FAILED tests/test_privacy_no_prompts.py::test_simple_workflow - openai.APIConnectionError: Connection error.
FAILED tests/test_prompt_management.py::test_prompt_management - openai.APIConnectionError: Connection error.
FAILED tests/test_sdk_initialization.py::test_resource_attributes - openai.APIConnectionError: Connection error.
FAILED tests/test_workflows.py::test_simple_workflow - openai.APIConnectionError: Connection error.

and
>           raise CannotOverwriteExistingCassetteException(cassette=cassette, failed_request=vcr_request)
E           vcr.errors.CannotOverwriteExistingCassetteException: Can't overwrite existing cassette ('/Users/maxzhang/Desktop/githubpjs/openllmetry/packages/traceloop-sdk/tests/cassettes/test_workflows/test_simple_workflow.yaml') in your current record mode ('none').
E           No match for the request (<Request (POST) https://api.openai.com/v1/chat/completions>) was found.
E           Found 1 similar requests with 0 different matcher(s) :
E
E           1 - (<Request (POST) https://api.openai.com/v1/chat/completions>).
E           Matchers succeeded : ['method', 'scheme', 'host', 'port', 'path', 'query']
E           Matchers failed :

errors and exceptions.
Have you  or @nirga encountered similar exceptions when using VCR?

@Humbertzhang I'd try to re-generate the cassettes. Comment out the OpenAI mock key from conftest.py and then run poetry run pytest --record-mode=once

Still got connection errors even i can curl request openai... Maybe u can help me with that @nirga ?
I have pushed my resolve conflict commits.

@nirga  not pushed successfully just now😅, now it is pushed

@Humbertzhang All tests pass now 🤩
Thanks so much for this, it's a significant project!
Can we just also fix and test the API base as well? (where @paolorechia and I commented above)

Hi @nirga  and @paolorechia, I have fixed the _get_openai_base_url func (@paolorechia's code and https://github.com/traceloop/openllmetry/pull/522 really helped me, thx!), it now can get url.
I also added assert checks for it in openai's tests/metrics tests.

does this also work when used with langchain via from langchain_openai import AzureChatOpenAI?

@nitin302 no, we need to add support in the langchain instrumentation separately


Testing - Move default build folder for Tensor/Voxel/Audio tests on HOST/HIP to CWD

@kiritigowda Please merge PR #276 first (changes for moving the output dump folders for QA/Unit/Performance testing)
Getting all tests to pass has many little changes if we move the build folder, so added a separate PR.
Once PR #276 is merged, this PR should show lesser changes in files, thus isolating changes needed ONLY for build folder change to CWD.

@r-abishek there are some merge conflicts on this PR

@kiritigowda Resolved

Merge pending CI - http://math-ci.amd.com/blue/organizations/jenkins/main%2Fprecheckin-computer-vision-rocm%2Frpp/detail/PR-279/2/pipeline


Fix for the issue 2475:

When a VM or Computer with SC4S shutdown suddently because electrical outage or other problems, SC4S container is not removed, so when you try to start again an error is fired with "Can not create, another container with SC4S name exists!"
Executing this ExecStartPre ensures that if the container exists is deleted. If the container doesn't exist the code || /bin/true makes the script continue.
Issue link: https://github.com/splunk/splunk-connect-for-syslog/issues/2475
@ikheifets-splunk is checking the issue

I have read the CLA Document and I hereby sign the CLA

recheck

I have read the Code of Conduct and I hereby accept the Terms

recheck


page index of searchCourse

If the page index is so large that there is no message, should I through an exception or return a null list?

Return an empty list.

what is the start value of page index, starting from 0 or 1

what is the start value of page index, starting from 0 or 1

0


Roll engine ef99738bafc0..72f747ae4300 (15 commits)

https://github.com/flutter/engine/compare/ef99738bafc0...72f747ae4300
git log ef99738bafc08861412655ec0a84e966c98c633a..72f747ae4300509116928350ae7d8affb92d791c --no-merges --oneline
72f747ae4 Roll fuchsia/clang/linux-amd64 from apsjZJ9ZIwlS08divc0QwxuNgIFeN4X-52mKR9d8qacC to zHiuOGMDwdWPUkV1B2fHKyRy2fGWGcUgfa2z6dyGAZQC (flutter/engine#10084)
bfe689ee8 Roll fuchsia/clang/mac-amd64 from USm3Qt9wen4bXoFF0T9Xt8NX-pyEfte2cTYi7geLAhQC to UYD9C8IEkWVi83ef4zwO1Ump2B8cP9Nd5WECjU8l3AUC (flutter/engine#10083)
cf5d76c17 [fuchsia] Add support for libs in packages (flutter/engine#10081)
c4c64896a Roll src/third_party/skia 829144cc76fe..d15571af2e1b (2 commits) (flutter/engine#10080)
25e5d597e Roll fuchsia/sdk/core/linux-amd64 from S_24mhItBb7pfUof8zsJPPSctC7xdlfjDEXEtyO25GYC to kU0OwJYlceN477ejdh9RXrl7Mp-wUTNvQozlaV96xmQC (flutter/engine#10079)
f7e0614b8 Change ParagraphBuilder to replace the parent style's font families with the child style's font families (flutter/engine#10074)
eb74f2ce7 One more luci fix (flutter/engine#10078)
6e732c7f1 Change flutter runner target for LUCI (flutter/engine#10075)
44ec59b70 Roll src/third_party/dart fedd74669a..9c148623c5 (70 commits) (flutter/engine#10066)
abf93699f Basic structure for flutter_jit_runner far (flutter/engine#10073)
3c76b90b2 Track clusters and return cluster boundaries in getGlyphPositionForCoordinates (emoji fix) (flutter/engine#10063)
e8fb641f6 Enable consts from environment in DDK for flutter_web (flutter/engine#10069)
bd3870219 [macos] Revert check on FlutterCodecs and refactor message function] (flutter/engine#10009)
d6e1a93e2 Roll src/third_party/skia e11dfd3da4d7..829144cc76fe (4 commits) (flutter/engine#10072)
d595af681 Roll fuchsia/sdk/core/linux-amd64 from XqtWTBni4xpYCTr7gqU7rFTuXNY1TZ_zOqBJrZM8c_kC to S_24mhItBb7pfUof8zsJPPSctC7xdlfjDEXEtyO25GYC (flutter/engine#10071)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (cbracken@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5054328178475008

Commit queue failed; closing this roll.


Bump version to 1.7.2-SNAPSHOT (gax) and 0.24.2-SNAPSHOT (gax-grpc)

Codecov Report

Merging #369 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #369   +/-   ##
=========================================
  Coverage     73.99%   73.99%
  Complexity      505      505
=========================================
  Files           132      132
  Lines          2646     2646
  Branches        184      184
=========================================
  Hits           1958     1958
  Misses          613      613
  Partials         75       75

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 8e4e7fb...9a05c98. Read the comment docs.


new feature?

Hi,
is there planned for adding supporting !admin [msg] via discord e.g?

Wonder if this is still maintained :(

i dont think it's maintained anymore. because there is weird bugs on orangebot.. like BO3.. where the 1stmap works fine. but 2ndmap is bugged.. where all players got "guns" after starting.. it should be knife.. no no.. "real guns" xD

hey guys :)
sorry i've been really busy with other projects + university
@MikkeIDK: Unfortunately i'm not planning to touch discord.
@huytc: is there something i can do for you? if it's not that big maybe i can implement it or hint you to the solution (if you have experience with javascript)

@dejavueakay it's alright. i have found some bugs.. dunno about it can be fixed from you? :D

btw @dejavueakay are you still active? - about you maybe can looking, why the orangebot don't starting/ from 2ndmap e.g !start de_dust2 de_inferno de_nuke > and it wont starting with scores from the de_nuke.
Thanks ;)

Hi there!
I still actively play CS:GO, but unfortunately due to university i lack the time to do any side projects :(
I'm sorry i cannot imagine what the problem might be because i have tested that feature and did not find any bugs - what do you mean exactly by "scores from de_nuke"?
Am 03.02.2019 um 20:14 schrieb Mikkel <notifications<EMAIL_ADDRESS>btw @dejavueakayhttps://github.com/dejavueakay are you still active? - about you maybe can looking, why the orangebot don't starting/ from 2ndmap e.g !start de_dust2 de_inferno de_nuke > and it wont starting with scores from the de_nuke.
Thanks ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dejavueakay/orangebot/issues/4#issuecomment-460079428, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AeEo8ZOQxzpku-4FKuMrr5wk_57J9Zrdks5vJzUhgaJpZM4YnmC7.

Hi @dejavueakay
It's because, if you are starting the orangebot on the server.
And setting the server to starting with 3 maps - as example: de_dust2, de_nuke, de_inferno.
I have experienced when I finish the first map and move on to the next map, so it should work without the score itself being fucked up.
I will try and test again, and then I just come back with picture if any.


Avoid attaching react component instances to window object

Hi,
the  user event handling registration process currently applied in src/views/main.tsx is not a proper solution. Each of the major components are registered to the global windows object in its component constructor.
window.x = this;

There is already a comment in each major component constructor that such event propagation currently applied is not a proper solution.
// FIXME: find a better design to propagate events.

We are using black-screen not as a standalone application but rather as a external git sub-module. For now we are stuck with version 0.2.13. In version 0.2.14 the old event handling registration was replaced with the current one.
Maybe this kind of problem could be solved through a HOC?
Thanks!

May I know what's HOC, @gurkerl83?
Also, if it's not a secret, I'm curios who are "we", and what's your use-case.

@shockone
A while ago I started watching the projects react-higher-event and react-outside-event. Maybe those libraries can help.
We are a small group of developers from various fields. Right no we are working on a privacy enhancing technology called Project Millipede more info at http://millipede.me/#home.
The primary use case is not to enable a terminal to look nice and provide a set of powerful auto complete mechanic. If I`m not mistaken black-screen listens to signals from a system's terminal such as bash and calls those very nice assistant features. The same applies to our use case with the difference that we are listening to application events triggered from within an Java application. From the perspective it is really no difference to a local git installation where black-screen provides auto complete features.
Short description:
We are using a framework called OSGi for application modularisation, service declaration...
Application modules are executed through an OSGi Runtime such as Apache Felix, Equinox...
In oder to provide the user with interactive capabilities we use a software container, which contains powerful features such as command declaration (Apache Karaf).
This enables us to create command modules, which can be administered through a typical terminal.
Black-screen and the auto complete features provide a way to unify the look and feel throughout multiple platforms providing the user a way to instruct the the application (command modules).
Thanks!


dockerise

Mapp should be dockerised.
Key features needed:

production flask server
redis server (stored appropriately)
tools script runnable

Help here is appreciated.

why should it be dockerised?

Done in https://github.com/compsoc-edinburgh/mapp-site/commit/15ee8995d768b623650602014b6252391ae7b5a5#diff-3254677a7917c6c01f55212f86c57fbf


A few improvements in code and CSS along with some config parameters.

Just a few improvements that I found useful:

Silence some php warnings.
Make better CSS alignment in totals.
Add config parameter: Control refresh rate.
Add config parameter: Add extra external CSS.

Hey there!
Would you mind adding some logic that defaults to 20 seconds if the $refresh_every_ms variable isn't set? (which would happen if people upgrade and don't spot the new config param).
Thanks!

Fixes #54

@lozzd Yeap, it makes sense to do it this way.
Thank you.

Thanks!

Cheers!


Create new release including httpserver

Hi!
I just installed the latest release on a Raspberry Pi (armv6) by following these directions

If using Linux: Download the latest release, unzip using tar -xzf go-chromecast.tar.gz, and install using sudo install ./go-chromecast /usr/bin/

Unfortunately, it seems that the latest release does not yet include the httpserver feature that was recently added. Could a new release/build be added that includes this feature?
Thanks!

Ah, yes sorry, I have been meaning to do this. I will get this done in the next couple of days :)

https://github.com/vishen/go-chromecast/releases/tag/v0.2.6 has been released which included the http server :)

Awesome, thanks so much!


Deal limit is incorrect

HubSpot documentation for Get All Deals describes the Limit optional parameter as defaulting to 100 but has a maximum value of 250.
When I retrieve all deals with a limit of 250 using HubSpot.NET an exception is thrown stating the following:
"Number of items to return must be a positive integer greater than 0, and less than 100 - you provided 250"

Caused by the HubSpotDealApi's List method requiring a ListRequestOptions parameter.
ListRequestOptions's Limit property has a upper limit of 100.

This is fixed in 0.6.17 - thanks to @Mark-RSK.


Compute extract path basename based on tarball name.  Add md5 support.

cc @chef-cookbooks/engineering-services
(Should be in delivery; will add it when I learn how.)

This has been replaced by the Deliverized version in https://github.com/chef-cookbooks/remote_install/pull/3


Niols/fix tools pre commit

This pull request ensures that the tools used by our pre-commit hooks and the CI are the exact same than the ones available in the nix develop .#default environment. This reduces the size of our derivations by factoring in some things (our SSDs should be happy) and this removes some discrepancies between environments, therefore closing #335.
Because of the aforementioned discrepancies, this pull request also re-formats all the files. The changes are minor but non negligible.

I'd just suggest using a .git-blame-ignore-revs

Done in 114d328.


add safeguard to circular references

Coverage remained the same at 100.0% when pulling 98edb3bf9d45eff1498fe0de254df3874d9d63e5 on feature/safeguard-circular-references into 61c88c1b10334b22adc4a6c36e5d52582b997f5b on master.


Bad connections pool implemetation

I've found multiple problems with current pool implementation:

Pool is blocking.
There is no async initialization which leads to next problems,
There is no way to set minimum number of connections that are created at pool initialization.
Instead of asyncronously blocking until connection is available, pool will raise exception:
https://github.com/NoneGG/aredis/blob/cec3cdabe67f45493ac413737d8579f46c27fded/aredis/pool.py#L225-L226
This makes any command invocation unsafe and requires user to do error handling. Current behaviour could be preserved with option though.
Addressing these issues will mean making backwards-incompatible changes (making get_connection async at least).

Some examples of pool implementations that could be referenced:

https://github.com/MagicStack/asyncpg/blob/master/asyncpg/pool.py
https://github.com/aio-libs/aioredis/blob/master/aioredis/pool.py

Thats affects me, too. Thats really disappoint.

I emailed @NoneGG asking about a possibility of releasing 2.0.0 with a few breaking changes addressing pool implementation and other issues

I emailed @NoneGG asking about a possibility of releasing 2.0.0 with a few breaking changes addressing pool implementation and other issues

May I know when version 2.0.0 is released?

May I know when version 2.0.0 is released?

I haven't got a response from maintainer yet, there haven't been any work on 2.0.0 so far.

Thanks for your efforts @Fogapod. I think there will be many new users here, since aioredis seems to have been abandoned. Let's hope this project can become a good replacement.

@Fogapod Aren't responses from the maintainer yet?
I really want to use aredis on my big projects.

@drgarcia1986 unfortunately, no. And I don't think I have time to work on aredis at this point.

@Fogapod maybe if @NoneGG add more maintainers, we can work together to fix bugs and rethink the connection pool design

We just hit this in production too:
https://gitlab.com/fluidattacks/product/-/issues/3874
I'm willing to contribute a patch, but I'll need a hand with the desired interface/implementation, at least a skeleton or a few words. This in order to keep the library useful to the most people as possible and not break too much compatibility


gravitas should contain logging equivalent options

When using a pacing function the option to change the gravitas needs to have the ability to pass in the equivalent terminology used by logging module.
@beat("msg", gravitas.INFO)
This would make using gravitas more accessible to folks who are already familiar with python logging.

It would be really nice if the gravitas argument accepted the adaptor values directly.
@beat("msg", logging.INFO)
aside("msg", allure.severity_level.MINOR)
We could still include gravitas.py values LIGHT etc which just point to the adaptor levels.

I wasn't considering the fact a test suite might use multiple adapters  (adaptor? adapter? am I British?) at same time.  Which seriously complicates trying to pass in the adapter value directly.

From our conversation though, it would be nice if each adapter supplied the appropriate equivalent, just keyed back to gravitas. For example, the StdOutAdapter can provide DEBUG = gravitas.AIRY, INFO = gravitas.LIGHT, etc. That would probably be nice!

I thought of a snag in this idea; indentation vs log level.  The indentation assumes that all messages get logged.  If we allow users to set the log levels of some messages the output could look really odd.  If there happens to be a way to take log level into account when deciding to indent, this can work.
I have doubts.

I think this might be a good place to use enum.   It allows the gravatas value types to still be int,  while guiding the user to hint that it only accepts the values found in the appropriate enum.

I think what I want to do is allow users to pass in either the int value in accordance with whatever Adapter they are using or pass in the Gravitas(StrEnum).  Is that too complex for folks to follow?
So the following two lines would be equivalent
aside("foo", gravitas=logging.DEBUG)  # int
aside("foo", gravitas=Gravitas.AIRY)  # Enum(str)
When they pass in the integer, the narrator just passes it through.  This assumes the user knows better if they are using more than one adapter.
When they pass the Enum, each adapter would have it's own translation of the gravitas levels.


Reflect Correct Node Version in manual-setup.adoc

redux of #64 so I could edit it a bit!
The suggested node version in the manual setup doc is behind what we actually use, so this brings the doc up to date.
Tagging @Shadowfiend for review.


v2.1-rc

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #63
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
The scope of this PR is to release all approved work from the v2.1-rc branch of work.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[x] Release v2.1-rc branch of work

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[BUG] isValid flag on submission is set wrongly when number field is added to the form

Environment
Please provide as many details as you can:

Hosting type

[] Form.io
[x] Local deployment

Version: N/A

Formio.js version: 4.13.9
Frontend framework: React
Browser: Google Chrome
Browser version: Version 95.0.4638.69 (Official Build) (64-bit)

Steps to Reproduce

Download the sample create-react-app project with reproducible example:
formio-validation-number.zip

In the app main catalog, execute npm install and then npm start to start the web app.

On the web app, you can see a form. By default, it renders a form (both forms' JSONs are hard-coded in formView.jsx file) with one text area and one checkbox. You can see that the text area control is required:

The text area has no value filled, so the whole form is invalid and shouldn't be submitted (it contains validation errors).

Open dev tools (F12) and see that when clicking on "Form without number" button (which simply re-renders this form), the submission parameter of onChange function has isValid flag correctly set to false:

This is so far correct.

Now click on "Form with number" button. This button does nothing more than changing the form prop of the Form component to the one with one additional control - a field of type number. When you click this button once, see what happens in the devtools console (F12) with isValid flag:

This is wrong. The isValid flag on submission object should still be false. Validation of the form is still not passing (form is still in an invalid state).
Expected behavior
When changing the form on Form component, the isValid flag on submission object passed in onChange callback function should not change. It should always reflect the actual validation status of the form.
Observed behavior
submission.isValid flag changes from false to true when only form definition is changed (number control is added).
I also noticed that it doesn't happen with all controls. If you add another text area, this problem doesn't occur.
More info
It might be related to #411
It's very important to fix that. It makes it impossible to block saving/submitting of the form when it has some validation errors, and you are using a custom submit button (not the FormIO's default one defined in form JSON as a component). It's not reliable at all and users of our application are getting frustrated with these strange validation issues.

I'm having the same sort of issue with the isValid flag.  For me, the isValid flag is reporting that the form is valid on initial render, but the Form's internal state seems to contradict that state since the included Submit button is disabled still.  Without access to the form's ref (#407), this makes using a custom submit button almost impossible with this library since I can't manually call the checkValidity function either.
@mm-dsibinski Did you manage to find any workaround to this isValid issue in the meantime?
Thanks!

@crcollver unfortunately no... Validation in FormIO component is totally unreliable. We've been already doing many "haxes" in the JSONs of the forms/submission objects for different things, because FormIO is veeery buggy, but haven't found any solution for valiation problems.

For me I just went ahead and coded my own check.
// data
const { o_form, o_submission } = props;  // feed your own stuff
const [construct] = React.useState(  o_form  ); // the formio json definition from database / etc.
const [originalSubmission] = React.useState( {data:   o_submission  } ); // initial form data to load.

const [canSubmit, setCanSubmit] = React.useState(true);
const [formReady, setFormReady] = React.useState(false);
const [submission, setSubmission] = React.useState({}); // value fed back from FormIO upon change.

// reactive events
const onChange = (submission) => {
	setSubmission(submission);
};
const onFormReady = (instance) => {
	setFormReady(true);
};

React.useEffect(() => {
	if (formReady) {
		let valid = submission.isValid;
		if (valid) { // isValid is seemingly flawed @ initialization: https://github.com/formio/react/issues/415
			valid = checkValidity(submission);
		}
		setCanSubmit(valid);
	}
}, [submission]);

// if any required field is blank, obviously the form isn't really valid. Supplemental fix for an issue #415.
const checkValidity = (data, _default) => {
	let valid = _default;
	for (let comp of construct.components) {
		if (comp.validate && comp.validate.required) {
			var checkForAnythingAtAll = data[comp.key];
			if (!checkForAnythingAtAll || checkForAnythingAtAll === '') {
				valid = false;
				break;
			}
		}
	}
	return valid;
};

// display
return (
<React.Fragment>
	<Form
		form={construct}
		submission={originalSubmission}
		formReady={onFormReady}
		onChange={onChange}
	/>
	<Button disabled={ !canSubmit }>
		Weee!
	</Button>
</React.Fragment>
);

React renders the form, then it triggers formReady, then it triggers onChange with initial empty(or default or provisioned) values for the form.
So it's 3 render cycles. 1st cycle:  isValid=false. Same for 2nd. 3rd cycle, isValid=true.
So idea in above is simply to override the isValid boolean with our own logic for the "initial" (3rd) render cycle.
You could supplement your own logic for checkValidity, if you don't have any required fields..
After that, the onChange event properly sends us isValid without any custom code needed (it starts working again).
Without a ref, perhaps we need some utility functions (execute: validate(data) )  exposed via using hooks to comply with react.
something like...
import {Form, useFormIO} from '@formio/react';
const {utility} = useFormIO();

.... code from above ...
valid = utility.Validate(submission);

Easier said than done I know. Just a suggestion. If I had time I'd dig into it and help, but i have none. I shouldn't even be here right now typing this.

We're currently addressing a backlog of GitHub issues. Closing this thread as it is outdated. Please re-open if it is still relevant. Thank you for your contribution!


Generate version before pull

As part of the pull process, it would be nice to have an option to generate a version for the version history before AG changes the object (for objects that support version history).
Seems pretty easy to do by calling function SVRS_AFTER_CHANGED_ONLINE_INT.
Note: Makes sense only for local packages since pulling with transportable packages generates the version automatically.

Does a version generated by this FM persist after transport?

yes, i believe so

Can be done by adding the function to DESERIALIZE_POSTPROCESS exit
https://docs.abapgit.org/ref-exits.html#deserialize_postprocess


Smart replace with "null" on readonly

Example:
class SomeClass
{
    public function __construct(private readonly $variable = 'default') {
    }
}

Applying Smart replace with "null" generates invalid code because readonly means its value cannot be changed after initialization and the ??= operator will generate error
In this case skip inspection for readonly arguments

Related issue #32

Thank you for your time in reporting this issue. The fix has been implemented in version 0.25.0 and is available exclusively through the Releases page. It will be implemented in the JetBrains Marketplace once this issue that are affecting some users are resolved (no ETA).
In response to your report:

The issue has been identified and fixed. Now, it will no longer suggest using null when the promoted property is readonly. I've proposed an idea to make this possible to the PHP team, but I don't think it will go very far (https://github.com/php/php-src/issues/12452).

Speaking "off-the-record" as a follow-up to that report, my recommendation is either:
(a) To not use a promoted property in the case where you need a non-null default value. Promoted properties aren't meant to be a replacement to defined properties, but merely shorthand to save developers from the boilerplate code of
public function __construct(int $a, int $b, int $c, int $d) {
    $this->a = $a;
    $this->b = $b;
 }`. Personally, I don't mix promoted and non-promoted properties in a class as that makes it harder to read the code: "I know there's an assignment to a property but I don't see it in the list of defined properties?" and such. It's much more suitable for "data purse"-type classes.

(b) To leave the default value as null and to apply the special logic at the time the value is needed. This has a significant benefit of allowing the property to retain its "default value"-ness; in this example, if the property has a value of `123` then you've lost (even if intentionally) the ability to distinguish between it having that value explicitly vs. it not having a value provided and you applying a default in its place. I also like this as it moves the concept of the default value to the location where the default actually has meaning: the place it's being used, whereas the constructor should have as little logic (business-wise or code-wise) as possible.

Unsurprisingly, the latter is what I would do myself.

If I understand correctly, I agree with you to some extent. If I've understood it correctly, it's precisely the concept that 0 and null are distinct and individually important values (where null means the absence of information, and 0 essentially means the value zero). However, the plugin suggests only when int is an acceptable value on its own, making it null to apply the "default" rule. But for cases where a distinction needs to be made, it should be done with caution. I prefer the application in the constructor, which, although I agree should be straightforward, is the best place to ensure that the entire application will respect the rule, and it wouldn't be necessary to use ?? all the time, for example (if that's what I understood from case B).

Yes, you're right. I'm thinking with more of a purist sense of design, and what you need here is something more practical - and something not as enthusiastic about making big changes to someone's code, too. And while I like the idea of preserving that "default value"-ness, I'll admit that it's just not something that ever really matters.
Demoting to a normal parameter is probably going to be the best solution for resolving the underlying issue of using a non-null default, as you've already considered. It's a little bit more intrusive, too, but I'll leave that to your judgment 👍


Update mapbox-gl-draw, fixing the active styling on the controls

Before:
https://user-images.githubusercontent.com/1664407/216993545-361ffb16-237a-4559-b761-432574e33654.mp4
After:
https://user-images.githubusercontent.com/1664407/216993568-aa367f9b-2af1-489e-93f7-c9893b861679.mp4
Note the polygon tool is shaded when active on the 2nd. The other changes in https://github.com/mapbox/mapbox-gl-draw/blob/main/CHANGELOG.md don't affect us.

I'll merge and only the dev deployment on Github will change. atip.uk is prod, deployed explicitly. We can decide how often to release fixes there.


The model training time is too long

I’ve trying to train a student model on my RTX 4070 12Gb.
The face training was quick, taking only 4 hours (with a target of 1 million steps).
However, the body training has already reached the tenth day and is only at 370,000 steps (with a target of 1.5 million steps), averaging 1500~2000 steps/hr.
Is this normal? Would you recommend ending the training early?
Or what size VRAM hardware should be recommended for training?
Thank you for your reply.

我嘗試在我的 RTX 4070 12Gb 上訓練學生模型。 面部訓練很快，只用了 4 個小時（目標是 100 萬步）。然而，身體訓練已經到了第十天，只有37萬步（目標150萬步），平均1500~2000步/小時。你會建議提前結束培訓嗎？或者應該推薦什麼尺寸的VRAM硬體進行訓練？感謝您的回復。

It's okay. I found out myself that body_morpher_batch_size was set too high, so I adjusted it to 6 or 4, and it's much faster.


[FLINK-13095][state-processor-api] Provide an easy way to read / bootstrap window state using the State Processor API

What is the purpose of the change
Provide an easy way to read / bootstrap window state using the State Processor API.
Brief changelog
e5dcf6f Adds support for listing namespaces for a state type from KeyedStateBackend. This is required as windows are encoded as namespaces in the state backend and up until now there has never been a way to query namespaces, all operations assume the calling code already knows what namespace they are interested in interacting with.
ffd3983 Refactors the KeyedStateInputFormat to support arbitrary user function types. It now follows the model of Task -> Operator -> Function where the input format is the task, StateReaderOperator is the low-level operator that can be implemented for any function type. In the future this allows us to support reading other low-level encodings such as stateful functions.
f6348bb Adds support for reading window operator state by implementing a WindowStateReaderOperator and WindowReaderFunction.
fbd88b7 Extracts the logic from WindowedStream into a builder class so that there is one definitive way to create and configure the window operator. I ran all the end to end tests to ensure everything works as expected.
bf4e549 Cursory refactoring of some test utilities
0169be0 Adds support for bootstrapping window state. While this commit contains a lot of code it is all boilerplate due to the many ways users can configure the window operator. All user-facing changes simply sit on top of the WindowOperatorBuilder.
Verifying this change
Unit and IT tests cases
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? JavaDoc

@flinkbot attention @tzulitai
The docs for the state processor api need a serious overhaul. If it's alright with you I'd like to leave this PR with just JavaDoc. I've already set aside some time next week to rewrite the docs for this component and will add window content at that time.


Migration to the new Backstage backend system

Summary

Hey!
Backstage is now using new backend system and it would be nice if Dynatrace plugin was also migrated to that new backend system.
Feature Suggestion

It would be great to have Dynatrace Plugin migrated to new backend system
Possible Implementation

Useful links to perform migration:

Plugin migration guide
Youtube video showing migration of a simple plugin
Documentation on Backend Services
Backend migration guide might come in handy

Hi,
I'm bumping this issue because it would help us a lot with our internal backstage instance.
In order to increase security (turn on authentication by default), we would like to leverage the new authentication system but as a prerequisite, all plugins have to be migrated to the new backend.
It would be great if you could plan this feature. Alternatively, we can also discuss if a contribution from our side makes sense.

Thanks for the feedback :)
We're currently eyeing July to work on this update. Would this timeline work for you?

That sounds good, thank you!

@MrManny We are wants to use this plugin. but we are new backend system. do you have exact ETA in july?

@MrManny We wants to use this plugin. but we are on new backend system. do you have exact ETA in July?

@tobigremmer-dt has been kind enough to take this into their capable hands :)
I would put the exact ETA at "soon, likely in the coming hours/days"


Add itemExtent and prototypeItem to DropdownButton and DropdownMenu

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Use case
As per #139908 and #52207, scrollbars are effectively unusable inside dropdowns with many items because of scroll animation jank.
Proposal
ListView can mitigate the animation jank using fixed itemExtent for all items. I propose Flutter adds an equivalent mechanism to DropdownButton and DropdownMenu. In the case of the DropdownButton it's a matter of passing those parameters from the constructor to the underlying ListView. The DropdownMenu seems to be harder to fix because it uses a more general MenuAnchor which is built as a Flex inside of SingleChildScrollView.

Hi @sashkent3, thanks for filing this issue. Can you split off the proposal for DropdownMenu into its own issue? It may need to handled differently compared to DropdownButton and we wouldn't want this getting closed if a fix for one is merged while the fix for the other hasn't merged

@danagbemava-nc Done, see #140057 for a similar issue about DropdownMenu. Should I edit this one to exclude DropdownMenu?

Btw as the DropdownButton fix seems trivial I believe I can submit the relevant PR right away. Should I?

@sashkent3, prs are always welcome. I will update this to remove DropdownMenu


Lazyloading would not reset the scrollbar location

Current version 0.1.3 would reset the scrollbar location to the top after reloading request is finished. My wish is the scroll inview position would stay where it was. So that the lazy loaing is really a "lazy loading" that 1. it happens in the background. 2. end users do not aware of it (no other change in GUI). Thanks!

I assume you are not using push? If blocking async-calls are used then delayed response will update the scroll position of parent back to position it had when lazyloading request was made. To avoid this use push, and handle any longer loading in separate thread.
To resolve non-push issues custom panel/window/tabsheet would be required.


chore(wasm): updates wasm + group icon

What this PR does 📖

Updates wasm to 1.6.1
Fixes permission issue with kicking users. A new permission to allow that was added
Updated various wasm typings in warp integration
Added group/chat icon (technically banner too but atm no way to change and/or display that)
Wired up the possibility to change group icon

Which issue(s) this PR fixes 🔨

Resolve #662
Resolve #771
Resolve #773

i had that happen once... and now i cant reproduce it anymore. tried different browsers, refreshing multiple times etc. can you test again?

So was able to test with Sara with the fix you gave us but I had some weird behavior that Sara wasnt able to replicate. If you cant replicate either its all good wont try to hold this up any longer just wanted to mention it
So with the group that I created everything works fine but when I go to the group that Sara created my entire app freezes up and I get the error that I posted below. If I refresh I can move around the app again until I go to the group Sara created then it breaks again. I created a group with another account and it also seemed to work fine but that was with another account on the same local network so maybe it could be something sometimes happening with users not over local net
Screen_Recording_2024-11-22_at_1.44.38_PM.mov

thats strange. as the data there shouldnt be undefined... i could for now add a check for that though

Noticed something while testing with Sara.
When a user tries to kick themselves from the group nothing happens on their side and they remain in the group.
I also still get a bug sometimes when whole app just freezes but it seems totally random
Flow: -UserB kicks themselves -On UserB side nothing changes and they appear as if they are in the group still -On UserA side it appears that UserB is no longer in the group -After UserB kicks themselves but groupchat remains on their side any messages sent in the groupchat userB will not receive.
There doesn't seem to be any error in chat when UserB kicks themselves
Screen.Recording.2024-11-25.at.10.02.48.AM.mov

do we even want someone to kick themselves?

Noticed something while testing with Sara.
When a user tries to kick themselves from the group nothing happens on their side and they remain in the group.
I also still get a bug sometimes when whole app just freezes but it seems totally random
Flow: -UserB kicks themselves -On UserB side nothing changes and they appear as if they are in the group still -On UserA side it appears that UserB is no longer in the group -After UserB kicks themselves but groupchat remains on their side any messages sent in the groupchat userB will not receive.
There doesn't seem to be any error in chat when UserB kicks themselves
Screen.Recording.2024-11-25.at.10.02.48.AM.mov

do we even want someone to kick themselves?

I mean right now thats the only way for a user to leave a group they dont wanna be in

ok. it seems like that we simply dont receive the conversation deleted event when kicking ourselfs. this is most likely a warp problem and thus i would say we make a ticket for it and handle it in another pr to not hold this up anymore


cli to ban/unban user?

any pivpn cli to ban/unban user? only ban, no revoke i test this https://docs.openvpn.net/command-line/managing-user-and-group-properties-from-command-line/ but no work. Thanks

I didn't dig in too much, but setting up openvpn client config directories ( aka ccd ) on server side and pushing a dummy route / subnet to the client you want to "ban" should do it. Later to "unban" just comment the line you've added previously.
Here's something related: https://superuser.com/questions/663024/using-ccd-openvpn-user-can-connect-but-cant-access-other-devices.

Rethinking on this and came up with the fact that what I said can be bypassed by setting the routes on client side, so if the user is technical he'll connect anyway.
To solve this you should set username and password as required to connect, too. In this case the user set the credentials along with the private key in the ovpn file, so they're not going to be re-typed every time. This way, if you want to ban an user just change the password and set it back to unban.

Don't see any benefit in such an option ... either you want the user there or not ... either the user has a certificate or not. easy. why complicate what is simple.
Closing issue.


feat(jest-runner): set exit code to 1 if test logs after teardown

Summary
Resolve https://github.com/facebook/jest/issues/10715

Test plan
Green CI

I'm not sure under which section should I put it in Changelog. fix, feat, or something else?

I'm not sure under which section should I put it in Changelog. fix, feat, or something else?

feat 🙂 Make sure to mark it as breaking as well

Thank you for doing this!

Does this change only affect non-watch mode? Because I can see that in Jest 27 tests still pass, except if I run it in non-watch mode, then the process will exit with exit code 1, but otherwise in watch mode, there's just a warning, but tests still pass.

That sounds like a bug. Could you open up a new issue?


MFA for Only Sensitive Parts of App, How?

You don't require Multi-Factor Authentication to access an application in general, but you do require it to access the sensitive portions within it

Can the above be expanded upon or linked to a doc that describes this further?  How would one do this?

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

ID: d8c394d1-e3b1-b88c-416c-3f8e0ab3ae3d
Version Independent ID: 8aafb9d8-72cb-c9b4-5a01-9e5633764f39
Content: Azure Active Directory B2C: Multi-Factor Authentication
Content Source: articles/active-directory-b2c/active-directory-b2c-reference-mfa.md
Service: active-directory-b2c
GitHub Login: @davidmu1
Microsoft Alias: davidmu

@spottedmahn Thanks for your feedback! We will investigate and update as appropriate.

@spottedmahn I have assigned the issue to the author to investigate and update as appropriate.

@spottedmahn We received this link from the engineering team, hope it might help: https://social.msdn.microsoft.com/Forums/en-US/f7a26ab7-940e-40a2-8147-29447f84314c/azure-b2c-step-up-authentication?forum=WindowsAzureAD. We have opened an item in our documentation backlog to have this investigated and updated. We'll respond back to you on this issue when we've done the work. For now, we'll close this GitHub issue.
#please-close

cool, thanks 🙏 @davidmu1


Ignore from download calculation

Codecov Report

Merging #115 (1582aa1) into master (5619c83) will decrease coverage by 0.40%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
- Coverage   63.11%   62.70%   -0.41%
==========================================
  Files         120      121       +1
  Lines        3096     3116      +20
  Branches      719      723       +4
==========================================
  Hits         1954     1954
- Misses       1131     1151      +20
  Partials       11       11

Impacted Files
Coverage Δ

...src/rules/diff-entry-download-size-limits/index.ts
90.00% <ø> (ø)

...src/rules/diff-entry-download-time-limits/index.ts
91.48% <ø> (ø)

...pack/src/rules/entry-download-size-limits/index.ts
95.83% <ø> (ø)

...pack/src/rules/entry-download-time-limits/index.ts
95.83% <ø> (ø)

packages/webpack-ui/src/init/init-menu.ts
0.00% <0.00%> (ø)

packages/webpack-ui/src/init/prepare/index.ts
0.00% <0.00%> (ø)

packages/webpack-ui/src/init/settings.ts
0.00% <0.00%> (ø)

packages/webpack-ui/src/pages/default.ts
0.00% <ø> (ø)

...ckages/webpack-ui/src/pages/diff/queries/chunks.ts
0.00% <ø> (ø)

...ges/webpack-ui/src/pages/diff/queries/dashboard.ts
0.00% <ø> (ø)

... and 4 more

Continue to review full report at Codecov.

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


[function](date function) add new date function  'to_monday'

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[x] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[x] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

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

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 38.42 seconds
load time: 565 seconds
storage size:<PHONE_NUMBER>9 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221026234740_clickbench_pr_34475.html


Update faraday-http-cache dependency from 1.0 to 2.0

The version 2.0 of faraday-http-cache was released 3 years ago. Pinning Danger to the version ~> 1.0 of that gem makes it incompatible with other recent gems.

          2 Warnings

:warning:
External contributor has edited the Gemspec

:warning:
.gemspec modified

  Generated by :no_entry_sign: Danger

          2 Warnings

:warning:
External contributor has edited the Gemspec

:warning:
.gemspec modified

  Generated by :no_entry_sign: Danger

👍


cuda::grid_executor does not compute the correct global function pointer used in all cases

basic_grid_executor::global_function_pointer is parameterized on the type of Function used to instantiate the kernel inside of basic_grid_executor::bulk_async(Function, shape_type). Instead, it needs to be parameterized on the Function and an optional Tuple template parameter.
Likewise, grid_executor::max_shape should take the type of Args which will be passed to the Function.
With this interface it's going to be fairly difficult to ensure that the thing returned by global_function_pointer is the actual kernel that will be used to implement a bulk_async with the given types.

The basic_grid_executor::launch function should take a void * to make it clear it's not going to wrap the launched thing in anything else.


fill_value support

Fill values are now part of the cube metadata
We should not be using numpy default fill-values to save data.
Todo:

Review all the file formats

loading must set cube.fill_value where appropriate
saving must require one, if data is masked
UM files can possibly implement format-specific default values

merge and concatenate to deduce and propagate fill_value (as currently done by biggus)
other cube operations and calculations likewise need to preserve or destroy fill_value

statistics ; arithmetic ; regridding

Note:
many tests may need reworking
expect that some of the current behaviour is flawed

Closed by #2492


To implement Receive Asset & Set Default Fee Rate

[x] Receive Asset (Free Faucet): Implement functionality for users to receive assets from a free faucet, ensuring seamless transfer and balance updates.

[ ] Set Default (Fee Rate): Establish a default transaction fee rate, automatically applied unless altered by the user, balancing cost and speed.

Closing this issue as discussed with dev.
Already have an issue in sprint 29 for second option (Default Fee rate)


[Webhooks] Undocumented workflow_job event

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/actions/managing-workflow-runs/reviewing-deployments
What part(s) of the article would you like to see updated?
I would like to see documentation added for the workflow_job event.
Additional information
I have some example payloads located here in octokit/webhooks
The webhook seems to fire for every job in a workflow

@wolfy1339
Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:

It seems the docs were just pushed out today.
That would close this issue.
👍

Thanks for the update @wolfy1339!


From private repo

@msmbaldwin : Thanks for your contribution! The author, @dougeby, has been notified to review your proposed change.


apt packages for bionic no longer working

Installing the apt packages (in a Docker image running ubuntu 18.04, in my case) no longer works. It worked less than a week ago.
I've re-installed the apt-keys and re-added the repo, with
RUN apt-key adv --keyserver keys.gnupg.net --recv-key F6E65AC044F831AC80A06380C8B3A55A6F3EFCDE || apt-key adv --keyserver hkp://keyserver.ubuntu.com:80 --recv-key F6E65AC044F831AC80A06380C8B3A55A6F3EFCDE
RUN add-apt-repository -y "deb http://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo bionic main" -u

but it doesn't help.
The error on apt-get update or apt-get install librealsense2-dkms is:
E: Failed to fetch http://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo/dists/bionic/InRelease  403  Forbidden [IP: <IP_ADDRESS> 80]
E: The repository 'http://realsense-hw-public.s3.amazonaws.com/Debian/apt-repo bionic InRelease' is not signed.

Broken for me as well.

Thanks for the report,  the public server is indeed not accessible during the last  hour.
We're troubleshooting the issue and we'll update accordingly.

All server operations were resumed.
@jackjansen ,@danielgordon10 - pls check now.

Confirmed working again. Thanks.

Thanks again. closed as resolved.


Small changes in linking certificates produced by renew/rekey

populate related certificates in detail DTO with certificates representing certificates for which certificate is source
when certificate is issued as result of renew/rekey, event history record should state issue certificate
differentiate between renew and rekey event in event history

make source certificate UUID foreign key with on delete set null