feat(react): add tabs to view assets screen

Renamed ViewFunds to ViewAssets throughout the codebase for better clarity.
Updated file names and corresponding imports to reflect this change.
Modified the styling of the tabs on the TransactionsScreen.
Updated the grid layout and spacing in the ViewNFTs component.
Streamlined token viewing logic by creating reusable content components for tokens and NFTs in ViewAssets, ViewNFTs, and ViewTokens.

PR-Codex overview
This PR restyles the Connect Wallet UI page, renames View Funds to View Assets, and updates icons and components for a better user experience.
Detailed summary

Restyled the Connect Wallet UI page
Renamed View Funds to View Assets
Updated icons and components for improved user experience

The following files were skipped due to too many changes: packages/thirdweb/src/react/web/ui/ConnectWallet/screens/ViewNFTs.tsx

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

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#3845  👈
#3844
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @gregfromstl and the rest of your teammates on  Graphite

Merge activity

Jul 27, 3:30 PM EDT: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.


Added github action workflow, plus needed tests

I do not think that github-actions runner will be able to run this pipeline, due to resource limitation

I do not think that github-actions runner will be able to run this pipeline, due to resource limitation

I'll fully admit I forgot about that. Did learn about actions though!

It would be any way useful to have target test in Makefile, be able to run tests locally

It would be any way useful to have target test in Makefile, be able to run tests locally

I agree with that.

Okay, so miraculously the tests passed. It says black on white that the machines have 7gb RAM available ref.
I would still like to close this PR, considering the checks should technically not pass. Anyone against that?

It would be any way useful to have target test in Makefile, be able to run tests locally

~I agree with that.~ Considering terraform runs in attached state an ansible task checking the health of all services is redundant. The terraform-task would never exit without everything being healthy.

Are you sure about this? Can you confirm that we have all services passed?

Strange.. It sure reports two cores and two threads https://github.com/fredrikhgrelland/data-mesh/pull/91/checks?check_run_id=780740006#step:3:1120 on "mac os level" but the hardware is 4 cores: https://github.com/fredrikhgrelland/data-mesh/pull/91/checks?check_run_id=780740006#step:3:1009 and memory is 12GB ? https://github.com/fredrikhgrelland/data-mesh/pull/91/checks?check_run_id=780740006#step:3:1007

Pretty decent CPU also https://github.com/fredrikhgrelland/data-mesh/pull/91/checks?check_run_id=780740006#step:3:1069

This has to be the actual hardware: https://everymac.com/systems/apple/mac_pro/specs/mac-pro-six-core-3.5-xeon-e5-gray-black-cylinder-late-2013-specs.html

Plenty med disk også: https://github.com/fredrikhgrelland/data-mesh/pull/91/checks?check_run_id=780826831#step:4:51165

Say what now... nice detective work, haha.
I guess we might as well keep the tests for now then, and remove it later if it for some reason starts failing.

Are you sure about this? Can you confirm that we have all services passed?

That answer was based on what you said earlier, which was that the nomad_job resource would run until the service is healthy in nomad, and by extension, consul.
After a little digging: ref, if detach = false, the provider will keep running until the deployment is successful, or it fails and returns an error.


Add Seed() function to API?

Library is really great for benchmark purposes when one needs to generate lots of random data to feed to some procesing.
But lack of Seed() function makes it difficult to debug code.

I’m open to adding this. Would you be able to send a pull request to add this?

Seed may be added to RNG, but it cannot be added to global rand funcs, since they depend on non-deterministic sync.Pool.
The addition of Seed to RNG would make the API incomplete and uglier.


Add portforward support.

Add port forward support. The implementation is the same with the dockershim.
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

@mikebrow Addressed comments.

Minor comments, LGTM

@abhinandanpb Addressed comments. Thanks for reviewing

Just updated this PR to use the containerd client.


[Feature Request] Coin & Fungible asset decimal parsing and display library

🚀 Feature Request Description
This should clearly show for a balance to convert the full u64 number to with decimals.
Such as 10000000 Octas should be 0.1 APT

Added here https://github.com/aptos-labs/aptos-ts-sdk/pull/482/files#diff-d9b0865f2e8f841410cd4faf3e1e4a1fb7cb0b545f583f6adaad5841d7eaa58dR35-R60


Orderbook Streaming

AFAIK, not yet implemented
Ref: https://github.com/stellar/go/blob/50091de20dff08100374ba2adfaa18f5e813a42e/services/horizon/internal/docs/reference/endpoints/orderbook-details.md

This has been implemented in the new horizonclient: https://godoc.org/github.com/stellar/go/clients/horizonclient#Client.StreamOrderBooks


Add ultraman

Summary
Add ultraman(yukihirop/ultraman)

The test fails because Formula/ultraman.rb does not exist in the main branch

LGTM


End to end tracing with Spring Cloud Sleuth

Goal: As a user of Spring Streaming Apps I want end to end tracing using Sleuth.
Acceptance Criteria

[ ] Have a sleuth injector (like the messaging one)
[ ] When a stream starts ensure a new span is created and when it's completed stop the span
[ ] Create a new trace id when no trace id is found in the original message, otherwise carry the found trace id through the system
[ ] Allow traces/spans to be named. By default use the name of the stream/app
[ ] Allow each app to add additional meta-data to a span

This issue was moved to spring-cloud-stream-app-starters/app-starters-release#38


Update for madMELA change

This simultaneously updates the manual to reflect changes in madMELA as well as changes the actual JHUGen area so that someone can compile said area.
It is in complement with https://github.com/JHUGen/JHUGenMELA/pull/60

I have to upload a new version of the library. Only problem is it's compiled through Python3 and some of the older versions of gcc don't like that 😢  - so give me a day or so. Compilation also takes a while each time, so bear with me


Unable to access documentation api

I had done setup to run articulate on local server but for me (http://localhost:7500/documentation)giving blank page as u can see in the below image

also it will be better if anyonecan guideme with documentation and tutorial of articulate for better understanding of articulate

My guess is ES failed too start. What is the output of docker-compose ps?
Please look at the 4th warning here https://samtecspg.github.io/articulate/#/getting-started/basic-installation.md if I am right you need to:
sudo chown -R 1000:1000 local-storage/elasticsearch

There's some documentation here and on our blog.

Is the main UI working?

hey @wrathagom  main ui is working fine see the below screenshot  image
and have run this command (sudo chown -R 1000:1000 local-storage/elasticsearch)even i run (http://localhost:7500/documentation) in postman to check

What command are you using to start Articulate?

i am using sudo docker-composer up as mention in repo


Add GitHub stars subscription gifAlert on new stars

Closes #3 (just need to change the repoOwner/repoName).
Also moves javascript into its own index.js file for easier formatting.

Tested with the lense of any user can stumble into this repo and fork this repo and have a bot. With that in mind, need an OneGraph APP_ID  and ENV var to pass that too since not every user will be able to use the same APP_ID.
My thought

instead of merging this live to gh-pages, it should go to a new branch, possibly called even keep it on main.
Add a Deploy to Netlify button that points to the MAIN branch and haves the user create an OneGraph account to provide their APP_ID. This will provide the best experience and the only confusion would be what is OneGraph? Which we can remedy by adding OneGraph context to the README.
Finally, the last thing is the need to include a login to connect to the bot. This is a hard stop because OBS presents a browser source plugin that is view only with no ability to interact with the authentication modal.

I have moved this to https://github.com/open-sauced/slaybot.
The limitation of having the login button and a running server. I will start using this in the next live stream. Excited to work on some improved CSS this weekend.


[FEATURE]: TiDB Support

Describe what you want
I want to use drizzle-orm to connect to TiDB Serverless to develop my application, but I found out it's not on the support list yet.

From the docs:

TiDB is compatible with the MySQL 5.7 protocol, common features of MySQL, and the MySQL ecosystem.

Did you try using the mysql2 driver to connect?

Hi, I think the mysql2 driver should be used to connect to TiDB Serverless. But it doesn't work in the edge environment that does not support TCP connection well.
Refer to this official do https://orm.drizzle.team/docs/installation-and-db-connection/mysql/planetscale, is it possible drizzle-orm support TiDB Serverless driver over HTTP?

The TiDB Serverless driver is not available as its' backend is still in our dev environment, it should be available in the next several weeks

@dankochetov PTAL, Can we support TiDB Serverless with drizzle-orm by the TiDB Serverless driver.

There is now a Mysql proxy available too. That should work in serverless

Driver support released in 0.31.2: https://orm.drizzle.team/docs/get-started-mysql#tidb-serverless

How about mysql 5.7 @dankochetov ? Drizzle support it?


[bug]: Toast - does not close when clicking/keyboard-enter the closing-X button on the right

Describe the bug
does not close when clicking/keyboard-enter the closing-X button on the right.
And also add type="button" to the example code on storybook-documentation, otherwise forms are submitted on clicking the x button.
Screenshots

@swisspost/design-system add type=button in the docs.

Similar to #4280, hiding the element on close click should be implemented for the moment. We'll use this ticket to update the documentation.


UAT Bug | Approve User Registration

Module: User Registration
User: Nyay Mitra
Issue: API Error when approving USer Registration request

resolved


Close on outgoing sequence number overflow

Implementation must either abandon an association or rehandshake. RFC 6347 Section 4.1.
Close connection for now.

rebased

@at-wat sorry this PR has been sitting so long!
IMO if you are left waiting more then 3 days, I am fine with you merging. I would rather see progress forward then you get block on other people :(


Exit button is oddly placed in share pane

See the "X" in the top right hand corner.

@TimonVS @daniellouwekooijmans looks like a styling conflict (http://cl.ly/1W2a3F0X223j)


Fix system call related date builtin functions

Related issues: #213, #691

Fixed 'ecma_date_local_tza' and 'ecma_date_daylight_saving_ta' date builtin helper functions
Added syscall of gettimeofday function to get the current system time and timezone.
Fixed related regression test files.

JerryScript-DCO-1.0-Signed-off-by: László Langó<EMAIL_ADDRESS>

@zherczeg, I've updated the PR.

@zherczeg, I've updated the PR.

@akiss77, @zherczeg, I've updated the PR.

LGTM

It's much better practice to have the #ifdef __cplusplus at the very beginning of the headers, not only where function declarations start. In jerry.h I've even spotted that those guards are used twice. So, I'd really prefer to have the order

copyright
#ifndef HEADERNAME_H #define HEADERNAME_H
includes
#ifdef _cplusplus #define extern "C" {
all header content (including typedes and all, not only function declarations)
#ifdef _cplusplus }
#endif // HEADERNAME_H

Something like that. Opinion?

@akiss77, the suggested style is convenient for me. The biggest problem here is the vera++ checker. We have to modify the rule and currently not sure how.
@zherczeg, @akiss77, do you mind if I raise an issue for this and do it in a separate PR?

@LaszloLango OK with me

A reverted the last commit, so this PR only contains the code related to the Date builtin, because the change of the EXTERN_C macro is not trivial at the moment. I raised an issue for that: #900 and pushed a new branch to my fork: https://github.com/LaszloLango/jerryscript/tree/refactor-extern-c-define
@zherczeg, @akiss77 please check this PR again. Is this ready for merge?

LGTM (FWIW)


bootstrapping cluster

use the term bootstrapping
deploy app as 3rd step

we are going to talk about "bootstrapping" during the hack
this big of a change to the hack this late is too risky, so we won't change the actual hack


SAA-840: Remove education levels from candidates response

Codecov Report
Patch coverage has no change and project coverage change: +0.07 :tada:

Comparison is base (368b175) 84.70% compared to head (a4fdc43) 84.78%.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #399      +/-   ##
============================================
+ Coverage     84.70%   84.78%   +0.07%
  Complexity     2029     2029
============================================
  Files           278      278
  Lines          7754     7747       -7
  Branches        437      437
============================================
  Hits           6568     6568
+ Misses          940      933       -7
  Partials        246      246

Impacted Files
Coverage Δ

...smanagementapi/model/response/ActivityCandidate.kt
0.00% <ø> (ø)

...tivitiesmanagementapi/service/CandidatesService.kt
48.11% <ø> (+2.16%)
:arrow_up:

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


Development Splitting cypress modeling spec to improve stability

Motivation and Context
 Right now the Modeling exercise e2e tests are dependent on another which leads to flakiness. It is also bad practice and against the non functional requirements we set for our selves. Therefore in this PR i make the tests independant and split them into multiple more readable spec files.

Description
 Im splitting ModelingExercise.spec.ts into
ModelingExerciseManagement.spec.ts
ModelingExerciseParticipation.spec.ts
ModelingExerciseAssessment.spec.ts
Steps for Testing

We have a guide for testing the cypress PRs HERE
Review Progress

Code Review

[ ] Review 1
[ ] Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

The tests now <a href="https://bamboo.ase.in.tum.de/chain/result/viewChainResult.action?planKey=ARTEMIS-AETBBT&buildNumber=453" all pass on the test plan.


Add AIYO Institute of Technology

@easygautam First stage of verification passed, official domain is aiyoit.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.


[Form Validation] Add a float/double validation rule

Hi,
Please shelf this until you are done with 2.0 release :)
I know there's integer validator available, but in my case almost all fields in the application are price/money fields and those sometimes need a floating point.  Maybe just add a number validator, which would accept all types of number.
Of course, meanwhile I can use regExp but what do you think?

I was trying to find the way to validate floats today and found your issue, I just created a PR so I hope Semantic team could migrate it soon :)

Thanks @TonnyORG for the reponse. I've actually created a custom rule in $.fn.form.settings.rules instead :) I think it's neater this way
$.fn.form.settings.rules.number = function(value) {
    return value.match(/^([0-9]+)|([0-9]*(\.[0-9]+))?$/);
};

and now I can do:
$(".ui.form").form({
    fields: {
        name: {
            identifier: 'name',
            rules: [
                 {type: 'number', prompt: 'Enter a valid number'}
            ]
        }
    }
});

Cheers!

Number and decimal was already added


getting error while running in google co-lab

i am getting error while running
model = build_model(input_shape=input_shape, output_channels=3)
"module 'tensorflow._api.v2.config' has no attribute 'experimental_list_devices'"
please help me out.

Please have a look at #40. This model was designed in TensorFlow 1.x API. It is not compatible with TensorFlow 2.x. Please install the package versions mentioned in #40 to make use of the model.


Version 2.0.0 breaks all devices, unable to fix

Installed the new version from HACS today. All devices immediately unavailable. Deleted and readded, appears success, and overheat+signal sensors are working, but all others unknown

HA logs show polling with success
2023-07-06 18:36:44.278 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.028 seconds (success: True)
2023-07-06 18:36:44.301 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.038 seconds (success: True)
2023-07-06 18:36:44.319 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.031 seconds (success: True)
2023-07-06 18:36:44.486 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.025 seconds (success: True)
2023-07-06 18:36:44.529 DEBUG (MainThread) [custom_components.tapo.coordinators] Finished fetching tapo data in 0.035 seconds (success: True)

What other config do you need? This all work previously.
I attempted to downgrade integration, this didn't help.  Also tried applying latest firmware updates.  Didn't help

tapo-logs.txt

Device was working before upgrade

Adding some debug to the code, it looks like PlugTapoCoordinator() is setup with self.has_power_monitor = True and nothing is calling enable_power_monitor() to turn it on.
If I hack the code to set self.has_power_monitor = True in the init then it all starts working again
class PlugTapoCoordinator(TapoCoordinator[PlugDeviceState]):
    def __init__(
        self,
        hass: HomeAssistant,
        device: PlugDevice,
        polling_interval: timedelta,
    ):
        super().__init__(hass, device, polling_interval)
        self.has_power_monitor = True # <==== here
        self.power_info = None
        self.energy_info = None

Yes, Confirmed that's working. Thanx

Aggiungendo un po 'di debug al codice, sembra che PlugTapoCoordinator() sia configurato self.has_power_monitor = Falsee nulla stia chiamando enable_power_monitor() per accenderlo.
Se hackero il codice da impostare self.has_power_monitor = Truenell'init, tutto ricomincia a funzionare
class PlugTapoCoordinator(TapoCoordinator[PlugDeviceState]):
    def __init__(
        self,
        hass: HomeAssistant,
        device: PlugDevice,
        polling_interval: timedelta,
    ):
        super().__init__(hass, device, polling_interval)
        self.has_power_monitor = True # <==== here
        self.power_info = None
        self.energy_info = None

I confirm. Thank you!

Would someone kindly explain how to get to this bit of code to modify. It's not configuration.yaml

Would someone kindly explain how to get to this bit of code to modify. It's not configuration.yaml

In "custom_components" you have a "tapo" folder. There you have a file called "coordinators.py". This fix is in this file.
And this worked.

Excellent - working fine now. Thanks very much indeed.

had the same issue, the fix above worked for me as well. thanks!

Thanks for hints, have you right.


crashed after server move

displaying error Parse error: syntax error, unexpected '<' in /home2/conc#####/public_html/convert.php(40) : eval()'d code on line 1
was  fine before move but i can not see the error

This file has been updated to Googles new system and URLS
You should bare in mind its just a sample file that grabs the Amount, Currency From and Currency TO
Side note: eval()'d code ? would not have been part of this original file


Video resize issues

Hi,
I have an issue with the video resize again (I opened the issue #97 in October 2014). I not sure if this is linked or not, this is why a created a new issue.
As you can see on the screenshot below, the first player failed to resize itself. This appends when I change the stream url (when loading the next stream from a playlist).

This appends rarely and randomly. I didn't find a way to reproduce the issue every time.
This problem appears since version 0.5.0, so I think it may be linked to the commit 8e9dea5bed8ae1b8fbdfabb915e4a6ba4ec82ff6 or 8e8352a49320d9024538fcb5ca17baefb4e7121b, but I don't know how...
When there is a gap in the the playlist (even a very small one), I stop and destroy the player and create it again when there will be a stream again. I do this for performance improvement since we can have up to 16 players at the same time. This is why I think it's more likely to be linked to 8e9dea5bed8ae1b8fbdfabb915e4a6ba4ec82ff6 and this had nothing to do with the fullscreen.
Any idea on this one ? I will continue to investigate but I have no clue yet.
Cheers
Gaétan

Have you gotten any further in investigating this? I have not been able to reproduce this or anything similar.
Does it occur in any version before 0.5.0 ?

Sorry I didn't do any further investigations. I'm working on another project in my company. I will be back in May or June and then I will have time for this.

I was thinking about something. When I play a record of a camera at a given time, I have to play then seek. This process id done really quickly (the seek is done as soon as the streamStarted event is received).
Is possible that the video size is in the meta information at the beginning of the stream but I invoke seek too fast and flash skip this information and continue playing without invoking onMeta (and thus the onResize)?
I'm not familiar with stream protocol, maybe someone can give me a int on this.

Hi,
I am working on the project with Gaetan. We still have this issue.
The last comment of Gaetan is maybe the cause of the issue. Do you have maybe more informations?

Hmm. Did you confirm that you really don't get onMeta? If that is the case, then this is probably the reason the video doesn't get resized.
I'm not sure why you wouldn't get the meta data though. That tag is put before any video data in the generated FLV, so it should arrive before streamStarted. This must be due to some internal mechanic in the netstream class.

Yes I can confirm now that I have not the onMeta event fired.
the first image is event logs when bug is happened

when it works...

I read the code of NetStreamClient.as again, and I see on line 123-127 that you use NetStream.Buffer.Full and not NetStream.Play.Start to fire the streamStarted event. This could be the problem.
In the documentation, they say that NetStream.Buffer.Full indicate that "the stream begins playing" but not that the stream is already started.

If you change to Play.Start does it work better? It really seems like this could be the issue altough the documentation is not quite clear on exactly when the events are emitted.


Add to Newsblur link don’t work if you’re not already logged in (better landing pages)

https://www.newsblur.com/?url=http://blog.newsblur.com/rss
If you don’t have an account, the position of the dialog is off/mostly out of screen. This has been broken for years … .
The dialog isn’t useful as it only shows an error about needing an account. Users should be prompted to login and then prompt them to add the feed again. Or if you register a new account, it should subscribe you to the referring feed like the promotional landing pages Feedly uses.
https://feedly.com/i/welcome/feed/http://blog.newsblur.com/rss

seems this to be an issue. Is anyone working on this?

Thanks for writing this up! I'm working on fixing all of these issues and this has now been fixed. If you're not logged in, you will now be presented with the login form, and then once logged in the feed url is preserved and easily added.

@samuelclay could you add a "Log in or sign up to subscribe to $feed_url" message above the login form? It kind of just drops you on the page with no explanation. Such a message would help explain what's going on and potentially land you some extra sign up from new users who stumbled upon a Subscribe link on an external website.

I think that's part of a broader redesign of the static pages, which I'm hoping to get to one day but is not higher priority than much of the other web issues. I'm going to table this for now, partially because designing that isn't straight-forward.


Write wiki page to go over testing

As a developer, I would like to have better documentation for testing so that we can have a consistent and robust test suite.

Wiki page finished here: https://github.com/maxrchung/S2VX/wiki/Testing

+1

👍🚀


Missing lumi information for RAW input files causes WorkQueueManager to fail LQ -> WMBS work acquisition

Impact of the bug
WMAgent (maybe only vocms0282)
Describe the bug
Agent vocms0282 has 3 workflows that are stuck in the agent because their LQE cannot be acquired and WMBS cannot be poppulated. Here is the error in the WorkQueueManager logs [1]
Looking at the first workflow and the block it's acquiring:
pdmvserv_Run2022A_EGamma_10Dec2022_221211_223528_7434, ['/EGamma/Run2022A-v1/RAW#b35ed21e-c1eb-44b7-8233-73146c5531d3']:
I don't see anything wrong in DBS and all 3 files are marked as valid:
https://cmsweb.cern.ch/dbs/prod/global/DBSReader/files?block_name=/EGamma/Run2022A-v1/RAW%23b35ed21e-c1eb-44b7-8233-73146c5531d3&detail=true
checking Rucio rules and replicas [2], I fail to see any problem either. The file 57392a41-7588-478c-8907-3d3fd262a05b.root can be listed in both DBS and Rucio.
How to reproduce it
Needs debugging
Expected behavior
Still unclear why it is even happening. But I would say that, if a block has only invalid files, that we should mark that GQE/LQE as failed and get it out of the way.
Additional context and error message
[1]
2023-02-07 18:32:57,111:140204633413376:INFO:WorkQueueBackend:Getting up to 200 available work from http://vocms0282.cern.ch:5984
2023-02-07 18:32:57,111:140204633413376:INFO:WorkQueueBackend:  for team name: None
...
2023-02-07 18:32:57,111:140204633413376:INFO:WorkQueueBackend:  with limit docs: 1000, and skip first 0 docs
2023-02-07 18:32:57,130:140204633413376:INFO:WorkQueueBackend:Retrieved 3 elements from workRestrictions list for: http://vocms0282.cern.ch:5984
2023-02-07 18:32:57,130:140204633413376:INFO:WorkQueueBackend:Accepting workflow: pdmvserv_Run2022A_EGamma_10Dec2022_221211_223528_7434, with prio: 110000, element id: 6bff81253c0a2c2c786ed94be272e2b0, for site: T2_DE_DESY
2023-02-07 18:32:57,130:140204633413376:INFO:WorkQueueBackend:Accepting workflow: pdmvserv_Run2022A_DisplacedJet_10Dec2022_221211_223342_4384, with prio: 110000, element id: bca5a7fc80773f93d8a899e667f7cf03, for site: T2_US_Nebraska
2023-02-07 18:32:57,130:140204633413376:INFO:WorkQueueBackend:Accepting workflow: pdmvserv_Run2022A_MuonEG_10Dec2022_221211_223701_5232, with prio: 110000, element id: ca33b8b8647772b3670bec89372b3000, for site: T2_US_UCSD
2023-02-07 18:32:57,130:140204633413376:INFO:WorkQueueBackend:  with limit docs: 1000, and skip first 1000 docs
2023-02-07 18:32:57,144:140204633413376:INFO:WorkQueueBackend:All the workqueue elements have been exhausted for: http://vocms0282.cern.ch:5984
2023-02-07 18:32:57,144:140204633413376:INFO:WorkQueueBackend:And 3 elements passed location and siteJobCounts restrictions for: http://vocms0282.cern.ch:5984
2023-02-07 18:32:57,144:140204633413376:INFO:WorkQueue:Got 3 elements matching the constraints
2023-02-07 18:32:57,340:140204633413376:ERROR:WorkQueue:pdmvserv_Run2022A_EGamma_10Dec2022_221211_223528_7434, ['/EGamma/Run2022A-v1/RAW#b35ed21e-c1eb-44b7-8233-73146c5531d3']:
failed to retrieve data from DBS/Rucio in LQ:
'/store/data/Run2022A/EGamma/RAW/v1/000/352/568/00000/57392a41-7588-478c-8907-3d3fd262a05b.root'
2023-02-07 18:32:57,521:140204633413376:ERROR:WorkQueue:pdmvserv_Run2022A_DisplacedJet_10Dec2022_221211_223342_4384, ['/DisplacedJet/Run2022A-v1/RAW#adf1cfa0-d923-438f-bc74-33662dc9b928']:
failed to retrieve data from DBS/Rucio in LQ:
'/store/data/Run2022A/DisplacedJet/RAW/v1/000/352/568/00000/fad54dbf-2e2b-4028-b6f3-52fce66a6c3c.root'
2023-02-07 18:32:57,776:140204633413376:ERROR:WorkQueue:pdmvserv_Run2022A_MuonEG_10Dec2022_221211_223701_5232, ['/MuonEG/Run2022A-v1/RAW#456381d1-049d-49c0-b1a5-5cf932a083d7']:
failed to retrieve data from DBS/Rucio in LQ:
'/store/data/Run2022A/MuonEG/RAW/v1/000/352/568/00000/be9d3332-784a-405b-8316-fdf331b6788f.root'
2023-02-07 18:32:57,832:140204633413376:INFO:WorkQueue:Injected 0 out of 3 units into WMBS

[2]
amaltaro@lxplus738:~ $ rucio list-rules "cms:/EGamma/Run2022A-v1/RAW#b35ed21e-c1eb-44b7-8233-73146c5531d3"
ID                                ACCOUNT            SCOPE:NAME                                                        STATE[OK/REPL/STUCK]    RSE_EXPRESSION                                                                                COPIES    EXPIRES (UTC)    CREATED (UTC)
--------------------------------  -----------------  ----------------------------------------------------------------  ----------------------  ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------  --------  ---------------  -------------------
2faaca2864f04d7c8ac77f3fc4b76ca7  wmcore_transferor  cms:/EGamma/Run2022A-v1/RAW#b35ed21e-c1eb-44b7-8233-73146c5531d3  OK[3/0/0]               T2_IT_Bari|T1_RU_JINR_Disk|T2_US_Nebraska|T2_HU_Budapest|T2_FR_GRIF_LLR|T2_DE_DESY|T2_BE_UCL|T2_BE_IIHE|T2_RU_JINR|T2_US_UCSD|T2_BR_UERJ|T2_IT_Legnaro|T2_US_Vanderbilt|T2_FR_GRIF_IRFU|T2_IT_Rome|T2_ES_CIEMAT|T1_UK_RAL_Disk|T2_UK_London_Brunel|T2_IT_Pisa|T2_US_Purdue|T1_DE_KIT_Disk|T2_CH_CERN|T1_IT_CNAF_Disk|T2_US_Wisconsin  1                          2023-01-19 09:28:05
cf1de0a802c947daa691b7bb6cda8cd3  wmcore_transferor  cms:/EGamma/Run2022A-v1/RAW#b35ed21e-c1eb-44b7-8233-73146c5531d3  OK[3/0/0]               T2_US_Caltech|T2_US_UCSD|T1_UK_RAL_Disk|T1_DE_KIT_Disk|T2_DE_DESY|T2_CH_CERN|T2_US_Florida|T2_TW_NCHC|T2_ES_IFCA|T2_US_Nebraska|T2_US_Wisconsin|T2_UK_London_Brunel                                                                                1                          2022-12-11 21:59:04
amaltaro@lxplus738:~ $

and
amaltaro@lxplus738:~ $ rucio list-files "cms:/EGamma/Run2022A-v1/RAW#b35ed21e-c1eb-44b7-8233-73146c5531d3"
+----------------------------------------------------------------------------------------------------+--------+-------------+------------+----------+
| SCOPE:NAME                                                                                | GUID   | ADLER32     | FILESIZE   | EVENTS   |
|----------------------------------------------------------------------------------------------------+--------+-------------+------------+----------|
| cms:/store/data/Run2022A/EGamma/RAW/v1/000/352/568/00000/57392a41-7588-478c-8907-3d3fd262a05b.root | (None) | ad:0b2d15c6 | 4.026 MB   |          |
| cms:/store/data/Run2022A/EGamma/RAW/v1/000/352/572/00000/449448fb-7d10-44b3-89f1-09158a1f54dd.root | (None) | ad:18e8b7e6 | 1.796 MB   |          |
| cms:/store/data/Run2022A/EGamma/RAW/v1/000/352/582/00000/e2236a5a-373e-43df-9da2-a757eccd3018.root | (None) | ad:d507e55e | 1.517 MB   |          |
+----------------------------------------------------------------------------------------------------+--------+-------------+------------+----------+

It looks like the problem lives in the DBS data discovery logic, which is performing at least these 2 actions:
a) using the listFileArray DBS client API, retrieve a list of valid files in a given block
b) then using the listFileLumis DBS client API, retrieves a list of tuples (run/lumi/eventCount) for a given block.
The query b) above can be mapped to the filelumis REST API, as follows:
https://cmsweb.cern.ch/dbs/prod/global/DBSReader/filelumis?block_name=/EGamma/Run2022A-v1/RAW%23b35ed21e-c1eb-44b7-8233-73146c5531d3&validFileOnly=1
By inspecting the URL above, one can confirm that only one file contains lumi information in DBS. Given that every single file in DBS is supposed to have AT LEAST 1 lumi registered in the database, this causes DBS3Reader to hit an exception that gets propagated all the way to WorkQueueManager.
The creation date of the files above match the dates that we were migrating DBSWriter from the Python to the Golang implementation:
cmst1@vocms0282:/data/srv/wmagent/current $ date -d @1653926004
Mon May 30 17:53:24 CEST 2022

@vkuznet Valentin, can you please confirm the debugging above. If I am not wrong, we had a couple of issues with that transition, like:
i) not properly splitting large blocks (> 500 lumis) in their relevant slices
ii) not properly rolling back transactions when one of the concurrent tasks failed in DBS Server.
Do you see any other possibility?
I haven't checked the other 2 workflows yet, but very likely they are hitting the same scenario.
Given that this is RAW data, there is no way that we can recover the run/lumi information and I believe the only option will be to invalidate the relevant files in both DBS and Rucio.

For the 3 files reported in the initial issue description, I can confirm that none of them have any lumis in DBS, as follows:
https://cmsweb.cern.ch/dbs/prod/global/DBSReader/filelumis?logical_file_name=/store/data/Run2022A/EGamma/RAW/v1/000/352/568/00000/57392a41-7588-478c-8907-3d3fd262a05b.root
and
https://cmsweb.cern.ch/dbs/prod/global/DBSReader/filelumis?logical_file_name=/store/data/Run2022A/DisplacedJet/RAW/v1/000/352/568/00000/fad54dbf-2e2b-4028-b6f3-52fce66a6c3c.root
and
https://cmsweb.cern.ch/dbs/prod/global/DBSReader/filelumis?logical_file_name=/store/data/Run2022A/MuonEG/RAW/v1/000/352/568/00000/be9d3332-784a-405b-8316-fdf331b6788f.root

Same issue has also affected workqueue elements sitting in the submit8 queue:
2023-02-07 21:33:45,603:140466668803840:INFO:WorkQueue:Got 5 elements matching the constraints
...
2023-02-07 21:33:49,027:140466668803840:ERROR:WorkQueue:pdmvserv_Run2022A_SingleMuon_10Dec2022_221215_174719_5888, ['/SingleMuon/Run2022A-v1/RAW#03e2d560-ef7f-4071-8f49-ce2adb45a070']:
failed to retrieve data from DBS/Rucio in LQ:
'/store/data/Run2022A/SingleMuon/RAW/v1/000/352/568/00000/06aebe9c-69d0-4f30-ac09-2267c34d0923.root'
2023-02-07 21:33:51,274:140466668803840:ERROR:WorkQueue:pdmvserv_Run2022A_BTagMu_10Dec2022_221211_223247_4770, ['/BTagMu/Run2022A-v1/RAW#30115150-de1a-4f1f-9794-79bead2c185d']:
failed to retrieve data from DBS/Rucio in LQ:
'/store/data/Run2022A/BTagMu/RAW/v1/000/352/568/00000/8cfb9284-ff5a-49ca-88d0-921f0673c27a.root'
2023-02-07 21:33:53,510:140466668803840:ERROR:WorkQueue:pdmvserv_Run2022A_JetHT_10Dec2022_221215_133251_28, ['/JetHT/Run2022A-v1/RAW#74fbecbf-0c06-46d4-ab09-781ef3f76047']:
failed to retrieve data from DBS/Rucio in LQ:
'/store/data/Run2022A/JetHT/RAW/v1/000/352/568/00000/faa1e582-a333-481c-a0ed-8b091710657d.root'
2023-02-07 21:33:55,722:140466668803840:ERROR:WorkQueue:pdmvserv_Run2022A_MET_10Dec2022_221215_174720_1948, ['/MET/Run2022A-v1/RAW#5f6d933d-63ec-426c-8fd1-30fe22816125']:
failed to retrieve data from DBS/Rucio in LQ:
'/store/data/Run2022A/MET/RAW/v1/000/352/568/00000/80aaece3-b944-420a-b1b0-79dfa093deb8.root'

which I verified with the filelumis API and indeed there is no output for them.

@vkuznet Valentin, do you have a way to find files inserted into DBS in the last year which are actually missing lumi information? I had the impression we had already done this exercise in the last year.

Alan, I checked directly on DBS DB side and confirm the following:
SQL> select file_id from cms_dbs3_prod_global_owner.files where logical_file_name='/store/data/Run2022A/EGamma/RAW/v1/000/352/568/00000/57392a41-7588-478c-8907-3d3fd262a05b.root';

   FILE_ID
----------
 885730637

SQL> select * from cms_dbs3_prod_global_owner.file_lumis where file_id=885730637;

no rows selected

SQL> select file_id from cms_dbs3_prod_global_owner.files where logical_file_name='/store/data/Run2022A/SingleMuon/RAW/v1/000/352/568/00000/06aebe9c-69d0-4f30-ac09-2267c34d0923.root'
  2  ;

   FILE_ID
----------
 885729557

SQL> select * from cms_dbs3_prod_global_owner.file_lumis where file_id=885729557;

no rows selected
SQL> select file_id from cms_dbs3_prod_global_owner.files where logical_file_name='/store/data/Run2022A/BTagMu/RAW/v1/000/352/568/00000/8cfb9284-ff5a-49ca-88d0-921f0673c27a.root'
  2  ;

   FILE_ID
----------
 885730997

SQL> select * from cms_dbs3_prod_global_owner.file_lumis where file_id=885730997;

no rows selected

So, for listed LFNs we do not have lumis in DBS which is what you found via filelumis API.
Why does it happen it is hard to say. Since you provided timestamp, indeed it correlates to migration to Go server. At that time we had couple of hick-ups, I even did manual fix in DB, and indeed it was related to wrong lumis splitting and to concurrency error.
Regarding finding files with missing lumis I think it can be done via proper crafted SQL but I need time for that and we should define time range for this search since FILES and FILE_LUMIS table are the most populated ones. So, if we want to run such kind of queries I need to know time-range.

Thank you for the confirmation, Valentin.
Once you come up with a proper SQL, could you please query for the following time range: from May 25 to June 5.
No science behind these numbers, just grabbing a close range to the date of the file above and the DBS migration.

So, for this time range I found 312 files with no lumis, you can see query and files over here: lxplus:/afs/cern.ch/user/v/valya/public/DBS/files-no-lumis.txt
head /afs/cern.ch/user/v/valya/public/DBS/files-no-lumis.txt
SQL> select f.logical_file_name from cms_dbs3_prod_global_owner.files f LEFT JOIN cms_dbs3_prod_global_owner.file_lumis fl on f.file_id=fl.file_id where f.creation_date>1653430000 and f.creation_date<1654400000 and fl.lumi_section_num is null;
/store/backfill/1/data/Tier0_REPLAY_2022/ZeroBias2/ALCARECO/SiStripCalZeroBias-PromptReco-v34/000/346/512/00000/b6f7505d-a8ec-4a79-beb8-94ffd175dfa4.root
/store/backfill/1/data/Tier0_REPLAY_2022/ZeroBias2/ALCARECO/TkAlMinBias-PromptReco-v34/000/346/512/00000/d5ba9221-ddfc-4ceb-b395-4e042f8e6344.root
/store/backfill/1/data/Tier0_REPLAY_2022/ZeroBias2/USER/LogErrorMonitor-PromptReco-v34/000/346/512/00000/0a836f4c-775b-47a2-b238-072867149d21.root
/store/backfill/1/data/Tier0_REPLAY_2022/ZeroBias6/ALCARECO/SiStripCalMinBias-PromptReco-v34/000/346/512/00000/8c5c6157-08df-45ce-aa64-539fe9682894.root
/store/backfill/1/data/Tier0_REPLAY_2022/ZeroBias18/ALCARECO/SiStripCalZeroBias-PromptReco-v34/000/346/512/00000/3e713282-3bea-4e91-aba8-68e17dfaec36.root
/store/backfill/1/data/Tier0_REPLAY_2022/ZeroBias18/ALCARECO/TkAlMinBias-PromptReco-v34/000/346/512/00000/e5522160-3758-4856-8988-338de53c9fa3.root
/store/backfill/1/data/Tier0_REPLAY_2022/ZeroBias4/ALCARECO/TkAlMinBias-PromptReco-v34/000/346/512/00000/4b6715fa-61c5-4319-a220-e8566e27bf1f.root
/store/backfill/1/data/Tier0_REPLAY_2022/ZeroBias4/ALCARECO/SiStripCalZeroBias-PromptReco-v34/000/346/512/00000/dbe02b7d-3eae-4875-96b3-a18bed8e1e5b.root
/store/backfill/1/data/Tier0_REPLAY_2022/ZeroBias16/ALCARECO/SiStripCalZeroBias-PromptReco-v34/000/346/512/00000/3ac2302b-4081-4ce9-8043-8767b578b557.root
...

I used the following time range in my query:
date -d @1653430000
Wed May 25 00:06:40 CEST 2022

date -d @1654400000
Sun Jun  5 05:33:20 CEST 2022

@amaltaro Is it confirmed we really can't recover the run/lumis from the input RAW files? Otherwise, if we can get this information from the input files, we can inject that into DBS, right? If we really can't recover this information then P&R was okay with invalidating the files (but recovering would be the first choice since this is data and not MC)

And, Alan asked me to perform two additional queries:

I run query to check if we have files with missing lumis for Jul1-Jul11th and I found nothing
then, he asked me to check wider range which I cover with Ma13-Jun16, you can find the file over here lxplus:/afs/cern.ch/user/v/valya/public/DBS/files-may13-jun16.txt and it has 1K files, with 164 RAW lfns.

@vkuznet , could you please filter your results and only shows the ones that does not match /*/backfill/* ?

@jhonatanamado , how about cat /afs/cern.ch/user/v/valya/public/DBS/files-may13-jun16.txt  | grep -v 'backfill' ? Doing this on ORACLE side is tedious process due to very large data volume in FILES and FILE_LUMIS tables.

@jhonatanamado I counted 52 Run2022A RAW files, retrieved with this command:
grep -v backfill /afs/cern.ch/user/v/valya/public/DBS/files-may13-jun16.txt | grep '/RAW/'

@vkuznet could you please try to pin point when we started having these problems and when it was resolved, from the database perspective? It could be that our range still does not cover all the cases.

Based on range I choose May13-Jun16, it is started on Tue May 17 16:10:49 CEST 2022 and ended on Tue May 31 15:23:30 CEST 2022.

@vkuznet thanks to German, we have a list of file name / run number / lumi number / event count in this text file:
/afs/cern.ch/user/c/cmst0/public/DBS/RAW_files_info.txt
would you be able to inject this information into DBS Server?

since it is delicate procedure I would not rush to do it today as I need to finish other stuff, and then I'll be away till next week Tue when I can do it properly. I hope it can wait till next week, ok?

Honestly, I would like to shut down those draining agents before the week is over, but I can definitely wait another week.
It shouldn't be a problem for the P&R team as well, but I let them confirm.
In the end, I totally agree with you. We need to be extremely careful with such operations in DBS.

Based on the initial list provided by Valentin in https://github.com/dmwm/WMCore/issues/11477#issuecomment-1422828886, with 999 files missing lumi information in DBS, I applied some filters and came up with 4 lists.
First list contains RAW data (52 files) which is going to be recovered in the beginning of the next week (see comments above):
https://amaltaro.web.cern.ch/amaltaro/forWMCore/Issue_11477/raw_data.txt
Second list contains PromptReco data in a variety of data tiers (with a total of 74 files). We are discussing with the T0 whether these need to be recovered or invalidated:
https://amaltaro.web.cern.ch/amaltaro/forWMCore/Issue_11477/prompt_reco.txt
Third list contains MC data (with 2 files), which I'd suggest to get them invalidated and removed in Rucio:
https://amaltaro.web.cern.ch/amaltaro/forWMCore/Issue_11477/mc.txt
Lastly, the fourth list contains backfill/REPLAY data (871 files), which I'd suggest to get them invalidated and removed in Rucio:
https://amaltaro.web.cern.ch/amaltaro/forWMCore/Issue_11477/backfill.txt
@haozturk can we go ahead and ask for the invalidation/removal of the 2 mc files in the 3rd list?
@germanfgv can we go ahead and ask for the invalidation/removal of the 871 backfill files in the 4th list?

@amaltaro, you can find the info about the prompt reco files here:
/afs/cern.ch/user/c/cmst0/public/DBS/PromptReco_files_info.txt

And yes, you can proceed and remove all the backfill files

File invalidation requested in: https://its.cern.ch/jira/browse/CMSTRANSF-483

@vkuznet Valentin, in case you haven't managed to fix these data in DBS server, could you please look into this tomorrow? German provided two text files to recover T0-related data, please see messages above.

@amaltaro , I"m working on it, please be patient as I'm dealing with production DB and do not want to screw up things. I'm doing it in safe phases, i.e. inserting data into TEMP table, then need to verify everything and finally I can make a merge.

@amaltaro , @germanfgv I performed necessary injection for two input files you provided:

/afs/cern.ch/user/c/cmst0/public/DBS/PromptReco_files_info.txt, 44822 entries
/afs/cern.ch/user/c/cmst0/public/DBS/RAW_files_info.txt, 41325 entries

After insert, I checked data by two approaches:

I used direct SQL query

select distinct l.lumi_section_num, l.run_num from cms_dbs3_prod_global_owner.files f
join cms_dbs3_prod_global_owner.file_lumis l on l.file_id=f.file_id
where f.logical_file_name='/store/data/Run2022A/Cosmics/ALCARECO/MuAlGlobalCosmics-PromptReco-v1/000/352/394/00000/9de2e984-f955-4233-9598-15cb821def15.root'
order by l.lumi_section_num

And, I used dasgoclient, e.g.

# set lfn
f=/store/data/Run2022A/Cosmics/ALCARECO/MuAlGlobalCosmics-PromptReco-v1/000/352/394/00000/9de2e984-f955-4233-9598-15cb821def15.root

# query block name for that lfn
./dasgoclient -query="block file=$f"
/Cosmics/Run2022A-MuAlGlobalCosmics-PromptReco-v1/ALCARECO#e8d72285-0767-4152-86ab-5b35db0258f1

# set block name
blk=/Cosmics/Run2022A-MuAlGlobalCosmics-PromptReco-v1/ALCARECO#e8d72285-0767-4152-86ab-5b35db0258f1

# query file,run,lumi for that block
/dasgoclient -query="file,run,lumi block=$blk"
/store/data/Run2022A/Cosmics/ALCARECO/MuAlGlobalCosmics-PromptReco-v1/000/352/394/00000/9de2e984-f955-4233-9598-15cb821def15.root 352394 [9,12...]

So, from my side the data looks fine. But I would appreciate if you'll perform your own test, e.g. using dasgoclient procedure I outlined using another lfn.

@vkuznet Valentin, from the WMAgent perspective (vocms0282), everything looks good and the 3 stuck workflows are now progressing. Thank you!

I can now confirm that all the relevant stuck workflows went through and the old agents that were hit by this issue are now clean and ready to be shutdown.
Given that this is not a problem in WMCore and an unexpected DBS scenario happened, I am closing this issue without any necessary fix. Please feel free to reopen it in case there is anything missing.


same task name in flow - legit?

i created a flow like this with these tasks  :
get_used_images:
do:
ops.get_used_images:
- all_containers_ids
get_used_images2:
do:
ops.get_used_images2:
- all_containers_ids
publish:
- usedImages: imageList
get_used_images:
do:
ops.get_used_images:
- all_containers_ids: usedImages
meanning same task name twice, result is that last task is not executed at all..
should this fail in compile time? or be supported and is a bug??

I think we should fail compilation when @avimora  will handle compiler errors

I think we should fail compilation when @avimora  will handle compiler errors

+1 for this issue. really annoying!!

+2 for that - assigning to Avi

related to #28

Since tasks are currently written as map properties inside the workflow, we loose the information of the duplicate tasks already at the YAML parsing level. We don't get it in the compiler.
workflow:
  task1:
   ...
   task1:
    ...

We can either change the tasks to a list:
workflow:
  - task1:
    ...
  - task2:
    ...

Or write our own parser.
@orius123  and everyone else, WDYT?

I think it should be a list anyways since order is meaningful

list


Can't watch my own pay streams

I've been playing around with setting my items to require payment to test my publishing app, but then I noticed that LBRY was going to require that I pay the vig for watching my own videos.  It seems like there should be a way to watch your own streams without having to pay yourself (and lose LBC due to transaction fees).

Thanks for opening this issue! This would most likely depend on https://github.com/lbryio/lbry/issues/803 unless we do some type of workaround using claim_list_mine to check ownership of a claim.
This situation will only occur if you delete the file after you publish it, otherwise, you should be able to play it after publishing since file_list will know that you published it.

This should be fixed in an upcoming SDK release.

This should be fixed in an upcoming SDK release.


[FP]: False Positive on grpc-netty / grpc-netty-shaded : 1.50.2.jar

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:netty:netty:1.50.2:*:*:*:*:*:*:*
CVE
CVE-2019-20444
ODC Integration
None
ODC Version
7.4.1
Description
False positive on library grpc-netty-shaded-1.50.2.jar
Find below the vulnerabilities reported by the dependency check tool:
CVE-2019-20444
CVE-2019-20445
CVE-2015-2156
CVE-2019-16869
CVE-2021-37136
CVE-2021-37137
CVE-2022-41881
CVE-2021-43797
CVE-2022-41915
CVE-2021-21295
CVE-2021-21409
CVE-2021-21290
CVE-2022-24823
CVE-2014-3488
I have gone through all these CVEs and could not find anything related to grpc-netty-shaded
The issues shown are related to other netty libraries.

@keerthanavenky Can you share more info on how you run it? The FP is expected to not happen for any of the 7.x versions. I tried reprocuction with both Maven-plugin and CLI and fail to get the FP CPE match you mention.

@aikebah we are leveraging the gradle plugin

@aikebah we are leveraging the gradle plugin

Could not reproduce with the following build.gradle.kts
plugins {
    kotlin("jvm") version "1.8.0"
    id("org.owasp.dependencycheck") version "7.4.1"
}

group = "com.example"
version = "1.0-SNAPSHOT"

repositories {
    mavenCentral()
}

dependencies {
    implementation("io.grpc", "grpc-netty-shaded", "1.50.2")
}

I was able to reproduce the finding with the CLI scanning the jar directly, I did have to run the scan twice, after the second run CVE-2019-20444 showed in the report
Steps:

Download the jar from https://mvnrepository.com/artifact/io.grpc/grpc-netty-shaded/1.50.2
CLI Command dependency-check --scan grpc-netty-shaded-1.50.2.jar --disableCentral

Understood, we don't run with --disableCentral, I only used the flag due to the issues lately with Maven Central. Thanks for the reply I think this answers why we are seeing the CVE
References:
https://github.com/jeremylong/DependencyCheck/issues/5342
https://github.com/jeremylong/DependencyCheck/issues/5341

I was able to reproduce the issue by setting up a structure similar to how our scan is performed.
Steps to Reproduce:

Create a new folder and a libs folder nested inside (ex: my-plugin/libs)
Download the https://mvnrepository.com/artifact/io.grpc/grpc-netty-shaded/1.50.2 jar and place the grpc-netty-shaded-1.50.2.jar inside the libs folder
From the top level folder, bundle the folder as a jar jar cvf plugin.jar  .
Install the plugin to your local .m2 repository apache-maven-3.8.7/bin/mvn install:install-file -Dfile=plugin.jar -DgroupId=com.example -DartifactId=my-plugin -Dversion=0.0.1 -Dpackaging=jar
build.gradle configuration (Note: this finding occurs on version 8.1.0 as well)

plugins {
    id 'java'
    id("org.owasp.dependencycheck") version "7.4.1"
}
repositories {
    mavenLocal()
}

dependencies {
    implementation 'com.example:my-plugin:0.0.1'
}

Run ./gradlew dependencyCheckAnalyze

I bundled the plugin jar with some additional libraries. What I noticed is the "package" field is discovered on some libraries but not all. I'm wondering if this is something on the DependencyCheck end or on the central search end, if it's on central search we can close this issue out.

@kevin-doyle Thanks for the detailed reproduction. Hope to get a detailed look into it over the weekend. I'd say that as the jar is downloaded from central it should've been matched in central search, so I'll try and see whether I can reproduce what you observed and can find out why it doesn't match.

It appears as if documentation is not in line with actual implementation. I can reproduce your case because with the defaults in the gradle plugin the central analyzer appears to not be enabled.
Can reproduce the FP and fix it by configuring centralAnalyzer to enabled in build.gradle
dependencyCheck {
    analyzers {
        centralEnabled = true
    }
}

Check the code-base and it's been disabled by default since v3.2.1 so the documentation is lacking an update on that.

@aikebah Thanks for all the help! I was able to resolve this on our end.


cxxbridge can't parse modules with inner attributes

Reproduction:
Put this code into test.rs:
#[cxx::bridge]
mod ffi {
    extern "Rust" {
        fn temp();
    }
}

mod c_ffi {
    #![allow(dead_code)]
    fn may_not_be_called() {}
}

Then:
cd cxx/gen/cmd
cargo run -- test.rs

Problem occurs:
error[cxxbridge]: expected square brackets
   ┌─ ../../build/rust/tests/bindgen_test/src/lib.rs:14:6
   │
14 │     #![allow(dead_code)]
   │      ^ expected square brackets

This seems to be because gen/cmd/src/gen/file.rs line 24 uses only parse_outer, as far as I can tell.
This problem is also known as https://issues.chromium.org/issues/372062067. It's not clear that inner attributes should actually be in use here, but also, cxxbridge probably shouldn't be the thing which gets upset about them.

Hello, I have a query. In the test.rs you gave, mod c_ffi is not modified with #[cxx::bridge], but as per the execution error report, the cxxbridge tool still checks mod c_ffi?

Yes. The cxx parser has to parse all the mods to check for the relevant attributes.

Thanks a lot for the quick reply!

It may be rare to use inner attributes when using cxx, but it will provide users with more flexibility if we introduce it.

Just found a related issue #903 , so parsing modules with inner attributes is indeed necessary if we intend to support #![cxx:bridge].


Any way to define number of contributors?

Some project may have a lot of contributors, but there's no need to show all of them. Is there any way to show only first ten etc?

not really, but you can fork it and replace this line:
       for contributor in data.contributors

by
       for contributor in data.contributors[10]

@tosh please close this issue


MSID: 91260 Version: 3 DOI: 10.1101/2<IP_ADDRESS>0678

MSID: 91260
Version: 3
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>0678
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=91260
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>0678

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/91260v3
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>0678
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/91260/v3/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

error: [journal-ref-article-title] This journal reference (id c1) has no article-title element.
error: [ref-year-value-1] Ref with id c7 has a year element with the value '1 1997' which contains a digit (or more) but is not a year.
error: [ref-year-value-1] Ref with id c40 has a year element with the value '7' which contains a digit (or more) but is not a year.
error: [ref-year-value-1] Ref with id c49 has a year element with the value '22' which contains a digit (or more) but is not a year.
warn: [article-email-corresp-author-count-equivalence] The number of corresponding authors (1: Teruhiko Yoshida) is not equal to the number of distinct email addresses (2<EMAIL_ADDRESS>yoshidateruhiko@g.ecc.u-tokyo.ac.jp). Is this correct?


Publish latest burst2safe version to PyPI and conda-forge

Jira: https://asfdaac.atlassian.net/browse/TOOL-3364
Note: The above link is accessible only to members of ASF.

Older versions of this package are available on PyPI and conda-forge:

https://pypi.org/project/burst2safe/
https://anaconda.org/conda-forge/burst2safe

We should add our automated PyPI publishing GitHub Actions workflow to this repo, and do both a PyPI and conda-forge release for the latest version. We should also add all the relevant links to our Releasing HyP3 SDK and other packages wiki article.

Looks like there's already a https://github.com/ASFHyP3/burst2safe/blob/develop/.github/workflows/build-and-deploy.yml workflow and a https://github.com/ASFHyP3/burst2safe/blob/develop/.github/workflows/build-and-deploy-test.yml workflow with a workflow_dispatch trigger, so we should update or replace those workflows.


[Das Schwarze Auge 5] High Contrast Mode (Feature Request)

The character sheet is difficult to read for vision-impaired people, due to the colourful background.
@mckee14 If it is not too much work, would it be possible to have a button in the options tab to replace the brown background picture with the plain white sheet background?
I have a vision impaired player who has trouble reading the black on brown sheet.
The "default" sheet without css loaded would be very readable for him though.

I have literally NO idea about CSS, so sadly no.
The background picture is the first thing in the CSS file, if you have a PRO account, you can set up your game with your own charsheet.  That way, you could use the files from the R20 repo and just delete the part with the background in the CSS file - it works, I tested it locally.
To change it depending on a checkbox, you would have to go via the R20 forum or some CSS school sites.

@HauRukh I'd suggest to ask this question https://github.com/Meteox/roll20-character-sheets/issues. This is the URL you get when you click on the feedback button in the actual sheet.
Replacing the background image upon clicking on a checkbox is a rather trivial topic in CSS so it should be no problem at all.

The original creator (Meteox) abandoned this project. If you can't get a response from him, you could also ask in the R20 forum and post a link to the rather small CSS file out of the github repository. I'm sure you will get the answer quick. Maybe discuss it with your friend beforehand: What colors would he like? Maybe you can make it even better that just remove the background image.

@HauRukh  if this is resolved, please close it

Probably won't happen


[Feature] Add Helmet Middleware for Enhanced Security

Overview : To improve the security of our Express application, we should implement the Helmet middleware. Helmet helps set various HTTP headers that protect against common vulnerabilities.
Benefits:

Adds security headers to HTTP responses, helping to mitigate risks such as:
Cross-Site Scripting (XSS)
Clickjacking
Content-Type sniffing
Provides a foundational layer of security for our application.

Hey @yashgabani845 Please assign me this issue

Hey @Yashgabani845 I have raised the PR
Sorry for the delay


namespace quals required to work with stan-dev/math issue feature/037…

Submisison Checklist

[x] Run unit tests: ./runTests.py src/test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary
Bring stan-dev/stan up to date with changes in stan-dev/math removing math.h includes.
Intended Effect
Make behavior consistent across platforms and avoid polluting top-level namespace.
How to Verify
Integration testing.
Side Effects
No.
Documentation
No.
Reviewer Suggestions
Anyone.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

…0-cp11-cmath-funs-2

Jenkins, retest this please.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/43287

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/50363

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/50395


Switch to Flask-JWT-Extended

Flask-JWT apparently doesn't support reading from cookie, while Flask-JWT-Extended does. This may help solving #6 and #11

Foolish me, I already use Flask-JWT-Extended.


Update flake

Blocked on https://github.com/paritytech/cargo-contract/issues/1058#issuecomment-1501225760


[Feature]: API Client can hit minimal log-cache API endpoints

Blockers/Dependencies
No response
Background
The CF CLI attempts to fetch app staging logs from log-cache as part of cf push.
It does this via the following process:

Discovers the address of the log-cache API via Cloud Controller's / root endpoint
Calls the log-cache API's /api/v1/info endpoint to discover version info
Calls the log-cache API's /api/v1/read/<app-guid> endpoint to fetch log envelopes

We will have more functional support for logs as part of a later story, but for our cf push MVP we just don't want these calls to error.
As a client of the API shim
I want minimal implementations of the log-cache endpoints to exist
So that I can use my existing code and scripts to push apps
Acceptance Criteria
Scenarios
API shim root endpoint
GIVEN I have deployed the API shim
WHEN I hit the root discovery endpoint
curl "https://api-shim.example.org"

THEN I I see a log-cache entry in the response self-referencing the API shim's address
{
  "links": {
    "self": {
      "href": "https://api-shim.example.org"
    },
    "bits_service": null,
    "cloud_controller_v2": null,
    "cloud_controller_v3": {
      "href": "https://api-shim.example.org/v3",
      "meta": {
        "version": "3.90.0+cf-k8s"
      }
    },
    "network_policy_v0": null,
    "network_policy_v1": null,
    "login": null,
    "uaa": null,
    "credhub": null,
    "routing": null,
    "logging": null,
     "log_cache": {
      "href": "https://api-shim.example.org/" <-------------------------- new entry
    },
    "log_stream": null,
    "app_ssh": null
  }
}

log-cache info endpoint
GIVEN I have deployed the API shim
WHEN I hit the /api/v1/info endpoint
curl "https://api-shim.example.org/api/v1/info"

THEN I I see the following hard-coded response
{"version":"2.11.4+cf-k8s","vm_uptime":"0"}

log-cache read endpoint
GIVEN I have deployed the API shim
WHEN I hit the /api/v1/read/<app-guid> endpoint
curl "https://api-shim.example.org/api/v1/read/<app-guid>?descending=true&limit=1&start_time=-6795364578871345152"

THEN I I see the following hard-coded response
{"envelopes":{"batch":[]}}

Note: The query params need to be accepted by the API, but do not need to be functional.
Dev Notes

Should we add these endpoints to the k8s API shim? Or should we listen on a different port and have a separate log-cache domain on our "system domain"?

For the beta adding them to the shim itself should be fastest. For a GA log-cache emulation service we should consider a separate component.

https://github.com/cloudfoundry/cf-k8s-controllers/blob/a93050e37b203a8f38fa6194a828946815a00cbf/api/presenter/root.go#L31

Closing: This was addressed in https://github.com/cloudfoundry/cf-k8s-controllers/pull/255.


Clear index entries when removing pods or ctrs

When removing a pod sandbox or one of its containers, don't forget to remove the ID of the item from the corresponding ID index.

LGTM


feat(worker-path): remove

Old worker routing system was only supporting:
/worker_path/status
/worker_path/agent_download
Now we can support these kind of paths
/worker_path/status
/status
/path1/path2/path3/path4/.../status
Cloudflare already checks for subdirectories, we don't have to match whole pathname. So with this PR we only check the related path
Also some code is refactored and added more tests for routing system

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Remove packages for pydantic compatibility

Poetry updates
This PR updates LangChains poetry file to remove
any dependencies that aren't pydantic v2 compatible yet.
All packages remain usable under pydantic v1, and can be installed
separately.
Bumping the following packages:

langsmith

Removing the following packages
not used in extended unit-tests:

zep-python, anthropic, jina, spacy, steamship, betabageldb

not used at all:

octoai-sdk

Cleaning up extras w/ for removed packages.
Snapshots updated
Some snapshots had to be updated due to a change in the data model in langsmith. RunType used to be Union of Enum and string and was changed to be string only.


Global shortcut to Clear stored content without having to reveal current stored content

Before Submitting Your Feature Request

[X] Check that there isn't already a similar feature request to avoid creating a duplicate.
[X] I have seen the FAQ.

Problem
So there's the "Clear" shortcut: ⌥⌘⌫
It seems that the only way to activate it is to have Maccy opened up (either by clicking on it, or opening it with the global shortcut).
Is it possible to make a global Clear shortcut, so one doesn't need to open up Maccy and reveal what's in the stored content? Often we need to copy-paste credential, for example imagine we copied a newly created authentication token, or something else sensitive. Quite often I am screensharing or having my system shared on another projector. It would be preferrable in such situations to Clear the content globally/directly without even opening Maccy (as that reveals the stored content).
I do appreciate the benefit of the two-step process at the moment (Open first, then Clear) as that lets the user eyeball what's about to be deleted. But sometimes we need to directly Clear without letting other eyeballs see the content 😝
Thanks for considering!
Solution
No response

You can add the custom Shortcut workflow - https://www.icloud.com/shortcuts/e756634285ef413db90a39fc6e91d9ec. Make sure to assign it to a global shortcut you prefer like this:

My bad, this is now fixed and will work correctly in the next version.

Once a new version of Maccy is released (for example 0.32.0), the existing shortcut will start working.


tflite unsupported ops:  Log1p, SparseReorder, SegmentSum

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (or github SHA if from source):

Provide the text output from tflite_convert
# Copy and paste here
tflite unsupported ops:  Log1p, SparseReorder, SegmentSum
Some of the operators in the model are not supported by the standard TensorFlow Lite runtime and are not recognized by TensorFlow. If you have a custom implementation for them you can disable this error with --allow_custom_ops, or by setting allow_custom_ops=True when calling tf.lite.TFLiteConverter(). Here is a list of builtin operators you are using: ADD, CAST, CONCATENATION, EXPAND_DIMS, FILL, FULLY_CONNECTED, GATHER, LESS, LOGISTIC, MUL, RESHAPE, SELECT, SHAPE, SOFTMAX, STRIDED_SLICE, SUM, TILE, UNIQUE, ZEROS_LIKE. Here is a list of operators for which you will need custom implementations: Log1p, SparseReorder.

Also, please include a link to a GraphDef or the model if possible.
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

Apologies that it doesn't work out for you. We are continuously adding new ops into tflite, in the mean time;
You can use TFLITE_BUILTINS to convert your model it will help to reduce custom implementations.
See https://www.tensorflow.org/lite/guide/ops_select
You will have to custom implement Log1p, SparseReorde ops.
See https://www.tensorflow.org/lite/guide/ops_custom


Deprecated Version of pusher-js is used

Hey,
it seems this package is using a very old version of pusher-js. This leads to dependency errors in my application.
Will there be an update soon?

I haven't had any use for the package as of late, but I'm open to pull requests


feat: Add missing semicolon to JS variables in block component_head_analytics_gtag_config to prevent JS errors

1. Why is this change necessary?
https://github.com/shopware/platform/issues/3314
2. What does this change do, exactly?
It adds a missing semicolon.
3. Describe each step to reproduce the issue or behaviour.
https://github.com/shopware/platform/issues/3314
4. Please link to the relevant issues (if any).
https://github.com/shopware/platform/issues/3314
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

🤖 Generated by Copilot at f79a288
Fixed a JS error caused by a missing semicolon after the window.gtagURL variable in the analytics.html.twig template. Added a changelog entry for the fix.

🤖 Generated by Copilot at f79a288

Add semicolon to JS variable window.gtagURL in analytics.html.twig to prevent JS errors (link)
Add changelog entry for the fix in _unreleased folder with issue number, author information and summary (link)

Thanks for providing the fix @tinect , the PR was just merged 💙


feat: canoncialize assets for macOS, Windows, and Linux

This PR adds the right relative pathing for assets when bundled through cargo-bundle.
It's never been this easy to include an image in your Rust .app! :)
I don't have any other machine than a mac box so I can't test alternate implementations.
https://crates.io/crates/directories

[x] macOS

These targets do not provide a straightforward way of bundling assets. I've opted to make it the asset dir configurable instead.

[ ] windows
[ ] linux (rpm)
[ ] linux (deb)

We will add windows/linux later, or anyone apsiring to figure it out can take care of it.
It seems that when building with cargo-wix, you can choose the resource directory. Same with linux. Either we'll work with the tauri config and get theirs working by default or come up with our owm.


Minor changes

This includes minor changes:

Add .gitattributes and .editorconfig to keep code consistent with what you expect. My default settings, for example, use spaces instead of tabs and unix line-endings, so I was constantly introducing those by accident. This seems to remedy that.
Fix warnings about classes that defined operator==() but not Equals(object) and GetHashCode().
Remove *_xp platform toolset from test project -- If you don't have this component installed (I don't) the test won't load the project and will fail.
Fix issues I was having with test output: My system include paths on my Win7 machine are different than what was expected, so the test projects use a property sheet to override them.

Great work @dakotahawkins! Thanks for submitting this pull request.


Add: Today's Shopping Choice

Channel Name
Today's Shopping Choice
Native Name (optional)
No response
Network (optional)
No response
Country
CA
Subdivision (optional)
CA-ON
City (optional)
Mississauga
Broadcast Area
s/CA-ON
Languages
eng
Categories (optional)
shopping
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://www.tsc.ca/pages/watchuslive?ic=HP_WatchUsLive&fm=top_header
Logo
https://src.tscimg.ca/Content/en_CA/Images/Marketing/2021/Masterbrand2_Campign/GiftCard/MB21_GiftCards_Footer.svg
Notes
Streaming URL:
https://tscamd.akamaized.net/hls/live/503340/TSCLive/video_0/video_0.m3u8

Already in the database: https://iptv-org.github.io/?q=TSC.ca


yaml.jar build.properties entry needs to be updated

The line in build.properties needs to be updated to point to the new jar (1.12).

Closed by #494


Implement columns bundling

Helps reduce the amount of columns in our stores as we encounter some Parquet implementation limitations.

https://github.com/uber/petastorm/pull/188

Currently on hold. We are not sure if it brings value.


Local pull request success status testing

Docs Build status updates of commit 4a5a65b:
:clock10: Pending: waiting for processors (13 builds ahead of you)
:warning: Docs Build is busy, currently there are 13 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 4a5a65b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

E2E_DocsBranch_Dynamic/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Possible error in getting_started.ipynb?

Hello,
I was looking at the source code and attached notebooks in the folder examples. In the evaluation cell of the getting_started.ipynb notebook, we can find:
print('%s  acc: %.3f  auc:%.3f' % (split, *metrics))

This is shown as to have printed train acc: 0.983  auc:0.834 when running the statement test('train'). However, looking at the evaluator.py file in MedMNIST, it seems that the evaluator object outputs the AUC first and then the accuracy. Consequently, the print statements in your notebook(s) may be switching the two metrics.
Let me know if this is right.
Best regards,
qlero

Hi @qlero ,
Thanks for figure it out!
I have quickly checked the code. It seems our mistake. We will fix it soon.
PS: our experiment codebase is correct. I suggest you to use that repo if you want to do more experiments. The notebook is only for quick exploration.
Jiancheng

Thank you very much. I'll refer to the experiment codebase if need be in the future.
Best regards

Hi @qlero ,
We have fixed the mistake. Thanks a lot!


what about docs?

what should we do for documentation on this project @nherman3?

[ ] item 1
[ ] item 2


github action move path check to PR only

if pr check is required and path for backend doesn't change then pr checks done pass need to examine more how to resolve this problem


Initial work on Host based regex master balancer

Tests still work in progress
@brianloss

@keith-turner

Just looking for some early-on comments from you two if you have time. The use case here is to be able to host tablets for a table on a set of tservers, to include the accumulo.metadata table.

@ctubbsii


Postman: international standing order consents

Description

Add the proper requests on postman collections to tests the international standing order consent

Acceptance criteria

[ ] All current postman tests pass.
[ ] The international sanding order consent request tests pass.

Hey team! Please add your planning poker estimate with Zenhub @frlaura-jianu @mariantiris


CLI Pattern argument

I'm using typeorm but I do compile my files before running and I don't want to use ts-node.
The problem is, that the clie is using the pattern .bundle.ts which is problematic when running fixtures using compiled files.
My proposal would be to introduce a CLI argument to allow custom search patterns for fixtures, something like
--pattern *.bundle.js
this would be a cool feature since the only other way would be to use the programmatic api.

Hello peanutbother!
Do you have any problems using a custom pattern? It should actuall work as argument, not as option, so just without --pattern


Make the comments that come from "Needs more info" determinations link to the determination (see the same/related one for linking to Activity items)

This is fixed.


NavBar improvements step 1

Moved the navigation buttons to the left of the path bar
Fixed spacing of the toolbar buttons

Could you share a screenshot?

@d2dyno1 This PR is part of a larger effort to redesign the navbar area, but this is what it looks like so far.

LGTM (actually it looks horrible but it's part of the plan right? 🤣)

LGTM (actually it looks horrible but it's part of the plan right? )

tbh I like this one more

LGTM (actually it looks horrible but it's part of the plan right? 🤣)

Exactly 😆

LGTM (actually it looks horrible but it's part of the plan right? rofl)

Exactly

Hmm. How will the final version of this redesign look alike?

@d2dyno1 There are several parts to this design change including #3184, there are also plans to add some shadows around the app. It's hard to describe how it will look, but this concept should give you the basic idea.

Ooof every time I see one of this mockups I cry for the lost sidebar extending all the way to the top 😂. Aware of the reason behind the change my gut says otherwise.

@gave92 What don't you like about the sidebar extending to top?

I meant I don't like too much the current design where the sidebar is not extented to the top. When the sidebar is made acrylic I miss the "old" design.

@gave92 Oh good, then I am sure you will like the new design when #3184 is merged 😄

I see that little progress circle in the bottom right. Will this replace this progress bar which indicates that the folder is being loaded? I'd appreciate that.

@R3voA3 The progress circle is just a concept, don't read much into it.

@R3voA3 The progress ring is just a concept, there isn't anything planned to change there.

Too bad =(

@d2dyno1 There are several parts to this design change including #3184, there are also plans to add some shadows around the app. It's hard to describe how it will look, but this concept should give you the basic idea.

That has... everything I wanted 😲Even the W10X quick actions below the navbar

@yaichenbaum one thing I noticed is that with the navigation buttons on the same row as the address bar on my screen the address bar becomes very cramped(in dual pane even more so and almost unusable). Just saying that this change would adapt with the window width.

@gave92 It's something to look into, I am curious if this is really an issue or not because the dual screen feature is mainly used on larger screens. Regardless, I plan on reducing the size of the text in the nav bar, so that should help with this issue.

Thanks, it's simply that currently I can use the dual pane feature with no issue, but with this change it gets a little awkward. Thinking that the search button and the others are going to also be moved into the same row.. :)

@gave92 The only other button that's going to be moved is the search button, it's something we should start discussing to find possible solutions. Thank you for bringing it up.

Wondering if you have considered adding the title of the app (and logo) to the top left of the NavBar. Probably good from a branding point of view, but also would be consistent with a lot of the other 'modern' apps. At the moment it sort of looks like the app is called 'favourites'. :) Screenshot below for example.

Not really sure where the title and logo would fit since the Tabs are taking the space and soon the nav bar.
Also, they should not try to be consistent with Microsoft apps since that's impossible :D

The Win32 Explorer doesn't use a logo or a title anyway, so as it is now Files is consistent with the legacy file browser

@yaichenbaum Any chance to see a screenshot? I need to see where
          <target state="new">Browse</target>         </trans-unit>         <trans-unit id="RecentLocations" translate="yes" xml:space="preserve">           <source>Recent locations</source>           <target state="new">Recent locations</target>
Is used so I can properly translate it.

Here is two good examples I think. Adding a title to the top of the navbar doesn't get in the way of tabs, or the navbar items.

Also here: https://www.behance.net/gallery/74300939/Windows-10-File-Explorer

@R3voA3

Wondering if you have considered adding the title of the app (and logo) to the top left of the NavBar. Probably good from a branding point of view, but also would be consistent with a lot of the other 'modern' apps. At the moment it sort of looks like the app is called 'favourites'. :) Screenshot below for example.

Thanks for bringing this up, it seems like Microsoft is moving back towards this trend, but as mentioned above, it wouldn't fit into our design right now.


feature request: Allow showing violations for only added or changed lines

Which reviewdog version do you use?
I'm guessing you're building the latest head ver, right?

and which reporter do you use? Can you write more information?
Originally, reviewdog actually showed violations for only added/changed lines, but it's changed by  #450.
You can use -filter-mode=added flag for the original behavior, but I'm currently considering to change the default value of -filter-mode to added as well before the new release. cc/  @Le6ow5k1

Implemented. https://github.com/reviewdog/reviewdog#filter-mode


Some more small fixes

Why the rename from coinPair to quotedCoin?

To maintain naming consistency and because it actually isn't the coin Pair (consisting of two coins). The proper term is quoted coin which is the one you are selling/using to purchase the other.

Merged into master branch. Thanks for the improvements!


[WebView] Crash on Android 10

Hi,
Our app is experiencing crashes with WebView, specifically on Android 10 only.
Here is the log:
java.lang.NullPointerException 	at android.view.inputmethod.InputMethodManager.startInputInner(InputMethodManager.java:1662) 	at android.view.inputmethod.InputMethodManager.checkFocus(InputMethodManager.java:1901) 	at android.view.inputmethod.InputMethodManager.isActive(InputMethodManager.java:1220) 	at io.flutter.plugins.webviewflutter.InputAwareWebView$1.run(InputAwareWebView.java:188) 	at android.os.Handler.handleCallback(Handler.java:888) 	at android.os.Handler.dispatchMessage(Handler.java:100) 	at android.os.Looper.loop(Looper.java:213) 	at android.app.ActivityThread.main(ActivityThread.java:8178) 	at java.lang.reflect.Method.invoke(Method.java) 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:513) 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1101)
Is this one of the known issues with WebView on Android 10 or related to any of those?
Thanks.

Hi @hoangduong1607
Can you please provide your flutter doctor -v, your flutter run --verbose and a minimal complete reproducible code sample?
Thank you

Hi @pedromassangocode
This is a first-party bug, so flutter doctor is not applicable. Could you please help to add first-party label for this bug? Thanks.

This is a first-party bug, so flutter doctor is not applicable.

@hoangduong1607
flutter doctor -v is very applicable and very relevant,
if you were to use an obsolete version of flutter for example
there would be little that flutter team could do to address the issue

Could you please help to add first-party label for this bug? Thanks.

for the issue to be workable needs to be reproducible, usually labels are added after reproducing the issue
please provide the information requested above

Can you please provide your flutter doctor -v, your flutter run --verbose and a minimal complete reproducible code sample?
Thank you

@hoangduong1607  please see https://github.com/flutter/flutter/issues/78405#issuecomment-803873017 and provide what was asked in https://github.com/flutter/flutter/issues/78405#issuecomment-800952048.

I am facing the same issue in my app which I have hosted on Google Play Store. It doesn't happen on my device but according to the play console crash report, more than 100 users have been affected by this issue. The one thing common in all such users was that they were using Android 10.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

Hi, we're experiencing the exact same issue in our app as well. Here is my flutter doctor output:
[✓] Flutter (Channel master, 2.3.0-17.0.pre.230, on Debian GNU/Linux rodete 5.10.28-1rodete1-amd64, locale en_US.UTF-8)
    • Flutter version 2.3.0-17.0.pre.230 at /usr/local/google/home/aweiland/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5cc7b6c3ea (2 minutes ago), 2021-06-07 10:33:00 -0700
    • Engine revision b4bce0cfaa
    • Dart version 2.14.0 (build 2.14.0-186.0.dev)

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/linux#android-setup for more details.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /opt/android-studio-with-blaze-4.2
    • Flutter plugin version 57.0.2
    • Dart plugin version 202.8488
    • Java version OpenJDK Runtime Environment (build 11.0.8+0-b944-P17168821)

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 91.0.4472.77

! Doctor found issues in 1 category.

Do note that we are also a Google app, and Google's internal release process is very different than the public Flutter-Android development processes.
Unfortunately, nobody on our team has been able to reproduce the issue locally. We do have over 1000 crash reports from users in production however. Thanks

I can also hazard a basic guess from looking at the stack trace that InputAwareWebView is posting a runnable, which leaks views. If that runnable then gets run after those views are detached, that's likely to result in crashes like this.

targetView and containerView needs to be null-checked

This seems to be a production issue and thus it would be difficult for us to reproduce locally. It was tested on an emulator and worked fine.

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on Microsoft Windows [Version 10.0.22000.120], locale en-US)
    • Flutter version 2.2.3 at C:\Users\Taha\Code\flutter_stable
    • Framework revision f4abaa0735 (6 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\Taha\Code\android-sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_SDK_ROOT = C:\Users\Taha\Code\android-sdk
    • Java binary at: C:\Users\Taha\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop for Windows (Visual Studio Community 2019 16.10.4)
    • Visual Studio at C:\Program Files (x86)\Microsoft Visual Studio\2019\Community
    • Visual Studio Community 2019 version 16.10.31515.178
    • Windows 10 SDK version 10.0.19041.0

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).

[✓] VS Code (version 1.59.0)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.25.0

[✓] Connected device (4 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 10 (API 29) (emulator)
    • Windows (desktop)                  • windows       • windows-x64    • Microsoft Windows [Version 10.0.22000.120]
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 92.0.4515.131
    • Edge (web)                         • edge          • web-javascript • Microsoft Edge 92.0.902.67

! Doctor found issues in 1 category.

I'm not sure if this the exact same bug but the following happens when clicking on the following link http://untis.at/ that is in an Android WebView (default: virtual display) on Android 10.
(Note: unsure if related, but the app doesn't have permissions communicate via cleartext)
Fatal Exception: java.lang.NullPointerException: Attempt to read from field 'android.view.WindowManager$LayoutParams android.view.ViewRootImpl.mWindowAttributes' on a null object reference
       at android.view.inputmethod.InputMethodManager.startInputInner(InputMethodManager.java:1626)
       at android.view.inputmethod.InputMethodManager.checkFocus(InputMethodManager.java:1865)
       at android.view.inputmethod.InputMethodManager.isActive(InputMethodManager.java:1184)
       at io.flutter.plugins.webviewflutter.InputAwareWebView$1.run(InputAwareWebView.java:26)
       at android.os.Handler.handleCallback(Handler.java:883)
       at android.os.Handler.dispatchMessage(Handler.java:100)
       at android.os.Looper.loop(Looper.java:214)
       at android.app.ActivityThread.main(ActivityThread.java:7356)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:491)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:940)


Loot notifications sending incorrect/wrong Araxxor loot

Checklist

[X] I'm reporting a problem with Dink
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
Dink is sending notifications for loot from Araxxor that it does not drop
Screenshots

Runelite Logs
client.log
Runelite Version
1.10.43

This is a shortcoming in runelite's loot tracker; see https://github.com/runelite/runelite/issues/18242#issuecomment-2336027507


Fix package CHANGELOG template

Aligns the package CHANGELOG.md.tmpl to the one in templates/plugin.
Fixes #76018.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I signed the CLA.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

it should be vice-versa

@nt4f04uNd I explained in #76018 why I think this is not the case.

Thanks for the contribution @creativecreatorormaybenot


[FEATURE] detect when we don't need sde

As @jfalcou mentioned, some of the github action servers actually don't need sde for avx512. Which would be nice  to use since sde is really slow

Fixed by reusing Circle CI that guarantees AVX512 machines


ClimateDevice is deprecated

HA 0.116 prints the following warning upon startup:
WARNING (MainThread) [homeassistant.components.climate] ClimateDevice is deprecated, modify UponorThermostat to extend ClimateEntity

Issue @24 is solved


Cannot Log in or Authenticate

Extension version: 0.11.2
VSCode Version:  1.38.1
OS: Mac OSX 10.14.4

Steps to Reproduce:

CMD + SHIFT + P to authenticate manually
Paste token generated from settings in github
Use the server github.com(not sure if this is the correct server)

Nothing happens

@OliverHuntsman Did the normal auth flow fail for you? Are you using GitHub enterprise?

Just nothing happened.
I’m not using an Github enterprise

@OliverHuntsman To clarify, when you first tried to authenticate, did you use the Github Pull Requests: Sign in to GitHub command?
Can you check that the folder you have open is pointing to a GitHub remote? (Run git remote -v)

FYI, I had the same issue and came here looking for clues. Turns out I had to open a folder in a new window with vscode to get this extension set up. I previously had a workspace open with multiple folders and there was no sign in button on the status bar, and manual setup didn't work. Might be something worth trying.

Here’s the issue
https://drive.google.com/a/gemmasutton.com/file/d/1RryZVvLaiUNmS3NGLkqKM7YBcdcs4R7p/view?usp=drivesdk
How do you mean you opened a folder in a new window? Which folder? Any folder?

That drive link needs sharing permissions, I'm not able to see it. To clarify what I said earlier, I opened one of my project folders which has a git repo in it. I'm not sure if the repo matters, you may be able to open any folder.

@RMacfarlane There is no Github Pull Requests: Sign in to GitHub command?

@lvarado I've changed the sharing now, the folder doesn't have a git repo in it bu tI get the same results with a folder which does have a git repo

@OliverHuntsman If you click on the GitHub icon on the activity bar on the right, what message is shown? Can you run the View: Toggle Output command (by Cmd + shift + p) and choose "GitHub Pull Requests" from the dropdown and copy the output from that?

Hi, no output

Ok, thanks, this helps. What's the result of running git remote -v in the terminal?

Sorry for the delay on replying

OK, so now I've set the remote repository as Github using the terminal and pull requests is ow working - thank you so much for your help.
FYI I used this command when I created a new repository in GitHub for an existing directory on my machine.


Build(deps-dev): Bump eslint-plugin-import from 2.28.0 to 2.28.1 (#5110)

Pull request opened by the merge tool on behalf of #5110

Merged in 4330cc695628cb2154704841fdb074b211bb4753