text:
withScope is marked as optional, but is required

In the README, it looks like withScope is marked as not required, but an error is thrown if you don't provide it. Looking at the code, it definitely seems to be required: https://github.com/BrunoScheufler/graphql-middleware-sentry/blob/master/src/index.ts#L75
Any reason it couldn't be defaulted to the identity function? If so I'd be happy to make a PR.

Thanks for adding the default 🙌 I'll make sure to release this as soon as possible 👍

:tada: This issue has been resolved in version 3.2.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

authors: BrunoScheufler cappslock


text:
change regexp to only match define at the beginning of the sentence

If I write something like this would define ... than the bot will search on urban. Maybe it would be a nice idea to change the regexp in a way that it only matches if the define is in the beginning?

That is a good call. Let me look at it over the weekend.
Thanks for the feedback!

Great. Thanks!

source: gharchive/issue

authors: BryanSaxon bitboxer

repo: BryanSaxon/hubot-urban-dictionary

license: mit

license_type: permissive

license_source: bigquery


text:
Should we drop the test script and just check in the test artifacts?

cc @chenglou
Other than potentially forgetting to check in the rendered artifacts and not having it be caught by CI, do we really lose anything by not having a test script and the explicit "expected" snapshots, compared to just using checked in artifacts as snapshots and the review process to verify them? It might even be a win, since there would be more eyes on it.

Yeah check the artifacts in!

authors: chenglou glennsl

repo: BuckleTypes/bs-webapi-incubator

license: MIT

license_source: github-api


text:
chore: update action.yaml

Description
Bind to 7.2.3 directly for a bit so that CI machines that have the faulty 7.2.2 version installed are updated to a working version.
Checklist

[x] Tested manually
[x] Unit tests pass with no errors or warnings
[x] Documentation updated (if applicable)
[ ] Reviewed by at least 1 other contributor

This is not the problem...

source: gharchive/pull-request

repo: BugSplat-Git/symbol-upload


text:
feat: ssr aws lambda apdapter

What is it?
New Adaptor for AWS Lamda
https://share.vidyard.com/watch/yKuXu8uiPpgRqHopySs9Pe?

[x ] Feature / enhancement
[ ] Bug
[ ] Docs / tests

Description
New implementation for deploy qwik add easy
Example: https://rsltkw254b.execute-api.us-east-1.amazonaws.com/
Checklist:

[ x] My code follows the developer guidelines of this project
[ x] I have performed a self-review of my own code
[ x] I have made corresponding changes to the documentation
[x ] Added new tests to cover the fix / functionality

serverles

We just made an update to how servers/adaptors are imported from Qwik City projects. Would you be able to update this PR to the last structure updates? https://github.com/BuilderIO/qwik/tree/cdea8ff43b2f7f3eab1634c795e360c8a5ae7ec9/starters/adaptors/cloudflare-pages
This will be a great addition, thanks!

New structure applied @adamdbradley  also i removed somes dependencies and moved a logic part to other package for avoid mix stranger somethings here.
https://github.com/leifermendez/serverless-plugin-qwik

Is this right for where the vite.config.ts file would go? starters/adaptors/aws-lambda/adaptors/aws-lambda/vite.config.ts

I need to review some files and make a test @adamdbradley I hope the next week launch an update

Hi @leifermendez 👋
Just scanning the open PRs and found this one 😄  a bit dusty but looks as good as new 🤣 what's the state of it?

Hi @leifermendez 👋 Just scanning the open PRs and found this one 😄 a bit dusty but looks as good as new 🤣 what's the state of it?

🙌 Hi @zanettin I need check this again because I remember this I left as draft for something but I not exactly the cause

@gioboa did i understand you right, that you have smth working in that regard?

Yep, I have it with edge AWS lambda

could we create a integration out of it? 😍

Yes, that's the plan

Bumping in excitement

authors: adamdbradley gioboa icarus-sullivan leifermendez zanettin

repo: BuilderIO/qwik


text:
fix(docs): add missing import statement of qwik-city menu

What is it?

[ ] Feature / enhancement
[ ] Bug
[X] Docs / tests

Description
fix(docs): add missing import statement of qwik-city menu
Use cases and why

One use case

Another use case

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

So good! thank you very much!

authors: cunzaizhuyi manucorporat


text:
Simplify Readme

It looks like the current Readme file has a lot of preview images, which may make it difficult for users to quickly find the information they need. Additionally, the link to the website is not prominently displayed.
We need to reduce the number of preview images in the Readme and make the link to the website more prominent. This will make it easier for users to get started with the project and find additional resources. I'd like to convert the HTML in the logo area to Markdown.
We also need to update the acknowledgments section: remove the reference to best-readme-template, as it is no longer being used in the project and add references to the Rust programming language and the MkDocs documentation.
Checklist

[x] reduce the number of preview images
[x] make the link to the website more prominent
[x] remove best-readme-template from the acknowledgments because it's no longer used
[x] add the rust language and mkdocs to the acknowledgments
[ ] convert the html in the logo area into markdown

Since the logo and the corresponding project name and description are all horizontally centered, we cannot use markdown for this and as such, have to leave it in html.

repo: Builditluc/wiki-tui


text:
Issue with Search in SavedProjects and SavedAPKs + My Drafts

While searching if any item is deleted, then on closing the search view, adapter still contains that item.
In Drafts, if an item is saved as project and on clicking back, if no drafts exists then, it should display proper empty message.

@opticod Can I work on it?

@codingblazer. You already are on more than 1 issue lemme work on it. ^^

Thank you Guys, but i have already fixed it. @codingblazer @octacode

@opticod Great !!
@octacode I don't have more than one issue assigned.

Closed via #277

authors: codingblazer octacode opticod

repo: BuildmLearn/BuildmLearn-Toolkit-Android


text:
All non-replaced players disappear

Worked fine last update, now everyone not using the custom models disappear

Related to this https://github.com/BunyaPineTree/LethalCompany_ModelReplacementAPI/issues/24#issuecomment-1879295918


text:
Support keywords (like returns, example,etc)

Like sphinx, can you provide keywords like returns or example to document such things?
Thanks a lot,

Absolutely not. The whole point of using Markdown was to encourage writing documentation using prose rather than be burdened with the mental overhead of remembering annotations.
(Not everyone agrees with this, and that's OK, but for better or worse, than is the trade off made by pdoc. See: https://github.com/BurntSushi/pdoc#motivation)

Ok. But how do you refer to a Class of your module? With a link?

Yup. See the docs "Linking to other identifiers": http://pdoc.burntsushi.net/pdoc

Thanks for references. What about external references like python json module?

@somenxavier Create a hyperlink.

authors: BurntSushi somenxavier

repo: BurntSushi/pdoc

license: unlicense


text:
Add regex responder capability

Work in progress. Lots of assistance from Matt.

I think that it would be good to create a regex type macro. If you want to update this implementation to work with the current Butler implementation then I'd love to discuss it.
I'm going to close this for now.

authors: keathley t-richards

repo: ButlerBot/butler


text:
[Chore] The different environments of the project.

Summary
Configure the .env files like development, staging, and production environments.
Acceptance Criteria

[x] Create the environment files
[x] Install the packages and Add the scripts.

The package json scripts was programmed and the files .env too for staging environment, production environment and development environment.


text:
🛑 C21CoasttoCoast is down

In f78da34, C21CoasttoCoast (https://www.c21coasttocoast.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: C21CoasttoCoast is back up in 1e988cd after 23 minutes.


text:
Enhance Build Script

I want to

Warn if the save json isn't found and copied into the artifact but not fail
For Mac, add the json in the zip archive output by Godot export
For Mac and Linux, name their archive files with the suffixes

#FirstTry !
Got all three done in d2657586caf064719b60a635ea6c1761b17ba5fd


text:
charts/gateway - redis integration

Description of the change

Updated image

Updated to Gateway 11.0.00_CR1

this will cause a restart if you are not overriding the default image

Redis Integration

Redis Configuration options for the Gateway (future use)
Added Redis SubChart

Ingress

Backend service is now more configurable allowing the management service to be exposed via ingress controller

this should only be done in environments where the ingress controller does not have a Public Address
ingress.rules[n]backend can be set to "management"

Restart on config change

A new flag has been added to facilitate auto redeploy of Gateways when there is a config change
Applies to the default config map only

does not include config.cwp, config.listenPorts or the Gateway Secret

MySQL subChart updated

Benefits

Updated Gateway Image to 11.0.00_CR1
Redis integration options
Remove the need to use rollout restart when updating configmaps
Route to the Management service via Ingress Controller

Drawbacks

This update will cause a rolling restart of the Gateway Deployment (new image)

Checklist

[X] Chart version bumped in Chart.yaml according to semver.
[X] Variables are documented in the README.md
[X] Title of the PR starts with chart name (e.g. [charts/gateway])
[X] If the chart contains a values-production.yaml apart from values.yaml, ensure that you implement the changes in both files

Closing - new PR here. https://github.com/CAAPIM/apim-charts/pull/283

repo: CAAPIM/apim-charts


text:
477072: Use opensuse/tumbleweed:latest as base image

https://internal.almoctane.com/ui/entity-navigation?p=131002/6001&entityType=work_item&id=477072

A developer build has not yet been created for this branch. Click here to go ahead and create the build...

CI Build Link:
http://sou-jenkins2.swinfra.net:8080/job/CAFapi/job/CAFapi~opensuse-java17-images~rory-tumbleweed~CI


text:
how to get the environment of precomputed_arrays in "so3.py"?

when I run the file of "eval_samples.sh", there is an error that "FileNotFoundError: [Errno 2] No such file or directory:***/precomputed/precomputed_arrays/.so3_omegas_array2.npy". But I dont know how to get the npy files? I am looking forward to your guidance. Thank you so much!

Thank you, the issue has been resolved.

Hi how did you resolve this?

Thank you, the issue has been resolved.

thanks! :)

authors: CAODH kelly1210 shekfeh

repo: CAODH/SurfDock


text:
🛑 Test Broken Site is down

In 846280b, Test Broken Site (https://radcord.xyz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Test Broken Site is back up in 4568074 after 19 minutes.


text:
🛑 RadCord Bot is down

In 35e09f1, RadCord Bot (https://radcord.xyz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: RadCord Bot is back up in 7ad8920 after 38 minutes.


text:
🛑 GF Status Dashboard is down

In 28cae65, GF Status Dashboard (https://gfstatus.xyz) was down:

HTTP code: 0
Response time: 0 ms

Resolved: GF Status Dashboard is back up in 48ccf65 after 14 minutes.


text:
Composite Attester/Device Description

As per RATS Architecture, Composite Device comprises of multiple sub-attester and a lead attester.
A more deeper discussion is required in CCC as to how different member organisation would want to see the Composite Attester Modelling and how the Format of the Attestation Token would span out meeting all the necessary security objectives

how different member organisation would want to see the Composite Attester Modelling
how the Format of the Attestation Token would span out meeting all the necessary security objectives

This sounds very interesting. Can you elaborate a bit more, e.g.,

what kind of modeling are you thinking about?
how is the format of attestation token related to security objectives?

how different member organisation would want to see the Composite Attester Modelling
how the Format of the Attestation Token would span out meeting all the necessary security objectives

This sounds very interesting. Can you elaborate a bit more, e.g.,

what kind of modeling are you thinking about?
how is the format of attestation token related to security objectives?

I think it'd be good to clearly state here what the expected outcome of the discussion would be.  A position paper?  Some form of input for the RATS / TCG / ${RELEVANT_SDO}?  Other?
(CC: @TheBankster who put forward the need for a "composite attesters" track in the SIG.)

@deeglaze on 27th Feb 2024 call:
A few separate/separable dimensions to consider:

How to bind evidence's topology together
How to express topology from the lead attester
How to express the accepted topology in attestation results

repo: CCC-Attestation/meetings


text:
Create parse-mega.js

Worker file for parse mega

Thanks, @bedwar14 ! Sorry this languished for awhile. We really appreciate it, and it looks good!

@AABoyles  No problem.  Was just looking for a first project I could contribute to.  Not used to the process so I didn't see the message till now.

repo: CDCgov/MicrobeTrace


text:
Rolling window for inference

This PR makes the inference problem into a sequence of inference problems with a rolling window lookback (default 35 time steps), start (Default 21 ts), and stride (default 7 ts) indicating the start and strides of the "current" time along with the size of the rolling window.
It also modifies methods that expected a fixed tspan to create a tspan from information in the inference config.
Closes #330

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.18%. Comparing base (d43f033) to head (b8a9982).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #335   +/-   ##
=======================================
  Coverage   93.18%   93.18%
=======================================
  Files          50       50
  Lines         528      528
=======================================
  Hits          492      492
  Misses         36       36

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

repo: CDCgov/Rt-without-renewal


text:
Enable replaceUnicodeWithAscii in the UP

User Story
As ReportStream, I want the ability replace unicode with ASCII for messages flowing through the UP, so that I can support the translation setting in the Covid pipeline.
Description/Use Case
The Covid pipeline has the ability to replace unicode with ASCII for messages being sent out. We need to enable this in the UP. This is currently only used for HL7 messages we need to find out if we need this for all formats.
Risks/Impacts/Considerations
Dev Notes

We need to find a way to enable replaceUnicodeWithAscii in the UP
The covid pipeline calls the unicodeToAscii function in Hl7Serializer.kt if the flag is set to true
We can probably re-use this function in the UP

Acceptance Criteria

The UP can replace unicode with ASCII for messages.

Will reopen if needed, not used by any receivers.

repo: CDCgov/prime-reportstream


text:
Design AuthN/Z Microservice Support for Okta Users

User Story

As a ReportStream maintainer,
I want the the AuthN/Z microservice to support Okta Users,
so that I do not have to support the legacy RS server-server AND Okta auth.
Description/Use Case

PR #UPDATE_JAMIE implements a MVP integrating the AuthN/Z microservice with the submission microservice for Okta Applications. We want Okta Users to also be able to use any microservice (or legacy app azure function) using the same auth microservice so that we do not have to support multiple auth options/implementations.
The new microservice supports the creation of Okta Applications which can be assigned the scope "sender" and be assigned to zero or more applications. If the Okta Application makes a request to the auth microservice, the service will check for the "sender" scope and that the Okta Application belongs to the Okta Group they are claiming to be a member of (via client_id header).
To make the new auth microservice support Okta Users, we will need to:

investigate how to reconcile the current pattern of determining scopes (Okta Group prefixes like: DH and DHSender) vs the new one (using actual scopes). It seems like Org names would no longer need prefixes like DH and DHSender.
gather requirements on what Okta Users should or should not be able to do and how their permissions should be managed.

Risks/Impacts/Considerations

Dev Notes

Acceptance Criteria

[ ] PRE-REQ/Blocker: Gathered Requirements from product
[ ] Created design detailing how Okta Users shall be supported with the new AuthN/Z service

@chris-kuryak  I don't think this is actually a blocker for the Microservice MVP deployment. I think the only blocker is a new ticket for integrating the new auth microservice into the history endpoint but it would be good to align on the future auth strategy sooner rather than later.


text:
Design - Discovery: Validation Service UI for a prospective Sender

Problem statement
As a prospective sender with no access to the protected ReportStream website, I want to be test my data file in CSV, HL7 or RADx-MARS format and get user friendly responses to understand, identify and fix issues with my data.
Things to consider:

Are new senders using custom schema?
Non-authenticated users looking to validate without support?

What you need to know

Mural board for Validation Service Deep Dive with Options
Provide user-interface that allows file drag&drop or upload feature and display of response to the file
There is an expectation of reusing CSV Uploader components and features and not a full new page

Acceptance criteria

[ ] Design/IA/Wireframes to allow a public user to do the following:
[ ] User is able to upload a file in HL7 or CSV format
[ ] User is able to view success or error msgs based on the validation of the data file
[ ] Determine where this tool fits in the IA
[ ] Author content around explaining the tool and how to use it (support /troubleshooting?)

To do

[ ] Verify what the intent is with files that get uploaded (are they kept?)
[ ] Review the current state of the feature-flagged validator (in order to keep any wireframes in sync)

FYI- @a-chappelear and @drew-usds will discuss. Not ready to refine yet.

Per design sync, @drew-usds & @victorialb-usds are going to meet with O&O for a bit of background/discovery work on this ticket (to be revised.)

Added some details to problem statement: - Are non-authenticated users looking to validate without support?

Specify the user group and segment to design for and why (API only; large senders, what would most reduce burden for state in their onboarding of senders?);
also take learnings from internal validator and the use case for which states would do this instead of reportstream and why? IL uses NIST. They might want to use our validator to directly connect senders. As per call with IL august 2022, it is not clear why they would directly connect rather than using RS.

I would suggest making an additional ticket for the for problem statement work (if not done in this sprint.)
For the AC on this  “Author content around explaining the tool and how to use it (support /troubleshooting?)” There is additional discovery work happening in other tickets on how a user could select a receiver (on the public side) and until that is determined/designed, it’s unclear how to write instructions.
—
This ticket does have a number of UI updates completed and should be done for both public and internal users. They are annotated and numbered in the Figma file, listing them also here:
15 - Adding “ReportStream” to the page title
8 - Move the filename info up to the top and above the messages/alerts
9 - Replace alert banners with the “slim” comp option, updated copy on all
12 - We only need “warnings” and “errors” mentioned once, replace them as headers and remove as table labels
13 - Updated copy
14 - “Cancel” is removed (not needed for a validator)
11 - Nothing to change in row headers, but there likely will be when we get to the error content / inventories in a coming ticket.
—
Some possible stories/ideas for later:

Consider creating a guide or documentation that a user could refer to on it’s use and for troubleshooting tips
Possibly add the ability to save or download results (if errors get really lengthy, it may be something someone wants to save and refer back to)

cc: @RakshindaAslam @bgantick @HeatherRienks

Should we add this to our O&O sync tmrw?
@drew-usds @victorialb-usds @a-chappelear  @RakshindaAslam

authors: HeatherRienks RakshindaAslam a-chappelear victorialb-usds


text:
Migrate to modular flyway (v10) + add postgres module

This PR bumps flyway from 9.3 to 10.0.1, adding a dependency for the newly introduced flyway postgresql module. Version 10 of flyway introduced modular support for databases. These dependencies need to be injected on the build classpath (as opposed to the project classpath).
Test Steps:

Gradle sync
Run test suite

Changes

Bump flyway to 10.0.1
Add modular dependency for flyway-database-postgresql to buildscript

Checklist
Testing

[x] Tested locally?
[x] Ran ./prime test or ./gradlew testSmoke against local Docker ReportStream container?

Process

[ ] DevOps team has been notified if PR requires ops support?

Linked Issues

Fixes #12247

@thetaurean We should keep an eye on the automatic deploy to staging when this gets merged

authors: mkalish thetaurean


text:
Experience/8959/replace organizations table

This PR replaces the table component with our custom table in /shared/Table. This will be the table component use across ALL tables going forwards in ReportStream. We're also adding sticky header functionality.
Been working closely with @jillian-hammer for the sticky header functionality and overall design. This is a 1:1 replacement -- any other edits across all tables will come in a later PR.
Example:
https://github.com/CDCgov/prime-reportstream/assets/516877/c8e0e831-a006-4db9-9feb-784150520288

thanks @etanb ! two design notes:

can you take a look at why the headers jump in position once you start scrolling? ideally, they'd stay in place. the scrollbox looks like it has extra inner padding at the top?
question: is it possible to keep the header row underline sticky, too?

After chatting with @jillian-hammer we're going to move forward without the internal scrollbar model for now. Here's the latest-and-greatest:
https://github.com/CDCgov/prime-reportstream/assets/516877/1c498b64-954e-4bdf-917f-58072c681250

authors: etanb jillian-hammer


text:
Shep Review

Great job on the FEC!!!
After going through the code base there are some bad habits I see developing that I would like to point out.
Taking consideration of these comments and making a cognitive effort to avoid similar habits in the future will make meaningful improvements to the efficiency and beauty of your code.
Whoops:

I saw lots of npm packages that can rob you of significant learning experiences, and if you're going to use them read the docs and don't use deprecated modules.

things to avoid:

using router for a single endpoint (currentProductInformation.js)
Bloated config files (package.json, "D": "^1.0.0"?)
Empty Files (leaving placeholders can lead to problems)
Mixing Fetch and Axios (stick with one or the other)
Mixed files (separation of concerns, keep similar items grouped)
Class components and Hooks (seriously, do not have the two in one repo, they are not interchangeable and can lead to unforeseen and very hard to correct bugs)
excessive css files (make css elements modular)
bloated state (WriteReview.jsx, focus on efficient use of data)
Unnecessary class components (Stars.jsx doesn't contain state)
no default when using Case (SungleStar.jsx)
hard to read code (AnswerForm.jsx, add a new line for each element property)
unused functions (AnswerList.jsx, ComponentDidMount())

AnswerForm.jsx and QuestionForm.jsx refactored for readability
AnswerList.jsx unused code clean up

authors: Blake-Whitham foodforcode

repo: CDMLifecycle/FEC


text:
Tweaks for Work/Grouping in play queue

I wanted to push the Artist/Album text in the heading onto a second line, but couldn't work out how.
Examples:

We should probably put this on hold for now (and also back out the commits you made for #992) because the whole work/grouping thing is causing some problems for MusicBrainz/Picard users who have those tags and didn't necessarily know it, until LMS9 started using them.

Why? What's wrong with the current code?

It's grouping the play queue by work (if that exists). This could potentially cause problems for releases tagged by Picard: unless you're using one of Picard's classical music plugins it will generally get WORK wrong, and for non-classical music, WORK has a different meaning in MB world.
The assumption that I was working on until now, that if the user has WORK/GROUPING tags it's because they wanted them, has turned out to be faulty: some Picard/MB users are discovering for the first time that their collection has these tags.
It's not a problem with browsing works, because the user can switch off Works in the interface prefs.

Changes reverted. Or rather, there still there just commented out...

authors: CDrummond darrell-k

repo: CDrummond/lms-material


text:
Style

Using a 4 spaces indentation, instead of 2
Defining simple styling rules for the future

Bitbucket: https://bitbucket.org/elen4/ceinms/ceinms/issue/7
Originally reported by: Claudio Pizzolato
Originally created at: 2014-06-10T02:24:49.660

FIne with me (even if I prefer two :D ) ... but change tab to space in editors.. two many tabs around :)

Original comment by: theBoss

and remember to avoid trailing whitespaces, git hates them!

Original comment by: elen4

repo: CEINMS/CEINMS


text:
🛑 e-INFRA CZ OpenStack site in Ostrava is down

In beccc49, e-INFRA CZ OpenStack site in Ostrava (https://identity.ostrava.openstack.cloud.e-infra.cz/v3/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: e-INFRA CZ OpenStack site in Ostrava is back up in 8e01b3e.

repo: CERIT-SC/uptime


text:
-#4689 se arreglaron los templates de correos

Se acomodaron los algunos templates en Español y se adecuaron las versiones en ingles correspondientes

Ari, ahi acomodé un poco los templates, no se si ya están bien así o hay que acomodarlos más

Excelente Santi!
Gracias

repo: CICBA/DSpace


text:
Add UI to change Sacct command line

This PR is in response to #15.
In the PR:

add support for -S/starttime and -E/endtime entry for sacct command
change config file to use above command line values (-d and -t flags have been removed)
add UI text showing above values
add UI inputboxes for the above commands (user can make change directly in JobHist panel by pressing the t key)

Extras (might move out to other PRs):

add refresh options by r key (currently only programmed for JobHist panel, can be extended)
add lipgloss styling to filter and time range values (not very well done, I don't have a very good sense of UI aesthetics)

Todos:

[ ] resolve user sacct accounts inconsistency, see below for info (1).

Further details (1):
sacct and sacctmgr have a somewhat subtle difference in behavior if they are called on a multi-cluster SLURM setup. By default the sacct only queries the local SLURM cluster for information, whereas sacctmgr queries (and modifies) all SLURM clusters. This means that when querying for user associated accounts, we get a list of accounts for all clusters, but calls to sacct with these might not match the local cluster.

Hey @hv15 ,
just to let you know i'm back online after the move. Checked out this PR and it looks great so far!
Love the use of the multiple textinputs to manipulate sacct line :rocket:
One small item to keep in mind while developing this feature, all new things shouldn't break the set minimum Width*Height:
https://github.com/CLIP-HPC/SlurmCommander/blob/21cee5ddc47eaad02dbdc37809f38085e194e6bf/internal/model/update.go#L344-L345
so while you're developing things that add lines/columns, be sure to turn on debug and test-resize your terminal to fit the minimum.
Can you just clarify for me the sacct/sacctmgs inconsistency? Not sure i understand?
Besides that, if i can be of any assistance on this, feel free to let me know, otherwise, amazing job!

@pja237  I've made sure that the height constraints aren't overstepped. The PR is 99% complete, the last bit is this issue with sacct/sacctmgr commands, which perhaps is not so much of an issue.
So on our cluster we have several SLURM clusters configured (different systems for different teams) that share the same slurmdb and job accounting. On our setup each SLURM cluster uses the same account associations (mostly), meaning that with the call of sacctmgr in scom to list out all the associate accounts we get duplicate accounts and/or accounts unique to a particular SLURM cluster. Now sacct is a very forgiving command, it neither checks the correctness of some parameters nor does it error out. This means that in most cases, sending a complete list of account associations for all SLURM clusters, even if we're only querying one of these clusters, should still work as we use the same account names between clusters (mostly).
The potential issue is if a user has a unique account association with one of the cluster, and does a query, sacct will return nothing unless we explicit query the cluster where the account association actually exists. There is no error or return code for this as far as I know, so we can't even check for this directly.
I think there are two ways to go about fixing this:

ask the user to select (or configure) the SLURM cluster(s) they want to interact with, and propagate this setting to all SLURM commands (using env SLURM_CLUSTERS for example).
do a complete query, setting the -L or -a flag for SLURM commands, so that we query all clusters (might cause permission errors)

Given the scope, maybe it makes sense to work on this in a separate PR?

Hey @hv15,
that was a massive bit of work you've pulled there, respect!
Agree with everything, few minor wishes from my side and a few bugs creeped in, it's all written above.
If i haven't been precise about something, just let me know and i'll clarify myself.
Besides that, great work! :+1:

Hi, was AFK for the last few days but am back now; I've pushed some new commits which

fixed Waiting for job history... line, this was a UI bug due to me adding an extra line without reducing the table height
fixed filter bug, ooops I had deleted the line when I was still learning the code
reworked key activation/deactivation, we now explicitly enable/disable keys which resolves issues like having t or r appearing in other tabs, or an issue that I ran into on doing shift-tab and then tab causing keys to mysteriously activate or deactivate without reason.
minor UI change to the help window (footer) to have it extend to the entire width of the shell window (could do the same for the main window, but I think one needs to carefully consider the column content).

I've given it a quick spin now and it looks very much polished and ready to roll-out.
Just let me know if it's all finished from your side before i release.
For the future, we should break up the work in smaller chunks, to keep the PRs from growing.
I know there is so much impromptu code inside, and lots of dead ends and anti-patterns and whatnots.
It's been a learning experience also for me using this architecture/framework so i was never quite sure how to best organize the code/data (still not sure in many cases :laughing: )
Knowing myself how tempting it is to stray from the path and start fixing legacy code, because so much could be refactored,  we should do it in smaller increments. Take note of something bad, make issue, fix in small increments i'd propose.
Perhaps keep a discussion thread/issue list/project tasks with the items that should/could be refactored?

Hi @pja237 yes its done now.
I agree, this PR got way to big :rofl:  I'll open up an issue about some of the ideas we discussed, maybe these will become PR :smile_cat:

repo: CLIP-HPC/SlurmCommander


text:
Resultaten pagina

Zullen we deze ook gaan mergen?

er zijn 2 conflicten, laten we die gaan oplossen!!

Ik zie dat de conflicten zijn opgelost! Laten we mergen

authors: YunusEmreCMD ashleyKeuning robbinux

repo: CMD-JOBS/CMD-JOBS


text:
Calling cpp_map_set with a key of "keys" results in corrupted state

Describe the bug
Calling cpp_map(SET "${instance}" keys "${blah}") results in overwriting the set of map keys, and calling cpp_map(SET...) with a different key afterward changes the value of the first mapping. This is due to how keys are stored, as seen here:
https://github.com/CMakePP/CMakePPLang/blob/b19617ebb542f1042699e8d0913f5f75792f397a/cmake/cmakepp_lang/map/map.cmake#L151-L160
A key is stored in the same way that the list of keys are stored, i.e. if the key is keys then it will resolve to the exact same global as the list of keys.
To Reproduce
Steps to reproduce the behavior:

Create a map
Set some mappings
Set a mapping with a key of keys
Set some more mappings
Get the mapping with a key of keys and observe the corrupted behavior

Expected behavior
Setting a mapping should not affect other mappings nor should it overwrite the list of keys
Additional context
This bug was discovered while fixing the tests/map/test_map.cmake test file, the affected section was the one named keys underneath test_cpp_map.
An easy solution is to add additional separators in the globals. For example, the list of keys can remain the same and a particular key could be stored underneath "${_ms_this}_keys_${_ms_key}" so that a key named keys would resolve to ..._keys_keys and therefore not conflict.

@AutonomicPerfectionist so if I'm understanding you correctly the proposal is to change line 155 of your code snippet to cpp_set_global("${_ms_this}_keys_${_ms_key}" "${_ms_value}")?
That sounds good to me. Maybe @zachcran can look into this? It should be pretty quick, but may involve tracking down more places where we access keys.

Correct, it would also involve changing cpp_map_get and cpp_map_append to point to those new globals as well

@AutonomicPerfectionist @ryanmrichard I'll take a look and see if I can get a PR in soon!

repo: CMakePP/CMakePPLang


text:
downloading the library script weighs 3.9mb

I have been using dash plotly for a long time and every application that I created on the first visit it loaded very slowly, regardless of the size of the application, even if it was a single-page or multi-page application, and I noticed that when we visit the application for the first time, third-party library scripts are loaded and they are usually more than 1mb and I think that this process slows down the initial download
in the next photo you can see the process of loading the script of third-party libraries

maybe I'm doing something wrong, I'll be glad to any advice to fix this problem

@sohibjon07012001 I recommend that you enable compress=True in dash.Dash(), to significantly reduce dependent file size, because of the rich functionality in fac, it is difficult to reduce the package size to very small, you can also use CDN to reduce your bandwidth consumption. See: https://github.com/CNFeffery/feffery-antd-components/blob/main/README-en_US.md#3-use-cdn

In my preview version website (http://fac-next.feffery.tech/) of fac 0.2.x version, via use the CDN of unpkg, the size is much smaller than 0.1.16:

[image: image.png]
Thank you for the quick answer after I put Serv_Locally = FALSE in
dash.dash () The script size decreased from 3.9MB to 915KB. I wanted to ask
the advice to the expense of one library Dash_Pivottable I can not reduce
its size, too, although the photo shows that in one of the applications the
size is reduced from 3.5MB to 1MB. I know this is not included in your
duties, but I would be grateful if you help.
пн, 27 мар. 2023 г. в 16:45, Feffery @.***>:

In my preview version website (http://fac-next.feffery.tech/) of fac
0.2.x version, via use the CDN of unpkg, the size is much smaller than
0.1.16:
[image: image]
https://user-images.githubusercontent.com/49147660/227932184-9dc07ec4-de51-45f8-a0ee-ee4c703e7ada.png
—
Reply to this email directly, view it on GitHub
https://github.com/CNFeffery/feffery-antd-components/issues/59#issuecomment-1484997438,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARCYTDQHJN5ZIEU4ECOQT5LW6F4WTANCNFSM6AAAAAAWI4E23Y
.
You are receiving this because you were mentioned.Message ID:
@.***>

did as you said put it in dash.Dash(compress=True) and removed
server_locally=False, but unfortunately loading the pivot_table library was
not required and the feffery_antd_components script became 3.9mb before.
Weird, I don't understand what could be the problem.
вт, 28 мар. 2023 г. в 12:28, Feffery @.***>:

@sohibjon07012001 https://github.com/sohibjon07012001 It's because of
the origin limitation of dash-pivottable, I suggest you to set
compress=True in dash.Dash(), then do not set server_locally, in this
way, related static resources should be compressed on the server based on
the flask-compress library:
[image: image]
https://user-images.githubusercontent.com/49147660/228160845-ff744306-851a-4b5c-93f0-dca2172551e1.png
—
Reply to this email directly, view it on GitHub
https://github.com/CNFeffery/feffery-antd-components/issues/59#issuecomment-1486352591,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARCYTDQHWHGMLG2HDY2OCJ3W6KHLVANCNFSM6AAAAAAWI4E23Y
.
You are receiving this because you were mentioned.Message ID:
@.***>

did as you said put it in dash.Dash(compress=True) and removed server_locally=False, but unfortunately loading the pivot_table library was not required and the feffery_antd_components script became 3.9mb before. Weird, I don't understand what could be the problem. вт, 28 мар. 2023 г. в 12:28, Feffery @.>:
…
@sohibjon07012001 https://github.com/sohibjon07012001 It's because of the origin limitation of dash-pivottable, I suggest you to set compress=True in dash.Dash(), then do not set server_locally, in this way, related static resources should be compressed on the server based on the flask-compress library: [image: image] https://user-images.githubusercontent.com/49147660/228160845-ff744306-851a-4b5c-93f0-dca2172551e1.png — Reply to this email directly, view it on GitHub <#59 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ARCYTDQHWHGMLG2HDY2OCJ3W6KHLVANCNFSM6AAAAAAWI4E23Y . You are receiving this because you were mentioned.Message ID: @.>

Here is a simple demo:
import dash
from dash import html
import feffery_antd_components as fac
import dash_pivottable
from dash.dependencies import Input, Output

app = dash.Dash(__name__, compress=True)

app.layout = html.Div(
    [
        fac.AntdAlert(
            message='This is a demo app.',
            showIcon=True
        )
    ]
)

if __name__ == '__main__':
    app.run(debug=False)

repo: CNFeffery/feffery-antd-components


text:
Collaboration service frontend

Finish the collabroation service frontend

your frontend editor can be implemented as part of the nice to haves (better code editor)


text:
React Initial Setup

Setup the React project with a skeleton

Completed


text:
Update the VS Code extension format

Describe the bug
VSCode has changed the way that extensions get loaded in the .devcontainers/devcontainer.json file.
To Reproduce
Steps to reproduce the behavior:

Open this project in VS Code and reopen it using container.
There will be no extensions loaded in the extension tab on the left.

Expected behavior
The specified extensions should get loaded as per the .devcontainers/devcontainer.json file.
Observed behavior
No extensions are loaded in the project.
Additional context
The solution is available at https://github.com/nyu-devops/project-template/blob/master/.devcontainer/devcontainer.json

Approved and merged. Moving this ticket to done.

authors: reubenabraham sakshamgarg


text:
Implement Add Shopcart Item API logic

Shopcart Item CRUD related tests require the Add Shopcart Item API to be ready in route.py, hence I implemented the logic first. Test cases in test_route.py and models.py are yet to be added.

Once PR #35 is merged to master, change PR base to master.


text:
[BE] FEAT: 댓글 기능 구현

작업 설명
꿈일기 피드와 게시글에서 댓글, 답글을 등록, 조회하는 기능을 구현하겠습니다.
접근 방법
댓글 조회

유저가 서버의 /api/comments/:filter-type/:id 엔드포인트로 접근한다.
filter_type으로 꿈일기 피드인지, 기타 게시글인지 판별한다.
게시글 타입 판별 후 dream_diary OR board 테이블에서 GET으로 받아온 id값에 맞는 글의 댓글들을 가져온다.

댓글 달기

작성란에 댓글을 작성 완료할 시 댓글에 해당하는 정보를 DB에 저장한다.

답글 달기

POST를 통해 매개변수로 넘어온 답글의 comment_id를 추출하여 parent_comment_id에 저장한다.
그 외 사항은 우 댓글 달기 내용과 동일.

대체 방안 (선택 사항)
comment table에 body가 없어서 추가해야할 것 같습니다!

comment table에 body가 없어서 추가해야할 것 같습니다!

헐 그러네요..?!
구현하실 때 DDL에 직접 추가해주시면 감사하겠습니다!
필드명은 다른 테이블과의 통일성을 위해서 content로 하는건 어떨까요?

넵 좋습니다!

추가로 댓글 삭제시에 Request가 /api/comments/:filter-type/:id로 되어있는데
/api/comments/:filter-type/:id/:comment-id로 comment-id도 함께 넘겨주어야 삭제가 가능할 것 같습니다.

추가로 댓글 삭제시에 Request가 /api/comments/:filter-type/:id로 되어있는데 /api/comments/:filter-type/:id/:comment-id로 comment-id도 함께 넘겨주어야 삭제가 가능할 것 같습니다.

엇 이걸 이제 봤네요..? 알겠습니다!!


text:
the number of frames in label and acoustic features are different: 1285 vs 1276 (Casting_Correct_0080)

How could i handle such error when i use magphase with merlin using constant frame rate ??

The wav has 103120 sample
which by sampling rate 16000 will be 6.445 second
and since frame shift is 0.005
so the wav must have 1289 frame
when i run the feature extractor , the output number of frames become 973 and by interpolation it become 1276 , so how could it be like the exact number of frames of the wav ?

Hi,
Thank you for posting. I have some questions:

Which data are you using?
Are you using the slt_arctic/s2 recipe?
Could you send me the example pair (wav and corresponding lab file), please?

Custom dataset
yes , i am using slt_arctic/s2 recipe
Sure , i will , here it is :
https://drive.google.com/drive/folders/1nJtKWbufEIUEY93H-zxRMH7uwPr8Psjh?usp=sharing

Also there are some waves that  repear produces empty file so it doesn't continue getting features for it , i don't know if it's a bug or what ?

I also faced this problem. I use the LJSpeech dataset and constant frame rate:
CRITICALListDataProvider: the number of frames in label and acoustic features are different: 1211 vs 1182 (LJ010-0079)

Have you solved the problem?I meet it too

I just commited a fixing for the inconsistency of number of frames given by REAPER (ec6ae28). That may solve the problem.

authors: 1271086950 ZhaoZeqing felipeespic rmmal

repo: CSTR-Edinburgh/magphase


text:
Extracting MGC parameter

Dear all,
Regarding to the WORLD vocoder, does the Merlin toolkit use to extract the MGC parameter by using "CheapTrick" spectral estimation algorithm?
Best regards,

yes, https://github.com/CSTR-Edinburgh/merlin/blob/master/tools/WORLD/test/analysis.cpp#L139

Thank you very much ...

repo: CSTR-Edinburgh/merlin


text:
add redirection to /administrator/

I find it confusing that when you type https://czertainly.domain/ you get an error saying just:
Error

no Route matched with those values.

This simple patch makes use of an nginx ingress configuration snippet to redirect the user to the right place https://czertainly.domain/administrator/ The reason why it is by default disabled is that snippet-annotations are also by default disabled in nginx ingress. According to docs they might introduce security risk on the cluster.

Looks like it affects certificate passing...

repo: CZERTAINLY/CZERTAINLY-Helm-Charts


text:
PyQt6 support?

I tried editing __init__.py mimicking  PyQt5 , copy-pasting and changing to PyQt6 where relevant, but running the aiohttp example using PyQt6.QtWidgets ends up with this:
Traceback (most recent call last):
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/asyncio/selector_events.py", line 261, in _add_reader
    key = self._selector.get_key(fd)
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/selectors.py", line 193, in get_key
    raise KeyError("{!r} is not registered".format(fileobj)) from None
KeyError: '6 is not registered'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/willyjl/Coding/spicetify-easyinstall/test.py", line 95, in <module>
    qasync.run(main())
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/site-packages/qasync/__init__.py", line 797, in run
    return asyncio.run(*args, **kwargs)
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/asyncio/runners.py", line 39, in run
    loop = events.new_event_loop()
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/asyncio/events.py", line 761, in new_event_loop
    return get_event_loop_policy().new_event_loop()
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/site-packages/qasync/__init__.py", line 775, in new_event_loop
    return QEventLoop(QApplication(sys.argv))
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/site-packages/qasync/__init__.py", line 335, in __init__
    super().__init__()
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/site-packages/qasync/_unix.py", line 198, in __init__
    asyncio.SelectorEventLoop.__init__(self, selector)
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/asyncio/unix_events.py", line 54, in __init__
    super().__init__(selector)
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/asyncio/selector_events.py", line 61, in __init__
    self._make_self_pipe()
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/asyncio/selector_events.py", line 112, in _make_self_pipe
    self._add_reader(self._ssock.fileno(), self._read_from_self)
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/asyncio/selector_events.py", line 263, in _add_reader
    self._selector.register(fd, selectors.EVENT_READ,
  File "/home/willyjl/.pyenv/versions/3.9.6/lib/python3.9/site-packages/qasync/_unix.py", line 112, in register
    notifier = QtCore.QSocketNotifier(key.fd, QtCore.QSocketNotifier.Read)
AttributeError: type object 'QSocketNotifier' has no attribute 'Read'
sys:1: RuntimeWarning: coroutine 'main' was never awaited

Is PyQt6 support planned? Am I missing something?

PyQt6 isn't fully supported yet, sorry. Some changes were made for PyQt6 but it's incomplete. I can't really say when/if I'll have time to finish it.

No worries, and thanks for even making this in the first place :D

I have created a PR #53 that supports PyQt6, hopefully it will be accepted soon.

@eyllanesc
I have installed your version but I have this error with from __feature__ import snake_case.
AttributeError: '_SimpleTimer' object has no attribute 'startTimer'
ERROR:qasync._QEventLoop:Task was destroyed but it is pending!

@sharpordie The explanation is simple: This library does not support the snake_case feature.

Should README.md be updated to reflect this change? It specifically states "PyQt5 or PySide2".

authors: TheGreatCabbage Willy-JL brianbienvenu eyllanesc sharpordie

repo: CabbageDevelopment/qasync


text:
Http url output support

Based off of Joey Parrish's HTTP Upload commit
Feature

Allow users to specify a HTTP url as an output
Segments and manifests will be uploaded to the specified url via HTTP PUT
Urls ending with a '/' will have the slash removed to prevent issues with Google Cloud

Testing

Added unit tests to check for config compatibility errors, such as specifying an HTTP url output with a "multi_period_input", a cloud URL, and with "segment_per_file" set to false

Shortcomings

The multi period input list feature is structured to use the local file system and does not easily support HTTP uploads. I will take a closer look to scope out the changes needed to support HTTP output and multi period input lists. Depending on the scope, I will add the changes to this PR or create a separate one addressing this issue.

Merged into Google's main branch! 🥳

repo: CaitlinOCallaghan/shaka-streamer


text:
Clear admin form after submission

currently add admin form doesn't clear name and email after submitted

done in #144

repo: CalNourish/pantry-webapp


text:
How to visualize generated data files

How to visualize generated data files，I hope you can provide some suggestions.

It depends on how big your data is. If you're looking at a small/medium amount of sequences, then python might work.
For larger data, we tend to use Java TreeView. It's old software, but it works well for visualizing large heatmaps.

authors: CK-X Jessime

repo: CalabreseLab/seekr


text:
alephone-marathon: add version 20230529

Added Aleph One (Marathon 1) package version 20230529.
No persistable data; user data is saved to %USERPROFILE%/Documents/Aleph One
Closes #987

[x] I have read the Contributing Guide.

/verify

Superseded by https://github.com/Calinou/scoop-games/pull/992.

authors: Calinou GriekseEi

repo: Calinou/scoop-games


text:
Instructions for adding APP ID /Credentials are missing in documentation README

Describe the bug
The README needs to provide instructions on how to setup AppID credentials for the TakeTwo
To Reproduce
There is no specific steps to reproduce. README is missing the required documentation on how to fill 3 fields
CLIENT_ID=
SECRET=
OAUTH_SERVER_URL=
Expected behavior
N/A
Screenshots
Attached is the section of the  README that needs updated documentation
Copy .env.sample file to .env file in the same directory. Fill out the CouchDB database and IBM App Id service credentials.
#CouchDB Credentials
DB_HOST=
DB_PORT=
DB_NAME=
DB_USERNAME=
DB_PASSWORD=
#IBM AppID Credentials
CLIENT_ID=
SECRET=
OAUTH_SERVER_URL=
Desktop (please complete the following information):
N/A
Smartphone (please complete the following information):
N/A
Additional context

Hi @shobhavijaystar thanks for this. I will add in some more information about how to get started with AppID on IBM cloud. We do want to make this part optional (#15) but it would still be good for people that want to use AppID.

@josiemundi I am not sure I understood the line "it would still be good for people that want to use AppID." I tried the steps I get an error on reading of DB_PORT as "ValueError: invalid literal for int() with base 10: 'None'". I have given the port number as 5984. I suspect this is not the right place and move it to Slack channel? Are you saying we really do not need this AppID section?

@shobhavijaystar are you trying to deploy the API. You have your env file but it's not working. The part you are referring to is for couch db not AppID and should look as follows:

Right now you need to have AppID on IBM cloud to get the demo working. Do you have AppID credentials? If not it won't work. We are working this next sprint to remove this dependency.
Hope this helps. Please reach out on Slack if you need any other help getting this up and running :)

reopening - not yet complete.

authors: demilolu josiemundi shobhavijaystar


text:
In Ch2_MorePyMC_PyMC3 Ln[11], running "lambda_1.random()[0]", got "IndexError: too many indices for array"

The codes in Ln[11] are：

%matplotlib inline
from IPython.core.pylabtools import figsize
import matplotlib.pyplot as plt
import scipy.stats as stats
figsize(12.5, 4)
samples = [lambda_1.random()[0] for i in range(20000)]
plt.hist(samples, bins=70, normed=True, histtype="stepfilled")
plt.title("Prior distribution for $\lambda_1$")
plt.xlim(0, 8);

And the program stopped at:

samples = [lambda_1.random()[0] for i in range(20000)]

with error:

IndexError: too many indices for array

So I tried to change it as:

samples= [lambda_1.random() for i in range(20000)]

and it worked.

I removed the [0] indexing and it worked. It looks like lambda_1.random() is a scalar and not an array.
samples = [lambda_1.random() for i in range(20000)]

as of numpy 1.18.5, you may need something like this:
samples = np.array([lambda_1.random() for i in np.arange(20000)]).reshape(-1,20000)[0]

authors: JamesCHub WillKoehrsen gaoleihlj


text:
loading and saving models with dill package

I'm not sure about workaround for saving and deleting permissions, so test could failed for that reason. Needed dill package for saving, loading and for test.

Changed requirements to include dill package by default

For tests to pass, we also need to update .tavis.yml line 55

Yeah, all checks passed for the first time.

Thanks for the contribution!

authors: CamDavidsonPilon aprotopopov

repo: CamDavidsonPilon/lifetimes


text:
Module 1.1 Assignment

Given what you have learned about Git, why would you want to use Git in your classroom?

To use industry-standard source control


text:
[Feat] - Gradle cleanup

There are some small unnecessary or deprecated thing in project's gradle files like:

Since Kotlin 1.4.0, there's no need to add standard library as a dependency
Since AGP 4.1 compileSdkVersion, targetSdkVersion and minSdkVersion are deprecated in favor of shorter properties of compileSdk, targetSdk and minSdk
Since AGP 4.2 java 1.8 support is enabled by default. We could mention this in ReadMe.

See original article for more info. I have these changes ready in my fork and could create a pull request if you are OK with it.

Hey @Alireza-Farahani thanks for raising this, please be free to create a PR so we can improve this ^^
We just need to be sure we keep backward compatibility.

authors: Alireza-Farahani Canato

repo: CanHub/Android-Image-Cropper


text:
[1.16]Flower Pot cause rankine plants dupe

Flower pot should become empty when click, however, ones with rankine plant does not, and gives one more plant each click.

Fixed as of the most recent hotfix (Hotfix 6).

authors: CannoliCatfish khjxiaogu

repo: CannoliCatfish/project-rankine


text:
adapts, demagogism

This pull request was generated by the 'mq' tool

/trunk merge

repo: Canva/mergequeue-trunk


text:
frosts, barndoor

This pull request was generated by the 'mq' tool

/trunk merge


text:
TESTING: Tejas/bp 141 poc for admin batch upload

@nohil2 Please review the db-gen.sql changes. They were required to get deletes to cascade as expected. The DEFERRABLE INITIALLY DEFERRED allows the constraint to be checked at the end of the transaction as opposed to end of statement/end of row
http://dbadailystuff.com/deferred-constraints-in-postgresql

@AaronCs no need to make a pr for BP-178 for the throwingOrder update, I've merged that branch


text:
👨💻 Member 1: Devise + Navigation + Recipes

👨💻 Member 1: Devise + Navigation + Recipes
Associated cards are #7, #9 & #10

[x] No linter error
[x] N+1 problem solved.
[x] Working navigation
[x] Working Authentication system with Devise
[x] Working Authorization with CanCanCan

#7

repo: CaptainBawa/Recipe-App


text:
🛑 Piglands is down

In 453c406, Piglands (https://piglands.ml/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Piglands is back up in ca8cb2f.

repo: CaptainQWasTaken/funkyducks-status


text:
封面问题

（请务必按照issue模板，修改后提交问题）
（ps 首页问题集锦和demo请先了解一下！）
问题描述：
看了问题锦集，
我目前 封面是有一个URL地址，视频是由一个url地址，但是我看到demo里面的
SampleCoverVideo是
封面和视频都是一个地址；
我想分开写
我应该如何修改
public void loadCoverImage(String url, int res) {
mCoverOriginUrl = url;
mDefaultRes = res;
Glide.with(getContext().getApplicationContext())
.setDefaultRequestOptions(
new RequestOptions()
.frame(1000000)
.centerCrop()
.error(res)
.placeholder(res))
.load(url)
.into(mCoverImage);
}
没有找到单独loadVideoURL的类似方法

既然是demo里的，你改一改方法呗，把loadCoverImage(xxx,xxx,xxx)不就好了

authors: CarGuo xyZangGit

repo: CarGuo/GSYVideoPlayer


text:
循环播放，就是播放完成以后继续重头播放，setLooping(true) 不起作用。

（！！！！请务必务必按照issue模板，修改 后提交问题！！！！，不按模板提Isuue删除处理）
（ps 首页问题集锦和demo请先了解一下！）
问题描述：
XXXX发生了这样的问题！
问题机型/系统：
例如 真机 小米note2 Android7.0
GSY依赖版本
例如 implementation 'com.shuyu:gsyVideoPlayer-java:7.0.1'
Demo中的复现步骤
Demo中的detailPlayer页面打开后XXXX会出现这个问题。
问题代码：(如果有)
xxxxxxx

问题log（如果有）

xxxxxxx

请修改模版，并标出 Demo 哪个页面复现

请修改模版，并标出 Demo 哪个页面复现，提供复现流数据

是重新播放，就是一个视频播放完成了。我想重新播放，是设置setLooping吗？

不是你理解的列表循环，是一个单个视频重新播放。

authors: CarGuo smilexuetao


text:
播放flv视频报错

无法播放flv视频：https://124.220.20.17:13016/nvr/video/proxy/ydaj0001/a85b5686-7b88-4ee6-af0f-09a1b909ff81.flv
所有手机
引用版本：    implementation 'com.shuyu:gsyVideoPlayer-java:8.1.2'
错误日志：
2023-03-06 13:03:14.225 15093-16489/com.kejin.hyt E/ExoPlayerImplInternal: Playback error
com.google.android.exoplayer2.ExoPlaybackException: Source error
at com.google.android.exoplayer2.ExoPlayerImplInternal.handleMessage(ExoPlayerImplInternal.java:579)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:202)
at android.os.HandlerThread.run(HandlerThread.java:65)
Caused by: com.google.android.exoplayer2.upstream.HttpDataSource$HttpDataSourceException: Unable to connect
at com.google.android.exoplayer2.upstream.DefaultHttpDataSource.open(DefaultHttpDataSource.java:356)
at com.google.android.exoplayer2.upstream.DefaultDataSource.open(DefaultDataSource.java:201)
at com.google.android.exoplayer2.upstream.StatsDataSource.open(StatsDataSource.java:84)
at com.google.android.exoplayer2.source.ProgressiveMediaPeriod$ExtractingLoadable.load(ProgressiveMediaPeriod.java:1015)
at com.google.android.exoplayer2.upstream.Loader$LoadTask.run(Loader.java:415)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
at java.lang.Thread.run(Thread.java:764)
Caused by: javax.net.ssl.SSLHandshakeException: java.security.cert.CertPathValidatorException: Trust anchor for certification path not found.
at com.android.org.conscrypt.ConscryptFileDescriptorSocket.startHandshake(ConscryptFileDescriptorSocket.java:219)
at com.android.okhttp.internal.io.RealConnection.connectTls(RealConnection.java:192)
at com.android.okhttp.internal.io.RealConnection.connectSocket(RealConnection.java:149)
at com.android.okhttp.internal.io.RealConnection.connect(RealConnection.java:112)
at com.android.okhttp.internal.http.StreamAllocation.findConnection(StreamAllocation.java:184)
at com.android.okhttp.internal.http.StreamAllocation.findHealthyConnection(StreamAllocation.java:126)
at com.android.okhttp.internal.http.StreamAllocation.newStream(StreamAllocation.java:95)
at com.android.okhttp.internal.http.HttpEngine.connect(HttpEngine.java:299)
at com.android.okhttp.internal.http.HttpEngine.sendRequest(HttpEngine.java:237)
at com.android.okhttp.internal.huc.HttpURLConnectionImpl.execute(HttpURLConnectionImpl.java:461)
at com.android.okhttp.internal.huc.HttpURLConnectionImpl.connect(HttpURLConnectionImpl.java:127)
at com.android.okhttp.internal.huc.DelegatingHttpsURLConnection.connect(DelegatingHttpsURLConnection.java:89)
at com.android.okhttp.internal.huc.HttpsURLConnectionImpl.connect(Unknown Source:0)
at com.google.android.exoplayer2.upstream.DefaultHttpDataSource.makeConnection(DefaultHttpDataSource.java:641)
at com.google.android.exoplayer2.upstream.DefaultHttpDataSource.makeConnection(DefaultHttpDataSource.java:543)
at com.google.android.exoplayer2.upstream.DefaultHttpDataSource.open(DefaultHttpDataSource.java:349)
at com.google.android.exoplayer2.upstream.DefaultDataSource.open(DefaultDataSource.java:201)
at com.google.android.exoplayer2.upstream.StatsDataSource.open(StatsDataSource.java:84)
at com.google.android.exoplayer2.source.ProgressiveMediaPeriod$ExtractingLoadable.load(ProgressiveMediaPeriod.java:1015)
at com.google.android.exoplayer2.upstream.Loader$LoadTask.run(Loader.java:415)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
at java.lang.Thread.run(Thread.java:764)
Caused by: java.security.cert.CertificateException: java.security.cert.CertPathValidatorException: Trust anchor for certification path not found.
at com.android.org.conscrypt.TrustManagerImpl.verifyChain(TrustManagerImpl.java:661)
at com.android.org.conscrypt.TrustManagerImpl.checkTrustedRecursive(TrustManagerImpl.java:539)
at com.android.org.conscrypt.TrustManagerImpl.checkTrusted(TrustManagerImpl.java:495)
at com.android.org.conscrypt.TrustManagerImpl.checkTrusted(TrustManagerImpl.java:418)
at com.android.org.conscrypt.TrustManagerImpl.getTrustedChainForServer(TrustManagerImpl.java:339)
at android.security.net.config.NetworkSecurityTrustManager.checkServerTrusted(NetworkSecurityTrustManager.java:94)
at android.security.net.config.RootTrustManager.checkServerTrusted(RootTrustManager.java:88)
at com.android.org.conscrypt.Platform.checkServerTrusted(Platform.java:197)
at com.android.org.conscrypt.ConscryptFileDescriptorSocket.verifyCertificateChain(ConscryptFileDescriptorSocket.java:399)
2023-03-06 13:03:14.236 15093-15093/com.kejin.hyt E/EventLogger: playerFailed [3.54]
com.google.android.exoplayer2.ExoPlaybackException: Source error
at com.google.android.exoplayer2.ExoPlayerImplInternal.handleMessage(ExoPlayerImplInternal.java:579)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:202)
at android.os.HandlerThread.run(HandlerThread.java:65)
Caused by: com.google.android.exoplayer2.upstream.HttpDataSource$HttpDataSourceException: Unable to connect
at com.google.android.exoplayer2.upstream.DefaultHttpDataSource.open(DefaultHttpDataSource.java:356)
at com.google.android.exoplayer2.upstream.DefaultDataSource.open(DefaultDataSource.java:201)
at com.google.android.exoplayer2.upstream.StatsDataSource.open(StatsDataSource.java:84)
at com.google.android.exoplayer2.source.ProgressiveMediaPeriod$ExtractingLoadable.load(ProgressiveMediaPeriod.java:1015)
at com.google.android.exoplayer2.upstream.Loader$LoadTask.run(Loader.java:415)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
at java.lang.Thread.run(Thread.java:764)
Caused by: javax.net.ssl.SSLHandshakeException: java.security.cert.CertPathValidatorException: Trust anchor for certification path not found.
at com.android.org.conscrypt.ConscryptFileDescriptorSocket.startHandshake(ConscryptFileDescriptorSocket.java:219)
at com.android.okhttp.internal.io.RealConnection.connectTls(RealConnection.java:192)
at com.android.okhttp.internal.io.RealConnection.connectSocket(RealConnection.java:149)
at com.android.okhttp.internal.io.RealConnection.connect(RealConnection.java:112)
at com.android.okhttp.internal.http.StreamAllocation.findConnection(StreamAllocation.java:184)
at com.android.okhttp.internal.http.StreamAllocation.findHealthyConnection(StreamAllocation.java:126)
at com.android.okhttp.internal.http.StreamAllocation.newStream(StreamAllocation.java:95)
at com.android.okhttp.internal.http.HttpEngine.connect(HttpEngine.java:299)
at com.android.okhttp.internal.http.HttpEngine.sendRequest(HttpEngine.java:237)
at com.android.okhttp.internal.huc.HttpURLConnectionImpl.execute(HttpURLConnectionImpl.java:461)
at com.android.okhttp.internal.huc.HttpURLConnectionImpl.connect(HttpURLConnectionImpl.java:127)
at com.android.okhttp.internal.huc.DelegatingHttpsURLConnection.connect(DelegatingHttpsURLConnection.java:89)
at com.android.okhttp.internal.huc.HttpsURLConnectionImpl.connect(Unknown Source:0)
at com.google.android.exoplayer2.upstream.DefaultHttpDataSource.makeConnection(DefaultHttpDataSource.java:641)
at com.google.android.exoplayer2.upstream.DefaultHttpDataSource.makeConnection(DefaultHttpDataSource.java:543)
at com.google.android.exoplayer2.upstream.DefaultHttpDataSource.open(DefaultHttpDataSource.java:349)
at com.google.android.exoplayer2.upstream.DefaultDataSource.open(DefaultDataSource.java:201)
at com.google.android.exoplayer2.upstream.StatsDataSource.open(StatsDataSource.java:84)
at com.google.android.exoplayer2.source.ProgressiveMediaPeriod$ExtractingLoadable.load(ProgressiveMediaPeriod.java:1015)
at com.google.android.exoplayer2.upstream.Loader$LoadTask.run(Loader.java:415)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
at java.lang.Thread.run(Thread.java:764)
Caused by: java.security.cert.CertificateException: java.security.cert.CertPathValidatorException: Trust anchor for certification path not found.
at com.android.org.conscrypt.TrustManagerImpl.verifyChain(TrustManagerImpl.java:661)
at com.android.org.conscrypt.TrustManagerImpl.checkTrustedRecursive(TrustManagerImpl.java:539)
at com.android.org.conscrypt.TrustManagerImpl.checkTrusted(TrustManagerImpl.java:495)
at com.android.org.conscrypt.TrustManagerImpl.checkTrusted(TrustManagerImpl.java:418)
at com.android.org.conscrypt.TrustManagerImpl.getTrustedChainForServer(TrustManagerImpl.java:339)
at android.security.net.config.NetworkSecurityTrustManager.checkServerTrusted(NetworkSecurityTrustManager.java:94)
at android.security.net.config.RootTrustManager.checkServerTrusted(RootTrustManager.java:88)
at com.android.org.conscrypt.Platform.checkServerTrusted(Platform.java:197)
at com.android.org.conscrypt.ConscryptFileDescriptorSocket.verifyCertificateChain(ConscryptFileDescriptorSocket.java:399)
at com.android.org.conscrypt.NativeCrypto.SSL_do_handshake(Native Method)
at com.android.org.conscrypt.SslWrapper.doHandshake(SslWrapper.java:374)
2023-03-06 13:03:14.236 15093-15093/com.kejin.hyt E/EventLogger:     at com.android.org.conscrypt.ConscryptFileDescriptorSocket.startHandshake(ConscryptFileDescriptorSocket.java:217)
... 22 more
Caused by: java.security.cert.CertPathValidatorException: Trust anchor for certification path not found.
... 34 more

Caused by: java.security.cert.CertificateException: java.security.cert.CertPathValidatorException: Trust anchor for certification path not found. 证书有问题，参考
https://github.com/CarGuo/GSYVideoPlayer/blob/afb74b0216797f95233aaffe06d1e702c838b4fd/app/src/main/java/com/example/gsyvideoplayer/GSYApplication.java#L79

authors: CarGuo caiyaoyao


text:
Implement operations on IDS profiles

This issue keeps track of the type of operations to be applied to the IDS profile.
These operations are applied in the create step.
Simple operations

[x] Simple numpy arithmetic operations

[x] Multiplication, i.e. (e.g. x * [0.75 , 0.95 , 1.05 , 1.25])  -> #63
[x] Addition, (e.g. for Zeff: x + [0.2,0.4,0.6,0.8]) -> #63

Note that a user needs information on absolute values of profiles for this to make sense

[ ] Linear displacement: keep one pivot point fixed and then modify everything inside of that linearly such that at rho=0 the maximum displacement is achieved

Operations based on error quantities

[ ] Addition/subtraction by factors of sigma (e.g. x + [0.25sigma, 0.5sigma, 1sigma, 2sigma])
[ ] Allow for symmetric and asymmetric errors
[ ] Sample based on error distribution, e.g. gaussian distribution sampling, must be able to set total number of runs (=number of samples to take) -> #145

Sampling methodology
To think about: sampling methodology. Especially if multiple profiles are being modified.

[x] Simple full lattice (Cartesian product, as currently implemented in matrix)
[x] Smarter samplers -> #71

[x] Latin Hypercube
[x] Low discrepancy sequences (Sobol)
[x] Halton sampler

Hi @aaronkho @jcitrin ,
I'm implementing sampling a distribution around the IDS using the error bars as the bounds. I wanted to check what you meant by 'asymmetric' sampling, i.e. if the upper and lower bounds are different. Seems to me like you would get a very 'disjoint' distribution by using this, so I'm wondering if this is the intent.
This post on SO illustrates well what I mean: https://stackoverflow.com/a/53048572

Relevant function in scipy? -> scipy.stats.skewnorm

The scipy skewnorm seems like exactly what we want here. However, I am
unsure how to convert the asymmetric errors into the "a" parameter needed
by the function. In this case, we can interpret the upper and lower error
values as the 16th and 84th percentiles, respectively.
On Tue., Jul. 5, 2022, 2:34 p.m. Stef Smeets, @.***>
wrote:

Relevant function in scipy? -> scipy.stats.skewnorm
https://docs.scipy.org/doc/scipy/reference/generated/scipy.stats.skewnorm.html
—
Reply to this email directly, view it on GitHub
https://github.com/CarbonCollective/fusion-dUQtools/issues/58#issuecomment-1175007936,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFTJRLJTRPBJVOP5K2NVO33VSQTW7ANCNFSM5YNUK4XQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

Yeah, exactly, for the symmetric error I have been assuming that the error describes the sigma of the data.
For asymmetric distributions, this is what the data model documentation has to say about the error values:

By convention, only the upper error node should be filled in case of symmetrical error bars. The upper and lower errors are absolute and defined positive, the effective values of the data will be within the interval [data-data_error_lower, data+data_error_upper]. Thus whatever the sign of data, data_error_lower relates to the lower bound and data_error_upper to the upper bound of the error bar interval.

The data I have worked with so far does not seem to have any ids with *_error_lower. So we can pick up on this later.
In any case, it would be good to know how the asymmetric error description is defined. And also what the the data represents in terms of mode, mean, or median.

How to interpret lower/upper:

Value itself is the mean
error_upper -> $1\sigma^+$, (~84th percentile)
error_lower -> $1\sigma^-$, (~16th percentile)
How to calculate a?

Some feedback from @aaronkho , seems like we implemented the IDS sampler in the wrong way, i.e. no double sampling.
For now: we should sample the whole range:

from -3 sigma to +3 sigma with 10 steps
support lists of numbers to multiply with sigma, [-2, -1, 0, 1, 2] * sigma

For later: we should support sampling discrete and continuous distributions:

Currently we support sampling a discrete distribution (i.e. list)
Latin-Hypercube should be able to sample a distribution
Distribution sampling should be weighted?
Rename matrix -> dimensions
Allow dimensions to be excluded from latin-hypercube sampling

Most of the work in this issue has been completed. I converted the remaining open item to a new issue: https://github.com/CarbonCollective/fusion-dUQtools/issues/307

authors: aaronkho stefsmeets

repo: CarbonCollective/fusion-dUQtools


text:
On-demand Push

It'd be nice to have an on-demand service for pushing from gateways and servers to Git servers before the scheduled time. It'd be nicer to be able to specify the repo to push. :-)

Implemented.

repo: CareyLabVT/SmartConnectedCommunities


text:
Pgcopy stream metrics

In order to measure the behavior of the copy endpoints (following https://github.com/CartoDB/cartodb-platform/issues/4414)
Things that are interesting to be measured:

query time,
size of payload,
format (text, csv, or binary)

Reviewers, take into account ---> https://github.com/CartoDB/cartodb-platform/issues/4414#issuecomment-390932252

repo: CartoDB/CartoDB-SQL-API


text:
Don't assume sh/bash on ./scripts/check-node-canvas.sh

For instance some distros uses an alias for sh poiting to dash.

cc @viddo

repo: CartoDB/Windshaft


text:
Load existing analysis from viz.json

Related to CartoDB/cartodb#7034.

Fixed in #1227.


text:
fixes bootstrap compatibilites, closes #87, #107

cc @javierarce

Caniuse reports that there are some issues with Android browsers (see "Known issues"), so I wouldn't recommend have a look. Other than that, :+1:

ok

@javierarce test on android and it works ok

:crossed_flags:

authors: javierarce javisantana


text:
Fix broken tests after merging with master

At least currently some failures like this one:
Failures:

  1) Api::Json::VisualizationsController behaves like visualization controllers main behaviour tests exclude_shared and only_shared filters
     Failure/Error: private_tables_enabled: true

       expected: true
            got: false

       (compared using ==)
     Shared Example Group: "visualization controllers" called from ./spec/requests/api/json/visualizations_controller_spec.rb:8
     # ./spec/support/factories/users.rb:68:in `create_user'
     # ./spec/requests/api/json/visualizations_controller_shared_examples.rb:105:in `block (3 levels) in <top (required)>'

The problem seems to be that the user is already created by a previous stage:
(rdb:1) user.valid?
false
(rdb:1) user.errors
{:email=>["is already taken"]}

hard to control as far as we have duplicate code (just check git grep 'client@example.com')

the user in section before(:all) was colliding with the one created in the include_context 'visualization creation helpers'

repo: CartoDB/cartodb


text:
Map on New edit metadata modal doesn't work with small polygons datasets

It never renders anthing to me, plus there is a white gradient overlapping it.

Try with https://team.cartodb.com/u/saleiva/tables/sf_building_footprints_1/map

The visual bug is already fixed (not deployed yet though), the other was due to a bug in cartodb.js, fixed in https://github.com/CartoDB/cartodb.js/pull/610, will have to update cartodb once it's merge there to fix this.

w/ fix :+1:

<3

authors: saleiva viddo


text:
Map not created after importing dataset from synced URL dataset

This problem is reproduced:

From maps view in dashboard click "New map"
Click "Connect dataset"
Import from URL (for example: http://www.naturalearthdata.com/http//www.naturalearthdata.com/download/110m/physical/ne_110m_coastline.zip)
Choose to sync every hour

The dataset is imported but the map is not created.
cc @xavijam

I feel this is due to a problem with the synchronization model. It doesn't allow to speficy if you want to create a vis or not, right @kartones?

It should work: https://github.com/CartoDB/cartodb/blob/master/app/controllers/api/json/synchronizations_controller.rb#L189
But please, provide more details, like:

Have you checked what payload is sent from the frontend to the backend? (if there is no create_vis param then might be indeed FE-related)
If create_vis is sent, have you tried debugging if the backend ignores the field when creating a synchronization?

PS: Might be worth to add to the imports/syncs log a line like... Will create visualization?true/false

This are the parameters sent to the backend (what I asked for details and didn't got an answer, anyway it's a minute to check):
{
   content_guessing: true
   interval: 3600
   type_guessing: true
   url: "http://..."
}

Plus what I said before about the backend supporting it this means that it really is a FE "issue".
By definition a Sync table is a dataset, and datasets do not have visualizations. Now we've added this option of creating a vis, but as they are created as new datasets, combined with my lack of Backbone knowledge I don't see where to plug the create_vis: true if it is a sync and I want a map... But I'll check where and play/learn a bit of Backbone

Branch 5806-create-sync-map-from-maps-page

authors: Kartones gfiorav xavijam


text:
communication with robot

research how to make robot communicate with PC, so it can recieve wheel commands and send sensor-data:

ROS2
MQTT
ros-js
....

So, I ended with MQTT:

install mosquitto broker
$ sudo apt-get install mosquitto
edit its configuration (/etc/mosquitto/conf.d/mosquitto.conf), so it will be possible to connect from remote hosts (robots, etc.)

allow_anonymous true
listener 1883

restart mosquitto
$ sudo service mosquitto restart
install ROS2 mqtt_client
on the robot install pip, python and mqtt-library

sudo apt-get install python3-pip
sudo pip3 install paho-mqtt

run simple test.py

#!/usr/bin/env python3

import paho.mqtt.client as mqtt

# This is the Publisher

client = mqtt.Client()
client.connect("localhost",1883,60)   # instead of localhost use your PC ip
client.publish("topic/test", "Hello world!");
client.disconnect();

repo: CatUnderTheLeaf/roboVision


text:
Planned Features v3.6.0

[ ] Browsing History
[ ] Site Security Indicator

Pull Request: #414
Please push changes to the v3.6.0 branch.

There will be 3 categories of website security:
Secure - Uses HTTPS with ideal encryption
Unsecure Uses misconfiguration https or doesn't handle data securely
Dangerous does not use HTTPS (uses http)

repo: CatalystDevOrg/Catalyst


text:
Add hover effect on social button

Related Issue(s)
Closes #58
Proposed Changes
Hey, @praveenscience I have added the hover effect on social buttons. Now the color changes to their respective official logo color on hover. Please review my pr and merge it. Thank you!
Type of Change

[ ] New Feature (Landing Page)
[ ] New Feature (App)
[ ] Bug (Landing Page)
[ ] Bug (App)
[ ] New Component (App)
[x] Enhancement
[ ] Documentation / ReadMe / Workflows
Other (If not in above options):

Your Idea for Level (GSSoC)

[ ] Level 1
[x] Level 2
[ ] Level 3

Screenshots (if applicable)

Additional Notes
Nothing

@AmanJain18 The PRs aren't getting labelled automatically! 😞

Thank you!

ya saw that, and fixes the issue in local, creating pr in few min, and what about SSoc Lable ?
@praveenscience

No idea about SSOC labels yet @AmanJain18

@praveenscience Please add level tag

Yes will we add it shortly...✌🏻

repo: CatsInTech/GitHub-ReadMe


text:
🛑 Nextcloud is down

In 1c4a7f1, Nextcloud (https://nextcloud.hiripple.com) was down:

HTTP code: 530
Response time: 56 ms

Resolved: Nextcloud is back up in 4d435f3 after 20 minutes.

repo: CelestialRipple/ripplelog


text:
🛑 HomeAssistant is down

In 05ec930, HomeAssistant (https://ha.hiripple.com) was down:

HTTP code: 530
Response time: 59 ms

Resolved: HomeAssistant is back up in 901f36a after 20 minutes.


text:
Improve embedded code formatting

Motivation
Some of the embedded code was still poorly formatted and not always
well recognized by decompilers.
Modifications

Remove colons from file paths.
Ensure that the formatter runs after any OpenAPI enrichers.
Add trailing semicolons to initializers and expression bodied
functions.
Add newlines after attribute lists.
Small perf bump by passing explicit workspace options to formatter.

/merge

repo: CenterEdge/Yardarm


text:
Allow optionally stronger enforcement to prevent silent property assignment

Assigning a value to a previously undefined property of a StoredObject instance raises a UserWarning today, but does not halt execution.
I'd like to see a way to make this raise a proper exception instead, thereby preventing accidentally missing a value due to a typo. This is especially important when using an interactive shell like Jupyter to work with models on the fly.
Proposed syntax:

class Foo(StoredObject):
    _meta: {
        'strict': True,
    }

foo = Foo()

foo.bar = 'my value'
# AttributeError: "bar" is not a defined property of the "Foo" model

Closes #26.

repo: CenterForOpenScience/modular-odm


text:
Fix Datetimes on Search Page

 Purpose
Make registration dates display the time they were created and not the current time. Address this trello card:
https://trello.com/c/PgBEhyUK/45-date-registered-displays-current-date-and-time
  Changes
Uses date_created attribute for the the time the registration was created opposed to the somewhat counter-intuitively date_registered which refers to the date the registration of project was created.
 Side Effects
None that I know of.

This doesn't look correct; why would we display the date registered as the project creation date?

I believe it's the date the node was created, not the original project.

I think you are using the wrong key; why not use date_registered? https://github.com/CenterForOpenScience/osf.io/blob/93ea883ff58102caae868450b906dc8cb3793c55/website/search/elastic_search.py#L193

@sloria I just realized that, but all my registered_dates are null, so I have the same problem.

@Johnetordoff See the code I linked to. The key is date_registered.

Yeah initially I thought that was the problem, but really I am just getting null values for date_registered, look:

I've confirmed through manual testing that date_created is the creation date of the original project, which is not what we want.

@Johnetordoff This indicates the bug is in our ES code, then.

ES?

Elastic Search. Indeed, format_result was not accessing the registration date correctly. I've fixed this in https://github.com/CenterForOpenScience/osf.io/commit/fbe50be9759b6ee8a9bc41390590725702cf7ce8

authors: Johnetordoff sloria


text:
[Feature] Institution Login for OKState University and Delegate Authentication to External CAS Server [#CAS-15][#OSF-7271]

Deployment Notes

Model Migration Required for osf_institution.
Populate Institution Required.

Tickets and PRs

Replaces #6689 and #6687
CAS PR: https://github.com/CenterForOpenScience/cas-overlay/pull/42
CAS Ticket: https://openscience.atlassian.net/browse/CAS-15
OSF Ticket: https://openscience.atlassian.net/browse/OSF-7271

Change target from develop to master. Failed to rebase or cherry-pick. Closed and replaced by #7120.


text:
Feature/Remove Submissions.js [PLAT-847]

Purpose
We have unused code causing confusion and slowing down page-load.
Changes
Removed submissions.js.
QA Notes
Double check meetings page (list of all meetings) and an individual meeting's page still works.
Documentation
None needed.
Side Effects

Ticket
https://openscience.atlassian.net/browse/PLAT-847

Unrelated test failure, didn't get this on my local travis.


text:
Fix DOI manage command

Purpose

Changes

QA Notes
Please make verification statements inspired by your code and what your code touches.

Verify
Verify

What are the areas of risk?
Any concerns/considerations/questions that development raised?
Documentation

Side Effects

Ticket

Cherry-squashed as 394623ae2279207667df7e0c099f6d514eed767c, consider this merged. :octocat:

authors: Johnetordoff mfraezz


text:
Add NOAA as a harvester

https://www.nodc.noaa.gov/access/services.html

@erinspace This can also be closed

repo: CenterForOpenScience/scrapi


text:
Refactor Cluster and Catalog mods

Describe This Problem
A rough dependency graph of related structures:

Cluster and CatalogManager are public, they may be used by Client or CeresMeta to modify catalog information (create, open or delete tables). This graph shows two paths of creating/droping a table. The green one is triggered by server itself and another orange one is triggered by CeresMeta

Green path
CatalogManager can create/drop tables when it has write access (the leader role). 3-1 in this path is notifying CeresMeta that a new table has been created, and 3-2 is an operation that happens simultaneously that create/drop that table in the leader itself.
Orange path
This path can be treated as counter part of the green one. CeresMeta may notify non-leader node that something has changed. It also goes through EventHandler to notify Cluster and let it take action.

Proposal
From my first impression, the dependency and call graph is complicated. I'd like to change relative parts in the following ways:

[ ] Combine VolatileCatalog and Cluster #245
Cluster is responsible to manage Tables and Shards while Catalog is responsible to manage Schemas which is a group of tables in another perspective different from Shard. Their functionalities have a big part that overlaps. It's natural to implement both Cluster and Catalog traits over one struct, so they can use one memory state and maybe one operation logic.
[x] Remove SchemaIdAlloc and TableIdAlloc #238
Id should be allocated by CeresMeta, not the server. And furthermore, the server does need not to allocate ID then create table. The entire procedure is accomplished by CeresMeta.
[ ] Split MetaClient
Current MetaClient provides two functionalities: poll update from CeresMeta and invoke CeresMeta's method. These correspond to the two ways above: green arrows are from server to CeresMeta and orange are from CeresMeta to server. We can split these two aspects, by making MetaClient an actual client that invokes RPC to CeresMeta and an Eventloop that keeps fetching updates from CeresMeta. This can also resolve the cyclic dependency between Cluster and MetaClient.

Additional Context
No response

During the development, I find it is hard to combine the VolatileCatalog and Cluster because create/drop/open/close procedures are too complex to be processed by the combined module in cluster mode, e.g. in the create table procedure, the combined one should be able to handle two different cases: create by user (the request will be forwarded to ceresmeta) and create by ceresmeta (do the real creation), and that is ugly.
Here is the new proposal, which will still keep the VolatileCatalog and Cluster separate, but the event handler will be removed, and the TableManager in the Cluster will be exposed and will be held by VolatileCatalog:
 ┌───────────┐
 │  Cluster  │───────Create/Drop/Open/Close table
 └───────────┘                │
       │                      ▼
       │             ┌─────────────────┐
       │             │TableManipulator │───────────┐
       │             └─────────────────┘           │
       │                                           ▼
       │                                   ┌──────────────┐
       │                                   │CatalogManager│
       │                                   └──────────────┘
       ▼                                           │
┌────────────┐                                     │
│TableManager│◀────────────────────────────────────┘
└────────────┘

As for the create/drop table procedure, a new proxy called TableCreator will be added to handle different cases:
         ┌─────────────┐
      ┌──│TableCreator │────┐
      │  └─────────────┘    │
      │                     │
      ▼                     ▼
┌───────────┐         ┌───────────┐
│ From user │         │ From meta │
└───────────┘         └───────────┘
      │                     │
      ▼                     ▼
┌───────────┐         ┌───────────┐
│Meta client│         │  Cluster  │
└───────────┘         └───────────┘
                            │
                            ▼
                      ┌───────────┐
                      │CatalogMana│
                      └───────────┘

authors: ShiKaiWi waynexia

repo: CeresDB/ceresdb


text:
Runtime crash related to CesiumIonServer in a packaged game when tilesets are only created at runtime

Reported here:
https://community.cesium.com/t/fatal-error-while-spawning-cesium3dtileset/34641
When a Cesium3DTileset is added via the Editor, it will have a reference to a CesiumIonServer instance (usually the default SaaS one) and so Unreal will know to automatically include this instance in a packaged game.
However, if your level does not have a Cesium3DTileset, but then you add one at runtime, Unreal won't know that the CesiumIonServer instance is required, and so it won't include it in the packaged game. As a result, the reference will be nullptr and the game will crash with an access violation.
This can be worked around by explicitly telling Unreal to include the CesiumIonServer. But it manifests as a mysterious crash to users, which is not great. We should address this by doing one or more of the following:

Tell the user at runtime that this is a problem and fail gracefully instead of crashing with an access violation.
Handle the possibility that there is no CesiumIonServer at runtime, and create one on the fly (like we already do in the Editor).
Always force the CesiumIonServer to be included in packaged games even if it doesn't appear to be used.

We should also consider if there are other assets that will be missing in this scenario.

I'm just adding stack trace from that forum post:
Unhandled Exception: EXCEPTION_ACCESS_VIOLATION reading address 0x0000000000000080
0x00007ff64b3086d7 AutoVrtlEnv.exe!FString::FString() []
0x00007ff65449a0a5 AutoVrtlEnv.exe!ACesium3DTileset::LoadTileset() [C:\TEMP\Bsim3d_2709503_12248\AutoVrtlEnv\Plugins\CesiumForUnreal\Source\CesiumRuntime\Private\Cesium3DTileset.cpp:1196]
0x00007ff65449a884 AutoVrtlEnv.exe!ACesium3DTileset::OnConstruction() [C:\TEMP\Bsim3d_2709503_12248\AutoVrtlEnv\Plugins\CesiumForUnreal\Source\CesiumRuntime\Private\Cesium3DTileset.cpp:660]
0x00007ff65080633c AutoVrtlEnv.exe!AActor::ExecuteConstruction() []
0x00007ff650791429 AutoVrtlEnv.exe!AActor::FinishSpawning() []
0x00007ff6507a8219 AutoVrtlEnv.exe!AActor::PostSpawnInitialize() []
0x00007ff651388bc7 AutoVrtlEnv.exe!UWorld::SpawnActor() []
0x00007ff65138904e AutoVrtlEnv.exe!UWorld::SpawnActor() []
0x00007ff654c3487b AutoVrtlEnv.exe!UWorld::SpawnActor<ACesium3DTileset>() [C:\Program Files\Epic Games\UE_5.3\Engine\Source\Runtime\Engine\Classes\Engine\World.h:3463]

authors: kring spikhoff

repo: CesiumGS/cesium-unreal


text:
Add support for glTF KHR_materials_clearcoat extension

Feature
The KHR_materials_clearcoat extension extends the PBR metallic roughness model to enable layering of a clear coating on top of an existing material. This clear coating can provide more realistic rendering of objects such as cars with a two-layered coating, where the color comes from the underlying layer.
CesiumJS does not currently recognize the extension parameters in a glTF. Here is how the clearcoat test model is currently rendered. Note how the first two columns appear identical:

Compare to the result from the reference implementation with the extension enabled:

To implement this extension, our PBR shaders will require a fairly significant refactoring.
Current flow
LightingStageFS does the following steps:

Initialize color with the material's diffuse color. But if the model has normals, we then:
Replace color with the result of czm_pbrLighting (in pbrLighting.glsl)
Add a calculated color from ImageBasedLightingStageFS. Both this and the previous step use the same light color / intensity.
Scale down by the material's pre-computed ambient occlusion
Add the material's emissive color
Apply tone mapping

Steps 2 and 3 each compute 3 parts:

Specular component (the product of a "microfacet distribution function" and a geometric attenuation/shadowing factor)
Diffuse component (a simple Lambertian scaling)
Fresnel reflection coefficient (used to scale and combine the specular and diffuse components)

The specular component has different paths for the anisotropic and isotropic cases.
Note that Step 3 is problematic for image-based lighting. If a user supplies environment maps, they probably expect those to be the only light source. Or if the user does want both point and environment lighting, they need to be able to scale them separately.
Step 4 applies the ambient occlusion to both image-based and point lighting. This is different from the reference implementation, which does not apply occlusion to the point lighting specular component.
Problems for clearcoat implementation
The clearcoat is layered on top of the base material. It contributes light via a specular component only. It also contributes a second Fresnel coefficient for use in combining the layers.
The obvious way to implement clearcoat is to re-use the specular calculations in both czm_pbrLighting and ImageBasedLightingFS to compute the clearcoat specular contribution. There are 2 main complications:

The specular and diffuse calculations are currently combined into the same functions. (clearcoat needs specular only)
The clearcoat Fresnel coefficient scales down the emissive color, but the PBR functions do not return this value to LightingStageFS where the emissive color is added.

Possible new code structure
czm_pbrLighting and ImageBasedLightingFS would be split into separate functions to compute specular and diffuse components and Fresnel reflection coefficients.
The refactored LightingStageFS would then:

Compute base layer specular components, for both point lighting and IBL (with separate light colors/intensities). This would call different functions depending on whether the material is anisotropic.
Compute diffuse components (again with separate light colors for point lighting and IBL)
Compute Fresnel reflection coefficients and apply to both specular and diffuse components
Compute clearcoat contribution and Fresnel coefficient
Scale diffuse components and IBL specular components (including clearcoat IBL) by ambient occlusion
Combine all base layer components, add emissive contribution
Scale by clearcoat Fresnel factor
Add clearcoat contribution
Apply tonemapping

repo: CesiumGS/cesium


text:
The GeoJSONandTopoJSON doesn't work normal when use Webgl2.

Sandcastle example: GeoJSONandTopoJSON.
Browser: Chrome 78.0.3904.108
Operating System: windows 10
code : var viewer = new Cesium.Viewer("cesiumContainer", {
contextOptions: {
webgl: {
antialias: true,
audioEngine: true,
deterministicLockstep: false,
lockstepMaxSteps: 4,
preserveDrawingBuffer: true,
stencil: true
},
requestWebgl2: true
}
});

I can see significant artifacts when running the GeoJSON example here with WebGL 2: https://sandcastle.cesium.com/index.html?src=GeoJSON and TopoJSON.html but those same artifacts are presents without loading any GeoJSON either. So I'm not sure there's a specific GeoJSON WebGL 2 issue here.
I'm closing this since CesiumJS does not officially support WebGL 2, but it is on the roadmap. See: https://github.com/CesiumGS/cesium/issues/797#issuecomment-617189407


text:
Fix bug with orthographic zoom

Reported on the forum.
camera._adjustOrthographicFrustum expects a parameter zooming which was not being passed, meaning the orthographic frustum was not being properly updated.

Thanks for the pull request @ggetz!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.

authors: cesium-concierge ggetz


text:
Use prop-types package instead React.PropTypes

React 15.5 brings a new package called prop-types: https://facebook.github.io/react/blog/#migrating-from-react.proptypes.
Please update definitions.

Any updates?

#19 merged, waiting for @CezaryDanielNowak to update npm registry.

Published.

authors: CezaryDanielNowak dalkir kkwiatkowski untone

repo: CezaryDanielNowak/React-dotdotdot


text:
Rust-analyzer underlining missing

Rust-analyzer gives useful information about which variables are mutable and which methods mutate through underlining. This extension used to respect that, but now there is no underlining anywhere.

I'm not sure when it stopped exactly, but I remember it did work around July 2023. It's too bad because I love this feature, but I also love the true blacks!

For anybody else who has the same issue, adding this to my settings.json did the trick
"editor.semanticTokenColorCustomizations": {
  "enabled": true,
  "rules": {
    "*.mutable": {
      "underline": true
    }
  }
},

Thank you! Sorry for my long hiatus on GitHub/my open source contributions at large. I will get this updated when I get the chance to get my local dev environment going.
If you have properly tested I don't mind accepting a PR (didn't check if you opened one), but to get the extension updated I'll have to fight the monster that is figuring out how to break into my azure account.

authors: ChadBailey JoeyBF


text:
Remove unused code from all actors

Summary of changes
Changes introduced in this pull request:

Remove unused types from all actor crates

Reference issue to close (if applicable)

Closes #22
Other information and links

Codecov Report
Patch coverage has no change and project coverage change: -0.73 :warning:

Comparison is base (f93fd4f) 14.23% compared to head (a88f4a6) 13.50%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
- Coverage   14.23%   13.50%   -0.73%
==========================================
  Files         225      200      -25
  Lines       21566    19957    -1609
==========================================
- Hits         3069     2695     -374
+ Misses      18497    17262    -1235

Impacted Files
Coverage Δ

account_v10/src/lib.rs
50.00% <ø> (ø)

account_v9/src/lib.rs
50.00% <ø> (ø)

cron_v10/src/lib.rs
50.00% <ø> (+16.66%)
:arrow_up:

cron_v9/src/lib.rs
50.00% <ø> (+16.66%)
:arrow_up:

datacap_v10/src/lib.rs
50.00% <ø> (ø)

ethaccount_v10/src/lib.rs
50.00% <ø> (ø)

evm_v10/src/lib.rs
50.00% <ø> (ø)

init_v10/src/lib.rs
50.00% <ø> (ø)

init_v8/src/lib.rs
50.00% <ø> (ø)

init_v9/src/lib.rs
50.00% <ø> (ø)

... and 25 more

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

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

authors: codecov-commenter sudo-shashank

repo: ChainSafe/fil-actor-states