text:
add ia3 peft support

we can support 4-bit IA3 once https://github.com/huggingface/peft/pull/864 is merged.

Hi - I wrote the PR for 4-bit IA3 - I adjusted my own installation of Axolotl to support IA3 (I didn't submit a PR as it was a hack based on rewriting existing LoRA support, and as I have literally no training or experience in coding, I wasn't confident in adding a new functionality without breaking everything else) and found IA3 ran properly for training with no other major changes required. Comparing my hack to this PR, the changes here seem near identical - fortunately I have found IA3 and LoRA are mostly interchangeable from a code perspective, there weren't any misleading adjustments I had to make to get it working. I did not test loading, inference or merging weights in Axolotl using IA3, as I did these tasks using my own scripts or adaptions of existing scripts.
The only points I would raise are that:

One of the principal benefits of IA3 is that it supports a vastly higher learning rate than LoRA (this is indicated in the original IA3 paper). This allows for high quality training to be performed with far fewer epochs. I have fine-tuned several models on complex technical documentation using LR ~0.004. This may be worth flagging to users in either a sample .yaml or in the readme itself, as missing this results in forgoing much of the improvement IA3 offers over LoRA.
With respect to target modules and feedforward modules, the PEFT library already contains default settings for the majority of existing model architectures (peft.utils.other.TRANSFORMERS_MODELS_TO_IA3_TARGET_MODULES_MAPPING and ... TRANSFORMERS_MODELS_TO_IA3_FEEDFORWARD_MODULES_MAPPING) which it assigns if peft.tuners.ia3._prepare_adapter_config does not receive a value. In practice, I have found that targeting all linear modules has a relatively minor effect on training speed and on the size of the adapter file (in that it doubles it from ~3MB to 6MB). It may be preferable to advise the user to allow PEFT to assign at least the feedforward module if the user is not confident in what they are selecting.
At least initially, the existing llama_attn_hijack_flash script did not seem to be properly recasting the model dtype to bf16/fp16 as required by Flash Attention when using IA3. I expect this was probably my fault, though I could not identify why it wasn't working (I inserted debug lines and the function was in fact being called). This led to tensor size mismatch or just Flash Attention throwing an error and terminating. Again, as a hack, I just shoved a forced recast into axolotl.utils.models.py. I note that the llama_attn_hijack_flash script has been re-organised since I last cloned the Axolotl repo, and so expect that this problem (if it wasn't my doing) may have been solved. Still - if you get tensor mismatches on initial tests, this would be where I would recommend looking first.
The current implementation of IA3 in PEFT does not support merging weights in 4-bit. I have gotten around this by loading the model in bf16 and merging the weights to that and then requantizing it. This is probably inefficient and, more importantly, for a ~34B model requires almost all my memory (24GB VRAM + 128GB CPU RAM), which I imagine is not feasible on most home PCs. I hope to put in a PR relating to merging 4-bit IA3 weights, but again, as I have no prior knowledge or experience of AI (or coding at all), I'm limited by how fast I can read the documentation.

Just a reminder, https://github.com/huggingface/peft/pull/864 has been merged.

I was trying PEFT's version of IA3 back in early August and it would not work, regardless of what I tried. I'm curious to see what this will produce and will test it as soon as I can.

@winglian any updates on this?

authors: His-Wardship Napuh creatorrr official-elinas winglian

repo: OpenAccess-AI-Collective/axolotl


text:
fix:replace mutex with atomic value in pika_conf

我看了你修改的几个变量，这些本来就是原子变量吧？

第一个修改是 e64f1fd
第二个修改是7de2f5e

source: gharchive/pull-request

repo: OpenAtomFoundation/pika

license_type: permissive

license_source: github-api


text:
Fix checkmark issues with berkeley task monitor

@hdefazio Could you add a summary of what this changed? I don't remember any more.

authors: Purg hdefazio


text:
chapter-2 notebooks

@rahul-packt : please take a look at this PR when you get a chance. Thanks!

license: MIT


text:
[DEV][SPR-2] Make book route to connect google API and fix error

resolved this issue!

source: gharchive/issue

repo: Patelzil/E-Book-Hub


text:
Gatherling, pdm, bot, ssh are all down

We will need @silasary 's  Linode creds to get anywhere with this I suspect.

Billing issue

authors: bakert silasary

repo: PennyDreadfulMTG/Penny-Dreadful-Tools


text:
Fix getInteger(String) always returning null

Fixed in v1.3.2.

repo: PerryPlaysMC/DynamicConfigurations


text:
Include LinkedIn & other social links into referral request emails

I am thinking user should set up their profile. When they send a request, the email will attached the user profile link.
Of course user profile route need to be changed to dynamic route.

I think we can add the receiver's social media url (if they have one) in the email first for now?

For this, better change '個人連結(選填)' in the profile page to be able to input multiple links as well, like we software engineer would have linkedin, fithub, personal profile website, but now we can only put one of them in the field

@tommyttf Great idea. Will create a issue for it

created issue https://github.com/Pjaijai/Referalah/issues/166

authors: Pjaijai thomassth tommyttf yaucp

repo: Pjaijai/Referalah


text:
Exclude example project from the code coverage

Codecov Report

Merging #37 (9c5f8ff) into develop (9a16c34) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff            @@
##           develop      #37   +/-   ##
========================================
  Coverage    60.48%   60.48%
========================================
  Files           84       84
  Lines         3204     3204
========================================
  Hits          1938     1938
  Misses        1266     1266

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

authors: RomanovaPrime codecov-commenter


text:
Fix grammar & make warnings less abrasive

This commit could've been pushed directly to main branch instead of through a PR.

authors: BobDotCom EmreTech

repo: Pycord-Development/guide


text:
Quick export to csv

Addressed in #7bb6930

repo: QUT-Digital-Observatory/youte


text:
! (shift-1) does not work

This requires a proper title and more details.

Is this a similar issue to #33?

Is this a similar issue to #33?

yes

repo: Quark-Games/YEquals


text:
Rocm refactor jaxlib

jenkins : retest Ubuntu-GPU-single please

jenkins : retest Ubuntu-GPU-single please

jenkins : retest Ubuntu-GPU-single please

repo: ROCmSoftwarePlatform/jax


text:
Estimation for interval censored data with left truncation

Hey @kshedden thanks for the PR, looks like checks are still failing for this, what's the status?

Codecov Report
Base: 99.84% // Head: 99.19% // Decreases project coverage by -0.66% :warning:

Coverage data is based on head (fc34ccb) compared to base (a0332d5).
Patch coverage: 95.49% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   99.84%   99.19%   -0.66%
==========================================
  Files          10       11       +1
  Lines         633      742     +109
==========================================
+ Hits          632      736     +104
- Misses          1        6       +5

Impacted Files
Coverage Δ

src/hazard_npmle.jl
95.09% <95.09%> (ø)

src/Surv.jl
100.00% <100.00%> (ø)

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

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

I would like to add one more feature and tests.  Also I need to resolve convergence errors on 1.2.

authors: RaphaelS1 codecov-commenter kshedden


text:
homework-1

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
41

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


text:
Windows upgrade for godot 4.1 not working

closed by #3

repo: RecBox-Games/godot-gamenite-controlpads


text:
make splits deterministic with seed

Awesome!
thanks!!

authors: radekosmulski vinid


text:
Add color to success logs

/verified

/verified

/lgtm

authors: myakove rnetser

repo: RedHatQE/openshift-cli-installer


text:
Fix when in some cases func to not have attr

/verified

Report bugs in Issues
The following are automatically added:

Add reviewers from OWNER file (in the root of the repository) under reviewers section.
Set PR size label.
New issue is created for the PR. (Closed when PR is merged/closed)
Run pre-commit if .pre-commit-config.yaml exists in the repo.

Available user actions:

To mark PR as WIP comment /wip to the PR, To remove it from the PR comment /wip cancel to the PR.
To block merging of PR comment /hold, To un-block merging of PR comment /hold cancel.
To mark PR as verified comment /verified to the PR, to un-verify comment /verified cancel to the PR.
verified label removed on each new commit push.
To cherry pick a merged PR comment /cherry-pick <target branch to cherry-pick to> in the PR.

Multiple target branches can be cherry-picked, separated by spaces. (/cherry-pick branch1 branch2)
Cherry-pick will be started when PR is merged

To build and push container image command /build-and-push-container in the PR (tag will be the PR number).
To add a label by comment use /<label name>, to remove, use /<label name> cancel
To assign reviewers based on OWNERS file use /assign-reviewers
To check if PR can be merged use /check-can-merge

Supported /retest check runs

/retest tox: Retest tox
/retest python-module-install: Retest python-module-install

Supported labels

hold
verified
wip
lgtm

/lgtm

authors: dbasunag myakove redhat-qe-bot2

repo: RedHatQE/python-utility-scripts


text:
instructions

add event listener for '?' Have it popup a modal with instructions on how to play the game.

repo: RickBacci/tron

license: mit

license_source: bigquery


text:
button toggleable and radio-group toggle single only

useless

repo: Rikarin/xui


text:
模块的重新整理

将原有的web模块细化为dependencies、share、web、validator、starter模块

repo: RovingSea/utility-framework


text:
[DRF] Utilizar Docker

Relacionado con #8


text:
[INTERNAL] CI: Test on Node.js v16

It's not running, it's configured as "Required" check.
This configuration needs to be changed, which hasn't happened, yet.
I'll do so now, and then we will be able to merge. But I wasn't able to add the new Node 16 runs as "Required". Those were not showing up in the list of checks. I need to follow-up on this.

Ok, so here it worked 👍🏻


text:
fix CI

closed by e973905456bc4250992798d4acf17a48a2564b40


text:
I am using ViewNode and I want to do runtime animation and clicks inside ViewNode, I tried below code and clicks are not working.

I am using  implementation 'io.github.sceneview:arsceneview:0.10.0' and  implementation("com.google.android.filament:filament-utils-android:1.31.4")
these both version are suitable for my project, I can not update the version. Clicks of imgProfile, imgCall and imgEmail are not working
this is the code I did
import android.animation.Animator
import android.animation.AnimatorListenerAdapter
import android.app.Activity
import android.content.Intent
import android.content.pm.PackageManager
import android.net.Uri
import android.os.Bundle
import android.os.Handler
import android.widget.TextView
import androidx.appcompat.app.AppCompatActivity
import io.github.sceneview.ar.ArSceneView
import io.github.sceneview.ar.node.ArNode
import android.view.MotionEvent
import android.view.View
import android.widget.ImageView
import android.widget.LinearLayout
import android.widget.Toast
import androidx.core.app.ActivityCompat
import androidx.core.content.ContextCompat
import androidx.core.content.res.ResourcesCompat
import com.bumptech.glide.Glide
import com.google.ar.core.HitResult
import com.google.gson.Gson
import io.github.sceneview.ar.arcore.position
import io.github.sceneview.ar.isFocusable
import io.github.sceneview.math.Position
import io.github.sceneview.math.Scale
import io.github.sceneview.node.ViewNode
data class Employee(
val LanId: String,
val Name: String,
val Email: String,
val Department: String,
val JobTitle: String,
val Manager: Employee,
val ManagerLanId: String,
val WorkPhone: String?,
val OfficeLocation: String,
val PhotoURL: String?,
val CellPhone: String?,
val UserPrincipalName: String,
val UserPhoto: String?
)
data class ApiResponse(
val HaveMore: Boolean,
val IsLock: Boolean,
val Employees: List,
val Count: Int,
val Success: Boolean
)
class MainActivity1 : AppCompatActivity() {
lateinit var arSceneView: ArSceneView
lateinit var viewNode: ViewNode
var isPlaced = false // Flag to control movement
private var employee: Employee? = null
private val CALL_PHONE_PERMISSION_CODE = 1001
private val originalScale = Scale(0.9f, 0.9f, 0.9f) // Original scale
private val smallScale = Scale(0.5f, 0.5f, 0.5f) // 50% of the original size

override fun onCreate(savedInstanceState: Bundle?) {
    super.onCreate(savedInstanceState)
    setContentView(R.layout.activity_main)

    arSceneView = findViewById<ArSceneView?>(R.id.sceneView).apply {

    }

    val jsonString = """
{

"HaveMore": false,
"IsLock": false,
"Employees": [
{
"LanId": "XYZ123",
"Name": "XYZ",
"Email": "XYZ@ocbc.com",
"Department": "Group Engineering - Digital Channels",
"JobTitle": "Assistant Vice President",
"ManagerLanId": "OCBCGROUP\ABC123",
"WorkPhone": "+6561234567",
"OfficeLocation": "SG",
"PhotoURL": "https://i.pravatar.cc/50?img=2",
"CellPhone": "",
"UserPrincipalName": "XYZ123@OCBCGROUP.OCBC.COM",
"UserPhoto": "https://i.pravatar.cc/50?img=2"
}
],
"Count": 1,
"Success": true
}
"""
    val gson = Gson()
    val apiResponse = gson.fromJson(jsonString, ApiResponse::class.java)
    employee = apiResponse.Employees.getOrNull(0)

//        val smallScale = Scale(0.5f, 0.5f, 0.5f) // 50% of the original size
    viewNode = ViewNode(arSceneView.engine).apply {
        loadView(
            applicationContext,
            lifecycle,
            R.layout.center_card_layout,
            onLoaded = { renderableInstance, view ->
                val imageView =
                    view.findViewById<ImageView>(R.id.profile_image)
                // Use the employee's image URL
                employee?.UserPhoto?.let { loadImageFromUrl(imageView, it) }

                val textView = view.findViewById<TextView>(R.id.profile_name)
                textView.text = employee?.Name
                val textViewJob =
                    view.findViewById<TextView>(R.id.profile_designation)
                textViewJob.text = employee?.JobTitle
                val txtDepartment =
                    view.findViewById<TextView>(R.id.txtDepartment)
                txtDepartment.text = employee?.Department
                val txtEmail =
                    view.findViewById<TextView>(R.id.txtEmail)
                txtEmail.text = employee?.Email
                val txtPhone =
                    view.findViewById<TextView>(R.id.txtPhone)
                txtPhone.text = employee?.WorkPhone
                val txtLocation =
                    view.findViewById<TextView>(R.id.txtLocation)
                txtLocation.text = employee?.OfficeLocation

                var shortAnimationDuration: Int =
                    resources.getInteger(android.R.integer.config_shortAnimTime)
                val imgCall =
                    view.findViewById<ImageView>(R.id.imgCall)
                val imgEmail =
                    view.findViewById<ImageView>(R.id.imgEmail)
                val imgProfile =
                    view.findViewById<ImageView>(R.id.imgProfile)
                val layData =
                    view.findViewById<LinearLayout>(R.id.layData)
                imgCall.setOnClickListener {

                }

                imgEmail.setOnClickListener {

                }

                imgProfile.setOnClickListener {

                }
            }) // Load your layout
        scale = smallScale // Set the decreased size

    }

    // Attach the ViewNode to the scene
    arSceneView.addChild(viewNode)
    viewNode.isFocusable = true

    // Keep the ViewNode centered with the camera
    arSceneView.onFrame = { frameTime ->
        if (!isPlaced) {
            // Keep the ViewNode at the center of the camera
            val cameraPosition = arSceneView.cameraNode.worldPosition
            viewNode.position = Position(
                cameraPosition.x,
                cameraPosition.y - 0.5f,
                cameraPosition.z - 1f
            ) // Adjust the z-offset as needed
        }
    }

    // Set up tap listener to place the ViewNode at the tap location
    arSceneView.onTapAr = { hitResult, motionEvent ->
        if (!isPlaced) {
            placeNodeAtTap(hitResult, motionEvent)
        } else {

//                Toast.makeText(applicationContext, "click on placed node", Toast.LENGTH_LONG)
//                    .show()
}
}
}
private fun placeNodeAtTap(hitResult: HitResult, motionEvent: MotionEvent) {
    val arNode = ArNode(arSceneView.engine).apply {
        position = hitResult.createAnchor().pose.position
    }
    // Place the ViewNode at the tap location
    arSceneView.addChild(arNode)
    viewNode.position = arNode.position // Move the viewNode to the tapped position
    viewNode.scale = originalScale
    isPlaced = true
    Toast.makeText(applicationContext, "Node placed", Toast.LENGTH_LONG).show()
}

private fun loadImageFromUrl(imageView: ImageView, imageUrl: String?) {
    Glide.with(this)
        .load(imageUrl)
        .circleCrop()
        .into(imageView)
}

private fun sendEmail(recipient: String, subject: String, message: String) {

}

Yes, it still require solution

repo: SceneView/sceneview-android


text:
Chage L2 proposal status after bridge

Current dependencies on/for this PR:

main

PR #172

PR #173   👈

This stack of pull requests is managed by Graphite.


text:
Speeding up actions

I think you've gotten a PR merged in to Selenium to fix these failing lint steps, right? Should we wait for that to be released?

authors: bandophahita perrygoy


text:
Update runtime.txt

OK


text:
Notatki ze spotkania 2

@avocadit podrzuciłbyś zdjęcie tablicy z końca?

@StanczakDominik done. Sprawdź, jak mógłbyś, czy we właściwym miejscu.;)

Tak, po prostu mi się pomieszały branche na których pracuję i nabrudziłem na masterze robiąc #22  >_< już posprzątałem, ale mogliście dostać maila że testy na masterze padły.

brakuje jednej osoby dla approve
@avocadit , @amjankowska ?

czemu nie zkomitowalo? Przeciez jest tutaj 2 approve..

czemu nie zkomitowalo? Przeciez jest tutaj 2 approve..

Tylko jeden z reviewerów ma odpowiednie uprawnienia.

wsensie ze ty niemasz opcji "approve"?

Przy moim approve jest, że "Only reviews by reviewers with write access count toward mergeability".

Ok poprawiłem, i już merge pull request zielony :) Choc coś dobrego w tym dniu)))

Przy moim approve jest, że "Only reviews by reviewers with write access count toward mergeability".

Ups, to mój błąd w takim razie - przepraszam! @okmechak dzięki za poprawienie :)

authors: StanczakDominik avocadit okmechak

repo: SocSIM/SocSIM


text:
Warcaster/resilent con (creed of arun) dont apply while wildshaped

War Caster doesn’t have Creed of Arum. Wildshape monsters use the monster CON.
Not an issue.

authors: Onlynoobs ThyWoof

repo: SolastaMods/SolastaUnfinishedBusiness


text:
Update Diagram for Central API Blog Post

Since merging #1003, it is now again possible to test patches locally.
For this I pulled the branch from this PR, checked it out locally and ran bash run-podman.sh. The local container builds the Jekyll webpage to be looked at http://localhost:4000.

authors: bitkeks o-otte

repo: SovereignCloudStack/website


text:
Various tweaks and improvements

Didn't reviewed, just merged lol

authors: ManlyMarco SpockBauru


text:
Fixed the read me agian

Sorry for taking so long and then declining this, cause I can't say I'm a fan of doubling the instructions to minimally change the name of the executable. It can't be tied to the OS with 100% accuracy anyways, I believe the note above should be enough.


text:
Create option for showing or hiding larva build times

Larva injections are now always visible when special skill is used

repo: StarkTemplar/starcraftbuilder


text:
changes related to logo and skip functionality

for one student we can save multiple subject marks .
when student is marked as absent its count is not shown in save status and scan status . and the information of that student is also not showing it in save status .
student list was not coming proper when we were switching different classes of same school .
local storage count was not proper it is resolved now . it was happening because we were using react old method(navigate now its replace with push) .
in local storage if student count is more than 50 it is showing alert message .

repo: Sunbird-Saral/Project-Saral


text:
fix: resolve annotation

Total Coverage: 90.57%

Coverage Report

File
Branch
Funcs
Lines
Uncovered Lines
src   shims.d.ts
100%
100%
100%
src/commonsrc/common/types   events.types.ts
100%
100%
100%
   meeting-colors.types.ts
100%
100%
100%
   participant.types.ts
100%
100%
100%
   realtime.types.ts
100%
100%
100%
   sdk-options.types.ts
100%
100%
100%
src/common/utils   do-request.ts
100%
100%
87.50%
20   index.ts
100%
100%
100%
   logger.ts
100%
100%
100%
   observable.ts
100%
100%
100%
   observer.ts
100%
100%
100%
   sleep.ts
0%
100%
100%
2src/components   index.ts
100%
100%
100%
   types.ts
100%
100%
100%
src/components/base   index.ts
100%
100%
100%
src/components/comments   index.ts
92%
97.06%
98.47%
123, 189, 443, 540, 556   types.ts
100%
100%
100%
src/components/comments/canvas-pin-adapter   index.ts
86.96%
96.30%
95.72%
231, 232, 233, 234, 235, 336, 338, 338, 343, 353, 389, 434, 448, 449src/components/presence-mouse   index.ts
88.89%
83.33%
89.19%
130, 131, 132, 142, 177, 178, 179, 180, 181src/components/realtime   index.ts
100%
100%
100%
src/components/video   index.ts
100%
100%
100%
src/core   index.ts
89.47%
100%
88.24%
49, 50, 69, 73, 75, 87src/core/launcher   index.ts
100%
100%
100%
src/servicessrc/services/api   index.ts
100%
86.67%
86.54%
104, 105, 106, 107, 94, 95, 97src/services/auth-service   index.ts
100%
100%
100%
src/services/browser   index.ts
100%
100%
100%
src/services/config   index.ts
100%
100%
100%
src/services/connection-status   index.ts
100%
100%
100%
src/services/event-bus   index.ts
100%
100%
100%
src/services/frame-brick-layer   index.ts
100%
100%
100%
src/services/limits   index.ts
75%
100%
100%
10src/services/message-bridge   index.ts
93.33%
100%
100%
80src/services/pubsub   index.ts
100%
90%
100%
src/services/realtime   index.ts
100%
100%
100%
src/services/realtime/ably   index.ts
56.49%
57.43%
70.20%
1009, 101, 1021, 1022, 1023, 1024, 1025, 1025, 1026, 1028, 1028, 1029, 1029, 1032, 1036, 1036, 1036, 1037, 1037, 1040, 1043, 1044, 105, 1080, 113, 1139, 1148, 1150, 1150, 1151, 117, 118, 1248, 1249, 1253, 1259, 1259, 1260, 1263, 1265, 1266, 1267, 1270, 1274, 1274, 1276, 1277, 1281, 1281, 1281, 1283, 1287, 1291, 1301, 1303, 1309, 1315, 1315, 1316, 1321, 1322, 1325, 1328, 1328, 1329, 1330, 1332, 1333, 1334, 1337, 1341, 1341, 1343, 1344, 1348, 1348, 1348, 1350, 1358, 1358, 1360, 1364, 1364, 1365, 1368, 1370, 1382, 1383, 1393, 1395, 1405, 1407, 1407, 1408, 1411, 191, 276, 432, 434, 444, 505, 515, 516, 537, 538, 540, 541, 543, 543, 544, 558, 560, 563, 565, 575, 576, 577, 578, 579, 579, 580, 581, 593, 603, 605, 606, 608, 609, 609, 615, 615, 615, 615, 615, 616, 617, 618, 618, 618, 618, 618, 619, 620, 620, 620, 620, 621, 624, 626, 626, 626, 626, 627, 628, 638, 656, 657, 667, 709, 710, 738, 740, 741, 742, 742, 743, 746, 753, 753, 754, 754, 756, 756, 756, 759, 760, 763, 763, 764, 765, 768, 769, 771, 776, 876, 876, 877, 880, 882, 882, 882, 883, 885, 939, 941, 942, 944, 946, 948, 948, 948, 948, 949, 950, 951, 954, 955, 958, 961, 961, 961, 961, 962, 97, 984, 985src/services/realtime/base   index.ts
66.67%
66.67%
87.18%
107, 107, 108, 111, 122, 122, 123src/services/remote-config-service   index.ts
100%
100%
100%
src/services/video-conference-manager   index.ts
64.10%
86.11%
93.62%
108, 112, 113, 114, 155, 160, 192, 193, 198, 199, 200, 203, 203, 204, 206, 207, 284, 319, 320, 321, 374, 408, 519, 531, 574, 578   types.ts
100%
100%
100%

Total Coverage: 97.54%

Coverage Report

File
Branch
Funcs
Lines
Uncovered Lines
srcsrc/web-componentssrc/web-components/base   index.ts
100%
100%
100%
src/web-components/base/styles   icon-button.style.ts
100%
100%
100%
   index.ts
100%
100%
100%
   sv-hr.style.ts
100%
100%
100%
   typography.style.ts
100%
100%
100%
   variables.style.ts
100%
100%
100%
src/web-components/comments   comments.ts
92.86%
87.50%
98.23%
44, 45, 61   index.ts
100%
100%
100%
src/web-components/comments/components   annotation-filter.ts
100%
100%
100%
   annotation-item.ts
100%
100%
100%
   annotation-pin.ts
88.89%
100%
100%
35   annotation-resolved.ts
100%
100%
100%
   annotations.ts
100%
88.89%
90.10%
39, 40, 41, 42, 56, 57, 58, 74, 75, 76   comment-input.ts
96.88%
94.12%
86.11%
100, 101, 102, 103, 104, 105, 106, 107, 108, 109, 110, 111, 112, 113, 114, 115, 116, 117, 118, 119, 120, 121, 122, 123, 124, 125, 126, 127, 128, 55, 99   comment-item.ts
100%
100%
100%
   content.ts
96%
81.82%
88.71%
104, 105, 35, 36, 39, 40, 41, 47, 48, 49, 63, 74, 75, 76, 77   delete-comment-modal.ts
100%
100%
100%
   float-button.ts
100%
100%
96.15%
41, 61, 62   index.ts
100%
100%
100%
   topbar.ts
80%
100%
89.80%
38, 39, 40, 41, 42, 43   types.ts
100%
100%
100%
src/web-components/comments/css   annotation-filter.style.ts
100%
100%
100%
   annotation-item.style.ts
100%
100%
100%
   annotation-pin.style.ts
100%
100%
100%
   annotation-resolved.style.ts
100%
100%
100%
   annotations.style.ts
100%
100%
100%
   comment-input.style.ts
100%
100%
100%
   comment-item.style.ts
100%
100%
100%
   comments.style.ts
100%
100%
100%
   content.style.ts
100%
100%
100%
   float-button.style.ts
100%
100%
100%
   index.ts
100%
100%
100%
   powered-by.style.ts
100%
100%
100%
   topbar.style.ts
100%
100%
100%
src/web-components/comments/utils   watermark.ts
100%
100%
90.77%
17, 18, 19, 20, 21, 22src/web-components/dropdown   index.style.ts
100%
100%
100%
   index.ts
100%
100%
100%
src/web-components/hello-world   index.ts
100%
100%
100%
src/web-components/icon   index.ts
100%
100%
100%
src/web-components/modal   index.ts
100%
100%
100%
   modal-container.ts
100%
100%
100%
   modal.ts
100%
100%
100%
src/web-components/modal/styles   index.style.ts
100%
100%
100%
src/web-components/presence-mouse   index.ts
71.43%
100%
97.20%
42, 43, 44, 59, 60, 74, 75   styles.ts
100%
100%
100%

repo: SuperViz/sdk


text:
[Translation] Add missing German translations and update existing ones

Thank you, @shochdoerfer!

Always happy to help :)

authors: lchrusciel shochdoerfer

repo: Sylius/InvoicingPlugin


text:
exe's unusable due to continous powershell popup

--windowed and --noconsole do not work

Fixed

repo: TXOG/Fluorite-Store


text:
chore: starting to test filter types

Where would attest be installed?

maybe another branch that actually uses the ExtractFeatureTypes should be made? We'll uncomment that code in that branch, run attest on both the alpha branch and that one.

Where would attest be installed?

Probably the root package.json devDeps. This way we can test examples, packages/* and anything else from there.
I don't have any strong opinion on this though. I did a global install of it locally and I am just sampling as I cd around the repo right now.

maybe another branch that actually uses the ExtractFeatureTypes should be made? We'll uncomment that code in that branch, run attest on both the alpha branch and that one.

yeah, this would be really helpful. Is it just a matter of uncommenting all the commented out types?
I was unsure if not all of them were commented for the same reasons or not.

assuming we want to close this one

authors: KevinVandy tylersayshi

repo: TanStack/table


text:
Can sharpcaster be used in mirror an android screen to chromecast?

Sharpcaster can be used to make chrome cast load the stream. It won't help you how to record your stream or how to serve that stream to the chrome cast.

Thanks for your reply. Is it possible to just send a bitmap to chromecast, I can only find mentions of providing a url.?

Nope. You would need to send a URL to chromecast.
URL could point to a webserver that you are hosting and then you could upload a bitmap to that webserver which then shows it at that url

I hope you don't mind the questions. As you know you can cast you android screen to a chromecast device using google home on android. Any ideas how that can work , I presume there are not uploading it to a server.  Maybe they are creating a streaming server on the android device.

Not at all.
That's a very good point and I had completely ignored that one.
Either there's something special on Chromecast protocol that allows that, but I haven't implemented which might be the case or they are using the android device as a server.
You could probably do a package capture on a network to see if the Chromecast starts to steam video from Android device. Not sure any other way to confirm that. I'm iOS and Windows user so don't currently have any android devices to test this.

authors: Tapanila noelitc

repo: Tapanila/SharpCaster


text:
minor: fix TAK_PLATFORM_ITSM value

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wheelwang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: tencentblueking-adm wheel-w

repo: TencentBlueKing/bk-chatbot


text:
feat(other): dbha查询接口返回增加tbinlogdumper信息

关联研发信息
dev_branch: feat/dbha_api_add_tbinlogdumper/#1724
target_branch: v1.2.0

repo: TencentBlueKing/blueking-dbm


text:
feat(mongodb): 部署时flow写集群元数据增加disaster_tolerance_level字段

关联研发信息
dev_branch: feat/meta_disaster_tolerance_level/#6713
target_branch: v1.5.0


text:
Develop

lo siento

repo: TheSoftwareDesignLab/RIPuppetCoursera


text:
Make sure port 4551 isn't already allocated before starting warcprox

Replaced with #14

Unreplaced with #14

repo: TheTechRobo/twitch-chat-getter


text:
unit test

Need to create unit test for game


text:
refactor: upgrade project

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

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

repo: TomaszKandula/SonarQubeOnAzure


text:
merge: stage to master

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

repo: TomaszKandula/TokanPages


text:
BestFTSO Flare Updates

Also update the logo URI to match your new address please.

Specifically https://github.com/TowoLabs/ftso-signal-providers/pull/281/files#diff-398d38a136234ec9c01612cf2c43d53ff4722e956fc16e1c28485ba4d0afa5b1R222

authors: RareData XDapps

repo: TowoLabs/ftso-signal-providers


text:
POST player facts API - Svelte

https://github.com/Tristandemuijnck/ultitv-api-svelte/commit/b8fb4ae6b97cd9cef6de65418d22cd7ecc3817f5
Added POST option for player facts


text:
Upgrade architecture graph for GeaFlow

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.东朔 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant fanzhidongyzby

repo: TuGraph-family/tugraph-analytics


text:
Test app r

Delete PR.


text:
Como jugador quiero que los enemigos esperen para pelear cuando haya más enemigos en combate

[ ] Añadir un nuevo estado de animación de espera (idle)
[ ] Implementar que el enemigo no pueda ser golpeado mientras espera

repo: UCM-FDI-DISIA/Selery-Studios


text:
FCM Multi-Host

This is the code running on fcm.distributor.unifiedpush.org rn, will merge later

repo: UnifiedPush/common-proxies


text:
Sweep: Test

Actions (click)

[ ] ↻ Restart Sweep

❌ Unable to Complete PR
Please add more details to your issue. I need at least 20 characters to generate a plan. Please join our Discord server for support (tracking_id=914842c9c4)
For bonus GPT-4 tickets, please report this bug on Discord (tracking ID: 914842c9c4).

🎉 Latest improvements to Sweep:

New dashboard launched for real-time tracking of Sweep issues, covering all stages from search to coding.
Integration of OpenAI's latest Assistant API for more efficient and reliable code planning and editing, improving speed by 3x.
Use the GitHub issues extension for creating Sweep issues directly from your editor.

💡 To recreate the pull request edit the issue title or description.
This is an automated message generated by Sweep AI.

repo: Upsonic/On-Prem


text:
[v1.3.0] Abstracted the cache driver

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

#25  👈
#24
dev

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

Join @GoodM4ven and the rest of your teammates on  Graphite

Merge activity

May 5, 2:58 AM EDT: @GoodM4ven started a stack merge that includes this pull request via Graphite.

repo: VPremiss/Livewire-Nonceable


text:
Tech debt: consider combining RuleState and RuleBuilder

And also consider combining TentativeRune with IRuneS somehow?

repo: ValeLang/Vale


text:
[Defect] CJK Vertical mode not able to type Space & Ogham typing issues

Fixed

repo: Vyshantha/multiscripteditor


text:
add g1

LGTM!

authors: WikKam ilyakuchynski

repo: WikKam/simple-react-ecommerce


text:
Cleaned code by using a hashmap

Thanks for contributing @Peu77!

repo: William-McGonagle/burm


text:
feat: add option to pascal case model names #71

@capaj This's the new generated file with the new option.
https://github.com/YassinEldeeb/prisma-tgql-types-gen/pull/72/files#diff-702315e2cb2f4ce9585f4d682d52266b45c61b7e9a74549f28754b1c9925aa39

I think it will break when importing that module. That import name has to be PascalCased too.

authors: YassinEldeeb capaj

repo: YassinEldeeb/prisma-tgql-types-gen


text:
Wyujie

bors r+


text:
Feature: Hover tiny session to reveal it

done

repo: Yukaii/film-scheduler


text:
friend functionality

@rostum rebase please

authors: Yurzs rostum


text:
Add files via upload

Adding changes.

repo: ZachMontgomery/PolyFits


text:
Properties for Spot

thank you for the PRs 🙏

authors: Marchl-Woid oblomov-dev


text:
Local Storage of Your Own Recipes

A few options:

User supplies the google sheets link
User uploads their own csv
Have a form and store in cookies

repo: acnolan/WhatShouldIEat


text:
feat(lint): create module spark-lint and import MaterialComposableUsageDetector

@soulcramer, it's ready to be reviewed. I'll add the packaging/export on another PR.

repo: adevinta/spark-android


text:
Build an extension that allows to identify in the website

Hi,
An interesting article about it.

@solygambas article is behind a paywall :( By I like the idea :)
Might be easy to do with basic data-attributes on spark components.

yup

authors: Powerplex andresin87 solygambas

repo: adevinta/spark


text:
Score Filter broken

Hey @adhesivecheese could you please fix the score filter, TYSM!

Hi there! It looks like the score parameter was replaced with min_score and max_score with the new API. This should be a pretty straightforward fix. I've done the changes in the scorefix branch that should take care of things, but I'll need to test before I can push it live. Anybody that feels like testing, please let me know!

@adhesivecheese Seems to be fixed now, thank you!

authors: Esiou NightxOwl adhesivecheese

repo: adhesivecheese/chearch


text:
Remove dormitory selection in LoginForm, if only one datasource is used

Done in b1d57ab, and then #471 removed the whole multi-datasource support.

repo: agdsn/sipa


text:
Added unit tests and convenience methods, fixed SonarCloud issues, include file cleanup

@akuker Any idea when you will have time to review? If you cannot review until Friday (tomorrow) morning CEST (my time), and in case you agree, I would like to add the changes from the code_cleanup branch to this PR. They address essentially the same: Unit tests, SonarCloud issues, removing duplication, general cleanup.

Hi @uweseimet - I’m out of town with only my phone, so the earliest I’ll be able to take a look is Saturday (or maybe Sunday, depending on jet lag). Feel free to queue up changes however is most efficient for you

@uweseimet Do you mean python and html issues reported by Sonarcloud? How can I see them?

@rdmark Just select the branch you are interested in on https://sonarcloud.io/project/overview?id=akuker_RASCSI.

@akuker After the review, I would like to merge this PR myself, because I want to clean up the final log message.

@uweseimet Thanks I found the relevant report. Glancing through the Python code smells, vulnerabilities and issues, there are a handful that are worth addressing. Others are known structural issues that pylint has reported already, but are hard to do anything about (complexity, nested if statements, etc.) without major refactoring.
As for the reported html issues: we deliberately use deprecated syntax in order to support ancient web browsers, so none of those are something that we want to address. If we can suppress html syntax deprecation warning that would be great.

@rdmark Do issues like those reported for python/common/src/util/unarchiver.py require a major refactoring?

@uweseimet I agree that unarchiver.py is the one module with low hanging fruit to potentially fix.
@nucleogenic Do you want to look into it, and perhaps fold changes into a new PR or your ongoing PR?

Hi both,
I will check out the SonarCloud reports soon and raise a new PR for any changes.
The unarchiver module is going to need refactoring to allow it to be unit tested, which is something I'm keen to investigate now we've got pytest set up.

Approved. feel free to merge!

@akuker Done, thank you for reviewing!

authors: akuker nucleogenic rdmark uweseimet

repo: akuker/RASCSI


text:
feat(scale_degree): add raised, lowered, major, and minor getters

Pull Request Test Coverage Report for Build 5224391927

18 of 18 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 5224020843:
0.0%

Covered Lines:
696

Relevant Lines:
696

💛  - Coveralls


text:
Last_part

Se agregó la última parte


text:
Create 32-nginx.sh

Tandoor addon seems to work, but it cannot find the port when using ssl.
I copied this from your repo, but maybe you know more how we can tell port 443 to port 8080 docker when using ssl.
I was hoping that this was loaded https://github.com/alexbelgium/hassio-addons-test/blob/master/tandoor_recipes/rootfs/etc/nginx/servers/direct.conf
through nginx.conf file.
https://github.com/alexbelgium/hassio-addons-test/blob/master/tandoor_recipes/rootfs/etc/nginx/nginx.conf

Need some more testing

repo: alexbelgium/hassio-addons-test


text:
feat(gzip): ask the API for gzipped answers (nodejs)

cc @redox

LGTM :+1:

authors: redox vvo

repo: algolia/algoliasearch-client-js


text:
fix spelling error in the readme

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.李括 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant SherLockOD

repo: alibaba/Pai-Megatron-Patch


text:
Address feedback for tasks 664 - 700 and fix a repeated misspelling across a lot of files.

Thanks!

authors: Palipoor danyaljj

repo: allenai/natural-instructions-expansion


text:
SCRUM-4645 Update AGM model to use associations for submitted objects

@markquintontulloch Everything looks good with the one exception being about whether we need association slots as per my comment

authors: chris-grove markquintontulloch


text:
Fix /exits/sync endpoint

rebase and the tests will pass

Cool, I think the dash changes are ready to merge when this lands: https://github.com/althea-net/althea-dash/pull/94

authors: asoltys jkilpatr


text:
Add Faith Adewuyi card info

PR Merged!
Thanks for contributing.

authors: amyyalex faithadewuyi

repo: amyyalex/simple-contribution


text:
add RadioHaiti

would probably be good, but doesn't feel necessary


text:
Attempt to read property "name" on null in Manage Company

Providing only your error message doesn't really help me track down what the issue could be. Can you provide more context on when this error occurs? Can you provide the stacktrace for me? Is this error with the latest release? Everything is working perfectly fine for me when visiting the "Manage Company" page. Have you created a company? If you have created a company, is the "name" column in the database null even though you provided a name when creating one?

I'm sorry, I managed to solve it, apparently when making the changes and migrations something was not done correctly with my db but I executed the php artisan migrate:fresh command and it was solved

authors: andrewdwallo zxstudio-llc

repo: andrewdwallo/erpsaas


text:
refactor code

ok


text:
Test auto-update outstanding PRs

Merging this one w/o approval as a test, will remove it in a couple hours

repo: ansible/ansible-navigator


text:
FEAT: Fields markers

@Samuelopez-ansys @SMoraisAnsys Codecov is not passing as the method works only in graphical mode (bug already reported)

repo: ansys/pyaedt


text:
Flu_df

closing merge to resolve discrpency


text:
-mistake

This is an accidental duplicate of issue #180. Please refer to that issue for the correct information and discussion.

repo: anthropics/anthropic-quickstarts


text:
chore: Superchain Crate

📚 $\text{Stack Overview}$
Pulls submitted in this stack:

#100 👈
main

This comment was automatically generated by st.

repo: anton-rs/superchain


text:
Update slack source 'Delay' description and example

all checks passed, may I get a merge please?

repo: apache/camel-kamelets


text:
Use project modules dependency management.

@arturobernalg Please consider investing your time and efforts into functional improvements.

repo: apache/httpcomponents-core


text:
[Subtask] Hive Backend Engine - GetTableTypes Operation

@yaooqinn, could you please assign this ticket to me?

@yaooqinn, could you please assign this ticket to me?

authors: SteNicholas yaooqinn

repo: apache/incubator-kyuubi


text:
OAK-10745 : added osgi configs for fullGc delayFactor & batch size

I'm wondering if we shouldn't also (or primarily) make PROGRESS_BATCH_SIZE configurable. Currently it's hardcoded to 10000 and we'd only be able to fiddle with the batch size, not the overall one. The overall one is however the one which controls how long fullGC occupies the scheduler thread. And in extremis we could set it to a ridiculously low value (say 10, then batch size also 10) and thus have near zero realistic performance impact on an otherwise very busy system. (Not suggesting 10 is a good value - but currently we wouldn't be able to configure that at all).

yes, let us add a config for PROGRESS_BATCH_SIZE as well.

The default for classic delayFactor is 0 - which I think is not ideal - zero means it does not delay at all. In production system that should or is rather set to 0.5 or 1 etc. I'm not suggesting to change the default for classic GC. But for the new full GC one, we could perhaps indeed use a different default. Perhaps even as high as 5 (which would mean max 20% time usage)?

I don't have any strong opinion here. Using default value 0 allows us to run it fast locally while testing.
If we use a different default value, even local testing would be slower. But for local, we can always change the code.
So I am fine with other default values.

Another aspect is that delayOnModifications (where the magic happens) is only invoked when deletion happens. But deletion isn't the only load on DocumentStore. There's also the reading and for fullGC the embedded verification that does many reads. For classic GC that probably applies as well to some extend - but the difference is : classic GC only ever works on deltas (as it existed since early oak days), it hardly has to scan a large amount. But full GC will do a one-off scan of large repositories thus do loads of reads. I'd suggest we introduce something like delayOnBatch - in addition to or perhaps actually in replacement of delayOnModification use.

Yes, we should add dealyOnModifications or delayOnBatch (or any other name) while collecting garbage as well.
Regarding embeddedVerifiacation, it is called before removing garbage, thus is already covered in the current code.

for collectGarbageOnDocument (which is used by oak-run) it doesn't seem useful for it to sleep, as it only ever runs on 1 document

For oak run, we could set the default value to 0. It would allow us to reuse the existing code.

I don't have any strong opinion here. Using default value 0 allows us to run it fast locally while testing.
If we use a different default value, even local testing would be slower. But for local, we can always change the code.
So I am fine with other default values.

Good point. I guess we have to choose between production and development unfortunately...

Regarding embeddedVerifiacation, it is called before removing garbage, thus is already covered in the current code.

the timer only measure removing garbage though - so that would have to be adjusted...

I have addressed your review comments here https://github.com/apache/jackrabbit-oak/pull/1795/commits/f6f8ce792f6f08c76fb03667be49993b6c6ef6e3

authors: rishabhdaim stefan-egli

repo: apache/jackrabbit-oak


text:
parquet: don't handle empty batches

Current dependencies on/for this PR:

main

PR #158   👈

This comment was auto-generated by Graphite.

repo: apibara/dna


text:
Devnet integration#93

This item belongs to payment request #00DAC5 on OnlyDust:

from fracek to ametel01
1 item included
$500 for 1 day of work

This item belongs to payment request #00DAC5 on OnlyDust:

from fracek to ametel01
1 item included
$500 for 1 day of work

Payment request #00DAC5 has been processed and payment is complete. See you around on OnlyDust.


text:
Fully adopt Sendable

@swift-server-bot test this please

@swift-server-bot test this please

Allocation counters regressions are unrelated to this PR:
https://github.com/apple/swift-nio-ssh/pull/152

Allocation have regressed because I marked the static function transform(:_) with @Sendable to workaround this issue: https://github.com/apple/swift/issues/61449
However, we can also just form a closure manually and it doesn't allocate any more.

authors: Lukasa dnadoba

repo: apple/swift-nio-ssh


text:
Move raw syntax nodes from gyb to code gen

@swift-ci please test

@swift-ci please test

repo: apple/swift-syntax


text:
[Epic] 2. Create module in package

Reopening the epic for QA tracking

repo: appsmithorg/appsmith


text:
[Bug - RTS] Activity status is shown as success even though the activity failed

Hey team! Please add your planning poker estimate with Zenhub @ayushpahwa @nsarupr

Duplicate of #32875

authors: nsarupr srix


text:
add bleed secrets onprem scanner to sscs helm chart

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Tamir Sinai seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant tamirsinai

repo: aquasecurity/aqua-helm


text:
Integrate MST into the DBSCAN driver

This is fine. We will need to revisit later the subdirectory and executable name.

Yeah. I think we'll need to move the whole machinery about loading the data into a separate file. That would leave with just simple drivers for dbscan, hdbscan, mst, etc.

Somehow, all these produce warnings:
view.extent(0) == bvh.size()
(int)view.extent(0) == bvh.size()
view.extent(0) == (int) bvh.size()

Does it mean that only (int)view.extent(0) == (int)bvh.size() will work?

Does it mean that only (int)view.extent(0) == (int)bvh.size() will work?

IIRC extent is of type View::size_type(which is signed for SYCL and unsigned otherwise but we want all of them to switch to signed eventually). So casting the right-hand side to that should work.

Kokkos::View::extent(Integral) -> size_t and ArborX::BVH::size() -> MemorySpace::size_type which depends on the backend you use.

@dalg24 thanks for the help!

authors: aprokop dalg24 masterleinad

repo: arborx/ArborX


text:
Delete requirements.txt

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:

authors: arjunsavel codecov-commenter

repo: arjunsavel/scope


text:
Variants

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

Comparison is base (0046a6e) 96.16% compared to head (1f12a01) 96.39%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
+ Coverage   96.16%   96.39%   +0.23%
==========================================
  Files          16       17       +1
  Lines         313      333      +20
==========================================
+ Hits          301      321      +20
  Misses         12       12

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

authors: aruizs codecov-commenter

repo: aruizs/unleash-client-cpp


text:
Update Composer dependencies (2024-06-08-00-18)

Visual regression test passed!
View the visual regression test report

repo: ashaal/elrom


text:
Horizontal feature post is not displaying correctly

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: auroral-ui/hexo-theme-aurora


text:
Upgrade testnet operator image to v0.1.3

LGTM


text:
chore: start testing on Bazel 8

The CI log output is extremely long. I don't see this with vanilla Bazel, maybe this can be fixed in Aspect CLI?

I haven't found time to investigate that logging issue but since Bazel 8 is released, I think we need to land this :)

authors: alexeagle fmeum

repo: bazel-contrib/bazel-lib


text:
Indigenous consultation level in history

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

:exclamation: No coverage uploaded for pull request base (develop@e7323b4). Click here to learn what that means.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #1820   +/-   ##
==========================================
  Coverage           ?   68.23%
==========================================
  Files              ?      252
  Lines              ?     8606
  Branches           ?        0
==========================================
  Hits               ?     5872
  Misses             ?     2734
  Partials           ?        0

Flag
Coverage Δ

epictrack-api
68.23% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

authors: codecov-commenter dinesh-aot


text:
Campus Code  in Offering

@HRAGANBC @CarlyCotton please confirm how valuable this would be for our Institution Offerings and help determine which Release this should belong to- Please Tag Accordingly

This seems like a uniquely BC Public issue, other PSIs have different locations (with different names). We don't incorporate "Campus Codes" in any of the fields collected for programs or offerings. Unless you had a different read on the question today? @HRAGANBC

@CarlyCotton @michesmith Agreed with you Carly, it's unique to Publics.
To solve the problem, we would need to either:

Work with Feds to allow Public institutions to operate with more than one Federal Institution Code (unlikely / lengthy); OR
Reconfigure Institution Location Code such that B.C. Publics can report 'iterations' of their main code to distinguish multiple campuses for the purpose of linking to Offerings, but ultimately only map the main code to the eCert file. For example, UBC could create multiple locations, one with a campus code AAUK and another with the campus code AAUK* to denote the secondary campus. Offerings could identify the suffixed * to know which offering apply to which location, while the eCert process ignores the * suffix and maps all requests to AAUK to ensure successful eCert process.

I think this is a change management consideration moreso than a critical requirement. At the end of the day, the above fix would NOT even solve the biggest issue - the need to have many distinct offerings, one for each campus.
I suggest definitely not pursuing this for MVP, but can label as enhancement for future review,

authors: CarlyCotton HRAGANBC michesmith

repo: bcgov/SIMS


text:
Feat/postgres

Resolves #58

repo: bcgov/greenfield-template


text:
Devops - ZEVA - convert existing deployment configs to helm templates

Backend bas been converted to helm templates. Most of frontend have been converted too. I'd like to build new frontend image in the way of similar as TFRS, as current frontend takes too many resources and take few minutes to load fir the initial visit.

The templates have been converted to helm chart, but the frontend webpack config needs some help from developer to properly setup.

repo: bcgov/zeva


text:
Clean repository after setup + hma + true range

hey could you push this to https://github.com/beatzxbt/mm-toolbox/tree/optimizations rather than main?
i have some changes to make to your code (types stuff) still and ill package this into a new version :)

repo: beatzxbt/mm-toolbox


text:
Sprint8 preview meeting setting

have some issue like null variables on video call service (unknown issue)????
everything else is fine.


text:
feat: add del method for cache module

Codecov Report
Attention: Patch coverage is 93.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 66.07%. Comparing base (e3661d3) to head (d0129d4).

Files
Patch %
Lines

pkg/cache/local_cache.go
85.71%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #28      +/-   ##
==========================================
+ Coverage   65.57%   66.07%   +0.49%
==========================================
  Files          24       24
  Lines         825      840      +15
==========================================
+ Hits          541      555      +14
- Misses        245      246       +1
  Partials       39       39

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


text:
Update Composer dependencies (2020-09-12-00-05)

Visual regression test failed!


text:
validator: don't display validator in menu for invited users

Closes https://github.com/betagouv/aplypro/issues/172

repo: betagouv/aplypro


text:
723 allow definition of gene panels based on gene ids backport

Codecov Report

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

:exclamation: Current head 0ee56e2 differs from pull request most recent head eccd155. Consider uploading reports for the commit eccd155 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##             anthenea     #726   +/-   ##
===========================================
  Coverage            ?   92.59%
===========================================
  Files               ?       17
  Lines               ?     4621
  Branches            ?      399
===========================================
  Hits                ?     4279
  Misses              ?      342
  Partials            ?        0

Flag
Coverage Δ

javascript
92.59% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out 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 at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: codecov-commenter holtgrewe

repo: bihealth/varfish-server


text:
Bevy 0.10.1 -> 0.11.0

Merged #22

license: Unlicense


text:
added images of the products and description

check both the files....which u find suitable update it to the main branch

No more required

repo: binitworld/Shop-Sphere


text:
/asyncquery endpoint added (recommended but not required) https://github.com/NCATSTranslator/ReasonerAPI/pull/255/files https://github.com/NCATSTranslator/ReasonerAPI/pull/263/files- implemented and deployed in dev instance for testing now.

accidental issue creation -- closing...


text:
fix amalgamated build

Why GetTempPathA doesn't have the issue?

authors: bkaradzic jamesfAnet

repo: bkaradzic/bx


text:
adding indel tests to hamming distance method

@huddlej I think I've fixed the issues outlined above - can you check and let me know?

@nandsra21 The most recent commits fix the indel counting logic, adding a check that excludes leading and trailing gaps from the total indel-based distances. Can you try updating the MERS workflow to use indel counts and then regenerate the MERS embeddings to see how it works? If it looks ok, we can update the SARS-CoV-2 workflow to use indels, too.

repo: blab/cartography


text:
Considering customize Segemented-IPC shuffle to reduce unnecessary header and footer for each segment

the IPC shuffle data file structure now becomes:
ipc length 1 (u64)
ipc length uncompressed 1 (u64)
ipc content 1 (compressed)
...
ipc content N (compressed)

also we turn arrow file format into stream format (https://arrow.apache.org/docs/python/ipc.html) which does not require a seekable reader. now we can implement shuffle reading with a simple InputStream and does not need to buffer anything in memory.

done

authors: richox yjshen

repo: blaze-init/blaze


text:
Bug: Fix issue with communication from web pages to extension

Not sure what this bug was, but we do have working web page to extension calling now.

repo: block-core/blockcore-wallet


text:
管理ページの記事一覧に public_id と公開日を出してはどうか

あと公開日の降順の方がいい気がする

repo: bm-sms/daimon-news-multi-tenant


text:
chore: Update .codegen.json with commit hash of codegen and openapi spec

Pull Request Test Coverage Report for Build 10943320184
Details

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
8 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.09%) to 70.815%

Files with Coverage Reduction
New Missed Lines
%

Box.Sdk.Gen/Networking/Fetch.cs
8
83.5%

Totals

Change from base Build 10920195724:
-0.09%

Covered Lines:
7581

Relevant Lines:
10667

💛  - Coveralls

authors: box-sdk-build coveralls

repo: box/box-dotnet-sdk-gen


text:
list of all harmonization outcomes <0

Must be sent to Mathijs

Technically if it is the 3 notebooks that aare the most important, we can't complete this harmonization, and therefore say which ones there were negative until the H dataset is OK'd

Issue is closed based on work of both M. D. (in R) and Makeda

repo: brainspinner/cvasl


text:
Build constraint models

Thank you so much for suggesting the write_expression() and the problem solved! Sorry for the back and forth. I should've check if hail have any way to deal with such things.

repo: broadinstitute/gnomad-constraint


text:
[DO NOT MERGE] - Docker is

The below passes the build and likewise so does "$${PLUGIN_OWNER}" = "protocolbuffers".
https://github.com/bufbuild/plugins/actions/runs/3424860057/jobs/5705043205
https://github.com/bufbuild/plugins/actions/runs/3424981941/jobs/5705293946
The Test step fails, but that's expected.

if [[ "$${PLUGIN_OWNER}" = "protocolbuffers" || "$${PLUGIN_OWNER}" = "grpc" ]]; then \
	touch $<; \
	$(DOCKER) $(DOCKER_BUILD_ARGS) \
		$(DOCKER_BUILD_EXTRA_ARGS) \
		--label build.buf.plugins.config.owner=$${PLUGIN_OWNER} \
		--label build.buf.plugins.config.name=$${PLUGIN_NAME} \
		--label build.buf.plugins.config.version=$${PLUGIN_VERSION} \
		-t $(DOCKER_ORG)/plugins-$${PLUGIN_OWNER}-$${PLUGIN_NAME}:$${PLUGIN_VERSION} \
		$(<D); \
	mkdir -p $(dir $@) && touch $@; \
else \
	echo "skipping $${PLUGIN_OWNER}/$${PLUGIN_NAME}"; \
fi

repo: bufbuild/plugins


text:
Add same product name example

Current dependencies on/for this PR:

main

PR #153

PR #157

PR #158

PR #159

PR #160

PR #165

PR #185   (this PR)

PR #186


text:
samples/multi-module: examine module registration a bit

LGTM

repo: bytecodealliance/wasm-micro-runtime


text:
Enable wamrc --enable-segue=<flags> to provide detailed options

what does "segue" stands for?

what does "segue" stands for?

Hi, the segue is an optimization technology to use the x86 segment register to improve performance, currently it is under testing, we may make it as an option in the feature.

authors: wenyongh yamt


text:
g3proxy: add h2 testcase

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

Project coverage is 25.42%. Comparing base (6add25a) to head (1fe5281).

:exclamation:  There is a different number of reports uploaded between BASE (6add25a) and HEAD (1fe5281). Click for more details.
HEAD has 1 upload less than BASE

Flag
BASE (6add25a)
HEAD (1fe5281)

g3proxy
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #382       +/-   ##
===========================================
- Coverage   48.80%   25.42%   -23.39%
===========================================
  Files        1091      658      -433
  Lines       98759    48485    -50274
===========================================
- Hits        48202    12325    -35877
+ Misses      50557    36160    -14397

Flag
Coverage Δ

g3proxy
?

lib
25.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests

authors: codecov-commenter zh-jq-b


text:
fix: process-compose invokation (use values from config as defaults)

Avoid:
❯ devenv up
Error: invalid argument "--tui=" for "-p, --port" flag: strconv.ParseInt: parsing "--tui=": invalid syntax

repo: cachix/devenv


text:
貌似失效？

自动完成模块用的是Linux的定时任务，默认是每周的周一下午一点，如果是windows运行需要自己写一下定时任务模块

不是，貌似获取不到截图链接了

报错信息提供一下


text:
Add Typescript types to Saving

Saving has already been updated.

repo: carbon-design-system/ibm-products


text:
airfile.rb: updated vendor comment

Merged as 97e4879a5ed73c95b07c615efe68446e3a6e1d80.

repo: caskroom/homebrew-cask


text:
build: upgrade to yarn v4

What if we... used pnpm instead? 👀

authors: nekowinston uncenter

repo: catppuccin/vscode


text:
Implement design specification

I pushed a new commit to main directly because all my changes have been reviewed by both of you. The changes are adding slice_test.py, removing pipeline_test, and fixing a small pylint issue. Thank you!

I highly appreciate your update Corbin! Again, I believe we should enumerate all pipeline methods with some tiny details. Thanks!

authors: ccharp nickjsr

repo: ccharp/dplyPY


text:
submission

You're still being flagged for duplicates: All combinations of task ID
column/output_type/output_type_id values must be unique.  Rows containing
duplicate combinations: 93, 94, 95, 96, 97, 98, 99, 100, 101, 102, 103, 104,
105, 106, 107, 108, 109, 110, …, 114, and 115  - Alex

repo: cdcepi/FluSight-forecast-hub


text:
Modeling for Group of Analyses

@bhavinbusa As discussed, I have uploaded an updated version of the model that includes "listOfPlannedAnalyses" that is created using a modified version of the NestedList class that now allows an "analysis" to be specified (as a reference to an instance of an "Analysis" class) in the same way that an "output" can be specified (as a reference to an instance of an "Output" class).
These changes have also be represented in the updated cmap.
A "Group of Analyses" is "sublist" within the "listOfPlannedAnalyses".
I have also updated the example yaml/json files to include a "listOfPlannedAnalyses" based on the CDISC Pilot SAP.  A few of the analyses in the STUDY POPULATION section have references to defined analyses (defined in the new "analyses" attribute of the ReportingEvent), but these definitions are not complete yet - they currently only include id and version, and (when needed) references to an analysis set and grouping.

It seems that there is no set/defined specification for the types of categorization of analyses/outputs that might be necessary.  The following are a few examples of ways in which analyses/outputs might be categorized:

"Group of Analyses"

Population Description
Safety
Efficacy

"Analysis Type", which seems to be an indication of the domain for the source data

Demographics
Adverse Events
Vital Signs

Source data class (as this might affect the types of analyses that are expected)

Events
Findings
Subject-level?

Type of analysis (some of which might only be applicable to certain classes of input data) and may indicate the structure of analysis dataset expected/required (e.g., BDS, OCCDS, TTE, etc)

Occurrence
Time to event
Change from baseline
Shift table

Importance for CSR

Top-line analysis
Other

Estimand Analysis Type

Primary Estimator
Sensitivity Analysis
Supplementary Analysis

It will be difficult to include specific representations of the categorizations in the model without having a clear idea of how categorization is performed and how the categories are used/interpreted.  However, in order to allow categorization to be applied, I have added "analysisCategorizations" as an attribute of the ReportingEvent with a structure that allows id and label to be defined for each categorization, and for each category within the categorization.  I have also included "subCategorizations" within Category to allow nested categorizations to be defined.  I have also added "categoryRefs" as a multi-valued attribute for both analyses and outputs, which will allow multiple categories to be referenced.

Examples files have been uploaded showing definition and use of analysis categories:

cdiscpilot01-sprint6.yaml
cdiscpilot01-sprint6.json

The following categorizations/sub-categorizations are defined in the example files:

Categorization: Group of Analyses

Category: Population Description
Category: Safety
Category: Efficacy

Categorization: Analysis Data Class

Category: Subject-level

Sub-categorization: Subject-level Data Type

Category: Demographics

Category: Events

Sub-categorization: Events Data Type

Category: Adverse Events
Category: Clinical Events
Category: Disposition

Sub-categorization: Events Analysis Type

Category: Occurrence
Category: Time-to-Event

Category: Findings

Sub-categorization: Findings Data Type

Category: Vital Signs
Category: Laboratory Tests

Sub-categorization: Findings Analysis Type

Category: Change from Baseline
Category: Shift Table

Categorization: Estimand Analysis Type

Category: Primary Estimator
Category: Sensitivity Analysis
Category: Supplementary Analysis

To show how these would be used, references to some of these categories have been added for the first defined analysis and the first defined output.

authors: ASL-rmarshall bhavinbusa

repo: cdisc-org/analysis-results-standard


text:
Update to habitat 0.74.0

Resolved in https://github.com/chef-cookbooks/habitat/pull/160

authors: chef-ci jonlives

repo: chef-cookbooks/habitat


text:
ci: add nodejs CI

I think we want automated tests for develop too, right?

Looks good. Will merge after checks complete, let's see how this works :)

authors: dvirberlo eternaleclipse

repo: chess-mazes/chess-mazes


text:
fix(console-exporter): rm redundant warning for debug exporter

Codecov Report

Merging #83 (d83cdc9) into main (ce03de6) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head d83cdc9 differs from pull request most recent head 2426a71. Consider uploading reports for the commit 2426a71 to get more accurate results

@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   87.87%   87.87%
=======================================
  Files          13       13
  Lines         495      495
=======================================
  Hits          435      435
  Misses         60       60

Impacted Files
Coverage Δ

cisco_telescope/options.py
97.72% <100.00%> (ø)

Continue to review full report at Codecov.

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

authors: codecov-commenter nemoshlag

repo: cisco-open/otel-py


text:
Added Lithuanian translations for entities

Thanks!

authors: cjaliaga drawgas

repo: cjaliaga/home-assistant-aquarea


text:
Test AWS provider v5

/terratest

repo: cloudposse/terraform-aws-ecs-atlantis


text:
docs: translate hertz/tutorials/example

ref: https://github.com/cloudwego/hertz/issues/48

authors: GuangmingLuo zhoulilily


text:
Fix rateParam bug in combineCards from py3 port

@amarini re: #770


text:
integration of qdrant --draft

is now integrated in service neural search

repo: code-kern-ai/embedders


text:
Remove CC registry

:+1:

authors: GordonDiggs wfleming


text:
refactor: delete env file

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

repo: codefactoryhu/pulumi-starter-kit


text:
sort on bills by clicking on column headers

this would simplify the search/sort process for bills by allowing us to remove the SortSelect option in BillSearch.tsx

Need to check with @jamesvas5307 what affordance we should use with the card-based layout

authors: alexjball jkoren

repo: codeforboston/advocacy-maps


text:
Add new volunteer Nkiruka Ebele

Hey @nkirukka looks good thanks for sharing :)

Thank you @nkirukka :)

authors: codewithshubhi codewithvoid nkirukka

repo: codewithvoid/AskDevs


text:
Replaced the 'µ' with 'u' in "µPickle"

Why?

Why?

It was the only occurrence of the word "µPickle". In all other places this librarry is called "uPickle".

authors: lefou nafg

repo: com-lihaoyi/upickle


text:
openpilot longitudinal: fix long press to increment by 5 mph

Reverted the change for now in #24161. This bug was asking to get written with the way it was written, so I added a comment too. This PR even further complicates this code - can we simplify it instead?

Will revisit with good tests first

authors: adeebshihadeh sshane

repo: commaai/openpilot


text:
Setup GitHub actions for main branch and tags

Closing, as this was already integrated on #12


text:
Fix outdated containerfile link and add link about vulkan

LGTM

authors: praveenkumar rhatdan

repo: containers/ai-lab-recipes


text:
build: flux on azure with usernetes

This was mostly OK, and Lise figured out infiniband I think. I'm going to merge for now, anticipating we will pick up more work on it when we get another Azure account setup.

repo: converged-computing/flux-usernetes


text:
Fixed the notes including the header.

Pull Request Test Coverage Report for Build 3291855368

15 of 16 (93.75%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 95.975%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

generate_changelog/cli.py
5
6
83.33%

Totals

Change from base Build 3291714952:
-0.4%

Covered Lines:
1023

Relevant Lines:
1056

💛  - Coveralls

Pull Request Test Coverage Report for Build 3291855368

-1 of 16 (93.75%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 95.975%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

generate_changelog/cli.py
5
6
83.33%

Totals

Change from base Build 3291714952:
-0.4%

Covered Lines:
1023

Relevant Lines:
1056

💛  - Coveralls

authors: coordt coveralls

repo: coordt/generate-changelog


text:
*: add basic readme

LGTM

authors: crawford jonboulle

repo: coreos/fuze


text:
"scope" for eo-maven-plugin "foreign" catalog

@rultor release, tag is 0.10.7

@yegor256 Done! FYI, the full log is here (took me 9min)

repo: cqfn/eo


text:
generate coverage during test-run so that codecov works

fabulous! (ack)

authors: chaudum mfussenegger

repo: crate/crash


text:
Merge changes from main branch

just take this and merge into your template manager plugin. no need for 2 plugins.

Are you sure? I guess it is related

yes, I'm sure.

authors: aljawaid creecros

repo: creecros/TemplateTitle


text:
[Feature] Payment Gateway Stripe

Fixes #7

repo: crescents-stack/boikontho


text:
Fix TestLOBPCG.test_maxit_None CUDA 12.2 CI failure

/test cuda122

/test cuda122

repo: cupy/cupy


text:
Create devcontainer.json

Dupe: #50

repo: curl/curl-for-win


text:
Fix the error reported by OTP and add exception handling to the library

Codecov Report
Attention: 108 lines in your changes are missing coverage. Please review.

Files
Coverage Δ

...ht/trap/datasource/gesture/TrapGestureCollector.kt
87.09% <100.00%> (+0.88%)
:arrow_up:

...ight/trap/datasource/gesture/TrapTouchCollector.kt
96.15% <100.00%> (+0.32%)
:arrow_up:

...ht/trap/datasource/gesture/TrapPointerCollector.kt
69.09% <88.88%> (+1.74%)
:arrow_up:

...ght/trap/datasource/gesture/TrapStylusCollector.kt
89.79% <88.88%> (-0.91%)
:arrow_down:

...cursorinsight/trap/datasource/TrapWiFiCollector.kt
70.88% <28.57%> (-4.46%)
:arrow_down:

.../datasource/gesture/internal/TrapWindowCallback.kt
87.50% <37.50%> (-10.12%)
:arrow_down:

...ap/datasource/sensor/TrapAccelerometerCollector.kt
58.62% <30.76%> (-10.95%)
:arrow_down:

...ght/trap/datasource/sensor/TrapGravityCollector.kt
58.62% <30.76%> (-10.95%)
:arrow_down:

...t/trap/datasource/sensor/TrapGyroscopeCollector.kt
58.62% <30.76%> (-10.95%)
:arrow_down:

...rap/datasource/sensor/TrapMagnetometerCollector.kt
61.53% <30.76%> (-13.47%)
:arrow_down:

... and 3 more

:loudspeaker: Thoughts on this report? Let us know!.

I've uploaded a new version

authors: codecov-commenter denessapi

repo: cursorinsight/ci-trap-android


text:
Allow to reuse previously uploaded data or model results for new jobs

Should be doable now with https://github.com/cybergis/cybergis-compute-core/pull/81 because it gives each HPC a /compute_shared mount.
When the PR hits production we will need to update documentation though so that users can utilize it.

authors: alexandermichels zhiyuli

repo: cybergis/cybergis-compute-core


text:
Sir berg patch 1

Sorry my bad! Wrong target branch I guess 😅


text:
Voting options must be unique

In the proposal creation view, add a validation to ensure that we don't have any repeating strings in the voting options (and display an error if there are repeating strings).

Right now, the validation function, trims the strings and check that they're unique. If not, it returns an error.

Approved

authors: EightRice fabiolalombardim thenerdcat

repo: dOrgTech/homebase-app


text:
feat: added labelChildren in TextField components

Good job!

repo: dankocher/metamodern-ui


text:
[library] Updated library to use production api endpoint to avoid cors auth error

Thanks!

authors: dwnoble shifucun

repo: datacommonsorg/website


text:
test: useCurrencyConfig test

Pull Request Test Coverage Report for Build 8547530959
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.4%) to 44.562%

Totals

Change from base Build 8536477965:
1.4%

Covered Lines:
465

Relevant Lines:
803

💛  - Coveralls

authors: coveralls thisyahlen-deriv

repo: deriv-com/traders-hub


text:
Release v2024.7.2

Closed by #160.

repo: deshima-dev/demerge


text:
Add two new sender features that has been added in Digipost.

We can now do this:
final SenderInformation senderInformation = sendClient.getSenderInformation(SenderId.of(SENDER_ID));

System.out.println(senderInformation.hasEnabled(SenderFeatureName.MASSEUTSENDELSE));
//true

repo: digipost/digipost-api-client-java


text:
Update PQS docs 2.10.0

Thanks for tagging me on this PR. The recent changes that I proposed are reflected in this update to the 2.10.0 docs. LGTM!

authors: filmackay wallacekelly-da


text:
Update deps

Addresses #79.

repo: digitalbazaar/zcap


text:
Prometheus integration

 Prometheus integration

 Prometheus integration

repo: dipdup-net/dipdup-py


text:
CI for Node

Initializes with two players


text:
Bump: repository-dispatch

Will be tracking and implemented in https://github.com/dnswarden/blocklist-staging/pull/28

authors: GiddyGoatGaming bhanupratapys

repo: dnswarden/blocklist-staging


text:
add way to change base value

for v0.1.0-alpha

repo: dotRarufu/spacify


text:
Volume chart

thanks for review and discussion @timothyhoover !

repo: doug-patterson/FMR-SearchKit-React


text:
Add a Codacy badge to README.md

@all-contributors

@all-contributors please add @dreambo8563 for code, doc and infra


text:
feat: support saga

能不能在对应 https://github.com/dtm-labs/dtmcli-java-sample  中提供一个saga的demo

可以，晚点补上

https://github.com/dtm-labs/dtmcli-java-sample/pull/15

authors: catcherwong li-xiao-shuang

repo: dtm-labs/dtmcli-java


text:
fix: path in api make target

@vladdoster thanks!

authors: duarteocarmo vladdoster

repo: duarteocarmo/boilerplate


text:
feat(proxy): Support UNSELECT

Pull Request Test Coverage Report for Build 7502569873

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.678%

Totals

Change from base Build 7491346348:
0.0%

Covered Lines:
477

Relevant Lines:
827

💛  - Coveralls

authors: coveralls duesee

repo: duesee/imap-flow


text:
Create Sign Up Screen

Finish Developing the Sign-Up Screen .


text:
Add new_XXXX to BatchedArray

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (e4da4aa) 100.00% compared to head (a7a8f1d) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #336   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           14        14
  Lines         1396      1414   +18
  Branches       154       157    +3
=========================================
+ Hits          1396      1414   +18

Files Changed
Coverage Δ

src/redcat/array.py
100.00% <100.00%> (ø)

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

authors: codecov-commenter durandtibo

repo: durandtibo/redcat


text:
Add cmake support and remove unneeded files

Adding Cmake Support


text:
testing font

Hi, what did you try to do ? Need any help with something ?

Hi, what did you try to do ? Need any help with something ?

Pls ignore, testing something, accidentally created pr while checking diff.

Alright no problem :)

repo: dylan-power/dinosaur-exploder


text:
Removed API Key

I have closed this PR because I was working on the main branch and I have now changed it to develop. So created a new PR from develop branch.

repo: eatozee/nftoupon


text:
Write best practice opensource PR guide

could be closed IMHO we already have a advisory for PR´s https://eclipse-tractusx.github.io/docs/oss/code-reviews#general-procedure-for-a-pr or should we update the description to it?

In my opinion the PR advisory is good enough, I would close this issue.

I think the same and will now actually close it

authors: FaGru3n Siegfriedk almadigabor

repo: eclipse-tractusx/sig-infra


text:
Release/3.5.0

sorry. my mistake

repo: eclipse-tractusx/traceability-foss


text:
Add Buildkite skeleton

LGTM! Thanks @picandocodigo

authors: ezimuel picandocodigo

repo: elastic/enterprise-search-php


text:
cmig is a default dir

@h1alexbel thank you for reporting this. I'll assign someone to take care of it soon.

@l3r8yJ this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2023-08-28T14:29:15.660457.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

repo: eo-cqrs/cmig


text:
flutterMapNavigation  event.BannerInstruction is Never trigerred.

Same as #258

authors: Akiitp eopeter


text:
test: Test PR from an unknown account -FOR TESTING ONLY - DO NOT MERGE

This PR was only to test if an external contributor is able to open a PR without scheduling the CI automatically. This works, someone frrom Epinio team should activate the CI, then the PR could be reviewed and accepted, then merging.
Close this fake PR.

authors: ldevulder ldevulder-SUSE-Test

repo: epinio/epinio


text:
Collated dependabot changes for CI actions

Codecov Report
Base: 61.61% // Head: 8.26% // Decreases project coverage by -53.35% :warning:

Coverage data is based on head (dd35b9d) compared to base (683c40d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #1921       +/-   ##
==========================================
- Coverage   61.61%   8.26%   -53.36%
==========================================
  Files         179      93       -86
  Lines       14523   10111     -4412
==========================================
- Hits         8949     836     -8113
- Misses       4468    9220     +4752
+ Partials     1106      55     -1051

Flag
Coverage Δ

acceptance-api
?

acceptance-apps
?

acceptance-cli
?

unittests
8.26% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

main.go
0.00% <0.00%> (-100.00%)
:arrow_down:

internal/api/v1/authtoken.go
0.00% <0.00%> (-100.00%)
:arrow_down:

internal/application/secret.go
0.00% <0.00%> (-91.49%)
:arrow_down:

internal/services/client.go
0.00% <0.00%> (-86.96%)
:arrow_down:

internal/api/v1/application/match.go
0.00% <0.00%> (-82.36%)
:arrow_down:

internal/configurations/configurations.go
0.00% <0.00%> (-82.06%)
:arrow_down:

helpers/string.go
0.00% <0.00%> (-80.00%)
:arrow_down:

internal/api/v1/application/delete.go
0.00% <0.00%> (-78.58%)
:arrow_down:

internal/auth/certs.go
0.00% <0.00%> (-77.78%)
:arrow_down:

internal/cli/usercmd/chart.go
0.00% <0.00%> (-77.19%)
:arrow_down:

... and 155 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 at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

Failure is known rails app issue with ruby version #1919

authors: andreas-kupries codecov-commenter


text:
feat(datepicker): highlight cells with data in

Could you link the PR?

authors: collinlokken pbullhove

repo: equinor/dm-core-packages


text:
Text Block component - link to another language

No changes needed. Link to different language link works for ingress and content in text block component. Verified for events, news, magazine and topic content.

Can confirm, and the action field for the text block component (link/image/file) has it through the linkSelector - perfect 👍

authors: SvSven nilsml padms

repo: equinor/energyvision


text:
feat: set minimum number of elastic instances

Tests pass 🎉

repo: equinor/terraform-azurerm-function-app


text:
Fix authentication to private repos rebar3 hex user auth --repo [REPO]

Hey @derekkraan can you provide steps to reproduce the issue?

Hey, @starbelly, yes. This is the short version since I'm just on mobile now. I am working on a private repo for hosting (among others) Erlang packages. Running rebar3 hex user auth --repo codecodeship results in the auth data being saved for hexpm (in the rebar3 state) and not for code code ship, resulting in further calls (such as rebar3 hex publish --repo codecodeship) failing due to missing keys.
I hope this makes sense. I can do a better write-up on Monday when I'm back in front of a computer if necessary.

@derekkraan Ahhh I see, I literally just reproduced the issue right before you replied. This LGTM!

This bug fix reveals some other latent bugs.

We're not setting api_organization to undefined in fetching / building hex config at runtime (line 241 of rebar3_hex_config.erl)
We're not setting api_repository for non-hexpm config, this is included by default for hexpm default config, but not so for this case. This is probably per testing with nothing but hexpm and in that referring to it as hexpm vs an alternative name.

Of course, there's a fun dialyzer error to fix as well :) I'm happy you have opened this PR though so we can solidify this use case!

@starbelly there seems to be some confusion in the codebase. I believe that dialyzer is complaining about the attribute name, which doesn't exist on the type config() (only repo_name). I am not really sure what to do about this, since rebar3_hex is using name all over the place. Is this a bug? Do I need to set both name and repo_name in rebar.config?
I added some code to make sure that api_organization and api_repository are both being populated with undefined when that is appropriate.

Setting name appears to be enough to also set repo_name. I think this PR is good to go.

@derekkraan LGTM! (for real), this helped uncover yet another bug, but I'll fix that up in another PR. Thank you!

authors: derekkraan starbelly


text:
[New practice exercise]: Rational numbers

Thanks!

repo: exercism/crystal


text:
Add kindergarten-garden

And that's all the 48in24 exercises.
https://exercism.org/challenges/48in24/implementation_status/jq

repo: exercism/jq


text:
Skip numpy module copy test for pypy

For the failing tests, cf https://github.com/explosion/confection/pull/11

authors: adrianeboyd svlandeg

repo: explosion/confection


text:
Create testcafes

Specifically for listing, editing and adding


text:
Fix a few URLs to the right website

@Meakk please quick review


text:
v5.2.1

特にやることなさそうなので明日くらいにはリリースしたい


text:
test: rename test to internal, external from puppeteer, non-puppeteer

Codecov Report

Merging #71 (89e0fbd) into master (0725aa4) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #71   +/-   ##
=======================================
  Coverage   91.42%   91.43%
=======================================
  Files          21       21
  Lines         490      502   +12
  Branches       84       85    +1
=======================================
+ Hits          448      459   +11
- Misses         42       43    +1

Flag
Coverage Δ

unittests
91.43% <ø> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/services/crash-service.ts
89.65% <0.00%> (+1.41%)
:arrow_up:

Continue to review full report at Codecov.

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


text:
lets disable groovy and kotlin by default for the release profile

[merge]


text:
Fix postcss.config.js snippet code

Thanks!

authors: jonovs necolas

repo: facebook/react-strict-dom


text:
State Recovery management implementation

Codecov Report

Merging #477 (2a7f4a3) into main (d2f498b) will increase coverage by 0.2%.
The diff coverage is 15.6%.

@@           Coverage Diff           @@
##            main    #477     +/-   ##
=======================================
+ Coverage   12.0%   12.3%   +0.2%
=======================================
  Files         29      31      +2
  Lines       8483    9048    +565
=======================================
+ Hits        1021    1110     +89
- Misses      7462    7938    +476

Impacted Files
Coverage Δ

src/checkpointd/opts.rs
0.0% <0.0%> (ø)

src/cli/command.rs
27.4% <0.0%> (-1.5%)
:arrow_down:

src/error.rs
0.0% <0.0%> (ø)

src/farcasterd/runtime.rs
0.0% <0.0%> (ø)

src/lib.rs
9.3% <0.0%> (ø)

src/swapd/runtime.rs
0.0% <ø> (ø)

src/walletd/runtime.rs
0.0% <0.0%> (ø)

src/rpc/request.rs
15.2% <17.9%> (+0.2%)
:arrow_up:

src/checkpointd/runtime.rs
30.7% <30.7%> (ø)

src/cli/opts.rs
46.5% <33.3%> (-0.4%)
:arrow_down:

... and 5 more

Continue to review full report at Codecov.

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

authors: TheCharlatan codecov-commenter

repo: farcaster-project/farcaster-node


text:
chore: editor format config

Thanks!

repo: farm-fe/farm


text:
register cemtatli

register


text:
pada input pengaduan, bila tersave juga tidak terhapus data sebelumnya

done


text:
upgrade to new model kit, finish sdk

Sorry Gábor for my request changes :D but good news these are super small ones :D

No worries :P

repo: feather-modules/feather-user-module


text:
📈 Instrument with fh-component-metrics

@paolobueno Should we merge this back to master, or just use it from the branch, like we do for the demo cloud app? I think this one could be merged more easily, since it doesn't contain the fork of fh-mbaas-api. What do you think?

@grdryn my :+1: on merging this to master, though I think it would require a readme update.
I can see that fh-component-metrics has some relevant info which could be linked to, but atm it only mentions 'RHMAP components' as opposed to a cloud app's context and seems a bit hard for customers to consume.
Do SaaS and on-prem deployments include influx+grafana? If so we could add their addresses\FH env-vars as extra defaults and mention them on the readme for this or fh-component-metrics'.
I'd also like @nialldonnellyfh to take a quick look if possible.

Do SaaS and on-prem deployments include influx+grafana? If so we could add their addresses\FH env-vars as extra defaults and mention them on the readme for this or fh-component-metrics'.

@paolobueno they don't include influx+grafana. @wei-lee is actively working on a statsd backend for fh-component-metrics though, here: https://github.com/feedhenry/fh-component-metrics/pull/6

Rebased on latest master

authors: grdryn paolobueno


text:
test issue

Thanks @5A11!

repo: fetchai/ledger-subquery


text:
add display name of contact

[ ] Transform contact entity to id, name, displayName and link

[ ] Migrate

[ ] Edit dependencies (Data, DTO, Controlers)

[ ] Test with postman

[ ] Resolving bugs

Time: 12hr

Deadline: 19/02

authors: Hahhhayam just-hoshinon

repo: fictadvisor/fictadvisor-api


text:
Updated privacy policy

Is there a rationale for this change, just so we're not mis-communicating something unintentionally?

repo: files-community/Files


text:
fix: add achive policy

This PR is included in version 1.0.2 :tada:

repo: finisterra-io/terraform-aws-sns


text:
Update _markdown.scss

👷 Deploy Preview for voluble-seahorse-25eb59 processing.

Name
Link

🔨 Latest commit
a38a206a605223f3784a2db4a164adf2297c593d

🔍 Latest deploy log
https://app.netlify.com/sites/voluble-seahorse-25eb59/deploys/652ae658d82c6400080924d8

repo: finolacahill/bluise


text:
Documentation is obsolete

Which parts of the documentation are you referring to specifically?

As far as I remember, not all of this framework's features are outlined in the documentation.

authors: LaurenzBeck firefly-cpp

repo: firefly-cpp/NiaAML


text:
Schedule Votte sync

Thanks @benhuangbmj !
We discussed, and chose 1. Just link

Thanks @benhuangbmj !
We discussed, and chose 1. Just link

Got you, @wilrnh . With this in place, I figure my work in the next couple of sprints will lean more toward grinding in Votte.

authors: benhuangbmj wilrnh


text:
Remove references to Observatory from flags and source code

@bkonyi is this one unblocked?

I think so, I just need to find some cycles to deal with it... :-)

authors: bkonyi christopherfujino

repo: flutter/flutter


text:
supports aarch64-apple-darwin: targets Apple silicon

Github CI doesn't support arm macs yet in my knowledge. I am waiting for those myself, but x86 darwin package works on mac just fine, without any performance issue.
I will still release arm versions as soon as github CI starts supporting arm macs.

@Sparkenstein Apple Silicone runners now available on Github.
Please check https://docs.github.com/en/actions/using-github-hosted-runners/about-github-hosted-runners/about-github-hosted-runners

Ah great, so macos-latest is the label for arm version, I am already using that. does that mean existing packages are already built for arm?

I think arch needs to be set, I can give it a stab. Will share the pr over the weekend.

Aarch64 images are now available to download from release tab

Weird, worked normally on my m2 mac. maybe your download was actually corrupt or something?

+1

Most likely this is because there are no version change in the release. will implement some new feature and do a version upgrade and release one more time this weekend. that should technically resolve the issue.
i am using the same release.yml for my another project fosslife/truthy. it works there so far I am assuming it should work here too

Apologies, it's the OSX shenanigans, I have only work macbook, I can't generate certificate from that. will try to get a resale macbook soon.
For now
xattr -r -c /Applications/dev-tools.app

Is the solution as mentioned by @reslear.
will mention this in README too.

authors: Puujee0238 Sparkenstein nileshmali reslear

repo: fosslife/devtools-x


text:
Replace default transaction hash in deploy with actual calculations

If it is not available in transaction info then it is not a problem. Can you check it @Radinyn and close if it is not a problem? 🥺

I think what can be problematic is that we are using same class hashes for multiple transactions. E.g. declare always uses TransactionHash::default().

authors: MaksymilianDemitraszek Radinyn cptartur

repo: foundry-rs/starknet-foundry


text:
Evaluate mobile camera capture

Have it working 'Good enough' for sandbox and testing
Included buttons for:

Turn Camera On
Turn Camera Off
Capture Image
Clear Image
Include Datestamp

repo: fps-solutions/FPS-Photo-Form


text:
Fix uploading artifact to existing artifact

Is this only needed for 7.9?

Is this only needed for 7.9?

No, every branch that has a matrix workflow.
Do you think this artifact name is good, or should I do something like java8/FrankDoc?

authors: philipsens tnleeuw

repo: frankframework/frankframework


text:
Feature/cloudwatch

Opening a new one.

repo: fvazquez-caylent/terraform-aws-emr


text:
TravelPerk

please fix the linting and codefactor issues and assign the PR

repo: fylein/fyle-integrations-settings-app


text:
Create designs for making player dice pools visible to all players

@gareththegeek I have forgotten what exactly we discussed here.
We want to reimplement a dice validation check, right? At least two dice that aren't d4s? When players have a valid dice pool they are considered to have "joined" the contest?
Then valid dice pools will show up for all players.
Do I have that right?

repo: gareththegeek/rollagon


text:
Fix: typo

Closing in favor of https://github.com/getsentry/sentry-php/pull/1725.

authors: Lisio cleptric

repo: getsentry/sentry-php


text:
Resolve CSV loader issue

Need to parse an external CSV to seed data for current LCS players.

repo: gflujan/LoLProWinsApp


text:
logo change

kk

kk

repo: gitpod-io/template-typescript-react


text:
Add basic content for documentation

@benborgers I removed the right side table of contents since I want to keep the docs a bit simpler. We don't have that much content so we can do two columns on desktop.

repo: glow-xyz/nftoken


text:
Django AD Integration

[ ] Login to Django using cyber.local credentials and a range security group.,

django


text:
Fix retry job

Codecov Report

Merging #64 (b5d9cc0) into master (f1ec2d0) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
+ Coverage   62.38%   62.39%   +0.01%
==========================================
  Files          15       15
  Lines        2608     2609       +1
==========================================
+ Hits         1627     1628       +1
  Misses        717      717
  Partials      264      264

authors: codecov-commenter goccy

repo: goccy/kubetest