Support BMM dequantize output bf16

Type of Change
feature
API not changed
Description
ILITV-2810: DistilBert support BMM dequantize output bf16
Expected Behavior & Potential Risk
The BMM int8 dequantize and cast can be fused, and BMM int8 output bf16.
How has this PR been tested?
Pre-CI and extension test.
Dependency Change?
No.

https://inteltf-jenk.sh.intel.com/job/intel-lpot-validation-top-extension-TFnewAPI/359/artifact/report.html


You're missing a release

You have to include a zip file with the mod in it as a release for it to work with the mod database. I already approved adding it into the db but didn't notice this, so it's not going to work yet. Just zip together the contents of ur mod and upload it and ur good (theres instructions on the mod template).

fixed


The SubQueryAndParamBinder must not ignore conversion exceptions

A conversion exception happening by an implicit cast must be thrown as-is and not be turned into a new one. Otherwise the information on which nesting level the error occurred is gone.
Fixes #16947.

retest this please, unrelated LR test failure

retest this please, another unrelated LR failure 🙄


build: What's the recommended IDE for Banshee development?

Once upon a time this was Monodevelop but since it looks like there's no development going on and it doesn't support the 15.x build tools it's probably time for a change of recommendation.
The recommended IDE should be Free Software but alternative suggestions are welcome.

GNOME Builder

Omnisharp integration;
Autotools support; (with some caveats)
readily available on GNU/Linux.

VS Code

Uses NRefactory AFAICT;
involves some setup;
manual download/install/update.

vim

With YouCompleteMe you get an Omnisharp server;
some setup involved for YouCompleteMe, otherwise none;
available on Mac, all GNU/Linuxes and Windows.


Settings: remove preview label

#57304
Either remove this statement completely

Or make it clickable to go from there to the old settings.json experience
Also the Settings(Preview) title feels too much for me if we already have the title

#57320


How to support Chinese?

LaTeX2e supports Chinese. But it fails when compiling contents with Chinese.

Hello @LinXueyuanStdio
You may check the CTex package and try to combine it with this resume project.


how to disable LSP warning, how to set LSP and how to uninstall specify LSP?

use this config:
let g:easycomplete_diagnostics_enable = 0
let g:easycomplete_lsp_checking = 0

duplicate of #131

You have do it manuly.

OK, what should I set, show errors but no warning?

rm -rf ~/.config/vim-easycomplete/servers/{$lsp-plugin-name}


Memory leak in server

Hi, I was reading through the code and I noticed this list of connections
https://github.com/programatik29/axum-server/blob/8574eaa2717803949877e8b00a26ea6a6f9604d9/src/server/http_server.rs#L116
it seems to grow every time the server accepts a connection and is never cleared out, so it might be an issue for a long-running server
I tried making lots of requests to the hello_world example server and after ~500,000 requests memory usage was about 1 GB

I will fix that today.


Cucumber not supported on XRAY API V2 for Jira Server

Hey there,
just wanted to use the XRAY Importer Github Action to inject our test results in Github Actions to our Jira. We use Jira Server and Xray format. Just saw that the importer run into problems:
Server responded with error (Unexpected token < in JSON at position 0): <?xml version="1.0" encoding="UTF-8" standalone="yes"?><status><status-code>404</status-code><message>null for uri: https://xxx.com/rest/raven/2.0/import/execution//multipart</message></status>
With a look on the URL, I noticed that there seems to be an issue within your xray-action. Noticed that you give the ability to use the test exec formats "xray" and "cucumber" to separate the API URL, but recording to XRAY API doc, only xray format is allowed (see https://docs.getxray.app/display/XRAY/v2.0#/Import/post-import-execution-multipart).

Thank you very much for the PR.
Looking into the changes. I am not certain on these changes. Looking into the sample code from xray I can see that their code differentiates for the raven endpoint. (and that code should be correct)
https://github.com/Xray-App/xray-code-snippets/blob/main/use_cases/import_automation_results/java/xray-code-snippets/src/main/java/com/xblend/xray/XrayResultsImporter.java#L229-L254

Hey @mikepenz . Just had another look into the XRAY API doc and didn’t noticed the following hint: „… all endpoints from version 1.0 are also available in version 2.0“. So you are right, the cucumber endpoint should still work.
Anyway, we need a solution for the ability to use xray format as by now the action will try to send the files to the endpoint „https://xxx.com/rest/raven/2.0/import/execution**//**multipart“ which causes to an 404. Maybe I will have time to dig into it this weekend.
BR, René


deploy admin console without minio

NOTE: Depends on changes in https://github.com/replicatedhq/kots-helm/pull/37
This PR updates the admin console add-on to deploy the KOTS Helm chart without MinIO.

I do still want to see a "install previous version and upgrade" test though

I do still want to see a "install previous version and upgrade" test though

Yup, planning to do that in this PR. It looks like there's some other test fixes needed since we're using a -build.x suffix.

I do still want to see a "install previous version and upgrade" test though

Yup, planning to do that in this PR. It looks like there's some other test fixes needed since we're using a -build.x suffix.

Apparently I added that to the tests since the last time we used a build suffix!


Update Variant.xml Outdated Comment

Comment was outdated - you can now get a script's class_name easily by calling get_global_name() on the script of a given object.

Thanks for contributing :) The commits need to be squashed into one before this can be merged. See here for one explanation on how to do do that.

Thanks for contributing :) The commits need to be squashed into one before this can be merged. See here for one explanation on how to do do that.

done!

Please change the name of the commit to be more descriptive, explaining what the change is

Please change the name of the commit to be more descriptive, explaining what the change is

updated

Thanks! And congrats for your first merged Godot contribution :tada:


Fix window does not show when already app is ready (broken in #145)

appReady() is not considered to be called in create(). So calling
it directly in create() does not work (I confirmed it at least in my app). #145 does not work.
My PR (#144) was care about it but it was closed.

@maxogden can you merge this patch please? It works well :)

What's up?

Hey @Alex-D and @rhysd, I implemented this fix on my branch, see https://github.com/amaurymartiny/menubar/pull/8.
It's also on npm<EMAIL_ADDRESS>

Fixed in https://github.com/maxogden/menubar/commit/251fb21de85b3a78dc564a08248e5a4f84a21d71


Null pointer dereference in mrb_gc_mark

The following input to mruby demonstrates a crash: 215761.txt
Note that I was only able to reproduce the crash using gcc on Ubuntu (and not with clang on OS X).
gdb output:
Program received signal SIGSEGV, Segmentation fault.
0x00000000004036e2 in mrb_gc_mark (mrb=0x6b0010, obj=0x10) at /home/clayton/git/mruby/src/gc.c:697
697	  if (!is_white(obj)) return;

(gdb) print obj
$1 = (struct RBasic *) 0x10

(gdb) bt
#0  0x00000000004036e2 in mrb_gc_mark (mrb=0x6b0010, obj=0x10) at /home/clayton/git/mruby/src/gc.c:697
#1  0x0000000000403415 in gc_mark_children (mrb=0x6b0010, gc=0x6b00e8, obj=0x743a40)
    at /home/clayton/git/mruby/src/gc.c:600
#2  0x0000000000404061 in gc_mark_gray_list (mrb=0x6b0010, gc=0x6b00e8) at /home/clayton/git/mruby/src/gc.c:970
#3  0x000000000040411a in final_marking_phase (mrb=0x6b0010, gc=0x6b00e8) at /home/clayton/git/mruby/src/gc.c:993
#4  0x0000000000404519 in incremental_gc (mrb=0x6b0010, gc=0x6b00e8, limit=2000)
    at /home/clayton/git/mruby/src/gc.c:1090
#5  0x0000000000404613 in incremental_gc_step (mrb=0x6b0010, gc=0x6b00e8) at /home/clayton/git/mruby/src/gc.c:1122
#6  0x00000000004047b9 in mrb_incremental_gc (mrb=0x6b0010) at /home/clayton/git/mruby/src/gc.c:1166
#7  0x000000000040302e in mrb_obj_alloc (mrb=0x6b0010, ttype=MRB_TT_STRING, cls=0x6ba510)
    at /home/clayton/git/mruby/src/gc.c:507
#8  0x0000000000413d57 in str_new (mrb=0x6b0010, p=0x711150 "215761.rb", len=9)
    at /home/clayton/git/mruby/src/string.c:59
#9  0x000000000041453e in mrb_str_new_cstr (mrb=0x6b0010, p=0x711150 "215761.rb")
    at /home/clayton/git/mruby/src/string.c:215
#10 0x000000000041c23a in mrb_restore_backtrace (mrb=0x6b0010) at /home/clayton/git/mruby/src/backtrace.c:407
#11 0x000000000042f1fe in mrb_exc_set (mrb=0x6b0010, exc=...) at /home/clayton/git/mruby/src/error.c:282
#12 0x000000000042f2ea in mrb_exc_raise (mrb=0x6b0010, exc=...) at /home/clayton/git/mruby/src/error.c:303
#13 0x0000000000408b82 in mrb_funcall_with_block (mrb=0x6b0010, self=..., mid=52, argc=1, argv=0x7ffffff3c5b0,
    blk=...) at /home/clayton/git/mruby/src/vm.c:395
#14 0x00000000004090af in mrb_funcall_argv (mrb=0x6b0010, self=..., mid=52, argc=1, argv=0x7ffffff3c5b0)
    at /home/clayton/git/mruby/src/vm.c:461
#15 0x000000000042c45a in const_get (mrb=0x6b0010, base=0x6bc4c0, sym=678)
    at /home/clayton/git/mruby/src/variable.c:913
#16 0x000000000042c608 in mrb_vm_const_get (mrb=0x6b0010, sym=678) at /home/clayton/git/mruby/src/variable.c:953
#17 0x000000000040a895 in mrb_vm_exec (mrb=0x6b0010, proc=0x73afc0, pc=0x725c00)
    at /home/clayton/git/mruby/src/vm.c:988
#18 0x0000000000409eec in mrb_vm_run (mrb=0x6b0010, proc=0x73afc0, self=..., stack_keep=2)
    at /home/clayton/git/mruby/src/vm.c:823
#19 0x0000000000412281 in mrb_run (mrb=0x6b0010, proc=0x73afc0, self=...) at /home/clayton/git/mruby/src/vm.c:2603
#20 0x0000000000408566 in ecall (mrb=0x6b0010, i=255) at /home/clayton/git/mruby/src/vm.c:312
#21 0x000000000040d8e9 in mrb_vm_exec (mrb=0x6b0010, proc=0x6b21a0, pc=0x71f308)
    at /home/clayton/git/mruby/src/vm.c:1682
#22 0x0000000000409eec in mrb_vm_run (mrb=0x6b0010, proc=0x6b21a0, self=..., stack_keep=2)
    ...

This issue was reported by https://hackerone.com/ssarong

I couldn't reproduce the problem. Hopefully fixed.

Yes, it stopped crashing after https://github.com/mruby/mruby/commit/ab25eaea2345577b4e28672908f0190ce6b21684.


SAP IAS Administrator - System user does not send User ID and Password and an email.

[As per the documentation it says that when an Administrator user is created if type "System" on SAP IAS we would receive an automatic email l to activate account and set a password for SAP Cloud Platform Identity Authentication Service, but SAP IAS is not doing that.  It only sends an email for Administrators of the type "User".
So in this scenario, what do I enter for user ID and Password - on the MS Azure when configuring Automatic Provisioning?
Please suggest
Thanks]

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

ID: 8e8629d0-8a5c-64ba-46f2-f8caa51b9126
Version Independent ID: 9eab5faf-da99-3fca-d7fd-a2814c72b136
Content: Tutorial: Configure SAP Cloud Platform Identity Authentication for automatic user provisioning with Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/saas-apps/sap-cloud-platform-identity-authentication-provisioning-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @twimmers
Microsoft Alias: thwimmer

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

Hi @manojkunta
Kindly look into below note for clarification in the document.
https://learn.microsoft.com/en-us/azure/active-directory/saas-apps/sap-cloud-platform-identity-authentication-provisioning-tutorial#:~:text=The admininistrator user in SAP Cloud Platform Identity Authentication must be of type System. Creating a normal administrator user can lead to unauthorized errors while provisioning.
If you are still facing the issue, you can reach out to below teams.
Support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1)
Thank you for your time and patience throughout this issue.


IA-152 Rename TenantInfo and TenantMetaInfo

This code is in the IA-152 PR so might as well close this one now


add message functions

Adding functionality to include 3 functions:

GenerateSession(region)
ReceiveMessages(session, url)
ResolveMessage(session, object)

The expected flow is to run as follows:

GenerateSession(region)

you can get a new session later if needed
This will require locally stored credentials for AWS, theoretically lambda functions should house this info already

ReceiveMessages(session, url)

The URL is the URL for the SQS Queue, each service will have their own
The session is what you get back from the GenerateSession command -OR- you can use the AWS SDK to get a new session manually
This will receive up to 10 messages with a 10 minute timeout before the message can be read again for another processing attempt.  This should allow ample time for processing but can be adjusted if needed.
This will return an SQSMessage object which contains the Body which will house the message and the Result which you would write the result to.

perform processing

This is where you would do anything you need to do with the message such as parse the message, run the commands, etc
Each service in theory would do their own thing with the parsed message

ResolveMessage(session, object)

The session is generally the session from step 1 and the object is the SQSMessage object.
This will write the message object into DynamoDB for future storage (thus the result is important to update on completion) and delete the message.  If it cannot update DynamoDB then it will not delete the message.
  This would be equivalent to (1) invalid data such as incorrect XML or (2) the call was successful.  If there was an internal error for some reason you would not call ResolveMessage so that the message can be run again.

Failing due to lack of AWS Credentials provided in test.  Need to update to include something for credentials.

Fixed and ready for review

This isn't working out due to inadvertent changes to the vendor directory.  Closing PR and trying again from master


Clarified use of STL with Minimal C++ Library

STL is not available, so the applications should not rely on it.
Probably just a little typo, but it confused me while reading :-)

Thanks, @itmm! Please check the contribution guidelines, esp. regarding how to get your commit message in the correct form. https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-message-guidelines


CAE is available in Azure AD Government

Hello - doc states "Continuous access evaluation isn't currently available in Azure Government GCC High tenants."
CAE is available and on by default for Exchange Online in AAD Gov tenants. Recommend updating to:
"Continuous access evaluation is available in Azure Government tenants (GCC High and DOD) for Exchange Online with other applications coming in the future."

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

ID: b5cf71de-20a1-6056-01cf-f44d9ef60ed3
Version Independent ID: 67cb36dd-918d-e41d-7890-53750957d2d6
Content: Continuous access evaluation in Azure AD - Microsoft Entra
Content Source: articles/active-directory/conditional-access/concept-continuous-access-evaluation.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

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

#label:"awaiting-product-team-response"


Cant open tab in chrome with live server

Hi! I'm new at coding, been using vscode with live server extension and i had no issue, the problem starts when i installed it in a different computer.
Every time i tried to open with live server there is a message "http://<IP_ADDRESS>:5500/index.html application not found"
I have already tried uninstalling both live server and vscode, also every other suggestion that i could find here.
I cant either use chrome console with js using script command (console.log)
Thank you in advance!

@Missangay are you sure that getting this?

"http://<IP_ADDRESS>:5500/index.html application not found"
If you lunch server and unfortunately it isn't open on the browser and you trying to get index.html and if it isn't in the application then you can get Cannot GET /index.html

Also console is not giving me back commands that i send

set the settings  "liveServer.settings.CustomBrowser":"chrome"

@Missangay Is this issue solved?

Closing this issue due to no feedback.

set your default browser to chrome


Shadowdarklings Import Pulls from System Compendium

When I use Shadowdarklings to import a character, the Ancestry, Class, Background, Deity, etc. fields pull from the system compendium instead of the user compendium.  Is there any way to fix this so it defaults to the appropriate user compendium?

It doesn't actually.  It will use them from any compendiums.  But they need to be proper compediums, not just folders in the Items section.  Also, if you're duplicating the core classes for some reason, it's likely that it will pick the first matching class it comes across.
If you have specific examples of issues, then please give more details and attach your custom compendium so we can test.

I have copied the core classes and every other core compendium so I could/can tweak some formatting to suit my eyes/brain better...particularly the Description section of the classes.  Below is how I've formatted the Priest class in my user compendium:

Each class has a linked critical failure journal entry or rollable table as applicable, respective spell progression tables, and linked spell tier list journals.
However, when I created a new NPC with the Priest class, it pulled the system compendium:

I have, for now, named my user/world compendium as Ancestries (Copy), Backgrounds (Copy), etc. as the default naming system.  Is there a way to supercede that so new characters pull from my user/world compendium (either by naming or other methods)?
I'd also be happy to share my custom compendium, but I'm unfamiliar with how to do that...I'm more of a organization person and don't really know a lick of coding :(

I have, for now, named my user/world compendium as Ancestries (Copy), Backgrounds (Copy), etc. as the default naming system. Is there a way to supercede that so new characters pull from my user/world compendium (either by naming or other methods)?

That won't do anything as it looks for the item types, not the names of the compendiums.
There won't be a way of disabling the core items until we get round to doing the built in character creator I'm afraid.

I'd also be happy to share my custom compendium, but I'm unfamiliar with how to do that...I'm more of a organization person and don't really know a lick of coding :(

We can certainly looks at making requested changes to Item descriptions if they're relevant.  Easiest way to share your compendium is to create an empty world with them imported and back that up (the internal Foundry backup tool will be fine) and attach it to this ticket for us to look at.

Got ya, so at present there's no way to toggle system items off.  Would a bypass/workaround be to go to each default item that I have adjusted and rename it?  E.g. the Priest class, rename the default Priest class item "xPriest"?  As an aside, I have manually input the Cursed Scroll classes (from all 3 Cursed Scrolls) and they work flawlessly with Shadowdarkling import (Knight of St. Ydris, Warlock, and Witch tested so far).  I don't know if that feedback is helpful or not but I figured I'd share.
I think the item descriptions for classes would do well with additional information - my intent for my personal game is to minimize how often players and the DM (me) need to tab back-and-forth to reference the PDF rulebook.  I know this ultimately comes down to licensing concerns with the ability to include the information, but I believe the relevant information is important for class item descriptions:

Link to rollable table for class's talent table.  This should also extend to additional tables like the Patron Boon rollable table for Warlock.
Link to critical failure relevant rollable table or journal entry for spellcasting classes.  Priest = Priest Penance; Wizard = Wizard Mishap; Witch and Knight of St. Ydris = Diabolical Mishaps, etc.
Link to entire spell list for class.  I have done this in a journal entry.  Said journal entry also links to every single spell's Spell Item for quick reference when spells can be learned or if a player would like to reference a spell's text quickly (especially for spells they do not currently know).
Optional I've gone as far as including spell breakdown for each caster class by spell Tier to make the list reference quick when learning a new spell due to leveling rewards.

Martial Class Item Description example:

Spellcasting Class Item Description example:

Each class links to "Level Progression:  which has the level advancement table from p. 39 of the rulebook.  The Warlock level advancement also links to a journal entry containing all of the Patron Boon tables so it's universal for all Warlocks, regardless of their Patron.

Honestly, that last bit about me sharing my compendia went over my head, so I've not attached anything.

Got ya, so at present there's no way to toggle system items off. Would a bypass/workaround be to go to each default item that I have adjusted and rename it?

No, that probably wouldn't work.  You'll just have to make do with the built in classes until we make the required changes.
We also can't add anything that isn't in the quickstart, as that's the agreement we have with Kelsey.

All understood :).  I'll just use my current method, no biggie :)

Issue #631 tracks the ability to filter out compendiums.  Will close this issue.


binding-generator doesn't handle CRLF in OpenCV header files

Operating system: Windows 11
OpenCV installation: Used 'choco install opencv'
OpenCV version 4.6.0
rustc version: 1.65.0

Env vars:
OPENCV_INCLUDE_PATHS=c:\tools\opencv\build\include
OPENCV_LINK_LIBS=opencv_world460
OPENCV_LINK_PATHS=C:\tools\opencv\build\x64\vc15\lib
Certain OpenCV functions - for example the cv::calib3d::init_undistort_rectify_map is not exported by the binding-generator due to a bug in handling of CRLF line endings.
Replacing the line endings in the OpenCV header file with LF instead of CRLF solves the problem. However, it would be much better if the binding-generator could handle CRLF as well.
A small working patch with a simple associated test can be found here:
https://github.com/grindvoll/opencv-rust/commit/dd54b2a6609ee644b7a1f63f058b6d7f51a394bc
I am happy to submit a pull request if wanted.

It's a great catch, thanks! I'll try to release the fix asap

Thanks, much appreciated!

This should be fixed in the latest 0.78.0 release!


Feature/password toggle

Description
This PR adds toggle password option and eye button in password section of login and signup page

This PR does the following:

Adds ... eye button and password toggle option
Fixes ... #59
Updates ...  updates login and signup page and their css. Added script to toggle between password and text type of input box

Changes
List the detailed changes made in this PR.

Added a new feature to ... toggle between visible and invisible password option

Testing Instructions
Detailed instructions on how to test the changes. Include any setup needed and specific test cases.

Pull this branch.
Run npm install to install dependencies.
Run npm test to execute the test suite.
Go to login Page or SignUp page and type password click on eye button

Screenshots (if applicable)

Additional Context
Any additional context or information that reviewers should be aware of.

This PR is based on the following... i am changing the type of input tag from password to text and text to password to make it visible. I am channging eye icon by adding a class

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x ] I have performed a self-review of my code
[x ] I have commented my code, particularly in hard-to-understand areas
[ x] My changes generate no new warnings
[x ] I am working on this issue under GSSOC

Request: Please add level 1 and gssoc-ext

@Suvadip-sana  By mistake I pushed node modules. Then I deleted that in my second commit.


Python f-string E999-Syntar Error invalid syntax

Hello,
I have experienced this error message
at this line:
self.assertRedirects(response, f'/lists/{new_list.id}/')
The ') signs are only underlined and flake says it is invalid syntax

What version of flake8 are you using?

I too have found this issue. It seems that it is running in python 2 but I do not know how to change this...

Likewise.

I had flake8 installed for both python2 and python3 and flake8 was being defaulted to python2, so I did the following

Since none of you have actually answered the question, I'm going to assume that you are all using an out of date version of flake8 with a known bug affecting all 9xx error codes. Please update your flake8 version (pip install -U flake8). This particular error should be fixed in flake8>=3.2.1.
Closing this as there has been no response from @DorMiX about this.

I use flake8 3.5.0 (mccabe: 0.6.1, pycodestyle: 2.3.1, pyflakes: 1.6.0) CPython 3.7.0 on Darwin, and the error still exist :(

@geniousisme Please head over to #626 and answer https://github.com/AtomLinter/linter-flake8/issues/626#issuecomment-411172768, thanks ;)

The solution that worked for me is that from @Trivikram1. I essentially had to do
pip uninstall flake8
pip3 install flake8

even though my original flake8 installation was installed to my python3 virtual env using pip, it was the python2 version of flake8. You can tell the difference by looking at the CPython version number printed by the flake8 --version command. Before reinstalling, the CPython version printed was "2.7.15", but after reinstalling with pip3, the CPython version was "3.6.5".

I have the 3.8.2 and still get the invalid syntax for f-strings.


Check for already successfull authentication before "reauthenticating"

This is an issue reported at https://wordpress.org/support/topic/authldap-doesnt-work-with-other-auth-plugins.

Hello heiglandreas,
sorry my mistake, I am use too much to PHP and WP, so steep learning curve for me.
I suggest better and simpler check based on WP_User object, this should do the trick:

  function authLdap_login($user, $username, $password, $already_md5 = false)
  {
    // don't do anything when authLDAP is disabled
    if (! authLdap_get_option('Enabled')) {
        authLdap_debug('LDAP disabled in AuthLDAP plugin options (use the first option in the AuthLDAP options to enable it)');
        return $user;
    }

    if (isset($user) && $user->ID) {
        return $user;
    }

    authLdap_debug("User '$username' logging in");

As for the rest of the answer why not provide configurable options of fallback? i.e. if user is already authenticated by previous hook would you really like to force http auth anyway?
Kind regards
Litin


CI jobs sometimes keep running

Some unit test may have a dead lock issue or something. I feel like log output always stops here when CI job is stuck.
make[4]: Entering directory '/home/jenkins/workspace/master/os_build/src/iocore/cache'
PASS: test_Alternate_S_to_L
PASS: test_Cache
PASS: test_Alternate_L_to_S
PASS: test_RWW
PASS: test_Alternate_L_to_S_remove_L
PASS: test_Alternate_L_to_S_remove_S
PASS: test_Alternate_S_to_L_remove_S
PASS: test_Alternate_S_to_L_remove_L
PASS: test_Update_L_to_S
PASS: test_Update_header

Build #18747 master ubuntu:23.04 gcc debug

CTest caught it on #10097. fedora38 gcc debug
The test output was very long, so I've included here only a snippet of a few lines right before the hang.
[Jul 26 18:10:26.682] Alternate_S_to_ DEBUG: <CacheRead.cc:413 (openReadFromWriter)> (cache_read_agg) 0x7f18b2bbd740: key: 72AF8021 http passed stage 1, closed: 1, frag: 10
[Jul 26 18:10:26.682] Alternate_S_to_ DEBUG: <CacheRead.cc:468 (openReadFromWriter)> (cache_read_agg) 0x7f18b2bbd740: key: 72AF8021 closed: 1, fragment: 10, len: 10485760 starting first fragment
[Jul 26 18:10:28.388] [ET_AIO 1:7] DEBUG: <CacheWrite.cc:391 (aggWriteDone)> (cache_agg) Dir /tmp/cachetest.QlOWjw/var/trafficserver/cache.db 16384:32766, Write: 11223040, last Write: 9125888
[Jul 26 18:10:28.388] [ET_AIO 1:7] DEBUG: <CacheWrite.cc:1059 (aggWrite)> (agg_read) copying: 0, 11223040, key: -1266361926
WARNING: waiting on lock UnixEThread.cc:134 (process_event) for UNKNOWN
WARNING: waiting on lock UnixEThread.cc:134 (process_event) for UNKNOWN
WARNING: waiting on lock UnixEThread.cc:134 (process_event) for UNKNOWN
WARNING: waiting on lock UnixEThread.cc:134 (process_event) for UNKNOWN
[Jul 26 18:10:28.427] [ET_NET 0] DEBUG: <CacheWrite.cc:1059 (aggWrite)> (agg_read) copying: 4096, 11227136, key: -1266361926
[Jul 26 18:10:28.427] [ET_NET 0] DEBUG: <CacheWrite.cc:1200 (openWriteCloseDir)> (cache_stats) Fragment = 11
[Jul 26 18:10:28.427] [ET_NET 0] DEBUG: <P_CacheInternal.h:580 (free_CacheVC)> (cache_free) free 0x7f18b2bbdb10
[Jul 26 18:11:25.814] [ET_AIO 7:7] DEBUG: <CacheWrite.cc:391 (aggWriteDone)> (cache_agg) Dir /tmp/cachetest.QlOWjw/var/trafficserver/cache.db 16384:32766, Write: 11235328, last Write: 11223040

97% tests passed, 1 tests failed out of 32

Total Test time (real) = 1503.13 sec

The following tests FAILED:
	  7 - test_cache_Alternate_S_to_L (Timeout)
Errors while running CTest

Build #19461 master rockylinux:8 clang debug asan

Build 22076 master ubuntu:22.04 gcc debug
RPRINT Cache_dir: corrupt_bucket test
    REGRESSION_RESULT Cache_dir:                                PASSED
REGRESSION TEST SDK_API_TSHttpConnectServerIntercept started


Check Siafund and file contract revision ephemeral outputs

see https://github.com/SiaFoundation/core/pull/43


v0.91: Vision: implement image augmentation for next()

Complete the implementation of image augmentation with the next() operator.

Added for next() and minibatch in v0.91


Q about enhanced endo's method

Dear Dr. Liu,
Thanks for sharing your invaluable asset.
As I read your paper, titled enhanced Endo's method, it was very interesting and informative. I appreciate it again. I was just wondering if this HAMS code has been updated following that article.
Thanks in advance for your answer.
Regards,
Ikjae

Dear Ikjae,
Thank you for your inquiry. I just see your message.
Actually, not yet, but I can send you via email the HAMS executable program compiled with Endo's method.
Regards,
Yingyi

Thanks for the response. I'd really appreciate it if you do that!! My email address is<EMAIL_ADDRESS>For now I have steady forward speed (Neumann-Kelvin) and zero speed deep water HOBEM codes, which I have written in MATLAB but will be translated to FORTRAN in the near future. I would like to contribute it to HAMS at some time later.
Thanks again!!
Ikjae


support docker machine?

Plans on supporting docker machine?

Yes, but I don't have a lot of time in this moment, so I can't promise you a release date for this :) feel free to submit a PR if you want

I will look through the code to see if I can find where it is hooking up to the docker host. Essentially we just want it to pull the DOCKER_HOST environment variable instead of being localhost only.


error with IO Generator (CRUD) generated files

I keep getting this error
::1:51393 [500]: /hello/ - Method yii\widgets\ActiveField::__toString() must not throw an exception, caught Error: Class 'common\models\Yii' not found in F:\yii2\helloproject\backend\views\hello\_search.php on line 0

::1:51312 [500]: /hello/create - Method yii\widgets\ActiveField::__toString() must not throw an exception, caught Error: Class 'common\models\Yii' not found in F:\\yii2\helloproject\backend\views\hello\_form.php on line 0

I generated with IO Generator (CRUD)
if remove codes below in F:\\yii2\helloproject\backend\views\hello\index.php works fine.
<?php echo $this->render('_search', ['model' => $searchModel]); ?>

<div class="search-form" style="display:none">
    <?=  $this->render('_search', ['model' => $searchModel]); ?>
</div>

Found the problem.
Always make sure the model extending the base model should have defined;
 use Yii;


Coerce result

Hi, as mentioned here I would like to use a validator to coerce my values, so validate should not use checkNew (like validateSchema).
validate should be able to be used on all methods and can be an after hook.
In my case I would have turned a tinyint knex into a boolean in an afterHook.

validateSchema uses Ajv which has a coersion option.

I have created a coerce hook which is substantially like validate without the checkContext restriction.


Bugs

Traceback (most recent call last):
File "/home/pi/venv38/lib/python3.8/site-packages/discord/ext/tasks/init.py", line 101, in _loop
await self.coro(*args, **kwargs)
File "/home/pi/FinTwit_Bot/src/cogs/loops/exchange_data.py", line 401, in restart_sockets
await self.start_sockets()
File "/home/pi/FinTwit_Bot/src/cogs/loops/exchange_data.py", line 488, in start_sockets
reply = await self.ws.recv()
File "/home/pi/venv38/lib/python3.8/site-packages/websockets/legacy/protocol.py", line 516, in recv
raise RuntimeError(
RuntimeError: cannot call recv while another coroutine is already waiting for the next message

Traceback (most recent call last):
File "/home/pi/venv38/lib/python3.8/site-packages/pycoingecko/api.py", line 29, in __request
response.raise_for_status()
File "/home/pi/venv38/lib/python3.8/site-packages/requests/models.py", line 953, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://api.coingecko.com/api/v3/coins/hoodler/
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/pi/FinTwit_Bot/src/util/tweet_util.py", line 197, in add_financials
volume, website, exchanges, price, change, ta = classify_ticker(
File "/home/pi/FinTwit_Bot/src/util/ticker.py", line 191, in classify_ticker
coin = get_coin_info(ticker)
File "/home/pi/FinTwit_Bot/src/util/ticker.py", line 43, in get_coin_info
coin_dict = cg.get_coin_by_id(id)
File "/home/pi/venv38/lib/python3.8/site-packages/pycoingecko/utils.py", line 12, in input_args
return func(*args, **kwargs)
File "/home/pi/venv38/lib/python3.8/site-packages/pycoingecko/api.py", line 140, in get_coin_by_id
return self.__request(api_url)
File "/home/pi/venv38/lib/python3.8/site-packages/pycoingecko/api.py", line 36, in __request
raise ValueError(content)
ValueError: {'error': 'Could not find coin with the given id'}

Traceback (most recent call last):
File "/home/pi/venv38/lib/python3.8/site-packages/tweepy/asynchronous/streaming.py", line 112, in _connect
await self.on_data(line)
File "/home/pi/FinTwit_Bot/src/cogs/loops/timeline.py", line 131, in on_data
await self.post_tweet(*formatted_tweet)
File "/home/pi/FinTwit_Bot/src/cogs/loops/timeline.py", line 169, in post_tweet
msg, channel = await self.upload_tweet(
File "/home/pi/FinTwit_Bot/src/cogs/loops/timeline.py", line 221, in upload_tweet
msg = await channel.send(embed=e)
File "/home/pi/venv38/lib/python3.8/site-packages/discord/abc.py", line 1065, in send
data = await state.http.send_message(channel.id, content, tts=tts, embed=embed,
File "/home/pi/venv38/lib/python3.8/site-packages/discord/http.py", line 254, in request
raise HTTPException(r, data)
discord.errors.HTTPException: 504 Gateway Time-out (error code: 0):

Failed to send request: HTTPSConnectionPool(host='api.twitter.com', port=443): Max retries exceeded with url: /1.1/friends/ids.json (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0xa3076e50>: Failed to establish a new connection: [Errno -3] Temporary failure in name resolution'))
Failed to get following ids
Exception in thread Thread-67:
Traceback (most recent call last):
File "/usr/local/lib/python3.8/threading.py", line 932, in _bootstrap_inner
self.run()
File "/usr/local/lib/python3.8/threading.py", line 1254, in run
self.function(*self.args, **self.kwargs)
TypeError: 'str' object is not callable
Exception in thread Thread-68:
Traceback (most recent call last):
File "/usr/local/lib/python3.8/threading.py", line 932, in _bootstrap_inner
self.run()
File "/usr/local/lib/python3.8/threading.py", line 1254, in run
self.function(*self.args, **self.kwargs)
TypeError: 'str' object is not callable

Traceback (most recent call last):
File "/home/pi/venv38/lib/python3.8/site-packages/tweepy/asynchronous/streaming.py", line 112, in _connect
await self.on_data(line)
File "/home/pi/FinTwit_Bot/src/cogs/loops/timeline.py", line 131, in on_data
await self.post_tweet(*formatted_tweet)
File "/home/pi/FinTwit_Bot/src/cogs/loops/timeline.py", line 154, in post_tweet
e, category = await add_financials(
File "/home/pi/FinTwit_Bot/src/util/tweet_util.py", line 243, in add_financials
if "Binance" in exchanges:
TypeError: argument of type 'NoneType' is not iterable


[Building a Cipher] Introduction to code comment seems to be out of sequence

Description
In the Building a Cipher project, comment is first mentioned in step 51:

After that, you'll see an error appear in the terminal. As a temporary fix, comment out the caesar() call.

But in step 59, we actually introduce comment and its syntax:

When coding, readability is key. Comments serve as effective notes, explaining the logic behind your code. [...]
In Python, you can write a comment using a #. Anything that comes after the # won't be executed.

Links

https://www.freecodecamp.org/learn/scientific-computing-with-python/learn-string-manipulation-by-building-a-cipher/step-51
https://www.freecodecamp.org/learn/scientific-computing-with-python/learn-string-manipulation-by-building-a-cipher/step-59

Suggestion
In step 51, we introduce function parameters, so I think mentioning comment here would be a little overwhelming. I think the comment introduction should either be moved to an earlier challenge or kept in step 59.
If we move the comment introduction to an earlier challenge, step 51 can be kept as-is.
But if we keep comment introduction in step 59, we should update step 51. Instead of asking the users to comment out the code, I wonder if we could ask them to remove the line entirely (though the code is added in step 50, so it's probably a little odd to immediately remove it in the next step).

This does seems to be out of sequence.
I would suggest to not comment out caesar() in step 51 and let the exception be(the resolution is there in the coming steps).
This part of the instruction can be updated from:
After that, you'll see an error appear in the terminal. As a temporary fix, comment out the caesar() call.

to:
After that, you'll see an error appear in the terminal. You'll work on the resolution in the coming steps.

If we don't ask to comment out the function call in step 51 and tell the campers they will work on fixing the error in the next steps, it would be logic to address the error in the console (TypeError: caesar() missing 2 required positional arguments: 'message' and 'offset') and require to pass the two positional arguments to the function call in step 52.
But then the function will seem to work fine, although it's still using the text and shift variable inside its body instead of the function parameters.
The alternative is to ignore the TypeError in the console for step 52, and address it directly in step 53. For that we should simply keep the function call instead of commenting it out and there is no need to swap the step order.
Honestly, I don't like particularly any of these options.


stats: 2.10.5 -> 2.10.6

Diff: https://github.com/exelban/stats/compare/v2.10.5...v2.10.6
Changelog: https://github.com/exelban/stats/releases/tag/v2.10.6
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 300731 run on aarch64-darwin 1

1 package built:

stats


Add Pool connection

Is it possible to add pool connection like Jedis ?

It is, we can look into it once we have some bandwidth. PR's are always welcome though. This shouldn't be difficult to implement as all the classes to implement it are already in place.


Dicer Vulnerability

Hello. Please update the multer dependency to fix the dicer vulnerability.
multer is a dependency of this package. Busboy is a dependency of multer. Dicer is a dependency of busboy in the version contained within this current tree.
Busboy and multer have both been updated to remove dicer as it is no longer supported. Currently this is causing a security vulnerability in our application and a stopgap solution must be implemented until this package is updated or the package is dropped.
https://nvd.nist.gov/vuln/detail/CVE-2022-24434

@JamesMessinger

Just linking the PR that addresses this for easy reference:
https://github.com/APIDevTools/swagger-express-middleware/pull/181


proposal: unicode/utf16: add DecodeRuneBytes

Proposal Details
Hello peeps. I'd like to propose to extract part of the existing code in the encoding/utf16 package into a function.
In particular these lines: https://github.com/golang/go/blob/master/src/unicode/utf16/utf16.go#L116-L130
Why?

The main reason for my use case is to avoid allocations.
The second reason is to be able to easily decode from a byte slice. Today this operation is not easily done without understanding the internals of utf16.

On existing utf16.DecodeRune
utf16.DecodeRune already exists but its usage is tricky with existing API. One can't use it in the same way as the well designed utf8.DecodeRune since utf16.DecodeRune expects one to have already finished the step of deciding whether the first rune in the []uint16 is a single "self" or a surrogate pair- and there's no API that helps one make this decision!
It is my understanding that utf16.DecodeRune not having the following signature was a missed opportunity to solving the issues I'm currently having:
func DecodeRune(buf []uint16) (r rune, size16 int)

How?
Adding a package level function. Unverified and untested function logic included for visual purposes.
func DecodeRuneBytes(srcUTF16 []byte, order16 binary.ByteOrder) (r rune, size int) {
	// UTF16 values.
	const (
		// 0xd800-0xdc00 encodes the high 10 bits of a pair.
		// 0xdc00-0xe000 encodes the low 10 bits of a pair.
		// the value is those 20 bits plus 0x10000.
		surr1 = 0xd800
		surr2 = 0xdc00
		surr3 = 0xe000
	)
	var r1, r2 rune

	slen := len(srcUTF16)
	if slen == 0 {
		return '\uFFFD', 1
	}
	r1 = rune(order16.Uint16(srcUTF16))
	if slen >= 4 {
		r2 = rune(order16.Uint16(srcUTF16[2:]))
	}
	var ar rune
	switch {
	case r1 < surr1, surr3 <= r1:
		// normal rune
		ar = r1
		size = 2
	case surr1 <= r1 && r1 < surr2 && slen >= 4 &&
		surr2 <= r2 && r2 < surr3:
		// valid surrogate sequence
		ar = DecodeRune(r1, r2)
		size = 4
	default:
		// invalid surrogate sequence
		ar = '\uFFFD'
		size = 1
	}
	return ar, size
}

Note: Here's an example of the API implemented in a package and it's usage.

I've just noticed that I'd also greatly benefit from adding a EncodeRuneBytes. This way conversion routines between different formats would be greatly simplified.
func EncodeRuneBytes(dst []byte, r rune) int

The DecodeRuneBytes function adds a new concept to the utf16 package, namely the encoding of sequences of UTF-16 codes as byte strings, with the concomitant need to specify the byte order.
If we make the byte encoding the caller's concern, then can't the problem be solved with code something like this?
   a := next()
   if !utf16.IsSurrogate(a) {
       return a
   }
   b := next()
   if !utf16.IsSurrogate(b) {
       return 0xFFFD
   }
   return utf16.DecodeRune(a, b)

(Replace 'next' with your favorite byte iterator.)

@adonovan Hmm, rune b should actually be checked with a different function. IsSurrogate does: surr1 <= r && r < surr3 while rune b is invalid if it does not fulfill surr2 <= r && r < surr3. But you are close.

CC @dsnet

The utf16 package is meant to be as minimal as possible. If you really need to decode bytes, it is easy to decode them outside the loop and then call DecodeRune with successive pairs of uint16 values.

This proposal has been declined as infeasible.
— rsc for the proposal review group


Enum cases incorrectly generated when containing enum name

Issue
When an enum case's first few letters contain the name of the enum itself (even if separated by underscores), the plugin strips those characters from the enum name.
Example
syntax = "proto3";

message Test {
    Promos promos = 1;

    enum Promos {
        PROMO_SCREEN = 0;
        TEST_ONE = 1;
    }
}

Actual
public enum Promos: SwiftProtobuf.Enum {
	public typealias RawValue = Int
	case creen // = 0
	case testOne // = 1
	case UNRECOGNIZED(Int)
}

Expected
public enum Promos: SwiftProtobuf.Enum {
	public typealias RawValue = Int
	case promoScreen // = 0
	case testOne // = 1
	case UNRECOGNIZED(Int)
}

Reproducibility
100% reproducible on SwiftProtobuf 1.0.3
Additional details
Python treats the above example as one would "expect", yielding PROMO_SCREEN and TEST_ONE cases (instead of CREEN and TEST_ONE like Swift does).
It's also worth noting that modifying the .proto file in the following way results in a different error:
message Test {
    Promos promos = 1;

    enum Promos {
        PROMO_SCREEN = 0;
        CREEN = 1;
    }
}

Error:

When enum name is stripped and label is PascalCased (Creen), this value label conflicts with PROMO_SCREEN. This will make the proto fail to compile for some languages, such as C#.

That said, it looks like protoc will prevent this particular example. However, we should be maintaining the names of enum cases within a given enum type to prevent confusion and maintain consistency with generated code.

If valid proto files generate invalid Swift code, that is certainly a bug.  Please provide more details so we can fix it.  (As you observed, your example does not demonstrate such a bug.)
As the error message points out, C# protobuf also strips enum names in a similar fashion, so there is ample precedent for this behavior.
We certainly cannot change this soon.   It would break existing code for people already using SwiftProtobuf.  We can only make such breaking changes at a major version, and we do not at this time have other changes that would justify creating a 2.0 release.
Note that I am happy to discuss such a change.  If there's a strong consensus that we should change it, then we can certainly put this into a list of potential changes for a future 2.0 release.

To provide some more history –
Top level enums in C (C++ generation), don't get scoped to the enum name, so the common practice is to repeat the enum name on all the values.  Lots of .proto files do this.  But for the languages that do scope the enum values to the names, the prefix makes for redundant names.
Since Apple also did this sorta prefix stripping for the System Frameworks with the updated Swift Naming guidelines in the 3.x/4.0 time frame, it seemed to make sense to try to model this on the proto side also to get the better scoping Swift provides.
The one you call out PROMO_SCREEN is a bug in the current impl that should get fixed.  Hopefully I'll get a chance to look at that in the near future is someone else doesn't beat me to it.

Finally got a chance to look back at the code this morning.  The logic in the generator matches what the C# ones does, and it also have the same weakness.  Both work by stripping the underscores characters and then just case insensitively matching.
We could retool things to use the underscores and or changes in case to indicate a word in the names, then match words.  The one gap that jumps out at me is an enum of foo, but a values of foobar and foobaz would no longer strip down to just bar and baz.  But maybe some of those things can be resolved by checking if the value was mixed case at all, and what the case of the first character after the prefix was.

Thanks for the details and context @thomasvl. The goal of prefix stripping in generated Swift code definitely makes sense.
If we're okay with making a change (which would be breaking, as @tbkka mentioned), I can take a look into it myself as well. It seems reasonable in this case to classify a "word" using both camelCase and SNAKE_CASE handling.
IMO it makes sense to avoid stripping prefixes for enums like the example you listed if they involve neither camel case or snake case:
enum foo {
    foobar = 0; // -> foobar
    foo_baz = 1; // -> baz
    foo_foobar = 2; // Disallowed by protoc
}

enum foo_bar {
    foo_bar_abc = 0; // -> abc
    foo_abc = 1; // -> foo_abc
}

enum MyFoo {
    MyFooBar = 0; // -> bar
    MyFooBaz = 1; // -> baz
}

There are a few interesting test cases in the code base which would likely change as well:
  (prefix: "foo_bar", str: "foobar_bAZ", expected: "bAZ" ),
  (prefix: "_foo_bar", str: "foobar_bAZ", expected: "bAZ" ),
  (prefix: "foo__bar_", str: "_foo_bar__baz", expected: "baz"),

With the new changes, these would become (notice that the number of underscores remains irrelevant as it does today):
  (prefix: "foo_bar", str: "foobar_bAZ", expected: "foobar_bAZ" ),
  (prefix: "_foo_bar", str: "foobar_bAZ", expected: "foobar_bAZ" ),
  (prefix: "foo__bar_", str: "_foo_bar__baz", expected: "baz"),
  (prefix: "FooBar", str: "FooBarBaz", expected: "baz"),
  (prefix: "foo", str: "_foo_bar__baz", expected: "bar_baz"),

What are your thoughts on this?

foo_foobar = 3; // Disallowed by protoc

What's the error on this one?  Sounds like they added some more checks since I last looked.
At first glance, it seems reasonable.  The test cases will also need to be expand for camel vs. underscore, as well as still ignoring the number of underscores:
  (prefix: "foo_bar", str: "FooBarBaz", expected: "Baz")
  (prefix: "FooBar", str: "foo_bar_baz", expected: "baz")
  (prefix: "foo__bar", str: "FooBarBaz", expected: "Baz")
  (prefix: "FooBar", str: "_foo__bar__baz", expected: "baz")
  ...

As far as breaking, we've got some other stuff we've been looking at doing that might also be breaking (I'm trying to get the cycles to work on them), so now is as good of time as any.  So if you are interested, go for it!

foo_foobar = 3; // Disallowed by protoc

What's the error on this one? Sounds like they added some more checks since I last looked.

Sorry, typo'd there - the one that fails is foo_bar, not foo_foobar in that case. The error is:

When enum name is stripped and label is PascalCased (Bar), this value label conflicts with foobar. This will make the proto fail to compile for some languages, such as C#.

It looks like some of the test cases you listed are in conflict with the ones I posted:
(prefix: "foo_bar", str: "foobar_bAZ", expected: "foobar_bAZ" )

versus:
(prefix: "foo_bar", str: "FooBarBaz", expected: "Baz")

(Note that the first one does not take into account underscores). Looking over it again, I think what you're proposing (second example above) makes sense and would still solve the original problem 👍 Hoping to get a chance to look into it soon, and I'll be sure to PR if so!

We are hitting this bug in C#.  It is clearly wrong and hard to fix outside of protoc given that it works for some client languages and not others.  I think it should be fixed asap and if you are worried about recompiling existing protos, add a command line option for the old behavior. If you won't do that, add a command line option to select the correct behavior.

Yes, not using swift. Just commenting that is it amazing this hasn't been fixed in general given how wrong the output is.


[Word Search] Details view enabled by default

Extension
https://www.raycast.com/rishabswift/word-search
Description
Add setting to enable Details View to be enabled by default.

Who will benefit from this feature?
Save some keystrokes for those that want to always have the details view available when reviewing spell results.
Anything else?
No response

@raycastbot close


Modified README.md

Changed URL for cloning project via git.
Added shto codeblocks and include leading $ sign for commands.
Added Troubleshooting section for Development section.
Changed string OSX to macOS.

Btw: you can fix the build by running yarn format before pushing (yep, haven't documented that yet, sorry ^^')


Fix colored font rendered in wrong size

Fix #24456
The character->rect.size is properly update at DynamicFontAtSize::_bitmap_to_character
so no need to multiply scale again.
It was changed with https://github.com/godotengine/godot/commit/5cd12f6649387f91d08fd17bf3c70e732798ab58#diff-fce1ec7b33979481c1af96f2d6be6686R565

Thanks!


Fix normalize-url

This PR removes the mandatory URL enoding domains, sometimes broken them because the official website expects an encoded route. Bug causing errors from normalize-url lib.

Awesome, might be worth to send a PR to the normalize-url package as well!


Arabic to Urdu Machine Translation

@lyeoni
In the case I want to train an Arabic to Urdu Machine Translation:

is that attainable using this project?
what options should be set in training?
do you suggest another github project?

@deepseek

Is that attainable using this project?

I don't know. I've never seent Arabic-Urdu paired corpus dataset

what options should be set in training?

It's up to your dataset size and resources. For example, If you have large corpus, I recommend to increase vocab_size (e.g. 30,000). and If you have many GPUs, I recommend to increase batch_size.

do you suggest another github project?

for translation? I recommend .. pytorch translation tutorial. But It uses seq-to-seq model.


COMPAT: remove deprecated np.unicode_ and np.string_

Closes #379
GeoPandas CI, where we now test with pyogrio main and numpy nightly (after #381 allowed that), caught usage of deprecated np.unicode_. Replacing with np.str_ as suggested by numpy.
~Shall I make an effort to add a CI env with nightly numpy?~ edit: I did

Thanks!


feat(tidb): add notify presubmit job for tidb

only notify when the monitor files changed.
Ref: #2277

/approve


Word racing stable branch first commit

Start the PR of word racing game, will push all updates into this PR

Abandoning this pull request as the work was not completed


[Pages]  New pages

We have to create the following pages:

About us
Explore (static for now)
Showcase
Social Links
Personalization
Profile Themes
API
Community
Help center
xProfile + Youth Icon
Open-source Software
Privacy
Terms

I would love to work on adding new pages. I can start off by doing work on About us page if I get the requirement mentioned correctly or the design to implement

I would love to work on adding new pages. I can start off by doing work on About us page if I get the requirement mentioned correctly or the design to implement

That's great @ghousetazeem but we don't have a design for it. I advice you to join our discord where we discuss things

Ok sure


feat: improve badges in README.md

Main improvements in this PR:
The aim in this PR is to fix #194. At the same time, by adding the version.txt file, the version badge will be fixed from ?.?.? to the right version, once this file ends up on the master branch.
I hereby confirm that I have:

[ ] Tested my code on my own computer for running the model
[x] Selected devel as a target branch

Not sure what's wrong with the version badge, which was certainly alright in this morning.

Oh that's a bit confusing, version.txt exists on master but not on devel.

Now the version badge is back to normal.

So, what about that version.txt file then? Should it or should it not be present on develop?

"files should not be added without necessity." - law of Ockham's razor

From what I remember, the problem we encountered with also having the version number/file on the devel branch was that it would become outdated unless we merged master into devel, which I don't think is consistent with the current workflow.

To resolve this PR, I propose that I remove the file version.txt for now, and maybe we can have a discussion about the workflow to see if there is a way to simplify it.

Agree to remove the version.txt that can only be existed in master branch.

The version.txt file has been removed - it will not be added to devel in this PR.