text:
Add a clear search button in the input field.

The approach to implement the clear search button is very similar to other clear search button like Search streams and Search people and it is kept consistent in every commit.
Discussion : https://chat.zulip.org/#narrow/stream/101-design/topic/No.20close(cross).20button
Fixes #17263
This is the list of the sections where the cross clear search button have to be implemented:

[x] SETTINGS / UPLOADED FILES  29bf5716f57a8167aaaa6222fe4675786803cd7f
[x] SETTINGS / MUTED TOPICS  45913aa5ba2f2e83351dcbf471c4619c7ab2f2f9
[x] SETTINGS / CUSTOM EMOJI  b670c5db079677382998da2fe5063caf75b5e439
[x] SETTINGS / USERS  cac139eeb1ea376a56fa1c76ad64e17331ca3923
[x] SETTINGS / DEACTIVATED USERS  5fff42b6893847ba12a57a8c3e23557fcaedb4db
[x] SETTINGS / BOTS  c7b875ec286ca051e523079c6b34562316cc2161
[x] SETTINGS / DEFAULT STREAMS  79dd5b3b1ed724a156022be7436b2e32e4d15a46
[x] SETTINGS / LINKIFIERS  e5f17193f88a38aff7d7200b67037c9e43e0cdd9
[x] SETTINGS / INVITATION  1f7f0c93c86010d3bb8d1b13c82887d9974779bb
[x] Manage streams/Stream settings/Stream membership  4d3b60ccf4a1e9946495252c44828c88446aceb8

Testing plan:  (This PR is also tested on the local host by @smitpatel24)

Run all the automated tests and it pass on all the commits.
Checked all the design factors(both in night/day mode) for all the 10 clear search buttons window.
Checked working of clear search button :
a. Clear search button should clear the input of the input box.
b. After clearing the input the table should be reloaded.
Checked all the new classes names and id names spellings.

Some points that I have kept in mind to have consistency:

Follow exactly the similar html code everywhere that is inspired from our old code of stream_search_section and user_search_section.
Always write the js code in the same function where we create a table for that respective window.
Call the function which contains the code to create table.

GIFs or screenshots:

I have fetched the branch locally with the change @nikhilmaske-2001 has made to review the buttons on my localhost. I have manually tested each button and they are working as expected.

Thanks @smitpatel24 . @timabbott PR is ready for review and all the necessary information is available in the description above.

@nikhilmaske-2001 do you plan to close this in favor of #19578?

@YashRE42 No, This PR is of no use and I am closing this PR.
#19578 is the PR for #17263.


text:
Fix compose top keyboard focus order.

Discussion - https://chat.zulip.org/#narrow/stream/9-issues/topic/compose.20tabbing.20focus

This look great to me, merged, thanks @amanagr!

source: gharchive/pull-request

authors: amanagr timabbott

repo: zulip/zulip

license_type: permissive

license_source: github-api


text:
message_list_view: Remove dead next_is_same_sender code.

The next_is_same_sender has no effect on the CSS of the message displayed, and the JS changes seem to have no effect too.
See cc8021a742491a04519544bda571206e0e23a842 for more details.
discussion: https://chat.zulip.org/#narrow/stream/6-frontend/topic/next_is_same_sender

Merged, thanks @amanagr!


text:
Make parameters optional in PATCH user_groups/int:user_group_id.

View that handled PATCH user_groups/<int:user_group_id> required
both name and description parameters to be passed. Due to this
clients had to pass values for both these parameters even if
one of them was changed.
To resolve this name description parameters to
PATCH user_groups/<int:user_group_id> are made optional.
Fixes: CZO link
Screenshots and screen captures:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

We need to document API changes in the API changelog. The new application feature tutorial and OpenAPI docs pages on ReadTheDocs cover how to do this, or just use git log --follow -p api_docs/changelog.md to get some examples.

I wanted to ask if we need a mention about this in changelog.md as a Feature level entry?

Yes, any change to the actual API interface should be documented there. It may not be important, but it's less work to just consistently document them than be constantly making decisions for each API change about which are "important enough" to document. (It's only a tiny bit of work to add Changes entries etc., once one knows how)

@timabbott I've updated the pull request to mention changes in api_docs/changelog.md and also added Changes entries for this endpoint in zerver/openapi/zulip.yaml.
This is ready for another review.

Merged, after some wording adjustments in the API docs, thanks @m-e-l-u-h-a-n!

authors: m-e-l-u-h-a-n timabbott


text:
ts: Migrate avatar.js module to TypeScript.

Migrates avatar.js module to TypeScript.

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

lgtm aside from https://github.com/zulip/zulip/pull/26624#discussion_r1314410686

Updates removed the default value for the max_file_upload_size param for the function upload_widget.build_direct_upload_widget.

Merged, thanks @Lalit3716 and @PIG208!


text:
stream_edit: Move set_stream_property to new stream_settings_api module

(78, 35) → (77, 36). Discussion.

Cool, yeah this seems like progress, merged, thanks @andersk!

authors: andersk timabbott


text:
analytics: Add mobile pushes forwarded column to remote activity page.

1st commit:

Adds a column to the remote servers activity page for a 7-day count of forwarded mobile push notifications from
RemoteInstallationCount for each server, which will be None if there is no data for the remote server with that ID.
This expands the current query for the new data since we're already getting data on all remote servers here.
Manually tested by adding a remote server and countstat via populate_analytics_db.py and rebuilding the dev database (tools/rebuild-dev-database) with those changes and comparing the query data to the database via ./manage.py dbshell.

populate_analytics_db for testing
replace FixtureData and insert_fixture_data function with below:
        import uuid

        from zilencer.models import RemoteInstallationCount, RemoteZulipServer

        remote_server = RemoteZulipServer.objects.create(
            hostname="zulip-remote.example.com",
            contact_email="admin@zulip-remote.example.com",
            plan_type=1,
            uuid=uuid.uuid4(),
            last_version="7.0",
        )

        FixtureData: TypeAlias = Mapping[Union[str, int, None], List[int]]

        def insert_fixture_data(
            stat: CountStat,
            fixture_data: FixtureData,
            table: Type[BaseCount],
        ) -> None:
            end_times = time_range(
                last_end_time, last_end_time, stat.frequency, len(next(iter(fixture_data.values())))
            )
            if table == InstallationCount:
                id_args: Dict[str, Any] = {}
            if table == RealmCount:
                id_args = {"realm": realm}
            if table == UserCount:
                id_args = {"realm": realm, "user": shylock}
            if table == StreamCount:
                id_args = {"stream": stream, "realm": realm}
            if table == RemoteInstallationCount:
                id_args = {"server": remote_server}

            for subgroup, values in fixture_data.items():
                table._default_manager.bulk_create(
                    table(
                        property=stat.property,
                        subgroup=subgroup,
                        end_time=end_time,
                        value=value,
                        **id_args,
                    )
                    for end_time, value in zip(end_times, values)
                    if value != 0
                )

        stat = COUNT_STATS["mobile_pushes_forwarded::day"]
        remote_server_data: FixtureData = {
            None: self.generate_fixture_data(stat, 1.5, 1, 3, 0.6, 8),
        }
        insert_fixture_data(stat, remote_server_data, RemoteInstallationCount)
        FillState.objects.create(
            property=stat.property, end_time=last_end_time, state=FillState.DONE
        )

2nd commit:

Adds a chart key to the top of the remote activity page. And, combines the support and stats links for a remote server into one column.
Added the chart key as a separate template that's conditionally added to a shared template. This will let us quickly iterate on adding more information to this particular page.
This is a separate commit so as not to delay merging first commit due to these changes.

Screenshots and screen captures:

Updated remote servers activity page

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @laurynmm!

authors: laurynmm timabbott


text:
WIP realm validation: show user if realm is OK

I've created a new branch where I've included all the changes I've done after the review. Below is a comment from another pull request, that I've pushed before that had some extra irrelevant commits in it:

I've submitted some of the changes, but I've left the check for the validity of the email. I am not sure how to proceed further. It's either of these two:
leave just one div where we notify the user about the errors for the email + subdomain
put the original div back where we notify about the email errors and in another div we notify about problems with the subdomain.

Closing in favor of the more current #7750.

authors: elenaoat timabbott


text:
Laravel 9 Version

TODO: Update to Laravel 9

Completed as of commit 91edf0ed304e5d21bb1a94174702de603d5550b9

source: gharchive/issue

repo: zuramai/kelaskita

license: MIT


text:
Equalizer in other way as usual

I have this code:

In this case is equalizer applied on both title classes. I need equalize content class too. Any chance to equalize only the same classes?

try this

maybe that's what you are looking for?

woohaaa.. that was fast! yep.. problem solved :D

the first div is only a helper div to initialize the second equalizer, because you cannot assign 2 equalizer to one element

yep.. I've got it.. I found it also in Docs.. I had to be blind :)
Now I need just watch #7223

repo: zurb/foundation-sites

license: mit

license_source: bigquery


text:
Javascript Error handling for foundation

There should be way to handle Javascript errors in foundation. There are multiple places in code which use something like this:
try {     <js_code> } catch (err) {     console.error(err); }
Which make impossible to report javascript errors in tools like Bugsnag, not just from foundation itself but many times other code not directly related to foundation.

there is hacky way to overload console.error but it is somehow risky, and some errors are reported twice - once in window.onerror event and on console.error event.

This is a great point, I 100% agree.
Do you have any examples of libraries that do this well you can point to?

(cc @zurb/yetinauts)

@stanogurnik @kball
One article to look into before starting on it!
https://www.sitepoint.com/proper-error-handling-javascript/

Yea, you don't need any library for that, simple callback would suffice. Something like
Foundation.registerDefaultErrorHandler(function(level, message, err) { doYourStuff(); // throw err; if you want propagate Error etc. }); would be great.

Would be good to get this in for 6.4

As an aside, @stanogurnik, maybe we should be thinking about replacing callbacks with promises where applicable and as needed.

@paxperscientiam there's no need to use promise there. Promises shine in async code and this is not that case. Only benefit would be common understanding of API but still callback is simpler/better/faster solution.

authors: IamManchanda colin-marshall kball paxperscientiam stanogurnik


text:
fixed dropdown mixin issue

Resolves: https://github.com/zurb/foundation-sites/issues/9836

Thanks for the feedback guys! I updated the PR :) let me know if there is anything else!

Merged!

authors: LaurelBCodin kball


text:
Removing duplicated android top-bar fix

Somehow this block was duplicated with 2 separate commits, removing one...

good catch!

authors: oller rafibomb

repo: zurb/foundation


text:
desktop sign up frame

created the desktop sign up frame, added the color style, added input fields, button, and all the necessary content.

https://www.figma.com/file/LEKdq0JqyuzzOGxv0tZWjh/WalletFriendly?node-id=1245%3A2782

authors: Powelldabig nkirusteph


text:
Route to post a vision

kindly review

Please make this PR to the staging branch instead

authors: Asylcreek mcsepro


text:
chore: add ESLint rule ot ensure all types used in a public CC API are exported

fixes: https://github.com/zwave-js/node-zwave-js/issues/6414

@zwave-js-bot automerge

repo: zwave-js/node-zwave-js


text:
chore: include node 19 in unit tests

Include node 19 in unit testing

Pull Request Test Coverage Report for Build 4253220500

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 23.836%

Totals

Change from base Build 4244486843:
0.0%

Covered Lines:
3818

Relevant Lines:
16918

💛  - Coveralls

repo: zwave-js/zwave-js-ui


text:
[bug] logging doesn't respect containers time / time zone settings

Before submitting a bug please read:
Version
App Version
1.1.1+94ff7ed.94ff7ed
Zwavejs Version
6.3.0-8f22052
Build/Run method

[X] Docker
[ ] PKG
[ ] Manually built (git clone - npm install - npm run build )

Describe the bug
I use the following in docker-compose to ensure the containers time matches the host.
    volumes:
      - ./store:/usr/src/app/store
      - /etc/timezone:/etc/timezone:ro
      - /etc/localtime:/etc/localtime:ro

This is correctly setting the date and time inside the container - confirmed by attaching to container and issuing the date command.
However logs now seem to be in UTC rather than local time.
This was working on 6.2.0 and 1.1.0
To Reproduce
Steps to reproduce the behavior:

set host to have a TZ of not UTC (e.g. LosAngeles)
build docker image from source  with both branches as master
ensure time is mapped as described above
on host use date command and note time
start container and attach to it with  docker exec -it zwavejs2mqtt  /bin/bash
in container use date command - time should match host time
use tail -f zwavejs_1.log - logs are in UTC time.

Expected behavior
Logs should be in local time as per previous builds, this appears to be regression.
Additional context
Previous working scenario was with pre-built docker hub master image, not ones i have built using the provided docker contrib file..

hmm adding this worked (as it is another solution to TZ)
What I don't get is why this wasn't needed on the containers for the last week or so, but is now.
I will retest once next official dockerhub master comes out and see if it is new behavior or dependent on build process.
    environment:
      - TZ=America/Los_Angeles

The TZ var has always been there in Docker containers

i know i think i didn't explain myself
The TZ is a fine workaround to a regression i am observing.
On master container from 4 days ago
      - /etc/timezone:/etc/timezone:ro
      - /etc/localtime:/etc/localtime:ro

works to set time zone for container AND logging was using this time
on the containers i built using Dockerfile.contrib it works to set the timezone in the container but logging ignores container time and uses UTC.

on the containers i built using Dockerfile.contrib it works to set the timezone in the container but logging ignores container time and uses UTC.

That file is to build images for testing purposes and it's not really maintained, @larstobi look at this if you want/have time, nothing so important btw :)

I think using the TZ variable is not only a workaround, but the correct solution. :-)

cool, glad i got an explanation :-)
@larstobi  i like the file mapping as in a pre-defined docker-compose it works for just about every system
but happy to snap to TZ as that seems to be the consensus, i can add TZ to our example dcker-compose in the repo.

i can add TZ to our example docker-compose in the repo if that works for folks?

Great idea! 😊

@scyto Go for it! Thanks :)

authors: larstobi robertsLando scyto


text:
Finding only a few of many QRCode symbols in one image

Hi, I'm trying to use zxing in a project where I need to split some information over multiple qr codes.
I cannot decode the following image (27 qr codes containing meaningless strings like "AAAAAAAA"):

I would expect the detection to work seamlessly, since the image is "perfect" (it's a not a picture taken from some camera, it's generated by the computer).
With the python wrapper version 0.2.2 only 4 qr codes out of 27 are decoded, while things are better with the online decoder, which finds 17 qr codes out of 27.
Any idea on what's going on and how to improve the result?
Thanks in advance!

The reason for your issue is the number 256 in this line. It limits the number of combinations of similar sized finder patterns to combine into a potential qrcode symbol. With 81 of those patterns, the theoretical number of combinations is  81! / (78! * 3!) = 85319.
A quick fix for this case is to simply change that 256 to say 256000. This increases the runtime from 4ms to 14ms, which is a lot better than I feared when I set the value to 256.
The situation would become substantially better if you had a photograph of those 27 symbols instead of a perfect rendering because then they would differ slightly in size and the size is used to sort them prior to trying the combinations. Is this rendering indeed what you want to process in the end or was it your attempt to make the input "easier" to process and debug the issue?

This worked perfectly, thank you so much!
About your question: this was a test during the development of the rendering part, to see if zxing was able to detect/decode what I was generating. I'm not sure about the real-life use case, it could be that the user will process only photographs, but it could also be that the user has the original rendered image and wants to decode the info from the qr codes. Since inference time is not a strict constraint, I will probably increase the limit as you suggested.
Thanks again!

Thanks for the hints, I'll keep you posted.
Closing for now!

I reopened it, since the library as is still has this (serious) limitation and I see the need to do something about it, sooner or later...

Very good idea. Temporarily, at least a little bit, we will increase this value a little in the main line.
Thank you

authors: adamponi axxel lykius

repo: zxing-cpp/zxing-cpp


text:
我想做一个相似问法的生成工具，请问有什么好的建议吗？

我想做一个相似问法的生成工具，请问有什么好的建议吗？不胜感激！

https://github.com/zxkmm/gpt2-ml-fixed/issues/9#issuecomment-1244904118


text:
Installing KLE Placer from PCM in KiCAD 7.0.8 does not work.

Hello, just as the title says.
After installing it via Plugin Manager, there is no icon in upper toolbar, and it's not showing under Tools menu -> External Plugins.
Restarting KiCAD and hitting Refresh Plugins button does not help as well.
As far as I know, KiCAD does not have error logs to see what's happening when it tries to load plugin, because even manual installation in KiCAD\7.0\scripting\plugins\kle-placer does not fix this issue.
Previously used with KiCAD 6.0.9 and there were no problems at all, but the whole project is on KiCAD 7.0 now, and can't be open with KiCAD 6.0, but I need to update the layout placement, since it all angled, and better just use KLE Placer instead of manually trying to align everything.
Thank you in advance!

PCM still not working correctly, but downloading this repo in .zip and unpacking it to KiCAD\7.0\scripting\plugins\kle-placer looks like works fine.

Hi thanks, I'll investigate the PCM.

authors: Iipal zykrah

repo: zykrah/kicad-kle-placer


text:
Add Token printing functions

As noted here, the code for printing out tokens as used by the Interpreter really belongs into the Token module.
This PR accomplishes that.

On a side note, the interpreter should - in an error case - probably try to judge whether the error was caused by a top-level Token or a BarToken and print with the according granularity (rather than always printing out the whole track).
(In fact, this should probably only be done on the TrackLexer's failure to correctly split it into bars.)
Might also want to think about printing some context, though.
This is all out of scope of this issue, though. Should rather be done with #45 .

repo: zzril/aula


text:
Keyword not supported: 'multipleactionresultsets'

Here is what to include in your request to make sure we implement a solution as quickly as possible.
1. Description
Describe the issue or propose a feature.
2. Exception
If you are seeing an exception, include the full exception details (message and stack trace).
Exception message:
Stack trace:

3. Fiddle or Project
If you are able,
Provide a Fiddle that reproduce the issue: https://dotnetfiddle.net/25Vjsn
Or provide a project/solution that we can run to reproduce the issue.

Make sure the project compile
Make sure to provide only the code that is required to reproduce the issue, not the whole project
You can send private code here: info@zzzprojects.com

Otherwise, make sure to include as much information as possible to help our team to reproduce the issue.
4. Any further technical details
Add any relevant detail can help us.

Hello @Hajar475 ,
We will close this issue as no detail has been provided.
Feel free to open a new issue with some details to help us to understand the problem.
Best Regards,
Jon

Sponsorship
Help us improve this library
Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function

repo: zzzprojects/EntityFramework-Plus


text:
Duplicate id when setting it using HtmlNode.Id

Description
You get Duplicate id when setting it using HtmlNode.Id. This seems to be happening when the node already had an Id and you set a different one.
Fiddle or Project
https://dotnetfiddle.net/OGnU9L
(its giving me an internal server error right now, so here is the code)
                var html = @"

                ";
		var htmlDoc = new HtmlDocument()
		{
			OptionEmptyCollection = true,
			OptionUseIdAttribute = true
		};
		htmlDoc.LoadHtml(html);
		var htmlBody = htmlDoc.DocumentNode.SelectSingleNode("//body");
		HtmlNode oldChild = htmlBody.ChildNodes[1];
		oldChild.Id = "test";
		Console.WriteLine(oldChild.OuterHtml);

Workarround: dtControl.SetAttributeValue("id", "test")

Weird but that definitely looks like an issue.
My developer will look at it.
Best Regards,
Jon

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Hello @EdwinStoteler ,
The v1.11.20 has been released.
Could you check it and let us know if everything work as expected now with the id?

Hello @JonathanMagnan ,
The problem seems to be solved, thx for the quick release.

authors: EdwinStoteler JonathanMagnan

repo: zzzprojects/html-agility-pack


text:
fix wrong file name

Thanks @alihardan !

authors: alihardan pdehaan


text:
feat: 🎸 Added 120 Questions in Software Development Track

Could you please revamp your questions to follow our specified format. Also, I noticed that you didn't add the answer for each questions there. Kindly rectify. @UmarAuna

@EmmanuelTheCoder If you kindly check, i added the correct answer

authors: EmmanuelTheCoder UmarAuna


text:
Update _sidebar.md

Closes #2


text:
added pre-commit.ci badge to README.md

so pre-commit.ci should now remove trailing whitespaces I intentionally added to the readme


text:
merge

merge branches new and main


text:
Fix DG

Codecov Report

Merging #209 (288f2f3) into master (4b546b8) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #209   +/-   ##
=========================================
  Coverage     70.91%   70.91%
  Complexity      851      851
=========================================
  Files           103      103
  Lines          2878     2878
  Branches        408      408
=========================================
  Hits           2041     2041
  Misses          680      680
  Partials        157      157

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

authors: JJoeYi codecov-commenter


text:
Add EditCommand implementation in DG

Codecov Report

Merging #108 (0e8cc9a) into master (3b32c3f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #108   +/-   ##
=========================================
  Coverage     71.19%   71.19%
  Complexity      472      472
=========================================
  Files            79       79
  Lines          1451     1451
  Branches        157      157
=========================================
  Hits           1033     1033
  Misses          374      374
  Partials         44       44

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

LGTM, remember to do the sequence diagram and activity diagram in v1.4


text:
Add dg page breaks final

Codecov Report
Base: 70.43% // Head: 70.43% // No change to project coverage :thumbsup:

Coverage data is based on head (19d8d2c) compared to base (b564c3c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #285   +/-   ##
=========================================
  Coverage     70.43%   70.43%
  Complexity      610      610
=========================================
  Files            97       97
  Lines          2006     2006
  Branches        237      237
=========================================
  Hits           1413     1413
  Misses          496      496
  Partials         97       97

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 tin-jy


text:
Add confirmation for clear command

See #167 for new resolution


text:
Update file names

Not sure whether to also rename "addressbook.json" in "data/addressbook.json". A lot of classes seem to reference "addressbook.json" for the storage path.


text:
UI improvement: output for find should prompt to search for a different field

Done


text:
build: raise libtiff min to 4.0 + self-build for libtiff (and libdeflate)

No objections -> merging

repo: AcademySoftwareFoundation/OpenImageIO


text:
Upgrade the dockerfiles of tensorflow-gpu

bors r+

authors: shiyingjin xidianwym

repo: Adlik/Adlik


text:
update packages

:x: Build TheIdServer 3.0.1-fix-dependencies1-0061 failed (commit https://github.com/Aguafrommars/TheIdServer/commit/3e5b5ab9ec by @)

:white_check_mark: Build TheIdServer 3.0.1-fix-dependencies.1+61.build.1386 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/3e5b5ab9ec by @)

authors: AppVeyorBot aguacongas

repo: Aguafrommars/TheIdServer


text:
update packages

:white_check_mark: Build TheIdServer 6.3.1-fix-dependencies1-0107 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/6809a5fcb8 by @)

:white_check_mark: Build TheIdServer 6.3.1-fix-dependencies1-0108 completed (commit https://github.com/Aguafrommars/TheIdServer/commit/45805e73d8 by @)


text:
feat: use service getters

🧡

/test

/test

ready to merge? ;)

authors: jarrvis wkolod

repo: Akord-com/akord-js


text:
Still print() messages not in the log though run with -u, need to use a real logging module

Done v1.3

repo: AlanFromJapan/GrblWebStreamer


text:
Intertypes, this time for real

@olynch, it looks like it isn't precompiling in GHA.
ERROR: The following 1 direct dependency failed to precompile:

SyntacticModels [22bb929c-8bcf-4852-b455-eb3e1675e09c]

Failed to precompile SyntacticModels [22bb929c-8bcf-4852-b455-eb3e1675e09c] to "/home/runner/.julia/compiled/v1.10/SyntacticModels/jl_dmXMSG".
WARNING: could not import DiagrammaticEquations.decapodeacset into ASKEMDecapodes
WARNING: could not import ASKEMDecapodes.decapodeacset into askemdecapodes
ERROR: LoadError: UndefVarError: `decapodeacset` not defined
Stacktrace:
 [1] top-level scope
   @ ~/work/SyntacticModels.jl/SyntacticModels.jl/src/decapodes.jl:17
 [2] top-level scope
   @ ~/work/SyntacticModels.jl/SyntacticModels.jl/src/decapodes.jl:14
 [3] include(mod::Module, _path::String)

authors: jpfairbanks olynch


text:
Update to 0.1.20

Closing and re-opening the PR due to a bad merge in the aggregate .gitmodules that's causing the builds to fail.

repo: AnacondaRecipes/gptcache-feedstock


text:
Update 1.8.5

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_3d1h6quvu2/clone/recipe/meta.yaml:28: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_3d1h6quvu2/clone/recipe/meta.yaml:29: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_3d1h6quvu2/clone/recipe/meta.yaml:56: license_file_overspecified: Using license_file and license_url is overspecified.
WARNING: /tmp/abs_3d1h6quvu2/clone/recipe/meta.yaml:56: missing_description: The recipe is missing a description
WARNING: /tmp/abs_3d1h6quvu2/clone/recipe/meta.yaml:57: missing_description: The recipe is missing a description
WARNING: /tmp/abs_3d1h6quvu2/clone/recipe/meta.yaml:57: license_file_overspecified: Using license_file and license_url is overspecified.
Errors were found

Linter check found the following problems:
The following problems have been found:
WARNING: /tmp/abs_devqn50zw4/clone/recipe/meta.yaml:55: missing_description: The recipe is missing a description
WARNING: /tmp/abs_devqn50zw4/clone/recipe/meta.yaml:56: missing_description: The recipe is missing a description
Warnings were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_01ktclprc_/clone/recipe/meta.yaml:29: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_01ktclprc_/clone/recipe/meta.yaml:30: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
Errors were found

repo: AnacondaRecipes/shapely-feedstock


text:
Figure out how to suppress printing of compiler generated classes

These are classes not in the original source code but generated on the fly by compilers

repo: AncientStudying/spoon-practice


text:
chore: simplify the code

Thanks for opening this! Can you run cargo fmt on it? We have a check that requires the project to be formatted with it.

done.

repo: ArroyoSystems/arroyo


text:
Fix docker builds by including arroyo-openapi

we could do something a bit hacky (parse the Cargo.toml, grep the dockerfiles for each of the crates)

Added a check for this in CI


text:
add confirmation on deletion

implemented

repo: ArtyomPanfutov/notes


text:
Add a new payment table specific to the payments for the whole API

Done with issue13

repo: Asone/graphQLN


text:
pls check  and confirm ARCHE document-property lists (sent via Mattermost)

see answer in mattermost

authors: linxOD timofruehwirth

repo: Auden-Musulin-Papers/amp-data


text:
xnat build v1.8.4.1

Should we just delete lines rather than commenting them out?

Sure, definitely

thanks for reminding, removed the commented lines now

waiting for Mark's confirmation that standard xnat 1.8.4.1 includes the fix to allow long aaf username

authors: andrewbrock-sahmri fxusyd

repo: Australian-Imaging-Service/xnat-build


text:
[#3170] Persistent streams - automatic configuration

@smcvb what do you think about this approach :)

@smcvb I am not sure if I understand you correctly, could you give me more details?

Of course, no worries. Let me break it down in the steps I'd foresee:

We introduce a Builder to the PersistentStreamMessageSourceDefinition, which could replace the constructor. The Builder will have all the fields of the constructor as separate methods, with validation. In doing so, this will mirror how all other infrastructure components of Axon Framework behave. This would net us with a PersistentStreamMessageSourceDefinition.Builder class, by the way.
I would rename the EventProcessingConfigurer.PersistentStreamMessageSourceDefinitionBuilder to EventProcessingConfigurer.PersistentStreamMessageSourceDefinitionConfigurationand, more importantly, let itextendsthePersistentStreamMessageSourceDefinition.Builderfrom step 1. If you're looking for an example, you can check theEventProcessingConfigurer.PooledStreamingProcessorConfigurationandEventProcessingConfigurer.DeadLetteringInvokerConfiguration`.
We add a registerPooledStreamingEventProcessorConfiguration(PersistentStreamMessageSourceDefinitionConfiguration) and registerPooledStreamingEventProcessorConfiguration(String, PersistentStreamMessageSourceDefinitionConfiguration) to the EventProcessingConfigurer. The first method becomes the default PersistentStreamMessageSourceDefinitionConfiguration configuration method,  and the latter becomes the processingGroup-specific PersistentStreamMessageSourceDefinitionConfiguration configuration method.
We implement the new methods on the EventProcessingModule. I again recommend checking out how we do this for the PooledStreamingProcessorConfiguration and DeadLetteringInvokerConfiguration and mirror their behavior as much as possible.
When the EventProcessingModule constructs a SubscribingEventProcessor we check whether there is a PersistentStreamMessageSourceDefinitionConfiguration present for the specific processing group. When yes, we use it. When not, we default to the normal behavior.
The Spring Boot Auto Configuration uses the new methods of the EventProcessingConfigurer to register persistent stream logic.

I think that's the flow in the rough. Does that help you further, @rsobies?

I would rename the EventProcessingConfigurer.PersistentStreamMessageSourceDefinitionBuilder to EventProcessingConfigurer.PersistentStreamMessageSourceDefinitionConfigurationan, more importantly, let it extends the PersistentStreamMessageSourceDefinition.Builder

@smcvb  I see a problem with this,  EventProcessingConfigurer has no access to PersistentStreamMessageSourceDefinition, because Axon configuration module doesn't have dependency to  Axon server connector module.
That's why I introduced the new functional interface in order to able to create PersistentStreamMessageSourceDefinition inside EventProcessingModule without directly referring to PersistentStreamMessageSourceDefinition

@smcvb I see a problem with this, EventProcessingConfigurer has no access to PersistentStreamMessageSourceDefinition, because Axon configuration module doesn't have dependency to Axon server connector module.
That's why I introduced the new functional interface in order to able to create PersistentStreamMessageSourceDefinition inside EventProcessingModule without directly referring to PersistentStreamMessageSourceDefinition

Darn, very good point, @rsobies.. I need to think a little for another angle.
That doesn't mean you cannot proceed, though. Perhaps you already have a good idea how to resolve this.

Alright, I gave it some time. Thanks for your patience, @rsobies.
The interface that is shared (or, not tied to Axon Server specifically), is the SubscribableMessageSourceDefinition.
In essence, you already provided the functional interface on the EventProcessingConfigurer for this.
If we rename the interface you called PersistentStreamMessageSourceDefinitionBuilder to SubscribableMessageSourceDefinitionBuilder, I think we keep that separation clear.
Differently put, the axon-config module shouldn't directly know about persistent streams; that's an Axon Server concern.
What it does know about, is a SubscribableMessageSourceDefinition.
So, adding registration methods for that to the EventProcessingConfigurer (similar to what I suggested in my earlier post), might be the way to go.
Although it means that the non-AxonServer solution does not have anything to construct these, I think that's fine.
Mainly because we are not providing this solution for non-AxonServer users.
With in hand, you should be able to register these builders based on application.properties to the EventProcessingConfigurer.
So, what do you think, @rsobies?

sorry for my absent :)

Although it means that the non-AxonServer solution does not have anything to construct these, I think that's fine.
Mainly because we are not providing this solution for non-AxonServer users.

ok, but it means that axon server users must be aware that SubscribableMessageSourceDefinitionBuilder must create PersistentStreamMessageSourceDefinition (if I am correct that is the only implementation, at least for now)

Now it is my turn to apologies for being late with replying...sorry, busy times here.

ok, but it means that axon server users must be aware that SubscribableMessageSourceDefinitionBuilder must create PersistentStreamMessageSourceDefinition (if I am correct that is the only implementation, at least for now)

That is true, of course! But, the EventProcessingModule isn't part of the Axon Server module, nor does it depend on that module. Hence, it shouldn't know the specifics about persistent streams.
In the documentation we can be specific about this point, though, through means of a sample.
Just to be clear, I mean the AxonIQ Docs here, not the JavaDoc.

authors: rsobies smcvb

repo: AxonFramework/AxonFramework


text:
Alliance got 5k gold, 15 lumber and a million exp, probably the leaver method broken

Closed by https://github.com/AzerothWarsLR/WarcraftLegacies/commit/8ac312999e4f37cda4f912d88c46c0bf40025cc9.

authors: SebastianTanoni YakaryBovine

repo: AzerothWarsLR/WarcraftLegacies


text:
c_bind for Native Base Rollup Circuit

Done


text:
Omit marshaller for output-only types

Fixes https://github.com/Azure/autorest.go/issues/797


text:
Regenerate azure-search-documents on New SHA

API changes have been detected in com.azure:azure-search-documents. You can review API changes here

/azp run java - search - tests

API changes have been detected in com.azure:azure-search-documents. You can review API changes here
API changes
-         public Mono<Void> resetDocuments(String indexerName, boolean overwrite, List<String> documentKeys, List<String> datasourceDocumentIds)
-         public Mono<Response<Void>> resetDocumentsWithResponse(SearchIndexer indexer, boolean overwrite, List<String> documentKeys, List<String> datasourceDocumentIds)
+         public Mono<Void> resetDocuments(String indexerName, Boolean overwrite, List<String> documentKeys, List<String> datasourceDocumentIds)
+         public Mono<Response<Void>> resetDocumentsWithResponse(SearchIndexer indexer, Boolean overwrite, List<String> documentKeys, List<String> datasourceDocumentIds)
-         public void resetDocuments(String indexerName, boolean overwrite, List<String> documentKeys, List<String> datasourceDocumentIds)
-         public Response<Void> resetDocumentsWithResponse(SearchIndexer indexer, boolean overwrite, List<String> documentKeys, List<String> datasourceDocumentIds, Context context)
+         public void resetDocuments(String indexerName, Boolean overwrite, List<String> documentKeys, List<String> datasourceDocumentIds)
+         public Response<Void> resetDocumentsWithResponse(SearchIndexer indexer, Boolean overwrite, List<String> documentKeys, List<String> datasourceDocumentIds, Context context)

authors: alzimmermsft azure-sdk

repo: Azure/azure-sdk-for-java


text:
Sync with bcda-aps upstream

Woops, meant to go the other way, sorry.

repo: BCDA-APS/tiled-template


text:
Multi-node etcd deployment

Assignee is None

Created by ritka-admin at 2022-12-03T09:10:10Z


text:
Allow the scheduler to shut down

this would alloow bstats to be used in fabric mods - otherwise, if there is no way to shut down bstats, and the fabric server won't shut down with bstats running.

Thank you!

authors: Bastian Tim203 onebeastchris

repo: Bastian/bstats-metrics


text:
devices: add taproot keypath spending support for bitbox02

Also please rebase and squash the commits - the older commits still have debug stuff in them

Also please rebase and squash the commits - the older commits still have debug stuff in them

Done! Thanks

@Beerosagos can add a commit here that bumps the bitbox02 dependency version?

@Beerosagos i tested this with taproot, p2wpkh and multisig. Please open the PR upstream :)

authors: Beerosagos benma

repo: Beerosagos/HWI


text:
Modular metrics

Probably we should close the pr until it is ready to merge?

ouch we now have conflicts with main here 🙈

/format

/format

authors: Hendrik-code neuronflow

repo: BrainLesion/panoptica


text:
Need to update the existing document for Fortify scans

Adding Chris to review the documentation created


text:
Feat weblate cidgravity main

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

repo: CIDgravity/CIDgravity-localization


text:
Feature add point highlight

closed


text:
route example

The only thing bothering me here is that route is not a route at all. At least not in the common sense of the word.
In other libraries, this process of searching a corresponding file is usually considered a "middleware" we use as a fallback when we can't find a route.
For instance,
https://github.com/expressjs/express/blob/2c47827053233e707536019a15499ccf5496dc9d/examples/static-files/index.js#L22

The only thing bothering me here is that route is not a route at all. At least not in the common sense of the word.

Its close enough for the review, and we can change it before the first Boost release. We need to get this in as soon as possible and then move on to the other things.

authors: alandefreitas vinniefalco

repo: CPPAlliance/url


text:
[TEST] Test access

Closing as this was to test access setup.

authors: AmyOctoCat mirenradia


text:
Reword users

any updates, here?

See #70

authors: CaptainSifff knarrff


text:
Ensure that secrets are hidden by default in audit logs

Coverage increased (+0.5%) to 87.48% when pulling d03213138bd76209b54175cd27924f2421651671 on feature/secure-audit-pages into f5281fc4afcb89ac3b3ae54abfc958c9d7dad0fc on main.

authors: ChrisMacNaughton coveralls

repo: CentauriSolutions/EyeDP


text:
[FSDE-7912] Migrate Divider component to new Vue documentation

You can check this PRs instance in https://6840f608a4c13bf54dd0b7c223dd55f0e2e4da64-chi-pr.enterprise.ctl.io

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/19/. :x:

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/20/. :white_check_mark:

You can check this PRs instance in https://f26a9f71e82b1a28d261f3ef82c78bf7094ebb47-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-204-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/21/. :white_check_mark:

You can check this PRs instance in https://f3e69bb4c2ebd8357e9bd9c82cc9d7487c0ae4a1-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-204-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/22/. :white_check_mark:

You can check this PRs instance in https://05649a6a8e920b6264af249bb6a527eadcdad437-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-204-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/23/. :white_check_mark:

You can check this PRs instance in https://aea052252f6a109847493a4b2eac9a9d6850471a-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-204-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline did not run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/24/. :x:

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/25/. :white_check_mark:

You can check this PRs instance in https://e4c2b9cf3b85306c16ad6da5b8d849da7b30e9ae-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-204-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/26/. :white_check_mark:

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/27/. :white_check_mark:

You can check this PRs instance in https://15915b5cb429cd389e028fc58732de7697994ebd-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-204-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/28/. :white_check_mark:

You can check this PRs instance in https://e80853e4ffa962695598e6242511e926527f9b2b-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-204-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/29/. :white_check_mark:

You can check this PRs instance in https://95e6515c147b5bdf1bb01eaa583e2391cdeeba56-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-204-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/30/. :white_check_mark:

You can check this PRs instance in https://165736ce0fc505612748587edc9b4ae3c8e6a121-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-204-ux-chi.kubeodc-test.corp.intranet/ (internal)

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-204/31/. :white_check_mark:

You can check this PRs instance in https://24cc65cc4b536bca25a0cd7ed44727ad0f2f2674-chi-pr.enterprise.ctl.io (external) and in https://nginx-pr-204-ux-chi.kubeodc-test.corp.intranet/ (internal)

authors: SCMAUTO evansito

repo: CenturyLink/Chi


text:
Create dev.tf

atlantis plan

atlantis plan

Ran Plan for dir: . workspace: default
Show Output
Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
+ create

Terraform will perform the following actions:

  # null_resource.example will be created
+ resource "null_resource" "example" {
      + id = (known after apply)
    }

  # null_resource.test will be created
+ resource "null_resource" "test" {
      + id = (known after apply)
    }

Plan: 2 to add, 0 to change, 0 to destroy.

:arrow_forward: To apply this plan, comment:

atlantis apply -d .

:put_litter_in_its_place: To delete this plan click here
:repeat: To plan this project again, comment:

atlantis plan -d .

Plan: 2 to add, 0 to change, 0 to destroy.

:fast_forward: To apply all unapplied plans from this pull request, comment:

atlantis apply

:put_litter_in_its_place: To delete all plans and locks for the PR, comment:

atlantis unlock

repo: ChiamakaObitube/platform-ops


text:
New Crowdin updates

Faudrait peut-être remove ça ? Y'avait pas mal de testes fait dessus

authors: BilelJegham PolThm

repo: ChouquetteCorp/beerbet


text:
Update contributors.js

Good job @harshit-sharma-gits
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow this steps.
Note that this repo is excluded from Hacktoberfest but this PR is still eligible for ClueLess Swags.

Star this repository. ⭐

SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .

Go to profile section from the top right hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard !
And don’t forget to share your achievement with your friends.

repo: Clueless-Community/first-contribution


text:
fix: #51 endless loop

Codecov Report

Merging #52 (ff75a40) into php (8bb7e94) will decrease coverage by 1.15%.
The diff coverage is 57.14%.

@@             Coverage Diff              @@
##                php      #52      +/-   ##
============================================
- Coverage     85.78%   84.63%   -1.16%
- Complexity       88       92       +4
============================================
  Files             8        8
  Lines           401      410       +9
============================================
+ Hits            344      347       +3
- Misses           57       63       +6

Impacted Files
Coverage Δ

app/Coding/Wiki.php
81.48% <52.94%> (-5.48%)
:arrow_down:

app/Commands/WikiImportCommand.php
90.44% <63.63%> (+0.93%)
: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 8bb7e94...ff75a40. Read the comment docs.

authors: codecov-commenter sinkcup

repo: Coding/coding-cli


text:
Started CC 071 Minesweeper

Sorry, messed up a bit 😅


text:
Java SDK : Account Transaction - Transfer With Memo

The java SDK already supports "TransferWithMemo".

repo: Concordium/concordium-java-sdk


text:
Bug fix, updated version number

Updated version number, minor bug fix


text:
Fix Chinese translations for "Roll".

Thanks for the contribution!

authors: Gateswong lantisnt

repo: CoreLootManager/CoreLootManager


text:
Mrrcury gentx.json

Sorry, the deadline has passed, thanks for offering to help out 🙏

authors: MercurySolar the-frey

repo: CosmosContracts/testnets


text:
Not main

No longer segfaults, application logic works.


text:
Add a standard "Contributing" section to the readme

https://github.com/CreativeSDK/phonegap-plugin-csdk-image-editor/issues/12


text:
Feat/cookie page functionality

Good work buddy! Looks much cleaner now IMO.
In addition to the few minor comments I've left for clean-up, there's a bug if you try to accept/reject when you're unauthenticated.
Steps:

Be signed out, with 0 cookies (for auth, or our site, etc.)
Be on homepage (obviously)
Click accept/reject
You're re-directed to DSi
Login
You'll be redirected back to either /cookies/accept or /cookies/reject; but you get a 405 error.
You can navigate back to a proper page (e.g. / or /self-assessment etc.) and clicking the buttons will work at this point at least.

By removing the authorize, the issue with the redirection has been fixed.


text:
Add extension to Find endpoint for Claim's matching policy

 Amend TRN lookup to use TRN match policy

 Amend Find endpoint to support matching policy required for Claim

repo: DFE-Digital/teaching-record-system


text:
try vanilla extract

PandaCSS보다 간단하지만 스타일을 별도의 파일에 작성해야한다는 점이 @nhistory 님과 저 모두 마음에 안 들었기 때문에 포기합니다.

repo: DaleStudy/dale-ui


text:
Tests for support of different cryptographic algorhitms

@tym2k1 should this still be in draft? Please also rebase onto uefi-sb-release and change target branch for this PR.

My bad. These changes were developed for the now merged qemu-rebase-pt2 and I forgot to rerun and adress changes to syntax after rebasing.

repo: Dasharo/open-source-firmware-validation


text:
Homologacao

Aprovado


text:
Read some interpolation tables into the Model struct

I've done some fixes here so we can use "Basin / subgrid_level" as you originally intended. I seem to have broken the test along the way though.
I could use more description of the design though. Main questions are:

Why do we need the "name" column?
Why distinguish between "subgrid level" and subgrid exporter"? Can they be named the same?

Good question about the name. I did some thinking: this stems from my idea that Ribasim would present 1:1 levels to MODFLOW, but it turns out this is impractical for multiple reasons, because we do not necessarily want the subgrid elements to align 1:1 (e.g. because the Ribasim domain is larger spatially than the MODFLOW domain, or vice versa).
The name would still be a useful entry to have when setting up the coupling schemes, but in that sense it's no different from the x and y columns that we discussed yesterday. In that case, all the grouping is a coupler problem only, and in Ribasim everything can be flattened (which also cleans up the BMI).
I guess the idea with exporter and level is that the exporter "does the exporting"; the level is just a level.
I'll omit the name column, and see whether the exporter / level thing can be cleared up somewhat.

authors: Huite visr

repo: Deltares/Ribasim


text:
Day 20: Mini Project - Deploy a Multi-Tier Web Application Using Docker Compose

Learned and completed Day 20 tasks.


text:
Latency monitoring

@swarajrb7 are you still working on this PR? Are you blocked by anything?

@JyotinderSingh i had some doubts i have mentioned them here  #1135

repo: DiceDB/dice


text:
To try out the latest state of this promising tool for EG-2

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2b05203). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff            @@
##             master       #1   +/-   ##
=========================================
  Coverage          ?   54.34%
=========================================
  Files             ?      188
  Lines             ?    12070
  Branches          ?        0
=========================================
  Hits              ?     6560
  Misses            ?     4951
  Partials          ?      559

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

authors: codecov-commenter heukai


text:
qasem sherfi

receive to my vertcion
vtc1q7edaqax9q04adkmj47p3dt8l2uv89t7zk82rx5

repo: EdgeApp/edge-currency-plugins


text:
WIP: add evaluation on JaQuAD

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

repo: EleutherAI/lm-evaluation-harness


text:
setup react app

Done in other PR

repo: Enrikerf/pfm


text:
add next batch of 5 puzzles

done

repo: Entkenntnis/robot-karol-web


text:
Threading Model Reinstated

Threading model redone - Seems to be "Slower" in terms of the gaps in the gannt are visibily larger and more spread out,  thoughts ??

Have tried under "release" and things seem better,  so i'm good to push ahead with this PR if you are.  Sorry for the size, but shall we have a chat to talk the changes through ?

Refactorings undone, any changes are now required for the gannt

What changed in Options.cs and some of the other files? Line endings? Spaces and tabs? It would be great if only your changes showed. If the files didn't change substantively, changes could be undone to make the pull request smaller/easier to understand. Hope this makes sense.

Nothing,  moved it to a Forms folder

Please do any refactoring in seperate PRs, thanks.

authors: ErikEJ daveTechLed

repo: ErikEJ/SqlQueryStress


text:
CI for Node

Inserts an 'X' into the top center (1 ms)


text:
Align with NOMAD docs

Pull Request Test Coverage Report for Build 9384287171
Details

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.03%) to 78.434%

Files with Coverage Reduction
New Missed Lines
%

pynxtools/dataconverter/validation.py
1
68.2%

pynxtools/dataconverter/helpers.py
1
79.21%

Totals

Change from base Build 9347831818:
-0.03%

Covered Lines:
2815

Relevant Lines:
3589

💛  - Coveralls

LGTM. I'm however not sure if we want to keep the vanilla NOMAD logo. It looks a bit like that this is the central documentation for NOMAD. This might confuse people and it might be good if we had a small pynxtools below or beside it or so. That way it would be clear that it belongs to nomad but is actually a different documentation.
Also it always confuses me that clicking on the logo brings me to the official nomad page. I'm used to landing on the entry page of the site I'm currently at. But it's the same for NOMAD itself so it's no real issue arising from this PR.

100% agree on both points. I guess the idea with the vanilla NOMAD logo is that it all points back to NOMAD, but pynx is in principle also standalone. Let's just keep it for now, I saw some discussion in discord about the particular UX when clicking on the button, so maybe this will be changed anyway.

authors: coveralls lukaspie

repo: FAIRmat-NFDI/pynxtools


text:
add Astro framework

@jamesdaniels, can you review this because i need it to host my project on firebase hosting.
thank you in advance.


text:
[core] fixed CONNECTION_STATE_MCS_CREATE_REQUEST parsing

@hardening @mfleisz ok, update the pr accordingly.

repo: FreeRDP/FreeRDP


text:
DX: Update PHPUnit config

Coverage remained the same at 92.846% when pulling d92b1ce88d7b1cd1867b11256b8a899b827697c4 on julienfalque:phpunit-config into 2cbb591bb68a6dfd7162de84f71e07e2349ad621 on FriendsOfPHP:master.

authors: coveralls julienfalque

repo: FriendsOfPHP/PHP-CS-Fixer


text:
Need a YouTube provider

Addressed in #13

repo: FritzAndFriends/TagzApp


text:
Select + JOIN: Syntax + Implementation

needs to differentiate between primary and foreign keys, which are not yet implemented

repo: FujiwaraChoki/SkibiDB


text:
added .env and package lock to .gitignore

bewm

authors: LockedJCE ZachariahKB


text:
Add README

Forgot to go back to main before creating the other branch 🤦♂

repo: GET-Prog/ant-colony-optimization


text:
pty dimensions configuration in SSHShellConnectionParameter

@corneliuslippert-gip can you fix the version references please. c.f. pull-request check
2022-11-08T13:08:03.7119717Z application SSHFile has an reference to application SSH which has the wrong versionName
2022-11-08T13:08:03.7120208Z versionName Reference:1.0.24
2022-11-08T13:08:03.7120693Z versionName in Source:1.0.25
2022-11-08T13:08:03.7121033Z file is ../../modules/xact/ssh/file/application.xml

authors: corneliuslippert-gip csc-gip

repo: GIP-SmartMercial/xyna-factory


text:
added my SmartGuard file

How do I do that?
On Mon, Nov 4, 2024, 11:23 AM Sayyad Aslam @.***> wrote:

@.**** requested changes on this pull request.
Please Submit the Screenshot in jpg format.
Create a new PR
image.png (view on web)
https://github.com/user-attachments/assets/ceab270e-7ce4-4006-91ae-748e87501825
—
Reply to this email directly, view it on GitHub
https://github.com/GSSoC24/Hack-Web3Conf/pull/16#pullrequestreview-2412199879,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD2GI7BPIDVCH4YBTS3GRY3Z64DV5AVCNFSM6AAAAABRCZPVFKVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDIMJSGE4TSOBXHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Just Upload the screenshot of it.
rename that ss as - name-projectname-date
You can make a New PR it'll be easy

i am unable to upload again
how ?

i am unable to upload again can u give access for this
On Mon, Nov 4, 2024 at 1:12 PM Sayyad Aslam @.***>
wrote:

Just Upload the screenshot of it.
rename that ss as - name-projectname-date
You can make a New PR it'll be easy
—
Reply to this email directly, view it on GitHub
https://github.com/GSSoC24/Hack-Web3Conf/pull/16#issuecomment-2454010282,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD2GI7CND7LX7P76RWHX5ELZ64QP5AVCNFSM6AAAAABRCZPVFKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJUGAYTAMRYGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

hi @23wh1a12c1 , Follow these steps:

Fork this repository.
Go to submit folder
Add your screenshot there (ss name : name-projectname-date
Create a pull request

It seems that you haven't forked the repo yet after I reviewed ur profile.
Will close this PR now,

I have uploaded can u please once check and tell me confirmation. Is that
correct ?
On Mon, Nov 4, 2024, 8:06 PM Sayyad Aslam @.***> wrote:

hi @23wh1a12c1 https://github.com/23wh1a12c1 , Follow these steps:

Fork this repository.
Go to submit folder
Add your screenshot there (ss name : name-projectname-date
Create a pull request

It seems that you haven't forked the repo yet after I reviewed ur profile.
Will close this PR now,
—
Reply to this email directly, view it on GitHub
https://github.com/GSSoC24/Hack-Web3Conf/pull/16#issuecomment-2454880914,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BD2GI7B3PZ4G7JBIFHNDVFLZ66A5XAVCNFSM6AAAAABRCZPVFKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJUHA4DAOJRGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


text:
[Question] is it possible to use phone's microphone in linux for voice/video chat with 2.1 update?

With --audio-source=mic, the device microphone is captured and played on the computer audio output (speakers).
It is not exposed as an audio input, so a priori you can't use as a computer microphone. But maybe it is possible with some PulseAudio or PipeWire magic, I don't know.

repo: Genymobile/scrcpy


text:
Update main branch in relation to original devito

I evaluated the tests that were not passing and they were due to some problems in the new multiparameter viscoacoustic, but they dont affect the rest of the package. I had to fix a conflict caused by another commit Gustavo made last week, and now its just waiting for the tests to run. Although I think it can be merged already.

repo: GeoSymCodes/devito


text:
Add JEST for better testing

I just went through https://jestjs.io/docs/getting-started. I will do more later, I just wanted to prepare a base. I think later it will consist of a lot of mocking.

I agree with your comment about mocking. We should avoid relying on responses from external APIs in our tests. Whatever you end up doing for your jest tests, can you run me through it so I can try to assist with creating some of these tests?

authors: GeorgeCiesinski Wissididom

repo: GeorgeCiesinski/poke-guesser-bot


text:
Update and publish ensemble tide modelling functionality

Currently ensemble model is producing incorrect results due to .dropna() removing any point containing missing results for any tide model (related to https://github.com/tsutterley/pyTMD/issues/366):
try:
        model_ranks_gdf = (
            gpd.read_file(ranking_points, engine="pyogrio")
            .to_crs(crs)
            .query(f"valid_perc > {ranking_valid_perc}")
            .dropna()[model_ranking_cols + ["geometry"]]
        )

Codecov Report
Attention: Patch coverage is 66.66667% with 6 lines in your changes missing coverage. Please review.

Project coverage is 80.9%. Comparing base (2018614) to head (c6da3fb).
Report is 3 commits behind head on main.

Files with missing lines
Patch %
Lines

eo_tides/model.py
40.0%
3 Missing :warning:

eo_tides/utils.py
71.4%
1 Missing and 1 partial :warning:

eo_tides/eo.py
0.0%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##            main     #32     +/-   ##
=======================================
- Coverage   81.3%   80.9%   -0.5%
=======================================
  Files          6       6
  Lines        632     644     +12
  Branches     110     111      +1
=======================================
+ Hits         514     521      +7
- Misses        71      75      +4
- Partials      47      48      +1

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

authors: codecov-commenter robbibt

repo: GeoscienceAustralia/eo-tides


text:
feat: integrate Beiboot version >= 0.11

Pull Request Test Coverage Report for Build 3263774500

4 of 10 (40.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.3%) to 63.834%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

getdeck/provider/beiboot/init.py
4
10
40.0%

Files with Coverage Reduction
New Missed Lines
%

getdeck/provider/beiboot/init.py
1
41.07%

Totals

Change from base Build 3263053107:
0.3%

Covered Lines:
1202

Relevant Lines:
1883

💛  - Coveralls

authors: Schille coveralls

repo: Getdeck/getdeck


text:
Add a post

:bird: A bird does not approve...

AAA

authors: GitChiripBot akoskovacs

repo: GitChirip/gitchirip-posts


text:
Do not search user paths for config defaults

LGTM

authors: cmccoy ehankland

repo: GoogleCloudPlatform/PerfKitBenchmarker


text:
Add feauture/create event comment notification to organizer by email

Please resolve conflicts in the listed files before merging

authors: NazarVavrushchak vulook


text:
Home Page Redesign

This PR is not ready for approval

Homepage Wave needs to be mobile optimized
The team needs to discuss color options
The team needs to discuss where Buttons are needed/not needed, and where they should link to
The team needs to discuss if photos should be implemented and which should be used


text:
Mise en place du front

Je propose Vite, Reactjs, et TailwindCSS.
En libraire de composant je propose Shadcn.

repo: GreenTreep/Frontend


text:
Added snippets

C'est top, merci !

authors: Grimille LoicDelplace

repo: Grimille/algo-vsc


text:
Feature: better logging

Codecov Report

Merging #233 (9d832ed) into main (47d0c87) will decrease coverage by 4.23%.
The diff coverage is 65.57%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #233      +/-   ##
==========================================
- Coverage   66.27%   62.05%   -4.23%
==========================================
  Files          33       34       +1
  Lines        1809     1858      +49
  Branches      399      373      -26
==========================================
- Hits         1199     1153      -46
- Misses        507      597      +90
- Partials      103      108       +5

Impacted Files
Coverage Δ

qgis_deployment_toolbelt/cli.py
84.78% <50.00%> (+7.85%)
:arrow_up:

qgis_deployment_toolbelt/utils/journalizer.py
67.27% <67.27%> (ø)

... and 6 files with indirect coverage changes

authors: Guts codecov-commenter

repo: Guts/qgis-deployment-cli


text:
[2022-4-16] JSX 和 Template 对比，各有什么优势？

JSX的本质就是JavaScript，能够支持更动态的需求，而 template 因为语法限制的问题，不够灵活
JSX在一个文件内可以返回多个组件
template解析时做了一些优化（Vue 本身的性能优化）

repo: HJY-xh/plantTrees


text:
Make Time selector draggable and intersection undraggable

done

repo: Harris-lans/Scheduler


text:
Enable FCM push notifications

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

repo: IBM/event-notifications-go-admin-sdk


text:
New prow cluster name

/retest

repo: IBM/ibm-cert-manager-operator


text:
Redesign

@dilpath how much progress did you make on this? is it still necessary to consolidate #17, #18 and #19 with this PR?

@dilpath how much progress did you make on this? is it still necessary to consolidate #17, #18 and #19 with this PR?

I made a little progress but currently have to take a break from work -- I should be able to apply your changes here this week.

@dilpath how much progress did you make on this? is it still necessary to consolidate #17, #18 and #19 with this PR?

I made a little progress but currently have to take a break from work -- I should be able to apply your changes here this week.

ping @dilpath

@dilpath how much progress did you make on this? is it still necessary to consolidate #17, #18 and #19 with this PR?

I made a little progress but currently have to take a break from work -- I should be able to apply your changes here this week.

ping @dilpath

Back now!

authors: FFroehlich dilpath

repo: ICB-DCM/fiddy


text:
[t5] add flan-t5

flan-t5 does not support interleaved pretraining

repo: IDEA-XL/PRESTO


text:
What's the use of FeatureSelector.py ?

It identify class ForwardFeatureSelector and class TwoStageFeatureSelector, but it seems that class OpenFE has already achieved every goal in the essay. One of your examples uses TwoStageFeatureSelector and the other only uses OpenFE. I didn't know the difference between them.

It has nothing to do with the paper. I just wanted to develop new selection
algorithms once.
Liu-200106 @.***> 于2024年5月23日周四 14:13写道：

It identifies class ForwardFeatureSelector and class
TwoStageFeatureSelector, but it seems that class OpenFE has already
achieved every goal in the essay. One of your examples uses
TwoStageFeatureSelector and the other only uses OpenFE. I didn't know the
difference between them.
—
Reply to this email directly, view it on GitHub
https://github.com/IIIS-Li-Group/OpenFE/issues/48#issuecomment-2126311190,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG2DAPVH5EUE22X27ILJFVTZDWCIJAVCNFSM6AAAAABIE6GSDWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRWGMYTCMJZGA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

repo: IIIS-Li-Group/OpenFE


text:
Develop gerard

Creo que es necesario revisar lo siguiente:

en build.gradle de mydata se ha añadido implementation project(path: ':BusinessAssistant-gencat'), pero no parece necesario
aunque añadir testcontainer es buena idea, se añade a los tests unitarios a la vez que h2.... si se incluye una, debería quitarse la otra

authors: GerardGurgui jonatanvicente


text:
horns

Check the new meta, add download to it. This is still the old meta and script I think.

Also use me gas his tutorial for conversion and volume normalization please.


text:
Select transform + paging in LINQ provider

@jeremydmiller I see some commented code here https://github.com/JasperFx/marten/pull/3348/commits/16fb2cfd235cf540aa4b31075abe0eaccd401d06
Why?

authors: jeremydmiller kjbtech

repo: JasperFx/marten


text:
@stu.acischools.k12.tr

I had some trouble creating the 'stu' domain in our atudent emails, than managed to make it. Do keep in mind the original website is acischools.k12.tr but student emails are stu.acischools.k12.tr

@norwegianbagel Pull request merged. Thank you!

authors: norwegianbagel philipto

repo: JetBrains/swot


text:
Work on terraform basics

The whole dev and production environment must be automated using terraform and few other tools like nomad , consul and vault etc..
This is continuous work
All required infra , app and production components must be automated

repo: JitenPalaparthi/cres-orchestration


text:
Added logoout.php

Completed


text:
Add generic interface tests from MLJTestIntegration

Codecov Report

Merging #22 (0125d75) into master (d44debb) will increase coverage by 3.84%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
+ Coverage   89.74%   93.58%   +3.84%
==========================================
  Files           1        1
  Lines         156      156
==========================================
+ Hits          140      146       +6
+ Misses         16       10       -6

Impacted Files
Coverage Δ

src/MLJXGBoostInterface.jl
93.58% <0.00%> (+3.84%)
:arrow_up:

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

@ExpandingMan

Thanks, I'll add these in my PR because that'll just be easier.

Marking as draft as #21 to incorporate.

Yeah, sorry, I was distracted by my XGBoost.jl PR.  I will return my attention to this when that is done.

I'm very stumped as to what's going on here... it seems to be complaining that it is getting the wrong num_class argument which apparently defaults to 1.  Part of the problem here is that with current XGBoost.jl it tries to return a completely wrong array for classification in which it cuts off the multi-dimensional output.  My guess is that this was never working properly but in the binary case it didn't matter because the second output didn't matter because it had to satisfy $1 - p_{1}$.  This is all going to have to get changed anyway if my PR gets merged because I'm outputting proper array sizes whereas the old version seems to be improperly flattening the arrays.
So if you don't mind I think I'd prefer to see if my PR to XGBoost.jl has a chance of being merged sometime soon before picking up the pieces of whatever happened here.

Yes, as I recall, the existing XBoost.jl flattens the output of probability arrays and special cases binary probabilities (only one instead of two, per observation). This required some logic to sort out, and it is quite possible your refactor has broken this.

Close as added in another merged PR

authors: ExpandingMan ablaom codecov-commenter


text:
Revert: Add SelectionRow and SelectionRows

Codecov Report

Merging #152 (b035f0a) into master (8a225c1) will increase coverage by 0.99%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #152      +/-   ##
==========================================
+ Coverage   93.97%   94.96%   +0.99%
==========================================
  Files          26       26
  Lines         846      795      -51
==========================================
- Hits          795      755      -40
+ Misses         51       40      -11

Impacted Files
Coverage Δ

src/tableselection.jl
90.00% <ø> (+7.28%)
:arrow_up:

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

authors: codecov-commenter eliascarv


text:
Test on neovim 0.5.1

Apparently the new diagnostics test is flaky...


text:
downloading the actual data

nice and thank you

authors: KathGo aanysofia


text:
Kie dev

Codecov Report
Patch coverage: 60.00% and project coverage change: +71.94 :tada:

Comparison is base (5419951) 0.00% compared to head (394ef0e) 71.94%.

: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
Additional details and impacted files
@@             Coverage Diff             @@
##           master      #27       +/-   ##
===========================================
+ Coverage        0   71.94%   +71.94%
===========================================
  Files           0        8        +8
  Lines           0     1201     +1201
===========================================
+ Hits            0      864      +864
- Misses          0      337      +337

Impacted Files
Coverage Δ

src/network/connection.cpp
79.16% <0.00%> (ø)

test/connection_test.cpp
100.00% <100.00%> (ø)

... and 6 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

authors: Kidsunbo codecov-commenter


text:
Text-to-CAD preview pane should share the configured mouse controls from the modeling pane

Okay, so after a bit of investigation I've determined that the THREE.js OrbitControls do not support our kind of implementation for pointer events. It has its opinions about the modifier key behavior baked into the implementation, and only exposed the ability for the consumer to switch off one of the interaction types (pan, rotate, or zoom), or reconfigure their mapping to individual buttons with their mouseButtons interface.
There are good lessons, however, in their implementation of the OrbitControls class, which is nicely written and not too hard to follow. We have talked elsewhere about making our control scheme more touch-friendly, and the OrbitControls class encapsulates that logic alongside mouse logic well.
I think we should make an OrbitControls class of our own that is maybe a little slimmer than THREE.js's, and use it both for controlling our stream orbit controls and this Text-to-CAD canvas (perhaps by providing a wrapper that makes our base class THREE.js-ready?).
I consider that more of a refactor than a bug fix, so I'm gonna take just a little more time to see if I can get a quick win here before moving on.

No, I don't think there is a quick win version of fixing this.
I was able to hack together something approximating the behavior here. However, this line shows just how opinionated the THREE.js OrbitControls class is. No matter what I do to tell it from the outside "do this action on mousemove", there is still logic for keydown handling right at the heart of it. So holding down the Shift, Control, or Meta keys when trying to orbit will force it to pan, and visa versa. Here are the control scheme opinions laid out in the OrbitControls as a doc comment.
I'm removing myself and moving on for now.

This isn't "blocked" by another external issue, it's just a high-effort low reward since at the moment most users don't even know that's a 3D scene, or even that we have a text-to-cad feature in the app.
The next step for this will be to try to write our own implementation of OrbitControls that honors our interaction guard system, perhaps borrowing from our CameraControls class (which was itself based on OrbitControls originally). Adapting CameraControls to not need to send to the engine, manage the clientSideScene, etc sounds like too much effort, but if splitting out a smaller class that just manages the interaction listeners is straightforward maybe we should do that, and then use that smaller class both in CameraControls and for this new use case.

authors: franknoirot greg-kcio

repo: KittyCAD/modeling-app


text:
Add aioclock

@Kludex Thanks, did the commit

repo: Kludex/awesome-pydantic


text:
防止sql注入

无

repo: KouShenhai/KCloud-Platform-Alibaba


text:
issue/236 Observing Conditions

Closing this because if I open the PR, I can't be the official reviewer. Qianjun will open a new one soon.

repo: LSSTDESC/RAIL


text:
Look at design level - best way to organize associated or derived genome editing files in LabCAS.

Review these slides - https://docs.google.com/presentation/d/1V_jYZ7dgDNkRi7qaVzP4T3IgQLDjMMKdxuYvkTpSc1Q/edit#slide=id.p

The like the idea of having the study definition files at collection file level. Derived and benchmark data will become either separate datasets or a single dataset. Will need a way to link the cell line files.

repo: Labcas-NIST/Labcas-ui


text:
edited file README.md

？

authors: ogios yourZaNoza

repo: LazyVim/LazyVim


text:
Feat modify UI

#2

repo: LspongebobJH/chatgpt-ui


text:
Feature/fix font familly

Thanks @CorentinSwiss4 for the fix and PR. 👍


text:
added code to HTML file

This pull request adds HTML

repo: MTS-sip/prework-study-guide


text:
changed

Not good

repo: MandlDevelopment/security-scan


text:
change title on README

dvxcvxcv

repo: Masterlb/reviewing-a-pull-request


text:
branch Cadastro usuario

Merge geral

authors: MatheusAmaros Thamiris-Higashiharaguti

repo: MatheusAmaros/tg-app


text:
tensorboard

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Could you plz tell me what is the purpose of this PR?

@FateScript
This will add the training loss graph and the learning rate graph to the tensorboard.

Interesting, I do not know whether the code is correct, but would certainly be useful to have the loss and maybe also metrics like MSE as well, wouldnt it ?

@FateScript any updates on this

LGTM.

@sauravkdeo  workflow failed here, plz lint your code. Thanks.

@FateScript  Done

@sauravkdeo  Too much code is modified in this PR, plz carefully lint your own modified code but not other part in the file. Thanks.

Code still failed with lint @sauravkdeo

@FateScript Done

@FateScript  Can you please approve this?

authors: CLAassistant FateScript mahnehsilla sauravkdeo sauravkumar-volt

repo: Megvii-BaseDetection/YOLOX


text:
New Crowdin updates

stop trying to delete stuff


text:
feat: add Export and Import funcs to support Marshal/Unmarshal

LGTM

repo: MetaDataLab/go-merkletree


text:
docs: fix recommendation comment

@mergify queue

@mergify rebase


text:
Update cloud-consolidation-managing-attributes.md

@yogkumgit can we merge this?

@CarolynRowe PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

authors: isrumnon scanum


text:
Merge Version 4.5 to live

Docs Build status updates of commit 6f19d32:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.gitignore
:white_check_mark:Succeeded

CONTRIBUTING.md
:white_check_mark:Succeeded

Docs/Features/key-exchange.md
:white_check_mark:Succeeded

Docs/Features/playready-features.md
:white_check_mark:Succeeded

Docs/Features/trusted-clocks.md
:white_check_mark:Succeeded

Docs/Overview/glossary-and-abbreviations.md
:white_check_mark:Succeeded

Docs/Overview/product-versions.md
:white_check_mark:Succeeded

Docs/Overview/what-is-new.md
:white_check_mark:Succeeded

Docs/Overview/what-is-new/what-is-new-4-5.md
:white_check_mark:Succeeded

Docs/PlayReady/images/caution.gif
:white_check_mark:Succeeded

Docs/PlayReady/images/client_security_level.png
:white_check_mark:Succeeded

Docs/PlayReady/images/commit_changes.jpg
:white_check_mark:Succeeded

Docs/PlayReady/images/edit_file.png
:white_check_mark:Succeeded

Docs/PlayReady/images/encryption_tools.png
:white_check_mark:Succeeded

Docs/PlayReady/images/howtowritemarkdownposter.PNG
:white_check_mark:Succeeded

Docs/PlayReady/images/image26_7.jpg
:white_check_mark:Succeeded

Docs/PlayReady/images/microsoft-logo.png
:white_check_mark:Succeeded

Docs/PlayReady/images/note.gif
:white_check_mark:Succeeded

Docs/PlayReady/images/playready_server_implementations.png
:white_check_mark:Succeeded

Docs/PlayReady/images/preview_changes.png
:white_check_mark:Succeeded

Docs/PlayReady/images/pro_with_prh_hexdump1.png
:white_check_mark:Succeeded

Docs/PlayReady/images/video-packaging-1.jpg
:white_check_mark:Succeeded

Docs/PlayReady/images/video-packaging-2.jpg
:white_check_mark:Succeeded

Docs/PlayReady/images/video-play-button-1.jpg
:white_check_mark:Succeeded

Docs/PlayReady/images/video-play-button-2.png
:white_check_mark:Succeeded

Docs/Specifications/playready-header-specification.md
:white_check_mark:Succeeded

Docs/images/caution.gif
:white_check_mark:Succeeded

Docs/images/client_security_level.png
:white_check_mark:Succeeded

Docs/images/commit_changes.jpg
:white_check_mark:Succeeded

Docs/images/edit_file.png
:white_check_mark:Succeeded

Docs/images/encryption_tools.png
:white_check_mark:Succeeded

Docs/images/howtowritemarkdownposter.PNG
:white_check_mark:Succeeded

Docs/images/image26_7.jpg
:white_check_mark:Succeeded

Docs/images/microsoft-logo.png
:white_check_mark:Succeeded

Docs/images/note.gif
:white_check_mark:Succeeded

Docs/images/playready_server_implementations.png
:white_check_mark:Succeeded

Docs/images/preview_changes.png
:white_check_mark:Succeeded

Docs/images/pro_with_prh_hexdump1.png
:white_check_mark:Succeeded

Docs/images/video-packaging-1.jpg
:white_check_mark:Succeeded

Docs/images/video-packaging-2.jpg
:white_check_mark:Succeeded

Docs/images/video-play-button-1.jpg
:white_check_mark:Succeeded

Docs/images/video-play-button-2.png
:white_check_mark:Succeeded

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

repo: MicrosoftDocs/PlayReady


text:
Create ocr for data safety tutorial

Thanks @Jak-MS  I will move this tutorial somewhere else

repo: MicrosoftDocs/azure-docs


text:
modified doc to reference the updated async methods

@Rodrigossz

Can you review this PR?

@AbhinavDAIICT - you can change the author or ms.author values in the public repo. You'll need to open a PR in the private repo to make those changes. See:

Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you make your changes in the private repo, review the articles on staging, and fix any validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@gahl-levy - GitHub won't let me assign this PR to you, but can you review it?

@Rodrigossz

Can you review this PR?

@AbhinavDAIICT - you can't change the author or ms.author values in the public repo. You'll need to open a PR in the private repo to make those changes. See:

Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you make your changes in the private repo, review the articles on staging, and fix any validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.

#label:"aq-pr-triaged" @MicrosoftDocs/public-repo-pr-review-team

Should I revert the author name and create a new PR?

#please-close

authors: AbhinavDAIICT Jak-MS Rodrigossz